[Harvesting] simulated_harvesting_masks passed to tt_SocDescriptor

[broskoTT]

Issue

Related to #439

Description

First of the changes preparing removal of harvesting logic inside Cluster.
perform_harvesting, and simulated_harvesting_masks arguments are now added to affect tt_SocDescriptor.
Had to remove logic converting harvesting_mask from physical layout coords to logical coords out of the CoordinateManager constructor.

List of the changes

• perform_harvesting now affects harvesting in tt_SocDescriptor
• simulated_harvesting_masks are now used for harvesting in tt_SocDescriptor
• get_tensix_harvesting_mask is a helper function which does both of these
• shuffle_tensix_harvesting_mask is now a static public function, which should be called prior to calling CoordinateManager
• CoordinateManager now accepts harvesting in logical coords.
• Due to deprecated get_harvesting_masks_for_soc_descriptors, there is an additional shuffle_tensix_harvesting_mask_to_noc0_coords which does the required translation.
• tt_SocDescriptor removed copy constructor, since it is already a default one.
• Other functions changed accordingly to pass this arguments

Testing

• Changed call to get_harvesting_masks_for_soc_descriptors, so it is tested with the new flow, through harvesting provided by socdescriptor class
• Added HarvestingShuffle tests in all arch tests

API Changes

There are no API changes in this PR.