Feature/change to converted file

[interim17]

website branch fix/conversion-filename-bug points at these changes to receive file that results from autoconversion

Time Estimate or Size

small-ish

Problem

The handling of autoconverted files was built in the viewer with the example viewer in mind, but now that we're integrating with the website repo, we need to make some minor changes to the viewer.

The way autoconverted files are currently handled in the viewer is that the viewer requests a file to be converted and immediately requests frame 0. However, in the website, we want to clear any previous trajectory in between these two steps, allowing users to resume interacting with the previous simulation if they cancel the autoconversion request before octopus has replied with the converted data.

Solution

• change SimulariumController's convertAndLoadTrajectory() to convertTrajectory(), and have that only handle sending the conversion request. The clearing of any previous trajectory and loading of frame zero can be done after, using changeFile()
• update changeFile() to include the option to not create a new remote simulator instance. Overall, if this.simulator is a remote simulator and the connectionParams indicate that the newly requested file also uses a remote simulator, we don't need to create a fresh remote simulator instance, we can keep using the existing one, but we don't. For non-autoconversion simulations, creating a fresh instance doesn't cause any issues, so it doesn't matter. For autoconversion, we will have already created a remote simulator instance for requesting the conversion, and we don't want to miss any responses from that by creating a new instance with a new websocket connection, so we need to skip creating a simulator. Right now, I'm only skipping this step for this autoconversion case, although we could expand and use this across remote simulator cases in the future
• allow the website to pass in it's own file name to be used internally for the autoconverted file, since we're generating that in the viewer anyways
• update the example viewer autoconversion form to use these new / updated methods
• point to the new octopus ecs staging instance, instead of the trial ec2 instance's IP address

with @ascibisz

Type of change

• Bug fix (non-breaking change which fixes an issue)

[github-actions[bot]]

jest coverage report 🧪

Total coverage

Status	Category	Percentage	Covered / Total
🔴	Statements	40.37%	2048/5073
🔴	Branches	43.02%	845/1964
🔴	Functions	36.79%	418/1136
🔴	Lines	40.59%	1961/4831

Status of coverage: 🟢 - ok, 🟡 - slightly more than threshold, 🔴 - under the threshold

Show files with reduced coverage 🔻

### Reduced coverage | Status | Filename | Statements | Branches | Functions | Lines | | :----: | :----------------- | :--------- | :---------------- | :-------- | :------ | | 🔴 | src/controller | 30.09% | 14.78% | 16.94% | 30.43% | | 🟡 | src/simularium | 60.32% | 57.69% | 49.14% | 61.19% | | 🔴 | RemoteSimulator.ts | 48.85% | 62.5% (-4.16% 🔻) | 27.27% | 52.1% | > Status of coverage: 🟢 - ok, 🟡 - slightly more than threshold, 🔴 - under the threshold