Add a fixit for attempting member lookup on Any.

What's in this pull request?

Add a fixit for cases where the user attempts dynamic member lookup on things with type Any.

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    All supported platforms     | @swift-ci Please smoke test and merge
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

A smoke test on macOS does the following:

Builds the compiler incrementally.
Builds the standard library only for macOS. Simulator standard libraries and
device standard libraries are not built.
lldb is not built.
The test and validation-test targets are run only for macOS. The optimized
version of these tests are not run.

A smoke test on Linux does the following:

Builds the compiler incrementally.
Builds the standard library incrementally.
lldb is built incrementally.
The swift test and validation-test targets are run. The optimized version of these
tests are not run.
lldb is tested.

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    All supported platforms     | @swift-ci Please test and merge
    OS X platform               | @swift-ci Please test OS X platform
    OS X platform               | @swift-ci Please benchmark
    Linux platform              | @swift-ci Please test Linux platform

Lint Testing
    Language     | Comment
    ------------ | -------------
    Python       | @swift-ci Please Python lint

Note: Only members of the Apple organization can trigger swift-ci.

...and make it general enough to handle any expression.
Suggest casting to AnyObject or a more specific type.
rdar://problem/27639935
(cherry picked from commit 56a5ef00a4234966698dda1d0aedded170a0b305)
(cherry picked from commit 4f5b80dd03c59aa0856832191c29a53cc502d5fd)

@swift-ci Please test.


Auto-salvage feature

https://github.com/longhorn/longhorn/issues/1046

@meldafrawi Can you help with the integration test and more manual testing?
Integration test cases:

test_ha_salvage should be run with this setting disabled.
Need a new test case test_ha_auto_salvage

At least need manual testing for the following scenarios:
All the test cases are with the setting enabled.
Test case 1:

A pod is running and using Longhorn volume.
Stop all the replicas of the longhorn volume at the SAME time.

Expect result

Volume recovers by itself.
Pod recovers by itself. (might need to set liveness probe like in https://github.com/longhorn/longhorn/blob/master/docs/recover-volume.md#recommended-setup-when-using-longhorn-volumes

Test case 2:

A pod is running and using Longhorn volume.
Stop one replica.
Wait for 30 seconds.
Stop all the other replicas

Expect result

Same as test case 1

Test case 3:

A pod is running and using Longhorn volume.
Stop one replica.
Wait for 2 minutes.
Stop all the other replicas

Expect result

Same as test case 1
The first stopped replica should remain in the Error state.

Test case 4

A pod is running and using Longhorn volume.
Stop the node network connection to other nodes to simulate a network outage.
Resume network connection after 1 minute

Expect result
Same as test case 1.
The test case 4 is the targeted scenario test case.

Validation: PASSED


release-download-count: not showing the count anymore

Description
As can be seen in the screenshots, the release count is not added to the assets anymore.
The setting is enabled, I even tried it without ad-block as I thought that might be the issue.
My token is set to never expire, although I think it's not used for the release count.

I made sure that the setting is enabled.

How to replicate the issue + URL
Navigate to https://github.com/Enubia/ghost-chat/releases/tag/v2.2.6
There should be a release download count next to some of the assets.
Extension version
23.8.3
Browser(s) used
Chrome Version 115.0.5790.170 (Official Build) (arm64)

Thanks for the report! This is due to:

https://github.com/refined-github/github-url-detection/pull/179

Solution:

add the "is single tag" detection the include array in https://github.com/refined-github/refined-github/blob/d8939719446c0504da7b151037f0aba197a4734c/source/features/release-download-count.tsx#L79-L84

Other features may be affected, it's worth a check on every isReleasesOrTags usage


About multivariate prediction

Hello, if i want to make a multivariate prediction, should the target be like this(in a list form)?
args.target = ['ws10_real', 'dir10_real', 'ws32_real', 'direction32_real', 'ws31_real', 'direction31_real']

Hi, if you want to make a multivariate prediction, you just need to set --features M, and the model will predict all input features.


Fixed initial conditions: global identifiability

In addition to knowing the ODE system and time series data for some outputs, one frequently also knows the initial conditions for some of the states. It would be great to use this information to clarify the identifiability analysis.
For an algorithm based on input-output equations, an interesting discussion of this problem can be found in this paper. To the best of my understanding, the paper does not give a complete algorithm but some ideas can be of interest.
Some interesting specific things we could do:

fixed initial conditions can be used when verifying the rank of the Wronskian. If it is still of full rank, then identifiability with these fixed initial conditions is at least as good as without them (that is, we do not loose identifiability because of hitting a singular point).
As suggested in the paper mentioned above, if the system is controllable from the fixed initial conditions, then again the result is at least as good.
What would be interesting to see is if we could get a different result of elimination by making some initial conditions fixed in the GenericPointGenerator used in the elimination process...

The functionality to add the known initial conditions (assuming that they are generic) has been added in release 0.5.4 (right now, for @ODEmodel macro only). An example is given in this tutorial


Corrected snippets for Equals

Fixes dotnet/docs#5158

@rpetrusha I've replaced the tabs with (4) spaces throughout the file.
Perhaps you should consider having a .editorconfig file in the root of the repository to help people following the code standard. This will of cause initially give you some larger diffs (due to tabs vs spaces etc), but it will make the samples more consistent on the long run.


The size of RAW and RGB images in the MIT-Adobe FiveK dataset is inconsistent

Hello, the RAW and RGB image sizes of the MIT-Adobe FiveK dataset I downloaded are inconsistent, how did you solve this problem?

Hi, we just use the raw datasets to output the sRGB images. Please have a look at this issue:
https://github.com/nothinglo/Deep-Photo-Enhancer/issues/38
To make things easier, you can directly download our processed sRGB low- and normal-light image pairs in the Baidu Disk (code:cyh2) and Google Drive
If you find our repo useful, please help us star it, thanks :)


Upgrade ex_rlp to 0.3.0

Hello @malroc thank you for your Pull Request, this is a great change, unfortunately we will not merge this. ex_rlp is already available in version 0.3.1, if our tests will pass with this version, we will directly upgrade to this newest version.


Draggable: Allow users to nullify the handle

Ref https://github.com/adobe-community/issues/issues/7311
Beff dialog derivatives that want the entire dialog to be draggable either need to target the inner content and add a js-drag-handle class (doesn't always cover the entire dialog), or nullify the handle option so that the entire popup is draggable.
I tried adding js-drag-handle as a container class of the dialog to no avail

:+1:

i dont understand why this would be necessary.  setting the entire container as the handle should work as long as the cancel area is also set correctly

please investigate whether this is necessary, but if so the code looks fine

:+1:

@jrefano In jqueryui/draggable, this selection is empty/falsy when we try to set the dialog's top-most container as the drag handle.
When we nullify the handle option, the guard defaults to true – allowing the whole dialog to be draggable.


[RFR] Fix ESI content negotiation for Applications and Tokens API segments

Purpose
With ESI enabled Application and Token API segments would return "" for each application or token. This fixes that problem.
Changes
In order to disable the browseable API for these endpoints the list of renderers was specified on the view. The renderer specified didn't process the response, removing quotes from around ESI tags. I changed the renderers list to the two renderers that will remove those quotes.
Side effects
I functionally tested the endpoints for applications and tokens using both the ?format=json and the content-type request header. They both returned correct responses and the endpoints did not appear in the browseable API locally. My local setup runs through varnish. It's possible that something is amiss @abought and @samchrisinger should probably take a look at it once it gets on staging to make sure it's secure.
Ticket
Flowdock link of the conversation regarding this.

One testing suggestion (until I can find my notes for how to set up varnish locally...)
The tokens endpoint has very specialized behavior- it is intended to hide the token value after the first visit. (and never serialize it again)
We should make sure that varnish doesn't inadvertently cache and return more information than we expect to be present, for subsequent requests to the token endpoint.

@abought The VCL (which is in the private-edge-configs repo) is "pass"ing anything based in the tokens namespace. It acts as nothing more than a proxy.
    # it's tokens, pass
    if (req.url ~ "^/v2/tokens/") {
        std.log("DEBUGGING: PASS -- TOKENS");
        return (pass);
    }

@abought Added todos.

@sloria and @icereval This is required for varnish on prod. It'd be nice if we could get this merged so QA could start testing it Monday or even later today on staging.
If it's a problem, let me know and I can turn it into a hotfix on top of 0.68.0. I thought it was going to be in 0.69.0 so I based it off develop. It should merge pretty cleanly regardless.
@icereval I undid the yapf formatting ;)

:shipit:

@cwisecarver @sloria this is becoming a bit of a blocker for me. Travis failures seem unrelated. Any reason not to merge?


Automate package signing

Regular gpg sign not working, need to utilize gpg-agent and appropriate rpmsign, debsign


IDs instead of [home, away, shirt_number]

Currently while building the datasets, the way to reference a team is by [home, away], and the way to reference a player is by combining [home, away] with the shirt number. I believe when available in the data, it would be better to index by the team and players IDs.
What are your thoughts on this?

Good one. I would like to make this configurable. When combining data from different providers it easier to use shirt numbers.
For what use case would you like the provider ids?
OR implement https://github.com/PySport/kloppy/issues/3 and add a little helper / layer to go from shirt number to provider id.
I think I’d prefer the last one. Would that work for you?

By last one you mean "go for the provider ids by default and have a mapper to go to shirt numbers"? I'd agree with that. The game model on #3 could have field to specify which provider are the tracking and event from and that can be used in the mapper to go from ids to shirt number.

Brainfart here for the internals: introduce a PlayerIdentifier type which holds IdentifierType (Jersey, Provider[Statsbomb], Provider[Metrica], etc) and the value.
and use player_id instead of jersey_number everywhere.
The PlayerIdentifier eq method should check type and raise exception when types don't match. This will prevent mistakes when trying to match different types of identifiers

Fixed in https://github.com/PySport/kloppy/pull/34


Users unknowingly removed from Distribution Groups

What App Center service does this affect?
Xamarin iOS app Distribution Group
Describe the bug
I had two users report that they were unable to get the app from the app center today. Yesterday everything worked fine. I looked at the Distribution and those two users were no longer in the list.
To Reproduce
I don't know how or why they were removed.
Expected behavior
I would expect that users wouldn't seemingly be removed from the list at random.
Screenshots
Desktop (please complete the following information):
Smartphone (please complete the following information):
Additional context
I would like some kind of notification or logging around adding/removing users from the distribution so I can see if someone unknowingly removed them or if this was a glitch on your side.

Hi, thanks for letting us know. Can you start an intercom ticket so we can have more info about your app to help debug this. You can create a ticket either through the ? on the top right or blue intercom button on the bottom on the screen. Someone from our engineering team will be able to help.


Beta

:tada: This PR is included in version 1.0.0-beta.44 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[BUG]

Describe the bug
¨Homebridge is sometime slow to respond, I got this error in the logs :
[6/17/2021, 9:18:11 AM] [homebridge-harmony] This plugin slows down Homebridge. The read handler for the characteristic 'On' was slow to respond! See https://git.io/JtMGR for more info.
[6/17/2021, 9:18:17 AM] [homebridge-harmony] This plugin slows down Homebridge. The read handler for the characteristic 'On' didn't respond at all!. Please check that you properly call the callback! See https://git.io/JtMGR for more info.
To Reproduce
None
Expected behavior
My devices become not responsive during a time
Screenshots
If applicable, add screenshots to help explain your problem.
IOS (please complete the following information):

Version 14.6
Device: iPhone 12 Pro Max

Logs
[6/17/2021, 9:18:11 AM] [homebridge-harmony] This plugin slows down Homebridge. The read handler for the characteristic 'On' was slow to respond! See https://git.io/JtMGR for more info.
[6/17/2021, 9:18:17 AM] [homebridge-harmony] This plugin slows down Homebridge. The read handler for the characteristic 'On' didn't respond at all!. Please check that you properly call the callback! See https://git.io/JtMGR for more info.
Config
"name": "Galaxy - Harmony",
"hubIP": "<IP_ADDRESS>",
"hubName": "Galaxy - Harmony",
"mainActivity": "Apple TV",
"DELAY_BEFORE_RETRY_AFTER_NETWORK_LOSS": 60000,
"HUB_CONNECT_TIMEOUT": 10000,
"HUB_SEND_TIMEOUT": 30000,
"showTurnOffActivity": "true",
"linkVolumeControlToTV": true,
"platform": "HarmonyHubWebSocket"
Additional context
Synology Docker DSM7
Node JS : 14.17.1
Nmp : 6.14.13
Thanks for the help

New error : This plugin slows down Homebridge. The read handler for the characteristic 'On' didn't respond at all!. Please check that you properly call the callback! See https://git.io/JtMGR for more info.

Hi,
don't post the same error each times it happens. This error is has it said due to a slow response from your hub whil the plugin requests its state.
You first have some things to try :

check network conectivity of your hub
Since you are on docker on synology, it might be related to host CPU / network usage also, or ressource available for docker itself and/or node.js ressources allocation. Y
You can try to publish in a separate bridge if you want to prevent this plugin to slow down others.

Your timeout is set to 30s, so, if the buh is slow to respond, indeed it will generate such warnings in homebridge.

Hi,
Sorry for the double post, and thanks for the response.
I have found my issue. It seems that I have an issue with my Apple TV doing the HUB. I turned off the two Apple TV's from doing the HUB, and let my HomePod do the job.
So far, it's working great. I let a day or two and close this topic.
Thanks, take care.


pkg/route: Add a package with route helpers

Constructing a usable URI for a route is non-trivial code that is useful in many route consumers.  Save consumers a few lines by creating a helper in a shared location for centralized maintenance.
Owners from router/OWNERS.
CC @Miciah

Some potential consumers:

installer, which currently assumes HTTPS and the spec host.
auth operator's RouteHasCanonicalHost, which becomes _, _, err := IngressURI(route, host); err != nil
openshift/cincinnati-operator#66

It might make sense to add a test case with Admitted=False.

Done with explicitly non-admitted ingress, host-agnostic in 08e24b164 -> c125aa6f7.

/lgtm

I'm generally wary of "bucket of helpers", but I'm willing to see where it goes.  The last three packages I saw like this took nearly six months to untangle enough to separate.
I would like to see pkg/route/routeapihelpers in case a different category of route related code arrives.
/approve
/hold
holding for package update.  Feel free to merge after.

/lgtm

/hold cancel


feature/big-endian

Tiny change, alters the endianness of the utils.compute_number() function. Current version is little-endian, update is big-endian. (so previously 1000[bin] would be 1[dec], whereas now 1000[bin] is 8[dec])

Thanks man


Question: user/app metadata

In Gotrue there are a concept of  user_metadata and app_metadata in access token.
user_metadata is custom/additional user properties that can changed by user (eg: avatar_url, address, etc)
app_metadata is custom/additional user properties that can only changed by admin (eg: roles, subscription status, etc)
Is Gothic also has this concept ?

Thanks for explanation, user Metadata is the app_metadata
I might be wrong, but I can't find admin API to update user Metadata, current update user function in admin API only update user Data
Maybe we should adding functionality to update Metadata in admin API, since only admin that has right to update user Metadata


Update node (ubuntu-20.04, arm64) docker build workflow

This PR is an attempt to fix the node (ubuntu-20.04, arm64) docker build space issues.
1.) I've made some changes so the docker images the workflow creates get cleaned up when it finishes, whether it finishes successfully or not. I did the by giving each build a unique name, and then removing those images after the build is complete. I set the cleanup to use "always()" so even if steps fail, this cleanup still runs.
2.) I've updated 'Dockerfile_Build' so it no longer copies all the source code inside the docker image. It instead just copies 'docker_build_entrypoint.sh', and then does the build inside the mounted volume. since it builds inside the mounted /data/ volume, it does not add space inside the docker image.
3.) I've added in an alternative config to run this on an ubuntu-latest runner. if the runner is switched to that, the build will use qemu. This is mainly so it is easier to switch if needed, like when someone tries to run this on their own fork and doesn't have the mac-os arm runner (like me in my fork)

One thing I do notice about not copying all files into the docker image like I was before is it seems like the build is much slower, which is unfortunate. I wonder if it was working in memory before?

Thanks for taking a look at this.
Maybe instead of the comment you can use:
${{ (github.action_repository == 'maplibre/maplibre-gl-native' && 'macos-12-arm') || 'ubuntu-latest' }}

(I hope the syntax is correct).
Then you don't have to change this file in a fork.

You could set node-build-${{ github.run_id }}-${{ github.run_attempt }} to an environment variable at the top of the workflow so you don't have to repeat it.
Otherwise LGTM.


Clarify use of installation recipes in the README

In the README, we say things like:
### powershell4

Include the `powershell4` recipe in a run list, to install PowerShell 4.0 is installed on applicable platforms. If a platform is not supported or if it already includes PowerShell 4.0, an exception will be raised.

But it looks like you just get a warning.
[2016-03-30T14:27:03+00:00] WARN: PowerShell 4.0 is not supported or already installed on this version of Windows: 6.3.9600

https://github.com/chef-cookbooks/powershell/blob/master/recipes/powershell4.rb#L52
It also looks like you just get a warning if the platform is not supported.
https://github.com/chef-cookbooks/powershell/blob/master/recipes/powershell4.rb#L56
I might simply remove the statement "If a platform is not supported or if it already includes PowerShell 4.0, an exception will be raised." I would have to see if the same logic is used by the other installation recipes.
I felt strongly enough to file an issue because exception implies failure to me, like I would have to write a guard around it. Using this recipe is much easier than doing that. ;)

I've clarified the behavior in the readme


Can I use this repo to check wallet balance and transfer some funds?

Hello:
I found there is web browser extension wallet: Suiet Wallet, and I want to know the following:
Does the wallet support tokens (like USDT) on HECO chain?
Can I use the repo (do you have NPM for Node.js?) to check wallet balance and transfer received funds to another address via different network: like receive USDT via HECO network, and send to another address via ERC20/BEP20 network?
Please advise,

Hello:
Do you have plan to support multi-chain later?
Thanks,

@zydjohnHotmail No, we don't have a plan to support multi-chain yet.

Thanks!


New C# subscriber

Description
I have created a new subscriber for c# in pubsub sample. I think this sample would be helpful for people who wants to start with asp.net core. In topic C have utilized dictionary to show different way of receiving message.
I have tested locally and it works.
I have updated the README file to the best of my knowledge but couldn't update the pictures (I hope you guys can help with that).
besides, I have added the yaml file for kubernetes deployment but couldn't test it because I don't have access to AKS.
Please let me know if I need to do anything else for this PR to be merged.

So in general we have sdk specific examples of dapr building blocks hosted with the SDKs in general. For the dotnet-sdk, you can find the examples here:
https://github.com/dapr/dotnet-sdk/tree/master/examples
For now I think adding a dotnet-sdk subscriber to this example complicates it a bit more than we'd like. We're currently discussion a larger effort that may reimplement the quickstarts using each one of the SDKs so that a dotnet developer can run the same examples native to dotnet as a python developer does, as a node developer does, and so on.

Thank you for your response, The beauty of this sample was using multiple technologies and I was excited to add C# version because I saw some questions about pub-sub in discord channel but if you guys are not interested I understand.

Please let me know if you want me to close this PR?

Hi @Arashrohani, thank you so much for this contribution.  I like it.  LGTM.  @msfussell could you please consider approving and merging?
Note these quickstarts will be moved to a repo called tutorials.  I think this is a nice addition.
You might also be interested in the new approach we have to quickstarts in the works.   This will mean that every feature and API will be done well in all major languages, including C#.  We're even showing the .NET 6 minimal API.  Could you please check this out?  https://github.com/dapr/quickstarts/tree/feature/new_quickstarts

Hi @paulyuk, Thank you.
I updated this branch with the base branch, what else should I do?
It says that I need to add sign off by author, Do you want me to do that?
Is Tests are going to run as part of pull request process or should I run them manually?
Thanks.

@paulyuk Hey Paul, I followed the instruction for DCO and updated the branch again. I'm waiting for your instruction for next step.

@Arashrohani Got it thank you much for doing the process.  Ok, I unblocked tests for you.  Let's give the tests workflow about 30 mins to see results!  If that looks good I think you'll see a looks good.

@paulyuk I updated the branch again, would you please run the tests again? I don't know what was the issue first time.

@paulyuk It seems that the order of some outputs has changed for calculator and causing the test to fail as far as I could understand from the logs.

Hi @paulyuk I was wondering what is gonna happen to this PR?

@Arashrohani Can you please pull the latest from master and rebase this?

@amulyavarote It's Done.

Can we make this .NET 6 minimal API based instead?

@davidfowl I was thinking about that also but wasn't sure when Should I do it? Now or after this branch was merged?

I would do it now.

@davidfowl Yeah It makes sense, I will upgrade to .NET 6 minimal API and push it up.

@davidfowl @amulyavarote It's been upgraded to ASP.NET Core 6 Minimal API.

@davidfowl @amulyavarote It's been upgraded to ASP.NET Core 6 Minimal API.

Can you please fix the build?

@amulyavarote Sorry about that, I just noticed that README file it's being used for testing also, I added csharp-subscriber to the makefile and update the README based on the outputs from the tests, hopefully these changes will fix it but How can I re-run the tests?

@amulyavarote Sorry about that, I just noticed that README file it's being used for testing also, I added csharp-subscriber to the makefile and update the README based on the outputs from the tests, hopefully these changes will fix it but How can I re-run the tests?

Yes! You can!

@amulyavarote Would you please let me know how? I thought that I can't do it because of this: "First-time contributors need a maintainer to approve running workflows".

@amulyavarote Would you please approve running the workflows again?

@Arashrohani I reran the workflows. Tutorials workflow is still failing. Can you please check that?
https://github.com/dapr/quickstarts/runs/6373558996?check_suite_focus=true

@amulyavarote How can I push my image to ghcr.io? I think the issue with K8s test is that my image has not been pushed to the registry (I don't think that I have access to do that myself), would you please help me on that?

@davidfowl @amulyavarote Removed the Swagger and added the missing steps in README file and tested locally with mm tool and everything passed except the K8s deployment because of the image, I tried to test that locally also but React-form was throwing exception, would you please re-run the workflows to test it?

@davidfowl Thanks, I don't know how to push csharp-subscriber image to the registry and I think that's the reason for failing tests. I can see the code for push in docker.mk file but when I run that code it just builds the images (I think it's related to some sort of access or something). @amulyavarote @paulyuk How can I push my image to the registry?

@Arashrohani Hi, Can you please rebase the changes with master and push the changes to fix the build?

@amulyavarote Hi, Sure, I will do that but as I mentioned before I don't think that fixes the build, I just checked and this is still missing: ghcr.io/dapr/samples/pubsub-csharp-subscriber:latest
this image has been used in csharp-subscriber.yaml and imagePullPolicy is always and I think because that image doesn't exist the build can not deploy the csharp-subscriber and fails.

@amulyavarote Hi, Sure, I will do that but as I mentioned before I don't think that fixes the build, I just checked and this is still missing: ghcr.io/dapr/samples/pubsub-csharp-subscriber:latest this image has been used in csharp-subscriber.yaml and imagePullPolicy is always and I think because that image doesn't exist the build can not deploy the csharp-subscriber and fails.

Got it! Let me check!

@amulyavarote I updated the code and pushed it up, I noticed that one step "publish docker manifest" is being skipped and I guess that's why the image is missing, would you please push it up and rerun the tests?

@amulyavarote I updated the code and pushed it up, I noticed that one step "publish docker manifest" is being skipped and I guess that's why the image is missing, would you please push it up and rerun the tests?

Hey @Arashrohani thank you much for this contribution.  Really nice.  You nailed it - we need your change to the makefile to get into the branch for the upload to happen.  This is a bit of a chicken and an egg situation.  Technically pushing the makefile change first as a separate PR is the ideal way to go.  But given low risk nature, I'm happy to just merge this and then we "test in prod".  You're cool to work with us to get tests passing after?

@paulyuk Hi Paul, Thanks, Yes, Of course I will work with you guys to get tests passing.


Investigate modification to LatestFileInPath Function to improve reliability

Consider indexing the log files in the folder and pick out the actual latest, not just the latest write time in the unexpected event that a log entry is written to an earlier log file.  This might be especially important if we rewrite the log file to correct errors.

This is related the to aaLogPathManager discussion in #26. The idea is aaLogPathManager would index the files and, as mentioned above, potentially take corrective action.


Unable to find a specification for SlideMenuControllerSwift

I just upgrade Xcode to 8.0 and my project to Swift3.0
After that i remove all pod and reinstall all of them again.
But i have a problem to run 'pod install', i get the following error:
Analyzing dependencies
[!] Unable to find a specification for SlideMenuControllerSwift
I have no idea to fix it. Please help me.

Is there any other framework okay?


[SPARK-24940][SQL] Use IntegerLiteral in ResolveCoalesceHints

What changes were proposed in this pull request?
Follow up to fix an unmerged review comment.
How was this patch tested?
Unit test ResolveHintsSuite.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

ok to test

Test build #94258 has started for PR 21998 at commit 8f0c578.

Test build #94258 has finished for PR 21998 at commit 8f0c578.

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

Merged build finished. Test PASSed.

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

LGTM


[Key Vault] Add Samples for ImportCertificate

Create samples for ImportCertificate and improve docs. When importing a PFX file, the user does not need to specify the content type. With a PEM file, they do. This currently isn't clearly documented in Go and lead to customer confusion in this issue.
Want to fix the gap in our documentation and create samples for uploading both file types.

Duplicate of #22475


[DoNotMerge]Network may release sdk generation

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and 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

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.

adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

/azp run automation - sdk

Can one of the admins verify this patch?

There seems to be a lot of Lint errors for this PR,  some are related to the new Lint rule we added in May. @number213 , could you help to address those issues?

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk


[ARCTIC-1332] Fix a ScheduledExecutorService with coolPoolSize = 0 consumes 100% CPU

Why are the changes needed?

fix #1332
Brief change log

do not create a ScheduledExecutorService with coolPoolSize = 0

How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[x] Add screenshots for manual tests if appropriate

[ ] Run test locally before making a pull request

Documentation

Does this pull request introduces a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

After fixed, CPU usage is normal


Add fix for inlining slices

See https://github.com/rdaly525/coreir/issues/895 for more context
This adds a step in the inlining process to insert intermediate wires for slice selects so the current inlining implementation works.  There might be alternative solutions to this, but hopefully this fixes the functionality bug with minimal changes to the code.  I played around with some other ideas but it seems pretty complicated to get this right in the actual inlining logic (need to map from indices of one slice to indices of another).  I think this gets us a baseline working version with the least amount of code, but may introduce some overhead in the output (extra wire instances), but I think this might be better than blasting the slices into individual bits.

Unfortunately it is not just inline that is affected, but any code that uses the "addPassthrough" method. Grep-ing through the source code it looks like at least 8 passes use this function, so we need to a more general solution. Also, if addPassthrough is safe for slices, then inlining is safe for slices.
Similar to your code, we can do a similar tactic of adding coreir.wire during the PTTraverse function (used by addPassthrough). Basically in the first line of that function, when the other wireable is a slice, stamp out a coreir.wire instance.
Your code does an additional optimization where it does not add "unneeded" coreir.wire instances. This should be easy to detect and cleanup in a separate pass. In fact that pass would be analogous to copy propagation and would be generally useful as a normal optimization pass.
Let me know if this makes sense. Apologies for not being clearer in the issue.


Add ticking session code for Unit 2, Session 2, First Program

Sure but first run the code and then create a pr


tf.reduce_sum(a) is slow

For large vectors, cost of tf.reduce_sum(a) is dominated by cost of range that's called to construct list of reduction indices.
Summing up 10k elements 1500 times, I get about 0.8 seconds spent in _sum while 2 seconds is spent in range. Also, transferring all those indices to sum probably slows things down. This only happens when static graph optimizations don't apply. Will update with reproducible benchmark in a bit

Here's the benchmark. I seem to have done something strange to my environment because in official version, reduce_sum is pretty fast, despite seemingly explicitly constructing 1 billion indices explicitly, hence closing this. Also, strangely enough, using @yuanbyu persistent tensors gives the result 2x faster
          numpy time 0.37053
             tf time 0.37053
       tf persistent 0.18634


What's the number of subchilds？

As far as I know，the number of childnode in radix tree is 2^n.But I look this code,the number is not limited? Can you explain it??? ^_^

Sorry for the delay, @kaikai-sk , but I was enjoying a vacation 🌴 🌞
I'll get back to you asap, but it's been awhile looking at this code, so I don't know the answer off the top of my head.

I wanna wait for your answer . What's more,in linux kernel , the number of childnode in radix tree is 2^n

Yeah, it definitely looks like that could be the case, although I'd have to dig some more to see if that's truly what happens. Specifically, it seems like this:
https://github.com/thegedge/radix-tree/blob/e20439a1fdedc9a93204c56beff4c48815912e10/src/main/java/ca/gedge/radixtree/RadixTree.java#L397-L415
could result in an arbitrary number of children.


Inherited method appears twice in allMethods

Given the following class structure:
protocol BaseProtocol {
	var variable: Int { get }
	func doSomething()
}

class BaseClass: BaseProtocol {
	var variable: Int = 0
	func doSomething() {}
}

protocol ExtendedProtocol: BaseClass {
	var externdedVariable: Int { get }
	func extendedDoSomething()
}

class ExtendedClass: BaseClass, ExtendedProtocol {
	var externdedVariable: Int = 0
	func extendedDoSomething() { }
}

When navigating the allMethods property of ExtendedProtocol, doSomething will appear twice, once with definedInTypeName = BaseClass and once with definedInTypeName = BaseProtocol.
Interestingly, allVariables only contains variable once.
Example stencil file:
{% for protocol in types.protocols where protocol.name|hasSuffix:"ExtendedProtocol"  %}
class Mock{{ protocol.name|replace:"Protocol","" }}: {{protocol.name}} {

{% for variable in protocol.allVariables|!definedInExtension %}
    {{ variable.name }} coming from {{variable.definedInTypeName}}
{% endfor %}

{% for method in protocol.allMethods|!definedInExtension %}
    {{ method.name }} coming from {{method.definedInTypeName}}
{% endfor %}

}

{% endfor %}

This is particularly a problem when trying to generate mocks. Using the current AutoMockable.stencil, the generated mock will not compile, because all the helpers generated for doSomething will be created twice.

We've faced the same issue when trying to build project on macOS 10.15.4.
However everything is ok on macOS 10.15 (19A602). It seems something has been broken in the latest macOS releases.

I think the reason is that definedInTypeName is not excluded from equality and as such when we flatten methods for allMethods it doesn't treat those methods as same, this was change made by @Antondomashnev a long time ago: https://github.com/krzysztofzablocki/Sourcery/commit/c73f18dbf125cb9c40b7fb33d00126f486239dc1
I honestly am not sure whether this should be excluded from equality or we need to deal with this other way (as I never relied on this defined variables), thoughts @Antondomashnev @ilyapuchka ?

I don't mind it being this way. My only problem is that the Mock template becomes useless in this case. I've even tried filtering out duplicates somehow, but was unsuccessful.

Any updates on this? 😞

Can we get an update on this?
protocol Foo {
    func doSomething()
}

protocol Foo2 {
    func doSomething()
}

protocol Foo3: Foo, Foo2 {
    func doSomethingElse()
}

The AutoMockable template still fails with this.

@Uncpy Nope, I wasn't able to fix it 😞
As a workaround, I removed:

Protocols requiring class conformance, so replace protocol SomeProtocol: SomeClass with protocol SomeProtocol: SomeClassProtocol.
Methods that appear in multiple protocols and instead move it to a parent protocol.

// Before:
protocol A {
    func doSomething()
    func doSomethingA()
}
protocol B {
    func doSomething()
    func doSomethingB()
}

// After:

protocol C { doSomething() }
protocol A: C { doSomethingA() }
protocol B: C { doSomethingB() }

I've only had these problems in a single instance every time, so it was easy to fix.
If the codebase gets more complex and this starts to appear more, I'm considering forking Sourcery...

I would gladly make a PR fixing this, but I would like some opinions from maintainers, if this is a bug or a "feature".
@krzysztofzablocki @Antondomashnev @ilyapuchka

I think just excluding definedInTypeName from equality will fix the issue but I'm not sure if that's what we want, @ilyapuchka @Antondomashnev what do you think?

Well, either way, the fact that allVariables and allMethods behave differently makes me think it's a bug.

Another possible solution I can see is to merge the methods from the two protocols, and make definedInTypeName an array. But that solution can get complicated real quick, because it will affect other fields as well. So unless there is need for it, it's probably not worth the time investment.

@lordzsolt yeah it's weird as both aren't excluding the definedInTypeName, how is sourcery registering the variables vs methods there?

Given these protocols:
protocol A {
    var value: Int { get }
    func doSomething()
}

protocol B {
    var value: Int { get }
    func doSomething()
}

protocol C: A, B {
    func somethingSomething()
}

This is the output:
    Variable: name = value, typeName = Int, isComputed = false, isStatic = false, readAccess = internal, writeAccess = , isMutable = false, defaultValue = nil, annotations = [:], attributes = [:], isFinal = false, isLazy = false, definedInTypeName = Optional(A), actualDefinedInTypeName = Optional(A)

    Method: name = somethingSomething(), selectorName = somethingSomething, parameters = [], returnTypeName = Void, `throws` = false, `rethrows` = false, accessLevel = internal, isStatic = false, isClass = false, isFailableInitializer = false, annotations = [:], definedInTypeName = Optional(C), attributes = [:]
    Method: name = doSomething(), selectorName = doSomething, parameters = [], returnTypeName = Void, `throws` = false, `rethrows` = false, accessLevel = internal, isStatic = false, isClass = false, isFailableInitializer = false, annotations = [:], definedInTypeName = Optional(A), attributes = [:]
    Method: name = doSomething(), selectorName = doSomething, parameters = [], returnTypeName = Void, `throws` = false, `rethrows` = false, accessLevel = internal, isStatic = false, isClass = false, isFailableInitializer = false, annotations = [:], definedInTypeName = Optional(B), attributes = [:]

As you can see, value with definedInTypeName = Optional(B)is not even present, whiledoSomething` is present twice.

@lordzsolt definitely something is off, I'll take a look today, can you tell me what behaviour would you expect here? it definitely should be consistent between variable and method but I wonder what the right behaviour is, I personally never used definedInTypeName nor do I see any common need for that so I'd opt in for skipping equality on those properties as for 99% of codegen needs it doesn't matter and it's same method

Yeah, I agree with you. For codegen reasons, there should be a single method.
Otherwise, everything will be generated twice and won't compile, because "redeclaration" errors.

@lordzsolt fixed this on master by matching the behaviours of allVariables, the reason there was a difference was because we filter out the variables composition when we flatten it, we do same for methods now

@krzysztofzablocki Could you please describe how to get old behavior with allMethods in 1.0.3?

@sochalewski why do you want to have duplicates? you should be able to get that by walking the inheritance / conformance hierachy

@krzysztofzablocki It seems that flattening sometimes works incorrectly. I've got a template to auto generate mocks which uses allMethods and most of mocks are generated correctly, but some of them not.
In example:
protocol RemoteUrlOpening: AnyObject {
  func open(_ url: URL)
}

//sourcery: AutoMockable
protocol UrlOpening: RemoteUrlOpening {
  func open(
    _ url: URL,
    options: [UIApplication.OpenExternalURLOptionsKey: Any],
    completionHandler completion: ((Bool) -> Void)?
  )
}

This treats open(_ url: URL) from the inherited protocol as a duplicate while it's not. Thus only `open(_:options: completionHandler:) appears in the generated mock. In 1.0.2 both functions appear in the mock.
On the other hand the following works properly even in 1.0.3:
//sourcery: AutoMockable
protocol UrlOpening: RemoteUrlOpening {
  func open(_ url: URL)
  func open(
    _ url: URL,
    options: [UIApplication.OpenExternalURLOptionsKey: Any],
    completionHandler completion: ((Bool) -> Void)?
  )
}

I believe filter in flattenAll(_:filter) should compare not only function's name, but its parameters too.

looks like we should use selectorName for flattening functions?

@sochalewski I added a test with the code you provided and the AST sourcery parses is correct https://github.com/krzysztofzablocki/Sourcery/commit/94d85c8ad99192b42dda2814470da6225f51b176
same when copying your code and running basic template:
// code
protocol RemoteUrlOpening {
  func open(_ url: URL)
}

protocol UrlOpening: RemoteUrlOpening {
  func open(
    _ url: URL,
    options: [UIApplication.OpenExternalURLOptionsKey: Any],
    completionHandler completion: ((Bool) -> Void)?
  )
}

// template
{% for method in type.UrlOpening.allMethods %}
{{ method.name }}
{% endfor %}

// output
open(
    _ url: URL,
    options: [UIApplication.OpenExternalURLOptionsKey: Any],
    completionHandler completion: ((Bool) -> Void)?
  )
open(_ url: URL)

Can you provide a failing spec or a running example?

@krzysztofzablocki As I mentioned I use allMethods combined with !definedInExtension.
protocol UrlOpening {
  func open(_ url: URL)
  func open(
    _ url: URL,
    options: [UIApplication.OpenExternalURLOptionsKey: Any],
    completionHandler completion: ((Bool) -> Void)?
  )
}

extension UrlOpening {
    func open(_ url: URL) {
        open(url, options: [:], completionHandler: nil)
    }
}

Let's go with the example above.
allMethods
// template
{% for method in type.UrlOpening.allMethods %}
{{ method.name }}
{% endfor %}

// output
open(_ url: URL)
open(
    _ url: URL,
    options: [UIApplication.OpenExternalURLOptionsKey: Any],
    completionHandler completion: ((Bool) -> Void)?
)
open(_ url: URL)

The above outputs open(_ url: URL) twice as the first comes from the protocol and the second from the extension.
allMethods|!definedInExtension, ex. 1
// template
{% for method in type.UrlOpening2.allMethods|!definedInExtension %}
{{ method.name }}
{% endfor %}

//output
open(_ url: URL)
open(
    _ url: URL,
    options: [UIApplication.OpenExternalURLOptionsKey: Any],
    completionHandler completion: ((Bool) -> Void)?
)

This works correctly. But going with a bit more complicated example (below) makes it wrong.
allMethods|!definedInExtension, ex. 2
//code
protocol RemoteUrlOpening {
  func open(_ url: URL)
}

protocol UrlOpening: RemoteUrlOpening {
  func open(
    _ url: URL,
    options: [UIApplication.OpenExternalURLOptionsKey: Any],
    completionHandler completion: ((Bool) -> Void)?
  )
}

extension UrlOpening {
  func open(_ url: URL) {
    open(url, options: [:], completionHandler: nil)
  }
}

// template
{% for method in type.UrlOpening.allMethods|!definedInExtension %}
{{ method.name }}
{% endfor %}

//output
open(
    _ url: URL,
    options: [UIApplication.OpenExternalURLOptionsKey: Any],
    completionHandler completion: ((Bool) -> Void)?
)

This outputs only open(_:options:completionHandler:) missing open(_:) at all.
Hope this is helpful to resolve the issue I encountered.

this isn't issue with method flattening but with type extension, I don't think that in the first example it should list 3 methods because the type really has 2 methods, one just have default implementation and from Sourcery standpoint that shouldn't be treated as separate method

@sochalewski take a look at https://github.com/krzysztofzablocki/Sourcery/pull/897 and the tests I added there, let me know if the changes make sense

@krzysztofzablocki I've checked changes from #897 out and it works well ❤


Headers get truncated on ':' character

On the Headers tab for the response view, if the head value contains a : character, the header gets truncated.

What app version are you using @eztinkerdreams. I remember fixing this a long time ago and can't seem to reproduce it on v5.2.0.

Ohh, nvm. I fixed it for the Request Headers tab, not Response headers. Got it.

Just noticed that this issue is on the wrong repository. I've moved it to https://github.com/getinsomnia/insomnia/issues/293


Persist user entered credentials

Ideally we shouldn't be using hard-coded credentials. These should be removed.
Instead once the user enter credentials for the first time, these should be persisted (on a per environment basis).

Completed and available in the latest version.


update back and pylint version

Description
Please summarise the changes.
Related issues
Please mention any github issues addressed by this PR.
Checklist

[ ] I have performed a self-review of my code.
[ ] I have commented hard-to-understand parts of my code.
[ ] I have made corresponding changes to the public API documentation.
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] I have updated the changelog with any user-facing changes.

would like to merge this without the tests working.


feat: icon rotate API

https://ant.design/components/icon/

rotate
Rotate degrees (added in 3.13.0, not working in IE9)
number
-

block collapse style


Add missing period in end of comment (NFC)

What's in this pull request?

Resolved bug number: (SR-)

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    OS X platform               | @swift-ci Please test OS X platform
    Linux platform              | @swift-ci Please test Linux platform

Note: Only members of the Apple organization can trigger swift-ci.

Oh! I thought it was just supposed to "represent a compound."
:-)
On Tue, Mar 29, 2016 at 9:13 PM, Chris Lattner<EMAIL_ADDRESS>wrote:

The comment is missing more than a period :-) It looks like someone forgot
to finish the sentence.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/apple/swift/pull/1926#issuecomment-203032348

Should I close this pull request or finish the sentence? e.g. "Represents a compound declaration name."

@danra Please finish the sentence, and squash the commits.  Thanks.

@tkremenek done


I can't scan NFC by asus nexus 7

I use the NFCForms in my project. I try to Installer the app for two phones. one is sony Z5, the other is nexus 7. I can use fine in sony Z5, but I can't scan in nexus 7.
nexus 7 doesn't happen in OnPause(), Why?
I try to find NFCtools, I can scan MifareClassic card in sony Z5 and nexus 7.

protected override void OnResume ()  { 			base.OnResume (); 			if (NFCdevice != null) { 				var intent = new Intent (this, GetType ()).AddFlags (ActivityFlags.SingleTop); 				NFCdevice.EnableForegroundDispatch 				( 					this, 					PendingIntent.GetActivity (this, 0, intent, 0), 					new[] { new IntentFilter (NfcAdapter.ActionTechDiscovered) }, 					new String[][] {new string[] { 							NFCTechs.Ndef, 						}, 						new string[] { 							NFCTechs.MifareClassic, 						}, 					} 				); 			} }
NfcAdapter.ActionTechDiscovered > NfcAdapter.ActionTagDiscovered
NfcAdapter.ActionTechDiscovered
Intent to start an activity when a tag is discovered and activities are registered for the specific technologies on the tag.
To receive this intent an activity must include an intent filter for this action and specify the desired tech types in a manifest meta-data entry. Here is an example manfiest entry:
NfcAdapter.ActionTagDiscovered
Intent to start an activity when a tag is discovered.
This intent will not be started when a tag is discovered if any activities respond to NfcAdapter.ActionNdefDiscovered or NfcAdapter.ActionTechDiscovered for the current tag.


Lifecycle diagram incorrect - onGameResize called before onLoad

Current bug behavior
onGameResize is called before onLoad.  This is different from what the lifecycle diagram shows.
Expected behavior
onGameResize is called before onLoad.  This is important to know as the documentation implies that you can use onLoad to initialize components
Steps to reproduce
Add the following code to any with FlameGame class:
  @override
  void onMount() {
    debugPrint('onMount');
    super.onMount();
  }

  @override
  void onGameResize(Vector2 size) {
    debugPrint('onGameResize');
    super.onGameResize(size);
  }

  @override
  Future<void>? onLoad() async {
    debugPrint('onLoad');
    await super.onLoad();
  }

Observe the sequence of output:
Restarted application in 344ms.
flutter: onGameResize
flutter: onLoad
flutter: onMount

Flutter doctor output
❯ flutter doctor -v
[✓] Flutter (Channel stable, 3.13.0, on macOS 13.4.1 22F770820d darwin-arm64, locale en-US)
    • Flutter version 3.13.0 on channel stable at /Users/john/Projects/flutter/flutter
    • Upstream repository git@github.com:flutter/flutter.git
    • Framework revision efbf63d9c6 (9 days ago), 2023-08-15 21:05:06 -0500
    • Engine revision 1ac611c64e
    • Dart version 3.1.0
    • DevTools version 2.25.0

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.1)
    • Android SDK at /Users/john/Library/Android/sdk
    • Platform android-33, build-tools 33.0.1
    • ANDROID_HOME = /Users/john/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14E300c
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)

[✓] VS Code (version 1.81.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.70.0

[✓] Connected device (3 available)
    • iPhone 14 Pro (mobile) • 5138CE7C-4E9A-4F03-90C7-5A0C7911ED73 • ios            •
      com.apple.CoreSimulator.SimRuntime.iOS-16-4 (simulator)
    • macOS (desktop)        • macos                                • darwin-arm64   • macOS 13.4.1 22F770820d
      darwin-arm64
    • Chrome (web)           • chrome                               • web-javascript • Google Chrome 115.0.5790.114

[✓] Network resources
    • All expected network resources are available.

• No issues found!

More environment information

Flame version: ^1.8.1
Platform affected: ios
Platform version affected: ios 14

Log information
None.
More information
I'd be happy to update the lifecycle diagram.

True, good catch!
That is actually the Component lifecycle. So the correct lifecycle (which is also verified in our tests) is:
          'onGameResize',
          'onLoad',
          'onMount',
          'update',
          'render',
          'onRemove',
          'onGameResize',
          'onMount',
          'update',
          'render',

I'd be happy to update the lifecycle diagram.

I'll assign you to the issue. :)

Hey @spydon  I changed the diagram, and I thought I would write a simple program to validate the call sequence.  What I'm seeing is the following:
          'onGameResize',
          'onLoad',
          'onMount',
          'render',
          'update',
          'onGameResize',
          'onRemove',
          'onLoad',
          'onMount',
          'render',
          'update',

This doesn't align with what you describe above.  If I create a FlameGame then attach it to a GameWidget, then  attach it to a different GameWidget I see the following:

I always see get a render before the first update (your order is reversed)
I get an onLoad again after re-attaching the FlameGame (you don't mention that above)
I see onRemove after onGameRestart and before onLoad after re-attaching, but only if I do await super.onLoad() in the onLoad callback (seems like something that needs to be documented to avoid resource leaks and incorrect behavior)

I'm concerned that I'm documenting unexpected or incorrect behavior.  I completely respect that I may also be doing something wrong.
You can check out the sources for jlyonsmith/flame_test to see how I tested.

Are all there cases only happening when you attach it to a new GameWidget and not before?

That sounds really weird, especially if it only happens after attaching to a new GameWidget.
Are you completely sure that it is the same game instance and not a new one?
Is this on main? Because there was some bugs regarding onRemove that has been fixed.

I was hoping to dash this one off quickly, but it seems like this is going to require a little more research.  Briefly, my answers to the above are:

Yes, I'm attaching a new GameWidget
It's a new instance; I'll update the test code to detach and re-attach the same instance
It's on the 1.8.2 release; I'll update my sample to use the tip of main.

The reversed update and render is odd though.  I'll set breakpoints and see if I can get to the bottom of that one.

OK, although I still see the render & update and onGameResize and onLoad event reversals, I'm assuming those are by design.  I misread my debug output and the onRemove appears to be in the correct place, before the next  onGameResize event.   Submitting an PR with the documentation updates.


Add Support remote audio microfon?

This is built-in feature which is available in Pro editions or higher.
Duplicate of #74.

Yes, but any way to wrap this feature for a Win 7 Home premium edition?


Nodepool controller collects nodes of all clusters instead of current one

What kind of request is this (question/bug/enhancement/feature request):
Bug/Enhancement
Steps to reproduce (least amount of steps as possible):

Deploy Rancher
Provision user cluster

Result:
Expected: nodepool controller retrieves nodes (.management.cattle.io) only from cluster namespace
Actual: nodepool controller retrieves all the nodes every time and then filter out by namespace
Related code: https://github.com/rancher/rancher/blob/v2.2.3/pkg/controllers/management/nodepool/nodepool.go#L171
Other details that may be helpful:
In case of many nodes (.management.cattle.io) it takes too long time to retrieve all nodes, so the controller is really slow.
In our case we have >120 clusters and >1300 nodes total. So k8s request lasts times longer (2-5 times difference) for non-namespaced request versus namespaced:
$ time kubectl get nodes.management.cattle.io --all-namespaces >/dev/null

real	0m2.627s
user	0m0.360s
sys	0m0.300s
$ time kubectl get nodes.management.cattle.io -n c-zxgw7 > /dev/null

real	0m0.653s
user	0m0.148s
sys	0m0.107s

Changing to namespaced list could help to support Rancher 2.4 Scaling Targets
Environment information

Rancher version 2.2.3

Local cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported): Hosted
Machine type (cloud/VM/metal) and specifications (CPU/memory): Baremetal

5 etcd nodes on baremetal 20 Cores / 64GB RAM / 480GB SSD / CentOS 7.6
5 controlplane nodes on baremetal 20 Cores / 64GB RAM / 480 SSD / CentOS 7.6
7 worker nodes on baremetal 20 Cores / 64GB RAM / 480 SSD / CentOS 7.6

Kubernetes version (use kubectl version):

Client Version: version.Info{Major:"1", Minor:"16", GitVersion:"v1.16.2", GitCommit:"c97fe5036ef3df2967d086711e6c0c405941e14b", GitTreeState:"clean", BuildDate:"2019-10-15T23:41:55Z", GoVersion:"go1.12.10", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"10", GitVersion:"v1.10.3", GitCommit:"2bba0127d85d5a46ab4b778548be28623b32d0b0", GitTreeState:"clean", BuildDate:"2018-05-21T09:05:37Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"linux/amd64"}

Docker version (use docker version) (local cluster):

Client:
 Version:      17.03.2-ce
 API version:  1.27
 Go version:   go1.7.5
 Git commit:   f5ec1e2
 Built:        Tue Jun 27 02:21:36 2017
 OS/Arch:      linux/amd64

Server:
 Version:      17.03.2-ce
 API version:  1.27 (minimum version 1.12)
 Go version:   go1.7.5
 Git commit:   f5ec1e2
 Built:        Tue Jun 27 02:21:36 2017
 OS/Arch:      linux/amd64
 Experimental: false

I've submitted 2 PRs to make List request Namespaced.
It affects 3 repos: rancher/norman, rancher/types (generated types), rancher/rancher.

Depends on https://github.com/rancher/rancher/issues/23934


Swipe to open HA sidebar changes tabs

In the ios HA app, swiping right over the left edge to reveal the HA app sidebar, Lovelace tabs change beneath it. I don't remember this being an issue before.
HA 2024.7.2, Swipe Navigation 1.13.3, HA app 2024.6.

Can confirm.

As stated in many places such as Reddit, HA Forum, GH issue, I think Home Assistant stopped supporting the gesture long ago. Nothing I can do here unfortunately.


[HackerNews] Fix tag containing comment

Caution: this is untested because I have trouble running the docker command, but it's minor

it already broken from before.
your suggestion suffers the same fate:
Details
Type: ErrorException
Code: 0
Message: Attempt to read property "innertext" on null
File: bridges/HackerNewsUserThreadsBridge.php
Line: 43

So, I don't know how to fix this; it now "works on my machine" (but I know how bad this sentence is)
Any help on what I can do to have a look at that ?

I dug into this a bit and it looks like the .innertext in line 43 is correct in general.
Is it possible that the innertext might be empty for some comments? If the comment is an image for example? I'm not sure of the capabilities of the thread board.
Can you add an error handling if the innertext of the comment is empty, then just set it to "" or something?
if (empty($element->find('span[class*="commtext"]', 0)->innertext))

No, HackerNews is a text-only message board. Moreove the error seemed to be related to the containing div being null. I put a null-check on it.
The build is broken because of #4310, fixed in #4311


RuntimeWarning: invalid value encountered in log   targets_dw = np.log(gt_widths / ex_widths) Command terminated by signal 11

i use my own datasets replace the voc2007 and have some issue. Can you please suggest solutions?
here is the log.
##`+ echo Logging output to experiments/logs/vgg16_voc_2007_trainval__vgg16.txt.2017-05-26_14-23-40
Logging output to experiments/logs/vgg16_voc_2007_trainval__vgg16.txt.2017-05-26_14-23-40

set +x
'[' '!' -f output/vgg16/voc_2007_trainval/default/vgg16_faster_rcnn_iter_70000.ckpt.index ']'
[[ ! -z '' ]]
CUDA_VISIBLE_DEVICES=0
time python ./tools/trainval_net.py --weight data/imagenet_weights/vgg16.ckpt --imdb voc_2007_trainval --imdbval voc_2007_test --iters 70000 --cfg experiments/cfgs/vgg16.yml --net vgg16 --set ANCHOR_SCALES '[8,16,32]' ANCHOR_RATIOS '[0.5,1,2]' TRAIN.STEPSIZE 50000
Called with args:
Namespace(cfg_file='experiments/cfgs/vgg16.yml', imdb_name='voc_2007_trainval', imdbval_name='voc_2007_test', max_iters=70000, net='vgg16', set_cfgs=['ANCHOR_SCALES', '[8,16,32]', 'ANCHOR_RATIOS', '[0.5,1,2]', 'TRAIN.STEPSIZE', '50000'], tag=None, weight='data/imagenet_weights/vgg16.ckpt')
Using config:
{'ANCHOR_RATIOS': [0.5, 1, 2],
'ANCHOR_SCALES': [8, 16, 32],
'DATA_DIR': '/media/y/B0AAA15CAAA11FB8/linux/tf-faster-rcnn/data',
'DEDUP_BOXES': 0.0625,
'EPS': 1e-14,
'EXP_DIR': 'vgg16',
'GPU_ID': 0,
'MATLAB': 'matlab',
'PIXEL_MEANS': array([[[ 102.9801,  115.9465,  122.7717]]]),
'POOLING_MODE': 'crop',
'POOLING_SIZE': 7,
'RESNET': {'BN_TRAIN': False, 'FIXED_BLOCKS': 1, 'MAX_POOL': False},
'RNG_SEED': 3,
'ROOT_DIR': '/media/y/B0AAA15CAAA11FB8/linux/tf-faster-rcnn',
'TEST': {'BBOX_REG': True,
'HAS_RPN': True,
'MAX_SIZE': 1000,
'MODE': 'nms',
'NMS': 0.3,
'PROPOSAL_METHOD': 'gt',
'RPN_NMS_THRESH': 0.7,
'RPN_POST_NMS_TOP_N': 300,
'RPN_PRE_NMS_TOP_N': 6000,
'RPN_TOP_N': 5000,
'SCALES': [600],
'SVM': False},
'TRAIN': {'ASPECT_GROUPING': False,
'BATCH_SIZE': 256,
'BBOX_INSIDE_WEIGHTS': [1.0, 1.0, 1.0, 1.0],
'BBOX_NORMALIZE_MEANS': [0.0, 0.0, 0.0, 0.0],
'BBOX_NORMALIZE_STDS': [0.1, 0.1, 0.2, 0.2],
'BBOX_NORMALIZE_TARGETS': True,
'BBOX_NORMALIZE_TARGETS_PRECOMPUTED': True,
'BBOX_REG': True,
'BBOX_THRESH': 0.5,
'BG_THRESH_HI': 0.5,
'BG_THRESH_LO': 0.0,
'BIAS_DECAY': False,
'DISPLAY': 20,
'DOUBLE_BIAS': True,
'FG_FRACTION': 0.25,
'FG_THRESH': 0.5,
'GAMMA': 0.1,
'HAS_RPN': True,
'IMS_PER_BATCH': 1,
'LEARNING_RATE': 0.001,
'MAX_SIZE': 1000,
'MOMENTUM': 0.9,
'PROPOSAL_METHOD': 'gt',
'RPN_BATCHSIZE': 256,
'RPN_BBOX_INSIDE_WEIGHTS': [1.0, 1.0, 1.0, 1.0],
'RPN_CLOBBER_POSITIVES': False,
'RPN_FG_FRACTION': 0.5,
'RPN_NEGATIVE_OVERLAP': 0.3,
'RPN_NMS_THRESH': 0.7,
'RPN_POSITIVE_OVERLAP': 0.7,
'RPN_POSITIVE_WEIGHT': -1.0,
'RPN_POST_NMS_TOP_N': 2000,
'RPN_PRE_NMS_TOP_N': 12000,
'SCALES': [600],
'SNAPSHOT_ITERS': 5000,
'SNAPSHOT_KEPT': 3,
'SNAPSHOT_PREFIX': 'vgg16_faster_rcnn',
'STEPSIZE': 50000,
'SUMMARY_INTERVAL': 180,
'TRUNCATED': False,
'USE_ALL_GT': True,
'USE_FLIPPED': True,
'USE_GT': False,
'WEIGHT_DECAY': 0.0005},
'USE_GPU_NMS': False}
Loaded dataset voc_2007_trainval for training
Set proposal method: gt
Appending horizontally-flipped training examples...
voc_2007_trainval gt roidb loaded from /media/y/B0AAA15CAAA11FB8/linux/tf-faster-rcnn/data/cache/voc_2007_trainval_gt_roidb.pkl
done
Preparing training data...
done
1528 roidb entries
Output will be saved to /media/y/B0AAA15CAAA11FB8/linux/tf-faster-rcnn/output/vgg16/voc_2007_trainval/default
TensorFlow summaries will be saved to /media/y/B0AAA15CAAA11FB8/linux/tf-faster-rcnn/tensorboard/vgg16/voc_2007_trainval/default
Loaded dataset voc_2007_test for training
Set proposal method: gt
Preparing training data...
voc_2007_test gt roidb loaded from /media/y/B0AAA15CAAA11FB8/linux/tf-faster-rcnn/data/cache/voc_2007_test_gt_roidb.pkl
done
328 validation roidb entries
Filtered 0 roidb entries: 1528 -> 1528
Filtered 0 roidb entries: 328 -> 328
2017-05-26 14:24:11.316553: W tensorflow/core/platform/cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use SSE4.1 instructions, but these are available on your machine and could speed up CPU computations.
2017-05-26 14:24:11.316569: W tensorflow/core/platform/cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use SSE4.2 instructions, but these are available on your machine and could speed up CPU computations.
2017-05-26 14:24:11.316572: W tensorflow/core/platform/cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use AVX instructions, but these are available on your machine and could speed up CPU computations.
2017-05-26 14:24:11.316575: W tensorflow/core/platform/cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use AVX2 instructions, but these are available on your machine and could speed up CPU computations.
2017-05-26 14:24:11.316577: W tensorflow/core/platform/cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use FMA instructions, but these are available on your machine and could speed up CPU computations.
Solving...
/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/gradients_impl.py:93: UserWarning: Converting sparse IndexedSlices to a dense Tensor of unknown shape. This may consume a large amount of memory.
"Converting sparse IndexedSlices to a dense Tensor of unknown shape. "
Loading initial model weights from data/imagenet_weights/vgg16.ckpt
Varibles restored: vgg_16/conv1/conv1_1/biases:0
Varibles restored: vgg_16/conv1/conv1_2/weights:0
Varibles restored: vgg_16/conv1/conv1_2/biases:0
Varibles restored: vgg_16/conv2/conv2_1/weights:0
Varibles restored: vgg_16/conv2/conv2_1/biases:0
Varibles restored: vgg_16/conv2/conv2_2/weights:0
Varibles restored: vgg_16/conv2/conv2_2/biases:0
Varibles restored: vgg_16/conv3/conv3_1/weights:0
Varibles restored: vgg_16/conv3/conv3_1/biases:0
Varibles restored: vgg_16/conv3/conv3_2/weights:0
Varibles restored: vgg_16/conv3/conv3_2/biases:0
Varibles restored: vgg_16/conv3/conv3_3/weights:0
Varibles restored: vgg_16/conv3/conv3_3/biases:0
Varibles restored: vgg_16/conv4/conv4_1/weights:0
Varibles restored: vgg_16/conv4/conv4_1/biases:0
Varibles restored: vgg_16/conv4/conv4_2/weights:0
Varibles restored: vgg_16/conv4/conv4_2/biases:0
Varibles restored: vgg_16/conv4/conv4_3/weights:0
Varibles restored: vgg_16/conv4/conv4_3/biases:0
Varibles restored: vgg_16/conv5/conv5_1/weights:0
Varibles restored: vgg_16/conv5/conv5_1/biases:0
Varibles restored: vgg_16/conv5/conv5_2/weights:0
Varibles restored: vgg_16/conv5/conv5_2/biases:0
Varibles restored: vgg_16/conv5/conv5_3/weights:0
Varibles restored: vgg_16/conv5/conv5_3/biases:0
Varibles restored: vgg_16/fc6/biases:0
Varibles restored: vgg_16/fc7/biases:0
Loaded.
Fix VGG16 layers..
/media/y/B0AAA15CAAA11FB8/linux/tf-faster-rcnn/tools/../lib/model/bbox_transform.py:26: RuntimeWarning: invalid value encountered in log
targets_dw = np.log(gt_widths / ex_widths)
Command terminated by signal 11
62.03user 5.27system 0:57.96elapsed 116%CPU (0avgtext+0avgdata 3723648maxresident)k
382896inputs+16outputs (296major+3462186minor)pagefaults 0swaps`

I also encounter the same error.
Before iteration 55, everything goes fine.
Fix VGG16 layers..
Fixed.
iter: 20 / 7000, total loss: 0.315415
 >>> rpn_loss_cls: 0.120561
 >>> rpn_loss_box: 0.016272
 >>> loss_cls: 0.137040
 >>> loss_box: 0.041542
 >>> lr: 0.001000
speed: 1.417s / iter
iter: 40 / 7000, total loss: 0.740965
 >>> rpn_loss_cls: 0.077266
 >>> rpn_loss_box: 0.005625
 >>> loss_cls: 0.416012
 >>> loss_box: 0.242062
 >>> lr: 0.001000
speed: 1.133s / iter
/notebooks/tf-faster-rcnn/tools/../lib/model/bbox_transform.py:31: RuntimeWarning: invalid value encountered in log
  targets_dh = np.log(gt_heights / ex_heights)
iter: 60 / 7000, total loss: nan
 >>> rpn_loss_cls: 0.681259
 >>> rpn_loss_box: nan
 >>> loss_cls: 2.784790
 >>> loss_box: 0.000000
 >>> lr: 0.001000
speed: 0.925s / iter

After iter=55, rpn_loss_box will become nan which caused by wrong value in bbox_transform.
lib/model/bbox_transform.py
Line20:
gt_heights = gt_rois[:, 3] - gt_rois[:, 1] + 1.0
gt_rois[:, 1] become greater than gt_rois[:, 3] and result gt_heights to negative value.
gt_rois[:, 3]  188.75
gt_rois[:, 1]  81918.8
gt_heights  -81729.0

My temporary solution is to ignore incorrect value (ymin > ymax).
Checking gt_boxes value as follows:
lib/model/train_val.py
https://github.com/endernewton/tf-faster-rcnn/blob/master/lib/model/train_val.py#L219
Line:219
      blobs = self.data_layer.forward()
	  if blobs['gt_boxes'][0][1] > blobs['gt_boxes'][0][3]:
        iter += 1
        continue

This modification could let the program running without getting 'Nan'.
Beside this temporary solution.
I start to figure out what is the cause of this error.
We could guess "ymin and ymax value" of brounding box ground truth is wrong.
However, after I exeamine my brounding box data, ymin is always smaller than ymax.

i am not sure your setting, your application.. it is hard to help. sorry

i run it again and it changed.  could you please tell me what happened?
Fix VGG16 layers..
Traceback (most recent call last):
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/script_ops.py", line 82, in call
ret = func(*args)
File "/media/y/B0AAA15CAAA11FB8/linux/tf-faster-rcnn/tools/../lib/layer_utils/anchor_target_layer.py", line 90, in anchor_target_layer
bbox_targets = _compute_targets(anchors, gt_boxes[argmax_overlaps, :])
File "/media/y/B0AAA15CAAA11FB8/linux/tf-faster-rcnn/tools/../lib/layer_utils/anchor_target_layer.py", line 163, in _compute_targets
return bbox_transform(ex_rois, gt_rois[:, :4]).astype(np.float32, copy=False)
File "/media/y/B0AAA15CAAA11FB8/linux/tf-faster-rcnn/tools/../lib/model/bbox_transform.py", line 26, in bbox_transform
targets_dw = np.log(gt_widths / ex_widths) if ex_widths != 0 else 0
ValueError: The truth value of an array with more than one element is ambiguous. Use a.any() or a.all()
2017-05-30 20:17:20.974332: W tensorflow/core/framework/op_kernel.cc:1152] Internal: Failed to run py callback pyfunc_2: see error log.
Traceback (most recent call last):
File "./tools/trainval_net.py", line 136, in
max_iters=args.max_iters)
File "/media/y/B0AAA15CAAA11FB8/linux/tf-faster-rcnn/tools/../lib/model/train_val.py", line 386, in train_net
sw.train_model(sess, max_iters)
File "/media/y/B0AAA15CAAA11FB8/linux/tf-faster-rcnn/tools/../lib/model/train_val.py", line 285, in train_model
self.net.train_step(sess, blobs, train_op)
File "/media/y/B0AAA15CAAA11FB8/linux/tf-faster-rcnn/tools/../lib/nets/network.py", line 374, in train_step
feed_dict=feed_dict)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 778, in run
run_metadata_ptr)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 982, in _run
feed_dict_string, options, run_metadata)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 1032, in _do_run
target_list, options, run_metadata)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 1052, in _do_call
raise type(e)(node_def, op, message)
tensorflow.python.framework.errors_impl.InternalError: Failed to run py callback pyfunc_2: see error log.
[[Node: vgg_16/anchor/PyFunc = PyFunc[Tin=[DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_INT32, DT_FLOAT, DT_INT32], Tout=[DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT], token="pyfunc_2", _device="/job:localhost/replica:0/task:0/cpu:0"](vgg_16/rpn_cls_score/BiasAdd, _recv_Placeholder_2_0, _recv_Placeholder_1_0, vgg_16/anchor/PyFunc/input_3, vgg_16/ANCHOR_default/generate_anchors, vgg_16/anchor/PyFunc/input_5)]]
Caused by op u'vgg_16/anchor/PyFunc', defined at:
File "./tools/trainval_net.py", line 136, in
max_iters=args.max_iters)
File "/media/y/B0AAA15CAAA11FB8/linux/tf-faster-rcnn/tools/../lib/model/train_val.py", line 386, in train_net
sw.train_model(sess, max_iters)
File "/media/y/B0AAA15CAAA11FB8/linux/tf-faster-rcnn/tools/../lib/model/train_val.py", line 105, in train_model
anchor_ratios=cfg.ANCHOR_RATIOS)
File "/media/y/B0AAA15CAAA11FB8/linux/tf-faster-rcnn/tools/../lib/nets/network.py", line 305, in create_architecture
rois, cls_prob, bbox_pred = self.build_network(sess, training)
File "/media/y/B0AAA15CAAA11FB8/linux/tf-faster-rcnn/tools/../lib/nets/vgg16.py", line 68, in build_network
rpn_labels = self._anchor_target_layer(rpn_cls_score, "anchor")
File "/media/y/B0AAA15CAAA11FB8/linux/tf-faster-rcnn/tools/../lib/nets/network.py", line 149, in _anchor_target_layer
[tf.float32, tf.float32, tf.float32, tf.float32])
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/script_ops.py", line 189, in py_func
input=inp, token=token, Tout=Tout, name=name)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/gen_script_ops.py", line 40, in _py_func
name=name)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/op_def_library.py", line 768, in apply_op
op_def=op_def)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 2336, in create_op
original_op=self._default_original_op, op_def=op_def)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 1228, in init
self._traceback = _extract_stack()
InternalError (see above for traceback): Failed to run py callback pyfunc_2: see error log.
[[Node: vgg_16/anchor/PyFunc = PyFunc[Tin=[DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_INT32, DT_FLOAT, DT_INT32], Tout=[DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT], token="pyfunc_2", _device="/job:localhost/replica:0/task:0/cpu:0"](vgg_16/rpn_cls_score/BiasAdd, _recv_Placeholder_2_0, _recv_Placeholder_1_0, vgg_16/anchor/PyFunc/input_3, vgg_16/ANCHOR_default/generate_anchors, vgg_16/anchor/PyFunc/input_5)]]
Command exited with non-zero status 1
18.59user 3.10system 0:24.67elapsed 87%CPU (0avgtext+0avgdata 3754036maxresident)k
659096inputs+16outputs (377major+1686078minor)pagefaults 0swaps

In the bbox_transform ,   the gt_width is odds. I alter it .  I can't ensure i am right. but it work.
cloes

I am getting a similar error. My ex_widths is coming to be nan after 100th iteration. This its giving a runtime warning and then exiting after a few more iterations. Any clues?
@HTLife

@xzy295461445, how did you alter it?  do you solve the problem?

@xzy295461445 ,  @HTLife , @abhiML  , I get the same problem with train my data , the rpn_box_loss is nan, after some research, it's because  in the file 'pascal_voc.py',  the function  '_load_pascal_annotation' has     Make pixel indexes 0-based,the code is :
x1 = float(bbox.find('xmin').text) - 1
y1 = float(bbox.find('ymin').text) - 1
x2 = float(bbox.find('xmax').text) - 1
y2 = float(bbox.find('ymax').text) - 1
but if your data  is not based 1, such as my data is based 0, then it will get -1 in the data, may be you can try to delete the -1 operation,hope helpful!

@xzy295461445 how did you alter it?Did you have solved this problem?

@VisintZJ  Can you train with the V0C datasets?

@xzy295461445  Yes, there is no question when I train with the VOC datasets

When I make the xml file of my own datasets, the width and height is contrary、

@xzy295461445 Thank you! I solve my problem after checking my training data sets and I found the reason——there are some wrong  data in my data. :(

@VisintZJ Hi there, I met this problem too, could you tell me how to solve this?

It is perhaps due to the errors of "bbox" coodinates ( x < 0 or x > img_width ) in your Annotations. (At least for my case)

if your dataset's bbox xmin = 0 or ymin = 0, you should change code in pascal_voc.py
# x1 = float(bbox.find('xmin').text) - 1
# y1 = float(bbox.find('ymin').text) - 1
# x2 = float(bbox.find('xmax').text) - 1
# y2 = float(bbox.find('ymax').text) - 1
to
# x1 = float(bbox.find('xmin').text)
# y1 = float(bbox.find('ymin').text)
# x2 = float(bbox.find('xmax').text)
# y2 = float(bbox.find('ymax').text)
if your dataset's bbox xmax= width or ymin = height ,you should change code in imdb.py
# boxes[:, 0] = widths[i] - oldx2 - 1
# boxes[:, 2] = widths[i] - oldx1 - 1
to
boxes[:, 0] = widths[i] - oldx2
boxes[:, 2] = widths[i] - oldx1
if your dataset's bbox xmax > width or ymin > height, you should delete it or relabel it

If you have checked the xmin ymin xmax ymax and ensure that xmin>0 and xmax<width, ymin>0 and ymax<height, but the problem is still there. Maybe you can try delete the file in /data/chache and rerun the code.


Delete more unnecessary stuff and add render preview

Testing out adding a render preview Github actions (if you want it here).
Also deleting some extra files you shouldn't need here.

This addresses  issue #8


KCC Folder

Hello.
This installation wiki says:
Move kindlegen.exe from Kindle Previewer into the same folder as KCC.
But when I download the installer, I don't see the folder. It's just the KCC app that opens. Where is the KCC folder?
For reference I am on windows and ran this installer.
https://github.com/ciromattia/kcc/releases/download/v5.6.2/kcc_5.6.2.exe

Updated the wiki, let me know if it works: https://github.com/ciromattia/kcc/wiki/Installation#windows-1
Also please try 5.6.3


add --no-ansi optional parameter

This PR adds an option to run the docker compose plugin with --no-ansi flag enabled. This is needed because sometimes ansi enabled outputs mess up the step timing in Buildkite's UI which ends up showing the timing in the wrong steps.

Thanks @gugahoi for the great PR!


Convert blueprints from using dep to go modules

Resolves paveroad-io/roadctl#115
Initial commit for datamgr - Makefile go.mod go.sum

Force pushed branch to fix misspelling of datamgr in commit message

Forget this branch, many template changes missing


pythonPackages.arelle: 2017-08-24 -> 18.3

Motivation for this change
Replaces #50269
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

cc @roberth

@GrahamcOfBorg build python3.pkgs.arelle

Success on x86_64-darwin (full log)
Attempted: python3.pkgs.arelle
Partial log (click to expand)
writing manifest file 'Arelle.egg-info/SOURCES.txt'
running build_ext

----------------------------------------------------------------------
Ran 0 tests in 0.000s

OK
see log in c:\temp\testLog.txt
pytestcachePhase
/nix/store/jvqp15hrs6a3x5gp7n6wbw07qb71dp47-python3.6-arelle-18.3

Success on aarch64-linux (full log)
Attempted: python3.pkgs.arelle
Partial log (click to expand)
writing manifest file 'Arelle.egg-info/SOURCES.txt'
running build_ext

----------------------------------------------------------------------
Ran 0 tests in 0.001s

OK
see log in c:\temp\testLog.txt
pytestcachePhase
/nix/store/9nh82sq57rbw9snh07xhbbmqy224sqrl-python3.6-arelle-18.3

Ran 0 tests in 0.000s

Please provide an appropriate checkPhase.

Failure on x86_64-linux (full log)
Attempted: python3.pkgs.arelle
Partial log (click to expand)

----------------------------------------------------------------------
Ran 1 test in 0.001s

FAILED (errors=1)
Test failed: <unittest.runner.TextTestResult run=1 errors=1 failures=0>
see log in c:\temp\testLog.txt
error: Test failed: <unittest.runner.TextTestResult run=1 errors=1 failures=0>
builder for '/nix/store/l6ps598pxw38zd42b90nys2lpjx6fdn7-python3.6-arelle-18.3.drv' failed with exit code 1
error: build of '/nix/store/l6ps598pxw38zd42b90nys2lpjx6fdn7-python3.6-arelle-18.3.drv' failed

That's strange. Why are the tests only run on x86_64-linux?

Ran 0 tests in 0.000s

Please provide an appropriate checkPhase.

Tests are actually being ran, but it's not reporting the output. I'll see what I can do

Ran 0 tests in 0.000s

Please provide an appropriate checkPhase.

Tests are actually being ran, but it's not reporting the output. I'll see what I can do

That gave me no end of encoding problems, so the answer seems to be "nothing"

@GrahamcOfBorg build python3.pkgs.arelle


Not compatible with PHP < 7.3

In composer.json the requirement for PHP shows
"require": {
        "php": "^7.1",

but actually requires 7.3, because of https://github.com/Ocramius/PackageVersions dependency
Can the dependency be locked to a version that works on 7.1? There's still projects that use this version of PHP

Wrong, Composer will require a version of ocramius/package-versions that is compatible with it (proof is our CI that tests against all versione of PHP from 7.1 to 7.4 :wink:) Also take a look at #850, you could find some comments helpful to solve the issue

Thanks @ste93cry, searched before for similar Issue, but haven't seen that one. Useful info there 👍🏼


Double check counts for the visual summary bar graphs

For the datasets in the DDE, I get different answers than what's in the bargraphs.  @candicecz can you check the calculations?
From a quick JavaScript calc, I get:
[{"technique":"transcriptomics","count":98},{"technique":"metabolomics","count":63},{"technique":"null","count":49},{"technique":"genomics","count":38},{"technique":"microbiome sequencing","count":36},{"technique":"proteomics","count":32},{"technique":"mRNA (microarray)","count":32},{"technique":"Bacteriological Assay","count":25},{"technique":"microRNA (microarray)","count":20},{"technique":"lipidomics","count":19},{"technique":"wgs","count":16},{"technique":".mzXML\\n.xlsx","count":13},{"technique":"RNA sequencing","count":13},{"technique":"RNA-Seq","count":13},{"technique":"transposon sequencing","count":10},{"technique":"immunological assay","count":9},{"technique":"DNAseq","count":8},{"technique":"Illumina PCR amplifed TnSeq ends","count":8},{"technique":"Illumina WGS","count":7},{"technique":"Mass Spectrometry","count":6},{"technique":"Microarray","count":5},{"technique":"Protein abundance","count":5},{"technique":"Protein phosphorylation","count":4},{"technique":"Illuminia gDNA sequence","count":4},{"technique":"Whole genome sequence","count":4},{"technique":"Proteomics experiment","count":4},{"technique":"Flow Cytometry","count":4},{"technique":"Nucleic Acid Sequencing","count":3},{"technique":"fastq","count":3},{"technique":"Functional genomics","count":3},{"technique":"Protein interaction experiment","count":3},{"technique":"metagenomics","count":3},{"technique":"temporal RNA-Seq","count":3},{"technique":"ChIP-seq","count":3},{"technique":"mRNA-seq","count":2},{"technique":"RNA-seq","count":2},{"technique":"Whole genome sequencing","count":2},{"technique":"serology","count":2},{"technique":"Post Translational Modification Analysis","count":2},{"technique":"Whole Genome Sequencing","count":2},{"technique":"Reverse Transcriptase-Polymerase Chain Reaction","count":2},{"technique":"Protein ubiquitination","count":2},{"technique":"Raw sequencing data for CRISPR KO screens","count":2},{"technique":"RNA Isolation","count":2},{"technique":"Metagenomics","count":2},{"technique":"Whole genome Sequencing","count":2},{"technique":"amplicon-sequencing (TRIP)","count":1},{"technique":"Lipidomics","count":1},{"technique":"miRNA-seq","count":1},{"technique":"FASTQ","count":1},{"technique":"Vital Signs Measurement","count":1},{"technique":"Gene Knockout","count":1},{"technique":"CRISPR","count":1},{"technique":"Pathology","count":1},{"technique":"pathology","count":1},{"technique":"Cell Sorting","count":1},{"technique":"Tn-Seq","count":1},{"technique":"expression profiling by array","count":1},{"technique":" Affinity Purification Mass Spectrometry (AP-MS)","count":1},{"technique":"Illumina metagenomic","count":1},{"technique":"Cytometry","count":1},{"technique":"MeDIP-seq","count":1},{"technique":"RNASeq","count":1},{"technique":"Clinical","count":1},{"technique":"animal data","count":1},{"technique":"Blood Chemistry Measurement","count":1},{"technique":"Microarray experiment","count":1},{"technique":"Microarray Analysis","count":1},{"technique":"Hi-C","count":1},{"technique":"IMMUNOLOGICAL ASSAY","count":1},{"technique":"Affinity Purification Mass Spectrometry (AP-MS)","count":1},{"technique":"GENOMICS","count":1},{"technique":"WGS","count":1},{"technique":"DNA Sequencing","count":1},{"technique":"Quantitative Reverse Transcriptase PCR","count":1},{"technique":"Illumina 16S","count":1},{"technique":"Fluorescence Activated Cell Sorting","count":1},{"technique":"Illumina transcriptomic","count":1},{"technique":"Microbiome analysis","count":1},{"technique":"metatranscriptome 16S","count":1},{"technique":"hemagglutination inhibition","count":1},{"technique":"HLA typing via sequencing","count":1},{"technique":"Whole Transcriptome Sequencing","count":1},{"technique":"flow cytometry","count":1},{"technique":"raw sequencing data for crispr ko screens","count":1},{"technique":"RNA Analysis","count":1},{"technique":"Rapid Antigen Test","count":1},{"technique":"metatranscriptomics","count":1},{"technique":"Blood Cell Count","count":1},{"technique":"microRtranscriptomics","count":1},{"technique":"Reduced-Representation Bisulfite Sequencing","count":1},{"technique":"Immunology","count":1},{"technique":"Proteomics","count":1},{"technique":"HLA sequencing","count":1},{"technique":"Protein Analysis","count":1},{"technique":"Protein Expression Analysis","count":1},{"technique":"Illumina meta-transcriptomic","count":1},{"technique":"Amplicon Sequencing (TRIP)","count":1}]

... which seems to mostly match R code to pull the data:

Note: quick check of pathogens look like they agree...


set temperature to 0.01 for better llm compatibility

#938
Change Temperature from 1e-8 to 0.01 for better llm Compatibility.

@0Falli0 sorry about the really big delay but which llm/provider are you trying to make this compatible with?

The main Issue is the fact that I am trying to make your metrics compatible with Mistral7b and Mixtral8x22 hosted via Huggingface Text Generation Interface(TGI). When provided a low temperature of 1e-8 for example the models just produce blank characters until the Token-Count is reached.


mydump: support multi bytes csv delimiter and separator

What problem does this PR solve?
partially reslove #321
What is changed and how it works?
Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Side effects

Possible performance regression
Increased code complexity

Related changes

Need to update the documentation
Need to be included in the release note

/run-all-tests

/run-all-tests tidb=pr/20191

/run-all-tests tidb=pr/20191


release_1.4.0

[x] Prep

[x] Create a release issue: release_1.4.0.
[x] Set freeze date freeze date [1/24/2020].

[ ] Branch Release (on or around freeze date)

[ ] Ensure all blocking milestone issues have been closed.
[ ] Merge the latest release into dev and push upstream.

[ ] Deploy and Test Release

[x] Review issues and ensure they all have a milestones attached. Link
[ ] Checkout release branch.
[ ] Run schemas/validate.py on each of the examples in examples/*, updating if necessary.
[ ] Check for obvious missing or inconsistent documentation in release.

[ ] Create Release Notes

[ ] Open newly edited/created files and manually curate for accuracy.
[ ] If any changes are made, ensure you commit branch again and document in commit what was changed.
[ ] Run sh scripts/log.sh from the repo root.
[ ] Check for obvious missing or inconsistent documentation in release notes (CHANGELOG.md).
[ ] Commit branch.

[ ] Do Release

[ ] Ensure all blocking milestone issues have been resolved.
[ ] Create and push new release tag.

[ ] Announce Release

[ ] Verify release included in https://github.com/biocompute-objects/BCO_Specification/releases
[ ] Review announcement on https://biocomputeobject.org/
[ ] Tweet docs news highlights link as @BioComputeObj on twitter. @kee007ney will provide an example in the future.
[ ] Email highlights to listserv @kee007ney will provide an example in the future..

[ ] Prepare for next release

[ ] Close milestone ${version} and ensure milestone ${next_version} exists.
[ ] Create release issue for next version release_${next_version}.
[ ] Close this issue.

I checked if the user guide is updated to match IEEE 2791-2020, but https://github.com/biocompute-objects/BCO_Specification/blob/1.4.0/docs/bco-domains.md#bco-domains says 1.3.0 in its spec_version.
The user guide no longer link or include the JSON schema files from https://gitlab.com/IEEE-SA/2791/2791-schema/
Many of the other pages have too strong recommendations compared to the standard, e.g. ETag mentions SHA256, but that would just be one of many ways to calculate it.
References to xref in https://github.com/biocompute-objects/BCO_Specification/blob/1.4.0/docs/usability-domain.md seem unfounded from IEEE spec/schema as usability_domain.json just say free text, while we find in user guide:

The usability_domain can accept template language to indicate values from the external_references.

This template language is not defined by IEEE 2791.
So overall these kind of general guidances or best practices (which do not exist in IEEE standard) should be separated out from guidance on what is required from the specification.

@stain
I posted an issue about the URLs.
Not sure if Josh has seen it. @kee007ney  or I will try to reach out again.
        "spec_version": {
            "type": "string",
            "description": "Version of the IEEE-2791 specification used to define this document",
            "examples": [
                "https://w3id.org/2791/"

The current schema files are a direct copy of the IEEE files, but they need to be addressed.
https://gitlab.com/IEEE-SA/2791/2791-schema/-/blob/master/2791object.json#L120


[cgal] Remove docs

Describe the pull request

What does your PR fix?
Remove share/doc and share/man
Use vcpkg_install_license

Thanks!


Docs improvement

Some minor docs improvements.

🎉 Thanks @federicobozzini, and welcome to the Veracruz project! 🎉
Can you amend this PR to also add your name and email address to the AUTHORS.markdown file?

Can you amend this PR to also add your name and email address to the AUTHORS.markdown file?

Done, thank you very much!

+1 reached, merging.


Added macOS environment

Added macOS for Travis testing to resolve https://github.com/caiogondim/logdown.js/issues/16.

Linux is enough for us. We are testing it against JS.
If it passes in Linux and not on macOS, it's not the library problem.
And we should use testing for correctness of the algorithm, not for E2E testing.

Regarding this topic, I'm very inclined to move all tests to Jest.

So we can close this PR & https://github.com/caiogondim/logdown.js/issues/16 ?

Yes!
Let's strive to make things simple =)

Ok! 🍻


Remove duplicate mod operation in GrpcConnectionPool

What changes are proposed in this pull request?
Remove duplicate mod operation in GrpcConnectionPool.
Why are the changes needed?
The mod operation is currently performed twice, which is duplicate.
Does this PR introduce any user facing changes?
No.

alluxio-bot, merge this please


Add PathAssert#hasDigest(MessageDigest, byte[])

Summary
I think this methods can be very useful for PathAssert, FileAssert and InputStreamAssert:

hasDigest(MessageDigest digest, byte[] expected)
hasDigest(MessageDigest digest, String expected)
hasDigest(String algorithm, byte[] expected)
hasDigest(String algorithm, String expected)
hasDigest(String algorithm, String provider, byte[] expected)
hasDigest(String algorithm, String provider, String expected)

Example
URL source = new URL("https://search.maven.org/remotecontent?filepath=org/assertj/assertj-core/2.9.0/assertj-core-2.9.0.jar");
assertThat(source.openStream()).hasDigest("SHA1", "5c5ae45b58f12023817abe492447cdc7912c1a2c");

P.S.
I can create PR if you approve such methods.

Digest expected variants:

byte[] - this variant maybe useful for some cases when digest calculated within tests.
String - But digest in string-representation is more useful for cases when developer already know digest in this form (hex).

Digest algorithms:

MessageDigest - core variant, all other variant will call this variant. And it's most powerful - developer can create what he want and what they need
String algorithm - more simple variant when used some basic digest algorithms (SHA1, MD5, e.t.c.): hasDigest("SHA1", "...") vs hasDigest(MessageDigest.getInstance("SHA1"), "...")
String algorithm + String provider - this less useful over all variants because it's only simplify hasDigest(MessageDigest.getInstance("SHA1", "provider?"), "...") into hasDigest("SHA1", "provider?", "...")

I think what

4 variants [(MessageDigest | String) x (byte[] | String)] will be very useful and need implemented
2 variants [(String, String) x (byte[] | String)] - can be eliminated

Sweet, let's go with the 4 variants first then if users ask for it we can always the two remaining ones.

Good.
I started preparing PR

This changes does not Java 8 specific but in 3406fc62cb20a5fb906584cbd7deaa97d7c2b8a3 you decide to stop work under branch 2.x.
Because of this I make PR from master branch.


Allow livewire.js to be loaded from root or subfolder

1⃣ Is this something that is wanted/needed? Did you create a feature-request issue first?
Yes. #84
2⃣ Does it contain multiple, unrelated changes? Please separate the PRs out.
No
3⃣ Does it include tests if possible? (Not a deal-breaker, just a nice-to-have)
No
4⃣ Please include a thorough description of the feature/fix and reasons why it's useful.
livewire.js had 404 error when the livewire app was not served by php artisan serve or from the root of a webserver
This change allows the livewire.js file to be loaded irrespective of whether the app is being served from a subfolder or the root folder of the web server or php artisan serve

Very cool!
Do you think the livewire() needs a bit more functionality around this? Such as the ability to force root?
D

@davidpiesse
I think that will be great. also checked out your PR. I think thats better since on further testing my PR doesnt work. Even though livewire.js was loaded correctly, the http requests to /livewire were unable to go the right url since the root of the app was not given.

This has been addressed in the latest release: v0.1.1
Thanks for the contribution though!

no problem..you are welcome..thanks too


Import wxFormBuilder Load From Resource

Description:

If we can find the resource file, we can then process it and convert any images into an Embed image type.

All three of the new languages support copying the original art to the language's output directory. While I don't think anyone using a resource file for images will be generating python, lua or php code, it would make sense to do this work for the 1.1 release. With the generation of the 3 additional languages, we're likely to get more people switching from wxFormBuilder to wxUiEditor, and that makes importing wxFormBuilder projects more important.

While copying art files was under consideration, it was dropped in favor of embedding since wxPython and C++ both support it. Support for wxLua and wxPHP was dropped, so we only need to deal with these two languages that both support embedding. While the ability to convert from a resource file to an embedded image would definitely be useful, it can wait until after the 1.1 release.