Skip to content

[MINOR]: Fix Downloads#121

Open
vibhatha wants to merge 4 commits intoLDFLK:mainfrom
vibhatha:fix-downloads
Open

[MINOR]: Fix Downloads#121
vibhatha wants to merge 4 commits intoLDFLK:mainfrom
vibhatha:fix-downloads

Conversation

@vibhatha
Copy link
Member

@vibhatha vibhatha commented Feb 1, 2026

The download zip option was downloading archived files. This PR updates it to latest files.

yasandu0505 and others added 3 commits February 1, 2026 06:47
* fix : fix some typo errors and column rows mismatch in data

* feat : new version of data

* feat : complete 2021 & 2022 data

* change the data structure

* feat : adding data contribution page

* feat : changing some data

* fix: minor newline removal

* fix: remove falty dir
- Updated update_dataset_index.py to generate ZIPs from data/statistics/
- Regenerated all year-based ZIP files with correct data
- Added 2025_Data.zip for new budget data

Downloads now match what's shown in the Interactive Browser.
- Removed find_missing_datasets.py script
- Removed call from prebuild.py
- missing-datasets.md is no longer generated
@gemini-code-assist
Copy link
Contributor

Summary of Changes

Hello @vibhatha, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

This pull request addresses a critical issue in the download functionality, ensuring that users receive the most current and published data when using the zip download option. It achieves this by refining the data source for zip generation and simplifying the overall build process by removing an unrelated script.

Highlights

  • Download Fix: The pull request resolves an issue where the download zip option was providing archived files instead of the latest data.
  • Targeted Data Source for Downloads: The update_dataset_index.py script has been updated to specifically generate ZIP files from the data/statistics directory, ensuring that only published data is included in downloads.
  • Script Removal: The find_missing_datasets.py script and its invocation in prebuild.py have been removed, streamlining the pre-build process.
Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help /gemini help Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution.

Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request addresses an issue with the download ZIPs by correctly updating the data path in scripts/update_dataset_index.py to data/statistics. This change ensures that only published data is included in the archives, which aligns with the PR's goal.

The PR also removes the find_missing_datasets.py script and its execution from the prebuild process. This seems to be a separate cleanup task. To maintain focus, it's often better to handle unrelated changes in a separate pull request. If this removal is intentional and part of a larger cleanup, please consider updating the PR description to reflect this for better tracking.

The code changes are correct, and I have no other issues to report.

@github-actions
Copy link

github-actions bot commented Feb 1, 2026

PR Preview Action v1.8.1

QR code for preview link

🚀 View preview at
https://LDFLK.github.io/datasets/pr-preview/pr-121/

Built to branch gh-pages at 2026-02-01 01:41 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

@github-actions
Copy link

github-actions bot commented Feb 1, 2026

📦 Documentation Build

Status Artifact
✅ Build successful Download docs-build-pr-121

To preview locally:

  1. Click the artifact link above
  2. Scroll to "Artifacts" section and download docs-build-pr-121
  3. Extract the zip file
  4. Run npx serve . in the extracted folder
  5. Open http://localhost:3000

Built from commit 9e3b40f

Copy link
Member

@zaeema-n zaeema-n left a comment

Choose a reason for hiding this comment

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

LGTM

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