search

steo85it / pyxover

Altimetry Analysis Tools For Planetary Geodesy
GNU General Public License v3.0
3 stars 3 forks source link

Combination number in pyXover.py #7

Open wdesprats opened 1 year ago

wdesprats commented 1 year ago

The sets in pyXover.py (misy) are hard coded, and depends on the instrument name. The combination number is passed as argument, but could the combination be directly chosen outside of Pyxover main?

For instance, passing misycmb[par] instead of par (= int(cmb_y_in))?

steo85it commented 1 year ago

Good idea! I totally agree, it would be much cleaner and easier to maintain. Thanks, will do!

wdesprats commented 1 year ago

misycmb_par was introduced to replace misycmb[par] in e63e4db. However, cmb_y_in is still used in pyxover.main() for file naming in BELA case. Some adaptation might be needed to only pass misycmb_par.