-
Notifications
You must be signed in to change notification settings - Fork 128
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
Are attributes supported with SST + FFS? #1006
Comments
Attributes are not yet supported in FFS marshaling. This is why we switched the default to BP marshaling prior to the SC tutorial. They’ll be supported eventually, but we have bigger fish to fry at the moment…
g
… On Nov 19, 2018, at 10:21 AM, Keichi Takahashi ***@***.***> wrote:
The heat transfer example in adiosvm/Tutorial/heat2d/cpp/ terminates with the following error when using SST engine and FFS marshaling:
gndx = 100
gndy = 100
rank 0 reads 2D slice 100 x 100 from offset (0,0)
ERROR: found null pointer in call to Attribute<T>::Data()
Usage: heatAnalysis input output N M
input: name of input data file/stream
output: name of output data file/stream
N: number of processes in X dimension
M: number of processes in Y dimension
It seems that InquireAttribute() is returning an empty attribute object.
https://github.com/pnorbert/adiosvm/blob/872dfdb2d78bd3d932509d22e3b7e361ee6b0084/Tutorial/heat2d/cpp/analysis/heatAnalysis.cpp#L133-L134 <https://github.com/pnorbert/adiosvm/blob/872dfdb2d78bd3d932509d22e3b7e361ee6b0084/Tutorial/heat2d/cpp/analysis/heatAnalysis.cpp#L133-L134>
Since the same workflow works with SST + BP, InSituMPI and BPFile, I think it is a problem with FFS marshaling.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub <#1006>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AA5fmQ_sJ6ZHBAICqos4Rf3H06vyzRKTks5uwsx6gaJpZM4YpUhT>.
|
Got it, I found this out because JAXA is mainly using this workflow for testing. I can just comment out these lines for now. |
Yeah, sorry. JAXA is a bit of a unique case here because of the architecture heterogeneity not being supported by BP and the attributes not being supported by FFS. I’ll try to get the attributes sorted reasonably soon.
g
… On Nov 19, 2018, at 1:35 PM, Keichi Takahashi ***@***.***> wrote:
Got it, I found this out because JAXA is mainly using this workflow for testing. I can just comment out these lines for now.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub <#1006 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AA5fmQoSvuSfGAsrC36BoRo2g9OweXoaks5uwvnhgaJpZM4YpUhT>.
|
Attribute support for FFS marshaling has been merged to master. Currently FFS marshaling supports attributes to the extent that BP does, which is not completely. In particular, the support ignores the need to make attributes work (still be applied) even if timesteps are dropped, or a reader arrives late to a stream. That work is ongoing in SST, but I'll close this now as basic support is in place. |
The heat transfer example in
adiosvm/Tutorial/heat2d/cpp/
terminates with the following error when using SST engine and FFS marshaling:It seems that InquireAttribute() is returning an empty attribute object.
https://github.com/pnorbert/adiosvm/blob/872dfdb2d78bd3d932509d22e3b7e361ee6b0084/Tutorial/heat2d/cpp/analysis/heatAnalysis.cpp#L133-L134
Since the same workflow works with SST + BP, InSituMPI and BPFile, I think it is a problem with FFS marshaling.
The text was updated successfully, but these errors were encountered: