Re: [PR] [QDP] Add TensorFlow .pb support to unified encode() path [mahout]

2026-01-21 Thread via GitHub


shiavm006 commented on PR #891:
URL: https://github.com/apache/mahout/pull/891#issuecomment-3777539474

   > You can test on colab! Btw, you remind me I have to update the colab 
script because a lot of things changes
   > 
   > edit: probably cannot. This is Kinda hard...
   
   ok done will do that 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]



Re: [PR] [QDP] Add TensorFlow .pb support to unified encode() path [mahout]

2026-01-21 Thread via GitHub


guan404ming merged PR #891:
URL: https://github.com/apache/mahout/pull/891


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]



Re: [PR] [QDP] Add TensorFlow .pb support to unified encode() path [mahout]

2026-01-20 Thread via GitHub


ryankert01 commented on PR #891:
URL: https://github.com/apache/mahout/pull/891#issuecomment-3776148437

   You can test on colab! Btw, you remind me I have to update the colab script 
because a lot of things changes


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]



Re: [PR] [QDP] Add TensorFlow .pb support to unified encode() path [mahout]

2026-01-20 Thread via GitHub


shiavm006 commented on PR #891:
URL: https://github.com/apache/mahout/pull/891#issuecomment-3776142066

   > @shiavm006 So far so good! Could you help test if it works locally and 
paste the result as part of the pr description? Thanks for your contributions!
   
   Can't test locally due to pre-existing macOS build issues in qdp-core


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]



Re: [PR] [QDP] Add TensorFlow .pb support to unified encode() path [mahout]

2026-01-20 Thread via GitHub


shiavm006 commented on PR #891:
URL: https://github.com/apache/mahout/pull/891#issuecomment-3776057594

   cc @ryankert01 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]