User metrics with optional platform specific db persistence #66

varoonp123 · 2023-08-04T20:13:23Z

Like #58 but with a massaged commit history.

Hoist the arcade-specific features (serial ports) to another crate to assist with conditional compilation. Hoist whatever bits were needed into the thetawave_interface crate to be used as something like a "service registry" with schemas+typedefs.
Implement a plugin to keep track of 4 caches of user metrics+mob kills for the current game and all completed games.
Implement a plugin to periodically persist the completed games caches to a local sqlite.
Show the metrics on the Game over, main menu, and victory screens.

…rs incremented during the game

…creen

…e/run

…e the layout later

…All mutations handled by 1 bevy plugin and reset on each new game

…ame_metrics to disk rather than doing a sqlite op whenever a mob is killed. This allows us to bulk increment the mob kill count sqlite table, which is fewer sqlite ops

…r of our plugin that handles separate resources. The game over screen now pulls from those newer structs with the current game stats

…t certain plugins into separate crates. This aids in conditional compilation and platform-specific functionality. Wasm is a bear

… hoist the db to another crate

…when the feature is not available. Hoist the db model/func return type to an interface to be used by the game over ui.

…ional compilation and doing more platform-specific behavior

…ches: (mobs killed, userStats) x (currentGame, completedGame)

…more 1-at-a-time increments.

…ng whenever we start a new game so that we can keep metric counters across games even without a database

…re actaully updating that now

counter metrics plugin

… things as resources

…eOver rather than OnUpdate

…e used by the plugin test

…me we write to the db for a given user and not log an error

… the game over screen

varoonp123 · 2023-08-04T21:27:04Z

crates/thetawave_arcade/Cargo.toml

+version = "0.1.0"
+edition = "2021"


Use consistent package version for all crates in the workspace.

varoonp123 · 2023-08-04T21:28:40Z

crates/thetawave_interface/Cargo.toml

+bevy_ecs_macros = "0.11.0"
+bevy_ecs = "0.11.0"


Use consistent bevy versions across all deps. Ideally we only update 1 place when they release a new version.

varoonp123 · 2023-08-04T21:31:13Z

crates/thetawave_storage/Cargo.toml

+thiserror = {workspace = true}
+derive_more = {workspace = true}
+rusqlite = { version = "0.29.0", features = ["bundled"] }
+dirs = "5.0.1"


I think directories is slightly higher leveled. Use that instead.

…ystem

cdsupina

Good one 🤣

varoonp123 added 30 commits August 4, 2023 02:59

First attempt at Sqlite DB for user stats with plugin. Persist counte…

05febba

…rs incremented during the game

Minimal example of persisted user stats showing up on the game over s…

7cdfc05

…creen

SpawnProjectileEvent now keeps track of the source Entity id

75c0b0f

Add a struct to keep track of metrics incremented during a single gam…

c613480

…e/run

flush shots fired to the db when exiting the game over screen

7b57239

haphazardly throw some stats onto the game over screen. Need to chang…

0e8c802

…e the layout later

Keep track of the entity (id) that fired each projectile

f867462

Reimplement program stat/metric counters to be completely in memory. …

1fc1a70

…All mutations handled by 1 bevy plugin and reset on each new game

Add the CurrentGameMetricsPlugin to the bevy app

06fbadb

The db plugin now flushes the data from the game::counters::current_g…

c08b9e6

…ame_metrics to disk rather than doing a sqlite op whenever a mob is killed. This allows us to bulk increment the mob kill count sqlite table, which is fewer sqlite ops

Rip out the old CurrentGameMetrics struct with a single field in favo…

fa8f444

…r of our plugin that handles separate resources. The game over screen now pulls from those newer structs with the current game stats

Minor stat reformatting on the game over screen

295a460

Move spawnable and states structs to an interface crate to later hois…

a22ac66

…t certain plugins into separate crates. This aids in conditional compilation and platform-specific functionality. Wasm is a bear

Move the game counters ECS type defs to the interfaces crate. Prep to…

1db5b7a

… hoist the db to another crate

Conditionally compile the thetawave_storage module with defaults for …

1356a56

…when the feature is not available. Hoist the db model/func return type to an interface to be used by the game over ui.

REFACTOR: Move the arcade module to a separate crate to aid in condit…

6a43ee8

…ional compilation and doing more platform-specific behavior

Add a new field to the database for the total number of shots hit

6cef787

Describe a different historical metrics interface that preserves 4 ca…

071dca3

…ches: (mobs killed, userStats) x (currentGame, completedGame)

Second attempt at storage plugin that only loads+dumps 2 structs. No …

b79af2e

…more 1-at-a-time increments.

Rewrite the game counters plug to do something analagous to log rolli…

7aad400

…ng whenever we start a new game so that we can keep metric counters across games even without a database

Use the newer 4-cache interface in the game over UI screen since we a…

6c2945e

…re actaully updating that now

Have a couple of structs derive clone to help with a test

a35732e

Add a test to check that player 1's shot counter increments in the

fba1669

counter metrics plugin

Rework the historical metrics interface a tiny bit so Bevy can derive…

21e110e

… things as resources

Add a test of the db storage plugin

1bc1873

BUG FIX: Increment the in-memory games lost counter when we enter Gam…

95394b4

…eOver rather than OnUpdate

Add a test for the in-memory mob desctroyed counters system

364de43

Run tests in CI now that we actually have some

b326f4f

Make the thetawave_storage db env var public to the crate so it can b…

4f3eaaa

…e used by the plugin test

BUG FIX: typo in a sql field name

e4855ff

varoonp123 and others added 8 commits August 4, 2023 15:21

Use an option type a bit more to more effectively handle the first ti…

49e1ff3

…me we write to the db for a given user and not log an error

BUG FIX: Pull from the correct cache when getting historical stats in…

cae06ef

… the game over screen

Get the db plugin test working

3679111

Do a bit more cargo work in CI

a78d4fd

add stats state for displaying historical metrics

fcb1bf0

add run stats to game over screen

183a02b

add run stats to victory screen

e6b2aa8

add historical metric to main menu and remove stats screen and state

738dcc3

varoonp123 force-pushed the vp/feature/platformSpecificPortability branch from 0279781 to 738dcc3 Compare August 4, 2023 20:23

varoonp123 marked this pull request as ready for review August 4, 2023 21:02

varoonp123 commented Aug 4, 2023

View reviewed changes

varoonp123 mentioned this pull request Aug 4, 2023

Use consistent package version for all crates in the workspace. #67

Closed

varoonp123 commented Aug 4, 2023

View reviewed changes

varoonp123 mentioned this pull request Aug 4, 2023

Use consistent bevy versions across all deps. Ideally we only update 1 place when they release a new version. #68

Closed

varoonp123 commented Aug 4, 2023

View reviewed changes

varoonp123 mentioned this pull request Aug 4, 2023

I think directories is slightly higher leveled. Use that instead. #69

Closed

varoonp123 and others added 2 commits August 4, 2023 16:39

Drop dead code

39a3b27

remove cleanup commponent for stats state and remove enter_mainmenu_s…

93c1433

…ystem

cdsupina approved these changes Aug 4, 2023

View reviewed changes

varoonp123 merged commit f353a8b into thetawavegame:main Aug 4, 2023
7 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

User metrics with optional platform specific db persistence #66

User metrics with optional platform specific db persistence #66

varoonp123 commented Aug 4, 2023 •

edited

Loading

varoonp123 Aug 4, 2023

varoonp123 Aug 4, 2023

varoonp123 Aug 4, 2023

cdsupina left a comment

		bevy_ecs_macros = "0.11.0"
		bevy_ecs = "0.11.0"

User metrics with optional platform specific db persistence #66

User metrics with optional platform specific db persistence #66

Conversation

varoonp123 commented Aug 4, 2023 • edited Loading

varoonp123 Aug 4, 2023

Choose a reason for hiding this comment

varoonp123 Aug 4, 2023

Choose a reason for hiding this comment

varoonp123 Aug 4, 2023

Choose a reason for hiding this comment

cdsupina left a comment

Choose a reason for hiding this comment

varoonp123 commented Aug 4, 2023 •

edited

Loading