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.

Sign up for GitHub

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

Jump to bottom

super shot #6

Open
krober10nd opened this issue Feb 11, 2021 · 3 comments
Open

super shot #6

krober10nd opened this issue Feb 11, 2021 · 3 comments
Assignees
@Olender

Comments

@krober10nd
Copy link
Collaborator

  • Enable several sources to be simulated at the same time.
@Olender
Copy link
Collaborator

Olender commented Jul 19, 2021

Has this already been fixed and tested with the new sources implementation?

@krober10nd
Copy link
Collaborator Author

krober10nd commented Jul 20, 2021 •
edited
Loading

No I have not tested this but it’s as simple as setting Source.current_source to a list the length of the number of sources. However to benefit from this you would have to change the gradient summation and the optimization algorithm.

@jinshanmu jinshanmu mentioned this issue Jan 17, 2024
Open
@Olender Olender mentioned this issue Jun 18, 2024
Open
Olender added a commit that referenced this issue Jun 25, 2024
@Olender
Merge pull request #6 from Olender/updating-readme
9cd0aca 
Updating readme
@Olender Olender self-assigned this Jun 27, 2024
@Olender
Copy link
Collaborator

Olender commented Jul 3, 2024

Supershots and gradients are already working on the referenced branch. I will merge as soon as I finish the other things in the branch (various regular shots in serial or spatial parallelism only).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Olender Olender

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

When branches are created from issues, their pull requests are automatically linked.

issue_0105-serial-shots-with-and-without-paralellism NDF-Poli-USP/spyro
2 participants
@krober10nd @Olender