Skip to content

Better derivative evaluation out of memory error message#1026

Merged
shamsmosowi merged 1 commit intodevelopfrom
feature/better-derivative-failure-message
Dec 26, 2022
Merged

Better derivative evaluation out of memory error message#1026
shamsmosowi merged 1 commit intodevelopfrom
feature/better-derivative-failure-message

Conversation

@bofeiw
Copy link
Contributor

@bofeiw bofeiw commented Dec 18, 2022

Derivative evaluation might fail due to Rowy Run out of memory.

Instead of "failed to fetch",
Screenshot 2022-12-19 at 04 36 24

display the issue and ask user to increase memory in GCP console.
Screenshot 2022-12-19 at 04 38 24

@bofeiw bofeiw requested a review from shamsmosowi December 18, 2022 17:40
@vercel
Copy link

vercel bot commented Dec 18, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
rowy-os ✅ Ready (Inspect) Visit Preview Dec 18, 2022 at 5:40PM (UTC)
1 Ignored Deployment
Name Status Preview Updated
rowy-typedoc ⬜️ Ignored (Inspect) Dec 18, 2022 at 5:40PM (UTC)

@bofeiw bofeiw changed the base branch from main to develop December 18, 2022 20:24
@shamsmosowi
Copy link
Contributor

failed to fetch can potentially caused by multiple reasons, would it be possible for the server to return an out of memory error or something more specific?

@bofeiw
Copy link
Contributor Author

bofeiw commented Dec 20, 2022

@shamsmosowi Unfortunately, that error message is out of our control. GCP cuts the execution off, leaving client side to have this failed to fetch error.

If the error is not caused by GCP cutting the execution off, we have the control and we can customise the UI. Let's keep an eye on it.

@shamsmosowi shamsmosowi merged commit 8915bb7 into develop Dec 26, 2022
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.

2 participants