Marchowes / pyProm

Surface Network Analyzer.
BSD 3-Clause "New" or "Revised" License
6 stars 2 forks source link

PSEUDO DISCUSSION: Current state of feature analysis. #41

Closed Marchowes closed 7 years ago

Marchowes commented 7 years ago

I've got a sort of mishmash of methods that seems to mostly work. I'll be using this to keep track of that.

Marchowes commented 7 years ago

Going to stick with the original fullscan type of analysis since all other methods introduce bugs with finding summits. The problems with saddles can be fixed with summit walks.

Marchowes commented 7 years ago

https://github.com/Marchowes/pyProm/pull/43 so far has fixed a number of bugs. We now have fully accurate Summit discovery for the first time. Saddle discovery still needs some work.

Marchowes commented 7 years ago

A number of variables in analyze() need to be "cleared on execution", summit lists and saddles mainly, but others as well.