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

UI crash at output dashboard with zipped exposure files #286

Closed
sambles opened this issue Jun 27, 2022 · 3 comments · Fixed by #298
Closed

UI crash at output dashboard with zipped exposure files #286

sambles opened this issue Jun 27, 2022 · 3 comments · Fixed by #298
Assignees
Labels
feature New feature or request
Milestone

Comments

@sambles
Copy link
Contributor

sambles commented Jun 27, 2022

Issue Description

Output Dashboard gray screen crashes when loading results from run using zipped exposure run (PiWind)

Version / Environment information

Oasislmf: 1.26.0
Platform branch: platoform-2.0-develop (Azure)
OasisUI: 1.11.3

Example data / logs

input$dt_analyses_rows_selected is changed to: 1
analysisId changed to 6
selected analysis ID is 6
updating result$dashboardanaID because submodulesList$step3_configureOutput$dashboardAnaID() changed to: 6
Navigate to Single browse
replacing icons
Updating preselected row because anaID() changed to 6
updating ana outputs data list
Warning in FUN(X[[i]], ...) : NAs introduced by coercion
Warning: Error in : Invalid: Parquet magic bytes not found in footer. Either the file is corrupted or this is not a parquet file.
  2: runApp
  1: oasisui::runOasisui
delete  

piwind_loc_oed.zip
piwind_acc_oed.zip

@RolandASc RolandASc self-assigned this Aug 24, 2022
@RolandASc
Copy link
Contributor

the crash happens because we read some data from the location file in order to produce the summary tab, and we haven't added any generic functionality to handle zipped exposure files

@RolandASc
Copy link
Contributor

it was simple enough to add some code to handle zipped exposure files in the main function retrieving that data, so I have implemented that and this issue is resolved now

it doesn't mean that we will have full feature support for zipped files across the board though, so maybe we'll want to do something more later

@RolandASc RolandASc added the feature New feature or request label Sep 19, 2022
@MattDonovan82
Copy link
Collaborator

@mtazzari would the functionality you mentioned last week regarding splitting the tar files help here?

The UI is generally slow when trying to export or even view the output files, especially if they are quite large.

RolandASc added a commit that referenced this issue Sep 21, 2022
fixed description for additional output files
@sambles sambles linked a pull request Nov 11, 2022 that will close this issue
@awsbuild awsbuild added this to the 1.11.5 milestone Nov 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants