Menu

#2421 Jython simplify to get parameters fails with code

nextrelease
open
nobody
None
5
2022-04-20
2021-12-18
No

Sadie has a case where the GUI isn't created properly, which prevents the script from being used. This happens a bit and I'm surprised there's not another ticket. But I'm adding it here and other similar cases should be added. I spent a good amount of time cleaning this up early this year, and it has worked pretty well. Also with completions there are issues, but this is a blocker and needs to be resolved.

Discussion

  • Jeremy Faden

    Jeremy Faden - 2021-12-18

    The code is at
    git@github.umn.edu:tetri006/RBSP_FU.git, under Master_dialplot.jy.

    I added a new action to the editor, [Right-Click]->Actions->Developer->"Show simplified for parameters"

     
  • Jeremy Faden

    Jeremy Faden - 2022-04-20

    Sadie provided another script (/home/jbf/ct/autoplot/u/2022/sadie/20220411/bin_chorus_micro_mageis.jy) which showed the problem, this time with:

    • try/catch wasn't properly handled. Now they are culled.
    • if/else prevented configuration from being loaded. Now the "correct" way to do this is to load a default config and then overwrite it.
    • ds[i]=... wouldn't be culled if ds hadn't been defined.

    ctrl-shift-f12 is now show parameters, so that this will be useful to the scientists as well. These were fixes similar to others, where the initial coding was flawed because I would ignore cases I didn't think about, instead of asserting I didn't handle them.

     

    Last edit: Jeremy Faden 2022-04-20
MongoDB Logo MongoDB