Problem with sending domains via AP

⚠ Please verify that this question has NOT been raised before.

[X] I checked and didn't find similar issue

🛡 Security Policy

[X] I agree to have read this project Security Policy

📝 Describe your problem
I have created API in Kuma also created Api on domain registrar side.
My task is to import hundreds of domains from the Kuma system.
I have written a python script that links the registrar and Kuma but on the Kuma side I have some problem.
I am using
kuma_api_url = ‘http://HERE_IS_MY_KUMA_UP:3001/api/v1/monitor’
kuma_api_key = ‘HERE_IS_MY_API’
But for example when going to pages:
http://HERE_IS_MY_KUMA_UP:3001/api/v1/monitor
http://HERE_IS_MY_KUMA_UP:3001/api/monitor
http://HERE_IS_MY_KUMA_UP:3001/monitor
I get a stub from kuma - page not found.
Also when I run the script I get errors in the console such as
Failed to add domain ***** to Uptime Kuma
📝 Error Message(s) or Log
No response
🐻 Uptime-Kuma Version
1.23.13
💻 Operating System and Arch
Ubuntu 24.04 LTS
🌐 Browser

🖥 Deployment Environment

Runtime: Docker version 24.0.7, build 24.0.7-0ubuntu4
Database: apt install mysql-client-core-8.0  # version 8.0.35-1ubuntu2, or
apt install mariadb-client-core    # version 1:10.11.6-1

I don't know where you installed or what you installed, but the information does not line up at all.
We currently don't have an api. Please refer to my comment here for your options https://github.com/louislam/uptime-kuma/issues/118#issuecomment-2143409606
Also maridadb is not a database option in v1.23.13. Please refer to #4500 for further details


add a docker setup for the demo

BTW, your video demo is amazing, I think it's the best explanation for blockchain I've seen so far. Great job! Thanks a lot!

Thanks for doing this!

@anders94 You're welcome! :-)


[BUG] 2.7.5 LDAP connector broken

Rancher Server Setup

Rancher version: 2.7.5-rc5
Installation option (Docker install/Helm Chart): docker

Information about the Cluster

Kubernetes version: 1.23
Cluster Type (Local/Downstream): local

User Information

What is the role of the user logged in?
admin to configure the LDAP
try any ldap user to connect after the configuration

Describe the bug

Just after upgrading from 2.6.8 to 2.7.5, I noticed en error to my LDAP configuration
I tried to disable the ldap connector (and loose every user configuration) and create a new one,  the same problem is present
c
To Reproduce

Create a fully working LDAP in rancher 2.6.X and update to rancher 2.7
Result
Just after the upgrade, we can see the following lines
2023/06/21 20:12:40 [INFO] [mgmt-auth-crtb-controller] Creating roleBinding for subject openldap_group://CN=nd_edition,OU=_Groupes,OU=Utilisateurs,DC=company,DC=com with role cluster-owner in namespace c-m-txpct7zr
2023/06/21 20:12:41 [INFO] [mgmt-auth-crtb-controller] Creating roleBinding for subject openldap_group://CN=nd_edition,OU=_Groupes,OU=Utilisateurs,DC=company,DC=com with role cluster-owner in namespace p-g48bl
2023/06/21 20:12:41 [INFO] [mgmt-auth-crtb-controller] Creating roleBinding for subject openldap_group://CN=nd_edition,OU=_Groupes,OU=Utilisateurs,DC=company,DC=com with role cluster-owner in namespace p-jwlqs
2023/06/21 20:12:41 [INFO] [mgmt-auth-crtb-controller] Creating roleBinding for subject openldap_group://CN=nd_edition,OU=_Groupes,OU=Utilisateurs,DC=company,DC=com with role cluster-owner in namespace p-psp9q
2023/06/21 20:12:42 [INFO] Creating clusterRoleBinding Group openldap_group://CN=nd_edition,OU=_Groupes,OU=Utilisateurs,DC=company,DC=com Role cluster-owner
2023/06/21 20:12:42 [INFO] Creating clusterRoleBinding Group openldap_group://CN=nd_edition,OU=_Groupes,OU=Utilisateurs,DC=company,DC=com Role cluster-owner
2023/06/21 20:12:43 [INFO] Creating clusterRoleBinding Group openldap_group://CN=nd_edition,OU=_Groupes,OU=Utilisateurs,DC=company,DC=com Role cluster-owner
2023/06/21 20:12:44 [INFO] Creating clusterRoleBinding Group openldap_group://CN=nd_edition,OU=_Groupes,OU=Utilisateurs,DC=company,DC=com Role cluster-owner
2023/06/21 20:12:45 [INFO] Redeploy Rancher Agents is needed for c-m-bjb86prh: forceDeploy=false, agent/auth image changed=true, private repo changed=false, agent features changed=false
2023/06/21 20:12:45 [INFO] Creating system token for u-st6exg52qq, token: agent-u-st6exg52qq
2023/06/21 20:12:45 [ERROR] error syncing 'u-nom4lfeabr': handler mgmt-auth-userattributes-controller: invalid server config. at least 1 server needs to be configured, requeuing
2023/06/21 20:12:45 [ERROR] Error refreshing token principals, skipping: invalid server config. at least 1 server needs to be configured
2023/06/21 20:12:45 [ERROR] available chart version (100.2.3+up0.5.3) for fleet is less than the min version (102.1.0+up0.7.0)
2023/06/21 20:12:45 [ERROR] Failed to find system chart fleet will try again in 5 seconds: no chart name found
2023/06/21 20:12:46 [ERROR] error syncing 'u-zavaqncbmy': handler mgmt-auth-userattributes-controller: invalid server config. at least 1 server needs to be configured, requeuing
2023/06/21 20:12:46 [ERROR] Error refreshing token principals, skipping: invalid server config. at least 1 server needs to be configured
2023/06/21 20:12:46 [INFO] Redeploy Rancher Agents is needed for c-m-5kr2sg25: forceDeploy=false, agent/auth image changed=true, private repo changed=false, agent features changed=false
2023/06/21 20:12:46 [INFO] Creating system token for u-yaoijgdp6m, token: agent-u-yaoijgdp6m
2023/06/21 20:12:47 [ERROR] Error refreshing token principals, skipping: invalid server config. at least 1 server needs to be configured
2023/06/21 20:12:47 [INFO] Refreshing driverMetadata in 1440 minutes
2023/06/21 20:12:47 [INFO] getReleaseConfig: reloading config for k3s
2023/06/21 20:12:47 [INFO] getReleaseConfig: reloading config for rke2
2023/06/21 20:12:47 [ERROR] Error refreshing token principals, skipping: invalid server config. at least 1 server needs to be configured
2023/06/21 20:12:47 [ERROR] Error refreshing token principals, skipping: invalid server config. at least 1 server needs to be configured

and just a few seconds
2023/06/21 20:12:48 [ERROR] Error refreshing token principals, skipping: invalid server config. at least 1 server needs to be configured
2023/06/21 20:12:48 [ERROR] Error refreshing token principals, skipping: invalid server config. at least 1 server needs to be configured
2023/06/21 20:12:49 [ERROR] Error refreshing token principals, skipping: invalid server config. at least 1 server needs to be configured
2023/06/21 20:12:49 [ERROR] error syncing 'u-zavaqncbmy': handler mgmt-auth-userattributes-controller: invalid server config. at least 1 server needs to be configured, requeuing
2023/06/21 20:12:49 [ERROR] error syncing 'u-pulf6lwmml': handler mgmt-auth-userattributes-controller: invalid server config. at least 1 server needs to be configured, requeuing
2023/06/21 20:12:49 [ERROR] Error refreshing token principals, skipping: invalid server config. at least 1 server needs to be configured
2023/06/21 20:12:49 [ERROR] Error refreshing token principals, skipping: invalid server config. at least 1 server needs to be configured
2023/06/21 20:12:49 [ERROR] Error refreshing token principals, skipping: invalid server config. at least 1 server needs to be configured
2023/06/21 20:12:49 [ERROR] Error refreshing token principals, skipping: invalid server config. at least 1 server needs to be configured
2023/06/21 20:12:49 [ERROR] error syncing 'u-p2ogoquf45': handler mgmt-auth-userattributes-controller: invalid server config. at least 1 server needs to be configured, requeuing
2023/06/21 20:12:49 [ERROR] Error refreshing token principals, skipping: invalid server config. at least 1 server needs to be configured
2023/06/21 20:12:49 [ERROR] Error refreshing token principals, skipping: invalid server config. at least 1 server needs to be configured
2023/06/21 20:12:49 [ERROR] Error refreshing token principals, skipping: invalid server config. at least 1 server needs to be configured
2023/06/21 20:12:49 [ERROR] error syncing 'u-pulf6lwmml': handler mgmt-auth-userattributes-controller: invalid server config. at least 1 server needs to be configured, requeuing
2023/06/21 20:12:49 [ERROR] Error refreshing token principals, skipping: invalid server config. at least 1 server needs to be configured
2023/06/21 20:12:49 [ERROR] Error refreshing token principals, skipping: invalid server config. at least 1 server needs to be configured
2023/06/21 20:12:49 [ERROR] Error refreshing token principals, skipping: invalid server config. at least 1 server needs to be configured
2023/06/21 20:12:49 [ERROR] Error refreshing token principals, skipping: invalid server config. at least 1 server needs to be configured

sometimes I see this line  after retrying to set the configuration
2023/06/21 20:19:36 [ERROR] API error response 422 for POST /v3/openLdapConfigs/openldap?action=testAndApply. Cause: servers=MissingRequired 422:

Expected Result

LDAP connection is still available
Screenshots

Additional context

Potentially related to https://github.com/rancher/rancher/pull/41869 ?


Initial mu differs from recommendation

You have:
self.mu = np.prod(self.D.shape) / (4 * self.frobenius_norm(self.D))
wherein forbenius_norm computes the 2-norm. However in the RobustPCA paper, we have the recommendation:
μ = n1 * n2 / 4 * |M|
"as suggested in Yuan and Yang", where |M| is the 1-norm of M. I suspect that parentheses have been omitted, and this is instead supposed to read:
μ = n1 * n2 / (4 * |M|)
However, even imputing (as you did) the missing parentheses, the norm should be of order 1, not order 2.

Thank you -- this is indeed a bug. It has been fixed. In my experiments, the precise choice of frobenius versus L1 norm for this computation didn't change results.

Came across this implementation since last using it a while back. Back then, the example didn't seem to converge nicely with the default parameters for a slightly different values for N, num_groups, num_values_per_group and p_missing.
For future reference, this was the effect of the μ parameter on the convergence (N = 1000, num_groups = 5, num_values_per_group = 100, p_missing = 0.05):


[spike] look into Apache Ozone as a drop-in replacement for minio for S3-compatible storage

We don't want to ship minio in the project because of its AGPL license. Look into whether Apache Ozone is a viable drop-in replacement for minio.
Acceptance criteria:
A document that:

compares Ozone to minio, including relevant pros/cons of each
gives a script of how to install ozone and how to hook it up to DSP
provides high level thoughts on how hard it would be to ship it in DSP out of the box
OR that describes why Ozone won't work

(prior work on a similar evaluation of Noobaa: https://docs.google.com/document/d/16yx9zNcrmBK9xyi_EiQrzykGgIuEfW-ZLvxgna9oV0o/edit)

Migrated to Jira: https://issues.redhat.com/browse/RHOAIENG-1674


Preview

📝 Changed routes:

/create
/

(view full size)
(view full size)

/p/[id]

(view full size)

And 3 other routes:

/api/publish/[id]
/api/term/[id]
/api/term

3 deleted routes:

/api/post/[id]
/api/post
/drafts

Commit d129929e7264490ae97898466ec1dcd6027b08b9 (https://climate-dictionary-3afqam1e3-tiny-factories.vercel.app).


[INTERNAL] sap.m.Link: documentation of rel fix

Some <code> tags were not properly closed which caused improper formatting in the Demo Kit - API reference (https://openui5.hana.ondemand.com/1.84.1/#/api/sap.m.Link%23controlProperties).

This PR fixes the issue above.

Hi @boghyon,
thank you for fixing those typos, and Happy New Year, BTW :-)
Kind Regards, Frank

Hi @boghyon,
thank you for fixing those typos, and Happy New Year, BTW :-)
Kind Regards, Frank

Happy new year. 🙂 Thanks for merging

Happy new year. 🙂 Thanks for merging


Retrieve macOS accent color

This introduces getControlAccentColor and implements it in macOS.
Fixes #41
Guess some API reimagination would be needed but at least it works I can say,

Partially addressed some of the reviews hopefully, https://github.com/material-foundation/material-dynamic-color-flutter/pull/42/commits/14c65f5170843bc0ab9deec9eb7d39663e8c6ab5

You'll also want to create a small example and tests so that the functionality you're adding can't be broken in the future.

Done the far I could.


htmlwhat package not providing correct feedback after code changes

Issue Description
The htmlwhat package, used for validating HTML code submissions, is not functioning correctly when I modify the HTML code being tested.
Steps to Reproduce

Code Example:
from htmlwhat.test_exercise import test_exercise

userhtml = """
<!DOCTYPE html>

        <title>Title</title>

My First Heading

"""

solutionhtml = """
<!DOCTYPE html>

        <title>First Webpage</title>

My First Heading

"""

test = "Ex().check_body().has_equal_attr()"

print(test_exercise(
    test,
    userhtml,
    solutionhtml,
))

Expected Behavior
The test_exercise function should provide accurate feedback on the differences between userhtml and solutionhtml, confirming correctness or pointing out discrepancies based on the test provided (Ex().check_body().has_equal_attr()).
Actual Behavior
Currently, the output from test_exercise does not accurately reflect the correctness of the HTML code when modifications are made to either userhtml or solutionhtml. Instead, it fails to detect changes or provides incorrect feedback.
Additional Information

htmlwhat is typically used in educational contexts to assess HTML assignments.
This issue affects the reliability and functionality of htmlwhat, which is crucial for providing accurate feedback on HTML correctness.

This issue impacts the reliability and functionality of the feedback mechanism crucial for assessing correctness in HTML submissions.

I am not getting your point as this is working fine and fif you are looking for the syntax error then at this stage it is not possible as we are using beautifulsoup to parse code which automatically correct html syntactically.
Provide more details on your problem.


Add sign up page based on intuit.com sign page

built with only html5 and css
the html page is validated for semantical errors with html-validator-cli
css passed all stylelint linting measures

Thank you @christianotieno for your kind suggestion I have considered all your comments and fixed me.


add "step" method as an alternative to state_processing and state_physics_processing

In many games (such as turn-based games or those with a completely fixed timestep), it does not make sense to use either _processing() nor _physics_processing() to house game logic. This PR allows the user to easily implement their own logic for updating states decoupled from the built-in callbacks.

This sounds pretty useful, thanks a lot for submitting it!

How do you config the step here? How would you use this in turn based games? I'm not building a turn based game now but I'm highly interested about how you'd use state charts with custom step for that kind of games

I have added a small demo showing this stepping mode and I have added some info to the documentation. Rebased & merged. Thanks a lot for submitting this!


Add repeat() for iterable datasets

Feature request
It would be useful to be able to straightforwardly repeat iterable datasets indefinitely, to provide complete control over starting and ending of iteration to the user.
An IterableDataset.repeat(n) function could do this automatically
Motivation
This feature was discussed in this issue https://github.com/huggingface/datasets/issues/7147, and would resolve the need to use the hack of interleave datasets with probability 0 as a simple way to achieve this functionality.
An additional benefit might be the simplification of the use of iterable datasets in a distributed setting:
If the user can assume that datasets will repeat indefinitely, then issues around different numbers of samples appearing on different devices (e.g. https://github.com/huggingface/datasets/issues/6437, https://github.com/huggingface/datasets/issues/6594, https://github.com/huggingface/datasets/issues/6623, https://github.com/huggingface/datasets/issues/6719) can potentially be straightforwardly resolved by simply doing:
ids.repeat(None).take(n_samples_per_epoch)
Your contribution
I'm not familiar enough with the codebase to assess how straightforward this would be to implement.
If it might be very straightforward, I could possibly have a go.

perhaps concatenate_datasets can already be used to achieve almost the same effect?

concatenate_datasets does the job when there is a finite number of repetitions, but in case of .repeat() forever we need a new logic in iterable_dataset.py


[FIELDTEST] Road labels are far too small during navigation

During navigation I mount the phone farther away than I hold it, so the somewhat small road labels we have now become too tiny to read during the navigation experience.

[x] Basic improvements
[ ] Polish improvements

Looking much better in master-569!

After a week away, using master-598, these are still a smidgen too small.

Geraldine landed fixes for these, time for another drive test.

done


Read-Only Mode

For users which not authorized to make changes, or pattern languages that should stay static, a read-only mode should just display the patterns

Not necessary anymore since #68 introduced an anonymous user (= not logged in). Removing all editing privileges from this user gives read only mode.


[NOT TESTED] etcd-operator: bump to v0.5.2

Note: v0.5.1 contains a breaking TLS change, please check the
release note before upgrading:
https://github.com/coreos/etcd-operator/releases/tag/v0.5.1

Closing, we have switched to on-host etcd in production..


🔄 Synced file(s) with BetaHuhn/github-files

Synced local file(s) with BetaHuhn/github-files.

Changed files

Synced local  .github/workflows/release.yml with remote  workflows/versioning/release.ymlCreated local  .github/workflows/node.yml  from remote  workflows/node/node.ymlCreated local  .github/workflows/latest-tag.yml  from remote  workflows/versioning/latest-tag.yml

This PR was created automatically by the repo-file-sync-action workflow run #629931511

:tada: This PR is included in version 0.2.15 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Designing a Production-Ready Kappa Architecture for Timely Data Stream Processing / Uber

https://eng.uber.com/kappa-architecture-data-stream-processing/
Key takeaway:
Out-of-order data is a big problem that can be mitigated but in some cases, it should be totally resolved. Usually, it resolved by using lambda architecture. Uber proposes their approach how to handle this problem in kappa architecture.

Published in Digest #3.


rule ordering static path vs set of items goes wrong

currently
/<a>/<b>/<c>
/static/<path:filename>

orders the path after the 3 chunks, which means it cant match static paths anymore

I am unable to recreate in Python 2.7 and 3.7. Here are my tests:
def test_stuff():
    map = r.Map(
        [
            r.Rule("/<a>/<b>/<c>", endpoint="stack"),
            r.Rule("/static/<path:filename>", endpoint="static"),
        ]
    )
    adapter = map.bind("example.org", "/")
    assert adapter.match("/static/hello.jpg") == ("static", {"filename": "hello.jpg"})
    assert adapter.match("/static/images/hello.jpg") == ("static", {"filename": "images/hello.jpg"})
    assert adapter.match("/x/y/z") == ("stack", {"a": u"x", "b": u"y", "c": u"z"})

I also ran them with the rules in reverse order:
 def test_stuff():
    map = r.Map(
        [
            r.Rule("/static/<path:filename>", endpoint="static"),
            r.Rule("/<a>/<b>/<c>", endpoint="stack"),
        ]
    )
    adapter = map.bind("example.org", "/")
    assert adapter.match("/static/hello.jpg") == ("static", {"filename": "hello.jpg"})
    assert adapter.match("/static/images/hello.jpg") == ("static", {"filename": "images/hello.jpg"})
    assert adapter.match("/x/y/z") == ("stack", {"a": u"x", "b": u"y", "c": u"z"})

If you can provide a way to recreate, I will re-open. Until then, I will close as I cannot recreate.


Patch 0.0.009

change form

I can't understand why you open this PR, could you give me some further description of this PR?


[PAPI-398] Update CoinGecko client

Resolves bug where price data was failing to deserialise if CoinGecko was used as pricing feed

Doesn't seem to actually resolve the issue :(


Adding inheritance modifiers

I changed the additive attributes to only work on lists, removed the important operator, and changed the '!' to work as an exclude in lists instead.

Looks good to me...


[bug] hook useSendTransaction doesnt work into mobile wallets (trust wallet, metamask)

Is there an existing issue for this?

[X] I have searched the existing issues

RainbowKit Version
0.12.2
wagmi Version
0.12.3
Current Behavior
i want to transfer eth from one wallet to another
Expected Behavior
on success, a modal window should open in the trust wallet confirming the transaction
Steps To Reproduce
i use the standard rainbowkit + wagmi + ethers example, in mobile wallets confirmation does not work anywhere, only the metamask extension works
const { config } = usePrepareSendTransaction({
    request: {
      to: toAddress,
      value: incomeVal ? utils.parseEther(incomeVal).toString() : undefined,
    }
  })

  const { data, sendTransaction } = useSendTransaction(config)

Link to Minimal Reproducible Example (CodeSandbox, StackBlitz, etc.)
No response
Anything else?
I've been trying to solve this problem for a week now, does no one notice this? Please tell me how to make transactions work in mobile wallets

I believe we fixed this with https://github.com/rainbow-me/rainbowkit/pull/1145. Just released as 0.12.7. Let me know if you're still experiencing the issue.


Component discovery on Opensuse Tumbleweed is inconsistent

Copied from https://github.com/microsoft/vscode-dotnettools/issues/49#issuecomment-2239902160

Hi, for me the 'solution (if we can name that..) is to just add using to namespace.
I don't know how does it work in more complex solutions - probably pain in the ... but it works for me.
Just add usings on the top of the file.
It looks like the _Imports.razor is not working as should be for IDE extension. (For build it's ok)
Attached screenshot before and after using.
What's more interesting after additional using i can remove it after build and IDE looks fixed for all other usings... really weird

is https://github.com/dotnet/vscode-csharp/issues/7625 related?

is dotnet/vscode-csharp#7625 related?

Probably not. This is specific to the OS. Other linux distros that we test with are not having this issue.


uhv: add http-parser permissive parsing for h1 codec

Commit Message: uhv: add http-parser permissive parsing for h1 codec
Additional Description: Honor the new uhv_enabled build flag to disable strict parsing within the H1 parsing library (http-parser). This is a reworking of the previous PR, #20528, but this time using the new compile-time option instead of making it configurable at runtime.
Risk Level: Low
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
Fixes #19750

@yanavlasov This should be good for review.

@alyssawilk this is special UHV build for H/1 parser with header validation disabled, using existing #define

@alyssawilk I have confirmed that removing the early return in both protocol_integration_test and http_inspector_test results in both failing when UHV is enabled:
[  FAILED  ] Protocols/DownstreamProtocolIntegrationTest.BadRequest/IPv4_HttpDownstream_HttpUpstreamNghttp2NoDeferredProcessing
[  FAILED  ] HttpInspectorTest.MultipleReadsHttp1BadProtocol (156 ms)


[TACHYON-1560] Rename tachyon.web.Utils to tachyon.web.WebUtils

tachyon.web.WebUtils is better for readability since there are multiple Utils class.
https://tachyon.atlassian.net/browse/TACHYON-1560

Can one of the admins verify this patch?

OK to test

Jenkins, test this please

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/8242/
Test PASSed.

Jenkins, ok to test

Jenkins, test this please

+1 LGTM
thanks for submitting the patch!

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/8245/
Test PASSed.

LGTM


should scale_penalty_with_samples = true be defualt?

Just sinking my teeth into MLLinearModels and I see that scale_penalty_with_samples = true is the default. Playing around with a number of toy datasets it seems that scale_penalty_with_samples = true does not produce intuitive results while scale_penalty_with_samples = false does, e.g.:
# create data
t = 1:0.001:10;
y = 10 .+ 10 * sin.(t) .+ 5 * t .+ randn(length(t)) * 2 .+ rand((zeros(round(Int64, length(t) / 5))..., 6, -8, 100, -200, 178, -236, 77, -129, -50, -100, -45, -33, -114, -1010, -1238, -2000), length(t))*.5;
X = hcat(ones(length(t)), sin.(t), t);
scatter(t, y; markerstrokecolor=:match, markerstrokewidth=0, label = "obs", ylim = (0, 70))

# Base LSQ model fit
θ = X \ y;
scatter!(t, X * θ, markerstrokewidth=0, label="Base lsq")

# index X[:,2:end] as fit includes offset by default
θ = fit(LinearRegression(), X[:, 2:end], y);
scatter!(t, hcat(X[:, 2:end], ones(length(t))) * θ, markerstrokewidth=0, label="linear")

θ = fit(HuberRegression(scale_penalty_with_samples=true), X[:, 2:end], y);
scatter!(t, hcat(X[:, 2:end], ones(length(t))) * θ, markerstrokewidth=0, label="huber")

θ = fit(HuberRegression(scale_penalty_with_samples=false), X[:, 2:end], y);
scatter!(t, hcat(X[:, 2:end], ones(length(t))) * θ, markerstrokewidth=0, label="huber-no_scale_penalty")

Given this, should scale_penalty_with_samples = false be made the default or is there a logical reason that it is not?

I think this stuff tends to depend on what/how you use MLJLM for.
Looping in @jbrea in since he's the one who introduced the change and had a good rationale for it (I think there's an issue where this stuff is discussed I'll try to find it)

@tlienart thanks for the link to the previous discussion. To me it seems likely a smaller value for lambda probably should have been set as the default. New users, if moving quickly, will assume that several of the models will give very bad predictions given the defaults. I realize that each use case is different but It seems to me that a lambda would be used in the minority of use cases.

In any case if you look at scikit learn for instance, which is one of the reference implementation out there, they use scaling and they use an L2 with a nontrivial lambda by default.
I personally don't have a strong opinion on this, to me it's a matter of having correct docs and potentially guiding the user in what they should do (eg HP tuning) ; if you feel that the docs were unclear please consider opening a PR for it.

I think expanding the docs with examples of the different regression options would be a good idea. I'll open a new issue for this with the hopes of supplying a PR soon.


esprima.js

How do you download or build "esprima.js"? Is there a guide somewhere? I see no mention for how you actually get "esprima.js" in the README. I'm not interested in installing it through NPM because I want to run it through Rhino.

It's mentioned in documentation. The easiest way is to grab the one from CDN, e.g. https://cdn.rawgit.com/jquery/esprima/2.7.0/esprima.js.

I'll close this. Let me know @cpburnz if you still have difficulties getting the file.

This is still not very easy to find, and I don't see anything in the docs. I'm not sure I'm doing it right. My best guess is from looking at the package.json and running npm run compile but there's no minified version. CDNs don't seem to have v4.

@simap It's in the documentation (Chapter 1, search for Using Esprima in a web browser).

@ariya Ah, I see it there now on a CDN link. I don't know why, but I did spent a lot of time looking for it on the site. I must have been looking for the wrong things, expecting a download, or how to build kind of thing. Thank you.


Production Environment Wiki

Title of ticket: Production Environment Wiki
Description
For production launch, team has discussed in retro and planning the need to set up a wiki document outlining key responsibilities, tasks, priorities and contacts for when the application moves to PROD. This will be an ongoing work in progress up to and likely after production launch.
DOD

[ ] Wiki page built and accessible to all relevant teams
[ ] Set up Section for priorities and decision making (ie: which issues constitute high priority/immediate action, how are decisions made, etc.)
[ ] Set up maintenance schedules, including list of scheduled tasks and those that will be assigned to those tasks
[ ] Establish a key contacts page/chain of command - who to contact and how, and their backups
[ ] Checklists and guidance documents for what to do when certain scenarios emerge
[ ] Include section on key information such as service accounts, and schedules for updates/password updates and administrators/owners for each account
[ ] Onboarding and offboarding documentation
[ ] Clear guidelines for user roles and permissions and how these will be handled

@sumathi-thirumani-aot @lmullane @KaraBeach @mpilchar
FYI - I have created a ticket for the production launch wiki. I expect some of these tasks will need to be handled/reviewed by the business, while others will be handled/reviewed by the dev team

@sumathi-thirumani-aot - I know you started this already - does this have duplications? Let's connect on this one and what's needed for Release.


Change to async calls once available

Update to async calls to currently sync versions.
Expected for Npgsql v.5.0.0

Use UnprepareAsync instead of Unprepare on NpgqlCommands #2944
Use GetColumnSchemaAsync instead of GetColumnSchema on DbReaders #3010

recheck

UnprareAsync is implemented, however GetColumnSchemaAsync won't be implemented in the future and isn't really required anyway.


[Explicit Module Builds] Change interface used by SwiftPM to communicate external dependencies.

Instead of SwiftPM passing in a set of external targets along with their dependency graphs, SwiftPM will now pass in:

A set of external targets along with the location of their resulting .swiftmodule files, as before
a collection of all modules seen so far across all targets already planned in this build.

The second point replaces previous behaviour, where SwiftPM only passed in the dependency graphs of targets which are dependencies of the target swift-driver is currently planning the build for. Having access to all previously-seen modules will allow us to be more intelligent with knowing what modules do not need scanning as well as what clang modules require a re-scan, as each re-scan updates Clang module's details with the set of PCMArgs their dependencies currently capture and propagate it up to the depending modules' build planning actions.
This PR produces an API change that is matched with a corresponding depending SwiftPM change: https://github.com/apple/swift-package-manager/pull/2959.
When used together with #279 and the respective SwiftPM changes, this change results in a significant reduction in the amount of versioned batched re-scanning we perform.
For example:
When constructing the build manifest for building SwiftPM, this change reduces the total sum of batch sizes of all versioned PCM re-scans involved in the build from 2525 to 303 (8.2x improvement).
Further followup change will address redundant re-scanning of Swift modules by using the additional information SwiftPM now provides to create more placeholder modules in the dependency scanning action input.

@swift-ci please test

@swift-ci please test

@swift-ci please test

I have added a temporary adapter to preserve the old API so that this change can be submitted without breaking the SwiftPM build. This is necessary because SwiftPM CI does not currently support cross-repository testing with the swift-driver. Once the corresponding SwiftPM change to adapt to the new API has been submitted, I will submit a followup patch to remove the legacy API adapter.


Restore last-resort expired untrusted intermediate issuers

Fixes #1997
Clone of #2004 for the master branch.

Thanks for the review.  Pushed and closed.


Feature basic level

Disable seeking healing kit for the boss
Modify boss special attack
Change wall height to avoid boss from escaping the area

Just realised this branch has conflicts. Could you resolve them and then make a new pull request, or update this one if possible?


Add support for Google Colab

Now localtileserver is working with JupyterHub. It would be nice to also make it compatible with Colab. The following two lines have no effect on Colab as it is not a JupyterHub.
import os
os.environ['LOCALTILESERVER_CLIENT_PREFIX'] = f"{os.environ['JUPYTERHUB_SERVICE_PREFIX'].lstrip('/')}/proxy/{{port}}"

@giswqs, it's been a long time since I've used Colab. Could you post a link to a repo and session I could launch to debug this?

I have no idea how to do this on google colab. It's seemingly not a Jupyter Hub/Lab instance and I have absolutely no idea how to expose a port on the runner. jupyter-server-proxy is really the only safe way to do this in cloud environments and I don't think that's possible with google colab

This is actually really easy! I figured it out while trying to get this to work for PyVista in https://github.com/pyvista/pyvista/issues/3851
I'll see if I can add support directly in localtileserver

@banesullivan That's awesome to hear. Can't wait to try it out.

It's as simple as:
from localtileserver import get_leaflet_tile_layer, examples, TileClient
from ipyleaflet import Map

client = TileClient(examples.get_data_path("bahamas_rgb.tif"))
client.client_host = f'localhost:{client.server.port}'

t = get_leaflet_tile_layer(client)

m = Map(center=client.center(), zoom=client.default_zoom)
m.add_layer(t)
m

But I'll make this a little easier in the next release

Excited to try this out. Got an error. I should probably wait for the next release.

yeah... ran into that too... Google Colab installs GDAL and their build is wack (and missing that module). This was handled in afe669cc80c1d29e65e1934956281c85c7935da9 and will be good on the next release

So, this is fixed by de4d2ef576e3e0c4f0ecaf39bdb044a02747b0cf
What do you think, @giswqs?
from localtileserver import get_leaflet_tile_layer, examples, TileClient
from ipyleaflet import Map

path = examples.get_data_path("bahamas_rgb.tif")

client = TileClient(path)
client.enable_colab()

t = get_leaflet_tile_layer(client)

m = Map(center=client.center(), zoom=client.default_zoom)
m.add_layer(t)
m

This gives me hope that localtileserver might work with all major Jupyter notebook platforms, including AWS SageMaker.
client.client_host = f'localhost:{client.server.port}'

The reason it works is actually pretty specific to colab supporting this localhost remapping. Reference https://colab.research.google.com/notebooks/snippets/advanced_outputs.ipynb

Outputs in the browser can request resources from the kernel by requesting https://localhost:{port}. The protocol will automatically be translated from https to http and the localhost will be the kernel executing the code.

You may not need that client.enable_colab() explicitly. You might be able to integrate this into TileClient.
import sys
if "google.colab" in sys.modules:
  self.client_host = "localhost"
  self.client_port = True

Ah, good point. This assumes google.clolab is only ever installed on colab... which is probably a safe assumption

Correct. The google.colab module can only be found on Colab. That module can't be found on any other Jupyter platforms or local computers.

Check out https://colab.research.google.com/drive/1h1iTDl4KqbXxKiyox9eM211vAg6_ONKT?usp=sharing (you have to restart the runtime after installing localtileserver)
0.6.4 now autodetects google colab and all should work

It worked like a charm! Thank you very much for this!


Podcasts fail to load on Android 6 devices if the SSL CA is Let's Encrypt

Describe the bug
Podcasts that use Let's Encrypt as their CA fail to load.
To Reproduce
Steps to reproduce the behavior:

Search for the 'Once Bitten' podcast.
Tap on the icon to load the podcast.
Busy spinner keeps on spinning,

Issue
The Let's Encrypt CA expired at the end of September 2021. This is causing issues with older Android devices which cannot update it's list of CA's. Later versions of Android are fine.

Fixed in latest feature branch. Testing before merging into master.


The boxes in the "Join Us Here" section are not of the same size.

Hay @SauravMukherjee44, This issue is already resolved by me. I request you to assign me this issue, and one more thing, there is a grammatical mistake that too has been resolved.😊

have a look at the issue.

Is there anyone else who can assign me to this.


[WIP] Group alerts based on user or platforms source

This PR will group the audit and infrastructure tenants into the platform alerts source, and application into user alerts source.
https://user-images.githubusercontent.com/5461414/223421482-0e6c10cd-b796-4768-956c-d000bdb7169e.mov
Image for testing: quay.io/gbernal/logging-view-plugin:alert-sources

LGTM


feat: Implement collections in x/oracle

@facundomedica , can you make it work to main?

This is ready to be merged, to do in follow ups:

Make QueryID consistent, use []byte everywhere if possible
Return errors anywhere needed, no panics
Misc cleanup, mostly check if the indexes added here are necessary or can be discarded


docs: v1alpha3 has changed so Keptn on non-k8s tutorial fails

The documented example for running Keptn in non-k8s no longer works because the spec for v1alpha3 has changed.
The existing YAML for a KeptnTask in the example looks like this:
apiVersion: lifecycle.keptn.sh/v1alpha3
kind: KeptnTask
metadata:
  name: runhelloworld1
spec:
  workload: "my-workload"
  workloadVersion: "1.0.0"
  appVersion: "1.0.0"
  app: "my-app"
  taskDefinition: helloworldtask
  context:
    appName: "my-app"
    appVersion: "1.0.0"
    objectType: ""
    taskType: ""
    workloadName: "my-workload"
    workloadVersion: "1.0.0"

But the v1alpha3 spec has now dropped spec.workload, spec.workloadVersion, spec.app and spec.appVersion to remove the redundancy and thus only pass these values in spec.context.
I think all that's required here is to remove the spec.... fields and things should be picked up from the context.... fields now.
Definition of Done
The example on the page above should work.

Confirmed that removing the spec... fields from the above does work. @StackScribe this should be a quick fix.


gtime在对time的零值格式化时不正确

1. What version of Go and system type/arch are you using?

go.1.15
2. What version of GoFrame are you using?

gf1.14.1
3. Can this issue be reproduced with the latest release?
4. What did you do?

like:
g.Dump(time.Time{}.String())
g.Dump("=>" + gtime.NewFromTime(time.Time{}).String() + "<=")
5. What did you expect to see?
一个有值，一个没有值，
在有些要求零值的datatime字段时会报错。
6. What did you see instead?
希望看看。

@jroam Yes, it's an empty string when you convert or print out gtime.Time. But if you do really want an initialized value, you cat do like this gtime.NewFromTime(time.Time{}).Time.String().


Fix compiler warnings for the iOS platform.

Fix compiler warnings for iOS platform. The related issue to it is #2715.

this won't work like this, I believe.
The macros are defined, but defined to "0", on the other platforms.
So TARGET_OS_IPHONE is defined, but defined to "0", building on Mac.
You will need to adjust your tests.
This has crept into the code in other places as well and will need fixing elsewhere too.

Do you know when TARGET_OS_IPHONE started to be ddefined to 0? I think this hasn't been the case in the past, has it?

Since at least 10.6:
{{{
cd /usr/include
$ ag TARGET_OS_IPHONE .
TargetConditionals.h
63:        TARGET_OS_IPHONE        - Generate code will run under iPhone OS which
92:    #define TARGET_OS_IPHONE            0
}}}

Ah right. if TARGET_OS_IPHONE is ok, which I think we use in some places. TThis one iss incorrecct indeed.

there's at least one more in the close_nocancel function to be fixed. Haven't searched for others as yet. Thanks!

@DeepakArora76 Can you include the feedback from the discussion above and rebase + tidy up the commit logs?

Code deleted by 47effc4bd39e4e13c59102bab3e789783725ecb6


Need clarification on max NICs for DCsv2 VM sizes

On the documentation for DCsv2-series VMs, the table states that the max NICs for each size is 2. Is this actually correct? The Azure CLI seems to return an error saying: "The number of network interfaces for virtual machine NAME exceeds the maximum allowed for the virtual machine size Standard_DC4s_v2. The number of network interfaces is 2 and the maximum allowed is 1."
 It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: aca0ec03-776b-fb48-3c1d-82dc11916534
Version Independent ID: 030f5f5d-992a-1ead-ec17-39faba84cef1
Content: DC-series - Azure Virtual Machines
Content Source: articles/virtual-machines/dcv2-series.md
Service: virtual-machines
Sub-service: vm-sizes-general
GitHub Login: @susaxen
Microsoft Alias: jushiman

Thanks for the feedback! We are currently investigating and will update you shortly.

Brenda is the PM for general purpose VMs, please sync with them on this issue.
@Karishma-Tiwari-MSFT
Their GH username: @brbell

@mmcrey, please help to address this issue on the DC-series.

Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

We are investigating this internally now, apologies I missed the prior tag from Brenda.

@joman-xmargin - Apologies for the long delay. It got assigned to the wrong person, reassigned to me and I missed it.
You are correct, there's a typo in the documentation. The hardware does support 2 vNICs, however it's misconfigured to allow a maximum of 1, it will take a few weeks to correct, but we've begun the work to fix. The individual who originally setup the configuration is no longer with the company, and hence it wasn't spotted.  In the interim, I updated the documentation to correct what is achievable today (which is 1 vNIC).
It's been a lot of time since you reported this, if you need 2 vNIC immediately, I recommend the DCv3 (https://docs.microsoft.com/en-us/azure/virtual-machines/dcv3-series) which are setup correctly to support 2-8 vNICs.
Hope this works. Please reach out to me via email if any other questions.

Thanks for providing feedback that helps improve our documentation. This issue has been resolved and we are closing the issue. #please-close


Add Callback Vars to flagset (#103)

$ ./basic -h

Usage:
  ./basic [flags]

Flags:
INFO:
   -n, -name string     name of the user
   -e, -email string[]  email of the user

ADDITIONAL:
   -ph, -phone string       phone of the user
   -add, -address string[]  address of the user
   -ut, -update             update this tool to latest version

$  ./basic -ut
checking if new version is available
updating tool....

@RamanaReddy0M Could you please also briefly update the readme file?


parseGame does not handle additional chars before move counter properly

When parsing a PGN, it seems additional chars before the move counter are not handled properly.
Notice the aa before move 10
1. e4 e5 2. Nf3 Nc6 3. Bb5 a6 4. Ba4 Nf6 5. O-O Be7 6. Re1 b5 7. Bb3 d6 8. c3 O-O 9. h3 Nb8 aa10. d4 Nbd7

When given to parseGame it returns:
...

    {
      "moveNumber": 9,
      "notation": {
        "fig": null,
        "strike": null,
        "col": "h",
        "row": "3",
        "check": null,
        "promotion": null,
        "notation": "h3"
      },
      "variations": [],
      "nag": null,
      "commentDiag": null,
      "turn": "w"
    },
    {
      "moveNumber": null,
      "notation": {
        "fig": "N",
        "strike": null,
        "col": "b",
        "row": "8",
        "check": null,
        "promotion": null,
        "notation": "Nb8"
      },
      "variations": [],
      "nag": null,
      "commentDiag": null,
      "turn": "b"
    },
    {
      "moveNumber": null,
      "notation": {
        "fig": null,
        "disc": "a",
        "strike": null,
        "col": "a",
        "row": "1",
        "check": null,
        "promotion": null,
        "notation": "aa1"
      },
      "variations": [],
      "nag": null,
      "commentDiag": null,
      "turn": "w"
    },
    {
      "moveNumber": 0,
      "notation": {
        "fig": null,
        "strike": null,
        "col": "d",
        "row": "4",
        "check": null,
        "promotion": null,
        "notation": "d4"
      },
      "variations": [],
      "nag": null,
      "commentDiag": null,
      "turn": "b"
    },
    {
      "moveNumber": null,
      "notation": {
        "fig": "N",
        "disc": "b",
        "strike": null,
        "col": "d",
        "row": "7",
        "check": null,
        "promotion": null,
        "notation": "Nbd7"
      },
      "variations": [],
      "nag": null,
      "commentDiag": null,
      "turn": "w"
    }
  ],
  "messages": []
}

Thank you for bringing that  to my attention, I will try to have a fix for that soon. Seems to be that the grammar is not strict enough. Btw, the move numbers will be ignored later on, so it is not so critical to have that ...

Checked it, and it seems some strange handling of 'aa' as a combination of discriminator? and column character (missing the row then??).  Seems to be more complicate than I thought.

Thinking twice, it seems just to be some legal notation, if you ignore the rules of chess, just take PGN as context. Let us see:

You have some normal moves, and have just read Nb8 (from black).
The next 3 characters are: aa1. The parser interprets:

a must be a discriminator, perhaps there are other options to go to a1.
a is then the target column, 1 the target row.

I have to think if there is some valid grammar rule, that ensures that combinations like that are then flagged as error. But hey, just typing anything in between legal notation may result in a legal move notation (not: move) ....

I have closed that issue, because I don't see any way to solve that in the parser. The parser has just a context free grammar, reading and interpreting the moves is done somewhere else. So no easy fix available.


docs(license): Update License Agreement terms

Motivation
Updating License Agreement as per Legal Team.

[OISY-122]


[WIP] Firefox Addon

This pull request aims to convert this web app to a Firefox Addon. The majority of the 100 files changed are simply moving everything at the root directory (not including the LICENCE and README) to the ./data directory, so they can be used by the addon.
What it does now:

Adjusts the user's settings to show this web app whenever a new window or tab is opened

What still needs to be done:

The URL bar still shows this ugly resource://... address, which really should be cleared
Tests? The Firefox Addon community seems to want tests, but I'm not sure they're applicable to this.

Cool and thanks for the help! I can write some tests/asserts to the angular side


How to enable onSwipeComplete and onBackdropPress propert at the same time?

Hello,
I have a problem like i write in title. My modal code is running when onSwipeComplete property is active. But it didn't run with onBackdropPress. I added <TouchableWithoutFeedback> instead of first <View>. At this time, onSwipeComplete property is not running correctly.
How can i run both of them at the same time?
You can my code as below which is running correctly with onSwipeComplete:
`<View>
          <Modal
            isVisible={visible}
            onBackdropPress={() => alert("ok.")}
            onSwipeComplete={() => this.onVisibleModal(false)}
            swipeDirection="down"
            style={{
              flex: 1,
              width: "100%",
              justifyContent: "flex-end",
              margin: 0
            }}
          >
            {this.renderModal()}
          </Modal>
        </View>
        </View>`

and renderModal() code:
renderModal = () => {
    const { source, pressedPhoto } = this.state;
    return (
      <View
        style={{
          flex: 1,
          justifyContent: "flex-end"
        }}
      >
        <View style={styles.containerModal}>
          <View style={{}}>
            <Text style={styles.modalTitle}>Grup Fotoğrafı Seç {height} </Text>
          </View>
          <View style={styles.containerScroll}>
            <Image
              style={{ width: 320, height: 320 }}
              source={{uri: source}}
            />
            <View />
          </View>

          <View style={styles.containerButton}>
            <TouchableOpacity
              activeOpacity={0.5}
              style={styles.buttonClose}
              onPress={() => this.onVisibleModal(false)}
            >
              <Icon
                name="x"
                size={styles.icon.fontSize}
                color="#47525E"
              />
            </TouchableOpacity>
            <TouchableOpacity activeOpacity={0.5}
            style={styles.buttonSave}
            onPress={() => {
              this.onVisibleModal(false);
              this.onSelectGroupProfilePicture(pressedPhoto);
            }}>
              <Text style={styles.buttonText}>Seç</Text>
            </TouchableOpacity>
          </View>
        </View>
      </View>
    );
  };

Anyone help ?

Hi!
I'm closing this issue because it's a question and not a bug report. Please don't take it personally, I'm just trying to make it easier to maintain this project by separating questions from bug reports 🙏.
If you're still looking for feedback/answers, could you please ask your question in a new discussion?
Thank you!


Modular compilation

Use this issue to track general modular compilation concerns.
This is very incomplete, but the general plan so far is this:
The binary format currently uses symbolic references, which generally become meaningless if the referenced file is recompiled. Name-based references were avoided in order to simplify generation of synthetic classes and members without needing to create unique names for them.
However, if we also emit a "header" file in which all method bodies are stripped out (but otherwise the same format), and that header file did not change, we can omit recompilation of dependent libraries because their symbolic references are known to remain intact.  And if the header file did change, the dependent libraries probably need recompilation for other reasons anyway.

Commit 3ac36acd4c54a3616ebb68d5cacb5b5c590a26c1 brings significant changes. I'm just going to paste its commit message here for reference:
The concept of a binary library file no longer exists.

A kernel file can contain any number of libraries, and some of these
libraries can be "external".  To reference a class or member from
another build, the class or member must be declared in an external
library.

Members in an external library contain all their type information,
but have no body.

Classes in an external library have their hierarchy information
present, but are not guaranteed to contain all their actual members.

The idea is that references themselves don't really cross module
boundaries, but rather refer to a local definition whose body is
contributed from elsewhere, much like 'external' members in Dart.

A modular backend such as DDC should be able to compile from one of
these kernel files without needing to load auxiliary information from
summaries or other kernel files.

For whole program transformations or backends, a linking step, which
is not yet implemented, must merge classes and members in external
libraries based on their name.

External libraries share the same IR and binary format as ordinary
libraries.  Transformations that affect the interface for a member
or class should transform the external libraries alongside with the
internal ones, ideally without needing to treat them any different.```


Support schema like django-hstore?

Any plans to support schema mode like django-hstore does?

Django hstore does a lot of things. This is not the scope of pgjson in my opinion.


Can’t see 4th option in menu

https://github.com/cqeta1564/maker_badge_fw/blob/88647deb2a6d44181ea10bfd973b126606164fd4/src/main.cpp#L143
Hi, there is a small problem in the code. All it needs is just to remove “Bold” from this line.
Thanks, M.

The problem was actually somewhere else. It's fixed now.


Feature/disable release status check

Disable release status check. Java package got released without approval. Even though release date is present in change log, change log parser didn't find date and as a result API check didn't enforce approval.
Disabling this check so all GA version package requires approval.

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow


Uncaught Error

Uncaught Error: EPIPE: broken pipe, write
Process exited with code 1
There is no problem with the CMD command line, but an error will be reported when sending a request in vs code debug mode

There is no problem with the CMD command line, but an error will be reported when sending a request in vs code debug mode

Can you give more details how you get this error? Reproducible steps and screenshots would be helpful.


refactor: overwrite ArbitrageMinGasPrice config from .005 to 0.1

Closes: #XXX
What is the purpose of the change
To address the failing arbitrage spam taking up block space, we will increase arbitrage-min-gas-fee to be near the order of magnitude of 0.1 OSMO.
CL Swap is around 600K gas wanted and 340K gas used. Let's assume a middle ground of 450K and aim to have the arb pay approximately 0.05 OSMO.
0.045 = 450_000 * 0.1 / 10**6

As a result, 0.1 is an appropriate choice.
Testing and Verifying
Tested the following vectors:

new config
does no overwrite other values
old value is "0.005" -> overwritten
old value is ".005" -> overwritten
old value is something else -> does not change

Documentation and Release Note

[ ] Does this pull request introduce a new feature or user-facing behavior changes?
[ ] Changelog entry added to Unreleased section of CHANGELOG.md?

Where is the change documented?

[ ] Specification (x/{module}/README.md)
[ ] Osmosis documentation site
[ ] Code comments?
[ ] N/A

why not make it just 1 osmo and leave this spam ahead?

why not make it just 1 osmo and leave this spam ahead?

Generally, I agree with increasing it even higher. I was worried about making too much of a rapid change for all nodes at once. This is already a 20x increase


Another version ruby

I have created another version of the fizz buzz algorithm  in Ruby.
I write another version because I just realised that it was already done lately.

Can't merge as it is now because all it does is adding a .DS_STORE file


启动报错ClusterBackend is not defined

启动之后报错NameError: name 'ClusterBackend' is not defined
Traceback (most recent call last):
File "C:\PyCharmProject\FunClip-main\funclip\launch.py", line 7, in
funasr_model = AutoModel(model="iic/speech_seaco_paraformer_large_asr_nat-zh-cn-16k-common-vocab8404-pytorch",
File "C:\Users\10379\AppData\Local\Programs\Python\Python310\lib\site-packages\funasr\auto\auto_model.py", line 138, in init
self.cb_model = ClusterBackend().to(kwargs["device"])
NameError: name 'ClusterBackend' is not defined

pip install funasr==1.0.25

(base) bobzheng@Bobs-MacBook-Pro FunClip % pip install funasr==1.0.25 sklearn
Looking in indexes: https://mirrors.aliyun.com/pypi/simple/
Requirement already satisfied: funasr==1.0.25 in /opt/anaconda3/lib/python3.11/site-packages (1.0.25)
Collecting sklearn
Using cached https://mirrors.aliyun.com/pypi/packages/46/1c/395a83ee7b2d2ad7a05b453872053d41449564477c81dc356f720b16eac4/sklearn-0.0.post12.tar.gz (2.6 kB)
Preparing metadata (setup.py) ... error
error: subprocess-exited-with-error
× python setup.py egg_info did not run successfully.
│ exit code: 1
╰─> [15 lines of output]
The 'sklearn' PyPI package is deprecated, use 'scikit-learn'
rather than 'sklearn' for pip commands.
  Here is how to fix this error in the main use cases:
  - use 'pip install scikit-learn' rather than 'pip install sklearn'
  - replace 'sklearn' by 'scikit-learn' in your pip requirements files
    (requirements.txt, setup.py, setup.cfg, Pipfile, etc ...)
  - if the 'sklearn' package is used by one of your dependencies,
    it would be great if you take some time to track which package uses
    'sklearn' instead of 'scikit-learn' and report it to their issue tracker
  - as a last resort, set the environment variable
    SKLEARN_ALLOW_DEPRECATED_SKLEARN_PACKAGE_INSTALL=True to avoid this error

  More information is available at
  https://github.com/scikit-learn/sklearn-pypi-package
  [end of output]

note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed
× Encountered error while generating package metadata.
╰─> See above for output.
note: This is an issue with the package mentioned above, not pip.
hint: See above for details.

pip install funasr==1.0.25 scikit-learn
...
Requirement already satisfied: sympy in /opt/anaconda3/lib/python3.11/site-packages (from torch->openai-whisper->funasr==1.0.25) (1.12)
Requirement already satisfied: networkx in /opt/anaconda3/lib/python3.11/site-packages (from torch->openai-whisper->funasr==1.0.25) (3.1)
Requirement already satisfied: jinja2 in /opt/anaconda3/lib/python3.11/site-packages (from torch->openai-whisper->funasr==1.0.25) (3.1.3)
Requirement already satisfied: fsspec in /opt/anaconda3/lib/python3.11/site-packages (from torch->openai-whisper->funasr==1.0.25) (2023.10.0)
Requirement already satisfied: regex>=2022.1.18 in /opt/anaconda3/lib/python3.11/site-packages (from tiktoken->openai-whisper->funasr==1.0.25) (2023.10.3)
Requirement already satisfied: MarkupSafe>=2.0 in /opt/anaconda3/lib/python3.11/site-packages (from jinja2->torch->openai-whisper->funasr==1.0.25) (2.1.3)
Requirement already satisfied: mpmath>=0.19 in /opt/anaconda3/lib/python3.11/site-packages (from sympy->torch->openai-whisper->funasr==1.0.25) (1.3.0)
(base) bobzheng@Bobs-MacBook-Pro FunClip % python3 funclip/launch.py
2024-05-08 08:32:10,092 - modelscope - INFO - PyTorch version 2.3.0 Found.
2024-05-08 08:32:10,092 - modelscope - INFO - Loading ast index from /Users/bobzheng/.cache/modelscope/ast_indexer
2024-05-08 08:32:10,114 - modelscope - INFO - Loading done! Current index file version is 1.14.0, with md5 b84e6dd4bc67ed4bbc318196f1d5760c and a total number of 976 components indexed
2024-05-08 08:32:10,643 - modelscope - INFO - Use user-specified model revision: v2.0.4
ckpt: /Users/bobzheng/.cache/modelscope/hub/iic/speech_seaco_paraformer_large_asr_nat-zh-cn-16k-common-vocab8404-pytorch/model.pt
2024-05-08 08:32:12,885 - modelscope - INFO - Use user-specified model revision: v2.0.4
ckpt: /Users/bobzheng/.cache/modelscope/hub/damo/speech_fsmn_vad_zh-cn-16k-common-pytorch/model.pt
2024-05-08 08:32:13,408 - modelscope - INFO - Use user-specified model revision: v2.0.4
ckpt: /Users/bobzheng/.cache/modelscope/hub/damo/punc_ct-transformer_zh-cn-common-vocab272727-pytorch/model.pt
2024-05-08 08:32:14,843 - modelscope - INFO - Use user-specified model revision: v2.0.2
ckpt: /Users/bobzheng/.cache/modelscope/hub/damo/speech_campplus_sv_zh-cn_16k-common/campplus_cn_common.bin
Traceback (most recent call last):
File "/Users/bobzheng/Documents/code/ai/partner/gao/FunClip/funclip/launch.py", line 7, in
funasr_model = AutoModel(model="iic/speech_seaco_paraformer_large_asr_nat-zh-cn-16k-common-vocab8404-pytorch",
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/opt/anaconda3/lib/python3.11/site-packages/funasr/auto/auto_model.py", line 138, in init
self.cb_model = ClusterBackend().to(kwargs["device"])
^^^^^^^^^^^^^^
NameError: name 'ClusterBackend' is not defined

Ensure scikit-learn>=1.3.0. It can be upgraded by running "pip install -U scikit-learn".

Thanks, it is OK


Use path-exists as fs.exists is being deprecated

Ha! @kevva already did it :p

Haha, missed this. Sorry :).


formula_installer: add env var to allow unbottled deps

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?
[ ] Have you successfully run brew man locally and committed any changes?

This will allow CI to be fixed for:

https://github.com/Homebrew/homebrew-core/pull/64175
https://github.com/Homebrew/homebrew-core/pull/64256
https://github.com/Homebrew/homebrew-core/pull/65300
https://github.com/Homebrew/homebrew-core/pull/66924

Review period will end on 2020-12-16 at 01:07:46 UTC.

A better short-term fix would be to pass --build-from-source in test-bot

Agreed that this is a better solution, this is more a test-bot issue than an actual brew issue.

Longer-term we should figure out how best to get actual bottles in Homebrew/homebrew-core to avoid slowing down CI runs so much by rebuilding unrelated formulae when they aren't the one being tested.

The long-term solution is to get the cellar information added at the right place in the bottle block, and fix brew merge. Once this is done we can start shipping bottles in homebrew-core. This might take a few months though because the code is complex, and it will take me some time to get it to work.
For the short-term, let's use the --build-from-source flag for the linux CI on homebrew-core. It allows maintainers to upstream the rest of the linux-only formulae while I continue to work on that cellar thing. It will also allow me to finish that endless xorg migration, I still need to migrate a few formulae, and for this I need the builds to be green in homebrew-core.
The number of builds for linux-only formulae is quite low IMHO, and the number of formulae is also low. It does not impact the mac builds at the linux builds will run on github-hosted runners, so build time is also less an issue.

Cool, closing this given that. https://github.com/Homebrew/homebrew-core/pull/66949 is the last step needed in the short-term fix.

Review period ended.


Update go-control-plane

Please provide a description of this PR:
Update go-control-plane dependency

@jacob-delgado: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-pilot-istiodremote-mc_istio
1ebd869001ec14eb843b4e2f1db58fe806bcd807
link
true
/test integ-pilot-istiodremote-mc

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

/test integ-pilot-istiodremote-mc

@jacob-delgado: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-pilot-istiodremote-mc_istio
1ebd869001ec14eb843b4e2f1db58fe806bcd807
link
true
/test integ-pilot-istiodremote-mc

integ-security-multicluster_istio
1ebd869001ec14eb843b4e2f1db58fe806bcd807
link
true
/test integ-security-multicluster

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

/test integ-security-multicluster


Does mdbook-catppuccin add help version info

E.g:
➜  mdbook-catppuccin -V
mdbook-catppuccin 0.1.1
➜

Right now, cli not show version help:
➜  mdbook-catppuccin -h
mdbook-catppuccin
A mdbook preprocessor that implements catppuccin flavours as default themes

USAGE:
    mdbook-catppuccin [SUBCOMMAND]

OPTIONS:
    -h, --help    Print help information

SUBCOMMANDS:
    help        Print this message or the help of the given subcommand(s)
    install     Install the necessary files needed and update the config to include them
    supports    Check whether a renderer is supported by this preprocessor
➜

Like mdbook cli:
➜  mdbook -V
mdbook v0.4.15
➜

Hi there, thanks for raising this!
I honestly didn't notice and am surprised to hear this wasn't included by default. Sorry about that!
I will make sure this is included in the next release where the assets have been updated!

okay


Update PointAnnotation.md

Description
The period is behind the br, I just moved it.
Fixes #
none

Added your feature that allows ...
Checklist

[ ] I have tested this on a device/simulator for each compatible OS
[ ] I formatted JS and TS files with running yarn lint:fix in the root folder
[ ] I updated the documentation with running yarn generate in the root folder
[ ] I mentioned this change in CHANGELOG.md
[ ] I updated the typings files (index.d.ts)
[ ] I added/ updated a sample (/example)

Screenshot OR Video

This was fixed by #1834 and could be closed.


Cleaning up the ActiveContainer test properly

Signed-off-by: Madhu Venugopal<EMAIL_ADDRESS>

LGTM

LGTM


how do I make splunk es to check my uploaded logs

I have installed splunk es app and uploaded botsv1.stream_http.json

but incident_review and ess_security_posture is not hitting any event

how do I make splunk es to check my uploaded logs and generate a list of alerts like below. Please note that I am not checking the logs forwarded by agent, but the log files uploaded on the browser side

thank you

The BOTS sample data is a single moment in time. So you need to ensure your ES Correlation searches are reviewing events for that time period.


Investigate potentional breaking changes in Envoy 1.20

Before shipping 1.12, we should take a close look at changes to Envoy for backwards compatibility
https://www.envoyproxy.io/docs/envoy/latest/version_history/current

config: due to the switch to using work-in-progress annotations and warnings to indicate APIs subject to change, the following API packages have been force migrated from v3alpha to v3
Impact envoyfilter users perhaps, but shouldn't impact us.
contrib images
https://github.com/istio/istio/issues/35336
dns_filter: dns_filter protobuf fields have been renumbered to restore compatibility with Envoy 1.18, breaking compatibility
possbile, but unlikely, envoyfilter breakage
extensions: deprecated extension names now default to triggering a configuration error. The previous warning-only behavior may be temporarily reverted by setting the runtime key envoy.deprecated_features.allow_deprecated_extension_names to true
VERY likely to break envoyfilters. We need to be very careful here - possibly set the flag for a bit
config: reduced log level for “Unable to establish new stream” xDS logs to debug.
we need to make sure our agent logging is up to date to make sense here
http: correct the use of the x-forwarded-proto header and the :scheme header. Where they differ (which is rare) :scheme will now be used for serving redirect URIs and cached content. This behavior can be reverted by setting runtime guard correct_scheme_and_xfp to false.
This stuff has broken our users constantly every release. Probably it will break users again - we will need to investigate more
access log: fix %UPSTREAM_CLUSTER% when used in http upstream access logs. Previously, it was always logging as an unset value.
might impact log output, but should be beneficial

cc @lambdai
@yangminzhu there are lots of ext_authz ones that may be worth a look

@lambdai @yangminzhu Are there any updates? Just want to make suer this is resolved before 1.12 needs to be released

nothing from my side, we have added the release note for 1.12, thanks for checking.


Print error on invalid call_group() calls

Closes #32186
If a method exists in the node, prints an error when the call failed.

Thanks!


⚠ Secret Website 3 has degraded performance

In b487160, Secret Website 3 ($SECRET_WEBSITE_3) experienced degraded performance:

HTTP code: 200
Response time: 2950 ms

Resolved: Secret Website 3 performance has improved in 6e5d00e.


feat: add CI for a scaled down version of a cancer WES pipeline

This PR adds a snakemake workflow that executes a snappy-pipeline (which in turn invokes snakemake, but, well…) to be run as as a CI target. The output is not validated yet, so for now this simply tests whether execution is successful or not.
The snappy-pipeline is a basic cancer_wes one with steps:

ngs_mapping
somatic_variant_calling
somatic_variant_annotation
somatic_variant_filtration

where ngs_mapping is done using bwa, variant calling is done with mutect2, variant annotation is done with vep, and filtration is done with bedtools.
The reference is restricted to chr12, the raw reads (from the TCRB dataset) are filtered down to those mapping to a specific region on chr12, and the vep cache is also restricted to the same region.
(It would probably make sense to restrict the trigger such that it only runs when any of the steps/wrappers used are changed, but I'll leave it set to run on PR and on push to main for now)

coverage: 85.742%. remained the same
when pulling 8d9198bbbc44c2e3ba4995bdaaf5d8bc046fdb1c on end-to-end-ci
into 6993233bc8b50641860d8a4bfc70b21a4ff09076 on main.


Run tests on ubuntu and windows

Codecov Report

Merging #19 (51019ae) into master (a3044b1) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master      #19   +/-   ##
=======================================
  Coverage   83.67%   83.67%
=======================================
  Files           2        2
  Lines          98       98
=======================================
  Hits           82       82
  Misses          8        8
  Partials        8        8

Impacted Files
Coverage Δ

govatar.go
78.94% <100.00%> (ø)

Continue to review full report at Codecov.

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


Double check A1Q1

Connection with https://github.com/lifewatch-eric/ecoportal_web_ui/issues/27
Seems that O'FAIRe can't deal with the array of identifiers.

fixed in #34


[FR] PSScriptAnalyzerSettings.psd1 hierarchy

I would like it if PSScriptAnalyzerSettings.psd1 was checked for based on a hierarchy:

Highest priority would be the workspace folder
Next to highest priority would be the the Packages/User folder
Previous to lowest priority would be the powershell.scriptAnalysis.settingsPath setting
Lowest priority would be the Packages/LSP-PowerShellEditorServices folder

Alternatively, have powershell.scriptAnalysis.settingsPath be an array with this as part of the default sublime-settings file:
		"powershell.scriptAnalysis.settingsPath": [
			"PSScriptAnalyzerSettings.psd1",
			"${packages}/User/PSScriptAnalyzerSettings.psd1",
			// "Recommended/placement/of/an/absolute/path",
			"${packages}/LSP-PowerShellEditorServices/PSScriptAnalyzerSettings.psd1",
		]

This helper package just provides a frontend for existing features of PowerShellEditorServices language server.
A feature request to use and maybe merge various settings files should probably be filed at language server's repository instead.

The point is powershell.scriptAnalysis.settingsPath to be a server-side setting to specify the configuration file. This plugin just provides a frontend for end users to modify it. The value is forwarded to the server as is without further evaluation.
It's up to the server to expand relative paths and lookup the file. The client (LSP / this plugin) is not involved.

Relative paths are resolved relative to root of workspace folders, by server.
Absolute paths can be used to force a certain file to be used.

As a workspace can contain multiple folders the server knows best which folder to search the file in, at any given time.
However project-specific settings can be used to create a lookup hierarchy.

A generic fallback can be defined in LSP-PowerShellEditorServices.sublime-settings
{
   "settings":
   {
      "powershell.scriptAnalysis.settingsPath": "${packages}/User/PSScriptAnalyzerSettings.psd1",
   },
}

A project-specific override can be specified via <Name>.sublime-project
{
   "folders": [
      {
         "path": ".",
      },
   ],
   "settings":
   {
      "LSP":
      {
         "PowerShellEditorServices": {
            "settings": {
               "powershell.scriptAnalysis.settingsPath": "PSScriptAnalyzerSettings.psd1",
            },
         },
      },
   },
}

Note

It's unlikely LSP-PowerShellEditorServices will provide a PSScriptAnalyzerSettings.psd1, thus related hierarchy is not relevant.

All settings in LSP-PowerShellEditorServices.sublime-settings are server defaults. So the value of powershell.scriptAnalysis.settingsPath is. As it is not intended to mess with users' project settings a hierarchy is to be set-up by users manually.

The illustrated strategy of using project-specific settings is aligned with VS Code's strategy of using workspace settings. The main difference is just VS Code using a .vscode/settings.json and ST relies on *.sublime-project files.


Merge from Virtomize - 2022-06-01

Checklist

[ ] mage test passes
[ ] unit tests are included and tested
[ ] documentation is added or changed

Fail to open


Test on my own data

Hi, Thank you for the awesome work.
I reproduce your work successfully and I want to use it to test with a VLP-16 lidar. The following figure is my .ply data:

The terminal has the following messages:
ERROR: In /build/vtk7-yd0MKW/vtk7-7.1.1+dfsg2/Common/ExecutionModel/vtkDemandDrivenPipeline.cxx, line 707
vtkCompositeDataPipeline (0x55e1b2536020): Input port 0 of algorithm vtkAppendPolyData(0x55e1b2535560) has 0 connections but is not optional.

I think the error is due to the sparsity of point cloud in the vertical direction, so I use another data:

But I got a same error:

Resolution of voxels is 0.01.
May I have your suggestions to fix this problem?
Any help is much appreciated:)

Hi akamob, I apologize for my late reply. I would suggest you to clear up the outlier of the input point cloud and try a larger resolution. For the applied methods and results of this program, you can find our research paper online now: https://www.nature.com/articles/s41598-022-05194-y. And I make a demonstration video to guide you how to use this program: https://youtu.be/XHyCV2DjnSg. Wish it helps.

Hi, @QiguanShu Thank you very much for your reply:)
I will read your paper and try to test.


Run the Linux Emulator on Windows OS

Is there any information how to run the Linux Emulator on Windows OS?
I'm interested in how to install the certificate for HTTPS as described here:
https://stackoverflow.com/questions/67839374/issue-with-https-connection-to-cosmosdb-emulator-in-linux-docker-container#67839374

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

ID: 0a59d2ab-cacd-5545-bddb-b9304f204605
Version Independent ID: bbcf9128-f9c3-fa84-b085-ba566f45d56f
Content: Run the Azure Cosmos DB emulator on Docker for Linux
Content Source: articles/cosmos-db/linux-emulator.md
Service: cosmos-db
GitHub Login: @StefArroyo
Microsoft Alias: esarroyo

Hi! Can you try using the Docker for Windows installation documentation? I think this will better fit your use case: https://docs.microsoft.com/en-us/azure/cosmos-db/local-emulator-on-docker-windows?tabs=cli

Hi @StefArroyo
It doesn't work with type=bind

Of course I can, but I am looking for the proper solution How to use Comsos Emulator with SSL certificate in Linux Container on Windows host?
It would be great to add some docs for that case (like existing docs for others Linux host, Mac host).
I'm sure that not only I will be run in on Windows. We need some docs. It is a request for docs.

@Marusyk why would you run a Linux container when your host is Windows OS based? We already have a Windows based Docker image and the documentation of how to run it and sounds like what you are trying to do for your case just does not make much sense. Please refer to: https://docs.microsoft.com/en-us/azure/cosmos-db/local-emulator-on-docker-windows?tabs=cli for instructions on how o do so.
If the "type=bind..." does not work for you, it's probably a configuration related issue, either the container does not have permissions to create that particular folder or it does not resolve/exist. Are you running from a regular Windows command prompt to allow %LOCALAPPDATA% to be resolved correctly?
You can also skip the whole "mount" option from your Docker command, but then you might not have access to the self signed emulator SSL certificate which is required by the Cosmos SDK based clients.

@milismsft

@Marusyk why would you run a Linux container when your host is Windows OS based?
because I (and my team)  use Windows for a development environment.  We use Visual Studio + Windows. But our services runs on Linux containers because we deploy them to a kubernetes cluster. A k8s doesn't support windows containers.
And you probably know that we cannot run Linux and Windows containers at the same time on Docker for Windows.

I will check in a few days

Hi @milismsft
But how can I use "mount"? What should I specify for destination?
"type=bind,source=%LOCALAPPDATA%\CosmosDBEmulator\bind-mount,destination=C:\CosmosDB.Emulator\bind-mount"

@Marusyk Are you asking how mount woks with Windows based Docker Cosmos DB emulator image or the Linux based on?
For Windows based replace in the above the "source" with some local path that you know the container will have full permissions, not just to read but also to write. For Linux based image, the default path for the emulator's data is "/tmp/cosmos" and you can opt to use that as the "destination" as long as the "source" you specify is fully accessible.
My recommendation is to try it out first in "attached" mode by overriding the start command for the image and instead run a batch/bash session (append the Docker start command line with "cmd.exe" or "/bin/bash" respectively).
For additional information on how Docker "mount" work please refer to https://docs.docker.com/storage/bind-mounts/

@milismsft thanks.
It seems I setuped https for my browser. But when I connect from VisualStudio app I got:

Response status code does not indicate success: RequestTimeout (408); Substatus: 0; ActivityId: c8a47800-e9b0-4331-99de-5691e3759b9e; Reason: (Message: Request timed out. More info: https://aka.ms/cosmosdb-tsg-request-timeout

@Marusyk The error you see in the VS app is most likely cause by how you setup your Cosmos client app to access the emulator's endpoint. By default a Cosmos client gets configured to access the endpoint in "Direct" mode and in your case, for the emulator running as a Docker container, you need to access that using the IP address of the instance rather than "localhost". Alternatively you can configure your application's Cosmos client to "Gateway" mode.
For any help troubleshooting emulator related issues, once you exhaust the steps we capture as part of the respective documentation page, there's an email address that you can use instead or just hit the Feedback/Report an issue link in the emulator's Data Explorer page.

@Marusyk I hope Milis's above comment helps you with the next steps. As mentioned above, we are documenting frequently used scenarios only. Because the outcome isn't a doc update, we will close this issue. This forum is for doc issues only, for technical troubleshooting can you please either:

Post the question on Microsoft Q&A or
File a support ticket

#please-close


Document MPMC behavior

I was surprised to find that cloning a Receiver and sending a message on the connected Sender results in the message showing up at only one of the downstream Receivers, but not both. Sending more messages results in a round-robin of sorts, where the Receiver next in line awaiting recv will get the next message.
This doesn't seem to be the implied MPMC behavior, though I'm not an authority on the matter.
Here's a minimal example in the form of a failing test:
    #[test]
    fn channel_sanity() {
        let (tx, rx1) = async_channel::unbounded::<u32>();
        let rx2 = rx1.clone();
        let t1 = smol::spawn(async move {
            let n = rx2.recv().await.unwrap();
            assert_eq!(n, 666);
        });
        let t2 =
            smol::spawn(async move {
            let n = rx1.recv().await.unwrap();
            assert_eq!(n, 666);
        });

        smol::block_on(async move {
            tx.send(666).await.unwrap();
            tx.send(123).await.unwrap();
            let ((), ()) = futures::future::join(t1, t2).await;
        });
    }

dup of https://github.com/smol-rs/async-channel/pull/37. see also https://github.com/smol-rs/async-broadcast

Ah, I see. I missed that update. Thanks for the clarity.


tfa ReduceLROnPlateau callback from Tf keras is not recognizing cohen kappa metrics direction in 'Auto' mode

Hi all,
the 'auto' mode in ReduceLROnPlateau  and ModelCheckpoint  are looking for specific string 'acc' in the name of the metrics to be monitor. this actually leads to unlickly scenarious of not working properly even while using metrics that are defined in tfa and hoping tf will be aware of the direction . this can be added in the doc to make the developers understand how to name their metrics or to set min max mode on their own.
thanks
https://github.com/tensorflow/addons/issues/1865

@yuvaramsingh94
Please provide details about what platform you are using (operating system, architecture). Also include your TensorFlow version.Request you to fill issue template.
Request yo to share colab link or simple standalone code to reproduce the issue in our environment.It helps us in localizing the issue faster.Thanks!

@ravikyram This Is as DOC improvment issue but he hasn't selected the doc template opening this.

I used colab. this is a simple one line code and its a doc issue . i do remember opening this issue under Documentation Issue .
at https://www.tensorflow.org/api_docs/python/tf/keras/callbacks/ReduceLROnPlateau
under Arguments -> mode we can find this line in auto mode, the direction is automatically inferred from the name of the monitored quantity.
this statement is misleading as it says auto mode can detect direction on its own . but, it does not work if the naming of the metrics does not fall within the criteria set in tf callback code as explained above ('acc' string is imp to determine the direction).
so requesting to modify this line to talk about the naming part of the metrics too. this is same for ModelCheckpoint  and few others.
thanks

Hi There,
This is a stale issue. As you are using an older version of tensorflow, we are checking to see if you still need help on this issue. Please test the issue with the latest TensorFlow (TF2.7 and tf-nightly). If the issue still persists with the newer versions of TF, please feel free to open it in keras-team/keras repository by providing details about the issue and a standalone code to reproduce the issue. Thanks!
Please note that Keras development has moved to a separate Keras-team/keras repository to focus entirely on only Keras. Thanks!


[ENH] Initial type and named object validator code

Reference Issues/PRs

What does this implement/fix? Explain your changes.

Does your contribution introduce a new dependency? If yes, which one?

What should a reviewer concentrate their feedback on?

Any other comments?

PR checklist

For all contributions

[x] I've reviewed the project documentation on contributing
[x] I've added myself to the list of contributors.
[x] The PR title starts with either [ENH], [CI/CD], [MNT], [DOC], or [BUG] indicating whether
the PR topic is related to enhancement, CI/CD, maintenance, documentation, or a bug.

For code contributions

[ ] Unit tests have been added covering code functionality
[ ] Appropriate docstrings have been added (see documentation standards)
[ ] New public functionality has been added to the API Reference

Codecov Report

Merging #122 (986a35e) into main (36167b7) will increase coverage by 2.83%.
The diff coverage is 95.68%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #122      +/-   ##
==========================================
+ Coverage   78.05%   80.88%   +2.83%
==========================================
  Files          23       30       +7
  Lines        1877     2197     +320
==========================================
+ Hits         1465     1777     +312
- Misses        412      420       +8

Impacted Files
Coverage Δ

skbase/validate/_types.py
76.19% <75.00%> (+13.69%)
:arrow_up:

skbase/lookup/_lookup.py
95.13% <100.00%> (-0.35%)
:arrow_down:

skbase/testing/utils/_conditional_fixtures.py
82.45% <100.00%> (ø)

skbase/utils/_iter.py
100.00% <100.00%> (ø)

skbase/utils/tests/__init__.py
100.00% <100.00%> (ø)

skbase/utils/tests/test_iter.py
100.00% <100.00%> (ø)

skbase/validate/__init__.py
100.00% <100.00%> (ø)

skbase/validate/_named_objects.py
100.00% <100.00%> (ø)

skbase/validate/tests/__init__.py
100.00% <100.00%> (ø)

...base/validate/tests/test_iterable_named_objects.py
100.00% <100.00%> (ø)

... and 3 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@fkiraly I've got updates and additional testing taken care of.
I'm thinking we initially restrict the named object format to the current (str, BaseObject) or Dict[str, BaseObject], then relax it in a follow-on PR. The reason I want to take a bit more time to get familiar with different use cases of named objects and consider if there are any specific patterns we want to check. For example, (str, BaseObject, str). And whether we want to allow certain dictionary formats for these options (maybe Dict[str, (BaseObject, str)] but that is just off the top of my head).


Do/new bar

Context

Implementation

Merge Checklist

[ ] Correct Target Branch
[ ] Pre-Review Diff Check
[ ] PR Description
[ ] Add Reviewers and Assignees
[ ] Review Complete
[ ] Rebase
[ ] Rebase Approved

Did you guys manage to get this on specific pages?


Add possible solutions to fix text overrun

As discussed in #22
Another alternative (and more concise solution) for this is in PR #44

I think this PR could bring better learning outcome than #44 with more alternatives. But to achieve this we need more elaboration, such as a side-by-side explanation mentioned in conversation in #44.


fix: Resolved incomplete error message for rating widget.

Description:

When the Allow Half Stars property is turned off and the user enters a decimal value for the Default Rating, an incomplete error message is displayed.
I have raised this PR in order to replace the error message for rating widget.

fixes: [Bug]: Incomplete error message for Rating widget #17654
Screenshots
Before resolving bug:

After resolving bug:

Summary by CodeRabbit

Enhancements

Updated validation message in the RateWidget component from "Allow half" to "Allow half stars" for better clarity.

Hello @riodeuno, Could you please review this PR when you have sometime.

Hi @shadabbuchh , Could you please review this PR
Thankyou!

Hi @shadabbuchh , Could you please review this PR

Cc: @jacquesikot

Hello @jacquesikot  , Could you please review this PR.


Module not found error when using Yarn 3 pnp

I'm getting this error when upgrading to Yarn 3 pnp:
Error loading the reporter: cypress-multi-reporters

We searched for the reporter in these paths:

 - /home/runner/work/provider-app/provider-app/cypress-multi-reporters
 - /home/runner/work/provider-app/provider-app/node_modules/cypress-multi-reporters

Someone else opened this same issue in the yarnpkg repo, but I think this is something that needs to be fixed here.

i don’t use pnp, you are welcome to fork the repo, try and fix it for your use case and provide a patch back 👍🏽


[libusb] Build error

Copy issue body from /home/anton/vcpkg/installed/vcpkg/issue_body.md

@J-YAA-i50-L Please provide the error log and steps to reproduce.

We don't have enough information to solve this issue, unfortunately; if you have any more information to help us solve this, please ping me and I will reopen this!


Promote configuration caching as a stable feature

The configuration cache is an incubating feature that significantly improves build performance by caching the result of the configuration phase and reusing this for subsequent builds.
This issue is about promoting the feature to stable.
This issue is not about making the configuration cache the default.
Once this issue is resolved the configuration cache can be considered a stable feature and requires to opt-in in order to use it.

Done through #24061


sudo make install with the default prefix (/usr/local) installs to locations it should not

Following the build instructions and not setting a prefix, or setting /usr/local for a system-wide install, the OE Makefile installs to the following locations:
$ grep -c "/usr/local/include/mbedtls" install_manifest.txt
70
$ grep -c "/usr/local/include/libc" install_manifest.txt
383
$ grep -c "/usr/local/include/libcxx" install_manifest.txt
175
This is going to come as an extreme and unwelcome surprise to a typical Linux developer, who will quite possibly already have the actual mbedtls library installed for example, and will find some parts of it altered.
All of the headers should instead be under /usr/local/include/openenclave, by default and when selecting /usr/local as a prefix.

Resolved by #815


Revisit transform function

Referencing pull request #119 and @harelba.
It still seems to me that this could be done easier (and potentially faster) without building your own module. Something along the lines of:
let content = fs.readFile(file);
let transform = file.test(/(^exports|module.exports)/m) ? require(file).transform : eval(content);

Note: The changes in Node v6.0 show that the module interface is not necessarily fixed and could change in the future (haven't checked if this implementation is broken in that version).
Also this needs to be documented in the readme.

Hi @mallocator , sorry for the late reply, I was a bit sick for a few days...
I believe that fn-encapsulate needs to be eventually treated as a 3rd party module and not an intrinsic part of es-exporter. In fact, I'll probably push it to npm at some point, relieving es-exporter from its source code and logic.
In regard to its implementation, I personally don't like eval(), since it allows for many issues which are mostly non-debuggable, not to mention the namespace mixups that can happen (e.g. someone will call the function with the same name as one of the es-exporter's function and override it without no one noticing).
In that regard, you are definitely right that compatibility with node v6.0 will be required by the fn-encapsulate module itself. I'll take a look at that and perhaps fix fn-encapsulate to something which will ensure compatibility.
btw, I have to say that me and my team have actually used the transform capability already several times, saving us hours of manual work and debugging.
Harel

Glad to hear that you guys can make good use of it. In the end that's why I published this tool to the community and still at least try to maintain it.
I agree, a separate module might be a better idea. Modulizer, ah no that already exists. Capsule - already taken. You'll find a good name. As a byproduct of so many things being broken by v6 right now, I realized that there has to be a way to get the Module version (new one is 48 I think). Based on that you could provide specific implementations for the individual interfaces, if there are such differences.
You could try to shield against eval()'s shortcomings, but I agree it's not the safest or easiest method. Somewhere I assumed that that was the original intention, maybe not in your pull request but somewhere in a community request. I think someone mentioned passing in a function from command line.
In any case, looking forward to the new module and I will integrate it when it's ready.

Great.
Really appreciate your efforts on this tool. I'm sure it helps lots of people all around.
I'll update once it's a separate module.
Harel

Yes, a little late, but I've moved on to other projects. Unfortunately, I can't point you to any other repo that forked and continued the work here, so you'll have to look for another solution, although I believe AI will soon make all of these little tools obsolete.
Most functionality was broken when ES moved to v6, and I don't even know what version we're on today, so it's no surprise it's broken. If there's huge demand I may come back to this as I was working on a more generic implementation with drivers for different DBs, a web GUI, and other little improvements. For now, consider this project abandoned 😢. One person can only do so much.


Formatting and small edits to "A Basic Segment Installation" [DOC-367]

@markzegarelli I have no idea why previous got added to this PR I created 🤷🏻♀


Resources get saved despite not belonging to the scene

Godot version:
Godot_v3.2.2-stable_x11.64
Issue description:
A node, which does not have an owner, has a variable; heightmap, which is not exported. On load an ImageTexture resource file is loaded to the hightmap. An EditorPlugin I'm working on uses heightmap.create_from_image(img) to change the ImageTexture. Then when I save something in the editor it will also save the changed ImageTexture.
This may be considered a bug, or intentional. But I certainly didn't expect it. I have many loaded resources and other files which are not resources (like json) so I want complete control of when they are saved for consistency and because it can take a while to save it all.
Using .duplicate() before changing, or just creating a new ImageTexture with the changes and then assigning to the variable prevents the editor from saving it. This makes sense as the resource which was loaded then remains unchanged, and the new ImageTexture doesn't even have a save location yet.
I think only resources which are referenced through an open scene, owned node, exported variable chain should be saved when saving in the editor.

The Inspector has the ability to keep multiple resources open simultaneously. Check its history dropdown. Likely the cause of some of this.

The resources are never viewed in the inspector.