-
Notifications
You must be signed in to change notification settings - Fork 2
Longitudinal Phase-Space Rework #85
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
Merged
Merged
Changes from all commits
Commits
Show all changes
29 commits
Select commit
Hold shift + click to select a range
48a8bd7
ADD: Define "time aware" pass methods
VitorSouzaLNLS 30fd1c8
ADD: Get time aware elements info from accelerator
VitorSouzaLNLS 0d2dd7c
ADD: defining path length kick function
VitorSouzaLNLS 79365b9
MNT: rename positions -> displacements
VitorSouzaLNLS 6aa4952
MNT: insert path length adjustment in line_pass & ring_pass
VitorSouzaLNLS 082f3e1
FIX: match unsigned int
VitorSouzaLNLS bd1dfe1
MNT: insert path length adjustment in track_findm66
VitorSouzaLNLS cd1a6d5
MNT: insert path length adjustment in track_findorbit6 & track_findor…
VitorSouzaLNLS 6e05e6f
MNT: adjust line_pass args in "optics.cpp"
VitorSouzaLNLS 59cb994
MNT: adjust line_pass args in "commands.cpp"
VitorSouzaLNLS 7b9dbad
MNT: adjust line_pass args in "tests.cpp"
VitorSouzaLNLS 8624405
MNT: adjust line_pass args in "test-kickmap.cpp"
VitorSouzaLNLS ad50bdb
MNT: adjust line_pass args in "interface.cpp"
VitorSouzaLNLS b03d5e3
FIX: mistyped variable and missing declaration
VitorSouzaLNLS e99345d
FIX: multiple declaration of element_offset
VitorSouzaLNLS 623e926
FIX: remove mistyped "]"
VitorSouzaLNLS 0e574f4
MNT: beautify "is_time_aware_pm" function
VitorSouzaLNLS 171e1e1
FIX: 0 -> UINT_MAX in "empty" time_aware_indices
VitorSouzaLNLS d6ca46f
IMP: avoid precision loss when obtaining the "time aware" elements' i…
VitorSouzaLNLS 978bd6c
MNT: Leave empty "time aware" indices/displacements for accelerators …
VitorSouzaLNLS a7c723d
ENH: improving functions: "get_time_aware_elements_info" and "adjust_…
VitorSouzaLNLS b45f323
MNT: match args for "get_time_aware_elements_info" and "adjust_path_l…
VitorSouzaLNLS 6b82a9a
DOC: rewrite comments
VitorSouzaLNLS 5a98142
ENH: optimize and beautify the usage of time aware info
VitorSouzaLNLS 0535045
MNT: match and propagate last modification
VitorSouzaLNLS 352ead7
MNT/DOC: edit some comments
VitorSouzaLNLS 3386063
ENH: reorder pos.push_back and adjust_path_length; add comments;
VitorSouzaLNLS 53bf0fb
Merge branch 'master' into rf-kick-rework
VitorSouzaLNLS 96692e3
Update version
VitorSouzaLNLS File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1 +1 @@ | ||
| 4.10.8 | ||
| 5.0.0 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
suggestions:
a) exchange the order of these two lines: pos.push_back and adjust_path_length. this is in orfer to calculate the variation of synchronous phase with Rf freq,we can use the z coordinate at the cavity entrance, not at its exit (more intuitive)
b) add a comment about how to calculate the synchronous phase, as described above (maybe a pyaccel method?)
c) add comment in the code documentation that adjustment of path length is not distributed in all elements (for performance reason)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changed ✓