Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

CoreCoord exceptions #518

Open
pjanevskiTT opened this issue Feb 6, 2025 · 0 comments
Open

CoreCoord exceptions #518

pjanevskiTT opened this issue Feb 6, 2025 · 0 comments

Comments

@pjanevskiTT
Copy link
Contributor

pjanevskiTT commented Feb 6, 2025

Sub issue of master issue #517

Each exception will have the proposal for additional fields apart from standard exception message

Proposing few exceptions that can be implemented

  • IncorrectDramHarvestingMask exception
class IncorrectDramHarvestingMask : public std::exception {
    // fields
   uint32_t incorrect_dram_harvesting_mask;
}
  • IncorrectEthHarvestingMask exception
class IncorrectEthHarvestingMask : public std::exception {
    // fields
   uint32_t incorrect_eth_harvesting_mask;
}
  • AmbigiousLogicalCoordinates, used when no CoreType for logical coords is specified
class AmbigiousLogicalCoordinates : public std::exception {
    // fields
   tt_xy_pair logical_coords;
}
  • InvalidCoordinate. used when all the fields are populated but the coordinate doesn't exist
class InvalidCoordinate : public std::exception {
    // fields
   CoreCoord logical_coords;
}

I will add more when I have ideas. Feel free to rename the exceptions to something more meaningful if you want

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

No branches or pull requests

1 participant