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

EfieldToVoltageConverter does not work for far away channels #159

Open
christophwelling opened this issue Nov 11, 2019 · 4 comments
Open
Labels
bug Something isn't working enhancement New feature or request

Comments

@christophwelling
Copy link
Collaborator

If a channel is too far away from the station center, the efieldToVoltageConverter does not work any more. When the starting bin for the signal in the channel trace is calculated, the result becomes negative, which causes an error.
This can be avoided by increasing the pre_pulse_time, but ideally this should work automatically.

@cg-laser
Copy link
Contributor

can you be more specific of the error? Stacktrace? I don't see why this would be a problem in general, probably the code needs to be made more robust.

@anelles
Copy link
Collaborator

anelles commented Dec 17, 2019

I would mark this as a science problem, rather than a computational problem. We have to develop algorithms that work with extended sources. The current code was never intended to work for more than one surface station.

@christophwelling
Copy link
Collaborator Author

No, this is a computing problem. The efieldToVoltageConverter should make the trace long enough to fit the pulse automatically, but for some reason it can happen that in line 182 it tries to use a negative start_bin, which throws an error. So there is something wrong with how the trace length is calculated vs. how that start_bin is calculated.
Practically, this seems to only happen for very large distances between the channel and the station center, so it's pretty low priority. I just made this issue to keep track.

@anelles
Copy link
Collaborator

anelles commented Jan 5, 2021

Also here, should we just document this issue or do something about it? Especially if it is only low priority as elaborated above.

@anelles anelles added bug Something isn't working enhancement New feature or request labels Jan 5, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants