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

Ai screens and such #390

Open
wants to merge 3 commits into
base: dev
Choose a base branch
from
Open

Ai screens and such #390

wants to merge 3 commits into from

Conversation

Merpblu
Copy link

@Merpblu Merpblu commented Jun 20, 2023

Ports some cool AI core screens from the Polaris codebase, cheers to the spriters that did these new AI sprites.

About the Pull Request

Adds in about 17 new AI screens and 5 new crash sprites for new and old AI core screens

Why It's Good For The Game

The players will have more options for an AI core screen.

Did you test it?

I have done a compile of the code no errors nor warnings to report, but I did not have time to test the game in a playable state.

Authorship

All of the new AI sprites were from the Polaris codebase and said spriters.

Changelog

🆑rscadd
/:cl:

Ports some cool AI core screens from the Polaris codebase, cheers to the spriters that did these new ai sprites.
@EgorDinamit EgorDinamit added 🔨1 Tweak A small change/addition to already existing systems. 🚢2 Ported This was fully or partially ported from another codebase. We thank the original authors! labels Jun 20, 2023
@EgorDinamit
Copy link
Member

I don't think it does anything on its own? You might have to add icon states wherever they are mentioned in the code itself.

@Merpblu
Copy link
Author

Merpblu commented Jun 20, 2023

I don't think it does anything on its own? You might have to add icon states wherever they are mentioned in the code itself.

Ah.... I did not know this till now

Merpblu and others added 2 commits June 29, 2023 16:09
Places in the code that I forgot to place in with the new AI core screen icons.
Copy link
Member

@EgorDinamit EgorDinamit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are some issues with rebasing/merging apparently, you deleted some of the newer PRs here.
If you need help I could try and implement AI screens into the game myself.

@Merpblu
Copy link
Author

Merpblu commented Jul 8, 2023 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔨1 Tweak A small change/addition to already existing systems. 🚢2 Ported This was fully or partially ported from another codebase. We thank the original authors!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants