Skip to content
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

rewrite w/ multiple dispatch #36

Merged
merged 29 commits into from
Feb 13, 2025
Merged

rewrite w/ multiple dispatch #36

merged 29 commits into from
Feb 13, 2025

Conversation

atharvas
Copy link
Collaborator

@atharvas atharvas commented Dec 25, 2024

Reimplementing key components of LaSR using multiple dispatch. The code compiles, but test coverage right now is basically 0.

[online test cases] (To be executed on github actions)

  • Add test cases to test LaSR integration (ie: use LaSROptions to execute SymbolicRegression.jl test cases).
  • Coverage for concept summarization.
  • Coverage for LLM crossover operator.
  • Coverage for all LLM mutation operators.
  • Coverage for concept evolution.
  • Expression printing and parsing homomorphism.
  • Test case for ensuring prompts are well-structured.
  • Prompt parsing test cases.

[offline test cases] (To be executed locally)

  • Test cases for LLM handshake
  • Test case for LaSR MLJ tutorial.

@atharvas atharvas self-assigned this Jan 6, 2025
@atharvas
Copy link
Collaborator Author

atharvas commented Jan 6, 2025

MilesCranmer#398 (comment) should address this. Test cases are passing locally now.

@atharvas atharvas merged commit 1df4e5c into master Feb 13, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant