Develop

[jaredraycoleman]

No description provided.

[Copilot]

Pull Request Overview

This PR implements various enhancements and new features across the codebase, including utility improvements, new scheduling algorithms, and updated visualization functions that support dynamic labeling and transcript callbacks. Key changes include:

• Adding a round method and new comparison operators in RandomVariable.
• Updating drawing functions to support dynamic font sizing and label rotation.
• Introducing MSTScheduler and integrating transcript callbacks and clustering in HEFT, CPoP, and other scheduler modules.

Reviewed Changes

Copilot reviewed 27 out of 29 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
src/saga/utils/random_variable.py	Added round and comparison operators; note round does not perform rounding.
src/saga/utils/draw.py	Updated drawing functions with dynamic labeling parameters and transcript callbacks.
src/saga/schedulers/mst.py	Introduced MSTScheduler for minimum start time scheduling.
src/saga/schedulers/heft.py	Enhanced HEFT scheduling with transcript callbacks and clustering support.
src/saga/schedulers/etf.py	Updated function signature to support optional clustering on scheduling.
src/saga/schedulers/data/wfcommons.py	Modified workflow conversion and network generation with optional random variable weights.
src/saga/schedulers/cpop.py	Added transcript callbacks and clustering support to CPoP scheduling.
src/saga/schedulers/init.py	Integrated MSTScheduler into the scheduler registry.
scripts/examples/*	Updated examples to incorporate new scheduler parameters and output file types.

Files not reviewed (2)

• scripts/examples/online/.gitignore: Language not supported
• scripts/examples/stochastic/.gitignore: Language not supported

Comments suppressed due to low confidence (2)

src/saga/utils/draw.py:579

• The variable 'font_size' is used but not defined in the current scope of gradient_heatmap. Ensure that 'font_size' is either defined or passed as a parameter to the function.

ax.set_xticklabels(xvals, rotation=rotate_xlabels, fontsize=font_size)

src/saga/utils/draw.py:580

• The variable 'font_size' is referenced here without a definition in the scope. Consider defining 'font_size' or making it a function parameter to ensure consistent behavior.

ax.set_yticklabels(yvals, fontsize=font_size)

[Copilot]

The round method currently does not apply any rounding logic and simply returns self. Consider implementing proper rounding on the underlying random variable data.

Suggested change

	return self
	rounded_samples = np.round(self.samples, n)
	return RandomVariable(rounded_samples)