fritz-marshal / fritz

Astronomical data platform for the Zwicky Transient Facility.
https://fritz.science
Other
24 stars 34 forks source link

Remove Kowalski as a submodules? #544

Open Theodlz opened 4 months ago

Theodlz commented 4 months ago

We don't deploy Kowalski alongside SkyPortal, and probably never will for good reasons (both systems, based on their hardware requirements, really belong on 2 - or more - different machines).

I'm wondering if it wouldn't make sense to remove Kowalski as a submodule here, from the config, and from the launcher commands. This would make the repo a little bit more maintainable, and limit confusion for contributors.

But also, I think that without Kowalski, Fritz can really be a solid example of what a system that needs to add features on top of vanilla SkyPortal should look like.

Theodlz commented 4 months ago

@mcoughlin @profjsb @stefanv any thoughts? Maybe something to discuss at the Fritz weekly meeting

Theodlz commented 4 months ago
mcoughlin commented 4 months ago

@Theodlz completely agreed.

stefanv commented 4 months ago

It would be good to have a way to do integration testing, if that isn't yet set up on either side.

Theodlz commented 2 months ago

@stefanv ah yes definitely. I think that can still happen on git. I'm thinking about pushing kowalski docker images to dockerhub soon, so not too hard to have it run here and pass it the SkyPortal token as an env variable (and vice versa).

Theodlz commented 2 months ago

And again this is just an idea. Documenting the existing code a bit mode and cleaning it up might be enough to separate the Fritz and Kowalski stuff enough that someone else can pretty much fork Fritz and modify it into their own skyportal custom app.

Theodlz commented 2 months ago

And first, I want to fix the existing integration testing with Kowalski