Skip to content

Add actual resource assets and change to ortho cam #32

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

Merged
merged 28 commits into from
Mar 10, 2023

Conversation

nucleartide
Copy link
Owner

@nucleartide nucleartide commented Mar 10, 2023

Quality Assurance

Take some screenshots as confirmation of quality assurance.

resources.mov

JIRA Ticket

Sorry folks, this is internal to my Notion workspace!

https://www.notion.so/nucleartide/Replace-dummy-resources-with-selected-3D-assets-85b6cdddaabe4474a57d97d73eefcdee?pvs=4

Development checklist

Review Files changed and check off the following:

  • General review to clean up your changes. Then,
  • Modular: Are you touching the scene file? If so, can you limit your changes to prefab files instead?
  • Editable: Has any configuration been moved into ScriptableObjects, so that modified values persist even after exiting Play mode?
  • Debuggable: Do you have debug views for the prefabs/systems that you are creating?
  • Update readme with new pull request.

Changelog

Provide a bulleted list of changes that were made in this pull request.

  • Replace terrain texture with ground texture from ADG Textures
  • Replace third-person perspective camera with orthographic camera
    • This suits the lack of height in the game’s terrain a little better.
  • Choose a bunch of assets from polyperfect low poly asset pack
  • Render newly selected assets instead of colored placeholder cubes
  • Update PlayerController to no longer rotate relative to Player Shoulder Target
  • Add TreeScaler script for visually scaling trees and their colliders by the same amount

How it works

Explain what you did, how you did it, and why you did it. Also any lessons that might be useful to others.

(Note to self: it's easiest to do a stream-of-consciousness writeup of what you did.)

In this pull request, I got to work replacing the placeholder resource cubes with actual assets.

There are 8 in total, but they required some shopping around in my polyperfect asset pack:

Wheat:

Screenshot 2023-03-10 at 5 39 16 PM

Water:

Screenshot 2023-03-10 at 5 39 08 PM

Sugar cane:

Screenshot 2023-03-10 at 5 39 03 PM

Stone:

Screenshot 2023-03-10 at 5 38 56 PM

Wood:

Screenshot 2023-03-10 at 5 38 44 PM

Iron ore:

Screenshot 2023-03-10 at 5 38 31 PM

Copper ore:

Screenshot 2023-03-10 at 5 38 29 PM

Coal:

Screenshot 2023-03-10 at 5 38 27 PM

Once the assets were selected, I made some minor adjustments so that the player collides with appropriate resources.

The player shouldn’t be able to walk on water, or walk through trees and stiff sugar canes, but they should be able to walk through other resources such as ores and stones.

I also added a scaling script to scale the trees by random amounts, to give some visual variety.

After selecting the resource assets, it became clear that a perspective camera would point out the unappealing flat-ness of my game:

Screenshot 2023-03-10 at 5 44 26 PM

Factorio hides this visual dimension with an orthographic camera, and I sought to do the same. Gameplay is 2-dimensional, so no need to complicate graphics:

Screenshot 2023-03-10 at 5 43 53 PM

With this change, at some point I may need to allow for zooming of the camera.

@nucleartide nucleartide changed the title Add actual resource assets Add actual resource assets and change to ortho cam Mar 10, 2023
@nucleartide nucleartide marked this pull request as ready for review March 10, 2023 22:49
@nucleartide nucleartide merged commit ae3f1a2 into master Mar 10, 2023
@nucleartide nucleartide deleted the jason/add-actual-resource-assets branch March 10, 2023 22:49
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

Successfully merging this pull request may close these issues.

1 participant