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

chemprop Web Prediction Interface Removed #515

Closed
euhruska opened this issue Oct 16, 2023 · 11 comments
Closed

chemprop Web Prediction Interface Removed #515

euhruska opened this issue Oct 16, 2023 · 11 comments
Labels
wontfix This will not be worked on
Milestone

Comments

@euhruska
Copy link

the link to the web prediction interface seems to be broken: http://chemprop.csail.mit.edu/

@euhruska euhruska added the bug Something isn't working label Oct 16, 2023
@kevingreenman
Copy link
Member

kevingreenman commented Oct 16, 2023

We have decided to remove support for the web interface in chemprop v2.0 (beta version available now on the v2/dev branch, stable version coming early 2024), so this probably will not be fixed as we finish the last few items of v1.x development in late 2023.

Will leave this issue open as a reminder to remove all references (README, etc) to the web.

@kevingreenman kevingreenman added this to the v2.0.0 milestone Oct 16, 2023
@kevingreenman kevingreenman added the wontfix This will not be worked on label Oct 16, 2023
@JacksonBurns
Copy link
Member

@kevingreenman if the site is currently down and we are not adding it to v2, should we officially remove the web page now? I think it would be better than leaving the page up as-is.

@kevingreenman
Copy link
Member

@JacksonBurns yeah, maybe not a bad idea. I'm currently trying to see how difficult it is to get it back up and running temporarily (it was easy to restart it when it went down last year, but it seems like there are some extra hurdles now). Also worth mentioning is that the website was where the pretrained models for the Cell paper and COVID work were hosted (see #255), so regardless of whether we take it down now or in a few months, we should talk to the authors of those papers to have them set up a Zenodo or some alternative way of storing the models long-term

@kevingreenman
Copy link
Member

Okay, I figured out that the website went offline on May 1, 2023 because the OS its virtual machine was running on (Ubuntu 18.04.2) was no longer supported by CSAIL as of that date. That's why the restart process from before isn't working.

@kevingreenman
Copy link
Member

I think restarting the website will require creating a new custom image with a supported OS (Ubuntu 22.04 or Ubuntu 20.04) and then starting a new instance from that image. I'll email CSAIL about how to do this.

@kevingreenman
Copy link
Member

Note (from @JacksonBurns): if we decide to keep the website running as a demo of the v1 code after we no longer maintain v1, we should add a big banner to the website noting that v1 is no longer maintained, and pointing the user to the v2 code

  • add banner to website

@Evert-Homan
Copy link

Hi,

Will there still be support in v2 for running a local version of the web site/server?

Thanks/Evert

@kevingreenman
Copy link
Member

Hi,

Will there still be support in v2 for running a local version of the web site/server?

Thanks/Evert

No

@Evert-Homan
Copy link

Hmmm, that's too bad.

Thanks/Evert

@kevingreenman
Copy link
Member

We apologize if the web interface is something you use often. We chose to remove it for a few reasons:

  • The companies that fund the development of the software don't use the web tool
  • It's a burden on the developers to also have to update the web tool when we add new features. Because of this, the web code hasn't been touched for quite some time despite the fact that we've added many new features in the rest of the code.
  • The web tool forces us to carry additional dependencies that make it difficult to use chemprop on Windows OS.

If we were to continue supporting a web tool, we would probably put it in a separate repo. However, we don't think we have the bandwidth to maintain the separate tool right now.

The web tool will remain available to you in the version 1 releases, so you're welcome to continue using it for what you've been using it for.

@JacksonBurns JacksonBurns changed the title web prediction interface broken chemprop Web Prediction Interface Removed Oct 20, 2023
@JacksonBurns JacksonBurns pinned this issue Oct 20, 2023
@JacksonBurns
Copy link
Member

I'm slightly modifying the title and will leave a note here for any newcomers.

Summary:

  • chemprop v1 supports running a web interface for prediction
  • previously, there was an MIT-hosted website that hosted an instance of this website
  • this website has gone down and will not be restored
  • chemprop v2 will not support a web interface natively; the feature will not be removed from v1

@JacksonBurns JacksonBurns closed this as not planned Won't fix, can't repro, duplicate, stale Oct 20, 2023
@JacksonBurns JacksonBurns removed the bug Something isn't working label Oct 20, 2023
@chemprop chemprop locked as resolved and limited conversation to collaborators Oct 20, 2023
@kevingreenman kevingreenman unpinned this issue Apr 22, 2024
@kevingreenman kevingreenman pinned this issue Apr 23, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

4 participants