Skip to content

feat: add load demo data button for the online demo webapp#60

Merged
gcroci2 merged 20 commits intodevfrom
online_demo_gcroci2
May 23, 2025
Merged

feat: add load demo data button for the online demo webapp#60
gcroci2 merged 20 commits intodevfrom
online_demo_gcroci2

Conversation

@gcroci2
Copy link
Contributor

@gcroci2 gcroci2 commented May 22, 2025

I successfully deployed the NPLinker webapp to nplinker-webapp.onrender.com using Render. The deployment is automatically linked to the main branch and updates with every push.

This PR adds a "Load Demo Data" button that allows users to instantly load sample data without manually downloading files. The button automatically downloads the mock dataset from the repository, processes it through the same pipeline as uploaded files, and provides immediate access to all webapp features for testing.

Reliable operation with multiple simultaneous users is ensured:

  • dcc.Store components are client-side by default, ensuring complete isolation between browser sessions
  • File uploads through dash-uploader automatically generate unique filenames using timestamps and random strings
  • Demo data loading includes unique ID generation to prevent filename collisions between concurrent requests

@gcroci2 gcroci2 linked an issue May 22, 2025 that may be closed by this pull request
@gcroci2 gcroci2 changed the base branch from main to dev May 22, 2025 11:46
@gcroci2 gcroci2 requested a review from CunliangGeng May 22, 2025 12:02
Copy link
Member

@CunliangGeng CunliangGeng left a comment

Choose a reason for hiding this comment

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

Great that you found a nice platform to host it!

Since now we have demo service, the README could be restructured to the following order: Online demo, Using the webapp, Installation.

@gcroci2 gcroci2 requested a review from CunliangGeng May 22, 2025 14:31
gcroci2 and others added 5 commits May 23, 2025 09:57
Co-authored-by: Cunliang Geng <c.geng@esciencecenter.nl>
Co-authored-by: Cunliang Geng <c.geng@esciencecenter.nl>
Co-authored-by: Cunliang Geng <c.geng@esciencecenter.nl>
Co-authored-by: Cunliang Geng <c.geng@esciencecenter.nl>
Co-authored-by: Cunliang Geng <c.geng@esciencecenter.nl>
gcroci2 and others added 5 commits May 23, 2025 09:57
Co-authored-by: Cunliang Geng <c.geng@esciencecenter.nl>
Co-authored-by: Cunliang Geng <c.geng@esciencecenter.nl>
@gcroci2 gcroci2 requested a review from CunliangGeng May 23, 2025 08:23
@sonarqubecloud
Copy link

Copy link
Member

@CunliangGeng CunliangGeng left a comment

Choose a reason for hiding this comment

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

:shipit:

@gcroci2 gcroci2 merged commit 5c1a3ec into dev May 23, 2025
3 checks passed
@gcroci2 gcroci2 deleted the online_demo_gcroci2 branch May 23, 2025 10:03
gcroci2 added a commit that referenced this pull request May 23, 2025
* update dev notes with branching and releasing sections

* change test to dev

* set debug false for release

* add detail about debug

* debug true

* remove darkmode components (#58)

* feat: add the possibility to change x-axis in the gm table plot (#59)

* add class_bgcs key to processed_data

* add dropdown option for x-axis

* match styles

* display none selector when there is not data

* make sure that the plot is reset to default x-axis when new file gets uploaded

* fix tests

* feat: make the cutoff filter for the scoring datatable clearer, and add filtering functionality to all columns (#57)

* make cutoff meaning clearer

* add filtering to the candidate links table

* remove unused confusing Aa box

* fix filtering row visualization

* add docs for filters in candidate links tables

* remove wrong operator

* add link to all filtering operators

* align placeholder and user input to the left

* feat: add load demo data button for the online demo webapp (#60)

* add demo data button

* add test for load demo data

* add docs for online demo

* add unique id to loaded demo data

* improve sections order

* add cleanup flag, defaulted to True

* add test for cleanup flag

* change default cutoff to 0

* add readme suggestions

* Update README.md

Co-authored-by: Cunliang Geng <c.geng@esciencecenter.nl>

* Update README.md

Co-authored-by: Cunliang Geng <c.geng@esciencecenter.nl>

* Update README.md

Co-authored-by: Cunliang Geng <c.geng@esciencecenter.nl>

* Update README.md

Co-authored-by: Cunliang Geng <c.geng@esciencecenter.nl>

* Update README.md

Co-authored-by: Cunliang Geng <c.geng@esciencecenter.nl>

* Update README.md

Co-authored-by: Cunliang Geng <c.geng@esciencecenter.nl>

* Update README.md

Co-authored-by: Cunliang Geng <c.geng@esciencecenter.nl>

* use conda instead of venv

* wrap subsections

---------

Co-authored-by: Cunliang Geng <c.geng@esciencecenter.nl>

* fix linter issue

* fix mypy issue

* add back ignore missing imports

---------

Co-authored-by: Cunliang Geng <c.geng@esciencecenter.nl>
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.

Online webapp demo

2 participants