-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
Fix bug in DenseScanner of DataProviderConverter. #1121
Fix bug in DenseScanner of DataProviderConverter. #1121
Conversation
@@ -47,6 +51,8 @@ def scan(self, dat): | |||
def finish_scan(self, argument): | |||
assert isinstance(argument, swig_paddle.Arguments) | |||
assert isinstance(self.input_type, dp2.InputType) | |||
if self.__mat__.dtype != numpy.float32: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
即使__mat__初始化成float32的类型,在numpy.append操作中,dtype还是有可能被修改成float64.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
什么情况下会中招?表现是啥
@@ -47,6 +51,8 @@ def scan(self, dat): | |||
def finish_scan(self, argument): | |||
assert isinstance(argument, swig_paddle.Arguments) | |||
assert isinstance(self.input_type, dp2.InputType) | |||
if self.__mat__.dtype != numpy.float32: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
什么情况下会中招?表现是啥
* test=document_preview * 0823 update FluidDoc,test=document_preview * add two files
No description provided.