urls.py : Implement Url Capturing Method according to newer version of Django

Inside the urls.py of analytics app. I see that we are using older regex method for url capturing
url(r'^realm_activity/(?P<realm_str>[\S]+)/$',analytics.views.get_realm_activity,name='analytics.views.get_realm_activity'),
Now Django (newer version) supports usage of
path('realm_activity/str:realm_str/',,analytics.views.get_realm_activity,name='analytics.views.get_realm_activity')
, for url capturing.
This is not an issue to be precise but I think we should always try to write our code according to the latest version. Please suggest you views should I work on this?

https://docs.djangoproject.com/en/2.1/ref/urls/#path is new in Django 2.0, and thus not supported in Django 1.11 (what Zulip is using).  Closing as wontfix, but I'll add a note to our Django 2.0 follow-up TODO list: https://github.com/zulip/zulip/issues/11341.  Thanks @imsubhams!


Support for stateful Lambdas and multi-shard Kinesis streams

Please review.
This should allow user-provided callables to read/write to the state table associated to a Lambda. It also supports multiple shards by having a separate DynamoDB table to keep the state of different shards.
User-provided callables can access the Lambda state table(s) like this:
import lambdautils.utils as utils

def my_callable(event, state_args=None):
    utils.set_state("my_state_key", "my_state_value", namespace="my_namespace", **state_args)
    assert utils.get_state("my_state_key", namespace="my_namespace", **state_args) == "my_state_value"

This change is

Review status: 0 of 13 files reviewed at latest revision, 4 unresolved discussions.

humilis_kinesis_mapper/resources.yaml.j2, line 101 [r1] (raw file):
use single table and add shard id to the namespace to potentially reduce the costs and deployment time

humilis_kinesis_mapper/lambda_function/handler/init.py, line 16 [r1] (raw file):
have a sentry reporting instead of silencing the exception and reraise

humilis_kinesis_mapper/lambda_function/handler/init.py, line 26 [r1] (raw file):
the same

tests/integration/humilis-kinesis-mapper.yaml.j2, line 24 [r1] (raw file):
init can be removed

Comments from Reviewable


crd example build tasks fails

iac\cdk8s\java\crd>npm run build

><EMAIL_ADDRESS>build
> cdk8s import --language java

Importing k8s v1.25.0...
Importing resources, this may take a few moments...
k8s
Importing resources, this may take a few moments...
jenkins.io
  jenkins.io/jenkins
Importing resources, this may take a few moments...
mattermost.com
  mattermost.com/clusterinstallation
Importing resources, this may take a few moments...
sample-controller.k8s.io
  sample-controller.k8s.io/foo
Error: Key '<<' contains non standard characters (Must match regex '/^(\w|\.|\/|-|#|,)*$/')
    at SafeReviver._sanitizeObj (d:\Apps\Dev\langs\scripting\js\Node\16.x\16.17.1\node_modules\cdk8s-cli\lib\reviver.js:40:23)
    at SafeReviver._sanitizeObj (d:\Apps\Dev\langs\scripting\js\Node\16.x\16.17.1\node_modules\cdk8s-cli\lib\reviver.js:47:22)
    at SafeReviver._sanitizeObj (d:\Apps\Dev\langs\scripting\js\Node\16.x\16.17.1\node_modules\cdk8s-cli\lib\reviver.js:47:22)
    at SafeReviver._sanitizeObj (d:\Apps\Dev\langs\scripting\js\Node\16.x\16.17.1\node_modules\cdk8s-cli\lib\reviver.js:47:22)
    at SafeReviver.sanitize (d:\Apps\Dev\langs\scripting\js\Node\16.x\16.17.1\node_modules\cdk8s-cli\lib\reviver.js:31:14)
    at safeParseYaml (d:\Apps\Dev\langs\scripting\js\Node\16.x\16.17.1\node_modules\cdk8s-cli\lib\util.js:155:17)
    at safeParseCrds (d:\Apps\Dev\langs\scripting\js\Node\16.x\16.17.1\node_modules\cdk8s-cli\lib\import\crd.js:147:46)
    at Function.fromSpec (d:\Apps\Dev\langs\scripting\js\Node\16.x\16.17.1\node_modules\cdk8s-cli\lib\import\crd.js:120:51)
    at runMicrotasks (<anonymous>)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)

when  serving version is upgraded to v0.16.0 to v0.18.0 (which is the latest)

Yeah the << characters came from a merge directive, which YAML technically supports but its very uncommon to exist in k8s manifests, so its ok for cdk8s to fail on it.
@sumupitchayan Can you update the CRD version we use though to the latest and correct one?


overlay should be injected into the page only once

Currently with a multi-entry setup, every entry's hot client will inject a copy of the overlay on to the page. When a compilation error is thrown, it will be sent to all these overlays regardless of which entry chunk it was associated with, therefore there will be multiple overlays open at the same time.
This PR simply ensures there will be only one overlay injected at a time.

I think I'd rather do this using an ID attribute on the created overlay element - so the logic is nicely contained inside the client overlay file.
It might also be worth making the overlay element creation lazy, so we're not adding an element unless there's an error.

Making it lazy would conflict with the ID attribute approach though, because when the clients are evaluated there would be no overlay injected, so there's no way to check for that ID.

Oh I see you mean checking for the id before appending...

Hmm, the issue is not only about the overlay - it's the problems function being called more than necessary. With multiple entries there will also be duplicate warnings in the console.

So, to clarify it a bit: for multiple clients on the same page, each client should its own processUpdate function, because only that client has access to the modules in the same chunk; but only one client should respond to the problems and success, because the errors are sent to every client regardless of which chunk they are associated with.

Ah, I see. The problems reporting needs to be a singleton on the page.
Is it possible to tie the errors to the bundles they're related to? The messages that come over the socket contain the bundle name & hash. I've never used multiple bundles, so I'm not clear how that part works, or how it fits together.

For now I reckon the best approach will be to create a little object/function for problem/success, which creates and holds a reference to the overlay module inside it - and then to memoise this into a singleton on window.
This will avoid the double-warn in the console too.

Made the reporter a singleton, also made the overlay only attached if there are errors.

Released in 2.10.0

:+1:


Add this project to the awesome-harbor list

This is an outstanding project. I think it is a perfect candidate to add it to the https://github.com/container-registry/awesome-harbor.

That looks like your project. ;) Feel free to add a link if you want.
I wish this project had a better name. Oh well.


Define dependencies in a manifest file

Done in https://github.com/e2jk/actioncam-upload/commit/d27128b6da29fa0dc0bf116b6ec9ffdbe7831084


Probability Table Enhancement

Changes

Miscellaneous code cleanup (mostly fixing nullable reference warnings, unused using notifications, etc)
Used some new C# index notation in ProbabilityTable to make some array indexes less verbose (shouldn't have a speed penalty)
Made Random and auto-property, since the only addition of the custom property setter was a null-check on set, and null shouldn't be passed to it anyway since the type is non-nullable

Allowing null and converting it to MizuchiRandom is also possible, however doing so properly would require one additional nullable type annotation; if you prefer this method I can switch to it)

Modified construction/Reset interface in ProbabilityTable to accommodate IEnumerables, as well as to allow the user to avoid copying of lists/arrays where it is undesirable (see notes)
Changed IndexOutOfRangeException to InvalidOperationException, since the definition as given in the C# documentation should fit slightly better

Notes
Most of the code cleanup changes were minor the primary change in this PR has to do with the Reset/constructor interface of ProbabilityTable.  The following summarizes the changes/intent:
First, any function or constructor that previously took a list of (TItem, double) tuples, now has two versions; :
public void Reset(IEnumerable<(TItem item, double weight)> items)
{
    // Call ToArray on items, and pass it to another Reset overload such that the newly created list
   // is used without making a copy
}

public void Reset(ref IReadOnlyList<(TItem item, double weight)> items)
{
  // Use the given list directly, without making a copy of it.
}

The idea here is to implement an interface in line with these goals:

Ensure a ProbabilityTable structure can be easily constructed from whatever data the user has available (by supporting IEnumerable)
Ensure the user is not forced to copy their data unnecessarily
Ensure the user's list will be copied unless they explicitly specify that they do not want to copy (to avoid subtle bugs), while remaining as syntactically succinct as reasonably possible for a user.

The concept is based upon the following pieces of information:

List and arrays both implement IEnumerable, and IEnumerable supports virtually every use case involving a collection, between direct implementations and generators via yield return.
Arrays and lists both implement IReadOnlyList
Parameters marked with ref keyword must be explicitly marked as ref when they are passed (the user cannot "accidentally" pass something as ref).

In practice, the idea looks something like this:
public class UsersSuperComplexItem
{
    public string Name;
    public double Probability;
    // Lots of other fields
}

var complexExample = new List<UsersSuperComplexItem>() { /* Put a bunch of items here */ };
var simpleExample = new List<(string item, double weight)>() { /* Put a bunch of items here */ };

var probTable = new ProbabilityTable();
// I can do this, and the list will be copied; Because the IEnumerable overload is called
probTable.Reset(simpleExample);

// But I don't want to copy the list, I'd rather just use the one I have... Ok, I can do this.
// But i'll never do it accidentally, unless I somehow add the 'ref' keyword to the front
// accidentally?
probTable.Reset(ref simpleExample);

// Complex data structure isn't an issue either, I can even use LINQ, if I dare
probTable.Reset(complexExample.Select(i => (i.Name, i.Probability)));

One thing to note; although I've seen the ref keyword used for stuff like this in a few places, this is this is not really the "intended" use of the ref keyword, per-se.  What ref really means, is, "pass a non-const pointer".  In this case, it's being used on a IReadOnlyList (already a reference type); so really what we're getting is a pointer to a pointer here.  That much said, one can argue that this isn't really a problem, for a few reasons:

Resets don't happen very often relative to NextItems, more than likely, so the extra overhead of a pointer dereference as it's passed along is not at all likely to be noticeable
The compiler may (or may not) optimize out the extra pointer anyway, since it's just immediately copied.

Again, the reason it's used this way here is to require the user to explicitly state when they are expecting to use data as-is with no copy by placing the ref keyword before it.  If you don't prefer this method, it is also possible to accomplish something similar using a boolean value like was originally in the code; however because lists, for example, convert implicitly to both IReadOnlyList and IEnumerable, using such a method may result in it being somewhat ambiguous what overload is being called if you're not quite careful with how the interface is set up.  Functionally, however, either is possible and I can adapt the PR to either preference.
The other change, is that functions taking two separate lists, one of items and one of weights, now take IEnumerables instead of IReadOnlyLists.  This fits the paradigm for the other overloads, since in any case these functions must result in a copy of the data (which is precisely what all the IEnumerable overloads result in.
It is worthy of note that the act of iterating over IEnumerables compared to IReadOnlyLists may involve more runtime overhead/performance impact, however in this case I believe it should be minor, for mostly reasons mentioned above:

Resets may not actually occur that often
The Reset function itself does a fair amount of allocating memory (lists, arrays, etc); so since the Reset function will take non-trivial time, the overhead of IEnumerable should not be noticable here

Feel free to let me know what you think and if you want any changes.

You're absolutely right; IEnumerable is a better choice for the most general type to accept. Reset() also shouldn't be called that often, since it runs in linear time but does do quite a bit of allocation. The idea is for NextItem() to be the most-commonly-called method, since it runs in O(1) and should be the bulk of the work. Using ref wasn't an idea that would ever occur to me, but it does make sense here. I'll review a little more, but I'm almost certainly going to approve this.


disable pymode folding default

 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.


Fixing kerberos package dependency issue

A number of fixes, as itemized:

Change import paths for github.com/jcmturner/gokrb5/* -> gopkg.in/jcmturner/gokrb5.v1/*
Update Gopkg.lock accordingly
Adding line end to files missing them
Reorder import list on class_mappings.go
Fix gen-protobuf.sh script and permissions
Generate latest message/*.pb.go files

I'm humbling requesting this PR be merged in order to re-add this SQL driver to usql (https://github.com/xo/usql). I was forced to remove this driver completely as the package path for the kerberos package changed (now using gopkg.in), and caused this dependency to no longer be go get-able.
Additionally, I did some other minor cleanup and regenerated the protobufs based on the latest stable protoc/protoc-gen-go binaries.

@kenshaw Thank you so much for this PR! Just a small comment in the review and it's good to be merged! 👍

I changed the .gitignore as requested, and rebased/pushed the changes to the PR.

@kenshaw Thanks! Just waiting for the tests to turn green before merging.

Released as v2.2.0.


zsh theme looks broken after phpbrew install

Hey, just tried phpbrew first time today.
Can't see any reference to how to resolve this or open/closed issues but looks like phpbrew has obscured my zsh theme.
I'm using zsh, ohmyzsh & spaceship-prompt
Have you any suggestions on how to resolve?
Output

Expected Result

Platform
OS:
macos bigsur 11.3.1
Running PHP:
7.4.16
Let me know if you need anything else from me.

I only use the zsh on the macOS 11.6 Bigsur and it doesn't have the issue:
cloudxmac@cloudxmacs-Mac ~ % phpbrew switch php-7.0.33
cloudxmac@cloudxmacs-Mac ~ % php -v
PHP 7.0.33 (cli) (built: Dec  5 2022 16:02:01) ( NTS )
Copyright (c) 1997-2017 The PHP Group
Zend Engine v3.0.0, Copyright (c) 1998-2017 Zend Technologies
cloudxmac@cloudxmacs-Mac ~ % sw_vers
ProductName:    macOS
ProductVersion: 11.6
BuildVersion:   20G165

I've resolved this by explicitly disabling the phpbrew prompt
In ~/.zshrc
export PHPBREW_SET_PROMPT=0
[[ -e ~/.phpbrew/bashrc ]] && source ~/.phpbrew/bashrc

It seems that the spaceship-prompt has the issue about Zsh theme.
And the resolved issue is available in the previous reply.


Data saving

I just wana say that in docs I think it could be more practical in some cases to just save ammount of item user has

There might be array holding all items players has or dictionary holding item key and item ammount... or mayby even array holding array or dictionary that has item reference/id and count
Also is it safe to use this for experimental project? I mean project idea that is in alpha/prototype.
And please add question template for issues (for questions 😄 )...

I believe for that to be possible we need #54 and #95
Pandora is currently in its alpha stage, however, considering your project is also in the alpha/prototype phase, it might be a good fit to experiment with. I recently began a new project and opted to use Pandora. This choice has not only accelerated my development process but also allowed me to contribute reporting bugs.

@pietru2004 thanks for writing this up! For questions, please checkout our Github Discussions
It is actually not obvious to go there so I'll update the README!

ok thanks for info


build docker image and push to docker hub

build docker image and push to docker hub, instead of build on user's desktop

why closed issue without any comment?

I'd be interested to know this as well... It doesn't seem like too much effort to add a docker build & push github action...
@driesvints Would a PR for such a thing be welcomed?

Heya, sorry I forgot to reply. At the moment we're not considering this.

Although I (obviously) don't entirely agree, I can understand not wanting to add more code/work/load to the project.
Thanks for the reply! :+1:

Sorry, I typed the wrong Issue number in my comment on Issue 435. The right number would be 81. Not 91.


[SPARK-19460][SparkR]:Update dataset used in R documentation, examples to reduce warning noise and confusions

What changes were proposed in this pull request?
Replace iris dataset with Titanic or other dataset in example and document.
How was this patch tested?
Manual and existing test

Test build #73304 has started for PR 17032 at commit 1f57467.

Test build #73304 has finished for PR 17032 at commit 1f57467.

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

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

Merged build finished. Test PASSed.

cc @felixcheung

Test build #73373 has started for PR 17032 at commit 233ebec.

Test build #73375 has started for PR 17032 at commit 0c05309.

Test build #73373 has finished for PR 17032 at commit 233ebec.

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

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

Merged build finished. Test PASSed.

Test build #73375 has finished for PR 17032 at commit 0c05309.

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

Merged build finished. Test PASSed.

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

Test build #73440 has started for PR 17032 at commit 5beca69.

Test build #73440 has finished for PR 17032 at commit 5beca69.

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

Merged build finished. Test PASSed.

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

@felixcheung I have made the changes per our review discussion. Thanks!

Test build #73509 has started for PR 17032 at commit b0585aa.

Test build #73509 has finished for PR 17032 at commit b0585aa.

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

Merged build finished. Test PASSed.

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

Test build #73608 has started for PR 17032 at commit 905ffde.

Test build #73608 has finished for PR 17032 at commit 905ffde.

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

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

Merged build finished. Test PASSed.

merged to master.


New package: Kingsoft.KMeeting version 1.25.0

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[x] Does your manifest conform to the 1.4 schema?

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

Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/110402&drop=dogfoodAlpha

This package didn't install in a Windows 11 VM:

12:44:12 PM: 2023-06-22 12:44:11.862 [CLI ] ShellExecute installer failed:<PHONE_NUMBER>

Application Log:

Faulting application name: kmeeting_1.25.130.exe, version: <IP_ADDRESS>, time stamp: 0x5c157f86
Faulting module name: System.dll, version: <IP_ADDRESS>, time stamp: 0x5c157efa
Exception code: 0xc0000005
Fault offset: 0x00001581
Faulting process id: 0x0x2174
Faulting application start time: 0x0x1D9A541E9B08ABE
Faulting application path: C:\Users\User\AppData\Local\Temp\WinGet\Kingsoft.KMeeting.1.25.0\kmeeting_1.25.130.exe
Faulting module path: C:\Users\User\AppData\Local\Temp\nsc3337.tmp\System.dll
Report Id: 7800ec43-fa6d-43b9-bd38-df778e5f31a2
Faulting package full name:
Faulting package-relative application ID:

This package didn't install in a Windows 11 VM:

Looks like a bug from the installer. You may need to run the installer twice.


cmd/go: import fails with subgroups

What version of Go are you using (go version)?

$ go version
go version go1.15.7 windows/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
set GO111MODULE=on
set GOARCH=amd64
set GOBIN=
set GOCACHE=C:\Users\me\AppData\Local\go-build
set GOENV=C:\Users\me\AppData\Roaming\go\env
set GOEXE=.exe
set GOFLAGS=
set GOHOSTARCH=amd64
set GOHOSTOS=windows
set GOINSECURE=
set GOMODCACHE=C:\Users\me\AppData\Local\go\pkg\mod
set GONOPROXY=
set GONOSUMDB=
set GOOS=windows
set GOPATH=C:\Users\me\AppData\Local\go
set GOPRIVATE=https://git.mycompany.com
set GOPROXY=https://proxy.golang.org,direct
set GOROOT=C:\Users\me\AppData\Local\go\go1.15.7
set GOSUMDB=sum.golang.org
set GOTMPDIR=
set GOTOOLDIR=C:\Users\me\AppData\Local\go\go1.15.7\pkg\tool\windows_amd64
set GCCGO=gccgo
set AR=ar
set CC=gcc
set CXX=g++
set CGO_ENABLED=1
set GOMOD=C:\Users\me\Documents\Code\me\_go\gologger\go.mod
set CGO_CFLAGS=-g -O2
set CGO_CPPFLAGS=
set CGO_CXXFLAGS=-g -O2
set CGO_FFLAGS=-g -O2
set CGO_LDFLAGS=-g -O2
set PKG_CONFIG=pkg-config
set GOGCCFLAGS=-m64 -mthreads -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=C:\Users\me\AppData\Local\Temp\go-build675008778=/tmp/go-build -gno-record-gcc-switches

What did you do?
Using a private GitLab server with subgroups, the go get command fails to find the appropriate root of the repository and attempts to download, giving exit status 128

What did you expect to see?
I expected it to download the module
What did you see instead?
go get -u -v
go: finding module for package git.mycompany.com/utilities/go/message-brokering/mqtt
get "git.mycompany.com/utilities/go": found meta tag get.metaImport{Prefix:"git.mycompany.com/utilities/go", VCS:"git", RepoRoot:"https://git.mycompany.com/utilities/go.git"} at //git.mycompany.com/utilities/go?go-get=1
get "git.mycompany.com/utilities/go/message-brokering": found meta tag get.metaImport{Prefix:"git.mycompany.com/utilities/go", VCS:"git", RepoRoot:"https://git.mycompany.com/utilities/go.git"} at //git.mycompany.com/utilities/go/message-brokering?go-get=1
get "git.mycompany.com/utilities/go/message-brokering/mqtt": found meta tag get.metaImport{Prefix:"git.mycompany.com/utilities/go", VCS:"git", RepoRoot:"https://git.mycompany.com/utilities/go.git"} at //git.mycompany.com/utilities/go/message-brokering/mqtt?go-get=1
get "git.mycompany.com/utilities/go/message-brokering/mqtt": verifying non-authoritative meta tag
get "git.mycompany.com/utilities/go/message-brokering": verifying non-authoritative meta tag
go: finding module for package git.mycompany.com/utilities/go/message-brokering/mqtt
gologger.go:8:2: module git.mycompany.com/utilities/go/message-brokering/mqtt: git ls-remote -q origin in
C:\Users\me\AppData\Local\go\pkg\mod\cache\vcs\05a6835aeda72f1a69ded8a71bbefb98908e5c080bafc868a0a1ff204278f2f5: exit
status 128:
        fatal: repository 'https://git.mycompany.com/utilities/go.git/' not found

Worth noting that I found a thread that appeared to show this issue as resolved (https://github.com/golang/go/issues/34094), but I am definitely still experiencing this. I even tried rolling back to 1.13.8 and it was still present. Also of note, I began on gitlab-ee 12.9.2, upgraded to gitlab-ee 13.8.1.

Is utilities a group, utilities/go a subgroup and utilities/go/message-brokering a repo?
Has this worked before for you?
 I have not tried using netrc, but I've never had auth issues. git.mycompany.com/user/project and git.mycompany.com/group/project have both always worked.

please try a netrc and this sounds more like a gitlab issue than ago one (see https://gitlab.com/gitlab-org/gitlab/-/issues/5253)

Indeed seemed to be a gitlab issue. Also of note Windows requires a _netrc instead of .netrc. That had me chasing my tail.


Fix #38 Lowers deployment target for Carthage

Fix #38
¯\_(ツ)_/¯
Title says it all

@serluca there's a branch bug/Carthage with the fix.
Have not being able to merge because Travis is having some issues ¯\_(ツ)_/¯
I'll merge once someone @ travis replies to my tweet

Hi @esttorhe ok great thanks, hope you fill find the solution for that

Hi @esttorhe did you manage to find the solution for this?


Alternate to Delete key (as Mac has no such key)

This needs testing, and the Guide to using the Editor needs updating

#217 covers the testing


New: Add /team page

I've added a /team page an updated the "About" link in the heading to be a dropdown that has "About ESLint" and "About the Team". The grid is responsive, going down to just two columns on mobile and up to four columns on larger screens.

Just a note about the data: I wrote a utility to create _data/team.json directly from the GitHub API. We can just run that as part of the build to update it automatically. It pulls from the GitHub team data.
And no, I didn't edit the data to put me first. That's the way it came out of GitHub. :)

re. eslintbot, it would be simpler to remove it from eslint-team. I've just done that now, so if the list is regenerated the bot will no longer appear.
It needs admin access to repositories in order to push releases, but it should already have that due to being an organization owner.

As a sidenote, I've was looking into whether it would be possible to get rid of the @eslintbot account entirely and only use the eslint[bot] GitHub app for releases. It seems like it's currently not possible because the bot needs to push release commits to a protected branch while overriding the requirement for PR approvals/status checks. This can only currently be done by real accounts (not GitHub apps) with admin access to the repo. (See here for more info.)

I re-ran the utility and updated team.json. I'll work to get that script integrated into the gensite build on the main repo.


Change: Multidex enabled

Pull request for issue #108

Thanks for this @barykaed !

yup multi-dex has become an issue. I actually had this included in another feature branch, but haven't gotten around to merging it yet.
I could trim the library/method count inclusion as there's really no good reason for this repo to exceed the method count but given that the objective of this repo is to be a playground of sorts,I want to include as many variations of the demo as possible.


add the domain itoaxaca

This domain belongs to the Instituto Tecnologico de Oaxaca

@ravenandhammer Please rename  lib/domains/mx/edu/itoaxaca to  lib/domains/mx/edu/itoaxaca.txt to reflect this repository rules. Thank you!

Sorry, my bad, i forgot the extension

@ravenandhammer Pull request merged. Thank you!


Allow 'Developer: Generate Color Theme From Current Settings' > 'workbench.action.generateColorTheme' from within extensions

Extension authoring doesn't expose many texteditor display behaviors (ie rows vs columns and so on).
In order to implement custom text editor-like interactions, authors must implement behaviors in a webview panel, and to preserve the feeling of an in-app editor experience, its reasonable to use an instance of the monaco editor.
Look and feel become an issue, as there is no way to ensure seamless style without painstakingly enumerating all the --vscode-* css vars and translating those styles through to the embedded monaco editor within the web-panel.
A user can run the Developer: Generate Color Theme From Current Settings command, which spits out all the requisite styles needed; however, when running that command from within the context of an extension, programmatically, the result is null/undefined.
If this command worked, those styles could be translated more easily.

Closing as we have no plans to change the Developer: Generate Color Theme From Current Settings command to be used programmatically

What of the color API you mention? Is that on roadmap, or is there anything like it that will allow extension authors to get active colors from the editor and recreate them in an embedded monaco editor?

I don't know what's the recommended way to embed a Monaco editor in a web view (or whether this is a good idea at all). Maybe @mjbvz or can give some insights.
If enumerating all colors is a needed part of it, we can investigate into a new API like getThemeColors(). We don't have such an API yet (and also no request for it), wo we would need an issue for it.

That's what my feature request is asking for. I assumed since the 'generate color theme` command was already available, making it available programmatically would be the lowest hanging fruit, but ultimately, I'd like to be able to ask vscode what its current theme is, or more specifically, what the texteditor colors are.


bug: Embeddings Incomplete with no way to fix it

Version
1.0.0
Describe the bug
Embeddings progress is stuck at 19% (hasn't changed in hours).  The error tooltip says click to resolve but clicking does nothing.

Expected behavior
Clicking the "Embeddings Incomplete" button in status bar would open a new tab to walk me through the process of re-generating embeddings index (or some other process that would  enable me to resolve the issue)
Additional context
No response

I'm also experiencing this. Has this been fixed?


Update AcmeClient.xml

Support for GratisDNS.dk

@ricobach Merged, thanks! In the future please create one pull request, this will make testing and review much easier.


Fix decimal equivalence with trailing zeroes

In the Equivalent logical specification, it mentions that "[f]or decimals, equivalent means the values are the same with the comparison done on values rounded to the precision of the least precise operand; trailing zeroes after the decimal are ignored in determining precision for equivalent comparison."  Would the correct reading of this be that, for 1.001 ~ 1.000:

The trailing zeroes of 1.000 are ignored in determining precision, so it is treated as 1 with precision 0
1.001 has three digits of precision and 1 has 0, so the comparison is done with the values rounded to the lower precision (0)
Truncating 1.001 to precision 0 yields 1, and 1 ~ 1 == true, so the expression evaluates to true

If so, I added this as a test case and have a proposed fix in the EquivalentEvaluator.  Otherwise, please let me know if I'm reading this the wrong way or misunderstanding!

Agree with the interpretation and the test case here, with the quibble that the description of the interpretation uses the word "truncate", but the specification is clear that values should be rounded to the least precision, not truncated.


Is there a plan to support Firefox?

What Chrome-only APIs are you using? https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/Browser_support_for_JavaScript_APIs

Hey @Hatko and @andreicristianpetcu - thanks for the question!
I'd love to support Firefox, but there are quite a few things in my queue ahead of this - a pull request would be quite welcome!
Cheers,
-Steven

@skoczen I don't think a PR is needed. You are not using any Chrome-specific APIs.
I just loaded it in Firefox and it works with no errors.

What is needed is a test that all the features work correctly in Firefox and a zip of the extension uploaded at https://addons.mozilla.org/en-US/developers/addon/submit/distribution


Support multiple entities form / questionnaire

Are you requesting a feature, reporting a bug or ask a question?
feature / question
What is the current behavior?
We would like to support multiple entities questionnaire forms.
In our situation, it's a medical form that the questionnaire form should display for each of the entities.
we discussing if the solution should be UI based only or be part of surveryjs solution API
for example, if there was an API that we can ask the same question for 1..N entities and upon survey submission, we will get the data for each entity.
also - it could be much more complex because each entity can get a different survey path as each entity answer can result in a different UI based on visibleIf condition.
Any idea/comment will be more than helpful.
best

@IDIBuild Do you mean you have the same survey, but you have only one additional invisible value, the entity? Then you may simply set survey.setValue("entityName", entitity1"); and you will get it in the final json data. Additionally you may use entityName in visibleIf property. You may set visibleIf on question/panel/page level, in case you want to hide/show several questions at one time.
Sorry, if I do not get you. Then please explain your scenario in another way.
Thank you,
Andrew

Hi @andrewtelnov
What I mean is that in our business scenario we got two major requests:
let's take a health insurance for example:
relevant questionnaire form may ask about more than one entity - for example - a family of
father + mother + child.
each entity in the family need to answer the questionnaire and we want to be able to fill the answers on the same survey and not fill separately.
hoping it's more clear this time :-)

@IDIBuild Yes, I have got it now.
Again, there are other questions. How are you going to store the data? If it is a JSON and the format something like this:
var entitiesData  = {
   "FamilyName": "Millers",
   "EntityCount" : "3",
   "EntitiesInfo" : [
      {"Name" : "Mark",
      "FullAge": "43",
      ....},  .....
    ]
}

In this case you may run N surveys. Create the JSON object and on completing every survey, push the survey.data (survey JSON result) into entitiesData.EntitiesInfo.
You have to use survey.onComplete event.
On completing the N-th survey, you may save entitiesData on your server.
The initial data entry, family name and information about person, could be done by using SurveyJS as well. On completing it, you may start N-th survey and add EntitiesInfo array into the initial/mail survey data.
Let me know if you need more help.
Thank you,
Andrew

@IDIBuild I have created a plunker example: https://plnkr.co/edit/n9FRZC?p=preview
It should do exactly what you are trying to do.
Thank you,
Andrew

@IDIBuild just want to followup. Does the example help or I did not get you correctly?
Thank you,
Andrew

Hi @andrewtelnov.
sorry for not responding.  Saw the plnkr solution and very cool.
My case's somehow different - let me try to explain.
I will define my entities as a predefined condition - for example, father + mother.
the question is: "Do you use any medical drugs?"
the layout should display this question for each entity separately.
and now for the more complex stuff - if the answer to the following  question is true there will be a follow-up question - let's say "Do you use a cholesterol reduce drug?" but the question should be displayed only for the father that says Yes to the previous question and not to the mother that answered No.
what can cause a different survey tree path and can be challenging at the UI level and the survey.js engine
what do u think? do I make any sense?
best

@IDIBuild I have added a new column into the matrix and new question into the entity survey with visibleIf criteria: https://plnkr.co/edit/n9FRZC?p=preview
We are currently working on our website redesign and we are looking for real and complex surveys. We feel that our platform is already good enough, but since it is open source, new and texts are written by non-native speakers, there is not enough trust in it.
I would be happy to create the full survey for you, in case you allow us to publish it on our new web site. You may remove some questions or  change the design/text before publishing.
Additionally, I see that we may do some new features into the library. Your survey will look and work perfectly and our library becomes better.
For example:

the code on mainSurvey.onMatrixRowAdded event can be removed if we add some options into matrix question/matrix column objects.
Right now I use a radiogroup for "Yes", "No". I may use the checkbox, but in this case the data will be stored as ["value"] and not as "Yes" or "No". An option to checkbox question is needed.

Do you have any interest in this? I would be happy to work together.
Thank you,
Andrew

Hi, Andrew.
we will be more than welcome to discuss it and display a demo.
took a simple document that describes a common form used by the insurance industry.
I will try to breakdown the main challenges:
the main challenge here is how to display a usable UI Survey that asks about more than one entity (in this case its' a family) and doesn't separate the UI to multiple instances of the survey.
the main issue here is that the customer will be able to share the same survey for all of the family and in some cases, we will be able to ask some more details (follow-up questions) about a specific family member.
as for the business point of view, we will need a different survey data structure for each of the entities so we can process it for each of the members.

see Family Members columns (page 3) - how can we supply a predefined list to be answered separately by each member.

see Systems Review table (page 4) - how can we handle a form questions based on a common category such as Hem/ID/Lymph? we need to display this questions as collapse that can be open/closed.

the main challenge here - how can we display a follow-up question?
a) in some cases it will be simple as you showed in the demo that in the next step we will ask a specific entity about his condition and then return to the main flow.
b) but what about a more complex scenario that the follow-up step is more than one question or even a sub-survey.
we thought about displaying  the survey as a flow that will help the user to understand that this part
or even be able to say: I will complete this part later  and in this case, at the end of the survey we will display the user with an option to continue where he's left

c) this brings me to the part of continuing where I left - when the user needs to fill a complex sometimes the user will want to stop and continue later - we also looking for a solution to rerun the survey from the part will left.

Hi @andrewtelnov
first of all and as I didn't manage to say it before - your project is amazing and didn't saw such advanced and friendly solution.
If I may I wanted to ask more technical questions:

I saw the feature/ng branch and it's will be really helpful to us if it will be integrated as part of the master as there are still allot of AngularJS developers.

in our case with have been created dedicated UI components framework based on Angular 1 and we want to use the same components as part of the survey framework and the same for the styling system - we using angular material 1.x.
is it going to be possible? can we use all the features with our custom widgets?

reduce the footprint of the framework - as we are planning to use our UI+Style frameworks, is there any way in the build process to remove default widgets and styling system in order to reduce the size of the target artifact?

Best,

@eladh Thank you for pdf document. I have now a better understanding what you are trying to do.
Let's make this survey. I would suggest to create several small parts in plunker first. So all issues with workflow and UI (I am talking about widgets and layout, not design) will be solved and cleared up. After that we may create the full survey and at the last stage apply the design/css.
We may do some experience with design/css in plunker as well in parallel.

Family Members. I will do a demo in plunker that does what you need. I understand that it should look as one survey. I will propose my UI and we will discuss it.
System Review. There are two solutions:

We may have a radio group question that ask: Do you fill the System Review? ["Yes" "No"] and a page or panel that is shown/hidden based on the answer
I may add a new property to the Panel object, expandState: string ('none", "expanded", "collapsed"). By default it is "none" and content of the Panel (questions and other Panels) are shown. Expanded - similar to "none" but a " - " button will be rendered before the title. On pressing it the Panel will be collapsed and only title is shown. Collapsed - opposite to expanded. If you think it is a better way, we implement it. I have just added it in our TODO list, since I think it is cool and we will do it sooner or later.

I need real examples to find out the solution. visibleIf or enableIf (this property will be in the next minor) properties may handle complex Boolean expressions like:
{a} != 'aaa' or {b} contains 2 and ({a} = "ccc" or {c} < 3)
It is not a problem at all. This question has been asked not the first time. All you need to store current page index, additionally to the data. On restarting the survey, apply the data:  survey.data = savedDataasJson; and survey page index: survey.currentPageNo = savedPageNoAsNumber. I would store the data in the system on every "next" page and additionally on timer, every x minutes.

Technical questions:

Angular 1.x. To be honest we don't want to work on Angular 1.x by ourselves. We have already Angular 2+ support. A company has asked us to help them with Angular 1.x. They do not want to use jQuery or Knockout versions. We have agreed to help them and created the branch so they may start. We promised to provide them any kind of help. They agreed to make it as Open Source project. We have created the branch, they said thank you and disappeared :-).
The current Angular 2 is based on react version + preact. You may use the same approach. It is probably the easiest way.
Yes, custom widgets are possible. We did not try it with Angular 1.x widgets. The widgets that work with DOM directly are easy. We need to see what you are trying to do.
We are using webpack: https://github.com/surveyjs/surveyjs/tree/master/src/entries. Questions and localization are done as plug-ins. It should not be a problem to remove or replace the most of them. I think it is doable.

I will be waiting for your comments. I will make a plunker demo for "Family Members", hopefully today.
Thank you,
Andrew

hi @andrewtelnov
About point 2 – adding new attribute seems to be the best flexible solution.
About point 3 – What I was trying to say, in a more specific way: let’s say that the father answered question “Do you have a heart condition” – Yes, and the mother answered – No
our logic decided that in this case a Panel (inner survey) need to be displayed.
This brings complex UI Decisions to make:
•	Do we saying to the user that we starting a new sub/inner survey about his heart condition?
•	A more explanatory flow navigation bar may need?
•	A complex survey can cause the user to ABEND the survey – a “Continue later” flow can be very helpful in this cases.
Do you think that in this cases (sub/inner survey) we will need to use the enableIf?
About point 4– it may be more complex to resolve – but let’s discuss it only if you think that the “Continue Later” makes sense in your end.

@eladh I have created a new example: https://plnkr.co/edit/j5DKTy?p=preview. It is about (1)
(2) - ok, we will do it. I think we will implement it in 5-10 days.
(3) It is not a problem. You may always ask do you want to answer these questions or continue (by pass them). If he says continue, just make those questions/panels/pages invisible by using visibleIf property. At the end of the survey, you may ask him about entering those question again, if he doesn't answered them yet.
Thank you,
Andrew

Hi @andrewtelnov
see the example: Family History (page 3), is there a possibility to display question matrix with Family Members columns?
as I stated in my last comment, in a case of an inner survey (as a follow-up procedure):
a) is there API to display some kind of a flow header that will display the current stage in the survey?
we are planning to allow our Super-Users to edit/modify the survey.
in this case supporting capabilities such as "Continue later" or supporting "Follow-up" questions/survey
without support from the framework.
we thinking to display a tree based editor that will help the users to understand the flow of the survey/
also in this case, if it makes sense I will be glad to supply more information
best.

@eladh Yes I will modify the display question matrix and will add the Family Members columns. I will do it today.
I think, I may modify the first matrix as well, so there will be only one survey and subsurvey will go away. Do you need this?
And I do not quite understand about the flow API. Do you mean show hide the progress bar or do you want to have an event to control the progress bar (% + page count and current page)?
Thank you,
Andrew

@eladh I will add visibleIf property into column first. It will allow to have a matrix as:
Relative | Name | Is Alive? | Age | Deceased Age | Cause of Death |
I will work on it right now. I remember several people have been asked about it as well. I did not completely understand the scenario.
After that I will make an example with Family Members as columns in the matrix.
Thank you,
Andrew

@eladh column.visibleIf property issue is here: https://github.com/surveyjs/surveyjs/issues/515

hi @andrewtelnov .
thanks for the time and effort its quite amazing :-)
a. when i define a Panel Entity (with sub-survey) it would be great if will be flow indication in the flow toolbar. some kind of a breadcrumbs. if i understood you expose a progress.html template that can only display progress but is there any possibility to display flow.html that will represent the header as flow.
b. the survey will be maintained by "Super Users" that will make changes such as  questions/answers/visibleIf....
c. in this case how can i support generic "Entity based flows" & "Continue Later" in a more generic way.
i would love to here your thoughts about it.
best,
elad.

@eladh I have implemented the column.visibleIf and here is the first matrix now (all custom code was gone): https://plnkr.co/edit/SYJBdx7AkjqOnXXrqTmV?p=preview
Please note, it uses the dev version.
However, before we go further, I just got another idea. I think it will be easy enough to add a feature as repeatable/dynamic Panel.
Right now Panel is just a container and nothing more. However, I believe I may create a dynamic Panel, that in fact will work exactly like matrix dynamic. Panels instead of Rows and  Questions instead of Columns.
So for every Family Member you will have the Panel, with questions similar to the columns in the current example + illness matrix. It will be done naturally, without any custom code.
What do you think? I guess it would solve the issue completely and will not need custom progress.
Thank you,
Andrew

hi @andrewtelnov.
i will try to explain - we have two modes of entities:

the first is Matrix based - Questions * Number of entities
in this case we don't see any usage of display the questions as the rows and the entities as columns.
the visibleIf at the column level will be great help to display only relevant entities.
in this scenario - we will not support a follow up questions in the same Matrix Question.

the second is Follow-Up questions based  - very similar to what you have done in this Plunker  after user submitted data we may decide that a follow up survey should be presented , and if I'm right survey.js Panel with VisibleIf condition will do the job , I'm i wrong ?

also - because "Follow-up" Questions may cause more complicated flow i ask for :
a) generic "Continue Later" - as explained before
b) some more advanced survey toolbar/header  that can display the current state.
my last issue was about how can we integrate "Continue Later" + "Follow Up questions" as part of the framework (Editor/JSON)
for example - how can i define the following : for each on the entities i would like to display a "Follow up questions" Panel if condition X is true.
for example - mark some part of the Questions as "Continue Later" without custom based VisibleIf complex rules.
best,
elad.

@eladh You want to have one Panel for one entity. At the current moment it will require to write a lot of tricky code.
I do not know about your deadline, but if you have a week, then I'm pretty sure, we will be able to create a new question type: QuestionDynamic. This question will have a template panel - actually the typical panel you have it now.
In the template panel you will have a radio group question and another nested panel.
A radio group is a question: Do you want to feel information about {relative} {name}? - Yes/No
If yes, the nested panel with all required questions about the Family Member will be shown.
The template panel will be generated for every Family Member.
In this case, an user may do not fill these information initially and do it later or do it for any Family Member(s) he wants by skipping the first x.
Does it make sense?
Thank you,
Andrew

hi @andrewtelnov
we have enough time - we only on the design stage.
as for the main issue - your idea seems great and seems that is solving the issue in a generic way.

what about the "Continue Next" option - do u see it as a framework capability so we can mark elements in the survey with this attribute without the need to write custom VisibleIf rule?

make the toolbar/header API more reach - so we can indicate certain events such as an inner panel or QuestionDynamic?

best,
Elad.

@eladh
(1) I think you will not need "Continue Next". Family Members question (panel dynamic) will be on separate page. By default for every family member only one radio group will be shown. If an end user press 'Yes' on this question then the panel with all questions for this family member will be shown. We will introduce expand/collapse feature for the panel, then an end-user may always collapse/expand the internal/nested panel with a lot of questions and keep the page not too long.
(2) Lets create the first version of QuestionDynamic and then we will what is missed here and add the required functionality.
Thank you,
Andrew

@eladh  Here is the link to the PanelDynamic developement: https://github.com/surveyjs/surveyjs/issues/525
You may add your thoughts about PanelDynamic there.
Thank you,
Andrew

@eladh Here is the prototype with QuestionPanelDynamic: https://plnkr.co/edit/aMcSnl?p=preview
I am going to work on design time right now for this question type.
There are several open design issues that has to be solved. We may discuss them:

Where to place the "Remove" button
Right now all questions inside QuestionPanelDynamic are treated as one question. I think it is better to change and number them as they were not inside the QuestionPanelDynamic.

There are some further improvements that I want to add:

"Do you want to enter the detail information?" Right now it is a radio group question. I want to make it as QuestionBoolean. The check will be next to the title and you may do no have a question number
age dropdown choices property look weird. I want to add a property with nested properties "startNumber", "endNumber" and step (1) by default. So you may define only 2 properties instead of a lot of choices items.

Thank you,
Andrew

@eladh I have updated the example: https://plnkr.co/edit/aMcSnl?p=preview
There are two new properties now:
renderMode: string, the default is list. In the example it equals to: progressTop (you can use progressBottom and progressTopBottom)
showQuestionNumbers: string, in this example it equals to "onPanel", but default it is "off". You may use "onSurvey" as well.
I think, I am mostly done with QuestionPanelDynamic. Could you please review the example and tell what is missed here?
Thank you,
Andrew

thanks .
will do

Hi @Andrew
i checked the functionality and its fit great into our scenario.


[Twitter] Handle multiple accounts?

Extension
https://github.com/raycast/extensions/tree/34c65c7f190ad172e46c75384600aa938f932e98/extensions/twitter/
Description
I'd like to be able to handle different accounts via this extension
Who will benefit from this feature?
Everybody who uses more than one twitter account.
Anything else?
No response

😬

Related to #269

ha that's great! Thanks!

👋🏻


ARROW-5654: [C++][Python] Add ChunkedArray::Validate method that checks chunk types for consistency, invoke in Python

This prevents users from constructing ChunkedArray with different data types

Codecov Report

Merging #4636 into master will increase coverage by 0.43%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #4636      +/-   ##
==========================================
+ Coverage   88.49%   88.92%   +0.43%
==========================================
  Files         890      710     -180
  Lines      110406    96710   -13696
  Branches     1418        0    -1418
==========================================
- Hits        97703    86000   -11703
+ Misses      12422    10710    -1712
+ Partials      281        0     -281

Impacted Files
Coverage Δ

cpp/src/arrow/table.h
100% <ø> (ø)
:arrow_up:

python/pyarrow/tests/test_table.py
99.62% <100%> (ø)
:arrow_up:

python/pyarrow/table.pxi
87.55% <100%> (+0.05%)
:arrow_up:

cpp/src/arrow/table.cc
97.03% <100%> (+0.07%)
:arrow_up:

cpp/src/arrow/table-test.cc
100% <100%> (ø)
:arrow_up:

go/arrow/ipc/writer.go

go/arrow/math/uint64_amd64.go

go/arrow/memory/memory_avx2_amd64.go

go/arrow/ipc/file_reader.go

js/src/builder/index.ts

... and 180 more

Continue to review full report at Codecov.

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

Yeah, that's something to be addressed separately.


Change welcome copy

capitalize "Member Portal"
keep "Interact Member Portal" words together (can either use   or white-space: nowrap; span

Before:

After:


Fix some installs and imports

Several fixes regarding old version of torchtext, some spaCy fixes and proper pip install strings for Google Colab

Hw01_NMT: why decrease N_EPOCHS in baseline from 10 to 3?

Fixed. And also added random_state to torch split because it caused some "reproducibility issues" with different vocab sizes.


How to change the ajax request url to render another datas?

I don't know if can do this with the docs and examples, is there a way to do this?
Thanks very much!

Not sure, I got the problem.
You can specify ajax url doing request to the server in controller methods
http://js-grid.com/docs/#grid-controller
Could you provide an example or more details about your case?

Thanks, this is the code
$("#xxx").jsGrid({
    width: "100%",
    paging: false,
    autoload: true,
    controller: {
        loadData: function() {
            var d = $.Deferred();
            $.ajax({
                url: "/a_list",
                dataType: "json"
            }).done(function(response) {
                d.resolve(response.data);
            });
            return d.promise();
        }
    },
    fields: [...]
});

After this grid loaded, I want change the ajax url to '/a_list?type=2', and render data to the same $('#xxx') element, is this possible?

Yes, it's possible, you can introduce url variable and change it on ajax done.
You can put this variable into the closure of self calling function like following:
loadData: (function() {
	var url = "/a_list";

	return function() {
		var d = $.Deferred();
		$.ajax({
			url: url,
			dataType: "json"
		}).done(function(response) {
			url = "/a_list?type=2";
			d.resolve(response.data);
		});
		return d.promise();
	}
})()

Not sure, what you try to achieve, but want to be sure you aware that loadData accepts pageSize and pageIndex parameters if pageLoading = true
See docs http://js-grid.com/docs/#loaddatafilter-promisedataresult

What I want to do is a filtering, different datas is loaded by different url, like '/a_list' and '/a_list?type=2', there is a select element in my page, and by different options, I want change the ajax url to get datas.

Ok, then define url outside the grid, change it on change event of your select, and call loadData method.
See following example (actual loading is not happen due to fake urls): http://jsfiddle.net/tabalinas/tjoag1ux/
App = {};

App.loadUrl = "/a_list";

$("#jsgrid").jsGrid({
    autoload: true,
    width: 200,
    controller: {
        loadData: function() {
            return $.ajax({
                type: "GET",
                url: App.loadUrl
            });
        }
    },
    fields: [
        { type: "text", name: "Name" }
    ]
});

$("#urlSelect").on("change", function(e) {
    App.loadUrl = $(e.target).val();
    $("#jsgrid").jsGrid("loadData");
});

Yes, this is exactly what I want. And thanks for this amazing grid plugin.

Thank you for the feedback, I appreaciate it.


Add support for regnet and stock and flow

This PR updates the mira_model context and the mira_config_edit contexts to support loading, editing and saving regnets and stock and flows. In both contexts the strategy is to store the schema_name on the context (self.schema_name) then pass this information throughout the context, agent and procedures to ensure that the frameworks are handled appropriately.

Notably, the stratification prompt had to be updated to ensure that if the stratification is to be performed on a regnet the structure argument is specified to be an empty array [] per Ben's guidance. See screenshot below for confirmation that this is now handled properly

This PR also includes updates to interact with HMI Server (not TDS) for the contexts referenced as well as for datasets and decapodes (untested).

I will update this to handle regnets using the new export functionality in https://github.com/gyorilab/mira/pull/276

This PR is updated to include the latest Mira support for regnet to JSON conversion.

This PR has been rebased to main and now has a replacement for DATA_SERVICE_URL with HMI_SERVER_URL more or less throughout the repo.


Update Update-SPHelp.md

Filled in placeholder text. Removed SharePoint 2010 applicability. Notated that this cmdlet does not currently function.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-server/Update-SPHelp.md
:white_check_mark:Succeeded
View (sharepoint-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


backup using cloud

step:-User navigates to wallet setting
step:-choose option backup via cloud.
User selects app-generated passwords and decides to get back and proceeds with creating their passphrase. but it brings the user where he has to select backup using cloud again and proceed further.
https://user-images.githubusercontent.com/107177634/183651983-0a97d38a-03b6-4abb-a27c-092b73c37e22.mp4

Completed


DO NOT MERGE: Only exists for sharing inline notes

Some inline notes I'm talking while reviewing the codebase for the refactor.

Intended to be draft PR


Fix race condition in my_init causing container never to exit

The following race condition existed, and happened to me in production:
root@182b305fa0d5:~# /sbin/my_init -- ls
*** Running /etc/my_init.d/00_environment.sh...
*** Running /etc/my_init.d/00_regen_ssh_host_keys.sh...
*** Running /etc/my_init.d/01_bootstrap_djbdns.sh...
*** Booting runit daemon...
*** Runit started as PID 26
*** Running ls...
*** ls exited with status 0.
warning: /etc/service/cron: unable to open supervise/ok: file does not exist
warning: /etc/service/dnscache: unable to open supervise/ok: file does not exist
warning: /etc/service/sshd: unable to open supervise/ok: file does not exist
warning: /etc/service/syslog-forwarder: unable to open supervise/ok: file does not exist
warning: /etc/service/syslog-ng: unable to open supervise/ok: file does not exist
*** Shutting down runit daemon (PID 26)...
At that point the process hangs forever, because I believe the following has happened:
my_init starts runit
main command exits
my_init tells runit to exit
my_init tells runit services to exit (nothing happens since they aren't running yet)
runit starts its services
runit exits
my_init checks to see if runit services are still running, and they are, so repeat forever
This patch solves this issue in my production environment.

Thanks for submitting this. I've merged your pull request with some changes.


Impersonation Handler

Overview
Bans people from changing their names to the names of the moderators.

also maybe check for people joining with the same name as a mod?

Is there work to be done on this? I haven't seen much communication on this feature in a while.


Time picker Error!

I have error when i use timePicker
An uncaught Exception occurred on "main" thread.
com.tns.NativeScriptException:
Calling js method onTouch failed

TypeError: Cannot read property 'is24HourView' of undefined
File: "file:///data/data/org.nativescript.mads/files/app/tns_modules/tns-core-modules/ui/gestures/gestures.js, line: 229, column: 40

StackTrace:
	Frame: function:'ModalDatetimepicker.pickTime', file:'file:///data/data/org.nativescript.mads/files/app/tns_modules/nativescript-modal-datetimepicker/modal-datetimepicker.js', line: 48, column: 39
	Frame: function:'exports.selectStartTime', file:'file:///data/data/org.nativescript.mads/files/app/views/sales/addplacement.js', line: 129, column: 11
	Frame: function:'_executeCallback', file:'file:///data/data/org.nativescript.mads/files/app/tns_modules/tns-core-modules/ui/gestures/gestures.js', line: 283, column: 27
	Frame: function:'TapAndDoubleTapGestureListenerImpl.onSingleTapUp', file:'file:///data/data/org.nativescript.mads/files/app/tns_modules/tns-core-modules/ui/gestures/gestures.js', `l

Hi @dyazincahya
Simply pass an empty object into
Like this
picker.pickTime({})
It shouldn't crash after that.
This is a known bug. It will be fixed with next release. :)

to add on to @shiv19 answer, when using TypeScript
            picker.pickTime({                 is24HourView: true             })
you can set it to false if you so wish


Duplicate content

Contributions

[X] I have read the contribution docs, and understand what's expected of me.

Link to the page on docs.getdbt.com requiring updates
https://docs.getdbt.com/faqs/Snapshots/snapshot-schema-changes
What part(s) of the page would you like to see updated?
https://docs.getdbt.com/faqs/Snapshots/snapshot-schema-changes is just a copy of information on https://docs.getdbt.com/docs/build/snapshots.
https://docs.getdbt.com/docs/build/snapshots dp doesn't event link back to any other Snapshot page. I would think you'd want to permanently redirect this page to the longer snapshot page to avoid the duplication.
Additional information
No response

Thanks, @alison985. One is an FAQ page. These sometimes have the same information as the articles, but we're enhancing and moving them soon, so this will no longer be an issue. We appreciate it!


Add configurable auto-checksum

Allow configuring the automatic checksum added to uploads, and test multipart uploads with all hash types.
Adds test for https://github.com/minio/minio/pull/20231 as well.

{"time":"2024-08-09T12:54:54.983665595Z","level":"ERROR","name":"minio-go: testPutMultipartObjectWithChecksums","duration":15136,"function":"PutObject(bucketName, objectName, reader,size, opts)","args":{"bucketName":"minio-go-test-kmscf5rwpbe4z5za","checksum":"SHA1","objectName":"zifzcffvyejqlcc3ix5rsqd6qg99et","opts":"minio.PutObjectOptions{UserMetadata: metadata, Progress: progress}"},"status":"FAIL","message":"checksum mismatch","error":"want FFOs4OZ97Ix0EXzmfYYXPTwbcLY=, got "}
panic: want FFOs4OZ97Ix0EXzmfYYXPTwbcLY=, got

We will need a new release that includes https://github.com/minio/minio/pull/20231

New release passes.


Delete specific episodes but no unmonitor series?

Looking into this and it seemed promising, but I am not sure if it has the feature I am looking for?
I would like to delete specific episodes of a show 7 days after they have aired, but still monitor the show and future episodes. Is this possible? I got it to delete the episodes but now Sonarr has it as unmonitored so it won't get future episodes.
Thanks

Are you using a media type of Episodes with an action of Unmonitor and delete episode? That should only be unmonitoring and deleting the episodes that match the rule.

Thank you I knew I had to be missing something. I had it on TV instead of episode, I have changed it and I believe it to work like I was hoping


Call to uuid() returns base64 encoded Java object name

OrientDB Version, operating system, or hardware.
v2.2.15
Operating System
Linux
It seems that a call to uuid() returns a base64 encoded java object name.
For instance,
select uuid() returns smth like rO0ABXNyAA5qYXZhLnV0aWwuVVVJRLyZA/eYbYUvAgACSgAMbGVhc3RTaWdCaXRzSgALbW9zdFNpZ0JpdHN4cImuHII+7ojuLqk+WyT1Rw0=.
When decoded it gives srjava.util.UUIDm/JleastSigBitsJmostSigBitsxp>.>[$G

hi @andreyvk
i've checked and it's a bug. You can workaround it now with
select format("%s",uuid())

@maggiolo00 Thanks! Looking forward for the fix

@maggiolo00 What's the bug? Can anybody from the team fix it?

@lvca
IMHO the bug here it's that the uuid function returns the Java Objects
select uuid()

Is this the expected behaviour ?

Hi,
Yes this for me is definitely a bug, uuid() shoud return a string with the base64 rapresentation of the 16 bytes of the UUID, not a java object, an alternative solution could be just return a byte[16].
Regards

Returning the string representation is the right thing to do from SQL.

Fixed in both 2.2.x and develop

Noted in the Release Notes


Canada and Switzerland subregions are merged

Canada (CAN) and Switzerland (CHE) subregions are mixed. CAN is missing and all of Canadian states showing up under CHE.

I think it may be fixed as of today?
I see 13 regions marked as *, CAN in the data plus "Total, CAN" which is likely a bug (it should be a country not a state - I reported it as #405.

@divisia sounds like a cache issue -- you have to hard refresh sometimes after we deploy.


Optimize the solver implementation and reduce allocations from ~1.15-1.20MB to ~38KB

Description
A friend of mine linked me this project, and I took a quick stab at optimizing the implementation to avoid unnecessary allocations that would significantly impact end user applications. Such GC impact is usually not well seen on microbenchmarks like the ones ran by BenchmarkDotNet since it invokes GC in-between the runs. Once I get to my Windows PC, I plan to see how this impacts the WinForms demo but it should be an improvement regardless.
I have changed the project to be multi-targeted in order for the collection expressions (like [..items]) to be able to use more efficient underlying APIs - TFM affects the Roslyn output, and NS2.0 is a really conservative target that prevents it from using any new API which is why it is recommended to provide both new and compat TFM package versions, if you want to do the latter in the first place. Adding System.Memory polyfill package for Span<T> allowed using span.CopyTo(dest) instead of Array's own .CopyTo(dest, idx) - the reason for this is array's one incurs the cost of C#'s original sin - array covariance and checks associated with it. Because spans do not allow this, their version of CopyTo is a simple length comparison and immediate call to the underlying CoreLib's Memmove implementation, something that array methods take extra steps to call into.
There are other small changes like ensuring that array references are not re-read per iteration from solver class fields, and are properly hoisted to locals to enable .NET to elide bound checks against them and clone the loops to provide bounds-check-less fast paths where the length bound cannot be proven. It's part of the reason why Particle is changed to be a struct and addressed via ref instead, another part is it improves data locality.
Last but not least, the biggest bottleneck here is the default and legacy Random implementation. When Random instance is constructed without a seed, a newer Xoshiro implementation is used which is significantly faster and has stable execution time (whatever happens with legacy one makes the results regress as they continue to get executed, and tends to cause massive bimodal gap between run latencies on Windows, but that's not just with Random, only that it exacerbates it). Unfortunately, it results in non-deterministic results with larger output variance, which makes tests crash unless the result precision tolerance is significantly relaxed, so I have only left the comment to consider this change and included reference numbers below.
In order to speed-up the algorithm further, it may be worth exploring custom RNG implementations that produce either more favourable distribution of values to this algorithm, or are faster, not being subject to the exact output the default Random with seed is limited to due to backwards compatibility considerations.
Numbers
Environment
// * Summary *

BenchmarkDotNet v0.14.0, macOS Sequoia 15.0 (24A335) [Darwin 24.0.0]
Apple M1 Pro, 1 CPU, 8 logical and 8 physical cores
.NET SDK 9.0.100-rc.1.24414.7
  [Host]     : .NET 8.0.8 (8.0.824.36612), Arm64 RyuJIT AdvSIMD
  DefaultJob : .NET 8.0.8 (8.0.824.36612), Arm64 RyuJIT AdvSIMD

Before the change

Method
Mean
Error
StdDev
Gen0
Allocated

SolveDemo
104.7 ms
0.20 ms
0.18 ms
166.6667
1.15 MB

Note: starts at ~90ms and then regresses
After the change (with seedless but non-deterministic Random impl. provided by new(), makes tests unhappy with much more significant output variation)

Method
Mean
Error
StdDev
Allocated

SolveDemo
65.51 ms
0.139 ms
0.123 ms
37.93 KB

After the change (with current Random impl. provided by new(0))

Method
Mean
Error
StdDev
Allocated

SolveDemo
98.67 ms
0.158 ms
0.132 ms
38.17 KB

Note: starts at ~77ms and then regresses

Hi @neon-sunset, thanks for these optimizations! I originally wrote this code just to be educational so there are lots of allocations and unnecessary copying for the sake of simplicity. At first glace it looks like this PR improves many of them!
I may pull some features out of SwarmFitter and move them into a LoggingSwarmFitter that is built for interactivity (inspecting fits along the way, allowing users to manually request additional iterations, randomization etc.)
I'll work on this PR soon (probably tonight), merge, and publish an updated package in good time.
Thanks again for your input! This is looking fantastic 🚀

Thank you too. Honestly, on low iteration counts with the demo WinForms applications there isn't even any significant impact since all of it runs under 1ms once the solver body hits Tier1 PGO-optimized compilation. I mostly care that if there are end users that push this library much further, their experience w.r.t memory usage and GC pause frequency could be significantly improved. Feel free to leave out the changes you consider not fit for this implementation, e.g. I avoided removing LINQ in parts that were not hot or not sufficiently impactful. The main two sources of allocations are documented so if only those are applied to this, or a similar implementation elsewhere - that's fine by me as well.
Cheers!

I mostly care that if there are end users that push this library much further

I'm on board! I wondered how many iterations it would take to solve equations with lots of unknown parameters. If I've learned anything over the years, it's that you can never predict the incredible things users will do with packages they find 🙃
Note that I'm about to start committing a lot of things to this PR so it may seem pretty large, but I'll try to keep it simple with descriptive commit messages then do a non-squashing merge at the end.
I'm excited to see how far I can get this tonight! 🚀

@neon-sunset I'm going to merge this now but keep working in the main branch.
You may wish to take a peek at my implementation of the IRandomNumberGenerator and the (what I think is performant?) default.
I'm surprised that it doesn't seem to be bit-perfect deterministic, in that the "best fit" from the test system seems to be slightly different on my local computer vs. the linux computer running tests in the CI system. I'm not sure if it's a floating point error accumulation thing or something else. I'm inclined to let it go, but if you want to go down that rabbit hole I'd love to know why I had to do this 189e831b430f4e87a0731a2a28b5b54e5d577ec6 to get the tests to pass in the CI system 🤷

Hi @neon-sunset, I made some good progress but am going to put this down tonight.

Method
Mean
Error
StdDev
Allocated

SolveDemo
2.347 ms
0.0837 ms
0.0046 ms
26.72 KB

Check out the demo app to see how I changed the behavior to support successive iterative improvement.
I think the benchmark should be more stable now, so let me know if you have ideas for further improvements and we can open up a new PR if you're still interested in diving deeper! Thanks again for your help on this. I'll probably work on this another day or so before publishing an updated package.


Update README.md

Corrected the statement which says users can not disable telemetry, but now users can disable by setting the environment variable OTEL_SDK_DISABLED to true.

Disclaimer: This review was made by a crew of AI Agents.
Code Review for PR #1694
Files Changed

README.md

Overview
This pull request updates the telemetry documentation section in README.md to accurately reflect that users can disable telemetry functionality using the OTEL_SDK_DISABLED environment variable.
Documentation Accuracy
The change is crucial for improving user understanding regarding telemetry controls. It corrects previously misleading information, emphasizing the user's ability to disable telemetry, thus aligning documentation with system functionalities.
Recommendations for Improvement

Content Clarity and Structure:

To enhance readability, consider the following restructuring:It's pivotal to understand that **NO data is collected** concerning:
- Prompts
- Task descriptions
- Agents' backstories or goals
- Usage of tools
- API calls
- Responses
- Data processed by the agents
- Secrets and environment variables

**Exception**: When the `share_crew` feature is enabled, detailed data such as task descriptions and agent backstories are collected.

**Disabling Telemetry**: Users can disable telemetry by setting:
```shell
OTEL_SDK_DISABLED=true

Detailed Documentation on Environment Variables:

Add specific examples for setting the environment variable across different environments (e.g., Windows, macOS, Linux).
Mention that a restart of the application might be required for changes to take effect.

Additional Information:

Include the default value of OTEL_SDK_DISABLED to inform users of the initial state.
Provide a verification method for users to confirm if telemetry is truly disabled, potentially through logs or commands.
Clarify implications regarding application functionality when telemetry is disabled, so users are fully informed.

Historical Context and Related Learnings
Although I could not retrieve related pull requests due to access issues, it’s essential to consider past modifications in README.md, especially regarding telemetry and privacy. Updates akin to this proposal have often led to enhanced user trust and reduced inquiries related to data privacy concerns.
Conclusion
✅ Approved with Suggestions
The PR significantly improves the accuracy of the documentation regarding telemetry functionalities. Merging the changes as they stand is fine, but I strongly recommend following up with the above suggestions for improving the clarity and comprehensiveness of the README.md documentation. Implementing these changes will undoubtedly benefit our user community by making the documentation more navigable and informative.

By adopting these changes, we can ensure that users understand their options fully and can configure the system according to their privacy preferences. Thank you for your attention to this important aspect of our documentation!

Thank you for catching this!


Correct a broken link on README

I've correct a broken link and its contexts on README.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #714   +/-   ##
=======================================
  Coverage   62.27%   62.27%
=======================================
  Files          18       18
  Lines        3979     3979
=======================================
  Hits         2478     2478
  Misses       1501     1501

Continue to review full report at Codecov.

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

👍


Nested class shadowing in "override" position

Aloha. This is a friendly note that Community Builds has discovered a potential issue you might run into when migrating to Dotty / Scala 3.x.
Ref https://github.com/scala/scala/pull/8705 / https://github.com/scala/bug/issues/8353
steps
Compile using the latest 2.13.x nightly (https://scala-ci.typesafe.com/job/scala-2.13.x-integrate-community-build/3267/) or 2.13.2 when it comes out.
sbt> set ThisBuild/resolvers += "scala-integration" at "https://scala-ci.typesafe.com/artifactory/scala-integration/"
sbt> ++2.13.2-bin-53ba87a!
sbt> compile

problem
Community Builds discovered there are some nested classes in "override" position.
[scala-swing] [warn] /home/jenkins/workspace/scala-2.13.x-integrate-community-build/target-0.9.16/project-builds/scala-swing-c3658cc87953fc195bdf67bc22f02da415303e04/src/main/scala/scala/swing/RichWindow.scala:42:9: shadowing a nested class of a parent is deprecated but trait InterfaceMixin shadows trait InterfaceMixin defined in class Window; rename the class to something else
[scala-swing] [warn] /home/jenkins/workspace/scala-2.13.x-integrate-community-build/target-0.9.16/project-builds/scala-swing-c3658cc87953fc195bdf67bc22f02da415303e04/src/main/scala/scala/swing/ScrollPane.scala:35:11: shadowing a nested class of a parent is deprecated but class Value shadows class Value defined in class Enumeration; rename the class to something else

expectation
Nothing for now, but consider renaming them when you can.

it turns out both cases produce compile errors with dotty, so they will have to be fixed in any case. My WIP solution is to rename InterfaceMixin in RichWindow to InterfaceMixin2, and make ScrollPane.BarPolicy a case class instead of an Enumeration.

@Sciss did this end up resolved somehow?

Yes, in 2fb8d2ea8a2a0bc493bbc001bb767c9d0db21d75 I did what I described above - rename InterfaceMixin in RichWindow to InterfaceMixin2, and make ScrollPane.BarPolicy a case class instead of an Enumeration. When using custom windows, you will get compile errors until you change to InterfaceMixin2 now.


Configuring backup tasks skipped when "restic_backups" is a dict

When the restic_backups object is a dictionary, everything after (BACKUP) reformat dict if necessary gets skipped.  If you instead use an array it works fine.
After some investigation, I think it's to do with how the aforementioned task works.  After placing a debug task just after it, it appears that restic_backups isn't transformed as wanted (i.e. it's the same dict).  This isn't because of the jinja expression - I'm using that to transform my dict at role invocation => it does what we want.
I recall reading somewhere that you can't set a fact on top of itself - if thats indeed the case then the fix is to edit that task such that it sets the new fact as a different name.

Another thought - looking at this SO thread answer it may actually be to do with variable precedence.
Since set_facts output has a low precedence, we end up not overriding it as we expect.  The workaround is the same: change the name of the new fact

@usmanatron Please share the Ansible version you experienced this with, and provide a playbook which reproduces it. See https://github.com/roles-ansible/ansible_role_restic/pull/122 for an example.
Also, test with the most recent version of this role. Before, there was a dependency on jmespath for this reformatting, but that dependency was just removed.

Hi @markstos - apologies for taking so long to look at this but I have finally re-tested and Im seeing the same issue.

The (BACKUP) reformat dict if necessary task does seem to run, its not skipped
I popped a debug statement just after this and I can see that the restic_backups object hasn't actually changed (i.e. its the same dict I passed in!)
Changing the name of the new fact in set_fact does indeed fix it - the debug'd object is now a list and the next tasks actually run
I wonder if this is somehow related to how I generate restic_backups... it's not a static dictionary, I generate it from some other config.  I suspect that means its at a different precedence which triggers the behaviour in that SO thread

I still currently believe that changing the fact name will fix it.  Im happy to make that change and create a PR?


Troubleshooters dec21

Changes / Comments
Changes to recent pulled version to correct bug caused by legacy code left in the html file and remove/replace redundant image files.
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[x] Does the pull request title have the sheet name(s)? Include each sheet name.
[x] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.


Sass pipe sees @import in comments

Hi.  Bootstrap has this comment in _reboot.scss:
// Wrap `@viewport` with `@at-root` for when folks do a nested import (e.g.,
// `.class-name { @import "bootstrap"; }`).

This creates an infinite loop in Pipe::Sass.

Thanks for reporting in.
Unfortunately, I won't have time to look at this until some time in the middle of February.


Alphabetically sorted aois.

Overview
Fixes this issue. Areas of interest are now sorted alphabetically in myGFW page and myGFW section in the map.

Looks ok. Can you check the map sort order. The map panel should be sorted by createdDate but it seems to have broken.

I think it's working, but I also think they requested it to be alphabetically sorted. I just restored the "sort by date" bit.

Looks ok. Can you check the map sort order. The map panel should be sorted by createdDate but it seems to have broken.

I think it's working, but I also think they requested it to be alphabetically sorted. I just restored the "sort by date" bit.

Yes but in the map case the specifically want it sorted by created date


add gitea config properties to enable reverse proxy authentication

Allow the following properties to be defined in Gitea's custom resource in order to enable reverse proxy authentication:

reverseProxyAuthenticationUser
enableReverseProxyAuthentication

@david-martin I'd be in favor of removing the deployProxy option from master for a few reasons:

Gitea already supports OpenID login and if we can make it work with self signed certs (via contribution) that would fit our needs.
Having the Oauth proxy in front of gitea makes repo access difficult from outside the cluster.

@david-martin I would also be in favour of removing the oauthproxy from master if we're not using it. There are also some issues with using this reverse proxy auth that we found when testing it. Gitea won't logout the user until the cookie created by oauthproxy is removed. Even if the user was logged out of Openshift, this cookie had to be deleted manually.
I'm ok with closing this PR and removing deployProxy from master.

I'm ok with closing this PR and removing deployProxy from master.

OK, closing.
Ping me on PR for removal of deployProxy option if you'd like


fixing random methods and make it overloaded

I created 2 random methods for double and int to make it work properly.

This won't work with angelscript


Fix ArgumentCountError exceptions

Issue #, if available:
#381
Description of changes:

Removed unused $value parameters to getSuccessUrl() and getFailureUrl() to fix Uncaught ArgumentCountError: Too few arguments to function.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Hi Jason! Thanks for contributing, unfortunately we already have the fix merged in the master, so closing this PR.
Appreciate your input here,
Tarini Shah


chore: Replace "syntax" blocks with specific languages

@nschonni Thank you for your housekeeping contributions overall. Much appreciated.
@lomayor @manikadhiman @AndreaBarr Would you be able to review the PR that introduces minor Markdown fixes to code blocks? Thanks.


Button Mapping issue

Cannot find file to button map. C:\Program Files (x86)\Steam\config\steamvr.vrsettings has 3 lines in it and when i add the button mapping lines it just gets overwritten by the 3 lines again. So i tried C:\Program Files (x86)\Steam\steamapps\common\SteamVR\resources\settings but when i put the button mapping lines in here it flips my display and nothing works correctly.

I already answered you.


bump aws java sdk to 1.11.704 to support k8s SA to IAM role binding

What changes were proposed in this pull request?

Why are the changes needed?

Does this PR introduce any user-facing change?

How was this patch tested?

Have we checked that this combination of aws sdk and hadoop still works for uploading to S3?

Have we checked that this combination of aws sdk and hadoop still works for uploading to S3?

As you shared with me offline, this should be working right?


if both preview and server are specifeid, only one of them will work.

some sort of cert issue.

fixed in https://github.com/liuweiGL/vite-plugin-mkcert/pull/28

https://github.com/liuweiGL/vite-plugin-mkcert/commit/bdbd92aeea912979b193b08b10fe7d555c288409


problem with the build

Hi,
Im trying to run vadeo on my arm64 but its giving me this strange error everytime.
Must be a build error from my side.
Its making a config.yaml dir instead of a file.
Im did  : docker build -t vadeo .
Can you help me with it?
it runs but it can't read the config.yaml file.

goroutine 20 [running]:
main.streamAudio(0x40000ace20)
/build/main.go:51 +0x204
created by main.start
/build/main.go:74 +0x2a0
config file err read config.yaml: is a directory

The error says your config.yaml is actually a directory. Do you have a real config.yaml file?

I deleted the config.yaml directory and replaced it with a config.yaml file.
Problem  is, it  automatically changing back to a directory.

Are you sure this is a problem with Vadeo and not how you're using Docker? Vadeo doesn't create directories. Please try it outside of Docker and track down where your issues are, it sounds like you're using Docker mounts incorrectly.

ok gone try installing tommorow outside docker

I will check it later. thxs

is this error because my system is linux/arm64/v8 and not v7.

It says ---> [Warning] The requested image's platform (linux/amd64) does not match the detected host platform (linux/arm64/v8) and no specific platform was requested in your log.

Hi,
Can I do something to fix it? I tryed overwriting everything with this file "vadeo_0.2.0_linux_armv7.tar.gz"
Doesnt that work because its armv7?

can you help me? The binary vadeo can't find ffmpeg.
Its installed but log says: ffmpeg not found.

How and where was ffmpeg installed?

Dunno where its installed I did sudo apt install -y ffmpeg

ok strange, tryed it again and now it works. Thanks man


Apply core migrations

changes the setting of typoScript.autoInclude to fusion.autoInclude
renamed root.ts2 to root.fusion
renames Resources/Private/TypoScript to Fusion

The other migrations applied no changes.

we should rename the .ts2 file to .fusion as well or do it when there's a migration for it..

I added the rename of LoginForm.ts2 to LoginForm.fusion


Group expenses my date

Closes https://github.com/scastiel/spliit2/issues/27

There are some TypeScript errors in your code, could you please fix them? You can run npx tsc --noEmit to see them if you can’t see them in your IDE.

Ah thanks! Done!
I also ran npx prettier . --write since I noticed you have a CI check for that as well

@scastiel Ready for another look!


Align Checkbox and Radio button icons to the top

Description
This library is fantastic! But our designs are a little different... we have some exceptional cases where the text for a radio button and checkbox is long, and the default of the library is to centre the "icons" in the middle, but we want to show it at the top of the container... is there any way of doing this?
Problem Statement
I can not make this work even using _stack
Proposed Solution or API
_stack={{ justifyContent: 'flex-start' }}

Alternatives
No response
Additional Information
No response

Hi @santomegonzalo, Property should alignItems: 'flex-start'
<Radio value="one" my={1} _stack={{ alignItems: 'flex-start' }}>
       Lorem ipsum dolor sit amet consectetur adipisicing elit. Quas eaque
       cupiditate harum error eligendi iste repellendus sed! Nobis eveniet,
       eligendi consequatur dolorum harum nesciunt a quo sed, nihil hic
       aspernatur.
     </Radio>


Add Delete Logfile data?

Godot: 3.2.1
Plugin: 0.9.0
Add a button to delete past logfiles
If someone had forgotten the token, and had to regenerate the token, the plugin seems like to always use the old token from logfiles. Or perhaps it's an isolated issue on my behalf? Please let me know as I'm not sure.
Also logfile is misspelled when hovering over the exclamation point. :-)

Hi @venska ,
Commit 398a7b9c8e69d76246175630b8fe42bdab544114 adds a new functionality to the Sign In screen:

Also fixed "logfile" misspelling.
Thank you very much


Packaging to windows 8.1 / windows Phone 8.1 issue

When validating an app package that contains this plugin, it fails with the following error:
File encoding test
FAILED
UTF-8 file encoding
Error Found: The UTF-8 file encoding test detected the following errors:
File plugins\nl.x-services.plugins.socialsharing\www\SocialSharing.js is not properly UTF-8 encoded. Re-save the file as UTF-8 (including Byte Order Mark).
Impact if not fixed: HTML, CSS, and JavaScript files must be encoded in UTF-8 form with a corresponding byte-order mark (BOM) in order to benefit from bytecode caching and to avoid other runtime error conditions.
How to fix: Open the affected file, and select "Save As..." option from the File menu in Visual Studio. Select the drop-down control next to the Save button and select "Save with Encoding..." option. From the Advanced save options dialog, choose the "Unicode (UTF-8 with signature)" option and click the OK button.

Recreated the file, don't think much will change for you though.
If you still have an issue with this please follow the instructions provided by the validator and send a PR.


Error when navigate to localhost:5000

Hi @AbelianKraun  currently we have some minor bug. Can you please take a look

Sure. I Will check.

Got it. Maybe some cache


[Question] Calculate key in repo

Can i do key calculation in fetcher function?
Example:
class InteractionRepoImpl {
private val circlesInfoStore by lazy {
    StoreBuilder
        .from(
            fetcher = Fetcher.of { _ ->
                val key = userRepo.user()?.id
                    ?: return StoreResponse.Error.Message("user not auth")

                remoteInteractionSource.loadInteractionCirclesInfo(userId, 4)
            },
            sourceOfTruth = ...
        ).build()
}

override fun interactionCirclesInfo(request: StoreRequest<Nothing>): Flow<StoreResponse<List<InteractionCircleInfo>>> {
    return circlesInfoStore.stream(request)
}

}

From store perspective you only have 1 key and it is nothing. This means memory cache and source of truth is going to treat every request as same key. I don't think this is what you want.

I want the repository to leave a method that only accepted StoreRequest, without a key, and already inside this method in the repository this key is calculated, or an error is given
like in my example
val userId = userRepo.user()?.id
                    ?: return StoreResponse.Error.Message("user not auth")

What do you expect the store to be doing rather than calling the fetcher directly? Store expects each request to have a unique key. This allows store to throttle or cache in a per key basis. If you only pass same key to store there little value in using it.

Hmm, I think I misunderstood the meaning of the key. I thought I was specifying a key to pass it to the remote data source to request data

What do you expect the store to be doing rather than calling the fetcher directly? Store expects each request to have a unique key. This allows store to throttle or cache in a per key basis. If you only pass same key to store there little value in using it.

This request takes userId as a key and returns a list of items. There will always be one key, the id of the current user, so I want to move the logic for getting the userId to one place, the repository.

I'm not sure store gives value for this use case. Since only managing one key you can call the fetcher/network source directly. Otherwise if the user if changes you won't get the new data but old cached data. Another way to do this is userrepo.id.map{store.stream(StoreRequest(it))

Also, what will happen if i pass key as Nothing?

What do you expect the store to be doing rather than calling the fetcher directly? Store expects each request to have a unique key. This allows store to throttle or cache in a per key basis. If you only pass same key to store there little value in using it.

What is the point then for me to use this library if I must manually call load from local and network storage? I'm just switching to library to not do it manually.
I have a lot of usecases where i once load list of items from api, cache it and then show them from cache with maybe parallel refresh request.

Also, how library understand when load info from network? For example, if local source return empty list, will library trigger fetcher?
    override fun saveInteractionCategories(categories: List<InteractionCategory>) {
        val json = gson.toJson(categories)
        categoryPreference.set(json)
    }

    override fun getInteractionCategoriesFlow(): Flow<List<InteractionCategory>> {
        return categoryPreference
            .asFlow()
            .mapLatest {
                it?.let{
                    gson.fromJsonWithType(it)
                        ?: emptyList()
                } ?: emptyList()
            }
    }


better interface for server side

Now I think maybe I should split js-bridge into js-base and js-server and js-client. Heck.


Audio output switcher

Description of the new feature / enhancement
A keyboard shortcut to switch between different audio output devices like the one for switching languages
There are similar apps already in existence:
https://github.com/xenolightning/AudioSwitcher_v1/
https://github.com/Belphemur/SoundSwitch
But would like to see it implemented in Windows.

Scenario when this would be used?
Fast switching between multiple devices, for example: from headphones to speakers when you want to share sound fast
Supporting information
No response

/feedbackhub


new initial password implementation

The way the initial password is configured has changed in Argo CD v1.9.  More information can be found here: https://github.com/argoproj/argo-cd/blob/master/docs/getting_started.md
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

The current command is working in the expected lab environment and you have duplicate ARGO_PWD so not sure what's the expected outcome here.