Error: Exception value cannot be null with new beta (<IP_ADDRESS>)

Hi,
I'm new to this project so forgive me if I'm bringing up a known issue.
On my fairly vanilla IIS 8.5 Windows 2012 R2 server I am getting an error at the end of the process of generating a certificate. Everything seems to have worked, the certificate is created successfully, bindings are updated, task is added to task scheduler, etc. but just before the "Would you like to start again?" question I keep getting:
EROR: Exception Value cannot be null.
Parameter name: source
Any ideas what this is? Is it something I'm doing or not doing or could it be a bug in this beta?
Thanks.

To add to the info above, I have noticed that the task in Task Scheduler also presents this same error, reproducible if I run the command line that is used for the task:
C:\letsencrypt-win-simple\letsencrypt.exe --renew --baseuri "https://acme-v01.api.letsencrypt.org/"
When I run that manually (or when Task Scheduler runs it) I get:

[INFO] Let's Encrypt (Simple Windows ACME Client)
[INFO] Version <IP_ADDRESS>878 (RELEASE)
[INFO] Please report issues at https://github.com/Lone-Coder/letsencrypt-win-simple
[INFO] Renewal period: 60
[INFO] Certificate store: WebHosting
[INFO] ACME Server: https://acme-v01.api.letsencrypt.org/
[INFO] Checking renewals
[EROR] Exception Value cannot be null.
Parameter name: source

Again, I'm not entirely sure if I've done something silly, but even with the error message all of the certs still seem to have been installed and are working/valid.
Thanks.

I think this is the same problem as I described here: https://github.com/Lone-Coder/letsencrypt-win-simple/issues/483#issuecomment-325982197

I agree it looks similar - but I have not used any previous versions of the program. The version I am running (<IP_ADDRESS>878) is literally the first version I have ever used. There has never been a previous version of the software on my server. The server is only about a week old too, and I am the only person who admins it. So if it is related to previous settings, I can only guess that this version of the program is creating old and new settings both by itself.
I hadn't realised that I could get the debug info so have just adjusted my config file and run the renewal process to generate the error again:

[INFO] Renewal period: 60
[INFO] Certificate store: WebHosting
[INFO] ACME Server: https://acme-v01.api.letsencrypt.org/
[DBUG] Settings {Renewals="The property accessor threw an exception: ArgumentNul
lException"}
[DBUG] Config folder: C:\ProgramData\letsencrypt-win-simple\httpsacme-v01.api.le
tsencrypt.org
[DBUG] Certificate folder: C:\ProgramData\letsencrypt-win-simple\httpsacme-v01.a
pi.letsencrypt.org
[DBUG] Loading signer from C:\ProgramData\letsencrypt-win-simple\httpsacme-v01.a
pi.letsencrypt.org\Signer
[DBUG] Getting AcmeServerDirectory
[DBUG] Send GET request to https://acme-v01.api.letsencrypt.org/directory
[DBUG] Loading registration from C:\ProgramData\letsencrypt-win-simple\httpsacme
-v01.api.letsencrypt.org\Registration
[INFO] Checking renewals
[DBUG] Exception ArgumentNullException {Message="Value cannot be null.\r\nParame
ter name: source", ParamName="source", Data=[], InnerException=null, TargetSite=
System.Collections.Generic.IEnumerable1[TResult] Select[TSource,TResult](System .Collections.Generic.IEnumerable1[TSource], System.Func2[TSource,TResult]), St ackTrace="   at System.Linq.Enumerable.Select[TSource,TResult](IEnumerable1 sou
rce, Func`2 selector)\r\n   at LetsEncrypt.ACME.Simple.Settings.get_Renewals()\r
\n   at LetsEncrypt.ACME.Simple.Program.CheckRenewals()\r\n   at LetsEncrypt.ACM
E.Simple.Program.Main(String[] args)", HelpLink=null, Source="System.Core", HRes
ult=-2147467261}
[EROR] Exception Value cannot be null.
Parameter name: source

I think the error message looks the same as the one you (nvanrooij) got when you tried to renew.
I wondered if the Default Web Site had any effect so I tried with it both present on the server and then removed from the server, but the results are the same.

Feels like a bug in 1.9.5, i'll look into it next week.

@nvanrooij you are right that seems to be the same issue

You can "fix" this error by creating a new registry key "Renewals" (Multi String value, REG_MULTI_SZ) in key "HKEY_LOCAL_MACHINE\SOFTWARE\WOW6432Node\letsencrypt-win-simple\https://acme-v01.api.letsencrypt.org/". You probably need to create the keys "letsencrypt-win-simple" and "https://acme-v01.api.letsencrypt.org/" within WOW6432Node first.

Thanks. I have created those registry keys and I now don't get the errors I described earlier.
Presumably this will be fixed in the next release, so it would be good to know if I can then remove the registry keys or if they need to stay (i.e. they should have been created on the first run and need to be there?).

I'm having the exact same issue with the same version and same brand new server, but creating the registry key did not work for me. Still got the same null value error.
Unfortunately it also sounds like I'll have to wait a week to try any other solutions since I have run into some sort of request limit...

[EROR] ACME Server Returned: Unexpected error - Response: {
"type": "urn:acme:error:rateLimited",
"detail": "Error creating new cert :: too many certificates already issued for exact set of domains: XXXX",
"status": 429
}`

Are you sure it's the same version? Did you definitely create the registry keys correctly?
Make a .reg file with this code in:
Windows Registry Editor Version 5.00
[HKEY_LOCAL_MACHINE\SOFTWARE\Wow6432Node\letsencrypt-win-simple\https://acme-v01.api.letsencrypt.org/]
"Renewals"=hex(7):00,00

and run it to add it to the registry, then try running letsencrypt-win-simple again and see if you still have a problem. It cured the error for me, but you haven't posted your error message so can't say for definite if it's the same as I was getting.

Yep, exact same version (<IP_ADDRESS>878) and same error message: "Exception Value cannot be null. Parameter name: source" The rest of the message is lost unless the logs are saved to disk somewhere.
I checked the registry keys again, and it looks like I missed the trailing slash on the URL. I triple checked everything I typed, but apparently still missed that slash.  :(
I'm guessing that was the problem, but now I have to wait a week for them to allow me to try again. I'll post an update at that time.

Sounds like you've implemented the fix correctly now. Shame you've hit your rate limit! Perhaps you have another domain you can try with for now?
I'd still like to know if we have to keep those registry keys in place once the bug is fixed, or if they can be removed once this is fixed? I imagine @WouterTinus will be able to tell us this once its fixed :)

I just tried a couple other domains on another Win 2012 server after applying the registry keys, and it worked flawlessly. Thanks!
Based on other threads I've seen, it sounds like the registry key is required for now, but the plan is to stop using the registry completely for version 2.0.

Yes, the keys are needed and will be created automatically again by the next beta. The problem was a simple missing null-check for the clean install scenario.

Wonderful! Thanks for your time spent on this! 🥇

Thanks, this work around worked for me as well.

Same here... Registry key add does not work, no clue why but I'll await the next beta..
Thanx for the great work!

@marcverkade did you hand type in the keys, or did you use the .reg code that fearworksmedia provided above?
Mine originally didn't work because I missed the trailing slash on one of the keys, even though I checked and re-checked my work several times. His .reg code worked on multiple servers for me.

Hai Jeff,
I copied and pasted the code of the site in a .reg file and after I imported that file, it created the extra registry node, but one level too high.
[cid:3336ac7f-b01c-44e3-8a22-0bddfa209a5d]
I created the reg-key manually and then it worked.
Thanx for your concern.
regards, Marc

Van: Jeff Richmond<EMAIL_ADDRESS>Verzonden: maandag 11 september 2017 16:43
Aan: Lone-Coder/letsencrypt-win-simple
CC: marcverkade; Mention
Onderwerp: Re: [Lone-Coder/letsencrypt-win-simple] Error: Exception value cannot be null with new beta (<IP_ADDRESS>) (#532)
@marcverkadehttps://github.com/marcverkade did you hand type in the keys, or did you use the .reg code that fearworksmedia provided abovehttps://github.com/Lone-Coder/letsencrypt-win-simple/issues/532#issuecomment-326778347?
Mine originally didn't work because I missed the trailing slash on one of the keys, even though I checked and re-checked my work several times. His .reg code worked on multiple servers for me.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Lone-Coder/letsencrypt-win-simple/issues/532#issuecomment-328587813, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AG3Eeu-_oVBRhrnKVnDxGNbGaeW-Dhodks5shWNDgaJpZM4PD9el.


feat(headless): Query Triggers

https://coveord.atlassian.net/browse/KIT-764

Manual tests worked :)


[VC] fix: register UWSOperationCounter

Signed-off-by: zhuangqh<EMAIL_ADDRESS>

/lgtm
/approve


[bug-1574]: Fix offline bundle registry references

Description
The images.manifest file prior to 1.7.0, used to update the bundle with the private registry when preparing the bundle, contains references with and without docker.io for some images:
# grep isilon scripts/images.manifest
dellemc/csi-isilon:v2.11.0
docker.io/dellemc/csi-isilon:v2.11.0

If a file contains docker.io/dellemc/csi-isilon:v2.11.0, preparing the bundle will replace dellemc/csi-isilon:v2.11.0 with the private registry: docker.io/private-registry.com/csi-isilon:v2.11.0. Then, docker.io/dellemc/csi-isilon:v2.11.0 is replaced with the private registry but now that string has no matches and the file is left with the faulty registry.
This fix is having all image references specify the registry.
https://github.com/dell/csm-operator/blob/main/bundle/manifests/dell-csm-operator.clusterserviceversion.yaml and https://github.com/dell/csm-operator/blob/main/operatorconfig/driverconfig/common/default.yaml, used to gather image names when building the bundle, are updated to reference all images with the public registry. Since 1.7.0, all of our images now reference quay.io except sdc, otel-collector, and nginx. These are updated to reference docker.io.
The sample files are updated to reference the registry to be consistent with the images.manifest file.
Made csm-offline-bundle.sh executable.
Adds a new unit test for code coverage improvement for PR check.
Caveats:

Since https://github.com/dell/csm-operator/blob/main/bundle/manifests/dell-csm-operator.clusterserviceversion.yaml references the latest images, the images.manifest file only has the latest images. The sample files for older releases are not updated with the private registry when preparing the bundle. This behavior has been in place since day one.
The OCP sample files are not updated with the private registry due to the image names using the redhat registry and SHAs instead of tags. The user will update these sample files manually.

GitHub Issues
List the GitHub issues impacted by this PR:

GitHub Issue #

https://github.com/dell/csm/issues/1574

Checklist:

[ ] I have performed a self-review of my own code to ensure there are no formatting, vetting, linting, or security issues
[ ] I have verified that new and existing unit tests pass locally with my changes
[ ] I have not allowed coverage numbers to degenerate
[ ] I have maintained at least 90% code coverage
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have maintained backward compatibility

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Please also list any relevant details for your test configuration
Built and prepared the bundle verifying that the private registry is correctly put in place in the yaml files that the user will use.
Built and installed the Operator via offline bundle.

have you tried installing with the bundle? how have we verified that it works other than looking through it?

Yes, this works. The issue that the customer observed with the RELATED_IMAGES environment variables is resolved and installation is successful, with relevant files updated with the registry.


[core] The agg table model supports replace types

Purpose
The agg table model supports replace types
Tests

API and Format

Documentation

@JingsongLi  PATL,thanks.


Accessibility of Thumbnail Strip

Is there anyway of making the thumbnail strip in the viewer accessible so that screen readers for those with limited vision can "see it" and navigate to it using the keyboard.

We don't have accessibility built in to the thumbnail strip yet, but we would certainly welcome it. Would you be up for adding it in?

@iangilman I would be happy to. Can you point me to the area of code where I might changes? I have looked over the code but haven't come to grips with this part. Also I am not a canvas expert, so I am not sure if I can insert the appropriate tags for the canvas elements?

Looks like the ReferenceStrip uses separate elements for each thumbnail. Here is where the container is created:
https://github.com/openseadragon/openseadragon/blob/master/src/referencestrip.js#L77
...and here is where the individual thumbnails are created:
https://github.com/openseadragon/openseadragon/blob/master/src/referencestrip.js#L185
There's also this element, that might be something to do with an earlier attempt at accessibility? I'm not sure what it's for. Anyway:
https://github.com/openseadragon/openseadragon/blob/master/src/referencestrip.js#L439

@iangilman I had a look but don't have the time to change it. I eventually implemented an external thumbnail strip. Here's our example: http://nladom-test.nla.gov.au/nla.obj-41292842/view

@terenceingram Looking good! Probably a better approach anyway.
Btw, we have multi-image support on master, soon to be released. If you want you can do things like this with it:
http://iangilman.com/osd/test/demo/m2/


There is something wrong with the "Possible Matches"

There is some sort of constant render issue going on in the Entity Detail component, but only when the "possible matches" section is present. You can see a large jump in memory usage, as well as the fact that the section's toggle handle does not toggle like the other sections.

Pull Requests:

https://github.com/Senzing/sdk-components-ng/pull/58


Query executed on Athena every time a panel is viewed, instead of caching

What happened:
Every time I view my panel, the graph is blank and a query to Athena is made. This happens whether the dashboard time range is relative or fixed (e.g. choosing a fixed range and refreshing the page to request the same range again). The query is static and contains no macros, so it is identical every time it is run.
If the time range is changed after initial load (e.g. dragging a box to zoom in), a new query is made instead of redrawing the existing graph.
What you expected to happen:
Opening a dashboard repeatedly avoids repeated calls to Athena for the same query, caching results instead. Zooming in does not result in a new query. Zooming out (up to the extents of the most recent query results) does not result in a new query.
How to reproduce it (as minimally and precisely as possible):

Create a panel.
Query:

select sum(duration_seconds)/3600 as duration_hours, date_trunc('day', date_parse(start_time,'%Y-%m-%d %H:%i:%s.%f')) as time from my_table
where mycol = 'MyValue'
group by date_trunc('day', date_parse(start_time,'%Y-%m-%d %H:%i:%s.%f'))
order by time

Save the panel to a dashboard.
Navigate to the dashboard.
In Athena, observe that a query was made.
Navigate away from the dashboard and navigate to it again.
In Athena, observe that another identical query was made.

Anything else we need to know?:
Environment:

Grafana version: 8.2.5
Plugin version: 1.0.0
OS Grafana is installed on: AMG
User OS & Browser: macOS 10.15.7, Firefox 91.4.0esr

There is no Cache tab in the data source settings and no "Cache timeout" option in query options for this data source.

Thanks for the issue, I see that you found the discussion at https://github.com/grafana/athena-datasource/discussions/86 so let's keep the conversation there for readability. I'll comment here in line anyway this time:

What you expected to happen:
Opening a dashboard repeatedly avoids repeated calls to Athena for the same query, caching results instead.

Yes, that's the goal of the Grafana query caching (more details in the discussion).

Zooming in does not result in a new query. Zooming out (up to the extents of the most recent query results) does not result in a new query.

This will only works if the query remains the same (with the caching system), which is the case of the example you posted I. If the query depends on the time range (e.g. using the timeFilter macro), the query will be different so a new query will be issued.

There is no Cache tab in the data source settings and no "Cache timeout" option in query options for this data source.

It's not clear to me if query caching is already or will be available in AMG, I have asked this to the team to get more info, I will reply once I get this information in the discussion.

2. Query:

Unrelated to this issue but I'd recommend you to use the macro $__dateFilter to filter by date, so you reduce the data queried :)


Store address hierarchy information in attributes instead of channel args.

This is more appropriate, since the attributes are not part of the key in the subchannel pool and therefore do not affect uniqueness.  The address hierarchy information is relevant only to LB policies and should not affect the subchannel identity.

Known issue: #23597


Remove per-region values for SNI router chart

They are tracked in the aws repo.

BTW, it doesn't looks like we need per preview env SNI router, but please correct me if I'm wrong @antonyc @SergeyMelnikov

BTW, it doesn't looks like we need per preview env SNI router

For cloud preview envs probably there is no need indeed.


Stats endpoint

@ksubrama you mentioned you had some pedant tests. If so, could you push them up.

👍


Optuna and fixes

Added Optuna tutorial
Added tutorial file names to index.rst
Remove imports from KD_Lib/__init__.py

Let me know if this is ready to merge or not.

Yup, good to merge

Good to merge now

Good to merge now

Resolve these conflicts once

Some problem with Black again ? Formatting has been done locally.

I don't think we should have anymore Black issues now

Update the black version, i ran it locally for your branch it works. Run pip install -U black


Casting a string to a DATE in MySQL results in 'unsupported Scan' error

Running SELECT CAST("2020-11-02" AS DATE) results in an error instead of the expected result. This only seems to happen if a connection is made with \c that is specifying a driver and params instead of only an url.
(not connected)=> \c mysql root:msandbox@tcp(<IP_ADDRESS>:8027)/test?sql_mode=STRICT_ALL_TABLES
Connected with driver mysql (8.0.27)
(mysql)=> SELECT CAST("2020-11-02" AS DATE);
error: mysql: sql: Scan error on column index 0, name "CAST(\"2020-11-02\" AS DATE)": unsupported Scan, storing driver.Value type []uint8 into type *time.Time
(mysql)=> SELECT CAST("2020-11-02" AS DATE) + 0;
 CAST("2020-11-02" AS DATE) + 0
--------------------------------
                       20201102
(1 row)

(mysql)=> SELECT CONCAT(CAST("2020-11-02" AS DATE),"");
 CONCAT(CAST("2020-11-02" AS DATE),"")
---------------------------------------
 2020-11-02
(1 row)

Looks like this depends on how \c is used:
(not connected)=> \c mysql://root:msandbox@<IP_ADDRESS>:8027/test
Connected with driver mysql (8.0.27)
my:root@<IP_ADDRESS>:8027/test=> SELECT CAST('2020-11-02' AS DATE);
 CAST('2020-11-02' AS DATE)
----------------------------
 2020-11-02T00:00:00+01:00
(1 row)

my:root@<IP_ADDRESS>:8027/test=> \c mysql root:msandbox@tcp(<IP_ADDRESS>:8027)/test
Connected with driver mysql (8.0.27)
(mysql)=> SELECT CAST('2020-11-02' AS DATE);
error: mysql: sql: Scan error on column index 0, name "CAST('2020-11-02' AS DATE)": unsupported Scan, storing driver.Value type []uint8 into type *time.Time

The usql driver mysql injects additional parameters. If you use the \c <driver> <params> format, it bypasses the URL parameter forcing that the mysql driver adds.
Try this:
\c mysql root:msandbox@tcp(<IP_ADDRESS>:8027)/test?sql_mode=ansi&parseTime=true&loc=Local

Please note that I chose to force ANSI mode only on MySQL, because without it, it would have required writing a different command parser for just MySQL. Also, because you really should be using ANSI mode ;)

I just tried this:
$ usql
Type "help" for help.

(not connected)=> \c my://
error: mysql: dial tcp <IP_ADDRESS>:3306: connect: connection refused
(not connected)=> \conninfo
(not connected)
(not connected)=>
(not connected)=> \c mysql root:P4ssw0rd@tcp(<IP_ADDRESS>:3306)?sql_mode=ansi&parseTime=true&loc=Local
error: mysql: invalid DSN: missing the slash separating the database name
(not connected)=> \c mysql root:P4ssw0rd@tcp(<IP_ADDRESS>:3306)/mysql?sql_mode=ansi&parseTime=true&loc=Local
Connected with driver mysql (10.6.5-MariaDB-1:10.6.5+maria~focal)
(mysql)=>  SELECT CAST('2020-11-02' AS DATE);
 CAST('2020-11-02' AS DATE)
----------------------------
 2020-11-02T00:00:00+07:00
(1 row)

(mysql)=>

Also note that with ANSI mode you'll need to use ' to quote strings. as " is for database identifiers.


Will this work for Electron renderer processes?

First off, I'm really excited about this module. Great work!
From the example, it looks like bytenode will compile Electron's main process files. Is there a way to compile the renderer process files? These may or may not have node enabled, but I'm not sure how they might have file system access. Is there another method for compiling them to bytecode?
Thanks!

In order to use bytenode, you must have an access to the Node.js' require function. So if the render process does not have Node.js enabled, you can't require neither bytenode nor any other node files.

With nodeIntegration turned on for an Electron BrowserWindow, you do have access to require(). I'll see if I can figure it out and maybe I can send you a new example project which compiles the renderer as well.
Thanks.

There are many issues (or questions) here to show how to compile a renderer process. I will try to find you some of them later.

Yes, it can, this is a demo I wrote, hope it will help you
demo
The encryption method is heretools/encode.js


HOSTEDCP-1994: Add generic create function for SecretProvider

What this PR does / why we need it:
Add generic create function for SecretProviderClass for ARO HCP.
Example usage - https://github.com/openshift/hypershift/pull/4884.
Which issue(s) this PR fixes:
Fixes #
Checklist

[ ] Subject and description added to both, commit and PR.
[ ] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

/retest

/lgtm

/test e2e-aks

/test e2e-aks

/retest

/retest

/test e2e-aks

/test e2e-aks

/test e2e-aws


v2.3.3 Imported Rancher cluster into incorrect Project

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

Import Rancher cluster
Namespace imported into All
Namespaces not imported into proper Projects

Result:
Other details that may be helpful:
Environment information

Rancher
v2.3.3

User Interface
v2.3.25

Helm
v2.14.3-rancher1

Machine
v0.15.0-rancher20-1

Installation option (single install/HA):
Single

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported): Custom
Machine type (cloud/VM/metal) and specifications (CPU/memory): VM (4 CPU/8GB)
Kubernetes version (use kubectl version):

ubuntu@dream:~/rancher-cluster$ rancher kubectl version
Client Version: version.Info{Major:"1", Minor:"16", GitVersion:"v1.16.0", GitCommit:"2bd9643cee5b3b3a5ecbd3af49d09018f0773c77", GitTreeState:"clean", BuildDate:"2019-09-18T14:36:53Z", GoVersion:"go1.12.9", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"16", GitVersion:"v1.16.3", GitCommit:"b3cbbae08ec52a7fc73d334838e18d17e8512749", GitTreeState:"clean", BuildDate:"2019-11-13T11:13:49Z", GoVersion:"go1.12.12", Compiler:"gc", Platform:"linux/amd64"}

Docker version (use docker version):

Client: Docker Engine - Community
 Version:           19.03.5
 API version:       1.40
 Go version:        go1.12.12
 Git commit:        633a0ea838
 Built:             Wed Nov 13 07:29:52 2019
 OS/Arch:           linux/amd64
 Experimental:      false

Server: Docker Engine - Community
 Engine:
  Version:          19.03.5
  API version:      1.40 (minimum version 1.12)
  Go version:       go1.12.12
  Git commit:       633a0ea838
  Built:            Wed Nov 13 07:28:22 2019
  OS/Arch:          linux/amd64
  Experimental:     false
 containerd:
  Version:          1.2.10
  GitCommit:        b34a5c8af56e510852c35414db4c1f4fa6172339
 runc:
  Version:          1.0.0-rc8+dev
  GitCommit:        3e425f80a8c931f88e6d94a8c831b9d5aa481657
 docker-init:
  Version:          0.18.0
  GitCommit:        fec3683

Importing a GKE cluster works as expected


Connect BBB + Motor Controller

1st Configuration Attempt:

connect 7.5v (5 x AA) batteries to motor controller board i.e. power circuit independent of BBB
connect 2 x 6v DC motors to motor controller
connect E1/M1, E2/M2 from motor controller to BBB
connect BBB ethernet to home router
connect 5v adaptor to BBB

Blocker:
When attempting to use the adafruit python IO library, I get the following error;
IOError: [Errno 2] No such file or directory: '/slots'
To resolve, I need to install the linux-3.8.13-boneX kernel that enables device tree overlay.
After doing some research I've come to the conclusion that trying to get the Beagleboard tailored Ubuntu 14.04 ARM for Beaglebone image to work by having to compile and patch the missing linux-3.8.13-boneX kernel for device tree overlay is not worth it.
Solution is either to interface to an Arduino (WiDo - Leonardo compatible), then to the motor controller OR try a RaspberryPi2 instead with their Ubuntu build.
Here are some links I found that may help others:

https://groups.google.com/forum/#!topic/beagleboard/olTY89KcMxg
http://www.element14.com/community/thread/31051/l/how-to-bb-view-on-latest-debian?displayFullThread=true

Looks like this could be a fix. Will try it tonight -https://github.com/mgaggero/beaglebone/wiki/Make-ADC-working-on-BBB---Ubuntu-kernel-3.14-

Moved this issue to the Ideas column as beaglebone may add cape manager to > linux-3.14* in the future


Videos not playing in Windows 11

I noticed this and seeing there's already an issue created for it, thought maybe I could add some of my findings. Not sure if it's helpful, but at least it'll be written up.
The first thing observed is the AVIFileOpen failed at CAVI.cs#40 from CDTX.cs#1522, but this actually seems to happen even when videos do work.
The actual problem appears to be just below foreach (CDirectShow cds in this.listDS.Values) , CTDX.cs#1529 .
Creating the FDK.CDirectShow instance at CDTX.cs#159 and then finally throwing exception at CDirectShow.cs#316 -- this.memoryRendererObject = new MemoryRenderer();
Actual exception is [ERROR] System.BadImageFormatException: Retrieving the COM class factory for component with CLSID {CE3CE3EE-5C4E-4BDC-A467-C068E1FC3DA5} failed due to the following error: 800700c1  is not a valid Win32 application. (Exception from HRESULT: 0x800700C1).
Replicated this from a fresh clone of the main branch a couple minutes ago --- to be complete here's the ffprobe output of the video I used for the song AloneBirthday:
ffprobe.log
Hopefully this trace provides some value for devs

Hello, thank you for your observations, currently for videos there is a file called FDK.MemoryRenderer.dll, accessed via a COM interface which is very likely x86-specific, it mights need a complete library rewamp to fix

same problem

Fixed since the .net7 migration


Vulnerability roundup 84: gnome-shell-3.34.4: 1 advisory

search, files

[ ] CVE-2019-3820 CVSSv3=4.3 (nixos-20.03)

Scanned versions: nixos-20.03: 82b5f87fcc7. May contain false positives.

20.03 is Frozen and EOL.
As such, issue is closed.


GEODE-9132: Cleanup PartitionedRegionClear

commit 425f25b3e0fbcd8681dc0f76340281b1ec143798
Author: Kirk Lund<EMAIL_ADDRESS>Date:   Fri Apr 9 17:24:26 2021 -0700

    GEODE-9132: Cleanup PartitionedRegionClearMessage

    * Use descriptive names for variables and methods
    * Use Objects.requireNonNull instead of Assert.assertTrue
    * Remove unnecessary uses of final, this, and super
    * Use static logger
    * Reformat some lines with weird formatting

commit a4fa93d30819924bd1ec291470c60743e950f89f
Author: Kirk Lund<EMAIL_ADDRESS>Date:   Fri Apr 9 15:22:46 2021 -0700

    GEODE-9132: Fixup PartitionResponse constructors

    * Chain constructors to only one constructor that calls super
    * Expose all arguments for dependency injection
    * Provide type to recipients

commit a2c41bf9a2bcb1b42d5fe6d85218b9ded8291ce1
Author: Kirk Lund<EMAIL_ADDRESS>Date:   Fri Apr 9 15:06:40 2021 -0700

    GEODE-9132: Delete ClearPRMessage

Since this PR was supposed to go to feature/GEODE-7665 and not develop, I'm closing it.


Add Banners to README

They should include:

npm
bundle size
build passing?
downloads

Fixed in version 0.3.1


Add posibility to specify snmp protocol version

Description:
Add posibility to specify version of snmp protocol version in configuration. Higher version number is needed to read 64-bit counters.
Related issue (if applicable): fixes #
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io #2621
Example entry for configuration.yaml (if applicable):

Checklist:
If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] New dependencies have been added to the REQUIREMENTS variable (example).
[ ] New dependencies are only imported inside functions that use them (example).
[ ] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

Hi @jhemzal,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

🎉  thanks


Low resolution text on high resolution (mac) displays

There have been various issues in the past, related to the resolution scaling on high dpi displays. This is one of them.
Using the standard hello world example, the text is rendered with lower resolution on retina displays.
This is because text is rendered on an intermediate texture before the frame buffer, and the size of this texture is based on the viewport, not the frame buffer. (Typically, a mac has 2x resolution scaling in both directions, meaning that the default viewport will be half width and half height of the frame buffer).
The desktop coordinate system is scaled, so when you request a 800x600 window, you get a 800x600 window in the desktop coordinate system, but it's resolution (frame buffer) is 1600x1200.
Related tickets: bitbucket #248 bitbucket #78
Example code:
__docformat__ = 'restructuredtext'
__version__ = '$Id$'

import pyglet

window = pyglet.window.Window()
label = pyglet.text.Label('Hello, world',
                          font_name='Times New Roman',
                          font_size=36,
                          x=window.width//2, y=window.height//2,
                          anchor_x='center', anchor_y='center')

@window.event
def on_draw():
    window.clear()
    label.draw()

pyglet.app.run()

Windows screenshot:

Mac screenshot:

( You might have to look close or zoom to notice that the text is low resolution. )

There is a workaround posted here. (Changing the viewport to force text rendering to use bigger textures.

@olehermanse we can have a look at putting the workaround directly into the default TextLayoutGroup: https://github.com/pyglet/pyglet/blob/master/pyglet/text/layout.py#L534
That way the state is set and unset properly when rending the text.

@benmoran56 What do you mean by setting the state? I think the proper fix would be to use the get_frame_buffer_size() method, that I saw you discussing in Discord, to set the size of the glyph texture, instead of the viewport size. But then you have to be careful when rendering the texture into the viewport, making sure to scale it based on the ratio of the framebuffer to the viewport.

I don't think my workaround is great. People want to be able to control the viewport size, and still have nice text, as can be seen in bitbucket #78.

Looks like I accidentally referenced this issue in an unrelated commit message. I guess we should be careful with using #<number> in commit messages and comments unless actually referencing issues or pull requests 😄


Clean elses

I've removed the elses when we have already returned something :put_litter_in_its_place:

As this overlaps with what's happening right now in odm-ng branch (I think) I'm leaving the merge decision up to @alcaeus :)

Didn't know that was a branch for next version development already :sweat_smile:

There's that big https://github.com/doctrine/mongodb-odm/pull/1553 that switches us to the new driver and moves all relevant things (including query/aggregation builders) from doctrine/mongodb into the ODM :)

I've tagged this for the 2.0 milestone and will take a look once #1553 has been merged down. I'll try to get that done within a week or so, after that improvement PRs such as this can be based on odm-ng (or master if I can finally push myself to just merge odm-ng down).

@carusogabriel could you rebase this pull request?

Sure, I’ll rebase and search for similars 👨💻

@alcaeus Only one test failed because of Mongo. Should we retrigger it?

Done.

Thanks @carusogabriel!


SelectableText.rich: Fix TextSpan.style not merged into default text style

fixes https://github.com/flutter/flutter/issues/94042

Before
After

import 'package:flutter/material.dart';

void main() => runApp(const MyApp());

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    const TextStyle textStyle =
        TextStyle(fontSize: 12.0, color: Color(0xff00ff00));

    return const MaterialApp(
      home: SelectableText.rich(
        TextSpan(
          text: 'Abcd',
          style: textStyle,
        ),
      ),
    );
  }
}

Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Google testing failed. Please contact a Google developer to investigate.

Trying again.

cc: @HansMuller
Another PR with failed Google testing, similar to the MaterialStateProperty.lerp PR.
Rebased this couple of times, I'm unable to find the cause.

It looks like this broke a few Scubas by changing line height or something like that.  Some text has moved vertically by <20px.  It looks acceptable to me, so I'm setting the Google tests to passing and merging this.
Sorry for the delay @TahaTesser!


Migrate to the new IPFS endpoint for uploads

Affected code:
https://github.com/kleros/t2cr-to-ipfs/blob/308c857226899ea540ac6eb467aee857c9a38c40/src/utils/ipfs-publish.ts#L17
Relates to https://github.com/kleros/builder-baseline/issues/12

Close by #8


Support of GeoJson Point for GeoPoint field

See https://github.com/opensearch-project/geospatial/issues/152
Signed-off-by: Heemin Kim<EMAIL_ADDRESS>Description
Supports GeoJson Point type in GeoPoint field
Issues Resolved
https://github.com/opensearch-project/geospatial/issues/152
Check List

[X] New functionality includes testing.

[X] All tests pass

[X] New functionality has been documented.

[X] New functionality has javadoc added

[X] Commits are signed per the DCO using --signoff
[X] Commit changes are listed out in CHANGELOG.md file (See: Changelog)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Will raise another commit with more unit tests in GeoPointFieldMapperTests.

Codecov Report

Merging #4597 (e6d6b8e) into main (bb47419) will increase coverage by 0.12%.
The diff coverage is 80.99%.

@@             Coverage Diff              @@
##               main    #4597      +/-   ##
============================================
+ Coverage     70.55%   70.67%   +0.12%
- Complexity    57432    57537     +105
============================================
  Files          4635     4635
  Lines        276645   276682      +37
  Branches      40489    40499      +10
============================================
+ Hits         195180   195549     +369
+ Misses        65094    64704     -390
- Partials      16371    16429      +58

Impacted Files
Coverage Δ

.../main/java/org/opensearch/common/geo/GeoPoint.java
87.75% <0.00%> (-0.91%)
:arrow_down:

.../main/java/org/opensearch/common/geo/GeoUtils.java
75.42% <81.31%> (-2.95%)
:arrow_down:

...index/mapper/AbstractPointGeometryFieldMapper.java
91.17% <85.71%> (+0.75%)
:arrow_up:

.../admin/cluster/reroute/ClusterRerouteResponse.java
60.00% <0.00%> (-40.00%)
:arrow_down:

...h/action/ingest/SimulateDocumentVerboseResult.java
60.71% <0.00%> (-39.29%)
:arrow_down:

...luster/routing/allocation/RoutingExplanations.java
62.06% <0.00%> (-37.94%)
:arrow_down:

...cluster/coordination/PublishClusterStateStats.java
33.33% <0.00%> (-37.51%)
:arrow_down:

...ensearch/client/indices/DetailAnalyzeResponse.java
20.54% <0.00%> (-34.25%)
:arrow_down:

...search/search/aggregations/pipeline/EwmaModel.java
24.44% <0.00%> (-33.34%)
:arrow_down:

...min/cluster/snapshots/get/GetSnapshotsRequest.java
52.63% <0.00%> (-31.58%)
:arrow_down:

... and 481 more

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

@VijayanB @nknize you're good with this?
@heemin32 you will have to resolve the CHANGELOG conflict pls

The failed test cases are not related with my cr. Also, it passed when I run the test in local box.

@reta Could you review this PR?

@heemin32 will need a manual backport to 2.x if you want this in a 2.x release

@heemin32 will need a manual backport to 2.x if you want this in a 2.x release

Created backport PR. https://github.com/opensearch-project/OpenSearch/pull/4842


Test: Automation shell

Test for #78497

[ ] Linux @weinand
[ ] macOS @RMacfarlane
[ ] Windows @sana-ajani

Complexity: 3
The new settings terminal.integrated.automation.shell.<platform> was introduced which when set will override the shell used by tasks and debug with whatever is specified. This will also cause shellArgs to get ignored.
To verify make sure it does as advertised for regular terminals (not touch), tasks and debug.

It seems the correct name of the setting is terminal.integrated.automationShell.<platform>.


Add search by arguments

This was regarding the issue #21

Hi @lilithxxx. There was a PR already opened for this feature. That has been merged. Thanks for contributing.


Setting enable_external_dns to true causes error

Welcome to Amazon EKS Blueprints!

[X] Yes, I've searched similar issues on GitHub and didn't find any.

Amazon EKS Blueprints Release version
4.4.0
What is your environment, configuration and the example used?
module "k8s_addons" {
source = "github.com/aws-ia/terraform-aws-eks-blueprints//modules/kubernetes-addons?ref=v4.4.0"
eks_cluster_id = data.aws_eks_cluster.region_main.id
K8s
enable_external_dns = true
}
What did you do and What did you see instead?
when setting the bool
enable_external_dns = true
Error: Either name or zone_id must be set
│
│   with module.k8s_addons.module.external_dns[0].data.aws_route53_zone.selected,
│   on .terraform/modules/k8s_addons/modules/kubernetes-addons/external-dns/data.tf line 1, in data "aws_route53_zone" "selected":
│    1: data "aws_route53_zone" "selected" {
Additional Information
No response

Hi @shawn-motley-dexcare, you will need to pass a value for eks_cluster_domain which is expected by the add-on as shown in this example.

closing with answer provided above

They're is actually a glitch here: the cluster_domain variable is used here: https://github.com/aws-ia/terraform-aws-eks-blueprints/blob/main/modules/kubernetes-addons/external-dns/main.tf#L78
Although it's supposed to be deprecated as it says here: https://github.com/aws-ia/terraform-aws-eks-blueprints/blob/main/modules/kubernetes-addons/external-dns/variables.tf#L20

this has been resolved here https://github.com/aws-ia/terraform-aws-eks-blueprints-addons/pull/101


fix(Headers): rename file extensions from .h to .hpp

BREAKING CHANGE: rename file extensions from .h to .hpp

:tada: This PR is included in version 7.0.0-rc.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 7.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Fix broken requirements.

Coverage decreased (-0.01%) to 91.366% when pulling db00bef2dad1253b92a5acdb5aeb74e8ef05a330 on akatsoulas:add-missing-deps into 1aee1fdefc6aef3ffcdd2bb7ffd5ceed08ab33cf on mozilla:master.


added back arrow keys as config option

Hello again!
This time I added back the arrow keys, should (hopefully) work with the current codebase this time.

sorry for taking a while :)

No worries, thank you for merging


[READ DESC] Front end design rework.

I admit, this code can be quite hacky at times when it comes to JavaScript. I was unable to test Oracle's recommendation feature locally on my machine due to some model issues.
Make sure to test this locally before putting it on the server, I'm not sure if the recommendations section will completely break from this. Also as of writing this, I noticed "Beatmap URL" should be changed to "Score URL", and the example should be changed to a proper score URL (not my profile, lol)

Recommendation fix was due to small typo in mod conversion from text to image. row['mod'] to beatmap['row']. Slightly changed descriptions.


Remote Rootfs backing images

Currently, to launch a qemu instance on a ciao compute node, you need to prepopulate the local image cache with backing images in /var/lib/ciao/images.  This is fine for demo purposes but isn't going to scale going forward.  We need a way for launcher to create a qemu rootfs image from a remote backing file.
Docker workloads do not have this problem as launcher can pull docker images from the docker hub.  However, we may chose to allow any solution we create for qemu images to work with docker images as well.

I've tested this on Single VM by mounting a cephfs over /var/lib/ciao/images.  The Single VM tests pass without any problems.  I won't submit the changes to Single VM to mount the ceph fs  as this slows things down considerably.  You need to wait for about 10 seconds after you start the ceph-demo container before you can mount the cephfs.


Add more flexibility in providing ffmpeg options when running locally

When running locally, it would be useful if there would be more flexibility in configuring the ffmpeg command.

Support 1-pass encoding
Support for not adding -b:v, -maxrate, -bufsize to the ffmpeg commandline

Fixed. The LocalPipelineConfiguration now has the properties singlePass and skipDefaultOptions.
If single pass set single pass transcoding will be used.
If skipDefaultOptions is set, -b:v, -maxrate, and -bufsize arguments will not be added to the ffmpeg commandline automatically


PAPP-30822: Updated requests modules

Manual Runs:
Current  -  https://cd.splunkdev.com/phantom-apps/app-tests/-/pipelines/13782106
Previous - https://cd.splunkdev.com/phantom-apps/app-tests/-/jobs/69944032
Cloud - https://cd.splunkdev.com/phantom-apps/app-tests/-/pipelines/13799716
Next - https://cd.splunkdev.com/phantom-apps/app-tests/-/pipelines/13799660

Due to an authorization issue in the pipeline
Ran all pipelines manually
hence,  merging it


properly quoted the default port parameter

This does not really add anything.
https://docs.puppetlabs.com/guides/style_guide.html#namespacing-variables
Looking at other modules Apache does not quote fixnums.
re-open if you feel strongly about this.


How to calculate new similarities_all.csv

After data processing and training process with 7.07 data, I don't know how to re-calculate the similarities_all.csv
P.S. I plotted the hero map successfully
Thanks a lot!

Dude! How are you doing? The patch has updated to 7.09 already. I plan to build some models to determine the advantage during different time slot, for example, 0-12 min 12-25min 25-40min >40min. I really need your kindly help about the similarities part! Thanks a lot

Hi! Please just tell me how to generate Similarity file, I could continue this project as possible as I could to follow the latest patch!

I'm sorry, but I really do not have the time to do this, even though I tried. As I previously said, I do not have the code anymore, but I can tell you what I did.
For each game in the dataset, I considered each hero to be a word, and each game to be a sentence. Therefore I did not use labels (who won the game). I then trained word2vec (skip-gram flavor) using tensorflow: word2vec example and when investigating similarities and clustering them using T-SNE, I discovered that even though the algorithm knew no relation between those heroes, it understood the heroes' roles. The similarity between two heroes is the cosine distance between two words' (Dota heroes) embeddings, so the shorter the distance, the more similar those heroes are (role-wise). I also used a special delimiter word, 'PAD' to split the radiant and dire teams.

@andreiapostoae Thank you, I will try my best to continue your work to update it to 7.11 following SteamDataBase! After you finish your Bachelor Thesis, We could discussion it


Feature request.

would it be nice if the pin settings can be configure in the admin panel for the following
LCD PIN
PUSH BUTTON
POWER LED INDICATOR
COIN LED INDICATOR
COIN PULSE
SET PIN (COINSLOT)
screen messages/slide for lcd
This will make the project more universal as it will work on all premade board for esp devices.

hi sir, here is the current premade board that I have. Was planning to test this, however it uses different set of pins.
D1 SDA (lcd)
D2 SCL (lcd)
D7 PUSH BUTTON
D5 SET PIN (COINSLOT)
D6 COIN PIN (COINSLOT)
D3 INSERT COIN INDICATOR (LED)
D4 POWER INDICATOR (LED)

@kuyajnonymous you can configure it in the admin config


Ephemeris source, reliability, and cost

In the ReadMe is says: “Chart data is loaded from a remote online ephemeris REST service.” Where does the ephemeris originate from (source)? This is more a question of curiosity.
A related and more significant question since I’m looking into using this as a module in an app, is: Is this service guaranteed to be offered in the future (reliability), and if so, for free (cost)? I would be unable to use this module if there’ll be a significant cost associated to it or if at some point the service is terminated.
Thank you!

ps: I’m excited for what you’re building!

Answer to #1 source: morphemeris, which uses swisseph.


docs: fix contrib/cni broken link

This change updates the install.md documentation to reference the new
cni directory location. This change also restores the previously
deleted README.md with updated instructions.
Signed-off-by: Tristan Cacqueray<EMAIL_ADDRESS>

bot, add author to whitelist

LGTM

@rh-atomic-bot r+


butcher-1.3

butcher-<IP_ADDRESS> (Lennart Spitzner<EMAIL_ADDRESS>@lspitzner) is out of bounds for:

[ ] brittany-<IP_ADDRESS> (==1.2.*). Lennart Spitzner<EMAIL_ADDRESS>@lspitzner. @lspitzner. Used by: library, executable, test-suite

Fixed in brittany-<IP_ADDRESS>.

Thanks!


git commands run in c9sdk location and not in workspace

When looking at the changes panel, the git log is shown from c9sdk's core repo, not from my workspace.
I guess it's that the vfs.spawn and execFile (why both?) don't always have their working dir set.

Same issue here, have you find out how can we bypass and set another path ?


Functions to get platform headers

As a library user, I want the library's HTTP client to include MIA Platform headers in his requests so that my micorservice can be integrated correctly into Mia Platform stack.
ACCEPTANCE TESTS

The HTTP client inclues MIA Platform headers in each of his requests.


Add "Rerun" button

Add a button on results page that triggers analysis of the current data all over again, exactly the way it happens after "uploading" a file on main page.
Related:

#91

This should now be resolved with #120


[stable/jenkins] Deprecate non-CasC values

Now Jenkins supports Configuration as Code, the configuration that is currently done by editing config.xml and other xml files should be marked as deprecated.
I suggest moving non-JCasC configuration to the bottom of the Readme table and as low as poss in the values. Mark them clearly as "deprecated" and add comments that these options will be removed shortly. Also warn the user in Notes.yaml output if these options are still being used.
Then after a grace period to allow users to wean themselves off the xml-munging methods, these options can be removed by a simplification-refactor.
Need to confirm these are covered by CasC, but I expect the settings to be deprecated will include:

Master.UseSecurity
Master.AuthorizationStrategy
Master.NumExecutors
Master.SecurityRealm
Master.ServicePort (? - might be a prereq for CasC)
Master.SlaveKubernetesNamespace
Master.SlaveListenerPort
Master.CSRF.DefaultCrumbIssuer
Master.ScriptApproval
Master.JenkinsAdminEmail
Master.JenkinsUrl (? - might be a prereq for CasC)
Master.HostName (? - might be a prereq for CasC)
Master.CLI
Master.CredentialsXmlSecret
Agent.*  - All agent values will be deprecated

I've deprecated as many of the items that are covered by CasC as I could find.  Let me know if there are any more.  The only way to reproduce the same configuration as the existing values.yaml is to enable Configuration as Code by default.   In the Readme and NOTES.txt I've added guidance for people who are still using deprecated values explaining how to use CasC instead.   No breaking changes, but some sections of config.yaml generated errors when I commented-out deprecated values, eg Agent so I had to insert {{- if .Values.Agent }} condition to prevent.
I've also:

Bumped CasC plugin to latest: from v1.5 to v1.6
Bumped git plugin from 3.9.1 to 3.9.3 to remove vulnerability warning

Let me know if anything is too opinionated or we we still need to use.  @torstenwalter \ @faizanahmad055, I'm not entirely sure the use cases for Master.AdditionalConfig, but confirm you can achieve what AdditionalConfig does using CasC.   Eg, it has keycloak support.
Hopefully this will put us on course to simplify this chart in due course.
I've not enabled auto-reload, but open to opinions on whether this feature is stable enough now.  We've had no issues with it.  Configuration as Code is part of the Jenkins project.  Without auto-reload, you have to reboot to make config changes, which isn't feasible except in a test environment.  Without auto-reload there's arguably little point in having Config as Code in this chart at all as it's merely "initial setup as code".  Not what the designers of CasC had in mind.  Your mileage may vary.  Anyway this is a bit outside the scope of this issue so I'll open a new one.

I had a look at the PR. It's a rather radical approach to reduce the tweaking options like that. For most of the ConfigScripts that should not be a problem as they are very short and copy and paste to customize them should not be to hard.
The only issue I see is with the kubernetes-slave as now people would have to copy the whole block even for small modification like using a different image or adjusting resource requests / limits.
Also the possibility to run agents in a separate namespace is gone.

Hi @torstenwalter.  I don't understand why changing a kubernetes-slave value (eg image or resources) requires "copying the whole block".
Re agents in a separate namespaces, are you referring to this:
jenkinsUrl: {{ template "jenkins.fullname" . }}:{{.Values.Master.ServicePort}}{{ default "" .Values.Master.JenkinsUriPrefix }}
Whereas currently config.yaml has:
{{- if .Values.Master.SlaveKubernetesNamespace }}
          <jenkinsUrl>http://{{ template "jenkins.fullname" . }}.{{.Release.Namespace}}:{{.Values.Master.ServicePort}}{{ default "" .Values.Master.JenkinsUriPrefix }}</jenkinsUrl>
...
{{- else }}
          <jenkinsUrl>http://{{ template "jenkins.fullname" . }}:{{.Values.Master.ServicePort}}{{ default "" .Values.Master.JenkinsUriPrefix }}</jenkinsUrl>
...
{{- end }}

Currently in this PR, we are only handling the else case.  Perhaps change values.yaml to:
jenkinsUrl: {{ template "jenkins.fullname" . }}:{{.Values.Master.ServicePort}}{{ default "" .Values.Master.JenkinsUriPrefix }}
# Replace the above line with the following if your master is in a different namespace from your agent:
# jenkinsUrl: http://{{ template "jenkins.fullname" . }}.{{.Release.Namespace}}:{{.Values.Master.ServicePort}}{{ default "" .Values.Master.JenkinsUriPrefix }}

?
Not as intuitive as simply defining SlaveKubernetesNamespace, but defining agents using CasC is far more flexible that the current config.xml-munging method, eg can have multiple agent templates, etc.  Also less maintenance of this chart is needed since users will be empowered to manage their own config.

What I meant was that the configuration looks like this:
   ConfigScripts:
      ...
      kubernetes-slave: |
        jenkins:
          clouds:
            - kubernetes:
                name: "jenkins-slave"
                serverUrl: https://kubernetes.default
                skipTlsVerify: true
                jenkinsUrl: {{ template "jenkins.fullname" . }}:{{.Values.Master.ServicePort}}{{ default "" .Values.Master.JenkinsUriPrefix }}
                connectTimeout: 0
                readTimeout: 0
                containerCapStr: 10
                maxRequestsPerHostStr: 64
                retentionTimeout: 5
                templates:
                  - name: "default"
                    label: "jenkins-slave"
                    idleMinutes: "1"
                    activeDeadlineSeconds: "120"
                    slaveConnectTimeout: "1000"
                    volumes:
                      - secretVolume:
                          secretName: mysecret
                          mountPath: "/var/myapp/mysecret"
                    containers:
                      - name: "jnlp"
                        image: "jenkinsci/jnlp-slave:3.27-1"
                        alwaysPullImage: true
                        workingDir: "/home/jenkins"
                        ttyEnabled: false
                        resourceRequestCpu: "200m"
                        resourceLimitCpu: "1000m"
                        resourceRequestMemory: "256Gi"
                        resourceLimitMemory: "2Gi"

Now if I want to change image: "jenkinsci/jnlp-slave:3.27-1"  then I need to copy kubernetes-slave: | ...  block to be able to modifiy the value. Or is there a way to override just that part?
That's no a big deal and definitely reduces maintenace effort, but on the other hand not everyone is experienced with casc. Which is a pleasure to work with compared to the xml alternative.
Would it do any harm if we use the url with the namespace suffix like below?
jenkinsUrl: {{ template "jenkins.fullname" . }}.{{.Release.Namespace}}:{{.Values.Master.ServicePort}}{{ default "" .Values.Master.JenkinsUriPrefix }}
In that case we could default to that and even add namespace: "{{ default .Release.Namespace .Values.Master.SlaveKubernetesNamespace }}"

to change image: "jenkinsci/jnlp-slave:3.27-1" then I need to copy kubernetes-slave: | ... block to be able to modifiy the value. Or is there a way to override just that part?

You just change the image value.  No need to copy the entire block.  I've just validated this in my test Jenkins.

namespace suffix?

I assumed if there's no namespace this would leave a trailing dot, but in fact, it suffixes .default, which is fine.  I've updated.  Nice suggestion.

'namespace: "{{ default .Release.Namespace .Values.Master.SlaveKubernetesNamespace }}"'

Not sure how this would work in the case of multiple agent templates in different namespaces.   Letting people manage their own config rather than trying to over-engineer this chart is a good general principle IMO.  Yes "not everyone is experienced with casc", but I think it's intuitive enough and well-documented.  I expect in 99% of cases, people will simply copy and paste from the demos, tweak a few values and be up-and-running.


Show also tags without release in releases sub-tab

Currently it is very confusing that tags that has no associated release does not show in Releases sub-tab of Releases tab.
Previous versions of Gitea did show them and also GitHub does it same way.

You can find them in Code -> Tags

Hm, github has a very unclear distinction between tags and releases, which has benefits and drawbacks..
Seing a chronologically sorted list of all tags+releases is a useful feature to check on updates to a project, so I'd agree to revert this change
For reference, this was introduced + discussed in https://github.com/go-gitea/gitea/pull/12096#issuecomment-695791548


Monitor the UDPST Test server execution rate

Is there any option available to monitor the Test server execution rate. How to identify the successful and failure rate from test server?
Please let us know if there any available metric to track the execution rate?

At this time there isn't a data source on the server providing statistics such as execution rate. One could enable verbose output (-v) and suppress sub-interval text (-s) so that the existing output logging provides test setup info...but this is obviously of limited usefulness in this context.
Internal server execution statistics (and I can think of several immediately) sounds like a worthwhile enhancement for a future release. We will certainly add it to our list. In this regard, what would be your preference for the mechanism and format for such a capability?


[debug] where to console.log human speaking after O.K. you can start talking!

Setup your demo code; the console log statements seem to indicate the system is working.
However there's no response after I start talking after this prompt:
O.K. you can start talking!
Where can I console log to catch my talking after O.K. you can start talking!?

Did you ever find a fix here @productboy

Update:
I changed the system message from this...
const SYSTEM_MESSAGE = 'You are a helpful and bubbly AI assistant who loves to chat about anything the user is interested about and is prepared to offer them facts. You have a penchant for dad jokes, owl jokes, and rickrolling – subtly. Always stay positive, but work in a joke when appropriate.';
to this...
const SYSTEM_MESSAGE = 'You are a helpful and friendly AI assistant who loves to chat about how to find homecare for people and their loved ones and is prepared to offer them tips for making a good match with a homecare provider. You have a penchant for favorite aunt wisdom. Always stay positive, but work in a warm, encouraging thought when appropriate.';
Then I called the number and it worked perfectly!


[EXP-27-227] add support for gnosis chain

add minimal support for gnosis chain since we are contemplating deploying there.

add to networks
update oracles

no registries to add yet

gdaddy safe:
https://gnosis-safe.io/app/gno:0x22eAe41c7Da367b9a15e942EB6227DF849Bb498C/balances
registry deployed:
0xe2F12ebBa58CAf63fcFc0e8ab5A61b145bBA3462
yvUSDC: 0x21d7b09bcf08f7b6b872bed56cb32416ae70bcc8

I'll see if I can figure it out

WIP https://github.com/yearn/yearn-exporter/pull/239
Will try to run some of these scripts later tonight

PR #239  has been closed and replaced with #289

Closing this as #289  has been merged


[NTR] B cell of anorectum

Note - please check that the term does not already exist (check OLS: https://www.ebi.ac.uk/ols/ontologies/cl)
Preferred term label:
B cell of anorectum
Synonyms
Definition (free text, please give PubMed ID in format PMID:XXXXXX)
B cell that is located in the anorectum. B cells are at the centre of the adaptive humoral immune system and are responsible for mediating the production of antigen-specific immunoglobulin (Ig). PMID: 31002794
Parent cell type term (check the hierarchy here https://www.ebi.ac.uk/ols/ontologies/cl):
B cell http://purl.obolibrary.org/obo/CL_0000236
What is the anatomical structure that the cell is a part of? Please check Uberon: https://www.ebi.ac.uk/ols/ontologies/uberon
UBERON:8410050 anorectum
Your nano-attribution (ORCID)
0000-0003-3440-1876
Any additional notes or concerns

Can't create as desired until Uberon releases 'anorectum'.

@sfexova
CL_0009050 (B cell of anorectum)


Would it be possible to switch to the MIT License ?

GPL is just so strict... :)
Or Apache 2.0 License maybe.

I prefer Apache 2.0.

OK! Give me some time to read the licenses first, but most likely I'll add one of the licenses (or even both).

+1 for apache2 of course :)

see http://choosealicense.com/ for example
For each license you also see what is Required, allowed and forbidden.
Don't know if you can combine them (not an expert either)

Apache V2 it is.
Dual licensing is not a problem. The licensees can chose themselves which license they want to use. However, Apache V2 is more liberal than GNU GPL in every respect, so keeping GPL is a moot point.

This would be awesome!

I've published RC2 of AngularFaces 2.1 under Apache License V2 (see http://search.maven.org/#search|gav|1|g%3A"de.beyondjava" AND a%3A"angularFaces-core"). However, Maven Central's license checker complains about a license incompatibility. I hope I can fix this.

Maven Central's license checker gives me a "go". Starting with AngularFaces 2.1, I put AngularFaces under the Apache License V2. You can download the current release candidate via Maven:
<dependency>
  <groupId>de.beyondjava</groupId>
  <artifactId>angularFaces-core</artifactId>
  <version>2.1.0-RC3</version>
</dependency>

or Gradle:
compile 'de.beyondjava:angularFaces-core:2.1.+'

or directly from Maven Central: http://search.maven.org/#search|ga|1|angularfaces.


Windows jdk12 build fails

Platform:
Windows
Architecture:
x86
The first attempt at a jdk12 Windows build fails early on accompanied by a worrying message:
WARNING: The toolchain version used is known to have issues. Please
consider using a supported version unless you know what you are doing.

See:
https://ci.adoptopenjdk.net/job/build-scripts/job/jobs/job/jdk12u/job/jdk12u-windows-x64-hotspot/5/console

    elif [ "${JAVA_TO_BUILD}" == "${JDK12_VERSION}" ]
    then
      TOOLCHAIN_VERSION="2017"
      export CONFIGURE_ARGS_FOR_ANY_PLATFORM="${CONFIGURE_ARGS_FOR_ANY_PLATFORM} --with-freemarker-jar=/cygdrive/c/openjdk/freemarker.jar --with-openssl=/cygdrive/c/progra~1/OpenSSL --enable-openssl-bundling"
    fi

Perhaps a bug in windows.sh - logic above looks fairly sane though.

Elsewhere in the output it says:
/cygdrive/c/cygwin64/tmp/openjdk-jdk12u-windows-x64-hotspot/workspace/build/src/build/windows-x86_64-server-release
using configure arguments '--verbose --with-version-opt=201903030947 --without-version-pre --with-version-build=30 --with-vendor-version-string=AdoptOpenJDK --with-vendor-url=https://adoptopenjdk.net/ --with-vendor-name=AdoptOpenJDK --with-vendor-vm-bug-url=https://github.com/AdoptOpenJDK/openjdk-build/issues --with-boot-jdk=/cygdrive/c/openjdk/jdk11 --disable-warnings-as-errors --with-toolchain-version=2013

Note the --with-toolchain-version=2013

Looks like TOOLCHAIN_VERSION=2017 is only set for openj9 builds.  TOOLCHAIN_VERSION=2013 is set for hotspot.

Similar failure on openj9 build though (with TOOLCHAIN_VERSION=2017): https://ci.adoptopenjdk.net/job/build-scripts/job/jobs/job/jdk12u/job/jdk12u-windows-x64-openj9/5/console
Building targets 'product-images legacy-jre-image test-image' in configuration 'windows-x86_64-server-release'
Compiling 8 files for BUILD_TOOLS_LANGTOOLS
Could not start process!  Failed with error 2: The system cannot find the file specified.

make[3]: *** [ToolsLangtools.gmk:40: /cygdrive/c/cygwin64/tmp/openjdk-jdk12u-windows-x64-openj9/workspace/build/src/build/windows-x86_64-server-release/buildtools/langtools_tools_classes/_the.BUILD_TOOLS_LANGTOOLS_batch] Error 126
make[2]: *** [make/Main.gmk:73: buildtools-langtools] Error 2

So, the main reason for the hotspot failure was a typo in the script file that doesn't pick the expected variables and even it was the build was not assigned to the correct build machine with VS2017 installed.
Those should be fixed by #940.

openj9 issue seems to be a completely different problem - maybe we should create another issue for that?

Raised https://github.com/AdoptOpenJDK/openjdk-build/issues/941 for the second issue.


[tool_crash] SocketException: Send failed, OS Error: No route to host, errno = 65

Command
flutter run --machine --start-paused -d 00008120-001C49281AF8C01E --devtools-server-address http://<IP_ADDRESS>:9100/ --target /Users/sharathchandra/Developer/flutter_app/lib/main.dart

Steps to Reproduce

...
...
...

Logs
SocketException: Send failed, OS Error: No route to host, errno = 65
#0      _NativeSocket.send (dart:io-patch/socket_patch.dart:1275:34)
#1      _RawDatagramSocket.send (dart:io-patch/socket_patch.dart:2590:15)
#2      MDnsClient.lookup (package:multicast_dns/multicast_dns.dart:219:22)
#3      MDnsVmServiceDiscovery._pollingVmService (package:flutter_tools/src/mdns_discovery.dart:232:66)
<asynchronous suspension>
#4      MDnsVmServiceDiscovery.firstMatchingVmService (package:flutter_tools/src/mdns_discovery.dart:188:56)
<asynchronous suspension>
#5      MDnsVmServiceDiscovery.getVMServiceUriForLaunch (package:flutter_tools/src/mdns_discovery.dart:433:50)
<asynchronous suspension>
#6      Future.any.onValue (dart:async/future.dart:628:5)
<asynchronous suspension>

[✓] Flutter (Channel stable, 3.24.5, on macOS 15.0.1 24A348 darwin-arm64, locale en-IN)
    • Flutter version 3.24.5 on channel stable at /Users/sharathchandra/dev/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision dec2ee5c1f (4 weeks ago), 2024-11-13 11:13:06 -0800
    • Engine revision a18df97ca5
    • Dart version 3.5.4
    • DevTools version 2.37.3

[✗] Android toolchain - develop for Android devices
    ✗ Unable to locate Android SDK.
      Install Android Studio from: https://developer.android.com/studio/index.html
      On first launch it will assist you in installing the Android SDK components.
      (or visit https://flutter.dev/to/macos-android-setup for detailed instructions).
      If the Android SDK has been installed to a custom location, please use
      `flutter config --android-sdk` to update to that location.

[✓] Xcode - develop for iOS and macOS (Xcode 16.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 16B40
    • CocoaPods version 1.16.2

[✗] Chrome - develop for the web (Cannot find Chrome executable at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome)
    ! Cannot find Chrome. Try setting CHROME_EXECUTABLE to a Chrome executable.

[!] Android Studio (not installed)
    • Android Studio not found; download from https://developer.android.com/studio/index.html
      (or visit https://flutter.dev/to/macos-android-setup for detailed instructions).

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

[✓] Connected device (3 available)
    • Sharath’s iPhone (mobile)       • 00008120-001C49281AF8C01E • ios          • iOS 18.1.1 22B91
    • macOS (desktop)                 • macos                     • darwin-arm64 • macOS 15.0.1 24A348 darwin-arm64
    • Mac Designed for iPad (desktop) • mac-designed-for-ipad     • darwin       • macOS 15.0.1 24A348 darwin-arm64

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

! Doctor found issues in 3 categories.

Flutter Application Metadata
Type: malformed
Version: 1.0.0+1
Material: true
Android X: false
Module: false
Plugin: false
Android package: null
iOS bundle identifier: null
Creation channel: null
Creation framework version: null

Closing this as a duplicate of https://github.com/flutter/flutter/issues/150131


Pouch-db does not return info when trying to get credentials for a new user

https://github.com/verida/verida-js/blob/bc29ebd6b887b505677e5d411a6e9b40c85ac7d9/packages/client-ts/src/context/engines/verida/database/db-encrypted.ts#L92
Current Behaviour:

Create a new account on vault
Go to data > credentials
Notice that the app shows white screen and does nothing

Finding is that, db Initialisation for credential is going to a time out state where the above line on client-ts does not return back any information and is stuck in the await request.

@tahpot please refer to this above issue. I will try resolving this.

@ram-verida is this on both Android and iOS?
I've seen similar unusual behaviour in Android.

@tahpot this finding has been made on Android only so far, yet to verify for iOS, will update.

@ram-verida This issue also exists on Android:  https://github.com/verida/vault-mobile/issues/202
I think Android has an underlying issue with CouchDB and I suspect this is the root cause of the Android issues.
We may need a call with @pkhien95 to help onboard you with more background information.
If this is an Android issue, this issue should probably be in the vault repo.

@ram-verida Should we close this issue assuming it's associated with the Android issues?

I'm closing this

Sorry for the delayed response, its ok that this is closed but an important thread for archives until we have truly resolved android bugs.


refactor: incremental update of inlay hints and more options

Previously, the inlay hints are dropped and requested as a whole on every update of text. This will cause apparent lag in large file, and tsserver is even less responsive as it's busy dealing with the large amounts of inlay hints. By requesting and updating the hints incrementally, the performance will be much better.
Three extra options are exposed to help customization.

inlay_hints_priority is used to resolve the conflict with other extmarks set on the same line.
inlay_hints_throttle configures the throttle time of requests as the text may change much frequently.
inlay_hints_format allow user to customize the highlight and text of different kind of hints, because the original text format of hint is confusing when messed up on the single line (param:  and : return).

I haven't done a ton of JavaScript work lately but this seems to be working well. Let me know what you think about the config issue, but other than that I think this is okay to merge.

Sorry for late response, previously I'm busy with personal stuff but these days I might have enough time to pick it up.
I think we should not rely on updatetime and bring the throttle time option back since there is an issue for CursorHold: https://neovim.discourse.group/t/psa-dont-rely-on-cursorhold-yet/1813. Also think setting default to 150ms is more reasonable since nvim now use it as default debounce time for lsp servers: https://github.com/neovim/neovim/pull/16908
And the another thing worth notice is that the changes introduce in https://github.com/jose-elias-alvarez/nvim-lsp-ts-utils/commit/4655fbcb85b55bc34c80aa3e9400ca069b3f1363 makes the throttle behavior actually becomes debounced since the on_lines events are accumulated and only the last response will be accepted and handled. I'm not sure whether it will be alright or better be reverted.

Sorry for late response, previously I'm busy with personal stuff but these days I might have enough time to pick it up.
I think we should not rely on updatetime and bring the throttle time option back since there is an issue for CursorHold: https://neovim.discourse.group/t/psa-dont-rely-on-cursorhold-yet/1813. Also I think setting default to 150ms is more reasonable since nvim now use it as default debounce time for lsp servers: neovim/neovim#16908
And the another thing worth notice is that the changes introduce in 4655fbc makes the throttle behavior actually becomes debounced since the on_lines events are accumulated and only the last response will be accepted and handled. I'm not sure whether it will be alright or better be reverted.

Yep, definitely agreed about updatetime and the default setting. I don't think debouncing changes is a big deal (it might reduce flickering, actually) so I'm happy with the way things are working now.

Everything looks good to me, and I've been using this without issues. Let me know when you think it's ready and I'll merge it.

Sure, I think there is no other issue.

Great – thank you for your work on this!


[SR-789] Remove localization of NSNumber.description

This is for SR-789 which reports that NSNumber.description is providing a localised string whereas it didn't do so previously.
In order to be behaviour compatible I've made a slight modification to the existing implementation to use a number formatter style of kCFNumberFormatterNoStyle rather than kCFNumberFormatterDecimalStyle.
At the same time I've done some refactoring to improve the delegation between description and descriptionWithLocale() as suggested in PR #277 and added tests for both description and descriptionWithLocale()

I will merge this and we can follow up on @phausler's suggestion.


Add possible return values of the string

I would love to have the information about possible values being returned - for example "Windows.Xbox", "Windows.Mobile", "Windows.Desktop" and other potentials, and under what circumstances they are returned.

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

ID: 4c62c532-03ab-7ceb-f17c-2071b848e696
Version Independent ID: 40064398-4737-7eeb-be33-5ba52b0acf3c
Content: AnalyticsVersionInfo.DeviceFamily Property (Windows.System.Profile) - Windows UWP applications
Content Source: winrt-api-build/xml/Windows.System.Profile/AnalyticsVersionInfo.xml
Product: uwp
Technology: winrt-reference
GitHub Login: @hickeys
Microsoft Alias: hickeys

Thanks for submitting your feedback and we apologize for the delayed response. I see that updates have not been made since the creation of this Issue, so I went ahead and created an internal work item to engage with our engineering folks, find the answers, and resolve this doc bug. The timeline for this may vary, so I will go ahead and close this and circle back when the changes are live.
We have moved to a new feedback system, and I want to ensure that we don't lose access to the feedback in our open issues. Though the Issue will be marked as Closed, we will monitor any replies that come afterwards so treat this like an open Issue. Many thanks.


Fix types

Mostly just replaced Boolean constructor with boolean type

While working to understand the ajv code, I wrote a lot more type definitions along these lines: https://github.com/forivall/ajv/tree/ts-jsdoc

does not apply to v7


fix: Update get-dependent-packages.sh

Add architecture argument to get-dependent-packages.sh
and download packages based on the architecture of the
remote node.
Signed-off-by: Kothapally Madhu Pavan<EMAIL_ADDRESS>

tested 3-27


Possibility to decode an extrinsic.

Should users be able to decode the xcm message transfer that they construct?

?

This is completed.


module 'treesj' not found

Hey, how do you do guys?
I'm trying to use the package in the following way:
use({
  'Wansmer/treesj',
  requires = { 'nvim-treesitter' },
  commit = '7be9ebd',
  config = function()
    require('treesj').setup({
      notify = true,
      dot_repeat = true,
      use_default_keymaps = true,
      check_syntax_error = true,
      max_join_length = 120,
      cursor_behavior = 'hold',
    })
  end,
})

But every time I start Neovim, I encounter this error:
packer.nvim: Error running setup for treesj: [string "..."]:0: module 'treesj' not found:
^Ino field package.preload['treesj']
^Ino file './treesj.lua'
^Ino file '/__w/neovim/neovim/.deps/usr/share/luajit-2.1.0-beta3/treesj.lua'
^Ino file '/usr/local/share/lua/5.1/treesj.lua'
^Ino file '/usr/local/share/lua/5.1/treesj/init.lua'
^Ino file '/__w/neovim/neovim/.deps/usr/share/lua/5.1/treesj.lua'
^Ino file '/__w/neovim/neovim/.deps/usr/share/lua/5.1/treesj/init.lua'
^Ino file '/home/adrianmiranda/.cache/nvim/packer_hererocks/2.1.0-beta3/share/lua/5.1/treesj.lua'
^Ino file '/home/adrianmiranda/.cache/nvim/packer_hererocks/2.1.0-beta3/share/lua/5.1/treesj/init.lua'
^Ino file '/home/adrianmiranda/.cache/nvim/packer_hererocks/2.1.0-beta3/lib/luarocks/rocks-5.1/treesj.lua'
^Ino file '/home/adrianmiranda/.cache/nvim/packer_hererocks/2.1.0-beta3/lib/luarocks/rocks-5.1/treesj/init.lua'
^Ino file './treesj.so'
^Ino file '/usr/local/lib/lua/5.1/treesj.so'
^Ino file '/__w/neovim/neovim/.deps/usr/lib/lua/5.1/treesj.so'
^Ino file '/usr/local/lib/lua/5.1/loadall.so'
^Ino file '/home/adrianmiranda/.cache/nvim/packer_hererocks/2.1.0-beta3/lib/lua/5.1/treesj.so'

Is there anything wrong with my configuration?

Hi! It looks like you don't have treesj installed. Try using :PackerInstall<Cr> or :PackerSync<Cr> - that should be help.
I'm closing the issue as this is it's not a treesj bug.

Actually, I ran it, and immediately after running it, I encountered this error.


enact deprecation for not-a-dictionary

resolves #3897
Description
Raise exceptions on the dunder methods of iter and len for all superclasses of FakeAPIObject.  Added new exception.  Removed deprecation associated with this.
Checklist

[x] I have signed the CLA
[ ] I have run this code in development and it appears to resolve the stated issue
[ ] This PR includes tests, or tests are not required/relevant for this PR
[ ] I have updated the CHANGELOG.md and added information about my change to the "dbt next" section.

Not sure where/how to test this.  Any pointers?


Support Slack projectChatLinkIcon

Currently, if projectChatLink is set in theme.config.js, the default icon is Discord.
Would love to see Slack support since Slack is very popular as well

This can be easily done:
import Slack from '@geist-ui/icons/slack'

export default {
  projectChatLink: "...",
  projectChatLinkIcon: <Slack />,


Update tested versions and PyPi classifiers.

Remove versions of Python (>3.5) and Django (everything except 2.2 and 3.0) that are no longer supported upstream from automated testing.
Update classifiers in setup.py to remove old Python versions.

Codecov Report

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

@@           Coverage Diff           @@
##           master      #92   +/-   ##
=======================================
  Coverage   77.35%   77.35%
=======================================
  Files           1        1
  Lines         106      106
  Branches       30       30
=======================================
  Hits           82       82
  Misses         19       19
  Partials        5        5

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 aeb46bd...c73335f. Read the comment docs.


Repeated operations and memory

I submitted this in the wrong place. Sorry.

I'm sorry -- this issue is in the wrong place. I'm going to close it and submit it in the correct place!

No problem. Answered here: https://github.com/jcupitt/libvips/issues/556


Remove JSONAssert from public interface

JSONAssert brings some nice options to customize how two objects are being compared. However, the API is a bit awkward and doesn't fit into our fluent/builder style DSL. As such, we should remove the public visibility of the JSONComparator class in JsonSnapshot and provide a nice fluent wrapper around it.

A better API has been implemented in 1.2.0. It can be used like this:
snapshot.assertThat(myself)
                .as(JsonSnapshot.withDefaultObjectMapper()
                        .withComparisonRules(rules -> rules
                                .jsonPathAt("address.city").ignore()
                                .jsonPathAt("birthdate").mustMatch(Pattern.compile("\\d{4}-\\d{2}-\\d{2}"))))
                .matchesSnapshotStructure();

Leaving this ticket open for now to remove the old method in next minor version.


Add the concept of "collection" to the compiler and remove the concept of "in-order AST nodes"

The Problem
Currently, the Sway compiler relies heavily on the concept of the AST nodes being ordered “correctly” in order to perform successful compilation. Out of this comes the additional requirement that dependencies be ordered correctly in a Sway project. The burden of deciding the “correct AST node ordering” is not placed on the user, but on the compiler. Currently, the compiler performs a pass before type checking to determine the correct order in which to type check the AST nodes.
Mathematically, this places restrictions on the dependency structure of a Sway project. Currently, Sway projects must be directed acyclic graphs (a tree).
This creates fundamental issues:

Mutually recursive dependencies are not currently supported, and would not ever be able to be supported without extensive special-casing.
Mutually recursive AST nodes within the same file are not currently supported, and would not ever be able to be supported without extensive special-casing.
This inhibits the introduction of recursive functions, recursive trait methods, recursive struct and enum declarations, and more.
If a user forgets to import a dependency, it is not possible to suggest an import to them without extensive special-casing.

The Solution
The order of AST nodes should not be a factor of importance inside the compiler. I propose that we shift the way that the compiler thinks about AST node ordering, and eliminate the concept of “correct AST node ordering.” This will mean that the order of AST nodes during type-checking will no longer have any effect on the output of the compiler.
This will open up Sway to a new domain of applications, as it would allow Sway projects to be directed cyclic graphs. (two direct mutually recursive dependencies are considered a 1-hop cycle)
I propose these new high-level compiler steps:

Parse all files in a Sway project.
Perform a “collection pass” on the untyped parsed AST and gather information about all of the AST nodes, without type checking. This will generate a “collection environment” containing key information about the AST nodes.
Perform type checking using the “collection environment,” allowing every AST node to have knowledge of the existence of every other AST node. This step will require us to introduce the concepts of a “declaration engine” that will operate similarly to the existing type engine. The declaration engine will work with the collection context to allow AST nodes to undergo type checking, without the need for dummy declarations, multiple passes, or substitutions. Additionally, the introduction of the declaration engine supports the introduction of other Sway features which are currently open on GitHub.
Generate IR and do codegen

The Roadmap
The implementation steps are (roughly) as follows in order:

[x] #1578
[x] #1810
[x] #1692
[x] #2636
[x] #1693
[ ] #1823
[ ] remove dead code (the node dependencies code, etc)

The original proposal is here: https://docs.google.com/document/d/1kwOk08-Ot7n2RCYCWNNtFCfQAxtlYN-DFHk2sxK139E/edit#

@tritao I was doing a little clean-up and realized this issue is marked as done in compiler board but still open. Is this issue still relevant?

#1823 discusses the very first IR generation that didn't have function calls but was inlining everything and proposes what we do have now, in the IR and in ASM generation, calls and jumps respectively. So I've closed it.


NormSohl from OPS portal: Merge from master to live

:warning: Validation status: warnings

File
Status
Preview URL
Details

msteams-platform/whats-new.md
:warning:Warning

Details

msteams-platform/resources/dev-preview/developer-preview-features.md
:warning:Warning

Details

msteams-platform/resources/schema/manifest-schema-dev-preview.md
:warning:Warning

Details

msteams-platform/includes/get-started/step1-prepare-for-dev.md
:warning:Warning

Details

msteams-platform/includes/get-started/step1-intro.md
:warning:Warning

Details

msteams-platform/whats-new.md

Line 15: [Warning] Invalid file link:(~/resources/schema/manifest-schema-dev-preview).
Line 15: [Warning] Invalid file link:(~/resources/dev-preview/developer-preview-features).
Line 16: [Warning] Invalid file link:(~/resources/dev-preview/developer-preview-intro).
Line 16: [Warning] Invalid file link:(~/concepts/cards/cards-reference#adaptive-card-supported-in-developer-preview-only).
Line 16: [Warning] Invalid file link:(~/concepts/cards/cards-actions#adaptive-card-actions-supported-in-developer-preview-only).
Line 17: [Warning] Invalid file link:(~/resources/dev-preview/developer-preview-intro).
Line 17: [Warning] Invalid file link:(~/concepts/bots/bots-files).
Line 17: [Warning] Invalid file link:(~/resources/dev-preview/developer-preview-features).
Line 18: [Warning] Invalid file link:(~/publishing/office-store-guidance).
Line 18: [Warning] Invalid file link:(~/publishing/office-store-checklist).
Line 19: [Warning] Invalid file link:(~/concepts/cards/cards-actions).
Line 20: [Warning] Invalid file link:(~/whats-new).
Line 21: [Warning] Invalid file link:(~/concepts/authentication/auth-tab-AAD).
Line 21: [Warning] Invalid file link:(~/concepts/authentication/auth-flow-tab).
Line 22: [Warning] Invalid file link:(~/resources/design/framework/command-box).
Line 23: [Warning] Invalid file link:(~/concepts/bots/bots-notification-only).
Line 24: [Warning] Invalid file link:(~/concepts/bots/bot-conversations/bots-conv-proactive).
Line 25: [Warning] Invalid file link:(~/concepts/cards/cards).
Line 25: [Warning] Invalid file link:(~/concepts/cards/cards-format).
Line 25: [Warning] Invalid file link:(~/concepts/cards/cards-reference).
Line 25: [Warning] Invalid file link:(~/concepts/cards/cards-actions).
Line 26: [Warning] Invalid file link:(~/get-started/app-studio-component-library).
Line 26: [Warning] Invalid file link:(~/get-started/get-started-app-studio).
Line 27: [Warning] Invalid file link:(~/concepts/bots/bots-context).
Line 28: [Warning] Invalid file link:(~/get-started/get-started-dotnet).
Line 33: [Warning] Invalid file link:(~/feedback).

msteams-platform/resources/dev-preview/developer-preview-features.md

Line 11: [Warning] Invalid file link:(~/concepts/bots/bots-files).
Line 12: [Warning] Invalid file link:(~/concepts/cards/cards-reference#adaptive-card-supported-in-developer-preview-only).
Line 12: [Warning] Invalid file link:(~/concepts/cards/cards-actions#adaptive-card-actions-supported-in-developer-preview-only).
Line 26: [Warning] Invalid file link:(~/resources/schema/manifest-schema-dev-preview).
Line 58: [Warning] Invalid file link:(~/concepts/bots/bots-notifications).
Line 62: [Warning] Invalid file link:(~/concepts/bots/bot-conversations/bots-conv-channel).
Line 83: [Warning] Invalid file link:(~/resources/schema/manifest-schema-dev-preview).
Line 85: [Warning] Invalid file link:(~/resources/general/developer-preview).

msteams-platform/resources/schema/manifest-schema-dev-preview.md

Line 10: [Warning] Invalid file link:(~/resources/dev-preview/developer-preview-intro).
Line 11: [Warning] Invalid file link:(~/resources/schema/manifest-schema).
Line 20: [Warning] Invalid file link:(~/resources/dev-preview/developer-preview-features).
Line 171: [Warning] Invalid file link:(~/concepts/bots/bots-create).
Line 223: [Warning] Invalid file link:(~/concepts/apps/apps-package#icons).
Line 285: [Warning] Invalid file link:(~/concepts/bots/bots-menus).

msteams-platform/includes/get-started/step1-prepare-for-dev.md

Line 10: [Warning] Invalid file link:(~/get-started/get-started-tenant).
Line 10: [Warning] Invalid file link:(~/get-started/get-started-tenant).
Line 15: [Warning] Invalid file link:(~/get-started/get-started-tenant).
Line 15: [Warning] Invalid file link:(~/get-started/get-started-tenant).

msteams-platform/includes/get-started/step1-intro.md

Line 3: [Warning] Invalid file link:(~/concepts/apps/apps-overview).
Line 3: [Warning] Invalid file link:(~/concepts/apps/apps-overview).
Line 5: [Warning] Invalid file link:(~/get-started/get-started-app-studio).
Line 5: [Warning] Invalid file link:(~/get-started/get-started-app-studio).

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.


Add RADIUS Authentication Support

We have a requirement to add RADIUS authentication servers/policy support.
Current Behavior
Creating and binding RADIUS server/policies is not currently possible.
Possible Solution
I have an implementation that's ready to go but thought I best raise an issue to see if there's anything in particular that's needed before submitting the PR!

Looks good to me @iainbrighton! Go ahead and put in the PR.

@devblackops Incoming.. 😃

This was implemented in PR #91


Lab: create Aurora cluster based on snapshot

Create A4L VPC
Aurora v2.07.1
Create 1 replica reader
Test access from bastion host

Use snapshot from #50

Will test with Aurora Serverless instead #52


[WIP] Test PR for making Row/ColDefinition styleable.

Just testing stuff out and im getting asserts on ControlCatalog... not sure how it happens though.

Didnt work, will try again sometime


Update readme

Make readme more attractive by adding emojis, animations, GIFs, and much more interactive graphics to make it more exciting to read.

Yeah Sure.

Sure

Do I add steps for contribution and table of content

yes please.

hi @ayushsarode this issue has been assigned to you for a week now, which is barring other contributors from contributing in it. Please update your status within 24 - hours of this message.

I'll make the PR tomorrow!
Sorry for Inconvenience.

Hi @ayushsarode the contribution of yours has been exceptionally delayed, so we have decided to remove you from this PR after 12 hours of this message.

Sure!
I'm sorry for Inconvenience, I'm busy in my college studies.

We understand your position and we really wanted you to contribute but as informed you before that it is barring other contributors from contributing and we feel really sad to turn them down so unfortunately, we are forced to do so. If you did not make PR in the given time this issue will be assigned to someone else.
Thank You.

hey @ashut0sh75
I can work on this issue. Please assign it to me.
Thanks!

Hey @ashut0sh75 you can assign the issue to @Vani177

Hi @Vani177 we are happy to have you. We are assigning you this issue.

Hi @Vani177 we are happy to have you. We are assigning you this issue.

Glad to contribute~

@Vani177  are you a hacktoberfest contributor ?? Would you like if we add Hactoberfest label on it ??
Please star our repository.

yeah sure!

i've completed my hactoberfest prs already btw, so it's up to you, there' particularly no need of the label

i've completed my hactoberfest prs already btw, so it's up to you, there's particularly no need of the label

ohh very well then

We are closing this issue. Thank you for your contribution @Vani177


Replace example window with dedicated example editor in the usd_qtpy lib

As a replacement to #8 this exposes the example editor to command-line to open with a USD file.
In the form of:
python -m usd_qtpy "/path/to/file.usd"

There's also some minor restructuring of a base class which isn't necessarily crucial to the change but had to be done either way, so I included it here.

@Sasbom would this work for you?
@MustafaJafar would you say this is an improvement on making it easier to get started? or harder? Since you ran the earlier example window before I wonder if that was easier. Note how the updated readme still provides examples to run directly from e.g. Python code as well. Does that suffice?

python -m usd_qtpy "/path/to/file.usd"

Looks good!


CAD-2050: fix leak/concurrent problem in TF.

description
Thomas Winant found a space leak in our recent addition of eliding mechanism for trace forwarder error messaging: https://github.com/input-output-hk/iohk-monitoring-framework/pull/590#commitcomment-43196801
checklist

[x] compiles (cabal v2-build or stack build)
[ ] tests run successfully (cabal v2-test or stack test)
[ ] documentation added
[ ] link to an issue
[ ] add milestone (the current sprint)

bors r+


added hyperlinks

code is clean. One UI suggestion: change the color of your nav-brand to background color. It is not visible in the current blue color. Good to merge.

Yeah, will add it to my CL's reset file asap 👍


Allow for playlists to be shown in SearchResults

This is probably more complicated than it seems.
First, the architecture of Streamus needs to change a bit. SearchResult has a reference to a Song object right now. Instead, it'll need to be able to have a reference to a Song or a Playlist.
It probably doesn't make sense to pull all of the Playlist's Songs when just showing the playlist in SearchResults. That means we'll have to pull songs once the user has shown intent to use that playlist. That's an async action where as currently all the data for a song is known making it a synchronous action.
Furthermore, one can drag-and-drop a playlist object into the stream. I suspect it would be best to convert that playlist object into a bunch of songs at that point instead of just showing it as one, playlist item. That'll have its own set of challenges.

This could be done progressively.
Step 1. Show playlists in the search results, but dragging it to the stream or saving to a playlist does nothing, all you can do is right-click and copy URL.
Step 2. Implement 'expanding' of the playlist into its songs when adding to the stream / saving to a playlist. This should be easy actually because this function already exists in the create playlist from URL dialog.
Actually now that I think about it step 2 should be pretty trivial anyway.

It will be cool if Streamus will offer the option to queue YouTube playlists without transforming the playlists in separate songs. So if a playlist is updated on the YouTube side, the user will still listen to all the new songs added in the playlist. And it would be even better if the user could mix songs and YouTube playlists in the Now Playing (Queue) section.

@michael-jerred I agree that expanding it is the way to go, just so much to do! I have barely gotten to code at all in a week :(
@silviubogan Your feature request is a duplicate of #16 :)

Closing in favor of #21


Automatic compaction ignores maxColumnsToMerge from the TuningConfig setting

Affected Version
The setting maxColumnsToMerge was introduced in version 0.21.0 but the problem was identified on version 0.22.1.
Description
Configuring a DataSource with an automatic compaction that includes maxColumnsToMerge will make the automatic compaction to ignore this field. This field is important to reduce and optimize memory usage, particularly for data sources with many dimensions.
The bug has been identified on the following classes:

server/src/main/java/org/apache/druid/client/indexing/ClientCompactionTaskQueryTuningConfig.java
server/src/main/java/org/apache/druid/server/coordinator/UserCompactionTaskQueryTuningConfig.java

Both of these classes are missing the maxColumnsToMerge property.
A fix was implemented to add the missing property to these two classes in our environment (fork). The fix was tested succesfully.
Pull request with the fix: https://github.com/apache/druid/pull/12551

The fix for this issue has been merged with the associated pull request.


Improvement : display rclone command line

It would be great to see the generated command line especially for Task.
Maybe it is already there but I did not see where.
Great effort!

when job is running next to cancel button there is button to copy rclone command to clipboard

Thx ... totally missed that!