Skip to content

read_interpolated_fsgrid_variable() bug fix#246

Merged
alhom merged 3 commits intofmihpc:masterfrom
kostahoraites:read_fsgrid_variable
Jun 19, 2024
Merged

read_interpolated_fsgrid_variable() bug fix#246
alhom merged 3 commits intofmihpc:masterfrom
kostahoraites:read_fsgrid_variable

Conversation

@kostahoraites
Copy link
Copy Markdown
Contributor

fixed small bugs in read_interpolated_fsgrid_variable(). The function was having trouble interpolating masked arrays, or fs grid vector variable data.

… was having trouble interpolating masked arrays, or fs grid vector variable data.
@kostahoraites kostahoraites marked this pull request as ready for review June 18, 2024 06:46
Comment thread pyVlsv/vlsvreader.py Outdated
if (len(periodic)!=3):
raise ValueError("Periodic must be a list of 3 booleans.")

\
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Um, what is this? ^^'

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks like a typo \

@alhom
Copy link
Copy Markdown
Member

alhom commented Jun 19, 2024

Merge conflict remains, otherwise ok.

@kostahoraites
Copy link
Copy Markdown
Contributor Author

kostahoraites commented Jun 19, 2024

is that because jonas's PR was basically fixing the same line?

--> indeed that seems to be the case. I'll edit to include his solution.

… the dimensionality of vector variable fsgrid data.
@alhom alhom merged commit a184153 into fmihpc:master Jun 19, 2024
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

Successfully merging this pull request may close these issues.

2 participants