llvmPackages_8.libllvm: Prevent allocator segfaults on PowerPC

Description of changes
LLVM 9 introduced a fix to LLVM's memory allocator on PowerPC. The underlying issue causes memory segmentation faults on at least powerpc64le-linux in LLVM 8. Since the commit fixing the problem applies cleanly to the affected LLVM version, I added it as a patch.
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

Closing since LLVM 8 has been dropped from Nixpkgs in #283551.


Error Publishing MQTT Message

Tried configuring to publish on insert get error:
"Cannot read properties of undefined (reading 'publish')"
Get error no matter what type of event is triggered
Saltcorn log:
TypeError: Cannot read properties of undefined (reading 'publish')     at Object.run (/home/saltcorn/plugin_packages/@saltcorn/mqtt/index.js:79:16)     at Trigger.runWithoutRow (/home/saltcorn/.local/lib/node_modules/@saltcorn/cli/node_modules/@saltcorn/data/models/trigger.ts:295:14)     at /home/saltcorn/.local/lib/node_modules/@saltcorn/cli/node_modules/@saltcorn/server/routes/actions.js:621:30     at processTicksAndRejections (node:internal/process/task_queues:95:5)

Looks like the client is not configured, have you set up the broker URL in the module configuration? Are you able to receive messages by adding a trigger on the MqttReceive event?

It is setup in the module and will receive MQTT messages it will just not let me send them. should is just be the host name or does it need mqtt:// or ws://?
Kevin Revelle
Director of Operations
[cid:c8f1ccb8-ca49-456a-941e-f7b7256e0105]https://upstreamproductions.ca

From: Tom Nielsen @.>
Sent: Wednesday, December 13, 2023 3:27 AM
To: saltcorn/mqtt @.>
Cc: Kevin Revelle @.>; Author @.>
Subject: Re: [saltcorn/mqtt] Error Publishing MQTT Message (Issue #1)
Looks like the client is not configured, have you set up the broker URL in the module configuration? Are you able to receive messages by adding a trigger on the MqttReceive event?
—
Reply to this email directly, view it on GitHubhttps://github.com/saltcorn/mqtt/issues/1#issuecomment-1853466823, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A3ZSRXIINRWIISO2LXDMHBTYJFRIFAVCNFSM6AAAAABASIY4N2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNJTGQ3DMOBSGM.
You are receiving this because you authored the thread.
The content of this email is confidential and intended for the recipient specified in message only. It is strictly forbidden to share any part of this message with any third party, without a written consent of the sender. If you received this message by mistake, please reply to this message and follow with its deletion, so that we can ensure such a mistake does not occur in the future.


NullPointer while fixing permissions

CommandAPI version
8.7.5
Minecraft version
1.19.3
Are you shading the CommandAPI?
No
What I did
I was editing the commands.yml file of my paper server to disable some commands.
Like this:
command-block-overrides: []
ignore-vanilla-permissions: false
aliases:
  help:
  - []
  '?':
  - []

What actually happened
By modifying commands.yml the commands 'help' and '?' have been removed. However, this caused the plugin to throw a NullPointerException when trying to fix command permissions, after my change.
What should have happened
I expected CommandAPI to simply ignore these now removed commands.
Server logs and CommandAPI config
I use the default config.yml for CommandAPI.
Error trace:
[09:35:17 INFO]: Running delayed init tasks
[09:35:17 WARN]: [CommandAPI] Task #3 for CommandAPI v8.7.5 generated an exception
java.lang.NullPointerException: Cannot invoke "org.bukkit.command.Command.setPermission(String)" because the return value of "org.bukkit.command.CommandMap.getCommand(String)" is null
        at dev.jorel.commandapi.CommandAPIHandler.fixPermissions(CommandAPIHandler.java:529) ~[CommandAPI-8.7.5.jar:?]
        at dev.jorel.commandapi.CommandAPI.lambda$onEnable$0(CommandAPI.java:199) ~[CommandAPI-8.7.5.jar:?]
        at org.bukkit.craftbukkit.v1_19_R2.scheduler.CraftTask.run(CraftTask.java:101) ~[paper-1.19.3.jar:git-Paper-404]
        at org.bukkit.craftbukkit.v1_19_R2.scheduler.CraftScheduler.mainThreadHeartbeat(CraftScheduler.java:483) ~[paper-1.19.3.jar:git-Paper-404]
        at net.minecraft.server.MinecraftServer.runServer(MinecraftServer.java:1114) ~[paper-1.19.3.jar:git-Paper-404]
        at net.minecraft.server.MinecraftServer.lambda$spin$0(MinecraftServer.java:316) ~[paper-1.19.3.jar:git-Paper-404]
        at java.lang.Thread.run(Thread.java:833) ~[?:?]
[09:35:17 INFO]: Done (4.501s)! For help, type "help"

Other
The best way I found to fix the bug was to re-add a value to the disabled commands. Like this:
command-block-overrides: []
ignore-vanilla-permissions: false
aliases:
  help:
  - minecraft:help
  '?':
  - minecraft:help

However, doing this does not disable the commands, it just creates aliases. It would be cool if CommandAPI ignored null commands, or even had an option to disable the permission fix, that creates problems sometimes.
Thank you for reading.

Thanks for raising the issue, I'll see if I can get a release out that fixes this within the next 3 days.
Also worth noting, aliases accepts an object with list values. As such, this is valid because help is a list containing minecraft:help:
command-block-overrides: []
ignore-vanilla-permissions: false
aliases:
  help:
  - minecraft:help
  '?':
  - minecraft:help

However, your example above uses a list of lists: you have help with a single list entry (denoted with the - character), which has the value of an empty list:
command-block-overrides: []
ignore-vanilla-permissions: false
aliases:
  help:
  - []
  '?':
  - []

Presumably you just wanted to assign an empty list (as opposed to a list containing an empty list), so you probably wanted something along the lines of this:
command-block-overrides: []
ignore-vanilla-permissions: false
aliases:
  help: []
  '?': []

Oh, thank you for pointing this out to me, I'm not really familiar with the yaml haha. Unfortunately doing it this way doesn't seems to change the error on my side, but I greatly appreciate your response! :)

@alkanife I have been having trouble reproducing the issue you've raised. Perhaps there may be some paper version differences or changes to the CommandAPI's config.yml file?
I'm running git-Paper-420 (you can find this with /version) with CommandAPI 8.7.5
[20:31:21 INFO]: This server is running Paper version git-Paper-420 (MC: 1.19.3) (Implementing API version 1.19.3-R0.1-SNAPSHOT) (Git: 81d7ff6)
You are running the latest version

That said, I did make an attempt to fix this issue! Since I was never able to reproduce the issue on my machine, it also means I can't test if this actually fixes the issue or not. If you would be willing to try this dev build of 8.7.6 and get back to me if it fixes your issue, that would be great: CommandAPI-8.7.6_24_Feb_2023_(08-25-50pm).zip
(Unzip the file and use the .jar file as a drop in replacement for CommandAPI-8.7.5.jar)

I was using the same paper version as you, and I never changed the CommandAPI configuration.
I tested your dev build and I'm happy to tell you that it fixes the problem! No more errors.
Thanks a lot :)

Fixed in version 8.7.6 (this is effectively identical to the dev build above, but it deserves a proper release)


[$100] Distance rates - Tax reclaimable can be equal Tax rates

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: v9.0.72-0
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: N/A
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/cases/view/3073283
Email or phone of affected tester (no customers): N/A
Issue reported by: Applause Internal Team
Action Performed:
Precondition: Distance rates and Taxes are enabled in workspace more features tab. Track tax is enabled in Distance rates settings.

Go to WS > Distance rates
Click on Default rate ($0,67 / mile)
Set Tax Rate
Click on Tax reclaimable on
Set value > Default rate (e.g. 0.68) > Save
Set value = Default rate (e.g. 0.67) > Save

Expected Result:
Step 5. An error appears 'The reclaimable portion must be less than the distance rate amount.'
Step 6. Value is not saved. Expect that 'The reclaimable portion must be less than the distance rate amount.' OR another error text. User can set 0.66 but not 0.67
Actual Result:
Step 6. Value is saved.
Workaround:
Unknown
Platforms:

[ ] Android: Standalone
[ ] Android: HybridApp
[ ] Android: mWeb Chrome
[ ] iOS: Standalone
[ ] iOS: HybridApp
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
https://github.com/user-attachments/assets/b863db73-69c2-4486-9ec8-7c659ebaf1a1
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~021867265132758522754
Upwork Job ID:<PHONE_NUMBER>758522754
Last Price Increase: 2024-12-12

Issue OwnerCurrent Issue Owner: @mananjadhav

Proposal
Please re-state the problem that we are trying to solve in this issue.
Distance rates - Tax reclaimable can be equal Tax rates
What is the root cause of that problem?
We are only checking that it is not greater here
https://github.com/Expensify/App/blob/48b80f65e80d304965cbeb3573d7d6678fb9bd3a/src/libs/PolicyDistanceRatesUtils.ts#L50-L51
What changes do you think we should make in order to solve the problem?

We should change the condition to >= and update the copy accordingly
What specific scenarios should we cover in automated tests to prevent reintroducing this issue in the future?
What alternative solutions did you explore? (Optional)

@muttmuure this an edge case and one liner change, could be tagged at $100

@FitseTLT's proposal looks good to me.
🎀 👀 🎀 C+ reviewed.

Yep proposal looks good! Given the testing steps and expectations, I don't think we want to allow it to be equal.

@dangrous The PR is ready for your review.


Support connection re-use for did:peer:2/4

See issue https://github.com/hyperledger/aries-cloudagent-python/issues/2703
This PR is WIP, reuse working for public DID and did:peer:2/4, still need to do additional testing and code cleanup.

adds additional meta-data to the wallet DID, to flag a did:peer for re-using in invitations
add the DID meta-data to the admin API GET /wallet/did response
add a flag to the OOB create invitation to request a unique did (default is to re-use the invitation DID in the wallet)
reuse works for did:peer:2 and did:peer:4, and also public DID
added separate flags to the demo for:

--public-did-connections - use the inviter's public DID in invitations, and allow use of implicit invitations
--reuse-connections - support connection re-use (invitee will reuse an existing connection if it uses the same DID as in the new invitation)
--multi-use-invitations - inviter will issue multi-use invitations
note that the --reuse-connections flag needs to be enabled in both Faber and Alice for the demo to enable connection reuse

Also added a few new integration tests with the --connection-reuse flag.

My thoughts on the questions:

reuse works for did:peer:2 and did:peer:4, and also public DID
should this be generic? and how to determine if the DID can be used for connection reuse? should it work for ALL DID methods?

The requirement for reuse in the spec. is that the service item in the invitation be a DID -- not a JSON structure such as happens with an unqualified DID (what we've mostly used to now) or a did:peer:1. As we are dropping support for either of those, likely any DID is valid -- since the DID is resolved.  did:peer:2/4 are magic in that by "resolving" them, the DIDDoc (the data that would be in the JSON structure) is decoded.  Note that did:key is not permitted as it does not have an endpoint to which messages can be sent.
Does that help?

adds additional meta-data to the wallet DID, to flag a did:peer for re-using in invitations

Perhaps this should also allow flagging ledger-based (published/public) DIDs?  E.g. any DID that supports "resuse"?

add the DID meta-data to the admin API GET /wallet/did response

Sounds necessary to me.

Is there any way we can pull out the create methods for did:peer:2/4?. It would be useful to have for integration testing, but I can also patch in the change from this PR.

adds additional meta-data to the wallet DID, to flag a did:peer for re-using in invitations

Perhaps this should also allow flagging ledger-based (published/public) DIDs? E.g. any DID that supports "resuse"?

It's already used for flagging public DID's, so I'm just piggybacking on an existing feature.

The requirement for reuse in the spec. is that the service item in the invitation be a DID -- not a JSON structure such as happens with an unqualified DID

OK this is clear, I'll update to check for a DID, and remove the specific did_peer checks that I added

For did:peer:4 should the connection use the long form or short form of the did?
      "my_did": "did:peer:4zQmbCiTt4dS2XZz9BMGYpZTKH92mYzq9Q1cavQPtfrFEfiV",
      "their_did": "did:peer:4zQmUpGK93RFrakSGmJbwL9QJXqs4uC2S8NQeSyzcomCqKzz",

vs ...
    "my_did": "did:peer:4zQmbCiTt4dS2XZz9BMGYpZTKH92mYzq9Q1cavQPtfrFEfiV:zCo9LoqasV3ZFnxnYNDU8yhBL9F6kGgrTrneiFJHM5Z8L7jw9Gj3H61gCqSKjiMxqWzztJykZfSAM5AXaeyD6wtFwjgyoPMZ6EYSoJ2HwFRd4oafTXtCvGmTC8E4FpjcFaQgnbe1uVbtRmZ1q96mPeMwsDszPiKuyH5D9rHULHkETFwd9hjPKSroX9XVoNxRaprNVr9JLWproTuDzCFwrRYou24dwVAUSsVtnM1ti2PJuZk3D5sdYqDeL8y4yw7WwVvzqd9jykuCt9nQyPBE2DE6saLSAyJRY3tYin2kcdeLCgWJfmaubDVqaEKQnFLZFJodbuts52PHhKGKcgoVXu4nKowE6zedeVtTmZEjn4USXg3oePgnQ47wDbFC5Z1nQgkZG3o7HtWWxmpYZQGqxxZxNDYckfLVxyAdNGZcXN2uHd6i788bc5B5tEEmBptAr9BeaggjobsZanaMDxNHkMLTHbVpn199pNwYXLwpZR8mdB4WMk99FQHyziWHbBLLUihEJqJzsdX88v8",
      "their_did": "did:peer:4zQmUwLj6dLCg2Sv9YirbFkzwFaMyRvmoqSn5Pge9fdP9ATi:zCo9LoqasV3ZFnxnYNDU8yhBL9F6kGgrTrneiFJHM5Z8L7jw9Gj3H61gCqSKjiMxqWzztJykZfSAM5AXaeyD6wtFwjgyoPMZ6EYSoJ2HwFRd4oafTXtCvGmTC8E4FpjcFaQgnbe1uVbtRmZ1q96mPeMwsDszPiKuyH5D9rHULHkETFwd9hjPKSroX9XVoNxRaprNVr9JLWproTuDzCFwrRYou24dwVAUSsVtnM1qintrVJ618X7sPxk13tCXhXHsMUHxK7PeL8sZG8dUFpc75k5waWuCTWK7gQhpvBGoijh4JCZMbqhFWGqUEwRZ3UbgC5f2DWM1MDJFbVtWmgkWtjtiEcJ1366gsLPRpuLYjPfAbMGRD6fXhFLaAmAxJjkEKpR4SvHUucdzv1LYeUZT6dMxf15dY5of2ca8YYSeGdpWYxrfpeMDkkfCSqXbTvXd8opaDEWjdmYMUnQVDBiWiW7C1cuPs73ewcCaeJP11yUbgD24qNtZZFLziaz3XoGRc5N1UfJ6HmCUKTi",

@dbluhm and @TelegramSam — can you please weigh in on this one to help Ian complete the implementation?  Thanks!

For did:peer:4 should the connection use the long form or short form of the did?

I'd say short form unless there's a strong reason not to (there isn't an obvious reason that comes to mind for me).

10 minute scan didn't reveal any red flags for me. I'll have to set aside some time to look deeper.

Integration tests have failed twice in a row, so I ran them locally and everything passed.


Design doc

This task is automatically imported from the old Task Issue Board and it was originally created by Wojciech Daniło.
Original issue is here.

No description provided.

Closing this as we have the document and have moved beyond it,


[NEW] Support "Hyperlink" of the message to be copied

Description
If someone long press a message, there should be an option to copy the hyperlink of it.

Already implemented by #1808.


Error : Video not available

[ X] I'm reporting a broken site support issue
[X ] I've verified that I'm running youtube-dl version 2019.07.12
[ X] I've checked that all provided URLs are alive and playable in a browser
[X ] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[ X] I've searched the bugtracker for similar bug reports including closed ones
[ X] I've read bugs section in FAQ

Verbose log
youtube-dl -v -F https://www.youtube.com/watch?v=PMYLjbUz34w&feature=youtu.be
[1] 1082
root@IS-52763:~# [debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-v', u'-F', u'https://www.youtube.com/watch?v=PMYLjbUz34w']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2019.07.12
[debug] Python version 2.7.9 (CPython) - Linux-3.16.0-6-amd64-x86_64-with-debian-8.6
[debug] exe versions: ffmpeg N-82067-g4d92bd3-static, ffprobe N-82067-g4d92bd3-static
[debug] Proxy map: {}
[youtube] PMYLjbUz34w: Downloading webpage
[youtube] PMYLjbUz34w: Downloading video info webpage
ERROR: This video is not available.
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl/YoutubeDL.py", line 796, in extract_info
    ie_result = ie.extract(url)
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl/extractor/common.py", line 530, in extract
    ie_result = self._real_extract(url)
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl/extractor/youtube.py", line 2097, in _real_extract
    raise ExtractorError(error_message, expected=True)
ExtractorError: This video is not available.

Description
Get ExtractorError: This video is not available. on many videos like the one above. Works absolutely fine in a browser.
EDIT: Well it works on my personal PC but not on my server. If a video is Geo-restricted I usually get a relevant youtube-dl error stating the same!!!
EDIT2: https://polsy.org.uk/stuff/ytrestrict.cgi?ytid=PMYLjbUz34w shows my server country to be valid for this video.

@dstftw Can you pls tell why would this be invalid. I have edited the issue with my findings.
EDIT: Well it works on my personal PC but not on my server. If a video is Geo-restricted I usually get a relevant youtube-dl error stating the same!!!
EDIT2: https://polsy.org.uk/stuff/ytrestrict.cgi?ytid=PMYLjbUz34w shows my server country to be valid for this video.

The error is self-explanatory. If you open it in browser from the same network interface you'll get the same error.


Remove FruitJuice

NOTE: Each discrete (stand-alone) request should be in its own issue.
Put "Remove ____" as Title
[x ] Done
Explain why it should be removed
FruitJuice is no longer available for purchase as of April 17th, 2022.
Additional context
Message on their website:

@danickblouin thx!


Can't compare things against null in asserts in const constructors

This fails in the analyzer with a CONST_EVAL_TYPE_BOOL_NUM_STRING:
class Test { }

class Foo {
  const Foo({
    this.test,
  }) : assert(test == null); // In constant expressions, operands of this operator must be of type 'bool', 'num', 'String' or 'null'.
  final Test test;
}

void main() { }

Constant expression were  changed in the specification for Dart 2 so that this should be allowed.
(The old "e1 == e2 is constant if" were updated to also say "or e1 or e2 is null").

Fixed in https://dart-review.googlesource.com/c/sdk/+/71340


Multiple source dirs

Motivation and Context

Fixes #55
Description

Change --service-path to accept multiple PathBuf
Add Horust::fetch_service_dirs method
Keep in memory list of directories, not an option

How Has This Been Tested?

I tested it manually on Linux machine plus I created a test that checks if all services were loaded from two different locations.
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Performance enhancement (non-breaking change which improves efficiency)
[ ] Code cleanup (non-breaking change which makes code smaller or more readable)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[x] Documentation (a change to man pages or other documentation)

Checklist:

[x] I have updated the documentation accordingly.
[x] I have added tests to cover my changes.

Awesome job, thanks!

@FedericoPonzi can we expect cargo release somewhere soon?

@esavier I've just pushed the latest cargo release (0.1.3) thanks!


Include API Version '2020-10-01-preview' with CloudServices Swagger in readme.azureresourceschema.md

This is required to ensure that ARM Schema can be generated using Swagger. Once the ARM Schema is generated, it is merged and maintained in https://github.com/Azure/azure-resource-manager-schemas.
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Changelog
Please ensure to add changelog with this PR by answering the following questions.

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[x] Other, please clarify

When you are targeting to deploy new service/feature to public regions? Please provide date, or month to public if date is not available yet.
When you expect to publish swagger? Please provide date, or month to public if date is not available yet.
If it's an update to existing version,  please select SDKs of specific language and CLIs that require refresh after swagger is published.

[ ] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[ ] PowerShell
[ ] CLI
[ ] Terraform
[ ] No, no need to refresh for updates in this PR

Contribution checklist:

[ ] I commit to follow the Breaking Change Policy of “no breaking changes
[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[ ] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
Adding a new service

[ ] Please ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from API Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable version with Breaking Change Validation errors
[ ] Updating API(s) in preview over 1 year

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

hi @myyyc could you pls confirm impact to Python SDK? thanks

hi @MYYyc could you pls confirm impact to Python SDK? thanks

Thanks @yungezz for your review. Just sharing my observation - it seems the label: "CI-BreakingChange-Python" is by default applied for all changes in the compute section: https://github.com/Azure/azure-rest-api-specs/commits/master/specification/compute; even when there are no changes to swagger or readme for those specific languages (like in this PR).

@sanchitkum , @yungezz , The Python SDK breaking change label is due to a different change that was checked in quite a while ago, it is unrelated to this PR.


Add black as official code style and add test in CI

Is your feature request related to a problem? Please describe.
Use black styling to enforce styling for new commits
Describe the solution you'd like
Make black the format for python files
Describe alternatives you've considered
yapf, pep8
Additional context
Add any other context or screenshots about the feature request here.

Why not YAPF? Support for 3.5 remains with configuration possibilities as well. Seems more extensible than Black

Why not YAPF? Support for 3.5 remains with configuration possibilities as well. Seems more extensible than Black

discussion 1
top comment is my reasoning

I would recommend going with Autopep8 for now instead of YAPF and Black. There is no delivery pipeline and the project is still nascent. A lot of features are up for debate right now, hence going for a comprehensive linter for a project which is still beginner-friendly seems to be cumbersome. Once features are finalized and a lot of the code becomes standardized, Black can then be used. Linting is difficult to adhere to when writing up new features. Linting heavily should be kept for a later priority.

I would recommend going with Autopep8 for now instead of YAPF and Black. There is no delivery pipeline and the project is still nascent. A lot of features are up for debate right now, hence going for a comprehensive linter for a project which is still beginner-friendly seems to be cumbersome. Once features are finalized and a lot of the code becomes standardized, Black can then be used. Linting is difficult to adhere to when writing up new features. Linting heavily should be kept for a later priority.

I disagree. It is not hard or too much to ask contributors to run black . on the repo before you commit. I don't quite see how having a strict formatter is getting in the way of developing features.
Not too sure what you mean by comprehensive. If I understand correctly, black only formats the Python files.
@hericlesme What do you think?

I would recommend going with Autopep8 for now instead of YAPF and Black. There is no delivery pipeline and the project is still nascent. A lot of features are up for debate right now, hence going for a comprehensive linter for a project which is still beginner-friendly seems to be cumbersome. Once features are finalized and a lot of the code becomes standardized, Black can then be used. Linting is difficult to adhere to when writing up new features. Linting heavily should be kept for a later priority.

Why autopep8?

PEP8 is he code formatting for Python to follow when writing clean and verbose code. This link would be a detailing of how maintainable code can be written. Seeing Linting and Code Analysis-Formatting as together, Flake8 would be a better choice since it is much more analytical giving Mccabe reports as well as linting with Pyflakes. Seeing as Flake8 is also used as industry standard for Python software as well as the tight integration with a lot of popular IDEs, it can be a good choice. This will provide a level of code analysis as well as properly formatted code according to PEP8 and PEP257 guidelines.

PEP8 is the code formatting for Python to follow when writing clean and verbose code. This link would be a detailing of how maintainable code can be written. Seeing Linting and Code Analysis-Formatting as together, Flake8 would be a better choice since it is much more analytical giving Mccabe reports as well as linting with Pyflakes. Seeing as Flake8 is also used as industry standard for Python software as well as the tight integration with a lot of popular IDEs, it can be a good choice. This will provide a level of code analysis as well as properly formatted code according to PEP8 and PEP257 guidelines. I have seen a lot of industry level code, that just does not have good code analysis.

I think you are missing the point.
From that link, I think the difference between flake8 and black is that black is a formatter? No? So we can use flake8 to generate the reports and such, but black would be used as the say all formatter? Referencing here

Black’s formatting rules are a superset of PEP 8. If you choose to use Black, you get PEP 8 compliance for free, plus some other style choices that the Black team sees as helpful for making maintainable code.

The Black team touts the tool as “The Uncompromising Code Formatter.” Practically, this means that Black comes with very few configuration options to affect the style. Limiting options for code formatting is A Good Thing™. With few options, code from project to project that is Black formatted looks very similar.

By using Black, we get the benefit of code that is similar which allows us to focus more on the content than trip over unfamiliar styling choices.

Flake8 can be used with Black as a formatter. Would be helpful for code analysis as well as formatting using black. Check out this package

However it is integrated can be changed, but I think applying a formatting style will make new commits just that much less to have to talk about. Anyone contributing will know exactly how the code should be formatted. Should I switch to flake8-black?

However it is integrated can be changed, but I think applying a formatting style will make new commits just that much less to have to talk about. Anyone contributing will know exactly how the code should be formatted. Should I switch to flake8-black?

So use the package and format it using Black. Flake8 will be able to provide the code analysis when needed as well, if we're setting up the Code Maintenance anyway.


fix: actually suppress slack error

Reason for Change

#6234
Followup to handle case where error occurs in Download or Validate steps (ahead of actual conversion)

Changes

add separate path for Download and Validate errors
add separate Deregister step to account for different json shape of input

Testing steps

tested in rdev ✅

Checklist 🛎

[ ] Add product, design, and eng as reviewers for rdev review
[ ] For UI changes, add screenshots/videos, so the reviewers know what you expect them to see
[ ] For UI changes, add e2e tests to prevent regressions

Notes for Reviewer

Confirmed no runs in dev or staging; merged.

Can you addd a picture of the machine to make it easier to understand the change


[VTK|OpenCV4|libharu:x64-windows] build failure: silent exit just after libharu[core,notiffsymbols] successful build/install

Host Environment

OS: Windows
Compiler: Microsoft (R) C/C++ Optimizing Compiler Version 19.28.29335 for x64

To Reproduce
Steps to reproduce the behavior:
./vcpkg install --triplet x64-windows vtk opencv4[VTK]
Failure logs
No real logs (Please tell me what could help..), but the last output is:
Building package libharu[core,notiffsymbols]:x64-windows... done
Installing package libharu[core,notiffsymbols]:x64-windows...
Installing package libharu[core,notiffsymbols]:x64-windows... done

>  command prompt (nothing more)

No errors are reported. But if I relaunch the same command (./vcpkg install --triplet x64-windows vtk opencv4[VTK]), the build continue and all package is rightly built, without any error.
And more problematic for me, is that the export command won't work:
vcpkg.exe export --triplet x64-windows --x-all-installed --7zip --output=perlimpinpin
will perform things, without errors, but at the end, no archive are present. It seems to block also on libharu. No problem without VTK, and with VTK alone.
Please tell me if there is a verbose/debug mode, or some logs, for this operation, so I can maybe give you better information.

Hi @dweckmann, thanks for reporting this issue!
Can you use command ./vcpkg install --triplet x64-windows vtk opencv4[VTK] --debug and provide the output?
Thanks.

 .\vcpkg.exe install --triplet x64-windows --debug vtk opencv4[VTK] > build.log
build.log

According to the log you provided, I found that the content of the final log is inconsistent with the information you provided before: it seems that vcpkg is stuck when preparing to build libogg[core]:x64-windows.
Please make sure that the anti-virus is turned off, and check whether the vcpkg process has exited through the task manager and whether the process has cpu/disk operations.
Thanks.

Hi @JackBoosY,
I've tested again, with antivirus fully deactivated and not only with directory "excluded", and I the problem seems to be gone.
Thanks for your help !

Since the issue seems to be resolved, I close it :)


DateInput onChange type oddities

The type for the DateInput's onChange handler seems to resolve to React.FormEvent<HTMLDivElement> | DateInputChangeEvent.

I would expect it to be DateInputChangeEvent.
Current work around is to explicitly set the type:

This should be fixed in v3.5.1.


[MLU] fix rnn with set projection zero

[MLU] fix rnn with set projection zero

Need to upgrade MLU SDK next week @ShawnNew


Create abstract class to define interface for tracker objects (Charuco/Mediapipe/Etc)

Tracker object Created...though need to implement that...


[iOS] not configuring locationsCollection/geofencesCollection will result in a crash

Your Environment

Plugin version: 0.1.0
Platform: iOS

Actual Behavior
On iOS not configuring either the locationsCollection or geofencesCollection can result in a crash when it tries to write the log to the not configured collection.
Expected Behavior
This should not crash the app. And remind the developer to configure the collectionName.
Steps to Reproduce
Don't configure the geofencesCollection like so:
                bgGeoFirebase.configure({                     locationsCollection: 'geopositions',                 });
Now trigger a geofence. This will crash the app.
Solution
You should check if the collectionName is not nil in the onPersist method (and give the developer a hint to configure the collection). So it will not try to write the a 'nil' path.
ps on Android I did not encounter this problem.

Where the crash log?

Here you go:

✅-[TSLocationManager locationManager:didUpdateLocations:] Acquired motionchange position: <+XX.XXXXXX,+X.XXXXXXX> +/- 65.00m (speed -1.00 mps / course -1.00) @ 11/09/2019, 13:42:30 British Summer Time
2019-09-11 13:42:30.441333+0100 MyApp[5352:2010775] ✅-[TSLocationManager locationManager:didUpdateLocations:] Acquired motionchange position: <+XX.XXXXXX,+X.XXXXXXX> +/- 65.00m (speed -1.00 mps / course -1.00) @ 11/09/2019, 13:42:30 British Summer Time
ℹ-[TSConfig persist]
2019-09-11 13:42:30.443148+0100 MyApp[5352:2010782] ℹ-[TSConfig persist]
2019-09-11 13:42:30.444093+0100 MyApp[5352:2010807] *** Terminating app due to uncaught exception 'FIRInvalidArgumentException', reason: 'Collection path cannot be nil.'
*** First throw call stack:
(0x18fd35c20 0x18fa500c8 0x100ad01f8 0x1008a2a04 0x1008d1dc0 0x1005a4f38 0x103099e48 0x10309b2a8 0x1030acccc 0x1030ad418 0x18fa42a60 0x18fa48c78)
libc++abi.dylib: terminating with uncaught exception of type NSException

Try installing from branch ios-default-collections
ionic cordova plugin add https://github.com/transistorsoft/cordova-background-geolocation-firebase.git#ios-default-collections

What exactly is different in this branch? (I was not able to spot any differences)

Forgot the commit.  Try again.


gui icon's getting broken after joining and leaving some job

Detailed description of the issue:
There is a bug in gui (gui from /joibs join command) when someone:

joins the job
leaves this job
this job's icon in gui gets broken - 2 screenshots examples
https://i.imgur.com/c1R7JO7.png
https://i.imgur.com/YqhRisB.png

Jobs version:
<IP_ADDRESS>
Server Type (Spigot/Paper/etc):
Paper
Server Version (using /ver):
This server is running Paper version git-Paper-196 (MC: 1.20.1) (Implementing API version 1.20.1-R0.1-SNAPSHOT) (Git: 773dd72)
Relevant plugins (Delete if this isn't needed):
CMILIB <IP_ADDRESS> (i also tried with <IP_ADDRESS> and still happening)

Can you elaborate more on this one? What exactly changes? As its hard to determine this from your images.

Can you elaborate more on this one? What exactly changes? As its hard to determine this from your images.

As i marked on the screenshot - on default page of /jobs join GUI there are added some random lines from the another page of that GUI (from page with earnings of that job - for example information about earnings from killing a player in Miner - Gornik job "Player 7.58$ 7.58pkt 7.50XP" - https://imgur.com/c1R7JO7 )

c5863bb121062bfdc3e8242eca6a61da4ce0a512


updates version to 10.0

This change is

Review status: 0 of 2 files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


HARP-9463 Added support for deprecated minZoomLevel, maxZoomLevel in …

…constructor of TileDataSource and OmvDataSource to improve backwards compatibility.
Signed-off-by: Marcel Pursche<EMAIL_ADDRESS>Thank you for contributing to harp.gl!
Before requesting a pull request, please remember to check the following documents:

contribution guidelines
coding style

If you are adding new functionality we would highly appreciate if you can describe what is the capability you are adding and even better if you can add some examples. Please also remember to add tests for it.
CI Check
Our bots will check whether your PR can be directly integrated into the mainline. We have some internal integration tests running on the background, our bots will inform you of the next steps and someone from our team will take a look and help if needed!
And please do not forget to sign-off your commit! You can read more about DCO here. But, in short, you just need to use git commit -s or append --signoff when you are committing to the repo.
Happy contributing!

Codecov Report

Merging #1398 into master will decrease coverage by 0.07%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master   #1398      +/-   ##
=========================================
- Coverage   54.98%   54.9%   -0.08%
=========================================
  Files         266     255      -11
  Lines       23761   23473     -288
  Branches     5328    5275      -53
=========================================
- Hits        13064   12889     -175
+ Misses      10697   10584     -113

Impacted Files
Coverage Δ

@here/harp-omv-datasource/lib/OmvDataSource.ts
55.73% <ø> (ø)
:arrow_up:

@here/harp-mapview-decoder/lib/TileDataSource.ts
62.5% <ø> (ø)
:arrow_up:

@here/harp-mapview/lib/DataSource.ts
90% <100%> (+12.72%)
:arrow_up:

...re/harp-text-canvas/lib/rendering/TextMaterials.ts
42.85% <0%> (-34.07%)
:arrow_down:

@here/harp-mapview/lib/ThemeHelpers.ts
6.55% <0%> (-0.68%)
:arrow_down:

@here/harp-omv-datasource/lib/clipPolygon.ts
96.96% <0%> (-0.47%)
:arrow_down:

...rp-mapview/lib/text/TextElementsRendererOptions.ts
100% <0%> (ø)
:arrow_up:

@here/harp-datasource-protocol/lib/Outliner.ts
100% <0%> (ø)
:arrow_up:

...ere/harp-materials/lib/LuminosityHighPassShader.ts
100% <0%> (ø)
:arrow_up:

@here/harp-test-utils/lib/WebGLStub.ts
100% <0%> (ø)
:arrow_up:

... and 32 more

Continue to review full report at Codecov.

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


laithong

hello

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

ID: 41f07171-f936-3d88-fbcb-d200605fd90d
Version Independent ID: 22aad290-7d51-429b-f59e-dd339355c753
Content: Quickstart: Call your Bing Custom Search endpoint using Node.js
Content Source: articles/cognitive-services/Bing-Custom-Search/call-endpoint-nodejs.md
Service: cognitive-services
GitHub Login: @aahill
Microsoft Alias: aahi

@Laithong Thanks for reaching out! Would you please provide some detail?

@Laithong do you still need help?

@Laithong Since he haven't heard back from you, we will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


[backport] Make device functions public

Backport of #6744

[automatic post] Jenkins, test this please.

Jenkins CI test (for commit 242f4ebc1061644798bfc0c67d843b5c8e70f664, target branch v6) failed with status FAILURE.

The test failures seem unrelated to this PR.


[SPARK-33824][PYTHON][DOCS] Restructure and improve Python package management page

What changes were proposed in this pull request?
This PR proposes to restructure and refine the Python dependency management page.
I lately wrote a blog post which will be published soon, and decided contribute some of the contents back to PySpark documentation.
FWIW, it has been reviewed by some tech writers and engineers.
I built the site for making the review easier: https://hyukjin-spark.readthedocs.io/en/stable/user_guide/python_packaging.html
Why are the changes needed?
For better documentation.
Does this PR introduce any user-facing change?
It's doc change but only in unreleased bracnhs for now.
How was this patch tested?
I manually built the docs as:
cd python/docs
make clean html
open

cc @srowen, @zero323, @viirya, @dongjoon-hyun, @erikerlandson, would you mind taking a quick look when you guys find some time?

Test build #132949 has started for PR 30822 at commit 414b135.

Test build #132949 has finished for PR 30822 at commit 414b135.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

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

retest this please

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

Test build #132952 has started for PR 30822 at commit 414b135.

Test build #132952 has finished for PR 30822 at commit 414b135.

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

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

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

Test build #132970 has started for PR 30822 at commit 68323e2.

Thank you guys. Merged to master and bracnh-3.1.

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

Test build #132970 has finished for PR 30822 at commit 68323e2.

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

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

Just in case anyone is interested in, my blog post was published yesterday here https://databricks.com/blog/2020/12/22/how-to-manage-python-dependencies-in-pyspark.html :-).


UsersIndex.create! not working with ES version 7.x

I have a users_index class as following:
app/indexes/users_index.rb
class UsersIndex < Chewy::Index
  define_type User.active do
    field :first_name
  end
end

With ES version 7.6.2, When I run UsersIndex.create!, it raises an error "type": "mapper_parsing_exception",         "reason": "Root mapping definition has unsupported parameters:
While when I do the same with version 6.8.8, it works fine.
I read that the body argument structure has been changed in ES version 7.
So, with version 7, request format should be:
PUT /users
{
  "mappings": {
      "properties": {
        "first_name": {
          "type": "text"
        }
    }
  }
}

While for earlier versions, it should be
PUT /users
{
  "mappings": {
     "user": {
        "properties": {
          "first_name": {
            "type": "text"
          }
      }
    }
  }
}

And, its failing because of this change.
I think changing chewy-5.1.0/lib/chewy/fields/root.rb#33 from mapping to mapping[name] should make it work fine.
Basically, it's returning the mapping hash as
{
    "user": {
        "properties": {
            "first_name": {
                "type": "text"
            }
        }
    }
}

While, for the 7.x versions, it should return as
 {
    "properties": {
        "first_name": {
            "type": "text"
        }
    }
}

I am facing same issue, after searching for a while, ES 7.x has removed mapping types https://www.elastic.co/guide/en/elasticsearch/reference/current/removal-of-types.html

@sachin1mittal is correct, returning mapping[name] is working and indexing mapping correctly for me.

related to #673

@sachalifs before it's fixed, you can try this nasty monkeypatch which fixes it for us on ES 7.8: https://gist.github.com/reneklacan/5ed4bd08b0ad43cd81cab38530ae0889

@sachalifs before it's fixed, you can try this nasty monkeypatch which fixes it for us on ES 7.8: https://gist.github.com/reneklacan/5ed4bd08b0ad43cd81cab38530ae0889

Maybe you want to create a PR with these fixes?

Please take a look at Chewy 7.0.0 release. Stay tuned for the 7.1 release, with dropped support for ES6.8 and removed deprecation warnings.


Add support for 'download thing and then unzip / untar / whatever' installer

After #3, the next way to make installers more flexible and easy to use is to let people apply some sort of 'unzip' to the artifact. This would help for legacy applications at ST that are 'built' by creating a tarball. It would also be useful for deploying WARs.

Further notes: this could be solved by just introducing the concept of a "post install hook". We could ship with some defaults like "unzip something", "untar/gz something", etc.

Upon further reflection, this probably doesn't need to depend on #3

Not happening.


Toolbar component

Closes
Update: Scope reduced, this only supports children of types: Button, Checkbox, Link, RSP ActionGroup
Guidance from accessibility:

ActionGroup should support role="group" so that you can have groups within a toolbar without nesting toolbars.

I've tested this out. If the toolbars are nested, then the outermost label is not read. If the inner toolbars are groups, then both labels are read as expected. This can be seen simply in this demo. I recommend popping out into a new window before using VO.
https://codesandbox.io/s/sleepy-northcutt-hfhwnr?file=/src/App.js
Implementing a subset of https://www.w3.org/WAI/ARIA/apg/patterns/toolbar/examples/toolbar/
Open questions, do we need the RSP component?
Answer: Not at the moment, we don't want this held up by a design review.

extra info: I've created a basic version of it so that we can test interactions with ActionGroup, which is the currently the primary use case. What I've implemented is pretty much what anyone else would need to implement to use it in Spectrum.

✅ Pull Request Checklist:

[ ] Included link to corresponding React Spectrum GitHub Issue.
[ ] Added/updated unit tests and storybook for this change (for new code or code which already has tests).
[ ] Filled out test instructions.
[ ] Updated documentation (if it already exists for this component).
[ ] Looked at the Accessibility Practices for this feature - Aria Practices

📝 Test Instructions:

🧢 Your Project:

I noticed when using Up/Down arrow on horizontal, focus stays within the groups (and vice versa for vertical). Is this expected?

This is expected https://www.w3.org/WAI/ARIA/apg/patterns/toolbar/ it just happens that ActionGroups use the opposing direction to navigate as well and I just let the child component handle the keys I don't. It's hard to know enough about the children to do anything other than this. For instance, controlling a slider vs navigating a menu vs numberfield spin button. So I think this is an ok interpretation.


NoClassDefFoundError: org/json/JSONArray

I download release and add the jar to my build path.
Then I add code:
String input = "A string 😄with a \uD83D\uDC66\uD83C\uDFFFfew 😉emojis!";
String result = EmojiParser.removeAllEmojis(input);

I get the error: NoClassDefFoundError: org/json/JSONArray
I found http://mvnrepository.com/artifact/org.json/json/20080701 which has a JSONArray class. But is that the correct JSONArray?

You want this as a dependency in your project:
https://github.com/stleary/JSON-java

Is that in the readme?

The README gives instructions to use the project with a package manager (Maven, Gradle...).
If you don't use any package manager, you have to resolve the dependencies yourself.
Feel free to send a PR to update the README with instructions if you want to help the next developer :)

And yeah, I used http://mvnrepository.com/artifact/org.json/json/ (see pom.xml)

Submitted a pull request.


Move uploadFiles onto ElementHandle

This removes page.uploadFile and frame.uploadFile

We can't land this until 1.0 since it's a breaking change. Let's mirror it on element instead


chore: create vester bot code and initial tests

Codecov Report

Merging #166 (13bb7d5) into develop (e71b8c3) will increase coverage by 2.14%.
The diff coverage is 68.49%.

@@             Coverage Diff             @@
##           develop     #166      +/-   ##
===========================================
+ Coverage    25.52%   27.66%   +2.14%
===========================================
  Files           17       18       +1
  Lines         1289     1359      +70
===========================================
+ Hits           329      376      +47
- Misses         960      983      +23

Impacted Files
Coverage Δ

src/vester.py
66.17% <66.17%> (ø)

config/constants.py
100.00% <100.00%> (ø)

src/aws.py
100.00% <100.00%> (ø)

src/token_utils.py
100.00% <100.00%> (ø)


add the remaining mediaelement events

Add remaining mediaelement events.
https://developer.mozilla.org/en-US/docs/Web/API/HTMLMediaElement

abort
emptied
canplaythrough
loadeddata
loadedmetadata
loadstart
ratechange
stalled
suspend
timeupdate

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

15 of 20 (75.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 76.71%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/mediaelement.js
15
20
75.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
1235

Relevant Lines:
1525

💛  - Coveralls

As I mentioned, I'm reluctant to accept this pull request for the following reasons:

It only covers the media element-specific events, which makes the other backends' API inconsistent
They are "backend" events which were never meant to be public (only the main wavesurfer events are officially public)
If we're bypassing the official/internal events distinction anyway, these events can be trivially subscribed to directly via wavesufer.backend.media.addEventListener. Making a single shortcut for it in the public API (e.g. wavesufer.addMediaListener) would make more sense.

Ultimately, it's unclear to me what issue this is solving. Has anyone requested this? If not, I suggest we close the PR.
Thanks @entonbiba!


no code completion;no Hover

Environment
OS: 6.8.0-48-generic Ubuntu22.04.5LTS GNOME42.9
neocmakelsp: 0.0.14
vscode: 1.95.3
no code completion

no Hover

Have you installed cmake?

Have you installed cmake?

Sorry, I forget to close this issue, just because my cmake is broken.

Please check your cmake.


rosconsole: 1.13.16-1 in 'melodic/distribution.yaml' [bloom]

Increasing version of package(s) in repository rosconsole to 1.13.16-1:

upstream repository: https://github.com/ros/rosconsole.git
release repository: https://github.com/ros-gbp/rosconsole-release.git
distro file: melodic/distribution.yaml
bloom version: 0.9.7
previous version for package: 1.13.11-1

rosconsole
* fix message in filter parameters (#43 <https://github.com/ros/rosconsole/issues/43>)

Waiting for other Melodic releases...


[Question]the question about updating the data on the web

Thank you for providing the best tool to monitor my server!
I have read all the code of this project in order to learn Flask.
But now I have an issue that I don't know the way you update the performance data to web in real time.
Thank you in advance!

https://github.com/Jahaja/psdash/blob/master/psdash/static/js/psdash.js


about:blank#blocked error from web bluetooth permission link

I get an about:blank#blocked page from CLICKING the "about://flags/#enable-web-bluetooth-new-permissions-backend" link in the code.circuitpython.org page
#17  is a pr that changes the link to chrome://flags/#enable-web-bluetooth-new-permissions-backend
[CLICKING added for clarity}

Now closed
Discussion in the pr.
As I understand it @makermelissa tested the change, and it didn't make an effect.
Cut and paste of the "about://flags/#enable-web-bluetooth-new-permissions-backend" link is the workaround.

Closing

I wonder if this could be fixed with a bit of JavaScript that sets the window.location when clicked? 🤔


Release zips without tests

The Release zip files with the tests included are rather enormous (>100 MB).  On a slow connection (for instance when having to log into corporate network through VPN), it takes absolutely ages(!) to check out the source of the repo.  Having the tests in the repo itself seems like a bit of an anti-pattern to me, but nonetheless, a useful compromise would be to also add test-free zip files because they would be much, much smaller (about 10 MB), and would result in a much faster download.

Also, I can't just download the header because I need the natvis file for Visual Studio and the cmake goodies.

There is a ongoing PR #1251 that aims at solving the issue, if you happen to know git-lfs, I'll gladly take your input on it!


Start the parser (WIP)

Description

Motivation and Context

Closes #
How Has This Been Tested?

[ ] I have tested using MacOS
[ ] I have tested using Linux
[ ] I have tested using Windows

Checklist

[ ] I have updated the documentation accordingly.
[ ] I have updated the tests accordingly.

Codecov Report

Merging #4 (c094d4c) into main (4c589d2) will decrease coverage by 2.04%.
The diff coverage is 88.88%.

@@             Coverage Diff             @@
##              main       #4      +/-   ##
===========================================
- Coverage   100.00%   97.95%   -2.05%
===========================================
  Files            3        4       +1
  Lines          354      391      +37
===========================================
+ Hits           354      383      +29
- Misses           0        7       +7
- Partials         0        1       +1

Impacted Files
Coverage Δ

parser/parser.go
78.37% <78.37%> (ø)

lexer/lexer.go
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Update keyword documentation for 1.7.2

I seem to have forgotten to update the documentation.
I'll try to get to that tonight.
@emanlove, feel free to go ahead and do it if you have a free minute or two.

Done.


remove wrong core group in PolicyRule

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: diemtvu
If they are not already assigned, you can assign the PR to them by writing /assign @diemtvu in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

security/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Hi @wackxu. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/assign @diemtvu

/lgtm
/approve

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: myidpt
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

security/OWNERS [myidpt]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@islinwb for ok-to-test

/ok-to-test


complex hotwords support #Current Model Limitations Discussion

Hi,
Thanks for your helpful research. I wonder if the current model can handle complex hot words like "Hey Siri" or just handle one word, like "Siri"?
My second question is about hot words that their pronunciation takes more than 1s, like"Hey XXXX." Does your model support changing the recording time?
Did you try to use cosine_similarity instead of Euclidian distance in inference time?
Thanks.

+1
We also want to use an about 2-seconds-long custom hotword, but with the current python -m eff_word_net.generate_reference method, the detection seems to be awkward.
So would like to have support changing the recording time too!

Sorry for the delayed response, the model was currently trained on single words , however its should work simple phrases like Hey xxx though . Moreover the current model was trained on 1 sec audio clippings , so bizare behviour might occur on trying to process audio clippings greater in length than 1 sec
The model was trained using Euclidean distance hence works on the same during inference time too
Coming to increasing hotword length, hotwords are usually small , may be we can extend the processing window to 1.5 sec , but 2 sec I am not really sure . Can you give a few examples where a hotword could be greater than 1.5 secs?
Kindly give you additional model suggestions in discussions page

Thanks for the Information.
An example of a long wake word is "Hey MercedesBenz".
Could you please provide the training steps?

sorry for the delay , didn't have time to clean the repository which held the training code , the same is built using keras https://github.com/Ant-Brain/wakeword_dataset_generator . It has both the training code and dataset generator code

Hey, thanks for this repo.
I can not find your training code here https://github.com/Ant-Brain/wakeword_dataset_generator . is it available in any other repo?

Extremely sorry for the delay , my bad forgot to add the notebook which contained the training code
https://colab.research.google.com/drive/1bdrdBZFi0I23JfCdNME_NnJTqbiO_U-R

Currently working on a newer model with better perfomance and higher hotword length, will be available in a month's time

Update
A newer model with better resilience to noise, 1.5 secs window support has been added to the flow . kindly check it out!!


Issue 360 existing organization id error

Implements: https://github.com/nearform/labs-authorization/issues/360

Cleaned up the Joi validation file,
Added Joi validation and tests for the Name field on orgs, users, teams and policies,
Team parent id Joi validation,
Added 400 bad request for cases when trying to create element with existing id. Added tests and error creation for orgs, users, teams and policies.

Coverage increased (+0.1%) to 96.15% when pulling 4a6ecaf2dd9457c29f3f75d501ee89240df72243 on issue-360-existing-organization-id-error into 0049b7003ec0d8a426e0476853f12a6e6b1aa103 on master.


show blips name when hovering

Any progress on this?

Any progress on this?

I have it in a local branch, I'll try to push whenever I have sometime


Fixes #407

Thanks i'll review that asap
Il lun 14 nov 2016, 18:05 Jean-Francois Larouche<EMAIL_ADDRESS>ha scritto:

You can view, comment on, or merge this pull request online at:
https://github.com/mcasimir/mobile-angular-ui/pull/408
Commit Summary

Fixes #407

File Changes

M src/js/gestures/swipe.js
https://github.com/mcasimir/mobile-angular-ui/pull/408/files#diff-0
(7)
M src/js/gestures/touch.js
https://github.com/mcasimir/mobile-angular-ui/pull/408/files#diff-1
(2)

Patch Links:

https://github.com/mcasimir/mobile-angular-ui/pull/408.patch
https://github.com/mcasimir/mobile-angular-ui/pull/408.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mcasimir/mobile-angular-ui/pull/408, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAUcIZ26mxFJlEPrQqE_VxQiNLlHADxtks5q-JS9gaJpZM4Kxjxi
.

No local lint build. Sorry for spam :/

Current coverage is 82.56% (diff: 100%)

Merging #408 into master will not change coverage

@@             master       #408   diff @@
==========================================
  Files            14         14
  Lines           562        562
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            464        464
  Misses           98         98
  Partials          0          0

Powered by Codecov. Last update 5ec10ac...859fc9e

Its fully tested now. I forgot a little detail for default behabior sorry.

Hello!
Any updates? Is this project still maintained?

Hi, yes it is. I'll look into that today.
Il lun 16 gen 2017, 02:59 Jean-Francois Larouche<EMAIL_ADDRESS>ha scritto:

Hello!
Any updates? Is this project still maintained?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/mcasimir/mobile-angular-ui/pull/408#issuecomment-272757398,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAUcIYiyPsGyf6HV93Qp01nCxOV0UZIyks5rSs7rgaJpZM4Kxjxi
.

Ill just continue on fork. Thanks a lot for the lib.


Data Manager implements container-interop

I have created a data-manager at https://github.com/chrismichaels84/data-manager that is fully implements container-interop.
At its core, data-manager is a simple container with some sugar. What makes it different is that everything is done using traits. There are several extra traits (di container, loading config files, collections, etc) that can be mixed and matched to integrate data-manager directly into an existing project.
Everything is well unit tested, documented, and follows psr conventions and SOLID principles.
I would love to chat about inclusion on this list.
Thank you.

Hey Michael!
Nice work! I've had a quick look at the documentation and it can really prove useful for managing configuration of projects.
Do not hesitate to open a PR adding your project on the README.md of container-interop and I'll accept it.

I've opened the PR and am closing this issue. Thanks!


Advanced search form

Thank you for creating this plugin. The faceting and indexing options greatly improve searching in Omeka.
However, we miss advanced search fields. Is this an option that you are considering for a future release? And in the meantime, would you be willing to point me in the direction of creating and advanced search form for my installation?
Thanks for considering!

Hi there,
I've been digging in to documentation to add advanced search capability to my Omeka Solr Search instance. I see that the fields used by the plugin are dynamic fields, including the element-text-id. However when I try to construct a query with these fields, my results are exactly the same as a general keyword search.
Is there something I can do to search within one of these dynamic fields?
Any tips you have are much appreciated!!

Just following up on what I discovered, to aid future inquirers...
The plugin's controller specifically filters out colons in the search term, at least in the "q" parameter: https://github.com/scholarslab/SolrSearch/blob/master/controllers/ResultsController.php#L119
To create a solr advanced search with this plugin, I first commented out the line above. Then I was able to construct pre-filtered query strings (eg. /solr-search?q=63_s:Frankfort+AND+62_s:1864). I didn't notice any negative consequences from commenting out line 119, however the plugin authors might be able to speak to the potential problems of this change.


Can't run AutoGen on linux: ImportError: Please install openai>=1 and diskcache to use autogen.OpenAIWrapper.

I tried #407   #311, and nothing help me.
I'm using Conda with pyautogen installed, on ubuntu.
What can I do? Is there any fix to the lib? must admit it's bit frustration...
Trying to run:

import autogen
from autogen import AssistantAgent, UserProxyAgent

config_list = [
	{
		"api_base": "http://<IP_ADDRESS>:8000",
		"api_key" : "NULL",
	}
]
llm_config = {
	"request_timeout" : 800,
	"config_list" : config_list
}

assistant = AssistantAgent(
    "assistant",
    llm_config = llm_config
)

user_proxy = UserProxyAgent(
    "user_proxy",
    code_execution_config = {
        "work_dir" : "coding"
    }
)

user_proxy.initiate_chat(
    assistant,
    message ="What is the name of the model you are based on?"
)

got:
autogen/oai/client.py", line 214, in create
    raise ERROR
ImportError: Please install openai>=1 and diskcache to use autogen.OpenAIWrapper.

Hi @barshag,
The error message you have above suggests that you have the latest version of autogen installed (which requires openai version > 1.00), but you have a lower version of open ai installed. To address this try the following:

Check your version of pyautogen (should be > 0.0.2)
Check your openai version (should be greater than 1.). I recommend upgrading with pip install -U openai

Let me know if this helps.

Closing this issue due to inactivity. If you have further questions, please open a new issue or join the discussion in AutoGen Discord server: https://discord.com/invite/Yb5gwGVkE5


Override Sphinx integrated search with our indexes

The Sphinx search "works", but our search returns more relevant results. This work will include an integrated front end interface for search results on generated documentation which pulls from our existing indexes.
Implementing this feature might be difficult on a wider basis, as themes all load their local _static/searchtools.js. The easiest place to start would be by reimplementing searchtools.js with a similar interface, or extending the existing implementation, and overriding this in our local template overrides path. This would only affect projects using our theme, or derivatives of, for now.
The current searchtools.js implementation uses it's own search index data and performs its own text normalization and result scoring. An overridden implementation would rely on an elastic search backend, with supplemental information from the existing implementation. The current index has information on domain objects, which aren't surfaced in the same way in the search index. Results could be merged into the same search results tree, for display output. Features like highlighting in search results and the current search UI elements will be maintained.

[x] Provide basic override of search tools javascript from Sphinx
[x] Test with other themes
[x] Test sub/super project display on listings
[x] Explore merging domain objects -- can we detect duplicates easily in search results, or are search results enough?
[x] Refactor to use search.lib
[x] Move searchtools override to RTD sphinx ext
[x] Fix linking to projects, add canonical url mapping for projects

This relies on rtfd/readthedocs-sphinx-ext#16

This is now unblocked. I created a sphinx_rtd_ext release branch of 0.6-alpha for staging some of our changes to moving changes into the ext. It would be good to get #1946 in this release as well.


[gn] python3 compatible

git-sync-deps also resolved
users can using either python2.7 or python3.x now. after clone, users can run: python3 tools/git-sync-deps, then run gn & ninja for a skia library

@googlebot I signed it!


Restrict safekeeper -> pageserver connection thread lifetime to the lifetime of connection from walproposer

Otherwise they breed uncontrollably.

@arssher Could you elaborate more about the behavior you observed? I would have assumed that it was at least partially handled by the page server only running one walreceiver connection per timeline.

#636 #117 #589 all relate to the similar issues and are on hold for now


Table Header on Every Page Break

I am rendering a long table so I have added pagebreak avoid so that my table rows don't get cut on end of every page.
Now I want to add thead on every pagebreak of the table. So, that header is there on every page where the table is present.
Using this CSS on thead if I print the page using chrome print command I get Thead on every page but using html2pdf I am unable to get thead on every page. Is there a functionality for this already or does this needs to be added?
    const pdfOptions = {
        margin: 10,
        filename: `Test.pdf`,
        image: { type: 'jpeg', quality: 0.98 },
        html2canvas: { scale: 2.5 },
        jsPDF: { format: 'a4', orientation: 'landscape' },
        pagebreak: { mode: ['css'], avoid: 'tr' }
    };
    html2pdf(Container, pdfOptions);

CSS:-
thead {
display: table-header-group;
}

I am having the same problem. I have no idea how to fix it. It works normally on Chrome. :(


isFluidWidth prop

Hey, first of all, great work on that fork!
Is there any way of mimicking isFluidWidth prop as it seems it was removed?
Thanks!

Hi @vojthor! The content is fluid width by default, unless you provide static sizing. Are you seeing a contrary behavior?

Hi @vojthor I will assume that you've resolved your question and will close this issue


Database should not store the kernel string rather a filepath that can be located

Easier debugging + significantly improves the legibility of the DB.

On thinking a bit more here, it's fine to store the kernel as a single file. However, there should be good tooling to be able to edit certain entries (especially the loopy transforms) of the table.


fix param's repeat in http callback

This PR try to fix #3930 .
Line 59 in trunk/src/protocol/srs_protocol_utility.cpp,  causes the repeat of the param
because in some case, tcUrl has already contained param, but argument param repeat it.
And after this line, it repeats
fullUrl += param.empty() ? "" : (param.at(0) == '?' ? param : "?" + param);

line 67 to 69 below are able to get the param, if append at line 59, it will repeat
fullUrl = fullUrl.substr(0, pos_query) // rtmp://ip/app/app2
                  + fullUrl.substr(pos_rslash) // /stream
                  + fullUrl.substr(pos_query, pos_rslash - pos_query); // ?k=v

TRANS_BY_GPT4

ok, now all checks have passed.
The problem is, when calling this function,
void srs_discovery_tc_url(string tcUrl, string &schema, string &host, string &vhost, string &app, string &stream, int &port, string &param)

if tcUrl contains param, the argrument param should be empty, or it will cause repeat in param
by adding a check in this function, I solved this problem, but I think the proper solution should be calling this function properly. And doing this will lead to a large amount of modifies to current code, some will bring new problem, so I didn't choose to do so.

And the same question also exist with arguement 'stream',
according to the test of this function, tcUrl shouldn't contain stream name, or streamUrl will repeat too! as shown in original issue #3930
but this problem can't be solved in the same way as the repeat of param

The function srs_discovery_tc_url is a utility function, please update the utests to cover your modification.

Thank you for replying!
Next, I would like to confirm with you what you want me to do.
My modification has passed the CI, indicating that it has passed all previous tests and has not introduced any new bugs.
My modification addresses this issue: when tcUrl contains param, passing a non-empty param does not cause param to be duplicated after calling this function.
So, when you ask me to add a test, are you referring to adding a test for the scenario where when tcUrl contains param, passing a non-empty param does not cause param to be duplicated after calling this function?
Additionally, there are tests for this utility function in the three test files trunk\src\utest\srs_utest_rtmp.cpp, trunk\src\utest\srs_utest_protocol2.cpp, and trunk\src\utest\srs_utest_rtmp.cpp. Should I add this test to one of these files, or should I add it to all of them, or should I create a separate test file?
This is my first pull request, and I really hope to get your kindly guidance. Thank you very much.

The function srs_discovery_tc_url is a utility function, please update the utests to cover your modification.

Thank you for replying!
Next, I would like to confirm with you what you want me to do.
My modification has passed the CI, indicating that it has passed all previous tests and has not introduced any new bugs.
My modification addresses this issue: when tcUrl contains param, passing a non-empty param does not cause param to be duplicated after calling this function.
So, when you ask me to add a test, are you referring to adding a test for the scenario where when tcUrl contains param, passing a non-empty param does not cause param to be duplicated after calling this function?
Additionally, there are tests for this utility function in the three test files trunk\src\utest\srs_utest_protocol.cpp, trunk\src\utest\srs_utest_protocol2.cpp, and trunk\src\utest\srs_utest_rtmp.cpp. Should I add this test to one of these files, or should I add it to all of them, or should I create a separate test file?
This is my first pull request, and I really hope to get your kindly guidance. Thank you very much.


Error: Unable to access jarfile target/my-app

`+ ./jenkins/scripts/deliver.sh
The following Maven command installs your Maven-built Java application
into the local Maven repository, which will ultimately be stored in
Jenkinss local Maven repository (and the "maven-repository" Docker data
volume).

mvn jar:jar install:install help:evaluate -Dexpression=project.name
[INFO] Scanning for projects...
[INFO]
[INFO] ----------------------< com.mycompany.app:my-app >----------------------
[INFO] Building my-app 1.0-SNAPSHOT
[INFO] --------------------------------[ jar ]---------------------------------
[INFO]
[INFO] --- maven-jar-plugin:3.3.0:jar (default-cli) @ my-app ---
[INFO]
[INFO] --- maven-install-plugin:2.4:install (default-cli) @ my-app ---
[INFO] Installing /var/jenkins_home/workspace/Github Pipeline/target/my-app-1.0-SNAPSHOT.jar to /var/jenkins_home/.m2/repository/com/mycompany/app/my-app/1.0-SNAPSHOT/my-app-1.0-SNAPSHOT.jar
[INFO] Installing /var/jenkins_home/workspace/Github Pipeline/pom.xml to /var/jenkins_home/.m2/repository/com/mycompany/app/my-app/1.0-SNAPSHOT/my-app-1.0-SNAPSHOT.pom
[INFO]
[INFO] ----------------------< com.mycompany.app:my-app >----------------------
[INFO] Building my-app 1.0-SNAPSHOT
[INFO] --------------------------------[ jar ]---------------------------------
[INFO]
[INFO] --- maven-help-plugin:3.4.0:evaluate (default-cli) @ my-app ---
[INFO] No artifact parameter specified, using 'com.mycompany.app:my-app:jar:1.0-SNAPSHOT' as project.
[INFO]
my-app
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  2.177 s
[INFO] Finished at: 2023-12-31T16:25:52Z
[INFO] ------------------------------------------------------------------------
+ set +x
The following complex command extracts the value of the  element
within  of your Java/Maven projects "pom.xml" file.
++ mvn help:evaluate -Dexpression=project.name
++ grep '^[^[]'
+ NAME='my-app
'
set +x
The following complex command behaves similarly to the previous one but
extracts the value of the  element within  instead.
++ mvn help:evaluate -Dexpression=project.version
++ grep '^[^[]'
+ VERSION='1.0-SNAPSHOT
'
set +x
The following command runs and outputs the execution of your Java
application (which Jenkins built using Maven) to the Jenkins UI.
java -jar target/my-app '-1.0-SNAPSHOT' '.jar'
Error: Unable to access jarfile target/my-app`

This will be resolved once #786 will be reviewed and merged to master.

I am new to Jenkins and doing a Jenkins tutorial and this is still an issue for me, the regex is not getting formed correctly at the last step of delivery, can you help me resolve this? Are there any changes to be done?
The following command extracts the value of the  element
within  of your Java/Maven projects "pom.xml" file.
++ mvn -q -DforceStdout help:evaluate -Dexpression=project.name
+ NAME='my-app'

set +x
The following command behaves similarly to the previous one but
extracts the value of the  element within  instead.
++ mvn -q -DforceStdout help:evaluate -Dexpression=project.version
+ VERSION='1.0-SNAPSHOT'
set +x
The following command runs and outputs the execution of your Java
application (which Jenkins built using Maven) to the Jenkins UI.
java -jar 'target/my-app-1.0-SNAPSHOT.jar'
Error: Unable to access jarfile target/my-app-1.0-SNAPSHOT.jar


The application does not display the “Packages” page properly arranged when the "short description" has different amount of characters.

Description:
When the “short description” of some “packages” is full of characters, the “Rooms & Tariff” site page does not arrange the packages page properly.
-Env:
Device: MacBook Pro
Google Chrome Version 54.0.2840.98 (64-bit)
REPRODUCIBILITY LEVEL:
Always
STEPS:
Steps

Open the Crafter studio with the Hotel_Info site.
On the site bar, locate the pages section.
Click on the edit button of the “Rooms & Tariff“  page.
On the rooms list, add 4 rooms with the next properties.
Package A: Fill all the fields and the “short Description” with the maximum characters limit (300).
Package B: Fill all the fields and the “short Description” with 100 characters.
Package C: Fill all the fields and the “short Description” with 20 characters.
Package D: Fill all the fields and the “short Description” with 150 characters.
Package E: Fill all the fields and the “short Description” with 150 characters.
Click on the Save and Close Button.
Go to the page preview of the “Packages” page.

CURRENT RESULT:
The application displays all the configured “Packages” but they are not arranged properly.
https://www.screencast.com/t/49lYtYRIWQn
EXPECTED RESULT:
The application should display all the configured packages properly arranged.

Fixed

Fixed

Bug verified and closed as fixed.


HTML item* attributes

Hi dude! I found that some of attributes are not correctly mapped to react props. For example, itemscope, itemtype, itemprop.

I think this issue should go to https://github.com/noraesae/react-attr-converter.

Thanks for your report though!


batch norm updates

Hi, any hint about how does batch norm update and why you do the following?
bn_updates = [(p, m * 0.05 + p * 0.95) for p, m in pop_updates]

Just trying to mimic the same behavior in Pytorch but not sure what are p and m and when this update happens during the execution.

This line estimate the population statistics during training using a moving average. p and m stand for the parameters before and after the update respectively.


Ignore egress requests updated more than 90 days ago

Description
Automatically omit egress requests updated more than 90 days ago from the API response to limit the size of the API body (AppSync has a 1MB limit).
Ideally the number of days should be configurable, but given the urgency of this change that can be added later.
This includes PR #11

Declaration : By submitting this pull request, I confirm that my
contribution is made under the terms of the Apache-2.0 license

I'll hold off tagging this until the number of days is configurable.


feat: warn on external goto navigation

in preparation for SvelteKit 2
related to #8775 / #11207
Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[x] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[x] This message body should clearly illustrate what problems it solves.
[ ] Ideally, include a test that fails without this PR but passes with it.

Tests

[x] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[x] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpm changeset and following the prompts. Changesets that add features should be minor and those that fix bugs should be patch. Please prefix changeset messages with feat:, fix:, or chore:.

Mhm ... I just did this PR, but now I'm wondering how much use the warning really is - you'll probably never see it because you navigate away from the page. Probably doesn't hurt though, and having the new option in SvelteKit 1 already is good.


Installation mode on Windows

Hey I've made a quick game that I'm trying to show at a big event and would like to not have to rely on bad wifi but am having a lot of trouble trying to piece together how to setup installation mode. I think I have my router correctly configured. Is it even possible on windows?

Installation mode should work fine on windows
What problem are you seeing specifically? What steps are you taking and what results do you see?
It's possible your router doesn't support installation mode (the netgear router I tried didn't support it)
Things to check:
You setup installation mode on your router, open a command prompt and type ipconfig

the IPv4 address should match the one you specified in your router configuration when assigning a static IP address for your computer
Then if you run the game, and then alt-tab back to the command prompt and type nslookup foo.com

The 1st and 2nd with the 3rd maybe being your Default Gateway from above numbers should match your IPv4 address from ipconfig
If those don't match then your router is not setup correctly or else you've maybe you manually put DNS addresses in your computer's network configuration and need to remove them.
If that all checks out then the question is are you testing Android or iPhone
It looks like Apple might have broken installation mode with the latest version of iOS. The simplest workaround at the moment is to edit Assets/WebPlayerTemplates/HappyFunTimes/hft/captive-portal/game-login.html and change it to this

<!DOCTYPE html>

  <meta charset="utf-8">
  <meta name="apple-mobile-web-app-capable" content="yes">
  <meta name="apple-mobile-web-app-status-bar-style" content="black">

  <meta name="HandheldFriendly" content="True">
  <meta name="MobileOptimized" content="320">
  <meta name="viewport" content="width=device-width, initial-scale=1.0, maximum-scale=1, user-scalable=no, minimal-ui">
  <meta name="format-detection" content="telephone=no" />
  <title>HappyFunTimes Redirect</title>
  <link rel="stylesheet" href="/hft/captive-portal/login.css">

HappyFunTimes

Please open your browser and go to hft.com

The only changes are the second
 line and commenting out the Start line

when i nslookup foo.com it prints
Server: UnKnown
Address: <IP_ADDRESS> (the correct IPv4 address for the router i set)
***UnKnown can't find foo.com: no response from server

this is what i setup on my router. is there something in unity i should be setting to look for the router?

You were running the game when you used nslookup?

Let me clarify that, you were running the game in installation mode when you typed nslookup?

i suppose not. i'm a little unclear on what i need to do for that? i was running that npm run start -- --dns bit  and i'm not sure what to have setup, if anything, in unity.

switched some things around, nslookup is now showing the right addresses but unity throws socketexception

Ugh! Sorry about that. I could have sworn I wrote docs but I can't find them anywhere 😓
There's no node or npm to run for the unity version.
You just pick Window->HappyFuntimes->Settings and check "Installation Mode" if you're in the editor. If you build your game to an exe then you have to run it from the command line with the argument --hft-installation-mode
Just remember to turn off installation mode when you're not with a configured router (as in your running things like normal, going through happyfuntimes.net). Because, when installation mode is used the game will not ping happyfuntimes.net because it assumes you're not on the internet.
I updated the docs a little

awesome i'm glad it was something easy! got it fixed just in time, here's my game i made using it running a jumbotron in Philadelphia https://www.instagram.com/p/BTsNTjuBEoe/?taken-by=space_bit


access entity components via keys that are a name field rather than class name

Great work on this library! This is minor, but...
I have run into a use case where it really makes sense to have a single entity component class, but many instances.
As I understand, currently, the components are accessed via a dictionary whose keys are the names of the component class:
https://github.com/google-deepmind/concordia/blob/01e4ad40e4721f35b884f1235f2b79b041a725e6/concordia/factory/agent/basic_agent.py#L196C3-L197C61
This restricts their use to only one instance, and so the only way to have many instances is to duplicate the same class, which is not really a good solution.
FIX: The obvious fix would be to add a name field to all component class definitions and use that as the key, rather than the class name.

Using the class name as the key is just a convention at the level of the agent factory. None of the code outside the factory file makes the assumption that the keys will be the class names. So you can name them however you like.

Ok! Good to know. Thanks.


Update dockerfile

I have created an updated version of dockerfile that include click and jupyter-book. at pull request #30

I think i needa add myst_nb import glue in the docker file later.

ok done, but please let me know if we need extra


Feat: Contact Us Page is Missing

Describe the feature
@Anushkabh
check this our, we can design /contact route for contact with us
Add ScreenShots

we can redesign somethig like this

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@dhruv8433 it has been fixed Please find new bug or feature


bug: Use process(css).then(cb) to work with async plugins

Provide environment information

  System:
    OS: macOS 13.3.1
    CPU: (10) x64 Apple M1 Pro
    Memory: 720.11 MB / 16.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 18.16.1 - ~/.nvm/versions/node/v18.16.1/bin/node
    npm: 9.5.1 - ~/.nvm/versions/node/v18.16.1/bin/npm

Describe the bug
When the steps are followed in the quick start, all goes well until running pnpm dev. At this point there is an error which reads:
@acme/expo:dev: iOS Bundling failed 2805ms
@acme/expo:dev: error: src/app/post/[id].tsx: /Users/charlieward/Desktop/smart-chef-cli/apps/expo/src/app/post/[id].tsx: Use process(css).then(cb) to work with async plugins

Link to reproduction
Screenshot attached
To reproduce
Follow each steps of the quick start with node version 18.16.1.
Additional information

use<EMAIL_ADDRESS>- will get this fixed. there are 2 open PRs for expo 49 and will get this fixed at the same tiem

I downgraded to<EMAIL_ADDRESS>and I'm still getting this.

I only get it on android tho, iOS builds work fine.

I'm still getting the same error when I switch to tailwind 3.3.2. I updated the package.json files of the app, deleted my pnpm.lock file, ran pnpm i again, but still get the same error Use process(css).then(cb) to work with async plugins
here's my package.json at the root:
{
  "name": "create-t3-turbo",
  "private": true,
  "engines": {
    "node": ">=v18.16.1"
  },
  "packageManager"<EMAIL_ADDRESS>  "scripts": {
    "build": "turbo build",
    "clean": "git clean -xdf node_modules",
    "clean:workspaces": "turbo clean",
    "db:generate": "turbo db:generate",
    "db:push": "turbo db:push db:generate",
    "db:studio": "pnpm -F db dev",
    "dev": "turbo dev --parallel",
    "format": "prettier --write \"**/*.{js,cjs,mjs,ts,tsx,md,json}\" --ignore-path .gitignore",
    "lint": "turbo lint && manypkg check",
    "lint:fix": "turbo lint:fix && manypkg fix",
    "type-check": "turbo type-check"
  },
  "dependencies": {
    "@ianvs/prettier-plugin-sort-imports": "^4.1.0",
    "@manypkg/cli": "^0.21.0",
    "prettier": "^3.0.0",
    "prettier-plugin-tailwindcss": "^0.4.1",
    "turbo": "^1.10.7",
    "typescript": "^5.1.6"
  }
}

Then in the package json in app/expo and app/nextjs I changed the tailwind dev depdendency to be: "tailwindcss": "^3.3.2"

Then in the package json in app/expo and app/nextjs I changed the tailwind dev depdendency to be: "tailwindcss": "^3.3.2"

That will still pull 3.3.3. Set the version to be exact:
"tailwindcss": "3.3.2"

Then in the package json in app/expo and app/nextjs I changed the tailwind dev depdendency to be: "tailwindcss": "^3.3.2"

That will still pull 3.3.3. Set the version to be exact:
"tailwindcss": "3.3.2"

I do realise you were not talking to me, but I did make sure in the lockfiles that version 3.3.2 was installed in all instances of tailwindcss + it only seems to happen on android builds for me.
Do you have any idea what might be causing it? By the looks of the error it's still related to postcss so it's still a tailwind issue, but I'm not sure what else to try

Do you have any idea what might be causing it? By the looks of the error it's still related to postcss so it's still a tailwind issue, but I'm not sure what else to try

Unfortunately no, I suggest posting in this issue and see if anyone knows what's up with android: https://github.com/postcss/postcss/issues/412

I use<EMAIL_ADDRESS>and it fixed the problem.

I use<EMAIL_ADDRESS>and it fixed the problem.

This solution also works for me. I also came across the solution on this post here.

Closing as completed. If you still encounter this I suggest following the postcss issue

i was facing same issue last two days but this work for me :
yarn add<EMAIL_ADDRESS>yarn add --dev<EMAIL_ADDRESS>

in package.json file, change "tailwindcss": "^3.3.2", to "tailwindcss": "~3.3.2",

i was facing same issue last two days but this work for me : yarn add<EMAIL_ADDRESS>yarn add --dev<EMAIL_ADDRESS>
Thank you, it's work to me


Verbose details not displayed

Spigot 1.12.2 latest
LuckPerms 4.2.36
I just noticed that verbose mode details are not displayed in the console/log. All I see is
VB > playerName
The details were displayed until recently, certainly end of May). I don't see a relevant configuration change since 4.2.2. Did I miss something?

Are you using a locale file?

Yes, I am. But I edited only the prefix.
# diff plugins/LuckPerms/lang.yml plugins/LuckPerms/releases/4.2.36/locale/en_US.yml
5c5
< prefix: "[LP] "
---
> prefix: "&7&l[&b&lL&3&lP&7&l] "

Well, that worked. But hardly the expected result. As an test, I tried
# LuckPerms Language File
# Locale: en_US (English)
# Author: Luck

prefix: "[LP] "
verbose-log: "VB > {0}"

(copied from the locale file, and again removing only the formatting codes). It displayed only
VB > playerName
In order to get the equivalent of the default verbose-log display (minus some of the formatting), I had to supply
verbose-log: "VB > {0} - {1} -{2} {3}"

Yes, the locale file is out-of-date at the moment.
The correct replacement is here:
https://github.com/lucko/LuckPerms/blob/6f96d9c7d161502d8abe09607b90ba4d1890ba62/common/src/main/java/me/lucko/luckperms/common/locale/message/Message.java#L70

Thanks.

I was leaving this open until the localization files were updated. Do you want me to open a separate ticket?

The locale files are generally updated once builds are pushed to Spigot/Ore as "stable". These changes aren't backwards compatible.

Thanks.


package-lock.json is old

Describe the issue

The version field of the package.json is 0.9.8, whereas as of package-lock.json is 0.9.2.
Found 23 vulnerabilities.

To Reproduce

clone the repo
npm install

added 1454 packages, and audited 1455 packages in 38s

280 packages are looking for funding
  run `npm fund` for details

23 vulnerabilities (3 low, 8 moderate, 12 high)

To address all issues, run:
  npm audit fix

Run `npm audit` for details.

git diff

--- a/package-lock.json
+++ b/package-lock.json
@@ -1,12 +1,12 @@
 {
   "name": "@goto-lab/pufu-editor",
-  "version": "0.9.2",
+  "version": "0.9.8",
...

Expected behavior

found 0 vulnerabilities
no version diff

Screenshots

Desktop (please complete the following information):

Windows 11 (WSLg)

Smartphone (please complete the following information):
N/A
Additional context
npm audit fix solved the problems, but breaks npm run storybook.

The npm run storybook broken problem was resolved by updating below package
npm install -D @storybook/react-vite

We have also updated the following:
npm install -D @storybook/test
npm install -D @storybook/react
npm install -D @storybook/addon-interactions
npm install -D tailwindcss

pufu-editor % npm audit
found 0 vulnerabilities

The above corrections are reflected in the PR https://github.com/goto-lab/pufu-editor/pull/27

Thank you for raising the issue.
Close this issue by RP #27 .


fix(card-group): card grid alignment

Related Ticket(s)
https://github.com/carbon-design-system/carbon-for-ibm-dotcom/issues/5730
Description
@shixiedesign I can fix the alignment perfectly when the card only includes text:

The problem happens when the card includes an image. From my tests, it's mathematically impossible to make it work unless we add a javascript fix that makes all images inside the card same height. To make the last card's content align with the grid, it needs to be the same width as the top card, which takes 33.333...% of the width. If you substract that width plus the two 16px gutters from 100%, we don't have enough space left to make the other two cards be the same row the same width as the last card, which causes their images to be shorter than the last card.

I can fix the image height problem by making them all the same width, but the last card wouldn't hang like the top card and image:

Not sure what to do?

Talked to Julio on Slack, the Card in card (image included) and all right-most card should not go beyond the pink line. So narrow grid mean 16px gutter and nothing hang-right

@shixiedesign fixed it! Now everything is aligned to grid, with no hang-right 👍


can't reload ExtraLib for LanguageService

delete this._extraLibs[filePath] return false
and not delete this._extraLibs[filePath]
see here https://github.com/Microsoft/monaco-typescript/issues/5

is a reason
get extraLibs(): { [path: string]: string; } { return Object.freeze(this._extraLibs); }

see here Microsoft/monaco-typescript#5


Update README.md

added a link to Contributing Guide Lines #33

This change has already been merged by another user


Investigate BDD patterns and Testing frameworks to replace the default testing Go lib.

As for know no BDD framework will be implemented. There is no integration directly with the GErkin/Cucumber feature files.


Changed Subscriptions title (test)

This is good, thanks for raising this test PR @louiseph . Closing this for now


Support the async put API provided by HTableMultiplexer to improve th…

…e write throughput in collector. #1683

If asyncOperation is not set, then execute put method instead of asyncPut method.
wait destory method until async queue is empty.

@emeroad review please :)

reviewed-by emeroad


Update docs for version 2.4.0

Please review and merge the pull request to update the docs for v2.4.0 release.

@rishikeshpalve Thanks!

Hello @rishikeshpalve
I noticed that you changed "TIBCO Business Studio for BusinessWorks development environment" to
"TIBCO Business Studio Container Edition development environment" in the PR but I see other examples of "TIBCO Business Studio Container Edition development environment" in the doc.
You might want to review all the TIBCO product names in this doc, just to protect your branding.

Thanks @jncd for pointing it out. @kgera-tibco is fixing it in the new PR we have created for version 2.4.3 release.

Thanks @rishikeshpalve  and @kgera-tibco