Skip to content

Add ASCIITexture and ASCIIEffect - #688

Merged
vanruesc merged 8 commits into
pmndrs:v7from
LucaArgentieri:v7
Feb 27, 2025
Merged

vanruesc merged 8 commits into
pmndrs:v7from
LucaArgentieri:v7

Conversation

@LucaArgentieri

@LucaArgentieri LucaArgentieri commented Feb 21, 2025 •

Copy link
Copy Markdown
Contributor

Related issue: #685

Description

  • Created ASCIITexture
  • Created ASCIIEffect
  • Created ascii demo in manual
  • Added unit test (commented ASCIIEffect test and ASCIITexture test)
  • Refactored files as codebase

Errors / Problems

  • Something wrong with blend effects shader in demos (not only ASCIIEffect demo), demo error only if changed
    ERROR: 0:570: 'blend34' : no matching overloaded function found
    
  • ASCIIEffect is visible but not working correctly, I stopped to add others demo folder bindings because effect is not working as it should

@vanruesc

vanruesc commented Feb 22, 2025 •

Copy link
Copy Markdown
Member

I'm confused, why was the other PR closed and a new one created?

Something wrong with blend effects shader in demos (not only ASCIIEffect demo), demo error only if changed

That's strange. I'll take a look.

@vanruesc

Copy link
Copy Markdown
Member

Something wrong with blend effects shader in demos (not only ASCIIEffect demo), demo error only if changed

That's strange. I'll take a look.

Fixed via 629fb8d

@vanruesc

Copy link
Copy Markdown
Member

Thanks for this, I'll take care of the remaining clean up work.

Also, if you close a PR by accident, it's better to reopen it instead of creating a new one.

@vanruesc
vanruesc merged commit 1208493 into pmndrs:v7 Feb 27, 2025
@LucaArgentieri

Copy link
Copy Markdown
Contributor Author

Thanks for this, I'll take care of the remaining clean up work.

Also, if you close a PR by accident, it's better to reopen it instead of creating a new one.

Great! Thank you very much, sorry if I caused some inconvenience

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