search

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 implemented crops from three seasons. #15

Closed hpeinar closed 7 years ago

hpeinar commented 7 years ago

Hi!

I've merged a PR into the planner that implements some new crops from three seasons. It would be very cool if you guys added the respective plants to the rendering api.

Planner save with all new crops: https://stardew.info/planner/white-zebra-77/ (few duplicates)

Upload.farm render of said plan: http://upload.farm/plan/1CDsDA

Laukei commented 7 years ago

Hi Henrik,

I've pushed the rendering patch @Sketchy502 just committed to upload.farm. It seems to render correctly: http://upload.farm/plan/1CDUhS

I've noticed it's not possible to put these crops on tilled soil, which leads to them coming out of the base of the map (as in my link) which I think isn't possible in Stardew Valley. Is it better for us to correct this by assuming that a crop means "crop on top of tilled soil"?

Hope all's well!

hpeinar commented 7 years ago

Hi!

Yeah, I think you should correct them to be on tilled soil. I'll probably update planner sprites sometime too, to make them show on tilled soil.

With best, Henrik.

Sketchy502 commented 7 years ago

Hey Henrik,

Completely forgot that the plants needed to be placed on tilled soil, my bad. Just pushed the changes now @Laukei can update the site when he gets a chance.

Regards, Ashley

Laukei commented 7 years ago

Pushed, appears to work great!