ENH: adding gabor filters to ndimage #4
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
updating this scipy to match the master
ryanlu41
pushed a commit
that referenced
this pull request
May 12, 2020
57f9aa928 Merge remote-tracking branch 'upstream/master' f6b518d13 Remove print statements 9970e1ca7 Change order of model status enums and export enums to make consistency automatic; found upstream bug in simplex solver 1e06c8a51 Merge pull request scipy#333 from ERGO-Code/dev-presolve e6b6f3162 actually moved files now 5d5faf597 scaffold files copied out. TestPresolve runs OK: now time to extract exec details from presolve. 689a24ebc path to files of HiGHS or scaffold fixed. TestPresolve works. c391eb362 instances added 6d339204b test presolve now not finding test files: the scaffold lib does not know where check/ is. it is easiest if we add files to scaffold and use them for test. ba10975f8 cmake fixes scaffold building with Highs.h in TestPresolve e53363dab added failing line to TestPresolve. cmake files fixes removed two issues. still get an error eef016f10 Merge pull request scipy#332 from ERGO-Code/ImproveCAPIexample aec2f6f5a Added a lot of comments to examples/call_highs_from_c.c 84e9f0e50 Names of arrays in examples/call_highs_from_c.c now match those in highs_c_api.cpp, and Highs_passLp is now correct in handling astart with only numcol entries 8b85654d8 command line options library moved to app/ so out of src/ 4d1534338 uses interface libraries in cmake b6a902fd6 Merge pull request scipy#318 from ERGO-Code/newlpfilereader 134c2b0f3 Merge branch 'master' into dev-presolve deed95b6e Merge remote-tracking branch 'ergo/master' 04472e386 fixed bug in .lp filewriter 8f54ecee3 fixed compilation issues 8483eaf12 Merge branch 'master' into newlpfilereader 79cec5994 Check primal feasibility to know if solution exists 02e0c8cb4 separated test from dev namespaces. next: add tests for component and dev 1fab41ba5 Merge pull request #4 from galabovaa/dev-presolve c15eeaa25 removed unnecesary code in LP filereader 94e82f41d added missing header 29055001c updated LP filereader d3108f2cc pull bugfix from reader repository f1b684001 replace LP filereader git-subtree-dir: scipy/optimize/_highs git-subtree-split: 57f9aa928c5232092819807e726d3444650d42ad
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Reference issue
Closes scipy#11583
What does this implement/fix?
This adds gabor_filters to the ndimage module. For verification of correctness of algorithm, see
https://github.com/NeuroDataDesign/mouselit/blob/master/ryan/Mouselight/Gabor%20Filter%20Verification.ipynb
Additional information
Created by Thomas Athey and Ryan Lu