Skip to content

Conversation

@bch0w
Copy link
Member

@bch0w bch0w commented Nov 20, 2024

Bugfixes from #47 #48

  • Remove plot axis formatter which forced axis bounds [0,1] in some instances
  • Bugfix synthetic waveform label was being incorrectly set
  • Changed misfit label $\chi$ format from float to scientific notation as misfit can be << 1
  • Allow Manager.load() to work even without valid StationXML attribute in ASFDataSet
  • Bugfix: no window selection did not allow measure to be called due to some
    incorrect referencing

bch0w added 3 commits August 22, 2024 14:01
* bugfix: when component list does not match the internal streams (e.g., seisflows inversion that is processing one component at a time), and no windows are selected, measure fails to run and throws an IndexError because it's tries to index out of the Stream. fixed to remove dependence on component list

* bump version and update changelog
* allow Manager.load() to work without the need for a StationXML attribute, which happens for synthetic only cases

* removed plotting axis formatter which was forcing axis bounds [0,1] when the fraction between min and max y lim values was <1, which will happen if only have positive or negative values, but we don't want this to happen because we will likely always want plots to straddle 0, especially when showing adjoint sources which will straddle 0 even if waveforms are all one value

* bugfix label for synthetic waveforms was incorrect

* change misfit label formatting on waveform plots from floating point to sci notation as misfit is usually << 1

* updating test figure

* bump patch version and update changelog
@bch0w bch0w merged commit 3605c7c into master Nov 20, 2024
@bch0w bch0w deleted the devel branch November 20, 2024 00:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants