Remove deprecated MOOSE code about moltres HOT 7 CLOSED

Oh I initially misunderstood you but I get what you mean now. Thanks for clarifying!

from moltres.

Thanks for the update. We'll track the eventual transition to Eigenvalue with #177.

from moltres.

Thanks for the heads up! The p parameter is easy enough to replace. The transition to Eigenvalue executioner will require more manhours because we'd have to change our Action files.

On your last point, would we have to manually turn on "Test deprecated" in every PR?

from moltres.

On the CIVET user interface, if you have sufficient permissions, you can turn on tests that don't normally run on PRs. For example I am also a maintainer of Zapdos, so if I look at an open PR, I can also see the option to turn on the deprecated test that would only otherwise run on the master branch:

Do you see this when looking at an open PR results page on CIVET for Moltres? I could check and then submit "Test deprecated" as an add-on for that Zapdos PR above.

from moltres.

I would only add this for the PR you submit to close this.....you don't necessarily want it running all the time, just when you're trying to clean up what is currently being flagged.

from moltres.

@cticenhour Is InversePowerMethod no longer deprecated? I noticed that the CIVET deprecation tests are passing, and there are some open issues about the new Eigenvalue executioner (e.g., idaholab/moose#20095)

from moltres.

You are correct. There seems to be some issues regarding the new executioner, so InversePowerMethod is no longer deprecated for the moment. If you have fixed up everything else, I would say this issue can be closed.

from moltres.