ARC icon indicating copy to clipboard operation
ARC copied to clipboard

Psi4 adapter

Open kfir4444 opened this issue 3 years ago • 15 comments

A functioning Psi4 adapter.

kfir4444 avatar Jun 28 '22 09:06 kfir4444

This pull request introduces 1 alert and fixes 2 when merging 09ac7f59fe4266905a2f8d67dc9065ea06bcab77 into ef0d5c690f648eabea7080c2af1e969ae8628646 - view on LGTM.com

new alerts:

  • 1 for Unused import

fixed alerts:

  • 2 for Unused import

lgtm-com[bot] avatar Jun 28 '22 21:06 lgtm-com[bot]

LGTM eventually completed after 12 hrs... It complains about unused imports

alongd avatar Jun 29 '22 00:06 alongd

This pull request introduces 2 alerts and fixes 4 when merging fe3ceb0485a8d88e75db4ee5f3b005ad8033cb80 into 2661d862f1b812ffd02a6c805f25f42bc0f1d8bb - view on LGTM.com

new alerts:

  • 1 for Unused import
  • 1 for Duplicate key in dict literal

fixed alerts:

  • 4 for Unused import

lgtm-com[bot] avatar Jul 06 '22 10:07 lgtm-com[bot]

This pull request fixes 4 alerts when merging ab998cfed03869f11e6dc648564129dacd85d3a2 into 2661d862f1b812ffd02a6c805f25f42bc0f1d8bb - view on LGTM.com

fixed alerts:

  • 4 for Unused import

lgtm-com[bot] avatar Jul 06 '22 11:07 lgtm-com[bot]

This pull request fixes 4 alerts when merging 9ea351ab6b6d5414b2660da749368f2452e7b9db into 542528b4e055a9bbbf3c686d35e6d07d77e288c3 - view on LGTM.com

fixed alerts:

  • 4 for Unused import

lgtm-com[bot] avatar Jul 10 '22 17:07 lgtm-com[bot]

This pull request introduces 2 alerts and fixes 4 when merging 5b04571cb3d3e1bcf589b1f36f4e6e647f7a82f1 into 6dc1de367139d08bf1169c09600a3e2970761782 - view on LGTM.com

new alerts:

  • 1 for Unused import
  • 1 for Unreachable code

fixed alerts:

  • 4 for Unused import

lgtm-com[bot] avatar Jul 31 '22 18:07 lgtm-com[bot]

This pull request introduces 3 alerts and fixes 4 when merging 4f0e6a07e1e8fbbe834fdde945c48c2320db1ec9 into 05e5ab6a8c23c85a175dc0f32d672f44e334febc - view on LGTM.com

new alerts:

  • 1 for Unused import
  • 1 for Unnecessary 'else' clause in loop
  • 1 for Variable defined multiple times

fixed alerts:

  • 4 for Unused import

lgtm-com[bot] avatar Aug 15 '22 12:08 lgtm-com[bot]

This pull request introduces 1 alert and fixes 4 when merging 2b0588c0a77b7883451dda2d4fbfc74f91917a6f into 05e5ab6a8c23c85a175dc0f32d672f44e334febc - view on LGTM.com

new alerts:

  • 1 for Unnecessary 'else' clause in loop

fixed alerts:

  • 4 for Unused import

lgtm-com[bot] avatar Aug 15 '22 12:08 lgtm-com[bot]

This pull request introduces 1 alert and fixes 4 when merging 58380b42a2478af65bd33273eb0fbd055a2b97a9 into 05e5ab6a8c23c85a175dc0f32d672f44e334febc - view on LGTM.com

new alerts:

  • 1 for Unnecessary 'else' clause in loop

fixed alerts:

  • 4 for Unused import

lgtm-com[bot] avatar Aug 15 '22 13:08 lgtm-com[bot]

This pull request introduces 1 alert and fixes 4 when merging dc41b2a59f6e056e61bfb9ba88137d57a6f13271 into 58b5698d093088186816a975de0a1a8ce519d3b8 - view on LGTM.com

new alerts:

  • 1 for Unnecessary 'else' clause in loop

fixed alerts:

  • 4 for Unused import

lgtm-com[bot] avatar Aug 16 '22 06:08 lgtm-com[bot]

This pull request introduces 1 alert and fixes 3 when merging c053c3c22ba2c7f850ef0db926c8e9a22ae1f7ab into 58b5698d093088186816a975de0a1a8ce519d3b8 - view on LGTM.com

new alerts:

  • 1 for Unnecessary 'else' clause in loop

fixed alerts:

  • 3 for Unused import

lgtm-com[bot] avatar Aug 23 '22 16:08 lgtm-com[bot]

This pull request introduces 1 alert and fixes 3 when merging 07c3ce8cf9c2c9deb61a0595038d524329e62366 into d60a51d1d97dc3ba088719d13d6ae7628158ce50 - view on LGTM.com

new alerts:

  • 1 for Unnecessary 'else' clause in loop

fixed alerts:

  • 3 for Unused import

lgtm-com[bot] avatar Aug 25 '22 18:08 lgtm-com[bot]

@kfir4444, take a look at the LGTM comments?

alongd avatar Aug 27 '22 22:08 alongd

This pull request fixes 3 alerts when merging abacd939be49f1be30bc4fd43c5b36a7ef318c11 into d60a51d1d97dc3ba088719d13d6ae7628158ce50 - view on LGTM.com

fixed alerts:

  • 3 for Unused import

lgtm-com[bot] avatar Aug 29 '22 17:08 lgtm-com[bot]

@kfir4444, can you remove the unused imports highlighted by LGTM?

alongd avatar Aug 30 '22 02:08 alongd