Skip to content
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

Requesting an empty run as part of a multi_run call with add_run_id_field=False fails #783

Open
jjakob03 opened this issue Dec 1, 2023 · 0 comments
Assignees

Comments

@jjakob03
Copy link

jjakob03 commented Dec 1, 2023

In the somewhat rare use case of

  • loading a run that does not return any entry (in my case, caused by usage of the selection argument)
  • together with at least one other run (so multi_run is called)
  • with the argument add_run_id_field=False

there is an error when trying to fetch the minimum time of an empty array in this line.

I ran into this when trying to load more than 70 runs at the same time with a quite tight selection applied, followed the warning suggesting to do run_id_as_bytes=True but then decided that I don't want to deal with the byte strings type that it produces and that I don't need the run_id at all.

To reproduce, load any list of run_ids with len(run_ids)>1 and data_type of your choice with an impossible selection, e.g.
st.get_array(run_ids,'event_info',selection='(s1_area>42)&(s1_area<42)',add_run_id_field=False).

@dachengx dachengx self-assigned this Dec 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants