No indication of rounded numbers in exported variables

Godot version
4.0
System information
Win 10, Ryzen 2600x
Issue description
In editor it looks like 0

But in reality it's not

If we open scene in code editor, we'll see it's 10^(-12) something

I think we should indicate whether some value is rounded or the exact. In my case it took me a hour to debug :(
What's interesting - even after I manually set 0, it didn't change to 0. It only changes if you set a different value first (e.g. 1) and then set it back to 0
Steps to reproduce

Export float variable
Set a small value
It'll display 0, but it isn't

Minimal reproduction project

Related to https://github.com/godotengine/godot/issues/33095.

Did you change the exported variable's type from float to int during development?

Can't remember tbh, this part wasn't touched for a couple of months. It could be the opposite tho, i.e. int -> float

It's easy to make the editor display a completely different float than what you see at runtime (tested in 4.2.2-stable and 4.3 beta3)

Create a new project
Editor -> Editor Settings... -> Interface -> Inspector
Change Default Float Step to 0.0005, click Save & Restart
Create a 2D scene and attach a script with the following:

extends Node2D

@export var val : float

func _ready():
  print(val)

In the inspector, change Val to 0.0005
(good) notice that Val says 0.0005
(good) Run the project (press F5), notice that the output says 0.0005
Editor -> Editor Settings... -> Interface -> Inspector
Change Default Float Step back to 0.001, click Save & Restart
(bug) notice that Val says 0.001
(bug) Run the project (press F5), notice that the output says 0.0005

This becomes a problem when working on a project with many people since Editor Settings isn't (shouldn't be?) shared.  This concern was originally brought up here: https://github.com/godotengine/godot/pull/30776#issuecomment-566208154
I think this bug, and the example above, come from the same issue: "exported floats are not accurately reflected in the editor when compared with their runtime value"


Create more repos like this.

Let's create some for typescript, react.js, graphql, mongodb, and redux.

Let's create.
I will give my contribution to react,js repo.

Lets do it !

i will give contribution to typescript and mongodb

I can help in react too

Let's create some for typescript, react.js, graphql, mongodb, and redux.

Please create one for Angular too!

Hello,
Are you done creating it, or How can I come in?

Hey...I want to contribute as well...please let me know about the repo. I can help with ReactJs and Redux toolkit.

lemme know if anyone of you is doing

Hi, guys I want to learn more about the repo I can contribute as well in reactjs, reduxtoolkit and firebase.

This will be done in the future for other languages. Stay tuned. Closing this for now.


MySQL connection

I am having trouble while connecting mysql in my perfect project. Can you provide a sample code  ?

import PerfectLib
import MySQL
let HOST = "your sql ip"
let USER = "your account"
let PASSWORD = "your password"
let SCHEME = "your scheme"
public func PerfectServerModuleInit() {
Routing.Handler.registerGlobally()
Routing.Routes["GET", ["/"]] = { (_:WebResponse) in return PerfectHandler() }
print("\(Routing.Routes.description)")

}
//Create a handler for index Route
class PerfectHandler: RequestHandler {
func handleRequest(request: WebRequest, response: WebResponse) {

    response.addHeader("Content-Type", value: "application/json")
    response.addHeader("Content-Type", value: "text/html; charset=utf-8")

    let mysql = MySQL()
    let connect = mysql.connect(HOST, user: USER, password: PASSWORD)
    if(connect)
    {
        let sres = mysql.selectDatabase(SCHEME)
        if(sres)
        {

            let sres2 = mysql.query("SELECT name,memo FROM Info")

            if(sres2)
            {
                let results = mysql.storeResults()!

                if(results.numRows()==0)
                {
                    do{

                        let encoder = JSONEncoder()
                        let data = try encoder.encode(["result": ""])
                        response.appendBodyString(data)
                    }
                    catch{
                        response.setStatus(500, message: "Could not create data")
                    }
                }
                else
                {
                    var dataArray:Array<AnyObject> = []
                    var dict = Dictionary<String,String>()
                    while let row = results.next() {

                        dict["name"]=row[0];
                        dataArray.append(dict)

                    }

                    print(NSJSONSerialization.isValidJSONObject(dataArray))

                    do {

                        let dataFinal = try NSJSONSerialization.dataWithJSONObject(dataArray, options:NSJSONWritingOptions(rawValue:0))

                        let string = NSString(data: dataFinal, encoding: NSUTF8StringEncoding)

                        let tee : String = string as! String
                        response.appendBodyString(tee)
                    }
                    catch{
                        response.setStatus(500, message: "Could not create data")
                    }

                }

                results.close()
            }
        }
        mysql.close()
    }
    response.requestCompletedCallback()
}

}
if you want to learn more , go to my Chinese blog:
http://blog.csdn.net/kinfey/article/details/50644752

thank you. Can you give tell me how MVC pattern implies in Perfect Swift?
Also, I want to build a backend for my mobile application.

okay i will write some tutorials about that . You can see my blog step by step
http://blog.csdn.net/u014388424/article/category/6092861

Thanks @lokinfey!

@sumantimalsina9  this is mvc pattern implies guideline for swift http://blog.csdn.net/kinfey/article/details/50754770

Thanks @lokinfey for helping here.
Closing issue. Thanks!


Prevent a non-numeric value encountered

Since PHP7.1 it returns always a non-numeric value encountered cause $decryptId is a string.

I don't know why I initialized $decryptId as a string in the first place. Thanks.


Text selected in peek does not transfer to search when Ctrl+Shift+F entered

Issue Type: Bug

Be in any view other than Search (optional, but adds evidence about the problem later).
Open a file.
Open a peek (e.g. peek a definition).
Select some text in the peek.
Enter Ctrl+Shift+F

:bug: the selected text does not transfer into the Search field, though the Search view appears and focus moves to that field.
VS Code version: Code - Insiders 1.56.0-insider (845efe5d8e31079cc858131c95c36103d9439e74, 2021-04-09T09:50:06.816Z)
OS version: Windows_NT x64 10.0.19042

@bpasero is this exposed in the editor service? Or is there any way for me to get the selected text out of the peek widget?

@roblourens embedded editors are actually not exposed through IEditorService. You could try ICodeEditorService#listCodeEditors to find it though.

Cool, that was relatively easy, thanks

/verified - thanks!


improvement: automatic detection of utc_offset and time_zone

The configuration settings utc_offset and time_zone should be derived from PC/Pi time settings. Especially in countries with dailight saving time it is too easy not to update the changed settings twice a year.
general time usage:

ftp_to_stanford.py uses UTC: yesterday = datetime.utcnow() - timedelta(days=1) - Used to calculate timestamp of yesterday.
sidfile.py uses UTC: utcnow = datetime.utcnow()
sidtimer.py uses UTC: self.utc_now = datetime.utcnow()
supersid/supersid_plot.py uses local time: Now = datetime.datetime.now() - Used to calculate timestamp of today or yesterday. Update to match ftp_to_stanford.py

utc_offset usage:

config.py: read the string from the configuration file
sidfile.py: write the string to the header

time_zone usage:

config.py: read the string from the configuration file
sidfile.py: write the string to the header with a fallback to self.sid_params['timezone']

additonal findings:

sidfile.py uses the non-existing 'timezone' in self.sid_params; one finding; to be removed
sidfile.py uses the non-existing 'loginterval' in self.sid_params; three findings; to be removed
sidfile.py uses the non-existing 'monitorid' in self.sid_params; one finding; to be removed

The problem to get a proper time_zone name is non-trivial.
English Ubuntu appears to be ok:
>>> import datetime
>>> datetime.datetime.now(datetime.timezone.utc).astimezone().strftime("%z")
'+0200'
>>> datetime.datetime.now(datetime.timezone.utc).astimezone().strftime("%Z")
'CEST'

German Windows shows German strings:
>>> import datetime
>>> datetime.datetime.now(datetime.timezone.utc).astimezone().strftime("%z")
'+0200'
>>> datetime.datetime.now(datetime.timezone.utc).astimezone().strftime("%Z")
'Mitteleuropäische Sommerzeit'

The following solution reports the same on both systems:
pip3 install tzlocal
>>> from tzlocal import get_localzone
>>> str(get_localzone())
'Europe/Berlin'

@sberl What are the expectations for time_zone reported in the file headers sent via FTP to Stanford University?

The data files from 2024-05-30 are using very different formats for UTC_Offset and TimeZone. Partially the offest does not match the time zone. Format and values of both show no common scheme.
Site,UTC_Offset,TimeZone
AGUSA3,0,NONE
Agua-Caliente,-0.0,Mountain Standard Time
BNA215,-06:00,CST
Bedford,-5,EST
CAS,-4,EDT
CHSAC1,0,GMT
CaGrBa,-00:00,EST
CiniselloBalsamo,+2.00,CEST
DE_St_George2,+02:00,CEST
EvansGA,-00:00,Eastern
Fuller,-00.00,Eastern Standard ET
GHM,-06:00,MST
Hermosillo,-07:00,MDT
JPO,-7,PST
Jaap,02:00,CEST
K4UL-1,-04:00,America/New_York
KA6AKH,-05:00,Eastern Time (EST)
KB5FIO,-07:00,MST_Arizona_No_DST
KOLHAPUR-2,UTC+05:30,Indian Standard Time (IST)
Kiel-Wellsee,+02:00,UTC
LottObs,20,EDT
MARSDEN01,-06:00,Central Standard Time (CST)
MHO,-5,Eastern Time (US & Canada)
MHO2,-5,NONE
OBS,0,GMT
PTO_SID,0,CST
RoswellMeteor,-7,America/Denver
SCO,-5,UTC
SSRO,-6:00,MST
Sirius,-5,HNE
UTP,19,GMT
VA3ROM,-00:00,America/Thunder Bay
XHS,-5,EASTERN STANDARD TIME
YADA,-5,CST
ZSPDRE,1,CET
tjb,0,GMT

The local time is never used for anything in the software either on the client, or on the stanford server.
I think the original vision was that the stanford web site would have an option display the plots in the local timezone of the user, but it was never implemented.
I think we can just leave the utc_offset parameter out, or just leave it as it is since nothing ever looks at the value.
We always have the option to go back later and derive the utc_offset from the (latitude, longitude, utc_time). I have code that does that in other contexts.

... just leave it as it is since nothing ever looks at the value


Is the refcoco-trained model available?

Hello,
It seems that only generic pre-trained models are available, and we have to fine-tune the model on the specific task, right? I am wondering if the refcoco-trained model is available for evaluation?
Thanks,
st2yang

Yes, you need to finetune them on downstream tasks. And I am sorry these models are not available since I didn't store them until now.

Hello, st2yang
Can you reproduce the results in the author's paper when fine-tuning the program (for the Refcoco dataset)? When I fine-tuned the test, the accuracy was only about 44 percent
Thanks

Hello,
It seems that only generic pre-trained models are available, and we have to fine-tune the model on the specific task, right? I am wondering if the refcoco-trained model is available for evaluation?
Thanks, st2yang

Hello, st2yang
Can you reproduce the results in the author's paper when fine-tuning the program (for the Refcoco dataset)? When I fine-tuned the test, the accuracy was only about 44 percent
Thank

@MI-WENYU Yes

@MI-WENYU Yes
@st2yang
Does the reproduction process use the original parameters? I have been unable to achieve the results in the paper. Could you please share your fine-tuned model with me? I would be very grateful.


[WIP] Migrate the desc statement from the old to new parser and analyzer

What type of PR is this：

[ ] bug
[ ] feature
[ ] enhancement
[x] refactor
[ ] others

Which issues of this PR fixes ：

Fixes #https://github.com/StarRocks/starrocks/issues/7285
Problem Summary(Required) ：

todo: need add ut  and migrate other desc

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Leo Cheng seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


CDN Icons

Hi! Any plans for supporting Fontawesome Icons in a format like :fa-[name]: or Material Icons ?

Currently, no.
You can add this kind of features via the WithRendererFunc option.

Can you please share an example for the same?

Just see tests.


EmrEtlRunner: simplify trail slash code now nils are supported

Following https://github.com/snowplow/sluice/issues/38, no need to do nil check

Closing since this was effectively done in r68 here


[Bug] ray.io_rayjobs.yaml missed in helm-chart/kuberay-operator/crds for v0.3.0

Search before asking

[X] I searched the issues and found no similar issues.

KubeRay Component
ray-operator
What happened + What you expected to happen
there are no ray.io_rayjobs.yaml in helm-chart/kuberay-operator/crds for v0.3.0 tag.
but it exists in master:
https://github.com/ray-project/kuberay/blob/master/helm-chart/kuberay-operator/crds/ray.io_rayjobs.yaml
it looks like this file should be in place for argocd deployment.
I faced with this error for kuberay-operatpr, it was just restarting all the time with this error in logs:
ERROR	controller-runtime.source	if kind is a CRD, it should be installed before calling Start	{"kind": "RayJob.ray.io", "error": "no matches for kind "RayJob" in version "ray.io/v1alpha1""}
sigs.k8s.io/controller-runtime/pkg/source.(*Kind).Start.func1.1
/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.11.1/pkg/source/source.go:137
k8s.io/apimachinery/pkg/util/wait.runConditionWithCrashProtectionWithContext
/go/pkg/mod/k8s.io/apimachinery@v0.23.0/pkg/util/wait/wait.go:233
k8s.io/apimachinery/pkg/util/wait.poll
/go/pkg/mod/k8s.io/apimachinery@v0.23.0/pkg/util/wait/wait.go:580
k8s.io/apimachinery/pkg/util/wait.PollImmediateUntilWithContext
/go/pkg/mod/k8s.io/apimachinery@v0.23.0/pkg/util/wait/wait.go:545
sigs.k8s.io/controller-runtime/pkg/source.(*Kind).Start.func1
/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.11.1/pkg/source/source.go:131
related issue:
https://discuss.ray.io/t/kuberay-operator-keep-restarting/7760/10
Reproduction script
you can just try to run it using scripts from readme in argocd. I faced with issue using these two argocd apps:
app 1:
apiVersion: argoproj.io/v1alpha1
kind: Application
metadata:
name: ray-operator-crds
spec:
project: default
source:
repoURL: https://github.com/ray-project/kuberay
targetRevision: v0.3.0
path: helm-chart/kuberay-operator/crds
destination:
server: https://kubernetes.default.svc
syncPolicy:
syncOptions:
- Replace=true
app 2:
apiVersion: argoproj.io/v1alpha1
kind: Application
metadata:
name: ray-operator
spec:
source:
repoURL: https://github.com/ray-project/kuberay
targetRevision: v0.3.0
path: helm-chart/kuberay-operator
helm:
skipCrds: true
destination:
server: https://kubernetes.default.svc
namespace: ray-operator
syncPolicy:
syncOptions:
- CreateNamespace=true
Anything else
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

It's fixed in master. Helm support will be stable in 0.4.0. For now, it's recommended to use the Helm charts from master.


Mozilla Add-ons: Resurrect Pages (isup edition) 2.1 Preliminary Reviewed

This version didn't pass full review because of the following issues:

content/resurrect.js
Synchronous HTTP requests can cause serious UI performance problems, especially to users with slow network connections.

Inserting un-sanitized remote content is a security problem. For example, JavaScript can be passed as URLs

You need to correct them to get full approval. Thanks.
Please note the following for the next update:

If the addEventListener() (or JQuery .on) is used once (e.g. for 'load', 'DOMContentLoaded'), it is good practice to  removeEventListener() (or JQuery .off) as soon as its function is called and free the resources.
elem.addEventListener(event, myfunction, false);
function myfunction() {
elem.removeEventListener(event, myfunction, false); // remove listener, no longer needed
// rest of the function code
}

elem.addEventListener(event, function myfunction(event){
elem.removeEventListener(event, myfunction, false); // remove listener, no longer needed
// rest of the function code
},false);

fixed in 3.0.0


cert-manager breaks namespace deletions

Describe the bug:
With installation of cert-manager, deletion of other namespace stuck in terminating state.
Expected behaviour:
namespace deletion should not be impacted by installation of cert-manager.
Steps to reproduce the bug:

install cert-manager with the manifest file
cert-manager-github.txt

kubectl apply -f cert-manager-github.txt

install a test namespace:

cat <<EOF | kubectl apply -f -
apiVersion: v1
kind: Namespace
metadata:
  name: test-foo

delete the test namespace and it will be stuck in terminating state
kubectl delete ns test-foo
delete the cert-manager installation
kubectl delete -f cert-manager-github.txt
the test namespace would get cleared up.

Anything else we need to know?:
With installation of cert-manager, kubectl api-resources is no longer able to list all APIs:
$ kubectl api-resources
error: unable to retrieve the complete list of server APIs: admission.certmanager.k8s.io/v1beta1: the server is currently unable to handle the request

This issue is similar to https://github.com/jetstack/cert-manager/issues/1355 but the cause is not improper deletion of cert-manager installation.
Environment details::

Kubernetes version (e.g. v1.10.2): v1.13.0
cert-manager version (e.g. v0.4.0): v.6.0
Install method (e.g. helm or static manifests): static manifests

/kind bug

+1

Found a workaround for the time being:
https://nasermirzaei89.net/2019/01/27/delete-namespace-stuck-at-terminating-state/

Ok, I think the renaming of namespace from cert-manager to platform-cert-manager caused the problem for me. I replaced "Namespace: cert-manager" with "Namespace: platform-cert-manager" in this file
cert-manager-working.txt. Of course, for the namespace resource itself.
I just re-dployed cert-manager with cert-manager-working.txt after I delete all resources I created with platform-cert-manager namespace. I no longer have problems with namespace deletion anymore.

When uninstall cert-manager, it's important that you delete all resources created by the installation manifest.
There are certain resources (notably, the ValidatingWebhookConfiguration resource) that are not namespaced, and as such, when you delete the cert-manager namespace the validating webhook still exists. Because the actual webhook pod runs in the cert-manager namespace however, it means that the webhook is no longer accessible which consequently means that Kubernetes garbage collector is unable to query the webhook, and so it blocks deletion of namespace resources (as it is unable to query the webhook in order to ensure that the namespace being deleted no longer contains resources)..
The TL;DR, is that you should uninstall cert-manager using kubectl delete -f cert-manager.yaml (where cert-manager.yaml is the file that you originally ran kubectl apply -f with).
Hope that helps! I'm going to close this issue, as it's not really a bug.
It'd be great to have an "uninstalling cert-manager" guide as part of our docs in future, as I think this would help with issues like this 😄
/close

@munnerz This is one of at least two issues I've just read where the answer appears to be "delete cert-manager properly"; however this issue occurs when you haven't deleted cert-manager. I'm seeing this issue occur on a cluster where I have cert-manager installed, as I want it to be, but it's presence appears to prevent any namespace deletion.

@munnerz I have this problem while using Helm (helm delete --purge cert-manager).
Just to clarify, shouldn't it remove all resources just as a manual installation would?

It'd be great to have an "uninstalling cert-manager" guide as part of our docs in future, as I think this would help with issues like this 😄

@munnerz waiting for it! 👍
It took me a long time googling for it until I found your comment:

The TL;DR, is that you should uninstall cert-manager using kubectl delete -f cert-manager.yaml (where cert-manager.yaml is the file that you originally ran kubectl apply -f with).

and so I understood that what I need was this simple command:
kubectl delete -f https://github.com/jetstack/cert-manager/releases/download/v0.8.1/cert-manager.yaml

😉

While this ticket is closed, isn't it worth considering that cert-manager breaks the ability to delete namespaces in general?  We regularly wipe namespaces for development and testing, but resources ultimately persist, unless we expressly delete them, and/or delete all certmanager resources first.  Is this expected behavior?
For future travelers, the only workaround I've found is:
for i in $(kubectl api-resources --namespaced | awk '{ print $1 }'  2>&1 | grep -v error | tail -n +2 | grep -v persistent ) ; do kubectl delete $i --all ; done

I want to restate that for me the problem isn't uninstalling cert-manager, but that resources that were created by cert-manager in another namespace aren't able to be deleted, so they hold up the deletion of the namespace at Terminating.
From trying to delete the resources individually, I get errors like:
Error from server: conversion webhook for &{map[apiVersion:cert-manager.io/v1alpha2 kind:Certificate metadata:map[creationTimestamp:2020-04-09T17:23:46Z
generation:1 name:ingress-tls namespace:development ownerReferences:[map[apiVersion:extensions/v1beta1 blockOwnerDeletion:true controller:true kind:Ingre
ss name:ingress uid:42957329-135a-44e4-8823-e104a94d9e74]] uid:897ca221-698a-4ea0-94b9-e092932ac467] spec:map[dnsNames:[<snip>] is
suerRef:map[group:cert-manager.io kind:ClusterIssuer name:letsencrypt-production] secretName:ingress-tls] status:map[conditions:[map[lastTransitionTime:2
020-04-09T17:23:46Z message:Certificate does not exist reason:NotFound status:False type:Ready]]]]} failed: Post https://cert-manager-webhook.cert-manage
r.svc:443/convert?timeout=30s: service "cert-manager-webhook" not found

For anyone wandering into here, I managed to solve this by updating my CRDs as noted here.


Unable to reproduce the paper results!

Hi,
I'm attempting to reproduce the paper's results, particularly using the provided datasets. On one dataset (under data/input/), I ran the model with the identical experimental settings as described in the paper. However, the model learns nothing. I was hoping you could help me figure out what I'm missing. thanks.

Same problem here. I've printed the output scores of the learned model with test data = train data, and it turns out that all scores are above 0.5 (i.e. all being classified to 1). One thing I noticed was that examples with label 0 tends to have scores less than 0.51, and those with label 1 tends to have scores above 0.51.

After months reviewing the data, I can tell you that the partial dataset is filled with errors - if not intentional. For example, in ffmpeg 27295 the method av_get_pixel_fmt() has been later changed to av_get_pix_format() which results in a different graph structure.  The fact that so many people can't reproduce the results makes me think Devign is worth throwing out of related works.  I will be switching to SARD and only SARD from now on. :)

After months reviewing the data, I can tell you that the partial dataset is filled with errors - if not intentional. For example, in ffmpeg 27295 the method av_get_pixel_fmt() has been later changed to av_get_pix_format() which results in a different graph structure. The fact that so many people can't reproduce the results makes me think Devign is worth throwing out of related works. I will be switching to SARD and only SARD from now on. :)

Hi，where can I find the dataset SARD you have mentioned?thanks!

@NikolasBielski
Hi，where can I find the dataset SARD you have mentioned?thanks!

@NikolasBielski Hi，where can I find the dataset SARD you have mentioned?thanks!

You can find it at https://samate.nist.gov/SARD/.


Do you have a demo running?

Is there any demo running somewhere?

not yet... right now the development as stopped


Create weekly newsletter

Hi, I currently use your service and receive emails, would you still like assistance with this or is this issue closed? Thanks and Happy New Year!

Hi, would you still like assistance with this or is this issue closed? Thanks and Happy New Year!


Remove title rename limitation from private-channels.md

Closes https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/9429.
You can change the title like any other meeting:

Learn Build status updates of commit 62bb666:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Teams/private-channels.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

@MikePlumleyMSFT
PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


HPPC-186: Sample to change the iteration order of KTypeScatterSet.toArray()

Sample code for https://issues.carrot2.org/browse/HPPC-186

Hi @bruno-roustant !
Please note that HPPCRT-18 was sperseded by : https://github.com/vsonnier/hppcrt/issues/38. In short, I still iterate "in reverse" in HPPC-RT but I also kept the "perturbation" policy in the end.

Hi @vsonnier !
Interesting, thanks for sharing.
Here the approach with a slot increment would pass. I think it is worth reviving this discussion again :)

Hello both (hi @vsonnier , long time no see!). Yes, it would be great to resign from the seed mix. I admit I can't remember whether this was considered at the time - it's been a while. I only remember I also didn't anticipate the original problem and I went the really defensive way of trying to absolutely minimize the chances of this happening in a production system. We do copy/ manipulate large associative arrays and if this every happens it's effectively a live deadlock...
If we can somehow prove the iteration order (or skip) prevents this with a really low-likelihood of happening then it'd be a great improvement. Remember those maps can be copied over and over between instances.
Separately from theoretical aspects there is also the practical aspect: seed mix adds overhead but it's fairly local and should play well with CPU caches, and even hotspot optimizations (in tight loops). The same with linear iteration - it is really cache-friendly. Adding skips will affect this property.

I'll take some time to think about how to prove the really low-likehood of collision avalanche.
About the seed mix overhead, I saw a consistent and visible perf impact during the numerous benchmarks we did when evaluating worm hashing. For all benchmarks, mixed hppc had visible overhead compared to scatter, due to this slightly more complex hash function, on multiple machines and with two different benchmark implementations (both based on JMH). So I'm convinced that the hotspot optimization does not have a good effect there. At least for jdk 11 and below. Maybe that would be different with C/C++.
For the cache-friendliness of the iteration, well I suppose the JMH benchmark will be the arbitrator. Based on the CollisionAvalanche I ran with the sample code, the small skip seems to work well since it provides 33% speed improvement for scatter hppc set compared to standard hppc set. We'll need to run more benchmarks for more sizes to confirm this.
Ok, this makes me confident I can continue to work on the subject. That's what I wanted to discuss first.

I agree the "scatter" version is slightly faster - it is! That's why I left it in the code (for some applications it's useful). The question remains what happens when you don't have full control over how your collections are being used - this is what I'm concerned with.
As for JVM/hotspot optimizations and cache influence: it's a really, really tough material to provide insight on since each and every architecture (not just the CPU but also memory controller, etc.) will have varying results...  I really miss simple computers sometimes when you could just read clock ticks out of each instruction and just know for certain...

I have tried a few things.
First I quickly verified that the small prime increment (let’s call it skip-iteration-order) is indeed an iteration order: it retrieves all slots and only once. I ran brute force from 2^0 to 2^28. It works also for any starting slot (could be random instead of always 0).
Then I instrumented KTypeHashSet to count each time we scan linearly the next slots after a hash collision. I called this collisionScanCount. I ran tests to copy Sets for many source and target sizes, focusing mainly on heavy loaded sets (as in CollisionAvalanche benchmark). Each time I measured the collisionRatio = collisionScanCount / targetSet.size() to estimate the penalty for collisions. For the case of heavy loaded set, I got collisionRatio=0.99 for KTypeHashSet and collisionRatio=0.75 for KTypeScatterSet. This is nearly 25% less collision slots scanned. And this corresponds to the perf gain I see for B004_HashSet_CollisionAvalanche: hppc_scatter is 28% faster.
Then I modified B004_HashSet_CollisionAvalanche to have copy cycles: a source set is copied to an empty target, then the target becomes the source and is copied to another target and so on. Same perf gain 28%.
Actually when a set is copied, the skip-iteration-order is used to read the source set but once copied the keys are in the target hash (mixPhi) order. So for any copy between two hppc sets the iteration order is always different than the target hash order. That’s why it avoids collisions. I don’t think we need to include randomness to ensure this property.
So this could protect forEach(), iterator(), toArray().
What if a non-hppc collection is copied to a hppc set? The source collection could have an iteration order based on the mixPhi order. This is actually the case with Fastutil and Koloboke collections, they use the same mixPhi. In this case only the current keyMixer approach guarantees the protection (I could imagine a buffered-shuffling iterable wrapper to iterate the keys in modified order, but the caller would have to remind it).

Can you file a patch with this, Bruno? Doesn't have to be commitable, I'd just look and give it some thought but it'd be clearer for me to see what you want to do with those prime increments, etc.

This PR already provides the code for protecting toArray().
I'll add to iteration protection for forEach() and iterator().

Hi Bruno.
bq. First I quickly verified that the small prime increment (let’s call it skip-iteration-order) is indeed an iteration order: it retrieves all slots and only once. I ran brute force from 2^0 to 2^28. It works also for any starting slot (could be random instead of always 0).
I think this one is relatively easy to prove mathematically too - if the "skip" is a prime m and underlying table is of size 2^n then the two are relatively prime to each other so for any sequence of i=0..(2^n-1), i * m mod 2^n is guaranteed to fill any bucket within 0..2^n-1. This also applies if you add an initial "shift" (it merely rotates the initial sequence of i's. The same idea is used in open addressing with double hashing.
As for practical repercussions - I think you may be right. I'll have to spend some time on this tinkering with actual possibilities but please do feel free to provide a patch that does implement this idea.

I pushed the code. Now it is on KTypeHashSet directly instead of KTypeScatterSet.
The idea is that each time we make a copy of a HashSet, the iteration order is kind of "shuffled" to ensure it is not the same as the hash order in the target set.
CollisionAvalanche benchmark before the change:
B004_HashSet_CollisionAvalanche.run __ HPPC _____________ 0.024 ± 0.002 s/op
B004_HashSet_CollisionAvalanche.run __ HPPC_SCATTER __ 1.062 ± 0.059 s/op
B004_HashSet_CollisionAvalanche.run __ FASTUTIL _________ 1.084 ± 0.087 s/op
B004_HashSet_CollisionAvalanche.run __ KOLOBOKE ________ 1.519 ± 0.036 s/op
CollisionAvalanche benchmark after the change:
B004_HashSet_CollisionAvalanche.run __ HPPC _____________ 0.024 ± 0.001 s/op
B004_HashSet_CollisionAvalanche.run __ HPPC_SCATTER __ 0.016 ± 0.001 s/op
B004_HashSet_CollisionAvalanche.run __ FASTUTIL ________ 1.058 ± 0.020 s/op
B004_HashSet_CollisionAvalanche.run __ KOLOBOKE _______ 1.529 ± 0.081 s/op

Looks good to me. I wonder if we should just drop the seed-mixed version entirely and just use this instead. The same would apply to maps too, I guess? I can fold this in and do this, let's see what happens. :)

Yes, cool! To drop the seed-mixed version is the next step, and apply to map.
I'm ready to help if you want. I could push the change for the map. I could also help to drop stuff.

Please do go ahead with this if you have spare cycles!

First step: I removed HashOrderMixingStrategy from KTypeHashSet (KTypeScatterSet to be removed).
In the process I got a significant test failure with HashCollisionsClusteringTest.testHashSetClusteringAtFront2().
Awesome tests!
It caught a naughty weakness of the hash order with increment: when we repeatedly iterate the keys of sets A1,A2,...,An (in the test the top 5K keys) and add them to the same set B, we actually follow the same hash iteration order for the sets A, so the distribution in B is stacking at the beginning of the array. With the very nice visualizeKeyDistribution() in this test I could confirm that. It provoked collision avalanches.
So I fixed this weakness by making the hash iteration order change between iterations. There is an iteration seed, which changes for each iteration start, that we use to determine the iteration slot start (not always 0) and the iteration increment (only needs to be an odd number). That fixes immediately the test (and the cool visualization shows a uniform distribution).
This works with multi-threads without synchronization.

The HashCollisionsClusteringTest.testHashSetClusteringAtFront2() test is fixed but the problem is not completely fixed.
The test reused the same A by clearing it and refilling it. So the iteration seed changed.
But if the test was creating new A instances each time, it would currently have the same initial seed.
I'll fix that in the next commit.

Hi Bruno. Just wanted to say I do follow your updates but haven't had the time to look at the code so far. Good the tests are catching odd cases - I'm sure there are more of these lurking around the corner. :)

The "format violations" can be automatically corrected - just run ./gradlew spotlessApply and it'll reformat the code. There is a Google format plugin for IntelliJ as well but they're not always in perfect sync (with respect to imports, for example). I am generally happy with the formatting done by google's formatter - it is very convenient with cross-code refactorings and generally leaves the code in much better shape over time.

I removed ScatterSet and ScatterMap. Actually now HashSet and HashMap are like old scatter versions with a kind of shuffled iteration order.
Tests are passing and the speed is good.
Next step will be to remove HashOrderMixingStrategy and associated classes.
During this change I fixed ObjectIdentityHashSet.hashKey to use System.identityHashCode() (it didn't use it although I think the intent was to use it like in KTypeVTypeIdentityHashMap).

Yep, I saw that. Looks like a legitimate bug to me.


[WIP] Refactor accounts and blocks services

Closes #588
Description
Refactor the services in pkg/flowkit/servies as outlined in #588

For contributor use:

[ ] Targeted PR against master branch
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work
[ ] Code follows the standards mentioned here
[ ] Updated relevant documentation
[ ] Re-reviewed Files changed in the Github PR explorer
[ ] Added appropriate labels

We should outline the interface in the issue first. There's no conclusion there on what will the final version be.

closing as more discussion is needed


Improvement for k8s.io/docs/setup/production-environment/container-runtimes/

Would be nice if the page put something in the CRI-O section on if/how CRI-O could be installed on RHEL8 without having to purchase OpenShift.
Thanx,
David Bandel
<EMAIL_ADDRESS>

/kind feature
/triage accepted
/language en
/priority backlog
Documenting container runtime deployment is valid; you need a container runtime to use Kubernetes.


Azure SDK Review - [Azure SDK - Azure.Core & Event Grid System Events]

New SDK Review meeting has been requested.
Service Name: Azure SDK - Azure.Core & Event Grid System Events
Review Created By: Ronnie Geraghty
Review Date: 10/02/2023 02:05 PM PT
Hero Scenarios Link: Not Provided
Architecture Diagram Link: Not Provided
Core Concepts Doc Link: Not Provided
APIView Links: .NET,
Description: Combine session for Azure.Core and Event Grid System Events
Detailed meeting information and documents provided can be accessed here

Meeting updated by Ronnie Geraghty
Service Name: Azure SDK - Azure.Core & Event Grid System Events
Review Created By: Ronnie Geraghty
Review Date: 09/25/2023 02:05 PM PT
Hero Scenarios Link: Not Provided
Architecture Diagram Link: Not Provided
Core Concepts Doc Link: Not Provided
APIView Links: .NET,
Description: Combine session for Azure.Core and Event Grid System Events
Detailed meeting information and documents provided can be accessed here

Meeting updated by Ronnie Geraghty
Service Name: Azure SDK - Azure.Core & Event Grid System Events
Review Created By: Ronnie Geraghty
Review Date: 09/25/2023 02:05 PM PT
Hero Scenarios Link: Not Provided
Architecture Diagram Link: Not Provided
Core Concepts Doc Link: Not Provided
APIView Links: .NET,
Description: Combine session for Azure.Core and Event Grid System Events
Detailed meeting information and documents provided can be accessed here

Meeting updated by Ronnie Geraghty
Service Name: Azure SDK - Azure.Core & Event Grid System Events
Review Created By: Ronnie Geraghty
Review Date: 09/25/2023 02:05 PM PT
Hero Scenarios Link: Not Provided
Architecture Diagram Link: Not Provided
Core Concepts Doc Link: Not Provided
APIView Links: .NET,
Description: Combine session for Azure.Core and Event Grid System Events
Detailed meeting information and documents provided can be accessed here

Meeting updated by Ronnie Geraghty
Service Name: Azure SDK - Azure.Core & Event Grid System Events
Review Created By: Ronnie Geraghty
Review Date: 09/25/2023 02:05 PM PT
Hero Scenarios Link: Not Provided
Architecture Diagram Link: Not Provided
Core Concepts Doc Link: Not Provided
APIView Links: .NET,
Description: Combine session for Azure.Core and Event Grid System Events
Azure.Core Topics:

Non-Azure Branding for TypeSpec

Detailed meeting information and documents provided can be accessed here

.NET Api View - https://apiview.dev/Assemblies/Review/5cd1d690711c418dae06f4661b576cb0/d7bd4797ccd14629b47ec38c103c2177?diffRevisionId=938b7a6fb6ac436fad73c85742e94178&doc=False&diffOnly=True
Swagger - https://github.com/Azure/azure-rest-api-specs/pull/25919/files


New address location not working

After pressing M key, input box for new address location appear and immediately disappear and screen is redrawn. I can't enter new location at all.

Fixed...


Handle Checked Exceptions With Lambda Expression

Task Description
We should introduce helper classes/methods like here to help us deal with limitations of lambda expressions and checked exceptions. We should at least create 2 classes/methods:

[ ] ThrowingFunction.unchecked
[ ] ThrowingConsumer.unchecked
[ ] ThrowingSupplier.unchecked

and try to get rid of UndeclaredThrowableException and RuntimeException calls wherever possible, as long as it does not make a code less transparent and clear.
Example of valid usage: consider this part of code. It could be rewritten like this
Connection connection = ThrowingSupplier.unchecked(dataSource::getConnection)
                                        .get();

We can do similar refactoring in the follow up tasks. We can also try to find loops in the source code, that were not able to use lambdas before, but will be using proposed classes/methods.
Acceptance Test

[ ] Building the code and running mvn spring-boot:run in the strongbox-web-core still starts up the application correctly.
[ ] Building the code and running the strongbox-distribution from a zip or tar.gz still works.
[ ] Running the strongbox-web-integration-tests still run properly.

Help

Our chat
Points of contact:

@carlspring
@sbespalov
@fuss86

@fuss86 Hi Przemyslaw, I can start to work on that one ;)

Welcome back, @adavid9 ! Feel free to dig into it!


Add user can change monster name

In case if  a user made a mistake on writing monster's name.

I like that fact that the name can't be changed. This way the monsters are more easily recognized. If you are used to interact in a battle with "Bubble" and suddenly it is call "Burst" then I will be confused.
If you made a mistake during creation you could create a new one with the correct spelling.
I could see also a solution that the monster name could be changed until it wakes up the first time.

I think we should add a "name change scroll" item, or something...


[Feature] Demo: Navbar component

❓ Type of change

[x] 📖 Documentation (updates to the documentation, readme or JSdoc annotations)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description
Add a Navbar component demo.
Hide/show on scroll.
Example : View on playground
📝 Checklist

[ ] I have linked an issue or discussion.
[ ] I have added tests (if possible).
[x] I have updated the documentation accordingly.
[ ] I have updated the changelog.

@titouanmathis @perruche What about creating a real component instead a demo ? Have you some ideas for renaming it ?


Check for cluster connection when public access disabled

If public access is disabled, it is no longer assumed that Rancher cannot communicate with the cluster API. Instead, Rancher attempts to contact the API. If this is successful, then Rancher can communicate with the cluster as normal (through the API access endpoint) instead of tunneling.
This is most important on cluster creation/import. In this case, since Rancher can still communicate with the API endpoint, the user does not need to run the import command that is displayed.
Issue:
https://github.com/rancher/rancher/issues/30403

Yes, the import command will pop up, but it can be ignored by the user if they have setup the VPC correctly.


WDIO8 / wdio-wait-for - Condition is not a function

After updating to WDIO8 and wdio-wait-for 3.0.1 I started to get this "Condition is not a function" whenever I try to use "urlContains"
Simple code :
import { urlContains } from 'wdio-wait-for';

export async function crossSellHandler(option: string) {
    await basePage.waitForPageStopLoadding()
    return await browser.waitUntil(urlContains("/test/"), { timeout: 60000, timeoutMsg: "Test page is not displayed" });

Error :
Error: Condition is not a function
at processTicksAndRejections (node:internal/process/task_queues:95:5)
at async World.crossSellHandler (...\test.ts:17:12)

Any clues where to look? The same code was properly working before updating.
FYI : It works fine with version 2.2.6

@LordPato thanks for filing the issue. You want to help us get on the bottom of this? With v3 we've rewritten the package to ESM which might be responsible for this. Let me know if we can help contributing to this project.

The way to make it work was to add another await before the urlcontains like
import { urlContains } from 'wdio-wait-for';
export async function crossSellHandler(option: string) {
await basePage.waitForPageStopLoadding()
return await browser.waitUntil(await urlContains("/test/"), { timeout: 60000, timeoutMsg: "Test page is not displayed" });
Which doesn't seems right. For the moment I rolled it back to 2.2.6 while I have a different branch using 3.0.1 to investigate furtgher.

It seems like the CJS export functions are written incorrect, e.g. with titleContains, instead of:
exports.titleContains = async function (expectedTitle: string) {
    const fn = (await importBrowserPromise).titleContains
    return fn.call(this, expectedTitle)
}

it should be:
exports.titleContains = function (expectedTitle: string) {
  return async function (this: WebdriverIO.Browser): Promise<boolean> {
    const fn = (await importBrowserPromise).titleContains
    return fn.call(this, expectedTitle)
  }
}

Do you want to help us get a fix for them in place?

Sure. I'll help you with it.


add next fee widnow rep stats to current fee window rep user stats, a…

…dded refresh fee window bounds when tokens are transferred, it's for participation tokens, mint log didn't come in
Need newest augur-node master
https://app.clubhouse.io/augur/story/15693/github-show-rep-on-next-fee-window
to repro:
confirm that when user fills up dispute bond and market goes to next fee window that their REP shows up in the staked REP totals in dispute header page.

Coverage decreased (-0.08%) to 64.489% when pulling b14c0a524be24eaa2a0f942411af639bba504cc2 on tomhaile/ch15693/github-show-rep-on-next-fee-window into cdb663b3207617e3b8d9263c900482c155081d4d on master.


Web page not loading completely

Hi,
First of all, i would like to thank you for all what you have done, great work!
But I have one issue:
When opening web page of the esp, page is not loaded completely (see attach).
Everything else works, I can listen to music, issue additional commands thru web and so on...
Can you, please help?
Thank you

Check if index.html in the data directory (the files in the SPIFFS) has the right length. It should be 6563 bytes.  If you connect a serial monitor, you should see a directory listing in the debug output.
An ESP-12 should have more than enough space to hold the files.
You could use the "show source" of your browser to examine the contents (specially near the end) of the index.html file.

I couldn't find the problem. File index.html is correct size and it is listed in serial output.
I have also tried to delete most of the stuff from index.html and leave just the header and one word in the body, then page was loaded completely. After that I have tried to add more and more stuff to body and then after some point page didn't want to load anymore. It looks like if the index.html grows over certain size then problem starts.
I have also tried to downgrade Arduino ide to 1.6.8 and ESP8266 package to 2.2.0 as stated in your PDF but it didn't help. I have also tried to flash esp as 1M(3M SPIFFS) to check if memory is corrupted, but behavior was the same.

I think your analysis is right.  Could you try to use a different WiFi network (ask a neighbour or a friend)?
What kind of ESP module are you using?

Hi.
After lots of troubleshooting I have found the problem. Cisco VPN client that I have installed on my PC changed MTU on all my network interfaces to 1300 (default in windows is 1500). And apparently expressIFs IP stack doesn't support MTU size less than 1460. After manually reverting my MTU to 1500 web page loaded successfully. You can close the issue.
Thank you for all your support!


Update add-clustervirtualmachinerole.md

I have found an issue related to this Cmdlet in a large enterprise, we can add a note and maybe the link of the blog to explain better.

@DanieleDeAn Thank you for the valuable find. The PR is being rolled review. Much appreciated.

@nenonix @JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@dariomws306 @get-itips @tseward
Peer review needed for this PR. Thanks.

Looks ok, but I think we can improve it. Maybe two different notes?
Note: This cmdlet cannot be run remotely without Credential Security Service Provider (CredSSP) authentication on the server computer.
Note: This cmdlet generate a .TMP file under the user temp folder, and you can't have more than 65535 .TMP files generated by this cmdlet, otherwise you will receive an exception "The File Exist". For more information, please go to:
https://techcommunity.microsoft.com/t5/ITOps-Talk-Blog/TQA-Add-ClusterVirtualMachineRole-fails-with-the-error-quot-The/ba-p/713344
What do you think?

Agree with @dariomws306

Also, I think the same note should be added to W2012 and 2012 R2 versions.

[!NOTE] stands out in documentation and is the standard for adding NOTES. See https://docs.microsoft.com/en-us/sharepoint/upgrade-and-update/install-a-software-update as one example near the top in the purple-ish section of a note.

[!NOTE] stands out in documentation and is the standard for adding NOTES. See https://docs.microsoft.com/en-us/sharepoint/upgrade-and-update/install-a-software-update as one example near the top in the purple-ish section of a note.

Hi @tseward many thanks for the info, now is clear, but is also clear that the preview do not read correctly "[!NOTE]" syntax, or may be I wrong something :). Many thanks

@JohanFreelancer9, @nenonix Hi,  I have modified the file based on your suggestion but probably I need to study more Github! :) because now i see that your info are outdated, sorry guys but this is a new world for me :), If I need to change something please let me know ;)

Copy Review - Please insert spaces as indicated.

Done and many tnx ;)

Hi @e0i we have a version ready to be merged in the master branch, I need to do something? :)
Many Thanks :)

Hello @DanieleDeAn, It seems that the notes need to be added for other supported Windows versions, via dariomws306's comment below,

Also, I think the same note should be added to W2012 and 2012 R2 versions.

The corresponding folders for these two Windows versions can be found under winserver2012-ps winserver2012r2-ps directories.
Thank you.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: DanieleDeAn sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Hi @e0i, you are correct ! I miss this info , I have modified also the version for 2012 and 2012R2, and many thanks to @dariomws306 ;).
Cheers!

@DanieleDeAn Thank you for your valuable contributions and taking the time to follow-up with reviewer's suggestions. The PR will be merged soon.

@AndreaBarr The PR is now ready to be merged. Thanks.


Check for the possibility that a TextureId is a number while cloning it.

imodel-native: https://github.com/iTwin/imodel-native/pull/820

Should team responsible for Revit Connector be contacted and ask that they bring TextureIds over as hexadecimal strings?

Should team responsible for Revit Connector be contacted and ask that they bring TextureIds over as hexadecimal strings?

100%. It is never correct to store element IDs in JSON as numbers, JavaScript represents them as doubles which lack sufficient precision. @timlawrence-bentley @jffmarker

Add more commits by pushing to the [nick/rendermaterial-enhancement](/iTwin/itwinjs-core/tree/nick/rendermaterial-enhancement) branch on iTwin/itwinjs-core.

I will reach out to them with more details, i.e. iModel this was observed in

It was recently discovered that another iModel has textureIds as numbers instead of hexadecimal strings, and there was no revit connector involved. The two mentioned to me were the Mstn and Navisworks connector. So one of the two, or both may also be bringing textureIds over as numbers.
This has caused crashes in transformation, so I've made a change to map these ids to invalid instead of erroring out on them. The reason for the crash is that findtargetElementId expects a string and not a number.
@jffmarker @timlawrence-bentley Could you have someone look at both connectors to see which is bringing textureIds over as numbers?

It was recently discovered that another iModel has textureIds as numbers instead of hexadecimal strings, and there was no revit connector involved. The two mentioned to me were the Mstn and Navisworks connector. So one of the two, or both may also be bringing textureIds over as numbers.
This has caused crashes in transformation, so I've made a change to map these ids to invalid instead of erroring out on them. The reason for the crash is that findtargetElementId expects a string and not a number.
@jffmarker @timlawrence-bentley Could you have someone look at both connectors to see which is bringing textureIds over as numbers? I don't have access to the iModel this was found in, but we could likely get that access if necessary. Let me know.

From skimming source, I think NWD is the culprit. I'll file a bug on the internal backlog.

It was recently discovered that another iModel has textureIds as numbers instead of hexadecimal strings, and there was no revit connector involved. The two mentioned to me were the Mstn and Navisworks connector. So one of the two, or both may also be bringing textureIds over as numbers.
This has caused crashes in transformation, so I've made a change to map these ids to invalid instead of erroring out on them. The reason for the crash is that findtargetElementId expects a string and not a number.
@jffmarker @timlawrence-bentley Could you have someone look at both connectors to see which is bringing textureIds over as numbers? I don't have access to the iModel this was found in, but we could likely get that access if necessary. Let me know.

From skimming source, I think NWD is the culprit. I'll file a bug on the internal backlog.

Could you send me a link to that bug so I can track it?

@mergifyio backport release/4.8.x


Forget Password Fix

Thanks for this awesome project,from the demo the forgot password url is not working.
Can you kindly fix the url.

@denzelwamburu Forgot password was never implemented. If you would like to see its implementation you can check out my other project hackathon-starter.


Automate upgrade test

We'd like to add the ability test the deis upgrade process in an automated fashion. This process currently is:

spin up the previous release of deis
deploy an app to it
verify that the app is running properly (smoke test)
tear down deis around the app
deploy the current version of deis
verify that the app is running properly (smoke test)

Ideally this test should be able to run on all providers, but for the first round of implementation I'm only looking to support Vagrant and EC2.

Ideally this test should be able to run on all providers,

I think EC2, Azure, and GCE are ones we should shoot for. Other providers don't have separate volumes and etcd would almost certainly fall to its knees under heavy IO. See https://github.com/deis/deis/issues/3648#issuecomment-121672023


Support force_path_style in s3 backend overrides

I'm seeing this error when I try running tflocal within a docker container with an s3 backend:
Successfully configured the backend "s3"! Terraform will automatically
use this backend unless the backend configuration changes.
Error refreshing state: RequestError: send request failed
caused by: Get "http://my-bucket-tfstate.localstack:4566/uat/uat.tfstate": dial tcp: lookup my-bucket-tfstate.localstack on <IP_ADDRESS>:53: lame referral

This is happening because the s3 backend code is missing the s3_use_path_style logic that's already in the provider overrides.
Unfortunately force_path_style is currently deprecated in favor of use_path_style, but I'm not seeing a mechanism for checking terraform version and switching between the two. That probably needs to be added in another PR, so I'm using force_path_style for the most compatibility for now.

Hi @pb-dod!
Can you please provide what version you are using of terraform, terraform-local and the aws provider.
Furthermore could you give some context how you run Localstack in your docker environment.

I'm on terraform 1.5.4 (released only a few months ago) and terraform-local 0.15.
Here's a gist with the Dockerfile, docker-compose file, backend.tf, and the shell script to run tests using tflocal: https://gist.github.com/pb-dod/52a14cee9d003a07250853cebc0f2164

Hi @pb-dod!
I have looked into your suggestion and unfortunately I must decline your PR idea, however I can give you 2 alternatives how to configure your compose file as the error is related to networking.
Alternative 1 (probably the simpler):
Set the network explicitly so you can pin your localstack instance to a fix IP and use it as a DNS server to serve localhost.localstack.cloud, this way you don't need to manage domains in your compose file. (Remove the S3_HOSTNAME from your script)
See example below:
version: "3.8"

services:
  localstack:
    container_name: "localstack_main"
    image: localstack/localstack:2.3.2
    ports:
      - "<IP_ADDRESS>:4510-4559:4510-4559"  # external service port range
      - "4566:4566"
    environment:
      - SERVICES=dynamodb,ec2,iam,kms,rds,route53,s3,ssm,sts
    networks:
      default:
        ipv4_address: <IP_ADDRESS>

  tester:
    build: .
    depends_on:
      - localstack
    environment:
      AWS_ACCESS_KEY_ID: test
      AWS_SECRET_ACCESS_KEY: test
    command: "./run-tests"
    dns:
      - <IP_ADDRESS>
    dns_search:
      - localstack

networks:
  default:
    driver: bridge
    ipam:
     config:
       - subnet: <IP_ADDRESS>/16
         gateway: <IP_ADDRESS>

Alternative 2 (unfortunately not that comfortable):
Set the network aliases for localstack, so docker routes the request to the right place (if you put localstack behind a reverse proxy like traefik and point *.localstack on it you can use this without configuring the aliases)
See example below:
version: "3.8"

services:
  localstack:
    container_name: "localstack_main"
    image: localstack/localstack:2.3.2
    ports:
      - "<IP_ADDRESS>:4510-4559:4510-4559"  # external service port range
      - "4566:4566"
    environment:
      - SERVICES=dynamodb,ec2,iam,kms,rds,route53,s3,ssm,sts
    networks:
      default:
        aliases:
          - s3.localstack
          - my-tfstate.s3.localstack

  tester:
    build: .
    depends_on:
      - localstack
    environment:
      AWS_ACCESS_KEY_ID: test
      AWS_SECRET_ACCESS_KEY: test
    command: "./run-tests"

Hope this helps, and thx for your patience.


File order not maintained

I call the closure compiler like:
java.exe -jar dev\jars\closure\compiler.jar --compilation_level WHITESPACE_ONLY --formatting PRETTY_PRINT --formatting PRINT_INPUT_DELIMITER --formatting SINGLE_QUOTES --js_output_file jQuery_a52d83c3.min.js --js jquery.js ui\core.js ui\widget.js ui\mouse.js ui\draggable.js ui\droppable.js ui\resizable.js ui\selectable.js ui\sortable.js ui\button.js ui\position.js

When I use the compiler.jar from github binary downloads compiler-20151216.tar.gz everything is fine. When I use compiler-20160208.tar.gz the file originial order is not maintained in the compressed file which breaks the code, as the js files have to be included in a specific order.
I tried the new parameter --dependency_mode NONE but it changed nothing. According to the docs should NONE be the default.
I'm on Windows 10, 64 bit JAVA 8 as provided by Oracle.

Thanks for the report. This should be fixed in #1549.
For now, the workaround is to specify multiple "--js" flags, like
--js jquery.js --js ui\core.js --js ui\widget.js, etc.


Allow rendering a text preview using custom fonts

Description
Allow usage of custom fonts for the text previews so non-latin chars (or chars not supported by the "goregular" font) can be shown
Related Issue
https://github.com/owncloud/ocis/issues/2620
Motivation and Context
Files containing non-latin can have a proper preview
How Has This Been Tested?

test environment:
test case 1:
test case 2:
...

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Technical debt
[ ] Tests only (no source changes)

Checklist:

[ ] Code changes
[ ] Unit tests added
[ ] Acceptance tests added
[ ] Documentation ticket raised:

Expected usage
Start the thumbnails service using the THUMBNAILS_TXT_FONTMAP_FILE pointing to a json file, for example THUMBNAILS_TXT_FONTMAP_FILE=/myfonts/fontMap.json. Note that all the fonts referred by the font map file must be downloaded manually and be present in the FS.
The json file must contain a json object with the following info:

A "fontMap" key, which contains a map of script => font_file
A "defaultFont" key, which contains the default font to be used in case the target script isn't found in the fontMap

An example of font map file can be the following
{
        "fontMap": {
                "Arabic": "NotoSansArabic-Regular.ttf",
                "Devanagari": "NotoSansDevanagari-Regular.ttf",
                "Latin": "NotoSans-Regular.ttf",
                "Hangul": "NotoSansMonoCJKkr-Regular.otf",
                "Han": "NotoSansMonoCJKsc-Regular.otf",
                "Hiragana": "NotoSansMonoCJKjp-Regular.otf",
                "Katakana": "NotoSansMonoCJKjp-Regular.otf"
        },
        "defaultFont": "NotoSans-Regular.ttf"
}

A full list of available scripts can be found in https://cs.opensource.google/go/go/+/refs/tags/go1.17.2:src/unicode/tables.go;l=3655
In case the target script isn't mapped, the default font will be used. The character of that script will likely appear as a white box instead of the expected character.
The font map contains paths to the font files. The paths can be absolute or relative as shown below:
{
        "fontMap": {
                "Arabic": "NotoSansArabic-Regular.ttf",
                "Devanagari": "NotoSansDevanagari-Regular.ttf",
                "Latin": "/customFonts/test001.ttf",
                "Hangul": "NotoSansMonoCJKkr-Regular.otf",
                "Han": "NotoSansMonoCJKsc-Regular.otf",
                "Hiragana": "NotoSansMonoCJKjp-Regular.otf",
                "Katakana": "NotoSansMonoCJKjp-Regular.otf"
        },
        "defaultFont": "/customFonts/test001.ttf"
}

If the path is absolute, the font will be loaded directly from that path.
If the path is relative, it will be relative to the directory where the font map file is placed. For example, if we assume that the "/fontMaps/fontMap1.json" file contains the content above, the "NotoSans*" paths will refer to "/fontMaps/NotoSans*"
This allows the usage of self-contained packages of fonts that are ready to be used. The thumbnails service would just need to point to the font map file contained in the package.
Known issues

Arabic (and likely any right-to-left) script won't be rendered properly. Glyphs will be printed left-to-right
Glyphs that might require repositioning or that are a combination of 2 or more glyphs, won't be rendered properly. This has been noticed with the Devanagari script such as विश्वस्त हुनुहोस् तपाइँको. Glyphs will be printed separately one after another.
Punctuation symbols might not be rendered properly in some circumstances. This usually involves using multiple scripts such as in 附加到电子邮件的未版本化文档以及安全性有问题的公共云中的影子 IT。. In the example, the last character is used in chinese or japanese languages, and latin fonts (such as NotoSans-Regular.ttf) doesn't contain that char. The problem is that the char is wrongly interpreted as latin because it's preceded by a latin char.

To Do List

Include tests
Check possible concurrency problems (docs say that the font.Face isn't thread-safe)

:boom: Acceptance tests Web-Tests-ocis-ocis-storage-1 failed. The build is cancelled...

We'll leave the known issues aside.
Both first and second issues are too complex to tackle on our own just for a text preview. It involves a text shaping mechanism which isn't present in the "golang.org/x/image" package, and I didn't found alternative packages (maybe https://github.com/benoitkugler/textlayout).
Note that this goes beyond the initial thought of the problem. It isn't "just" draw the chars from right to left, or reposition a char, but substituting a combination of glyphs for a different one.

:boom: Acceptance tests accountsUITests failed. The build is cancelled...

:boom: Acceptance tests Web-Tests-ocis-ocis-storage-1 failed. The build is cancelled...

:boom: Acceptance test localApiTests-apiArchiver-ocis failed. Further test are cancelled...

Rebased. Note that there have been conflicts and the code has been slightly changed:
juan@juan-VBox:~/src/ocis/ocis$ git rebase origin/master
First, rewinding head to replay your work on top of it...
Applying: Allow rendering a text preview using custom fonts
Using index info to reconstruct a base tree...
M	thumbnails/pkg/config/config.go
A	thumbnails/pkg/flagset/flagset.go
Falling back to patching base and 3-way merge...
CONFLICT (modify/delete): thumbnails/pkg/flagset/flagset.go deleted in HEAD and modified in Allow rendering a text preview using custom fonts. Version Allow rendering a text preview using custom fonts of thumbnails/pkg/flagset/flagset.go left in tree.
Auto-merging thumbnails/pkg/config/config.go
CONFLICT (content): Merge conflict in thumbnails/pkg/config/config.go
error: Failed to merge in the changes.
Patch failed at 0001 Allow rendering a text preview using custom fonts
hint: Use 'git am --show-current-patch' to see the failed patch
Resolve all conflicts manually, mark them as resolved with
"git add/rm <conflicted_files>", then run "git rebase --continue".
You can instead skip this commit: run "git rebase --skip".
To abort and get back to the state before "git rebase", run "git rebase --abort".

thumbnails/pkg/flagset/flagset.go -> deleted
thumbnails/pkg/config/config.go -> adjusted
thumbnails/pkg/config/mappings.go -> adjusted

I can recheck tomorrow to ensure it still works after the rebase.

Looks good to me. Ready to merge.


Coinbase Wallet extension crashing

Seems to be crashing every single time, sometimes showing the page crashed screen with the message saying Out of memory.
Version 109.0.5362.0 (Official Build) (64-bit), Windows 10 64bit Home, 8gb ram

Thanks but I’m intrested in avx2 builds of chrome in general. I’ll wait till it gets fixed then.

Well I can't even run the latest build, using the mini installer.

@skver0 @gz83 Yeah you must not be doing something right. Thorium has thorium.exe not chrome.exe.


Improve truss watch: respect .truss_ignore when calculating patches

:rocket: What
Although we currently account for truss ignore patterns when detecting file changes with watchfiles.watch, we do not account for the ignore patterns when calculating patches or the truss hash and signature. This means that changes in ignored files will still be patched. This PR fixes that, along with some refactoring and bug fixes.
I didn't profile this thoroughly, but from running on a truss with a .mypy_cache and 1000 model files this doesn't have much positive or negative impact on performance. I think this change is mostly valuable for correctness (we shouldn't calculate patches for ignored files, and we should ensure that we're matching the ignore patterns correctly).

:computer: How

Updates TrussHandle.calc_patch to pass in ignore patterns when calculating patches, hash, and signature
Fixes logic for calculating unignored paths in calc_truss_patch. Uses the pathspec library, which supports .gitignore style pattern matching
Refactors other areas where we calculate unignored paths to use the same logic. This is in truss/util/path
Adds unit test coverage

See my inline comments throughout this PR for the rationale for other small changes.

:microscope: Testing

Unit tests
Tested truss watch locally

FYI @squidarth / @bolasim, I probably won't get around to updating this PR until I'm back from my OOO. Feel free to review when I come back if that helps minimize context switching


[Foundation] Correct sequence initializers for Data when repeating:count: is called and add a memset fast-path

This is a cherry-pick for a critical issue already merged into master

@swift-ci please test and merge


Metadata support for safetensors

Proposed changes
Adds metadata support for safetensors, Closes #633
Checklist
Put an x in the boxes that apply.

[x] I have read the CONTRIBUTING document
[x] I have run pre-commit run --all-files to format my code / installed pre-commit prior to committing changes
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have updated the necessary documentation (if needed)

added some aliases so its a little more readable, but if not desired i can undo that commit

addressed the comments in the latest commit, not finding instances of safe_tensors what file is it in?

not finding instances of safe_tensors what file is it in

Sorry I meant we don't treat is as two words (e.g. we don't do safe_tensors we do safetensors

@dc-dc-dc would you mind adding one more test in python to cover the binding?


Unexpected keyword 'USING' when missing semicolon between link properties

When trying to use this schema:
  type Foo;
  type Bar {
    link l -> Foo {
      a -> bool # missing semicolon
      b -> bool;
      c -> bool;
    };
  };

This error is produced:
error: Unexpected keyword 'USING'
  ┌─ /home/dnwpark/test/dbschema/futures.esdl:4:1
  │
4 │ using future nonrecursive_access_policies;
  │ ^^^^^ error

This is the futures.esdl referenced:
4:    using future nonrecursive_access_policies;

This error is confusing when compared with the error produced by missing semicolons between type properties. This schema:
  type Bar {
    a -> bool # missing semicolon
    b -> bool;
    c -> bool;
  };

Produces this error:
error: Missing ';'
38 │       a -> bool # missing semicolon
   │ ╭──────────────^
39 │ │     b -> bool;
   │ ╰────^ error

When the schema is sent to server it is joined with semicolons. I've tweaked the parser to never ignore semicolons while recovering from the error so this error is not reported at the correct location.


Add MediathekArr

It's work in progress but already working stable for Sonarr and is used by users in production.
https://github.com/PCJones/MediathekArr

Please fix the lint error. Similar to my previous question, I would like to know what it actually does, as I can't really grasp what the goal is from the readme.

For this, it acts as an indexer and downloader for the website of the German public broadcasters ARD and ZDF. It uses an API to get everything downloadable from these websites and can also download it.

Sounds good to me. I've added an additional commit on top to satisfy the linter.


Reload the configuration on external changes

At the moment the configuration server only updates the configuration sent to agents when a save happens via the REST API. It would be nice to automatically detect changes of the used folder in the file system and reload the configuration on this occasion.

This feature has been implemented using #791


Complex division

Resolves #96.

Checklist

Please ensure the following tasks are completed before submitting this pull request.

[x] Read, understood, and followed the contributing guidelines, including the relevant style guides.
[x] Read and understand the Code of Conduct.
[x] Read and understood the licensing terms.
[x] Searched for existing issues and pull requests before submitting this pull request.
[x] Filed an issue (or an issue already existed) prior to submitting this pull request.
[x] Rebased onto latest develop.
[x] Submitted against develop branch.

Description

What is the purpose of this pull request?

This pull request:

adds robust complex division using the algorithm from A Robust Complex Division in Scilab

Questions

Any questions for reviewers of this pull request?

The benchmarks run and compute a result, but I haven't put much effort into making them particularly informative/useful. Do the results go somewhere or do something? I suspect that Math.random might eat up most of the time so that the benchmarks aren't all that representative of the actual performance comparison. Is there something I can do to make them better?
Should I probe it with thousands of examples? I instead aimed for just enough cases to achieve 100% test coverage (which isn't trivial do to all the switching going on).
Finally, how do you feel about the format:
function complexDivision ( re1, im1, re2, im2, output ) {
  output = output || [];
  ...
  output[0] = re;
  output[1] = im;
  return output
}

instead of
function complexDivision ( re1, im1, re2, im2 ) {
  ...
  return [re, im];
}

so that in case you use this heavily you can avoid allocating thousands of two-element arrays. Perhaps a better approach is that cdiv.js has that call signature and the main export allocates storage when necessary. Perhaps this is premature optimization, but leaving the option to avoid lots of extra allocation seems friendly.

@stdlib-js/reviewers

And not to get ahead of things, but do you prefer this rebased into a single commit? Is it best to do that last, once it's determined mergeable? If at all?

Re: rebase. I can do it on my end on merge.

Re: providing an output array. That is an idea. If so, we would need to update other base complex functions. Were we do support providing an output array (or maybe we force this on consumers), I might want the destination array to be first. e.g.,
cdiv( out, 1.0, 2.0, 3.0, 4.0 );

@Planeshifter You have thoughts on this?

Supporting a destination array would take us away from other langs, e.g., Go, but maybe that is okay.

@kgryte anecdotally I've heard of lots of small allocations adding up to kinda thrashing memory/garbage collection, so I hated to make avoiding unnecessary allocation impossible. Honestly the main use case (perhaps) would be to use these functions elsewhere within stdlib, i.e. product of complex matrices. If that were the case, it would definitely be worthwhile to avoid any extra allocation.

I think having a parameter for the output array makes sense for performance reasons; also prefer it to be the first parameter. While these packages will mostly be used elsewhere in stdlib - as Ricky correctly notes, my hunch is that it's best to not force it on users and make it optional though.

@rreusser Supplementary info: https://gist.github.com/ggggggggg/7904745 with possibly additional hard divisions.

I've made the following changes:

Above lint and code fixes.
Added cdiv( output, ... ) style format to output to preallocated storage.
Verify against Julia result for all combinations of all four arguments across 2^linspace(-1023, 1023, 9) (including infinity / NaN).

It could still be worthwhile to test the hard cases though. If nothing else, I'm encouraged to see that it at least seems to handle the very large/very small cases somewhat sensibly.

Could be worthwhile to add those cases though and test the number of bits correct. 😬

Agreed. Feel free to add.

@rreusser Don't need to change any of the commit messages, but, for future reference, here is the commit message guide: https://github.com/stdlib-js/stdlib/tree/develop/docs/style-guides/git#commits.

Ah, thanks. Sorry this has been an exceptionally involved review process. I'm not used to such vigorous review so that I push perhaps a bit too frequently. Plus there are lots of conventions to get used to here. Its greatly appreciated though. And it'll get easier. 👍

@rreusser No worries! This is a learning process for us, as well!! It has been quite illuminating all the various conventions that we have failed to document or are not addressed during linting, etc. Your work (and peristence) is greatly appreciated!

@rreusser Any other changes you need to make? Meaning are we ready for final review?

@kgryte Yeah, I'm content at the moment.

@rreusser To be clear, although this will change in the future when we transition to conventional changelog, our current commit message convention is
This commit will "Fix REPL and JSDoc omissions in complex division"

Ah, got it. "This commit will." Internalizing now.

Yeah, you would probably only have gotten that by clicking on through to the embedded link: https://chris.beams.io/posts/git-commit/. :(

Works for me.

Well, thankfully, also on the TODO list is to have a commit message linter combined with some NLP to protect you from yourself. ;)


Micronaut Function (Serverless) to Oracle Cloud / Gradle Guide Does Not Work OOTB

Expected Behavior
Following the steps in https://guides.micronaut.io/latest/micronaut-oracle-function-gradle-java.html should work as described.
Actual Behaviour
Following guide doesn't produce a working OCI Function: (1) required dependencies are missing and (2) native Executable cannot be built.
Steps To Reproduce
Following the steps in the guide I've generated a sample Function Application for Serverless using the Micronaut Launch. The build pipeline takes care of the docker login and uses the following command to build & push the docker image to OCIR:
chmod 777 ./gradle
./gradlew dockerPush -x test

After creating & setting up an Application and Function, I've tried to invoke the function using:
oci fn function invoke --function-id <OCID> --file "-" --body ""

The function invocation fails with status 502. Inspecting the OCI-side logs, the following exceptions seemed relevant:
Exception java.util.NoSuchElementException: No http provider available; add dependency on one of the oci-java-sdk-common-httpclient-* choices, e.g. oci-java-sdk-common-httpclient-jersey [in thread \"main\"]
...
java.lang.ClassNotFoundException: javax.ws.rs.ProcessingException

The problem seems to be caused by missing dependencies required by the function code:
@ReflectiveAccess
public String handleRequest() {
    String tenancyId = tenantIdProvider.getTenancyId();
    return "Your tenancy is: " + tenancyId;
}

When replaced by a simple string, the function works just fine:
@ReflectiveAccess
public String handleRequest() {
    return "Hello from Function!";
}

To make the original sample code working, I had to add the following dependencies to the dependencies section of the build.gradle file:
implementation("javax.ws.rs:javax.ws.rs-api:2.1")
implementation("com.oracle.oci.sdk:oci-java-sdk-common-httpclient-jersey")

At this point, I'm able to successfully run the sample Function as described by the guide. Then I wanted to complete the native executable part. I've modified the build pipeline to use this code to build & push the docker image to OCIR:
chmod 777 ./gradle
./gradlew dockerPushNative -x test

The build of the native executable container fails, this is the relevant part of the output:
EXEC: Error: Classes that should be initialized at run time got initialized during image building:
EXEC:  io.micronaut.core.reflect.ClassUtils was unintentionally initialized at build time. To see why io.micronaut.core.reflect.ClassUtils got initialized use --trace-class-initialization=io.micronaut.core.reflect.ClassUtils

At this point I'm unable to complete the guide, I don't know how to tweak the native image. Seems similar to https://github.com/micronaut-projects/micronaut-oracle-cloud/issues/761.
Environment Information
Using a Build Pipeline of an OCI DevOps project.
Example Application
No response
Version
4.4.3

Update after checking with @n0tl3ss: the correct fix for the Java part is replacing:
implementation("javax.ws.rs:javax.ws.rs-api:2.1")
implementation("com.oracle.oci.sdk:oci-java-sdk-common-httpclient-jersey")

by:
implementation("io.micronaut.oraclecloud:micronaut-oraclecloud-httpclient-netty")

Unfortunately doesn't fix the native executable part.

Update for the native executable part, as suggested by @n0tl3ss. The build.gradle file requires these changes:
graalvmNative {
    binaries.configureEach {
        buildArgs.addAll(
             "-H:+StaticExecutableWithDynamicLibC", "-H:+ReportExceptionStackTraces",
             "-Dfn.handler=example.micronaut.Function::handleRequest",
             "--initialize-at-build-time=io.micronaut.core.reflect.ReflectionUtils",
             "--initialize-at-build-time=io.micronaut.core.reflect.ClassUtils",
             "--initialize-at-build-time=example.micronaut",
        )
    }
}

and making sure that the file DOES NOT contain this line (the guide suggests to manually add it):
baseImage('frolvlad/alpine-glibc:alpine-3.12')

The function source in Function.java needs to look this way (@ReflectiveAccess used for the class):
package com.example;

import io.micronaut.core.annotation.ReflectiveAccess;
import io.micronaut.oraclecloud.core.TenancyIdProvider;
import io.micronaut.oraclecloud.function.OciFunction;
import jakarta.inject.Inject;
import jakarta.inject.Singleton;

@Singleton
@ReflectiveAccess
public class Function extends OciFunction {
    @Inject
    TenancyIdProvider tenantIdProvider;

    public String handleRequest() {
        String tenancyId = tenantIdProvider.getTenancyId();
        return "Your tenancy is: " + tenancyId;
    }
}


rotate world screen

my output is rotated 90 degrees, is there an easy way to rotate the world box?
PS thanks for sharing this.  I had a bunch of issues with your ilda file renderer in your other library but this is working at rendering fonts.  I don't have an encoder hooked up yet.
PPS First thought is active noise canceling for the galvo, wonder how hard that would be.

Thanks! I'm sorry you had issues with the ILDA project - I'll take a look at the issues on it.
The simplest way is to swap the X and Y galvo inputs and if things are the wrong way round reverse the polarity on whichever is upside down or back to front. May take a bit of experimenting to get it right.

yup, i had my wires wrong, figured it out!  Text is rendering ok.  Trying to get an animated clock running, seems my aspect ratio is a bit stretched out, not sure if that is  the galvo or my setup.


[hi] Incorrect Title in home page

In kubernetes hindi localization home page, there is an improper Title as कुबेरनेट्स रिलीज नोट्स where as the description describes about how to download kubernetes.
So instead of कुबेरनेट्स रिलीज नोट्स there should be कुबेरनेट्स डाउनलोड करें

/sig docs
/language hi

/assign

raised #45457 which solves this

/triage accepted


Is there no onCopy event?

I'm going through some events and I couldn't find this one for p for example.

I think there's actually a few more. Is this because these events are not high priority or is there some technical limitation?

It's just that adding event props is a manual process and no one has needed this even yet. Event PRs are always welcome.

I made a PR with the events I could make work with react-testing-library.

Fixed by #105


It seems REGISTER_MODULE_INITIALIZER(memfs_malloc ...) never real registered itself.

I was playing around hugetlbfs with tcmalloc, however I couldn't get tcmalloc to touch huge pages. So I debugged some and observed the following code never really registered the allocator. Did I do something wrong??
REGISTER_MODULE_INITIALIZER(memfs_malloc, {
  if (FLAGS_memfs_malloc_path.length()) {
    SysAllocator* alloc = MallocExtension::instance()->GetSystemAllocator();
    HugetlbSysAllocator* hp =
      new (hugetlb_space.buf) HugetlbSysAllocator(alloc);
    if (hp->Initialize()) {
      MallocExtension::instance()->SetSystemAllocator(hp);
    }
  }
});

@alk

Ah, I see. Anyone with cmake will have a wrong link sequence. It's a bug.


Hot start cv

This branch isn't ready to merge but I want to go ahead and create a PR to keep things organized.
An issue I'm running into with real-world regularized estimation problems is that when doing CV the grid search sometimes tries a regularizing constant which leads to singularity issues with the Gamma estimates.  To get around this, I'm updating the CV algorithm to stop when the regularizer tries to zero out too many coefficients.
This does change things such that the parallelization now happens over the training/test samples and each alpha is tested sequentially.  One nice side effect of this is that we can use a "hot-start" approach, which feeds the estimated Gamma, Factor values from the previous step into future cross-validation steps (speeds up the convergence some).
Will say more on this once things are finalized.

This sounds like a great feature to add. What exactly do you mean by "zero out too many coefficients", i.e. how is too many defined?

If you choose a large enough regularizing constant (alpha in our case) then you can end up with linearly dependent columns in Gamma (too many elements have been zeroed out).  In terms of the implementation, I'm catching the LinAlgError which is thrown by our attempted Cholesky decomposition when this happens.
Since each training/test partition is fit separately (this is where the parallelization can happen), when this LinAlgError is caught we stop attempting larger alpha values and return the MSE path up to that point.
Currently, fit_cv (which essentially replaces the earlier fit_path method) returns a matrix of MSEs where columns correspond to training/test partitions and rows correspond to alpha values.  Since the alpha path for each training/test partition may stop at different points, rows after a given training/test partition stopped contain NaNs.
It might make sense to do some more handling of this matrix internally (e.g. instead of returning the entire matrix, drop rows which contain a certain number of NaNs and then calculate the mean for each alpha).  I'm still thinking about this (open to input) but figure it makes sense to return too much instead of too little to start.
Let me know if any of that isn't clear.

While I'm discussing this, here is something of a roadmap for what I'd like to have together before I'm really happy with the cross-validation approach:

We should provide some mechanism to properly scale the possible alpha path based on the data.
Instead of returning the MSE matrix we should provide some aggregates as well as something like a path of full data fits.

For a lot of these, I'm digging into how glmnet handles their cross-validation because I think we can follow that approach closely.

Closing this for now as this approach isn't currently active.


Notes appear in random order

Let's say you have a folder with several notes in it. If you select the folder and then export all the notes in the folder to PDF, the notes appear in a random order in the PDF file. I would expect them to appear in the same order as they occur in the folder.

I've added a feature that generates a toc.md file from folders. You can use this feature to generate the toc.md file and then edit the order of the files in it.


Do not reuse old node Id's

See: https://github.com/deso-protocol/core/commit/c8a0a4c129a8d571f932373620d13458f7a0e310, you should not reuse old node id's. It's confusing with old posts that were already tagged with that old node id..
Just don't do that in the future..

It was a placeholder name - see PR https://github.com/deso-protocol/core/pull/359 for context

Ok, thanks for clarifying


🌐 Add norwegian translation for web page

Went through all the keys in the translation file for Norwegian and translated sentences and phrases into ones commonly used in Norwegian.
Left some technology related keywords in their original language, as some of them don't make sense to translate into their Norwegian equivalent. We use a lot of English words in our daily conversations, technologists perhaps most of all.
Added Norwegian to the list of languages on the landing page as well, and changed the order of the languages based on the alphabetical order of their locale. Hope that's fine ✌
Also noticed a bug in the "coming soon" tag for each lesson in the Popover in the Navigation bar, but I'll open a new Pull Request for fixing that.
Edit: PR for badge translation bug (https://github.com/elixirschool/school_house/pull/202)

I don't think we have anyone working on Norwegian translations right now (@doomspork would know better though), and if you're interested in helping out more with them then we can create an Elixir School team for Norwegian translators and add you to it!

I would love to contribute to the Norwegian translations!
I was considering starting on some of them yesterday, but I was unsure whether or not the lessons will/should be written as livebook .livemd's or not. If yes, then I figured I might be introducing more debt if I wrote the new translated files in the "olden ways". 🤔
Would be really exciting to explore writing the lessons in .livemd, it's a banger technology! 💥


docs(pie-docs): DSW-1942 adds cookie banner overview docs

Describe your changes (can list changeset entries if preferable)
Adds the Overview section for the cookie banner
Author Checklist (complete before requesting a review)

[x] I have performed a self-review of my code
[x] I have added thorough tests where applicable (unit / component / visual)
[x] I have reviewed the Docs site preview
[ ] If there are visual test updates, I have reviewed them properly before approving

Reviewer checklists (complete before approving)
Reviewer 1 - Kev

[x] I have reviewed the Docs site preview
[x] If new links have been added, I have tested them in the preview
[x] If there are visual test updates, I have reviewed them

Reviewer 2

[ ] I have reviewed the Docs site preview
[ ] If new links have been added, I have tested them in the preview
[ ] If there are visual test updates, I have reviewed them

Given the failing browser test is unrelated to this change, and these changes are going into another branch and not main. I am merging 👍 We will not merge the other branch into main until everything is green.


Register coding.is-cool.dev

This will be the website for my coding community - The Coding Empire. The domain would also be used for an email address.
Unfinished website link: https://the-coding-empire.github.io/website/

CLA Assistant Lite bot: Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.

I have read the CLA Document and I hereby sign the CLA

You can retrigger this bot by commenting recheck in this Pull Request

I have read the CLA Document and I hereby sign the CLA


mate.mate-backgrounds: 1.24.0 -> 1.24.1

Motivation for this change
Update to version 1.24.1
Things done

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

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

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

Port to meson

build: add meson config files to EXTRA_DIST

Done.


Passthrough of POI DataFrame columns

It would be really nice if all columns ~(or all columns that are not :measure and :position)~[^1] defined in a POI dataframe would be passed to the output DataFrame.
If a DataFrame like this
points_of_interest = DataFrame([
	19900 1 "view point"		"front";
	20000 1 "distant"		"front";
	21000 1 "main"			"front";
	26000 1 "clearing signal"	"rear";
], [:position, :blockid, :label, :measure)

would keep :blockid and :label in the output of trainrun() this would be really nice and much more flexible.
With this behaviour more information for later calculation or display could be added.
[^1]: I think the original position could be very important in the calculated DataFrame. :position and :measure should be kept too

I think it would make sense not to transform POIs from DataFrame to Dict, from Dict to NamedTuple and from NamedTuple to Dict and from Dict to DataFrame.
I have not reviewed the whole code, but maybe it would be possible to directly use and modify the DataFrame?


Missing Binding samples in the BitSlider demo page

Missing Binding samples in the BitSlider demo page

I'm closing this in favor of #5525