UniStuttgart-VISUS / damast

Code for the DH project "Dhimmis & Muslims – Analysing Multireligious Spaces in the Medieval Muslim World" (VolkswagenFoundation)
MIT License
10 stars 1 forks source link

Infotext Pictures - Update as soon as the public instance is up and running #109

Closed rpbarczok closed 2 years ago

rpbarczok commented 2 years ago

Just as a reminder: In the infotext pictures are used that still include the shiite subgroups. To prevent confusion on the side of the users we should update them as soon as we have the live version of the visualisation

tutebatti commented 2 years ago

Thanks for the reminder, this is noted already.

tutebatti commented 2 years ago

@mfranke93 We would probably need to take the screenshots either from the version to be hosted in Berlin or run another testing instance, right?

tutebatti commented 2 years ago

I have been able to make screenshots for the startpage, i.e. not for the info texts that should not show anything not included in the public instance.

mfranke93 commented 2 years ago

We would probably need to take the screenshots either from the version to be hosted in Berlin or run another testing instance, right?

If the Shi'ites should be condensed (and generally no data not in the public version should be visible), then yes.

tutebatti commented 2 years ago

If the Shi'ites should be condensed (and generally no data not in the public version should be visible), then yes.

So yes. :)

tutebatti commented 2 years ago

I am waiting for feedback regarding the possible base maps in the public version to update the screenshots. Other than that, we could do it now. Here, too, (cf. #7) I wonder if we should make a difference between what will be in the repo and what is shown in the public version.

mfranke93 commented 2 years ago

What would be the advantage of not having the new screenshots in the repo?

tutebatti commented 2 years ago

There's no advantage in that sense, I'm just thinking of different base maps and also of different data and data structure (in case of the 5-Shiis etc.). If that's not a problem, the screenshots can be the same in both repo and public version.

mfranke93 commented 2 years ago

The way I see it, the software in the repo and the data that is shown are independent, so the repo does not have to represent either of the versions that are currently running regarding which religions, data, or even map layers are shown. So regarding maintainability (see my response in #7) putting those screenshots in would be best.

On Mon, Apr 11, 2022 at 01:08:15AM -0700, Florian Jäckel wrote:

There's no advantage in that sense, I'm just thinking of different base maps and also of different data and data structure (in case of the 5-Shiis etc.). If that's not a problem, the screenshots can be the same in both repo and public version.

— Reply to this email directly, [1]view it on GitHub, or [2]unsubscribe. You are receiving this because you were mentioned.Message ID: @.***>

References

Visible links

  1. https://github.com/UniStuttgart-VISUS/damast/issues/109#issuecomment-1094682335
  2. https://github.com/notifications/unsubscribe-auth/ACHQP3UKAPCQTT2IPBZYVFDVEPMW7ANCNFSM5Q3RXZZA

-- Max Franke, M.Sc. VIS, University of Stuttgart Universitätsstraße 38 70569 Stuttgart-Vaihingen

Office: 1.438 Phone: +49 711 685 88435 Web: http://go.visus.uni-stuttgart.de/max.franke

tutebatti commented 2 years ago

Since we now use the same map style as before, we do not need to change the screenshots showing maps. However, there are some instances where the three Shiite subgroups are showing. This is not a huge problem, but I would change it soon. Do you, @mfranke93, want to change them or should I do that?

mfranke93 commented 2 years ago

Do you, @mfranke93, want to change them or should I do that?

I would say you do it, and put in a PR.

For the visualization documentation, I added this disclaimer, alongside the map content copyright statement, in the introduction, and that should suffice in my opinion:

The map material visible in the screenshots is copyrighted by © Mapbox and © OpenStreetMap. The screenshots were created using a larger, private dataset, and the shown religious groups and places might differ from those visible in public versions and the public dataset.

tutebatti commented 2 years ago

For the visualization documentation, I added this disclaimer

That's a very good solution!

I will get to the screenshots soon.