Support matching URI-encoded requests against the precache manifest

Library Affected:
workbox-webpack-plugin
Browser & Platform:
all browsers
Issue or Feature Request Description:
the generated service worker doesn't not encode file names of the cached files.
For instance, the plugin will generate the following code.
e.precacheAndRoute([
    ...
    {url:"/ui/immutable/npm.@material.23acbb8b21aa4990d57d.bundle.js",revision:null},
    ...
]

However, when webpack-html-plugin will generate the following html.
...
<script defer="defer" src="/ui/immutable/npm.%40material.23acbb8b21aa4990d57d.bundle.js"></script>
...

Note that the @ is encoded to %40.
As a result, the file is not cached by the service worker and the page has to hit the web server to retrieve that file. This can be problematic when a new version is released and the original file no longer exist on the server. The old HTML file may still be served by the service worker, and the page may run into 404 when trying to get the dependencies.
I believe encoding the URL is the correct implementation. So we should fix it on the workbox side.

Hmm, I'm not really sure that encoding the @ in URLs automatically is the right thing for Workbox to do. My understanding is that URLs with unencoded @ in them are widely supported by browsers, and introducing a breaking change is something we'd like to avoid. (I also think that there would be issues around, e.g., whether to encode a   as %20 or +, since both are valid.)
What you could do is use the [urlManipulation option](https://developers.google.com/web/tools/workbox/modules/workbox-precaching#custom_manipulations0 that can be passed in to precacheAndRoute() to tell workbox-precaching which URLs it should consider a precaching match. That function could decode any entities in the incoming URL and return that as a potential match.
Using this does require that you switch to the InjectManifest mode of the workbox-webpack-plugin instead of GenerateSW, since you need more control over the source of your service worker.
// Inside your swSrc file:
import {precacheAndRoute} from 'workbox-precaching';

precacheAndRoute(self.__WB_MANIFEST, {
  urlManipulation: ({url}) => {
    const decodedURL = decodeURIComponent(url);
    return [decodedURL];
  }
});

If that doesn't work for you, let us know and we can revist.

I fixed it with npm-patch. So strictly speaking, this is not blocking me.
I think your concern about backward compatibility makes sense. However, given @ is a reserved character in
https://datatracker.ietf.org/doc/html/rfc3986#section-2.2
Is it possible that we at least add some documentation or show a warning when @ is in the precached filename?
A lot of npm packages are prefixed with @ and webpack-html-plugin has 6.6m download, I reckon many users will run into this issue. They probably just didn't realise it because the page is only broken when a dependency with @ is updated.

Thinking more about it, it's probably "safe" (from a semver perspective) if we added something like that custom urlManipulation function that called decodeURIComponent() to run as part of the default logic in workbox-precaching that tries to find equivalent cache entries for an incoming URL:
https://github.com/GoogleChrome/workbox/blob/f6fb94a9d8b922fdcb758129d94e6d2a959f2167/packages/workbox-precaching/src/utils/generateURLVariations.ts#L24-L29
So maybe we can just do that (assuming the logic shared above works; we'd add in a test case to confirm) and we wouldn't need to warn folks to begin with—it would just start working as expected.

I think you are right.
Upon further investigation. I think this might be an issue with html-webpack-plugin rather than with workbox.
See https://github.com/jantimon/html-webpack-plugin/issues/1771

Hi there,
Workbox is moving to a new engineering team within Google. As part of this move, we're declaring a partial bug bankruptcy to allow the new team to start fresh. We realize this isn't optimal, but realistically, this is the only way we see it working. For transparency, here're the criteria we applied:

For PRs, we closed everything that doesn't target the current v7 branch.
For Issues, we closed everything that was created before 2024.

Thanks, and we hope for your understanding!
The Workbox team


Build bionic base image in Jenkinsfile.gitian

We updated gitian builds to use bionic, so we should also update the Jenkinsfile

Note: should also update docs probably.


Typo

Typo

Hi @coezbek, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

Thanks for the fix!


Feature/liayoo/tta

DONE

Added blockchainConfig to node status
Added /blockchain_config client API
Added validators, globalTimeSyncStatus to consensus status

TODOs

Add numberToBlockInfo mapping in BlockPool & /get_block_info_by_number client API
Extend tools/transfer/sendTransferTx.js
Update deploy_on_prem.sh (on bitbucket)

Addressed all comments, PTAL @platfowner


Added the ability to import a certificate from file without a private key.

There is a need to import certificates without a private key, but there was no way to do this by importing from a file, only by importing from a base64 string, which is not convenient.
Added a check for privateKeyPath, if it is not set, then a certificate without a private key is imported.

thanks. I'll include this in the latest release.


how to get class/struct by name?

how to get class/struct by name string?
struct Person{}
let info = typeInfo(of:"Person")

Unfortunately this is not possible. If I recall correctly it may be possible to get the type/type info from the mangled name, but not just the name itself.

@wickwirew Does the library support getting a type, and then type info by mangled name?

@edudnyk unfortunately it does not


drivers/console/xtensa_sim_console: force \r\n byte sequence for newline

Force \r\n byte sequence for newline for the Xtensa simulator console driver. This effectively mirrors the behavior of the UART console driver.

Hey @dcpleung, could you perhaps take a look at this PR? I'm asking because you're the maintainer of the Xtensa arch in Zephyr, and it looks like this specific file is not covered by any of the defined areas of maintenance.

Do you need this to run on Windows? Any side effects if running on Linux?

Hi, thanks a lot for your reply :)

Do you need this to run on Windows?

No.

Any side effects if running on Linux?

No.
I'm recording simulation logs, and because of that, I'd like to get the console drivers to exhibit the same behavior across all platforms. The Xtensa simulation console driver seems to be the odd one here. Running the same app, it was outputting only \n whereas the UART console driver always ended lines with the \r\n sequence.
You can see the relevant UART console driver code here: https://github.com/zephyrproject-rtos/zephyr/blob/main/drivers/console/uart_console.c#L97-L100. This was first introduced in 2016 (commit), so it looks like it's a rather well-established behavior at this point.


Dm/#144569/update readme branch

Update readme and license files to accurately reflect Pedigree Technologies usage of this repository.

https://pedigreetechnologies.tpondemand.com/entity/144569


website: establish a no-me-too policy

@adg and I realized our "no-me-too" policy has only ever been "documented" in the golang-dev mailing list. Let's make it more official.

I wrote this: https://github.com/golang/go/wiki/NoMeToo
Thoughts? @ianlancetaylor @rsc ?

@adg, should we link that from somewhere else? Although most people arriving at a github issue and me-tooing it will probably be arriving there from a direct link or search, so linking it from e.g. https://golang.org/doc/contribute.html might not be too effective. Still, can't hurt.
We can also just let people of the policy by cc'ing them from this bug if/when we delete comments.

We can also just let people of the policy by cc'ing them from this bug if/when we delete comments.

I think we should do this, at least. But I don't see much point in documenting it elsewhere, because people who do this aren't going to read our contribution guidelines.

I wrote this: https://github.com/golang/go/wiki/NoMeToo

Thoughts?

I'd like advice on this text with regard to tone. I don't want it to come across as passive aggressive.

The text you wrote on the wiki LGTM.

@adg That wiki page currently says:

we cannot sort issues by "reactions" (yet)

That is no longer true, it's now possible to sort issues by most reactions. E.g.:
https://github.com/golang/go/issues?q=is%3Aopen+is%3Aissue+sort%3Areactions-%2B1-desc

@shurcooL oh, that's great! I've replaced the paragraph you quoted with this text:

Instead of commenting, you may add a :+1: reaction to the comment or issue.

@runningmaster, per the NoMeToo policy, we've deleted your +1 comments at https://github.com/golang/go/issues/14861

@timoteoponce, per the NoMeToo policy, we've deleted your "+1 for this!!" comment on #10010

@marcopeereboom, please note the https://golang.org/wiki/NoMeToo policy (per your comment on #15405)


[mbx,keymgr_dpe] Disable unimplemented stress tests

This PR disables the unimplemented stress tests from their respective IP simulation configuration. This removes them from the full configuration that is inferred when using dvsim with all, all_once, or nightly as the test input list.

I noticed some tests are unmapped for Mailbox, which I'm proposing to fix in https://github.com/lowRISC/opentitan/pull/21333


'division_safe' fix applied

This fix used instead of libfuturize.fixes.fix_division.
Division
Integer division (rounding down):
# Python 2 only:
assert 2 / 3 == 0

# Python 2 and 3:
assert 2 // 3 == 0

“True division” (float division):
# Python 3 only:
assert 3 / 2 == 1.5

# Python 2 and 3:
from __future__ import division    # (at top of module)

assert 3 / 2 == 1.5

“Old division” (i.e. compatible with Py2 behaviour):
# Python 2 only:
a = b / c            # with any types

# Python 2 and 3:
from past.utils import old_div

a = old_div(b, c)    # always same as / on Py2

This one, I think, requires some thought and we should make an attempt to discern which of these things are expecting ints and which are expecting floats as a result.
This will take some effort, but will leave us with better, less ugly code.

@ticoann  As we discussed on meeting this PR could be closed.
I making changes "by hand", results can be viewed here: https://github.com/dmwm/WMCore/pull/8125


Delete Mozilla winter of security

If you are adding a SoC/WoC, please make sure

[ ] You have added the SoC/WoC in README.md file in correct table
[ ] You have added the SoC/WoC in index.html in correct table
[ ] ensure Website is tested on a deployment and looks okay

If you are fixing an issue please add

[ ] fixes issue #issue_number to PR description
fix issue #154

Ok, that means #154 should be closed then


Error handling

Using an incorrect key doesn't throw any exceptions. I just get stuck waiting forever for the repository to be ready.
key = GetSecretValue(f"featurehub.{app}.{env}") #this contains an old key. I've recreated the app and forgot to change the secret value.
config = FeatureHubConfig(edge_url, [key])
asyncio.run(config.init())

while not config.repository().is_ready():
    time.sleep(0.1)

I'd expect config.init() to throw an exception or something, or at least have a "status" method so I can see something is wrong.

Hey @ilyatovbin-pp - the connection is asynchronous so it cannot throw exceptions. The underlying streaming thread has a
cancelled" field when a 404 is returned from the server, currently only "stopped" is exposed, but we could expose cancelled as well, so you could check if  the edge connection was cancelled?


pr: run tests

@shashanksinha89 hmm any idea why some tests are failing/canceled?

@shashanksinha89 hmm any idea why some tests are failing/canceled?

installed missing dependencies.

@shashanksinha89 can you resolve the merge conflicts please? Thanks in advance!

@shashanksinha89 can you resolve the merge conflicts please? Thanks in advance!

fixed.


Develop a way to run "manage.py cleartokens" periodically

https://django-oauth-toolkit.readthedocs.io/en/latest/management_commands.html

Does this need to run on production specifically? If so, this could be implemented as a workflow that uses something like https://github.com/marketplace/actions/heroku-run to log into the dyno and run the command as a cron job

That action itself is not complicated if we want to remake something ourselves (not keen on depending on an Action that isn't well maintained or adopted)


[New game]: Super Mario Run

🎮 Game Request
Super Mario Run is a game that brings the classic side-scrolling platformer action of the Mario franchise to your fingertips. With simple touch controls, players guide Mario through levels filled with obstacles, enemies, and coins. The game combines precision jumping, power-ups, and engaging level design to offer a fun and challenging experience.
Point down the features

Endless Runner Gameplay: The game can feature an endless runner style where Mario continuously runs forward, and players must control his jumps and movements to navigate through levels and avoid obstacles.
Level Editor: Create and customize your own Mario levels, placing platforms, enemies, and obstacles to design unique challenges.
Score Leaderboard: Compete and strive to achieve the highest score in each level, aiming for the top spot on the leaderboard.
Saved Levels Library: Access and play previously saved levels.
Sound Effects: Immerse yourself in the game with lively sound effects that enhance the gameplay experience, making it more engaging and enjoyable.

Select program in which you are contributing
GSSoC23
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @Vedansh-777 , You can't have another issue before completing the previous one 😀
you already have the following 1 open issues 👀 ! :
#1697

Hey @Vedansh-777 ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇

Hey @Vedansh-777 !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

/assign


Added default customDomainHttpsParameters value

As issue #11567 says, there is an issue with enabling HTTPS for custom domains in the CDN. This is caused by the API requiring a customDomainHttpsParameters parameter. However, in the current version, this property is optional, causing problems.
This PR adds a hopefully reasonable default value if the property has not been set. This should make it usable if left out, and make it backwards compatible to v4.2 where it worked without providing it.

@ramya-rao-a Is there a way to add this default configuration in code generator ? maybe change the swagger ? Otherwise it may get override when next time release it ?

@qiaozha I would expect any default values to be configured in the swagger file
@joheredi Can you confirm?

@qiaozha I would expect any default values to be configured in the swagger file
@joheredi Can you confirm?

I just confirmed with Service Team, the certicateType default value is actually related with the profile SKU. We can't approve this PR.
It depends on the profile SKU - Standard_Microsoft uses dedicated, Standard_Akamai, Standard_Verizon, and Premium_Verizon use shared. Also the protocol type depends on the SKU, you can see in the example I sent earlier: https://github.com/Azure/azure-cli/blob/31584aaca4d5f02fe795f1d75172e38965ad67ab/src/azure-cli/azure/cli/command_modules/cdn/custom.py#L558-L585
@ramya-rao-a @joheredi Then in this case, Is there a way to put this kind of logic in the swagger ?

Ok, that makes sense, and I could update the PR with logic similar to the Python example you provided. However, it doesn't feel very valuable if there is no way to get it into the code.
I would say that the Swagger would be able to define the properties as mandatory, which would make it a breaking change, but it wouldn't be able to add custom logic. On the other hand, I assume that there has to be some way to get the code in there during generation as well.
Do you want me to update the PR with the SKU code, or should I just abandon it? Unfortunately it causes problems for people, so I think solving it would be the best solution...

Ok, that makes sense, and I could update the PR with logic similar to the Python example you provided. However, it doesn't feel very valuable if there is no way to get it into the code.
I would say that the Swagger would be able to define the properties as mandatory, which would make it a breaking change, but it wouldn't be able to add custom logic. On the other hand, I assume that there has to be some way to get the code in there during generation as well.
Do you want me to update the PR with the SKU code, or should I just abandon it? Unfortunately it causes problems for people, so I think solving it would be the best solution...

It seems unlikely for user to add that complex default value logic in the swagger. @lsmith130 Is it possible to add the default value logic in the service backend side? If yes, will it take a long time to get it deployed to prod?
@ChrisKlug I would suggest to change the default logic in the JS SDK to unblock customer first.
@ramya-rao-a @joheredi How do you think of it?

Well, for the sake of things, I added the code for creating different values per SKU. If nothing else, it might help someone implement it properly in the future...

Adding this default logic in the swagger doesn't seem possible, and the logic seems very specific to this service to be added in the generator. If I'm understanding this correctly, it seems like this is a bug either in the Swagger where the parameter is marked as optional, or in the service where it is returning an error for an optional parameter not being set.
If it is a Swagger error, I think the right thing to do would be to update it to have this property  as required. Unfortunately it will be a breaking change
If it is a Service bug it should be fixed in the service and since no Swagger fix would be needed, there won't be breaking changes
@qiaozha who can help us confirm if this is a service bug?

@lsmith130 is from the service team. Could you please confirm that? Thankx

I agree with you that either the swagger has problem or the service side need to add that default logic to handle null. Just think both of the two processes might take a couple of days. To temporarily fix customer issue, maybe approve this PR is the best? @joheredi

I did some digging in the service code, it looks like this was intended to be optional, but our request validation is setup in a way that makes the arguments required with no easy workaround. Fixing this in the service would require significant rearchitecture of this feature, unfortunately. If possible, please handle this on the client side.

@ramya-rao-a @joheredi Then in this case. We should approve this manual fix right?
@ChrisKlug  Could you bump a minor version in sdk/cdn/arm-cdn/package.json and sdk/cdn/arm-cdn/src/cdnManagementClientContext.ts for this package

Minor version has been bumped to 5.1

@ramya-rao-a @joheredi Could you please let me know if this works based on the comments above? If there's no objections from you, I will merge this PR.

Cool! I have added region comments to mark the code I have added. Hopefully this won't take up anymore of your time! Cheers!


remove dead code in authn plugin.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: yangminzhu
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:

pilot/pkg/networking/plugin/authn/OWNERS [yangminzhu]

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

/lgtm

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: quanjielin, yangminzhu
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:

pilot/pkg/networking/plugin/authn/OWNERS [yangminzhu]

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

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: diemtvu, quanjielin, yangminzhu
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:

pilot/pkg/networking/plugin/authn/OWNERS [diemtvu,yangminzhu]

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

New changes are detected. LGTM label has been removed.

@yangminzhu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests-v1alpha3.sh
90e398e5ffd6f44d9e1d6dfeeed08213bc8cbd9b
link
/test e2e-bookInfo-envoyv2-v1alpha3

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. I understand the commands that are listed here.

/test e2e-bookInfo-envoyv2-v1alpha3

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: diemtvu, quanjielin, yangminzhu
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: myidpt
If they are not already assigned, you can assign the PR to them by writing /assign @myidpt 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:

OWNERS

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

/lgtm

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: diemtvu, quanjielin, sebastienvas, yangminzhu
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: myidpt
If they are not already assigned, you can assign the PR to them by writing /assign @myidpt 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:

OWNERS

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

@wattli Could you help approve this? Diem and Sebastien have approved but we need someone from the istio/OWNERS. Thank you!


resources.Get errors when given an empty string

What version of Hugo are you using (hugo version)?

$ hugo version
hugo v0.101.0-466fa43c16709b4483689930a4f9ac8add5c9f66+extended windows/amd64 BuildDate=2022-06-16T07:09:16Z VendorInfo=gohugoio

Does this issue reproduce with the latest release?
Yes

Observed Behavior
When resources.Get "" is run, ie. resources.Get is given an empty string, it errors out and fails the build. .Resources.Get does not have the same issue.
execute of template failed at <resources.Get>: error calling Get: dirs not supported resource types: &{0xc002d15350 0xc002bc8c80}

Expected Behavior
When an empty string is passed to resources.Get, it will return a false, as .Resources.Get does, and continue processing the build.

I'd like to work on this bug, have been using Hugo for a few years and would love to contribute!

The issue reported here has apparently been fixed: https://github.com/gohugoio/hugo/pull/10444
I can still trigger the error by passing one or more spaces.

Closed with https://github.com/gohugoio/hugo/pull/10444


Anime names should be title case

Select your issue type: (check at least one)

[ ] Bug
[ ] Question
[x] Suggestion
[ ] Other (please describe):

Describe your issue:
Anime name is the title param of the grabbed links and in metadata.json should be in title case (ex: Boruto) and not upper/lower case (ex: BORUTO, boruto).

Fixed in d14e285a44277d5f2a9e14c40e8af31566513add


HHH-13645 : StatsNamedContainer#getOrCompute throws NullPointerException when computed value is null

https://hibernate.atlassian.net/browse/HHH-13645

Thanks Gail! Added a comment so to remember the implementation choices and merged.


strange behavior of the reconnectStrategy callback

Hi
I've noticed that when using the reconnectStrategy feature , there is always a first call to the given callback with a 0 passed as the first callback parameter
If I understand the documentation right, this parameter stand for the number of retrial, so always been called with a 0 value at startup is a bit puzzling, if not awkward, especially because we log.warn the retrial count in this callback 😋
Is it possible to avoid this initial call ?
If not, is it relevant to condition the log.warn with a retrialNbr > 0 ?
Thanks in advance for your feedback about it
Environment:

Node.js Version: 14.20.0
Redis Server Version: 7.0.5
Node Redis Version: 4.1.0
Platform: Mac OS Monterey 12.6

Sorry, made a wrong analysis of our logs. Don't care about this issue


repeat posts

For some reason  mattermost is being updated through RSS with duplicate articles that have already been posted.  I'm pretty sure I don't have two instances of the same RSS feed running.  However I do have several instances running which all post to different mattermost channels.
Thanks

maybe the content creator pushed an update to thr original article?


feat: objet 3D boxes visualization support

Hi Zirui,
A proposition to support object visualization with the current axis convention.
thanks
Pierre

Thanks a lot!


Readme updates

No changes to the API, only README updates.

Minor README reconfiguration
Fixed architecture diagram
Uploaded sample non-technical diagram

Please review the new diagram and let me know if it needs any changes!

Adding a non-technical diagram will help our stakeholders better understand their product. I like the look of this diagram, and look forward to updating the documentation with the features we intend to build out imminently.

Great work, makes it clearer, of course that is my opinion and I have background knowledge of it. Hopefully it makes sense to Avista


Contribution

Hey there, love the idea I would like to help contribute, what sort of stuff is in the roadmap so I can take a look at it and see how I could help out.
Regards
Andy

Thanks for all the contribs and feedback on v0.x!


Delete tests/test.ts

Hi @BunnyBytes39, thank you for your PR!
The changes should be done from and to the develop branch.
Also, why would you want to remove the tests/test.ts file? Please provide explanation.


[Bug]: Missing credentials error when use azureADTokenProvider to call Azure OpenAI

Language
Javascript/Typescript
Version
latest
Description
I got follow error when using azureADTokenProvider
OpenAIError: Missing credentials. Please pass one of apiKey and azureADTokenProvider, or set the AZURE_OPENAI_API_KEY environment variable.
Reproduction Steps

Use azureADTokenProvider as follow

Run the app

Found the root cause, this line sets the wrong parameter:
https://github.com/microsoft/teams-ai/blob/main/js/packages/teams-ai/src/models/OpenAIModel.ts#L234

@blackchoey could you elaborate? Are you saying endpoint is incorrect?

@corinagum Let me create a PR to fix it:) Will try to submit a PR within this week.


Insert project description into metadata (FF)

followup of #18

Functionality:
Our extension produces noice, while you are normally surfing the web. In the background different websites will be opened. These websites are random but connected to a specified stereotype.
Key Features:

Browse the web while your user profile is obscured by fake activity!
Random traffic is generated to obscure your real user profile!
You pick what your fake profile should look like!
Illegal traffic is prevented by using up-to-date blacklists!
If you have no data flat: define the limit of traffic generated!

same here - where has this been applied to the manifest - or is this just a new text

Link: Website
https://mystique-extension.jimdo.com/

	"description": "Our extension produces noice, while you are normally surfing the web. In the background different websites will be opened. These websites are random but connected to a specified stereotype.",

	"author": "DHBW.CAS.ADM.2K17",
	"homepage_url": "https://mystique-extension.jimdo.com/",

#38 use updated text from here


Request for CLI driven IoT Hub jobs functionality how-to

The IoT Hub jobs how-to documentation shows job support via SDKs but also including a CLI driven experience leveraging the az iot hub job command group would be great and supporting of CLI focused personas.

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

ID: 3e182997-86b8-60ee-353d-f500de072893
Version Independent ID: a88cceb1-7e3d-b30b-6cd6-22dbdec1c660
Content: Schedule jobs with Azure IoT Hub (Python)
Content Source: articles/iot-hub/iot-hub-python-python-schedule-jobs.md
Service: iot-hub
GitHub Login: @kgremban
Microsoft Alias: kgremban

Hi @digimaun, thank you for the suggestion. I think that including CLI capabilities would be a good addition to this set of content. I'm going to make an item for this work in our backlog so that it can be triaged and prioritized appropriately. I'll send you a pointer once that is created.
In the meantime, I'm going to close this issue. I'll make a note to post here again once the work is complete.
#please-close


[v0.1.0] Release Tracker

We cut a release branch for the 0.1.0 release.
This issue is for coordinating and tracking cherry-picks to the release branch. Things might start getting hectic if all of us cherry-pick things to the release branch at the same time, so if you want something cherry-picked to the release branch, comment with a link to a PR that has been merged into the main branch and @zou3519 will cherry-pick, ensure the tests pass, and reach out if anything went wrong.

[ ] PR #411  Because of ghstack, it's commit on main is 1986cf7

[ ] #412 d775bc17217c93545672a819c85f776b827ffd1f
[ ] #510 30f742d98fd04c0714810b90f56f7e0bfdf5e7e1
[ ] #511 30f742d98fd04c0714810b90f56f7e0bfdf5e7e1
[ ] #521 057c3cacade334effd1210eb98382400988849d5
[ ] #558 54c867145bee73b52fd137f717090720f888b3cb
[ ] #523 1fa54342eab96040ad6a71bbe7348bd7d0ffeb8b

AOT Autograd Documentation stuff

[ ] https://github.com/pytorch/functorch/pull/577
[ ] https://github.com/pytorch/functorch/pull/580
[ ] https://github.com/pytorch/functorch/pull/585

We've released!


Disable scorecard test

In its current form, the scorecard test configuration is often a barrier to passing the validation pipeline. Though that is usually not because the Operator is lacking but because scorecard makes some assumptions that do not apply to every Operator.
We need to make scorecard more permissive and configurable for these use cases. Until this is achieved, scorecard will be disabled for the community pipeline.

/hold


Custom portal generation: throw_item trigger consumes whole stack

This might be a relatively silly bug to report but I've noticed that portals with the "throw_item" trigger are activated regardless of the stack's amount, consuming 1 to 64 items. It would be nice if the activation would only consume 1 item from the stack at most. (Unless you'd want to support specifying a stack size so it would have to be at least a certain amount of items.)

As I tested it only consumes 1 item. Maybe the item stack entity teleported through the portal?

I can't reproduce the original issue anymore, but I'm still noticing multiple items can disappear from the stack, especially if the portal creation fails because generate_frame_if_not_found is set to false. (Though do note that I am still using version 0.36 at this moment - we'll have to update our pack again some time soon.)


Support more skos properties

In the long run, wee should support all skos properties. For now, we will cover those we need for the vocabularies we publish. https://github.com/VIVO-DE/destatis_faecherklassifikation – which is published at https://test.skohub.io/acka47/skos/onto.tib.eu/destf/cs/index.html –  has the following SKOS properties that are not covered yet: notation, narrowerTransitive, broaderTransitive and note.

On test, see e.g. https://test.skohub.io/literarymachine/skos/onto.tib.eu/destf/cs/0120.html
Note that the transitive properties are only added to the JSON, I did not know how to represent them in the UI in a useful way.

Everything is there. But I am missing the notations in the tree and am not sure whether they really should just be shown in brackets after the prefLabel. I would like to try out putting the notation before the prefLabel – in the tree as well as in the detail view – as is e.g. done in https://nwbib.de/subjects or in Skosmos (see e.g. https://finto.fi/ykl/en/).

I would like to try out putting the notation before the prefLabel – in the tree as well as in the detail view

On test, see e.g. https://test.skohub.io/literarymachine/skos/onto.tib.eu/destf/cs/040.html

+1


Make site exciting to watch

Add animations, transitions (not too much) to make clock interesting to watch

have you found a .gif you would want to use for the animated background?

have you found a .gif you would want to use for the animated background?

Raw svg code from bgjar.com trust you can take it from here

@RDjarbeng , the code you pasted is incomplete.


Using from google colab notebook

Good day Seth,
I have been going over your work and as I am about to import the libraries, the code throws an import error, "cannot import name 'downsample_mono'." I am not sure which version of the 'clean' library did you import but could you kindly assist me in solving this error?
I am using Google Colab so I could utilize the GPU.
Thank you for supplying us with such, we are really grateful for it. Thank you for your time.

I downloaded the code to my google drive and mounted to the notebook instance.
You need to uncomment and install the requirements.txt.
I've never attempted to run this stuff from colab, but technically it could work. I did get a lot of warning when installing requirements.txt, so just might just want to install things one by one as you need them.
sys.path.append adds the mounted instance into PYTHONPATH, so you can make imports from clean.py.

Thank you very muxh for your help and swift response.
Just one last point which I have to resolve.
When I am running the prediction code I get thrown into this error: "Exception: Data must be 1-dimensional." Is there a way to solve this error which you could share with me.

I never tested this in google colab, so I'm not going to attempt to solve some error with data I don't have access to running in an instance that requires all the packages to be revised.


add default value getter

Add a getter to retrieve a knob's default value. Useful when having to reset a knob to its default state.

function needs "const" attribute, also please put it just before the setDefault call
thanks

thanks!


Upgrade Symfony to 6.3.3 & other vendors

For some reason, EA fails with custom routes... See https://github.com/EasyCorp/EasyAdminBundle/issues/5864
Hence the change in tests.

Indeed, not practical for this story of a road that fails. Hoping it will be fixed soon.
PR OK :)


Filtering duplicates from primary impacts performance

These lines from get_metrics() look to negatively impact performance pretty significantly, esp. when the query has filters applied.  Guessing it forces a read of the entire primary dataset even when only some of it is needed.
WITH filtered_primary AS (
    SELECT * FROM(
        SELECT *,
            row_number()
        OVER(
            PARTITION BY value_time, location_id
            ORDER BY reference_time desc
            ) AS rn
        FROM read_parquet("/home/jovyan/data/protocols/retro/timeseries/usgs/*.parquet")
        )
),

34 seconds vs 4 seconds when filtering 3 years of retrospective data at all USGS stations to a single USGS station.
We might need a flag to only include this when the user requests that the primary data be "de-duped".

@mgdenno I pushed some commits adding this flag to this branch.  Wondering how you want to handle the versioning since we'll be at v0.2.9 when #92 is merged.
I could tack this de-dupe flag on to #92 or add a pre-release version here (ie, v.0.2.9.beta)? Any preference?
Weirdly I didn't see much impact of removing duplicates on query performance when testing against Katie's 2023 Hilary dataset (locally).  Not sure if I'm missing something or the dataset is too small to make a difference but just wanted to make a note

I think going to v0.2.10 is perfectly valid.

Just a note, we determined that filtering duplicates after the join is much faster, so the code snip looks like this:
, windowed AS (
            SELECT * FROM(
                SELECT *,
                    row_number() OVER(PARTITION BY value_time, primary_location_id, configuration ORDER BY reference_time desc) AS rn
                FROM joined
                ) t
            WHERE rn = 1
        )
        , metrics AS (
            SELECT
                joined.primary_location_id,joined.configuration
                , sum(primary_value - secondary_value)/count(*) as bias
            FROM
                windowed as joined
            GROUP BY
                joined.primary_location_id,joined.configuration
        )


add antrea in network addon list

Antrea project is a k8s network CNI which has been added in https://kubernetes.io/docs/concepts/cluster-administration/networking/#antrea, add it to addon installation page.

/check-cla

Hi @daminisatya @kbarnard10, Could you help to review? thanks!

/approve


*.mitx.mit.edu certificate renewal

Hi Michael Davidson,
Your TLS certificate "*.mitx.mit.edu" is set to expire on 05/20/2023 at 11:59 PM UTC.
To avoid service disruptions, we recommend you replace this certificate now.

Once we get a renewed certificate for this it will need to be updated in:

Fastly
AWS ACM
Vault QA and Production


Jv rox 12304 remove unused scripts in circleci directory

Removed unused scripts in circleci directory.
Checklist

[ ] Investigated and inspected CI test results

Testing Performed
See above.

/test ubuntu-1804-lts-ebpf-integration-tests


Suspend

Use this functionality by calling tw_lp_suspend in your event handler. An example is:
if( lp->gid == 0)
{
    printf("LP 0 is suspended at TS = %lf - should only get this once!!\n", tw_now(lp));
    tw_lp_suspend( lp, 0, -31459 );
    return;
}

Current coverage is 73.30%

Merging #75 into master will decrease coverage by -0.21% as of 41015bb

@@            master    #75   diff @@
=====================================
  Files           25     25
  Stmts         2447   2469    +22
  Branches         0      0
  Methods
=====================================
+ Hit           1799   1810    +11
  Partial          0      0
- Missed         648    659    +11

Review entire Coverage Diff as of 41015bb

Powered by Codecov. Updated on successful CI builds.

So I pulled in your suspend flag fix and re-tested the codes suite. The new suspend code works as expected!

Great news!
Thanks,
Chris

Christopher D. Carothers
Director, Center for Computational Innovations
Professor, Department of Computer Science
Rensselaer Polytechnic Institute
110 8th Street
Troy, New York 12180-3590
e-mail<EMAIL_ADDRESS>web page: www.cs.rpi.edu/~chrischttp://www.cs.rpi.edu/~chrisc
phone:<PHONE_NUMBER>tel:(518) 276-2930
fax:<PHONE_NUMBER>tel:(518) 276-4033

From: John Jenkins<EMAIL_ADDRESS>Sent: Monday, November 02, 2015 10:50 AM
To: carothersc/ROSS
Cc: Carothers, Christopher D.
Subject: Re: [ROSS] Suspend (#75)
So I pulled in your suspend flag fix and re-tested the codes suite. The new suspend code works as expected!
—
Reply to this email directly or view it on GitHubhttps://github.com/carothersc/ROSS/pull/75#issuecomment-153059385.

closed #15


Ubuntu 18.04 support

When can we expect Ubuntu 18.04 inclusion in ASR?

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

ID: 4d92f050-73a2-c519-2046-2aba7caeecb0
Version Independent ID: e0fe7de0-ce70-de87-32c7-fe498552f9be
Content: Migrate AWS VMs to Azure with the Azure Site Recovery service
Content Source: articles/site-recovery/migrate-tutorial-aws-azure.md
Service: site-recovery
GitHub Login: @rayne-wiselman
Microsoft Alias: raynew

Thanks for your feedback. The current article is up-to-date. We'll keep it up-to-date it with new releases as they happen.
#please-close

added triage labels


Why is Opaleye disable from nightly?

@bergmark Is probably the best person to handle this
This is a response to https://github.com/fpco/stackage/issues/1907.  I'm posting this as a new issue since there are so many mentions on the original.
Can you explain why Opaleye is disabled?  It says "BLOCKED aeson 1.0".  Opaleye has aeson < 1.1 as you can see here: https://hackage.haskell.org/package/opaleye-<IP_ADDRESS>
It is true, however, that I made an oversight and the tests have not had their aeson dependency bumped.  Is this causing the problem?  If so, could a Hackage admin bump that dependency for me?  I am currently away so cannot do it at the moment.

opaleye is disabled because it happened before the aeson 1 support. We have opaleye under expected-test-failures which means we build the test so we do care about that constraint.
I'm a hackage trustee so I can bump the upper bound.

Thanks, yes please bump the upper bound.

Thanks.  I have an issue to fix this upstream too: https://github.com/tomjaguarpaw/haskell-opaleye/issues/207


Possible to get full resolution images?

The API provides the 320x320 and 750x750 versions of the image, but is it possible to get the full 1080x1080(or x1350), or is this a limitation of using the private API? Thanks!

Hey @SunburstEnzo,
please refer to #244 about the future of SwiftyInsta.


Implement "Clear" button for both Dashboards

When pressed, clear all the selections that has made by the user

Implemented successfully for both dashboards


Add hover template option to plots

https://plotly.com/javascript/hover-text-and-formatting/

Better and custom hover text for complex plots

add option to plot component for string providing label of value and add this label to the hover info in an informed way ;-)

Same problem as #27

X and Y data title
give x and y more descriptive
make decimal places configurable


Traffic Signal made by RoadRunner is available?

Hi, I'm using Carla for fun.
I am interested in road traffic and make some road.
But after 'road.fbx and road.xodr' import to carla well, no car stop to my traffic signals.
How can i stop the autopilot car at traffic light?
Thank you in advance.

Hi MingooJ,
Did you manage to solve this issue? I have the same problem. Thanks

Hi Mkarami3,
Unfortunately, this problem still unsolved. but, i generate traffic light of UE4, it works.
However, complex traffic can't create with it's traffic system. So, i still try solving it.
Thanks.

@MingooJ please provide further information. Which version of CARLA are you using? OS? etc.
@XGodina could you help with this?

Hi, @germanros1987
I used CARLA 9.8 in Ubuntu18.04 and made map using RoadRunner.
I made a traffic light on the roadrunner, but it doesn't seem to work on the CARLA.
And my roadrunner was a trial version, so I can't do anything right now.
I wish it could be of help someone's help.
Thanks.

@MingooJ you need to use 0.9.9 to be able to ingest traffic lights automatically from OpenDRIVE (generated by RoadRunner). Then, you need to cook the contents using the docker provided by CARLA.
@bernatx @sergi-e could you please add more details so that @MingooJ can have a smooth journey?

Hi, @MingooJ !
Indeed, the automatic ingestion of landmarks was added by CARLA 0.9.9. If you download package 0.9.9 and follow this tutorial to add the map, you should be able to run your custom map in CARLA with traffic lights in it.


[Docs] Fix  offset prop default value

Hello, I've noticed that the default value of props.offset for <Popover> component is actually 8 not 0 as the current docs states.
https://github.com/adobe/react-spectrum/blob/80fcbf54206ca1ffb011f28fabcfc982d97bca5f/packages/react-aria-components/src/Popover.tsx#L126-L130
This PR corrects the default value of props.offset to 8 by re-defining the offset prop and omitting the originally inherited one from PositionProps (as a minor consequence of this, the order of the prop table in the docs is a little different)
I'm not sure if this is the best way to override the default value of some inherited props, so let me know if there's a better way.
Thanks.
✅ 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).
[x] 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:

Visit http://localhost:1234/react-aria/Popover.html#props
Check if the default value of the prop offset is 8.

🧢 Your Project:

GET_BUILD


Not working on Windows on ARM

I'm getting an issue that the application is not supported on windows that run on ARM. Sadly the batch file mentioned in the readme does not exist. Otherwise I would have tried to just change the platform target. Kindest Regards

Sorry to disappoint you but running Daedalus on Windows running on ARM is not supported.


Keycaps In Readme

Sorry to reach out via an issue, but I was wonder what the keycaps are in this picture and where I might be able to find them. I have a CRKBD on order and looking to flesh it out with switches and caps. Thanks in advance!

Probably https://make.dmm.com/item/1012817/
Related: https://github.com/foostan/crkbd/issues/13

Thanks @l4u , trying to order off the Japanese site and not having any luck. If anyone knows how to get em, please tag me!


docs(): add polished docs (integrations section) #67

What: add polished documentation from Glamorous GitHub README to website

Why: integrate documentation to the website

How: copied and formatted polished integration documentation from GitHub README into the polished component of website

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@4332d28). Click here to learn what that means.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master     #141   +/-   ##
=========================================
  Coverage          ?   96.56%
=========================================
  Files             ?       58
  Lines             ?      233
  Branches          ?       25
=========================================
  Hits              ?      225
  Misses            ?        7
  Partials          ?        1

Impacted Files
Coverage Δ

pages/integrations/content/polished.js
100% <100%> (ø)

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 4332d28...b45393a. Read the comment docs.


Fix test with setupTestData(), replace with setUp()

Replace setUpTestData with setUp
setUpTestData should not have items that don't support deepcopy. e.g. the database model items

This is a PR branched from the #3018 PR to be able to get to master quickly.


Service gracefully shutdown

Add gracefully shutdown functionality, but still does not implement lifecycle for service.

I think this is superseed by #27 ? Should we close it @al8n ?

Close to support #27

I think this is superseed by #27 ? Should we close it @al8n ?

Yes


Bug: Can't install without prompts

Environment
Node version: v16.17.1
npm version: 8.15.0
Local ESLint version:
Global ESLint version:
Operating System: Ubuntu 22.04
What parser are you using?
Default (Espree)
What did you do?
I am trying to install ESLint with all defaults be running npm init with --yes option, but it doesn't work.
$ npm init --yes @eslint/config
? How would you like to use ESLint? …
  To check syntax only
▸ To check syntax and find problems
  To check syntax, find problems, and enforce code style<!-- Paste your code here -->

What did you expect to happen?
All options are chosen by default.
What actually happened?
Command hangs waiting for input.
Participation

[ ] I am willing to submit a pull request for this issue.

Additional comments
No response

Hi @abitrolly, thanks for the issue!
This was discussed in https://github.com/eslint/create-config/issues/23 and we decided not to support --yes for the reasons described in the linked issue thread.

@mdjermanovic understood. From #23.

@aladdin-add's original use case, clarified in https://github.com/eslint/create-config/issues/23#issuecomment-1087163068, was to have a non-interactive setup that could be run in an unsupervised scenario.

So what is the official way to install ESLint non-interactively without --yes? The only thing that comes to my mind is to throw in some temporary config. I don't know.

You can install eslint package:
npm install eslint --save-dev

Then, you should add a config file. A simple config to lint es5 browser scripts can be:
// .eslintrc.js
module.exports = {
    extends: ["eslint:recommended"],
    env: {
        browser: true
    }
};

This enables recommended built-in rules and enables browser globals.


feat: Add support for smithy.test#smokeTests for Java

Issue #, if available:
Fixes #298
Description of changes:
This trait is ideal for filling a testing gap in our TestModels. Because our strategy for mapping Smithy shapes to Dafny types enforces constraints statically, it's not possible to express invalid input without lying to Dafny through axioms that aren't actually true, or in some cases (such as @required) not possible at all. smokeTests instances generate tests that specify inputs at the Smithy level using document types (i.e. more or less JSON valued arguments to the trait), which generate target language tests using the customer-facing interfaces such as builders to create input, and can therefore express invalid input as well.
The key new component in the implementation is the ModeledShapeValue.shapeValue method, which creates a code block in Java expressing the given Shape value. Similar logic already appears in other smithy code generators, although mostly for HTTP Protocol Compliance Tests instead. I mainly imitated the structure of smithy-python's logic, but was also inspired by smithy-typescript and smithy-rs.
Because this feature introduces a new category of generated code (Smithy-generated tests) this PR also adds an --output-java-test option with SmithyDafnyMakefile.mk support. We will likely need to add a similar parameter for the other languages. It may make more sense to migrate to smithy-build.json first though: https://github.com/smithy-lang/smithy-dafny/issues/356.
It also adds ./gradlew test as an additional step for make test_java, since running the Java @Tests is distinct from executing the transpiled dafny test program.
Finally, it also improves the CLI interface to print out validation events even if there are only warnings. Besides providing previously hidden but useful information in general, this provides a way to warn against using smokeTests outside of test models - since it doesn't/can't support @reference shapes it's not terribly useful as a testing method for local services themselves.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

I left a blocking comment that is not really blocking,
but I do wish we would not put in a Reference Trait Validator
that we know only validates a particular condition...
That is asking any user to look up what it does,
rather than telling them in the name that it is limited.

I actually made a commit to do this and thought I pushed before merging, but the push failed without my noticing. :P I've queued it up for my next PR though.


ISSUE-169: fix gradle demo

Fixed Gradle demo
Updated PF4J version for the demo
Added a README that explains the demo

Coverage remained the same at 56.981% when pulling 7cbb12a71ab4a5083d285386567127ff4b57a5bf on acesar:ISSUE-169 into 42aaee15591e241add08323c76f4f813867f996b on decebals:master.

Thanks!


Implement class regex for custom class name

We are using clsx in our project and with tailwind we could use
	"tailwindCSS.experimental.classRegex": [
		["clsx\\(([^)]*)\\)", "(?:'|\"|`)([^']*)(?:'|\"|`)"]
	],

in settings.json to get intellisense.

It would be great if windi can also implement this.

+1. This would also be a  desired feature for us


Getting an error while loading CSV file with single column

Describe the bug
Getting an error while loading CSV file with single column
To Reproduce
Generate CSV file for example:
header_name
hello
world

Expected behavior
Create dataframe with single column
Screenshots
Could not determine delimiter. Make sure a delimiter is shown the same number of times on every row in the file.

Fixed in v1.1.15


Cache expiration should have an option to base maxEntries on last accessed time

Library Affected:
workbox-cache-expiration
Currently the cache expiration plugin takes a maxEntries option that removes old entries based on their added date. I think it'd be a nice addition to add an option to remove entires based on their "last accessed" date.
The use cases is a static assets cache where JS or CSS entries could quickly become obsolete, but image or SVG assets might have been originally added long in the past but are still currently referenced on the page.
Expiring entries based on their "last accessed" date would help provide a convenient way to auto expire assets that are no longer referenced.

You're describing the current documented behavior:

This plugin can be used in the Workbox API's to regularly enforce a limit on the age and / or the number of cached requests.
Whenever a cached request is used or updated, this plugin will look at the used Cache and remove any old or extra requests.
When using maxAgeSeconds, requests may be used once after expiring because the expiration clean up will not have occurred until after the cached request has been used. If the request has a "Date" header, then a light weight expiration check is performed and the request will not be used immediately.
When using maxEntries the last request to be used will be the request that is removed from the Cache.

In all cases, expiration is done on a least-recently-used basis, where "used" could be a read or a write (i.e., "last accessed").

Maybe I'm missing something, but the only place the timestamp of the IDB entry is updated is in CacheTimestampModel.prototype.setTimestamp method. And the only place that method is called is in CacheExpiration.prototype.updateTimestamp, which is only called from the plugin's cacheDidUpdate lifecycle method.
That means it's not called when it's used; it's only called when it's updated. This also reflects what I'm seeing in my testing.

Oh, yikes. I could have sworn we were also calling updateTimestamp() in the cachedResponseWillBeUsed callback (which is definitely what the docs imply). Good catch.

@prateekbh has volunteered to look into this, likely for the v5 timeframe.


New version: anchor.cli 0.0.23-wingettest2

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[x] Is there a linked Issue?

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Automated with GoReleaser
Microsoft Reviewers: Open in CodeFlow

@microsoft-github-policy-service agree [company="Anchor Security, Inc."]

@microsoft-github-policy-service agree company="Anchor Security, Inc."

@benburkert @stephengillie Can we close this? I'm fixing the PackageIdentifier via automation and this will be replaced with a new PR.

#149561 is the new PR


add prefill option and logic to populate the slabs with data

This allows us to quickly fill a slab's heap with printed numeric keys/values, this is useful for performance tests.
Cache's RSS, which is primarily driven by the amount data stored in slab, is an important factor for performance. As memory gets bigger on modern hardware, it takes longer to fill the heap by external traffic. For example, if we want to warm up a 32GB heap, using 64 bit key+val sizes, we will need to create 300M keys. If we use 100K qps to warm it up, it will take almost an hour. This clearly makes realistic load testing fairly slow and expensive to conduct. In comparison, prefilling the memory inside the server can make this process at least an order of magnitude faster.
The implementation introduces a master switch prefill, and 3 options that control how prefilling should be performed, regarding key size, value size, and number of keys inserted. It also provides a summary of the operation including timing in the log.

Example:
With config:
prefill: yes
prefill_ksize: 5
prefill_vsize: 10
prefill_nkey: 16

We can verify the keys:
get 00000
VALUE 00000 0 10
0000000000
END
get 00009
VALUE 00009 0 10
0000000009
END
get 00015
VALUE 00015 0 10
0000000015
END
get 00016
END

Summary in log looks like this:
prefilling slab with 10000000 keys, of key len 32 & val len 32, in 19.424 seconds


GetAssemblyTypesWithDynamicLinqTypeAttribute fails to type load failures

For .NET Framework, AbstractDynamicLinqCustomTypeProvider.GetAssemblyTypesWithDynamicLinqTypeAttribute() calls Assembly.GetExportedTypes() and handles a resulting ReflectionTypeLoadException.
Assembly.GetExportedTypes() will actually throw FileNotFoundException if an assembly cannot be loaded.
ReflectionTypeLoadException is only thrown by Assembly.GetTypes().
As a result, if there's a failure loading a type caused by a missing assembly, none of the types marked with DynamicLinqTypeAttribute in the assembly will be returned.

@philr Did you encounter this issue in your application, or did you check the source code?
Because the logic in the code is that if an assembly cannot be loaded, the definedTypes will be empty and no types will be returned for that assembly, but the next assembly will be analyzed.

@philr
If you have time, can you verify that PR?

@StefH I've added a review to the PR with a few comments.


Pass full built ROCKs metadata to testing env

Pass the entire list of built ROCKs metadata produced by the build_rocks.yaml workflow as an environment variable to the testing steps.
You can check out what the env var looks like in this run.

The CI only fails on the Rocks upload. Let's merge it.

Just a minor note, it might be a bit cumbersome to define that huge BUILT_ROCKS_METADATA env variable when testing locally.


XKB Keyboard support is required to accurately determine keyboard characters

Expected Behavior
My app should start (simple code from the docs)

Current Behavior
It gives me this error:
λ node .
load_input_helper [1918]: Using XFree86 keyboard layout.
load_input_helper [1920]: XKB support is required to accurately determine keyboard characters!
Speicherzugriffsfehler (Speicherabzug geschrieben)
radsteve@radsteve in ~/folder is  v1.0.0 via  v15.14.0 took 177ms
[⚡] ×

Your Environment

IOHook Version: 0.9.3
Environment name and version (e.g. Chrome 39, node.js 5.4, electron 11.2): node.js 15.14.0
Operating System and version (desktop or mobile): Garuda Linux

Having the same issue :(


Same library class implemented in two places. One of the two will be used. Which one is undefined.

Related Env info
Mac OS X 10.14.6.
python                           3.6.1
mujoco-py                 <IP_ADDRESS>
spinup                    0.2.0
glfw                      1.10.1
The env is basically the result of running the official setup pipeline.
Problem
The algorithm could run properly but the visualizing window did not showed up when I ran the DDPG algorithm.
It said
objc[64756]: Class GLFWApplicationDelegate is implemented in both path2mujoco/mujoco200/bin/libglfw.3.dylib (0x154a8b778) and path2spinningupLibrary/lib/python3.6/site-packages/glfw/libglfw.3.dylib (0x154b3d6d0). One of the two will be used. Which one is undefined.
objc[64756]: Class GLFWWindow is implemented ...
objc[64756]: Class GLFWContentView is implemented
objc[64756]: Class GLFWWindowDelegate is implemented
It seems that the GLFW related library is defined in both Mujoco and GLFW library. So how to deal a problem like this?

I think you may have the wrong mujoco-py. Gym should require a mujoco-py < 2.0.
https://github.com/openai/gym/blob/master/setup.py#L14
Try uninstalling mujoco-py and reinstalling via
pip install gym[mujoco,robotics]
and see if that repairs things.

I reinstall the Mujoco and here I have
mujoco-py                 <IP_ADDRESS>
But the same problem still exists.

@Jack12xl I am having a similar issue, where you able to figure out a solution?

I just delete the libglfw.3.dylib file under site-package/glfw, and it seems to work for me.

@VentusYue I tried this, and it returns this error: distutils.errors.LinkError: command '/usr/local/opt/llvm/bin/clang' failed with exit status 1.
I suggest you do not delete the file.


Move binding.js to repository root

Removes the need for involving the path module.

Wanted to mention that this PR solves a problem getting this library to work with Electron (electron-rebuild  specifically).
Error without it:
App threw an error during load
Error: No native build was found for platform=darwin arch=x64 runtime=electron abi=110 uv=1 libc=glibc node=16.17.1 electron=22.0.3 webpack=true

Until this PR is merged, a quick work around in package.json:
  "resolutions": {
    "udx-native": "github:holepunchto/libudx.git#move-binding-js"
  },

Thank you @kasperisager!


[WebApp] sync function triggers can return 200

In https://github.com/Azure/azure-rest-api-specs/blob/master/arm-web/2016-08-01/swagger/WebApps.json#L9667 and https://github.com/Azure/azure-rest-api-specs/blob/master/arm-web/2016-08-01/swagger/WebApps.json#L10659, only 204 is listed as a possible response.
However, in live testing, we discovered that server can also return 200.

I see a 200 response when I call the 2018-02-01 API as well.

@Nking92 that should be a simple fix, would it be possible to address it?

I think the WebApps_SyncFunctionTriggersSlot may have the same issue, but I have not tested to confirm.

@Nking92 can we get an update when this will be fixed?

Any updates on this issue? The latest stable spec still is not updated to expect 200 responses.

https://github.com/Azure/azure-rest-api-specs/blob/31294f31d4a3f7576117cfa8c58f61d05b24b885/specification/web/resource-manager/Microsoft.Web/stable/2018-11-01/WebApps.json#L17134

It seems ridiculous that this issue is open for more than 5 years now and either anybody cares or is able to fix it. Could someone please finally fix this. I really dislike catching success exceptions:
From the new Azure.ResourceManager.AppService SDK:
Azure.RequestFailedException: Service request failed.
Status: 200 (OK)
Content:
{"status":"success"}


Weird SQLite Error on Startup

13|counter | [2024-04-14 08:28:05][INFO] Running v10.1.0-rc.1 of counterparty-lib.
13|counter | [2024-04-14 08:28:05][INFO] Running v10.1.0-rc.1 of counterparty-server.
13|counter | [2024-04-14 08:28:05][INFO] Connecting to address indexer.
13|counter | [2024-04-14 08:28:05][INFO] Acquiring lock.
13|counter | [2024-04-14 08:28:05][INFO] Connecting to database (SQLite <IP_ADDRESS>).
13|counter | ERROR:root:SQLITE_LOG: recovered 642 frames from WAL file /home/adam/.local/share/counterparty/counterparty.db-wal (283) SQLITE_NOTICE_RECOVER_WAL
13|counter | [2024-04-14 08:28:05][INFO] Starting API Server.
13|counter | [2024-04-14 08:28:05][INFO] Resuming parsing.
13|counter | [2024-04-14 08:28:06][INFO] New block inserted 839147

13|counterparty  | ERROR:root:SQLITE_LOG: statement aborts at 22: [
13|counterparty  |         SELECT * FROM balances
13|counterparty  |         WHERE (address = ? AND asset = ?)
13|counterparty  |         ORDER BY rowid DESC LIMIT 1
13|counterparty  |     ] database schema has changed (17) SQLITE_SCHEMA
13|counterparty  | [2024-04-14 08:28:21][INFO] Block 839147 parsed, ledger hash is 8a8ee58ad33a530a54630cc57c251ff9f7eb22827867c016ae67bd79b592bfc6 and txlist hash is 9c664aa0a9e17d3b569fac54ddc42809dc2c338a62d12a07352d7953e88b9909
13|counterparty  | [2024-04-14 08:28:21][INFO] Block: 839147 (15.67, hashes: L:2bfc6 / TX:b9909 / M:8ce3b)
13|counterparty  | [2024-04-14 08:28:24][INFO] New block inserted 839148
13|counterparty  | [2024-04-14 08:28:27][ERROR] Key error in addrindexrs still exists!!!!!
13|counterparty  | [2024-04-14 08:28:27][WARNING] tx missing in rawtx cache: '1cd237a14fb463ff2ff8fae0dd62126b2bd1d1d140c94f1486c26d880fa40b49' -- txhash_list size: 2990, hash: b2bc4ded94244c52f4caa994baf374e5 / raw_transactions_cache size: 2990 / # rpc_batch calls: 2990 / txhash in noncached_txhashes: True / txhash in txhash_list: True -- list []
13|counterparty  | [2024-04-14 08:28:32][INFO] Credit 1 A3552076589448406000 to 171EqT3RUgyLbwRDhmMfhXPnLws4AXLWxy

What I think is going on:

DB didnt shut down correctly, SQLite is doing what is should and recovering from write ahead log
DB connection is opened in read-only mode and as such doesn't need to acquire writer lock ( which could not be acquired until after recovery is completed )
Query is made against table which is being modified by WAL recovery process, error thrown

Thoughts @ouziel-slama?

hum.. I don't think so.. I fixed the bug which caused the presence of a wal file and the error message appears even when there is no recovery.

Hmm, I think we should change the "Lock acquired." log to info level.
# server.py#50
def get_lock():
    logger.info("Acquiring lock.")

    # Cross‐platform.
    if platform.system() == "Darwin":  # Windows or OS X
        # Not database‐specific.
        socket_family = socket.AF_INET
        socket_address = ("localhost", 8999)
        error = "Another copy of server is currently running."
    else:
        socket_family = socket.AF_UNIX
        socket_address = "\0" + config.DATABASE
        error = (
            f"Another copy of server is currently writing to database {config.DATABASE}"
        )

    lock_socket = socket.socket(socket_family, socket.SOCK_DGRAM)
    try:
        lock_socket.bind(socket_address)
    except socket.error:
        raise LockingError(error)  # noqa: B904
    logger.debug("Lock acquired.")

(That said, it's been a while since I saw this error message!)

Even though we don't have a callstack here, it's something like this:
ledger.get_balance
<message>.parse
blocks.parse_tx
blocks.parse_block
blocks.follow
server.start_all

This would all happen before DB recovers from WAL file.

this happens systematically on the first RPC call, but not on the next one! and there is no WAL recovery.

ERROR:root:SQLITE_LOG: statement aborts at 43: [
        SELECT d.*, a.asset_longname
        FROM dispensers d
        LEFT JOIN assets a ON a.asset_name = d.asset
        WHERE (tx_hash = ?)
        ORDER BY d.rowid DESC LIMIT 1
  (17) SQLITE_SCHEMA

FWIW I get this error when the first call is get_dispenser_info!

Solved by  #1701


Move Canonical.Ubuntu v16.04 -> Canonical.Ubuntu.1604

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Might as well use the vanity link since both seem to result in domain validation errors


Fix IE bug in the docs.css for "docked" sub navbar in fixed position.

Added an overriding filter: progid:DXImageTransform.Microsoft.gradient(enabled=false) to the .subnav-fixed class that fixes the bug caused on nav dropdown menus when "docked" in the fixed position.

If by bug you mean issue #1869 then this fix unfortunately doesn't work for me ...

Now all that's missing is the overlapping static navbar-fixed-top dropdowns. I played around with the z-indexes, changing .subnav-fixed to around 1029 makes the menu go underneath the top menu.
but for some reason it loses about 2 pixels in height, I'll have to look into this ...


Wait until DOM is loaded

Wait until the DOM is loaded until do any changes to the DOM.
Fix some issue(still has another one) on duckduck.com

All code this contains is over #87


Fix selling giving odd values

This eliminates the forcing of all values to gold and actually spreads it over to the different monetary units.
On the side it also removes double update which likely causes some other problems with selling.
Closes #4

Thanks a lot for the pull request. For some reasons, GitHub notifications landed in my spam and I wasn't aware of that contribution


Stats Benchmarks

Adds microbenchmarks for OpenCensus Stats.

Looks like rebase created a merge commit.

Ta - thanks!


Select Data Source - Does not expand on filter

OS: windows 10
DBeaver version: 5.1.6 CE
Description:
When typing in the filter field and connections are in folders. It only filters the folders themselves. It would be more better to have auto expand on the dialog when typing the filter. So when you select the field it would auto generate to expand based on search. If you have a long folder structure it can be very cumbersum.

Agreed


On Colab, UnicodeDecodeError

Hello, I am using pygtp4all on Colab, but I often encounter the following error:
---------------------------------------------------------------------------

UnicodeDecodeError                        Traceback (most recent call last)

[<ipython-input-2-1545ea4aa0da>](https://localhost:8080/#) in <cell line: 8>()
      6
      7 model = GPT4All_J('./ggml-gpt4all-j-v1.2-jazzy.bin')
----> 8 model.generate("Once upon a time, ", n_predict=55, new_text_callback=new_text_callback)

[/usr/local/lib/python3.9/dist-packages/pygptj/model.py](https://localhost:8080/#) in generate(self, prompt, new_text_callback, n_predict, seed, n_threads, top_k, top_p, temp, n_batch)
    122
    123         # run the prediction
--> 124         pp.gptj_generate(self.gpt_params, self._model, self._vocab, self._call_new_text_callback)
    125         return self.res
    126

UnicodeDecodeError: 'utf-8' codec can't decode byte 0xad in position 0: invalid start byte

Colab soure code:
!pip install -q pygpt4all
!wget https://gpt4all.io/models/ggml-gpt4all-j-v1.2-jazzy.bin

# https://github.com/nomic-ai/pygpt4all
from pygpt4all.models.gpt4all_j import GPT4All_J

def new_text_callback(text):
    print(text, end="", flush=True)

model = GPT4All_J('./ggml-gpt4all-j-v1.2-jazzy.bin')
model.generate("Once upon a time, ", n_predict=55, new_text_callback=new_text_callback)

I am getting this issue as well on Ubuntu 22.04 VirtualBox VM, I am just trying to run the code example in README file

Also getting this on Ubuntu 20,04  (WSL2) using python 3.11.2. Tried using a fresh venv and installing below but still same problem.
pygpt4all                1.0.1
pygptj                   1.0.5

im experiencing this as well on Ubuntu 22.04, python 3.10.9.
i to was just trying to rund the code example from the readme. i had some additional errors with unknown tokens (see the following additional output):
---------------------------------------------------------------------------
model.generate("Once upon a time, ", n_predict=55, new_text_callback=new_text_callback)
gptj_generate: seed =<PHONE_NUMBER>
gptj_generate: number of tokens in prompt = 3
gpt_tokenize: unknown token ' '
gpt_tokenize: unknown token 'a'
gpt_tokenize: unknown token 'm'
gpt_tokenize: unknown token 'e'
gpt_tokenize: unknown token ','
gpt_tokenize: unknown token ' '
Once upon tiTraceback (most recent call last):
  File "/opt/python/mambaforge/envs/cbot4me/lib/python3.10/site-packages/IPython/core/interactiveshell.py", line 3505, in run_code
    exec(code_obj, self.user_global_ns, self.user_ns)
  File "<ipython-input-6-0c9a6b32721a>", line 1, in <module>
    model.generate("Once upon a time, ", n_predict=55, new_text_callback=new_text_callback)
  File "/opt/python/mambaforge/envs/cbot4me/lib/python3.10/site-packages/pygptj/model.py", line 124, in generate
    pp.gptj_generate(self.gpt_params, self._model, self._vocab, self._call_new_text_callback)
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xad in position 0: invalid start byte

separately, i did try to apply this update mentioned in the comment relating to the similar (but different) issue #61. i actually did get it to successfully generate some text, but it ended up failing again after generation of just a few tokens.

Thanks @LeMoussel for reporting the issue.
I just tried it on Colab and you are right, I don't know why building it from source does not raise this error! Maybe something related to the pre-built wheels process!
I pushed a new version of pygptj v1.0.8 to handle the error as @cbrendanprice mentioned, but it still not working as expected.
from pygptj.model import Model

def new_text_callback(text):
    print(text, end="")

model = Model('/home/su/Downloads/ggml-gpt4all-j.bin', log_level=logging.ERROR)
model.generate("Once upon a time, ", n_predict=55, new_text_callback=new_text_callback)

I tried a lot but couldn't figure out yet why the prebuilt wheels are not working properly!
Please let me know if you have any idea!

Same error for me (Ubuntu 22.04 running on WSL2), tried with both pygptj and pygpt4all.

Same problem on Ubuntu 20.04
gptj_generate: seed =<PHONE_NUMBER>
gpt_tokenize: unknown token ' '
gpt_tokenize: unknown token 'a'
gpt_tokenize: unknown token 'm'
gpt_tokenize: unknown token 'e'
gpt_tokenize: unknown token ','
gpt_tokenize: unknown token ' '
gptj_generate: number of tokens in prompt = 3

Hello guys,
Please update the package and give it a try now ?
I believe the issue should be solved.
(ensure you have pygptj version 1.0.10).

It works for me. Thx!

HI, It's OK with pygptj version 1.0.10


URL parsing for redis

fixes #50

Thanks for the effort, great work
@TroyDota
Please adjust the README for the redis storage and add a unittests, then i will merge it promptly

Thanks for the effort, great work
@TroyDota
Please adjust the README for the redis storage and add a unittests, then i will merge it promptly

could you add something for this in the README for the redis storage and a unittest

Test and documentation added in #64

your work is included in the other pull(#64) request, thanks for the effort and contribution


not an issue: freaking love this theme

you're such an amazing designer!!!!!! i won't use this for my personal blog today but please know i am obSESSED with this gorgeous zola theme. thank you for gifting the world your creativity!

Just star it on GitHub. Don't need an issue for that. Making it look more buggy.


I cannot set the value of the select2 box dynamically.
I'm mainly interested in clearing it - so .select2('val', '')
Here's an example jsfiddle where I've got a 2-level hierarchy, and 2 select2 boxes for each level.
The buttons do completely unexpected things.
JSFiddle: http://jsfiddle.net/PlaceboZA/5casr/5/
NOTE: This might be related to #135 (setting val not working), but to be honest, the fiddle in #135 appears to work for me. Feel free to merge issues if you prefer.
Regards

d


ENH: stats: Integrate library UNU.RAN into scipy.stats

Trying to build a thread-safe API

@mckib2 Yay, windows builds pass now. I added LLVM and MinGW to path as suggested in https://github.com/scipy/scipy/issues/13717#issuecomment-852388251


(urql-plugin) - add additionalTypenames functionality

When working with document-based-caching in urql we had the caveat that when an empty list was returned this query would never be invalidated.
This can be fixed by passing additionalTypenames to the useQuery hook so the doc-cache is aware of what the dependencies are.
Now we are passing it all the time, I could see if it's a list and only pass it then, up to you what you prefer.
docs link
I made this a draft PR so we can discuss on what standards you'd like to see, I tried copying from other plugins.
Todo:

Fragments containing types

This is cool, thanks @JoviDeCroock !
I'm not familiar with urql and all it's features, but this seems like something we can support.
Does this hooks replaces the other hooks generated by the codegen? because it seems like the name useXYZQuery is the same.
Maybe @mxstbr can help with reviewing it?

@dotansimha it replaces the usual query since we add the memoized context, this would if the user chooses to use document-cache prevent weird bugs when the return value would for instance be this:
query todos { id }
const result = { todos: [] }

now we don't know that todos has a dependency on __typename: Todo and it won't automatically invalidate when a mutation has a result of Todo. This makes life a bit easier on the user so it should just replace the normal query when the user chooses for this approach.

I'll defer to @JoviDeCroock who makes urql—if he thinks we should do this, let's do it! :wink:

@JoviDeCroock I think we can continue with that PR. What are the missing pieces here? is there something I can do to help?

@dotansimha if you could point me where to do the fragments stuff that would be awesome, I'll probably have time to look at it this weekend.

Hi, what is blocking this PR?

@dotansimha I couldn't find a way to add fragments, if you could point me at an existing place that would be great. This is finished apart from that https://github.com/dotansimha/graphql-code-generator/pull/3648/files#diff-cb39cefa28181b86b615240373524d1aR715

@JoviDeCroock I'm curious, did you ever figure out how to get the typenames from fragments?  I see the PR is still open, so I'm guessing no.  Thanks.


added s16 bp

Yo marvelco do u know where can i find the ids for items for s17 bp

I got the mcp already


multiple BN windows causes errors

  File "/opt/local/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/threading.py", line 953, in run
    self._target(*self._args, **self._kwargs)
  File "/Users/jwiens/Library/Application Support/Binary Ninja/repositories/community/plugins/seeinglogic_ariadne/ariadne/server.py", line 32, in run_http_server
    with HTTPServer((address, port), handler) as httpd:
  File "/opt/local/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/socketserver.py", line 452, in __init__
    self.server_bind()
  File "/opt/local/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/http/server.py", line 136, in server_bind
    socketserver.TCPServer.server_bind(self)
  File "/opt/local/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/socketserver.py", line 466, in server_bind
    self.socket.bind(self.server_address)
OSError: [Errno 48] Address already in use

Probably just need to more cleanly detect if it's in use already and warn but only if the user tries to use it? IE, optimize for the most common case where they're not planning on using the plugin and don't need to be getting extra alerts about it.

Thanks for letting me know, I didn't expect this use-case... but now Ariadne supports it by using the next available port when the port is already bound!
The plugin logs the IP and HTTP port for the web UI the user should use for the current instance on startup.
The websocket port that handles comms between the web UI and the core will also use the next available port number, though that should be completely transparent to the user.


add "files" to package.json for distribution

Currently this package cant be used because the .gitignore excludes /dist from the package on release. npm docs
this change adds "files" field to package.json which takes precedence over .gitignore or .npmignore etc.

Hi @roggervalf. Hmm, NOW it shows up for me as well. First time i installed it the "dist" folder was omitted until i manually ran npm run build in the node_modules/bullmq-otel folder. So i immediately assumed it was a misconfiguration on your end.
So i guess this is not an issue after all ¯_(ツ)_/¯

@philipodev you are actually right, the dist directory is in npmignore and I just tested to install the package and there is no dist files there.

@fgozdz can you look into this? I think that besides adding files in package.json, you also need to remove the /dist directory from .npmignore

very odd because at first i didnt have the dist folder, but now i do on every install (could be a cache thing). or maybe works differently between npm, yarn and pnpm (im using pnpm)?

sorry @philipodev I thought that this was base bullmq repo. I could validate that dist is not being published for this repo


MINIFICPP-265 Disabled fsanitize=address for now, due to overly-verbo…

…se log messages. This may be re-enabled in a more controlled fashion later on.
Thank you for submitting a contribution to Apache NiFi - MiNiFi C++.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[x] Does your PR title start with MINIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[x] Has your PR been rebased against the latest commit within the target branch (typically master)?

[x] Is your initial contribution a single, squashed commit?

For code changes:

[x] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[x] If applicable, have you updated the LICENSE file?
[x] If applicable, have you updated the NOTICE file?

For documentation related changes:

[x] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

Opened this off a bit too soon. Let me go in and change those commented lines into deleted lines.

Fixed. Ready for review/merge now.

looks good.


Update salt requirement

Codecov Report
Patch has no changes to coverable lines.

:exclamation: Current head c0b2fd5 differs from pull request most recent head f468144. Consider uploading reports for the commit f468144 to get more accurate results

:loudspeaker: Thoughts on this report? Let us know!.