Pull Request #3
Open
Pull Request #3
Conversation
This file contains hidden or 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
Contributor
|
Hmm, this is strange. It looks like some of this commit is new and other parts of it are undoing some of @tvoran's pep8 fixes (e.g., adding random spaces, making multi-line strings back into a long single line, etc.). Can you try merging just your new code (for running large datasets) with the latest automaton master code and updating the pull request? |
dmurtari
pushed a commit
to dmurtari/automaton
that referenced
this pull request
Feb 10, 2014
Added lib/config.py with classes that retrieve and store options from the config files Implemented classes Cloud, Clouds, Cluster, Clusters Update cu-csc#1: changed Log.info to Log.debug Update cu-csc#2: removed etc/automaton.conf Update cu-csc#3: added a check before registering a key Update cu-csc#4: replaced string concatenation in %s Update cu-csc#5: added file default locations in the help for command line arguments Update cu-csc#6: replaces old function names with log_info and get_fqdns
dmurtari
pushed a commit
to dmurtari/automaton
that referenced
this pull request
Feb 10, 2014
Added config files: global.conf, benchmarking.conf, clouds.conf
dmurtari
pushed a commit
to dmurtari/automaton
that referenced
this pull request
Apr 30, 2014
Added lib/config.py with classes that retrieve and store options from the config files Implemented classes Cloud, Clouds, Cluster, Clusters Update cu-csc#1: changed Log.info to Log.debug Update cu-csc#2: removed etc/automaton.conf Update cu-csc#3: added a check before registering a key Update cu-csc#4: replaced string concatenation in %s Update cu-csc#5: added file default locations in the help for command line arguments Update cu-csc#6: replaces old function names with log_info and get_fqdns
dmurtari
pushed a commit
to dmurtari/automaton
that referenced
this pull request
Apr 30, 2014
Added config files: global.conf, benchmarking.conf, clouds.conf
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
No description provided.