geoschem / integrated_methane_inversion

Integrated Methane Inversion workflow repository.
https://imi.readthedocs.org
MIT License
26 stars 23 forks source link

updates from sarah for calc_sensi #84

Closed laestrada closed 2 months ago

laestrada commented 1 year ago

We should evaluate whether these changes should go into the standard IMI code. Sarah writes: "An important thing to note though is that while it speeds things up, it does require more memory so maybe would depend on the use case if one wants to use my version or the current version. I think there are a lot of things that are different for the large domain case…Like memory default settings, etc."

msulprizio commented 2 months ago

@laestrada @sarahhancock Can this PR be closed? I imagine it's quite outdated at this point. If these updates are still needed, would it be best to create a new PR?

laestrada commented 2 months ago

This will be addressed by James PR speeding up calc_sensi