Decouple topologies and operators

[whzup]

Description

Removed the compute_velocity and compute_position methods in the topologies to decouple them.

Related Issue

#259

Motivation and Context

There was no reason for the methods to be in the topologies. This change makes the backend more coherent.

How Has This Been Tested?

Tests run.

Screenshots (if appropriate):

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

• My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have read the CONTRIBUTING document.
• I have added tests to cover my changes.
• All new and existing tests passed.

[ljvmiranda921]

Hey @whzup ! Thanks for this! Let me check this over the weekend :)

[ljvmiranda921]

Thanks for this @whzup ! Just for formatting, can you run black on all affected files?

[ljvmiranda921]

I like this PR, we're removing more code (feels good!). Let's fix the conflicts first (mostly imports because of black) then this is good to merge!

[ljvmiranda921]

Let's fix some conflicts first (some import formatting due to black), then we're good to merge!

[stale]

Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward?

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.