Eagle-1194: Remove 'keyAttribute' attribute from 'field' structure in graph JSON

[james-strauss-uwa]

We are moving away from the 'keyAttribute' system and towards using a Graph Configuration to overwrite specific parameters in a Logical Graph Template.

[sourcery-ai]

🧙 Sourcery has finished reviewing your pull request!

---

Tips

• Trigger a new Sourcery review by commenting @sourcery-ai review on the pull request.
• Continue your discussion with Sourcery by replying directly to review comments.
• You can change your review settings at any time by accessing your dashboard:
  • Enable or disable the Sourcery-generated pull request summary or reviewer's guide;
  • Change the review language;
• You can always contact us if you have any questions or feedback.

[sourcery-ai]

Hey @james-strauss-uwa - I've reviewed your changes and they look great!

Here's what I looked at during the review

• 🟢 General issues: all looks good
• 🟢 Security: all looks good
• 🟢 Testing: all looks good
• 🟢 Complexity: all looks good
• 🟢 Documentation: all looks good

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

_{Help me be more useful! Please click 👍 or 👎 on each comment to tell me if it was helpful.}

[james-strauss-uwa]

I guess I need to re-generate the pickled versions of the graphs in daliuge-translator/test/dropmake/pickle/

[myxie]

@james-strauss-uwa this is another one I have missed. I am assuming these changes will need to be merged at some point; I am wondering if you would be okay with me doing the changes at the same time I created the new entry in EAGLE_test_repo?

[james-strauss-uwa]

@myxie Yes, that would be great thanks.