Skip to content
This repository has been archived by the owner on Mar 24, 2021. It is now read-only.

Clean up NuRadioRecoio #252

Open
christophwelling opened this issue Aug 31, 2020 · 1 comment
Open

Clean up NuRadioRecoio #252

christophwelling opened this issue Aug 31, 2020 · 1 comment
Assignees
Labels
enhancement New feature or request

Comments

@christophwelling
Copy link
Collaborator

The way NuRadioRecoio works right now is very confusing and can easily lead to errors. For example, the __scan_files method overwrites itself with another function that doesn't even take the same parameters the first time it is called. There has to be a better way to handle this.

@anelles
Copy link
Collaborator

anelles commented Jan 5, 2021

I moved this to won't fix before the merge, because it will require significant work and will take longer. If anyone thinks we should do this before merging with NuRadioMC, please speak up.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants