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

fast likelihood #24

Open
tdixon97 opened this issue Sep 8, 2024 · 1 comment
Open

fast likelihood #24

tdixon97 opened this issue Sep 8, 2024 · 1 comment
Labels
enhancement New feature or request low priority

Comments

@tdixon97
Copy link
Collaborator

tdixon97 commented Sep 8, 2024

Can we use array programming (in future) like Luigi did to speed the likelihood?

@tdixon97 tdixon97 added enhancement New feature or request low priority labels Sep 8, 2024
@tdixon97
Copy link
Collaborator Author

tdixon97 commented Sep 15, 2024

Tried lots of stuff

  • array programming - probably doesn't help
  • strong typing variables - made no difference
  • @simb and @inbounds - make no difference
  • removing global variables - helped alot in small tests, only a bit in our case

Perhaps the issue is all of our logical statements making it hard for the compiler to optimise, should do more tests...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request low priority
Projects
None yet
Development

No branches or pull requests

1 participant