Support text alignment 'justify' in BitmapText

Description of change
Add 'justify' feature to bitmap texts. This aligns the text both right and left using varying spaces between words
roundPixels = true, is recommended for small font sizes.
Pre-Merge Checklist

[x] Tests and/or benchmarks are included
[x] Documentation is changed or added - only inside code
[x] Lint process passed (npm run lint)
[x] Tests passed (npm run test)

TestJustify.zip

Codecov Report

Merging #6968 into dev will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               dev     #6968   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           15        15
  Lines          671       671
=========================================
  Hits           671       671

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 a4a408b...30612c1. Read the comment docs.

@bigtimebuddy ,
In fact, I was looking at the standard text as well, to see if I can add it there as well, but it is a bit more complicated as the text is not placed char-by-char, but full lines. Playing with that is more tricky, and I don't want to mess it up.
shai

No worries, that's a more complicated task, for sure.


Removes rkt from list of orchestrators

Description

As per @ahrkrak , we don't support this (particularly in v3) and OK to remove.

Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note

None required

@emanic the integration branch has been merged to master - please re-open this against the master branch.


iesvirgendelcarmen.com

Institution/School Name
IES Virgen del Carmen (Jaén)
Instiution/School Website
https://www.iesvirgendelcarmen.com
Vocational Training Moodle (Computer Science)
https://www.iesvirgendelcarmen.com/moodle

@rafaies Pull request merged. Thank you!


Any use case, to use this over require?

Just curious, maybe I should know something that's better than what I use right now, require?

From the readme:

Import a module like with require() but from a given path

Instead of importing relative to the current file, you can import relative to any file. So you can for example import a module relative to the user's current directory.
Example: https://github.com/sindresorhus/atom-esformatter/blob/b3c939215993f3c53ec66377ed6b631c025a97b2/index.js#L18


Updating location parameter for playbooks

Required items, please complete
Change(s):

See guidance below

Reason for Change(s):

See guidance below

Version Updated:

Required only for Detections/Analytic Rule templates
See guidance below

Testing Completed:

See guidance below

Checked that the validations are passing and have addressed any issues that are present:

See guidance below

Guidance <- remove section before submitting

Before submitting this PR please ensure that you have read the following sections and filled out the changes, reason for change and testing complete sections:
Thank you for your contribution to the Microsoft Sentinel Github repo.

Details of the code changes in your submitted PR.  Providing descriptions for pull requests ensures there is context to changes being made and greatly enhances the code review process.  Providing associated Issues that this resolves also easily connects the reason.

Change(s):

Updated syntax for XYZ.yaml

Reason for Change(s):

New schema used for XYZ.yaml
Resolves ISSUE #1234

Version updated:

Yes
Detections/Analytic Rule templates are required to have the version updated

The code should have been tested in a Microsoft Sentinel environment that does not have any custom parsers, functions or tables, so that you validate no incorrect syntax and execution functions properly.  If your submission requires a custom parser or function, it must be submitted with the PR.

Testing Completed:

Yes/No/Need Help

Note: If updating a detection, you must update the version field.

Before the submission has been made, please look at running the KQL and Yaml Validation Checks locally.
https://github.com/Azure/Azure-Sentinel#run-kql-validation-locally

Checked that the validations are passing and have addressed any issues that are present:

Yes/No/Need Help

Note: Let us know if you have tried fixing the validation error and need help.

References:

Guidance for Detection checks
General contribution guidance
PR validation troubleshooting

Hi @v-sabiraj, please check with validations.

Hi @v-sabiraj, please look into validations. Thanks.

Hi @anki-narravula, please provide your feedback on this. Thanks

Hi @anki-narravula, could you please provide your feedback on this. Thanks

Hi @anki-narravula, please provide your feedback on this. Thanks

Hi @anki-narravula, could you please provide your feedback on this. Thanks

Hi @anki-narravula, please provide your feedback on this. Thanks

Hi @v-sabiraj @anki-narravula, please provide your feedback on this.

Hi @v-sabiraj @anki-narravula, could you please provide your feedback on this. @v-sabiraj Also resolve branch conflicts. Thanks

Hi @v-sabiraj @anki-narravula, please provide your feedback on this. @v-sabiraj Also resolve branch conflicts. Thanks

Hi @anki-narravula, please provide your feedback on this. @v-sabiraj Also resolve branch conflicts. Thanks

Hi @anki-narravula, could you please provide your feedback on this. @v-sabiraj please resolve branch conflicts. Thanks

Hi @anki-narravula, please provide your feedback on this. @v-sabiraj Also resolve branch conflicts. Thanks

Hi @v-sabiraj, please resolve branch conflicts. Thanks

Hi @v-sabiraj, please resolve branch conflicts. Thanks

Hi @anki-narravula, please provide your feedback on this. @v-sabiraj Also resolve branch conflicts. Thanks

Hi @v-sabiraj, please resolve branch conflicts. Thanks

Internal request as per author's confirmation closing for now.


[Docs] Apply the code inline styles for "render"

Thanks for the suggestion.
"render method" not only refers to render of class components but also function components. So using the code formatting is not covering all cases.

Got it. Sorry. I've missed this part.


multiple Daikin devices

I have 4 Daikin devices, but I can only integrate one. How do I integrate the others? In the terminal they are successfully paired with “pair ”. But not visible in the Daikin integration

At the place where you enter the BT MAC Address you just have to add all 4 mac addresses, separated by spaces if I remember right.

Not recognized, I tried also comma.
@mduran80 if you have an idea, I am stuck, I opened another ticket.

"devices": "Comma-separated list of BRC1H Bluetooth addresses",
@wilhelmf-devel check in the code, comma should be used.
https://github.com/mduran80/daikin_madoka/blob/main/strings.json


Upgrade to use Cirq 0.11

I think cirq.Qid isn't directly compatible so we'll need to make some changes within FQE.

I assume we tried the same thing which led to an AttributeError with cirq.Qid. Interestingly this had nothing to do with cirq.Qid but rather a broken Cirq installation.
pip install openfermion==1.0.0  # Installs cirq==0.10.0
pip install cirq==0.11.0

At this point the Cirq installation is no bueno.
import cirq
cirq.Qid  # AttributeError: module 'cirq' has no attribute 'Qid'
cirq.Circuit  # AttributeError: module 'cirq' has no attribute 'Circuit'
cirq.__version__  # AttributeError: module 'cirq' has no attribute '__version__'

cc @balopat as an FYI, maybe this will pop up elsewhere?

a new version of openfermion as been pushed to pypi 1.1.0 which functions with cirq.0.11.0. Moving forward we need to force FQE to use these versions.

Thanks @ncrubin - Done in #91!


Random crashes?

node:events:505
throw er; // Unhandled 'error' event
^
TypeError: The compressed data passed is corrupted
at Decoder._decode (C:\Users\serap\Desktop\discordspeechbot\discord-speech-recognition\examples\simpleBot\node_modules\prism-media\src\opus\Opus.js:64:25)
at Decoder._transform (C:\Users\serap\Desktop\discordspeechbot\discord-speech-recognition\examples\simpleBot\node_modules\prism-media\src\opus\Opus.js:204:22)
at Decoder.Transform._write (node:internal/streams/transform:184:23)
at writeOrBuffer (node:internal/streams/writable:389:12)
at _write (node:internal/streams/writable:330:10)
at Decoder.Writable.write (node:internal/streams/writable:334:10)
at AudioReceiveStream.ondata (node:internal/streams/readable:754:22)
at AudioReceiveStream.emit (node:events:527:28)
at AudioReceiveStream.Readable.read (node:internal/streams/readable:527:10)
at flow (node:internal/streams/readable:1011:34)
Emitted 'error' event on Decoder instance at:
at Decoder.onerror (node:internal/streams/readable:773:14)
at Decoder.emit (node:events:527:28)
at emitErrorNT (node:internal/streams/destroy:157:8)
at emitErrorCloseNT (node:internal/streams/destroy:122:3)
at processTicksAndRejections (node:internal/process/task_queues:83:21)
This is periodically happening when I use SimpleBot. I don't really see how it's something I wrote, though I did modify it a bit.

I have it joining a channel, and it doesn't leave it just continues functioning when you trigger it.

try to update @discordjs/voice and @discordjs/opus packages and tell me if that fixed the issue for you

I did and it did not
Sent from my iPhone

On Jul 29, 2022, at 3:05 PM, Bartosz Gotowski @.***> wrote:

try to update @discordjs/voice and @discordjs/opus packages and tell me if that fixed the issue for you
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.

are you using web version of discord?

Try to use the newest version for discord.js v14, it shouldn't happen now


fetch: automatically trust keys

Code for trust moved to common/fetch_keys.go for sharing between
commands 'rkt trust' and other fetch commands.
The keys are only trusted automatically if they are fetched via https
and the images are fetched via the discovery protocol.
Partially fixes #481.

I rebased this to fix a conflict. PTAL.
Summary of the patch:

Move code from rkt/trust.go to common/fetch_keys.go
rkt/images.go calls common/fetch_keys.go to fetch the keys
Delete function CheckSignature: it was not used (only the method was used)
pkg common now requires pkg pkg/keystore. Change pkg/keystore so that it does not depend on common to avoid circular dependencies.
Remove common.DefaultSystemConfigDir: it was only for the tests anyway, so it does not need to be kept in sync
replace stderr by fmt.Fprintf in common/fetch_keys.go because stderr is only in pkg main
improve message Trusting %q for prefix %q [without fingerprint review|after fingerprint review.]

I moved back fetch_keys.go in the main package and the diff is now smaller, with shorter function prototypes. PTAL.

This implementation prints
Trusting "https://coreos.com/dist/pubkeys/aci-pubkeys.gpg" for prefix "coreos.com/etcd" without fingerprint review.

everytime you fetch an ACI. That should only be printed the first time you trust a key.
You should check if we already have that key and skip that message in that case.
Another way could be checking the signature first and, if we fail, try to download the key and check again.

The new implementation either prints:
rkt: searching for app image coreos.com/etcd:v2.0.0
prefix: "coreos.com/etcd"
key: "https://coreos.com/dist/pubkeys/aci-pubkeys.gpg"
gpg key fingerprint is: 8B86 DE38 890D DB72 9186  7B02 5210 BD88 8818 2190
	CoreOS ACI Builder<EMAIL_ADDRESS>Trusting "https://coreos.com/dist/pubkeys/aci-pubkeys.gpg" for prefix "coreos.com/etcd" without fingerprint review.
Added key for prefix "coreos.com/etcd" at "/etc/rkt/trustedkeys/prefix.d/coreos.com/etcd/8b86de38890ddb7291867b025210bd8888182190"
Downloading signature from https://github.com/coreos/etcd/releases/download/v2.0.0/etcd-v2.0.0-linux-amd64.aci.asc
Downloading signature: [=======================================] 819 B/819 B
Downloading ACI: [=                                            ] 129 KB/3.7 MB

or
rkt: searching for app image coreos.com/etcd:v2.0.0
prefix: "coreos.com/etcd"
key: "https://coreos.com/dist/pubkeys/aci-pubkeys.gpg"
gpg key fingerprint is: 8B86 DE38 890D DB72 9186  7B02 5210 BD88 8818 2190
	CoreOS ACI Builder<EMAIL_ADDRESS>Key "https://coreos.com/dist/pubkeys/aci-pubkeys.gpg" already in the keystore
Downloading signature from https://github.com/coreos/etcd/releases/download/v2.0.0/etcd-v2.0.0-linux-amd64.aci.asc
Downloading signature: [=======================================] 819 B/819 B
Downloading ACI: [=============================================] 3.7 MB/3.7 MB

So if the file in the keystore is an empty file (key disabled by the administrator), it will not be overwritten during the next rkt fetch.

The Semaphore test TestImplicitFetch fails but it is not related to this patch: it is the same reason as https://github.com/coreos/rkt/pull/1194#issuecomment-129350838 : AWS S3 is currently failing.

Ok, LFAD I think. There was one thing I was wondering about, though.


Inherit snapshot ids for manifest entries

This PR addresses #504.

I've updated this PR with some recent progress. I am working on tests. In addition, there are a couple of open points to discuss.

Looks like this conflicts with the recent update to rewrite manifests. Can you update, @aokolnychyi?

This PR is ready for a closer look.

I run a few basic benchmarks locally (e.g. add/read 100 manifests that are 275KB in size and contain 10000 entries).
With this PR:
Benchmark                                      Mode  Cnt  Score   Error  Units
AppendManifestsBenchmark.fastAppendManifests     ss    5  2.468 ± 0.087   s/op
AppendManifestsBenchmark.mergeAppendManifests    ss    5  4.489 ± 0.103   s/op
ReadManifestsBenchmark.basicPlanFiles            ss    5  0.489 ± 0.103   s/op

Before this PR:
Benchmark                                      Mode  Cnt  Score   Error  Units
AppendManifestsBenchmark.fastAppendManifests     ss    5  6.387 ± 1.068   s/op
AppendManifestsBenchmark.mergeAppendManifests    ss    5  8.007 ± 0.217   s/op
ReadManifestsBenchmark.basicPlanFiles            ss    5  0.496 ± 0.051   s/op

Right now, we still read manifests to get stats. I think we can address that in a follow-up PR.

@aokolnychyi, the implementation looks correct to me. My only remaining concern is our forward-compatibility guarantee that older readers will continue to be able to read tables written by future versions within the format version in metadata.
Technically, this breaks that guarantee for tables that are appended to using manifests. I think that means we should test that older readers can read tables written with this that don't have appended manifests (I think Avro will throw a runtime error if it encounters a null snapshot ID, but will allow attempting to read). We should also probably add a feature flag to turn on this breaking behavior -- that way you can opt into using manifest files without a snapshot ID, knowing that it will break older readers. What do you think?

I started a thread on the dev list to discuss breaking changes to the format and how to handle them. I proposed the feature flag approach.

The feature flag approach makes sense to me in this case. Then what about exposing a table property to enable metadata inheritance (false by default) and rewriting appended manifests if that property is false? That will complicate the logic for cleanup but it will keep format forward-compatible by default.

Then what about exposing a table property ... and rewriting appended manifests if that property is false?

That's what I was thinking. I don't think we should add a flag that turns on or off inheritance, though. When reading, we must always add the inherited data. Otherwise we can corrupt other places in metadata. I'd prefer a specific flag to allow writing manifests without snapshot IDs. If it is allowed, then we can append manifests and otherwise we have to rewrite them.

Sorry, that’s what I meant by enabling inheritance as well.

I am going to merge this one.

Thanks for the review, @rdblue and @chenjunjiedada!

Thank you for building the framework for inheritance! This is really helpful.


 I finished coding on this PR)
[x] All changes have test coverage
[x] Code is well-documented

Changes

[x] fix the content of last batch in bptt_batchify

Job PR-120/1 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-120/1/index.html

Job PR-120/2 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-120/2/index.html


Bugfix: Normalize workspaceFolder name

workspaceFolder name with case-sensitive cause match failed

@kh2se2013 Thanks for the pull request! Is this similar the issue brought up here?
@Peppie23 https://github.com/ryanluker/vscode-coverage-gutters/issues/188#issuecomment-544208927
Side note: there is an issue with github actions running builds on forked repositories that I will have to look into...

@kh2se2013 Github actions for forked repositories is fixed https://github.com/ryanluker/vscode-coverage-gutters/pull/242 once you get back to me with an answer for https://github.com/ryanluker/vscode-coverage-gutters/pull/241#issuecomment-544293183 we can adjust or merge this into master and I will make a quick release for 2.4.2.

Very nice vscode extension

@kh2se2013 Thanks for the pull request! Is this similar the issue brought up here?
@Peppie23 #188 (comment)
Side note: there is an issue with github actions running builds on forked repositories that I will have to look into...

LGTM
Very nice vscode extension

@kh2se2013 excellent, thanks again for the fix!
2.4.2 should go out soon once I get around to finalising the testing.


document statistics

should document how each statistic is calculated, what the units are, etc

Reaction time: How is he reaction
time calculated? Time between target appearance and
movement onset?
Movement time: how is movement time calculated? It seems that this is the time only when the
cursor is moved (e.g. when the cursor is not moved, 0 is displayed)? Would it be possible to
add an additional movement time parameter (e.g. overall movement time) defined as the time
between target appearance until the target is reached?
Distance & RMSE: how are the distance and RMSE calculated? For example, the distance and
RMSE seems to be very small when the cursor is moved in the opposite direction. What is the
unit of the parameters (e.g. cm)?


Next or Previous post helper Not working in API v2

next or previous post helper Not working in API v2 - https://docs.ghost.org/api/handlebars-themes/helpers/prev_next_post/
{{#next_post}}

What's Next? - {{title}}

{{else}}

Oops No more New posts! Goto → <a<EMAIL_ADDRESS>{{/next_post}}

it does not fetch the Post URL it fetch the Homepage URL

What's Next? - Post description

PS: working well in API v0.1

Hi @mskian! This seems like the same issue that was already fixed here & here. Please feel free to reopen if it's a different case :)


Fix logical errors in byteStride calculations

Fixed loading failure issue when byteStride  description is 0 in buffer view

@Yeah-Zi can you explain more why this is needed? If a bufferView has byteStride explicitly specified to be zero, that's a broken glTF. The value is required to be greater than or equal to 4:
https://registry.khronos.org/glTF/specs/2.0/glTF-2.0.html#_bufferview_bytestride

Thank you for providing the standard format specification for gltf.
Why I need this is because the customer provides data with a byteStride of 0, it can be loaded normally in CesiumJS, while Cesium For Unreal cannot

@kring Perhaps some log information can be added here, as loading failure does not result in any errors. It took me some time to debug this error

i found how cesiumjs due with this situation:
https://github.com/CesiumGS/cesium/blob/main/packages/engine/Source/Scene/GltfPipeline/getAccessorByteStride.js

Perhaps some log information can be added here, as loading failure does not result in any errors.

I'm not opposed to adding some more logging but, in general, cesium-native is a loader, not a validator. A proper glTF validator (like this one: https://github.khronos.org/glTF-Validator/) should easily detect invalid glTF files like this.
However, in this case, since your change matches what CesiumJS is already doing, and it has very little cost, I'm happy to merge it. Can you please sign the Contributor License Agreement first:
https://github.com/CesiumGS/cesium/blob/master/CONTRIBUTING.md#contributor-license-agreement-cla
Thank you for the PR!

@kring I have signed the CLA

Thanks @Yeah-Zi!


FileNotFoundError:

FileNotFoundError: [Errno 2] No such file or directory: 'C:\Users\Mr.Wang\.keras\datasets\cifar-10-batches-py\data_batch_1'
I‘m using this code "cifar10.load_data()"
but when I run get this error

This might help you.

It looks like you haven't used a template to create this issue. Please resubmit your issue using a template from here. We ask users to use the template because it reduces overall time to resolve a new issue by avoiding extra communication to get to the root of the issue. We will close this issue in lieu of the new one you will create from the template. Thank you for your cooperation.


Add Cytomining ecosystem visualizations, configure GitHub Pages, and add to GitHub data

This PR makes a number of changes largely focused on increasing the amount of data extracted, creating visualizations, and configuring GitHub pages content for with the visualizations.
Thank you for any feedback you may have! Please don't hesitate to let me know if you have any questions.

Thanks @gwaybio! Going to merge this in.


"list instances" does not return instance

Hello,
thank you for the cool tool.
I created EC2 instances in the eu-west-1, and the following command does not see any instances:
./ssm-sh-darwin-amd64 list instances -p default
Instance ID | Name | State | Image ID | Platform | Version | IP | Status | Last pinged
where aws command returns 2 instances:
aws ec2 describe-instances --profile default | gron | grep InstanceId
json.Reservations[0].Instances[0].InstanceId = "i-0ebxxxxd8a2";
json.Reservations[1].Instances[0].InstanceId = "i-0183xxxxf4d1";

it was due incorrect permissions.


MM-41342 Revert fixes related to using team sidebar in products

As discussed in https://community-daily.mattermost.com/core/pl/7rqqd1g1pig4xb1pp4j7ybnmqo, we're reverting these changes since the root CSS for the page is fragile, and those changes ended up breaking the position of some elements (notably the centre pane in Playbooks/Boards and the announcement bar/sidebar in Channels)
Ticket Link
https://mattermost.atlassian.net/browse/MM-41342
Related Pull Requests
https://github.com/mattermost/focalboard/pull/2263
Screenshots

Before
After

Desktop View*

Mobile View

*: Note that the overlap of the team sidebar is a pre-existing issue with the announcement bar (https://mattermost.atlassian.net/browse/MM-40887)
Release Note
NONE

@harshilsharma63  Please help prioritize PR review.

/e2e-test

Triggering e2e testing with options:
<nil>

Successfully triggered e2e testing!
https://git.internal.mattermost.com/qa/cypress-ui-automation/-/pipelines/147942

Tested this and https://github.com/mattermost/mattermost-webapp/pull/9756, the respective issues are fixed on both.
Waiting on e2e results now.

Skipping the second dev review on this since this is just a revert and I want to get the other PRs unblocked

Cherry pick is scheduled.


Do we must implement Xamarin.Auth in CustomRenderer?

Hi! I'm working with Xamarin.Forms, as the title. Do we must implement Xamarin.Auth in CustomRenderer?
Thank you!

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

ID: aa8bb9e0-5898-0d11-0213-fd4d8c8d0297
Version Independent ID: 2f46e1f7-164f-2f32-70c0-f28919d16c5c
Content: Authenticating Users with an Identity Provider - Xamarin
Content Source: docs/xamarin-forms/data-cloud/authentication/oauth.md
Product: xamarin
GitHub Login: @davidbritch
Microsoft Alias: dabritch

There is a sample linked at the start and end of the page, it does not appear to use a custom renderer, however platform-specific code is required in both the iOS and Android projects (such as the initialization code on both platforms, the OpenUrl method on iOS, and CustomUrlSchemeInterceptorActivity on Android).

Thank you Sir!
Is there any update for this package, it's too long from last update?


Error handling

Fluent doesn't support error handling. If a transformation fails (e.g. extracting a value from None) an exception is thrown.
This PR makes error handling more explicit by returning an Either[Error, Result].

Codecov Report

Merging #8 into master will decrease coverage by 4.54%.
The diff coverage is 75.6%.

@@            Coverage Diff             @@
##           master       #8      +/-   ##
==========================================
- Coverage   89.47%   84.93%   -4.55%
==========================================
  Files           3        4       +1
  Lines          57       73      +16
  Branches        3        0       -3
==========================================
+ Hits           51       62      +11
- Misses          6       11       +5

Impacted Files
Coverage Δ

src/main/scala/fluent/TransformError.scala
33.33% <33.33%> (ø)

src/main/scala/fluent/package.scala
50% <50%> (-50%)
:arrow_down:

src/main/scala/fluent/internal/Transformer.scala
88.52% <82.35%> (+0.28%)
:arrow_up:

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 80d8139...da72dbf. Read the comment docs.


ci-multicluster: Fix post-test information gathering

This fixes two issues with the ci-multicluster log gathering:

cilium clustermesh status can unfortunately block even when
not using --wait.
See cilium/cilium-cli#384
This causes a failing job to be cancelled, thus causing any
post-failiure steps to be skipped as well.
This commit works around the issue by manually adding a timeout to
the post-failiure cilium clustermes status invocation.

Sysdumps were not collected for both clusters. This is fixed by
manually switching to each cluster context using kubectl.

I have successfully tested this in https://github.com/cilium/cilium/pull/16705 - the artificial failure caused the correct double sysdump to be created: https://github.com/cilium/cilium/pull/16705/checks?check_run_id=2954156565
Artifacts:

cilium-sysdump-out.zip.zip
logs_126246.zip

We should probably also make a note of removing the workarounds once we bump up the CLI version to a release with the fix for https://github.com/cilium/cilium-cli/issues/384.

Thanks LGTM. Do you want to make changes in cilium-cli (https://github.com/cilium/cilium-cli/blob/daed305be87be2ba609a077d766ddd8b404c5623/.github/workflows/multicluster.yaml#L131-L138) accordingly for retrieving sysdump for both clusters? Otherwise I'll make a note of doing that later.

Oh, good catch. I will open a PR on cilium/cilium-cli as well then.


Consider adopting the proposed Port Attributes API

Original post: https://github.com/microsoft/vscode/issues/115616#issuecomment-1520449147
In a remote environment (Codespaces, Dev Containers, etc.) VS Code will automatically detect and forward ports that it finds processes listening on so that users can access the remote application they're developing from their local machine. This includes debugees, such as a Python program that the user is debugging.
 This results in noisy extra ports being forwarded. We've seeing this in other extensions, such as the JS Debug extension.
The proposed solution for this is to allow extension to provide port attributes for ports that they know about. The JS Debug extension has done this for the debug ports it uses: https://github.com/microsoft/vscode-js-debug/blob/0d0704da3726653e2d806be528b8ab51fa952cf0/src/ui/portAttributesProvider.ts#L39-L55
We're working on finalizing the port attributes API, so I don't expect too many more changes. In support of finalizing the API, and to remove the port-clutter that comes from forwarding Python extension debug ports, it would be great if the Python extension adopted the proposed port attributes API.

Python folks, if you want I can specifically assign a Python extension dev to my port attributes test plan item for endgame next week.

Here are few scenarios we are creating ports, and where I think forwarding will be required and where it is not.
Debugger scenarios
I had a discussion with debugpy team, the underlying debugger where we were planning on using this API. The debugger itself communicates with subprocess that it starts over ports, but those don't need to forward. Those are ephemeral and can be ignored for this case.
Cases to consider:

debugpy listen mode: In this mode VS Code launches the Debug Adapter with the intent that the Debug adapter will listen for connections from the debugger. This is a scenario where the debugger is installed independently of VS Code or Python extension, say like in a container that the user wants to debug. The client that launched debug adapter will have a port provided by the user that it opens in a listening mode, and the debugger in the container connects to the debugger as an outgoing connection from the container.

debugpy connect mode: In this mode VS Code directly connects to debugpy which is running in server mode. This is another case where typically it is a container or a remoter server that is running debugpy in it and may not contain VS Code server in it.

I don't think the above two cases are where the API will be used.
Web app scenarios
These are cases that involve running some web app framework, like django, fastapi, flask etc. There are two possibilities here:

Run without debugging/run in terminal case: This is where the web app is run either without debugger or is being run in terminal directly by user or run in terminal using a command. If we can detect the port that is opened by the web app framework then we could provide the right attributes for it to be forwarded.

Debugging a web app launched with server ready: Server ready is a VS Code launch json configuration that automatically detects web server string in the terminal using patterns provided by the user. This is often used to trigger browser launch after the server has started. This seems like a case where, VS Code itself should be able to detect and forward the port without requiring input from extension?

Debugging a web app launched without out server ready: This is another case where we will have to monitor the content being sent to the terminal, but a key difference here is that debugger can capture stdout and redirect it as a DAP message. So extension can use DAP tracker API from VS Code to monitor and provide port attributes.

Data Science

Tensor Board: I think tensor board uses ports in some form so this API could be used there to provide the right attributes.

The debugger scenarios sound reasonable to me.
For the web app scenarios:

Run without debugging/run in terminal case: This is where the web app is run either without debugger or is being run in terminal directly by user or run in terminal using a command. If we can detect the port that is opened by the web app framework then we could provide the right attributes for it to be forwarded.

VS Code will automatically forward this anyway. VS Code detects ports in two possible ways based on what settings the user has: watching the proc filesystem and/or watching terminal/debug output. It sounds like this is also desirable behavior.

Debugging a web app launched with server ready: Server ready is a VS Code launch json configuration that automatically detects web server string in the terminal using patterns provided by the user. This is often used to trigger browser launch after the server has started. This seems like a case where, VS Code itself should be able to detect and forward the port without requiring input from extension?

Yes, the port will be automatically forwarded.

Debugging a web app launched without out server ready: This is another case where we will have to monitor the content being sent to the terminal, but a key difference here is that debugger can capture stdout and redirect it as a DAP message. So extension can use DAP tracker API from VS Code to monitor and provide port attributes.

If you want to have the port forwarded in this case then you shouldn't have to do anything.

@paulacamargo25 I'm ready to finalize the port attributes API. I'd love to get feedback from you or someone else on the Python team before I do so. Is adopting this API work that is planned for September?

@alexr00 This is planned for September Milestone.
Adding debugger owners to provide any input they have. @int19h @AdamYoblick

@alexr00 I had a look, and my main concern is the inherent asynchrony between the debugger processes and VSCode. For internal ports, to avoid clashing with anything else, the debugger typically does listen(0) to let the OS allocate an ephemeral port - so it doesn't actually know the port number until it starts listening. On the other hand, it needs to communicate that port number back to the Python extension for the latter to be able to respond to providePortAttributes correctly. We can communicate it using a custom DAP message or through some other mechanism, but the problem is that, so far as I can tell, there's no guarantee that providePortAttributes call is scheduled after that custom message has been parsed. I would expect similar issues with any out-of-proc DAP implementation that uses sockets internally to communicate between its own components.

@int19h you are correct that there's no guarantee that providePortAttributes will happen after your custom message. However, this will likely not be an issue because VS Code will not be very quick to call providePortAttributes:

First, we have to actually discover the port. For the purposes of this issue, debug ports will only ever be discovered through polling, which means we likely already have a delay. The minimum polling interval is 2 seconds, but we use a moving average to calculate it based on how fast the machine is, so it can be greater.
Next, VS Code has to communicate the discovered port from the remote machine to the local machine. This will add whatever the network latency is to the delay.
Finally, VS Code's call to providePortAttributes will always be cross-process, so that will also add delay.

Given these inherent delays, which can easily be > 1 second, and will almost always be hundreds of milliseconds, I don't think we should let perfect be the enemy of better here.


share: ensure DA for heights for which the data was requested

We currently serve data and do DAS independently, but it may be possible to serve shares or blobs without checking DA for the particular height. We should change that and block calls like GetSharesByNamespace and all the BlobModule getters until the data is available via the SharesAvailable call.
For context, this was part of various sync discussions and finally came out on the lower level in Rollkit https://github.com/rollkit/rollkit/pull/986#discussion_r1236661154

We should be careful with all the SharesAvailable implementations we have for diff node types we use. We need to audit that for possible locking issues this may create in this code path

Well, it needs #1595, because it implies the SharesAvailabe is regularly called by our users and the DASer, but we don't want to have parallel data requests.

We should change that and block calls like GetSharesByNamespace and all the BlobModule getters until the data is available via the SharesAvailable call.

This behaviour shouldn't be mandatory, it should be optional. There are valid use cases for getting data before DASing in completed. For example, a sequencer might want to get a head-start in making the next block, or they might be happy being a super-light client (which is still better than relying on a centralized RPC endpoint which is what Ethereum sequencers do now).


x/net/html: doesnt support attributes without values ie <script async "foo.js">

What version of Go are you using (go version)?

$ go version
go version go1.14.4 windows/amd64

Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
set GO111MODULE=
set GOARCH=amd64
set GOBIN=
set GOCACHE=C:\Users\rhaksi\AppData\Local\go-build
set GOENV=C:\Users\rhaksi\AppData\Roaming\go\env
set GOEXE=.exe
set GOFLAGS=
set GOHOSTARCH=amd64
set GOHOSTOS=windows
set GOINSECURE=
set GONOPROXY=
set GONOSUMDB=
set GOOS=windows
set GOPATH=C:\Users\rhaksi\go
set GOPRIVATE=
set GOPROXY=https://proxy.golang.org,direct
set GOROOT=c:\go
set GOSUMDB=sum.golang.org
set GOTMPDIR=
set GOTOOLDIR=c:\go\pkg\tool\windows_amd64
set GCCGO=gccgo
set AR=ar
set CC=gcc
set CXX=g++
set CGO_ENABLED=1
set GOMOD=C:\dev\bitbucket\kidoz-team\bidder-server\go.mod
set CGO_CFLAGS=-g -O2
set CGO_CPPFLAGS=
set CGO_CXXFLAGS=-g -O2
set CGO_FFLAGS=-g -O2
set CGO_LDFLAGS=-g -O2
set PKG_CONFIG=pkg-config
set GOGCCFLAGS=-m64 -mthreads -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=C:\Users\rhaksi\AppData\Local\Temp\go-build383668822=/tmp/go-build -gno-record-gcc-switches

What did you do?
markup := `<!DOCTYPE html>
<html>
	<head>
		<title>.</title>
	</head>
	<body>
		<script async src="test.js"></script>
	</body>
</html>
`
r := strings.NewReader(markup)
doc, err := html.Parse(r)
if err != nil {
	return
}
var writer bytes.Buffer
err = html.Render(&writer, doc)
if err != nil {
	return
}
fmt.Println(writer.String())

What did you expect to see?
The async attribute should have no value.
<html>
	<head>
		<title>.</title>
	</head>
	<body>
		<script async src="test.js"></script>
	</body>
</html>

What did you see instead?
The async attribute is rendered with an extra ="" added to it.
<html>
	<head>
		<title>.</title>
	</head>
	<body>
		<script async="" src="test.js"></script>
	</body>
</html>

async is one of the 25 boolean attributes, it should be either present (value of true) or absent (value of false), and shouldnt have a value assigned.
The Boolean Attributes
allowfullscreen
allowpaymentrequest
async
autofocus
autoplay
checked
controls
default
disabled
formnovalidate
hidden
ismap
itemscope
loop
multiple
muted
nomodule
novalidate
open
playsinline
readonly
required
reversed
selected
truespeed

Aside from visual noise and extra bytes, does the added ="" cause any issues when the HTML is parsed?
/cc @mikioh @bradfitz @ianlancetaylor per owners.

For the one example I ran across when putting html through the parser and back work in the latest version of chrome for specifically the extra ="" being assigned to the async boolean attribute it does seem to work. In the sense that the html seems to render and the javascript seems to work correctly, although I cant say if the async attribute is still being obeyed (but probably)
On the other hand its an open question if it works in every version of every browser on every platform for every boolean attribute, even just the subset still commonly in use, because that I couldn't tell you. I suppose its possible. Its also of course impossible to make my minifier truly minify while using html.parse/render.
I imagine many html validators would fail the resulting output with an error for assigning a value to a boolean attribute. Although in my particular narrow use case this wont affect me.
Cheers

As an other perspective, the current behavior is valid as per the whatwg spec:
https://html.spec.whatwg.org/multipage/common-microsyntaxes.html#boolean-attributes
Since most browsers probably respect the whatwg spec, the html code rendered by this package can be expected to work on browsers properly.


Amazon Login button

Hi!
very nice job can you add also login with Amazon button?
Thanks.
L.

I agree, especially since it could be used for amazon checkout.

Yeah, i totally agree

There is an Amazon font awesome icon, so this would make sense: <i class="fa fa-amazon"></i>

+1


Did some repos get tinier?

figure out if it's my code or the repos.
It's probably my code.

there's something wrongggg with the byes of code calculations. This is clearly two repos, not a single repo that replaced its entire codebase

\on the other hand, some of the repos are fine - this is clearly a stable older repo:

github action where I check out the repo and then do cloc, at the dates  of month0 and month12?  https://github.com/AlDanial/cloc


Jest configurations not supported

I am trying to setup Jest so that I can have my test files inside of my lib folder. Just based on preference. The problem here is that if a build was performed, it generates those test files in the build folder. Normally I can just use the testPathIgnorePatterns config in Jest. But currently you are not allowing me to override that particular config. Are there any reasons for this?

Hi there!
I'll have to look at what's going on with the Jest config here. But in this case, the issue is that the build command generated the test files as well right? So you don't want these files to be transpiled?

Yep, I guess its a bit of both. In this case, definitely would be good to stop the build command generating the test files. At the same time, would be good to have more options in configuring Jest as per usual Jest configurations (including option to use a separate jest.config instead of placing it inside of package.json. I can see this being a requirement for another project that I am wanting to transition to use the CDK and therefore the SST if my proof-of-concepts goes well.

Yeah let me have a look at the Jest config options and see what we are not exposing right now.

@justindra Right now there's a set of supported options available to be overridden. It's roughly based on the Create React App project.
https://github.com/serverless-stack/serverless-stack/blob/master/packages/cli/scripts/config/createJestConfig.js#L44
We can change this, let me know which ones you want added.

The one I currently need right now is the testPathIgnorePatterns  key. I was just wondering what the reason behind only supporting these particular keys, instead of allowing everything?

Since the entire Jest process is managed internally, it's not completely configurable. But if there are specific options that need to be added, we'll definitely find a way to expose them.
Let me have a look at the testPathIgnorePatterns for now.

Oh btw, I should mention that you can still pass in testPathIgnorePatterns through the command line.
But just pushed out a release fixing this.
https://github.com/serverless-stack/serverless-stack/releases/tag/v0.4.2

Since the entire Jest process is managed internally, it's not completely configurable.

This also prevents running tests individually via WebStorm's Jest run configuration.
Rather than hide the process, perhaps tools like ts-node could help to run tests. I use ts-node on other Typescript projects to run tests directly from my IDE and via the CLI, without having to transpile the TS to JS first.


Fixing Mono compile issues

fixes issue #186

Don't merge this yet, i still have some errors and i'm trying to find them since i'm already here.


cant run PySilon.bat

ive built the bot as followed and ive been trying to run the program in visual code studio so i can put the token for my bot into the gui but i keep running into a error within the PySilon.bat code, ill run it in the terminal and it always shows this and doesnt show me the ui, please help i have no clue on where to continue and or how to resolve this
_tkinter.TclError: Can't find a usable init.tcl in the following directories:
C:/Users/darkf/AppData/Local/Programs/Python/Python313/lib/tcl8.6 C:/Users/darkf/AppData/Local/Programs/Python/lib/tcl8.6 C:/Users/darkf/AppData/Local/Programs/lib/tcl8.6 C:/Users/darkf/AppData/Local/Programs/Python/library C:/Users/darkf/AppData/Local/Programs/library C:/Users/darkf/AppData/Local/Programs/tcl8.6.14/library C:/Users/darkf/AppData/Local/tcl8.6.14/library
This probably means that Tcl wasn't installed properly.
#===============================================================#
Software terminated.

Give us a Star on Github, this would really help us grow!
https://github.com/mategol/PySilon-malware

Also, please don't send this malware using websites like
Workupload or googledrive because they will scan the malware
and keep track of it and other ocurrences, which will result
in more detections in the future, please send it to people in
a zip archive with a password, or use services like anonfiles

Thank You!
#===============================================================#
Press any key to continue . . .

do you have python installed? try uninstalling python and reinstalling it but make sure you select add to path or it won't work

i too got the same error tried uninstalling but it isnt working


Edit table on Preview section

You have planned in future make simple edit table on Preview page (merge/separate cells, move data from one cell to another and etc) before export?

Hi @sashsvamir, we've considered it, but I think we decided not to. We think Tabula is best suited to do only one thing, that is, extracting data from PDFs. If you want to edit the output spreadsheets, you should use a dedicated spreadsheet editing tool like LibreOffice Calc, Google Sheets or Microsoft Excel. Those tools do a far better job at editing spreadsheets than Tabula would.


What we share on share button to other platform

@Matrixbirds We have to build share button and share which method we get data to live streaming to other platform like whatsapp

could this method helps you?
https://docs.agora.io/en/Voice/API Reference/java/classio_1_1agora_1_1rtc_1_1_rtc_engine.html#aeb1a5691094a10cb047d106d6c6c32b7


bug: ng-switch with collection-repeat

Type: bug
Platform: all
Having two collection-repeats in the same page with a ng-switch breaks the scroll when "switching". The result is somewhat funny though (when you try to scroll, the items disapear from top to bottom (as they're being redrawn like it is scrolling, but the scroll is absolutelty locked))
Plunker here:
http://plnkr.co/edit/MNTJq0QUbRZxiJumoVL9?p=preview
Just scroll a bit to check it works initially, click on "two" (now it doesn't scroll, but there're only 3 elements (as expected)) and when you go back to "one" the bug is clearly visible when you try to scroll down.
I wasn't sure to flag this as a bug or as a documentation thingy, as the docs say:
edit: Important update below!

4 Each collection-repeat list will take up all of its parent scrollView's space. If you wish to have multiple lists on one page, put each list within its own ionScroll container.
5 You should not use the ng-show and ng-hide directives on your ion-content/ion-scroll elements that have a collection-repeat inside. ng-show and ng-hide apply the display: none css rule to the content's style, causing the scrollView to read the width and height of the content as 0. Resultingly, collection-repeat will render elements that have just been un-hidden incorrectly.

But the thing here is that ng-switch doesn't use ng-show/ng-hide, but it just comments out the whole DOM. Maybe you should change the docs to say that there shouldn't be two collection-repeats in the same scrollView at all?
In my case there's an easy fix: Put only one collection repeat "pointing" to an array in the scope, and switch this array in the controller depending on which button is pressed. I tried it this way and it works perfectly (there's then only 1 collection-repeat) (collection-repeat is not inside an ng-switch)
EDIT: Just checked what happened in the case where only one tab has collection-repeat, and it's broken as well. Plunker or it didn't happen! http://plnkr.co/edit/Nr2sxcGql3WoRQtNnjMd?p=preview

Fixed. The problem was that collection-repeat's logic expected that the scrollView would always get destroyed with the repeater, not re-used later.
collection-repeat overrides the scrollView's content sizing and onscroll behavior. Now, when it's destroyed, it restores that behavior to normal and resizes the scrollView.


Platform: link against Pathcch.lib when using pathcch.h

The MSDN documentation indicates that you should link against the
Pathcch.lib import library when using functions from pathcch.h which
can also be verified by use functions failing to link due to unresolved
symbols.  Add the missing linking to enable autolinking for WinSDK.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

@swift-ci please smoke test

CC: @egorzhdan


Does eventmesh support JMS ?

Does eventmesh support JMS ? How can we migrate to eventmesh ?

No plan for ActiveMQ support for now. May be we will support it one day.


Writer a separate operator to install KubeAGI with a CRD

we should define a CRD to make KubeAGI installation quite easy, so we can install u4a/cluster/kubeagi-system together without a deep understanding how they work.

@nkwangleiGIT should we also install cluster-component,u4a-component,kubebb-component in this operator?

yes, one command install using the CRD + controller, cover all components required.


adjusting bonds section to temp remove from UI while we update the bo…

…nds for new ones.
Issue Link
No issue Link.
Description
Changes were small, I simply removed the bonds mapping and put a placeholder in place while we work on the bond list.
How To Review This
You can review the changes by running local or using the AWS Amplify Preview URL.
Checklist:

[ ] I have run yarn lint and there are no lint or typescript errors.
[ ] The production build passes.
[x ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed locally.

References
None.

Running lint checks and making another commit, DO NOT MERGE


Error on imhist when img data is ColorTypes

Getting this error with both RGB and Gray ColorTypes :
using TestImages, Images

img=testimage("mandrill")
img2=testimage("lena_gray_256")

imhist(img2)

ERROR: ArgumentError: StepRange should not be used with floating point
 in steprange_last(::ColorTypes.Gray{FixedPointNumbers.UFixed{UInt8,8}}, ::Float64, ::ColorTypes.Gray{FixedPointNumbers.UFixed{UInt8,8}}) at ./range.jl:27
 in imhist(::Images.Image{ColorTypes.Gray{FixedPointNumbers.UFixed{UInt8,8}},2,Array{ColorTypes.Gray{FixedPointNumbers.UFixed{UInt8,8}},2}}, ::Int64, ::ColorTypes.Gray{FixedPointNumbers.UFixed{UInt8,8}}, ::ColorTypes.Gray{FixedPointNumbers.UFixed{UInt8,8}}) at /home/jarvis/.julia/v0.5/Images/src/algorithms.jl:1219
 in imhist(::Images.Image{ColorTypes.Gray{FixedPointNumbers.UFixed{UInt8,8}},2,Array{ColorTypes.Gray{FixedPointNumbers.UFixed{UInt8,8}},2}}) at /home/jarvis/.julia/v0.5/Images/src/algorithms.jl:1213
 in eval(::Module, ::Any) at ./boot.jl:225
 in macro expansion at ./REPL.jl:92 [inlined]
 in (::Base.REPL.##1#2{Base.REPL.REPLBackend})() at ./event.jl:46

@timholy Did you get the time to go through this issue? I needed this to test histeq with proper ColorType images. Right now I can only use images with Number type.
Even if you could give me some hints as to what I am doing wrong or what needs to be added, it would be great. Since StepRange seems to be working with Float, I am a bit confused by this error. Also, this works without any error -
ColorTypes.Gray(1.0):ColorTypes.Gray(0.2):ColorTypes.Gray(1.8)

By the way, if you click the "watch" button at the top right, you'll get the same notifications about new issues, pull requests, and comments that I get.

Done :)


[ionic v4 with react] Integrating @ionic/core 4.0 beta defineCustomElements error

Hi everyone,
I'm having issues integrating Ionic Web Components to a React project.
So far, I have followed these instructions:
https://stenciljs.com/docs/framework-integration/#react
However, I get an error saying that defineCustomElements is not a function?
Error screenshot

Sandbox code

yup same issue here.
Here is why is this happening:

if you're using webpack or rollup, they'll consume module package.json field, which points to es2015 module files ( not es5 )

which doesn't export defineCustomElements. This function is exported only from the main bundle, which is wrong.

Quick Fix

hope this is gonna be resolved sooner than later ✌

Awesome @Hotell!
I have changed the path to my Ionic JS library import, no errors.
Have you had to add anything else? My components styles are looking all wrong. Perhaps an import statement just for the stylesheet files?

you need to import ionic.css

but I'll save you the effort, I've tested today ( it took me almost whole day poor me...) ionic v4 components with react, and, well. It doesn't work ( also as ionic core devs stated on twitter "ionic v4 is primarily for Angular, react is not supported, yet" . Don't do the same mistake as I did :)

Oh, I see... thanks! (I was also driving insane).
Yeah, it's confusing. My understanding is that thanks to Ionic 4 components being developed in Stencil, they should be framework agnostic. Seems like that's not entirely true.

This is issue should be resolved in beta.17. If I am wrong, please write here!


[Bug] array_zip run failed

Search before asking

[X] I had searched in the issues and found no similar issues.

Version
doris-1.2.4-rc01-Unknown
What's Wrong?
Executing the array_zip function according to the official website document, returned failure
select array_zip(array('a', 'b'), array('c', 'd'));

What You Expected?
The following is an example of an official document containing the expected running results

How to Reproduce?
Deploy a Doris cluster with version 1.2.4-rc01-Unknown and execute the official document example of the array_zip function
The example SQL is as follows:
select array_ zip(array('a', 'b'), array('c', 'd'));
Anything Else?
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

This code was not merged into 1.2


Site replaced

@davetaz I see that https://csvlint.io has completely changed to https://github.com/theodi/csvlint.io, so I assume this repo should be archived?

@Floppy Yes. I built a new version. It still uses the gem which I've also made some changes too and sent you a PR.
https://github.com/Data-Liberation-Front/csvlint.rb/pull/299


[SYSTEMML-902] Improve FrameObject toString output

Override toString for FrameObject, similar to MatrixObject.
Example of outputting MatrixObject and FrameObject via MLContext, before PR:
scala> val res = ml.execute(dmlScript)
res: org.apache.sysml.api.mlcontext.MLResults =
  [1] (Matrix) tA: Matrix: scratch_space//_p7667_<IP_ADDRESS>//_t0/tA_290, [3 x 2, nnz=6, blocks (1000 x 1000)], binaryblock, dirty
  [2] (Frame) tAM: org.apache.sysml.runtime.controlprogram.caching.FrameObject@6c6c4579

After PR:
scala> val res = ml.execute(dmlScript)
res: org.apache.sysml.api.mlcontext.MLResults =
  [1] (Matrix) tA: Matrix: scratch_space//_p42840_<IP_ADDRESS>//_t0/tA_1, [3 x 2, nnz=6, blocks (1000 x 1000)], binaryblock, dirty
  [2] (Frame) tAM: Frame: scratch_space//_p42840_<IP_ADDRESS>//_t0/tAM, [3 x 2, nnz=6, blocks (1000 x 1000)], binaryblock, dirty

@mboehm7
Is the conditional "if (_metaData instanceof NumItemsByEachReducerMetaData)" check necessary for frames or can this be removed?

no it's not required as frames are not supported in the MR backend yet.

@mboehm7 Thank you


Add retries to cephadm package installs

This task seems to fail a lot during multinode automated deployments. Adding retries to help it along. Currently testing: https://github.com/stackhpc/stackhpc-kayobe-config/actions/runs/12481486518

Retested, working https://github.com/stackhpc/stackhpc-kayobe-config/actions/runs/12481486518


caddy service caddyplug install filemanager fail

Info:

Docker version ($ docker --version): Docker version 18.03.1-ce, build 9ee9f40
Laradock commit ($ git rev-parse HEAD): 876935452ec75fb19f75fef386234dc38e7c78de
System info (Mac, PC, Linux): Linux
System info disto/version: Ubuntu 16.04LTS

Issue:

Expected behavior:

Reproduce:
add filemanager to ARG plugins

Relevant Code:

You can try it with the latest version and if there is a problem I will update it again.


Currently, starting a new project throws the error "Error: Invalid sdkVersion."

Description
Following the instructions in the read.me gives the error "Invalid sdkVersion"
Expected Behavior
If someone is following instructions from the readme shouldn't it work?
Observed Behavior
I had to change the sdkversion of expo and react-native. I also had to clear the watchman cache and delete node_modules and reinstall packages. This could have been avoided if the sdk version was set to version that worked.
Error: Invalid sdkVersion. Valid options are 10.0.0, 11.0.0, 12.0.0, 13.0.0, 14.0.0, 15.0.0, 16.0.0, 17.0.0, 18.0.0, 19.0.0, 20.0.0, 21.0.0, 22.0.0, 23.0.0, 24.0.0, 25.0.0, 7.0.0, 8.0.0, 9.0.0

Environment
Please run these commands in the project folder and fill in their results:

npm ls react-native-scripts: 1.13.1
npm ls react-native: 0.54.0
npm ls expo:  26.0.0
node -v: 9.8.0
npm -v: 5.6.0
yarn --version: 1.5.1
watchman version:  4.9.0

Also specify:

Operating system: Mac high sierra 10.13
Phone/emulator/simulator & version: iOS sim v10

Changing expo to v25 and react-native to v52 fixed this error. Just wondering why it's set to an unsupported sdk version.
Reproducible Demo
Just follow the readme.
create-react-native-app my-app
cd my-app
npm run start

Can you share the results of running the following in the terminal: exp -V, mine is 51.4.0 and it works fine... 0.0

Mine is ‘50.0.5’

Updated exp to 51.4.0 and still get the following error.
Error: Invalid sdkVersion. Valid options are 10.0.0, 11.0.0, 12.0.0, 13.0.0, 14.0.0, 15.0.0, 16.0.0, 17.0.0, 18.0.0, 19.0.0, 20.0.0, 21.0.0, 22.0.0, 23.0.0, 24.0.0, 25.0.0, 7.0.0, 8.0.0, 9.0.0

try rm -rf ~/.expo/cache

Doing that in addition to also reseting and reinstalling the expo app on the ios sim made it work. Thanks for the quick reply.


Add HLS support to Elemental wrapper

I believe that this changes looks good, I just have that question regarding the structs looking the same.
Thank you!


Add missing License notice text

Description
Fixes license notice header message for some files. The last line of the header was missing.
Release Note

None required

/sem-approve


Bug: Basket ball & block by block extension

Describe the bug
BasketBall game
--> There are errors in gsap cdn link when chrome extension is loaded..
--> script is not working and size of chrome window is also negligible..
Block by block
--> The chrome extension window is not visible..
--> styling need to be fixed..
To Reproduce
Steps to reproduce the behavior:

Go to chrome://extensions
Click on load unpacked
Scroll down to AtoZ learning or Cgpi calc
See error

Expected behavior
These extensions should work properly on chrome.
Screenshots

Desktop (please complete the following information):

windows (chrome).

Additional context
please assign me under GSSoC'24...

Closing this as exists #991 #2083


Adds japanese language.

This PR also makes some modifications to regex extraction to not use word boundary chars sometimes.

Current coverage is 89.01%

Merging #254 into master will increase coverage by +0.04% as of d1b59a5

@@            master    #254   diff @@
======================================
  Files          189     191     +2
  Stmts         5975    6000    +25
  Branches         0       0
  Methods          0       0
======================================
+ Hit           5316    5341    +25
  Partial          0       0
  Missed         659     659

Review entire Coverage Diff as of d1b59a5

Powered by Codecov. Updated on successful CI builds.


Broken conversation link

The link in "Setup" of the last version is broken (404).

Thanks for letting me know, it has been fixed!

Your welcome :)


Remove deprecation note

Remove note (and then we'll delete the master branch as the note stated)

/lgtm

/lgtm


WARNING - UnicodeDecodeError of bytes b'\xad'

Hello,
When running with model ggml-gpt4all-j-v1.3-groovy.bin, I have a lot of {model.py:85} WARNING - UnicodeDecodeError of bytes b'\xad' in the INFO log.
This is related to an Unicode error in pygptj/model.py (line 85 as specified).
Not blocking as this is a warning.

Hello @pycloux,
It is the same issue as #76
Please update the package and give it a try now ?
I believe the issue should be solved.
(ensure you have pygptj version 1.0.10).

Hi @abdeladim-s,
Just tested.
It fixed the issue for me.
Thanks 👍


LINK inside FOR throwing 'INTERNAL_ERROR' in v3.1.1

Links were working fine (although without styling) in v3.0.0. I spotted you fixed the styling issue in 3.1.1 (thanks!) so I upgraded. Unfortunately I'm now getting an error from line 350 in processTempalate.js.
After a bit of debugging I've found that links are fine outside of FOR loops but throw the error when placed inside them. The link doesn't have to reference the FOR to cause the error,
+++LINK ({ url: 'http://www.test.com', label: 'Link' })+++
for example still causes the error to be thrown.
Thanks again for creating and maintaining this!

After looking through the code a bit more it seems like its being triggered on line 429:
'ctx.textRunPropsNode = nodeIn;'
Removing it fixes the error being thrown, but breaks the styling.

please fix it.

@Suley @piemasters i've built a test case for this in #108
I could not reproduce the error on the 4.0.0 release candidate. Can you verify that the test document I included in #108 contains the error case you mentioned?
If the test document resembles the one that errors out, then I guess the next major release will solve this issue. We'll release 4.0.0 soon. I'll leave the regression test case in.

@jjhbw Thanks. Really appreciate your hard working. I will try it later!

If the test in PR #108 correctly covers the case (let me know if it doesn't!) then I think this bug has been fixed with yesterday's 4.0.0 release. Let me know if it still occurs! I'll merge the regression test just to be sure.


Change default version to 4.1 for RHEL7 and derivatives and change the way the yum repo is declared.

This pull request contains two changes:

Change the way the yum repo is declared
This patch is needed because the default package.io module doesn't support RHEL 7 and derivatives and we don't want to depend on another module if it is not needed.

Default version for RHEL & derivatives is changed to 4.1
The packages in EPEL and Packagecloud are broken for version 7. 4.1 fixes these problems.

Hi,
Thanks for the pull request, I'm aware of the yumrepo issue - I have a branch that is being worked on at the moment to fix that, it should be ready for merging early next week (time permitting of course).
Thanks,
Craig

Hi,
Thanks for message, looking forward to it.
Rik


Fix for Parkmobile app broken/not loading when any unified hosts applied

I was confused as to why parkmobile app suddenly quit working one day on my android device. I eventually narrowed it down to something in the master hosts file and found it to be this entry:
api.branch.io
Whitelisting the above solved the problem and the app worked normally. Can this entry be removed from the hosts? Thanks

Hi @Zackptg5 thank you for researching this.
The branch.io domain is adware, analytics, mobile revenue, etc.
Domain api.branch.io was added in commit 1e52f6dd0d92b5d946 in April 2018.
I don't see that changing, Zac.  Convince me otherwise; make a case.   A hard dependency by Parkmobile would not make a case; it's a bug in Parkmobile, amirite?

I have no idea honestly. The tcpdump shows that it uses api.branch.io (present in master hosts) and cdn.branch.io (not present in master hosts - maybe you might want to add this one too if you're not going to remove api).
The app itself appears to use google maps so I'm wondering if the branch.io is used in relation to that and mapping out the user's location?
Parkmobile seems to use localytics and crashlytics for most of it's ad related tracking though which are blocked fine. Below are the logcats of opening the app with api.branch.io blocked and not blocked if you want to look at the difference. In the blocked one, this line stands out to me: 03-14 17:24:57.223 26308 26308 I BRANCH SDK: Trouble initializing Branch.  Branch API Error: poor network connectivity. Please try again later.
After this line, the app pretty much dies and that's when I close it so my guess would be that this behavior is intentional. It makes it sound like it's required for the functionality of the app but I don't know for sure
BranchBlocked-StuckOnSplashScreen.txt
BranchNotBlocked-Works.txt


Semgrep

Notes

A common pattern is to read and write file paths obtained from the
platform's vault API. There's around 100 plus apps that do something like
def get_file_path():
    files = phantom.get_vault_info(...)
    for f in files:
       if CONDITION:
          return = f['file_path']
 ...
def handle_action():
    fp = get_file_path()
    ...
    with open(fp) as f:
        ...

and Semgrep isn't able to detect the call to the phantom.get_vault_info(...) from within the call stack. Since there's too many false positives with the current filesystem-access check, and I haven't figured out a way to support the above pattern, I'm removing the rule for now.

Adding a rule to check for https://splunk.atlassian.net/browse/PAPP-24779

Updating logging-sensitive-data rule to not consider string literals

did you reopen a ticket for it?
I realized it would be simpler if we just ran these rules through the github pipeline, then we don't need to open a separate MR with ProdSec and wait for them to update their Docker image

also, while not ideal what do you think about preemptively adding the semgrep ignore comment to all the existing cases (basically push it to the next branch) and letting people add the exception when they introduce new instances of it? I ask since the alternative would be manually having to check this, no?

The issue would be manually combing through the findings to distinguish between false/true positive 😅 We can add a related question to the PR template for now. Ideally, filesystem access should be control by the run-time environment and not statically 😦

Ideally, filesystem access should be control by the run-time environment and not statically

that's a good point. especially now that we've had a push to be as hands off as possible in the developer supported community apps. kind of raises a more general platform architecture question of what degree of sandboxing we have. it's not the end of the world if an app nukes itself or its own state but it really shouldn't be able to mess with other apps or the rest of the system's code


Add Arabic translations

Thanks to @nasser-glx!

Yeah, I recommend you create your own PR so you can make edits

The Arabic translation could use some polishing. Will find time to give some suggestions

Sounds good, feel free to create your own PR so you can edit it

Moved to https://github.com/commaai/openpilot/pull/25153


How to use runtimeConfig

Hello,
i wonder if its possible to use the Nuxt RuntimeConfig with electron and how.
The .Env gets installed  path/to/installation/resource/.env but changing them does not do anything.
I need to set on the client a global variable, an ID eg.:  "Client1"
The official electron docs arent helpful with runtime configurations. Do you know a way to make use of it?

when generating the app, there should be an index.html near the path/to/installation/resource/.././
In that index.html are all variables inside a script section.


Fix "time range in seconds" calculation in flow query stats

Also remove time range props from NetflowTopology component as it isn't used there anymore
Related to NETOBSERV-602

/ok-to-test

pre-merge tested.
/qe-approved

/approve


waitForStatefulsets in e2eutils

Description of the change:

I am writing e2e tests for https://github.com/druid-io/druid-operator, we need to check the status of both deployments as well as statefulsets. As of now the e2eutil only had a function for supporting waitForDeployments.
This PR does the same action for statefulsets.
Checklist
If the pull request includes user-facing changes, extra documentation is required:
[ ] Add a new changelog fragment in changelog/fragments (see changelog/fragments/00-template.yaml)
[ ] Add or update relevant sections of the docs website in website/content/en/docs

I would agree with @camilamacedo86.
The v0.19.x branch is receiving only bug fixes at this time.

@joelanford @camilamacedo86
Thanks for your reply.
Few doubts i have for e2e in case i want to run them,  does that mean these tests arn't available for 1.x.x ?
I mean where is the code base for e2e for 1.x.x ? dint find the e2e utils in here https://github.com/operator-framework/operator-lib
Thank you :)


Update domains.json

I am unable to initiate pull request for adding awake.eth to the subdomain contract.

Why?

I transferred domain to contract, but when I put a pull request it won't
let me.  I must not be on the correct subscription.
James K. McCall
On Thu, Oct 31, 2019, 5:02 PM Nick Johnson<EMAIL_ADDRESS>wrote:

Why?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ensdomains/subdomain-registrar/issues/84?email_source=notifications&email_token=ACR77NNV456LAKVMSRD5XZLQRNPW7A5CNFSM4JFSN3GKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECZQEYA#issuecomment-548602464,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACR77NMIQ2BBZKKIDDI4PPLQRNPW7ANCNFSM4JFSN3GA
.

No "subscription" is required to submit a PR to GitHub. I can't help unless you can be more specific than "it won't let me".

I go to the pull request at
https://github.com/ensdomains/subdomain-registrar/pulls
I am able to click on the new pull request button.

The create pull request button wasn't illuminated.  I have tried to copy
what was done earlier.  I was able to create pull request in gh-pages.
Apologies, for making a mess of it.

On Thu, Oct 31, 2019 at 7:36 PM Nick Johnson<EMAIL_ADDRESS>wrote:

No "subscription" is required to submit a PR to GitHub. I can't help
unless you can be more specific than "it won't let me".
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ensdomains/subdomain-registrar/issues/84?email_source=notifications&email_token=ACR77NPUHWDNJRWR62DPOK3QROB2FA5CNFSM4JFSN3GKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECZXPIQ#issuecomment-548632482,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACR77NIDS6LGYJPBDAEUGYDQROB2FANCNFSM4JFSN3GA
.

--
James McCall | Origination and Servicing | McFinney Agri-Finance, LLC |
Website http://www.mcfinney.farm |  vcard
https://s3-us-west-2.amazonaws.com/vcardsjkm/James++McCall.vcf


Please update the README.md to include more information about the purpose of this repository

I syncronised the info with the DAS Board site (http://dasboard.eu/specifications/sip).


Fixes for building on Windows

Use dlfcn-win32 for dlopen and friends
Use official CMake find_package for LLVM and Python (only on Win32)
Replaced a few POSIX-only functions with their portable equivalents.
Use _popen and family on Windows instead of popen

Hey! Thanks for the PR. There seems to be minor issues that prevent the linux version from building, but happy to merge when it passes. Getting it integrated with the CI will be harder though, as it requires to run the windows on a different agent

@ptillet I was planning to just build on GitHub hosted machines.
The problem is that I would have to supply a prebuild LLVM somehow, because the one distributed officially does not include CMake config files for some unknown to me reasons. If you'd be fine for me to package my own build and upload to GitHub releases to be downloaded like the one setup.py downloads for Ubuntu, that would be stellar.

@lostmsu Would you mind discussing it more directly in the Slack ? You can join here: https://join.slack.com/t/triton-lang/shared_invite/zt-yjvv7235-uhvujP1Jb5KGlM1cMltEog. For security reasons, I would refrain from using third-party binaries in the Triton build process -- LLVM official binaries being an exception to that. Maybe we can find a way around the missing CMake config files

This is still ready for CI.


Generate SQLAlchemy models based on the current schema, using SQLAcodegen

I ran the model generation, surprisingly it did not end up generating Table definitions at all.
It still lacks of some Validation (However in some places it managed to generate those checks) however some of the checks defined on the Django side still need to be implemented. (Most of them are fairly easy though). Although, I believe opening a Draft Pull Request to discuss further things to do/implement is vital.
Originally, everything was in one file, so I took the liberty and split them into multiple files. It had some issues with DeletedMessages and Messages but I believe I managed to resolve this.
Closes  #16

Should we remove the Api prefix from models, and add uppercase letters where reasonable? Such as ApiOfftopicchannelname -> OffTopicChannelName?
I think multiple modules is fine, but we should have an __init__.py in there importing them so we can import the models from a single import.

Yes, sorry. The class names
were generated that way, there is a naming convention of some sort.
I'll remove the API prefix and add inits. Also, how does the overall picture looks like? It had some problem with the messages module, but I fixed it, I believe.

Should we remove the Api prefix from models, and add uppercase letters where reasonable? Such as ApiOfftopicchannelname -> OffTopicChannelName?
I think multiple modules is fine, but we should have an __init__.py in there importing them so we can import the models from a single import.

I've added the changes mentioned, however I'm not sure I understand why the max line length is 88.

Yes, sorry. The class names were generated that way, there is a naming convention of some sort.

No need to be sorry at all 🙂 I realized there was some convention. Thanks for updating it!

Also, how does the overall picture looks like? It had some problems with the messages module, but I fixed it, I believe.

With overall picture, do you mean whether this can un-drafted? From a quick view, it looks good to me, I can give it a thorough review once it's ready to go.

Sorry, that was the wrong button....

Yes, sorry. The class names were generated that way, there is a naming convention of some sort.

No need to be sorry at all 🙂 I realized there was some convention. Thanks for updating it!

Of course, it was a bit inconsistent with this prefix.

Also, how does the overall picture looks like? It had some problems with the messages module, but I fixed it, I believe.

With overall picture, do you mean whether this can un-drafted? From a quick view, it looks good to me, I can give it a thorough review once it's ready to go.

There's only one check remaining that has to be implemented, and that is the check in the User model that ensures that the role being added is an existing Role. For that, I have to create a session, and in order to create this session I'll need a database connection string, so I believe we might have to add it to the environment variables. However, after that and testing the checks out, it should be ready to go! 👍

However it's not the scope of this PR, would it mean a problem if I added PostgreSQL to the docker-compose file, and the drivers to the poetry environment? @jchristgit
(I have the changes prepared locally, since I was testing the models, but haven't added them to the PR just yet.)

However it's not the scope of this PR, would it mean a problem if I added PostgreSQL to the docker-compose file, and the drivers to the poetry environment? @jchristgit

No, not at all, go PostgreSQL!!! 🐘

However it's not the scope of this PR, would it mean a problem if I added PostgreSQL to the docker-compose file, and the drivers to the poetry environment? @jchristgit

No, not at all, go PostgreSQL!!! 🐘

Alright! 👍
Apart from the linting errors(Should we change the max-line-length to 120?) , I believe it's done. I have tested out few of the checks and relationships, they seem to work just fine.

This PR looks good to me now. Two small points:

Could you look into fixing CI for our tests to pass?
Since we now have the model definition in two projects, it would be great to maybe add some form of reminder to the site pull requests about whether they update the model definitions in a way that requires an update here. I do not think we should put a freeze to model changes on the old site, at all. First off, it's hard to implement, and second we still want to be able to make modifications. @jb3 would it be possible to create a pull request template in the site repository that goes something like "If you have updated the models: [] I have created a corresponding PR to update the models in the API project, [] I have not" or something.

This PR looks good to me now. Two small points:
* Could you look into fixing CI for our tests to pass?

* Since we now have the model definition in two projects, it would be great to maybe add some form of reminder to the site pull requests about whether they update the model definitions in a way that requires an update here. I **do not** think we should put a freeze to model changes on the old site, at all. First off, it's hard to implement, and second we still want to be able to make modifications. @jb3 would it be possible to create a pull request template in the site repository that goes something like "If you have updated the models: [] I have created a corresponding PR to update the models in the API project, [] I have not" or something.

Could you look into fixing CI for our tests to pass?
As for now, it wouldn't make sense to add Postgres to the CI, since we don't have any relating tests. As a possible solution, maybe I could make "database_url" Optional for now?
Also, am I also supposed to fix linting as well? I've added a new configuration that is similar to the ones in other pydis projects, should I PR a new config?

As for now, it wouldn't make sense to add Postgres to the CI, since we don't have any relating tests. As a possible solution, maybe I could make "database_url" Optional for now?

Could we get away with setting that to some bogus value for now? I don't think making it optional makes sense from a deployment nor testing perspective. 99% of what the API does needs the database.

Also, am I also supposed to fix linting as well? I've added a new configuration that is similar to the ones used in other pydis projects, should I PR the new config? (I mean flake config)

Yes, feel free to add it into this PR.

As for now, it wouldn't make sense to add Postgres to the CI, since we don't have any relating tests. As a possible solution, maybe I could make "database_url" Optional for now?

Could we get away with setting that to some bogus value for now? I don't think making it optional makes sense from a deployment nor testing perspective. 99% of what the API does needs the database.

Also, am I also supposed to fix linting as well? I've added a new configuration that is similar to the ones used in other pydis projects, should I PR the new config? (I mean flake config)

Yes, feel free to add it into this PR.

I've already added, and based on that config the linting should pass.
@jchristgit


The app crashing after pressing certain keys

The program works perfectly with all keys except one(in MacOS) that is the esc key. As soon as one presses the esc key,
thread '<unnamed>' panicked at /Users/purushottamsharma/.cargo/registry/src/index.crates.io-6f17d22bba15001f/rustyvibes-1.0.9/src/play_sound.rs:53:82: called Result::unwrap()on anErrvalue: Os { code: 2, kind: NotFound, message: "No such file or directory" } note: run withRUST_BACKTRACE=1environment variable to display a backtrace thread '<unnamed>' panicked at /Users/purushottamsharma/.cargo/registry/src/index.crates.io-6f17d22bba15001f/rustyvibes-1.0.9/src/play_sound.rs:49:60: calledResult::unwrap()on anErr value: PoisonError { .. }
such errors pop up and no further key press produces any sound.

might be an issue with a particular soundpack, "esc" key seems to be working for me with the soundpack I use

Can you please tell a fix for it? I am using the cherrymx-blue-pbt sound pack.


On iOS, using DatePickerDialog with showGeneralDialog function causes the dialog to have a weird scrolling behavior

Steps to reproduce
I am trying to make an animation for dialog popups in my app to blur the background with a smooth animation as they popup. For this I used the showGeneralDialog function instead of the usual showDialog as it has a transitionBuilder.
To reproduce:
Simply call showGeneralDialog and inside its builder callback return a DatePickerDialog.
Expected results
when calling with showDialog

Actual results
when calling with showGeneralDialog
iOS:

Android:

As you can see for some reason the day selection buttons seem to be initially scrolled down on iOS
Code sample
Here is a small example showing my exact usecase with the desired blur animation
Code sample
import 'dart:ui';

import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        colorScheme: ColorScheme.fromSeed(seedColor: Colors.deepPurple),
        useMaterial3: true,
      ),
      home: const MyHomePage(title: 'Flutter Date Picker bug'),
    );
  }
}

class MyHomePage extends StatelessWidget {
  const MyHomePage({super.key, required this.title});

  final String title;

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        backgroundColor: Theme.of(context).colorScheme.inversePrimary,
        title: Text(title),
      ),
      body: const SizedBox.shrink(),
      floatingActionButton: FloatingActionButton(
        onPressed: () {
          showBlurredBackgroundPopUp(
            context: context,
            builder: (context) {
              return DatePickerDialog(
                firstDate: DateTime.now(),
                lastDate: DateTime.now().add(const Duration(days: 365)),
              );
            },
          );
        },
        child: const Icon(Icons.calendar_month_rounded),
      ),
    );
  }
}

Future<T?> showBlurredBackgroundPopUp<T extends Object?>({
  required BuildContext context,
  required Widget Function(BuildContext context) builder,
}) async {
  return await showGeneralDialog<T>(
    context: context,
    barrierDismissible: true,
    barrierLabel: '',
    barrierColor: Colors.transparent,
    transitionDuration: const Duration(milliseconds: 500),
    pageBuilder: (context, anim1, anim2) {
      return builder(context);
    },
    transitionBuilder: (context, anim1, anim2, child) {
      final double blurAnimationValue = Curves.easeOut.transform(anim1.value);
      return BackdropFilter(
        filter: ImageFilter.blur(sigmaX: 10 * blurAnimationValue, sigmaY: 10 * blurAnimationValue),
        child: FadeTransition(
          opacity: anim1,
          child: child,
        ),
      );
    },
  );
}

Screenshots or Video
No response
Logs
No response
Flutter Doctor output
Doctor output
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.24.5, on macOS 15.1 24B83 darwin-arm64, locale en-EG)
[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 16.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2024.2)
[✓] VS Code (version 1.95.3)
[✓] Connected device (6 available)
[✓] Network resources

• No issues found!

Thanks for the report @ahmednoaman-git
I do see the same behavior as below:

Although we need to confirm if this is occuring due to blur effect applied in the background or not. Can you try without blur effect or BackDropFilter to check if the behavior still persist or not ? Also, I tried with official example with which the cut-off at the bottom doesn't occur:

Yes, @darshankawar. Using just the plain function showGeneralDialog leads to the same behavior
import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        colorScheme: ColorScheme.fromSeed(seedColor: Colors.deepPurple),
        useMaterial3: true,
      ),
      home: const MyHomePage(title: 'Flutter Date Picker bug'),
    );
  }
}

class MyHomePage extends StatelessWidget {
  const MyHomePage({super.key, required this.title});

  final String title;

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        backgroundColor: Theme.of(context).colorScheme.inversePrimary,
        title: Text(title),
      ),
      body: const SizedBox.shrink(),
      floatingActionButton: FloatingActionButton(
        onPressed: () {
          showGeneralDialog(
            context: context,
            pageBuilder: (context, anim1, anim2) {
              return DatePickerDialog(
                firstDate: DateTime.now(),
                lastDate: DateTime.now().add(const Duration(days: 365)),
              );
            },
          );
        },
        child: const Icon(Icons.calendar_month_rounded),
      ),
    );
  }
}

@darshankawar The official example works because showDatePicker dialog is internally just calling showDialog rather than showGeneralDialog

Thanks for the update. Please see https://github.com/flutter/flutter/issues/130691#issuecomment-1639675057 reasoning as why it doesn't work with showGeneralDialog and we should be using showDialog as it supports M3. You may also try to wrap showGeneralDialog with SafeArea to see if it works.
But based on the linked comment, I will close this as not an issue. If you disagree, write in comments and I'll reopen it.


stdcommands: improve 8ball command

Remove the AYLIEN plugin, because it is broken anyway.
The API is either dead or inaccessable, so there is no point in keeping
it around.
Secondly, re-add the 8ball command formerly under aylien/ to stdcommands.
Improve it by actually having more answers than a yes or no as it formerly
was due to AYLIEN being broken.
Signed-off-by: Luca Zeuch<EMAIL_ADDRESS>

it would be an august move to post eightball code as well : )

it would be an august move to post eightball code as well : )

Haha, yeah. Forgot to add these. Pushing now.


opt add pick rowset to compact interval config

Proposed changes
Make pick rowset to compact interval configurable.
Problem summary
Describe your changes.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run buildall

run buildall

run buildall

run buildall

run buildall


UI - combine raw and tidy into the same tab, only show tidy?

I don't think the distinction between raw vs tidy is going to be very obvious to users.
One option would be to have them in the same tab with a button to go from one to the other.
Another option would be to only show tidy, and allow raw to be downloaded as a csv. I'm not sure how we would expect users to interact with the raw tab at the moment (other than they might want to download it)

decision: don't show raw, but have button to download somewhere (in the summary tab?) using exportSummarisedResult function


Implement parsing of the place

Whois, metadata.

https://github.com/joepie91/python-whois
https://github.com/secynic/ipwhois


Operations between numpy.array and scipy.sparse matrix return inconsistent array type

It appears that adding or subtracting numpy.ndarrays with scipy.sparse matrices returns a numpy.matrix.
Is the inconsistency in returned array type (see code below) a bug or is it intentional?
It is confusing because different operations return either numpy arrays or numpy matrices, and since we are working with arrays that are often inter-converted between sparse and dense, it would be very helpful to have consistency in output array type (ie. operations between sparse and dense always return either a numpy array or always a numpy matrix.)
It makes sense that operations between like types return the same type as the input type. However, it would be nice if operations between numpy.arrays and scipy.sparse matrices would return always the same array type.
Reproducing code example:
import numpy as np
from scipy.sparse import csc_matrix

mat = np.array([[1,1],
                [1,0]])
sp_mat = csc_matrix([[1,1],
                     [1,0]])

mat + mat
>>> array([[2, 2],
           [2, 0]])

sp_mat + sp_mat
>>> <2x2 sparse matrix of type '<class 'numpy.int64'>'
	with 3 stored elements in Compressed Sparse Column format>

mat + sp_mat
>>> matrix([[2, 2],
            [2, 0]], dtype=int64)

sp_mat * mat
>>> array([[2, 1],
           [1, 1]], dtype=int64)

sp_mat == mat
>>> matrix([[ True,  True],
            [ True,  True]], dtype=bool)

Scipy/Numpy/Python version information:
print(scipy.__version__, numpy.__version__, sys.version_info)
>>> 0.19.0 1.12.1 sys.version_info(major=3, minor=6, micro=0, releaselevel='final', serial=0)

For me, the issue here is that numpy.array + scipy.sparse returns a numpy.matrix. I think the ideal behavior is:

numpy.array + scipy.sparse returns a numpy.array
numpy.matrix + scipy.sparse returns a numpy.matrix

This issue always applies to subtraction. Actually the way we noticed it was from code like:
numpy_array -= scipy_sparse_matrix

This command changed the type of numpy_array to numpy.matrix which caused downstream problems.

In general, the rule we try to follow is: if you replace the sparse matrix with a numpy matrix, the resulting type should not change (unless the result itself is sparse).
Following this rule:

array -= sparse converting to matrix is certainly a bug (and I can reproduce it locally).
array + sparse returning matrix is working as intended (even though it's a bit strange).
array * sparse returning array is a bug.

In general, the rule we try to follow is: if you replace the sparse matrix with a numpy matrix, the resulting type should not change (unless the result itself is sparse).

@perimosocordiae so it sounds like goal is to imitate the type behavior of numpy.matrix. Let's work with the following definitions:
import numpy
from scipy.sparse import csc_matrix

data = [[1,1],
        [1,0]]

array = numpy.array(data)
matrix = numpy.matrix(data)
sparse = csc_matrix(data)

The following pure numpy operations all return a matrix: array + matrix, matrix + array, array - matrix, matrix - array. Hence according to @perimosocordiae's rule, array + sparse, sparse + array, array - sparse, sparse - array should all return a matrix (which they do). However, since array += matrix and array -= matrix, keep array as an array, so should array += sparse and array -= sparse.
For the dot product case, I will use @ rather than * for clarity. array @ matrix and matrix @ array both return a matrix. Hence, array @ sparse and sparse @ array should return a matrix, but instead they return an array.

(unless the result itself is sparse).

@perimosocordiae how do you know whether the output should be sparse? For example, why doesn't matrix @ sparse return sparse?
From the scipy.linalg docs:

Despite its convenience, the use of the numpy.matrix class is discouraged, since it adds nothing that cannot be accomplished with 2D numpy.ndarray objects, and may lead to a confusion of which class is being used.

This makes a lot of sense to me. In fact, for our project, we've tried to avoid using matrix at all. Primarily because we sometimes have to do elementwise multiplication. The separate operators for dot product and elementwise multiplication is helpful. However, scipy.sparse seems pretty committed to the matrix rather than ndarray design.
So my question is, for a project that mixes 2d-arrays and scipy.sparse matrices, should we just migrate from ndarray to matrix entirely? While this forces us to use the "discouraged" class, it's an uphill battle to use ndarrays when everything is always getting converted to matrices?

Your analysis looks correct to me.
We determine if the output is sparse without considering the contents of the operands, based on the worst-case behavior. Thus, sparse - dense is always dense, even if the particular dense operand happens to be mostly zeros. In the case of dot products, the worst-case is that the dense operand has no zeros and the sparse operand is full-rank, which will result in an entirely dense result.
In general, I agree that users should avoid numpy.matrix whenever possible. Unfortunately, the scipy.sparse API design decision to mimic numpy.matrix can't be changed now. There's a growing list of efforts to produce a sparse ndarray-like library that may someday make it into scipy, but that will take considerable time.
To answer your question, I suggest taking a close look at the operations producing your arrays/matrices, and deciding what the appropriate type is on a case-by-case basis. It's fairly common to have separate code paths for sparse vs dense inputs, as the time/space complexity concerns often change pretty significantly between these cases. If you find that you often have sparse matrices that end up densifying, it might be faster/cheaper to avoid the sparse representation altogether.

@perimosocordiae
Hi, is there any work going on to fix the bug you outlined above?

array -= sparse converting to matrix is certainly a bug

gh-7826 has more discussion of that particular problem, and why it's not trivial to fix. I like the suggestion in https://github.com/scipy/scipy/issues/7826#issuecomment-437642900 about raising a warning, though.
As far as I know there aren't any open PRs along those lines yet.

@perimosocordiae thanks!


on arm it downloads the x86 version...

Results in an error .asdf/installs/crystal/0.33.0/bin/shards: cannot execute binary file: Exec format error
Can be reproduce on raspberry pi...

I'm pretty sure I can fix it relatively quickly, but I don't have the bandwidth to handle this right now, could you wait a little longer? If the response is slow, don't hesitate to ping as needed. or, if you open a PR, I will take the time to review it.

it should be pretty easy to check that platform is either x86 or x86_64, but installing the arm version may be more difficult given that they do not distribute a binary for it.
After a bit of research it seems like cross compilation is needed for it to work.
Maybe a pre-compiled/cross-compiled version could be distributed here as a bootstrap and then used to compile the latest version from source ?


help

it wont let me open it or work at all i cant open it on the bookmark bar

Please elaborate on what you mean by "open it or work on it at all"

Also, which bookmarklet are you referring to?

I think I know. Thank you it has been fixed.


If Gson mapper is chosen, the oauth token returned is empty

Hi,
When I set the property spring.http.converters.preferred-json-mapper=gson in application.properties and add the gson dependency, the oauth token is not returned (the json returned is empty).
Setting the property to spring.http.converters.preferred-json-mapper=jackson makes it work again.
We use Gson as main parser and we cannot move to Jackson, do you have an idea about this issue?
I created a project for this issue (https://github.com/marcellom/sample-oauth2-spring-gson), you can clone it and follow the instructions in the README.
Dependencies:

Spring Boot: 1.5.6.RELEASE
Gson: 2.8.0
spring-security-oauth2: 2.2.0.RELEASE

Thank you in advance

I solved it by adding a custom GSON serializer. I can close this issue


feat: added warning if app already deployed

I think the point raised by @FabienArcellier is important.
@raaymax I'm guessing it makes more sense to merge this PR and then implement in CLI refactor PR though, right?

agree, with both

--quitet is important I will add it.
first merge this and then add it to click

@ramedina86 can we merge this one?

I was just waiting for --quiet has this been implemented?

Ach a little miss understanding 😅 I will add it to click PR


completion errors

I started to get these errors since lately and it's very annoying, because it asks to reload my current file because of that and if I agree, I'm losing my changes...
Traceback (most recent call last):
  File "C:\Users\korostelev\sublime\sublime_plugin.py", line 304, in on_activated
    callback.on_activated(v)
  File "C:\Users\korostelev\sublime\Data\Packages\Haxe\HaxeComplete.py", line 530, in on_activated
    return self.on_open_file( view )
  File "C:\Users\korostelev\sublime\Data\Packages\Haxe\HaxeComplete.py", line 550, in on_open_file
    self.highlight_errors( view )
  File "C:\Users\korostelev\sublime\Data\Packages\Haxe\HaxeComplete.py", line 504, in highlight_errors
    right = len(line[:right].decode("utf-8"))
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xd0 in position 4: unexpected end of data

Can't reproduce.
Can you provide some test project?

Temporarily wrapped the code that is causing the issue in try block.