support for Snowflake - updated of PR #2274

pls review an updated version of PR #2274
Original version can't be merged because of conflicts

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: olavloite:x: Roman SolodovnichenkoRoman Solodovnichenko seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

I'm going to close this - v6.1 will be out with Snowflake support next week. Thanks to everyone who contributed!


500 Server Error with 3000+ Data

So I have a table full of data. About 3500 records. And I am using serverSide to load the data. But it is giving me DataTables warning: table id=data-table-all - Ajax error. For more information about this error, please see http://datatables.net/tn/7 error. If I reduce the amount of data on my table to 2000, it works as intended, and there is no error. I am not sure whats wrong here. Is it my query? Here is my query:
$array = Reservation::with('restaurantoption')->with('islandhost')->with('mealplan')->with('guest')->with('nationality')->with('restaurant')->with('user')->withCount('guestpaxes')->where('restaurant_id', $auth->restaurant_id)->latest()->get();
return datatables()->of($array)->toJson();

Route:
If I open the route in Chrome (Visit the Link) it shows all the data with no problem
Route::get('/reports/display-report-data/all-data', 'Reservation\ReportController@getAllReservations')->name('get.all.reservations.report');

And here is front page:
$('#data-table-all').DataTable({
                responsive: true,
                order:[[0,"dec"]],
                processing: true,
                serverSide: true,
                ajax: '{{ route('get.all.reservations.report') }}',
                columns: [
                    {data: 'id', name: 'id'},
                    {data: 'guest.name', name: 'guest'},
                    {data: 'room_no', name: 'room_no'},
                    {data: 'table_no', name: 'table_no', defaultContent: "null"},
                    {data: 'guestpaxes_count', name: 'guestpaxes_count'},
                    {data: 'status', name: 'status'},
                    {data: 'nationality.name', name: 'nationality'},
                    {data: 'mealplan.name', name: 'mealplan'},
                    {data: 'islandhost.name', name: 'islandhost', defaultContent: ""},
                    {data: 'restaurant.name', name: 'restaurant'},
                    {data: 'restaurantoption.name', name: 'restaurantoption', defaultContent: ""},
                    {data: 'reservation_date', name: 'reservation_date'},
                    {data: 'reservation_time', name: 'reservation_time'},
                    {data: 'free_meal', name: 'free_meal'},
                    {data: 'served_by', name: 'served_by', defaultContent: ""},
                    {data: 'user.name', name: 'user', defaultContent: "null"},
                    {data: 'cancel_reason', name: 'cancel_reason'},
                    {data: 'canceled_by', name: 'canceled_by'},
                    {data: 'created_at', name: 'created_at'},
                ],
                "createdRow": function (row, data) {

                    $('td', row).eq('5').html($('<span>').attr({
                        class:  (data.status == 1) ? 'text-success' : ( (data.status == 2) ? 'text-danger': 'text-warning' )
                    }).html((data.status == 1) ? 'Confirmed' : ( (data.status == 2) ? 'Denied': 'Pending' )));

                    $('td', row).eq('13').html($('<span>').attr({
                        class:  ((data.free_meal == 1) ? 'text-success': 'text-danger' )
                    }).html((( (data.free_meal == 1) ? 'YES': 'NO' ))));

                }
            });
        });

I know it's a bit messy sorry. If I remove /comment the serverSide, it works normally but very slow (Obviously because it is loading all data on load).
The error on console is very very long but I will paste it here anyways (Please have mercy sensei):
http://test.com/restaurant-reservations/reports/display-report-data/all-data?draw=1&columns%5B0%5D%5Bdata%5D=id&columns%5B0%5D%5Bname%5D=id&columns%5B0%5D%5Bsearchable%5D=true&columns%5B0%5D%5Borderable%5D=true&columns%5B0%5D%5Bsearch%5D%5Bvalue%5D=&columns%5B0%5D%5Bsearch%5D%5Bregex%5D=false&columns%5B1%5D%5Bdata%5D=guest.name&columns%5B1%5D%5Bname%5D=guest&columns%5B1%5D%5Bsearchable%5D=true&columns%5B1%5D%5Borderable%5D=true&columns%5B1%5D%5Bsearch%5D%5Bvalue%5D=&columns%5B1%5D%5Bsearch%5D%5Bregex%5D=false&columns%5B2%5D%5Bdata%5D=room_no&columns%5B2%5D%5Bname%5D=room_no&columns%5B2%5D%5Bsearchable%5D=true&columns%5B2%5D%5Borderable%5D=true&columns%5B2%5D%5Bsearch%5D%5Bvalue%5D=&columns%5B2%5D%5Bsearch%5D%5Bregex%5D=false&columns%5B3%5D%5Bdata%5D=table_no&columns%5B3%5D%5Bname%5D=table_no&columns%5B3%5D%5Bsearchable%5D=true&columns%5B3%5D%5Borderable%5D=true&columns%5B3%5D%5Bsearch%5D%5Bvalue%5D=&columns%5B3%5D%5Bsearch%5D%5Bregex%5D=false&columns%5B4%5D%5Bdata%5D=guestpaxes_count&columns%5B4%5D%5Bname%5D=guestpaxes_count&columns%5B4%5D%5Bsearchable%5D=true&columns%5B4%5D%5Borderable%5D=true&columns%5B4%5D%5Bsearch%5D%5Bvalue%5D=&columns%5B4%5D%5Bsearch%5D%5Bregex%5D=false&columns%5B5%5D%5Bdata%5D=status&columns%5B5%5D%5Bname%5D=status&columns%5B5%5D%5Bsearchable%5D=true&columns%5B5%5D%5Borderable%5D=true&columns%5B5%5D%5Bsearch%5D%5Bvalue%5D=&columns%5B5%5D%5Bsearch%5D%5Bregex%5D=false&columns%5B6%5D%5Bdata%5D=nationality.name&columns%5B6%5D%5Bname%5D=nationality&columns%5B6%5D%5Bsearchable%5D=true&columns%5B6%5D%5Borderable%5D=true&columns%5B6%5D%5Bsearch%5D%5Bvalue%5D=&columns%5B6%5D%5Bsearch%5D%5Bregex%5D=false&columns%5B7%5D%5Bdata%5D=mealplan.name&columns%5B7%5D%5Bname%5D=mealplan&columns%5B7%5D%5Bsearchable%5D=true&columns%5B7%5D%5Borderable%5D=true&columns%5B7%5D%5Bsearch%5D%5Bvalue%5D=&columns%5B7%5D%5Bsearch%5D%5Bregex%5D=false&columns%5B8%5D%5Bdata%5D=islandhost.name&columns%5B8%5D%5Bname%5D=islandhost&columns%5B8%5D%5Bsearchable%5D=true&columns%5B8%5D%5Borderable%5D=true&columns%5B8%5D%5Bsearch%5D%5Bvalue%5D=&columns%5B8%5D%5Bsearch%5D%5Bregex%5D=false&columns%5B9%5D%5Bdata%5D=restaurant.name&columns%5B9%5D%5Bname%5D=restaurant&columns%5B9%5D%5Bsearchable%5D=true&columns%5B9%5D%5Borderable%5D=true&columns%5B9%5D%5Bsearch%5D%5Bvalue%5D=&columns%5B9%5D%5Bsearch%5D%5Bregex%5D=false&columns%5B10%5D%5Bdata%5D=restaurantoption.name&columns%5B10%5D%5Bname%5D=restaurantoption&columns%5B10%5D%5Bsearchable%5D=true&columns%5B10%5D%5Borderable%5D=true&columns%5B10%5D%5Bsearch%5D%5Bvalue%5D=&columns%5B10%5D%5Bsearch%5D%5Bregex%5D=false&columns%5B11%5D%5Bdata%5D=reservation_date&columns%5B11%5D%5Bname%5D=reservation_date&columns%5B11%5D%5Bsearchable%5D=true&columns%5B11%5D%5Borderable%5D=true&columns%5B11%5D%5Bsearch%5D%5Bvalue%5D=&columns%5B11%5D%5Bsearch%5D%5Bregex%5D=false&columns%5B12%5D%5Bdata%5D=reservation_time&columns%5B12%5D%5Bname%5D=reservation_time&columns%5B12%5D%5Bsearchable%5D=true&columns%5B12%5D%5Borderable%5D=true&columns%5B12%5D%5Bsearch%5D%5Bvalue%5D=&columns%5B12%5D%5Bsearch%5D%5Bregex%5D=false&columns%5B13%5D%5Bdata%5D=free_meal&columns%5B13%5D%5Bname%5D=free_meal&columns%5B13%5D%5Bsearchable%5D=true&columns%5B13%5D%5Borderable%5D=true&columns%5B13%5D%5Bsearch%5D%5Bvalue%5D=&columns%5B13%5D%5Bsearch%5D%5Bregex%5D=false&columns%5B14%5D%5Bdata%5D=served_by&columns%5B14%5D%5Bname%5D=served_by&columns%5B14%5D%5Bsearchable%5D=true&columns%5B14%5D%5Borderable%5D=true&columns%5B14%5D%5Bsearch%5D%5Bvalue%5D=&columns%5B14%5D%5Bsearch%5D%5Bregex%5D=false&columns%5B15%5D%5Bdata%5D=user.name&columns%5B15%5D%5Bname%5D=user&columns%5B15%5D%5Bsearchable%5D=true&columns%5B15%5D%5Borderable%5D=true&columns%5B15%5D%5Bsearch%5D%5Bvalue%5D=&columns%5B15%5D%5Bsearch%5D%5Bregex%5D=false&columns%5B16%5D%5Bdata%5D=cancel_reason&columns%5B16%5D%5Bname%5D=cancel_reason&columns%5B16%5D%5Bsearchable%5D=true&columns%5B16%5D%5Borderable%5D=true&columns%5B16%5D%5Bsearch%5D%5Bvalue%5D=&columns%5B16%5D%5Bsearch%5D%5Bregex%5D=false&columns%5B17%5D%5Bdata%5D=canceled_by&columns%5B17%5D%5Bname%5D=canceled_by&columns%5B17%5D%5Bsearchable%5D=true&columns%5B17%5D%5Borderable%5D=true&columns%5B17%5D%5Bsearch%5D%5Bvalue%5D=&columns%5B17%5D%5Bsearch%5D%5Bregex%5D=false&columns%5B18%5D%5Bdata%5D=created_at&columns%5B18%5D%5Bname%5D=created_at&columns%5B18%5D%5Bsearchable%5D=true&columns%5B18%5D%5Borderable%5D=true&columns%5B18%5D%5Bsearch%5D%5Bvalue%5D=&columns%5B18%5D%5Bsearch%5D%5Bregex%5D=false&order%5B0%5D%5Bcolumn%5D=0&order%5B0%5D%5Bdir%5D=dec&start=0&length=10&search%5Bvalue%5D=&search%5Bregex%5D=false&_=1578940884709

What could be the problem here? I have searched all over Git issues and Stackoverflow and I could not find anything relating to my issue.

Don't use collection for large datasets. Remove get().
$array = Reservation::with('restaurantoption')
->with('islandhost')
->with('mealplan')->with('guest')->with('nationality')
->with('restaurant')->with('user')
->withCount('guestpaxes')
->where('restaurant_id', $auth->restaurant_id)
->latest();

return datatables()->of($array)->toJson();

Hello
Thank you so much. This solves my issue. I had one more issue I noticed after removing get. On my local server, I get the data instantly. It doesn't even take a second. But on my Digital Ocean server, it takes about 6 seconds to load. When I SSH and Htop the server, it shows that when requested, mySQL takes 100% usage from CPU to complete one single request. Is it because my query is too big?

Hello
Thank you so much. This solves my issue. I had one more issue I noticed after removing get. On my local server, I get the data instantly. It doesn't even take a second. But on my Digital Ocean server, it takes about 6 seconds to load. When I SSH and Htop the server, it shows that when requested, mySQL takes 100% usage from CPU to complete one single request. Is it because my query is too big?

that could be the case, try to toSql() and run the resulted query first. and see does it also happen if you do normal query? If yes then you should find another way to query the data. Also if you use agregate, try to use indexes(on mysql, I don't know on other database engine).


feat: Starknet RPC " get_transaction_by_hash" call

Description
Implement "get_transaction_by_hash" Starknet RPC call
Starknet-RS Implementation
https://github.com/xJonathanLEI/starknet-rs/blob/da72d0130eb48cc84288c9948d983f5f78cf6054/starknet-providers/src/jsonrpc/mod.rs#L187
Acceptance Criteria

Beerus can query a Starknet's block number
Associated cli command
Associated rest api wrapper
Helios integration
Units tests
Integration tests

I'll try to do it!

I'll try to do it!

Thanks, assigned it to you


netrw windows are not ignored, even if added to the config

:set filetype? in a netrw window returns "netrw".
this config, however, does not seem to take any affect, windows.nvim still auto-resizes the window
require("windows").setup {
	autowidth = {
                enable = true
        }
	ignore = {
                filetype = {"netrw"}
        }
}

Commenting to say that i've been having the same issues, not sure whats causing it. Did you ever find a solution?

Coming back to comment that this seems to be only an issue if it is the only vertical split that you have opened. Say you already have 2 vertical splits open, then proceed to open netrw in another vertical split. It is resized. Otherwise if you only have one buffer open and open netrw in a vertical split, it will resize itself.


[SMALLFIX]Use static imports for standard test utilities for class DefaultAsyncPersistHandlerTest

[SMALLFIX]Use static imports for standard test utilities for class DefaultAsyncPersistHandlerTest

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/17819/
Test PASSed.

Merged build finished. Test PASSed.


Update gtk to v0.15

Gtk has new release. We should update to it.
The crates that we own should also update are:

[x] libappindicator-rs
[x] libayatana-appindicator

Looks like there's a regression and currently be blocked by this issue
https://github.com/gtk-rs/gtk3-rs/issues/718


framebuffer error

Hi, working with distributed RL algorithms I found quite confusing bug in dm suite:
when it comes to spawning a bunch of environments (like 32/64) dm_suite just goes to
core.py Offscreen framebuffer is not complete, error *****

This bug occurs every time when number of envs > number of cpus threads.
Is there any way to solve it?

UPD: Solved with MUJOCO_GL="osmesa".

I am having the same issue. What was the exact change that you made ?

Use instructions from https://github.com/deepmind/dm_control#additional-instructions-for-linux
export MUJOCO_GL="osmesa" before script
now, you can run the script


fix: await bundle finish before open browser url to prevent a 404 page

This PR's fix the 404 page in the first view of the application.
Note: tests are already breaking before this PR.

@danielbarion Very nice! Good job! I'm having this problem!

Hi @danielbarion,
Thanks for you pull request.
Can you please provide some instructions on how to reproduce the issue? Maybe a sample project that I can download to reproduce it myself?
By the way, tests are running fine both on GitHub Actions and locally for me using<EMAIL_ADDRESS>Which version are you using?
If I run the tests with your changes they fail. I believe that is because you have removed the line that actually tells browsersync to reload every time there is a change:
bs.reload(bundle.dest);

I would appreciate if you could update the pull request so that the tests pass with your changes.

I've published a new version using writeBundle. I don't think it will resolve your issue but it is indeed more correct (see https://github.com/4lejandrito/rollup-plugin-browsersync/commit/b20db01009b61943e9a018db49651419c9007ccf).

Hi again @danielbarion,
I have published a prerelease (1.3.0-alpha.1) with a change that could fix your issue. Can you please confirm?
npm i<EMAIL_ADDRESS>

Hey @4lejandrito!
I'll take a look into this alpha version, thank you!
I'm using Node v13.13.0.
I'll provide a test repository in few minutes

The example with the "error" (the plugin opens the browser window before the bundle finish)
https://github.com/danielbarion/browsersync-example/
Yeah, the alpha version works as expected! Thank you!
Here is the Alpha version branch:
https://github.com/danielbarion/browsersync-example/pull/1

Thanks a lot! It failed with the current version and works with the alpha. I just published 1.3.0.
I am closing this pull request.
Thanks for reporting it.

Fixed by https://github.com/4lejandrito/rollup-plugin-browsersync/commit/1f2265b610532e848d414c124c4c4f1f13fdc4ff.


I am unable to set the card to its original position( Now it's Discarded Automatically) Once I move card.

hi, do you mean that when you move a little you cant set it back and it will always swipe?

if this is the problem you need to add logic to swipeEnd() callback.
@Override
public boolean swipeEnd(int section, float distance) {
//Toast.makeText(this,"swipe end",Toast.LENGTH_SHORT).show();
Log.i("LogTag","SwipeEnd: "+distance);
if(distance > 450)
return true;
else
return false;
}
//450 is the minimum distance that user have to move the card for swipr, you can play with that parameter as you want.

Yes.Exactly it does.
On Tue, Aug 1, 2017 at 11:44 AM, maksim<EMAIL_ADDRESS>wrote:

hi, do you mean that when you move a little you cant set it back and it
will always swipe?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/wenchaojiang/AndroidSwipeableCardStack/issues/86#issuecomment-319278247,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AbBf3fyU9kdkGlEjWkSrlDvGWabreGUBks5sTsI1gaJpZM4N4SFI
.

Let me try. I will let u know.Thanks for reply.
On Tue, Aug 1, 2017 at 11:51 AM, maksim<EMAIL_ADDRESS>wrote:

if this is the problem you need to add logic to swipeEnd() callback.
@override https://github.com/override
public boolean swipeEnd(int section, float distance) {
//Toast.makeText(this,"swipe end",Toast.LENGTH_SHORT).show();
Log.i("LogTag","SwipeEnd: "+distance);
if(distance > 450)
return true;
else
return false;
}
//450 is the minimum distance that user have to move the card for swipr,
you can play with that parameter as you want.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/wenchaojiang/AndroidSwipeableCardStack/issues/86#issuecomment-319279247,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AbBf3Z6EPhgBnwx_x4abUpEbwUYQpzr9ks5sTsPbgaJpZM4N4SFI
.

another way to control minimum travel for discard is :
private CardStack cardStackView = new CardStack();
int minimumTravelToDiscard = 400;
cardStackView.setThreshold(minimumTravelToDiscard );


[BUG] same photo's createDateTime property on iOS and android will get different date.

Describe the bug
Copied photo or photo download from cloud.
createDateTime property can not get EXIF photo capture time on android but yes on iOS.
复制后的照片，建立时间属性在iOS跟android 上会取得不一样的照片建立日期，ios 取得照相时间，android取得档案复制建立时间
Expected behavior
createDateTime property can get photo EXIF capture date on iOS and android or provide a captureDateTime to get EXIF capture date.
I know we can get EXIF by originFile but this function need 1x secs on 300 photos.
建立时间属性在iOS上可以顺利取得照相时间跟EXIF中的资讯相同，希望在android上也是可以有相同的逻辑。
或是提供一个 captureDateTime 可以取得照相时间。
我知道使用 originFile 可以取得 EXIF 资讯，但是使用场景是会一次取得几百张照片，再把日期合成在照片上。以300张照片来说，使用 originFile 会耗时十几秒。
Smartphone (please complete the following information):

Device: samsung note10+, iphone13
OS: android 11, iOS 15

没有明白你想说的问题…

Hello!
I guess I am experiencing the same issue..
Describe the bug
The createDateTime information is not correctly picked from the EXIF info in Android.
To reproduce

Take a photo that was taken some time ago (for instance I have a photo taken on Dec 26th 2021 at 21:03)
Download it on your phone (I downloaded it on my phone on Jan 11th 2022 at 14:16)
Load it as AssetEntity using the plugin
The createdDateTime given by this plugin is:

Jan 11th 2022 - on Android ❌
Dec 26th 2021 - on iOS ✅

I double-checked the EXIF info of the Photo and ModifyDate, CreateDate and DateTimeOriginal are all equal to 2021:12:26 21:03:19 (offset is +01:00).
So, finally, the createDateTime on Android is not correctly taken from the EXIF.
For photos taken directly with my smartphone the problem does not occur, that's why @KenLuo writes

Copied photo or photo download from cloud.

Thank you

Thanks for the detailed info. Will take further investigations to verify.

@gianmarcocalbi Can you verify if the linked PR fix this issue?

@gianmarcocalbi Can you verify if the linked PR fix this issue?

@AlexV525, the commit in the PR #682 solves the issue 🚀🚀
I tested with the following dependency
photo_manager:
    git:
      url: https://github.com/fluttercandies/flutter_photo_manager
      ref: 7fdc2101265324e72128f109f132655966485dae

Thank you!

感謝🙏
Alex Li @.***>於 2022年1月18日 週二，下午5:55寫道：

Closed #650
https://github.com/fluttercandies/flutter_photo_manager/issues/650 via
#682 https://github.com/fluttercandies/flutter_photo_manager/pull/682.
—
Reply to this email directly, view it on GitHub
https://github.com/fluttercandies/flutter_photo_manager/issues/650#event-5909289642,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAQOMUD4XGY446C2Y5EZAJ3UWU2KHANCNFSM5JF5AGEA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
<fluttercandies/flutter_photo_manager/issue/650/issue_event/5909289642@
github.com>

--
Regards,
羅志堅 Ken Luo

@gianmarcocalbi  How do you use to get the right  creatTime? I tried as follows, but still could not get the right time.

@gianmarcocalbi How do you use to get the right creatTime? I tried as follows, but still could not get the right time.

I was reading assetEntity.createDateTime..
anyway

make sure you're using a version of the plugin including the fix because it has not been released to a stable version yet
double check that your photo does actually have the EXIF information you expect, it should be something like CreateDate (there are many free tools to do such or you may use the dart exif package).


gzip sizes

This is a nice plugin, but it should be possible to take the actual files content (not just the size) and run an on-the-fly gzip (e.g using zlib) to calculate the correct gzipped size  (at least a better approximation) ? Does the code use the file content or just the sizes ?
zlib.gzip(code, (err, zipped) => {           // zipped.length         })

So there are some challenges here. First off - I'm trying to keep this as consistent as I can for both the plugin and the web version. In the web version - all we have to go on is the stats.json, which includes:

the post-loader and pre-plugin source for each module (e.g. after babel, but before minification), and that size in bytes
the actual (post-plugin) size of the output bundles

It does not include the actual source of the output bundles - which means that we don't have anything to actually run gzip on. And even if we did - the best we could do is gzip the entire bundle and then use the post-loader size percentages to estimate a gzip size for each module. This is what I'm doing for the minified sizes already.
It might be possible to work around some of these issues in the plugin version alone - since it can get access to the actual bundle source. Some other tools like webpack-bundle-analyzer do some clever things like parsing the bundle source and figuring out where each module is located in it - which yields a very accurate minified size. However running gzip on those modules one-at-a-time yields an overestimate since the bundle as a whole gzips much more efficiently. I'm thinking though - that multiplying the percentage by the total gzipped size would probably be close enough.
So that might be an option for the plugin version...

@chrisbateman wow. great to see such a response. I wasn't aware that there is a client side version of this (I forgot about the hosted version with the upload ability) which obviously does not see the source anymore. But the work you did do, and so quickly, is spot on. An estimate on gzip per file is pretty good.
Of course the entire pack might more efficiently pack, but the reason I asked for source level compression is to detect the impact of included code that is harder to compress (eg embedded base 64 encoded jpegs or similar which usually dont compress as well as the rest).  Of course these cases are rare, I just happened to have something like this.

sorry did not want to close it, in case you might want to track this

Hiya! Do you think we could combine our efforts for this idea and webpack-bundle-analyzer somehow? I feel like a unified solution to get the stats out from webpack builds would benefit us both, and the reporters could diverge freely if we get to separate the plugin code and the reporter from each other.
I'm looking at solving https://github.com/th0r/webpack-bundle-analyzer/issues/32 right now, and would want to collaborate with other npm packages trying to solve the same issue

@valscion The main complication here is that this is both a plugin and a website - and the website only gets the stats.json data.
What if we had a module that was soley responsible for stats extraction - and it could accept either stats.json alone - or that plus the other info you can get from from a plugin?

It seems that webpack-bundle-analyzer needs access to the filesystem to get other sizes besides the parsed one. The package has a CLI with it that can be fed a stats.json file but unless also given a bundle directory location, it can't tell gzip and minified sizes, too.
I'm not sure how everything works under the hood but I'll see if we could do with just the stats.json if it would contain the sources, too


I/O Error: EOF Occurred on start button in AS

The phone was connected with USB and after I pressed the AndroidWifiADB button I disconnected the phone. On start button I got I/O Error: EOF.

@mladenbabic can you try with the new version I've released today? Version 2.0

I'm having the same issue after downloading the latest version today (v2.2).

Guys can you send me the steps to reproduce and the devices you are using?


Added support for happiness-format

The standard-linter supports the happiness format so I thought I add it to the formatter as well.

Thanks for this, looks good.  Most people using this package are likely using the linter as well, so I think it  makes sense to keep them somewhat in sync.


Add read only prop

Disabled prop removes the component, that's not the behaviour i need, a read-only prop should show the three and it's value but don't react to any clicks on it's check-boxes

I've just come across the same problem too.  It would be good to have a read-only prop to show the select with the value set but completely inactive.

Is this repo still maintained?

The workaround would be to disable each node in the options array, thus you can open the tree but all items are disabled


Fix GCS bucket name validation

Motivation and context

It is not possible to attach a GCS cloud storage bucket which name contains a dot. As stated here, GCS bucket names allow dots. This is a major problem for us, and this PR fixes this issue.
How has this been tested?

No tests required.
Checklist

[x] I submit my changes into the develop branch
[x] I have created a changelog fragment
[x] I have updated the documentation accordingly
[x] I have added tests to cover my changes
[x] I have linked related issues (see GitHub docs)
[x] I have increased versions of npm packages if it is necessary
(cvat-canvas,
cvat-core,
cvat-data and
cvat-ui)

License

[x] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.

@jootten, Hello, thank you for the contribution!
Looks like it also affects attaching AWS S3 buckets. Could you please add  . directly to COMMON_ALLOWED_RESOURCE_NAME_SYMBOLS?

@Marishka17
Could you please take a look?


Replace IAllocator with reference counted struct

I'll approve this to keep PR size under control. Subsequent work should add attributes to allocator interface appropriately.

There are still three dub projects failing due the breaking changes you introduce, so we have to wait a bit anyhow. Also I feel like this PR is merged a bit too quickly. It still contains debug work and as such a big breaking API change, at least having a motivation in the changelog for why the IAllocator interface needs to be bloated with incRef/decRef would be nice.

There are still three dub projects failing due the breaking changes you introduce, so we have to wait a bit anyhow.

Happy merging & tagging a new release:

[ ] https://github.com/dlang/ci/pull/93
[ ] https://github.com/dlang/dub/pull/1307
[ ] https://github.com/rejectedsoftware/diet-ng/pull/44

CC @s-ludwig

There are still three dub projects failing due the breaking changes you introduce, so we have to wait a bit anyhow. Also I feel like this PR is merged a bit too quickly. It still contains debug work and as such a big breaking API change, at least having a motivation in the changelog for why the IAllocator interface needs to be bloated with incRef/decRef would be nice.

@edi33416 could you please insert a changelog before going on vacation?

Dont forget to close https://issues.dlang.org/show_bug.cgi?id=15509

Looks like processAllocator setter doesn't implement proper multithreading - it decrements reference before storing allocator. Also using setter before getter will still trigger initialization in getter.

cc @edi33416

Ack. Will follow up with a PR to fix this.
@d-random-contributor thanks for pointing those out


[AA4PP Docs] Profile settings documentation

Thanks for this documentation.  It would be nice to see some documentation that describes what data to put into the system to make the recommended deployment user settings and deployment profiles that match watch is shown in the walkthrough videos.
You can reverse engineer some of the settings from the screen shot but it is not obvious or easy to see unless on a large monitor.  The screenshot does not show the recommended button text labels, and it would be very helpful to document their recommended values as well.
Adding a table to the document that lists each of the different recommended profiles to create, along with all the recommended settings for each of them would make it very clear to people setting this up how to proceed.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 5190b0eb-7cda-9642-07aa-519f28a08c9d
Version Independent ID: 5190b0eb-7cda-9642-07aa-519f28a08c9d
Content: Configuring the ALM Accelerator for Power Platform App - Power Platform
Content Source: power-platform/guidance/coe/setup-almacceleratorpowerplatform-deployment-profiles.md
Service: power-platform
Sub-service: guidance
GitHub Login: @mikefactorial
Microsoft Alias: v-mikeochs

@TBag thanks for this adding a link to the April release docs issue in coe-starter-kit to track this and resolve in the next release.


Connect to dotnet-profiler to collect testwise coverage

@rottkit Thank you for your first round of review! I addressed your comments in the code and replied to the comments in this merge request.
Regarding testing on SAP I agree that we need to accomplish this before publishing a new version. If I find some time tomorrow, I will have a look.

Thanks for your rework @r2h2
I think you missed a bunch of the comments. Please have a look again :-)

Thanks @rottkit for your helpful feedback and sorry for missing several comments in the first round!
I think I have addressed all your comments.
I tested the current version at ILP (fine) and MR. Unfortunately, the SCOV main switch is currently disabled at F57 and I reached out to their pool to let it turn on again. At least, the test start and stop events were correctly detected and I also got feedback from Teamscale that (a) the main switch is not enabled and (b) there was no coverage recording session for my user which could have been stopped (which is expected in case of disabled SCOV). I will re-test when the SAP team has re-enabled the SCOV and keep you posted.

@rottkit Thanks again for having a look at my changes! Now this is ready for review round 2 :)
Update from MR:
I tested this implementation on a SAP testing environment and all events (start, stop, update) and requests seem to work as expected, i.e., coverage recording start + stop, error on multiple active test sessions in ADO, report upload to Teamscale.
The test scenario was a bit limited as I have no permission to execute any monitored code on the system, so the coverage report was empty. Still, I think we can expect the event and data processing pipeline to work. WDYT?


iOS: App freezes as sync is finishing

Twice today I’ve launched NetNewsWire and it froze at about 80% finished with a sync to Feedbin. The app will eventually crash/be killed and then on subsequent launches, the app never finishes initialising. The Feeds screen appears, but no rows are filled and the app is unresponsive. It eventually crashes or gets killed. This behaviour persists until the device (iPhone X) is restarted.
There’s one curious thing I notice when the initial freeze occurs. The ‘Today’ and ‘Unread’ Smart Feed unread counts don’t match. They should, though, because I haven’t got any old unreads.
(I should note that I haven’t experienced it any other time this week, but that it started happening today, after updating to the latest iOS 13.2 beta last night.)
subsequent-NetNewsWire-2019-10-26-143709.ips.beta.txt
initialcrash-NetNewsWire-2019-10-26-143529.ips.beta.txt

I’m pretty sure that this is the UserDefaults hanging bug.

I wondered after I saw your #work post, but I thought I'd just file this in case it was different and to give you the unread count information.

Thanks for the report. I probably should have filed this one myself. :-)

I caught this one in the act today and was able to make some progress.  I caught it in the debugger while it was launching and hanging.  It turns out that the culprit is BGTaskScheduler.
For a while I thought the problem was in how we were utilizing the UserDefaults.  I thought this because when we wrote to the UserDefaults the system hung.  It turns out that we were actually getting hung in the UserDefaults.didChangeNotification that was trying to schedule BGTaskScheduler.
Now that I had the issue correct, I could actually get some traction while searching the web and came up with this.  Not very helpful, but misery loves company.
https://twitter.com/marcoarment/status/1182085151746117632?lang=en
This matches my experiences exactly.  The problem goes away on device restart.
This is clearly an Apple failure on two fronts.  First their daemon is unresponsive.  I'm assuming launchd doesn't fix this because it didn't crash.  Secondly, their BGTaskScheduler.shared.submit is throwable and should throw an exception when the daemon is unresponsive.
Hopefully Apple gets this fixed soon.

I think we have this one resolved.  Will reopen if anyone experiences hanging problems again.


[Model Upgrade] upgrade mistral-7b-instruct-v0.1 to mistral-7b-instruct-v0.2

Model Mistral-7b-instruct-v0.2 has been released for a while now, we should upgrade anything using this model. I understand that granite is our default but we still provide it as a working model so the version should get bumped.
@MichaelClifford

This should be handled via renovate.
LGTM

@Gregory-Pereira have you validated that this model works? I was under the impression V2 was not compatible with llamacpp.

Update: Tested with our llamacpp model server image. mistral-7b-instruct-v0.2 seems to work well. 😃 Not sure where I got the impression it didn't work.


upgrade to calico v3.22.0

Towards https://github.com/giantswarm/giantswarm/issues/19842

Needs to be tested.

closing this as we won't be doing it


[Polly] fs "name" is deprecated and has been renamed to "id".

Description
When running any test in jest my console shows this warning

[Polly] fs "name" is deprecated and has been renamed to "id".

I have no idea if this is bad and I have to change anything in my test environment or is it an old dependency?
But it's spamming my logs. Is this something I can change?
I'm using "setup-polly-jest" package
Config
Copy the config used to setup the Polly instance:
const options: PollyConfig = {
        persister: "fs",
        adapters: ["xhr"],
        recordFailedRequests: false,
        recordIfMissing: false,
        expiryStrategy: "warn",
        matchRequestsBy: {
            headers: false,
        },
    };

    return setupPolly(options);

Dependencies
{
    "@pollyjs/adapter-node-http": "4.3.0",
    "@pollyjs/adapter-xhr": "4.3.0",
    "@pollyjs/core": "4.3.0",
    "@pollyjs/persister-fs": "4.0.0",
   "setup-polly-jest": "0.9.0",
"jest": "26.1.0",
    "jest-localstorage-mock": "2.4.2",
"ts-jest": "26.1.3",

}

Environment
Yarn 2.1.1
node 12.18.2
win32 10.0.17134

oh dear... I just saw that the version of @pollyjs/persister-fs is not the same like pollyjs and other components! This was the solution!


Fix metadata check in generate script

I've moved the metadata check where other checks are made and just skip adding the theme object to the list of theme objects if some required metadata property is missing (currently only one required is "name").
Previously the check was done way later in the function to_zola_folder and a directory with an empty index.md was already created for the invalid theme, which would break the site building.
This should fix the current issue with the build automation of the themes site that I saw in the netlify checks for the already created pull requests.

Thanks! I did get the PRs on zola so it works!


Make the release automatic

Issue

ref #439

Overview (Required)

Modified play-publisher settings
Added metadata files

This has been almost done but publisher-keys.json doesn't work well because of the authorization issue 🙇  I'll ask to the member who is the owner account.

Finally, it works!

🎉

Thanks Let's merge this🚀


refactor: interface output

enforce parsing output to FunctionSignature and FunctionComment
change parsing data type from Map to regular Object
rename several functions to be more explicit

I like the design that is emerging. This refactoring is in the right direction. It makes the code cleaner and easier to extend (good respect of the open close principle ;-) )

thank you 🤝🤝🤝
working on it


game: add observers which were forgotten

In 4ff0cd1, adding the observer wrapper was forgotten.
The observer in YourHand will be made in #58

Thank you!


Unable to import functionapp api to APIM

i'm trying to import the function app api to apim with below command and getting 401 error.
Import-AzApiManagementApi -Context $context -SpecificationUrl https://publiclyaccessibleurl.com/test/items -SpecificationFormat OpenApi -Path deployCD
Response status code does not indicate success: 401 (Access
Denied).., Target= representation]
At line:1 char:1
Import-AzApiManagementApi -Context $context -SpecificationUrl https:/ ...
is there any way we can provide the authentication, please:? Please help me to understand
also please help me to understand the specification url, i dont want to use the specification path as needs to automate the task.
Can we import the new function app to apim direclty with the above command ? Please

@arpitmishra-eaton Apologies for the late reply. Thanks for reaching out to us and reporting this issue. We are looking into this issue and we will provide an update.

@arpitmishra-eaton AFAIK, there isn’t any PowerShell command or Rest API available to import Azure Function app into APIM at present. Usually, the command APIM supports is to import swagger file or WSDL file.
There is a certain work around available now which enables us to achieve the automation process. Firstly we need to create an open API definition file for the targeted Azure Function app, and then we can import this swagger file into our APIM using PowerShell command Import-AzApiManagementApi. .
Importing the new Function App to APIM by using Import-AzApiManagementApi. PS cmdlet is explained in detail in this techcommunity blog. Please follow the same and let me know if you face any issues.

@arpitmishra-eaton I wanted to do quick follow-up to check if you had a chance to look at my above comment. Please let us know if you had any updates on this. Awaiting your reply.


Decrementing product stock by cartItem qty

When I am adding item into cart. My code like this.
$cart = Cart::add($product->id, $product->title, $quantity, $product->price, ['stock' => $product->stock]);
Cart::associate($cart->rowId, 'App\Product');

After the order finished. I just wanna decres the stock of the product table, so, how I do this ? Can you guys explain how ?
The first I test is decres in cartItem like this.
foreach (Cart::content() as $item) {
    $item->product->decrement('stock', $item->qty);
}

But, it's returned Call to a member function decrement() on null.
So How I do this correctly ?

@Crinsane  Any Help ? :(

Solved
$item->model->decrement('stock', $item->qty);

Glad you figured it out


get some common options into host config

use start_puppetdb helper instead of the sleep hammer
increase wait time to start puppetserver otherwise the pre-suite will often fail.

@er0ck sorry, I did not see this PR yesterday. There is a conflict in the config now. Can you rebase and resolve?

@johnduarte rebased, added another commit with more fixes.  13 tests passing


Volume attach option cannot get correct specified access protocol

Is this a BUG REPORT or FEATURE REQUEST?:

Uncomment only one, leave it on its own line:
/kind bug

What happened:
When execute volume attachment via API request like below body, pb.CreateVolumeAttachmentOpts.GetAccessProtocol() outputs "iscsi" even though specified "fibre_channel".
{
  "name": "sample-volume-attachment",
  "description": "This is a sample volume attachment for testing",
  "volumeId": "1a32ab20-7801-48a2-9298-a768a11b9b1e",
  "mountpoint": "/mnt",
  "userId": "777",
  "hostInfo": {
    "platform": "ubuntu",
    "osType": "linux",
    "ip": "<IP_ADDRESS>",
    "host": "osds02",
    "initiator": "21:00:00:e0:8b:05:05:04 "
  },
  "AccessProtocol": "fibre_channel"
}

However I can get AccessProtocol as "fibre_channel" in response body.
{
    "id": "12383020-00e3-48b8-8775-b48ebcdaada6",
    "createdAt": "2019-05-03T05:31:40",
    "updatedAt": "",
    "tenantId": "e93b4c0934da416eb9c8d120c5d04d96",
    "userId": "777",
    "volumeId": "1a32ab20-7801-48a2-9298-a768a11b9b1e",
    "mountpoint": "/mnt",
    "status": "creating",
    "metadata": {
        "Group": "grp01",
        "Iqn": "iqn.2007-11.com.nimblestorage:1a32ab20-7801-48a2-9298-a768a11b9b1e-v4831f4cd0455709c.00000031.4c648451",
        "LunId": "064831f4cd0455709c000000000000000000000031",
        "PoolId": "0a4831f4cd0455709c000000000000000000000001"
    },
    "hostInfo": {
        "platform": "ubuntu",
        "osType": "linux",
        "ip": "<IP_ADDRESS>",
        "host": "osds02",
        "initiator": "21:00:00:e0:8b:05:05:04 "
    },
    "connectionInfo": {
        "data": {
            "attachment": "attachment"
        },
        "additionalProperties": {
            "attachment": "attachment"
        }
    },
    "accessProtocol": "fibre_channel"
}

What you expected to happen:
I want to get correct AccessProtocol by using pb.CreateVolumeAttachmentOpts.GetAccessProtocol().
How to reproduce it (as minimally and precisely as possible):
You may be able to see same issue when put http request like above example body.
Anything else we need to know?:
Environment:

Hotpot(release/branch) version:
OS (e.g. from /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Others:

@xxwjj Please take a look.

@fideltak You mean you can get  the right answer in the response but got the wrong access_protocol inside controller module, right?

@fideltak Oh I guess I have found the reason, if you look at here, the accessProtocol value is fetched from StoragePoolSpec structure, so you need to configure the AccessProtocol field in pool config file.

@leonwanghui Thanks! I missed it...


scripts/cli: use distro kernel as default for all tests

Avoids using existing kernel on test nodes and always installs distro kernel if no option provided
Fixes: http://tracker.ceph.com/issues/20203
Signed-off-by: Vasu Kulkarni<EMAIL_ADDRESS>

@liewegas, we could use your input on this

:+1:


Add OtoNoke by Creepy nuts

Song Name
OtoNoke
Artist Name
Creepy nuts
Source
Youtube
Youtube Link
https://m.youtube.com/watch?v=nSVHMVYxJy8&pp=ygUTb3Rvbm9rZSBjcmVlcHkgbnV0cw%3D%3D
Direct File Link
No response
Song ID
360804
Start Offset [ms]
0
Extra
Created from Auto Nong.
Created for level: 109016278

reject
popular songs like the sakupen hell song are too inconvenient to be replaced by smth unrelated and are therefore usually rejected.
Note that you can still use the song yourself, but it won't be public


some updates for plotting

The triangle plot is now updated, to fix some issues pointed out by @sthater (Sabine, please take a look and see if all is ok now!). Moreover, the plots are now accepting kinmapchi2 as a valid value for which_chi: this should solve the plotting issues pointed out here.

I added to one-liner changes :)  Tested on my ATLAS3D galaxies. Further testing should not be necessary. Thanks for taking care of this, Alice!
Below, I show the effect of the second commit:


Import fails with index check error (v.4.1)

Describe the bug
Import always ends with an index check error (tried on different PCs and servers):
With v.4.0 everything is OK.
+ '[' -f tiger-nominatim-preprocessed.csv.tar.gz ']'
+ sudo -E -u nominatim nominatim admin --check-database
2022-08-26 20:09:08: Using project directory: /nominatim
2022-08-26 20:09:08: Checking database
Checking database connection ... OK
Checking for placex table ... OK
Checking for placex content ... OK
Checking that tokenizer works ... OK
Checking for wikipedia/wikidata data ... WARNING

Wikipedia/Wikidata importance tables missing.
Quality of search results may be degraded. Reverse geocoding is unaffected.
See https://nominatim.org/release-docs/latest/admin/Import/#wikipediawikidata-rankings

Checking indexing status ... Failed
The indexing didn't finish. 61006 entries are not yet indexed.

To index the remaining entries, run:   nominatim index

Checking that database indexes are complete ... OK
Checking that all database indexes are valid ... OK
Checking TIGER external data table. ... not applicable

To Reproduce
I use this docker-compose file:
version: "3.6"

services:
    nominatim:
        container_name: nominatim
        image: mediagis/nominatim:4.1
        restart: always
        ports:
            - "6080:8080"
        environment:
            PBF_URL: https://download.geofabrik.de/russia-latest.osm.pbf
            REPLICATION_URL: https://download.geofabrik.de/russia-updates/
            NOMINATIM_PASSWORD: very_password_123
            UID: 2082
        volumes:
            - nominatim-pg:/var/lib/postgresql/14/main
        shm_size: 8gb

volumes:
    nominatim-pg:
        driver: local
        driver_opts:
            o: bind
            type: none
            device: /media/dev/osm

Expected behavior
Successful import
Screenshots & Logs
full_log.log
Desktop / Server (please complete the following information):

OS & Version: Ubuntu 22.04
Docker Version: 20.10.12
Nominatim Version: 4.1

https://github.com/mediagis/nominatim-docker/discussions/331 reports the same. Can you try adding sudo -E -u nominatim nominatim index --threads $THREADS before sudo -E -u nominatim nominatim admin --check-database in init.sh ?

Thank you!
Everything is OK!
log2.log

Should we do this in the initialisation by default, @mtmail ?


Using SDK 7.0.200, DotNetCoreCLI@2 "dotnet pack" causes NETSDK1194: The "--output" option isn't supported when building a solution.

Required Information
Entering this information will route you directly to the right team and expedite traction.
Question, Bug, or Feature?
Type: Bug
Enter Task Name: DotNetCoreCLI
Environment

Server - Azure Pipelines:  GF.MS.DB build 95030

Issue Description
When using DotNetCoreCLI@2 with pack as command, and a project that defines
        <Content Include="Resources\xx" Pack="true" PackagePath="contentFiles/xxxx">
            <PackageCopyToOutput>true</PackageCopyToOutput>
        </Content>

Using .NET SDK 7.0.102 (via UseDotNet@2) the build runs as expected
Using .NET SDK 7.0.200 (via importing 7.x, since yesterday), the build throws an error: Microsoft.NET.Sdk.Solution.targets(36,5):  error NETSDK1194: The "--output" option isn't supported when building a solution.

Task logs
   1:2>Target "ValidateToolsVersions" in file "/home/vsts/work/1/s/GF.MS.DB.Infrastructure.sln.metaproj" from project "/home/vsts/work/1/s/GF.MS.DB.Infrastructure.sln" (entry point):
       Task "Error" skipped, due to false condition; ('$(MSBuildToolsVersion)' == '2.0' and ('$(ProjectToolsVersion)' != '2.0' and '$(ProjectToolsVersion)' != '')) was evaluated as ('Current' == '2.0' and ('' != '2.0' and '' != '')).
   1:2>Done building target "ValidateToolsVersions" in project "GF.MS.DB.Infrastructure.sln".
   1:2>Target "ValidateProjects" in file "/home/vsts/work/1/s/GF.MS.DB.Infrastructure.sln.metaproj" from project "/home/vsts/work/1/s/GF.MS.DB.Infrastructure.sln" (entry point):
   1:2>Done building target "ValidateProjects" in project "GF.MS.DB.Infrastructure.sln".
   1:2>Target "_CheckForSolutionLevelOutputPath" in file "/opt/hostedtoolcache/dotnet/sdk/7.0.200/Current/SolutionFile/ImportAfter/Microsoft.NET.Sdk.Solution.targets" from project "/home/vsts/work/1/s/GF.MS.DB.Infrastructure.sln" (target "pack" depends on it):
       Using "NETSdkError" task from assembly "/opt/hostedtoolcache/dotnet/sdk/7.0.200/Sdks/Microsoft.NET.Sdk/targets/../tools/net7.0/Microsoft.NET.Build.Tasks.dll".
       Task "NETSdkError"
   1:2>/opt/hostedtoolcache/dotnet/sdk/7.0.200/Current/SolutionFile/ImportAfter/Microsoft.NET.Sdk.Solution.targets(36,5): error NETSDK1194: The "--output" option isn't supported when building a solution. [/home/vsts/work/1/s/GF.MS.DB.Infrastructure.sln]
       Done executing task "NETSdkError" -- FAILED.

I'm seeing a similar error.  It also looks like the DotNetCoreCLI@2 task is using a prerelease version of msbuild.
The pipeline specifically installs .NET 7.0.2.
Tool to install: .NET Core sdk version 7.0.x.
Found version 7.0.200 in channel 7.0 for user specified version spec: 7.0.x

The DotNetCoreCLI@2 is run with the following config
          - task: DotNetCoreCLI@2
            inputs:
              command: publish
              publishWebProjects: false
              projects: $(Build.SourcesDirectory)/mypath/MyProject.sln
              arguments: '--configuration ${{ parameters.buildConfiguration }} --output $(Build.ArtifactStagingDirectory)/MyProject --runtime win-x64 --no-self-contained'

On MSBuild version 17.4.1 --output is supported when compiling solutions.  All tasks running with DotNetCoreCLI@2 are logging a prerelease version of MSBuild

MSBuild version 17.5.0-preview-23061-01+040e2a90e for .NET

The --output problem looks related this issue that's generating quite a bit of discussion
https://github.com/dotnet/sdk/issues/30625
Microsoft response
https://github.com/dotnet/sdk/issues/30624#issuecomment-1432118204
@rvairavelu, I thought this was due to the prerelease MSBuild, but now I'm not so sure.  Should I open up a separate issue for prerelease MSBuild.

To add to my original issue: The problem shows up only when "packagesToPack" is an empty string or a directory name (causing the task to take the .sln file), and only from 7.0.200 on. Specifying a list of csproj files or a wildcard spec makes the issue go away.
This issue has stirred up a lot of steam though, and seems to be connected with 7.0.200 using some MSBuild preview that introduces incompatibilities, since this works without an issue in 7.0.102


To be more specific platform use case, updated document with RID value

To be more specific platform use case, updated document with RID values explicitly. This could be also treated as solution for issue: https://github.com/MicrosoftDocs/openapi-docs/issues/11
fixes #11

@nagarajDeveloper please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
@microsoft-github-policy-service agree [company="{your company}"]

Options:

(default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.

@microsoft-github-policy-service agree

(when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.

@microsoft-github-policy-service agree company="Microsoft"

Contributor License Agreement

@microsoft-github-policy-service agree


Remove file license blocks per discussion in 672

See: https://github.com/npgsql/Npgsql.EntityFrameworkCore.PostgreSQL/pull/672#issuecomment-431682630

@YohDeadfall wrote:

Looks like some files contains encoding preamble, but others not. Could you fix it?

I'm happy to fix it....but any advice on how to do so without editing each file manually?

Using StreamWriter and some console app? (:

At the risk of showing my tenuous familiarity with encodings...
It looks like they are all (already) set as UTF-8. It seems like the diff shows some deleted/re-inserted based on how I edited the files.
This is just a hunch, but I think the files where the BOM was deleted/re-inserted are the files that opened in Rider with their license blocks already collapsed (which I then deleted while collapsed). Perhaps in that case rider deletes (then re-inserts) the preamble?

If no one objects, I'm planning to merge this at 1 PM UTC today. Just post back if you want me to hold off for any reason.


XDS_UPSTREAM doesn't work in user gateway

(NOTE: This is used to report product bugs:
To report a security vulnerability, please visit https://istio.io/about/security-vulnerabilities
To ask questions about how to use Istio, please visit https://discuss.istio.io)
Bug description
as per https://istio.io/v1.7/docs/reference/commands/pilot-agent/#envvars setting XDS_UPSTREAM variable doesn't work, meaning the discovery address doesn't change, user gateway logs show the default service address (istiod.istio-system) as discoveryAddress
Affected product area (please put an X in all that apply)
[ ] Docs
[x] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
[ ] Upgrade
Affected features (please put an X in all that apply)
[ ] Multi Cluster
[ ] Virtual Machine
[ ] Multi Control Plane
Expected behavior
discoveryAddress should be set as per XDS_UPSTREAM env variable
Steps to reproduce the bug

install 1.7 istio with revision option

istioctl install --set revision=1-7-7

deploy user gateway with XDS_UPSTREAM variable pointing to istiod-1-7-7.istio-system (it will keep on waiting because gateway doesn't become ready)

cat <<EOF | istioctl manifest install -f -
---
apiVersion: install.istio.io/v1alpha1
kind: IstioOperator
metadata:
  name: test

spec:
  profile: empty
  components:
    ingressGateways:
    - name: istio-ingressgateway
      enabled: false

    - name: test-gw
      namespace: default
      enabled: true
      label:
        istio: test-gw

      k8s:
        env:
        - name: XDS_UPSTREAM
          value: istiod-1-7-7.istio-system

        service:
          type: ClusterIP
          ports:
          - port: 80
            name: http
            targetPort: 8080
          - port: 443
            name: https
            targetPort: 8443
EOF

in another terminal, check XDS_UPSTREAM is set to correct value but pilot agent doesn't honor that

$ k describe deploy/test-gw |grep XDS_UPSTREAM
      XDS_UPSTREAM:              istiod-1-7-7.istio-system

$ k logs deploy/test-gw |grep discoveryAddress
discoveryAddress: istiod.istio-system.svc:15012

Version (include the output of istioctl version --remote and kubectl version --short and helm version --short if you used Helm)
$ istioctl version --remote
client version: 1.6.14
istiod version: 1.6.14
istiod version: 1.7.7
pilot version: 1.5.10
data plane version: 1.7.7 (1 proxies)
$ kubectl version --short
Client Version: v1.19.3
Server Version: v1.16.0
How was Istio installed?
istioctl install --set revision=1-7-7
Environment where the bug was observed (cloud vendor, OS, etc)
Additionally, please consider running istioctl bug-report and attach the generated cluster-state tarball to this issue.
Refer cluster state archive for more details.

as a workaround, istio-system/istio-1-7-7 configmap can be copied to target namespace as istio name after removing some un-needed fields like annotations, creationTimestamp, resourceVersion, selfLink, uid

Are you actually trying to use XDS_UPSTREAM? Its a super experimental feature that doesn't really do anything in release-1.7 and was removed fully since then.
For other settings - this is a duplicate of https://github.com/istio/istio/issues/27265

i want my user gw to connect to istio 1.7 canary revision so came across XDS_UPSTREAM

This is not the correct way to do it, see https://istio.io/latest/docs/reference/config/istio.mesh.v1alpha1/#ProxyConfig. It still will not work due to https://github.com/istio/istio/issues/27265 though. Can you follow that for workarounds + the future fix? Thanks!

https://github.com/istio/istio/issues/27265#issuecomment-692055112 workaround works as expected


Validate that machine pool availability zones belong to the selected region

What this PR does / why we need it
Sometimes when configuring node pools, users may forget to choose the right availability zones, and select AZs that differ from the selected region. This makes cluster creation to fail, but the user needs to investigate why the cluster is not coming up. Instead, it would be better to fail earlier, explaining the user what the error is.
Checklist

[X] Updated CHANGELOG.md.

Trigger E2E tests

/run cluster-test-suites

/run cluster-test-suites

/run cluster-test-suites

/run cluster-test-suites

/run cluster-test-suites

Good ol' China...


Add Kronos

Pull Request Description
Kronos is an NTP client library written in Swift. It supports sub-seconds
precision and provides an stable monotonic clock that won't be affected by clock
changes.
Acceptance Criteria
To be accepted into the Swift source compatibility test suite, a project must:

[x] be an Xcode or swift package manager project
[x] support building on either Linux or macOS
[x] target Linux, macOS, or iOS/tvOS/watchOS device
[x] be contained in a publicly accessible git repository
[x] maintain a project branch that builds against Swift 3.0 compatibility mode
and passes any unit tests
[x] have maintainers who will commit to resolve issues in a timely manner
[x] be compatible with the latest GM/Beta versions of Xcode and swiftpm
[x] add value not already included in the suite
[x] be licensed with one of the following permissive licenses:

BSD
MIT
Apache License, version 2.0
Eclipse Public License
Mozilla Public License (MPL) 1.1
MPL 2.0
CDDL

[x] pass ./check script run

Looks good. Thanks!


Flaky crash in dart::SampleBlockBuffer::BuildProcessedSampleBuffer

A dry-run of a CL that changes only dart library code has resulted in flaky crash, see log:
TESTEE ERR:
===== CRASH =====
si_signo=Segmentation fault(11), si_code=1, si_addr=0xb8
version=3.1.0-edge.516f238aa6c053bb89d3d9b8ad8c1b2884ebfa9f (be) (Tue Jun 27 09:02:28 2023 +0000) on "linux_x64"
pid=207579, thread=207586, isolate_group=main(0x56125a447ab0), isolate=(nil)((nil))
os=linux, arch=x64, comp=no, sim=no
isolate_instructions=561258ca0d20, vm_instructions=561258ca0d20
fp=7f5513aa94c0, sp=7f5513aa9470, pc=561258ecde10

TESTEE ERR:   pc 0x0000561258ecde10 fp 0x00007f5513aa94c0 dart::SampleBlockBuffer::BuildProcessedSampleBuffer(dart::SampleFilter*, dart::ProcessedSampleBuffer*)+0xb0
  pc 0x0000561258ed21a0 fp 0x00007f5513aa9500 out/ReleaseX64/dart-sdk/bin/dart+0x23761a0
  pc 0x0000561258ed2122 fp 0x00007f5513aa95a0 dart::Profile::Build(dart::Thread*, dart::SampleFilter*, dart::ProcessedSampleBufferBuilder*)+0x1f2
  pc 0x0000561258ecfa12 fp 0x00007f5513aa9cd0 dart::Profiler::ProcessCompletedBlocks(dart::Isolate*)+0xe2

TESTEE ERR:   pc 0x0000561258ecfcbf fp 0x00007f5513aa9cf0 out/ReleaseX64/dart-sdk/bin/dart+0x2373cbf
  pc 0x0000561258e1241b fp 0x00007f5513aa9d40 dart::IsolateGroup::ForEachIsolate(std::__2::function<void (dart::Isolate*)>, bool)+0x10b
  pc 0x0000561258e1a436 fp 0x00007f5513aa9d70 out/ReleaseX64/dart-sdk/bin/dart+0x22be436
  pc 0x0000561258e11b99 fp 0x00007f5513aa9dc0 dart::IsolateGroup::ForEach(std::__2::function<void (dart::IsolateGroup*)>)+0xa9
  pc 0x0000561258e18ed8 fp 0x00007f5513aa9e00 dart::Isolate::VisitIsolates(dart::IsolateVisitor*)+0x38
  pc 0x0000561258ecf8f8 fp 0x00007f5513aa9e30 dart::SampleBlockProcessor::ThreadMain(unsigned long)+0xa8
  pc 0x0000561258ec8fe6 fp 0x00007f5513aa9ef0 out/ReleaseX64/dart-sdk/bin/dart+0x236cfe6
-- End of DumpStackTrace

/cc @bkonyi

cc @rmacnak-google

We still see flaky crashes, today for example in log
/======================================================================================\
| pkg/dds/test/get_cached_cpu_samples_test broke (Pass -> RuntimeError, expected Pass) |
\======================================================================================/

--- Command "vm" (took 12.000285s):
DART_CONFIGURATION=ReleaseX64 CHROME_PATH=/b/s/w/ir/third_party/browsers/chrome/chrome/google-chrome out/ReleaseX64/dart-sdk/bin/dart --enable_asserts --sound-null-safety -Dtest_runner.configuration=unittest-asserts-release-linux --ignore-unrecognized-flags --packages=/b/s/w/ir/.dart_tool/package_config.json /b/s/w/ir/pkg/dds/test/get_cached_cpu_samples_test.dart

exit code:
255

stdout:
00:00 +0: No UserTags to cache

TESTEE OUT: The Dart VM service is listening on http://<IP_ADDRESS>:42747/Hb-RwzLq1mA=/

TESTEE ERR: vm-service: isolate(4939056402051283) 'main' has no debugger attached and is paused at start.  Connect to the Dart VM service at http://<IP_ADDRESS>:42747/Hb-RwzLq1mA=/ to debug.

00:01 +1: Cache CPU samples for provided UserTag name

TESTEE OUT: The Dart VM service is listening on http://<IP_ADDRESS>:42523/yA-wOQ6SlZE=/

TESTEE ERR:
===== CRASH =====
si_signo=Segmentation fault(11), si_code=1, si_addr=0xb8
version=3.2.0-edge.b6409c0f04bf5d04692a1040decc5709c6b09b79 (be) (Tue Aug 8 09:13:51 2023 +0000) on "linux_x64"
pid=130226, thread=130229, isolate_group=main(0x55589f0919b0), isolate=(nil)((nil))
os=linux, arch=x64, comp=no, sim=no
isolate_instructions=55589e4eeba0, vm_instructions=55589e4eeba0
fp=7fefa05984c0, sp=7fefa0598470, pc=55589e71dca0
  pc 0x000055589e71dca0 fp 0x00007fefa05984c0 dart::SampleBlockBuffer::BuildProcessedSampleBuffer(dart::SampleFilter*, dart::ProcessedSampleBuffer*)+0xb0
  pc 0x000055589e722030 fp 0x00007fefa0598500 out/ReleaseX64/dart-sdk/bin/dart+0x23d2030
  pc 0x000055589e721fb2 fp 0x00007fefa05985a0 dart::Profile::Build(dart::Thread*, dart::SampleFilter*, dart::ProcessedSampleBufferBuilder*)+0x1f2
  pc 0x000055589e71f8a2 fp 0x00007fefa0598cd0 dart::Profiler::ProcessCompletedBlocks(dart::Isolate*)+0xe2
  pc 0x000055589e71fb4f fp 0x00007fefa0598cf0 out/ReleaseX64/dart-sdk/bin/dart+0x23cfb4f
  pc 0x000055589e66183b fp 0x00007fefa0598d40 dart::IsolateGroup::ForEachIsolate(std::__2::function<void (dart::Isolate*)>, bool)+0x10b
  pc 0x000055589e66a0b6 fp 0x00007fefa0598d70 out/ReleaseX64/dart-sdk/bin/dart+0x231a0b6
  pc 0x000055589e660f69 fp 0x00007fefa0598dc0 dart::IsolateGroup::ForEach(std::__2::function<void (dart::IsolateGroup*)>)+0xa9
  pc 0x000055589e668a08 fp 0x00007fefa0598e00 dart::Isolate::VisitIsolates(dart::IsolateVisitor*)+0x38
  pc 0x000055589e71f788 fp 0x00007fefa0598e30 dart::SampleBlockProcessor::ThreadMain(unsigned long)+0xa8
  pc 0x000055589e718fb6 fp 0x00007fefa0598ef0 out/ReleaseX64/dart-sdk/bin/dart+0x23c8fb6
-- End of DumpStackTrace

@a-siva Could you find someone to fix this?

I am able to reproduce this consistently on a debug build and the stack trace is as follows
#0  __pthread_kill_implementation (threadid=<optimized out>, signo=signo@entry=6, no_tid=no_tid@entry=0) at ./nptl/pthread_kill.c:44
#1  0x00007ffb110a118f in __pthread_kill_internal (signo=6, threadid=<optimized out>) at ./nptl/pthread_kill.c:78
#2  0x00007ffb110534a2 in __GI_raise (sig=sig@entry=6) at ../sysdeps/posix/raise.c:26
#3  0x00007ffb1103d4b2 in __GI_abort () at ./stdlib/abort.c:79
#4  0x0000558c3678db3e in dart::Assert::Fail (this=<optimized out>, format=<optimized out>)
    at out/DebugX64/../../runtime/platform/assert.cc:55
#5  0x0000558c36b7faed in dart::CodeLookupTable::Build (this=this@entry=0x7ffb103c2c70, thread=<optimized out>)
    at ../../runtime/platform/assert.h:30
#6  0x0000558c36b7deee in dart::CodeLookupTable::CodeLookupTable (this=0x7ffb103c2c70, thread=0xb4651)
    at out/DebugX64/../../runtime/vm/profiler.cc:1436
#7  dart::ProcessedSampleBuffer::ProcessedSampleBuffer (this=0x7ffb103c2c40) at out/DebugX64/../../runtime/vm/profiler.cc:1735
#8  dart::SampleBlockBuffer::BuildProcessedSampleBuffer (this=0x558c38d8e2f0, filter=0x7ffb10a4cba8, buffer=<optimized out>)
    at out/DebugX64/../../runtime/vm/profiler.cc:732
#9  0x0000558c36b89cd7 in dart::ProfileBuilder::FilterSamples (this=this@entry=0x7ffb10a4c238)
    at out/DebugX64/../../runtime/vm/profiler_service.cc:1007
#10 0x0000558c36b84fd4 in dart::ProfileBuilder::Build (this=this@entry=0x7ffb10a4c238)
    at out/DebugX64/../../runtime/vm/profiler_service.cc:968
#11 0x0000558c36b84d29 in dart::Profile::Build (this=0x7ffb10a4cac0, thread=0x7ffafc000dc0, filter=0x7ffb10a4cba8,
    sample_buffer=0x558c38d8e2f0) at out/DebugX64/../../runtime/vm/profiler_service.cc:1503
#12 0x0000558c36b80fb2 in dart::Profiler::ProcessCompletedBlocks (isolate=isolate@entry=0x558c38da8bc0)
    at out/DebugX64/../../runtime/vm/profiler.cc:1804
#13 0x0000558c36b8192f in dart::SampleBlockProcessorVisitor::VisitIsolate (this=<optimized out>, isolate=0x558c38da8bc0)
    at out/DebugX64/../../runtime/vm/profiler.cc:1825

I believe the crashes started happening after https://dart-review.googlesource.com/c/sdk/+/295940
This changed SampleBlockProcessorVisitor::VisitIsolate to use EnterIsolateGroupAsHelper from what used to be
EnterIsolateAsHelper, so now thread->isolate() is null or potentially uninitialized but there is code further down (e.g: FlushSampleBlocks) which assumes there is an isolate.

Ah, I see. Yeah, I also noticed that ASSERT(isolate != nullptr) was failing in debug mode, but I wasn't able to figure out why. Thanks for taking a look!


Add mergeRollupTask delay metrics

When operating merge/roll-up tasks in production, we should have
a metrics that indicates the progress or delay of merge/roll-up
task. mergeRollupTaskDelayInNumTimeBuckets is computed as follows:
(current_time - watermark - bufferTime) / bucketTime

@Jackie-Jiang When I made the change, I checked on the leader status part. Currently, we have 2 different concept of isLeader.

isLeader boolean value to invoke nonLeaderCleanUp() for PinotTaskGenerator.

We invoke nonLeaderCleanUp() when we invoke the task schedule using API because we pass isLeader=false (e.g. PinotTaskManager:scheduleTask() - L459.
We do not invoke nonLeaderCleanUp() when the task schedule is triggered via periodic task because we hardcoded isLeader=true (e.g. PinotTaskManager:processTables() - L485).
So, isLeader value here is a bit misleading because it is actually not related to the lead controller for the table. nonLeaderCleanUp() simply gets called when the task schedule is called by API call (not periodic task).

Lead controller for a table via Helix controller separation.

LeadControllerManager.isLeaderForTable() will tell us whether the current controller is the leader for the given table.

So, here are the things that I considered to handle for metrics:

We need to reset the metric when the current controller is no longer the leader for this particular table

I check it via leadControllerManager.isLeaderForTable() and clean up the metric.

Behavior for API call vs periodic task

In case we manually invoke the API, I wanted to keep the leader controller for this table to emit metrics since it's fine to have ~1hr delay for these metrics to get bumped up. It's more confusing if we see metrics are starting to get emitted from multiple controllers for a single table. Given that the manual trigger will be a rare case, I think that it's fine with updating the metrics only from the periodic tasks.
I get your point that the current code will clean up the metrics because candidateMergeTables has only 1 table. This is not the intended behavior. I will make the adjustment. I want to make the entire metrics to be No OPS in case of API schedule invoke.

Handle the case where the merge task configs get removed or mergeLevel is removed

For periodic task cases, this should be handled by the cleanUpCandidate check. I will address the case for the API invoke case.
I missed the mergeLevel change part. I will address it.

@Jackie-Jiang @mqliang @jtao15 I updated the cleanup logic to make it a bit more readable.
@Jackie-Jiang for the critical issue of deleting all metrics in case of ad-hoc scheduling. I will address the issue from the next pr by separating the path for adhoc vs periodic scheduling. As we discussed, we can add the new API with metricsEnabled flag.


Empty 'args' value in Neural Language Modeling "Training a transformer language model with the CLI tools" example model

🐛 Bug
The model trained (in Colab) according to instructions in Neural Language Modeling "Training a transformer language model with the CLI tools" example model has an empty 'args' value resulting .pt model. This contrasts with pretrained models downloable in the same page and creates problems with conversion by CTranslate2, since the converter expects keys in 'args'.
To Reproduce
Follow closely instructions in https://github.com/facebookresearch/fairseq/blob/main/examples/language_model/README.md
Expected behavior
The resulting model should have filled 'args' fields.
Environment

fairseq Version (installed through pip):
How you installed fairseq (`pip):
Free Google colab environment

@lancioni This issue is only with the CLI version right? Does the Python code version work fine?

I didn't try actually. But I understand the CLI version is just calling the Python code, isn't it?


[Dialog] The component prop is not available in DialogTitleProps

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯
When using the component property on the ModalTitle component in Typescript code, an error is thrown saying that this property is not available in DialogTitleProps.

For the code:
<DialogTitle component="h4">Title</DialogTitle>

The following HTML is successfully rendered using an h4 tag:

Test

But I got the error:
Type error: Type '{ children: string; component: string; }' is not assignable to type 'IntrinsicAttributes & DialogTitleProps'.
  Property 'component' does not exist on type 'IntrinsicAttributes & DialogTitleProps'.

Expected behavior 🤔
I suppose this error shouldn't appear, since the h4 component is correctly taken in account during rendering
Steps to reproduce 🕹
Check the following sandbox: https://codesandbox.io/s/rough-microservice-9f7jp0?file=/src/index.tsx
Context 🔦
No response
Your environment 🌎

`npx @mui/envinfo`
  Don't forget to mention which browser you used.
  Output from `npx @mui/envinfo` goes here.

Thanks for the report.
It is a bug.
I opened a pull request for the fix here.


Dropping layers (server-side) while layers are loaded in QGIS must be prevented

Already loaded layers will lose the respective server-side updatable view if the layers are dropped by the user via the Layer Loader dialog.
If this is the case, the loaded layers (actually the whole QgsLayerTreeGroup) must be dropped

Solved in v. 0.8.6


Protocol 7.0

I've upgraded my NSP to protocol 7.0. With these changes to CereLink, my applications (some Python-based, some not) seem to be working well. I've mostly tested in Windows. I tested briefly in Mac and Linux but maybe further testing there is needed.

Thanks for the PR.
I have not checked but the uint16 for list seems like a bug in the original code, that you fixed.


feature/clean-schema-api

This PR implements a more consistent interface for the DataFrameSchema and SeriesSchemaBase class and its subclasses.
The SeriesSchemaBase.__call__ method executes coercion logic and calls to the validate method that executes the validation checks. The method takes either a dataframe or series, validating the series or column in the dataframe. A nice outcome of this is the Column objects can now be used as stand-alone validators, in case user just wants to check a single column in the dataframe.
More improvements need to be made to the interface, such as DataFrameSchema implementing coercion logic in __call__ and eventually deprecating validate from the public interface (need to assess user impact, if any... for now keeping the DataFrameSchema as it is).

implement index and multiindex coercion logic, this was previously not working.
also add documentation for stand-alone column validation.

Codecov Report

Merging #160 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #160   +/-   ##
=======================================
  Coverage   95.12%   95.12%
=======================================
  Files           9        9
  Lines         513      513
=======================================
  Hits          488      488
  Misses         25       25

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0de799f...f2a9157. Read the comment docs.


How does Javy support Node.js & NPM modules?

Usually JS script requires other npm modules. Is Javy able to compile them together? I tried to have code require("firebase-admin/app") to use firebase sdk in my code but it's not working.

Is Javy able to compile them together?

You need to use esbuild or another bundler to combine your JavaScript files into a single JS file.
Javy only includes APIs present in the ECMAScript 2020 Language specification. It does not include support for NodeJS APIs that are not present in that specification. That is, Firebase client support is unlikely to work.

Got it. Thank you.


IE11 via webchat-es5.js bundle - AC button text is cut off

While helping out on #1454, I noticed that IE11 cuts off the adaptive card button text.
Tried looking at adaptive cards on the browser here  and it doesn't load at all. @compulim could I get your input?

Don't know why flex-basis: 100% is killing it, it works when I removed it. And need to investigate if it's on AC side or our side.

Note to investigators

[ ] Checks if Adaptive Card rendered on a plain HTML page on IE11 will repro or not

[ ] If it repro, file a bug to Adaptive Card
[ ] If no, find out which CSS styles in Web Chat is offending


feat: impl SIMD for function read_payload

SIMD:
web-socket (send):  5.188728ms
web-socket (echo):  14.103772ms
web-socket (recv):  8.076615ms
web-socket:         27.369312ms
Native:
web-socket (send):  5.22461ms
web-socket (echo):  15.035612ms
web-socket (recv):  8.022805ms
web-socket:         28.283226ms

Why not SIMD ?

SIMD is platform depended, so we need to write SIMD instruction for most platforms behind #[cfg(..)] flags.
SIMD is difficult, Sometime manual SIMD implementation even make execution slower.
Most of the time compiler is good enough. And generate efficient SIMD instruction.

We should implement separate SIMD specific benchmarks.


CI - Address Failures

Expected Behavior
CI should not fail as often
Current Behavior
CI seems to fail quiet a bit and I think this is just a side effect of how slow the VMs are in GitHub.
I see a few possibilities here:

More retries within tests

While this might some some failures, it won't solve all

Retries for the whole pipeline

This might require brining in a retry action, so we can retry the whole filed job.

Self-hosted runners

This would allow us to have higher performance nodes than what gitlab offers.  I have some free compute and could host some if it comes to this.

I think this is just a side effect of how slow the VMs are in GitHub.

@timothystewart6 I assume you are talking about the "Test" workflow?
I did just take a look and found that out of the last 25 runs of the workflow, 12 runs failed:

6 due to conflicts in requirements.txt:

#184
#181
#178
#173
#168
#164

4 due to a stricter molecule.yml schema in molecule 4.0.2:
- #185
- #179
- #167
- #165
2 due to flaky playbook execution:

#183
#176

So, let's look at the problems:

conflicts in requirements.txt: These are bound to re-occur in dependabot's merge requests as with a "flat" requirements.txt file, we do not differentiate between direct dependencies (stuff that we directly use) and indirect dependencies (stuff that we do not directly use but our dependencies or other indirect dependencies) use. So if any package in our dependency tree has an upper version limit on any of its dependencies and dependabot updates it, we will get these failures. The solution to this is separating our direct dependencies from the "whole, flat" list of dependencies, e.g. using pip-compile (which is also supported by dependabot).
a stricter molecule.yml schema in molecule 4.0.2: In the default and ipv6 scenario's molecule.yml, I have been using YAML anchors so that common properties of the nodes (like OS, hardware ressources) do not have to be repeated. The "anchored" nodes live in a key that ansible-lint does not read out. Since molecule 4.0.2 has changed the schema validation implementation, these additional nodes are now no longer accepted. We can either:

Get rid of the anchors and repeat that information.
Open up a discussion in the molecule project and talk about if and how our use case would be supported.

flaky playbook execution: These failures could potentially be caused by poor performance in the GitLab runners.

@sleiner Thank you for the analysis!  I am going to spin up some self-hosted runners to see if that eases the failures due to performance.
RE:  requirement.txt good to know!  I will see if I can straighten this out.  I am a fan of freezing the requirements however it might be causing too much churn having depdendabot looking at our child dependencies.
RE: molecule.yml schema, good to know.  That's too bad we can't use anchors but I am ok with repeating since it's something that shouldn't change too often.

RE: requirement.txt good to know! I will see if I can straighten this out. I am a fan of freezing the requirements however it might be causing too much churn having depdendabot looking at our child dependencies.

If we were not freezing dependencies right now, CI would be broken because of the schema changes in molecule 4.0.2, so I agree that this is sensible for this project. Just to make sure that we are on the same page: With pip-compile, you would still freeze the dependencies, but separate the frozen environment (requirements.txt) from your "known requirements" (requirements.in: only direct dependencies, usually only lower bounds on version numbers, except when you're sure a newer version would break something). Through that separation, you get multiple advantages, like...

dependabot can keep the whole frozen environment up to date (currently, if any package introduces a new dependency, it would not be added to requirements.txt - likewise if we do not need some indirect dependency anymore, it would currently not remove it from the list but keep on updating it).
dependabot can avoid conflicts in requirements.txt(as seen in the original post): requirements.in contains the actual constraints. When freezing requirements (thus creating requirements.txt, everything else can be changed as needed (usually the latest available and compatible version),
Some users might not want to install all of the frozen dependencies into a separate venv, but use the project as part of another venv. For that case, they need to ensure that the direct dependencies are met, but additional constraints on all indirect dependencies will just cause conflicts for them.

Also, if this was a "real Python project", one should probably use pyproject.toml instead of requirements.txt as well as a dependency manager that comes closer to the state of the art (in my experience, pdm is currently in the lead, other contestants are poetry and pipenv) - but for the scope of this project (we just need to install some Python tools), the really basic requirements.txt approach with pip-compile seems well-suited to me.

I think these can be addressed in separate PRs.  Thank you again for your insight!

I decided not to pursue self-hosted runners, at least in Kubernetes and for now.  https://github.com/techno-tim/k3s-ansible/pull/136


Use correct dtype in Tensor when data is an ndarray

Upon tryign to run codellama example with GPU=1 on M2 Macbook Air 16GB, I was getting following errors
AttributeError: 'E_12_4_8_16_2_4' was not found as a program info attribute or as a kernel name
pyopencl._cl.LogicError: clCreateKernel failed: INVALID_KERNEL

The kernel had was taking in a double parameter, e.g.
__kernel void E_12_4_8_16_2_4(__global float* data0, const __global float* data1, const __global double* data2) {
but was assigning to float value
float val4 = data2[(gidx1*32)+(lidx3*2)+(gidx0*128)];
and after some digging I realized Tensor was loading in ndarray with incorrect dtype

Changes made in tinygrad/:
------------------------------------------------------------
files                             insertions       deletions
------------------------------------------------------------
tinygrad/tensor.py                         1               1
------------------------------------------------------------
lines added in the tinygrad folder: 0

Neat, can you add a test to make sure this doesn't regress?

Nice!


Requires IndustricalCraft 2 (IElectricItem) and thus crashing on refuel.

Installing IC2 seems to elimintate the crash.
Stack trace:
java.lang.NoClassDefFoundError: ic2/api/item/IElectricItem
	at org.squiddev.cctweaks.integration.IndustrialCraftIntegration$1.canRefuel(IndustrialCraftIntegration.java:24)
	at org.squiddev.cctweaks.core.turtle.TurtleFuelRegistry.getProvider(TurtleFuelRegistry.java:26)
	at dan200.computercraft.shared.turtle.core.TurtleRefuelCommand.execute(TurtleRefuelCommand_Rewrite.java:31)
	at dan200.computercraft.shared.turtle.core.TurtleBrain.updateCommands(TurtleBrain.java:995)
	at dan200.computercraft.shared.turtle.core.TurtleBrain.update(TurtleBrain.java:183)
	at dan200.computercraft.shared.turtle.blocks.TileTurtle.func_145845_h(TileTurtle.java:278)
	at net.minecraft.world.World.func_72939_s(World.java:1939)
	at net.minecraft.world.WorldServer.func_72939_s(WorldServer.java:489)
	at net.minecraft.server.MinecraftServer.func_71190_q(MinecraftServer.java:636)
	at net.minecraft.server.MinecraftServer.func_71217_p(MinecraftServer.java:547)
	at net.minecraft.server.integrated.IntegratedServer.func_71217_p(IntegratedServer.java:111)
	at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:427)
	at net.minecraft.server.MinecraftServer$2.run(MinecraftServer.java:685)
Caused by: java.lang.ClassNotFoundException: ic2.api.item.IElectricItem
	at net.minecraft.launchwrapper.LaunchClassLoader.findClass(LaunchClassLoader.java:101)
	at java.lang.ClassLoader.loadClass(ClassLoader.java:425)
	at java.lang.ClassLoader.loadClass(ClassLoader.java:358)
	... 13 more

Which version of Minecraft? Can I also have your logs?

1.7.10
crash report:
http://pastebin.com/hPyM7wem
log (not much info in it):
http://pastebin.com/fEzM5aY2

Uninstalled ic2 and tried to reproduce again to get more detailed logs
For a few refuels it was fine, but then finally crashed
Short description of my actions:
Went through the enhanced portal to the mined are where 2 advanced turtles are located.
Refueled them with block of charcoal and some raw coal, successfully.
Started their mining task (those 2 are equipped with chunkloader from peripherals++)
Killed some zombies and skeletons in the dungeons
Went back trough the portal
Put one redstone dust into a simple turtle without any upgrades and commanded it to refuel.
And it crashed as before, complaining that IElectricItem is not found
Here is the detailed log:
detailed_log.txt

Sorry for taking so long to get round to this. There isn't much I can do about this: by the looks of it some mod is packaging the IC2 API but leaving out a load of files. You can try setting turtle.euRefuelAmount to -1 or, if that doesn't work, including the IC2 API in the mods folder.

I'd look at MFR, according to your crash report it has loaded it's IC2 integration.
	UCHIJAAAA	MineFactoryReloaded|CompatIC2{1.7.10R2.8.0} [MFR Compat: IC2] (MineFactoryReloaded-[1.7.10]2.8.0-104.jar)
I would've added AE2, but it claims that IC2 integration is off.


Test on node 9.x

Is there a reason we don't test on node 9?

Probably just because the last time I looked at the Travis matrix Node 9 wasn't out. I'll queue a Travis run.

Green like money. LGTM.


Update get/setBucketPolicy methods to use files instead of pkg/policy

Also removed ListBucketPolicies API as we move to json file based
policy handling.
Fixes #943

Update api.go to update the API version we need to move to 6.0.0 here.

@nitisht, second @krishnasrinivas suggestion on using policy as a string or passed in as a file.

updated the PR @harshavardhana @poornas @krishnasrinivas PTAL

@krishnasrinivas could you pls take a look again


Angular Tutorial Deploy Fail

Environment information
System:
  OS: Windows 11 10.0.26120
  CPU: (20) x64 12th Gen Intel(R) Core(TM) i7-12700H
  Memory: 1.14 GB / 15.71 GB
Binaries:
  Node: 20.12.0 - C:\Program Files\nodejs\node.EXE
  Yarn: 1.22.19 - ~\AppData\Roaming\npm\yarn.CMD
  npm: 10.8.0 - C:\Program Files\nodejs\npm.CMD
  pnpm: undefined - undefined
NPM Packages:
  @aws-amplify/backend: 1.0.0
  @aws-amplify/backend-cli: 1.0.1
  aws-amplify: 6.3.2
  aws-cdk: 2.140.0
  aws-cdk-lib: 2.140.0
  typescript: 5.4.5
AWS environment variables:
  AWS_NODEJS_CONNECTION_REUSE_ENABLED = 1
  AWS_SDK_LOAD_CONFIG = 1
  AWS_STS_REGIONAL_ENDPOINTS = regional
No CDK environment variables

Description
During the Amplify Gen 2 Angular Quick start at step 6. Implement login UI. The deployment fails due to low production budget threshold in the Angular.json
         "configurations": {
           "production": {
             "budgets": [
               {
                 "type": "initial",
                 "maximumWarning": "500kb",
                 "maximumError": "1mb"
               },

A mention of this would be handy to users
main-2OXBZKPV.js | main | 739.26 kB | 178.16 kB
styles-B2HMUAM4.css | styles | 270.78 kB | 20.75 kB
polyfills-S3BTP7ME.js | polyfills | 33.24 kB | 10.76 kB
| Initial total | 1.02 MB | 209.67 kB

Hey, thanks for raising this! I'm going to transfer this over to our documentation repository to track this.


chore: what to test update

Fails

:no_entry_sign:
Every PR deserves some kind of description. Please make sure to give this a good description.

  Generated by :no_entry_sign: Danger Swift against e4d7a2187d995d1a933df10b76a7657b32916ed3


translateEmailStringToOrderLanguage / translateEmailStringToObjectLanguage compatibility issue for WooCommerce Germanized

2021-02-21T00:48:24+00:00 CRITICAL Uncaught Error: Call to undefined method Hyyan\WPI\Emails::translateEmailStringToOrderLanguage() in /www/htdocs//*******/wp-content/plugins/woocommerce-germanized/includes/compatibility/class-wc-gzd-compatibility-woo-poly-integration.php:135
Stack trace:
#0 /www/htdocs///wp-includes/class-wp-hook.php(289): WC_GZD_Compatibility_Woo_Poly_Integration->translate_order_subject('Zahlung erhalte...', Object(Automattic\WooCommerce\Admin\Overrides\Order))
#1 /www/htdocs//a******/wp-includes/plugin.php(212): WP_Hook->apply_filters('Zahlung erhalte...', Array)
#2 /www/htdocs//a*/wp-content/plugins/woocommerce/includes/emails/class-wc-email.php(381): apply_filters('woocommerce_ema...', 'Zahlung erhalte...', Object(Automattic\WooCommerce\Admin\Overrides\Order), Object(WC_GZD_Email_Customer_Paid_For_Order))
#3 /www/htdocs//ae/wp-content/plugins/woocommerce-germanized/includes/emails/class-wc-gzd-email-customer-paid-for-order.ph in /www/htdocs/*******/******/wp-content/plugins/woocommerce-germanized/includes/compatibility/class-wc-gzd-compatibility-woo-poly-integration.php in Zeile 135

This is an error coming from the WooCommerce Germanized plugin.
In their class-wc-gzd-compatibility-woo-poly-integration.php file there are two references to a function that has been renamed in WooPoly.
return $this->get_pll_email_instance()->translateEmailStringToOrderLanguage( $subject, $object, 'subject', $email_id );
should be changed to:
return $this->get_pll_email_instance()->translateEmailStringToObjectLanguage( $subject, $object, 'subject', $email_id );
And:
return $this->get_pll_email_instance()->translateEmailStringToOrderLanguage( $heading, $object, 'heading', $email_id );
should be changed to:
return $this->get_pll_email_instance()->translateEmailStringToObjectLanguage( $heading, $object, 'heading', $email_id );

thanks, this is next bug for me :D
i hope the problem with Klarna can be solved.
Mit freundlichen Grüßen
Nuriyev
Am So., 21. Feb. 2021 um 22:57 Uhr schrieb mrleemon <
<EMAIL_ADDRESS>
This is an error coming from the WooCommerce Germanized plugin.
In their class-wc-gzd-compatibility-woo-poly-integration.php file there
are two references to a function that has been renamed in WooPoly.
return
$this->get_pll_email_instance()->translateEmailStringToOrderLanguage(
$subject, $object, 'subject', $email_id );
should be changed to:
return
$this->get_pll_email_instance()->translateEmailStringToObjectLanguage(
$subject, $object, 'subject', $email_id );
And:
return
$this->get_pll_email_instance()->translateEmailStringToOrderLanguage(
$heading, $object, 'heading', $email_id );
should be changed to:
return
$this->get_pll_email_instance()->translateEmailStringToObjectLanguage(
$heading, $object, 'heading', $email_id );
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/hyyan/woo-poly-integration/issues/540#issuecomment-782934415,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AF4O5CC6EQRT6UI4VW7ID4TTAF6VXANCNFSM4X7KXFNA
.

sorry false email...
ok, i will write to the Germanized Plugin. for update. Thank you.
Mit freundlichen Grüßen
Nuriyev
Am So., 21. Feb. 2021 um 23:01 Uhr schrieb Kamran Nuriyev <
<EMAIL_ADDRESS>
thanks, this is next bug for me :D
i hope the problem with Klarna can be solved.
Mit freundlichen Grüßen
Nuriyev
Am So., 21. Feb. 2021 um 22:57 Uhr schrieb mrleemon <
<EMAIL_ADDRESS>
This is an error coming from the WooCommerce Germanized plugin.
In their class-wc-gzd-compatibility-woo-poly-integration.php file there
are two references to a function that has been renamed in WooPoly.
return
$this->get_pll_email_instance()->translateEmailStringToOrderLanguage(
$subject, $object, 'subject', $email_id );
should be changed to:
return
$this->get_pll_email_instance()->translateEmailStringToObjectLanguage(
$subject, $object, 'subject', $email_id );
And:
return
$this->get_pll_email_instance()->translateEmailStringToOrderLanguage(
$heading, $object, 'heading', $email_id );
should be changed to:
return
$this->get_pll_email_instance()->translateEmailStringToObjectLanguage(
$heading, $object, 'heading', $email_id );
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/hyyan/woo-poly-integration/issues/540#issuecomment-782934415,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AF4O5CC6EQRT6UI4VW7ID4TTAF6VXANCNFSM4X7KXFNA
.

Apparently, this has been fixed in WooCommerce Germanized 3.3.7:
https://es.wordpress.org/plugins/woocommerce-germanized/#developers

I see that's 8 releases ago for WooCommerce Germanized so I think we can close here then.


Issue pulling Lenovo Drivers into the Driver Automation Tool

Driver Automation Tool Version Version 7.0.4
When attempting to find and download Lenovo drivers from the "Make&Model Selection" tab, I am not seeing our Thinkpad X1 Yoga Gen 6 model in the list. I also note that I am not seeing many newer Lenovo models listed. I did confirm this in discussion with Lenovo.
When I check the "Process Log" tab, I see, "Found 77 Lenovo model driver packs for Windows 10 64 bit". I have attached the Lenovo portion of the driverautomation.log.
I'm not sure where to go from here and Lenovo did advise to log this issue in this forum. They are also investigating.
Thank you for this wonderful tool! I truly appreciate the work you do.
-Matt Breuer
DAT_Lenovo.log

Hi there,
You have to pick Windows 21H2 or 21H1 (etc) and then select Lenovo. The Windows 10 option (without build) only seems to work reliably for Dell models.


Rate limits configured with environment variables

:tada: This PR is included in version 1.24.9 :tada:
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.24.9 :tada:
Your semantic-release bot :package::rocket:


Toolbar DW Icon should link back to the website

When a user clicks on the datawake icon in the left corner of the plugin, the datawake website should be opened.

To clarify behavior, if already logged in the user should be taken to the default dashboard, otherwise they'll be taken to the login page.
I

This is not happening.  Actually it's entering the URL to the base site URL twice.  For example, when you click on the DW Icon it tries to take you to a page like:
http://:3000http://:3000


Travis: Build master with ghc-8.2

We were using ghc-head for the master branch.

@hvr If this looks good for you I will land this and fix the testsuite failures in subsequent PR (namely https://github.com/haskell/haddock/pull/602).

Thanks @hvr!


Backport HSEARCH-4260 to branch 5.11 - Display Java version during builds

https://hibernate.atlassian.net/browse/HSEARCH-4260
Backport of #2615

Merged, thanks!


Yousef/master/remove lowercase on value

why this merge request closed?
Usually, IDs are case-sensitive, so I cannot use the ID in the data to locate options reasonably.


Create Guid Command

Basically what the VS tool does

we don't need all of those formats, maybe some config options in the user prefs for a default format
the ux could be after the command is executed a notification bar appears at the top
new Guid "{6D39E137-512B-4447-86E7-EAB1057215E1}" added to clipboard

as long as the notification can automatically dismiss itself after a few seconds.

https://marketplace.visualstudio.com/items?itemName=nwallace.createGUID  ??

You want to take it as a dependency? It comes up enough in .net programming that we should have it OOB.

I don't want it all if there is other plugin doing the same.


Fix typo: 'ssh' field should be 'auth' in source-packages documentation

I used authMethod: sshPrivateKey in source-packages and got the following error:
"failed to evaluate function: upstream team, auth method 'sshPrivateKey' require auth specification"

It appears that the field ssh in the documentation is incorrect and should be auth instead.

Thanks!


Implement Proper Event Store

Currently events are just held in memory.
A proper event store needs to be implemented

this is no longer needed; the decision was made to flow events rather than storing them. When there arises a need for historical event lookup we can revisit.


remove inheritable properties from reset

inheritable styles like color inherit from their parent so adding --color: initial to the reset is redundant. we can clean this up and it will also allow things like style={css({ '--height': 'var(--font-size)' })}. we would also need to do #217 for that to work though.

they're not redundant bcos they allow properties to toggle revert-layer.