-
Notifications
You must be signed in to change notification settings - Fork 10
refactor!: massive cleanup #293
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
Changes from 46 commits
Commits
Show all changes
56 commits
Select commit
Hold shift + click to select a range
ff7c211
style: fix indentation in `.java` files
c-dilks 605697e
build: use the same compiler plugins
c-dilks 4d0680e
refactor!: remove `main` methods from monitoring classes
c-dilks c4aab6d
perf: remove unused variables, includes, and dead code
c-dilks f7fe5ff
refactor!: remove `plot` methods from monitoring classes
c-dilks 1f506a9
refactor: `ana_2p2` should only call monitor's `processEvent` and `wr…
c-dilks aa4d5b0
refactor!: combine POM modules and organize source tree
c-dilks d240836
fix(ci): build-artifact path
c-dilks 04516f4
fix: package name is `clas.timeline`, not `clas12.timeline`
c-dilks 37f85c1
fix: classpath env var
c-dilks 76037e3
fix: do not require groovy JARs to be in caller classpath
c-dilks b82d1b8
fix: DRY violation of classpath array in `environ.sh`
c-dilks ab4fee1
refactor: rename `main` Java and Groovy files
c-dilks 02f4986
fix: include `groovy-yaml` as a dependency
c-dilks 702b167
fix: try `snakeyaml`
c-dilks 75eddac
build: bump version to 3.0.0
c-dilks 52ed214
ci: tell dependabot to track `groovy.jfrog.io`
c-dilks f44713b
build: revert to `groovy-yaml`; still doesn't fix the bug
c-dilks 0af7444
fix: simplify `install.sh`
c-dilks cd484c4
fix: do not use `run-groovy`
c-dilks 026786a
fix: eliminate `$JYPATH`, which is only used by `run-groovy`
c-dilks 1c93065
fix: use maven-packaged `groovy`, and use `snakeyaml` for parsing
c-dilks 6abed8c
fix: full path to `run-groovy-timeline.sh`
c-dilks 4e5238e
refactor!: remove `$TIMELINE_GROOVY_OPTS` env var
c-dilks 77bc86c
refactor!: remove unused methods in `org.jlab.clas.timeline.histograms`
c-dilks 5e6d8d1
doc: respect combination of `monitoring` and `detectors` maven modules
c-dilks 495b6ba
refactor!: remove `occupancies.java`, for BST and BMT occupancy
c-dilks 4fca8ae
refactor!: remove `deuterontarget.java`
c-dilks e48f63a
refactor: better names for histogramming Java classes
c-dilks 1d9900f
refactor!: remove `dst_mon` histogramming class
c-dilks 604f670
refactor!: remove unused CTOF and FTOF analysis `groovy` classes
c-dilks e3ec7e4
fix(build): `groovy-yaml` is in `groovy-all`
c-dilks 2ac5970
build: bump version to 2.99.0
c-dilks 295d235
refactor: rename analysis class methods...
c-dilks 5a12fca
Merge branch 'main' into v3
c-dilks 8b1cf96
Merge branch 'main' into v3
c-dilks f83c135
fix: consistent RCDB URIs
c-dilks 318d35e
fix: do not require any of `coatjava`'s environment
c-dilks 46f7e48
fix: all `java` calls should use `TIMELINE_JAVA_OPTS`
c-dilks 666c980
fix: remove last remaining `CLASSPATH`
c-dilks 23fccfe
style: snake case
c-dilks 58f74b8
style: remove trailing whitespace
c-dilks 5573d38
doc: more details in setup.md
c-dilks 6ffc5d7
modified: doc/setup.md
c-dilks 3c1dc1b
refactor: more renamings
c-dilks f4de52f
fix: executable path
c-dilks 155abdc
Merge remote-tracking branch 'origin/main' into v3
c-dilks 3ddab69
build: prefer 'plugins-release-local' artifactory repo, which has ONL…
c-dilks 778d5c7
fix: ignore more `jdtls` dotfiles
c-dilks 19f3dd5
Merge remote-tracking branch 'origin/main' into v3
c-dilks e53c33f
build: test `jfrog` repo `plugins-release`
c-dilks a1f4ecd
fix: bump slurm mem request
c-dilks cff4d0e
fix: source env vars in generated scripts
c-dilks 5157761
fix: limit heap space for groovy too
c-dilks 3953c8a
fix: use same options for `java` and `groovy`
c-dilks 822e5a0
build: bump version
c-dilks 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
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
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.
Uh oh!
There was an error while loading. Please reload this page.