isi-vista / adam

Abduction to Demonstrate an Articulate Machine
MIT License
11 stars 3 forks source link

Add STEGO & Mask R-CNN API models #1185

Closed lichtefeld closed 1 year ago

lichtefeld commented 1 year ago
lichtefeld commented 1 year ago

🙃 Ok, I'll aim to fix PyCharm's greedy renaming process before you review tomorrow.

lichtefeld commented 1 year ago

It may be best to setup a shared-env on the cluster which we all have access to so that submitting the job for the server can be done by any of us. We'll have to manage by hand making sure we don't start overlapping servers or provide a way to pass in a port number but that should be doable and I'm unsure a more sophisticated automation solution is needed for this project.

spigo900 commented 1 year ago

@lichtefeld Sounds good to me. Will you set up the shared-env? And would the shared-env go in the script or in the README? I'm thinking the script makes sense since it's shared.

lichtefeld commented 1 year ago

@spigo900 I can set up the shared env if that's preferred. I'd point this submission script at environment on the cluster so there's no modifications needed. I'll include instructions on configuring a local shared environment in the readme for completeness.

lichtefeld commented 1 year ago

@boyleconnor I've reorganized the Readme some as well as merged in suggestions of yours.

boyleconnor commented 1 year ago

@lichtefeld I approved so you don't have to come back to get my feedback again but I recommend some changes in a comment