-
Notifications
You must be signed in to change notification settings - Fork 19
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
Showing
2 changed files
with
27 additions
and
7 deletions.
There are no files selected for viewing
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
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
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -3,7 +3,7 @@ | |
INSTALL_REQUIRES = [ | ||
'numpy', | ||
'scipy', | ||
'matplotlib', | ||
'matplotlib>=1.3.0', | ||
'obspy'] | ||
|
||
setup( | ||
|
f08ecee
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@megies Do you think
matplotlib>=1.3.0
could be changed to
matplotlib>=1.1.1
matplotlib 1.1.1 is the last stable version on Debian and Ubuntu. Thus it is not very easy to install seedlink_plotter for some users.
f08ecee
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm.. I can't find the reason for requesting 1.3.0 right now.. did you try and run it on 1.1.1 (all 2-3 different options single/multistation etc.)? If it runs.. sure. Any version that runs the code is fine.
f08ecee
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have tested with 1.1.1 without errors. So I lowered the requirement for matplotlib.
f08ecee
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, good to hear it works on 1.1.1 also!