Skip to content
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

Updates to Schema, Tests, Workers, Documentation #1848

Merged
merged 40 commits into from
May 27, 2022
Merged

Updates to Schema, Tests, Workers, Documentation #1848

merged 40 commits into from
May 27, 2022

Conversation

sgoggins
Copy link
Member

Updates tested and functional.

Biki-das and others added 30 commits April 24, 2022 23:21
Signed-off-by: Biki Das <bikid475@gmail.com>
```
2022-04-14 17:10:28,581,581ms [PID: 984587] workers.pull_request_worker.50471 [DEBUG] Extracting data from source in pr model. ERROR: invalid literal for int() with base 10: '909941810.0'
Traceback (most recent call last):
  File "/bigdisk/augur-vmware/workers/pull_request_worker/pull_request_worker.py", line 554, in pk_source_increment_insert
    prs_insert = [
  File "/bigdisk/augur-vmware/workers/pull_request_worker/pull_request_worker.py", line 558, in <listcomp>
    'pr_src_id': int(str(pr['id']).encode(encoding='UTF-8').decode(encoding='UTF-8')),#1-22-2022 inconsistent casting; sometimes int, sometimes float in bulk_insert
ValueError: invalid literal for int() with base 10: '909941810.0'
2022-04-14 17:10:28,581,581ms [PID: 984587] workers.pull_request_worker.50471 [INFO] 0 insertions are needed and 0 updates are needed for pull_requests
2022-04-14 17:10:28,581,581ms [PID: 984587] workers.pull_request_worker.50471 [INFO] Preparing to enrich data.

2022-04-14 19:07:07,129,129ms [PID: 984587] workers.pull_request_worker.50471 [DEBUG] Extracting data from source in pr model. ERROR: 'cntrb_id'
Traceback (most recent call last):
  File "/bigdisk/augur-vmware/workers/pull_request_worker/pull_request_worker.py", line 554, in pk_source_increment_insert
    prs_insert = [
  File "/bigdisk/augur-vmware/workers/pull_request_worker/pull_request_worker.py", line 572, in <listcomp>
    int(pr['cntrb_id']) ## cast as an int because of an otherwise inexplicable error.
KeyError: 'cntrb_id'
2022-04-14 19:07:07,130,130ms [PID: 984587] workers.pull_request_worker.50471 [INFO] 0 insertions are needed and 0 updates are needed for pull_requests
2022-04-14 19:07:07,130,130ms [PID: 984587] workers.pull_request_worker.50471 [INFO] Preparing to enrich data.
```
Added literal syntax to create the data structure
augur_new_contributors preserves all the prior data used in the visualization API, but removes repo_id parameter. The materialized view performs 89 times faster, @andrewbrain
@sgoggins sgoggins self-assigned this May 27, 2022
@sgoggins sgoggins added installation Related the installation process API Related to Augur's metrics API metric Related to metrics, new or otherwise CHAOSS Issues that relate directly to our goal of being a good reference implementation of CHAOSS metrics documentation Updates documentation usability Usability related issues (bugs, features, etc) server Related to the Augur server docker Related to our Docker images database Related to Augur's unifed data model security Related to keeping Augur secure add-feature Adds new features critical-fix Should be addressed before any other issue/PRs dependencies Pull requests that update a dependency file python Pull requests that update Python code devops Development Operations labels May 27, 2022
@sgoggins sgoggins merged commit 08ef23d into main May 27, 2022
@sgoggins sgoggins deleted the spg-patch-abg branch November 21, 2022 14:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
add-feature Adds new features API Related to Augur's metrics API CHAOSS Issues that relate directly to our goal of being a good reference implementation of CHAOSS metrics critical-fix Should be addressed before any other issue/PRs database Related to Augur's unifed data model dependencies Pull requests that update a dependency file devops Development Operations docker Related to our Docker images documentation Updates documentation installation Related the installation process metric Related to metrics, new or otherwise python Pull requests that update Python code security Related to keeping Augur secure server Related to the Augur server usability Usability related issues (bugs, features, etc)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants