Skip to content

Conversation

@zhan4329
Copy link
Contributor

@zhan4329 zhan4329 commented Oct 31, 2025

This pull request merges all six open PRs and introduces some other necessary changes. Here are the details.


New Features from Previous PRs

Code Refactoring

  • Refactored some codes to make the ui more user-friendly
  • Refactored some codes for better logic

Dependency Management

  • Added two new packages (colorcet and datashader) to requirements.txt to support the newly introduced heatmap mode in scatter plot
  • Updated environment.yml:

Development and Deployment Infrastructure

  • Updated Dockerfile:
  • Updated .gitignore to exclude an unnecessary Jupyter-related file format

Documentation

  • Updated README.md to include names of this year's Data Mine Team contributors and TA
  • Added a CONTRIBUTING.md with links to the copilot-based instructions as well as SPAC Package project's contributing guide

Remarks About the Six Open PRs

Remarks About Other Changes

  • We are not fully confident of the changes we made to Dockerfile, especially the Python version. We haven't found any new issues though.

root and others added 30 commits July 6, 2025 10:40
zhan4329 added a commit to Saran-Nag/SPAC_Shiny-Purdue-2025 that referenced this pull request Nov 1, 2025
Small fixes:
- Update environment.yml to make it consistent with requirement.txt

Comments:
- Update .gitignore to include Jupyter section
- Write clearer comments in Dockerfile; remove redundant comments

Files clean up:
- Remove redundant quickstart guide
zhan4329 added a commit to Saran-Nag/SPAC_Shiny-Purdue-2025 that referenced this pull request Nov 1, 2025
Small fixes:
- Update environment.yml to make it consistent with requirement.txt
- Update README.md to include TA's name

Comments:
- Update .gitignore to include Jupyter section
- Write clearer comments in Dockerfile; remove redundant comments

Files clean up:
- Remove redundant quickstart guide
@zhan4329 zhan4329 force-pushed the merge/high-school-prs-issue-7 branch 4 times, most recently from fa81212 to 3dc2a08 Compare November 1, 2025 01:01
small fixes
- Update environment.yml to make it consistent with requirement.txt
- Update README.md to include TA's name

docs clean-up
- Update .gitignore to include Jupyter section
- Write clearer comments in Dockerfile; remove redundant comments
- Remove redundant quickstart guide

server files clean-up: anno_vs_anno_server.py, feat_vs_anno_server.py, nearest_neighbor_server.py, scatterplot_server.py

ui files clean-up: anno_vs_anno_ui.py, annotations_ui.py, feat_vs_anno_ui.py, features_ui.py, scatterplot_ui.py
@zhan4329 zhan4329 force-pushed the merge/high-school-prs-issue-7 branch from 3dc2a08 to 53e8c5f Compare November 1, 2025 03:00
@zhan4329 zhan4329 marked this pull request as ready for review November 1, 2025 16:14
@zhan4329
Copy link
Contributor Author

zhan4329 commented Nov 1, 2025

@georgezakinih Hi George, could you review this Pull Request for high schoolers' PRs as well as some other necessary changes done by us? Thank you!

@zhan4329 zhan4329 force-pushed the merge/high-school-prs-issue-7 branch from c0c4621 to 95d41db Compare November 3, 2025 18:10
@zhan4329 zhan4329 closed this Nov 4, 2025
zhan4329 added a commit to Saran-Nag/SPAC_Shiny-Purdue-2025 that referenced this pull request Nov 5, 2025
small fixes
- Update environment.yml to make it consistent with requirement.txt
- Update README.md to include TA's name

docs clean-up
- Update .gitignore to include Jupyter section
- Write clearer comments in Dockerfile; remove redundant comments
- Remove redundant quickstart guide

server files clean-up: anno_vs_anno_server.py, feat_vs_anno_server.py, nearest_neighbor_server.py, scatterplot_server.py

ui files clean-up: anno_vs_anno_ui.py, annotations_ui.py, feat_vs_anno_ui.py, features_ui.py, scatterplot_ui.py
zhan4329 added a commit to Saran-Nag/SPAC_Shiny-Purdue-2025 that referenced this pull request Nov 5, 2025
…AP#61

small fixes
- Update environment.yml to make it consistent with requirement.txt
- Update README.md to include TA's name

docs clean-up
- Update .gitignore to include Jupyter section
- Write clearer comments in Dockerfile; remove redundant comments
zhan4329 added a commit to Saran-Nag/SPAC_Shiny-Purdue-2025 that referenced this pull request Nov 5, 2025
…AP#61

small fixes
- Update environment.yml to make it consistent with requirement.txt
- Update README.md to include TA's name

docs clean-up
- Update .gitignore to include Jupyter section
- Write clearer comments in Dockerfile; remove redundant comments
zhan4329 added a commit to Saran-Nag/SPAC_Shiny-Purdue-2025 that referenced this pull request Nov 5, 2025
…AP#61

small fixes
- Update environment.yml to make it consistent with requirement.txt
- Update README.md to include TA's name

docs clean-up
- Update .gitignore to include Jupyter section
- Write clearer comments in Dockerfile; remove redundant comments
zhan4329 added a commit to Saran-Nag/SPAC_Shiny-Purdue-2025 that referenced this pull request Nov 6, 2025
…AP#61

small fixes
- Update environment.yml to make it consistent with requirement.txt
- Update README.md to include TA's name

docs clean-up
- Update .gitignore to include Jupyter section
- Write clearer comments in Dockerfile; remove redundant comments
zhan4329 added a commit to Saran-Nag/SPAC_Shiny-Purdue-2025 that referenced this pull request Nov 6, 2025
…AP#61

small fixes
- Update environment.yml to make it consistent with requirement.txt
- Update README.md to include TA's name

docs clean-up
- Update .gitignore to include Jupyter section
- Write clearer comments in Dockerfile; remove redundant comments
zhan4329 added a commit to Saran-Nag/SPAC_Shiny-Purdue-2025 that referenced this pull request Nov 7, 2025
…AP#61

small fixes
- Update environment.yml to make it consistent with requirement.txt
- Update README.md to include TA's name

docs clean-up
- Update .gitignore to include Jupyter section
- Write clearer comments in Dockerfile; remove redundant comments
zhan4329 added a commit to Saran-Nag/SPAC_Shiny-Purdue-2025 that referenced this pull request Nov 7, 2025
small fixes
- Update environment.yml to make it consistent with requirement.txt
- Update README.md to include TA's name

docs clean-up
- Update .gitignore to include Jupyter section
- Write clearer comments in Dockerfile; remove redundant comments
- Remove redundant quickstart guide

server files clean-up: anno_vs_anno_server.py, feat_vs_anno_server.py, nearest_neighbor_server.py, scatterplot_server.py

ui files clean-up: anno_vs_anno_ui.py, annotations_ui.py, feat_vs_anno_ui.py, features_ui.py, scatterplot_ui.py
zhan4329 added a commit to Saran-Nag/SPAC_Shiny-Purdue-2025 that referenced this pull request Nov 7, 2025
small fixes
- Update environment.yml to make it consistent with requirement.txt
- Update README.md to include TA's name

docs clean-up
- Update .gitignore to include Jupyter section
- Write clearer comments in Dockerfile; remove redundant comments
- Remove redundant quickstart guide

server files clean-up: anno_vs_anno_server.py, feat_vs_anno_server.py, nearest_neighbor_server.py, scatterplot_server.py

ui files clean-up: anno_vs_anno_ui.py, annotations_ui.py, feat_vs_anno_ui.py, features_ui.py, scatterplot_ui.py
zhan4329 added a commit to Saran-Nag/SPAC_Shiny-Purdue-2025 that referenced this pull request Nov 7, 2025
zhan4329 added a commit to Saran-Nag/SPAC_Shiny-Purdue-2025 that referenced this pull request Nov 11, 2025
zhan4329 added a commit to Saran-Nag/SPAC_Shiny-Purdue-2025 that referenced this pull request Nov 12, 2025
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.

7 participants