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.

Sign up for GitHub

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

Jump to bottom

rerun analysis does not allow me to select entry point #62

Closed
ablack3 opened this issue Aug 5, 2024 · 5 comments
Closed

rerun analysis does not allow me to select entry point #62

ablack3 opened this issue Aug 5, 2024 · 5 comments
Assignees
@SergeySukhanov @dmitrys-odysseus
Labels
bug Something isn't working Complexity: XL Tested
Milestone
v2.0.4

Comments

@ablack3
Copy link
Collaborator

ablack3 commented Aug 5, 2024

If I select rerun and try to choose the entry point there is no option for an entry point.

image
@ablack3
Copy link
Collaborator Author

ablack3 commented Aug 13, 2024 •
edited
Loading

Also when the app is in Docker mode and I rerun an analysis I am not able to enter the url to the docker image. Instead I have to select the environment from a dropdown similar to tarball mode.

image

@konstjar konstjar added the bug Something isn't working label Aug 20, 2024
@konstjar
Copy link
Contributor

konstjar commented Oct 4, 2024

The second part of the issue was resolved in #71.

@konstjar konstjar added this to the v2.1.0 milestone Oct 4, 2024
dmitrys-odysseus pushed a commit that referenced this issue Oct 17, 2024
#62 When sending data to EE, avoid nested archives and stream instead…
ad78d89 
… of writing to temp files
dmitrys-odysseus pushed a commit that referenced this issue Oct 17, 2024
#62 Update GET /api/v1/analysis/{id} endpoint to provide list of file…
4169fed 
…s to show as potential entry points
dmitrys-odysseus pushed a commit that referenced this issue Oct 17, 2024
#62 When sending data to EE, avoid nested archives and stream instead…
209e3f5 
… of writing to temp files
dmitrys-odysseus pushed a commit that referenced this issue Oct 17, 2024
#62 Update GET /api/v1/analysis/{id} endpoint to provide list of file…
b606ffc 
…s to show as potential entry points
dmitrys-odysseus pushed a commit that referenced this issue Oct 17, 2024
#62 Update GET /api/v1/analysis/{id} endpoint to provide list of file…
30b24e7 
…s to show as potential entry points
dmitrys-odysseus pushed a commit that referenced this issue Oct 17, 2024
#62 When sending data to EE, avoid nested archives and stream instead…
f975c37 
… of writing to temp files
dmitrys-odysseus pushed a commit that referenced this issue Oct 17, 2024
#62 Update GET /api/v1/analysis/{id} endpoint to provide list of file…
3ba6c63 
…s to show as potential entry points
dmitrys-odysseus pushed a commit that referenced this issue Oct 18, 2024
#62 When sending data to EE, avoid nested archives and stream instead…
dc9d600 
… of writing to temp files
dmitrys-odysseus pushed a commit that referenced this issue Oct 18, 2024
#62 Update GET /api/v1/analysis/{id} endpoint to provide list of file…
4baaafe 
…s to show as potential entry points
dmitrys-odysseus added a commit that referenced this issue Oct 21, 2024
@dmitrys-odysseus
Merge pull request #100 from OHDSI/issue-62-rerun-entry-point
e6cabd4 
#62 Update GET /api/v1/analysis/{id} endpoint to provide list of files to show as potential entry points
@dmitrys-odysseus
Copy link
Contributor

The endpoint GET /api/v1/analysis/{id} response is updated to provide files field, that contains a list of analysis file names

@konstjar
Copy link
Contributor

@SergeySukhanov can you please update UI?

@konstjar konstjar reopened this Oct 23, 2024
@ttemnikova
Copy link

I was able to select an entry point
image

The docker image filed is editable, and the typed value is taken in account in the request
image

@ttemnikova ttemnikova added Tested and removed Merged labels Nov 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@SergeySukhanov SergeySukhanov

@dmitrys-odysseus dmitrys-odysseus

Labels
bug Something isn't working Complexity: XL Tested
Projects
None yet
Milestone
v2.0.4
Development

No branches or pull requests
5 participants
@konstjar @SergeySukhanov @ablack3 @dmitrys-odysseus @ttemnikova