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

GeoBrowser(CSV) does not display a CSV file #96

Open
emanueldima opened this issue Dec 9, 2020 · 8 comments
Open

GeoBrowser(CSV) does not display a CSV file #96

emanueldima opened this issue Dec 9, 2020 · 8 comments
Labels
bug Something isn't working

Comments

@emanueldima
Copy link
Collaborator

@fungunga We tested the GeoBrowser with a csv file but it does not display anything on the map, only the first two columns of the csv file (attached, renamed to .txt because github would not attach it otherwise).

all-test-geo.csv.txt

@emanueldima emanueldima added the bug Something isn't working label Dec 9, 2020
@fungunga
Copy link
Contributor

fungunga commented Dec 10, 2020

@emanueldima
Copy link
Collaborator Author

Ok, so the documentation says these fields are mandatory:

  • Address
  • TimeStamp, or TimeSpan:begin and TimeSpan:end
  • Longitude
  • Latitude

Do any other columns matter, or are they silently ignored? Would any csv work as long as it has these columns?

@fungunga
Copy link
Contributor

I do not know either :-) Maybe you would need a title row? --> also improve error handling, and maybe documentation, too!

@fungunga
Copy link
Contributor

It does work with the following file (created with the Datasheet Editor): https://cdstar.de.dariah.eu/dariah/EAEA0-0CB7-1F90-DF1B-0
In short that would be:
Address,Longitude,Latitude
University of Vienna,16.36077,48.213023
Karl Franzens University Graz - FÖDT/OEDT,15.447049,47.07874
Karl Franzens University of Graz - Romance Studies,15.45751183,47.07057311
Austrian Academy Corpus,16.37727769,48.20909235

Without time fields you just get places visualized.

@fungunga
Copy link
Contributor

@emanueldima
Copy link
Collaborator Author

The switchboard does not yet distinguish between pure plain/text files and text/comma-separated-value files. You have to go to the switchboard and change the mediatype manually:
Screenshot 2021-04-19 at 14 37 30

@fungunga
Copy link
Contributor

But we give the correct mimetype with the switchboard call, and this mimetype is existing in the Switchboard, too. Don't you take the minetype from the API call, if existing?

@emanueldima
Copy link
Collaborator Author

In some cases the provided mimetype cannot be trusted and because of that we currently ignore it. I think you're right that we should not ignore it in this particular case (clarin-eric/switchboard#197)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants