Skip to content

Conversation

@mutatrum
Copy link
Collaborator

@mutatrum mutatrum commented Nov 19, 2025

Fixes #1362

This PR was mostly generated by Google's Antigravity IDE. It's an experiment from my side. Things it did well:

  • Analyzing the application
  • Finding a place to put new components
  • Discovering and properly using existing code like hashSuffix and tooltip-icon (only pointing out that they exist);
  • Help with naming things
  • Verifying correct usage (see change in home.components.ts)

Things it struggles with:

  • Styling, it didn't grasp the theme system, so all colors are hard-coded
  • Burn through credits. Depleted credits of 3 out of 5 models. Some models you can only ask one or two things before running out. Others you can use for quite a while.

Total development time: ~2 hours. This was almost exclusively front-end, where the integration shines. Not sure what it'll do with a back-end.

image

@github-actions
Copy link

github-actions bot commented Nov 19, 2025

Test Results

23 tests  ±0   23 ✅ ±0   0s ⏱️ ±0s
 1 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit 62ca46c. ± Comparison against base commit d647eb8.

♻️ This comment has been updated with latest results.

@Travetown
Copy link

Screenshots are urgently needed! 😂

// live data
this.info$ = interval(5000).pipe(
startWith(() => this.systemService.getInfo()),
startWith(0), // Emit immediately on subscription
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Incorrect usage found by AI

@Travetown
Copy link

I would remove the description in the table and just leave the legend.
1P is not really important. Maybe add 10, 50, and 100 G?

@mutatrum
Copy link
Collaborator Author

I would remove the description in the table and just leave the legend. 1P is not really important. Maybe add 10, 50, and 100 G?

I did not interfere at all with the text or code of the new page. I didn't even look at the code of the new page. I did check changes in existing code, and ran it on actual hardware. This PR is more of an experiment in how it handles the codebase, which didn't disappoint on almost every aspect. Also, I'm not an Angular expert, so I cannot really comment on code quality here, hence it's up for review. Content and text we can finetune later, when we deem the code is acceptable.

I do like the P in the table as that's the future where we're going, the network diff will keep going up. It's a nice book-end of the table and whenever we cross that, we can add E.

@Travetown
Copy link

Travetown commented Nov 19, 2025

I don't want a P! 😤 That's a terrible thought. My poor Bitaxe will then only be worth remembering.

@mutatrum mutatrum changed the title Add solo change page Add solo chance page Nov 24, 2025
@mutatrum mutatrum force-pushed the feature/solo-mining-calculator branch 2 times, most recently from c2401ad to 3245228 Compare December 15, 2025 14:03
Fixes bitaxeorg#1362

This PR was mostly generated by Google's Antigravity AI. It's an experiment from my side. Things it did well:

 * Analysing the application
 * Finding a place to put new components
 * Discovering existing components (with some help)
 * Help with naming things
 * Verifying correct usage (see change in `home.components.ts``)

Things it struggles with:

 * Styling, it didn't grasp the theme system, so all colors are hardcoded
@mutatrum mutatrum force-pushed the feature/solo-mining-calculator branch from 3245228 to 2cc4fc3 Compare December 15, 2025 14:11
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.

Add a solo chance calculator

3 participants