[SPARK-13781][SQL] Use ExpressionSets in ConstraintPropagationSuite

What changes were proposed in this pull request?
This PR is a small follow up on https://github.com/apache/spark/pull/11338 (https://issues.apache.org/jira/browse/SPARK-13092) to use ExpressionSets as part of the verification logic in ConstraintPropagationSuite.
How was this patch tested?
No new tests added. Just changes the verification logic in ConstraintPropagationSuite.

cc @marmbrus @yhuai

cc @marmbrus @yhuai

Test build #52765 has started for PR 11611 at commit 1872334.

LGTM pending tests

Test build #52765 has finished for PR 11611 at commit 1872334.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/52765/
Test PASSed.

Merged to master.


Replace std::fs to tokio::fs.

This approach is hard to fix the issue. I'll retry with other methods.


Option to clear or disable cache when using lazy:true

Describe the feature
It would be great if there could be an option to disable cache when using lazy:true.
I have a problem where i use mergeLocaleMessage() in my code on some route changes.
Why because, I like to fetch translations based on route.
In my scenario, i have A LOT of pages with A LOT of translations for each page and it would be expensive to load them all on initial page load.
So now I am fetching translations on route change and then using mergeLocaleMessage() to add them.
But the cache breaks everything if the initial route is different. It loads only translations on client-side.
Little basic reproduction: https://stackblitz.com/edit/nuxt-starter-nus1pf?file=app.vue
Additional information

[ ] Would you be willing to help implement this feature?
[ ] Could this feature be implemented as a module?

Final checks

[X] Read the contribution guide (The contribution guideline of nuxt-modules/i18n is compliant with Nuxt too).
[X] Check existing discussions and issues.

I could use this feature as well.

Any updates on this?

This would be useful for me too.


Libretro compilation issue on Windows using msys2

Hi,
I tried to compile the core using msys2
git clone --recursive https://github.com/tiberiusbrown/Ardens
cd Ardens
mkdir build && cd build
cmake -DCMAKE_BUILD_TYPE=Release -DARDENS_LLVM=0 -DARDENS_DEBUGGER=0 -DARDENS_PLAYER=0 -DARDENS_LIBRETRO=1 ..
make -j$(nproc)

But I could not launch the core with error failed to load the core.
According to Dependencies, it seems it does not find :

Should not we add -static-libgcc -static-libstdc++ ?
Thank you.

Hi, thanks for letting me know. Are you willing to test the msys2_static_libgcc branch for me with your MSYS2 install? If it works, I will merge it.

Hi,
I confirm it is working using msys2_static_libgcc branch.
ardens_libretro.dll if needed.
Thank you.


Plug-in ended with uncaught signal: 5)

Describe the bug
Adding the package to my iOS 16 project causes the error Plug-in ended with uncaught signal: 5)
To Reproduce
Steps to reproduce the behavior:

Add Swiftdown as a dependency via the Xcode dependency manager
See error

Additional context
I'm using Version 14.3.1 on a project for iOS 16

It is an issue with SwiftLint, you can install it through brew (I am actually not sure if you need to do that), as for the plug-in issue refer to this https://github.com/realm/SwiftLint/issues/1466#issuecomment-297265863
It helped me with my local environment, but unfortunately I have encountered this exactly issue when deploying to Xcode Cloud.


Fixes spaces in run command argument paths

Fixes running the workspace as a godot project or editing the workspace in the godot editor.  In PowerShell, paths with spaces in the argument would cause the commands to fail.

Just a heads up, your second change (replacing the backslash with a tick mark) doesn't work on unix based systems. I'm guessing that's why this hasn't been merged yet. Perhaps the editor path should be quoted, just like you did with this._rootDir, then hopefully we'll have compatibility with both Windows/Unix.
At least that's how I fixed it locally on my macOS system (quoting both paths) and it works nicely.

Hello,
This was actually something I tested first and was unable to get working properly.  PowerShell does not seem to like encapsulating argument 0 in quote marks from vscode;  it refuses to execute a fully qualified path specified this way and I'm not exactly sure why.
Perhaps there's a way to branch this block to execute only in w Windows environment, but I'm not sure exactly how to do that in typescript.  I'm open to suggestions for a better workaround!

Here is more information.  Vscode ignores the $SHELL environment variable on Windows in lieu of defaulting to PowerShell; this is probably by design and won't be fixed.
https://code.visualstudio.com/docs/editor/integrated-terminal
A workaround for Windows users may be to change terminal.integrated.shell to point at ubuntu bash, but this requires the user to install the Linux subsystem for Windows 10.

@nobuyukinyuu  Infact lot of windows user still using Windows7 and use cmd.exe as the default terminal of VSCode. So we need a common way to solve this problem on windows.

I can close the pull request and file this under issues instead if you prefer.  I'm a bit inexperienced and it's unlikely that I'll be updating this branch to support all the possibilities, since I'm not sure the best way to handle it.
As you've pointed out that cmd.exe, ps and bash all have different ways of parsing the arguments.  The executable name may not be guaranteed, but it may be possible to check the value of terminal.integrated.shell within the extension to determine how to modify spaces into literals before passing it to the integrated terminal.

After thinking over your comments, I believe I've built a working solution. I just tested it under Windows using both cmd.exe and PowerShell, then on macOS using bash and all succeeded. Happy to share it here or do a separate pull request, whichever is appropriate.

Thanks for your contibution! I'm merging #56 to solve this problem.


[SPARKTA-274] Fault tolerant

Description
With this PR Sparkta should create from a checkpoint path a SparkStreamingContext with the saved data.

+1
Jira 2w jaja

We should delete the policy checkpoint when updating the policy

test-at ISocketOParquetOperatorsIT fails


issue 1328 Docu keywords

No coding changed, just docu for keywords
go_to
type_secret
select_options_by
drag_and_drop_by_coordinates

Fixes #1328

There is small space issue, could you fix it and then this ready to go.


Appsync custom lambda pipeline resolver

Which Category is your question related to?
Amplify CLI Version
4.29.1
What AWS Services are you utilizing?
AppSync, Lambda
Provide additional details e.g. code snippets
I have my resolvers generated for me via the graphql directive @model
type MyType @model

I'd like to prepend custom lambda resolver before the VTL mapper to achieve following scenario
1. a graphql mutation is invoked mutation CreateMyType(
    $input: CreateMyTypeInput!
    $condition: ModelMyTypeConditionInput
  )

2. My custom lambda function which would return additional attributes

3. Amplify VTL mapper, where I could reference the previous lambda result as $ctx.prev.result

Background - to create a new model I need some attributes to be either fetched or calculated in my lambda. I also want to use the generated VTL resolver.
Is it achievable in CLI to combine these functionalities?

@janfabian
You can provide custom resolvers to override the CLI generated ones.
But I don't think use you can use lambda function to dynamically add fields into the definition of a type annotated by @model.

@UnleashedMind thanks for the response


New extension

Describe the extension
Extension for connecting SDK Yandex.Games (https://yandex.ru/games/docs/). The extension provides the following features:
integration of ad display (full-screen banner, rewarded ad);
automatic switch to full screen mode;
setting the screen orientation at the start of the game;
Checklist

Extension has a proper name and description.
Extension has tags (for example: "platform, brick, breakable").
All behaviors have a description.
All functions (actions, conditions, expressions) have descriptions.

YaGamesSDK_Extension.zip
I confirm that this extension can be intergrated to this GitHub repository, distributed and MIT licensed.
Extension file
Finally, attach the .json file of your extension here.
You also may have to create an account on GitHub before posting.
If your extension is high quality and useful, it will be added to the list of GDevelop community extensions.
When you're ready, remove this last paragraph and click on "Submit new issue". Thanks 🙌

Nice :)
I've made a few fixes:

Don't use _variables, or anything that has an underscore. It's private and will break in the future. Instead, use getVariables().
Careful about the indentation of your code, it's cleaner when all brackets are properly indented and much quicker to understand.
I've added support for the next version of GDevelop to use "booleans" (True/False or Yes/No) parameters, so I've changed the extension to use them for the parameters of the action to initialize the SDK.

I've published it, thanks again! 👍

Careful about the indentation of your code, it's cleaner when all brackets are properly indented and much quicker to understand.

Little tip: you can type ctrl-shift-f to let Monaco reindent the code for you.

I've added support for the next version of GDevelop to use "booleans" (True/False or Yes/No)

I can't seem to find that commit on the GDevelop repository 🤔 did you already push it?

It's here :) https://github.com/4ian/GDevelop-extensions/commit/2d45c763c2c120cdc6c6394c875b8fa95e78bce5

Little tip: you can type ctrl-shift-f to let Monaco reindent the code for you.

Nice! I did not know about this :)

I meant the commit to pass booleans as functions parameter on the main GDevelop repository

Oops forgot to push it 😄 It's here: https://github.com/4ian/GDevelop/commit/a0e0fdf6e1cfe965dff9fb976d135bd8c9517716


Update use cases

I think we have many more use cases out there, but just added a few of the larger ones that could be of interest and where we are allowed to share

Very nice thank you very much for the contribution 🥰


Master

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Parameters used
Expected behavior
Observed behavior
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Platform
What was the B&R Automation version used when bug was observed
Additional context
Add any other context about the problem here.

[Aplicar]


APK download

Nice to see a tracking-free FOSS app! Will there be a way to download the APK if one has no access to Play Store? One way would be to tag your releases and attach the APK to the tag.
One more question: What for does the app need the Internet permission if it is just tracking the user's mood?

Thanks for bringing this to my attention.
I've tagged the current release with an APK and will be sure to do so for any future releases.
MiniMoods requests internet permissions for crash management.

Thanks a lot! Yes, I see the app is using Sentry.io. I didn't check it out yet, but I guess you made that opt-in (so reporting is done with the users' consent). If I guessed right, your app should qualify for listing at F-Droid – the place where the free/libre open-source Android apps live. If you like, I can help you getting it there. Until then: with the next sync in about 11h from now, your app will become available in my repo here. If that suits you, you can add this badge to your Readme to link there, next go the Play Store badge :smiley:
Thanks again, and all the best for your project!

@CampbellMG can you confirm Sentry is configured opt-in (i.e. the user is consulted BEFORE sending any data)? Because then I could remove the Tracking AntiFeature from your app's listing.


Feature/spm manually set bundleresourcepath

Swift Package Manager (SPM) additional PR.
This PR adds one commit over bfernandesbfs SPM PR
This adds a new bundleResourcePath variable to FBSnapshotTestController which can be manually set as the root folder where reference images are stored, which is needed for SPM. After this change there are now 4 ways of setting reference image directories instead of 3:

Set the preprocessor macro FB_REFERENCE_IMAGE_DIR to a double quoted c-string with the path. This only works for Objective-C tests.
Set an environment variable named FB_REFERENCE_IMAGE_DIR with the path. This takes precedence over the preprocessor macro to allow for run-time override.
(New way) Set FBSnapshotTestCase.bundleResourcePath as the root folder where reference images are stored.
Keep everything unset, which will cause the reference images to be looked up inside the bundle holding the current test, in the Resources/ReferenceImages_* directories.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: bfernandesbfs:x: stareque-atlassianYou have signed the CLA already but the status is still pending? Let us recheck it.

@bfernandesbfs @alanzeino

Feel free to make this as a PR in https://github.com/alanzeino/ios-snapshot-test-case


issue with constantLine and threshold functions while requesting data in json format.

Hi, There is an issue that graphite does not return atleast two values for the constantLine and threshold functions
Question : http://stackoverflow.com/questions/20656683/missing-writing-values-in-json-format?noredirect=1#comment30925172_20656683

related: #236


Caddy's import path has changed

Caddy's import path (and Go module name) has changed from
github.com/mholt/caddy

to
github.com/caddyserver/caddy

Unfortunately, Go modules are not yet mature enough to handle a change like this (see https://golang.org/issue/26904 - "haven't implemented that part yet" but high on priority list for Go 1.14) which caught me off-guard. Using Go module's replace feature didn't act the way I expected, either. Caddy now fails to build with plugins until they update their import paths.
I've hacked a fix into the build server, so downloading Caddy with your plugin from our website should continue working without any changes on your part, for now. However, please take a moment and update your import paths, and do a new deploy on the website, because the workaround involves ignoring module checksums and performing a delicate recursive search-and-replace.
I'm terribly sorry about this. I did a number of tests and dry-runs to ensure the change would be smooth, but apparently some unknown combination of GOPATH, Go modules' lack of maturity, and other hidden variables in the system or environment must have covered up something I missed.
This bash script should make it easy (run it from your project's top-level directory):
find . -name '*.go' | while read -r f; do
	sed -i.bak 's/\/mholt\/caddy/\/caddyserver\/caddy/g' $f && rm $f.bak
done

We use this script in the build server as part of the temporary workaround.
Let me know if you have any questions! Sorry again for the inconvenience.

Hi @mholt , thanks for the informing us :)
I'm using Windows so I cannot run your command. But I just simply replace all the github.com/mholt/caddy occurrence to github.com/caddyserver/caddy in: https://github.com/casbin/caddy-authz/commit/d21039f3f56735e3072883f6b75c4ce587061284.  I hope this fixed the issue.

Aye, that should do it. Thank you!! And sorry again.


BUG: Missing modality tag when reading meta-image format.

The modality of a meta image is not stored as part of the metadata dictionary and was not copied into the ITK metadata dictionary. This change addresses the issue.

Thanks @zivy I'm sorry I'm not very familiar with this part of ITK! So a few questions:

Doesn't the modality already get "encapsulated" as par of the "AdditionalReadFields"? Here: https://github.com/InsightSoftwareConsortium/ITK/blob/1780a31476d63a3f67b97c50037bf8880224f3b9/Modules/IO/Meta/src/itkMetaImageIO.cxx#L408-L416
How does VTK (vtkMetaImageReader, vtkMetaImageWriter) deal with the Modality?
Are there other fields that might also need to be encapsulated, like PatientName, ScanID, etc? Or is specifically the Modality missing?

Hello @N-Dekker,
The "Modality" tag is treated in a unique manner by the metaIO module, converted from string and stored as enum here, so it is separate from the additional read fields stored here. As a result a MetaImage always has a modality (often MET_MOD_UNKNOWN) (see values here) and we can expect to always get a value even if it just means "don't know".
I have no idea why the modality was singled out in this way, possibly @aylward may remember though looks like the decision was made 22 years ago which is amazing in terms of software longevity but the rational may be lost in the mists of time :foggy: .

The basis was that every image has a modality.   So, it shouldn't be
optional meta data.
On Thu, Mar 28, 2024 at 4:25 PM Ziv Yaniv @.***> wrote:

Hello @N-Dekker https://github.com/N-Dekker,
The "Modality" tag is treated in a unique manner by the metaIO module,
converted from string and stored as enum here
https://github.com/InsightSoftwareConsortium/ITK/blob/1780a31476d63a3f67b97c50037bf8880224f3b9/Modules/ThirdParty/MetaIO/src/MetaIO/src/metaImage.h#L354,
so it is separate from the additional read fields stored here
https://github.com/InsightSoftwareConsortium/ITK/blob/1780a31476d63a3f67b97c50037bf8880224f3b9/Modules/ThirdParty/MetaIO/src/MetaIO/src/metaObject.h#L44.
As a result a MetaImage always has a modality (often MET_MOD_UNKNOWN) (see
values here
https://github.com/InsightSoftwareConsortium/ITK/blob/1780a31476d63a3f67b97c50037bf8880224f3b9/Modules/ThirdParty/MetaIO/src/MetaIO/src/metaImageTypes.h#L22-L40)
and we can expect to always get a value even if it just means "don't know".
I have no idea why the modality was singled out in this way, possibly
@aylward https://github.com/aylward may remember though looks like the
decision was made 22 years ago which is amazing in terms of software
longevity but the rational may be lost in the mists of time 🌁 .
—
Reply to this email directly, view it on GitHub
https://github.com/InsightSoftwareConsortium/ITK/pull/4549#issuecomment-2026053260,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACEJLYWPPOC2ASGU3WFYR3Y2R4DJAVCNFSM6AAAAABFNJTK2OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMRWGA2TGMRWGA
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Stephen R. Aylward, Ph.D.
Chair, MONAI Advisory Board
Senior Director, Strategic Initiatives, Kitware

For reference, this is the post that triggered this response. The problem is not necessarily that it is not part of the metadata dict, but rather that there is no method to actually get this info after reading the image, at least in SimpleITK.

The basis was that every image has a modality. So, it shouldn't be optional meta data.

Thanks for your feedback, Stephen. But isn't the concept of image modality very specific to medical images? And isn't the meta image format more generally applicable, also outside the medical field?
I noticed that the list of supported modalities is very short:  CT, MR, NM, US, and OTHER
https://github.com/InsightSoftwareConsortium/ITK/blob/1780a31476d63a3f67b97c50037bf8880224f3b9/Modules/ThirdParty/MetaIO/src/MetaIO/src/metaImageTypes.h#L24-L29
Compared to the list of 51 modalities listed as Acquisition Modality: https://dicom.nema.org/MEDICAL/Dicom/current/output/chtml/part16/sect_CID_29.html
In order to preserve the value of an image modality, I think it would be better to store it as a text string. That would allow MetaIO to be future prove.

One thing...these changes should go in the MetaIO repo, not in ITK.   The MetaIO repo is automatically pulled into ITK using a script.  If the changes aren't upstreamed, they will be overwritten with the next update.

Thanks for reviewing, @aylward I don't think this PR is meant to change Kitware's MetaIO library. It only adjusts the ITK wrapper, itk::MetaImageIO.

Ah.  My mistake.  Approved!
Stephen R. Aylward, Ph.D.
Chair, MONAI Advisory Board
Senior Director, Strategic Initiatives, Kitware
On Fri, Mar 29, 2024 at 12:38 PM Niels Dekker @.***>
wrote:

Thanks for reviewing, @aylward https://github.com/aylward I don't think
this PR is meant to change Kitware's MetaIO
https://github.com/Kitware/MetaIO library. It only adjusts the ITK
wrapper, itk::MetaImageIO.
—
Reply to this email directly, view it on GitHub
https://github.com/InsightSoftwareConsortium/ITK/pull/4549#issuecomment-2027466103,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACEJL65LHHUNVRAJMC3LMDY2WDHZAVCNFSM6AAAAABFNJTK2OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMRXGQ3DMMJQGM
.
You are receiving this because you were mentioned.Message ID:
@.***>

@aylward thanks for approving, will merge the fix now.


[xlsxio] new port

Describe the pull request

What does your PR fix?

port xlsxio

Which triplets are supported/not supported?

Yes

not clean, redo

You don't need to close PRs if you want to clean up. You can force-push a clean history. However I would recommend this only for PRs in draft state.
Note that in the vcpkg repo, PRs are squashed when merged, and only the commit messages remain from intermediate steps.

You don't need to close PRs if you want to clean up. You can force-push a clean history. However I would recommend this only for PRs in draft state.
Note that in the vcpkg repo, PRs are squashed when merged, and only the commit messages remain from intermediate steps.

The branch for PR is base on a wrong commit.
I think force-push should work as the branch name doesn't change, anyway I'll try later.
Thanks

force push let you do almost everything. You could even push a totally different repository branch.
but if you force push while PR is closed, you cannot reopen it anymore if I remember correctly github policies


add Syllables property to Word

Create a Syllable token schema
Both Word and Syllable have a phonemes attribute (array of phoneme tokens) - it's up to the user which they prefer to use
Probably need to create distinct schemas for Phoneme (type) and Phone (token)

Information about syllable structure of words can already be stored in a phonetic transcription. Alternatively, users could add project-specific fields to their data.


Update blobfuse2-configuration.md

Add '--use-adls=true' to blobfuse driver when using data lake storage

Learn Build status updates of commit 298f2bd:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/storage/blobs/blobfuse2-configuration.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

@jimmart-dev
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
Note that the new "Note" on line 73 must be formatted with Markdown callout syntax before merging.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

I am checking with the PM on this. I know that parameter can be used with the blobfuse2 mountv1 command, but I think it might be obsolete for BF2.

With BlobFuse2, that parameter is no longer required (or supported) as a CLI parameter when mounting a blob. It is specified in the config file now (example below from the baseConfig.yaml file):
azstorage:
Required
type: block|adls <type of storage account to be connected. Default - block>
It can however be used when using the ‘blobfuse2 mountv1’ command to mount a blob using the old v1 method.

#hold-off


[1.13.x] devmapper: Return device id in error message

Backport to 1.13.x of https://github.com/docker/docker/pull/29917
I often get complains that container removal failed and users got following
error message.
"Driver devicemapper failed to remove root filesystem 18a69ba82aaf7a039ce7d44156215012d703001643079775190ac7dd6c6acf56:Device is Busy"
This error message talks about container id but does not give any info
about which particular device id is busy. Most likely device is mounted
in some other mount namespace and if one knows the device id, they
can try to do some debugging figuring which process and which mount
namespace is keeping the device busy and how did we reach that stage.
Without that information, it becomes almost impossible to debug the
problem.
So to improve the debuggability, when device removal fails, also return
device id in error message. Now new message looks as follows.
"Driver devicemapper failed to remove root filesystem 18a69ba82aaf7a039ce7d44156215012d703001643079775190ac7dd6c6acf56: Failed to remove device dbc15bdf9994a17c613d8ef9e924f3cffbf67f91e4f709295c901ad628377991:Device is Busy"
Signed-off-by: Vivek Goyal<EMAIL_ADDRESS>
- What I did
- How I did it
- How to verify it
- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

LGTM

LGTM but since it's not a P0, I'm going to close this for now, we will merge it in 1.13.1 if we ever do one.


Feature request: mapping to append comment to current line

NERDCommenter has the ability to append a comment to the current line, which I find really useful.
The default mapping is <leader>cA and it appends comment delimiter(s) to the end of the line and leave you in insert mode so you can start typing your comment.
I would love it if this feature were available in kommentary.

Agree that this would be nice to have, for the moment I implemented it myself:
function! AppendComment()
  let commentChar = split(&commentstring, '%s')[0]
  execute 'normal! A ' . commentChar . ' '
  startinsert!
endfunction
nnoremap  <silent> <leader>C  :call AppendComment()<CR>

Hi, thanks for your interest in kommentary, unfortunately I'll not be adding any new features, I'm sorry I never got to this issue.
If anybody is still interested in this, you can achieve this by defining a custom function, see https://github.com/b3nj5m1n/kommentary#advanced-configuration.


Unable to install Dawn

I tried to install the package but it doesn't work for me

I will be closing, it works now

Ok, I'm glad it's fixed. If the problem persists feel free to reopen this issue or open a new one.


chore(actions): migrate run_actions to node

In order to migrate all our actions to node, first we needed to migrate the run_action.sh entrypoint itself. The thrust of this effort is to remove most if not all of our bash code, so that:

we no longer require cygwin
contributors need know one fewer language 😅
we have more flexibility overall, for example, i was able to very easily able add a time to completion in ms, here:

Also, multiple "action roots":

Indenting doesn't work while running sh actions for the moment, but the intent is to move them all over to mjs ones anyway.

Sorry, had to resolve the comments to keep everything straight


client: clarify encoding of ClientTlsConfig field data

What was changed
Add a _pem suffix to both ClientTlsConfig fields, and remove the client_ prefix (which is redundant with the ClientTlsConfig type name) to try to balance the length increase.
Why?
While trying to get a Rust SDK worker connected to Temporal Cloud, I got a TonicTransportError(tonic::transport::Error(Transport, PrivateKeyParseError)). This turned out to because I was passing DER-encoded private key and certificate to ClientTlsConfig instead of PEM. tonic's Identity::from_pem() makes it obvious that they should be PEM, but that context isn't obvious when looking at the rustdoc for temporal_client::ClientTlsConfig (and in fact, because it uses a Vec<u8> type instead of a String type, it suggests using DER over PEM).
Checklist

How was this tested: no logic affected, so relying on the compiler
Any doc updates needed: should be reflected in rustdoc automatically

I've rolled back the field rename, and added a commit that cleans up a clippy issue likely new in Rust 1.76.

@djc This and your other PR need a rebase/merge. Normally I can do that from right in here but the option isn't showing up. I'm guessing maybe because you haven't allowed commits from maintainers: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/working-with-forks/allowing-changes-to-a-pull-request-branch-created-from-a-fork

@djc This and your other PR need a rebase/merge. Normally I can do that from right in here but the option isn't showing up. I'm guessing maybe because you haven't allowed commits from maintainers: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/working-with-forks/allowing-changes-to-a-pull-request-branch-created-from-a-fork

Hmm, I'm not seeing this checkbox show up.
Is there any chance/method by which you can put me on an allowlist to allow CI to run without waiting for explicit approval? What with the timezone differences, these feedback cycles are pretty slow. :(

I've changed it to only requiring approval for new contributors.

Thanks! Rebased this on top of current master, I guess that will do the job?

The remaining test failure looks unrelated to this change?

The remaining test failure looks unrelated to this change?

Yeah it flakes occasionally. I'll re-run it and merge it.


Documentation: how to logout

Documentation (README) has no mentions of logout. Please suggest, whether it is available as part of the SDK?

Hi,
This is available by calling to logout function on AppIDAuthorizationManager:
AppIDAuthorizationManager appIDAuthorizationManager = new AppIDAuthorizationManager(AppID.getInstance());

appIDAuthorizationManager.logout(context, listener);


When passing browseWithoutSelecting and a value, month/year selector doesn't work

Hey, thanks for your work, the component is amazing.
I've seen that whenever you pass browseWithoutSelecting and a value, the month and year selectors (arrows included) don't work.
Here's  a REPL replicating the bug: https://svelte.dev/repl/2c23cf5f7ae048e1808948bf1b3d2860?version=3.57.0

Thanks for the report. Would appreciate a PR

Fixed in v2.4.1


Cache command picks for the duration of the quick pick

Fixes #200548
The PR seems to work, the main things that need clarity are:

Whether this is the right place to do the caching, I'm sure there's a more elegant way to do this.
Does it invalidate the cache correctly, are there edge cases I'm missing?
Should we do this more generally? I'm not sure how dynamic providers can be.

It essentially removes this getCommandPicks entirely when filtering, until the provider changes:

Below are some numbers in milliseconds of the duration of _getPicks before and after when filtering 1+ letters.
Before

12.72
10.05
17.49
10.90
11.11
avg 12.45

After:

3.39
2.11
3.88
4.46
4.21
avg 3.61 (-8.84, -72%)

Since the change reduces the work by about 9ms, this is expected to more often than not display the results in 1 less frame on a 60Hz display and almost always in >= 1 less frame on a 144Hz display.

One of the things I'm currently worried about with this change is that I wanna know what happens when you start the Command Palette right away at start up... I worry that that initial list doesn't contain all commands because some components are delayed in their initialization. We do wait a bit but it might not be enough for low end machines.
I'd be fine accepting this and see what insiders has to say though. cc @bpasero for any historical insight.

@TylerLeonhardt could we do a quick command count check and reload to avoid problems there? We could move the caching into getCommandPicks itself with that? We could also cache getCodeEditorCommandPicks and getGlobalCommandPicks independently (post mapping into picks to save more time).
@bpasero sure it would be better to not need to do this, but 9ms is a significant price to pay on every keystroke. I think the above suggestion mostly addresses your concern.

Stale


Athena Connection are not ingesting Views

Affected module
Ingestion
Describe the bug
Ingestions does not bring the views.
To Reproduce
when I run the ingestion with a view it does not bring anything not tables or views.
If I run the ingestion with only tables it then brings the tables.
Screenshots or steps to reproduce

Expected behavior
Upon selection the option for view I expect to have both tables and views displayed in Athena
Version:

OpenMetadata version: 0.12.1

Just to clarify it looks like the issue is that the ingestion cannot be reused. If I add an new Database and Ingestion it works.
So what it may need to be looked at is the way to be able to reuse then ingestion with new parameters i.e ticking or not the view. to bring different tables.

I test out and found that Athena Connection is ingesting Views. confirm with @jsuarez-ravenpack  also on slack.

@MilanBariya can we close this ?

@MilanBariya can we close this ?

Yes @harshach we can close.


[Velox to Substrait] Add converter from AggregationNode to AggregateRel

Hi @mbasmanova , I have updated it, please help take a review, Thanks!

Hi @mbasmanova , I have updated it, please help take a review, Thanks!

Hi @ZJie1, I was on vacation last week. I'll take a look today.

linux-parquet-s3-build CI job failed. Would you rebase to see if the failure goes away?

@ZJie1 I'm seeing test failures and crashes:
[ RUN      ] VeloxSubstraitRoundTripPlanConverterTest.sumGlobal
velox/exec/tests/utils/QueryAssertions.cpp:245: Failure
Expected equality of these values:
  rowType->size()
    Which is: 3
  dataChunk->GetTypes().size()
    Which is: 2
Wrong number of columns

and
[ RUN      ] Substrait2VeloxPlanConversionTest.q6

stderr:
I0608 07:44:02.013142 11707 Compression.cpp:460] Initialized zstd compressor with compression level 7
I0608 07:44:02.019433 11707 WriterShared.cpp:468] Stripe 0: Flush overhead = 52304, data length = 1175, pre flush mem = 440087, post flush mem = 490797. Closing = true
third-party-buck/platform010/build/libgcc/include/c++/trunk/bits/stl_vector.h:1063: std::vector::const_reference std::vector<std::shared_ptr<const facebook::velox::core::FieldAccessTypedExpr>>::operator[](std::vector::size_type) const [_Tp = std::shared_ptr<const facebook::velox::core::FieldAccessTypedExpr>, _Alloc = std::allocator<std::shared_ptr<const facebook::velox::core::FieldAccessTypedExpr>>]: Assertion '__n < this->size()' failed.
*** Aborted at<PHONE_NUMBER> (Unix time, try 'date -d @1654699442') ***
*** Signal 6 (SIGABRT) (0x755900002dbb) received by PID 11707 (pthread TID 0x7fa1798d50c0) (linux TID 11707) (maybe from PID 11707, UID 30041) (code: -6), stack trace: ***
    @<PHONE_NUMBER>01104e folly::symbolizer::(anonymous namespace)::innerSignalHandler(int, siginfo_t*, void*)
                       ./folly/experimental/symbolizer/SignalHandler.cpp:449
    @<PHONE_NUMBER>00f771 folly::symbolizer::(anonymous namespace)::signalHandler(int, siginfo_t*, void*)
                       ./folly/experimental/symbolizer/SignalHandler.cpp:470
    @<PHONE_NUMBER>000000 (unknown)
    @<PHONE_NUMBER>09c9f3 __GI___pthread_kill
    @<PHONE_NUMBER>04450c __GI_raise
    @<PHONE_NUMBER>02c432 __GI_abort
    @<PHONE_NUMBER>2bae4b std::__replacement_assert(char const*, int, char const*, char const*)
                       ./third-party-buck/platform010/build/libgcc/include/c++/trunk/x86_64-facebook-linux/bits/c++config.h:535
                       -> ./velox/exec/Aggregate.cpp
    @<PHONE_NUMBER>470608 std::vector<std::shared_ptr<facebook::velox::core::FieldAccessTypedExpr const>, std::allocator<std::shared_ptr<facebook::velox::core::FieldAccessTypedExpr const> > >::operator[](unsigned long) const
                       ./third-party-buck/platform010/build/libgcc/include/c++/trunk/bits/stl_vector.h:1063
                       -> ./velox/exec/HashAggregation.cpp
    @<PHONE_NUMBER>46bd93 facebook::velox::exec::HashAggregation::HashAggregation(int, facebook::velox::exec::DriverCtx*, std::shared_ptr<facebook::velox::core::AggregationNode const> const&)
                       ./velox/exec/HashAggregation.cpp:99
    @<PHONE_NUMBER>598725 std::_MakeUniq<facebook::velox::exec::HashAggregation>::__single_object std::make_unique<facebook::velox::exec::HashAggregation, unsigned long&, facebook::velox::exec::DriverCtx*, std::shared_ptr<facebook::velox::core::AggregationNode const>&>(unsigned long&, facebook::velox::exec::DriverCtx*&&, std::shared_ptr<facebook::velox::core::AggregationNode const>&)
                       ./third-party-buck/platform010/build/libgcc/include/c++/trunk/bits/unique_ptr.h:962
                       -> ./velox/exec/LocalPlanner.cpp

@ZJie1 I'm going to fix these problems and land the PR. Let's prioritize adding Substrait tests to CircleCI to get a timely and clear signal about the test failures next time.

Sure. Thanks!

@ZJie1 I'm going to fix these problems and land the PR. Let's prioritize adding Substrait tests to CircleCI to get a timely and clear signal about the test failures next time.

Sure. Thanks!


Caddy throws when using hugo

Using 0.8.
With this config:
http://localhost:8888 {
  root public
  hugo {
    args "--theme=hyde"
  }
}

I get this error:
Activating privacy features... done.
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xb code=0x1 addr=0x0 pc=0x2dfe21]

goroutine 1 [running]:
github.com/hacdias/caddy-hugo/utils.Run(0x208e8d2c0)
	/home/caddy/go/src/github.com/hacdias/caddy-hugo/utils/utils.go:178 +0x2b1
github.com/hacdias/caddy-hugo.Setup(0x208eea100, 0x0, 0x0, 0x0)
	/home/caddy/go/src/github.com/hacdias/caddy-hugo/hugo.go:26 +0x7d
github.com/mholt/caddy/caddy.loadConfigs(0x89c510, 0x9, 0x2208e4a2a0, 0x208f79050, 0x0, 0x0, 0x0, 0x0, 0x0)
	/tmp/custombuild_6545_142287504/src/github.com/mholt/caddy/caddy/config.go:162 +0x1a8a
github.com/mholt/caddy/caddy.Start(0x2208e4bd50, 0x208f79020, 0x0, 0x0)
	/tmp/custombuild_6545_142287504/src/github.com/mholt/caddy/caddy/caddy.go:138 +0x2c3
main.main()
	/tmp/custombuild_6545_142287504/src/github.com/mholt/caddy/main.go:99 +0x5fe

goroutine 5 [syscall]:
os/signal.loop()
	/usr/local/go/src/os/signal/signal_unix.go:21 +0x1f
created by os/signal.init·1
	/usr/local/go/src/os/signal/signal_unix.go:27 +0x35

goroutine 9 [chan receive]:
github.com/mholt/caddy/caddy.func·012()
	/tmp/custombuild_6545_142287504/src/github.com/mholt/caddy/caddy/sigtrap_posix.go:19 +0x2d9
created by github.com/mholt/caddy/caddy.trapSignalsPosix
	/tmp/custombuild_6545_142287504/src/github.com/mholt/caddy/caddy/sigtrap_posix.go:72 +0x25

goroutine 8 [chan receive]:
github.com/mholt/caddy/caddy.func·010()
	/tmp/custombuild_6545_142287504/src/github.com/mholt/caddy/caddy/sigtrap.go:31 +0x110
created by github.com/mholt/caddy/caddy.trapSignalsCrossPlatform
	/tmp/custombuild_6545_142287504/src/github.com/mholt/caddy/caddy/sigtrap.go:41 +0x25

goroutine 10 [select]:
github.com/mholt/caddy/caddy/letsencrypt.maintainAssets(0x208ee71e0, 0x1, 0x1, 0x0)
	/tmp/custombuild_6545_142287504/src/github.com/mholt/caddy/caddy/letsencrypt/maintain.go:34 +0x7cd
created by github.com/mholt/caddy/caddy/letsencrypt.Activate
	/tmp/custombuild_6545_142287504/src/github.com/mholt/caddy/caddy/letsencrypt/letsencrypt.go:146 +0x113a

This would be good to bring up with @hacdias - open this issue at https://github.com/hacdias/caddy-hugo - thanks!

Ah, sorry about that, posted it here for others who see this issue: https://github.com/hacdias/caddy-hugo/issues/34


[Arith][GPU]Rewrite simplify fix for Vectorized Cooperative Fetching

This pr is part of #5883 , fix for the rewrite_simplify error when doing vectorized cooperative fetching in some cases.
Code generated with bug is shown like this:
A.shared[ramp(((ax0.ax1.fused.outer.outer*256) + (threadIdx.x_1*4)), 1, 4)] =
(float32x4*)A_2[(((broadcast(((floordiv(blockIdx.x, 4)*32768) + (ax0.ax1.fused.outer.outer*2048)), 4) + (floordiv(ramp((threadIdx.x_1*4), 1, 4), broadcast(64, 4))*broadcast(512, 4))) + broadcast((k.outer.outer*64), 4)) + floormod(ramp((threadIdx.x_1*4), 1, 4), broadcast(64, 4)))])

Which will finally lower to wrong CUDA C instructions.
This should be simplified to generate the correct RampNode:
A.shared[ramp(((ax0.ax1.fused.outer.outer*256) + (threadIdx.x_1*4)), 1, 4)] =
(float32x4*)A_2[ramp((((((floordiv(blockIdx.x, 4)*32768) + (ax0.ax1.fused.outer.outer*2048)) + (floordiv(threadIdx.x_1, 16)*512)) + (k.outer.outer*64)) + (floormod(threadIdx.x_1, 16)*4)), 1, 4)])

Then main problems inside this expression are:
threadIdx.x_1 = [0, 64]
floordiv(ramp((threadIdx.x_1*4), 1, 4), broadcast(64, 4)) * broadcast(512, 4)
floormod(ramp((threadIdx.x_1*4), 1, 4), broadcast(64, 4))

should be simplified to:
threadIdx.x_1 = [0, 64]
broadcast(floordiv(threadIdx.x_1, 16)*512), 4)
ramp(floormod(threadIdx.x_1, 16)*4, 1, 4)

@merrymercy Our former simplify rules will cause some extra bug, I find a better way to fix this.
@tqchen For the UTs, I'm not sure if there's any better way to check if all of the inner AST blocks are RampNode, the current UTs I added can still pass even if the vectorize failed.
cc @minminsun @FrozenGene @yangjunpro

@jcf94 we should add a testcase to test_arith_rewrite_simplify, by constructing the case and verify the rules you have worked. Use isinstance(x, tvm.ir.Ramp) to assert the ramp node

Thanks @jcf94 we should add a testcase to test_arith_rewrite_simplify, by constructing the case and

verify each of the rule added in this PR.
Use isinstance(x, tvm.ir.Ramp) to assert the ramp node
You mentioned a bug in the previous rule, it would be great if the testcase covers the bug you mentioned

Commemts are all addressed.

Add several test cases in tests_arith_rewrite_simplify for simplify rules
Update test_target_codegen_cuda UTs & use pre post function to check the RampNode patterns
The bug metioned above is introduced by our former implementation, currently everything work fine

@jcf94  Did our old rule affect the correctness of common operators?

@jcf94 Did our old rule affect the correctness of common operators?

Yes, with those rules several other UTs will fail, they're actually not always correct.
For example we have:
    TVM_TRY_REWRITE_IF(floordiv(x * c1 + y, c2), floordiv(x * c1, c2),
                       c1.Eval()->value > 0 && c2.Eval()->value > 0 &&
                       c2.Eval()->value % c1.Eval()->value == 0 &&
                       CanProveGreaterEqual(-y.Eval(), -c1.Eval()->value + 1));

while floordiv(x * 4 + 4, 8) cannot be simplified to floordiv(x * 4, 8).

Thanks @jcf94 @merrymercy . this PR is now merged


Build system

I tried to clone the repository and build it on two machines
simply by using git clone and build.ps1
On both machines I get the following error
Building C:\Users\Alex\Documents\projects\external\Npgsql.EntityFrameworkCore.PostgreSQL
dotnet-install: .NET SDK version 1.0.0-preview2-003121 is already installed.
log  : Restoring packages for C:\Users\Alex\Documents\projects\external\Npgsql.EntityFrameworkCore.PostgreSQL\.build\project.json...
error: Unable to resolve 'NuGetPackageVerifier (>= 1.0.1)' for '.NETFramework,Version=v4.5.1'.
log  : Lock file has not changed. Skipping lock file write. Path: C:\Users\Alex\Documents\projects\external\Npgsql.EntityFrameworkCore.PostgreSQL\.build\project.lock.json
log  : .build\project.json
log  : Restore failed in 1546ms.

Errors in .build\project.json
    Unable to resolve 'NuGetPackageVerifier (>= 1.0.1)' for '.NETFramework,Version=v4.5.1'.
Using makefile: .build\shade\makefile.shade
& : The module '.build' could not be loaded. For more information, run 'Import-Module .build'.
At C:\Users\Alex\Documents\projects\external\Npgsql.EntityFrameworkCore.PostgreSQL\.build\KoreBuild.ps1:78 char:2
+ &"$koreBuildFolder\Sake\0.2.2\tools\Sake.exe" -I $koreBuildFolder\sha ...
+  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : ObjectNotFound: (.build\Sake\0.2.2\tools\Sake.exe:String) [], ParentContainsErrorRecordException
    + FullyQualifiedErrorId : CouldNotAutoLoadModule

Sake has not been downloaded into said folder and is not contained in the zip file that you are downloading from https://github.com/aspnet/KoreBuild/archive/release.zip.
How can I get it to work?
Thanks

With the switch to Microsoft's new csproj format, Npgsql no longer uses build.sh/build.ps1. Simply open the solution in VS2017 and build, or install dotnet cli, go into src/Npgsql and do dotnet restore followed by dotnet build.

ok, thanks for the answer


compile error

when attempting to "infect" the apk, it says "failed to compile infected apk". is there a way to fix this?

@anirudhmalik same problem here with:
https://www.apkmirror.com/apk/team-videostream/videostream-chromecast-mobile/videostream-chromecast-mobile-1-18-02-27-release/videostream-chromecast-mobile-1-18-02-27-android-apk-download/
xhunter_v1.6.apk
OruxMaps7.4.26.apk
https://oruxmaps.en.softonic.com/android
I am trying with WhatsApp (for testing purpose), currently building.
Android version 12 December 5 2022
kernel 5.4.147-qgki-g397a12867cb4
Phone if relvant Realme GT 5G
It is kind of useless app if the building with most of the .apk fails.


fix: remove unneeded Sync bounds

These Sync bounds make it hard to integrate with other tooling, and don't seem to be actually needed.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Note: looks like the non core s3s crates still require it, will look into if I can fix this

probably not easily mergeable given the Sync requirements from the aws sdk/hyper-body


RuntimeError: Unable to start Tika server.

I am running a simple sample script , and facing the below issue,
The below script is running on Ubuntu 18.04 system
from tika import parser
#file = 'path/to/file'
file_data = parser.from_file('DataEnginner1.txt')
text = file_data['content']
print(text)

Below is the error stack trace :
2020-03-12 15:34:57,611 [MainThread  ] [WARNI]  Failed to see startup log message; retrying...
2020-03-12 15:35:02,617 [MainThread  ] [WARNI]  Failed to see startup log message; retrying...
2020-03-12 15:35:07,620 [MainThread  ] [WARNI]  Failed to see startup log message; retrying...
2020-03-12 15:35:12,624 [MainThread  ] [ERROR]  Tika startup log message not received after 3 tries.
2020-03-12 15:35:12,628 [MainThread  ] [ERROR]  Failed to receive startup confirmation from startServer.
---------------------------------------------------------------------------
RuntimeError                              Traceback (most recent call last)
<ipython-input-79-0b160c12afff> in <module>
      3 #file = 'path/to/file'
      4 # Parse data from file
----> 5 file_data = parser.from_file('DataEnginner1.txt')
      6 # Get files text content
      7 text = file_data['content']

~/.local/lib/python3.6/site-packages/tika/parser.py in from_file(filename, serverEndpoint, service, xmlContent, headers, config_path, requestOptions)
     38     '''
     39     if not xmlContent:
---> 40         output = parse1(service, filename, serverEndpoint, headers=headers, config_path=config_path, requestOptions=requestOptions)
     41     else:
     42         output = parse1(service, filename, serverEndpoint, services={'meta': '/meta', 'text': '/tika', 'all': '/rmeta/xml'},

~/.local/lib/python3.6/site-packages/tika/tika.py in parse1(option, urlOrPath, serverEndpoint, verbose, tikaServerJar, responseMimeType, services, rawResponse, headers, config_path, requestOptions)
    335         status, response = callServer('put', serverEndpoint, service, f,
    336                                       headers, verbose, tikaServerJar, config_path=config_path,
--> 337                                       rawResponse=rawResponse, requestOptions=requestOptions)
    338
    339     if file_type == 'remote': os.unlink(path)

~/.local/lib/python3.6/site-packages/tika/tika.py in callServer(verb, serverEndpoint, service, data, headers, verbose, tikaServerJar, httpVerbs, classpath, rawResponse, config_path, requestOptions)
    528     global TikaClientOnly
    529     if not TikaClientOnly:
--> 530         serverEndpoint = checkTikaServer(scheme, serverHost, port, tikaServerJar, classpath, config_path)
    531
    532     serviceUrl  = serverEndpoint + service

~/.local/lib/python3.6/site-packages/tika/tika.py in checkTikaServer(scheme, serverHost, port, tikaServerJar, classpath, config_path)
    598             if not status:
    599                 log.error("Failed to receive startup confirmation from startServer.")
--> 600                 raise RuntimeError("Unable to start Tika server.")
    601     return serverEndpoint
    602

RuntimeError: Unable to start Tika server.

@suresh96458 please make sure you have Java running on your system. You need java and check the pre-reqs section of the README.md.


LG Refrigerator showing same "Set Temp" in "Current Temp"

I have a LG Refrigerator GC-C247UGLW.
It is showing the same set temperature in the current temperature field also.
If I change the set temperature, Current temperature changes too instantly to the same value.
Describe Your Problem:

Logs:
[6/2/2022, 2:20:25 AM] [HB Supervisor] OS: Linux 4.19.103-boot2docker x64
[6/2/2022, 2:20:25 AM] [HB Supervisor] Node.js v16.15.0 /usr/local/bin/node
[6/2/2022, 2:20:25 AM] [HB Supervisor] Homebridge Path: /usr/local/lib/node_modules/homebridge/bin/homebridge
[6/2/2022, 2:20:25 AM] [HB Supervisor] UI Path: /usr/local/lib/node_modules/homebridge-config-ui-x/dist/bin/standalone.js
[6/2/2022, 2:20:28 AM] [Homebridge UI] Homebridge Config UI X v4.46.0 is listening on :: port 8581
[6/2/2022, 2:20:28 AM] [HB Supervisor] Starting Homebridge with extra flags: -I -P /homebridge/node_modules
[6/2/2022, 2:20:28 AM] [HB Supervisor] Starting Homebridge with custom env: {"DEBUG":"true"}
[6/2/2022, 2:20:28 AM] [HB Supervisor] Started Homebridge v1.4.1 with PID: 14184
[6/2/2022, 2:20:29 AM] Loaded config.json with 3 accessories and 3 platforms.
[6/2/2022, 2:20:29 AM] Loaded 1 cached accessories from cachedAccessories.
[6/2/2022, 2:20:29 AM] ---
[6/2/2022, 2:20:30 AM] [Homebridge UI] Starting terminal session
[6/2/2022, 2:20:30 AM] Loaded plugin<EMAIL_ADDRESS>[6/2/2022, 2:20:30 AM] Registering accessory 'homebridge-dummy.DummySwitch'
[6/2/2022, 2:20:30 AM] ---
[6/2/2022, 2:20:31 AM] Loaded plugin<EMAIL_ADDRESS>[6/2/2022, 2:20:31 AM] Registering platform 'homebridge-lg-thinq.LGThinQ'
[6/2/2022, 2:20:31 AM] ---
[6/2/2022, 2:20:31 AM] Loaded plugin<EMAIL_ADDRESS>[6/2/2022, 2:20:31 AM] Registering platform 'homebridge-mi-waterpuri.MiWaterPurifierPlatform'
[6/2/2022, 2:20:31 AM] ---
[6/2/2022, 2:20:31 AM] Loaded plugin<EMAIL_ADDRESS>[6/2/2022, 2:20:31 AM] Registering accessory 'homebridge-mijia-camera.MijiaCamera'
[6/2/2022, 2:20:31 AM] ---
[6/2/2022, 2:20:31 AM] Loaded plugin<EMAIL_ADDRESS>[6/2/2022, 2:20:31 AM] Registering platform 'homebridge-config-ui-x.config'
[6/2/2022, 2:20:31 AM] ---
[6/2/2022, 2:20:31 AM] Loading 3 platforms...
[6/2/2022, 2:20:31 AM] [Config] Initializing config platform...
[6/2/2022, 2:20:31 AM] [Config] Running in Service Mode
[6/2/2022, 2:20:31 AM] [LGThinQ] Initializing LGThinQ platform...
[6/2/2022, 2:20:31 AM] [MiWaterPurifierPlatform] Initializing MiWaterPurifierPlatform platform...
[6/2/2022, 2:20:31 AM] [MiWaterPurifierPlatform] [MiWaterPurifier][INFO]*********************************************************************
[6/2/2022, 2:20:31 AM] [MiWaterPurifierPlatform] [Miwaterpurifier][INFO]                      Miwaterpurifier v1.0.3                           *
[6/2/2022, 2:20:31 AM] [MiWaterPurifierPlatform] [Miwaterpurifier][INFO] GitHub: https://github.com/biner88/homebridge-mi-waterpuri     *
[6/2/2022, 2:20:31 AM] [MiWaterPurifierPlatform] [Miwaterpurifier][INFO]                                                                    *
[6/2/2022, 2:20:31 AM] [MiWaterPurifierPlatform] [Miwaterpurifier][INFO]*********************************************************************
[6/2/2022, 2:20:31 AM] [MiWaterPurifierPlatform] [Miwaterpurifier][INFO]start success...
[6/2/2022, 2:20:31 AM] [MiWaterPurifierPlatform] [Miwaterpurifier][INFO]device size: 1, accessories size: 6
[6/2/2022, 2:20:31 AM] Initializing platform accessory 'TDS OUT'...
[6/2/2022, 2:20:31 AM] Initializing platform accessory 'TDS IN'...
[6/2/2022, 2:20:31 AM] Initializing platform accessory 'PP cottonfilter remaining'...
[6/2/2022, 2:20:31 AM] Initializing platform accessory 'Front active Carbonfilter remaining'...
[6/2/2022, 2:20:31 AM] Initializing platform accessory 'Ro Filter Remaining'...
[6/2/2022, 2:20:31 AM] Initializing platform accessory 'Rear active carbonFilter remaining'...
[6/2/2022, 2:20:31 AM] Loading 3 accessories...
[6/2/2022, 2:20:31 AM] [Test] Initializing DummySwitch accessory...
[6/2/2022, 2:20:31 AM] [Mi Camera XXX2] Initializing MijiaCamera accessory...
[6/2/2022, 2:20:31 AM] [Mi Camera XXX1] Initializing MijiaCamera accessory...
[6/2/2022, 2:20:31 AM] [LGThinQ] Loading accessory from Homebridge cache: LG Refrigerator
Setup Payload:
X-HM://0023ISYWYIIDJ
Enter this code with your HomeKit app on your iOS device to pair with Homebridge:

    ┌────────────┐
    │  │
    └────────────┘

[6/2/2022, 2:20:31 AM] Homebridge v1.4.1 (HAP v0.10.2) (Homebridge C15F) is running on port 52696.
[6/2/2022, 2:20:32 AM] [Mi Camera XXX2] connecting to camera at ...
[6/2/2022, 2:20:32 AM] [Mi Camera XXX1] connecting to camera at ...
[6/2/2022, 2:20:33 AM] [Mi Camera XXX2] current power state: on
[6/2/2022, 2:20:33 AM] [Mi Camera XXX1] current power state: on
[6/2/2022, 2:20:35 AM] [homebridge-mi-waterpuri] This plugin generated a warning from the characteristic 'Current Ambient Light Level': characteristic value expected valid finite number and received "NaN" (number). See https://homebridge.io/w/JtMGR for more info.
[6/2/2022, 2:20:35 AM] [Mi Camera XXX2] connecting to camera at ...
[6/2/2022, 2:20:35 AM] [Mi Camera XXX1] connecting to camera at ...
[6/2/2022, 2:20:35 AM] [Mi Camera XXX1] current power state: on
[6/2/2022, 2:20:36 AM] [Mi Camera XXX2] current power state: on
[6/2/2022, 2:20:37 AM] [LGThinQ] Successfully connected to the ThinQ API.
[6/2/2022, 2:20:38 AM] [homebridge-mi-waterpuri] This plugin generated a warning from the characteristic 'Current Ambient Light Level': characteristic value expected valid finite number and received "NaN" (number). See https://homebridge.io/w/JtMGR for more info.
[6/2/2022, 2:20:38 AM] [Mi Camera XXX2] connecting to camera at ...
[6/2/2022, 2:20:38 AM] [Mi Camera XXX1] connecting to camera at ...
[6/2/2022, 2:20:39 AM] [Mi Camera XXX2] current power state: on
[6/2/2022, 2:20:39 AM] [Mi Camera XXX1] current power state: on
[6/2/2022, 2:20:39 AM] [LGThinQ] [LG Refrigerator] Setting up device!
[6/2/2022, 2:20:39 AM] [LGThinQ] Restoring existing accessory: some-id-here: LG Refrigerator (REFRIGERATOR GC-C247UGZV.ALWQEBN)
[6/2/2022, 2:20:39 AM] [LGThinQ] Start MQTT listener for thinq2 device
[6/2/2022, 2:20:41 AM] [homebridge-mi-waterpuri] This plugin generated a warning from the characteristic 'Current Ambient Light Level': characteristic value expected valid finite number and received "NaN" (number). See https://homebridge.io/w/JtMGR for more info.
[6/2/2022, 2:20:43 AM] [LGThinQ] Successfully connected to the MQTT server

Plugin Config:
{
            "country": "IN",
            "language": "en-IN",
            "auth_mode": "token",
            "refresh_token": "",
            "username": "",
            "password": "",
            "devices": [
                {
                    "id": "",
                    "name": "LG Refrigerator",
                    "type": "REFRIGERATOR",
                    "ref_express_freezer": true
                }
            ],
            "thinq1": false,
            "refresh_interval": 60,
            "platform": "LGThinQ"
        }

Screenshots:

Environment:

Plugin Version: v1.2.15
Homebridge Version:  [v1.4.1]
Node.js Version: v16.15.0
NPM Version: 8.5.5
Operating System:  Windows in Hyper V (Ubuntu Focal Fossa (20.04.4 LTS))

is your model showing current temperature separate with desired temperature ?
please capture some screenshot from LG ThinQ mobile app, let see

is your model showing current temperature separate with desired temperature ?
please capture some screenshot from LG ThinQ mobile app, let see

The Lg thinq mobile app only shows the desired temperature.
I thought via homebridge, i could get the current temperature somehow.
I tried homeassistant, but it also doesn't show the current temp.
I bought this refrigerator this year only, so its fairly new model. So current temperature sensors should be present there.

unfortunately we can't get current temperature, api responsed desired temperature only

unfortunately we can't get current temperature, api responsed desired temperature only

:(
Anyways, can't do anything if its not available via API.
Feel free to close the issue. Will see in future if there comes a way to the temp.
Meanwhile, I'll start looking into external temp sensor that I can somehow place inside refrigerator.
Will comment here if I find a decent way.


TODO annotation updated after removing duplication

Annotation changes after removing  Issue duplications.
@dr-orlovsky do let me know if you see any more. Will update in this PR.

Oohh.. Something went wrong while merging.. Will check..

Fixed..


Antiword and UnRTF failing

Hi,
First of all, I'd like to thank you for your awesome repo!
However, I was testing it, and run into some errors. The PDF extractor lambda works good. However, when I tried the office extractor lambda, it failed, both with an RTF and a DOC files.
This are the messages:
For UnRTF:
 "reason": "Exception while executing ['/var/task/bin/unrtf', '-P', '/var/task/lib/unrtf', '--text', u'/tmp/intelllex_dZROnq.rtf']: Command '['/var/task/bin/unrtf', '-P', '/var/task/lib/unrtf', '--text', u'/tmp/intelllex_dZROnq.rtf']' returned non-zero exit status 1 (output=No config directories. Searched: /var/task/lib/unrtf\n)"
For Antiword:
"reason": "Exception while executing ['/var/task/bin/antiword', '-t', '-w', '0', '-m', 'UTF-8', u'/tmp/intelllex_pLX1jK.doc']: Command '['/var/task/bin/antiword', '-t', '-w', '0', '-m', 'UTF-8', u'/tmp/intelllex_pLX1jK.doc']' returned non-zero exit status 1 (output=I can't find the name of your HOME directory\nI can't open your mapping file (UTF-8.txt)\nIt is not in '/.antiword' nor in '/usr/share/antiword'.\n\tName: antiword\n\tPurpose: Display MS-Word files\n\tAuthor: (C) 1998-2005 Adri van Os\n\tVersion: 0.37  (21 Oct 2005)\n\tStatus: GNU General Public License\n\tUsage: antiword [switches] wordfile1 [wordfile2 ...]\n\tSwitches: [-f|-t|-a papersize|-p papersize|-x dtd][-m mapping][-w #][-i #][-Ls]\n\t\t-f formatted text output\n\t\t-t text output (default)\n\t\t-a <paper size name> Adobe PDF output\n\t\t-p <paper size name> PostScript output\n\t\t   paper size like: a4, letter or legal\n\t\t-x <dtd> XML output\n\t\t   like: db (DocBook)\n\t\t-m <mapping> character mapping file\n\t\t-w <width> in characters of text output\n\t\t-i <level> image level (PostScript only)\n\t\t-L use landscape mode (PostScript only)\n\t\t-r Show removed text\n\t\t-s Show hidden (by Word) text\n)"
Do you know what the reason might be? I just used apex deploy from a cloned version of your repo, with my IAM role. From what I can see, it kind of looks like is looking for a  lib folder where instead seems to be a lib-linux_x86 folder. Although I'm not sure and it might have nothing to do with it.
Please, any pointers would be very welcomed. I can do more testing if you point me in the right direction.
Thanks!
Santiago.

I think I have narrowed it down a little bit.
It seems that the folder where the lib folder should be, in this case, is the office_extractor folder inside functions. However, there is no lib folder there, nor instructions to copy that folder over.
May that be the source of the problem?

Yes, apparently that was the problem. I created a lib folder inside the office_extractor folder, and in there, a symlink to the unrtf folder. Run apex deploy again, and now it works as expected! Great!


Adding Bugsnag NDK causes some instrumentation tests to crash.

Expected behavior
Bugsnag ndk doesn't crash when under test
Observed behavior
When I call leaveBreadcrumb with bugsnag ndk support enabled, my app will sometimes crash while testing.
Steps to reproduce
Add
Observable.fromCallable(
                        new Callable<Void>() {
                            @Override
                            public Void call() throws Exception {
                                Bugsnag.leaveBreadcrumb("some non-null value" + "some other non-null value");
                                return null;
                            }
                        })
                .subscribeOn(Schedulers.io())
                .subscribe();

from a Activity under instrumentation test.  This doesn't happen all the time, but it's pretty consistently reproducible on this specific test.
Version
NDK 1.1.2 w/ Bugsnag sdk 4.3.1
Additional information
I THINK this is only due to the ndk.
Can't comment on Issues?
Some users have been unable to comment on Github issues when an adblocker extension is enabled.
We recommend temporarily disabling the extension, or if that fails, contacting<EMAIL_ADDRESS>

@maltzj I've created a branch to reproduce this. I've tried running the test both with the leaveBreadcrumb method present, and commented out, and both times I observe:
Test running failed: Instrumentation run failed due to 'Process crashed.'
In RxJava2 there was a change which means it's not permitted to emit null, and attempting to do so will result in a call to onError. Adding a consumer for onError would prevent an uncaught exception from crashing the instrumentation process, e.g.:
.subscribe(System.out::println, Throwable::printStackTrace);

Let me know if I've got the wrong end of the stick here and I can look into this further.

That may be true in RxJava2 land; however, we're currently living in RxJava 1 land (at least for this code).  Also, this code doesn't crash in our non-test application, nor does it consistently fail, which is what I'd expect if there were an RxJava bug.

Ah right, my bad! I'll try again with RxJava1 and post results.

The following test reproduces the issue 100% of the time:
public void leaveBreadcrumb() {
        for (int k = 0; k < 100; k++) {
            Observable.fromCallable(
                    new Callable<Void>() {
                        @Override
                        public Void call() throws Exception {
                            Bugsnag.leaveBreadcrumb("some non-null value" + "some other non-null value");
                            return null;
                        }
                    })
                    .subscribeOn(Schedulers.io())
                    .subscribe();
        }
    }

Interestingly using Schedulers.immediate() passes the test, so that may be a potential workaround if you're able to inject the scheduler in for testing purposes. The Logcat seems to suggest that the NDK notifier is trying to free memory which isn't allocated - I'll have to look into this a bit further.
ndk_logcat.txt

gentle poke

@maltzj we've tracked it down to a couple of thread safety issues when adding breadcrumbs, I've linked a PR which partially fixes this if you're interesting in an explanation. Still investigating a more general fix that will prevent this altogether.

Curious if there are any plans to move forward on this, or is it going to be marked as won'tfix?


Connection issue with Android demo project

Hi there, I'm running the Android demo, it works without issue for SC2, whereas there is a small problem after capturing with SC and going back to the preview page, sometimes it's freezing and sometimes it throws an error like the video shows.
The error is as follows
E/ThetaViewModel$startPreview: com.ricoh360.thetaclient.PreviewClientException: Forbidden         at com.ricoh360.thetaclient.PreviewClientImpl.responseStatus(PreviewClient.kt:367)         at com.ricoh360.thetaclient.PreviewClientImpl.access$responseStatus(PreviewClient.kt:73)         at com.ricoh360.thetaclient.PreviewClientImpl$responseStatus$1.invokeSuspend(Unknown Source:14)         at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:33)         at kotlinx.coroutines.DispatchedTask.run(DispatchedTask.kt:106)         at kotlinx.coroutines.internal.LimitedDispatcher.run(LimitedDispatcher.kt:42)         at kotlinx.coroutines.scheduling.TaskImpl.run(Tasks.kt:95)         at kotlinx.coroutines.scheduling.CoroutineScheduler.runSafely(CoroutineScheduler.kt:570)         at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.executeTask(CoroutineScheduler.kt:750)         at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.runWorker(CoroutineScheduler.kt:677)         at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.run(CoroutineScheduler.kt:664)
Thanks in advance!!!

@yuehengshiCarmedia2p0 Thank for the defect report.
I observed same error with Theta SC. I will investigate this issue.

I have fixed this defect.  Please wait for the next release.


double-beep: Watch jpg4png.com

double-beep requests the watch of the watch_keyword jpg4png\.com. See the MS search here and the Stack Exchange search here.

jpg4png\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Makyen in Charcoal HQ

I'm not sure why this shows as closed. It was fully merged.


when make GR from PO amount not include Discount amt.

when make GR from PO amount not include Discount amt.

https://github.com/nfco/netforce/commit/66cf06478f96f73b079c65b24b3a4a340d6015b3


chore(merge-back): 1.134.0

See CHANGELOG

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: 9e3474ffc910136d9afe9c84aa78fe03fb5c8f4f
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v1
Commit ID: 9e3474ffc910136d9afe9c84aa78fe03fb5c8f4f
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


382 add define-xml-path

Not 100% sure on the scope of this change, it seems we point to the define-xml from a few different places so there are quite a few changes here. Let me know if I'm on the right track and how best to add more tests for this.

@mdeforestbrown just need to fix the lint errors. You can install the linter in the github repo by running pre-commit install in the root of the repo


api for making bulk atributions

I have just discovered this repository and I guess this api is for recovering data from bionomia (which is great).
But I wonder if there are chances to use an api for making attributions, instead of having to browse bionomia.net website and manually attribute records.
I am not meaning bulk attributions related to a GBIF dataset curated by me, which I already can bulk-attribute by populating with ORCID & wikidata IDs and then republishing to GBIF.
I mean that I know of many other GBIF records which I do not upload to GBIF, for example:

Duplicated specimens of my own-curated collections, stored in collections at other institutions.
Other GBIF specimens collected/identified by people that I already know their ORCIDs (mostly because these people are also present in labels of my institution's collections). If data provider is not providing people IDs, I could do it.

Is there anyway that authenticated users can somehow update structurated data like this to bionomia.net?

gbifID
recordedById
identifiedById

3031150476
Q6117770,Q11703496

3829872920

Q11703496

1935887248
Q5707146
Q6117770

...
...
...

I suppose it wouldn't harm to re-attribute data which have already been attributed by other people.
If that ever happened, you just can keep the original attribution.
(also, if both differ you could take profit of this for detecting possible errors).
Thanks
@abubelinha

Thanks @abubelinha for the idea. What you'd like is a write API OR an interface to upload a crafted csv file structured similarly to the table you included here and/or a textarea box to paste records in a web form. This is interesting. We'd first want to follow the recommendations for how to construct an entry for recordedByID or identifiedByID as URIs separated by pipes. I'd also have to resolve each of the entries on-the-fly because Bionomia might not know anything about them so would have to first create the person via calls to ORCID or wikidata. In principle, it's no different than what I already do prior to bi-monthly data refreshes from GBIF. In this case, I use a processing queue on my laptop with a csv export at the end for which I then make bulk attributions. My structure is a pivot from yours because of this need to call wikidata and ORCID and so I have a column of unique identifiers for people with two other columns containing comma-separated values of gbifIDs for the actions collected and identified.

I wonder if it would be way easier to provide our data as remote csv files which you can read and process.
In that case, we could authenticate attributors by simply providing ORCID in a left column.

gbifID
recordedById
identifiedById
attributor_ORCID

3031150476
0000-0001-7618-5230
Q6117770,Q11703496
0000-0001-7618-5230

3829872920

Q11703496
0000-0001-7618-5230

1935887248
Q5707146
Q6117770
0000-0001-7618-5230

...
...
...
...

That left column wouldn't be necessary right now (I would be the only attributor of my file records).
I was just thinking that attributions might be done as a collaborative task (i.e. several alumni during a class) and it would be easier for me if I can upload all their work in a single file, myself.
Of course you could then process this file at any time, checking first for people's IDs found in recordeById and identifiedById columns.
If you find this suggestion easier to accomplish than using the api, then you could simply let bionomia users profile to enter an optional url for providing such a remote csv file.


feat: Update Rust builder, add target flag

Description
Updated rust builder, add new configuration options for building step. Replaced bash scripts with python one.
Related Issue(s)
Part of the https://github.com/input-output-hk/hermes/issues/20
Description of Changes

replaced std_build.sh and std_check.sh with Python implementation std_check.py, std_build.py, and std_docs.py, docs generation step moved into the separate script.
modified cargo config.toml files, provided new target.wasm32-unknown-unknown compile options, removed --locked flag from all aliases (because we are going to remove Cargo.lock files from the repo), disabled cargo progress bar and disabled verbose.
some other cleanups

updated docs:
https://input-output-hk.github.io/catalyst-ci/branch/feat_rust_wasm/guides/languages/rust/


[REQ] Legacy Lighting Engine option.

Acknowledgement of preliminary instructions

[X] I have read and acknowledged the preliminary instructions.

What idea do you have?
Would this be a possible option in the lighting section. Add on to the 3 ones with the legacy 2017 lighting. Not sure which phase you would set that too though.
It could look something this.

Not possible. I would've added it if it was.


Plugin release 4.0.0

Description
Replace this paragraph with a description of what this PR is doing. If you're modifying existing behavior, describe the existing behavior, how this PR is changing it, and what motivated the change.
Related Issues
*Replace this paragraph with a list of issues related to this PR from the issue database. Indicate, which of these issues are resolved or fixed by this PR.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]).
This will ensure a smooth and quick review process. Updating the pubspec.yaml and changelogs is not required.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[x] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (flutter analyze) does not report any problems on my PR.
[x] I read and followed the Flutter Style Guide.
[x] I signed the CLA.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[x] No, this is not a breaking change.

It looks like the iOS unit test testEncodeDecodeRequestConfiguration failed: https://github.com/googleads/googleads-mobile-flutter/actions/runs/7093442769/job/19306819114?pr=972

It looks like the iOS unit test testEncodeDecodeRequestConfiguration failed: https://github.com/googleads/googleads-mobile-flutter/actions/runs/7093442769/job/19306819114?pr=972. From the release notes it looks like there have been some changes to GADRequestConfiguration in some recent releases: https://developers.google.com/admob/ios/rel-notes

Yeah, it seems that new changes will be needed to be able to update to SDK version 10.14.0. For this release, I changed it from 10.9.0 to 10.11.0 as expected here. I created a backlog task to work on updating the deprecated methods for iOS.


✨ [Update] BoxMatcher matching criteria

Added an additional validity criterium in get_valid_matrix, which masks out anchors from targets, that are too large to predict with the given reg_max and stride values.
Implemented a new function: ensure_one_anchor, which adds a single best suited anchor for valid targets without valid anchors. It is a fallback mechanism, which enables too small or too large targets to be trained to be predicted as well, even if not perfectly.
Fixed the filter_duplicate function to use the topk_masked iou_mat for the selection, which previously sometimes matched invalid targets to anchors with duplicates.
Updated docsstrings across the BoxMatcher functions to match the changes.
Implements #124

Hey @henrytsui000,
I'm going to be testing these changes in the upcoming days. I created a pull request so you can try them out as well if you'd like :)

Hi,
Thank you for your contribution! I’m currently busy with a rebuttal deadline for ICLR, so my replies might be a bit slow for the time being. Thank you for your understanding!
Best regards,
Henry Tsui

Hey @henrytsui000,
Resolved the recently emerged merge conflict.
My proposed changes are looking good in my own tests :) I'd like you try them out :)
Best regards,
Adam

Thanks!
I’ll try this on my local setup later.
By the way, I implemented EMA for single GPU training, and the convergence looks great. However, when training on multiple GPUs, something seems to be missing—the mAP is significantly lower.
Best regards,
Henry Tsui


Why is this link broken hlep

Notebook:
https://github.com/fonsp/disorganised-mess/blob/master/language detection.jl
Direct binder link:
https://mybinder.org/v2/gh/fonsp/pluto-on-binder/master?urlpath=pluto/open?url=https%3A%2F%2Fgist.githubusercontent.com%2Ffonsp%2Fd88b3adc8c958dce780cf7c4df8fa437%2Fraw%2F2a8a5bdc6d8b1bcb4462006f62ead2f52fec17bf%2Fturtles.jl
is broken :(
But if you open the binder to the main menu, and paste the notebook link the blue box, it works.

Maybe the URL needs to be encoded twice?

@dralletje

@fonsp close this thank you


Watch ambassador-certs and ambassador-cacert secrets to avoid need to restart Ambassador

When setting up TLS termination following https://www.getambassador.io/user-guide/tls-termination, it took me a while to realize that I needed to restart Ambassador for it to detect the ambassador-certs secret I had created. The same way ambassador/kubewatch.py watches services for annotation changes, it could watch the ambassador-certs secret (and the ambassador-cacert secret while we're at it).
The watch loops would run concurrently. The official Kubernetes client isn't asyncio-friendly at the moment https://github.com/kubernetes-client/python/issues?utf8=✓&q=asyncio, but concurrent.futures with threads or raw threading would do the trick. If time allows, this multi-watch logic could be implemented upstream in the Kubernetes client: https://github.com/kubernetes-client/python/issues/30
In the meantime, a bold warning in the documentation would be helpful.

Thanks for the bug report. This makes sense. If you're able to do a documentation PR on the TLS guide, that would be greatly appreciated.

I'm curious if this is something that's being looked at as a roadmap item? We're trying to avoid having to build tooling around this or switch gateways. LetsEncrypt (via cert-manager) is going to rotate our cert every 90 days and automatically update ambassador-certs.

@kevinohara80 Very much so! We've actually been sorting out the path forward on Ambassador's roadmap just recently, so I expect to have a better sense of exactly when this will get handled shortly.

@KowalczykBartek volunteered to work on this earlier today :tada:


Tensorflow version

Hello !
I heard that it's necessary to use Tensorflow 0.12, because with newer versions it doesn't train.
Is it true ? What is the reason for this ?

I think it works with latest version of tf for latest update.


Clean up on game join/player respawn packets

Signed-off-by: liach<EMAIL_ADDRESS>

Closing for https://github.com/FabricMC/yarn/commit/c8192e4d369f0ef977bd123833248ac1da384753#diff-6a22d519560ac2eb75d7c5b07339f3c6R7


Added code to allow specifying a prefix when storing the bundles on the cloud

This is useful when the project is using the --no-git flag, but has multiple environments, such as production, qa and dev.

Will look at this soon, thanks for waiting 👍


Enhancement/link to posts

The post id is now part of the URI
User searches:
/posts?q=Jugendarbeit&location=&radius=&page=1
User opens post:
/posts/8bba27c43f202713c745f2d1704a1725?q=Jugendarbeit&location=&radius=&page=1
User closes post:
/posts?q=Jugendarbeit&location=&radius=&page=1

The implementation is technically correct

- the best kind of correct


webpack build

I can't find instruction on building the latest sources (webpack version)
what packages are needed and what is the build process ?
Thx!

I did yarn install then yarn build.
cat packages.json givies you a list of options under "scripts".
JF

OK it would be great to have a build document .. especially for those like me new to yarn etc
I installed yarn, shx, karma, karma-phantomjs-launcher
OK I installed yarn (new to all this), then went to the node_modules\cornerstone-wado-image-loader directory and did 'yarn build'
I get:
C:\src\WamiPACS 3.x\ZeroFP\node_modules\cornerstone-wado-image-loader>yarn build
yarn build v0.24.5
$ npm run clean:dist && npm run build:es6 && npm run build:codecs

><EMAIL_ADDRESS>clean:dist C:\src\WamiPACS 3.x\ZeroFP\node_modules\cornerstone-wado-image-loader
> shx rm -rf dist

><EMAIL_ADDRESS>build:es6 C:\src\WamiPACS 3.x\ZeroFP\node_modules\cornerstone-wado-image-loader
> npm run test && npm run version && npm run webpack && npm run doc:generate

><EMAIL_ADDRESS>test C:\src\WamiPACS 3.x\ZeroFP\node_modules\cornerstone-wado-image-loader
> npm run test:phantom

><EMAIL_ADDRESS>test:phantom C:\src\WamiPACS 3.x\ZeroFP\node_modules\cornerstone-wado-image-loader
> karma start config/karma/karma-phantom.js

23 05 2017 10:04:24.567:ERROR [config]: Error in config file!
 TypeError: Path must be a string. Received undefined
    at assertPath (path.js:7:11)
    at Object.resolve (path.js:186:7)
    at Object.<anonymous> (C:\src\WamiPACS 3.x\ZeroFP\node_modules\cornerstone-wado-image-loader\config\webpack\webpack-base.js:3:22)
    at Module._compile (module.js:570:32)
    at Object.Module._extensions..js (module.js:579:10)
    at Module.load (module.js:487:32)
    at tryModuleLoad (module.js:446:12)
    at Function.Module._load (module.js:438:3)
    at Module.require (module.js:497:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (C:\src\WamiPACS 3.x\ZeroFP\node_modules\cornerstone-wado-image-loader\config\webpack\webpack-dev.js:1:80)
    at Module._compile (module.js:570:32)
    at Object.Module._extensions..js (module.js:579:10)
    at Module.load (module.js:487:32)
    at tryModuleLoad (module.js:446:12)
    at Function.Module._load (module.js:438:3)

npm ERR! Windows_NT 10.0.14393
npm ERR! argv "C:\\Program Files\\nodejs\\node.exe" "C:\\Program Files\\nodejs\\node_modules\\npm\\bin\\npm-cli.js" "run" "test:phantom"
npm ERR! node v6.9.5
npm ERR! npm  v3.10.10
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>test:phantom: `karma start config/karma/karma-phantom.js`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>test:phantom script 'karma start config/karma/karma-phantom.js'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the cornerstone-wado-image-loader package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     karma start config/karma/karma-phantom.js
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs cornerstone-wado-image-loader
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls cornerstone-wado-image-loader
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     C:\src\WamiPACS 3.x\ZeroFP\node_modules\cornerstone-wado-image-loader\npm-debug.log

npm ERR! Windows_NT 10.0.14393
npm ERR! argv "C:\\Program Files\\nodejs\\node.exe" "C:\\Program Files\\nodejs\\node_modules\\npm\\bin\\npm-cli.js" "run" "test"
npm ERR! node v6.9.5
npm ERR! npm  v3.10.10
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>test: `npm run test:phantom`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>test script 'npm run test:phantom'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the cornerstone-wado-image-loader package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     npm run test:phantom
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs cornerstone-wado-image-loader
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls cornerstone-wado-image-loader
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     C:\src\WamiPACS 3.x\ZeroFP\node_modules\cornerstone-wado-image-loader\npm-debug.log

npm ERR! Windows_NT 10.0.14393
npm ERR! argv "C:\\Program Files\\nodejs\\node.exe" "C:\\Program Files\\nodejs\\node_modules\\npm\\bin\\npm-cli.js" "run" "build:es6"
npm ERR! node v6.9.5
npm ERR! npm  v3.10.10
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>build:es6: `npm run test && npm run version && npm run webpack && npm run doc:generate`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build:es6 script 'npm run test && npm run version && npm run webpack && npm run doc:generate'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the cornerstone-wado-image-loader package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     npm run test && npm run version && npm run webpack && npm run doc:generate
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs cornerstone-wado-image-loader
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls cornerstone-wado-image-loader
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     C:\src\WamiPACS 3.x\ZeroFP\node_modules\cornerstone-wado-image-loader\npm-debug.log
error Command failed with exit code 1.


Correct csh syntax for activate_virtualenv

The . operator in csh executes a directory, which is the wrong behaviour here. At best, when pipenv is installed system-wide (preferably via the operating system package manager), the result is a permission denied error from the shell.
As csh uses the source command, reflect that here.

Github's direct editing is causing more problems wrt indenting at the very least. Closing and opening a new PR when this is sorted without using said direct editing.


Update wireshark casks - launchctl

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} reports no offenses.
[x] The commit message includes the cask’s name and version.

I added /Library/LaunchDaemons/org.wireshark.ChmodBPF.plist to delete by mistake, it should have been added to launchctl.

Merged as 3af99d84462036088a98e6a93de241bfa02501a8.


Sema: Ban uncallable protocol member operators

Member operators of concrete nominal types must declare at least one parameter with that type, like
struct S {
  static func +(lhs: S, rhs: Int) -> S {}
}

For protocol member operators, we would look for a parameter of type Self, or an existential type any P. While the latter was consistent with the concrete nominal type case, it was actually wrong because then the resulting interface type does not give the type checker any way to bind the Self type parameter.
There were two existing test cases that now produce errors, which I believe is now correct. While this is technically a source break, because these bogus operators seemingly cannot be witnessed or called, such a protocol probably had no conforming types.
Fixes https://github.com/apple/swift/issues/73201.

@swift-ci Please smoke test


setAlpha and setOpacity have no effect

Summary
The setAlpha and setOpacity methods don't seem to have any effect on the TinyColor instance.
Possible cause
Looking at the source, I believe this is because these methods are defined like this:
TinyColor setAlpha(int alpha) {
  _color.withAlpha(alpha);
  return this;
}

However, the withAlpha method returns a new Color instance:

Returns a new color that matches this color with the alpha channel replaced with a (which ranges from 0 to 255).

so this doesn't actually change _color.
Context
This bug was originally submitted to https://github.com/FooStudio/tinycolor/issues/14 and was copied here.

I can take a look at this!

Should be _color = _color.withAlpha(alpha);


[5.8] Documentation fix in helpers.php

The e helper does encode special characters instead of escaping them. This PR will make the documentation more clear.

Hmm, I'm in two minds about this. New comers often search for "how to escape html", rather than "encode".

Word escape could be used somewhere in official documentation, then newcomers will find it using the search.


Opening an SVG gives random noise

How to reproduce
https://moellenstedt.iff.kfa-juelich.de/user-redirect/image-viewer/?path=%2Fstorage%2Fer-c-data%2Fadhoc%2Fvadimpcm%2Fpaper%2Fprocessing%2Fcrystallography%2Fpeakassignment.svg&redirects=1
What happens
I get an array of random noise.
What should happen
The SVG image or a helpful error message (format not supported and such) is shown.

Thanks for the report. The random noise is the fallback when either rosettasciio or skimage can't load the requested data, which is the case for SVG files. When I work on #11 you would hopefully get an error message instead.
That said, Panel does support SVG display so we can add another code path to display SVG files using their widget rather than the LiberTEM-panel-ui implementation which only works on raster data.


Fix test for Stripe.URI

Partially addresses https://github.com/robconery/stripity-stripe/issues/51, e.g. fixes the test.

Coverage remained the same at 17.153% when pulling 77a505cdaa2dcde39710efe198545f5ea86e233e on gmile:fix-uri-test into c2f6ae30d1336480134952f965915d99beef15ab on robconery:master.


⚠ Thumbnails API Endpoint has degraded performance

In 05c8e10, Thumbnails API Endpoint (https://thumbnails.roblox.com/v1/assets?assetIds=82738847&format=Png&isCircular=false&size=30x30) experienced degraded performance:

HTTP code: 200
Response time: 2298 ms

Resolved: Thumbnails API Endpoint performance has improved in 78f8214 after 2 minutes.


Changing the Displayname for OpsMx Enterprise Operator for Spinnaker

Signed-off-by: dvropsmx<EMAIL_ADDRESS>Thanks submitting your Operator. Please check below list before you create your Pull Request.

Flat operator directory structure is obsolete from 23-rd of October 2019, only nested directory structure will be accepted.

New Submissions

[ ] Does your operator have nested directory structure?
[ ] Have you selected the Project Community Operator Submissions in your PR on the right-hand menu bar?
[ ] Are you familiar with our contribution guidelines?
[ ] Have you packaged and deployed your Operator for Operator Framework?

[ ] Have you tested your Operator in all supported installation modes?
[ ] Is your submission signed?

Updates to existing Operators

[ Y ] Is your new CSV pointing to the previous version with the replaces property?
[ Y ] Is your new CSV referenced in the appropriate channel defined in the package.yaml ?
[ Y ] Have you tested an update to your Operator when deployed via OLM?
[ Y ] Is your submission signed?

Your submission should not

[ ] Modify more than one operator
[ ] Modify an Operator you don't own
[ ] Rename an operator - please remove and add with a different name instead
[ ] Submit operators to both upstream-community-operators and community-operators at once
[ ] Modify any files outside the above mentioned folders
[ ] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

[ ] Description about the managed Application and where to find more information
[ ] Features and capabilities of your Operator and how to use it
[ ] Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

[ ] Human readable name and 1-liner description about your Operator
[ ] Valid category name1
[ ] One of the pre-defined capability levels2
[ ] Links to the maintainer, source code and documentation
[ ] Example templates for all Custom Resource Definitions intended to be used
[ ] A quadratic logo

Remember that you can preview your CSV here.
--
1 If you feel your Operator does not fit any of the pre-defined categories, file an issue against this repo and explain your need
2 For more information see here

/test CVP

/test CVP

@mvalahtv , Shall i push the same changes to "upstream-community-operators" folder now for complete? Please confirm.

@dvropsmx, Since this PR is for community-operators directory, please make new PR for upstream. And yes you can push same changes. Just new PR.

/test CVP

/test CVP

/test CVP

/test CVP

/test CVP

/test CVP

/test CVP

/test CVP

/test CVP

/test CVP

/test CVP

/test CVP

@dvropsmx Did you create pull request from private repository ?
Gitbub says: it is created from unknown repository
@J0zi @mvalahtv retesting will not help for unknown repository I already explain that few times

@dvropsmx Source repository for this PR was deleted and we cannot run tests. I am closing it now. Can you please recreate new PR if still needed? THanks


Add the TKP feature item.

add the tkp feature item.

Please describe the core capabilities of this feature.

Have been updated


Can't build on Android with React Native 0.74

Environment

Dev OS: OSX 14.1.2
@rnmapbox/maps version: 10.1.19
React Native version: 0.74
Expo version: ❌

Steps to reproduce
npx<EMAIL_ADDRESS>init MapboxRn74 --version 0.74
cd MapboxRn74
yarn add @rnmapbox/maps

Setup MapBox as explained here : https://rnmapbox.github.io/docs/install?install-module=yarn

yarn android

Result
e: file:///Users/frezal/Sources/MapboxRn74/node_modules/@rnmapbox/maps/android/src/main/java/com/rnmapbox/rnmbx/components/AbstractEventEmitter.kt:59:5 'addEventEmitters' overrides nothing

Complete output
yarn run v1.22.22
$ react-native run-android
info A dev server is already running for this project on port 8081.
info Installing the app...
> Task :gradle-plugin:checkKotlinGradlePluginConfigurationErrors
> Task :gradle-plugin:compileKotlin UP-TO-DATE
> Task :gradle-plugin:compileJava NO-SOURCE
> Task :gradle-plugin:pluginDescriptors UP-TO-DATE
> Task :gradle-plugin:processResources UP-TO-DATE
> Task :gradle-plugin:classes UP-TO-DATE
> Task :gradle-plugin:jar UP-TO-DATE
> Task :app:checkKotlinGradlePluginConfigurationErrors
> Task :app:generateCodegenSchemaFromJavaScript SKIPPED
> Task :app:generateCodegenArtifactsFromSchema SKIPPED
> Task :app:generatePackageList
> Task :app:preBuild
> Task :app:preDebugBuild
> Task :app:generateDebugBuildConfig UP-TO-DATE
> Task :rnmapbox_maps:preBuild UP-TO-DATE
> Task :rnmapbox_maps:preDebugBuild UP-TO-DATE
> Task :rnmapbox_maps:writeDebugAarMetadata UP-TO-DATE
> Task :app:checkDebugAarMetadata UP-TO-DATE
> Task :app:generateDebugResValues UP-TO-DATE
> Task :rnmapbox_maps:generateDebugResValues UP-TO-DATE
> Task :rnmapbox_maps:generateDebugResources UP-TO-DATE
> Task :rnmapbox_maps:packageDebugResources UP-TO-DATE
> Task :app:mapDebugSourceSetPaths UP-TO-DATE
> Task :app:generateDebugResources UP-TO-DATE
> Task :app:mergeDebugResources UP-TO-DATE
> Task :app:packageDebugResources UP-TO-DATE
> Task :app:parseDebugLocalResources UP-TO-DATE
> Task :app:createDebugCompatibleScreenManifests UP-TO-DATE
> Task :app:extractDeepLinksDebug UP-TO-DATE
> Task :rnmapbox_maps:extractDeepLinksDebug UP-TO-DATE
> Task :rnmapbox_maps:processDebugManifest UP-TO-DATE
> Task :app:processDebugMainManifest UP-TO-DATE
> Task :app:processDebugManifest UP-TO-DATE
> Task :app:processDebugManifestForPackage UP-TO-DATE
> Task :rnmapbox_maps:compileDebugLibraryResources UP-TO-DATE
> Task :rnmapbox_maps:parseDebugLocalResources UP-TO-DATE
> Task :rnmapbox_maps:generateDebugRFile UP-TO-DATE
> Task :app:processDebugResources UP-TO-DATE
> Task :rnmapbox_maps:checkKotlinGradlePluginConfigurationErrors
> Task :rnmapbox_maps:generateDebugBuildConfig UP-TO-DATE
> Task :rnmapbox_maps:javaPreCompileDebug UP-TO-DATE
> Task :app:javaPreCompileDebug UP-TO-DATE
> Task :app:mergeDebugShaders UP-TO-DATE
> Task :app:compileDebugShaders NO-SOURCE
> Task :app:generateDebugAssets UP-TO-DATE
> Task :rnmapbox_maps:mergeDebugShaders UP-TO-DATE
> Task :rnmapbox_maps:compileDebugShaders NO-SOURCE
> Task :rnmapbox_maps:generateDebugAssets UP-TO-DATE
> Task :rnmapbox_maps:packageDebugAssets UP-TO-DATE
> Task :app:mergeDebugAssets UP-TO-DATE
> Task :app:compressDebugAssets UP-TO-DATE
> Task :app:desugarDebugFileDependencies UP-TO-DATE
> Task :app:checkDebugDuplicateClasses UP-TO-DATE
> Task :app:mergeExtDexDebug UP-TO-DATE
> Task :app:mergeDebugJniLibFolders UP-TO-DATE
> Task :rnmapbox_maps:mergeDebugJniLibFolders UP-TO-DATE
> Task :rnmapbox_maps:mergeDebugNativeLibs NO-SOURCE
> Task :rnmapbox_maps:copyDebugJniLibsProjectOnly UP-TO-DATE
> Task :app:mergeDebugNativeLibs UP-TO-DATE
> Task :app:stripDebugDebugSymbols UP-TO-DATE
> Task :app:validateSigningDebug UP-TO-DATE
> Task :app:writeDebugAppMetadata UP-TO-DATE
> Task :app:writeDebugSigningConfigVersions UP-TO-DATE

> Task :rnmapbox_maps:compileDebugKotlin FAILED
49 actionable tasks: 5 executed, 44 up-to-date

info 💡 Tip: Make sure that you have set up your development environment correctly, by running npx react-native doctor. To read more about doctor command visit: https://github.com/react-native-community/cli/blob/main/packages/cli-doctor/README.md#doctor

e: file:///Users/frezal/Sources/MapboxRn74/node_modules/@rnmapbox/maps/android/src/main/java/com/rnmapbox/rnmbx/components/AbstractEventEmitter.kt:59:5 'addEventEmitters' overrides nothing

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':rnmapbox_maps:compileDebugKotlin'.
> A failure occurred while executing org.jetbrains.kotlin.compilerRunner.GradleCompilerRunnerWithWorkers$GradleKotlinCompilerWorkAction
   > Compilation error. See log for more details

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights.
> Get more help at https://help.gradle.org.

BUILD FAILED in 13s
info Run CLI with --verbose flag for more details.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Fixed by: #3461

I confirm that it works. Thanks for the quick answer. 👍
For the record I had to update to 10.1.20.

Worked for me! 🙂 Thank you so so much, I've been trying to fix this issue for 3 days now, amazing!

can i see your package.json sir ?


GH-36815: [C#]: Enable net472 tests under Windows

What changes are included in this PR?
The primary C# test project is updated to run for .NET 4.7.2 only on Windows platforms.
Are these changes tested?
Yes.

Closes: #36815

Okay, this seems to work.

@github-actions crossbow submit verify-rc-source-csharp-*

@github-actions crossbow submit verify-rc-source-windows

Ah, our RC verification script https://github.com/apache/arrow/blob/main/dev/release/verify-release-candidate.bat doesn't test the C# part...
We need to add it. Could you open an issue for it?
Does the following log show that this case was tested?
https://github.com/apache/arrow/actions/runs/5626539222/job/15247546402?pr=36818#step:6:68
Test run for D:\a\arrow\arrow\csharp\artifacts\Apache.Arrow.Tests\Debug\net472\Apache.Arrow.Tests.dll (.NETFramework,Version=v4.7.2)
Microsoft (R) Test Execution Command Line Tool Version 17.6.3 (x64)
Copyright (c) Microsoft Corporation.  All rights reserved.

Starting test execution, please wait...
A total of 1 test files matched the specified pattern.

Passed!  - Failed:     0, Passed:  1110, Skipped:     0, Total:  1110, Duration: 5 s - Apache.Arrow.Tests.dll (net472)

I clearly should have come back to this before checking in the Duration change (which has now broken the .NET 4.7.2 build).
Opened #38298.

@github-actions crossbow submit verify-rc-source-windows