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

Won't Import Spansh CSV #1

Open
Wombat1501 opened this issue Feb 19, 2023 · 5 comments
Open

Won't Import Spansh CSV #1

Wombat1501 opened this issue Feb 19, 2023 · 5 comments

Comments

@Wombat1501
Copy link

As the title says. Say the file is not a valid file.

@KitsuneNoBaka
Copy link

It imports Road2Riches spansh cvs, but not Road2Exobiology

Road2Riches CVS
"System Name","Body Name","Body Subtype","Is Terraformable","Distance To Arrival","Estimated Scan Value","Estimated Mapping Value","Jumps" "BD+33 4395","BD+33 4395 A 1","Water world","Yes","304.295127","274004","995547","2" "HIP 114458","HIP 114458 A 2","Earth-like world","No","346.979084","266717","969071","2" "HIP 104179","HIP 104179 4","High metal content world","Yes","491.518702","191221","694769","3" "HIP 104179","HIP 104179 5","High metal content world","Yes","880.225912","188695","685591","0" "HIP 16887","HIP 16887 1","Water world","Yes","445.113266","275451","1000804","5"

Road2Exobiology
"System Name","Body Name","Body Subtype","Distance To Arrival","Landmark Subtype","Value","Count","Jumps" "Sesuang","Sesuang A 6","Icy body","288.154661","Fonticulua Fluctus","20000000","20","1" "Borfor","Borfor 1 c","Rocky body","762.982748","Cactoida Vermis","16202800","20","2" "Borfor","Borfor 1 d","Rocky body","767.200511","Cactoida Vermis","16202800","20","0" "Borfor","Borfor 1 b","Rocky body","768.64627","Cactoida Vermis","16202800","20","0" "HIP 82068","HIP 82068 5 c","Rocky body","786.262387","Cactoida Vermis","16202800","20","3" "Daruwutja","Daruwutja B 5","High metal content world","760.642688","Stratum Tectonicas","19010800","20","5" "Daruwutja","Daruwutja B 4","High metal content world","781.991118","Stratum Tectonicas","19010800","20","0"

@Sigtrap2074
Copy link

I'm having similar issues. I can import neutron plots without problem, but I've been trying to import a Spansh Ammonia Worlds route, to no avail.

@Wombat1501
Copy link
Author

Wombat1501 commented Mar 3, 2023 via email

@Artoria2e5
Copy link

Artoria2e5 commented Aug 2, 2023

The router is looking for the exact road2riches header:

road2richesimportheader = "System Name,Body Name,Body Subtype,Is Terraformable,Distance To Arrival,Estimated Scan Value,Estimated Mapping Value,Jumps"

elif headerline == road2richesimportheader:

It only takes the information in body name, subtype, and the usual system and jumps columns. All these columns are also present in the Road2Exobiology CSV.

For a band-aid fix, just add this below the first linked line,

road2exoimportheader = "System Name,Body Name,Body Subtype,Distance To Arrival,Landmark Subtype,Value,Count,Jumps"

and change the second linked line to

elif (headerline == road2richesimportheader) or (headerline == road2exoimportheader):

For a long-term fix (assuming Frontier adds ANOTHER thing that makes you scan planets and spansh makes another roads to X -- unlikely lol), this thing should not be matching exact headers, but just checking for whether column titles exist.

@norohind
Copy link
Owner

Hello, guys. For some reason I wasn't receiving notifications from this repo, I'm sorry for such long respond time.
I agree the code for csv parsing should be more unified than it is now. However, I don't currently play ED much so probably won't get my hands to fix it.
I have just merged a PR #3 which declares support for routes by galaxy plotter. Please feel free to test if it's so. Others PRs are welcome.
If you have a better fork of original project, I can specify link to it in README and archive this repo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants