[DEVX-1565] Add APAC in saucectl

Proposed changes

Types of changes

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices apply)

Checklist

[ ] Lint and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)
[ ] I have updated the json schema (if appropriate)

Further comments

@mhan83 Done :D

@tianfeng92 What framework / combo did you test it with?
Here's playwright (win 10 + firefox):
10:15:44 ERR Failed to start suite. error="job start failed; unexpected response code:'400', msg:'unexpected status code from ondemand: 400 details: {\"value\":{\"error\":\"invalid argument\",\"message\":\"No matching browser/platform was found\",\"stacktrace\":\"\"}}'" suite="Firefox using global mode"

Closing this for now, since platform changes are still very far out.


Corrupt packets trought mobile network

Hello. As far as I know, there is no cancer is RUDP. but do not know whether it guarantees the safety of the package?
In general sent Xml
rak_peer-> Send (ptr, size, HIGH_PRIORITY, RELIABLE_ORDERED, 0, to_guid, false);
And sometimes it comes distorted, and always the same
Here - the most interesting thing in the end, as if a piece of torn out and crooked inserted ago
Normal - http://www.everfall.com/paste/id.php?8g41jw03yyf2
Damaged - http://www.everfall.com/paste/id.php?ivmdr8fa0cc8
This crap happens rarely, but it happens only when the server Win64 client communication via Android 2G 3G or 4G ...

I think there is a good chance this is your issue: https://github.com/OculusVR/RakNet/issues/50


feat: handle macOS ctrl key

fix #154

Ahhh thanks! I couldn't find this simple keycode for Mac cmd key :) Didn't even check the Windows key one.


Fix encoding of byte values when passed as strings

Codecov Report

Merging #8 into master will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master        #8   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            3         3
  Lines           64        65    +1
  Branches        16        17    +1
=========================================
+ Hits            64        65    +1

Impacted Files
Coverage Δ

src/eip-712.ts
100.00% <100.00%> (ø)

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 7939dba...494d817. Read the comment docs.


Better handeling of image for the interests.

In place of having to tweak the Interests.js file why no having it checking for a name match in the images folder ?
for exemple if my interest is "football" it would check for football.png/jpg/gif then only show the wildcard.
this would allow for more hands-off approach when it comes to a non forked container.

There is an undocumented property with name "image" in interests json. E.g. interests: [{ name: "football", image: "https://cdn.dribbble.com/users/923520/screenshots/4436772/salah-animation-gif.gif", name: "skiing", image: "https://lehasvv2009.github.io/resume/static/media/snowboarding.30404734.gif" }].


getbestblockhash method doesn't allow to set client timeout?

Hey @Mukhametvaleev , thanks for taking an interest in this library and raising the issue (even though you promtply closed it ;) ).

Actually, to have the ability to set a timeout on every RPC call is a good idea.
Let me think it through a then come up with potential solutions.
Are you in a rush having this feature implemented? It has been some time I touched the code, so I guess I could squeeze in some other ideas that have been brewing in my mind as well. ;)
Thanks again and cheers!
Libor

Hey @Mukhametvaleev , thanks for taking an interest in this library and raising the issue (even though you promtply closed it ;) ).
Actually, to have the ability to set a timeout on every RPC call is a good idea.
Let me think it through a then come up with potential solutions.
Are you in a rush having this feature implemented? It has been some time I touched the code, so I guess I could squeeze in some other ideas that have been brewing in my mind as well. ;)
Thanks again and cheers! Libor

I end up with:
rpc = BitcoinRPC(
    url,
    user,
    password,
    timeout=None,  # <– HERE
)


Remove unused Marker from Dispatcher struct

Signed-off-by: Julius Volz<EMAIL_ADDRESS>

Please make sure to rebase after #2904 is merged.

Rebased, thanks.

Thanks


[Roman Numerals] add practice exercise (v2)

Let's try this again, on a new day.
For the week of [02-06]. The example.R doesn't exactly showcase R's best features, but it's the best I've come up with so far.

I have some draft approaches ready to PR, but I'll wait until this exercise is merged so I have a base for adding the .approaches directory.

Once the exercise is merged I can rebase the approaches and PR them. There might be better solutions in there, that can be copied across later.

I've just skimmed through the approaches in your branch which look good - thanks. I'm sure we can get those merged in over the weekend ahead of next week's #48in24


pub upgrade starts finding a solution with a more strict constraint

See attached files.
Basically:
Case one dependency on x: >= 0.1.0 <0.2.0
Fails
Case one dependency on x: >= 0.1.2+3 <0.2.0
Succeeds?

Attachments:
less_strict_output.txt (21.45 KB)
less_strict_pubspec.yaml (380 Bytes)
more_strict_output.txt (8.40 KB)
more_strict_pubspec.yaml (382 Bytes)

This issue has been moved to dart-lang/pub#978.


Long line issue

Hi,
when I have a long line code inside my page, it would 'overflow'...
Like this:

BTW anatole is an extraordinary work!

Hi there,
Thanks for bringing this issue to my attention.
I've tweaked the css for the code blocks. Please see my commit 27289efeb428d8a8ba527ffe78b2ecf1d2002f60. Instead of the ugly overflows, it's now adding a horizontal scrollbar. This behaviour is very similiar to how github gists is handling too wide code blocks.


Install protobuf through apt-get instead of pip

Add python3-protobuf to the existing apt-get install command, instead of installing it with pip

Fixing... Stand by for another PR.


Add to list of STAC extensions

Hey there,
I think it would be nice to add the general-purpose extensions (so all except the "linz" extension) to the stac-extensions list at https://stac-extensions.github.io/
This would help others who are looking for something similar.
Thanks!

Thanks @m-mohr. That sounds like a good idea.
We were also wondering if/when we could move some of them to the stac-extensions github organisation. What are your thoughts on that? We were thinking we could test out some of the concepts ourselves first and then propose them as a community extension?

@l0b0 This would be the first one to add I think. https://github.com/linz/stac/tree/master/extensions/quality

Just found this issue again, sorry.
Yes, I think it is a good idea if you don't mind the effort. Otherwise, you can also simply add it to the table of extensions and link to this repository. Happy either way, it's just important that people get aware of all the extensions out there so that we don't reinvent the wheel and get as much feedback as possible.

Thanks @m-mohr good idea to just add them to the table at the very least. Will do that.


error[E0277]: [Spans<'_>; 4] is not an iterator

I am running Parrotsec OS
Distributor ID:	Debian Description:	Parrot OS 5.0 (LTS) Release:	5.0 Codename:	bullseye
I tried to install ttyper.
Cargo was already installed
cargo --version cargo 1.46.0
I got the following error after running
cargo install ttyper
error[E0277]: [Spans<'_>; 4] is not an iterator
Details to the error are in the attached file.

This is probably due to the fact that old versions of Rust did not implement IntoIterator on arrays due to the lack of const generics. You can fix this by using a more recent Rust version.


Go fullscreen and hide the cursor during the devices check

Some browsers will ask the user's permission to go fullscreen or hijack the cursor. It's better if it happens here, not during the first training trial.

Decide not to bother - first training trial is a good enough place to give the permissions.


As Uhlissa UTA, I want to be notified when tickets join the queue without having the tab open.

Change tab title to notify user of new tickets in queue
Play a notification sound when new tickets are created

@maddyandrews What is done here will be very useful for the rest of the site!


Game does not starts with Krastio 2 and Bullet Tracers

"RenaiTransportation": RenaiTransportation/data-final-fixes.lua:778: attempt to index field 'action_delivery' (a nil value)
stack traceback:
RenaiTransportation/data-final-fixes.lua:778: in main chunk
factorio-current.log

idk if something changed but I just tried it and the game loads and plays with those mods installed, I shot a few nests and the bullet tracers worked

Confirming something changed. Something is got fixed


In iOS, when I try to send a message, keyboard hides right away

Describe the bug
I was able to display the widget properly, although when I try to type something in the keyboard, it hides right away
To Reproduce
Steps to reproduce the behavior:

Go to component where ChatView is, try to type in the keyboard.
Keyboard shows and hides right away.

Expected behavior

Keyboard doesn't hide

Screenshots
If applicable, add screenshots to help explain your problem.
https://github.com/SimformSolutionsPvtLtd/flutter_chatview/assets/12133723/5a8b487a-ad8c-4cbd-a334-d9b9a6323d89
Smartphone (please complete the following information):

OS: iOS 15 Pro Max
Version 2.0.0

Additional context
Add any other context about the problem here.

Hello @vincegnzls,
Since we are unable to reproduce this, can you please try this with the example app of this package? Otherwise, you can also share a minimal reproducible code with us so that we can look into it better. Thanks!

Hello @vincegnzls, Since we are unable to reproduce this, can you please try this with the example app of this package? Otherwise, you can also share a minimal reproducible code with us so that we can look into it better. Thanks!

https://github.com/SimformSolutionsPvtLtd/flutter_chatview/assets/63441861/9e08ca5e-791c-46dd-a8e5-4b4e5681d878
I have the same issue, when i using auto_route, Nested Navigation

@giangphan11 @vincegnzls Could you please provide a reproducible example so that we can investigate this further?

I am closing this issue due to inactivity. If the issue persists in Chatview 2.0.1, feel free to reopen it.

Hey folks, I have conducted deep investigation to the library and found the root cause. PR #258 will attempt to fix the issue.


Replace org.json JSON library

I have a rather unusual request. Unireset-java uses the org.json JSON library, which is considered non-free software by some parties such as Debian or Google (https://wiki.debian.org/qa.debian.org/jsonevil). Thus, the library cannot be used in Debian, Ubuntu, Android. As a consequence, unireset-java cannot be packaged, too.
Is it possible to replace the library by json-simple or by a cleanroom reimplementation of org.json (available under com.android.json in debian)?

This issue had practical implications in Apache Jackrabbit Oak, see this issue regarding the removal of org.json:json and more followup work to (sadly) stop using Unirest in our project.

As a side note: For Apache and the JSON license cf. also https://lwn.net/Articles/707510/

FWIW, the unirest-java package is working fine in Debian using the cleanroom implementation Oliver refers to in the request.  The only difference is behavior for users is that JSONException is thrown in some additional cases.

The openjson project offers org.json's functionality under the package com.github.openjson with the additional exceptions @tmancill mentions. Thus, a port to openjson should be easy. In case this project is still maintained, I am willing to create a PR.

There are several significant deviations in openjson from the original, there are the checked exceptions yes (gross) but also JSONArray is not iterable.   This would result in quite a few breaking changes for consumers. I'd prefer a implementation that actually met all the same contracts as the original, OR go with something entirely different like Jackson or GSON.

Closing this for now. If someone really has a problem with the clause they can always exclude that dependency and not use the built in json. I don't see any implementation which is compatible with the current version and wouldn't cause issues. The only real solution is changing to GSON or Jackson, which might happen, but in a future major version

Bummer...  Instead of carrying a Debian package that has to be patched to use a JSON implementation that isn't supported by the upstream author, I will work on removing unirest-java from Debian in buster + 1.

yea, sorry, like I said, the "cleanroom" implementation doesn't even work, a whole bunch of tests break when attempting to use it. If you can find one that actually meets the contract then I might be open to it.

Okay, cool.  I'm to see if we can't make https://github.com/openjson/openjson more generally useful.

One thing that would help a lot is if it shared a namespace with org.json. Namespeces aren't like maven package names and there is nothing that says you can't occupy the same package. Heck Google didn't when they implemented all of Java.
If we are going to take a breaking compiler error then I would almost rather switch to gson

We are going to have a major disruption when we eventually upgrade to Apache Http Client 5. I think at that time we will simply replace this library with GSON or Jackson

I'd recommend standard javax.json

standardized
efficient
simple, no reflection magic
powerful, full programmatic access to all JSON operations

One of the implementations: https://github.com/eclipse-ee4j/jsonp

I would be also happy if it could be replaced by e.g.:
com.google.gson 2.8.2
The reason is, that I would like to use Unirest in an Eclipse project. The library org.json was rejected, due to license issues. Gson is fine:
https://download.eclipse.org/tools/orbit/downloads/drops/R20190602212107
Hence, getting rid of org.json would increase the chances of Unirest to be used in Eclipse projects.

@eselmeister Please check Eclipse internal IPZilla for org.json.

The latest available org.json version is 1.0.0:
https://download.eclipse.org/tools/orbit/downloads/drops/S20190819183153/
There will be no update to a newer version due to the license issues
already mentioned: "The Software shall be used for Good, not Evil.".
Am 25.08.19 um 09:25 schrieb Oliver Kopp:

@eselmeister https://github.com/eselmeister Please check Eclipse
internal IPZilla for org.json.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Kong/unirest-java/issues/176?email_source=notifications&email_token=AAFHUTYD7CZNPU74ZCBXHXDQGIXV5A5CNFSM4CTI76WKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5COC7Q#issuecomment-524607870,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAFHUT46LIG55MPH7EGACN3QGIXV5ANCNFSM4CTI76WA.

--
OpenChrom - the open source alternative for chromatography / mass spectrometry
Dr. Philip Wenig » Founder »<EMAIL_ADDRESS>» http://www.openchrom.net

I'm starting to work on this, it might be a couple weeks until I'm done. I'm going to create classes that mostly match the public signatures of org.json but are actually powered by Gson.
This should give most people using the json stuff a pretty easy upgrade path (just change the namespace to kong.unirest.json). It also gives this project some ability to make the classes better because quite frankly it's not even terribly good compared to what jackson and gson offer for json tree objects.
I am NOT going to match org.json's utility classes for dealing with XML, csv, cookies, etc.
The reason I'm going with gson is mostly because Jackson is pretty big, lots of libraries, and more projects use it and use fairly complicated configurations. I don't want my dependency to conflict with it.

That sounds like a good plan. The latest available and checked version
in Eclipse Orbit is v2.8.2:
https://download.eclipse.org/tools/orbit/downloads/drops/S20190819183153/
If you will use a newer one, please let me know:
https://github.com/google/gson/releases
In such a case, I will create a CQ to get into Eclipse Orbit.
Am 25.08.19 um 18:36 schrieb Ryan Bergman:

I'm starting to work on this, it might be a couple weeks until I'm
done. I'm going to create classes that mostly match the public
signatures of org.json but are actually powered by Gson.
This should give most people using the json stuff a pretty easy
upgrade path (just change the namespace to kong.unirest.json). It also
gives this project some ability to make the classes better because
quite frankly it's not even terribly good compared to what jackson and
gson offer for json tree objects.
I am NOT going to match org.json's utility classes for dealing with
XML, csv, cookies, etc.
The reason I'm going with gson is mostly because Jackson is pretty
big, lots of libraries, and more projects use it and use fairly
complicated configurations. I don't want my dependency to conflict
with it.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Kong/unirest-java/issues/176?email_source=notifications&email_token=AAFHUT7A55JYFUINQO363CTQGKYHNA5CNFSM4CTI76WKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5CW6AQ#issuecomment-524644098,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAFHUT7SNKKQQAV4MGXUTBTQGKYHNANCNFSM4CTI76WA.

--
OpenChrom - the open source alternative for chromatography / mass spectrometry
Dr. Philip Wenig » Founder »<EMAIL_ADDRESS>» http://www.openchrom.net

Update: new version will be out this weekend!

This is complete in Unirest 3.0.00!
I decided to do a cleanroom implementation of org.json's public interfaces but powered everything with gson. This means Unirest consumers should only need to change their package names. background and details are in the upgrade guide https://github.com/Kong/unirest-java/blob/master/UPGRADE_GUIDE.md

Excellent, thank you @ryber !

Perfect, thanks a lot.
Am 22.09.19 um 17:33 schrieb tmancill:

Excellent, thank you @ryber https://github.com/ryber !
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Kong/unirest-java/issues/176?email_source=notifications&email_token=AAFHUTZ2GL3NPYQ2OLZA653QK6F4XA5CNFSM4CTI76WKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7JIWPQ#issuecomment-533891902,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAFHUTZS3QQ2VQHXBHFSIGLQK6F4XANCNFSM4CTI76WA.

--
OpenChrom - the open source alternative for chromatography / mass spectrometry
Dr. Philip Wenig » Founder »<EMAIL_ADDRESS>» http://www.openchrom.net

Hi folks,
thanks for your work.
Unirest 3.4.02 has been added to Eclipse Orbit now:
https://bugs.eclipse.org/bugs/show_bug.cgi?id=560175
https://git.eclipse.org/r/#/c/157750/


fix: support-node-workspace-plugin-prerelease

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[x] Make sure to open an issue as a bug/issue before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[x] Ensure the tests and linter pass
[x] Code coverage does not decrease (if any source code was changed)
[ ] Appropriate docs were updated (if necessary)

Fixes #2248 🦕

@chingor13 tested and validated using my local branch of https://github.com/Seekr-io/release-please-action

@Animalmix55 shouldn't the version be incremented to x.x.x-beta.x instead of x.x.x-beta ?

@Animalmix55 shouldn't the version be incremented to x.x.x-beta.x instead of x.x.x-beta ?

The initial prerelease comes with no .#, subsequent prereleases begin appending and iterating the .#.

I'm also interested in this fix, is there any way to get it approved/merged?

I need to be able to rebase/update from main to merge this. Can you either rebase/merge from main or enable the "Maintainer may modify" checkbox?


[Feature] - Add the ability for a stream to send to a thread.

Is would be useful to be able to send long streams of output to a threaded message.
The first message (or leading comment) goes to the selected channel and then all subsequent messages are threaded to the original.

#85 PR created

merged for inclusion in 1.7.3 release

Is 1.7.3 released?
I just installed via homebrew, and I have version 1.7.2. I tried the --thread parameter based on the README, but got this error message:
Incorrect Usage. flag provided but not defined: -thread

@bcdady 1.7.3 release published now


Invalid item in chat with item2chat

Invalid item in chat with item2chat please help me

Duplicate of #779


Use the queryset returned by SnippetViewSet.get_queryset() in other views

Is your proposal related to a problem?
This is a similar problem described in wagtail-nest/wagtail-modeladmin#21, but for snippets.
When you override SnippetViewSet.get_queryset(request), the resulting queryset is only used for the index view. This means that if you put certain filters to prevent users from accessing certain objects, the "detail" view e.g. edit, delete, inspect, etc. can still be accessed via the URL.
The documentation does mention that the get_queryset() method override is for the index view, but for the reasons stated above, it would be nice if this extends to the other views as well.
Describe the solution you'd like
One way to achieve this would be to allow the queryset to be defined as a class/instance attribute on those views (some of them that extend from Django likely support this already), then pass the SnippetViewSet.get_queryset function down to the view as the queryset, then update the view code to check if the queryset is a callable. If so, call it with the request object, then use it to do the rest of the view logic.
Describe alternatives you've considered
Alternatively, just like ModelAdmin, we could leverage the fact that snippet models have the viewset accessible via model.snippet_viewset, and use it directly in the view. However, this does mean that the view now "knows" about the existence of the viewset, which is undesirable as the viewset is a higher abstraction on top of the view. Not to mention that this doesn't sit well with our work on making snippets features more generic in #10740.

+1 Just stumbled over this while using ModelViewSet and SnippetViewSet. I've automatically assumed that the overridden get_queryset() works similar to django's ModelAdmin :cold_sweat:
Re-using that existing pattern used for the IndexView sounds good, maybe we could also move that logic into the parent ModelViewSet?
Maybe it makes sense to add some kind of get_instance_specific_view_kwargs(), which could be used by all views based on a single object.

I was thinking about trying a PR here, so I tried to get a better overview of the used classes. Of course there are multiple ways to solve it, so I would like to see if my plan seems good to you:

The BaseObjectMixin (link) seems quite a good place to implement the queryset-resolution (check if set, check if callable,...).
Find all used classes within the ModelViewSet and SnippetViewSet, which do not yet inherit from the BaseObjectMixin. The InspectView (link) is one example, but there are more (PreviewRevision,...) where I think this would fit quite well. This would reduce duplicate code (e.g. things like pk_url_kwarg).
Move SnippetViewSet's get_queryset() (link) into ModelViewSet, and set the queryset-kwarg in ModelViewSet.get_index_view_kwargs() instead of SnippetViewSet.get_index_view_kwargs().
Find the existing get_*_view_kwargs() or *_view() and pass queryset=self.get_queryset.
I thought I would not implement a new get_instance_specific_view_kwargs for now, as I've wrote in the previous comment(?).
Tests:

Actually, the tests should not fail after the change, as the default was always to use the models default manager/queryset.
I would then add new test-classes which override the default queryset and then add simple GET-tests for all the used view classes, which should result in 404s.

The change does affect many places, but it seems quite straight forward, but I'm not sure if I oversee something here :sweat_smile:

Thank you for breaking it down @th3hamm0r!
I think my comment in https://github.com/wagtail/wagtail/pull/10621#issuecomment-1712079277 and my recent PR #11005 are of relevance here.
 I know this somewhat goes against my preference of keeping things flat in my linked comment. And it does mean we lose control over the code.
However, at least we're only making use of a mixin, not a base view class, so it only gives us methods/properties without really "controlling" the flow of the view process other than the things that happen in get_queryset. Thus, it doesn't suffer from issues such as get_object being called later in the view process when we might need it in setup()/dispatch().

Never mind, let me start over.

Yeah I think BaseObjectMixin is a good place to put it. I was considering using Django's SingleObjectMixin instead, but it seems to have a bunch of other stuff we don't really need. It'd probably be a good idea to follow the signatures though, so you can benefit from the familiarity of Django's views.
Yep that is a good idea.
Might be better to move it to get_common_view_kwargs, but we need to be sure as there might be cases where we don't want the queryset to be passed automatically.
See 3.
We'll just need to add tests for the other views in the tests I added in #11005.

I think having the base queryset customisable at the ViewSet level is a valid use case for reasons already outlined in this issue. The queryset isn't final, the views can add their own default additions on top (e.g. search and filtering, adding annotations).
I think having the request available to the customisation point (like the current SnippetViewSet.get_queryset) would be important for most use cases, as otherwise this customisation isn't really useful. I previously discussed in https://github.com/wagtail/wagtail/pull/10851#discussion_r1315047040 whether it we should allow access to the view instance itself (so that you have access to other things, not just the request), but as per the discussion there, it's not a really good idea.
Before we proceed with this, I'd like to highlight this comment:

...I think we're better off embracing the fact that methods on the viewset aren't going to be as capable as methods defined on the views themselves, and that if you want to do anything that relies on tapping into the state of the views, you should override the view instead.

I think we should allow exposing the base queryset (with access to the request) in the viewset be an exception here. What do you think @gasman?
Let's also discuss this at the next core team meeting.

Yeah I think BaseObjectMixin is a good place to put it. I was considering using Django's SingleObjectMixin instead, but it seems to have a bunch of other stuff we don't really need. It'd probably be a good idea to follow the signatures though, so you can benefit from the familiarity of Django's views.

:+1:

Yep that is a good idea.

A special example is the HistoryView, which is both a detail view for the model and an IndexView, but it still seems to make sense to use the BaseObjectMixin for the get_object()-functionality here. The get_base_queryset() is already a custom version, which does not use the queryset attribute. So this shouldn't collide with the BaseObjectMixin I think.

Might be better to move it to get_common_view_kwargs, but we need to be sure as there might be cases where we don't want the queryset to be passed automatically.
See 3.

You're right :+1:. Initially I thought just of the CreateView as an example which probably does not need the queryset, so it spoke against get_common_view_kwargs for me, but actually, most views could benefit from getting the base-queryset passed, and if not, you could still overwrite the kwarg when you instantiate your custom view (if it requires a different queryset).

We'll just need to add tests for the other views in the tests I added in Allow IndexView's base queryset to be customised via ModelViewSet.get_base_queryset() #11005.

:+1:

I think we should allow exposing the base queryset (with access to the request) in the viewset be an exception here. Of course, with the queryset being used in multiple views, developers have to be mindful of the returned queryset (ensuring it's not tailored to a specific view). What do you think @gasman?

Even though Django's modeladmin is structured a little differently, it still feels natural when the base-queryset is defined via the viewset. At the end, every view can build on it or not, but adapting the core views to consistently use it seems to be a good direction for me.

+1 for this feature.
I'd like to question whether passing the request object is the right choice - it seems to me that the primary (only?) motivation for sharing the get_queryset method between views is access control, in which case just passing the user object ought to be sufficient. I think it's better to be conservative about the amount of context we give to get_queryset, to discourage developers from filtering on things like URL parameters that are really specific to IndexView. It's not so much "I don't trust developers to use this responsibly", and more about nudging them to think about the correct place to put the logic - "is this dealing with access control, or the UI of the index view?"
Happy to give way on this if we can think of other scenarios for SnippetViewSet.get_queryset() beyond limiting the objects globally and per-user. Maybe something like how the locale URL parameter is passed around? (But even there, I don't think there's a need to filter the queryset anywhere but IndexView.)
Incidentally, I'd like to see this using the inject_view_methods mechanism introduced in #10851. Having the views implement a special case for "this object claims to be a queryset, but might actually be a callable" feels hacky, and obviously it's better from a code quality POV if we don't have multiple mechanisms for doing the same thing in different places.

TL;DR: I understand your points, but I would vote for making an exception with get_queryset and pass the request to it. It defines what the ModelViewSet manages and I would allow the developers adapt it centrally based on the request, not only the user object.
Looking at the current passed kwargs of all the views in the ModelViewSet here, beside many args that won't change for a viewset once instantiated, there are multiple cases, where passing a request would be nice, e.g. to quickly adapt the "list_display" for the index view based on the request. But here, I'm 100% with you, I wouldn't start passing the request object just because it is a central place. Its very easy to add a IndexView-subclass and adapt the list_display there based on the request (softly directing the dev to put the code where it belongs to :wink: ).
On the other hand, changing the central base queryset, which is probably one thing that is required by nearly every view class of a ModelViewSet (except the CreateView), based on the request, would require to adapt and think of every view class that is used in the viewset. And since it's so common to have a central get_queryset with a passed request, I would make this one exception.

I'd like to question whether passing the request object is the right choice - it seems to me that the primary (only?) motivation for sharing the get_queryset method between views is access control, in which case just passing the user object ought to be sufficient.

Over the years, I've used various values from request to customise the return value for a get_queryset() method:

Values from request.GET (e.g. to avoid complex query optimisations when the q param is present, to get around search backend limitations)
request.resolver_match (e.g. to apply certain filters for some views and not others, without having to customise multiple view classes)
Multi-site/multi-tenancy related lookups (where it's common to pass the request on to identify the site/tenant of interest)
Attribute values set by custom middleware

While it's certainly good to think about what we pass around and why, the downside of that is the restrictions it can introduce. While there could sometimes be a better way to do something, getting the job done quickly is often just as important (maybe you're working on a quick POC, or maybe you're new to Django/Wagtail and simply aren't aware of the alternatives). Being able to count on having access to the request in situations like this offers a lot security to devs, especially if they've spent a significant amount of time working with Django views or modeladmin (where passing request around is as common as day).

OK, I'm persuaded :-) Let's go with request!


dico: missing python and guile modules

Describe the bug
The dico package is built without Python and Guile module support even though they are added as dependencies, so the intention seems to have them included.
It can be seen in the build logs:
...
checking for a Python interpreter with version >= 3.0... none
...
Dico configured with the following settings:

GSASL ........................... yes
Readline ........................ yes
Preprocessor .................... NULL

Guile ........................... no
LDAP ............................ no
PAM ............................. no
Perl-compatible regexs .......... yes
Python .......................... no
WordNet ......................... no

Default server .................. gnu.org.ua
Autologin file .................. .dicologin

https://hydra.nixos.org/build/240690147/nixlog/1
And the modules are missing in the package files:
$ ls /nix/store/gwys6p4hmlpsj9h4rd4dfkygxnwj1j7v-dico-2.11/lib/dico/
dictorg.la  echo.la  gcide.la  metaphone2.la  nprefix.la  outline.la  pcre.la  stratall.la  substr.la  word.la
dictorg.so  echo.so  gcide.so  metaphone2.so  nprefix.so  outline.so  pcre.so  stratall.so  substr.so  word.so

I haven't investigated why Python and Guile are not detected by autoconf, I would expect having them as buildInputs should be enough.
Steps To Reproduce
Try to use a Python module in the dicod configuration.
Expected behavior
The Python and Guile modules should work (or be optional and have a way to add them).
Notify maintainers

@lovek323

Seems the dependencies just need to be added to nativeBuildInput as well so autoconf can discover them, I'll look into making a PR.


Proposal: change the responce code in eventshub receiver for dropped events

I am writing an e2e test for apiServerSource's retry feature, but the cloudevents.Client only retries when the HTTP response code is one of  these.
I want to use eventshub to simulate a events lost scenario but the eventshub returns http.StatusConflict when dropping events. and the http.StatusConflict is not one of the codes which cloudevents.Client retries.
So I have a question. Is there any special reason to choose  http.StatusConflict ?
Can we change it to 429(Too Many Requests) or 503(Service Unavailable) or 504(Gateway Timeout) for some tests scenario like retring?
Or is there any advices for this e2e test?

@vaikas What do you think of this

Not totally sure if this would fit your use case, but for RabbitMQ Broker, I ended up using this:
https://github.com/knative-sandbox/eventing-rabbitmq/blob/main/pkg/dispatcher/dispatcher.go#L80
With some more context here:
https://github.com/cloudevents/sdk-go/pull/596

Not totally sure if this would fit your use case, but for RabbitMQ Broker, I ended up using this:
https://github.com/knative-sandbox/eventing-rabbitmq/blob/main/pkg/dispatcher/dispatcher.go#L80
With some more context here:
cloudevents/sdk-go#596

Thanks!  It is very helpful!

Great to hear :) Ping me if you need some more help :)

@vaikas  Hi, what do you think we add a new env DropEventsResponseCode to config the status code returned when dropping events ?
I think in my scenario(https://github.com/knative/eventing/pull/5457), it might not be good to modify the default strategy of cloudevent.Client
in pkg/adapter for the convenience of writing tests, because It may affect other components that use cloudevent.Client.

@xinydev makes sense to me, and it will be a lot useful for testing the behavior of some specific status codes.

/assign


What about custom groups?

"If you create an environment within a YAML, contributors and project administrators will be granted Administrator role. This is typically used in provisioning Dev/Test environments."
=> We have custom groups in our projects - When env is created automatically, these groups are added as readers. It would be nice if we have a possibility to change the standard behaviour of the automatic creation - so that it is possible to set own standard permissions (like template or sth.)

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

ID: 77d95db6-9983-7346-d0eb-4b7443e4e252
Version Independent ID: 0a22cccc-318d-592f-d1ab-09ec01d88087
Content: Environment - Azure Pipelines
Content Source: docs/pipelines/process/environments.md
Product: devops
Technology: devops-cicd-process
GitHub Login: @juliakm
Microsoft Alias: jukullam

Thanks for the suggestion! The product teams aren’t always watching this repository, so to make sure it gets in front of them for triage, please submit your idea here.


ACN_CD_QualysKB_Parser01

Proposed Changes

Qualys KB Parser

Sample Data

Not required, but recommended - If Detections are added later, validations will fail on all the Detection YAML's because the table schema cannot be identified since this is a Custom Log. While not required here, to prevent any issues with future Detection addition, you may want to add a Custom Table schema here - https://github.com/Azure/Azure-Sentinel/tree/master/.script/tests/KqlvalidationsTests/CustomTables. They are simple JSON format. The name of the table goes at that top in the name field, in your case it would be "QualysKB_CL". Then the schema follows in the Properties key value pair section.

@chicduong - do you want to include this or do later?

@shainw

Not required, but recommended - If Detections are added later, validations will fail on all the Detection YAML's because the table schema cannot be identified since this is a Custom Log. While not required here, to prevent any issues with future Detection addition, you may want to add a Custom Table schema here - https://github.com/Azure/Azure-Sentinel/tree/master/.script/tests/KqlvalidationsTests/CustomTables. They are simple JSON format. The name of the table goes at that top in the name field, in your case it would be "QualysKB_CL". Then the schema follows in the Properties key value pair section.

@chicduong - do you want to include this or do later?

@shainw I added the file, but I'm still unclear on what property key value pair should be added here. Should I add all the ones we're expecting?

@shainw

Not required, but recommended - If Detections are added later, validations will fail on all the Detection YAML's because the table schema cannot be identified since this is a Custom Log. While not required here, to prevent any issues with future Detection addition, you may want to add a Custom Table schema here - https://github.com/Azure/Azure-Sentinel/tree/master/.script/tests/KqlvalidationsTests/CustomTables. They are simple JSON format. The name of the table goes at that top in the name field, in your case it would be "QualysKB_CL". Then the schema follows in the Properties key value pair section.

@chicduong - do you want to include this or do later?

@shainw I added the file, but I'm still unclear on what property key value pair should be added here. Should I add all the ones we're expecting?

@chicduong - Sorry for not being clear.  Yes, you want to include the entire schema, so yes all of them.  Within reason.  If someone submits a detection with a field that is new after this, then the validation will fail, but we can simply add that into the JSON when that occurs.  Make sense?

@shainw updates made to the validation file.

@shainw updates made to the validation file.

@shainw, the new added schema looks good to me. But, please verify as well. Thank you1


Fix typo in readme

@swift-server-bot test this please

@swift-server-bot test this please

thanks @bryanbartow

@tomerd Of course. Always happy to do the heavy lifting ;)


32 es api tunnel

Add a new Api endpoint serving Elasticsearch query language. Only open for GET requests. Thereby, the full query language of ES is available, withour needing to open the ES port, which can be dangerous in terms of index manipulation.

Please merge the commits of Volker first, or cherry-pick the last two commits.


Wrong pasteo token cause runtime panic

Describe the bug
While testing i discovered a case where the library will panic instead of throwing an error. Explicitly while attempting a wrong token "v4.local.1234".
The error is
rusty_paseto-0.6.0/src/core/paseto.rs:766:47:
range end index 32 out of range for slice of length 3

To Reproduce
just try to decrypt "v4.local.1234"
Expected behavior
i was expecting to catch the error same as other errors.
Additional context
I think this problem could be solved by checking the length of the third part. I am interested to create a PR for this as first contribution.

I was unable to reproduce this. I wrote a test to check for this but it provides an error in the result as expected. If you've got an example I can repro from I'll be happy to look into it again.


BUG: C++ wrong null value for enum

Custom nullValue specified for enum type
<enum name="Enum1" encodingType="int16" presence="optional" nullValue="10000">
    <validValue name="Val1">-1000</validValue>
    <validValue name="Val2">2005</validValue>
    <validValue name="Val3">3000</validValue>
</enum>

is not taken into account when generating C++ code:
class Enum1
{
public:

    enum Value
    {
        Val1 = (std::int16_t)-1000,
        Val2 = (std::int16_t)2005,
        Val3 = (std::int16_t)3000,
        NULL_VALUE = (std::int16_t)-32768 // <---- WRONG
    };
    ...
};

Reproduced with v1.7.4

This is not a bug. If you turn on schema validation the follow error is emitted:
[Error] :36:91: cvc-complex-type.3.2.2: Attribute 'presence' is not allowed to appear in element 'enum'.
[Error] :36:91: cvc-complex-type.3.2.2: Attribute 'nullValue' is not allowed to appear in element 'enum'.

Just like mentioned in #529, I think any example mentioned in the spec is a valid one and I as a developer would expect it to be supported by the compiler. The spec contains the following example of optional boolean:
<enum name="optionalBoolean" encodingType="uint8" presence="optional" nullValue="255" semanticType="Boolean">
    <validValue name="false">0</validValue>
    <validValue name="true">1</validValue>
</enum>

Seeing this I won't think twice before attempting something like this in my schema. Yes, I'm aware that table in section "<IP_ADDRESS>" doesn't contain the "presence" and "nullValue" properties, but still, maybe it's worth considering to be supported.

This is a mistake in the spec. I've raised an issue but have little influence over the specification.
https://github.com/FIXTradingCommunity/fix-simple-binary-encoding/issues/61


Issue with multiple path backup

If I specify two paths, Rustic seems to ignore the glob that is set in the config file. If I backup the paths separately, it works fine. If I use full paths (/home/nick/ rather than ~/) the problem persists
nick@DESKTOP-DIFRTR1:~/devilbox$ rustic -r rclone:onedrive:wsl2 backup ~/apps/cloudflare/ ~/apps/haystack-app/ --dry-run
using config /home/nick/.config/rustic/rustic.toml
[00:00:00] reading index...               ████████████████████████████████████████         16/16
[00:00:00] getting latest snapshot...     ████████████████████████████████████████         13/13
[00:00:15] backing up...                  ████████████████████████████████████████  93.88 MiB/93.88 MiB  5.98 MiB/s   (ETA 0s)                                                                                Files:       8490 new, 0 changed, 0 unchanged
Dirs:        20463 new, 0 changed, 0 unchanged
Added to the repo: 14.1 MiB (raw: 36.9 MiB)
processed 8490 files, 93.9 MiB
snapshot 00000000 successfully saved.
nick@DESKTOP-DIFRTR1:~/devilbox$ rustic -r rclone:onedrive:wsl2 backup ~/apps/haystack-app/ --dry-run
using config /home/nick/.config/rustic/rustic.toml
[00:00:00] reading index...               ████████████████████████████████████████         16/16
[00:00:01] getting latest snapshot...     ████████████████████████████████████████         13/13
[00:00:00] backing up...                  ████████████████████████████████████████   6.34 MiB/6.34 MiB   16.85 MiB/s  (ETA 0s)                                                                                Files:       199 new, 0 changed, 0 unchanged
Dirs:        7 new, 0 changed, 0 unchanged
Added to the repo: 3.9 MiB (raw: 6.4 MiB)
processed 199 files, 6.3 MiB
snapshot 00000000 successfully saved.
nick@DESKTOP-DIFRTR1:~/devilbox$ rustic -r rclone:onedrive:wsl2 backup ~/apps/cloudflare --dry-run
using config /home/nick/.config/rustic/rustic.toml
[00:00:00] reading index...               ████████████████████████████████████████         16/16
[00:00:02] getting latest snapshot...     ████████████████████████████████████████         13/13
[00:00:00] backing up...                  ████████████████████████████████████████  54.09 KiB/54.09 KiB  5.00 MiB/s   (ETA 0s)                                                                                Files:       4 new, 0 changed, 0 unchanged
Dirs:        6 new, 0 changed, 0 unchanged
Added to the repo: 16.1 kiB (raw: 56.8 kiB)
processed 4 files, 54.1 kiB
snapshot 00000000 successfully saved.

Thanks for reporting! Actually this is designed behavior. The reason is: if you backup ~/apps/cloudflare/ ~/apps/haystack-app/ as a "multi-source" it is still only possible to specify common backup option for this source. That is, you can't specify separate glob-files for both paths or let rustic respect .gitignore files for the first path but ignore them for the second path.
To specify common options, you have to give all paths in the config file like this:
source = "/home/nick/apps/cloudflare /home/apps/haystack-app"
I think this behavior needs better documentation, so I'll leave this open as docu topic.
In future we may add a mutlibackup command which basically just runs multiple backup steps in a row (generating multiple snapshots). After #411 there could then be also a --merge options to this command to directly merge the generated snapshots into one.

Would multibackup be able to use individual sources within a single config file?

Yes, definitely. This, and multiple types of sources (local FS, stdin, maybe others to come in future).
But I think if #472 is coming, the command to run all sources defined in a config file would be then rustic mutlibackup instead of the now-used rustic backup.


Not all C# Code files are yet converted to  CRLF

After the changes done in #5970 (Update .gitattributes) a lot of file changes start appearing when the files were not touched.
@crutkas Were you still converting some files? Looks like not all files in the solution are converted.
ℹ Computer information

Windows Sandbox (Build 20185)
Git version 2.28.0.windows.1

📝 Provide detailed reproduction steps (if any)

Ensure git is installed (default settings)
Ensure Visual Studio (Code) is installed (default settings)

Clone the solution
Open the solution in Visual Studio (Code)

✔ Expected result
No Modified Files
❌ Actual result
A lot of file changes (131 to be precise)
📷 Screenshots

A git diff on one of the files shows the following
 git diff --color --word-diff-regex=. src/core/Microsoft.PowerToys.Settings.UI/Views/KeyboardManagerPage.xaml.cs

diff --git a/src/core/Microsoft.PowerToys.Settings.UI/Views/KeyboardManagerPage.xaml.cs b/src/core/Microsoft.PowerToys.Settings.UI/Views/KeyboardManagerPage.xaml.cs
index 2326470..d741823 100644
--- a/src/core/Microsoft.PowerToys.Settings.UI/Views/KeyboardManagerPage.xaml.cs
+++ b/src/core/Microsoft.PowerToys.Settings.UI/Views/KeyboardManagerPage.xaml.cs
@@ -1,23 +1,23 @@
<EF><BB><BF>// Copyright (c) Microsoft Corporation[-^M-]
// The Microsoft Corporation licenses this file to you under the MIT license.[-^M-]
// See the LICENSE file in the project root for more information.[-^M-]
[-^M-]

using Microsoft.PowerToys.Settings.UI.ViewModels;[-^M-]
using Windows.UI.Xaml.Controls;[-^M-]
[-^M-]

namespace Microsoft.PowerToys.Settings.UI.Views[-^M-]
{[-^M-]
    /// <summary>[-^M-]
    /// An empty page that can be used on its own or navigated to within a Frame.[-^M-]
    /// </summary>[-^M-]
    public sealed partial class KeyboardManagerPage : Page[-^M-]
    {[-^M-]
        public KeyboardManagerViewModel ViewModel { get; } = new KeyboardManagerViewModel();[-^M-]
[-^M-]

        public KeyboardManagerPage()[-^M-]
        {[-^M-]
            InitializeComponent();[-^M-]
            DataContext = ViewModel;[-^M-]
        }[-^M-]
    }[-^M-]
}[-^M-]

Note the "[-^M-]" which mentions newline changes. (Removal of the "newline" character)

Crap :(. This should be an easy PR


Question: Usage graph for single class

Hi. I'm trying to analyze usage for single class on whole project.
It's possible to create such configuration?

You can use the filePattern option to analyze only a specific class.
Instead of filePattern: '*.php' like in the example, use filePattern: 'MyClass.php'.


Jaeger tracing

[x] Use tagged Jaeger version - f209812d
[x] Change the picture in the root readme with jaeger

Updated: this is getting to the final state. Now we are waiting for a tagged version of Jaeger, then it can be tested and merged.

@pavolloffay Tested invoking api-gw via curl, and scaling down aloha to test fallback - all displayed correctly in Jaeger UI.

Updated version of Jaeger OC template with readinessprobe.
@rafabene all other PRs have been merged, what is missing to merge this one?

PR merged


Fixed errors on build and run.

Brought up code to a working state. TLS 1.1 or higher is now required by Salesforce - forcing TLS 1.2. Updated NewtonSoft to the highest working version (8.0.3). Updated app.config with comments on where to find required values instead of including values which don't work.

Thanks for these additions!


Bug: invite modal dropdown

What happened?
The invite expiration dropdown gets cut off if it exceeds the modal height.

It is caused by the overflow being set to hidden on div.zipline-ScrollArea-root zipline-1deqy7c.
Version
upstream (ghcr.io/diced/zipline:trunk)
What browser(s) are you seeing the problem on?
No response
Zipline Logs
No response
Browser Logs
No response
Additional Info
No response

I can't seem to reproduce this issue in a fresh container using the same docker image that you specified. Are you sure you're this image: ghcr.io/diced/zipline:trunk?

If you are using ghcr.io/diced/zipline or ghcr.io/diced/zipline:latest as the docker image, it may be likely that this bug is already patched in ghcr.io/diced/zipline:trunk.

If you are using ghcr.io/diced/zipline or ghcr.io/diced/zipline:latest as the docker image, it may be likely that this bug is already patched in ghcr.io/diced/zipline:trunk.

Ah yes my bad, I am indeed using ghcr.io/diced/zipline as the image. I am still new to using GitHub and Docker so I apologise for any mistakes or confusion haha.


[Frontend] Admin Button

Describe the bug
Admin button do not need "?"
To Reproduce
Steps to reproduce the behavior:

Go to homepage

Expected behavior
Admin button doesnt need "?" and can have hovering animation and similar color to that of the home button
Screenshots

Desktop (please complete the following information):

OS: Window
Browser Edge
Version 127.0.2651.86

Have you read the Code of Conduct?
yes

It's intentional :)


How to deal with gencode gtf file

How to deal with gencode gtf file

InPACT is designed to work with annotations from RefSeq, and currently, the GENCODE GTF file format is not compatible with InPACT.

Thank you, another question is [Salmon](https://github.com/COMBINE-lab/salmon) is used to index and quantify the transcriptome, and then the usage is calculated. which fasta file is used to generate salmon index?

The FASTA file is generated from merged.gtf produced from "InPACT_transcript". Tools like gffread can be used to extract the transcript sequences from the GTF file and generate the required FASTA file.


refactor: get_headers conformance tests

Last of the conformance test refactoring.

Nice! 🙌
Would basic_query.rs fit in this new query dir?

I've moved the file in, and fixed the match len nit (x2).
I also renamed get_header.rs to be plural - this matches the message struct etc.


drivers/sensor/st: Fix wrong data byte swap for be

Remove wrong calls to sys_le16_to_cpu() in all cases in which the 16/32 bit data is built from register raw bytes with a proper bit shift operation of the MSB. In this cases performing an additional byte swapping would result in wrong data on big endian machines. This includes all data retrieved with stmemsc raw_get() API calls.
Fix #75758

I'm a little confused, it's fine if we want access to the raw numbers, but the get function should convert things to the system endianess.

I removed all the calls to sys_le16_to_cpu() when the data is already in the right system endianness. And this happens when the int16_t or int32_t are constructed from lsb/msb bytes with the shift operands. In some cases the constructions happens inside the raw_get() APIs, e.g. :
        iis3dhhc_acceleration_raw_get(data->ctx, raw_accel);
-       data->acc[0] = sys_le16_to_cpu(raw_accel[0]);
-       data->acc[1] = sys_le16_to_cpu(raw_accel[1]);
-       data->acc[2] = sys_le16_to_cpu(raw_accel[2]);
+       data->acc[0] = raw_accel[0];
+       data->acc[1] = raw_accel[1];
+       data->acc[2] = raw_accel[2];

If we do not remove the  sys_le16_to_cpu() call, then in the big endian case the data will be swapped.
Let me know if it makes sense to you.

@yperess still any doubts?

Any feedback on this fix?

@kartben do you possibly have any feedback on this fix? Thanks

@avisconti sorry, I am not really on expert on that stuff and was hoping maybe other sensor maintainers/collaborators could chime in to confirm it's the right approach. Thanks!

Even if big endian architectures are not so frequent I would like to see whether this approach is correct or not. Who can be added to the list of reviewers? @fabiobaltieri Fabio, do you know if there is anyone with some expertise on endianity? Thanks

Even if big endian architectures are not so frequent I would like to see whether this approach is correct or not. Who can be added to the list of reviewers? @fabiobaltieri Fabio, do you know if there is anyone with some expertise on endianity? Thanks

If I understand this correctly: we're ~reverting these BE/LE~ omitting these sys_le16_to_cpu conversions because the ST HAL is already taking care of it.

Yes, correct.

May we capture that in the drivers code? Personally, I wouldn't have inferred it, had I not looked at the GH Issue.

Do you mean if we can reproduce it on a big endian system? If so, unfortunately I do not have access to such h/w setup.
And yes, the endianness it is always confusing me, but I think that in this case the issue is really there.

The code changes look fine. It's clear that the byteswapping helpers aren't needed when the byte manipulation is done in the zephyr driver; what's a little less clear is when the ST HAL handles it under the hood. It would get repetitive to comment in every driver, so could you at least mention it in the commit message?

May we capture that in the drivers code? Personally, I wouldn't have inferred it, had I not looked at the GH Issue.

Do you mean if we can reproduce it on a big endian system? If so, unfortunately I do not have access to such h/w setup.

Sorry - I just meant if we could drop a comment explaining. Just what Maureen said.

As a side-note: A few months back I was able to compile/run big-endian on Linux with QEMU (using qemu-mips and mips-linux-gnu-gcc).

As a side-note: A few months back I was able to compile/run big-endian on Linux with QEMU (using qemu-mips and mips-linux-gnu-gcc).

Yeah, I was looking into something like that in fact, just to make some compile tests and clear my mind against any possible doubt. But I think that theory is pretty clear. So I will re-elaborate the commit message adding some more info as requested by Maureen and you.

Added a better commit header and copied here on this PR as well.

repushed just to give a kick to ci to run again


Bug: False format for list slicing with self.x as index

# file foo.py
a = list(range(5))
b = 2
print(a[1:b])

class Foo:
    def __init__(self):
        self.a = list(range(5))
        self.b = 2

    def bar(self):
        # The next line will be formatted to self.a[1 : self.b]
        print(self.a[1:self.b])

Running ruff format tmp.py will change the self.a[1:self.b] to self.a[1 : self.b] which is a conflict with flake8: E203. However the a[1:b] is not touched. No extra configs are needed.
$ ruff --version
ruff 0.6.5

I believe this is intended behavior. It doesn't conflict with Ruff's E203, and is in line with Black's style: https://black.readthedocs.io/en/stable/the_black_code_style/current_style.html#slices

PEP 8 recommends to treat : in slices as a binary operator with the lowest priority, and to leave an equal amount of space on either side, except if a parameter is omitted (e.g. ham[1 + 1 :]). It recommends no spaces around : operators for “simple expressions” (ham[lower:upper]), and extra space for “complex expressions” (ham[lower : upper + offset]). Black treats anything more than variable names as “complex” (ham[lower : upper + 1]).

Thank you!


Trigger on load event not working? (lazy loading)

Hi there!
I've really been having a lot of fun with Intercooler; however, I am having trouble replicating your lazy element loading example, as specified.
http://intercoolerjs.org/examples/lazyload.html
I've been able to generate a mock response by setting "ic-trigger-on" to various mouse events and "scrolled-into-view", but when I set it to "load" I get a 404 error.
Is this a bug? Are the docs out of date? For the record, I am using version #0.9.
Thanks.

If you are getting a 404 on the load action only, but other actions are working, I would guess that mockjax isn't loaded yet to handle the request when the load event is firing.
Maybe move your mockjax initialization above the the intercooler code?

Hi carsongross, thanks for the rapid response!
I looked into that, but I already am loading mockjax before intercooler (using headjs which ensures that).
Here is the relevant HTML:

Here is the relevant JS:
if (window.Intercooler !== undefined && window.jQuery.mockjax !== undefined) {
					          console.log("Intercooler and mockjax are ready!");
					          $.mockjax({
						            url: "/about/eliot/1",
                        responseTime:500,
						            responseText: "Available for hire.",
                        response: function(settings) {
                            console.log("bio1");
                        }
					          });
                    //next
                    $.mockjax({
						            url: "/biograph",
                        responseTime:500,
						            responseText: ""
					          });
				        }

In your estimation, does anything overtly wrong stand out?

Can you ensure that this code:
$.mockjax({
url: "/biograph",
responseTime:500,
responseText: ""
});
}
is run before the load event is triggered?  Make sure it is not in a jQuery onLoad callback.  Maybe just dump it into a script tag directly on the page:

Given that the other trigger actions are working, it seems like it has to be a timing issue.
Cheers,
Carson
On December 28, 2015 at 4:04:31 PM, Chris<EMAIL_ADDRESS>wrote:
Hi carsongross, thanks for the rapid response!
I looked into that, but I already am loading mockjax before intercooler (using headjs which ensures that).
Here is the relevant HTML:

Here is the relevant JS:
if (window.Intercooler !== undefined && window.jQuery.mockjax !== undefined) {
console.log("Intercooler and mockjax are ready!");
$.mockjax({
url: "/about/eliot/1",
responseTime:500,
responseText: "Available for hire.",
response: function(settings) {
console.log("bio1");
}
});
//next
$.mockjax({
url: "/biograph",
responseTime:500,
responseText: ""
});
}
In your estimation, does anything overtly wrong stand out?
—
Reply to this email directly or view it on GitHub.

Sorry, could you clarify "jQuery onLoad callback"? Thanks.

Ah, sorry, I'm being loose with my language.  So the "load" action is fired once the page has loaded, using the jQuery "ready" handler:
https://api.jquery.com/ready/
Are you putting that $.mockjax({...}) code inside a jQuery ready handler, or is it in a top level javascript tag?
Given that things are working when you use other events, my guess is that the
$.mockjax({...})
code is executing after intercooler is firing the request, so mockjax isn't catching it, leading to the 404 (since a real ajax request is being made.)
Does that make sense?

Yes, that does make sense! Thank you @carsongross . I will report back once I've fully heeded your advice.

Hi Carson, I appreciate you working with me here.
Perhaps there is some subtlety I am not grasping?
Here is what I'm working with now:
    (function () {
        head.ready( function () {
        if (window.jQuery.mockjax !== undefined && window.Intercooler
!== undefined) {
            //
            console.log("jQuery is ready for mockjax!");
            console.log(document.readyState === 'complete');
            $.mockjax({
	  	          url: "/biograph",
                responseTime:500,
	  	          responseText: ""
	          });
        }
        });
    })();

console.log(document.readyState === 'complete') yields false as
desired; that is the function is fired before the 'ready' event is
fired. Still, I am getting a 404 error.
Mouseover, etc work in this case as well.
Thoughts?
On 12/28/15 7:35 PM, Carson Gross wrote:

Ah, sorry, I'm being loose with my language. So the "load" action is
fired once the page has loaded, using the jQuery "ready" handler:
https://api.jquery.com/ready/
Are you putting that $.mockjax({...}) code inside a jQuery ready
handler, or is it in a top level javascript tag?
Given that things are working when you use other events, my guess is
that the
$.mockjax({...})
code is executing /after/ intercooler is firing the request, so mockjax
isn't catching it, leading to the 404 (since a real ajax request is
being made.)
Does that make sense?
—
Reply to this email directly or view it on GitHub
https://github.com/LeadDyno/intercooler-js/issues/70#issuecomment-167684712.

Ah, so that’s the problem: you are initialzing the mockjax handler in the “ready” function of the head element.
You need to move that code out to a top level script block:

so that it will execute before all the load events do.
Cheers,
Carson
On December 28, 2015 at 5:23:29 PM, Chris<EMAIL_ADDRESS>wrote:
Hi Carson, I appreciate you working with me here.
Perhaps there is some subtlety I am not grasping?
Here is what I'm working with now:
(function () {
head.ready( function () {
if (window.jQuery.mockjax !== undefined && window.Intercooler
!== undefined) {
//
console.log("jQuery is ready for mockjax!");
console.log(document.readyState === 'complete');
$.mockjax({
url: "/biograph",
responseTime:500,
responseText: ""
});
}
});
})();

console.log(document.readyState === 'complete') yields false as
desired; that is the function is fired before the 'ready' event is
fired. Still, I am getting a 404 error.
Mouseover, etc work in this case as well.
Thoughts?
On 12/28/15 7:35 PM, Carson Gross wrote:

Ah, sorry, I'm being loose with my language. So the "load" action is
fired once the page has loaded, using the jQuery "ready" handler:
https://api.jquery.com/ready/
Are you putting that $.mockjax({...}) code inside a jQuery ready
handler, or is it in a top level javascript tag?
Given that things are working when you use other events, my guess is
that the
$.mockjax({...})
code is executing /after/ intercooler is firing the request, so mockjax
isn't catching it, leading to the 404 (since a real ajax request is
being made.)
Does that make sense?
—
Reply to this email directly or view it on GitHub
https://github.com/LeadDyno/intercooler-js/issues/70#issuecomment-167684712.

—
Reply to this email directly or view it on GitHub.

Closing, we are handling offline.


Proposal: Ability to avoid certain modules/files/folders during debugging process - "smart code stepping"

VSCode Version: 1.1.0
OS Version: OS X

Could it be possible to blacklist certain modules (let's say given libraries within node_modules from showing up during the Step Over / In / Out debugging process?
For example, let's say Library -Z- is a common one through out your Node application, but it's very low-level, it would be incredible to have some sort of Flag to say "Skip during debugging process" for the entire (lets say) folder, that way you could continue (F10 etc) and never end up in those files / locations.
Just an idea, I'm not sure how feasible it is.

@MarkPieszak VS Code 1.1 supports something like this already: see section "Smart code stepping" in https://code.visualstudio.com/updates#_nodejs-debugging
Currently this only works for a very specific use case, but it should be possible to extend this to your scenario as well.

Thanks @weinand I'm looking at it right now, you guys are one step ahead of me, glad it's at least in the works! :+1: I'll close this out since it's already on your guys radar.

@MarkPieszak let's keep this feature request open until your scenario is actually supported...

Sounds good to me, I can keep testing it as features get released as well.

@MarkPieszak great! I'll let you know.

just to keep things connected:  #3215

This feature has been implemented in November 2016.


Cache downloadBoost file

Description
When I build my project for release — every time it downloads boost
Reproduction Steps and Sample Code
# react-native/ReactAndroid/build.gradle
task downloadBoost(dependsOn: createNativeDepsDirectories, type: Download) {
    // Use ZIP version as it's faster this way to selectively extract some parts of the archive
    src 'http://mirror.nienbo.com/boost/1.57.0/boost_1_57_0.zip'
    // alternative
    // src 'http://mirror.nienbo.com/boost/boost_1_57_0.zip'
    onlyIfNewer true
    overwrite false
    dest new File(downloadsDir, 'boost_1_57_0.zip')
}

./gradlew assembleRelease
:ReactAndroid:downloadBoost
Solution
It will be cool to cache downloaded files
Additional Information

React Native version: 0.33.0
Platform: Android
Development Operating System: Mac
Dev tools: Terminal

I'm tired of downloading a file

Building 82% > :ReactAndroid:downloadBoost > 34,11 MB/105,37 MB downloaded

It's not clear to me, from the issue description, what sort of impact this has and why it should be changed. Seems like if downloading this file repeatedly is a problem, a simple PR can be provided to cache the file.

I think that this issue is resolved now.
After deleting the downloaded boost file (at the time of writing, on master this is '/react-native/ReactAndroid/build/downloads/boost_1_63_0.tar.gz'), repeated executions of this task will realize that it's already been downloaded.
On clean run:
 ./gradlew :ReactAndroid:downloadBoost
Incremental java compilation is an incubating feature.
:ReactAndroid:createNativeDepsDirectories UP-TO-DATE
:ReactAndroid:downloadBoost
Invalid cookie header: "Set-Cookie: logged_in=no; domain=.github.com; path=/; expires=Sun, 04 Oct 2037 11:18:53 -0000; secure; HttpOnly". Invalid 'expires' attribute: Sun, 04 Oct 2037 11:18:53 -0000
Download https://github.com/react-native-community/boost-for-react-native/releases/download/v1.63.0-0/boost_1_63_0.tar.gz

BUILD SUCCESSFUL

Total time: 33.848 secs

After first run:
./gradlew :ReactAndroid:downloadBoost
Incremental java compilation is an incubating feature.
:ReactAndroid:createNativeDepsDirectories UP-TO-DATE
:ReactAndroid:downloadBoost UP-TO-DATE

BUILD SUCCESSFUL

Total time: 4.943 secs

Head at: eae4fe810f43266ee54a3bb89558621324d7a326

This was fixed recently.


Incorrect color contrast on vim with st

I'm using vim 8.1 (with +termguicolors) inside st under dwm. I applied the nord theme to st. I installed nord trough Plug as recommended.
But for some reason some colors are incorrect inside vim... For instance as you can see on the screenshots line numbers and comments color are to close to the background color.
More, there is now visual selection color at all. Everything else looks ok trough...

Hi @loustak :wave:, please note that the linked st theme is not an official theme but a contribution of a Nord user. I've not checked if the used colors are matching Nord's styles for syntax highlighting so your problem might be related to this.
I've quickly checked the proposed theme and the adjustment of Nord's comment color is not included. This explains why comments and the line numbers using a darker color ( nord3) instead of the new #616e88 value ( nord3 brightness increased by 10%).
Also another problem might be that you Vim and terminal is not configured to use true colors. Can you please try to add the following config to your .vimrc and check if the problem still occurs:
if (has("termguicolors"))
  set termguicolors
endif

Also please make sure you're running at least Nord Vim v0.10.0 or higher, otherwise the new comment color is also not supported.

Thanks for the answer @arcticicestudio. I tried changing the color inside my st configuration to the new nord3 color. No changes.
I checked my vim and tried set termguicolors it make all the text goes white:

I checked the nord vim version inside nord.vim installed by plug, and I'm using v0.12.0:
let s:nord_vim_version="0.12.0"

I found a solution: https://github.com/vim/vim/issues/993#issuecomment-255651605
Adding the following lines to my .vimrc solved all my colors problems:
" set Vim-specific sequences for RGB colors
let &t_8f = "\<Esc>[38;2;%lu;%lu;%lum"
let &t_8b = "\<Esc>[48;2;%lu;%lu;%lum"

Did the same thing, however I still can't get the current line highlight to work.
Maybe that requires a different sequence?

@DictumMortuum you are right. I didn't noticed it first but I have the exact same issue. So I reopen it.

Has someone found a solution to also enable highlight support for the current cursor line?
I haven't installed and configured st so having some help here would be nice to resolve this ticket.

I'd be happy to help, but I'm not sure what I should do - do you want me to try anything?

Does it work if you're disabling italic font rendering in both st and the specific Nord theme configuration? Might be possible that there are conflicts with the terminal sequences.

Hi !
St + vim works perfectly for me with this configuration. :wink:
St + Vim

St + Tmux + Vim

" ~/.vimrc

if exists('+termguicolors') && ($TERM == "st-256color" || $TERM == "tmux-256color")
	let &t_8f = "\<Esc>[38;2;%lu;%lu;%lum"
	let &t_8b = "\<Esc>[48;2;%lu;%lu;%lum"
	set termguicolors
endif

let g:nord_italic = 1
let g:nord_italic_comments = 1
let g:nord_underline = 1
let g:nord_uniform_status_lines = 1
let g:nord_uniform_diff_background = 1
let g:nord_cursor_line_number_background = 1
colorscheme nord

# ~/.tmux.conf

# $TERM, fix colors
set -g default-terminal "tmux-256color"
set -ga terminal-overrides ",*256col*:Tc"

Source : https://github.com/tmux/tmux/issues/1246#issue-292083184

For my case, I'm not running vim inside tmux.
@cyrilaugier can you provide the versions of st, vim and nord please? Because I have the same setup without tmux and I don't get the current line highlighted.

@cyrilaugier Thanks, this helps a lot to encircle the root cause :+1:
@loustak I guess I've also misunderstood your actual problem: I thought you're having wrong styles for the current line highlighting (like e.g. too dark or totally wrong color), but I guess this is also just a Vim setup problem on your side.
Have you enabled Vim's cursorline option? Otherwise the current line won't be highlighted at all :smile:
You can add set cursorline to your vimrc to enable it by default or run the command from within Vim.

Okay, I'm stupid, I just forgot to add the cursorline option...
The issue seems close now.

@loustak Not stupid, human. I could tell you a lot of private or job-related development/coding situations   where I was like "wow, the fix was a one-liner" after spending hours with debugging :upside_down_face:
Glad to see it works for your now.
@DictumMortuum Could you please also check if you're just missing the cursorline option? Otherwise we need to check other settings that might conflict with the theme.

Currently on vacation, but I believe that I don't have the cursor line
option enabled. Will check and update you as soon as possible !
 I could tell you
a lot of private or job-related development/coding situations where I was
like "wow, the fix was a one-liner" after spending hours with debugging 🙃
Glad to see it works for your now.
@DictumMortuum https://github.com/DictumMortuum Could you please also
check if you're just missing the cursorline option? Otherwise we need to
check other settings that might conflict with the theme.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/arcticicestudio/nord-vim/issues/166?email_source=notifications&email_token=AAR7GLE7BY6XI4H74GKX6NLQBKGXFA5CNFSM4H3LCV6KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD23SQNQ#issuecomment-515319862,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAR7GLDQS6VAAOHTX7MM2ILQBKGXFANCNFSM4H3LCV6A
.

Confirmed, I'd forgot to set the cursorline option. I think that we can close!


[Accessibility] Ctrl+M (editor.action.toggleTabFocusMode) does not work inside terminal input field

CC @meganrogge
Reproducible Steps

Load screen reader.

Open terminal.

Focus in terminal input area.

Press Ctrl+M to toggle tab-key move (editor.action.toggleTabFocusMode).

Notes: I don't hear any change. It does not seem to work.

Expected Behavior
Pressing Ctrl+M key needs to toggle the tab-key move mode.
System Info
Version: 1.75.0-insider (system setup)
Commit: f1b09237cd400d92987cf7a9681995e8011ef54c
Date: 2023-01-23T11:53:59.088Z
Electron: 19.1.9
Chromium: 102.0.5005.194
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22621
Sandboxed: Yes

This works for me when I add editor.action.toggleTabFocusMode to terminal.integrated.commandsToSkipShell, so we can do that

On macOS, this works for me. It's part of our default terminal.integrated.commandsToSkipShell so surprising that it wouldn't work. will try windows

Yep does not work on windows

the logs show that the keybinding works and  Invoking command editor.action.toggleTabFocusMode. but then it doesn't happen
2023-01-25 06:25:58.152 [info] [KeybindingService]: / Received  keydown event - modifiers: [ctrl], code: ControlLeft, keyCode: 17, key: Control
2023-01-25 06:25:58.152 [info] [KeybindingService]: | Converted keydown event - modifiers: [ctrl], code: ControlLeft, keyCode: 5 ('Ctrl')
2023-01-25 06:25:58.153 [info] [KeybindingService]: \ Keyboard event cannot be dispatched in keydown phase.
2023-01-25 06:25:58.301 [info] [KeybindingService]: / Soft dispatching keyboard event
2023-01-25 06:25:58.302 [info] [KeybindingService]: | Resolving ctrl+M
2023-01-25 06:25:58.304 [info] [KeybindingService]: \ From 1 keybinding entries, matched editor.action.toggleTabFocusMode, when: no when condition, source: built-in.
2023-01-25 06:25:58.306 [info] [KeybindingService]: / Received  keydown event - modifiers: [ctrl], code: KeyM, keyCode: 77, key: m
2023-01-25 06:25:58.306 [info] [KeybindingService]: | Converted keydown event - modifiers: [ctrl], code: KeyM, keyCode: 43 ('M')
2023-01-25 06:25:58.307 [info] [KeybindingService]: | Resolving ctrl+M
2023-01-25 06:25:58.307 [info] [KeybindingService]: \ From 1 keybinding entries, matched editor.action.toggleTabFocusMode, when: no when condition, source: built-in.
2023-01-25 06:25:58.308 [info] [KeybindingService]: + Invoking command editor.action.toggleTabFocusMode.
2023-01-25 06:25:58.447 [info] [KeybindingService]: / Soft dispatching keyboard event
2023-01-25 06:25:58.448 [info] [KeybindingService]: | Resolving ctrl+M
2023-01-25 06:25:58.449 [info] [KeybindingService]: \ From 1 keybinding entries, matched editor.action.toggleTabFocusMode, when: no when condition, source: built-in.

it only gets run when there's an editor view model, which there isn't when the terminal is focused
https://github.com/microsoft/vscode/blob/fc5d2e8d7ef21179bf86c79a7755b426b8a3baea/src/vs/editor/browser/coreCommands.ts#L34-L46


GPU Flow - color convert - RGB output cases

@LakshmiKumar23 ready to merge ?

@LakshmiKumar23 ready to merge ?

@kiritigowda Yes, it can be merged


chore: upgrade to latest controller deps

Upgrade to latest controller and dependencies.
Upgrades tonic to the latest version.
Description

Motivation and Context

Regression

How Has This Been Tested?

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added unit tests to cover my changes.

bors merge


Forces the tests for calendar date conversion to use UTC instead of system time.

This is a fix for SR-171.

looks good to me, as a good rule of thumb for the darwin foundation tests we prefer to have local machine independent tests if at all possible (unless we are testing the local machine setting of course)


Mysql container not starting due to change in mysql 8.4

Description / Beschreibung
In mysql 8.4 he "MySQL Native Password" plugin is no longer enabled by default. The mysql container fails to start.
Affected versions / Verwendete Versionen
mysql:8 pointing to mysql 8.4
For me the solution was to change in docker-compose.yml:
command: --default-authentication-plugin=mysql_native_password
to
command: ["mysqld", "--mysql-native-password=ON"]

Hey @rotegras, thanks for the hint! I’m currently not very deep into the topic, but maybe we can completely remove the usage of the default authentication plugin. As far as I remember, we had to use it at the beginning of MySQL 8.0.x for compatibility reasons?

I've created a PR. #46  that does the same

Fixed with #46


removed screenshots from tutorials and restructured hands-on from non-dip

variant-analysis/non-dip
@bebatut

Hey @wm75 would you have time to check this today during the co-fest?

This cannot get merged as is since some of the boxes replacing the images are insufficient.
I will work on completing them, and leave this PR open until I'm done.

Sounds good @wm75, tagging as work-in-progress until then :)

@wm75 This PR is quite old, and has accumulated a fair number of conflicts now, what would you like to do with it? Happy to leave it open if still useful, but if not then maybe we close it?

@wm75 @polkhe my gentle annual reminder about this very old PR, let me know what you think, are you still willing and able to finish it? is it still relevant? should we close it? up to you, but let me know :)

looks like most of what this PR was aiming to do is already present in the main branch, so closing this. Thanks!


[tool_crash] FileSystemException: Failed to decode data using encoding 'utf-8', null

Command
flutter pub get

Steps to Reproduce

...
...
...

Logs
FileSystemException: Failed to decode data using encoding 'utf-8', null
#0      _File._tryDecode (dart:io/file_impl.dart:564:7)
#1      _File.readAsStringSync (dart:io/file_impl.dart:584:7)
#2      ForwardingFile.readAsStringSync (package:file/src/forwarding/forwarding_file.dart:96:16)
#3      ErrorHandlingFile.readAsStringSync.<anonymous closure> (package:flutter_tools/src/base/error_handling_io.dart:222:22)
#4      _runSync (package:flutter_tools/src/base/error_handling_io.dart:597:14)
#5      ErrorHandlingFile.readAsStringSync (package:flutter_tools/src/base/error_handling_io.dart:221:12)
#6      new SettingsFile.parseFromFile (package:flutter_tools/src/base/utils.dart:145:36)
#7      updateLocalProperties (package:flutter_tools/src/android/gradle_utils.dart:231:29)
#8      AndroidProject.ensureReadyForPlatformSpecificTooling (package:flutter_tools/src/project.dart:534:12)
#9      FlutterProject.ensureReadyForPlatformSpecificTooling (package:flutter_tools/src/project.dart:352:21)
<asynchronous suspension>
#10     PackagesGetCommand.runCommand (package:flutter_tools/src/commands/packages.dart:175:5)
<asynchronous suspension>
#11     FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:1209:27)
<asynchronous suspension>
#12     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#13     CommandRunner.runCommand (package:args/command_runner.dart:209:13)
<asynchronous suspension>
#14     FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:281:9)
<asynchronous suspension>
#15     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#16     FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:229:5)
<asynchronous suspension>

[✓] Flutter (Channel stable, 3.3.4, on Microsoft Windows [version 10.0.22000.978], locale fr-BE)
    • Flutter version 3.3.4 on channel stable at C:\Users\Hleysonn\OneDrive\Bureau\N_Flutter\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision eb6d86ee27 (7 days ago), 2022-10-04 22:31:45 -0700
    • Engine revision c08d7d5efc
    • Dart version 2.18.2
    • DevTools version 2.15.0

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at C:\Users\Hleysonn\AppData\Local\Android\sdk
    • Platform android-33, build-tools 33.0.0
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 11.0.12+7-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[✗] Visual Studio - develop for Windows
    ✗ Visual Studio not installed; this is necessary for Windows development.
      Download at https://visualstudio.microsoft.com/downloads/.
      Please install the "Desktop development with C++" workload, including all of its default components

[✓] Android Studio (version 2021.2)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+7-b1504.28-7817840)

[✓] VS Code (version 1.72.1)
    • VS Code at C:\Users\Hleysonn\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.50.0

[✓] Connected device (4 available)
    • Android SDK built for x86 (mobile) • emulator-5554 • android-x86    • Android 10 (API 29) (emulator)
    • Windows (desktop)                  • windows       • windows-x64    • Microsoft Windows [version 10.0.22000.978]
    • Chrome (web)                       • chrome        • web-javascript • Google Chrome 106.0.5249.103
    • Edge (web)                         • edge          • web-javascript • Microsoft Edge 106.0.1370.37

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

Flutter Application Metadata
Type: app
Version: 1.0.0+1
Material: true
Android X: false
Module: false
Plugin: false
Android package: null
iOS bundle identifier: null
Creation channel: stable
Creation framework version: fb57da5f945d02ef4f98dfd9409a72b7cce74268
Plugins
cloud_firestore-3.5.1
cloud_firestore_web-2.8.10
firebase_auth-3.11.2
firebase_auth_web-4.6.1
firebase_core-1.24.0
firebase_core_web-1.7.3
firebase_storage-10.3.11
firebase_storage_web-3.3.9
flutter_plugin_android_lifecycle-2.0.7
flutter_stripe-4.0.0
google_sign_in-5.4.2
google_sign_in_android-6.1.1
google_sign_in_ios-5.5.0
google_sign_in_web-0.10.2
image_picker-0.8.6
image_picker_android-0.8.5+3
image_picker_for_web-2.1.10
image_picker_ios-0.8.6+1
path_provider-2.0.11
path_provider_android-2.0.20
path_provider_ios-2.0.11
path_provider_linux-2.1.7
path_provider_macos-2.0.6
path_provider_windows-2.0.7
stripe_android-4.0.0
stripe_ios-4.0.0

Closing this as a duplicate of https://github.com/flutter/flutter/issues/65021.
Kindly follow up in the issue linked above for updates. Please upvote the issue linked above as well.
Thank you


nginx validation enabled without interactive mode

Changes:

Enabled nginx image validation without interactive mode

Type of change

[ ] Bug fix
[ ] New feature
[ ] Update feature
[ ] Refactor code
[ ] Translation to code
[ ] Translation to crowdin
[ ] Script configuration
[ ] Improve performance
[ ] Style only
[ ] Dependency update
[ ] Documentation update
[ ] Release

closing this PR as its no longer needed


Laravel npm run watch error while refreshing Vite

I'm using the latest Laravel 9 that comes with Vite, instead of Mix. Vite works much faster than Mix but there's one very annoying issue.
In package.json I've the following line, so I can use the regular 'npm run watch' as I had on Mix. Here's my code:
"scripts": {
    "dev": "vite",
    "build": "vite build",
    "watch": "vite build --watch"
},

I work on production server, so I don't need 'npm run dev' and instead of that I'm using 'npm run build'. So far so good, now the issue:
When I write 'npm run watch' while the process is running, I'm getting the error on the browser:
    Vite manifest not found at:
/home/username/public_html/cms/public/build/manifest.json

The marked error is this line:
@vite(['resources/sass/app.scss', 'resources/js/app.js'])
When the process is finished, the error is gone and the changes are applied.
If I change something in my CSS again, the same error appears while it's processing.
When I was using Mix in my other projects there wasn't any error when I write "npm run watch", I just wait for the changes to apply without any error while processing.
Is there any solution for this?

This is unrelated to this package, you seem to use the official Vite integration. See their own  docs: https://laravel.com/docs/9.x/vite#main-content
I think you can probably solve your issue by setting this option in vite.config.ts though.

Hello,
Thank you for your answer. I've tried your suggestion but the issue still remains.
Here's my code of vite.config.js:
import { defineConfig } from 'vite';
import laravel from 'laravel-vite-plugin';

export default defineConfig({
    plugins: [
        laravel({
            input: [
                'resources/sass/app.scss',
                'resources/js/app.js',
            ],
            build: {
                emptyOutDir: false,
            },
            refresh: true,
        }),
    ],
});

Thank you very much for your help, that actually fixed the issue!
I'm sorry for posting in the wrong repository but at least there's a right person to help here. :)


Dev eene

practice lecture2 on 01-23-2024

My GitHub interface is different from what I see in the recorded lecture from 01-23-2024


[AutoPR machinelearningservices/resource-manager] [Do Not Merge] SSH and Quota changes

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/6432

This PR has been merged into https://github.com/Azure/azure-sdk-for-go/pull/5601


Add main declaration to package.json

Added main declaration pointing to bootstrap-select entry point to allow bundling via browserify. This will enable the common vendor browserify recipe.

+1

The main declaration already exists in package.json.


[BUG] window can be null when calling Zotero.getMainWindow()

https://github.com/windingwind/zotero-plugin-toolkit/blob/a6a90697788a10a3f11e22372dceef7ba076cf70/src/basic.ts#L120-L130
L129 may return null when the main window has been unloaded.
It will cause a Type Error in the invoker, e.g., ztoolkit.log().

Thanks for the feedback.


Signup: Errors even if successfully signed up

Reproducing: Go to /login/signup and input a new, unique screenname & username.
Expected: Redirected to login screen
Actual: Error "This user profile already exists"

Solved in 04cb815


snapshotCanvas function bug in CanvasRenderer

Version

Phaser Version:  3.55.2

Operating system: doesn't matter

Browser: doesn't matter

Description

Trying to get a snapshot from the container, when using Canvas Renderer. There seems a reference issue.
    snapshotCanvas: function (canvas, callback, getPixel, x, y, width, height, type, encoderOptions)
    {
        if (getPixel === undefined) { getPixel = false; }

        this.snapshotArea(x, y, width, height, callback, type, encoderOptions);

        var state = this.snapshotState;

        state.getPixel = getPixel;

        CanvasSnapshot(this.canvas, state);

        state.callback = null;

        return this;
    },

The function uses CanvasSnapshot(this.canvas, state); but there is no this.canvas property in there, there is canvas argument, that I think might be there or this.gameCanvas could be placed instead of this.canvas.
Example Test Code

  protected async generateImage(): Promise<
    Phaser.Display.Color | HTMLImageElement
  > {
    return new Promise<Phaser.Display.Color | HTMLImageElement>((resolve) => {
      const texture = this.add.renderTexture(
        0,
        0,
        this.gameObject.width,
        this.gameObject.height,
      );
      texture.draw(this.gameObject);
      texture.snapshot(resolve, 'image/png', 1);
    });
  }
}

Additional Information

@samme thanks for the fast fix. Is there any information on when this will be available?

I don't know but you can use:

phaser-arcade-physics.js.zip
phaser-arcade-physics.min.js.zip
phaser-facebook-instant-games.js.zip
phaser-facebook-instant-games.min.js.zip
phaser-ie9.js.zip
phaser-ie9.min.js.zip
phaser.js.zip
phaser.min.js.zip

Thank you for submitting this issue. We have fixed this and the fix has been pushed to the master branch. It will be part of the next release. If you get time to build and test it for yourself we would appreciate that.


XKRX test_calculated_against_old_csv redundant?

Hi @elbakramer, I'm migrating the test modules to a new base and have come across XKRXCalendarTestCase.test_calculated_against_old_csv which is obviously specific to XKRX.
Is there any reason for continuing to test a slice of the schedule against the old csv even though the full schedule now equates with the new xkrx.csv? Or can this test and xkrx_old.csv now be removed?
Thanks.

#96 removed XKRXCalendarTestCase.test_calculated_against_old_csv and xkrx_old.csv.


fix(extension): send balance UI no longer glitches when input field loses focus

Checklist

[X] JIRA - <link>
[] Proper tests implemented
[ ] Screenshots added.

Proposed solution
Currently in Lace when the input field loses focus (blur) the UI re-renders with the review-transaction greyed disabled for a split second, If the user changes focus by clicking the button, it needs to click twice. This is caused by a specific flag that checks whether the max ADA available was already computed to be used in a specific button of the UI. I removed this check, now the UI re-renders with the button enabled. Removing this check doesn't seems to have any negative effect in the MAX ADA functionality.
The interface now doesn't require a double click on the review button
Testing
Describe here, how the new implementation can be tested.
Provide link or briefly describe User Acceptance Criteria/Tests that need to be met
Screenshots
Attach screenshots here if implementation involves some UI changes

Allure Report
allure-report-publisher generated test report!

processReports: ✅ test report for bd80413c

passed
failed
skipped
flaky
total
result

Total
32
0
4
0
36
✅


Asterisk escaping in (relative or absolute) URLs doesn't work

E.g.:

var s83 = "Essai 57 : asterisk within a URL is not encoded, example/some/**/\*search-phrase\*. 37e ***chose importante***."
var s83 = "Essai 57 : asterisk within a URL is not encoded, http://example/some/**/\*search-phrase\*. 37e ***chose importante***."
var md = window.markdownit();
md.render(s83);

Result: search-phrase
Expected: in CM reference impl, for instance, these escapes work, & search-phrase is surrounded by asterisks & not emphasized.

That's expected behaviour for autolinks extension, not related to commonmark. If you need correct work for such case - use explicit markup as in spec. See previous closed issues with explanations.
PS. Please, use online demos for examples, when possible, as said in issue template.

Just for the record, from the PoV of the considerations raised in the OP, commonmark.js works better (\ escaping as expected).
P.S. While markdown-it is slightly smaller in size, which some could consider as an advantage.

This compare us not relevant, because commonmark.js does NOT supports linkification (where this problem exists in md-it). If you think different - please provide samples via online demos (markdown-it and dingus).
Those package have completely different targets. While commonmark,js provides rigid reference implementation, markdown-it provides spec implementation + flexibility (extendability). In this scope size means nothing.


[Update Snap] Aleph Zero

Checklist
All items in the list below needs to be satisfied.

[x] Is the summary of the change documented in this ticket?
[x] Has a MetaMask Snaps team member reviewed whether the changes need to be vetted?
[x] If there are changes that need to be vetted, attach a description and the relevant fixes/remediations to this issue.
[ ] The corresponding pull request in this repo has been merged.

Summary of changes:
Updating the snap icon to a higher resolution (and hence, bumping the snap version in the registry). Updating the snap website URL (pointing to the wallet integration).
PENDING query from the Builder.


Try LE Secure connections first before falling back

This PR fixes #12
It turns out that some hearing aids will allow bonding without LE Secure connections, but then refuse to grant access to the ASHA GATT characteristics.
Pico-ASHA now attempts bonding with LE Secure Connections first, and then disables LE SC as a fallback if it fails.
The prior behavior was the opposite, it upgraded to LE SC if bonding failed.

Sigh... this (or something else I've done) seems to have broken reconnections when moving in/out of range.
Having to manage the whole pairing/bonding process is an absolute PITA :(

Is there a command so that we can choose which devices to connect to? One apparent issue that happens is that devices like Resound advertise two different bluetooth addresses. I'm not sure what the difference is but one is named GN and one is  "User's hearing aids". I'll try checking into it myself too

Hi @sk408 could you please open a new issue to discuss this? Thank you.

Ok, am going to merge this as-is. I have another branch I'm working on that I'm hoping to implement various improvements and fixes.


(#104) Add Leaderboard Functionality

Description
This PR adds Leaderboard functionality, similar to Redis Leaderboard Overview.
The leaderboard is implemented using a Sorted Set backed by a SkipList implementation.
Instead of storing the actual strings for keys, we store pointers to these keys. Similarly for the values, we only store pointers to the actual values stored in the store. This allows us to keep the data structure space efficient.
We also need to make sure that the KV pairs stored in the ZSet do not collide with the KV pairs in the general store.

[x] Implement SkipList
[x] Implement SkipList-backed Zset (SortedSet)
[x] Add ZSet Tests
[x] Add Benchmark
[ ] Add user-facing API with key/value refs

Issue
#104
How was the patch tested?
Added unit tests + existing tests


verible: 0.0.3471 -> 0.0.3515

Description of changes
Update Verible to latest. A few fixes in particular in the language server.
For the nix package: we don't need the patch anymore as this is fixed upstream.

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

I've attempted to see if Darwin would work, but it gets some ominous LIBTOOL issue. Possibly something to do with the default toolchain that bazel is using there.

https://logs.ofborg.org/?key=nixos/nixpkgs.285692&attempt_id=8d03840e-f5ed-4bc5-8a37-5c69ce12da77
https://logs.ofborg.org/?key=nixos/nixpkgs.285692&attempt_id=6104eda3-10a9-4fe5-924e-564c7366cada

I suppose bazel/nix folks work on that separately, so for now, let's keep Darwin-Build disabled (  @aherrmann, you are working on bazel related things, right ? Is this expected, or could there be an easy workaround ? )

Result of nixpkgs-review pr 285692 run on x86_64-linux 1

1 package built:

verible