Sketchy502 / SDV-Summary

An application to display a summary of the player from a Stardew Valley save file.
206 stars 21 forks source link

Planner will implement fence with torches on it #13

Open hpeinar opened 7 years ago

hpeinar commented 7 years ago

Hi again!

So I have a plan to implement fence with torch into the planner and was wondering if you guys are also able to render it? We can agree on the resource names here, current fences are named as "fence, wood-fence, stone-fence... etc" and I'd like to name the torched fences "torch-fence, torch-wood-fence, torch-stone-fence... etc"

Edit: It also appears that planner currently does not have wooden lamp posts "wood-lamp-post & iron-lamp-post" available, this is also something I'll add.

Edit2: These things went like, new objects are the following: torch-fence torch-stone-fence torch-wood-fence torch-iron-fence torch-hardwood-fence wood-lamp-post iron-lamp-post

With best, Henrik.

Sketchy502 commented 7 years ago

Hey,

The naming convention is completely fine with us. We have no specific naming convention on the API side, it's all based on what names you use.

I'm currently in the process of rewriting how we handle rendering in an attempt to fix collision issues and various other issues. Once done, I'll update the API parser and plans should start rendering correctly. The error with lamp posts is most likely to do with the API parser as well. I'm looking to get everything sorted by the end of this coming week/start of the next.

Laukei commented 6 years ago

@Sketchy502 Did we ever do this?

hpeinar commented 6 years ago

https://stardew.info/planner/shaggy-husky-73/

=> Bottom ones are "with torches", they seem to render as regular fence at the moment