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

New Update fails to import Paramname: itemLotParam_enemy - Fieldname: lotItemBasePoint0x due to number being larger than 34463. #794

Open
greetthemoth opened this issue Dec 14, 2023 · 1 comment
Labels
enhancement New feature or request

Comments

@greetthemoth
Copy link

greetthemoth commented Dec 14, 2023

In the past when i attempted to import an ItemLotParam_enemy csv, the values associated with the fields lotItemBasePoint0x would be clamped down to 34463 if the number in the csv was higher, this was very useful for my workflow, but in the new update this doesnt seem to happen. I hope this is not an intentional change.

Ive recently updated to the new version (1.8.1 - 1.9.1)and realised i couldnt import a CSV file that had worked just fine in my previous version, after some investigating i figured out the issue was fairly benign, so i hope it gets fixed.

@kingborehaha
Copy link
Collaborator

I'm personally a little conflicted, since on one hand it's obviously more convenient. But on the other, if you don't know what you're doing (or what value types are) this behavior would cause confusion.

Maybe adding a prompt to clamp going forward and it explains what's happening if you do.

@Philiquaz Philiquaz added the enhancement New feature or request label Jun 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants