Skip to content

Conversation

@DanteNiewenhuis
Copy link
Contributor

@DanteNiewenhuis DanteNiewenhuis commented Nov 4, 2025

Summary

Reduced RAM usage of OpenDC by picking better datatypes and merging Task and ServiceTask.

Implementation Notes ⚒️

Reduced RAM usage with the following changes:

  1. Removed Flavor and ServiceFlavor and moved the information to ServiceTask directly

  2. Merged Task and ServiceTask. They contained the same information, so it was just using more RAM.

  3. Updated variables of ServiceTask to reduce RAM

External Dependencies 🍀

N / A

Breaking API Changes ⚠️

  • Task does not exist anymore, all previous usages of Task are updated.

Simply specify none (N/A) if not applicable.

@DanteNiewenhuis DanteNiewenhuis self-assigned this Nov 4, 2025
@codecov
Copy link

codecov bot commented Nov 4, 2025

Codecov Report

❌ Patch coverage is 70.18349% with 65 lines in your changes missing coverage. Please review.
✅ Project coverage is 57.36%. Comparing base (59898b8) to head (fa7cbb7).
⚠️ Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
.../opendc/compute/simulator/service/ServiceTask.java 67.70% 25 Missing and 6 partials ⚠️
...mulator/scheduler/timeshift/MemorizingTimeshift.kt 0.00% 6 Missing ⚠️
...tlin/org/opendc/compute/workload/WorkloadLoader.kt 16.66% 4 Missing and 1 partial ⚠️
...endc/compute/simulator/service/ComputeService.java 92.30% 0 Missing and 2 partials ⚠️
...otlin/org/opendc/compute/simulator/host/SimHost.kt 50.00% 0 Missing and 2 partials ⚠️
...simulator/scheduler/weights/VCpuCapacityWeigher.kt 0.00% 2 Missing ⚠️
...simulator/scheduler/weights/VGpuCapacityWeigher.kt 0.00% 2 Missing ⚠️
...mulator/telemetry/parquet/DfltTaskExportColumns.kt 66.66% 1 Missing and 1 partial ⚠️
...g/opendc/compute/workload/ComputeWorkloadLoader.kt 90.47% 1 Missing and 1 partial ⚠️
...endc/trace/formats/workload/FragmentTableWriter.kt 0.00% 2 Missing ⚠️
... and 8 more
Additional details and impacted files
@@             Coverage Diff              @@
##             master     #379      +/-   ##
============================================
- Coverage     57.39%   57.36%   -0.04%     
  Complexity        1        1              
============================================
  Files           337      335       -2     
  Lines         11213    10788     -425     
  Branches       1041     1037       -4     
============================================
- Hits           6436     6188     -248     
+ Misses         4412     4235     -177     
  Partials        365      365              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@DanteNiewenhuis DanteNiewenhuis merged commit 71f6361 into atlarge-research:master Nov 4, 2025
5 checks passed
@DanteNiewenhuis DanteNiewenhuis deleted the memory_update branch November 4, 2025 20:09
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