mountetna / monoetna

mono-repository version of etna projects
GNU General Public License v2.0
1 stars 0 forks source link

File / Folder UIs for Vulcan #1249

Closed dtm2451 closed 1 year ago

dtm2451 commented 1 year ago

Overall:

PR To Dos:

***ToDo via portainer:

dtm2451 commented 1 year ago

One BIG note (reminder to check, and do, if needed!): may need to add metis_host to the production vulcan config in portainer! Otherwise, deploy likely breaks all vulcan steps that run a script.

dtm2451 commented 1 year ago

I think we discussed giving pointers to reviewers as focus/entry points... Primarily, I'd love if you ran the workflow:

  1. Put some tsv or csv in dev-metis for a project
  2. Follow code edits in this slack to let the dev systems communicate and use proper targets.
  3. Try out the "Visualization from File" workflow!

Some minor questions:

maflaven commented 1 year ago

I tried changing the metis file to see if it triggers a refresh on the "Review your data frame" step. It only refreshes if the review step is committed.

image

Review step won't refresh in this case.

dtm2451 commented 1 year ago

(Edit: nolonger confused. You're seeing behavior that works entirely as expected!)

I'm confused by the error case you are seeing in https://github.com/mountetna/monoetna/pull/1249#issuecomment-1716105336 @maflaven.

Perhaps we can slack or zoom to figure out how you got to the state you are in as I can't understand how you got there. It's confusing to me because the way things should work, and do for me in the gif attached, is that as soon as you 'commit' to using that 'Pancan_samp...' file, the workflow should refresh and clear or fill in next steps based on matching hashes with the new workflow state. Admittedly, I could understand how this part of the state system works better, so I am very interested in understanding your workflow state as it seems like it does point to a bug.

For me, if I change the path / 'File or Folder' target & commit... Changing from the original ".tsv" to the ".csv" I had also previously used successfully, when I hit 'Commit' the "Review your data frame" remains because there is a past state matching the '.csv' file. (And it looks the same because the ".tsv" and "*.csv" files are equivalent.) If I instead change to the 'xneo1_diagtestnew.csv' file that I'd never used before, when I hit 'Commit' the "Review your data frame" step completely disappears as I expect it should. file-choice-update

dtm2451 commented 1 year ago

Ohhhhhhh! not a bug case... the state did update successfully =). Notice in your image that it refers to 224 rows and 2 columns. I think that matches your file? Certainly doesn't match the test data I'd shared!