Write up PD Community Guidelines

Rough draft:

We respect the WotC "Commitment to Each Other".

Put fun first. Keep it friendly, respectful and fun.
Welcome everyone. Magic Online first-timers, people coming back to the game for the first time in a long time, children, j_meka … everyone!
Respect your environment. This means no "BM" in Magic Online chat. No unpleasantness in Discord. No going to Discord to talk about how terrible your last opponent was (they are probably there reading what you are saying!)
Follow the rules. Although Magic Online enforces a lot of the rules for us there are still bugs and other issues that require players to cooperate to resolve. If in doubt talk to the Tournament Organizer or a Discord moderator.

Do we need more than the four points from the WotC code? Something about no ban chat? :D

"BM" is too heavy on the jargon.  Just say "Bad Manners"

I'll spell it out explicitly and include some examples. I thought BM was a more widely known phrase than it is but you're the second one to call it out :D

b853e0bf.


Update cats-mtl to 1.4.0

About this PR
📦 Updates org.typelevel:cats-mtl from 1.3.1 to 1.4.0
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

🔍 Files still referring to the old version number
The following files still refer to the old version number (1.3.1).
You might want to review and update them manually.
website/package-lock.json
website/yarn.lock

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.typelevel", artifactId = "cats-mtl" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.typelevel", artifactId = "cats-mtl" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, version-scheme:early-semver, old-version-remains, commit-count:1

Superseded by #314.


Configurable throttling for iroh-net relays

Some people have offered to run "altruistic relays" for iroh-net. However, in order to run such an altruistic relay, you would need a way to throttle all connections that don't switch to direct after some time, otherwise there would be a chance that the operator of an altruistic relay would end up with a huge bandwidth bill.
This is something we ourselves might need at some point, e.g. if a heavy app that builds on iroh-net goes viral. So it might be a good idea to build this in proactively.

For context (albeit out of the scope of this issue):
An altruistic relay would be discoverable from a Mainline hardcoded topic (like content-discovery example).
By default, Mainline returns ~20 random addresses so you get the load-balancing you need.
But that topic might be spammed, so the ultimate solution might be a separate DHT (using the rust's Mainline library) so that only "good" relays are kept in the routing table, so nodes would test each other's for holepunching, and not just responsiveness for ping requests.


On boarding scene - The emotes do not have SFX

Explorer Alpha build versions:
0.25.26
Issue Description:
During the Bazier's quest , you need to click on certain emotes in order to continue. The emotes do not have SFX when clicking on them.
STR:

Open v0.25.26
Go to /world cryptonico.dcl.eth.
Go to Bezier's quest
Make three emotes
Validate that none of those emotes have sounds

Expected behaviour:
The emotes must have sounds when activating them
Current behaviour:
The emotes do not have SFX when activating them during the quest
Evidence:
https://github.com/user-attachments/assets/bb14ce90-bc9f-425a-b3d8-6703a16df375
Additional Notes:
Windows 11 pro
16 GB RAM
📂 Scene Repository
https://github.com/agustincarriso/dcl-sdk7-onboarding

Duplicated from : https://github.com/decentraland/unity-explorer/issues/1920


create_zello_connection: exception: module 'websocket' has no attribute 'create_connection'

I am not sure where I am going wrong. I have followed the steps, but clearly I have either changed something or I have missed something. Attached is the what is returning when I run python3 zellostream.py
pi@pi:~/zellostream $ python3 zellostream.py ALSA lib pcm.c:2660:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.front ALSA lib pcm.c:2660:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.rear ALSA lib pcm.c:2660:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.center_lfe ALSA lib pcm.c:2660:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.side ALSA lib pcm.c:2660:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.surround21 ALSA lib pcm.c:2660:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.surround21 ALSA lib pcm.c:2660:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.surround40 ALSA lib pcm.c:2660:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.surround41 ALSA lib pcm.c:2660:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.surround50 ALSA lib pcm.c:2660:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.surround51 ALSA lib pcm.c:2660:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.surround71 ALSA lib pcm.c:2660:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.iec958 ALSA lib pcm.c:2660:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.iec958 ALSA lib pcm.c:2660:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.iec958 ALSA lib pcm.c:2660:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.modem ALSA lib pcm.c:2660:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.modem ALSA lib pcm.c:2660:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.phoneline ALSA lib pcm.c:2660:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.phoneline Cannot connect to server socket err = No such file or directory Cannot connect to server request channel jack server is not running or cannot be started JackShmReadWritePtr::~JackShmReadWritePtr - Init not done for -1, skipping unlock JackShmReadWritePtr::~JackShmReadWritePtr - Init not done for -1, skipping unlock Cannot connect to server socket err = No such file or directory Cannot connect to server request channel jack server is not running or cannot be started JackShmReadWritePtr::~JackShmReadWritePtr - Init not done for -1, skipping unlock JackShmReadWritePtr::~JackShmReadWritePtr - Init not done for -1, skipping unlock ALSA lib pcm_oss.c:377:(_snd_pcm_oss_open) Unknown field port ALSA lib pcm_oss.c:377:(_snd_pcm_oss_open) Unknown field port ALSA lib pcm_a52.c:823:(_snd_pcm_a52_open) a52 is only for playback ALSA lib conf.c:5200:(snd_config_expand) Unknown parameters {AES0 0x6 AES1 0x82 AES2 0x0 AES3 0x2  CARD 1} ALSA lib pcm.c:2660:(snd_pcm_open_noupdate) Unknown PCM iec958:{AES0 0x6 AES1 0x82 AES2 0x0 AES3 0x2  CARD 1} ALSA lib pcm_usb_stream.c:486:(_snd_pcm_usb_stream_open) Invalid type for card ALSA lib pcm_usb_stream.c:486:(_snd_pcm_usb_stream_open) Invalid type for card ALSA lib pcm_dmix.c:1075:(snd_pcm_dmix_open) unable to open slave Cannot connect to server socket err = No such file or directory Cannot connect to server request channel jack server is not running or cannot be started JackShmReadWritePtr::~JackShmReadWritePtr - Init not done for -1, skipping unlock JackShmReadWritePtr::~JackShmReadWritePtr - Init not done for -1, skipping unlock 2023-01-03 20:43:38,174 E start_audio: input_pulse_name is Virtual_Sink.monitor 2023-01-03 20:43:38,206 E start_audio: output_pulse_name is Virtual_Sink 2023-01-03 20:43:40,245 E create_zello_connection: exception: module 'websocket' has no attribute 'create_connection' 2023-01-03 20:43:40,246 W main: cannot establish connection for incoming 2023-01-03 20:43:41,306 E create_zello_connection: exception: module 'websocket' has no attribute 'create_connection' 2023-01-03 20:43:41,306 W main: cannot establish connection for incoming 2023-01-03 20:43:42,369 E create_zello_connection: exception: module 'websocket' has no attribute 'create_connection' 2023-01-03 20:43:42,369 W main: cannot establish connection for incoming 2023-01-03 20:43:43,410 E create_zello_connection: exception: module 'websocket' has no attribute 'create_connection' 2023-01-03 20:43:43,411 W main: cannot establish connection for incoming 2023-01-03 20:43:44,451 E create_zello_connection: exception: module 'websocket' has no attribute 'create_connection' 2023-01-03 20:43:44,451 W main: cannot establish connection for incoming 2023-01-03 20:43:45,494 E create_zello_connection: exception: module 'websocket' has no attribute 'create_connection' 2023-01-03 20:43:45,494 W main: cannot establish connection for incoming 2023-01-03 20:43:46,540 E create_zello_connection: exception: module 'websocket' has no attribute 'create_connection' 2023-01-03 20:43:46,540 W main: cannot establish connection for incoming 2023-01-03 20:43:47,584 E create_zello_connection: exception: module 'websocket' has no attribute 'create_connection' 2023-01-03 20:43:47,584 W main: cannot establish connection for incoming ^C2023-01-03 20:43:48,617 E main: keyboard interrupt caught 2023-01-03 20:43:48,619 I main: terminating
Here is the config.json set up (I am removing the Password and issuer.
`
"username": "Username",
"password": "password",
"zello_channel": "FDH P25 Traffic",
"issuer": "key",
"vox_silence_time": 3,
"audio_threshold": 1000,
"input_device_index": 1,
"input_pulse_name": "Virtual_Sink.monitor",
"output_device_index": 1,
"output_pulse_name": "Virtual_Sink",
"zello_input_sample_rate": 16000,
"audio_input_sample_rate": 48000,
"in_channel_config": "mono",
"audio_input_channels": 1,
"audio_output_sample_rate": 48000,
"audio_output_channels": 1,
"audio_output_volume": 1.5,
"logging_level": "info",
"TGID_in_stream": true,
"TGID_to_play": 58917,
"UDP_PORT": 9123
}
`

I suspect that you may be using a different websocket package than what this script is expecting.  It expects to use this one:
https://github.com/websocket-client/websocket-client
which should be installed by doing pip3 install websocket-client
You can check to see what Python is using for the websocket package by doing this from a Python terminal:
import websocket
help(websocket.__package__)

You should then get something that looks like this:
Help on package websocket:

NAME
    websocket

DESCRIPTION
    __init__.py
    websocket - WebSocket client library for Python

    Copyright 2022 engn33r

    Licensed under the Apache License, Version 2.0 (the "License");
    you may not use this file except in compliance with the License.
    You may obtain a copy of the License at

        http://www.apache.org/licenses/LICENSE-2.0

    Unless required by applicable law or agreed to in writing, software
    distributed under the License is distributed on an "AS IS" BASIS,
    WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
    See the License for the specific language governing permissions and
    limitations under the License.

If you get something else, you may have a different websocket package installed that's being used instead of the one that's expected.


Windows: Refactor pkg\plugins

Signed-off-by: John Howard<EMAIL_ADDRESS>I was going through still trying to get test-unit running on Windows when I stumbled across the plugins package which was erroring out. This is very Unix specific in implementation, so I ended up refactoring it to have the minimal pieces needed to compile docker on Windows. It was slightly complicated by the fact that libnetwork relies on certain structures, functions and variables exported by this package. I have put some TODO Windows where with some libnetwork refactoring and revendor, these can also be moved to _unix files.

@jhowardmsft experimental fail is related

@LK4D4 Huh! Thanks. Investigating...

@LK4D4 Yeah, was an easy fix. A couple of no-op methods were missing for graphdriver plugins present in experimental. Added "build Windows experimental" to my list of checks before PR submission! And hopefully I'll remember to run it!

LGTM

@jhowardmsft is this for TP4?

Just a general change

I'm not sure, plugins should work on windows as well(it's just HTTP). @jhowardmsft Could you point me to unix-specific details there?

Plugins work over http. This is not a unix only feature. This does not LGTM.

Hmmm. Closing and will revisit.


Don't open random devices while cleaning up.

Fixes #7022
In pull request #6432 a change was made to keep the handles to the
random devices opened in order to avoid reseeding problems for
applications in chroot environments.
As a consequence, the handles of the random devices were leaked at exit
if the random generator was not used by the application. This happened,
because the call to RAND_set_rand_method(NULL) in rand_cleanup_int()
triggered a call to the call_once function do_rand_init, which opened
the random devices via rand_pool_init(void).
Thanks to GitHub user @bwelling for reporting this issue.

I'll merge this evening.

Editorial note: I intend to apply two small cosmetical changes to the commit message when merging:
Don't open random devices while cleaning up.
->
rand_lib.c: Don't open random devices while cleaning up.

the random devices via rand_pool_init(void).
->
the random devices via rand_pool_init().

Ok, merged. Thanks!


Visual ILS Lock (suggestion)

I have been trying to come up with a good looking / minimal visual ILS lock indicator:
Anyone else have any ideas?

That looks very good! I like the main one and alt 2; I think 1 and 3 are too complicated while 4 has a square which IMO doesn't fit with the theme. Do you have a branch with this written?

no, was just trying out ideas in photoshop.
Yea, I liked those the best also, leaning towards alt #2 as it seams the cleanest.
And here is another one:

I can pretty easily implement this if we can pick one style. Do you prefer the main one or alt 2?

main one.    ( Looking at alt#2 now, its sort of looking like a mag glass... )

I'm refactoring the infobox drawing code at the same time so this may take a day or two.

... or longer. Sorry about the huge delay, but real life got in the way (as it often does).

"Real life" is there such a thing! :P
On Tue, May 12, 2015 at 1:30 PM, Jon Ross<EMAIL_ADDRESS>wrote:

... or longer. Sorry about the huge delay, but real life got in the way
(as it often does).
—
Reply to this email directly or view it on GitHub
https://github.com/zlsa/atc/issues/93#issuecomment-101411308.

Seriously though, no worries.

I got the graphics portion working for it, just haven't dug through the code to find where to see if a plane has ILS enabled or locked.

If the aircraft's mode == "landing", then ILS is locked. Thanks for working on it.

Got a working local branch of it, but not sure how to get it pushed up to GetHub.

@xenocide0 Does this help? https://help.github.com/articles/set-up-git/

Can this issue be closed @zlsa?

Yes, thanks @Maverick283.


Resolved issue 177 - Using default value in set_value with Form_validation not working

Fix in reference to #177.

This was out of date ... resolved in #1089.


Input translation (Completed)

#4
Changes

Added a file with all available English translations.
Added Input field and submit button to select a translation
Users can now enter abbreviated translation name to select that that translation.
Added error message that displays if the user does not select an adequate option.
Added CSS file with some styles.
Ignored a validation made in browserAction.js that restricted the possible translations.

Looks really good! Thanks for correcting the spelling mistakes as well! Appreciate it!
I guess your right about not having to double check input, I just thought maybe somehow users could fool around with inner HTML, but on background scripts they can't. I think that's not the case, I was wrong, so you're correct in what you're saying.


Future of the project

Not sure where else this project should be going, have a few issues that are high level that can be done

Read from .babelrc to get info: https://github.com/babel/babel-eslint/issues/573
Remove deps and use peerDeps given this project is the intersection of eslint/babel and can use the user's dep versions?
Finish TS Support: https://github.com/babel/babel-eslint/issues/505
Smoke test on ESLint/this project (related to https://github.com/babel/babel-eslint/issues/62)
Easier setup/install? maybe rename to @babel/eslint or @eslint/babel or even just expose this package itself in another scoped package?
Should this package move under the eslint org? I don't think it matters as long as everyone as access to github/npm
What do we do with https://github.com/babel/eslint-plugin-babel? I made https://github.com/babel/eslint-plugin-babel/issues/135 to talk about how it can be more easily maintained and inform the implementation of ESLint if a syntax does get to JS

Anything else?

Definitely would love to see the package to be renamed to @babel/eslint or @eslint/babel depending on the Org that maintains the package.
Putting everything under @eslint org makes more sense in term of where the maintains happens since the code is related to ESLint compatibility with Babel, no other way around.


Convert StructureMatcher inline code to HTML for partial link

Before:

After:

Btw, is this repo an official part of Matbench or just similar in spirit but a separate effort?

Thanks @janosh! Also, unofficial (will add to README, good clarification) but I'm hoping will become part of Matbench eventually as part of the official leaderboard #63

Also, unofficial (will add to README, good clarification) but I'm hoping will become part of Matbench eventually as part of the official leaderboard #63

Nice! I think it would be cool to have this in Matbench!


[1.3] Group Key Management Cluster XML has to be updated as per spec

Reproduction steps

[ ]  ClusterRevision value has to be added in XML. The value is 2.

Screenshot:

[ ] “GroupKeyMap” and “GroupTable” attributes length is mentioned as ”254 ” in XML, But Spec is not saying about it.

Screenshot:

[ ]   In GroupKeySecurityPolicyEnum, CacheAndSync field comformance is missing in XML.

Screenshot:

Spec Reference:
https://github.com/CHIP-Specifications/connectedhomeip-spec/blob/master/src/data_model/Group-Key-Management-Cluster.adoc#group-key-management-cluster
XML Reference:
https://github.com/project-chip/connectedhomeip/blob/master/src/app/zap-templates/zcl/data-model/chip/group-key-mgmt-cluster.xml

@Ashwinigrl XML does not have a way (or a reason) to represent enum value conformance.


Router::dispatch() directs to '/' when $_SERVER['SCRIPT_NAME'] == $_SERVER['REQUEST_URI']

I'm trying to incorporate Slim v 3.x and encountered a problem when $_SERVER['SCRIPT_NAME'] and $_SERVER['REQUEST_URI'] are identical (in our case, say both equal '/documentation/test/foo'). In this case, Slims Uri::createFromEnvironment() function sets the basePath property of the $uri object to '/documentation/test/foo' and the path property to '/'. When Router::dispatch() fires, it ignores the basePath and attempts to route to '/'.
I may have a naive understanding of this virtualPath and basePath distinction that Uri::createFromEnvironment() is making, but one would imagine that in a clean apache httpd.conf case, without virtual hosts or re-write rules configured, that you would encounter this situation where SCRIPT_NAME == REQUEST_URI commonly. Shouldn't Router::dispatch() incorporate the "basePath" in the uri used to dispatch the request? Something like:
    $basePath = rtrim($request->getUri()->getBasePath(), '/');
    $path = ltrim($request->getUri()->getPath(), '/');

    if (!empty($path) || (empty($basePath) && empty($path))) {
        $path = '/' . $path;
    }

    $uri = $basePath . $path;

    return $this->createDispatcher()->dispatch(
        $request->getMethod(),
        $uri
    );

Can you give us an example of the situation that leads to both $_SERVER['SCRIPT_NAME'] equaling $_SERVER['REQUEST_URI'] ?

Hey, sure, though I'm not sure in what way you mean for an example. I'll give you all I know. When I halt execution right before the return of Uri::createFromEnvironment(), I can verify that both of those server variables equal (these are dummied, but the same sort of thing I'm seeing) "/documentation/api/view", which is the route entered into, let's just say for clarity, a normal GET route in a browser to "/documentation/api/view". At this point, basePath is "/documentation/api/view" and path is "/".
Apache-wise, we're using virtual hosts with mod rewrite in place to route to appropriate routing scripts. So, for instance:
RewriteCond %{REQUEST_URI} ^/documentation
RewriteRule .* /www/<stuff here>/documentation/routes/main.php [NS,L,QSA]

This file, main.php, is where the Slim\App is instantiated, configured with paths to respond to, and where run() is called on it.
Another thing I've noticed is that in version 4.x, the references to basePath have been reduced. I'm hoping this means that the concept is being phased out? One of the critical places where basePath still factors in on 4.x is in URI::createFromEnvironment() at the part relevant to this ticket:

if (stripos($requestUri, $requestScriptName) === 0) {
            $basePath = $requestScriptName;
        } elseif ($requestScriptDir !== '/' && stripos($requestUri, $requestScriptDir) === 0) {
            $basePath = $requestScriptDir;
        }
        if ($basePath) {
            $virtualPath = ltrim(substr($requestUri, strlen($basePath)), '/');
        }

I'd like to mention that the solution that I've employed locally is actually commenting out this entire block and therefore completely gutting this basePath/virtualPath distinction.

Hi, this problem occurred when I was using php-cli server in my develop environment. When I access to /aabb.txt, the SCRIPT_NAME is /aabb.txt, others are /index.php. To fix this problem, I have to manually write a line such as $_SERVER['SCRIPT_NAME'] = '/index.php' in my index.php file.
I was referred to the implementation in Symfony HttpFoundation, usingSCRIPT_FILENAME and SCRIPT_NAME in the same time is a good idea to avoid this problem.

Yes there is an open problem with the php-cli web server which they have indicated that they will not be fixing.
@DanKottke Yes we are deprecating it :)

I am still having this problem with apache2.4.27-2ubuntu3 and Slim 3.9.0.
I am using this rewrite;
RewriteEngine On
RewriteCond %{REQUEST_FILENAME} !-f
RewriteCond %{REQUEST_FILENAME} !-d
RewriteRule ^ index.php [QSA,L]

And all URLs are resolving to / and not being routed where they should be.
Here is my $_SERVER;
Array
(
    [SCRIPT_URL] => /basket
    [SCRIPT_URI] => http://<IP_ADDRESS>/basket
    [HTTP_HOST] => <IP_ADDRESS>
    [HTTP_USER_AGENT] => Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:57.0) Gecko/20100101 Firefox/57.0
    [HTTP_ACCEPT] => text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8
    [HTTP_ACCEPT_LANGUAGE] => en-GB,en;q=0.5
    [HTTP_ACCEPT_ENCODING] => gzip, deflate
    [HTTP_COOKIE] => XDEBUG_SESSION=XDEBUG_ECLIPSE
    [HTTP_CONNECTION] => keep-alive
    [HTTP_UPGRADE_INSECURE_REQUESTS] => 1
    [HTTP_CACHE_CONTROL] => max-age=0
    [PATH] => /usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
    [SERVER_SIGNATURE] => <address>Apache/2.4.27 (Ubuntu) Server at <IP_ADDRESS> Port 80</address>

    [SERVER_SOFTWARE] => Apache/2.4.27 (Ubuntu)
    [SERVER_NAME] => <IP_ADDRESS>
    [SERVER_ADDR] => <IP_ADDRESS>
    [SERVER_PORT] => 80
    [REMOTE_ADDR] => <IP_ADDRESS>
    [DOCUMENT_ROOT] => /var/www
    [REQUEST_SCHEME] => http
    [CONTEXT_PREFIX] =>
    [CONTEXT_DOCUMENT_ROOT] => /var/www
    [SERVER_ADMIN] => webmaster@localhost
    [SCRIPT_FILENAME] => /var/www/index.php
    [REMOTE_PORT] => 38632
    [GATEWAY_INTERFACE] => CGI/1.1
    [SERVER_PROTOCOL] => HTTP/1.1
    [REQUEST_METHOD] => GET
    [QUERY_STRING] =>
    [REQUEST_URI] => /basket
    [SCRIPT_NAME] => /basket
    [PHP_SELF] => /basket
    [REQUEST_TIME_FLOAT] =><PHONE_NUMBER>.615
    [REQUEST_TIME] =><PHONE_NUMBER>
)

As you can see SCRIPT_NAME is not correct, which leads the path to be incorrect during routing. Is there a solution to this which doesn't involve manually setting SCRIPT_NAME  in my PHP? (feels like a hack)

I confirm the same bug with the latest slim version.

If this can be fixed in a B/C way I'm interested.
Also, can someone test if this is a problem for Slim 4?

I'm removing the Slim 4 label from this as it is not an issue there. URI comes from ServerRequestInterface which is determined by the PSR-7 implementation that you choose since #2529 was merged.

We could do with a reproducible project with .htaccess file and php -S command line.

Hunting around for the cause of this and possible workarounds, I found this long-standing bug report in Apache: https://bz.apache.org/bugzilla/show_bug.cgi?id=40102 Although dating to Apache 2.0, I believe this is the same issue I'm seeing on Apache 2.4
Based on that discussion, the key here seems to be the "context" within the Apache configuration:

In server-wide configuration, or a <VirtualHost> block, SCRIPT_NAME will end up equal to REQUEST_URI, breaking Slim's automatic base path detection.
In a <Directory> or <Location> block, or in a .htaccess file, mod-rewrite sets SCRIPT_NAME as expected.

This explains why not everyone sees this bug, because they're using a .htaccess rather than editing <VirtualHost> block in the main server config. It also gives a workaround, since surrounding the redirect with <Location /> ... </Location> is enough to change the context.
An additional workaround mentioned in the Apache bug is to use the [PT] (pass-through) flag on the RewriteRule. This forces it to rewrite URL-to-URL rather than URL-to-filename, which seems to cause SCRIPT_NAME to populate correctly.

All that being said, it would be nice if this could be mitigated in Slim as well.
If the detection code cannot be fixed, perhaps a setting could be added in some way to disable it, for people running into this issue. Unfortunately, \Slim\Http\Uri::createFromEnvironment is static, and only has access to the environment collection, so I'm not sure how to do that other than putting something into the environment data.
Alternatively, we might be able to detect the broken case by comparing SCRIPT_FILENAME and SCRIPT_NAME: in the expected case, SCRIPT_FILENAME is the full path on disk for the same file as SCRIPT_NAME; in the broken case, SCRIPT_NAME is an unrelated string from the requested URL.
if ( substr($env->get('SCRIPT_FILENAME'), -strlen($requestScriptName)) != $requestScriptName ) {
    $requestScriptName = '';
    $requestScriptDir = '';
}

Adding that just after the definition of $requestScriptName in Uri::createFromEnvironment seems to leave the feature working in the PHP built-in server, and when Apache sets SCRIPT_NAME correctly, but disable it (and so let the router work) when SCRIPT_NAME is bogus.
I don't have environments on hand to test it under other setups, though, like CGI, Apache+FPM, or Nginx.

Actually, combining that detection with mmelvin0's fix above might make it work for everyone:
$requestScriptFullPath = $env->get('SCRIPT_FILENAME');
if ( substr($requestScriptFullPath, -strlen($requestScriptName)) != $requestScriptName ) {
    // SCRIPT_NAME appears to have been set wrong by the server
    // Attempt to recover the correct value by taking everything in SCRIPT_FILENAME not also in DOCUMENT_ROOT
    $documentRootNoSlash = rtrim($env->get('DOCUMENT_ROOT'), DIRECTORY_SEPARATOR);
    if (strpos($requestScriptFullPath, $documentRootNoSlash) === 0) {
        $requestScriptName = substr($requestScriptFullPath, strlen($documentRootNoSlash));
        $requestScriptDir = dirname($requestScriptName);
    }
    else {
        // If SCRIPT_FILENAME and DOCUMENT_ROOT don't match either, just disable the base path detection
        $requestScriptName = '';
        $requestScriptDir = '';
    }
}

Again, though, I'm not confident how these vars will behave under other setups.

I'm closing this as the Slim 3 branch is now in maintenance mode only for security fixes.


About the implementation with the CLIP model

Could you release the code for the implementation using the CLIP model?

Regrettably, I can't release the code regarding CLIP due to internal reasons. However, we used the CLIP model from OpenCLIP. You can modify our code from ViT to CLIP without much effort using OpenCLIP.

Regrettably, I can't release the code regarding CLIP due to internal reasons. However, we used the CLIP model from OpenCLIP. You can modify our code from ViT to CLIP without much effort using OpenCLIP.

Ok, so can you provide details of your implementation, including base/ incremental learning rate, epoch, etc?

I also encountered this situation. One possibility is that your python version is not 3.8. Just change it to 3.8.

Regrettably, I can't release the code regarding CLIP due to internal reasons. However, we used the CLIP model from OpenCLIP. You can modify our code from ViT to CLIP without much effort using OpenCLIP.

Ok, so can you provide details of your implementation, including base/ incremental learning rate, epoch, etc?

Sorry for the delayed reply.
I set learning rate to 5e-5 and trained 1epoch for both base and incremental sessions in CIFAR-100.


Minor: Fix long running RangeQueryIntegrationTest

Summary

Noticed that RangeQueryIntegrationTest is taking ~approx 20 - 30min to run
Upon deep dive in logs, noticed that there were error for consumer rebalancing and test was stuck in loop

[2024-11-24 18:13:17,428] Unable to commit as we are in the middle of a rebalance, will try again when it completes.
[2024-11-24 18:13:17,428] Processed 0 total records, ran 0 punctuators, and committed 0 total tasks since the last update.
[2024-11-24 18:13:17,428] Invoking poll on main Consumer.
[2024-11-24 18:13:17,529] Main Consumer poll completed in 101 ms and fetched 0 records from partitions [].
[2024-11-24 18:13:17,530] Processing tasks with 1 iterations.
[2024-11-24 18:13:17,530] Processed 0 records with 1 iterations; invoking punctuators if necessary.
[2024-11-24 18:13:17,530] 0 punctuators ran.
[2024-11-24 18:13:17,530] Committing all active tasks [] and standby tasks [] since<PHONE_NUMBER>530ms has elapsed (commit interval is 100ms).

[2024-11-24 18:13:17,530] Unable to commit as we are in the middle of a rebalance, will try again when it completes.
[2024-11-24 18:13:17,530] Invoking poll on main Consumer.
[2024-11-24 18:13:17,632] Main Consumer poll completed in 102 ms and fetched 0 records from partitions [].
[2024-11-24 18:13:17,632] Processing tasks with 1 iterations.
[2024-11-24 18:13:17,632] Processed 0 records with 1 iterations; invoking punctuators if necessary.
[2024-11-24 18:13:17,632] 0 punctuators ran.
[2024-11-24 18:13:17,632] Committing all active tasks [] and standby tasks [] since<PHONE_NUMBER>632ms has elapsed (commit interval is 100ms).

Seems like due to same application.id across tests, Kafka Streams application is failing to track its state correctly across rebalances.

Test
Before Fix:

After Fix:

Merged #17933 into trunk


Use fog-aws instead of fog proper.

Fog is going through a process where they split out all of their
providers into separate gems. Since dragonfly-s3_data_store is targeted
at only aws we can use the fog-aws gem directly. This has the benefit of
not requiring applications to pull in all of fog (which is something
like 20 gems at the moment).
This should help reduce the memory overhead required for loading fog.

:+1:

Sounds good to me, although it would be a good idea to warn of this in the changeling - if the user is not aware of this change and if they have an older version of fog in their gem file already (either directly or as a dependency of something else) then they'd end up with the aws bit of fog loaded twice which could conceivably do odd things

+1 poor memory
dragonfly/s3_data_store: 19.3555 mb
  fog: 19.2852 mb
    fog/joyent: 6.6797 mb
      fog/joyent/compute: 6.6289 mb
        net/ssh: 6.2969 mb
          net/ssh/transport/session: 3.3906 mb
            net/ssh/transport/algorithms: 2.3555 mb
              net/ssh/transport/kex: 0.7031 mb
              net/ssh/transport/hmac: 0.5586 mb
              net/ssh/buffer: 0.3945 mb
              net/ssh/transport/cipher_factory: 0.3047 mb
            net/ssh/transport/packet_stream: 0.5859 mb
          net/ssh/authentication/session: 1.4102 mb
            net/ssh/authentication/key_manager: 0.6367 mb
          net/ssh/connection/session: 1.0547 mb
            net/ssh/connection/channel: 0.3203 mb
    fog/rackspace: 2.1914 mb
      fog/rackspace/auto_scale: 0.4297 mb
    fog/hp: 1.9258 mb
      fog/hp/block_storage: 0.5195 mb
        fog/hp/core: 0.3711 mb
      fog/hp/storage: 0.3359 mb
    fog/google: 1.1602 mb
      fog/google/compute: 0.7656 mb
    fog/openstack: 1.1016 mb
      fog/openstack/compute: 0.332 mb
    fog/softlayer: 0.7422 mb
    fog/internet_archive: 0.6328 mb
      fog/internet_archive/storage: 0.5898 mb
    fog/ecloud: 0.5977 mb
      fog/ecloud/compute: 0.5977 mb
    fog/cloudstack: 0.3984 mb
      fog/cloudstack/compute: 0.3984 mb
    fog/ibm: 0.3828 mb
    fog/libvirt: 0.3594 mb
      fog/libvirt/compute: 0.3516 mb
    fog/xenserver: 0.3125 mb
      fog/xenserver/compute: 0.3125 mb
fog/aws: 10.7031 mb
  fog/aws/core: 8.4258 mb
    fog/xml: 4.8281 mb
      nokogiri: 4.7969 mb
        nokogiri/xml: 2.3711 mb
          nokogiri/xml/node: 0.4766 mb
        nokogiri/html: 1.0313 mb
          nokogiri/html/element_description_defaults: 0.6328 mb
        nokogiri/css: 0.8047 mb
          nokogiri/css/parser: 0.4219 mb
    fog/core: 3.2188 mb
      fog/core/collection: 0.4648 mb
  fog/aws/compute: 0.3477 mb
  fog/aws/elb: 0.3008 mb

Why not use aws-sdk then? Any memory benchmark for this?

Memory consumption dropped significantly after switching to a branch from collectiveidea
Here is the gemfile
gem 'dragonfly-s3_data_store', require: nil, github: "collectiveidea/dragonfly-s3_data_store", branch: "use-fog-aws"

memory benchmark on require using derailed_benchmark
dragonfly/s3_data_store: 1.1953 mb
  fog/aws: 1.1953 mb
    fog/aws/core: 1.1953 mb
      fog/xml: 1.1719 mb
        nokogiri: 1.168 mb
          nokogiri/xml: 0.6797 mb

I've created a fork which requires fog/aws/storage directly as you suggested. It does not improve memory footprint in my benchmarks. I presume because in production only used classes are actually required. Regardless I like to require only the smallest necessary file and this seems to be the one.
https://github.com/MSchmidt/dragonfly-s3_data_store/tree/use-fog-aws
(fork also includes support for eu-central-1 AWS region)

@MSchmidt You should checkout this post to benchmark memory consuption after rails boot http://www.schneems.com/2014/11/07/i-ram-what-i-ram.html
It might be hard to notice in production when you have a lot of gem loaded or multiple web workers.
derailed_benchmark, bumbler gems and this onliner helped me tracking down slow requires and excessive memory usage from rubygems on load.
time bin/rails r 'p $LOADED_FEATURES.length'

I look forward to see how much impact on your rails boot after switching to fog/aws

thanks for this.
it's merged in, and I've stuck with require 'fog/aws' rather than require 'fog/aws/storage', as the latter seemed to raise errors, and I'm not sure how fully supported it will be as fog evolves.
as for using aws-sdk gem directly, I'd be up for this if it was better in any way, though don't have time to do it myself, so if anyone wanted to do a PR for that and it was demonstrably better (quicker/more robust, etc.) then I'd definitely consider pulling that in
Thanks!

:+1:


Build errors under Ubuntu 18.04 with GCC 9

When I build under Ubuntu 18.04, I get:
FAILED: src/source.p/source.cpp.o
c++ -Isrc/source.p -Isrc -I../src -I../include/loguru -Iinclude -I../include -Isrc/test_utils -I../src/test_utils -I../include/test -fdiagnostics-color=always -D_FILE_OFFSET_BITS=64 -Wall -Winvalid-pch -Wnon-virtual-dtor -Wextra -Wpedantic -std=c++17 -g -pthread -MD -MQ src/source.p/source.cpp.o -MF src/source.p/source.cpp.o.d -o src/source.p/source.cpp.o -c ../src/source.cpp
In file included from ../include/redis-cpp/stream.h:33,
                 from ../src/source.cpp:37:
../include/redis-cpp/detail/stream.hpp:109:18: error: 'io_context' in namespace 'boost::asio' does not name a type
  109 |     boost::asio::io_context io_context_;
      |                  ^~~~~~~~~~
../include/redis-cpp/detail/stream.hpp:110:42: error: 'io_context_' was not declared in this scope
  110 |     boost::asio::ip::tcp::socket socket_{io_context_};
      |                                          ^~~~~~~~~~~
../include/redis-cpp/detail/stream.hpp:110:53: error: could not convert '{<expression error>}' from '<brace-enclosed initializer list>' to 'boost::asio::ip::tcp::socket' {aka 'boost::asio::basic_stream_socket<boost::asio::ip::tcp>'}
  110 |     boost::asio::ip::tcp::socket socket_{io_context_};
      |                                                     ^
      |                                                     |
      |                                                     <brace-enclosed initializer list>
../include/redis-cpp/detail/stream.hpp: In lambda function:
../include/redis-cpp/detail/stream.hpp:82:57: error: 'io_context_' was not declared in this scope
   82 |                 boost::asio::ip::tcp::resolver resolver{io_context_};
      |                                                         ^~~~~~~~~~~
../include/redis-cpp/detail/stream.hpp:82:68: error: no matching function for call to 'boost::asio::ip::basic_resolver<boost::asio::ip::tcp>::basic_resolver(<brace-enclosed initializer list>)'
   82 |                 boost::asio::ip::tcp::resolver resolver{io_context_};
      |                                                                    ^
In file included from /usr/include/boost/asio.hpp:63,
                 from ../include/redis-cpp/detail/stream.hpp:17,
                 from ../include/redis-cpp/stream.h:33,
                 from ../src/source.cpp:37:
/usr/include/boost/asio/ip/basic_resolver.hpp:67:12: note: catestdate: 'boost::asio::ip::basic_resolver<InternetProtocol, ResolverService>::basic_resolver(boost::asio::io_service&) [with InternetProtocol = boost::asio::ip::tcp; ResolverService = boost::asio::ip::resolver_service<boost::asio::ip::tcp>]'
   67 |   explicit basic_resolver(boost::asio::io_service& io_service)
      |            ^~~~~~~~~~~~~~
/usr/include/boost/asio/ip/basic_resolver.hpp:67:12: note:   conversion of argument 1 would be ill-formed:
In file included from ../include/redis-cpp/stream.h:33,
                 from ../src/source.cpp:37:
../include/redis-cpp/detail/stream.hpp:83:83: error: no matching function for call to 'boost::asio::ip::basic_resolver<boost::asio::ip::tcp>::resolve(std::remove_reference<const std::basic_string_view<char>&>::type, std::remove_reference<const std::basic_string_view<char>&>::type)'
   83 |                 auto endpoints = resolver.resolve(std::move(host), std::move(port));
      |                                                                                ^

This is using the following packages installed:
apt-get update -qq && \
  apt-get install --assume-yes --no-install-recommends \
  ca-certificates \
  build-essential \
  wget \
  ninja-build \
  pkg-config \
  cmake \
  ninja-build \
  libboost-all-dev \
  libcurl4-openssl-dev \
  libpcap-dev \
  libavahi-client-dev \
  python3-pip \
  software-properties-common

add-apt-repository ppa:ubuntu-toolchain-r/test && \
  apt-get update -qq && \
  apt install --assume-yes gcc-9 g++-9 && \
  update-alternatives --install /usr/bin/gcc gcc /usr/bin/gcc-9 60 --slave /usr/bin/g++ g++ /usr/bin/g++-9

So I have g++ (Ubuntu 9.4.0-1ubuntu1~18.04) 9.4.0 and libboost-all-dev version <IP_ADDRESS>ubuntu1.

Seems that when you use the mhier/libboost-latest PPA and install libboost1.74 it works.

Hi.
Thanks for the request.
I'd tried to build the library on my system with boost 1.71 (available out of the box within ubuntu 20.04) and everything worked. I've mentioned version 1.71 on the README page. I'm not sure about the minimum version, it's likely 1.67. You could try if you like and report the result.
Good luck.
Regards

Yeah I think it's simply the fact that 18.04 does not bundle the required libraries. But then again, it's outdated and it only comes with GCC 7 out of the box. I'll close this issue then. Thanks for your help!


useMatches type inference

I found myself wanting to access a loader response generated by remix-typedjson via useMatches and ran into difficulty trying to specify the type of the data attribute.
I tried doing various incantations of:
match as Awaited<Promise<ReturnType<typeof loader>>>

In so doing I was never able to get the typedJson unwrapped from the response type, the closest I managed to get was:
Response & { typedjson(): Promise<{ foo: string, bar: number }> }

When Remix 1.10 lands useRouteLoaderData (https://reactrouter.com/en/main/hooks/use-route-loader-data) seems like it'll offer a better model for accessing loader data from elsewhere in the tree much more elegantly than the useMatches route so perhaps I'm barking up the wrong tree entirely.

For now, I think the simplest way is to do:
import { loader as rootLoader } from '~/root'

type RootLoaderData = typeof rootLoader
const matches = useMatches()
const data = matches[0].data as RootLoaderData

And change which loader type you're trying to access. But yeah, useRouteLoaderData will let you specify a generic loader type, so DX is better.

So what I was doing was fairly similar - but because the loader returns a json from remix-typedjson I've struggled to get the typeof loader to behave.
I think what I need to emulate is what's happening in the useTypedLoaderData (not the actual JSON parsing but the type-assignment bits) but basically trying to "tear off" the TypedJsonResponse type wrapped around the result similar to what the Awaited type allows one to do for promises.
I put together a small example repo at: https://github.com/lukebowerman/remix-example/blob/main/app/routes/index.tsx#L7-L17 to try to show what I'm bumping into.

Thank you @marcojakob - that was exactly what I was looking for :)


Fix GitHub Deployments example color

The example color currently appears as grey on the homepage:

render expects the state to be passed in as uppercase.

Warnings

:warning:
This PR modified service code for github but not its test code. That's okay so long as it's refactoring existing code.

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @PyvesB!

  Generated by :no_entry_sign: dangerJS against 4f8ee4f6a0f512ae6e74cbccadd7a60fa606a7b8


Ci debug

Purpose

Linked issue: close #xxx

Tests

API and Format

Documentation

Sorry.....Merge it to main branch by mistake....Now, I revert it in 294....


Performance Issues

Is there anything that can be done about load times for this theme?  Every time we switch pages, each item takes about 1 second to render, and they do it one at a time.  When a light switch is clicked, the styling disappears for 0.5 seconds, and comes back, making it look "broken".  Running it on a Fire 10 HD.  Any other theme doesn't experience this issue.
Love this theme, just want it to perform as quick and smooth as other themes.  If anyone has suggestions, that's good too!
Thx!!

I would start with disabling the texture using the entity you were instructed to create. It will look flatter (seems many people prefer this) and it will reduce the number items that need to load. The textures are a bit heavy.
Beyond that, you could try putting Cloudflare (free) in front of your site if it's exposed to the internet externally. It will cache some of the assets which should improve load times, especially if the system you are hosting it on is lower power. Ultimately, though, it comes down to the hardware that hosts Home Assistant, the network it lives in, and the hardware that displays it. By necessity, this theme is heavier than most other themes.

Thanks. Can you clarify what you mean by texture? We’re running it all locally. It works fine when accessing in my iPhone. It’s just locally in the Fire tablets where it’s slow to render. Might just be the complexity of the theme as we don’t have issues with other themes.

Yes, unfortunately the theme is heavy and adds a lot of extra CSS. The only other thing I would suggest is simplifying the dashboards that load on the problematic devices.


feat: draft release support

I started with the generic builder and will continue with the others as well if you accept the solution:
Fixes #1476
I tried that in osv-scanner project and it worked:

Job: _https://github.com/developer-guy/osv-scanner/actions/runs/4536204956/jobs/7992687991_
Release: _https://github.com/developer-guy/osv-scanner/releases/tag/untagged-5bf09e3accd2794d0d10_

PTAL @laurentsimon @asraa @ianlewis

Thanks! Can you:

[ ] Add draft-release to the Go builder?
[ ] Add draft-release to the docs for both the generic generator and Go builder?
[ ] Add a note to the CHANGELOG.md?

ready for review @ianlewis

Thanks! Could you add the draft-release input to the docs for the generic generator, docker based builder, and Go builder ?
We should also have a note in the CHANGELOG.md for the v1.6.0 release.

[x] Add draft-release to the docs for the generic generator
[x] Add draft-release to the docs for the Go builder?
[x] Add draft-release to the docs for the docker-based builder?
[x] Add a note to the CHANGELOG.md?


Using radian with chemacs.

I tried to use chemacs  to test radian but it doesn't work.
How to configure it with chemacs?

Have you followed the symlinking instructions in the README? I don't use this tool (Radian implements a profile switching system itself, by means of the USER_EMACS_DIRECTORY environment variable), but it seems like it should work fine to link the requisite Radian files into a separate directory to serve as user-emacs-directory, and then configure this directory for use with chemacs.

I just added the following:
(("default" . ((user-emacs-directory . "~/.emacs.d")))              ;; DoomEmacs, my default emacs
 ("prelude" . ((user-emacs-directory . "~/EMACS_CONFIGS/prelude"))) ;; Prelude
 ("spacemacs" . ((user-emacs-directory . "~/EMACS_CONFIGS/spacemacs/.emacs.d"))) ;; Spacemacs

 ("radian" . ((user-emacs-directory . "~/EMACS_CONFIGS/radian/emacs/"))) ;; radian. By Straight.el package maker.
 )

It usually work like this
Changed this line too:
    (defvar radian-local-init-file "~/.emacs.d/init.local.el"
      "File for local customizations of Radian.")

to be:
    (defvar radian-local-init-file "~/EMACS_CONFIGS/radian/emacs/"
      "File for local customizations of Radian.")

but still having the same issue.

You need to follow the symlinking instructions in the README. The emacs subdirectory of this repository will not function as a user-emacs-directory. If you don't want to link the files directly into ~/.emacs.d, you need to create a new directory and link them into that directory, as per the instructions.
Separately, I have pushed a commit which makes sure that various other functions in Radian will function as desired with user-emacs-directory set to a nonstandard value.

Thanks for the support.
You can close the issue. I test straight on my setup and it downloaded about 340mb of data by cloning repositories of each package I have on my init file, which is a very slow process unfortunately.
Off topic:
What's the point of having Melpa in straight if download the repos of a package? why can't it just download directly from melpa? Is there an option for that?
The only package I need for an external repos is Bookmark+.
And straight download about 100mb of data for that package alone.

If you don't want to clone Git repositories, then it sounds like you don't want to use straight.el. The entire point of straight.el is to clone Git repositories. Downloading tarballs from MELPA is an antipattern given the design goals of straight.el and is completely unsupported.
But of course it also needs to clone MELPA. Otherwise how would it know where to find the Git repositories for packages?

Thanks for the explanation :)

See also https://github.com/raxod502/straight.el/issues/149 for improving performance of cloning Git repos initially.

Thanks :)

Can you explain how to set it up for chemacs 2.0 if possible? It occupies emacs.d. Iv'e tried this, but with the require init files simlinked into USER_EMACS_DIRECTORY I'm getting:
error: Recursive load, /home/uri/.config/radian/init.el, /home/uri/.config/radian/init.el, /home/uri/.config/radian/init.el, /home/uri/.config/radian/init.el, /home/uri/.config/radian/init.el, /home/uri/src/radian-emacs/emacs/init.el, /home/uri/.emacs.d/init.el

I believe you would want to have ((user-emacs-directory . "~/.config/radian")) if that's where you are symlinking the dotfiles, no?

AFAIK user-emacs-directory is the base directory of the distribution itself ... I'm note sure then where to link my personal config files...

Radian expects to be symlinked, hence the recursive load error. If you don't want to symlink into ~/.emacs.d directly, e.g. because you're using chemacs, I would suggest creating a directory like ~/.config/radian, and then symlinking the Radian dotfiles into that directory. I.e.,

~/.config/radian is an ordinary directory
~/.config/radian/init.el is a symlink to ~/src/radian-emacs/emacs/init.el
~/.config/radian/straight/versions/radian.el is a symlink to ~/src/radian-emacs/emacs/versions.el
~/.config/radian/init.local.el is your local configuration file for Radian

Then you would set user-emacs-directory to ~/.config/radian.


Feature Request: Add Notifications

I suggest to add notifications when the balance changes. Basically scanning for SendTx transaction that were not self created.

Done :)

Oh wow, that was fast :-D


[Bug] Failing to copy libraries with Unity 2022 and maintemplate enabled

[REQUIRED] Please fill in the following fields:

Unity editor version: 2022.1.0f1
Firebase Unity SDK version: 8.10.1
Source you installed the SDK: tried both .unitypackage and Unity Package Manager
Problematic Firebase Component: All
Other Firebase Components in use: N/A
Additional SDKs you are using: None
Platform you are using the Unity editor on: Windows
Platform you are targeting: Android
Scripting Runtime: IL2CPP

[REQUIRED] Please describe the issue here:
When maintemplate.gradle is enabled and any firebase package is imported, the dependency manager fails to copy the files to the correct location in GeneratedLocalRepo. Builds and Force Resolve both fail with the error:
Unable to copy Assets\Firebase\m2repository\com\google\firebase\firebase-analytics-unity\8.10.1\firebase-analytics-unity-8.10.1.srcaar to Assets\GeneratedLocalRepo\Firebase\m2repository\com\google\firebase\firebase-analytics-unity\8.10.1\firebase-analytics-unity-8.10.1.aar.  Assets\GeneratedLocalRepo\Firebase\m2repository\com\google\firebase\firebase-analytics-unity\8.10.1\firebase-analytics-unity-8.10.1.aar will not be included in Gradle builds. Reason: Failed to copy Assets\Firebase\m2repository\com\google\firebase\firebase-analytics-unity\8.10.1\firebase-analytics-unity-8.10.1.srcaar to Assets\GeneratedLocalRepo\Firebase\m2repository\com\google\firebase\firebase-analytics-unity\8.10.1\firebase-analytics-unity-8.10.1.aar due to System.IO.DirectoryNotFoundException: Could not find a part of the path 'D:\unity\Projects\testu22\Assets\GeneratedLocalRepo\Firebase\m2repository\com\google\firebase\firebase-analytics-unity\8.10.1\firebase-analytics-unity-8.10.1.aar'.
When browsing the directory, it has been incorrectly created as
Assets\GeneratedLocalRepo\Firebase\m2repository\com\google\firebase\firebase-analytics-unity\8
instead of
Assets\GeneratedLocalRepo\Firebase\m2repository\com\google\firebase\firebase-analytics-unity\8.10.1
Steps to reproduce:
Have you been able to reproduce this issue with just the Firebase Unity quickstarts (this GitHub project)?
Yes, it also happens with an empty project with just maintemplate.gradle enabled and any firebase package imported
What's the issue repro rate? (eg 100%, 1/5 etc)
100%
What happened? How can we make the problem occur?
Create a project, switch to Android, enable maintemplate.gradle and import any firebase package. When attempting to build or use the EDM force resolve option, errors are shown and the build fails.
If you have a downloadable sample project that reproduces the bug you're reporting, you will
likely receive a faster response on your issue.
Relevant Code:
N/A

I'm having the same problem on Mac with Apple Silicon

the same problem
Unity 2020.1.4f1
packages:
external-dependency-manager 1.2.171
GooglePlayGamesPlugin v0.11.01
Firebase Unity 9.1.0

has anyone solved this problem?

has anyone solved this problem?

for us it's work only with disabled mainTemplate in Unity Player Settings

has anyone solved this problem?

for us it's work only with disabled mainTemplate in Unity Player Settings

Thanks

❗  Finally I found the reason why it's happening. There is a Unity bug, it's already registered, thus it's not related to Firebase at all: https://issuetracker.unity3d.com/issues/folder-name-is-truncated-when-dot-is-used-in-the-name
Be more concretically, you can find this code in FileUtils class that is part of AndroidResolver project: https://github.com/googlesamples/unity-jar-resolver/blob/master/source/VersionHandlerImpl/src/FileUtils.cs#L635
When recursion goes to dotted part of the path (8.10.1 in your case), CreateFolder method returns an error.
To solve the problem you can do one of the following things:

use explicit resolving as described before: libraries stay in Plugins folder;
downgrade to Unity version where the issue doesn't reproduce (can't imagine that someone will do this 😆 );
replace the whole code of CreateFolder method with native Directory.CreateDirectory (requires manual resolver compiling).

Simple yet Annoying Bug.
Found a simple solution :
No need to disable anything.
It simply cannot find/create the folder with the same version number in Generated Local Repo Directory if created through Unity Asset Browser.

Simply go to the directory through Windows Explorer and in my case it was  "[THE  LOCATION   OF  YOUR GAME PROJECT]\Assets\GeneratedLocalRepo\Firebase\m2repository\com\google\firebase\firebase-analytics-unity".
Or
In Unity Asset Browser, browse to "Assets\GeneratedLocalRepo\Firebase\m2repository\com\google\firebase\firebase-analytics-unity\firebase-analytics-unity" and right-click, then "Show in Explorer".

Delete any existing folder and refresh in unity asset browser

Create a new folder with version number, mine was 9.3.0 and check inside unity if the folder has been synchronized. Only a folder with your corresponding version number should exist.

Do the same for all firebase folders, and then run Force resolve.

Voila!! Android Dependencies have been resolved.

Simple yet Annoying Bug.
Found a simple solution :
No need to disable anything.
It simply cannot find/create the folder with the same version number in Generated Local Repo Directory if created through Unity Asset Browser.

Simply go to the directory through Windows Explorer and in my case it was  "[THE  LOCATION   OF  YOUR GAME PROJECT]\Assets\GeneratedLocalRepo\Firebase\m2repository\com\google\firebase\firebase-analytics-unity".
Or
In Unity Asset Browser, browse to "Assets\GeneratedLocalRepo\Firebase\m2repository\com\google\firebase\firebase-analytics-unity\firebase-analytics-unity" and right-click, then "Show in Explorer".
Delete any existing folder and refresh in unity asset browser
Create a new folder with version number, mine was 9.3.0 and check inside unity if the folder has been synchronized. Only a folder with your corresponding version number should exist.
Do the same for all firebase folders, and then run Force resolve.

Voila!! Android Dependencies have been resolved.

it's simple, but it's not work if you frequencly updated plugins, we need worked, not "magical" solution

it's simple, but it's not work if you frequencly updated plugins, we need worked, not 🔧"magical" solution It seems only like temp solution

Of course, its a temporary solution. Its for the devs to fix it permanently as it is present in many versions of the editor, and I prefer not to touch the mess that the devs created, so hence my solution which hardly takes some time even if we add a couple different versions.

Hey, @chkuang-g has already created PR-543 and after some time it will be merged to the unity-jar-resolver repo.

Simple yet Annoying Bug.
Found a simple solution :
No need to disable anything.
It simply cannot find/create the folder with the same version number in Generated Local Repo Directory if created through Unity Asset Browser.

Simply go to the directory through Windows Explorer and in my case it was  "[THE  LOCATION   OF  YOUR GAME PROJECT]\Assets\GeneratedLocalRepo\Firebase\m2repository\com\google\firebase\firebase-analytics-unity".
Or
In Unity Asset Browser, browse to "Assets\GeneratedLocalRepo\Firebase\m2repository\com\google\firebase\firebase-analytics-unity\firebase-analytics-unity" and right-click, then "Show in Explorer".
Delete any existing folder and refresh in unity asset browser
Create a new folder with version number, mine was 9.3.0 and check inside unity if the folder has been synchronized. Only a folder with your corresponding version number should exist.
Do the same for all firebase folders, and then run Force resolve.

Voila!! Android Dependencies have been resolved.

that's worked, thank you!!

Fix In Review
https://issuetracker.unity3d.com/issues/folder-name-is-truncated-when-dot-is-used-in-the-name


Pilot agent won't start if SDS is used for control-plane security.

Epic: https://github.com/istio/istio/issues/15267
Issue: if controlPlaneAuthPolicy is enabled, but SDS is used to communicate with Pilot - the sidecar will keep looking for mounted certificates.
Desired behavior, taking into account 'upgrade' and gradual adoption:

Pilot agent should verify that the SDS UDS path exists:
-- use SDS_PATH environment variable, with a list of ":" separated paths ( like CLASSPATH or GOPATH).
-- if SDS_PATH is not specified, default to '/var/run/sds/uds_path'
if any of the paths in SDS_PATH exists ( file.exists ) - do not monitor /etc/certs and don't delay the startup until /etc/certs is populated.
include the SDS path that was found in the "metadata" that is passed to pilot.

Same thing for SDS_TOKEN_PATH - with the default set to
'/var/run/secrets/tokens/istio-token:/var/run/secrets/kubernetes.io/serviceaccount/token'
The defaults are matching the default installation option for Istio.

most change in pilot agent was in my original https://github.com/istio/istio/pull/13593, difference is using env var instead of proxy argument.
btw, why sds_path and sds_token_path are list of string(instead of one), what's pilot expected behavior is multiple values are set ? only first value is used or generate multiple sdsconfig ? if later, how to match uds path/token path, do we plan to support one envoy fetch certs from different services(on same node) , does that work ?

List of paths for UDS is P2 - it's fine to have only one, with the default set. The use case is support of other SDS providers, like the SPIRE agent (or other SDS implementations). Technically we can map whatever host paths they have to a constant name.
For token - the point is to verify if the 'trustworthy' token is set first, and if yes use it. Otherwise fall back to the less secure token. And for VMs - they may get tokens from different locations. So that would be P1 or P0 for usability.
Yes, it is possible for some workloads to use a different SDS provider - in cases of upgrades, migrations, etc. There is a document in Env WG discussing safe upgrade - anything that relies on a global needs to be fixed, and the fact SDS server has a fixed UDS path is a problem. If users want to test a new version of the SDS provider, or a different implementation - they would run them side-by-side, and annotate the
test workloads to use the new one.

I left few additional comments on #13593 - if you can create a separate PR with just the pilot-agent changes (i.e. check if UDS and which token exist, pass to pilot - without the additional setup complexity and risks) we can probably have the fix merged sooner.

sounds like a plan, I will create a new PR only for pilot-agent first; then we may change pilot code where sdsconfig is constructed, replacing global flag with sds related values passed by pilot-agent.
sg we should make uds path/token path configurable for each sidecar, IMO it's fine to just pass a single string env var(both uds path/token path) to pilot-agent and let sidecar injector to handle the value to set, trustworthy jwt is injected by sidecar injector, so let sidecar injector to handle this align better; single value simply logic in pilot agent, and support the use cases(ex point to different sds provider, choose which token to use)

I don't think we have a use case to making it configurable - the problem we have is that we have far too much complexity in configuring and troboubleshooting. I am ok with having an env var to override the path for testing or VMs or special cases - but it must have a default value and by default should just work.  If it is ok to use /etc/certs as a fixed path ( with some env to override that we added recently ) - it should be the same for the uds path.
Simpler is better.
For tokens - we certainly need to check both locations, if the 'trustworthy' JWT is installed we use it otherwise the old k8s one. Again - the second is hardcoded anyways, and hardcoding the first is not
a problem, just like we hardcode the location of the certs. Any injector or tool using a different kind of token can just place it in the specified location.

So the minimum requirements for pilot-agent are:

check the hardcoded UDS path. If file is present - send it to pilot as metadata.
check a hardcoded list of 2 tokens. First found - send it to pilot as metadata.

Optional:

env variable to override the hardcoded UDS path. We can add it later (in 1.4).
env variable to override the search path for tokens

For SPIRE agent ( or other agents  that authenticate pod by UDS socket, and don't need token ) -
we will still enable SDS if only the UDS path is sent to pilot.

this is done.


Export themes with css class.

Please compile the css with an additional class so that we can switch styles in runtime just by toggling some class like flatpickr-theme-dark, etc. This way, it is trivial to add a light theme and a dark theme to the webpack assets, and switch between those as per user config without reloading the browser.
Your Environment

flatpickr version used: 4.6.3
Browser name and version: Firefox 67
OS and version: Linux/Ubuntu 18.04

@aero31aero Sounds like a great idea. I will see if I can work it out without breaking backwards compatibility.

Was this ever implemented in some way, some way to dynamically choose the theme if the component is compiled into a solution?

Also wondering if this was ever implemented?


AMIs in eu-*

Hey,
I'm a total AWS noob. This is literally the reason why I've signed up. I've tried to copy cloudygamer-loader5 from us-west-1 to eu-west-2 but I'm getting the error "You do not have permission to access the storage of this ami". Is that something you can fix?
Thanks and happy new year.

hrm, forgot to also set the Snapshot permissions. try it now. i'll also make some copies since i don't intend on changing the loader any time soon

cool, i started copies to all regions with g2.2xlarge instances, should be done shortly. happy new year!

FWIW there weren't any in us-east-1, so I copied it there too (public).

@ericzinnikas i took a quick look, ami-f57d63e2 should be the AMI there?

This is the only thing that comes up when I search by name (my AMI), searching with that ID shows 0 results.  Is it set to public?


[BUG] Update generate_config to add container_names, or deprecate and update the tests that require it

Describe the bug
We should update the generate_config command to support adding & populating the container_names argument if we continue to use it. Currently, only the rolling_update, rolling_downgrade and rolling_version_change tests need this argument and they're currently excluded from the certification? So, as long as these tests remain excluded, this ticket doesn't need to be addressed.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Device (please complete the following information):

OS [e.g. Linux, iOS, Windows, Android]
Distro [e.g. Ubuntu]
Version [e.g. 18.04]
Architecture [e.g. x86, arm]
Browser [e.g. chrome, safari]

How will this be tested? aka Acceptance Criteria (optional)
(optional: unnecessary for things like spelling errors and such)
Once this issue is address how will the fix be verified?
Additional context
Add any other context about the problem here.

NOTE: you can enable higher logging level output via the command line or env var. to help with debugging
# cmd line
./cnf-testsuite -l debug test

# make sure to use -- if running from source
crystal src/cnf-testsuite.cr -- -l debug test

# env var
LOGLEVEL=DEBUG ./cnf-testsuite test

Also setting the verbose option for many tasks will add extra output to help with debugging
crystal src/cnf-testsuite.cr test_name verbose

Check usage documentation for more info about invoking commands and logging

@HashNuke @agentpoyo @denverwilliams is still issue still relevant for CNTI?

Yes issue still relevant. Tagging appropriately for contributor visibility.
Notes for contributors
The generate_config helps generate a cnf-testsuite.yml file if the cnf is a helm chart or a manifest directory. It does not currently generate any config options relevant for running the rolling_* tests as mentioned in the ticket description.
spec/setup_spec.cr has some automated tests that have example usage of this command.
We could generate these fields as comments in the resulting config file, along with example values and display a note in the command's output that these fields need to be set appropriately for the relevant tests to run (instead of being skipped).
If we go any further and try to fetch previous versions of the container images in order to set useful values for the rolling_* config fields, then we would have to handle a few more scenarios (Example: What to do if the images are from a private registry? Which containers do we use for the test?)

Even with new config generation (#2138) - this issue will still stay still relevant. As I mentioned in #1321, I think that these tests should be redesigned, and possibly, after this redesign, additional info in CNF config wouldn't be needed.
On the other hand, I think, that config generation primarily serves for new testsuite users, and adding configuration for this small, non-cert test group is not that necessary.


Added Microsoft eCDN 1st party solution

I would recommend the Peer5 logo, which is being sunsetted following the company's acquisition by Microsoft in Aug 2021, should be removed from the initial image.

Docs Build status updates of commit 5d7aa8d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Teams/teams-live-events/what-are-teams-live-events.md
:white_check_mark:Succeeded

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

Docs Build status updates of commit 4b257f8:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Teams/teams-live-events/what-are-teams-live-events.md
:white_check_mark:Succeeded

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

@MicrosoftHeidi Can you help us to merge this pull request? Thanks!
@PeerDiego

#sign-off

Hello @MicrosoftHeidi. Could we get your assistance getting this one merged as well? Thanks.

Learn Build status updates of commit 827939b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Teams/teams-live-events/what-are-teams-live-events.md
:white_check_mark:Succeeded

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


[curl] Mitigate issue #14834: port broken on Linux

Describe the pull request

What does your PR fix?

Mitigate issue #14834: port broken on Linux
The more complete fix will come from the pull request #14807. However, since curl on Linux is broken right now, I think it might be worth it to merge this branch quickly so that curl will work again on Linux.

Which triplets are supported/not supported? Have you updated the CI baseline? Nothing changed here

Does your PR follow the maintainer guide? Yes

No, this will break the port using curl pkgconig.

No, this will break the port using curl pkgconig.

Well, yes and no 🙂 . As explained above in my comment, this line was added ~10 days ago in #14350. It only affect the behaviour on Linux, and right now, because of this line, the port doesn't work at all on Linux
(neither does anything that depends, directly or transitively, on curl). Hence, I'd argue that this is a good first step , and then your pull request #14807 will provide a more complete fix that allows to use pkgconfig on Linux. 🙂

No, this will break the port using curl pkgconig.

Well, yes and no 🙂 . As explained above in my comment, this line was added ~10 days ago in #14350. It only affect the behaviour on Linux, and right now, because of this line, the port doesn't work at all on Linux
(neither does anything that depends, directly or transitively, on curl). Hence, I'd argue that this is a good first step , and then your pull request #14807 will provide a more complete fix that allows to use pkgconfig on Linux. 🙂

I was doing some tests for pull request #14807, and in the midst of these tests, I've realized that for I reason that I don't understand, I'm not able to reproduce the problem anymore. I guess that's a good news, although I'm curious and I'd like to understand what happen :laughing: . I'm closing and I'm going to leave a comment on issue #14834 to see if the issue is solved for other people to. I'm going to continue to follow #14807 (I'll add a comment there in ~1h if everything goes well).

I was doing some tests for pull request #14807, and in the midst of these tests, I've realized that for I reason that I don't understand, I'm not able to reproduce the problem anymore. I guess that's a good news, although I'm curious and I'd like to understand what happen :laughing: . I'm closing and I'm going to leave a comment on issue #14834 to see if the issue is solved for other people to. I'm going to continue to follow #14807 (I'll add a comment there in ~1h if everything goes well).


Bug in Unlock Function

In the current Unlock, we fetch the value of the receiver and the transferred value from rlpReceipt, however rlpReceipt is not used in checkReceiptProof => Malicious users can use whatever rlpReceipt value they want.

rlpReceipt and recValue are actually the same thing, so the rlpReceipt input paramter can be removed.
blockHashes of trx and receipts are the same (they should be). Hence, we can use just 1 of them.
If both proofs (checkReceiptProof and checkTxProof) are valid, then we can assured that the transaction and corresponding receipt is valid.


Hide  link button on link form when no link is added

Description
When there is no link added or edited, I think we don't need to show the link button on  the right side you can see the snapshots for reference, as when we click on the button it redirects us to on this URL
https://linkfree.io/api/profiles/sumitunique777/links/undefined
Screenshots

Additional information
No response

Thank you for raising 👍  But then we can only show the preview after a URL has been entered, but people will want to see how their link name is displayed before they add a url - I am not sure what the solution is, so for now I will close this issue  - hopefully they will realise if they event entered a link they can't click it to go to a link lol


Coordinate Conversion: "Copy All" button does not include the input coordinate

Widget
Coordinate Conversion
Version of widget
15 December 2016
Bug or Enhancement
When you click on "Copy All" button, see screenshot below. The input coordinate is not copied with all the output coordinates. If you're copying all the coordinates, you want to know what the input coordinate is associated with all the output values.

Clicking "Copy All" button using the coordinate shown in the screenshot above and pasting into a text files gives the following:
18.67S 023.24E
18° 40.08'S 23° 14.14'E
18° 40' 4.8"S 23° 14' 8.6"E
407FY19
34KGE3581934434
34K GE 35819 34434
34S<PHONE_NUMBER>434
In ArcMap and ArcGIS Pro addins, clicking "Copy All" will give you the following, where the first line is the input coordinate:
34KGE3581934434
18.67S 23.24E
18° 40.08'S 23° 14.14'E
18° 40' 4.8"S 23° 14' 8.6"E
407FY19
34KGE3581934434
34KGE3581934434
34K<PHONE_NUMBER>434
Repo Steps or Enhancement details

Fix please verify

@adgiles Verified fixed on build 16 Jan 2017 10:41. Closing issue.


perf: use total cmp for ordered float

TL;DR: PartialOrd impl for f32 requires two cmp instructions and PartialOrd::lt is not reduced to a single instruction. With total_cmp we avoid this overhead. See https://godbolt.org/z/ez4EaEs3W
Note: maybe we could gain more perf by implementing a NaN-less cmp?
V1:
Running query with nprobes: 1, refine_factor: 0
Time: 652.877µs

Running query with nprobes: 10, refine_factor: 0
Time: 818.418µs

Running query with nprobes: 50, refine_factor: 0
Time: 1.406433ms

Running query with nprobes: 100, refine_factor: 0
Time: 1.747243ms

Running query with nprobes: 512, refine_factor: 0
Time: 5.754752ms

V3 before perf fix
Running query with nprobes: 1, refine_factor: 0
Time: 752.659µs

Running query with nprobes: 10, refine_factor: 0
Time: 2.379638ms

Running query with nprobes: 50, refine_factor: 0
Time: 9.518057ms

Running query with nprobes: 100, refine_factor: 0
Time: 17.623498ms

Running query with nprobes: 512, refine_factor: 0
Time: 39.43036ms

V3 after perf fix
Running query with nprobes: 1, refine_factor: 0
Time: 644.933µs

Running query with nprobes: 10, refine_factor: 0
Time: 1.512715ms

Running query with nprobes: 50, refine_factor: 0
Time: 5.974101ms

Running query with nprobes: 100, refine_factor: 0
Time: 9.848213ms

Running query with nprobes: 512, refine_factor: 0
Time: 23.058848ms

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 77.58%. Comparing base (4812ac0) to head (4f96145).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3059   +/-   ##
=======================================
  Coverage   77.57%   77.58%
=======================================
  Files         240      240
  Lines       78636    78636
  Branches    78636    78636
=======================================
+ Hits        61005    61012    +7
- Misses      14496    14498    +2
+ Partials     3135     3126    -9

Flag
Coverage Δ

unittests
77.58% <100.00%> (+<0.01%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Can we (or do we) tell how many transcribers are active?

This info is included in FreeBMD newsletters and is also in the latest FreeREG newsletter. It would be very useful if we could have a similar metric for FreeCEN. I imagine it could be difficult to know in FreeCEN...

Both FreeBMD and FreeREG have always worked on the basis of transcriber uploading and processing their transcriptions (file) it is possible to define some metric for active. e.g. uploaded a file within the last year, 6 months or 3 months; or been active i.e. uploaded a file.
Since FreeCEN until very recently worked by transcribers giving their work to a coordinator who then uploaded the file and the file never identified its transcriber there are no systematic metrics.  With the advent of CSVProc for FreeCEN it is possible to get some sense of activity. I say sense because only a very few counties are actually using CSVProc. The current statistic says that 106 transcribers have uploaded a file in the year that CSVProc has been available. But who knows how many more are active the old way?

I do not think we can tell how many are active at this time with any certainty, maybe something to revisit at a later date.
Take my FreeCEN group for sample, while I have 13 I would consider active at present, only four at this time have made use of CSVProc.  Rest are not at this time for various reasons, mostly as they are transcribing large pieces which they started before FreeCEN 2 became live.  Would however expect some of these to start using CSVProc when they start transcribing under the FreeCEN2 system.
One also has to consider the length of time it takes to transcribe a piece, and also the delay in between transcribing a piece, and having it proofread

Thanks all.  I think a (very rough!) calculation could be made along the lines of:
Count the CSV transcribers who have uploaded, and the number of records: get an 'average records per transcriber' number.
Count the number of records uploaded via VLD and divide by the number of records per transcriber.
Reprocessing might make that impractical, though.
While VLD transcribers may be taking longer to complete (for very understandable reasons), they'll complete more, and previous year's transcriptions will be counted, although some this years will not be counted -  so the numbers should balance out.

Once vld uploads are eliminated we should be able to d a count of the number of volunteers who have been active on the system in the past xxx months. However as long as we have vld uploads happening this count cannot be achieved.

We can do it easily for FreeCEN2. We can define it as (for instance) the number of volunteers who have uploaded a test in the last 6 months.
The fly in the ointment is VLD files again. There is no way of knowing who worked on them. They are all uploaded by the Coordinator. The transcriber and proofreader may never have even signed in to the FreeCEN site.
Geoff

Draft spec for report sent to Denise. Awaiting reply so that we can progress this.
Geoff

Captured in 1551, closing


How to I fix invalid-input-response ?

I am getting this error.
{     "class": "Lukeraymonddowning\\Honey\\Exceptions\\RecaptchaFailedException",     "message": "The following errors were returned from the recaptcha token: invalid-input-response",     "code": 0,     "file": "/var/app/current/vendor/lukeraymonddowning/honey/src/Captcha/Recaptcha.php:37",     "trace": [         "/var/app/current/vendor/lukeraymonddowning/honey/src/Captcha/Recaptcha.php:18",         "/var/app/current/app/Http/Controllers/Auth/RegisterController.php:70",         "/var/app/current/app/Http/Controllers/Auth/RegisterController.php:76",         "/var/app/current/vendor/laravel/ui/auth-backend/RegistersUsers.php:34",         "/var/app/current/vendor/laravel/framework/src/Illuminate/Routing/Controller.php:54",         "/var/app/current/vendor/laravel/framework/src/Illuminate/Routing/ControllerDispatcher.php:45",         "/var/app/current/vendor/laravel/framework/src/Illuminate/Routing/Route.php:262",         "/var/app/current/vendor/laravel/framework/src/Illuminate/Routing/Route.php:205",         "/var/app/current/vendor/laravel/framework/src/Illuminate/Routing/Router.php:695",         "/var/app/current/vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php:128",         "/var/app/current/app/Http/Middleware/RedirectIfAuthenticated.php:26",         "/var/app/current/vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php:167",         "/var/app/current/vendor/laravel/framework/src/Illuminate/Routing/Middleware/SubstituteBindings.php:50",         "/var/app/current/vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php:167",         "/var/app/current/vendor/laravel/framework/src/Illuminate/Foundation/Http/Middleware/VerifyCsrfToken.php:78",         "/var/app/current/vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php:167",         "/var/app/current/vendor/laravel/framework/src/Illuminate/View/Middleware/ShareErrorsFromSession.php:49",         "/var/app/current/vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.
Token refreshes time is 'token_refresh_interval' => 100000,

I have the same problem.
Any suggestion / solution?


Any point to use xformers with Flux?

First of all, thanks for this great repo! I'm running out of vram on 24gb trying to train a 128 rank Flux lora. When I install xformers and try to use it, I get an error:
Traceback (most recent call last):
  File "D:\ai-toolkit\run.py", line 90, in <module>
    main()
  File "D:\ai-toolkit\run.py", line 86, in main
    raise e
  File "D:\ai-toolkit\run.py", line 78, in main
    job.run()
  File "D:\ai-toolkit\jobs\ExtensionJob.py", line 22, in run
    process.run()
  File "D:\ai-toolkit\jobs\process\BaseSDTrainProcess.py", line 1701, in run
    loss_dict = self.hook_train_loop(batch)
  File "D:\ai-toolkit\extensions_built_in\sd_trainer\SDTrainer.py", line 1483, in hook_train_loop
    noise_pred = self.predict_noise(
  File "D:\ai-toolkit\extensions_built_in\sd_trainer\SDTrainer.py", line 891, in predict_noise
    return self.sd.predict_noise(
  File "D:\ai-toolkit\toolkit\stable_diffusion_model.py", line 1650, in predict_noise
    noise_pred = self.unet(
  File "D:\ai-toolkit\venv\lib\site-packages\torch\nn\modules\module.py", line 1553, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "D:\ai-toolkit\venv\lib\site-packages\torch\nn\modules\module.py", line 1562, in _call_impl
    return forward_call(*args, **kwargs)
  File "D:\ai-toolkit\venv\lib\site-packages\diffusers\models\transformers\transformer_flux.py", line 400, in forward
    encoder_hidden_states, hidden_states = torch.utils.checkpoint.checkpoint(
  File "D:\ai-toolkit\venv\lib\site-packages\torch\_compile.py", line 31, in inner
    return disable_fn(*args, **kwargs)
  File "D:\ai-toolkit\venv\lib\site-packages\torch\_dynamo\eval_frame.py", line 600, in _fn
    return fn(*args, **kwargs)
  File "D:\ai-toolkit\venv\lib\site-packages\torch\utils\checkpoint.py", line 488, in checkpoint
    ret = function(*args, **kwargs)
  File "D:\ai-toolkit\venv\lib\site-packages\diffusers\models\transformers\transformer_flux.py", line 395, in custom_forward
    return module(*inputs)
  File "D:\ai-toolkit\venv\lib\site-packages\torch\nn\modules\module.py", line 1553, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "D:\ai-toolkit\venv\lib\site-packages\torch\nn\modules\module.py", line 1562, in _call_impl
    return forward_call(*args, **kwargs)
  File "D:\ai-toolkit\venv\lib\site-packages\diffusers\models\transformers\transformer_flux.py", line 201, in forward
    attn_output, context_attn_output = self.attn(
ValueError: not enough values to unpack (expected 2, got 1)

Hello @henryfw
So far I was reading , Xformers can have compatibility issues with flux1-dev model.
I'm currently using CUDA 12.4 and PyTorch 2.4.0 combination, without xformers. The same for comfyui.
The training with a person was successful so far (ca. 1h, 24 minutes)


Fixes for stellarTransactionId and Customers

PR Checklist
PR Structure

[ ] This PR has reasonably narrow scope (if not, break it down into smaller PRs).
[ ] This PR avoids mixing refactoring changes with feature changes (split into two PRs
otherwise).
[ ] This PR's title starts with name of package that is most changed in the PR, ex.
paymentservice.stellar, or all or doc if the changes are broad or impact many
packages.

Thoroughness

[ ] This PR adds tests for the most critical parts of the new functionality or fixes.

What
Small fixes.

Make it possible to update stellarTransactionId in patch (previously missed updates from payment observer). Also add it to getTransaction
Fix amountExpected in getTransaction (see comment)
Update customers logic. Now sender is always user account address (regardless of type). Receiver is set to null for deposit, and to withdraw_anchor_account for withdraw.

Why
To make reference server work
Known limitations
N/A

Reference Server Preview is available here:https://anchor-ref-pr757.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr757.previews.kube001.services.stellar-ops.com/

Can we have sender and receiver refer to the same public key? SEP-24 transactions are often referred to as "me-to-me" since the user is sending to themselves, making themselves the receiver too.

Reference Server Preview is available here:https://anchor-ref-pr757.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr757.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr757.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr757.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr757.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr757.previews.kube001.services.stellar-ops.com/


Error of missing patch names

Removed patches, then got an error of No schedule patch named xxxx which should be ignored.

Fixed by https://github.com/dtaniwaki/cron-hpa/commit/9f4b4384e411f4406f375951243b2871319bc15d.


Test for non-existing resources

This PR is based on work by @surilindur in PR #84 but created as a new PR to incorporate changes. PR #84 will be closed.
The original work was to add tests for operations against non-existing (fictive) resources. It also made improvements to the way test resources are created and their access controls are set up. This PR contains the original work plus an additional commit which extracts common code into a shared utils feature. It also improves performance by re-using resources that are only used in read tests. There are a couple of changes to the expected response codes that need to be highlighted too.
The tables suggest 404 in cases where the agent can know that the resource doesn't exist however it was also argued that 401/403 are valid options since the agent does not have the correct access and security issues should be processed ahead of CRUD operations.

POST to a fictive resource with inherited read access - both 403 and 404 are valid.
DELETE a fictive resource with inherited read access - both 403 and 404 are valid.

When a DELETE succeeds, CSS returns a 205 but the HTTP spec also allows 200, 202 and 204.
Lastly there are 3 cases where CSS returns a 404 which I think is incorrect and it should be 401/403. This is when the agent has W access (inheritable) to a container and attempts to delete a fictive resource. The tables make it clear that this should not be a 404 as the agent does not have R access to the container. This appears to be related to a line commented out in CSS's tables: https://github.com/CommunitySolidServer/CommunitySolidServer/blob/main/test/integration/PermissionTable.test.ts#L108
I think the next steps are to split out the write tests from here and link them to the relevant requirements in the spec, but I suggest we do that as follow-on work.
Please can you review this and then I can merge and get these tests into the test suite.

Great work! Have you also ran these tests against the other servers, or only against CSS?

I've run them against ESS too and there are a couple of issues I've raised
with the team, plus the fact that it doesn't yet implement n3 patch, of
course.
I haven't tried it on NSS.
On Mon, 10 Oct 2022, 18:53 Michiel de Jong, @.***>
wrote:

Great work! Have you also ran these tests against the other servers, or
only against CSS?
—
Reply to this email directly, view it on GitHub
https://github.com/solid-contrib/specification-tests/pull/95#issuecomment-1273645228,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACICDC3VFV2D5DVK3CESEQLWCRJZLANCNFSM6AAAAAAQ7YEYJE
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
This e-mail, and any attachments thereto, is intended only for use by the
addressee(s) named herein and may contain legally privileged, confidential
and/or proprietary information. If you are not the intended recipient of
this e-mail (or the person responsible for delivering this document to the
intended recipient), please do not disseminate, distribute, print or copy
this e-mail, or any attachment thereto. If you have received this e-mail in
error, please respond to the individual sending the message, and
permanently delete the email.

There are a number of errors with NSS, some already seen but a number of
new ones related to these codes, which do appear to be errors. One thing
we'll have to think about is whether the successful PATCH requests can also
return 200 where the test allowed 201, 204, 205.
On Tue, 11 Oct 2022, 21:46 Kjetil Kjernsmo, @.***>
wrote:

@.**** approved this pull request.
The return codes look good to me, but I think we should run this on NSS
too before merging.
Also, I can't claim to understand the setup code now, but I'm confident in
your abilities around that.
—
Reply to this email directly, view it on GitHub
https://github.com/solid-contrib/specification-tests/pull/95#pullrequestreview-1138134455,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACICDC7NUIAAQTQPFIN7OXLWCXG27ANCNFSM6AAAAAAQ7YEYJE
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
This e-mail, and any attachments thereto, is intended only for use by the
addressee(s) named herein and may contain legally privileged, confidential
and/or proprietary information. If you are not the intended recipient of
this e-mail (or the person responsible for delivering this document to the
intended recipient), please do not disseminate, distribute, print or copy
this e-mail, or any attachment thereto. If you have received this e-mail in
error, please respond to the individual sending the message, and
permanently delete the email.

There are a number of errors with NSS, some already seen but a number of new ones related to these codes, which do appear to be errors.

Alright, sounds good!

One thing we'll have to think about is whether the successful PATCH requests can also return 200 where the test allowed 201, 204, 205.

OK! It seems 200 would be OK if the server returns a representation of the resource.

I've sorted the PATCH tests to allow 200. The PUT and POST tests should also allow 200, except where creating a new resource via PUT in which case only 201 is allowed. This resolves a number of the issues with NSS and the remaining new errors are related to NSS allowing deletion of resources despite having insufficient access.
I also removed a test which was duplicated by these tests.
If there are no objections, I will merge these tests based on the existing approval by the end of the week and release a new version.

Hello all, Request to clarify on notion of the fictive resources.
As per solid-protocol, uri slash semantics are necessary but not sufficient for a containment statement to be valid. Implies,

if a/b is known to be existing contained resource,  then we can derive that a/ must be it's parent container, as the condition is necessary.
But we cannot say a/b must be a contained child resource of a/ if we only know existance of a/ container, just because of slash delimited uri path. For, uri convention satisfaction alone is not sufficient to deduce such triple. Thus a/b._acl may be aux of a/b, instead being child of a/. Or it can be totally a barred name in resource naming policy. Or may not be an ldp::Resource at all.

Thus we can say a/b is child of a/, only after minting the child node, and <a/> ldp::contains <a/b> triple in the container rep. Till then such relation doesn't hold by mere satisfaction of one of the uri conventions.
Thus if a/b doesn't exist as child of a/ yet, then any acl::default authorizations in acl of a/ doesn't apply or inherit to a/b. As, acl:default applies only to resources that are ensured-sans-heuristics as children of a/ through ldp:contains.
And until, a request target c/r is ensured to be child of c/, we should not see c/ as any relavent resource for operations targeting  c/r.
Am i missing something in this?
@edwardsph , @kjetilk , @csarven .

@damooo Many thanks for your attention to detail here! I believe that you're right here, and that we should give this proper treatment, especially in light of aux resources and how we distinguish those from normal resources.
Can I ask you to open a separate issue on this on the spec repo?

I agree that the specification should further clarify membership/containment. It was brought up before [citation needed].
I would add that we may need to allow auxiliary resources to have their own ACLs.


sokol_gfx: Ability to get underlying handles without using internal API

sokol_gfx: Ability to get underlying handles without using internal API.
It is currently possible to get the underlying handles through something like this:
info = sg_query_XXX_info(handle);
internal_data = _sg_lookup_XXX(&_sg.pools, info.slot.res_id);
internal_data->gl.YYY // or whatever

However, if you are using sokol_gfx not in C or C++ and binding to it, you cannot do this second step to get the necessary data.
One of the reasons I require this is for doing glReadPixels, since sokol_gfx does not have an equivalent.

I would recommend that you create a small "extension header" with additional functions you need to access the required private data and functions (or even create a "high level" function which wraps glReadPixels().
Basically create a small header which is split into a declaration part (which declares your own extension function), and an implementation part which has the actual code.
Then include that header in the same file where your sokol_gfx.h implementation is included, after sokol_gfx.h. That way your own extension code can access all the "guts" (private functions and data) of sokol_gfx.h, including all the backend-specific stuff.
Like this basically (for the source file which includes the implementations):
#define SOKOL_IMPL
#define SOKOL_GLCORE33
...
#include "sokol_gfx.h"
#include "sokol_gfx_ext.h"

The sokol_gfx_ext.h header would contain your own extension functions, and this can access the private data and functions of all sokol-headers included before it.
And everywhere else in the project where you need to call the extension functions, you would also include sokol_gfx_ext.h right after sokol_gfx.h to access the new public functions.

Okay, and thank you for the quick answer. That's fine by me and what I have been doing already. Just wondering behind the rationale that was all.


Use shipping recipient's name in shipping address

r? @romain-stripe
In our Payment Request integration, we use event.payerName for the shipping recipient's name, even though we don't ask for requestPayerName: true. This results in name always being null.
Since the address is used for shipping, we should use event.shippingAddress.recipient instead.

Perfect, thank you for the fix!

Oops, accidentally closed before merging! Thanks again!


ACR with private endpoint - Web Apps

Hi,
On this page: https://docs.microsoft.com/en-us/azure/container-registry/container-registry-private-link
It's in the Important section written: "Instances of certain Azure services including Azure DevOps Services, Web Apps, and Azure Container Instances are currently unable to access the container registry."
This looks for me that is not in line with what the Azure App Service Team have blog'ed about here: https://azure.github.io/AppService/2021/07/03/Linux-container-from-ACR-with-private-endpoint.html

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

ID: 8e01945a-5892-a416-716f-c499f110586c
Version Independent ID: e2aef346-266f-3e69-842d-58004084609b
Content: Set up private endpoint with private link - Azure Container Registry
Content Source: articles/container-registry/container-registry-private-link.md
Service: container-registry
GitHub Login: @dlepow
Microsoft Alias: danlep

@runemy Thanks for the feedback! I have assigned the issue to content author to check and update the document as appropriate. We will update you accordingly.

@dlepow Can you please check and add your comments on this doc update request as applicable

@runemy - Thanks for the feedback. I'm gathering more information and plan a doc update in the near future.

Thanks. Looks like that this is only supported for linux containers right now. The AppService setting WEBSITE_PULL_IMAGE_OVER_VNET=true needs to be set.

#assign: @tejaswikolli-web

Thank you for your feedback on the documentation! This article has been updated since this issue was filed. We're closing the issue at this time, but please let us know if this issue persists.
#please-close


PyPI: Very long license badge

Are you experiencing an issue with...
shields.io
🐞 Description
Per PEP 621 a Python package can be distributed with a license metadata field set to either file: link_to_license_file or text: some_license_text.
Packages that set file instead of text can end up with very long content. This seems to be valid, as demonstrated by the long version content you can see on Bokeh's PyPI page, this project setting file indeed.
🔗 Link to the badge
This leads to a very long badge :)
https://img.shields.io/pypi/l/bokeh

💡 Possible Solution
Shouldn't the badges be built from the License classifier instead of the core license metadata?

This is an amusing example :)
One issue here is that there is nothing in the JSON response from PyPI (e.g: https://pypi.org/pypi/bokeh/json ) that explicitly says whether the contents of license is file or text so we'll have to use some judgement. I'll get a PR up shortly..

For what it's worth this issue triggered a longer discussion on DPO.


Update pydantic

Thank you! I think this is covered by https://github.com/tiangolo/fastapi/pull/10344, so I'll now close this one. ☕


Rishi microsoft

Removed the variable name timeRequired.

I think you need to increase the package.json version?

I think you need to increase the package.json version?