the problems exp37

[liangaomng]

dear sir,

Thanks for this codes. It shows this error.

In the exp37, Number of control unknowns: 12288

Step = 1
Traceback (most recent call last):

b.AddDomainIntegrator(mfem.VectorDomainLFIntegrator(self.rhs_cf))

File "/mfem/_ser/lininteg.py", line 367, in init
self._ir=ir
NameError: name 'ir' is not defined. Did you mean: 'dir'?

[bluthej]

Hey, I ran into the same issue. I'm not too familiar with the PyMFEM codebase, but it looks like files are generated with Swig, and for some reason in the release there is an inconsistency between the signature of the constructor of VectorDomainLFIntegrator and its body. The signature reads def __init__(self, QF):, and there is no ir parameter, so when you get to self._ir=ir it errors out.

When I pip install the package from source, the signature of the constructor is def __init__(self, QF, ir=None):, so ir is indeed present, I'm not sure how it can be different in the release... It looks like the last time this was touched was on the day that 4.8.0.1 was releases, but it may have been fixed since then.

Anyway, I guess if you can't pip install it and have to use the release you can patch this function signature and it should work (I tried exp37 like this and it seems to work).

[sshiraiwa]

The signature of VectorDomainLFIntegrator has changed, while ago and wa not updated in the released version. This will be addressed in next release, which I am still working on.