proposal: x/crypto/x509roots: new module

#43958 is the accepted proposal for introducing the x509.SetFallbackRoots API, and had a rough outline of the bundle module. We want to have a slightly different API for the bundle package than was originally outlined there, so instead of continuing the discussion in the previous issue, I'm opening a new proposal that is tightly focused on x/crypto/x509roots (the name was previously decided on, this proposal will not rehash that discussion).
x/crypto/x509roots will be a submodule of x/crypto, containing the root package which contains the NSS certdata.txt parser, and a package x/crypto/x509roots/fallback, which registers the fallbacks on import.
API for x/crypto/x509roots:
// Package x509roots provides functionality for parsing NSS certdata.txt
// formatted certificate lists and extracting serverAuth roots. The parser
// provided by this package is very opinionated, only returning roots that are
// currently trusted for serverAuth. As such roots returned by this package
// should only be used for making trust decisions about serverAuth certificates,
// as the trust status for other uses is not considered. Using the roots
// returned by this package for trust decisions should be done carefully.
//
// Some roots returned by the parser may include additional constraints
// (currently only DistrustAfter) which need to be considered when verifying
// certificates which chain to them.
package x509roots

// NSSConstraint is a constraint to be applied to a certificate or
// certificate chain.
type NSSConstraint any

// NSSDistrustAfter is a NSSConstraint that indicates a certificate has a
// CKA_NSS_SERVER_DISTRUST_AFTER constraint. This constraint defines a date
// after which any certificate issued which is rooted by the constrained
// certificate should be distrusted.
type NSSDistrustAfter time.Time

// NSSCert represents a single trusted serverAuth certificate in the NSS
// certdata.txt list and any constraints that should be applied to chains
// rooted by it.
type NSSCert struct {
	// Certificate is the parsed certificate
	Certificate   *x509.Certificate
	// Constraints contains a list of additional constraints that should be
	// applied to any certificates that chain to Certificate. If there are
	// any unknown constraints in the slice, Certificate should not be
	// trusted.
	Constraints []NSSConstraint
}

// ParseNSSCertData parses a NSS certdata.txt formatted file, returning only
// trusted serverAuth roots, as well as any additional constraints.
func ParseNSSCertData(r io.Reader) ([]NSSCert, error)

API for x/crypto/x509roots/fallback (taken verbatim from #43958, other than package name changes):
// Package fallback embeds a set of fallback X.509 trusted roots in the
// application by automatically invoking [x509.SetFallbackRoots]. This allows
// the application to work correctly even if the operating system does not
// provide a verifier or system roots pool.
//
// To use it, import the package like
//
//	import _ "golang.org/x/crypto/x509roots/fallback"
//
// It's recommended that only binaries, and not libraries, import this package.
//
// This package must be kept up to date for security and compatibility reasons.
// Use govulncheck to be notified of when new versions of the package are
// available.
package fallback

cc @FiloSottile

LGTM. Only returning serverAuth makes sense per #26624.
One thing I'm worried about is that if we ever add a new constraint to NSSCert, code written against an old version will consider them unconstrained. Is it worth making the field a []NSSConstraint so code can panic on unknown values?
If we ever switch to a different root store, we can add new functions to x509roots and replace the one we use in fallback.
nit: do we usually use nil *time.Time or zero time.Time as the missing value sentinel?
Not necessary to ship the package, but we should soon after implement automation to notice changes to the pool, and mail CLs to update the submodule and file a govulncheck entry.

One thing I'm worried about is that if we ever add a new constraint to NSSCert, code written against an old version will consider them unconstrained. Is it worth making the field a []NSSConstraint so code can panic on unknown values?

Oh hm, good point. Do you imagine the NSSConstraint would be something like type NSSConstraint any and then we'd have type NSSDistrustDate time.Time etc, with a comment along the lines of "callers should not trust any certificate which contains constraints which they do not understand".

I couldn't come up with anything more elegant than that, so yeah :)

Note that the 1.20 release notes mention this package, which does not yet exist, so it would be good to prioritize this.  Or change the release notes.

We have an inflight CL for this, I'd like to get this in ASAP rather than updating the release notes, but at some point that may be prudent.

(I realized I forgot the golang/go prefix for the Fixes line 🤦)

This proposal has been added to the active column of the proposals project
and will now be reviewed at the weekly proposal review meetings.
— rsc for the proposal review group

Does the NSS prefix need to be on every function, or is that implied by the x509roots package name?

I think we previously discussed making the name of the package somewhat generic, in part in case we wanted to provide multiple different trust stores in the future (hence it's x509roots, not nssroots). Currently all of these functions and types are extremely specific to the NSS list, so if we strip the prefix some context is lost if we want to add non-NSS things in the future. It's not particularly pretty, but I think it somewhat future-proofs the package.

We need x/crypto/x509roots/fallback, which has no API.
Do we need to export a parser for NSS files?
Or maybe the NSS parser should be x509roots/nss?
Do we want to enable others to use the parser?
If we just need it for the fallback, it can be internal.

Oh sorry, I completely missed your last comment after we talked.
Switching to Certificate definitely makes sense and adding Kind on Constraint seems reasonable, and I like that it prevents everything from being a valid Constraint.
I think moving the module boundary probably does make sense. The initial rationale is as you described, along with giving us the ability to use entries in the vulnerability database as a way to signal to users that they should update the module when the baked in list changes.
One of the reasons we initially put the parser and fallback packages in the same module was that any changes to the former were likely to require an update to the latter, so bundling them together would allow us to avoid having to do chained tagging/dependency updates, but since we want to allow users to use just the parser, I think the argument you present does make sense (splitting the fallbacks into their own isolated module also means that if we do use vulnerability database entries as an update signal, we won't create noise in symbol/package unaware vulnerability scanners, which only operate at the module level, for users of just the parser code).
If we do make this split I guess the question is where do we put the module boundary. Should x/crypto/x509roots/fallback be the separate module, or should x/crypto/x509roots be the module, and we move the nss package somewhere else? I think the former probably makes more sense in terms of organizational clarity.

I agree that x/crypto/x509roots/fallback should be its own module; everything else is part of x/crypto.

Sounds good to me.

Based on the discussion above, this proposal seems like a likely accept.
— rsc for the proposal review group

No change in consensus, so accepted. 🎉
This issue now tracks the work of implementing the proposal.
— rsc for the proposal review group


Add vercel quick deployment

Add Vercel Quick Deployment
I really love this project! To make it easier for users to deploy their own JSON editor, I've added a Vercel quick deployment option.
Changes

Added Vercel deployment button and instructions to the README.
Included steps for setting up necessary environment variables after deployment.
Added @types/jest dependency to resolve Vercel deployment issues.

Rationale
Vercel offers a simple and fast way for users to deploy web applications. By adding this quick deployment option, we make it easier for individuals to set up their own instance of JSON For You, potentially increasing adoption and usage of the tool.
The addition of @types/jest was necessary to resolve type-related issues during the Vercel build process.

Testing
I've tested the Vercel deployment process and confirmed that it works correctly when following the provided instructions. Test link: https://json4u-psi.vercel.app/editor
Please let me know if you'd like any changes or have any questions about this pull request. Thank you for considering my contribution!

@rubickecho 感谢你的贡献，但是我没 get 到为啥用户会需要自行部署 vercel，可以解释一下吗？已经提供了 dockerfile 可以在本地部署，网站也提供了海外和国内两个版本。

@rubickecho 感谢你的贡献，但是我没 get 到为啥用户会需要自行部署 vercel，可以解释一下吗？已经提供了 dockerfile 可以在本地部署，网站也提供了海外和国内两个版本。

是的，这个需求应该偏个人向

期望以最简单的方式（动动鼠标）部署，dockerfile 还需要一些简单的服务器相关知识
数据隐私问题，有些工作开发的 json


PlayerTick not called anymore

I'm sure this worked before, but now it doesn't. PlayerTick somehow doesn't get called anymore, meaning the buff expiry isn't called, meaning quantum buffer doesn't expire (when it is marked to expire)

Nevermind it does get called (I was just testing it incorrectly). There must be another reason why the buff expiry wasnt working

I'm going to assume I did something wrong when I ran lua in the console while testing. If this happens again I'll find out soon enough. Closing for now


Only return matching URLs

@simonrjones can you please review this when you have time?
I had also considered that we could reduce the code here further by just using the foreach loop to handle results regardless of how many are returned (this would get rid of the if/else statement). What do you think?

Just to note as well that I've tested this fix locally, and it seems to work as desired.


Default battery last replaced date to device created date

Checklist

[X] I have filled out the template to the best of my ability.
[X] This only contains 1 feature request (if you have multiple feature requests, open one feature request for each feature request).
[X] This issue is not a duplicate feature request of previous feature requests.

Is your feature request related to a problem? Please describe.
Rather than unknown set the battery replaced date to a sensible default
Describe the solution you'd like
Is date-created available in the HA device registry? If so, would it be possible to write a one-time thing to go thru all devices in Battery Notes, and set “Battery last replaced” on any device having null / Unknown value, to the date that the device was added to HA? --Could be useful to provide a decent approximation of how old some of the batteries are.
Describe alternatives you've considered
N/A
Additional context
Date created/modified was added to HA 2024.8, when Battery Notes min version reaches that investigate if those dates can be accessed via the device

A fuller description is in the discussion #2223 but when creating the default last replaced data also add a default last seen.


Add language to user's profile

Add an editable optional field 'locale' to the user's profile. Display the UI according to the selected language.
Format 2 letters code (ISO 639-1).
Format BCP-47
Related to gbif/registry#213

This might already have been discussed and I just missed the discussion. If so, then I do not want to open it again.
But I'm confused about what our prefered language codes are these days. And perhaps it doesn't have to be the same.

We have 3 letter in the enum API http://api.gbif.org/v1/enumeration/basic/Language
2 letter, 3 letter and locale in https://api.gbif-uat.org/v1/vocabularyLanguage (btw @marcos-lg can 2 letter codes really be 3 letters long?)
The website use 2 letter with an optional country qualifier to capture traditional vs simplified chinese.
The registry use 3 letters for dataset language

With 2 and 3 letters language codes we cannot capture traditional vs simplified chinese. I do not know how much we intend to capture with these codes. Should we use something like BCP-47 instead of ISO 639-1?
I'm confused about where scripts recides - is it a part of language or locale or something else? As I understand it, writing system and country/regional variation in speech belong in the language code.

@MortenHofft let's use BCP-47 then. Matt tested that and it seems it suits us well

Backend functionality is on dev now

User's locale stored in the 'settings' field


Search not working on custom domain

Hello, thank you for making this great theme
But, I've problem with my website (https://bytadit.com)  which is not showing content highlight when typing some words on search.

I inspected its element and got some error messages :

According to correlated issue i found here (https://github.com/zzossig/hugo-theme-zzo/issues/214), i have a solution to change my baseURL in config.toml from https://bytadit.com into https://bytadit.netlify.app (because i hosted on netlify), and it works!!.

But i faced another problem, my address bar is always change to https://bytadit.netlify.app, even i type bytadit.com. I want it stayed to my custom domain (bytadit.com), not netlify domain. I know it's because of the changed baseURL, but if i change the baseURL to my own domain (like before), the search  doesn't work again. What should i do to solve that?

@bytadit  Just checking, if you have found a solution to this?
I have same issue for my site, search works on local but not on custom domain.
I have looked at this search issue 214 and all of it's comments but not helpful.
@zzossi  If you could help on this, would be great. Thanks.

I have resolved this now for my site  https://www.neerajsidhaye.com
I had same issue of CORS. I have custom domain on AWS Route 53 and was using AWS Amplify for build and publishing.
I am now my site live using GitHub Pages and using Git Actions for build, deploy and publish as soon I commit. I am still using custom domain on AWS Route 53.
I think, moving to github pages and adding a CNAME entry into the DNS ( route 53 ) , has resolved the CORS error and eventually the search.

I have resolved the search function for my site  https://www.neerajsidhaye.com
I had same issue of CORS. I have custom domain on AWS Route 53 and was using AWS Amplify for build and publishing.
Now, my site live using GitHub Pages and using GitHub Actions for build, deploy and publishing as soon I commit.
I am still using custom domain on AWS Route 53.
I think, moving to github pages and adding a CNAME entry into the DNS ( route 53 ) , has resolved the CORS error and eventually the search.

Hi,
Can you provide an example of the CNAME? I am familiar with it, however I would like to know the pattern. You do not have to provide a real domain name it can be example.com and how the CNAME record should look like. I suppose it will be different for Cloudflare, AWS, Google Cloud etc.

Hi @sysadmin-info
I just visited your site [https://sysadmin-info.netlify.app/en/]
It seems to be there is still an issue regarding to "search" function.
I have fixed it by entering the value of [CNAME/Custom domain] into the BaseURL on config.toml file instead the "abczyx.github.io"
That mean the correct will be BaseURL = "https://sysadmin-info.netlify.app/", let's try it on your side
You can visit here to take the explanation: #241

Hi,
I already fixed it a very long time ago but I do not remember now what I
did. The site is working very long at sysadmin.info.pl.
Kind regards
pon., 15 kwi 2024, 18:35 użytkownik Trinh Thanh Thien <
@.***> napisał:

Hi @sysadmin-info https://github.com/sysadmin-info
I just visited your site [https://sysadmin-info.netlify.app/en/]
It seems to be there is still an issue regarding to "search" function.
I have fixed it by entering the value of [CNAME/Custom domain] into the
BaseURL on config.toml file instead the "abczyx.github.io"
That mean the correct will be BaseURL = "
https://sysadmin-info.netlify.app/", let's try it on your side
You can visit here to take the explanation: #241
https://github.com/zzossig/hugo-theme-zzo/issues/241
—
Reply to this email directly, view it on GitHub
https://github.com/zzossig/hugo-theme-zzo/issues/456#issuecomment-2057300875,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFULAOVWFN5FU5DSAQF4PF3Y5P6VTAVCNFSM5H6KIXJKU5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TEMBVG4ZTAMBYG42Q
.
You are receiving this because you were mentioned.Message ID:
@.***>


Compile with 64 bit flags for Mac

This is needed for Mac OS X Catalina because it only supports 64 bit binaries.
When you run this you will see changes to libusbx and libwebsockets directories. Since these are submodules, I am not sure what the process is for updating them.
I am also not sure if there are any tests to make sure the binary still behaves the same compiled as 64bit vs 32bit.
Fixes #132

Thanks for this PR, I could compile them perfectly. As far as I checked (just webinterface and a few color-thingies with Julia) – it behaves the same.

This worked great for me as well. Really saved my butt on a project as I'd already assembled the hardware and done the testing on a different machine.


Remove bdist_wininst, Filter out distutils deprecation

Summary of changes
Python 3.10.0a5 related fix / workaround. See individual commits.
Closes https://github.com/pypa/setuptools/issues/2558
Pull Request Checklist

[ ] Changes have tests -- nope, not sure how
[x] News fragment added in changelog.d/ -- not yet

@jaraco Is this a way to go for now?

Apologies for letting this linger so long. I’ll try to get to it today.

At one point, it was proposed that Setuptools might continue to support bdist_wininst after the removal from Python, or at least it might enable support on Python 3.10+. This approach takes a different tack and accelerates the deprecation and imposes it on older Pythons. I'm okay with that, because those users can still use older Setuptools as a stopgap.

Amedned. I've also extended the filterwarnings commit message a bit.

@jaraco Could you please merge this PR? It has been approved 3 weeks ago.

Yes. Thanks for your patience.


activemq mysql persistent

mysql persistent not working: hanging in  INFO | Using Persistence Adapter: JDBCPersistenceAdapter(org.apache.commons.dbcp2.BasicDataSource@3e08ff24)
- is docker network problem? mq problem? or jdbc connection to mysql problem

is docker network problem? mq problem? or jdbc connection to mysql problem?


5.0.2 -> 4.1.4 Compatibility

Hello,
First, Thank you for such an excellent project.
A question: upon upgrade to 5.0.2 from a 4.1.4 based project I receive a number of compile errors.  Is 5.0.2 designed to be backwards compatible with 4.1.4 syntax and functionality?
Thank you!
-J

Good day!
If you need 4.1.4 version, you can install this version. It simple.
In 5.0 versions I do syntax more simple, I recommend you do small changes in your code. See Readme with examples usage, so simple)
Thanks for feedback!


linux: support hierarchies and cgroup2 for mem constraint lookup (rebase)

Rebase of https://github.com/libuv/libuv/pull/2323, which I'm hoping to help push over the line.
As far as I could see, here were only a couple of remaining review comments by @bnoordhuis:

You parse /proc/self/mountinfo to determine if the system uses cgroups v1 or v2? Is that necessary? Doesn't the format of the memory field in /proc/self/cgroups tell you that?
Would open("/sys/fs/cgroup/unified", O_DIRECTORY|O_CLOEXEC) work? It's a v2 system when it's there, it seems reasonable to assume v1 when it's not. (Related subject: I prefer openat() over path munging.)

I don't know whether the cgroupv2 controller is guaranteed to be mounted at /sys/fs/cgroup/unified. The cgroups man pages only state that modern systemd auto-mounts the cgroup2 filesystem there, nothing more general. So for now, I've left the mountinfo parsing in, but happy to change that.

I think it should fall back to /sys/fs/cgroup/memory/memory.limit_in_bytes if procfs isn't mounted. Right now it seems to default to 0, no limit.

Done.
cc @vtjnash @kjin

On an Alpine Linux (OpenRC, not systemd) system, the cgroup2 mount also appears at /sys/fs/cgroup/unified, if that's helpful at all.

I'll review this when I get back from vacation, around the 25th. Thanks for the PR, @maleadt.

@bnoordhuis Friendly bump; have you had the time to review this yet? Thanks!

I had this on my list of PRs to review this week but looks like I'm not going to get around to it due to time constraints.

Superseded by https://github.com/libuv/libuv/pull/3744


Getting evaluation metrics like prediction interval coverage probability(PICP)

Hello ,I would like to ask few questions on evaluation :
1.I am getting results like  "MSE": 3660.0208333333335,
"abs_error": 29178.109375,
"abs_target_sum": 19184.365234375,
"abs_target_mean": 33.306189643012154,
"seasonal_error": 2.728197371922594,
"MASE": 18.567732821857597,
"sMAPE": 1.3565907568101567,
"MSIS":<PHONE_NUMBER>427731,
"QuantileLoss[0.1]": 6339.818550300599,
"Coverage[0.1]": 0.3958333333333333,
"QuantileLoss[0.5]": 29178.10952758789,
"Coverage[0.5]": 0.7552083333333334,
"QuantileLoss[0.7]": 33780.337918281555,
"Coverage[0.7]": 0.7725694444444444,
"QuantileLoss[0.75]": 32101.51523065567,
"Coverage[0.75]": 0.7725694444444444,
"QuantileLoss[0.8]": 29448.640180969236,
"Coverage[0.8]": 0.7760416666666666,
"QuantileLoss[0.85]": 25970.12318687439,
"Coverage[0.85]": 0.78125,
"QuantileLoss[0.9]": 21592.694770431517,
"Coverage[0.9]": 0.796875,
"QuantileLoss[0.95]": 15795.05863990784,
"Coverage[0.95]": 0.8038194444444444,
"RMSE": 60.49810603096045,
"NRMSE": 1.8164223130715684,
"ND": 1.5209317075927002,
"wQuantileLoss[0.1]": 0.330467986448713,
"wQuantileLoss[0.5]": 1.520931715546463,
"wQuantileLoss[0.7]": 1.7608264597544852,
"wQuantileLoss[0.75]": 1.6733165178243905,
"wQuantileLoss[0.8]": 1.5350333368446543,
"wQuantileLoss[0.85]": 1.3537129255827818,
"wQuantileLoss[0.9]": 1.1255360553572664,
"wQuantileLoss[0.95]": 0.823329750395175,
"mean_wQuantileLoss": 1.2653943434692412,
"MAE_Coverage": 0.12352430555555556
now i want to calculatePI coverage probability (PICP), PI normalized averaged width (PINAW),coverage width-based criterion (CWC) which requires
1.Lower and upper bounds of ith PI
2.Predicted mean observed values
How these metrics can be used to give me the above required values

Hi @parimuns,
the metrics you wish to calculate (PICP, PINAW, CWC) are not currently implemented in the GluonTS Evaluator. If you want to compute them, you have two options:

Extend the Evaluator class to compute the metrics you want. This should be straightforward and is probably the way to go if you want to compute these metrics for several models going forward. If you go down this route you can also consider opening a pull request and contribute these changes back to GluonTS.

Obtain the forecast distributions directly from the model and do the evaluation "by hand", e.g. you can use

https://github.com/awslabs/gluon-ts/blob/4bf7f520c60af177942b4e513286a50650bd8e3d/src/gluonts/evaluation/backtest.py#L39-L46
to obtain pairs of true values and forecasts (represented as Forecast objects), which you can then compare to compute your required metrics.
Note that the Coverage[q] metrics in the output you posted are essentially "one-sided" versions of PICP, i.e. for a given quantile level q, Coverage[q] is the fraction of observations that falls below the q-quantile of the predictive distribution.

Thanks for the help.But if we want to calculate the CRPS ,how we can evaluate that ?

Hi @parimuns,
please see this answer from @jgasthaus below regarding CRPS (from #516):

Hi @alexhallam,
the QuantileLoss here is the exactly the pinball loss, scaled by a factor of two (to make it agree with the absolute error for q=0.5), defined as:
https://github.com/awslabs/gluon-ts/blob/08ab434b1e0946c21010ddbb4248288f8f043599/src/gluonts/evaluation/_base.py#L346-L352

Please reopen if you have further questions.


hillas_parameters_2 is inside ctapipe.reco.hillas, not ctapipe.reco

Small fix in camera_display.ipynb notebook.

in fact, you should just use hillas_parameters, not hillas_parameters_2 (the first is an alias for the second). I cleaned that up recently, so all notebooks should just drop the _2


Update _zaraxis.aiml

Merged my previous request with _zaraxis.aiml and submitted. This must be present!

No closing , syntax error.

:+1:

Only syntax error to fix now. :+1:
BTW, you can fix it by editing the file in your branch you want to merge. Let me know once it's ready.

Added closing li on the file in my branch. It apparently lost my change somehow, so I had to edit it all over again.

You updated master, while you want to merge patch-2.

Miyuutsu  wants to merge 1 commit into JustArchi:master from Miyuutsu:patch-2

You should edit patch-2 branch then: https://github.com/Miyuutsu/ArchiBoT/commits/patch-2

Still new to GitHub. Fixed.

:+1:


🔃 Jokes not refreshing quickly

Refreshing the jokes takes about an hour due to image caching.
Can be solved by solving #4

@karmanya007 this was the issue that you worked on, lol.

fixed in #18 by @karmanya007

😁

I'm pretty sure you don't need 1 hour of cache since you are not hitting any APIs you are just getting the jokes from the local file system.

@anuraghazra should I remove it? would it mess up the randomization time?

I think you can remove the caching safely and if you got lot more serverless execution in future you can make it 30min cache..

@anuraghazra thanks for the friendly tip! I would remove it.
Previously when I did that, on Vercel deployment it was working fine,
but on Github READMEs it was not working.
Would have to do some testing.


Use GCC for LD on MIPS processors

Summary
I built GCC 11.2 with crosstool-ng and found that the linking process failed.  This led me to issue #3826, which is the same issue but for ARM.  I applied effectively the same patch as #3836 in order to get it to compile
Impact
Seems to be minimal.  Other compilers may have issues.
Testing
Nuttx now compiles with newer versions of GCC.  This shouldn't have any effect on the actual compiled code.  I have not tested with other compilers apart from the one crosstool-ng built one.

@rm5248 there are some compile errors. Please reference the last arm Toolchain.defs too.

@rm5248 very nice!!! What board did you use for test? How did you generate the toolchain? I tried this way https://acassis.wordpress.com/2022/08/06/using-crosstool-ng-to-create-mips-toolchain/ but it didn't create a mips-unknown-elf-gcc! My goal is to get NuttX running on Dmitry visit card: https://dmitry.gr/?r=05.Projects&proj=33. LinuxCard and on this uMIPS emulador.

@xiaoxiang781216 I did notice an error and fixed it, it should be good now.  The build still seems to be failing, but that appears to be a different issue with the compiler.

@acassis Based off of this mailing list post, I just started with the mips-unknown-elf toolchain and modified it slightly to build for the PIC32

@xiaoxiang781216 I did notice an error and fixed it, it should be good now. The build still seems to be failing, but that appears to be a different issue with the compiler.

Look like the endian of compiled generated code is different from the expectation of link script. Please try the different value of OUTPUT_FORMAT in the link script.

@xiaoxiang781216 I did notice an error and fixed it, it should be good now. The build still seems to be failing, but that appears to be a different issue with the compiler.

Look like that the endian of compiled generated code is different from the expectation of link script. Please try the different value of OUTPUT_FORMAT in the link script.

Yes, the mips-unknown-elf toolchain is big-endian by default, I needed to change it to little-endian

@acassis Based off of this mailing list post, I just started with the mips-unknown-elf toolchain and modified it slightly to build for the PIC32

What did you do after "ct-ng build" ?

@acassis Based off of this mailing list post, I just started with the mips-unknown-elf toolchain and modified it slightly to build for the PIC32

What did you do after "ct-ng build" ?

Added it to the PATH and set CROSSDEV:
export CROSSDEV=mipsel-elf-
export PATH=/home/robert/x-tools/mipsel-elf/bin:$PATH

Here's the output from ct-ng savedefconfig:
CT_CONFIG_VERSION="4"
CT_ARCH_MIPS=y
CT_OMIT_TARGET_VENDOR=y
CT_ARCH_LE=y
CT_ARCH_FLOAT_SW=y
CT_STATIC_TOOLCHAIN=y
CT_LIBC_NONE=y
CT_CC_GCC_mips_plt=y
CT_CC_LANG_CXX=y
CT_DEBUG_GDB=y
CT_ZLIB_NEEDED=y

Thank you @rm5248 !! @xiaoxiang781216 maybe the issue is caused by incompatibility with pinguino after this change. But since Robert is also using little endian, but issue could be specific of pinguino

As far as I am aware, all of the PIC32 processors are always little-endian, so it seems that if there is a problem with the big vs. little endian that is a problem with the compiler that is used and/or the configuration of nuttx.

At least a while back I ran some testing under qemu. But looks like you might have also been on that thread.
https://lists.apache.org/thread/hk8gdtr495qbl2fg4yx1gfotm319npmy

after looking at this more, everything builds fine for me with the current master branch.  I think I was in some weird state where I had either tried to use two different compilers and/or I had leftover data from something old.
Closing as this is not an issue.


Validation issue in ratner stock

I am getting the following error while downloading the data sets
  File "/home/anand/dirichlet/TCPD/build_tcpd.py", line 115, in <module>
    main()
  File "/home/anand/dirichlet/TCPD/build_tcpd.py", line 110, in main
    func(name, script)
  File "/home/anand/dirichlet/TCPD/build_tcpd.py", line 85, in collect_dataset
    return run_dataset_func(name, script, "collect")
  File "/home/anand/dirichlet/TCPD/build_tcpd.py", line 81, in run_dataset_func
    func(output_dir=dir_path)
  File "./datasets/ratner_stock/get_ratner_stock.py", line 170, in collect
    write_patch(json_path, target_path=json_path)
  File "./datasets/ratner_stock/get_ratner_stock.py", line 65, in wrapper
    raise ValidationError(target)
tcpd.ratner_stock.ValidationError: ./datasets/ratner_stock/ratner_stock.json

Please see the downloaded dataset here
I am also facing issue #3

Fix released, please try again


Can't compile on macOS (zlib issue)

I am attempting to build on macOS 10.14.4.  I get this:
error: could not find native static library zlibstatic, perhaps an -L flag is missing?
I seem to be unable to make it happy about zlib, trying both the system zlib and zlib installed through homebrew.  It seems to be looking for zlibstatic, not sure if there is a naming issue here.
OpenEXR and ILMbase seem to be working fine.

Thanks for reporting your issue!
Have you tried setting the ZLIB_DIR environment variable that the openexr-rs build uses?
If you already have, and it's still failing, can you find zlib on your system and let me know what library files there are for it?
Unfortunately, I don't have an Apple computer to test on, so I'll need your help to get this sorted out.

The below are the system zlib
/usr/lib/libz.dylib
/usr/lib/libz.1.2.11.dylib
/usr/lib/libz.1.2.8.dylib
/usr/lib/libz.1.2.5.dylib
/usr/lib/libz.1.dylib
/usr/lib/libz.1.1.3.dylib

/usr/include/zlib.h

The below are zlib installed using homebrew
/usr/local/opt/zlib//lib/libz.dylib
/usr/local/opt/zlib//lib/pkgconfig
/usr/local/opt/zlib//lib/pkgconfig/zlib.pc
/usr/local/opt/zlib//lib/libz.1.2.11.dylib
/usr/local/opt/zlib//lib/libz.a
/usr/local/opt/zlib//lib/libz.1.dylib

/usr/local/opt/zlib//include/zlib.h
/usr/local/opt/zlib//include/zconf.h

I have tried ZLIB_DIR in many ways, including pointing it to a zlib that i compiled from source.  I'm sure I'm missing something basic here.  I have also tried manually changing "zlibstatic" in your build script to "libz" and that made Rust happy, but once it got to gcc, it failed.

Hmm... honestly, I'm pretty stumped then. :-/
@Ralith Do you have a MacOS system you could check this out with?

I do not. You'd probably want to set the lib name to "z", not "libz", or just use pkg-config.

@Ralith
Okay, thanks!
@bryab
Since neither me nor Ralith have MacOS systems to test on, I think we'll be of limited help.  Give Ralith's suggestions a try if you can, and if it works, let us know!  If not, then if you have the time to do a more thorough investigation yourself that would be amazing, but no obligation of course.  If you do figure out the issue, I will happily accept a pull request that fixes things on MacOS.
I'll leave this issue open until we have a fix.

Thanks a bunch for looking at this.  I will put some time into it when I can.

This was fixed by installing pkg-config via homebrew.


Compatibility detection between serialized models after version upgrades

Is your feature request related to a problem? Please describe.
Dependabot sent us https://github.com/nextcloud/mail/pull/3610 and I'm a bit hesitant integrating this change because our installations have existing models persisted and it's not obvious for a user of the library whether those have to be discarded or can be reused (class props are still the same in the new version).
For some environments it might be OK to "just" retrain the model before the new code goes into production, but it's not really the case for how we use rubix-ml in Nextcloud Mail. The model is trained in the background, for each user's account. If we discard the model there is a time window where no classification is possible. Some processes rely on this, though, so it would give sub-par UX at best.
Describe the solution you'd like
I think it should be technically feasible to have a version number or similar as property of each model. This number is touched (incremented) every time the data structure changes. Then if php successfully deserialized the class you can compare the restored value with the default of a new class. If there's a mismatch, you throw.
Describe alternatives you've considered
n/a
Additional context
I guess this is somewhat related to https://github.com/RubixML/RubixML/issues/108 and https://github.com/RubixML/RubixML/issues/58

Is "detection" enough to solve your issue @ChristophWurst?

@ChristophWurst @simplechris I like the idea of versioning persistable objects, however, if the serilaized data and class property definitions don't match up PHP will throw an exception upon calling unserialize(). We can catch this exception, however, we cannot read the version property in this case because we don't have an instance of the persistable object yet because it failed to deserialize. Is this going to be a problem?

If php can't even unserialize the string then we are out of luck anyway. I'm more worried about the nifty cases where for example a property was nullable before but in the new version of the code it's always assigned a value. Php will (I think) happily unserialize. The resulting object is semantically faulty. Does that make sense?
I did this quick test in two interactive shells:
Shell 1
php > class A {
php {   private $a;
php {   private $b;
php {
php {   public function __construct() {
php {     $this->a = 3;
php {   }
php { }
php >
php > $a1 = new A();
php > echo serialize($a1);
O:1:"A":2:{s:4:"Aa";i:3;s:4:"Ab";N;}
php > file_put_contents('/tmp/a.ser', serialize($a1));

Shell 2
php > class A {
php {   private $a;
php {   private $b;
php {
php {   public function __construct() {
php {     $this->a = 4;
php {     $this->b = 2;
php {   }
php { }
php > $a1 = new A();
php > echo serialize($a1);
O:1:"A":2:{s:4:"Aa";i:4;s:4:"Ab";i:2;}
php > $a1ser = file_get_contents('/tmp/a.ser');
php > echo $a1ser;
O:1:"A":2:{s:4:"Aa";i:3;s:4:"Ab";N;}
php > $a1orig = unserialize($a1ser);
php > var_dump($a1orig);
php shell code:1:
class A#2 (2) {
  private $a =>
  int(3)
  private $b =>
  NULL
}

As you can see, php happily unserializes the object because no new properties were added. Even worse, I tried with a third shell to see what happens if you add a new property:
php > class A {
php {   private $a;
php {   private $b;
php {   private $c;
php { }
php > $a1ser = file_get_contents('/tmp/a.ser');
php > $a1orig = unserialize($a1ser);
php > var_dump($a1orig);
php shell code:1:
class A#1 (3) {
  private $a =>
  int(3)
  private $b =>
  NULL
  private $c =>
  NULL
}

So php is quite "flexible" and will just set the new properties to null. There was not an exception because of a mismatching property.

@ChristophWurst yup makes sense, thanks for the information. I've had similar results. To me, it seems like the best solution is one implemented in the PHP core no?
As @simplechris mentioend, we are working on revamping the entire persistence subsystem for version 1.0.0. Maybe we can come up with something if we put our heads together.

What if we put a $version property on every Persistable object and compared that to a class constant that we increment everytime the class definition changes? Perhaps this could be done using the new PHP 7.4 __unserialize() magic method. If there is no $version property (it's null as I believe this would be the behavior or unserializing an object without the version property) then we can assume the version is 0.
Any thoughts?

Also, this is sorta related. In this commit https://github.com/RubixML/RubixML/commit/59bafb781dd70e03d16c2771551c75c8ef287fb2 we now catch missing class definitions (missing, not semantically faulty like you say) and throw a more helpful exception in both Native and Igbinary serializers.

@ChristophWurst A prototype solution to this can be found in https://github.com/RubixML/ML/pull/158
The idea is a custom file file format with metadata and automatically tracking class revisions. Your early feedback would be appreciated!

@ChristophWurst A prototype solution to this can be found in https://github.com/RubixML/ML/pull/158
The idea is a custom file file format with metadata and automatically tracking class revisions. Your early feedback would be appreciated!


Store * and + operator implementations in TypeValues

Certain operations like + or * cannot be defined on LHS.
For example to be able to embed Starlark in Bazel, implementation
should be able to override + for list and select types:
[] + select({...})

To solve this issue Python types define a pair of operators, for
plus it is __add__ and __radd__. Former is called on LHS, and
if it is not defined, __radd__ is used on RHS.
We could do the same, but alternatively we can store operator
implementations for each type pair in TypeValues object, this way
operators become symmetrical between LHS and RHS.
This diff implements storing + and * in TypeValues, the rest
operators (including unary operators) are still implemented as
virtual functions of TypedValue.

This PR is obsolete, closing


Add a bit of CI for the project

Currently when we are pushing something in the master branch we do not perform any checks at all and sometimes our containers are broken after commits which seemed to be pretty safe. Like that case when updating a minor version of the osprey package screwed everything up.
I was wondering if there is a way of running docker-compose up in Travis CI and checking at least that localhost:8055 gives you a 200.

This could be accomplished by running a simple mocha test to do the above


Next 15 breaking Async Request APIs

Description
With the breaking change of the Next.js version 15 of the Async Request APIs we need to change the implementation of receiving the headers in Server-Side Rendering.
Currently the following error occurs, when trying to access const t = useTranslations() in an Server Component:
[ Server ]  Error: Route "/[locale]" used `headers().get('X-NEXT-INTL-LOCALE')`. `headers()` should be awaited before using its value. Learn more: https://nextjs.org/docs/messages/sync-dynamic-apis

Verifications

[X] I've verified that the problem I'm experiencing isn't covered in the docs.
[X] I've searched for similar, existing issues on GitHub and Stack Overflow.
[X] I've compared my app to a working example to look for differences.

Mandatory reproduction URL
https://github.com/amannn/next-intl-bug-repro-app-router
Reproduction description
git clone https://github.com/amannn/next-intl-bug-repro-app-router
cd next-intl-bug-repro-app-router
pnpm i
pnpm up next@^15
pnpm dev

Expected behaviour
Repository should be updated to support next@15

duplicate of https://github.com/amannn/next-intl/issues/1375#issue-2552102735

Please see: https://github.com/amannn/next-intl/issues/1375#issuecomment-2439437363


Added usage instructions for config-engine-lite, fixes #68

Use "hltopo.json" or "lltopo.json" as example filenames to distinguish one from the other. I think "my-topology.json" might be unclear to the user which one it refers to.

You're right, we should use consistent naming. Latest commit fixes this.


Fix case tLetin in function map_constr_with_binders

The body of the letin was accidentally overriden

Thanks, good catch!


Update and rename sources/talk/20211213 How I use open source to desi…

…gn my own card games.md to translated/talk/20211213 How I use open source to design my own card games.md

提交译文


Staging maclist blacklist whitelist

Subject: Added feature so ,that the Cloud SDK can define a list of unauthorized devices by MAC address
Test Done:

/usr/opensync/tools/ovsh u Wifi_VIF_Config mac_list_type:=blacklist/whitelist/none mac_list:=
'["set",["aa:4e:3f:f1:c9:60","0C:2F:B0:27:0D:80"]]' -w if_name==home-ap-u50 /home-ap-24
uci show or cat /etc/config/wireless

Hi John,
I’m using Eclipse ,it has default indentation styles. It is changing indentation styles based on that. I’ll figure it out.
Thank you.
Nagendrababu.
From: John Crispin<EMAIL_ADDRESS>Reply-To: Telecominfraproject/wlan-ap<EMAIL_ADDRESS>Date: Tuesday, July 14, 2020 at 3:11 AM
To: Telecominfraproject/wlan-ap<EMAIL_ADDRESS>Cc: Nagendrababu Bonkuri<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [Telecominfraproject/wlan-ap] Staging maclist blacklist whitelist (#12)
@blogic commented on this pull request.

In feeds/wlan-ap/opensync/src/platform/openwrt/src/lib/target/src/vif.chttps://github.com/Telecominfraproject/wlan-ap/pull/12#discussion_r454149685:

       SCHEMA_SET_STR(vstate.mac, mac);

  else

          LOGN("%s: Failed to get base BSSID (mac)", vstate.if_name);

  vif_state_security_get(&vstate, tb);

  if (vconf) {

         LOGN("%s: updating vif config", radio);

         vif_state_to_conf(&vstate, vconf);

         radio_ops->op_vconf(vconf, radio);

  }

  LOGN("%s: updating vif state", radio);

  radio_ops->op_vstate(&vstate, radio);

  return true;

 struct blob_attr *tb[__WIF_ATTR_MAX] = { };

Hi,
your editor has reformatted a whole function again. What editor are you using if I may ask ?
John
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/Telecominfraproject/wlan-ap/pull/12#pullrequestreview-447844574, or unsubscribehttps://github.com/notifications/unsubscribe-auth/APZR4XQWSUGUL4HKAERJIGLR3QAKBANCNFSM4OY2K74A.

Hi,
Yes I did, I was working on staging-osync2.0. So I checked out to pending branch, and did merge my previous working branch on staging-osyn2.0 to local branch on pending. Because of that it might stuck in?
Thank you,
Nagendrababu.
From: John Crispin<EMAIL_ADDRESS>Reply-To: Telecominfraproject/wlan-ap<EMAIL_ADDRESS>Date: Tuesday, July 14, 2020 at 3:12 AM
To: Telecominfraproject/wlan-ap<EMAIL_ADDRESS>Cc: Nagendrababu Bonkuri<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [Telecominfraproject/wlan-ap] Staging maclist blacklist whitelist (#12)
@blogic commented on this pull request.

In patches/wlan-ap/0007-ipq40xx-TP-Link-AP2220-support.patchhttps://github.com/Telecominfraproject/wlan-ap/pull/12#discussion_r454150201:

@@ -233,6 +233,7 @@ index<PHONE_NUMBER>..1746f51d54

                status = "okay";

                switch_mac_mode = <0x0>; /* mac mode for RGMII RMII */

                switch_initvlas = <0x0007c 0x54>; /* port0 status */

++                     witch_initvlas = <0x0007c 0x54>; /* port0 status */
this must have snuck in, did you do some merge commit with git causing this ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/Telecominfraproject/wlan-ap/pull/12#pullrequestreview-447845197, or unsubscribehttps://github.com/notifications/unsubscribe-auth/APZR4XQEKNWL7WQVD7EAWZDR3QAN5ANCNFSM4OY2K74A.

merged into pending, Thanks !

merged into pending, Thanks !

Thank you John, for your inputs.


Build against libxcrypt on Linux

sulogin is currently linked against libcrypt.so.1 which we want to phase
out. For details, see:
https://github.com/conda-forge/linux-sysroot-feedstock/issues/52

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

The logs show
INFO (util-linux,sbin/sulogin): Needed DSO lib/libcrypt.so.2 found in conda-forge/linux-64::libxcrypt==4.4.36=hd590300_1

, which is the change we want.

Adding automerge label with blessing from https://github.com/conda-forge/util-linux-feedstock/pull/37#issuecomment-1868320576 :

lets see if this works, if so, you could just simplify your conda-build-config file in the merge request you are proposing and merge it.

:bow:

i  was faster ;)

i was faster ;)

Gonna get you next time :P.

🐄 🔫

🤠


Close workers & dispatchers when all connections are closed

Or else applications using it will never shutdown, since the worker / dispatcher thread persists.

Fixed by https://github.com/gudzpoz/r2dbc-jdbc/commit/d5ab0605e880150e5613af3bcc7b8fe5e82c47d5


bug: angular 9 cli livereload -l cause cordova_not_available cordova.js not injected

Bug Report
Ionic version:

[x] 5.x
Current behavior:

ionic cordova run ios -l
casue cordova_not_available
Expected behavior:

works fine with angular 8 cli 8.3.25
Steps to reproduce:

https://github.com/mohamedsharaf/i5ng9
clone repo and run in ios simulator or device with live reload
npm i
ionic cordova run ios -l
you will see that cordova not injected cordova_not_available
Related code:

insert short code snippets here

Other information:

Ionic info:

Ionic:

   Ionic CLI                     : 6.0.2 (/usr/local/lib/node_modules/@ionic/cli)
   Ionic Framework               : @ionic/angular 5.0.0
   @angular-devkit/build-angular : 0.900.1
   @angular-devkit/schematics    : 9.0.1
   @angular/cli                  : 9.0.1
   @ionic/angular-toolkit        : 2.1.2

Cordova:

   Cordova CLI       : 9.0.0<EMAIL_ADDRESS>   Cordova Platforms : ios 5.1.1
   Cordova Plugins   : cordova-plugin-ionic-keyboard 2.2.0, cordova-plugin-ionic-webview 4.1.3, (and 4 other plugins)

Utility:

   cordova-res : 0.9.0
   native-run  : 0.3.0

System:

   Android SDK Tools : 26.1.1 (/Users/mohamed/Library/Android/sdk/)
   ios-deploy        : 1.10.0
   ios-sim           : 8.0.2
   NodeJS            : v12.11.1 (/usr/local/Cellar/node/12.11.1/bin/node)
   npm               : 6.13.7
   OS                : macOS Catalina
   Xcode             : Xcode 11.3.1 Build version 11C504

mohamed@mbp i5ng9 %

Thanks for the issue. It looks like the issue is caused by this: https://github.com/ionic-team/angular-toolkit/issues/184.
There is a PR that is open and should be merged soon: https://github.com/ionic-team/angular-toolkit/pull/201

This is still a problem for me
Ionic:

   Ionic CLI                     : 6.1.0 (/usr/local/lib/node_modules/@ionic/cli)
   Ionic Framework               : @ionic/angular 5.0.0
   @angular-devkit/build-angular : 0.900.2
   @angular-devkit/schematics    : 9.0.2
   @angular/cli                  : 9.0.2
   @ionic/angular-toolkit        : 2.1.2

Cordova:

   Cordova CLI       : 9.0.0<EMAIL_ADDRESS>   Cordova Platforms : not available
   Cordova Plugins   : not available

Utility:

   cordova-res : not installed
   native-run  : 0.3.0

System:

   ios-deploy : 1.10.0
   ios-sim    : 8.0.2
   NodeJS     : v10.16.3 (/usr/local/bin/node)
   npm        : 6.13.7
   OS         : macOS Catalina
   Xcode      : Xcode 11.3.1 Build version 11C504

@danieljameswilliams Can you please open an issue here: https://github.com/ionic-team/angular-toolkit along with a reproduction?


Fix/improve webfields

Support referrer parameters in all the webfields.
Show banner to the parent group for all the webfields, including landing pages.
Make the decision home page optional when running post decision stage.
Handle 'Reject' tab as optional.
Compute 'venue' and 'venueid' for accepted papers. (In the future we can show the decision page using the venueid instead of querying the decision notes).
Refactor home page webfield, some options were in the header json and we don't need them.

Please review, feedback is welcome!

Thanks @zbialecki , I have a question for you: for the roots groups I'm not using the banner, I'm calling to the wecolme function, do you think we should add a link to go back to the openreview home page or all venues?

Honestly for that case I would probably not show the banner at all. If you don't call one of the banner functions it defaults to being hidden.

Honestly for that case I would probably not show the banner at all. If you don't call one of the banner functions it defaults to being hidden.

Do you mean to not call OpenBanner.welcome()?

Yes


Opensource : 6.15 Explain everything about Ember.js

Description 📜
Opensource : 6.15 Explain everything about Ember.js
NOTE 👇
Please put your doc 6.Popular_Projects with the name 6.15_EmberJS.md

Domain
Open Source
Type of Contribution
Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

/assign

@Harshkumar62367 Any updates on this?

I will submit a pr tonight sir.


UI-router $state.go has more than one implementation

$state.go and $state.transitionTo accept either a String or a state object (IState). The interface only specifies the String version.

I have a pull request here: https://github.com/borisyankov/DefinitelyTyped/commit/6612806e0a1be58b4cfe37775c0f48ec3809fcfd

closed by #5653


Features: Devise

Hello reviewer! :wave:
This PR uses the devise gem to implement authentication.

Can register a new user.
User logs in with a combination of email and password.
Hashed passwords should be stored in the database.
Ask for confirmation of email.
Modifies the views in devise for registration and login to match the wireframes and your styling.

After successfully singing up, remember to confirm your account. Since we don't have an email service yet, do it manually from the rails console like this:
user = User.last
user.confirmed_at = Time.now
user.save

Now you should be able to log in without a problem
Additionally, updates the README file to include how to run fast reload with Webpack.
Thank you for your feedback, is greatly appreciated! :fireworks: :rocket:

Hi @D4nielJ 👋,
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉

Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


feat: Add cargo machete

closes #439

adding cargo machete to check unused dependencies
CI fails on error

Thanks a lot for your contribution. However, the ASF has it's own limitation about actions that can use:

bnjbvr/cargo-machete@main is not allowed to be used in apache/iceberg-rust.

We will raise a ticket to the ASF infra to review and allow this action.

I used that repo because that is why the github readme recommends for cargo machete.
we can also run cargo machete on our own and parse its output. please let me know if that would make the process easier @Xuanwo

Thanks @vaibhawvipul for this pr, due to ASF infra policy, this third-party github action is not allowed. I've left some suggestion to improve.

Thank you.

due to ASF infra policy, this third-party github action is not allowed. I've left some suggestion to improve.

I'm going to submit a ticket to the ASF infra. We can wait for a bit.

Cool, let wait for a moment.

Apologies, I missed some discussion about using cargo install directly. I'm fine with merging this PR first and switching to GitHub Action versions once it's approved.


UCP: Show progress when switching frontend page

Description
This task is to show progress by the progress bar when switch frontend pages.
Score

301

Mentor(s)

@baurine

Contact the mentors: #tidb-challenge-program channel in TiDB Community  Slack Workspace
Recommended Skills

React

@baurine  Hi, mentor, is react-top-loading-bar ok for this ?

Looks like a good idea


Use of !$ in ruby 1.9

GeoKit uses the pattern "message string" + !$
!$ is the exception object and is not implicitly convertible to a string (at least in ruby 1.9)
Thus the catenation fails and throws another exception. I replaced with, e.g. in GoogleGeocoder:
      logger.error "Caught an error during Google geocoding call: address: #{address.to_s} error: #{$!.inspect}"

adding .to_s explicit string conversion to the catenation would also work, but this shows both the error class and the message.

I have stumbled upon this $! in one of the forks (very old one) that the app which I maintaint use. This has already been rewritten in the gem

Only found one use of this, in geocoders.rb, and it appears to work. Open a new issue if you see more problems, thanks :)


Feature: Allow a second row of tabs

This has been tested in the wild (it is deployed currently) and I am confident that this update is backwards compatible. However, I'll wait for Götz to have a look at it before merging.

At the moment, the number of bottom and top tabs is fixed. If a combination does not exist, the frame slot remains empty.
In the future, the second row of tabs should depend on the first: Given some top tab is selected, only show bottom tabs that we have data for.


Getting Started web editor link to ml5 account

Filing an issue for to linking to an ml5 web editor sketch whenever this gets merged and the example upload script runs. https://github.com/ml5js/ml5-examples/pull/143

We have this live now!


Adds SecuritySchemes to merged generated open api spec file

Relates with: #qubIT-Omnis-4314

PR checklist

[ ] Read the contribution guidelines.
[ ] Pull Request title clearly describes the work in the pull request and Pull Request description provides details about how to validate the work. Missing information here may result in delayed response from the community.
[ ] Run the following to build the project and update samples:./mvnw clean package
./bin/generate-samples.sh
./bin/utils/export_docs_generators.sh

Commit all changed files.
This is important, as CI jobs will verify all generator outputs of your HEAD commit as it would merge with master.
These must match the expectations made by your contribution.
You may regenerate an individual generator by passing the relevant config(s) as an argument to the script, for example ./bin/generate-samples.sh bin/configs/java*.
For Windows users, please run the script in Git BASH.
[ ] In case you are adding a new generator, run the following additional script :./bin/utils/ensure-up-to-date

Commit all changed files.
[ ] File the PR against the correct branch: master (7.0.1 - patch release), 7.1.x (minor release - breaking changes with fallbacks), 8.0.x (breaking changes without fallbacks)
[ ] If your PR is targeting a particular programming language, @mention the technical committee members, so they are more likely to review the pull request.

@joaojtmarques-qubit since I am preparing our forked released hooked to the latest release they have (that's the 7.0.0) your branch is really ahead with commits we shouldn't have.
Can you branch from the new branch which is master-7.0.0 so we don't have more than 30 commits enterting the master? Thanks.


F12 Go to definition not working for signature files in referenced project.

Latest nightly

This is probably related to #2902.

I can't repro this - could you reopen with exact repro steps please and sample project? thanks

I can't reproduce this outside of VIsualFSharp.sln.


stdDate() is not functional

stdDate() is not functional from api/helpers/dateops.php#L168-L177, following is the test snippet used
$actual = stdDate();
$expected = date('D, d M y H:i:s O');
$this->assertEquals($expected,$actual);

Fixed in latest commit.


w00tw00t

Harmless most of the time, but no one should try to access to any file with w00tw00t...
This is a trigger scenario and will act on the first access.
type: trigger
name: si/w00tw00t
description: "detect w00tw00t"
debug: false
# request on xmlrpc
filter: "evt.Meta.log_type == 'http_access-log' && evt.Parsed.file_name contains 'w00tw00t.at.ISC.SANS.DFind'"
groupby: evt.Meta.source_ip
#capacity: 1
#leakspeed: "60s"
blackhole: 5m
labels:
 service: http
 type: scan
 remediation: true

WIP https://github.com/crowdsecurity/hub/pull/147

WIP https://github.com/crowdsecurity/hub/pull/147

merged, thanks


Unable to add integration

When trying to add the intergration i get the following error:
Config flow could not be loaded: {"message":"Invalid handler specified"}

Apologies, notice this was due to my yaml. previous defined it.


Discover Go and JS nodes

I believe this requires adding an mDNS implementation that is compatible with the Go and JS nodes. Not sure if this should be addressed in an IPFS implementation or in rust-libp2p.

@adamski there's no hurry with this, this IPFS implementation won't be useful for a while...

@adamski this requires fixes/changes in the libp2p stack. see https://github.com/libp2p/rust-libp2p/pull/968

Adding an Identify behaviour fixed the connection problems with jsipfs. Let me know if you are still running into issues...

Looks like this issue can be closed.


put_assoc/3 does not respect prefix on parent or association structs

Environment

Ecto version (mix deps): 3.7.2
Database adapter and version (mix deps): postgrex 0.15.10

Current behavior
ParentSchema
|> Repo.get!(1, prefix: "tenant")
|> Repo.preload(:teams) |> Ecto.Changeset.change()
|> Ecto.Changeset.put_assoc(:assoc, [Repo.get!(ChildSchema, 1, prefix: "tenant")])
|> Repo.update()

INSERT INTO "join_table" ("parent_schema_id","child_schema_id") VALUES ($1,$2) [1, 1]

### Expected behavior

INSERT INTO "tenant.join_table" ("parent_schema_id","child_schema_id") VALUES ($1,$2) [1, 1]

Note: `Repo.update(prefix: prefix)` produces the desired query, but is inconsistent with how prefixes are handled in other places.

Is this only for many to many?

Sorry yes I've only observed this on many to many associations


Serverless: Apache Pulsar Functions, stars = 1,728

Pulsar Functions is a light-weight computing framework developed for lambda-style stream-native processing. Along with the message system, Apache Pulsar provides an installable platform for applications to run lambda-style/serverless functions along with messaging/streaming systems.
Details: https://pulsar.incubator.apache.org/docs/latest/functions/overview/
Github repo: https://github.com/apache/incubator-pulsar/tree/master/pulsar-functions
Pre-submission checklist:
Please check each of these after submitting your pull request:

[x] Are you only including a repo_url if your project is 100% open source? If so, you need to pick the single best GitHub repository for your project, not a GitHub organization.
[x] Is your project closed source or, if it is open source, does your project have at least 250 GitHub stars?
[x] Have you picked the single best (existing) category for your project?
[x] Does it follow the other guidelines from the new entries section?
[x] Have you included a URL for your SVG or added it to hosted_logos and referenced it there?
[x] Does your logo clearly state the name of the project/product and follow the other logo guidelines?
[x] Does your project/product name match the text on the logo?
[x] Have you verified that the Crunchbase data for your organization is correct (including headquarters and LinkedIn)?
[ ] 2 minutes after opening the pull request, the CNCF-Bot will post the URL for your staging server. Have you confirmed that it looks good to you and then added a comment to the PR saying "LGTM"?

Deploy preview for landscape failed.
Built with commit b162cc5ae84b2106f727320f1ced6b3fa4838d56
https://app.netlify.com/sites/landscape/deploys/5b119c91dd6a544214e49177

Pulsar is currently listed under Streaming: https://landscape.cncf.io/grouping=landscape&landscape=streaming&selected=pulsar
Each GitHub repo can only be used in a single section.
Closing now, but feel free to open a new PR where you move Pulsar from Streaming to Installable Serverless platforms.

Gotcha. Thank you @dankohn


No lr_scheduler in full_finetune_distributed

Learningrate scheduler isn't set up in the  full_finetune_distributed training loop, only exists in the lora finetuning.

Hi @jxmsML thanks for pointing this out. We do only support fixed learning rate in the full finetune recipes, but I agree we should support schedulers there like we do in LoRA. There's not really a technical blocker for this, so if you're interested to open a PR for the feature we'd love to have it. Otherwise we hope to add it soon!

Duplicated in #1308


onInputFocus display all the suggestion in dropdown format

Hi, I'm trying to create an input filed that display all the suggestion in a dropdown format when the input is focused and when user searches it should short list the suggestions.
Please let me know if anyone has any idea to implement this, thanks.

Hi @jigarvishwakarma, did you try using the suggestions prop ?


Build compatible for Visual Studio 2019 & Android Studio

I was trying to reference the cryptopp as a submodule, but they just failed to build directly in VS&AS IDE.(I knew it can be compiled mannually from cmake command line, but it's more complicated, for example, android studio has at least 4 architectures to build for the DLL)
I have committed 2 pathches for both IDE.
With this patch, the cryptopp can be built directly on VS&AS.
And my project can reference and compile it together as a lib without mannual build each architectures or debug release versions. Tested on vs2019 cmake project and Android Studio 3.5.3 with NDK r20.

Thanks.


DOC: update the description of the carpetplot in the sample report

Is this taken directly from somewhere? I thought we took an actual output report here, so I'm surprised this is the one change.

@effigies  While visualizing a report from fMRIPrep from v22.0.0, I noticed the edge is not mentioned in the text describing the carpetplot that is why I thought of quickly adding it. Does it reply to your question ?


Meet a problem while upgrade tikv to v5.1.0 and enable raft-engine

hi,
I'm trying to upgrade my tikv cluster to v5.1.0 to test raft-engine's performance, but I met some problem
my old version cluster is using two disks to store raftdb and rocksdb(by mount /var/lib/tikv/raft to another disk A). But when I try to do the update, i found raft-engine is using /var/lib/tikv/raft-engine dir. If i remount diskA to /var/lib/tikv/raft-engine, tikv cannot dump old raftdb's log.
So my question is how can i upgrade my cluster, can use raft-engine with two disks and don't need to lose my old raft's log?
Thx~

Besides, i also try to just upgrade tikv from raftdb to raft-engine with using one disk, but got a FATAL device or resource busy after raft-engine dumped raft's old logs. It seems tikv cannot remove the old log file?
[2021/07/05 03:42:22.027 +00:00] [INFO] [engine.rs:437] ["Recovered raft log Append.265."]
[2021/07/05 03:42:22.029 +00:00] [INFO] [engine.rs:479] ["Recover raft log takes 17.234144924s"]
[2021/07/05 03:42:22.031 +00:00] [INFO] [raft_engine_switch.rs:244] ["Start to scan raft log from RocksEngine and dump into RaftLogEngine"]
[2021/07/05 03:42:22.031 +00:00] [INFO] [raft_engine_switch.rs:253] ["Scanned all region id and waiting for dump"]
[2021/07/05 03:42:23.185 +00:00] [INFO] [raft_engine_switch.rs:258] ["Finished dump, total regions: 218; Total bytes: 463632899; Consumed time: 1.15466225s"]
[2021/07/05 03:42:23.686 +00:00] [FATAL] [lib.rs:462] ["called `Result::unwrap()` on an `Err` value: Os { code: 16, kind: Other, message: \"Device or resource busy\" }"] [backtrace="stack backtrace:\n   0: tikv_util::set_panic_hook::{{closure}}\n             at components/tikv_util/src/lib.rs:461\n   1: std::panicking::rust_panic_with_hook\n             at library/std/src/panicking.rs:595\n   2: std::panicking::begin_panic_handler::{{closure}}\n             at library/std/src/panicking.rs:497\n   3: std::sys_common::backtrace::__rust_end_short_backtrace\n             at library/std/src/sys_common/backtrace.rs:141\n   4: rust_begin_unwind\n             at library/std/src/panicking.rs:493\n   5: core::panicking::panic_fmt\n             at library/core/src/panicking.rs:92\n   6: core::result::unwrap_failed\n             at library/core/src/result.rs:1355\n   7: core::result::Result<T,E>::unwrap\n             at rustc/16bf626a31cb5b121d0bca2baa969b4f67eb0dab/library/core/src/result.rs:1037\n      server::raft_engine_switch::rename_to_tmp_dir\n             at components/server/src/raft_engine_switch.rs:37\n   8: server::raft_engine_switch::check_and_dump_raft_engine\n             at components/server/src/raft_engine_switch.rs:265\n      server::server::TiKVServer<engine_rocks::engine::RocksEngine>::init_raw_engines\n             at components/server/src/server.rs:1130\n      server::server::run_tikv\n             at components/server/src/server.rs:152\n   9: tikv_server::main\n             at cmd/tikv-server/src/main.rs:181\n  10: core::ops::function::FnOnce::call_once\n             at rustc/16bf626a31cb5b121d0bca2baa969b4f67eb0dab/library/core/src/ops/function.rs:227\n      std::sys_common::backtrace::__rust_begin_short_backtrace\n             at rustc/16bf626a31cb5b121d0bca2baa969b4f67eb0dab/library/std/src/sys_common/backtrace.rs:125\n  11: main\n  12: __libc_start_main\n  13: <unknown>\n"] [location=components/server/src/raft_engine_switch.rs:37] [thread_name=main]

@Itachi666 Hi, so sorry for the late response.
The first issue is a bug caused by incomplete canonicalization, I am fixing it in tikv/tikv#10615.
For the second issue, it looks like you opened two TiKV instances simultaneously, and there is a conflict between their operations on the same files.
I have this suspicion because the log Start to scan raft log from RocksEngine and dump into RaftLogEngine means transferring logs from RocksEngine to RaftEngine, while the panic stacktrace check_and_dump_raft_engine suggests it is transferring in the opposite direction.


auth-user & auth-pass not working

It looks like the auth-user and auth-pass parameter isn't working
pgweb auth-user=myuser --auth-pass=mypass --user postgres --host <IP_ADDRESS> --db MyDB --bind=<IP_ADDRESS>

I can access the site directly with no problem without password, it goes directly to this page(I've restarted httpd & pgweb, disconnected pgweb and restart, also tested in chrome incognito):

I see my problem now, missed the -- for auth-user


[BUG] Missing SASL Config Due to String Casing

Describe the bug
I was trying to debug why my SASL configs were not being loaded.   Everything looked correct, except the usingSasl() function below was still returning false.  This was because it checks specifically for uppercase strings, whereas our config was in  lowercase.
    private function usingSasl(): bool
    {
        return $this->securityProtocol === 'SASL_PLAINTEXT' || $this->securityProtocol === 'SASL_SSL';
    }

Because Kafka itself seems to understand the lowercase sasl_plaintext values, I suggest we change the function to ignore case by throwing a strtolower in there.
To Reproduce
Steps to reproduce the behavior:

Setup SASL, but use lowercase sasl_plaintext as the security protocol.
See that SASL configs are not loaded via getConsumerOptions()

Expected behavior
I would expect the SASL configs to be loaded properly regardless of the casing of their values.

Makes sense to me!

@nickpoulos @rtuin released on v1.6.1. I'll add this to v1.4.x and v1.5.x later today.


NX endorses spec7

We are still discussing this SPEC and decided to open this PR to have a place where we can make comments and vote. We will leave this PR open for a week or two.

I approved this PR for NetworkX endorsement of SPEC7 understanding that it applies only to the API for NumPy random number generators (as stated in the "Scope" part of the SPEC. The first paragraph of the SPEC seems to be much broader in scope than NumPy rngs, but the Scope section does reduce attention to NumPy which is what most of this ecosystem will be using.
I don't think the NetworkX will exactly follow the recommended API because of our use of Python's random.random as well as numpy.random and our tooling to provide a unified RNG interface for both. But I think the project endorses the ideas in this SPEC and will work toward them within our other constraints.


How to manage multiple proxy containers?

If the node serving the proxy container goes down, swarm will take some time to start it somewhere else. So my backends will be unaccessible for a while. So it makes sense to have multiple instances (perhaps a global proxy service).
But in that case, how do I reconfigure the proxy with new services?
The only thing I can do right now is to make an external process aware of the swarm nodes where the proxy is running and reconfigure on those nodes.
Just wanted to check if you have a better solution.

The idea is as follows.
If a request has "distribute=true", the instance that receives it will resend it to all the other instances. The same logic will be applied to both reconfigure and remove request types.
What do you think? Would that solve the problem you explained?

That would work.
Are you thinking about inspecting the 'proxy' service to figure out the IP addresses of containers running on the 'proxy' network?

With the new networking, you can inspect DNS record tasks.[SERVICE_NAME]. That entry returns the IPs of all the containers. The trick is that it is available only inside a container that belongs to the same network. You cannot do that from "outside". The tasks.[SERVICE_NAME] DNS record is one of the still undocumented Docker 1.12 features.
When the proxy receives a request to reconfigure a service and distribute=true is present in the query, it will take the name of the service, find all the IPs in the DNS tasks.[SERVICE_NAME] and resend the same request to all of them.
I just finished the code but did not have time to do integration tests. It's very late where I live (5:15h). Tomorrow I'll run them and, if everything works correctly, make a new release. Unlike most other tests, those are a bit tricky to automate.

The feature is done. An example can be found in https://github.com/vfarcic/docker-flow-proxy/blob/master/articles/swarm-mode.md#scaling-the-proxy .
Can you please try it out and let me know if it works as expected.

@vfarcic Thanks for the quick implementation. Will try it out and report and issues.

Please reopen the ticket if you experience a problem.


updating error

Please describe what you were doing when this error happened.
Specifications

OS: win32
IPFS Desktop Version: 0.11.2
Electron Version: 8.2.4
Chrome Version: 80.0.3987.165

Error
TypeError: stopIpfs is not a function
    at quitAndInstall (C:\Users\Mannix\AppData\Local\Programs\IPFS Desktop\resources\app.asar\src\auto-updater\quit-and-install.js:13:24)
    at Immediate.<anonymous> (C:\Users\Mannix\AppData\Local\Programs\IPFS Desktop\resources\app.asar\src\auto-updater\index.js:87:9)
    at processImmediate (internal/timers.js:439:21)

i got a notifcation to update ipfs, after saying it should restart, it crashed,

Ref #1509


Demo App on material.angular.io not working

Bug, feature request, or proposal:
The Demo App on material.angular.io is not working properly.
What is the expected behavior?
Demo App loads.
What is the current behavior?

It says "Loading document..." on the page
StackTrace with message "Unhandled Promise rejection: Cannot read property 'document' of undefined"
Whole Page unresponsive

What are the steps to reproduce?
Open https://material.angular.io/guide/material2-app

Duplicate of #3388

Duplicate of #3388


Create the guideline to use Service Mesh with Dapr

Describe the proposal
We need to provide the guideline and best practice for users who want to use Service Mesh with Dapr. We need to have test result by running actions with service meshes and provide the instructions.
Copied from https://github.com/dapr/dapr/issues/467

@orizohar - Given the community call today, this would be good to get into the docs

Any news on best practices with service mesh? Our deployment environment specifically has istio, but any guidance would be welcome.

Agreed. It looks like the sidecars are managing everything for Service Invocation component but would be good to learn why we should use that over a service mesh because that's not clear in the FAQ imo.
Even then, it's hard to scale based on the current solution which could be improved if it would comply with SMI Spec as per FAQ and use something like KEDA to scale on.

Is it possible for the self-hosted mode to run dapr-mesh(or with other service-mesh toolkit)? If so, would you consider providing guidelines for this?
K8s typically runs the dapr runtime in container-based environment. But It still have many services still running without containers. They may need to use dapr-mesh, including to communicate with K8s services, to make it easier to migrate K8s services in the future.

Partly addressed by https://github.com/dapr/docs/pull/1350

Is it possible for the self-hosted mode to run dapr-mesh(or with other service-mesh toolkit)? If so, would you consider providing guidelines for this?
K8s typically runs the dapr runtime in container-based environment. But It still have many services still running without containers. They may need to use dapr-mesh, including to communicate with K8s services, to make it easier to migrate K8s services in the future.

We currently do not provide any guidance on running service meshes on self-hosted mode. We only support service meshes on Kubernetes: https://docs.dapr.io/concepts/service-mesh/
We're happy to discuss this on the https://aka.ms/dapr-discord server if you want to try it out. We'd also love to eventually add docs on self-hosted service meshes if you'd like to contribute.


Unable to re-create environment

Mōrena.
I am unable to recreate a conda environment from the provided yml file.
Conda initially fails to find three packages:
  - gfortran_impl_linux-64==7.5.0=hdf63c60_6
  - gxx_impl_linux-62==7.5.0=hdf63c60_6
  - gcc_impl_linux-64==7.5.0=hd420e75_6

removing the =hdf63c60_6 suffix lets conda identify gfortran_impl_linux-64 and gcc_impl_linux-64, but there remains no candidate for gxx_impl_linux-62. There is a gxx_impl_linux-64 though.
Changing the packages in the environment.yml file to:
  - gfortran_impl_linux-64==7.5.0
  - gxx_impl_linux-64==7.5.0
  - gcc_impl_linux-64==7.5.0

lets Conda solve the environment, but results in it finding conflicts. It then searches for incompatible package and finds a whole bunch of conflicts (in the attached txt file). Mostly in R packages, so I have no idea if the conflicts are a result of changing the names of the three packages above, or if they're there anyway.
Thanks
Ben.
conflicts.txt

It's not super important - the more basic environment creation process does work.

I trimmed down the environment file to match what we are creating in the Docker container. Please try the attached environment file and let me know if it works for you.
environment.yml.zip

All good. Thanks.


OLM kustomization overlay OCP vs k8s

Currently we provide the community with a single manifest bundle that mostly supports OCP.
It might be a good idea to only use kustomiaztion overlays to be able to separate config like seucirty context.
For inspiration look at:
https://github.com/grafana/loki/pull/7308

Worked on in: https://github.com/grafana-operator/grafana-operator-experimental/pull/72


vm: apply storage sku on attaching data disks

Fix #4103

[x] The PR has modified HISTORY.rst with an appropriate description of the change (see Modifying change log).

Command Guidelines

[x] Each command and parameter has a meaningful description.
[x] Each new command has a test.

(see Authoring Command Modules)

Codecov Report

Merging #4111 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #4111   +/-   ##
=======================================
  Coverage   70.49%   70.49%
=======================================
  Files         437      437
  Lines       28134    28134
  Branches     4316     4316
=======================================
  Hits        19832    19832
+ Misses       7018     7014    -4
- Partials     1284     1288    +4

Impacted Files
Coverage Δ

...zure-cli-vm/azure/cli/command_modules/vm/custom.py
75.52% <ø> (ø)
:arrow_up:

...dback/azure/cli/command_modules/feedback/custom.py
31.25% <0%> (ø)
:arrow_up:

src/azure-cli-core/azure/cli/core/util.py
70.66% <0%> (ø)
:arrow_up:

...nent/azure/cli/command_modules/component/custom.py
16.23% <0%> (ø)
:arrow_up:

...li-cloud/azure/cli/command_modules/cloud/custom.py
13.79% <0%> (ø)
:arrow_up:

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 163b548...ef767c1. Read the comment docs.


Code improvements

Custom exception instead of just printing the error
Don't use raw types for parametized classess
Use Apache's Lang Validator (Because it has a better ide implementation)
Code Formatting / Naming is sometimes a bit inconsistent (Throwable e / Throwable ex)
Always use the absolute filepath in error messages. Makes debugging a lot simpler

Hey Justin,
Thank you for providing these enhancements. We will consider adding it.
Regards,

Leonhard


Animation / free mpl styling args

This PR brings mpl animations to plot.scene, which is the most relevant figure to see the evolution of the optimization or sampling procedure.
If called with a single Scene, it behaves as is. But if it get a list of scenes, it will turn in into an animation, where the images that depend on the state of the model (the model image itself, rendered, and residuals) will be updated. To get the list of scene from the fitter, use the callback argument:
scenes = []
def scene_saver(scene, *args):
    scenes.append(scene)

scene_ = scene.fit(obs, max_iter=200, e_rel=0, callback=scene_saver)
ani = plot.scene(scenes, obs, show_model=True, show_rendered=True, show_observed=True, show_residual=True)

The output can be displayed inline in jupyter or saved to file, e.g. this gem with a terrible initialization:
https://github.com/pmelchior/scarlet2/assets/1463403/0b2df65f-ba2b-48cc-bbb6-6c8136da67e8
This PR also replaces hard-coded styling parameters, like size of the title fonts or colors of the source markers, with kwargs, so that they can be overwritten by the user:
plot.observation(obs,
    add_labels=False,
    show_psf=True,
    fig_kwargs={"figsize": (6,4)},
    title_kwargs={"color": "red", "size": 22}
)

I should add that I had some trouble with the animation showing in PyCharm, but that seems to be a problem of the IDE. And for saving, the best quality and smallest file sizes are from the ffmpeg writer,
ani.save("test.mp4", writer="ffmpeg")

which you have to install separately:
conda install -c conda-forge ffmpeg

The scene returned in callback needs to run through the constraints otherwise colours/morph incorrect.  By adding
                # report current iteration results to callback
                if callback is not None:
                    if constraint_fn is not None:
                        scene_frame = _constraint_replace(scene, constraint_fn)
                    else:
                        scene_frame = scene
                    callback(scene_frame, convergence, loss)

this is solved, in a relatively straightforward way
https://github.com/pmelchior/scarlet2/assets/44228257/d417f08e-3ef4-4c79-b644-dc8cafc44b02

Also, this should work on most (maybe all) IDEs for visuals
from IPython.display import HTML
%matplotlib notebook
# create the animation
ani = plot.scene(scenes, obs, norm=norm, show_model=True, show_rendered=True, show_observed=True, show_residual=True)
# Show the animation
HTML(ani.to_html5_video())

Good catch with the constraints. I should have seen this myself, with the colors going wild in the example above. Repeated now with the change:
https://github.com/pmelchior/scarlet2/assets/1463403/3d7e7247-714a-4050-a343-39e817f010ad
Still very bad initialization (just 2 sources, one with uniform random morphology...), but that looks like a fit now...


update react-native-web

Please update react-native-web since it is very outdated (1.6.0 now) and provide a Hook or a way to perform platform specific code
eg:
if(platform==='web') {

} else if(platform ==='android' || platform ==='ios'){

}

You need to check Platform.OS in your code.


Missing information about Expression-type parameters

[EDIT by guardrex to update the metadata]
When @binding-Xxx is used, there is a also a "hidden" parameter, named XxxExpression (of Expression type).
Please document this one also, including the nested component binding. Should the parent set the Expression parameter of the child with its own value?

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

ID: c3128506-d228-35ca-e34a-183a881acdec
Version Independent ID: c3128506-d228-35ca-e34a-183a881acdec
Content: ASP.NET Core Blazor forms and input components
Content Source: aspnetcore/blazor/forms-and-input-components.md
Product: aspnet-core
Technology: aspnetcore-blazor
GitHub Login: @guardrex
Microsoft Alias: riande

Hello @Timores ... Sorry for the delay. I'm OOF for a few days and will get back to you by the end of the week.

Hello @Timores ... Sorry for the delay. I'm OOF for a few days and will get back to you by the end of the week.

Hello @Timores ... I'm back.
There's no "@binding" directive in Blazor. I also don't know about any "hidden" expression parameter. No such content/guidance ever arrived for me to document. It's probably best if I direct you to open an issue directly for the product unit. Please open a new issue at ...
https://github.com/dotnet/aspnetcore/issues
You'll probably need to give them more information than you stated here. Can you point to what you're talking about in reference source or provide them an example in your new issue? I'm not sure if they will know what you're referring to without additional details.
Please add ...
cc: @guardrex https://github.com/dotnet/AspNetCore.Docs/issues/28785

... to the bottom of your opening comment so that I can follow along. I might re-open this for doc work depending on what they say.

OK, it's @bind-Xxx, not @binding-Xxxx, my bad. I am surprised that such a simple error causes one to dismiss a comment.
Guess it's not worth commenting.

I didn't close the issue on that basis. I literally have no personal knowledge of any hidden parameter for binding. You didn't explain out what this "hidden" parameter is or where it is (i.e., reference source). Because I have no understanding of it, it falls more under the "support request" type of issue as far as the docs repo is concerned, so it should go to the product unit for now. If you add the line that pings me to your issue over there and cross-links this issue, I'll follow along until I understand what content should be added or updated. Then, I'll re-open this issue for work. That's just the way that we work here. We try to keep the backlog as clean as possible and as focused on actionable work items as we can.

When you use @bind-Value on a component, it expects the Value property and ValueChanged callback, but it also sets a ValueExpression property of type Expression.
See https://github.com/dotnet/aspnetcore/blob/main/src/Components/Web/src/Forms/InputBase.cs, lines 32 to 49 (at the time of writing).
And the SetParameterAsync method, lines 191 to 232 I quote a small part:
if (ValueExpression == null)
{
throw new InvalidOperationException($"{GetType()} requires a value for the 'ValueExpression' " +
$"parameter. Normally this is provided automatically when using 'bind-Value'.");
}
If you want to create your own component which uses, say, InputNumber, while providing the binding ability (which is nicely explained in the docs), you also need to forward the ValueExpression property to InputBase.
This is something that took me a long time to figure out and I thought that it could help others.

By "hidden", I meant that it seems to happen automagically and I could not find any mention of it in the docs.
There was no intent of saying that anyone is trying to hide information. Apologies if it was unclear.

I see ... so this pertains to built-in forms components' binding scenarios. I didn't pick up on that from your opening post.
We keep the general subject matter on forms components over in ...
https://learn.microsoft.com/aspnet/core/blazor/forms-and-input-components
... and you're correct that we don't cover ValueExpression in that scenario, probably because it's dependent on a framework implementation for the built-in components that's subject to change at any time.
I'll update the metadata for this issue because if we did cover anything like this it would be for the forms topic. I'll take a closer look at this later today or tomorrow morning. I probably will need to consult with the product unit to see how deep into the weeds that they want to get with the framework implementations for those components. They often say that they don't want to cover API aspects that are entirely up to arbitrary design decisions made at a point in time for them.

Other components like the Telerik Blazor suite seem to depend on this...
Don't know if this is the right thing to do for them, but this is how it is.

I read a bit on that parameter that Chris published. The product unit decided not to show/explain how to build components from InputBase, so it wouldn't be expected to see this coverage without additional coverage on the whole scenario. Since the product unit ruled it out, we don't need to address it in the article. I'm sure that they'll keep an 👂 open on requests tho going forward. If enough developers ask them, they might agree to cover it. I'll ...
cc: @mkArtakMSFT
... so that he's aware that a request came in.
Artak ... TL;DR ... @Timores is asking for coverage on building bespoke InputX-type of components from InputBase, but the PU ruled that out a long time ago.

I am not asking for specific documentation on building components out of InputX classes. Using this classes was just an example.
I am asking to document the fact that the Value and ValueChanged properties are not the only ones being magically set when using the @bind syntax. I think that other developers would benefit from knowing about ValueExpression (again using 'Value' as an example)
The doc page linked to this discussion is quite good but is, IMHO, missing the explanation of the Expression property.

I hear you, but the product unit (PU) doesn't agree (yet anyway 😄 ... they might change their mind at some point in the future). The most likely reason (AFAICT) that a dev would need to read about this param in the doc is for (as you said) advanced use cases working directly with InputBase, which the PU (product unit) ruled out right from the start. When these articles go up, the PU reviews them and calls out if they feel anything is missing. They didn't flag this param for coverage at the time.
Besides, it's documented via the API docs ...
https://learn.microsoft.com/en-us/dotnet/api/microsoft.aspnetcore.components.forms.inputbase-1?view=aspnetcore-7.0
... and all of the built-in components in the Built-in input components section cross-link to the API docs, where devs can see that they all either inherit from InputBase or ComponentBase ... and then they can research from there and study how the PU composed these.
As I said right from the start ... and as I still suspect ... the PU may feel that InputBase is an internal framework implementation for these InputX components and subject to change at any time without notice. They often decide not to cover such aspects in the articles because there are too many of these internal implementations churning away over the years to keep up with in docs and for advanced use cases. For this type of param, they may just need to not document it 😄 in order to be able to efficiently change the framework on-the-fly without the fear of breaking articles.
Of course, the reference source is also available and considered "documentation" of sorts for how the PU designed InputBase to work. It's also covered in any number of blog posts on the subject, including Chris's blog, which comes up early in search results. The articles in the Blazor docs node are considered introductory coverage for the most common use cases. This one falls outside of that scope ... for today ... we'll see if they change their mind in the future on it.


[elastic] Following changes of syncing to the upstream.

:x: Build go-langserver 1.0.158 failed (commit https://github.com/elastic/go-langserver/commit/8580a0fcfc by @movie-travel-code)

:white_check_mark: Build go-langserver 1.0.159 completed (commit https://github.com/elastic/go-langserver/commit/034056b4f2 by @movie-travel-code)

:x: Build go-langserver 1.0.160 failed (commit https://github.com/elastic/go-langserver/commit/4f89d7366e by @movie-travel-code)

:x: Build go-langserver 1.0.161 failed (commit https://github.com/elastic/go-langserver/commit/6aa9587d36 by @movie-travel-code)

:x: Build go-langserver 1.0.162 failed (commit https://github.com/elastic/go-langserver/commit/9b00a8091c by @movie-travel-code)

:x: Build go-langserver 1.0.163 failed (commit https://github.com/elastic/go-langserver/commit/ae256ff955 by @movie-travel-code)

:x: Build go-langserver 1.0.164 failed (commit https://github.com/elastic/go-langserver/commit/eb14d39eda by @movie-travel-code)

:x: Build go-langserver 1.0.165 failed (commit https://github.com/elastic/go-langserver/commit/c6495466e4 by @movie-travel-code)

:x: Build go-langserver 1.0.166 failed (commit https://github.com/elastic/go-langserver/commit/9cb0075f8f by @movie-travel-code)

:x: Build go-langserver 1.0.167 failed (commit https://github.com/elastic/go-langserver/commit/ef80d166cd by @movie-travel-code)

:x: Build go-langserver 1.0.168 failed (commit https://github.com/elastic/go-langserver/commit/71beb20949 by @movie-travel-code)


Updated index of Linux of free-courses-en.md

What does this PR do?

Add Resource(s)
Remove Resource(s)

For resources
Description
Develop a good working knowledge of Linux using both the graphical interface and command line, covering the major Linux distribution families.
Linux powers 100% of the world’s supercomputers, most of the servers powering the Internet, the majority of financial trades worldwide, and over two billion Android devices. In short, Linux is everywhere. It appears in many different architectures, from mainframes to server to desktop to mobile and on a staggeringly wide variety of hardware.
Moreover, 80 percent of hiring managers reported that they will prioritize hiring Linux talent relative to other skills areas, and 47 percent of hiring managers say they’re more likely to hire a candidate with Linux certification.
This course explores the various tools and techniques commonly used by Linux system administrators and end-users to achieve their day-to-day work in a Linux environment. It is designed for experienced computer users who have limited or no previous exposure to Linux, whether they are working in an individual or Enterprise environment.
Upon completion of this training, you should have a good working knowledge of Linux, from both a graphical and command-line perspective, allowing you to easily navigate through any of the major Linux distributions. You will be able to continue your progress as either a user, system administrator, or a developer using the acquired skill set.
Why is this valuable (or not)
yes
How do we know it's really free?
https://itsfoss.com/free-linux-training-courses/
For book lists, is it a book?
it's a  verified online course
Checklist:

[x] Not a duplicate
[ ] Included author(s) if appropriate
[ ] Lists are in alphabetical order
[ ] Needed indications added (PDF, access notes, under construction)

Good resource. The linter is picky about spaces, please revise.

what happened with #3701 ?


fix: make CircularButton responsive

FX-4179
Description
According to BASE design, icons acting as buttons should accommodate for bigger icons.
How to test

Temploy
Check the Button story
There's a new Responsive story for the CircularButton
Resize the screen to see how the icon changes

Screenshots

Before.
After.

Development checks

[x] Add changeset according to guidelines (if needed)
[x] Read CONTRIBUTING.md and Component API principles
[x] Make sure that additions and changes on the design follow Toptal's BASE design, and it's been already discussed with designers at #-base-core
[x] Annotate all props in component with documentation
[x] Create examples for component
[x] Ensure that deployed demo has expected results and good examples
[x] Ensure the changed/created components have not caused accessibility issues. How to use accessibility plugin in storybook.
[x] Self reviewed
[x] Covered with tests (visual tests included)

All development checks should be done and set checked to pass the
GitHub Bot: TODOLess action

PR commands

List of available commands:

@toptal-bot run package:alpha-release - Release alpha version
@toptal-anvil ping reviewers - Ping FX team for review

PR Review Guidelines

When to approve? ✅
You are OK with merging this PR and

You have no extra requests.
You have optional requests.

Add nit: to your comment. (ex. nit: I'd rename this variable from makeCircle to getCircle)

When to request changes? ❌
You are not OK with merging this PR because

Something is broken after the changes.
Acceptance criteria is not reached.
Code is dirty.

When to comment (neither ✅ nor ❌)
You want your comments to be addressed before merging this PR in cases like:

There are leftovers like unnecessary logs, comments, etc.
You have an opinionated comment regarding the code that requires a discussion.
You have questions.

How to handle the comments?

An owner of a comment is the only one who can resolve it.
An owner of a comment must resolve it when it's addressed.
A PR owner must reply with ✅ when a comment is addressed.

@toptal-anvil ping reviewers

@sashuk I like it, but we need to carefully think of the implementation details. We can't have it import all the icons directly since it will no longer be possible to tree shake them out of the bundle. Lazy loading is an option, however with dynamic imports icons will still end up in the bundle, even not loaded on the client. Since we have hundreds of icons, this can very severely impact the total bundle size.

@mkrl Yep, I'll bring this up on Daily to check if we can have a task / spike for that, thanks!

I've updated this PR to take responsive value from Picasso Provider into account. This PR is now based on https://github.com/toptal/picasso/pull/3823 to make use of the responsive icon set, the base will be changed to master once it is merged.


AN-3641/downstream receipt succeeded

Add receipt succeeded to logs (not filter, still parse logs on failed receipts).

[ ] FR required

Add receipt succeeded filter to usn supply (in silver, to be further deprecated)

[ ] FR required
[ ] Already on Pool Events, but FR not yet done

Added to horizon receipts

[ ] FR required to this plus horizon decoded logs

Added to social receipts & addkey

[ ] FR required to this plus downstream social models

Elevated col to actions model and filtering addkey on it. No change to functioncall yet

[ ] FR required

Disable legacy silver nft mints model

Logs and Actions Events will take a long time to FR - consider backfilling the column rather than a full FR

Largely successful FR and test on the following selection:
-s silver__usn_supply_s3 silver__nft_mints_s3 models/silver/horizon models/silver/social
Except for social addkey model:
23:44:42  Completed with 1 error and 0 warnings:
23:44:42
23:44:42  Failure in test not_null_silver_social__addkey_signer_id (models/silver/social/silver_social__addkey.yml)
23:44:42    Got 2885 results, configured to fail if >100
23:44:42
23:44:42    compiled Code at target/compiled/near/models/silver/social/silver_social__addkey.yml/not_null_silver_social__addkey_signer_id.sql
23:44:42
23:44:42    See test failures:
  --------------------------------------------------------------
  select * from NEAR_DEV.not_null_silver_social.addkey_signer_id
  --------------------------------------------------------------

Look into why with next step(s)


Error: NameError: name 'file' is not defined

NameError: name 'file' is not defined.
please help.

That alone doesn't tell me anything. There's a reason there's a template.
Possible duplicate of #2

Try updating and using it now, switched to usinf the method provided in webui to get the base directory of the extension

Assuming duplicate of #2


Move arguments as well as closure in call macro

Codecov Report

Merging #941 (326a2e5) into main (9cd44c8) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #941      +/-   ##
==========================================
- Coverage   84.30%   84.30%   -0.01%
==========================================
  Files         388      388
  Lines       40847    40882      +35
==========================================
+ Hits        34436    34465      +29
- Misses       6411     6417       +6

Impacted Files
Coverage Δ

lib/ffi/src/future/call.rs
100.00% <100.00%> (ø)

lib/hir-mir/src/expression_compiler.rs
73.60% <0.00%> (-0.41%)
:arrow_down:

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 9cd44c8...326a2e5. Read the comment docs.