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

Fix #51 #57

Merged
merged 9 commits into from
Jul 5, 2023
Merged

Fix #51 #57

merged 9 commits into from
Jul 5, 2023

Conversation

dbrakenhoff
Copy link
Collaborator

Solve multiprocessing issue, add notebook comparing fortran/numba with/without multiprocessing.

- remove f2py from elements and model class
- move besselaes module logic to bessel.py
- refactor bessel function calls
- set bessel method in init
- show difference fortran numba
- with and without multiprocessing
@dbrakenhoff
Copy link
Collaborator Author

@mbakker7 I merged the dev branch into mine... I realized too late that master contained the latest changes. Not sure if there were changes on dev that aren't supposed to be pulled into master?

@mbakker7
Copy link
Owner

mbakker7 commented Nov 9, 2022 via email

@dbrakenhoff
Copy link
Collaborator Author

I removed the dev merge commit and redid the master merge. So I think it should be good now.

@dbrakenhoff dbrakenhoff requested a review from mbakker7 June 1, 2023 15:10
@mbakker7 mbakker7 merged commit 6f682eb into master Jul 5, 2023
@mbakker7 mbakker7 deleted the solve_mp branch July 5, 2023 10:50
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.

2 participants