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

2024 07 16 photo video for vsa kek testimport #316

Merged
merged 12 commits into from
Dec 20, 2024

Conversation

sjib
Copy link
Contributor

@sjib sjib commented Jul 16, 2024

  • added inspectiondata folder with fotos and video (cut video to around 35 MB) to not conflict with github max 100 MB.
  • move that folder before publication of plugin @ponceta
  • changed prefix to ch20zzsh for KEK inspection data to have valid Swiss prefix

@sjib sjib self-assigned this Jul 16, 2024
@sjib sjib added enhancement New feature or request plugin Concerns the wastewater plugin labels Jul 16, 2024
@sjib sjib requested a review from ponceta July 16, 2024 14:34
@sjib sjib added this to the TEKSI Wastewater 2024.0 milestone Jul 16, 2024
@sjib
Copy link
Contributor Author

sjib commented Jul 16, 2024

@ponceta Here with the inspection data folder and a few more corrections. I have tested the import - it works

20240712_import_kek_adapt_path
On import you should adapt the path of the the inspectiondata folder to your local system.

It still needs some work with the file tab - see #272 (comment), but here is some KEK data so we are able to work on this. I try to comment in the issue on what I think would be needed there.

@ponceta
Copy link
Member

ponceta commented Jul 19, 2024

We have to host it somewhere so we don't have a plugin that weights 100mb

@domi4484 any experience with heavy media files accessibility?

Users should be able to download these easily.

@domi4484
Copy link
Contributor

@ponceta 100mb is a limitation of the QGIS plugin repository?

The pictures added here are pretty small and if with video the whole plugin stays by around 40mb I think there is no need to take them out of the package.

In the plugin we already do an on the fly download of Interlis binaries, and that would be possible for other data as well. Depending how "heavy" the files could still be hosted on github parallel to the plugin directory. Otherwise it would need some sort of other server/infrastructure.

@ponceta
Copy link
Member

ponceta commented Nov 26, 2024

@sjib are you ok if we host the video somewhere on the web?

Example on Vimeo :
https://vimeo.com/1033484838/9c11ac73e0?share=copy

or youtube via TEKSI, or anywhere where it's free and easy to access (without having to reference it on the web)?

And then add the online link to the xtf data. This would challenge the document mangement part as we sometimes have mixed ressources (Local disks, local shares, SFTP, Professional electronic document management, Online ressources.

@sjib
Copy link
Contributor Author

sjib commented Nov 29, 2024

@sjib are you ok if we host the video somewhere on the web?

Example on Vimeo : https://vimeo.com/1033484838/9c11ac73e0?share=copy

And then add the online link to the xtf data. This would challenge the document mangement part as we sometimes have mixed ressources (Local disks, local shares, SFTP, Professional electronic document management, Online ressources.

@ponceta I need a stable URI to create an additional data_media object that represents our vimeo video folder
Can you create a video group with a specific name for the example video?

So I can split https://vimeo.com/1033484838/ as absolute path, /xxx/ the new group folder' as relative path and 9c11ac73e0 filename (identifier) of the file

CLASS Datentraeger EXTENDS VSA_BaseClass =  
  ATTRIBUTE
    !!@ comment = "Beschreibt die Art des Datenträgers"
    Art: MANDATORY (     
      andere,
      CD,
      Diskette,
      DVD,
      Festplatte,
      !!@ comment = "Lokaler Server (im Gegensatz zu Webserver)"
      Server,
      Videoband,
      !!@ comment = "Webserver mit URI (https://de.wikipedia.org/wiki/Uniform_Resource_Identifier)"
      Webserver
    );
    !!@ comment = "Bemerkungen zum Datenträger"
    Bemerkung: TEXT*80;
    Bezeichnung: MANDATORY TEXT*60;
    !!@ comment = "Zugriffspfad zum Datenträger. z.B. DVD-Laufwerk -> D: , Server -> //server/videos, Harddisk -> c:/videos . Kann auch eine URL sein. Bei einem analogen Videoband leer"
    Pfad: Pfad;
    !!@ comment = "Ort, wo sich der Datenträger befindet"
    Standort: TEXT*50;
END Datentraeger;

@sjib
Copy link
Contributor Author

sjib commented Nov 29, 2024

  • take out video from plugin when ok with link on vimeo

@sjib
Copy link
Contributor Author

sjib commented Dec 19, 2024

@ponceta Can we check at the vimeo link so this can be merged for 2024.0?

@ponceta
Copy link
Member

ponceta commented Dec 20, 2024

VIMEO requires a credit card.

I hosted it under our GIS youtube channel in the "non repertoriee" so only people with the link can see it :

https://youtu.be/D1TcLrBPYxQ

Path would then be https://youtu.be/
and file id would be D1TcLrBPYxQ

@sjib
Copy link
Contributor Author

sjib commented Dec 20, 2024

@ponceta Ready to review from my side

@ponceta ponceta merged commit e6ab890 into main Dec 20, 2024
4 checks passed
@ponceta ponceta deleted the 2024-07-16-photo-video-for-vsa-kek-testimport branch December 20, 2024 09:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request plugin Concerns the wastewater plugin
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants