Repository navigation
Replacement of Class Referenced with std::shared_ptr #541
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
Closed
Closed
Changes from all commits
Commits
Show all changes
53 commits
Select commit
Hold shift + click to select a range
718d665
removed workflows for this complete seperate branch
JanNiklasB efb8c8b
Merge remote-tracking branch 'origin/CPP23CRPropa' into CRRW_CRPropa
JanNiklasB cee805c
replaced AssocVector..h with unordered_map
JanNiklasB 17bfae7
replaced Clock with chrono
JanNiklasB 63e1a05
Replaced Referenced class with shared_ptr
JanNiklasB 50e27cb
wrapped bare new calls in shared_ptr
JanNiklasB 7de1e3a
reworked memory in tests
JanNiklasB 7075b4f
replaced pointer with ref_ptr
JanNiklasB fed3880
bugfix
JanNiklasB 6fff4e0
added virtual destructors for base classes
JanNiklasB 9871af2
added overloads for raw pointer to every function
JanNiklasB 52e2b2a
added operator== to ref_ptr
JanNiklasB d248a39
bugfix
JanNiklasB a0c3da6
removed _ptr from ref_ptr
JanNiklasB 7c6f7eb
reverted tests and fixed tabs
JanNiklasB 0bdfe9b
changed occurences of stack pointer to references
JanNiklasB 30e7021
added _raw_ptr only for stack pointer reference
JanNiklasB 9cb2cad
small bugfix, still encountering double frees
JanNiklasB a670578
added get_shared functions and changed 0 to NULL
JanNiklasB 1a69f4e
added clone tests
JanNiklasB 841c9cf
removed .get() functions where they don't belong
JanNiklasB 118840c
fixed emission call behaviour
JanNiklasB 6fd8e84
reimplemented workflows
JanNiklasB 61bf8b5
fixed wrong indents
JanNiklasB 4f5d0bb
removed raw pointer overloads
JanNiklasB fbc9ed7
added --output-on-failure to ctests
JanNiklasB bd67452
switched operator== to operator T*
JanNiklasB 4c16b51
added shared_ptr to some classes, most derived still missing
JanNiklasB f50b43b
attempted fix for Vector3 python issue
JanNiklasB dfa4c92
Merge remote-tracking branch 'origin/master' into ReferencedClassRemoval
JanNiklasB 345d55a
Merge remote-tracking branch 'MASTER/master' into ReferencedClassRemoval
JanNiklasB 9508933
changed <> back to ""
JanNiklasB d965f5a
reversed other changed that were not needed
JanNiklasB fd21a47
Attempt to fix "will be initialized after" warnings
JanNiklasB 8beacb3
reverted back workflows
JanNiklasB 74a35dc
Resolved issue encountered in the ubuntu workflows
JanNiklasB a6b10a0
Added weights to calculation of avg energy
JanNiklasB c49f021
Merge remote-tracking branch 'CRPropa/CRPropa3/master' into Reference…
JanNiklasB 39927d2
Merge remote-tracking branch 'CRPropa3/master' into ReferencedClassRe…
JanNiklasB 1b6880f
Merge remote-tracking branch 'GDMarco/PositionDependentPhotonFieldOpt…
JanNiklasB cc61f33
changed `operator =` argument to `nullptr_t`
JanNiklasB 44b314e
added instructions for plugin compability with older CRPropa Versions
JanNiklasB 99849b2
replaced Version.h by configurable Version.h.in to define new CRPROPA…
JanNiklasB 273a0d4
included crprop/Version.h in module so user have immidiate acces to i…
JanNiklasB 622295a
Merge remote-tracking branch 'CRPropa3/master' into ReferencedClassRe…
JanNiklasB fa3bd7c
updated plugin to now removed `Referenced` class
JanNiklasB d03144c
Added changes to CHANGELOG.md
JanNiklasB 8f2694e
reverted unintended changes
JanNiklasB ca30769
reversed meaningless changes in tests and corrected error in CMakeLis…
JanNiklasB e85204c
added venv install to old-crpropa tests
JanNiklasB bf8157c
fix not working tests by activating python venv in every step
JanNiklasB a95e7ca
fixed plugin not finding crpropa in venv and _crpropa.so not finding …
JanNiklasB 1825f3d
fixing plugin tests (#5)
JanNiklasB 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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -12,6 +12,7 @@ install/ | |
| lib/ | ||
| share/ | ||
| doc/api/ | ||
| .vscode/ | ||
|
|
||
| !final_doc/ | ||
| !conda-bld/build.sh | ||
|
|
||
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
Oops, something went wrong.
Oops, something went wrong.
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.
If I understand the logic correctly, this will test the old CRPropa version 3.3 with the plugin template at this point, not the new plugin with your compatibility check.
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.
yes, somehow I did not think about this, but yes the plugin template is also set to 3.3 .
Pulling the 3.3 repository manually and installing it should fix it.