Skip to content

Conversation

@sg-s
Copy link
Collaborator

@sg-s sg-s commented Nov 18, 2025

changes

ability to run a detailed local server that responds exactly like the real backend, but obviously runs instantly, using fixtures to construct responses.

what

Screen.Recording.2025-11-18.at.2.00.54.PM.mov

@sg-s sg-s self-assigned this Nov 18, 2025
@sg-s sg-s requested a review from a team as a code owner November 18, 2025 19:02
@sg-s sg-s changed the title Abfe mock Local server, mocked responses and testing of notebooks Nov 18, 2025
@BrianB2
Copy link

BrianB2 commented Nov 18, 2025

Logo
Checkmarx One – Scan Summary & Details8d299f7c-fa0f-4b1f-9730-6fcc823dd6fe

Great job! No new security vulnerabilities introduced in this pull request


Use @Checkmarx to reach out to us for assistance.

Just send a PR comment with @Checkmarx followed by a natural language request.

Examples: @Checkmarx how are you able to help me? @Checkmarx rescan this PR

Comment on lines 46 to +48

protein.upload()
ligand.upload()
protein.upload(client=client)
ligand.upload(client=client)
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

need to pass this to make sure this uses the correct client (context)

Comment on lines -422 to 423
def watch(self):
def watch(self, *, interval: float = 5.0):
"""Start monitoring job progress in real-time.
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

support for faster updates, e.g., in a demo

Copy link
Contributor

@aregsyan aregsyan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is excellent!

@sg-s sg-s merged commit 5eb3f66 into main Nov 19, 2025
13 of 14 checks passed
@sg-s sg-s deleted the abfe-mock branch November 19, 2025 02:34
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

Successfully merging this pull request may close these issues.

4 participants