Skip to content

Igor's fixes.#15

Closed
samcunliffe wants to merge 33 commits into
mainfrom
niu-igors-fixes
Closed

Igor's fixes.#15
samcunliffe wants to merge 33 commits into
mainfrom
niu-igors-fixes

Conversation

@samcunliffe
Copy link
Copy Markdown
Member

Propagate @IgorTatarnikov's fixes from the @neuroinformatics-unit.

adamltyson and others added 30 commits September 4, 2024 11:00
* Added stars column

* Changed readme
* Set build deploy action to at 6 am UTC every day

* Offset from top of hour to avoid congestion

* Backend implementation for contributor count

* Switch collaborator count to contributor in frontend

* Removed commented out code

* Added collaborator fetch back

* Removed console.log statement
* Added page for NIU repose

* Updated favicon

* Update gitignore in data to ignore all json

* WIP add NIU tab

* WIP backend for multiple orgs

* Backend now takes array of organizations and fetches all

* Front end renders both orgs

* Correct tab highlighting

* Fixed broken import

* Fixed linting rules
* Added page for NIU repose

* Updated favicon

* Update gitignore in data to ignore all json

* WIP add NIU tab

* WIP backend for multiple orgs

* Backend now takes array of organizations and fetches all

* Front end renders both orgs

* Correct tab highlighting

* WIP fetch conda data

* WIP fetch conda data

* Fixed broken import

* Fixed linting rules

* Fixed package.json

* Cache the conda download workflow once a month

* Revereted basePath change

* Updated commit on cacche action

* Moved fetch_parquet to fetchers directory

* Added missing single quote

* Added parquetjs to requirements

* Backend implementation for conda numbers with duckdb

* Fixed filtering and added condaDownloads to front end

* Fixed duplicate code in backend/src/index.ts

* Moved duckdb dependency to backend

* Legacy packages are only loaded if org is brainglobe

* Change cache path for conda-packages

* Removed unnecessary interface
* Priorites NIU content

* Updated favicon

* Rename link from niu to brainglobe
* Avoid looking for the zeroth month for last months conda downloads

* Adjust year if the last month is 12

* Add sleep and retry to PePy query

* Break out of the outer loop as well

* Update sleep time to 1 minute

* Catch not found, always return null when failed

* Update error handling in pepy fetching, set currYear correctly when breaking loop
…, or course (#20)

* Exclude repos in the exluded list, starting with slides, or course

* Remove console.log statement used for testing

* Update utility function

* Make sure conda cache is actually used

* Update sleep time to 70s instead of 60s

* Filter in discussions and issues to make sure all repos have all field
…tributors

Add a max retries to fetching contributors
* Remove projects from repository data fetching, only projectsV2

* Tracks num pepy requests globally, limits calls when near limit

* Removes extra logging used for debugging
@samcunliffe samcunliffe changed the title Niu igors fixes Igor's fixes. Oct 6, 2025
Comment thread .github/workflows/nextjs.yml Outdated
Comment on lines +67 to +81
- name: Restore cache
uses: actions/cache@734d9cb93d6f7610c2400b0f789eaa6f9813e271 # v3
- name: Restore .next cache
uses: actions/cache@v4
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use commit hash here.

Comment thread app/docs/definitions.md
## Welcome to the Open-Source Metrics Documentation

#### How to Read World Health Organization Open-Source Dashboard
#### How to Read SWC/GCNU Neuroinformatics Unit Open Source Dashboard Open-Source Dashboard
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
#### How to Read SWC/GCNU Neuroinformatics Unit Open Source Dashboard Open-Source Dashboard
#### How to Read the UCL Open Source Dashboard

Comment thread app/docs/definitions.md
#### How to Read SWC/GCNU Neuroinformatics Unit Open Source Dashboard Open-Source Dashboard

The Dashboard is currently a snapshot of data from the **World Health Organization** Repositories. All the metrics are based on the _Last Updated Date_ reflected on the top of the dashboard.
The Dashboard is currently a snapshot of data from the **SWC/GCNU Neuroinformatics Unit** Repositories. All the metrics are based on the _Last Updated Date_ reflected on the top of the dashboard.
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
The Dashboard is currently a snapshot of data from the **SWC/GCNU Neuroinformatics Unit** Repositories. All the metrics are based on the _Last Updated Date_ reflected on the top of the dashboard.
The Dashboard is currently a snapshot of data from **UCL**'s repositories. All the metrics are based on the _Last Updated Date_ reflected on the top of the dashboard.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Remove/update logos

Comment thread config.yml Outdated
# Organization to pull metrics from
organization: 'github-community-projects'
# Organization to pull metrics from (must be an array)
organization: ['brainglobe', 'neuroinformatics-unit']
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
organization: ['brainglobe', 'neuroinformatics-unit']
organization: ['ucl']

Comment thread config.yml
# GitHub Pages path to the repository
# Used for handling relative paths for assets
basePath: '/org-metrics-dashboard'
basePath: '/dashboard'
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
basePath: '/dashboard'
basePath: '/org-metrics-dashboard'

@samcunliffe
Copy link
Copy Markdown
Member Author

Need to remove some NIU specifics.

  • Remove NIU specifics.

@samcunliffe samcunliffe marked this pull request as draft October 6, 2025 15:30
@samcunliffe
Copy link
Copy Markdown
Member Author

Will do this via #17

@samcunliffe samcunliffe closed this Oct 9, 2025
@samcunliffe samcunliffe deleted the niu-igors-fixes branch October 15, 2025 17:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants