Skip to content

Conversation

@hhkit
Copy link
Collaborator

@hhkit hhkit commented Jan 27, 2025

This is not a fully engineered fix (many design solutions to consider for extensibility), but for now, I've put together this quickfix for the compile errors.

@hhkit hhkit changed the title Compile error hotfix [IFRT] Compile error hotfix Jan 27, 2025
@mofeing
Copy link
Collaborator

mofeing commented Jan 27, 2025

Thanks @hhkit! I'll wait until tomorrow's meeting because they resolve some questions about this.

@hhkit
Copy link
Collaborator Author

hhkit commented Jan 28, 2025

No worries, just wanted to take some stuff off your plate so you can do the real work. :)

@mofeing
Copy link
Collaborator

mofeing commented Jan 28, 2025

yes! really appreciate it!

@mofeing mofeing merged commit 90f608d into EnzymeAD:ss/ifrt Feb 1, 2025
9 of 11 checks passed
@hhkit hhkit deleted the hhkit/ifrt-patch branch February 1, 2025 21:53
mofeing pushed a commit that referenced this pull request Feb 7, 2025
* hotfix compile errors

* format
mofeing pushed a commit that referenced this pull request Feb 8, 2025
* hotfix compile errors

* format
mofeing pushed a commit that referenced this pull request Feb 9, 2025
* hotfix compile errors

* format
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