MSS icon indicating copy to clipboard operation
MSS copied to clipboard

GUI FOR AUTOMATED PLOTTING 2- update the GUI

Open Preetam-Das26 opened this issue 1 year ago • 7 comments

Purpose of PR?:

Fixes #

Does this PR introduce a breaking change?

If the changes in this PR are manually verified, list down the scenarios covered::

Additional information for reviewer? : Mention if this PR is part of any design or a continuation of previous PRs

Does this PR results in some Documentation changes? If yes, include the list of Documentation changes

Checklist:

  • [ ] Bug fix. Fixes #
  • [ ] New feature (Non-API breaking changes that adds functionality)
  • [ ] PR Title follows the convention of <type>: <subject>
  • [ ] Commit has unit tests

Preetam-Das26 avatar Aug 15 '24 21:08 Preetam-Das26

On an operation the filename column is used (ok), but an operation has no extension

I start with AIRS(ECMWF) and click then Add - something of the data is changed

example_add

How and What?

ReimarBauer avatar Aug 16 '24 09:08 ReimarBauer

On an operation the filename column is used (ok), but an operation has no extension

I start with AIRS(ECMWF) and click then Add - something of the data is changed

example_add example_add

How and What?

lookslike I had a transmission failure.

ReimarBauer avatar Aug 16 '24 09:08 ReimarBauer

when I resize it also or also change the width of the column the left treewidget keeps a lot of space. Could you enable that both take only the space they needs?

image

ReimarBauer avatar Aug 16 '24 09:08 ReimarBauer

While the UI in the Multilayer selector can use direct the cached images from the defined dir in the configuration. For me it is tmp/msui_wms_cache the autoplot does on each load the capabilities. This needs usually only done once. This gives a delay. Please look if you can skip this "get capability" request.

example_wmsc

ReimarBauer avatar Aug 16 '24 09:08 ReimarBauer

When storing such a configuration one has to add .json. There is suppot by qt for this https://doc.qt.io/qt-6/qfiledialog.html#defaultSuffix-prop

reloading from file works too. :)

Download Plots crashes

default
Style 'autolog' not found for layer 'CLaMS_PL.mole_fraction_of_CH3Br_in_air_pl'.
log  | 
Style 'autolog' not found for layer 'CLaMS_PL.mole_fraction_of_CH3Br_in_air_pl'.
Usage: autoplot_gui [OPTIONS]
Try 'autoplot_gui --help' for help.

Error: Invalid value for '--intv': '' is not a valid integer.
Speicherzugriffsfehler

ReimarBauer avatar Aug 16 '24 10:08 ReimarBauer

Restoring the level is not implemented?

ReimarBauer avatar Aug 16 '24 10:08 ReimarBauer

The recent change creates an error for me

problem

ReimarBauer avatar Aug 19 '24 08:08 ReimarBauer

Because of the github behaviour change, this crashes

File "/opt/conda/envs/mss-develop-env/lib/python3.11/site-packages/_pytest/assertion/rewrite.py", line 184, in exec_module
    exec(co, module.__dict__)
  File "/__w/MSS/MSS/conftest.py", line 39, in <module>
    import tests.constants as constants
  File "/__w/MSS/MSS/tests/constants.py", line 40, in <module>
    SHA = repo.head.object.hexsha[0:10]
          ^^^^^^^^^^^^^^^^

we have an updated stable merged to develop https://github.com/Open-MSS/MSS/pull/2537/files#diff-2ba1f11e715e19bf607aa39b8541650e9cc7062cea55288accced904af34d3ca

ReimarBauer avatar Sep 26 '24 08:09 ReimarBauer

with temporary applying https://github.com/Open-MSS/MSS/pull/2537/files#diff-2ba1f11e715e19bf607aa39b8541650e9cc7062cea55288accced904af34d3ca

542 passed, 13 skipped, 32 warnings in 434.61s (0:07:14)

ReimarBauer avatar Sep 30 '24 18:09 ReimarBauer