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

Update instruction to allow for running locally (not through docker) #85

Closed
AlexAxthelm opened this issue Feb 27, 2024 · 2 comments
Closed

Comments

@AlexAxthelm
Copy link
Contributor

          Once we merge in this batch, I'll go through and see if I can get it working on bare metal? I think if mostly involves changing some things in the docker-compose, but dealing with variable inheritence across this and #71 open at the samer time is tricky.

Originally posted by @AlexAxthelm in #66 (comment)

@jdhoffa
Copy link
Member

jdhoffa commented Feb 27, 2024

The main purpose of this workflow is to:

  • Scrape index data
  • Pull a transition monitor image and run PACTA on the scraped data

As such it is fundamentally dependent on building off of a base workflow.transition.monitor docker image:

FROM ghcr.io/rmi-pacta/workflow.transition.monitor:main

At this stage, I don't plan to support an API to run this through RStudio as it would either involve:

  • jumping through hoops to figure out how to also run "Transition Monitor" itself through RStudio (e.g. by cloning the relevant repositories and calling source() on the scripts)
  • adding some system() lines in the .R script to run the Docker image

Neither of these options seem to be worth the extra maintenance burden.

If either of you see it differently and wish to assign yourself this ticket then go ahead, otherwise I will close this issue.

FYI @cjyetman

@AlexAxthelm
Copy link
Contributor Author

Closing as noted above.

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

No branches or pull requests

2 participants