Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Poketch digital clock app #388

Merged
merged 17 commits into from
Feb 20, 2025
Merged

Conversation

lucie-sharpe
Copy link
Contributor

@lucie-sharpe lucie-sharpe commented Feb 17, 2025

Documented OV26 which is the digital clock app on the poketch. Some more work could be done on the function PoketchDigitalClock_CopyDigitTilemap in poketch_digital_clock_graphics.c

@lucie-sharpe lucie-sharpe marked this pull request as draft February 17, 2025 00:25
@lucie-sharpe lucie-sharpe marked this pull request as ready for review February 17, 2025 00:55
Copy link
Contributor

@cbt6 cbt6 left a comment

Choose a reason for hiding this comment

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

  • Use "digital watch" instead of "digital clock" since that's what it's called in the game
  • Remove outdated files (e.g. overlay026/ov26_022561C0.c, overlay026/ov26_02256404.c, etc.)


#define NUM_TASK_SLOTS 8

#define POKETCH_CLOCK_DIGITS_TILEMAP_IDX 25
Copy link
Contributor

Choose a reason for hiding this comment

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

This should probably be defined somewhere else together for all indexes of files in graphic/poketch.narc, but idk where would be appropriate.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@lhearachel commented on #378 "These are fine for now, but we will want to replace them with an naix generated by re-packing this NARC once we unpack them into the resource tree."

I am not sure about the process of unpacking NARCs into the resource tree.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Unpacking is a bit of a process unto itself, but probably one that we should do sooner rather than later, since there are multiple hands touching the Poketch subsystem at the moment. A lot of it is going to be manual work and tinkering to figure out what values align with what using a previewer like Tinke or some-such.

@@ -878,11 +878,11 @@ Overlay overlay25
Object main.nef.p/src_overlay025_ov25_02255DBC.c.o
}

Overlay overlay26
Overlay poketch_digital_clock
Copy link
Collaborator

Choose a reason for hiding this comment

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

thought: I'm making a note for myself whenever I get back onto source-tree reorganization that the poketch is sufficiently large as to deserve its own top-level folder (similar to field, battle, and contest).

@lucie-sharpe
Copy link
Contributor Author

lucie-sharpe commented Feb 18, 2025

I have made the changes suggested. I did not feel a constants file was needed so i have just used the local header file. Tell me what you think of the naming conventions used?

Copy link
Collaborator

@lhearachel lhearachel left a comment

Choose a reason for hiding this comment

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

Just one little quibble below; otherwise looks great!

@lhearachel lhearachel self-assigned this Feb 19, 2025
@lhearachel lhearachel merged commit a9bf501 into pret:main Feb 20, 2025
2 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants