dcbaker commented 2 years ago

This is still a Work in Progress, but I'm putting it here in case others want to add things, or make comments (and to see the work underway). This is mostly working for the main interpreter, but there are issues with the introspection code that isn't working yet.

lgtm-com[bot] commented 2 years ago

This pull request introduces 2 alerts when merging ab06a462ff9892acc594f04f7a43429a2f4be12d into db680a78d11c45064281dbc5a158919323dda666 - view on LGTM.com

new alerts:

2 for Unused import

lgtm-com[bot] commented 2 years ago

This pull request introduces 3 alerts when merging e8fdd9f76357d736141a0f2b221db200af10eb03 into c6862744cb91e1fff0ed2f18a1bb353e3292df28 - view on LGTM.com

new alerts:

3 for Unused import

lgtm-com[bot] commented 2 years ago

This pull request introduces 3 alerts and fixes 1 when merging c7574fa6ad8c42a802b275185d63866a1e6d9aa7 into 5d0538d2357259c49ea6abd7793584bbf04091d2 - view on LGTM.com

new alerts:

1 for First parameter of a method is not named 'self'
1 for Unused import
1 for Wrong number of arguments in a class instantiation

fixed alerts:

1 for `__init__` method calls overridden method

codecov[bot] commented 2 years ago

Codecov Report

Merging #9950 (02e4f54) into master (5ca5e0c) will decrease coverage by 1.61%. The diff coverage is n/a.

:exclamation: Current head 02e4f54 differs from pull request most recent head 7caea98. Consider uploading reports for the commit 7caea98 to get more accurate results

@@            Coverage Diff             @@
##           master    #9950      +/-   ##
==========================================
- Coverage   68.89%   67.28%   -1.62%     
==========================================
  Files         406      203     -203     
  Lines       88236    43957   -44279     
  Branches    19593     9692    -9901     
==========================================
- Hits        60788    29575   -31213     
+ Misses      22851    12192   -10659     
+ Partials     4597     2190    -2407

Impacted Files	Coverage Δ
modules/cuda.py	`0.00% <0.00%> (-72.64%)`	:arrow_down:
templates/cudatemplates.py	`37.50% <0.00%> (-62.50%)`	:arrow_down:
compilers/cuda.py	`19.63% <0.00%> (-45.40%)`	:arrow_down:
dependencies/cuda.py	`20.19% <0.00%> (-42.79%)`	:arrow_down:
compilers/mixins/clang.py	`56.09% <0.00%> (-14.64%)`	:arrow_down:
scripts/coverage.py	`56.43% <0.00%> (-7.93%)`	:arrow_down:
compilers/detect.py	`41.95% <0.00%> (-5.02%)`	:arrow_down:
interpreter/type_checking.py	`65.23% <0.00%> (-4.17%)`	:arrow_down:
linkers/linkers.py	`56.53% <0.00%> (-1.21%)`	:arrow_down:
compilers/mixins/gnu.py	`80.46% <0.00%> (-1.11%)`	:arrow_down:
... and 221 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

lgtm-com[bot] commented 2 years ago

This pull request introduces 4 alerts and fixes 1 when merging 5ee28be1503c0b7116fae1bc72f383677527cd15 into 9b9154017e9edc7b5b3bd93dce74ffa9c8b22a03 - view on LGTM.com

new alerts:

2 for Unused import
1 for First parameter of a method is not named 'self'
1 for Unused local variable

fixed alerts:

1 for `__init__` method calls overridden method

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert and fixes 1 when merging ab1a7d5681d8f83068231804c899146289b16ef4 into 9b9154017e9edc7b5b3bd93dce74ffa9c8b22a03 - view on LGTM.com

new alerts:

1 for Unused import

fixed alerts:

1 for `__init__` method calls overridden method

lgtm-com[bot] commented 2 years ago

This pull request introduces 2 alerts and fixes 1 when merging 406e756c63f207f7b7c2ce64d1726032d5c4510a into 9b9154017e9edc7b5b3bd93dce74ffa9c8b22a03 - view on LGTM.com

new alerts:

2 for Unused import

fixed alerts:

1 for `__init__` method calls overridden method

lgtm-com[bot] commented 2 years ago

This pull request introduces 3 alerts and fixes 1 when merging 9976b3436ca3fadd535ab2b86fb050e53ac3094c into 3e2dba5b7cd107f60474b3cb01c307a9e2354868 - view on LGTM.com

new alerts:

2 for Unused import
1 for `__init__` method calls overridden method

fixed alerts:

1 for `__init__` method calls overridden method

lgtm-com[bot] commented 2 years ago

This pull request introduces 2 alerts and fixes 1 when merging 011b6108500a73928faa31a2bcd2dd182db6165f into 9061c3a52d355e478cbf83a6110b4cc3d2330ba6 - view on LGTM.com

new alerts:

1 for Unused import
1 for `__init__` method calls overridden method

fixed alerts:

1 for `__init__` method calls overridden method

lgtm-com[bot] commented 2 years ago

This pull request introduces 2 alerts and fixes 1 when merging 5f407b815bad58ab1bb18ba31dd9dcf5c2fe0cb7 into 9061c3a52d355e478cbf83a6110b4cc3d2330ba6 - view on LGTM.com

new alerts:

1 for Unused import
1 for `__init__` method calls overridden method

fixed alerts:

1 for `__init__` method calls overridden method

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert and fixes 1 when merging 4718dcf0be226888b522a1f7b9a93c226fce0f47 into c3ef52311af84d100be220ae8a9ed58e50100808 - view on LGTM.com

new alerts:

1 for `__init__` method calls overridden method

fixed alerts:

1 for `__init__` method calls overridden method

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert and fixes 1 when merging bba873e3a3cb4bd08ffa9620608db545282fd00b into b69d4f38e6f4488da42402a19b921a9012692ffe - view on LGTM.com

new alerts:

1 for `__init__` method calls overridden method

fixed alerts:

1 for `__init__` method calls overridden method

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert and fixes 1 when merging ffa10e872df1a83cc9284dfcd431f4ed71dcc7d3 into b69d4f38e6f4488da42402a19b921a9012692ffe - view on LGTM.com

new alerts:

1 for `__init__` method calls overridden method

fixed alerts:

1 for `__init__` method calls overridden method

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert and fixes 1 when merging d56ceb206c675991061008d4df1985768073c902 into b69d4f38e6f4488da42402a19b921a9012692ffe - view on LGTM.com

new alerts:

1 for `__init__` method calls overridden method

fixed alerts:

1 for `__init__` method calls overridden method

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert and fixes 1 when merging 9cc64c226901f43e73c5f2055af34a9a6285a830 into b69d4f38e6f4488da42402a19b921a9012692ffe - view on LGTM.com

new alerts:

1 for `__init__` method calls overridden method

fixed alerts:

1 for `__init__` method calls overridden method

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert and fixes 1 when merging 6db7911874be08a6c129675265bec678827f8af0 into b69d4f38e6f4488da42402a19b921a9012692ffe - view on LGTM.com

new alerts:

1 for `__init__` method calls overridden method

fixed alerts:

1 for `__init__` method calls overridden method

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert and fixes 1 when merging bb919ac9ed48c041f4b108700df0aa2aa0411304 into 4ad869cd849313f88497b1124a23c8188561a554 - view on LGTM.com

new alerts:

1 for `__init__` method calls overridden method

fixed alerts:

1 for `__init__` method calls overridden method

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert and fixes 1 when merging ecbcc23abd39d82b86b709668183f5cc513ddd8f into 4ad869cd849313f88497b1124a23c8188561a554 - view on LGTM.com

new alerts:

1 for `__init__` method calls overridden method

fixed alerts:

1 for `__init__` method calls overridden method

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert and fixes 1 when merging 88fa19d5f1b89883639c34874fa9cffcc5b3623f into 4ad869cd849313f88497b1124a23c8188561a554 - view on LGTM.com

new alerts:

1 for `__init__` method calls overridden method

fixed alerts:

1 for `__init__` method calls overridden method

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert and fixes 1 when merging 3a001e494dfbf5755f56f42431107c5db264e95c into f380862284c9187f8c4496068f11bcc3ca245fec - view on LGTM.com

new alerts:

1 for `__init__` method calls overridden method

fixed alerts:

1 for `__init__` method calls overridden method

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert and fixes 1 when merging 26969dc4d7a35e6acd4ccf93f6fa7e94bd1dcaca into f380862284c9187f8c4496068f11bcc3ca245fec - view on LGTM.com

new alerts:

1 for `__init__` method calls overridden method

fixed alerts:

1 for `__init__` method calls overridden method

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert and fixes 1 when merging d51fab1d18e0d4bdb2b6458fc37308e0ae9de432 into f380862284c9187f8c4496068f11bcc3ca245fec - view on LGTM.com

new alerts:

1 for `__init__` method calls overridden method

fixed alerts:

1 for `__init__` method calls overridden method

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert and fixes 1 when merging f3e2616f7bf64b26743556342e3f8994f135aed1 into f380862284c9187f8c4496068f11bcc3ca245fec - view on LGTM.com

new alerts:

1 for `__init__` method calls overridden method

fixed alerts:

1 for `__init__` method calls overridden method

lgtm-com[bot] commented 2 years ago

This pull request introduces 2 alerts and fixes 1 when merging 683d21143b2e06c899426d93b9cae58600a4675c into 16ed82a62f535475b0eff6acc2d7c34c5265c16f - view on LGTM.com

new alerts:

1 for Unused import
1 for `__init__` method calls overridden method

fixed alerts:

1 for `__init__` method calls overridden method

lgtm-com[bot] commented 2 years ago

This pull request introduces 2 alerts and fixes 1 when merging 86060c8fc9faeb6735936fb7b213c4d6bc6f2140 into 16ed82a62f535475b0eff6acc2d7c34c5265c16f - view on LGTM.com

new alerts:

1 for Unused import
1 for `__init__` method calls overridden method

fixed alerts:

1 for `__init__` method calls overridden method

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert and fixes 1 when merging 08a7c06fe38c894145905adaf8e07ee22f09552c into f3ba24f2892fa4ccf1c6c198190f43d4da44a761 - view on LGTM.com

new alerts:

1 for `__init__` method calls overridden method

fixed alerts:

1 for `__init__` method calls overridden method

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert and fixes 1 when merging 2a911d22554246d7d3433950a38f0a79c63d5138 into d85a9733e6af45c9e3111f23ecec9e614eaeaeeb - view on LGTM.com

new alerts:

1 for `__init__` method calls overridden method

fixed alerts:

1 for `__init__` method calls overridden method

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert and fixes 1 when merging b4e9fbd8b994cdefe61cc9aa03da603d1577b0a9 into d85a9733e6af45c9e3111f23ecec9e614eaeaeeb - view on LGTM.com

new alerts:

1 for `__init__` method calls overridden method

fixed alerts:

1 for `__init__` method calls overridden method

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert and fixes 1 when merging 968e890a0018f8057ef03d7c524ec5376c240f56 into d85a9733e6af45c9e3111f23ecec9e614eaeaeeb - view on LGTM.com

new alerts:

1 for `__init__` method calls overridden method

fixed alerts:

1 for `__init__` method calls overridden method

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert and fixes 1 when merging 8bfc2d8dc188b6c3a4475387ece0cb9b89fb73be into d85a9733e6af45c9e3111f23ecec9e614eaeaeeb - view on LGTM.com

new alerts:

1 for `__init__` method calls overridden method

fixed alerts:

1 for `__init__` method calls overridden method

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert and fixes 1 when merging 553871efb281945800fa5aa609c3d3cb31ec94c9 into ee7a7fec10298e8dabc423f6163e866d19473e7f - view on LGTM.com

new alerts:

1 for `__init__` method calls overridden method

fixed alerts:

1 for `__init__` method calls overridden method

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert and fixes 1 when merging 8e56e5a566cbc1e89733ec016e9bb42def7413e0 into ee7a7fec10298e8dabc423f6163e866d19473e7f - view on LGTM.com

new alerts:

1 for `__init__` method calls overridden method

fixed alerts:

1 for `__init__` method calls overridden method

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert and fixes 1 when merging ffc42d2c6dd51748c91cb49ce4ae91f2ddf9f0aa into ee7a7fec10298e8dabc423f6163e866d19473e7f - view on LGTM.com

new alerts:

1 for `__init__` method calls overridden method

fixed alerts:

1 for `__init__` method calls overridden method

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert and fixes 1 when merging 879f8b6ec693e1ee212c28ce29405e344c91fa70 into 9b2142800658e2a63def39f2cfb1ec4cb18795bd - view on LGTM.com

new alerts:

1 for `__init__` method calls overridden method

fixed alerts:

1 for `__init__` method calls overridden method

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert and fixes 1 when merging 898f5324bcf37c06eb2aeb23db6934ebe7b7cbae into 9b2142800658e2a63def39f2cfb1ec4cb18795bd - view on LGTM.com

new alerts:

1 for `__init__` method calls overridden method

fixed alerts:

1 for `__init__` method calls overridden method

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert and fixes 1 when merging e082cfb1c744bdc565e5dbcfdd2f57d80e1adf93 into b49b9f52b29896cce58a1e3dbeb1b6cf54420d45 - view on LGTM.com

new alerts:

1 for `__init__` method calls overridden method

fixed alerts:

1 for `__init__` method calls overridden method

lgtm-com[bot] commented 1 year ago

This pull request introduces 2 alerts and fixes 1 when merging 5626a25c0c90e8730ead33f0ecf8d102c3d862ff into f0e9a44d41e2040e14dbd07a288654415c6b595a - view on LGTM.com

new alerts:

1 for Unused import
1 for `__init__` method calls overridden method

fixed alerts:

1 for `__init__` method calls overridden method

lgtm-com[bot] commented 1 year ago

This pull request introduces 1 alert and fixes 1 when merging 854c7913f9dc948b0c4362be95cbfb7b5b45f5bb into f0e9a44d41e2040e14dbd07a288654415c6b595a - view on LGTM.com

new alerts:

1 for `__init__` method calls overridden method

fixed alerts:

1 for `__init__` method calls overridden method

lgtm-com[bot] commented 1 year ago

This pull request introduces 1 alert and fixes 1 when merging ad8e0c03694205284f7a409ecd10dbea0c9290d4 into f0e9a44d41e2040e14dbd07a288654415c6b595a - view on LGTM.com

new alerts:

1 for `__init__` method calls overridden method

fixed alerts:

1 for `__init__` method calls overridden method

lgtm-com[bot] commented 1 year ago

This pull request introduces 1 alert and fixes 1 when merging 0e0c58630662cbb17e41b14b02a2531a160d1774 into f0e9a44d41e2040e14dbd07a288654415c6b595a - view on LGTM.com

new alerts:

1 for `__init__` method calls overridden method

fixed alerts:

1 for `__init__` method calls overridden method

lgtm-com[bot] commented 1 year ago

This pull request introduces 2 alerts and fixes 1 when merging ab807e4ed96e95aaedd9474adec621fbdb471d0c into f0e9a44d41e2040e14dbd07a288654415c6b595a - view on LGTM.com

new alerts:

1 for Variable defined multiple times
1 for `__init__` method calls overridden method

fixed alerts:

1 for `__init__` method calls overridden method

lgtm-com[bot] commented 1 year ago

This pull request introduces 1 alert and fixes 1 when merging e7e2956c1262a969b569996d8eedba4abfa5d568 into f0e9a44d41e2040e14dbd07a288654415c6b595a - view on LGTM.com

new alerts:

1 for `__init__` method calls overridden method

fixed alerts:

1 for `__init__` method calls overridden method

lgtm-com[bot] commented 1 year ago

This pull request introduces 1 alert and fixes 1 when merging 35ff35688318c8e2de5abf21694a0a556a14bdc3 into b88bec0ee84f607108cd57594815400e7627cbc9 - view on LGTM.com

new alerts:

1 for `__init__` method calls overridden method

fixed alerts:

1 for `__init__` method calls overridden method

lgtm-com[bot] commented 1 year ago

This pull request introduces 1 alert and fixes 1 when merging 4650d173959ca2f0bff0e43305176d20870d0476 into b88bec0ee84f607108cd57594815400e7627cbc9 - view on LGTM.com

new alerts:

1 for `__init__` method calls overridden method

fixed alerts:

1 for `__init__` method calls overridden method

lgtm-com[bot] commented 1 year ago

This pull request introduces 1 alert and fixes 1 when merging 7caea983d896b961128d0036860b6fc36f29a92f into 5ca5e0c900340110eb26bcf4491033e9d1bacf08 - view on LGTM.com

new alerts:

1 for `__init__` method calls overridden method

fixed alerts:

1 for `__init__` method calls overridden method

eli-schwartz commented 1 year ago

A bunch of these commits now landed separately, time to rebase.

Since the current PR is 47 commits, covering 37 files and 4,000 lines of changes -- and the rebase will only make it a little better -- I would really appreciate if some of these changes could be split out into their own series to make review easier. :)

dcbaker commented 1 year ago

Yeah, I’m working on splitting out the first 10 or so patches that cover most of the interpreter changes for a first batch

mesonbuild / meson

Use typed_*_args for build_targets #9950

Codecov Report