Skip to content

iterPars need to match full parameter names #55

Closed

Description

@pavoljuhas

it would allow to constrain all equal-type atoms to the same ADPs. It is currently annoyingly clunky.

Activity

  1. ycexiao commented on Jul 29, 2025

    @ycexiao
    Contributor

    @sbillinge it can not be fixed quickly. I think we can put it in the next release.

    Though the implementation might be simple, one of the tests depends on the iterPars. This test will be skipped unless diffpy.srreal and diffpy.structure are installed. Now diffpy.srreal is not available for python==3.13. We might want to wait until diffpy.srreal is successfully updated, and then we can run all the tests that involve iterPars.

  2. stevenhua0320 commented on May 21, 2026

    @stevenhua0320
    Contributor

    @sbillinge I believe we can close this as we fixed via #183

  3. sbillinge commented on May 22, 2026

    @sbillinge
    Contributor

    @sbillinge I believe we can close this as we fixed via #183

    Can we confirm this and I will close

  4. stevenhua0320 commented on May 22, 2026

    @stevenhua0320
    Contributor

    @sbillinge I believe the PR #183 have fixed this issue that equal type atoms should have same ADP by doing a global recursive search. We have incorporated some test to test the behavior and why we did not close this issue immediately is because I forgot to make the Close issue comment in the PR.

  5. sbillinge commented on May 30, 2026

    @sbillinge
    Contributor

    closed by #183

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions