Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Blazor Sample App #779

Merged
merged 17 commits into from
Apr 21, 2022
Merged

Blazor Sample App #779

merged 17 commits into from
Apr 21, 2022

Conversation

kilngod
Copy link
Contributor

@kilngod kilngod commented Mar 29, 2022

Blazor sample application

@kilngod kilngod changed the title Initial Commit Blazor Sample App Mar 29, 2022
@m4rs-mt m4rs-mt added this to the v1.2 milestone Apr 2, 2022
@m4rs-mt m4rs-mt added enhancement feature A new feature (or feature request) labels Apr 2, 2022
@m4rs-mt
Copy link
Owner

m4rs-mt commented Apr 4, 2022

@kilngod Thanks for this great sample 🥇. Would it be possible to disable secure connections via https on the localhost side in the Blazer website configuration? This seems to cause some problems from time to time on various test machines.

@kilngod
Copy link
Contributor Author

kilngod commented Apr 6, 2022

@m4rs-mt I'll take a look at removing the https:// from the sample app Friday.

@kilngod
Copy link
Contributor Author

kilngod commented Apr 6, 2022

@m4rs-mt Marcel, turns out I had time this morning to sort out removing https:// actually simple to remove.

Copy link
Owner

@m4rs-mt m4rs-mt left a comment

Choose a reason for hiding this comment

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

Great work! I had some problems running this example on a computer without OpenCL or Cuda compatible devices, as the backend throws some OutOfRange exceptions when clicking some buttons. Should we add a note that at least an OpenCL or Cuda compatible accelerator should be available?

Another more high-level question: can we remove the pages Basic Canvas Test and JavaScript notes from the sample?

Copy link
Owner

@m4rs-mt m4rs-mt left a comment

Choose a reason for hiding this comment

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

Great job 👍 Thank you for addressing my comments.

@m4rs-mt m4rs-mt merged commit c8227bc into m4rs-mt:master Apr 21, 2022
@kilngod kilngod deleted the BlazorDemoApp branch December 27, 2023 20:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement feature A new feature (or feature request)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants