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

Schema: rewrite to run local only and handle newer schema versions #3117

Open
wants to merge 12 commits into
base: dev
Choose a base branch
from

Conversation

mashehu
Copy link
Contributor

@mashehu mashehu commented Aug 14, 2024

No description provided.

Copy link
Member

@mirpedrol mirpedrol left a comment

Choose a reason for hiding this comment

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

I'm not sure if it's because I am trying on gitpid, but I get an error when opening the URL
image
These are the debug logs:

DEBUG    Starting new HTTP connection (1): localhost:8000                                                connectionpool.py:243
DEBUG    POST request received                                                                                    server.py:44
DEBUG    "POST /process-schema HTTP/1.1" 200 -                                                                   server.py:102
DEBUG    http://localhost:8000 "POST /process-schema HTTP/10" 200 None                                   connectionpool.py:546
DEBUG    Got response from nf-core API: received                                                                  utils.py:479
INFO     Opening URL: http://localhost:8000/schema_builder.html?schema_path=nextflow_schema.json                 schema.py:908
INFO     Waiting for form to be completed in the browser. Remember to click Finished when you're done.           schema.py:910
                                                                                                                              
⣾ Use ctrl+c to stop waiting and force exit.DEBUG    Starting new HTTP connection (1): localhost:8000                                                connectionpool.py:243
DEBUG    "GET /process-schema?schema_path=nextflow_schema.json HTTP/1.1" 200 -                                   server.py:102
DEBUG    http://localhost:8000 "GET /process-schema?schema_path=nextflow_schema.json HTTP/10" 200 None   connectionpool.py:546
DEBUG    Got response from nf-core API: waiting_for_user                                                          utils.py:479

item.a_path for item in repo.index.diff(None)
]:
if not Confirm.ask(
f""":exclamation_mark: '{str(self.schema_filename)}' has uncommitted changes. These will be overwritten in the following steps. Do you still want to continue""",
Copy link
Member

Choose a reason for hiding this comment

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

I think this is always thrown? I just tested with a fresh pipeline and I see this logging:

INFO     [✓] Default parameters match schema validation                                                                       
INFO     [✓] Pipeline schema looks valid (found 32 params)                                                                    
INFO     Writing schema with 32 params: 'nextflow_schema.json'                                                                
🚀  Launch web builder for customisation and editing? [y/n] (y): 
❗  'nextflow_schema.json' has uncommitted changes. These will be overwritten in the following steps. Do you still want to 
continue [y/n] (y): 

@mashehu mashehu added the WIP Work in progress label Aug 20, 2024
@mashehu
Copy link
Contributor Author

mashehu commented Aug 20, 2024

Hmm, I think gitpod is looking at the wrong port. Let me check.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
WIP Work in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants