8000 Expand literal function descriptor language [in progress] by sritchie · Pull Request #262 · sicmutils/sicmutils · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Expand literal function descriptor language [in progress] 10000 #262

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 2 commits into from

Conversation

sritchie
Copy link
Member
@sritchie sritchie commented Feb 8, 2021

Full port of litfun.scm and the function descriptor language from scmutils.

== Literal Function Descriptor Language

We support some of the possible literal function types; these are type signatures like (-> (UP Real Real) Real). But litfun.scm has support for a richer language.

  • study litfun.scm and port the full scope of function descriptor language, including lots of helper functions like Lagrangian. There are also many examples here that would make great tests. This can live in sicmutils.abstract.function.cljc
  • there is also a "typed function" we don't have that would be interesting to study.
  • Forms like (UP* Real) should work, with no explicit arity.
  • add 0-arity of Hamiltonian back in: ([] '(-> (UP Real (UP* Real) (DOWN* Real)) Real))
  • add 0-arity of Lagrangian back in: ([] '(-> (UP Real (UP* Real) (UP* Real)) Real)). the current impl can't handle stars with no numbers.

@sritchie sritchie force-pushed the sritchie/litfun branch from 3aaf865 to 123bf04 Compare May 3, 2021 13:41
@sritchie sritchie changed the title Working on literal functions [in progress] Expand literal function descriptor language [in progress] May 3, 2021
@codecov-commenter
Copy link

Codecov Report

Merging #262 (71687e8) into master (5385c8e) will decrease coverage by 0.37%.
The diff coverage is 33.69%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #262      +/-   ##
==========================================
- Coverage   83.98%   83.61%   -0.38%     
==========================================
  Files          97       97              
  Lines       12872    12941      +69     
  Branches      674      675       +1     
==========================================
+ Hits        10811    10821      +10     
- Misses       1387     1445      +58     
- Partials      674      675       +1     
Impacted Files Coverage Δ
src/sicmutils/abstract/function.cljc 61.42% <33.69%> (-22.26%) ⬇️
src/sicmutils/numsymb.cljc 92.71% <0.00%> (-0.49%) ⬇️
src/sicmutils/polynomial.cljc 83.12% <0.00%> (+0.15%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5385c8e...71687e8. Read the comment docs.

@sritchie sritchie added this to the 0.21.0 milestone Nov 21, 2021
@sritchie sritchie mentioned this pull request Nov 27, 2021
@sritchie sritchie added the kernel Remaining items from scmutils kernel. label Nov 28, 2021
@sritchie sritchie modified the milestones: 0.21.0, 0.22.0 Jan 23, 2022
@sritchie sritchie force-pushed the main branch 3 times, most recently from afa6dd7 to c33ef42 Compare February 15, 2022 22:57
@sritchie
Copy link
Member Author

Moved to Emmy.

@sritchie sritchie closed this Mar 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kernel Remaining items from scmutils kernel.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Flesh out literal fn syntax for structured args, values to match scmutils
2 participants
0