Re: [PR] fix(operator): convert booleans to Python-compatible values in BubbleChart/ContourPlot/TernaryPlot [texera]
aglinxinyuan merged PR #4238: URL: https://github.com/apache/texera/pull/4238 -- 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] fix(operator): convert booleans to Python-compatible values in BubbleChart/ContourPlot/TernaryPlot [texera]
carloea2 commented on PR #4238: URL: https://github.com/apache/texera/pull/4238#issuecomment-4265883546 Thanks. -- 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] fix(operator): convert booleans to Python-compatible values in BubbleChart/ContourPlot/TernaryPlot [texera]
aglinxinyuan commented on PR #4238: URL: https://github.com/apache/texera/pull/4238#issuecomment-4265860337 > Why hasn't this PR been merged? Jiadong is busy. I will merge it now. -- 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] fix(operator): convert booleans to Python-compatible values in BubbleChart/ContourPlot/TernaryPlot [texera]
carloea2 commented on PR #4238: URL: https://github.com/apache/texera/pull/4238#issuecomment-4265835022 Why hasn't this PR been merged? -- 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] fix(operator): convert booleans to Python-compatible values in BubbleChart/ContourPlot/TernaryPlot [texera]
carloea2 commented on PR #4238: URL: https://github.com/apache/texera/pull/4238#issuecomment-4008683721 @chenlica this PR LGTM -- 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] fix(operator): convert booleans to Python-compatible values in BubbleChart/ContourPlot/TernaryPlot [texera]
xuang7 commented on PR #4238: URL: https://github.com/apache/texera/pull/4238#issuecomment-4008481421 > @carloea2 Is your suggested change taken care of? If so, we can merge this PR. Yes, I've applied @carloea2 suggested fix. -- 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] fix(operator): convert booleans to Python-compatible values in BubbleChart/ContourPlot/TernaryPlot [texera]
chenlica commented on PR #4238: URL: https://github.com/apache/texera/pull/4238#issuecomment-4002123442 @carloea2 Is your suggested change taken care of? If so, we can merge this PR. -- 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] fix(operator): convert booleans to Python-compatible values in BubbleChart/ContourPlot/TernaryPlot [texera]
xuang7 commented on PR #4238: URL: https://github.com/apache/texera/pull/4238#issuecomment-4001777150 > LGTM. Please explain briefly how the current fix solves the problem Thanks for the suggestion. I have updated the PR description with a short explanation. -- 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]
