forked from hackforla/311-data
-
Notifications
You must be signed in to change notification settings - Fork 0
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
Syncing to latest updates. #1
Merged
Merged
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
Bring up to date with Dev
Bring up to date with master
--sqlIngest:DataHandler now has dumpCsvFile method --returns string buffer object for download
Added CSV stream to sqlIngest.py
added dataAnalysis onboarding document draft
Added first iteration of onboarding script, does not auto-install
Reporting endpoint for biggest offenders
--Amended docker onboarding information --Grammar corrections
Updated onboarding doc
further updates to onboarding doc
…eter of the DataHandler().dumpCsvFile method
…; querying for json results successful; also joined ../ to filepath in DataHandler.loadData method
…pt utils.py as it's referenced by other files
…cstrings to functions
Implement redux
Updated webpack to support the CD scripts
Added sodapy to requirements.txt
Making sure CI runs on all PRs
Update master with latest version of dev
Bring dev up to date with upstream
Sanity checked all existing functionality Trying to add postgres utils to action Trying standalone installer Switching to on_push for testing Switching to on_push for testing Eureka, libpq on linux will fix the backend build
Added linting and correctly implemented the backend github action
No Flake errors/warnings allowed
--orm configuration files now in databaseOrm.py package --updateDatabase method of DataHandler now allows partial updates of records in database --runs roughly 90 seconds per 1000 records --around 5000 new records can be expected per day --fixed fetchSocrata method of DataHandler to correctly grab number of records specified in querySize parameter
Bringing up to date with main repo
--documented updateDatabase method --added a couple syntax/linting fixes to sqlIngest file
added changed in local fork to feature modules
Added partial update capability to sqlIngest
Dev update
Updating master with lastest dev
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.
No description provided.