-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feature updates and new README #3
base: master
Are you sure you want to change the base?
Conversation
feat(readme) updated readme to current workflow feat(restart_server.sh) changed to use python3 by default
|
||
mkdir superman | ||
cd superman | ||
git clone https://github.com/all-umass/superman-web |
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 think it may be easier to keep the two repositories separate:
pip3 install git+https://github.com/all-umass/superman.git
git clone https://github.com/all-umass/superman-web
cd superman-web
pip3 install matplotlib tornado==4.4.2 pyyaml h5py pandas pywt sklearn
|
||
pip install --only-binary :all: superman matplotlib tornado pyyaml h5py pandas | ||
While 2.7 and 3.4+ are supported, I've only tested with 3.4+ |
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.
Our primary deployment is still using 2.7, so both are tested.
|
||
For running tests, you'll want: | ||
pip3 install matplotlib tornado==4.4.2 pyyaml h5py pandas pywt sklearn |
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.
Ideally we should put these in a requirements.txt file, but that can come later.
for k, cls, mname in info.get('metadata', [])] | ||
if info.get('vector', False): | ||
loader_fn = _generic_vector_loader(meta_mapping) | ||
if entries: |
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.
To avoid indenting everything here, you can do:
if not entries:
continue
geoiplookup $1 | sed 1d | cut -d: -f2 | cut -d' ' -f3- | cut -d, -f2,3 | xargs | ||
geoiplookup $1 | sed 1d | cut -d: -f2 | \ | ||
cut -d' ' -f3- | cut -d, -f2,3 | \ | ||
tr "'" '^' | xargs |
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.
What's the replacement of '
with ^
for?
feat(restart_server.sh) Defaults to python3 rather than python
feat(README.md) Updated readme due to workflow update
feat(backend/dataset_loaders.py) Operates with a blank dataset file if no pre-defined datasets exist