Atlas icon indicating copy to clipboard operation
Atlas copied to clipboard

config_gis

Open davidhcar opened this issue 1 year ago • 6 comments

Looking at https://github.com/OHDSI/Broadsea-WebTools/blob/master/config-gis.js Is there an example/sample for the config_gis. or what is the purpose of this .js? I understand the config_local though. Could someone please clarify.

davidhcar avatar Mar 28 '23 16:03 davidhcar

If WebAPI is compiled without webapi-gis profile (it applies to Broadsea), than config-gis.js file should be empty.

konstjar avatar Mar 31 '23 18:03 konstjar

This JavaScript file is a reference to this component: https://github.com/OHDSI/atlas-component-geospatial and the corresponding WebAPI component: https://github.com/OHDSI/webapi-component-geospatial. I'm not well versed in how these work but hoping this answers your questions. If you have any questions about those components, please re-open this issue.

anthonysena avatar Mar 31 '23 20:03 anthonysena

I'd like to re-open this. We don't use the webapi-gis profile, but Atlas continues to look for it anyway, and then throws a console error when it doesn't find it.

It doesn't hinder anything, but we should re-evaluate.

alondhe avatar Jan 20 '24 00:01 alondhe

To add: obviously ours is via Broadsea. Is it possible the Atlas Dockerfile isn't handling the lack of webapi-gis profile correctly?

alondhe avatar Jan 20 '24 18:01 alondhe

If I look at the original repo, the file and Dockerfile is correct: https://github.com/OHDSI/Broadsea-WebTools/blob/master/Dockerfile#L64

But Dockerfile in the ATLAS repo does not contain it. https://github.com/OHDSI/Atlas/blob/master/Dockerfile

As a quick fix the Dockerfile could be extended with additional command: RUN touch /usr/share/nginx/html/atlas/js/config-gis.js && chown 101:101 /usr/share/nginx/html/atlas/js/config-gis.js

konstjar avatar Jan 20 '24 20:01 konstjar

Ah, good catch @konstjar . I'll raise a PR.

alondhe avatar Jan 23 '24 16:01 alondhe