-
Notifications
You must be signed in to change notification settings - Fork 217
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
compute server: http interface #7943
Open
williamstein
wants to merge
42
commits into
master
Choose a base branch
from
compute-http-server
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
williamstein
commented
Oct 15, 2024
- url to get basically a single cocalc project running on a compute server, e.g., file system navigation, terminals, jupyter, file editing, etc.
- direct connection between browser and compute server
- everything is sync'd with main project in cocalc.com
… server in dev mode
- also found a cache header bug in hub express server (only impacts cocalc-docker so not a real issue).
motivation: I'm trying to decide what architecture to use for the compute server's web server, and it would be really helpful to better understand what we do now, in order to see the pros and cons.
motivation is so it can be used in a compute server too; also this helps better organize and clean up code. more to come.
- obviously way better approach than trying to put server in @cocalc/sync where it can only go with a bunch of brittle runtime insertions - very natural module to define
invalid cell type
… run -- it's a button with an optional dropdown, and the button brings up the full assistant dialog with proper context, focused on this cell.
- needed to support this functionality outside of the home base project
- this breaks compute, will fix later
…e the raw url to a file in a project, in many different ways...; replace all of these by one
… server - we have to implement a little of this to make the frontend happy - implemented ping for starters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.