search

NCAR / VAPOR

VAPOR is the Visualization and Analysis Platform for Ocean, Atmosphere, and Solar Researchers
https://www.vapor.ucar.edu/
BSD 3-Clause "New" or "Revised" License
178 stars 49 forks source link

Messed up params #1669

Open shaomeng opened 5 years ago

shaomeng commented 5 years ago

In SettingsParams, so many parameter items share the same const strings. I'm pretty sure the params database isn't functioning well under this circumstance. Maybe this is the cause of half-paralyzed Settings tab...

This code snippet is just one example.

https://github.com/NCAR/VAPOR/blob/e8dbaf8cbdd68fbb54d7d5d2a7470f61e58534c2/apps/vaporgui/SettingsParams.cpp#L50-L53

shaomeng commented 5 months ago

Problem still exists. Bumping up the priority.