[BUG] - signalling server didn't response to UE-App with PixelStreaming plugin

UE Version:
UE 4.27.2
Problem component
Signalling Server, Pixel Streaming plugin
Description
My goal is deploying UE-Rendering on localhost with multi instance in LAN.
First, start the signalling server by run Start_SignallingServer.ps1, then start the UE-App. I have a UE-App named VRPS.exe, but i found that VRPS.exe cannot connected with signalling server. specifically, this problem occurs accidently.
Steps to Reproduce:

Enable PixelStreaming plugin in UE4.27.2, and i have checked Additional Launch Parameters in Edit setting. here is my parameters:

-AudioMixer -PixelStreamingIP=localhost -PixelStreamingPort=8888 -RenderOffScreen

Start Matchmaker server it listening at HttpPort=100, MatchmakerPort=9999
Start Signalling server, i changed UseMatchmaker, HttpPort and StreamerPort, here is my config.json of signalling server:

{
	"UseFrontend": false,
	"UseMatchmaker": true,
	"UseHTTPS": false,
	"UseAuthentication": false,
	"LogToFile": true,
	"HomepageFile": "/www/player.html",
	"AdditionalRoutes": {},
	"EnableWebserver": true,
	"MatchmakerAddress": "",
	"MatchmakerPort": "9999",
	"PublicIp": "localhost",
	"HttpPort": 81,
	"HttpsPort": 443,
	"StreamerPort": 881
}

in terminal of matchmaker server, it shows:
09:23:40.667 HTTP listening on *:100
09:23:40.667 Matchmaker listening on *:9999
09:23:46.611 Message TYPE: connect
09:23:46.612 Adding connection for localhost with playerConnected: false
09:24:16.608 Message TYPE: ping

in terminal of signalling server, it shows:
09:23:46.595 WebSocket listening to Streamer connections on :881
09:23:46.595 WebSocket listening to Players connections on :81
09:23:46.608 Http listening on *: 81
09:23:46.611 Cirrus connected to Matchmaker :9999

Start VRPS.exe from WindowsNoEditor in terminal as following:

hesetone@LAPTOP-UPU06LAF  D:\Projects\UnrealProjects\VRPS\ApplicationPackage\ALL\WindowsNoEditor                                                     [09:27]
❯ tasklist|findstr VRPS
hesetone@LAPTOP-UPU06LAF  D:\Projects\UnrealProjects\VRPS\ApplicationPackage\ALL\WindowsNoEditor                                                          [09:29]
❯ .\VRPS.exe -AudioMixer -PixelStreamingIP=localhost -PixelStreamingPort=881 -RenderOffScreen
hesetone@LAPTOP-UPU06LAF  D:\Projects\UnrealProjects\VRPS\ApplicationPackage\ALL\WindowsNoEditor                                                          [09:30]
❯ tasklist|findstr VRPS
VRPS.exe                     12744 Console                    1      5,860 K
VRPS.exe                     14152 Console                    1    744,428 K
hesetone@LAPTOP-UPU06LAF  D:\Projects\UnrealProjects\VRPS\ApplicationPackage\ALL\WindowsNoEditor                                                          [09:30]
❯ netstat -ano|findstr :881
  TCP    <IP_ADDRESS>:881            <IP_ADDRESS>:0              LISTENING       23812
  TCP    [::]:881               [::]:0                 LISTENING       23812

it shows that VRPS.exe didn't connect with signalling server when PixStreamingPort=881. the expected behavior did not occur.
Expected behavior

in terminal of signalling server, it should output a line:

hh:mm:ss.xxx Streamer connected: ::1

Platform (please complete the following information):

OS: [Windows 10]
Browser + version [chrome  114.0.5735.199]

Additional context
i have check the task manager of Windows System, the process of VRPS.exe has started successfully, but it cannot connect with signalling server.

ohhhhhh, it is just a mistake cause by VPN.if you have setup VPN service on your platform, it is possible to cause error in connection between UE-App and Signallingserver.


Installation fails on make build due to lack of node-sass support for Node 12.x

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/741
Originally created by @gtsiolis on 2019-06-25 09:54:27

Problem
Currently, the docs do not mention a strict requirement for a Node version besides the docs section which mentions Node 8.x in https://gitlab.com/meltano/meltano/commit/b47bc4d487e8dff515bad98fc03c22c7e7e9cee4#9f7cbce127b94a94ee14460050fbff2ce58417d0_8_7, following the requirement of VuePress which is ok.

COMPATIBILITY NOTE VuePress requires Node.js >= 8.

However, when trying to contribute by installing from source, running make build fails due to lack of node-sass support for Node 12.x. This was added in node-sass 4.12.0 (see https://github.com/sass/node-sass/pull/2633).
This happens only if the user is using Node 12.x. Installation works fine when using Node 8.x and 10.x.
Proposal
Update node-sass from 4.9.0 to 4.12.0. This could also affect the prerequisites being added in https://gitlab.com/meltano/meltano/merge_requests/688.

View 4 previous comments from the original issue on GitLab


not working in 1.20 version

please fix it

I'm not sure what you mean by this, it should work with 1.20 with no problems. If you're referring to 1.20.2 however than this is due to some behind the scenes changes made in the game, and I haven't updated the mod to work with 1.20.2 yet

I just meant version 1.20 . I can't launch Minecraft with your mod on it. Writes that it's a mod Pyro frost requires version 1.20.1 . I have the latest version fabric loader

This mod is very important for me, because I want to include it in my build (if you don’t mind)

I definitely meant version 1.20. My Minecraft won't launch with your mod. Error: Mod pyrofrost Requires Minecraft version 1.20.1. I have the latest version fabric loader

I definitely meant version 1.20. My Minecraft won't launch with your mod. Error: Mod pyrofrost Requires Minecraft version 1.20.1. I have the latest version fabric loader

Are you sure you didn't accidentally install the 1.20.1 version of the mod?

Oh yeah, it's because it's strictly set for 1.20.1 in the fabric.json file. That's probably why it isn't working then.


Times out when using states (Alexa.CreateStateHandler)

I'm using states in my handlers. In my event for the test, I add STATE: '_READY' to attributes. As soon as I do this, the test throws a timeout error.

Never mind. I needed to add

I never used Alexa and or Lambda to create an Alexa Skill (is this what it is called?).So it might be that the mock context doesn't support some things. If you could show me the docs and also a little example of what you are doing, I might be able to see what's going wrong.

https://paste.laravel.io/KLn6o
This test passes as we're calling the SayHello intent directly. On the other hand, If you go to line 88 and change the intent name to NewSession, the test will fail as SayHello is never called. In an Alexa skill, this.emit('SayHello') would call SayHello. That doesn't happen in the test and I get the timeout error below:
Testing Hello
1) "before all" hook

0 passing (2s)
1 failing

1) Testing Hello "before all" hook:
 Error: Timeout of 2000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.


Special Instruction while inferencing locally deployed model

Deployed SQLCoder-34B Locally, But couldn't provide special instructions as shown in Demo. Where & How can I write special instructions ?

If i am not wrong the special instructions appears in the Instructions section, This notebook in the repo contains the inference of the model with the schema and question, where Adhere to these rules: is what ig to be the special instructions section.

Hi there! You can add instructions anywhere in the prompt you construct – either in the instructions section as @Kirushikesh mentioned, or elsewhere in the prompt. The exact location will vary depending on your specific use-case.

Also @rishsriv, thanks for the reply. What is the actual prompt format used for finetuning the codellama model, mostly i found the Task/Database Schema/Answer, but in the notebook i found a new format of Instructions/Input/Response.

It's a mixture of formats – you should pretty good results with the file in prompt.md


Realm delete works only for empty registrations

when you try to delete a realm it gives the error "invalid type" but I think the problem is that
the realm must no longer have identiy providers, users, clients configured before it can be deleted.
rendere più chiaro un messaggio di errore "invalid type"
in fact, if you delete the components in the realm, the realm get deleted with the delete function

Thanks for the report. The message is generic because there is a bug in the removal.
There is no need to clean up a realm before removing it: everything related should be contextually removed.
This is a bug.


Introduce a new not-passable-entity detector

Such as yellow solid line. @lqy0057

But not traffic vehicles!

We have Lane detector, Side detector and Vehicle Lidar now!


Implement User Class

Implement the User class with necessary attributes.

[ ] Create the User class.
[ ] Define the attributes: id, name, familyName, fractions.


Introduce Feathub configuration infra

What is the purpose of the change
This PR introduces infrastructure for configuration to better manage all the configuration available in FeatHub.
Brief change log

Introduces infrastructure for configuration

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

@lindong28 Could you review this PR?

@lindong28 Thanks for your review! I updated the PR. Could you please take another look?

Thanks for the update! LGTM.


JQuery Error when Editing Finding

When editing a finding, there is a JS JQuery error. For example, when clicking on "alerts" ring at the top right corner of the page, content won't be displayed.

https://github.com/DefectDojo/django-DefectDojo/pull/6619


[Asgardeo to Choreo Hub] Improve the capabilities for debuging production issues

Description:

$subject

Completed the task with following PRs:

https://github.com/wso2-enterprise/azure-websubhub/pull/4
https://github.com/wso2-enterprise/azure-websubhub/pull/5


In R-CNN evaluation, key 'binary_cls' does not exist

Hello. I'm trying to replicate the R-CNN results on VOC/COCO but I'm running into an issue with the voc_coco_vmf.py and voc_coco_knn.py metric computations.
The line:
id_val_data = torch.stack(id_val_data['binary_cls']).cpu()
Errors because there is no binary_cls key stored in the probabilistic_scoring_res_odd_{}.pkl files.
How is this computed?
Thanks.

hi, @MatthewInkawhich , thanks for the info! It is indeed a bug, I have fixed it.

please retry and let me know if there are any further issues on that.


⚠ GitLab Server has degraded performance

In 5035fb8, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 13974 ms

Resolved: GitLab Server performance has improved in 5360668.


Remove unused design-system pieces

Highlights:

All table js has been removed. Table css associated with row-links and sortable tables has been removed.
dom-traverse.js has been removed, relying instead on browser native methods
the signature of multiselectutils' create method was simplified
Other minor removals

Any interest in spinning out separate PRs from this for the logical blocks (tables, multiselect, utilities) for easier review and diagnosing the test failures? I can if you're busy wrapping up loose ends.

Yeah that makes sense to me, will do this afternoon


Updates and improvements

I updated this a bit :) This should take care of #1

Mind giving us a video tutorial on how to use it ? Just for the noobs between us, please.

Videos don't usually link well on Github, but I did update the README though. There is now a config.json file to put the e-mail address into. That allows for each script to get the target e-mail from one location. All of the required modules are now placed into a file named requirements.txt. This allows everything to be installed with one command pip install -r requirements.txt. From there, you just run the scripts (python file.py).

Can I ask about implementing this on Linux? :)

@retropele You should run pip install -r requirements.txt and all of the python dependencies should be installed *provided that you already installed Python.
For setting up Selenium, you should consult a source that has information relevant to the specific flavor of Linux that you are using. (For instance, if you're using Arch, consult the Arch wiki ect.)


Euchre: Bid alone more often

Fix #12
Also updates the bot to account for receiving something worthwhile from partner in call-for-best.
Lastly updates bot to be a bit more cautious in call-for-best scenarios with unprotected lower cards in an off-suit.

There's one additional change to the play-of-the-hand I may try to address before this lands. Currently if the bot is missing the highest trump it always leads low from the trump suit. This causes it to miss making potentially winnable alone hands.
If the bot has bid alone, the opponents haven't taken a trick yet, and the bot is leading, it should lead its highest trump to maximize the chances of taking all tricks (in case the high trump is either in partner's hand or the blind).


fix: append coreAnalyzer if active_filter is empty and integration is added

Closes #412
📑 Description

✅ Checks

[x] My pull request adheres to the code style of this project
[ ] My code requires changes to the documentation
[ ] I have updated the documentation as required
[ ] All the tests have passed

ℹ Additional Information

(matthisholleville) ➜  k8sgpt git:(fix/integration-and-filters) go run main.go filters list
Active:
> PersistentVolumeClaim
> Ingress
> Node
> Deployment
> ReplicaSet
> StatefulSet
> CronJob
> Pod
> Service
Unused:
> HorizontalPodAutoScaler
> PodDisruptionBudget
> NetworkPolicy
(matthisholleville) ➜  k8sgpt git:(fix/integration-and-filters) go run main.go integration activate trivy
...
Activated integration trivy
(matthisholleville) ➜  k8sgpt git:(fix/integration-and-filters) go run main.go filters list
Active:
> Deployment
> Ingress
> Node
> Service
> CronJob
> ReplicaSet
> PersistentVolumeClaim
> VulnerabilityReport (integration)
> Pod
> StatefulSet
Unused:
> HorizontalPodAutoScaler
> PodDisruptionBudget
> NetworkPolicy

👍


Update nf-fileapi-writefile.md

Fix broken link to Synchronization and File Position section

@ryan-weil : Thanks for your contribution! The author(s) have been notified to review your proposed change.


How to add extra options (a la "allow_extra_args", etc.) to a command's help?

First Check

[X] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the Typer documentation, with the integrated search.
[X] I already searched in Google "How to X in Typer" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to Typer but to Click.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
import typer

app = typer.Typer

@app.command(
    context_settings = {"allow_extra_args": True, "ignore_unknown_options": True}
)
def main(
    ctx: typer.Context,
    some_common_option: str = typer.Option(default='', help='Some Help'),
  ):
  """This is the command's overall description
  """
  handle_extra(ctx.args)

# I do this to work around the lack of support of multiple, multi-valued options (that is, repeated multi-valued options, like `--set-program-opt <program> <opt> <value>`)

Description
The above example should already output some nice help for the existing options.
I know that proper support for this sort of thing has been requested (or at least asked about; #387 for instance), but it's also possible to do things even crazier than that by handling your own extra options/arguments (variable multi-valued options? IDK people do some crazy things... And I'm one of them.)
The question is, then, is there a simple method of adding additional help bits on this command to properly document the extra options I am adding?
Operating System
Linux
Operating System Details
No response
Typer Version
0.6.1
Python Version
3.9.7
Additional Context
No response

I guess this could also extend to completion, but one thing at a time 😅

This is a big hack, but I did something similar by using Python's exec to invoke dynamically generated command line content. Something similar might work for you If you can get past the ugliness (and this is really ugly).
https://github.com/davidbrownell/v4-Common_Foundation/blob/c37f42ae79a00fd368cc3b3d41da0f47e2bd4f9c/Libraries/Python/Common_FoundationEx/src/Common_FoundationEx/BuildImpl/__init__.py#L246

Nop, there's no way to do dynamic things like this. But I think that for your use case, the first think would really be request for support for multiple multi-valued options, right?

Right; the easier? Better? option would be to implement support for multiple multi-valued options, as in #459 (or #362).  Would you be willing to give those a look?  Thanks!

Thanks! I'll check that out. 🤓


override the version number that is read from pubspec.lock file for all native plugins when generating the local Maven repository.

Fixed https://github.com/flutter/flutter/issues/152538
override the version number that is read from pubspec.lock file for all native plugins when generating the local Maven repository.
This pull request addresses this issue by obtaining the version number from the pubspec.lock file, which contains the actual version numbers of plugins in pub.dev. When determining the version number that should be used in the AAR file, most plugin's defined versions are set to '1.0' in their respective android/build.gradle files. To avoid significant changes, this pull request selects the plugin's version published on Pub server.

There is no issue linked to this pull request. If necessary, I could create one.
This pull request has no changes in the flutter/tests repo. If a test is necessary, I could create one.
Pre-launch Checklist

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

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

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
View this failed invocation of the CLA check for more information.
For the most up to date status, view the checks section at the bottom of the pull request.

I've changed my git mail to the correct one.

@reidbaker  Could you please help review this pull request? Thanks!

I am trying to get a better understanding of when pubspec.lock files are expected to change.

I am trying to get a better understanding of when pubspec.lock files are expected to change.

The rule of package management for Pub dictates that the pubspec.lock file stores the version numbers of every library used. Therefore, if I change the version number in pubspec.yaml and then run flutter pub get, or upgrade all upgradable packages with flutter pub update, it may affect the version constraints in pubspec.lock. Eventually, this may change the content of pubspec.lock file.

This pull request and the issue are a bit hard to follow. I am going to try to reproduce the issue on my machine to see if that helps me understand the issue better.
In general we do not like to add dependencies we do no control. Especially for code used to build assets our developers use.
Right, perhaps I could use a simplified function to process YAML files instead of this gradle library.

Also passing in a boolean for "isModuleProject" implies more than is needed for configureProject. I suggest renaming the booling to something more tightly scoped to what happens when the boolean is true/false.
Should I use the project name as a parameter for the function "configureProject"? Something like :flutter or :app?

Finally this pull request, as written, has confused multiple contributors and I get the sense that the pr description and the bug were written in a hurry. To help this pr land assume the reviewers have a broad knowledge of flutter but have less experience in this specific area when communicating what is changing or what needs to be changed.
Got it, I'll add more descriptive details to the issue and PR descriptions.

@reidbaker Is the last comment here enough to move forward with? Or, is this PR stuck and should we consider closing this?

I believe most plugin maintainers should and will update the plugin's version number when they modify the code and re-publish it.

It's impossible to re-publish the same version; I'm talking about cases like editing the source of a path based plugin, or using an unpinned git source for a plugin, where the source can easily change without the version changing.

Even for locally sourced plugins, running flutter build aar triggers the gradle assemble task, which always uses the latest code to generate the aar file.

But then under your proposed change, you would have different build output with the same version number, which directly violates the purpose of build-number.

I don't have a good solution for this https://github.com/flutter/flutter/issues/152538#issuecomment-2265075353 at the moment. Personally, I think if users are including dependencies via a git path or a local path, they should be responsible for updating the uploaded aar file after each build. Alternatively, perhaps we could add a build flag to 'flutter build aar' that forces all dependencies to use the version number specified by the '--build-number' flag?

Alternatively, should we include more details in the generated pubspec.lock file, such as dependency type? This would allow us to apply the version number passed from --build-number to both git based and path based dependencies.

Hello. Unfortunately, it looks like we aren't able to make progress with this PR due to its complexity and available reviewer bandwidth. Thank you for your contribution.


linux-64 gsl 2.7 version: libgsl.so links to libgsl.so.25.1.0

Solution to issue cannot be found in the documentation.

[X] I checked the documentation.

Issue
This is a problem in the tarball posted at https://anaconda.org/conda-forge/gsl/files.
As noted elsewhere:

https://github.com/bluesky/libhkl/issues/20

linux-64/gsl-2.7-he838d99_0.tar.bz2
When the tarball is expanded (by conda install), its include file says it is version 2.7 (as expected) but the libgsl.so file links to libgsl.so.25.1.0.
These steps to reproduce (in a clean VM):
conda create -y -n test
conda activate test
conda install -y conda-forge::gsl
grep VERSION $CONDA_PREFIX/include/gsl/*.h
ls -lAFgh $CONDA_PREFIX/lib | grep libgsl.so

The tarball has the incorrect content:
(gsl) root@683c40875983:/opt/conda/pkgs/gsl-2.7-he838d99_0# grep VERSION ./include/gsl/*
./include/gsl/gsl_complex.h:     defined(__STDC_VERSION__) && (__STDC_VERSION__ >= 201112L) /* C11 */
./include/gsl/gsl_version.h:#ifndef __GSL_VERSION_H__
./include/gsl/gsl_version.h:#define __GSL_VERSION_H__
./include/gsl/gsl_version.h:#define GSL_VERSION "2.7"
./include/gsl/gsl_version.h:#define GSL_MAJOR_VERSION 2
./include/gsl/gsl_version.h:#define GSL_MINOR_VERSION 7
./include/gsl/gsl_version.h:#endif /* __GSL_VERSION_H__ */
(gsl) root@683c40875983:/opt/conda/pkgs/gsl-2.7-he838d99_0# ls -lAFGh ./lib/libgsl.so*
lrwxrwxrwx 1 root   16 Jul 18 19:07 ./lib/libgsl.so -> libgsl.so.25.1.0*
lrwxrwxrwx 1 root   16 Jul 18 19:07 ./lib/libgsl.so.25 -> libgsl.so.25.1.0*
-rwxr-xr-x 2 root 3.4M Jul 15  2021 ./lib/libgsl.so.25.1.0*

Installed packages
(gsl) root@683c40875983:/opt/conda/pkgs/gsl-2.7-he838d99_0# conda list
# packages in environment at /opt/conda/envs/gsl:
#
# Name                    Version                   Build  Channel
_libgcc_mutex             0.1                 conda_forge    conda-forge
_openmp_mutex             4.5                       2_gnu    conda-forge
gsl                       2.7                  he838d99_0    conda-forge
libblas                   3.9.0           22_linux64_openblas    conda-forge
libcblas                  3.9.0           22_linux64_openblas    conda-forge
libgcc-ng                 14.1.0               h77fa898_0    conda-forge
libgfortran-ng            14.1.0               h69a702a_0    conda-forge
libgfortran5              14.1.0               hc5f4f2c_0    conda-forge
libgomp                   14.1.0               h77fa898_0    conda-forge
libopenblas               0.3.27          pthreads_hac2b453_1    conda-forge

Environment info
(gsl) root@683c40875983:/opt/conda/pkgs/gsl-2.7-he838d99_0# conda info

     active environment : gsl
    active env location : /opt/conda/envs/gsl
            shell level : 2
       user config file : /root/.condarc
 populated config files : /opt/conda/.condarc
          conda version : 24.5.0
    conda-build version : not installed
         python version : 3.10.14.final.0
                 solver : libmamba (default)
       virtual packages : __archspec=1=skylake
                          __conda=24.5.0=0
                          __glibc=2.31=0
                          __linux=5.15.0=0
                          __unix=0=0
       base environment : /opt/conda  (writable)
      conda av data dir : /opt/conda/etc/conda
  conda av metadata url : None
           channel URLs : https://conda.anaconda.org/conda-forge/linux-64
                          https://conda.anaconda.org/conda-forge/noarch
          package cache : /opt/conda/pkgs
                          /root/.conda/pkgs
       envs directories : /opt/conda/envs
                          /root/.conda/envs
               platform : linux-64
             user-agent : conda/24.5.0 requests/2.31.0 CPython/3.10.14 Linux/5.15.0-112-generic ubuntu/20.04.6 glibc/2.31 solver/libmamba conda-libmamba-solver/24.1.0 libmambapy/1.5.8
                UID:GID : 0:0
             netrc file : None
           offline mode : False

This is interesting, indeed the soversion should be 26 according to http://git.savannah.gnu.org/cgit/gsl.git/diff/configure.ac?h=release-2-7&id=659dcc296cc31532d133d5374ce1baf82f280610 . Perhaos something is going wrong when we call autoreconf ? I am definitely not an autotools expert, but we can look in cleaning this in https://github.com/conda-forge/gsl-feedstock/pull/60 (we can't just change the soversion of 2.7 as it would result in a ABI break for gsl 2.7).
However, while it would be interesting to understand why this is happening, this should not a be a problem in conda-forge, as you can't have multiple gsl version installed in the same conda environment, so even if two different versions have the same soversion number, I do not think it should be a problem. What kind of failure you are experiencing related to this?

Just to clarify, the soversion 25 is the one of gsl 2.6 (see http://git.savannah.gnu.org/cgit/gsl.git/tree/configure.ac?h=release-2-7&id=659dcc296cc31532d133d5374ce1baf82f280610), not the one of gsl 2.5 .

What happens is that my package (libhkl) fails to run its functions that require libgsl because it cannot find the expected libgsl.so.27.  And stops cold dead.
I don't know about soversion 26, this issue is about the linux-64/gsl-2.7 he838d99 tarball.  Likely it is mislabeled (which AFAIK happens during autoreconf).  My package works when I rename the libgsl.so.25.0.0 file to libgsl.so.27.0.0.
With so many downloads of the gsl-2.7 package, it surprises me that I am first to identify this.  Double-checked to verify that my report is based on download file from anaconda.org.

stops cold dead

Actual traceback is in the details dropdown of this issue comment:
https://github.com/bluesky/libhkl/issues/20#issue-2409824154

What happens is that my package (libhkl) fails to run its functions that require libgsl because it cannot find the expected libgsl.so.27. And stops cold dead.

But your software it expects libgsl.so.27 ? If you are building it against the conda-forge's gsl 2.7.0 package, you should expect libgsl.so.25 or whatever soversion we used. How you end up requiring .27, also considering the .27 is the soverion of 2.7.1, not of 2.7.0 .

With so many downloads of the gsl-2.7 package, it surprises me that I am first to identify this. Double-checked to verify that my report is based on download file from anaconda.org.

Indeed, I regularly run software that links gsl 2.7, so I am surprised as well, and I suspect there is something different in your software w.r.t. to other consumers.

Could be.  I'm not building with the conda-forge compilers and that might be part of the problem (that my package expects libgsl.so.27 instead of just libgsl.so).  (I'm building my package on Debian with the packages downloaded via apt install.)
BUT ... Ignoring my software completely,
Download the described gsl-2.7 package from conda-forge and open it up.  It contains a file libgsl.so.25.1.0.  That's the problem.  Here, I show the tarball downloaded from anaconda.org as opened in Window's 7-zip program:

Compare with linux-64/gsl-2.7.1 from the anaconda channel:

My package works with this one.

This looks like the same situation: https://github.com/samtools/bcftools/issues/1698

Ok, first I will try to do a bit of recap.
I started looking a bit more in the soversion situation, and I think it is correct that 2.7 has the 25 soversion. Indeed, if you check in http://git.savannah.gnu.org/cgit/gsl.git/tree/configure.ac#n10, you see that the soversion is actually reset when the GSL_AGE is reset to 0, so for the latest releases we have the current situation:

gsl version
libgsl version
libgsl soversion
file list in a linux distro  to verify the soversion

gsl-2.6
libgsl 25:0:0
25
https://pkgs.alpinelinux.org/contents?branch=v3.12&name=gsl&arch=x86_64&repo=community

gsl-2.7
libgsl 26:0:1
25
https://pkgs.alpinelinux.org/contents?branch=v3.14&name=gsl&arch=x86_64&repo=community

gsl-2.7.1
libgsl 27:0:0
27
https://pkgs.alpinelinux.org/contents?branch=v3.16&name=gsl&arch=x86_64&repo=community

gsl-2.8
libgsl 28:0:0
28
https://packages.debian.org/experimental/arm64/libgsl28/filelist

As a reference for the soversion name I used alpine and debian, but you can check yourself in any distro by looking at https://repology.org/project/gsl/versions .
Indeed, the 2.7 case is a bit strange as the README reported that "(***) There was an error on this release. Age should have been reset to 26:0:0". So in theory the correct soversion for 2.7 should have been 26 instead of 25, but we can't change it without breaking existing packages in conda-forge built against gsl 2.7.
So, having said that, it seems to me that gsl 2.7 containing libgsl.so.25 make sense and is aligned with the rest of linux distributions, unless I am missing something.

Could be. I'm not building with the conda-forge compilers and that might be part of the problem (that my package expects libgsl.so.27 instead of just libgsl.so). (I'm building my package on Debian with the packages downloaded via apt install.)

Probably I am missing something on this, but how does that work? If you are building a library against gsl 2.7.1 in debian, why it should work when you install gsl 2.7 (not 2.7.1) from conda-forge? And even if you use exactly the same version of the dependencies, there is nothing that ensures (or promises) that the ABI between the Debian-built library and the conda-forge-built library is the same.
Just to understand, why don't you simply build the package from source in https://github.com/conda-forge/hkl-feedstock/blob/main/recipe/build.sh from the conda-forge dependencies instead of doing a binary repackaging of binaries built on Debian against debian provided dependencies?

This looks like the same situation: samtools/bcftools#1698

I am not sure that is related, it is not even clear if they refer to a conda package or not.

Your table is very helpful to understand that the version (and soversion) numbers of gsl and libgsl are not identical.
There are plans to refactor the hkl-feedstock in a future release to use the conda-forge compilers.
Thanks for your assistance.


various contact-bundle fixes

changed contact bundle according to bernds suggestions
tasks:

[x] changed datagrid view-decorator translations (images to cards, table to list)
[x] removed sulu-signet from contact cards
[x] made avatar field default in contact controller
[x] hidden info fielt in contact card if no info is present
[x] activated selected counter for category-datagrid in contact-detail and account-detaul
[x] shrinked navigation dropdown padding (in husky) and imported new husky
[x] replaced default avatar image with fa-icon
[x] replaced login-success icon with svg
[ ] test coverage
[ ] gather feedback for my changes

informations:

q
a

Fixed tickets
None

BC breaks
None

Documentation PR
None

@nnatter acknowledged. Please make it mergeable


doc: make error checking in ticket handling code explicit

The example code doesn't show that MAC and ecryption methods require error checking.
While this wasn't really a problem in the pre-provider days, now any calls can fail, so examples should be explicit about error checking.

Checklist

[x] documentation is added or updated

sanitizers run failing in "test_threads with FIPS" doesn't seem to be related to my changes...

@paulidale yes, I know; problem is that omission of error checking is not obvious to everybody. With words like "Reference Implementation", users are not likely to question lack of error checking.
So I think we should be more explicit about it; if you'd prefer a general "Error checking and memory management omitted for readability." before all examples I could add it too...

This pull request is ready to merge

Merged to master. Thank you for your contribution.


Logs from hs fail to propogate up into Google Cloud logging

https://cloud.google.com/python/docs/reference/logging/latest/handlers-structured-log#google.cloud.logging_v2.handlers.structured_log.StructuredLogHandler

we added loggers to stdout in https://github.com/CUAHSI/platform-recipes/pull/23
but every log shows up as error level in GCP
so need to use
https://cloud.google.com/python/docs/reference/logging/latest/handlers-structured-log#google.cloud.logging_v2.handlers.structured_log.StructuredLogHandler


Switching channel is not correctly described

File: docker-for-mac/install.md
"Uninstall the current version of Docker Desktop and install a different version (Stable or Edge), or reset your VM disk."
The suggestion "reset your VM disk" as a way to switch channel (e.g. from Edge to Stable) does not give any useful information. In particular, if I reset the VM image in the Docker (edge) settings then nothing obvious happens that would let me switch back to Stable channel.

Thanks for sharing the issue. This is no longer part of the topic.


feat: export TemplComponent interface.

Pretty much the title. ^

If you're already using and importing Templ in your project, wouldn't it just be better to use the templ.Component interface which is the exact same?
By having users depend on htmx.templComponent on my library, whenever Templ makes a change to the templ.Component interface we have to maintain it always because projects might break, as opposed to only Response.RenderTempl() breaking.


Add funhouse-noota build target

@tyeth  I merged the NOOTA build configuration into CI. This PR is re-running as it previously failed on NOOTA.
I will review this when tagged for approval

Adjusted as discussed @brentru, ready for review


vector

Hi
How to use vector in circle image view????
Thanks

You shouldn't. Please have a look at my comment on #138 as to why this is.

README has been updated with info on this.


Wrong Error message in design_matrix module

l.317. Add a test to enforce correct behavior.

can you please explain what is the wrong error message? I can take care of this.

look at nistats.design_matrix.py , l.317, the error message is ill-formed.


Implement Youtube-DNN ranking model

🌟 New model addition
Model description
The youtube-dnn paper defined a two-stage recommender and proposed  two architectures for retrieval and ranking steps. The purpose of this request is to add the ranking architecture of Youtube-DNN (Figure 7 of the paper) to merlin.models.

Open source status

[ ] the model implementation is available: (give details)
[ ] the model weights are available: (give details)
[ ] who are the authors: (mention them, if possible by @gh-username)

tracked via https://github.com/NVIDIA-Merlin/Merlin/issues/279

closing this ticket as the scope is more detailed in this RMP ticket


urxvt symbola

Hi friends,
I've been trying the setup described on the readme page for URXVT so:
URxvt.font: xft:DejaVuSansMono:size=12,xft:symbola:size=10:minspace=False

I run Arch Linux so I installed the package "ttf-symbola" from the AUR using yay.
It is not giving me the nice cloud icons when i call
curl "wttr.in/Sydney?format=v2"

Would you have any idea what I m doing wrong?
I also tried letterspace=0 but no success.
Basically I get empty boxes or weird characters instead of clouds. A couple of small clouds appear though.

@The-Sound: I believe, we need your help again. Your instructions from https://github.com/chubin/wttr.in/commit/e1ccc04f3a7a2b532022f908340c006b6058969d seem to miss some point. Do you have any ideas?

After testing a while, I'm gonna go ahead and say it is a URXVT related issue. See, URXVT has the chance of taking fallback fonts on configuration, but sometimes  it fails to change them. In this particular case, symbola has a few glyphs, thus, is chosen to appear, but it lacks all the glyphs, so you're seeing boxes.
If you want to test it, run :
urxvt -fn "xft:symbola:pixelsize=10"
This will open a new instance of URXVT with the picked font, if you use curl "wttr.in/Sydney?format=v2" you will see the boxes aren't there anymore.
The only way I can thing to fix it from Igonr's side (which is not up to him, since this is an URXVT issue) would be that if it detects a font with broken characters, it won't load. But again, this is a stab in the dark and not wttr.in issue.

that if it detects a font with broken characters, it won't load

We could add an option for that; the server itself can't detect it of course, but it possible, that the user specifies an additional option for this. But the question remains: what should be displayed instead of the emojis in this case?
Maybe we should contact urxvt developers and try to clarify it with them?

I think it would be the best option. Tried to search some issues about it but I must be searching something wrong. The primal issue is, when URXVT needs to show a double sized character, it won't. In theory, is an URXVT thing because the actual characters from Symbola are single sized. That means, they shouldn't be double the size you have on your .Xresources. I was testing it again and if you put URxvt*font: xft:symbola:size=10 on your .Xresources file, you can check that if you double click them after asking for curl wttr.in/Sydney?format=v2 they will show [?] on the double sized them, just like this:

For some reason, that won't happen if you go like the above settings main font: 10; symbola font: 5 because when trying to double it up, won't be bigger than the main font.

I've found this discussion, where they are talking about colourful emoji fonts support:
http://lists.schmorp.de/pipermail/rxvt-unicode/2020q1/002553.html
It does not look like the patch will be applied, and the support
will be added to URXVT soon, but who knows.
That would be a possible solution to our problem at least.
@The-Sound I think that the problem is even a little bit bigger than you think.
In some of this cases it is not double-width emojis as you maybe think, but it
is multibyte grapheme clusters, which I have no idea how could be displayed
in terminals without emoji support (and without grapheme clusters support I assume).
So I think we need to add simple emojiless unicode support to wttr.in first,
and then experiment with it. Because otherwise we have almost no chance that
we manage to display it in urxvt (and similar terminals) properly


feat(create_rspress): support cancel operation

Summary
Hi~ I noticed that create-rspress cannot exit during creation. I hope this could help.
Related Issue

Checklist

[x] I have added changeset via pnpm run change.
[ ] I have updated the documentation.
[ ] I have added tests to cover my changes.

Thank you

Maybe you can add changeset for patch change by the following tip:

Maybe you can add changeset for patch change by the following tip:

Yes, sure thing. thank you for reminder~

Hi~ I added changeset, if anything is inappropriate just let me know.


This is doing a top-up every start, which transfers some values of tokens to the bot's accounts.
The things left to do are:

Manage spammers' balances.


fix(deps): :arrow_up: update all non-major dependencies

This PR contains the following updates:

Package
Change
Age
Adoption
Passing
Confidence

@sentry/angular-ivy (source)
^7.80.0 -> ^7.80.1

aws-cdk
^2.108.0 -> ^2.108.1

Release Notes

getsentry/sentry-javascript (@sentry/angular-ivy)
v7.80.1
Compare Source

fix(astro): Adjust Vite plugin config to upload server source maps (#9541)
fix(nextjs): Add tracing extensions in all serverside wrappers (#9537)
fix(nextjs): Fix serverside transaction names on Windows (#9526)
fix(node): Fix tRPC middleware typing (#9540)
fix(replay): Add additional safeguards for capturing network bodies (#9506)
fix(tracing): Update prisma span to be db.prisma (#9512)

aws/aws-cdk (aws-cdk)
v2.108.1
Compare Source
Bug Fixes

elasticloadbalancingv2: access logging fails on imported bucket (#27948) (8243601), closes #27928

Alpha modules (2.108.1-alpha.0)
v2.108.0
Compare Source
Features

lambda: add python 3.12 runtime (#27967) (505907a)

Alpha modules (2.108.0-alpha.0)
v2.107.0
Compare Source
Features

lambda: node20 runtime (#27897) (a5acd0f)

Alpha modules (2.107.0-alpha.0)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

⚠ Artifact update problem
Renovate failed to update an artifact related to this branch. You probably do not want to merge this PR as-is.
♻ Renovate will retry this branch, including artifacts, only when one of the following happens:

any of the package files in this branch needs updating, or
the branch becomes conflicted, or
you click the rebase/retry checkbox if found above, or
you rename this PR's title to start with "rebase!" to trigger it manually

The artifact failure details are included below:
File name: package-lock.json
npm WARN cli npm v10.2.4 does not support Node.js v20.2.0. This version of npm supports the following node versions: `^18.17.0 || >=20.5.0`. You can find the latest version at https://nodejs.org/.
npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/@angular/core
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/@angular/animations
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
npm ERR!
npm ERR!
npm ERR! For a full report see:
npm ERR! /tmp/renovate/cache/others/npm/_logs/2023-11-17T21_48_46_428Z-eresolve-report.txt

npm ERR! A complete log of this run can be found in: /tmp/renovate/cache/others/npm/_logs/2023-11-17T21_48_46_428Z-debug-0.log


"Node 'Const' is not unique" Using C++ api on windows to freeze graph

I use FreezeSavedModel fucntion in source code https://github.com/tensorflow/tensorflow/blob/v1.12.0/tensorflow/cc/tools/freeze_saved_model.cc to freeze graph, and use tensorflow::WriteBinaryProto to export pb file. it report an error: "Node 'Const' is not unique". Then I check code in FreezeSavedModel fucntion, I find out in function ConvertVariableToConstant(https://github.com/tensorflow/tensorflow/blob/v1.12.0/tensorflow/cc/tools/freeze_saved_model.cc#L149),   after running
 const_node->set_op("Const");

the op and name and device of const_node all are set to "Const". And I check the address of member op_, name_, device_ of const_node, they are all the same.
Similarly, if you runconst_node->set_name(variable_node.name()); , the op and name and device of const_node all are set to name of this variable_node.
System information

Windows 10
TensorFlow installed from (source or binary): source
TensorFlow version (use command below): v1.12.0
Bazel version (if compiling from source): bazel 0.15
CUDA/cuDNN version: 10/7
GPU: 1050ti

I use vs2015 update3 to compile tensorflow and add some msvc symbols manually.

In order to expedite the trouble-shooting process, please provide a code snippet to reproduce the issue reported here. Thanks!

tensorflow::SavedModelBundle saved_model_bundle;
tensorflow::MetaGraphDef *meta_graph_def = &(&saved_model_bundle)->meta_graph_def;
*meta_graph_def->mutable_graph_def() = graph_def;
saved_model_bundle.session->Create(graph_def);
saved_model_bundle.session->Run({ { "save/Const", t } }, {}, { "save/restore_all" }, nullptr);
tensorflow::GraphDef frozen_graph_def;
std::unordered_set<string> inputs;
std::unordered_set<string> outputs;
TF_CHECK_OK(FreezeSavedModel(saved_model_bundle, &frozen_graph_def,
		&inputs, &outputs));

the definition of FreezeSavedModel is in https://github.com/tensorflow/tensorflow/blob/v1.12.0/tensorflow/cc/tools/freeze_saved_model.cc

tensorflow::SavedModelBundle saved_model_bundle;
tensorflow::MetaGraphDef *meta_graph_def = &(&saved_model_bundle)->meta_graph_def;
*meta_graph_def->mutable_graph_def() = graph_def;
saved_model_bundle.session->Create(graph_def);
saved_model_bundle.session->Run({ { "save/Const", t } }, {}, { "save/restore_all" }, nullptr);
tensorflow::GraphDef frozen_graph_def;
std::unordered_set<string> inputs;
std::unordered_set<string> outputs;
TF_CHECK_OK(FreezeSavedModel(saved_model_bundle, &frozen_graph_def,
		&inputs, &outputs));

the definition of FreezeSavedModel is in https://github.com/tensorflow/tensorflow/blob/v1.12.0/tensorflow/cc/tools/freeze_saved_model.cc

Hey, try to edit your freeze_saved_model.cc.
before:
void ConvertVariableToConstant(const NodeDef& variable_node,
const Tensor& variable_value,
NodeDef* const_node) {
const_node->set_name(variable_node.name());
const_node->set_op("Const");
(*const_node->mutable_attr())["dtype"] = variable_node.attr().at("dtype");
variable_value.AsProtoTensorContent(
(const_node->mutable_attr())["value"].mutable_tensor());
}
change:
void _ConvertVariableToConstant(const NodeDef& variable_node,
const Tensor& variable_value,
NodeDef const_node) {
*const_node = variable_node;
const_node->set_name(variable_node.name());
const_node->set_op("Const");
(*const_node->mutable_attr())["dtype"] = variable_node.attr().at("dtype");
variable_value.AsProtoTensorContent((*const_node->mutable_attr())["value"].mutable_tensor());
}
It is worked for me.

@linyuhui I am facing similar issue on windows. Were you able to solve the issue?
For my work I have build Tensorflow from source, and I have added missing protobuf symbols in def_file_filter.py.tpl file, to resolve the protobuf related linking errors.


Provide Diff statistics

Hey there,
we have a Google internal Use case, where it would be nice to support Diff statistics. Any chance that we can get this into go-cmp?
I thought about something like:

Total: Total number of Differences (each line counts as a difference)
Additions: Number of additions
Changes: Number of Changes
Removals: Number of Removals

As first draft a total number is sufficient.

This is highly specialized and I don't think there needs to be first-class support for it in cmp.
You can accomplish this yourself using a custom Reporter: https://pkg.go.dev/github.com/google/go-cmp/cmp#Reporter
The output of Diff is implemented in terms of a Reporter. You can look at the implementation in the cmp package and take the relevant parts to gather the statistics you care about.


Update docs to clarify RCT_REMAP_METHOD usage.

Motivation (required)
The current docs (iOS) for promises make use of the RCT_REMAP_METHOD macro but it's not clear about the proper usage for the macro. Specifically, it points to a selector of resolver:rejector: and makes it seem like findEvents is assumed as both the alias and method name.
This has led to some confusion (#9070) on how to use RCT_REMAP_METHOD.
Test Plan (required)
No tests needed since this is just a small docs update.

@ericvicenti The RCT_REMAP_METHOD example is confusing. Previously the example defined a method findEvents on the JS sideand a method-(void)resolver:rejecter:on the Objective-C side, which makes little sense as a method name (and could have overlap if defined multiple times). The correct name for that method would befindEventsWithResolver:rejecter:`, which is the change here.

hi @ericvicenti, sorry if it wasn't clear. I just want to make the docs show a realistic method signature for findEvents.
Current version of findEvents method according to the docs:
-(void)resolver:(RCTPromiseResolveBlock)resolve rejecter:(RCTPromiseRejectBlock)reject
{
}

Ideal version of findEvents:
-(void)findEventsWithResolver:(RCTPromiseResolveBlock)resolve rejecter:(RCTPromiseRejectBlock)reject
{
}


Blank map?

I'm not quite sure what I'm doing wrong - help would be appreciated.
I've tried including the base component in my application:
{{google-map lat=0 lng=0}}

And all I get is a blank/empty div I've tried setting other lat and lng values but to no avail.
Everything is installed properly - I get the following when booting the ember app:
DEBUG: [google-maps] creating new map: {zoom: 5, mapTypeId: roadmap, center: (0, 0)}
The DOM elements appear to be there, but I can't see anything!

You need to define a with and height to the canvas div, look at the docs in the wiki of this repo (can't link to the right page now, on my phone)

:+1: Thanks.
Sorry - I thought I had done this (I did read through the wiki). But it turns out I got myself confused between a few files...... :)


Wiki page contains links to malware and ipgrabbers

The home page of the wiki was edited recently to contain an iplogger link, and the Development Kit page was edited to include an iplogger link and far more concerningly links to a fake msys2 installer uploaded to a different github. This installer drops the NetSupport RAT among other nasty things for infostealing and command and control.
Likely the entire wiki needs to be reviewed for malware / malicous links - I have removed the ones that i have identified from the Home and Development Kit pages but the correct links need to be added again (see revision history).
This situation is very concerning as anyone who downloaded and ran the fake msys2 installer linked on the development kit page now has a compromised system - the file has a large amount of padding to evade detection by AV and in my testing the default Windows Defender was not concerned by it.

The Development Kit (legacy) page is also affected and I have removed ipgrabbing links

Thank you @Modularis1 for fixing this! I think you caught all affected links.
I set the permissions of the wiki to contributors only now.


extra key commands for editor

when a traj is selected, "p" to toggle pluck; "d" to toggle dampen.
Also, perhaps left and right arrow to move to next or previous traj

I can't think of any others that need to be implemented at the moment. But, it would be good to have these better documented, as well as having some visual reference on the page, so users can better tell which modes they have selected / choose a different mode if they can't remember the key commands.

There should be a project to redo the editor component. Will be a difficult and time-consuming project, because it is so large, and needs to be much more modular. This would be folded into that project.

A ModeToggle component


[Feature request] :ignore_keys option

I have a use-case where I often need to compare 2 hashes where I don't care about the values of particular keys. For example ones which contain timestamps, or the following example which contain a 'duration' key (representing milliseconds):
h1 =
{
  "fonts": {
    "value": [],
    "duration": 93
  },
  "domBlockers": {
    "duration": 0
  },
  "fontPreferences": {
    "value": {
      "default": 146,
      "apple": 146,
      "serif": 146,
      "sans": 144,
      "mono": 119,
      "min": 10,
      "system": 144
    },
    "duration": 117
  },
# ...
}

h2 =
{
  "fonts": {
    "value": [],
    "duration": 78
  },
  "domBlockers": {
    "duration": 0
  },
  "fontPreferences": {
    "value": {
      "default": 146,
      "apple": 146,
      "serif": 146,
      "sans": 144,
      "mono": 119,
      "min": 10,
      "system": 144
    },
    "duration": 134
  },
# ...
}

I can ignore differences in the 'duration' values with a block:
diff = Hashdiff.diff(h1, h2) { |path, _e, _a| true if path.split('.').last == 'duration' }
#=> [['~', 'fontPreferences.value.system', 144, 146]]

But I'd love to be able to write something like this, where :ignore_keys can take a single value or array of keys to ignore:
diff  = Hashdiff.diff(h1, h2, ignore_keys: 'duration')

The logic would be the same as above, i.e. ignored keys would have to be at the end of each path. My guess is this may be a common enough use-case that a shorthand option like this would be valuable to users.
I might have a stab at a PR if you like the idea. Awesome library btw.

+1

Thanks for the detailed explanation of the use case @MatzFan . It sounds like a useful feature.
A PR is welcome from anyone who has bandwidth to do it.

V1.1.0 now contains this feature, kudos to @MatzFan 🎉


Dependencies Not Isolated Between Tests in SetupAsync

I need a way to ensure that each test receives a fresh instance of AppDbContext to prevent data leakage and interference between test test assembly.
In the current implementation, SetupAsync is invoked twice, once per TestBase<App> implementation.
However, it receives the AppDbContext from the last ConfigureApp call, leading to potential data leakage and test interference.
Code Snippet
public class App : AppFixture<Program>
{
    protected override void ConfigureApp(IWebHostBuilder a)
    {
        var name = $"ConnectionTests-{Guid.NewGuid()}";
        // set DatabasePath to :memory: for in-memory database
        a.ConfigureAppConfiguration((_, b) => b.AddInMemoryCollection(new Dictionary<string, string>
        {
            [Constants.Configuration.UseInMemoryDatabase] = "true",
            [Constants.Configuration.DatabasePath] = name
        }!));
    }

    protected override async Task SetupAsync()
    {
        {
            using var scope = this.Services.CreateScope();
            var db = scope.ServiceProvider.GetRequiredService<AppDbContext>();

            try
            {
                await db.Database.MigrateAsync();
            }
            catch (Exception)
            {
            }
        }
    }
}

// Adding Infrastructure:
// This gets called several times per test, but it receives both "dbPaths" at least once.
services.AddDbContext<AppDbContext>(opt =>
{
    var useInMemory = configuration.GetValue<bool>(Constants.Configuration.UseInMemoryDatabase, false);
    var dbPath = configuration.GetValue<string>(Constants.Configuration.DatabasePath);

    if(useInMemory)
    {
        if(string.IsNullOrEmpty(dbPath))
        {
            dbPath = "DefaultName";
        }

        var constring = string.Equals(dbPath, Constants.Configuration.InMemoryDatabase)
            ? "Data Source=:memory:" // non-shared in-memory database
            : $"Data Source={dbPath};Mode=Memory;Cache=Shared"; // shared in-memory database

        var keepAliveConnection = new SqliteConnection(constring);
        keepAliveConnection.Open();
        opt.UseSqlite(keepAliveConnection);
    }
    else
    {
        if(string.IsNullOrEmpty(dbPath))
        {
            throw new InvalidOperationException("Database path is not set");
        }

        opt.UseSqlite($"Data Source={dbPath}");
    }
});

Problem
SetupAsync gets called twice, once per test, but it receives the AppDbContext from the last ConfigureApp call. This behavior can cause data to persist between tests, breaking test isolation.

ConfigureApp is called twice, once per TestBase<App> implementation. It generates a unique database name for each test.
The AddDbContext delegate is called multiple times per test assembly, but it seems to receive the correct database names. (2 Test cases -> 1st db name, 2nd db name, 1st again, exception before 2nd)
The AppDbContext in SetupAsync contains always the same database name, the last one from ConfigureApp.

Steps to Reproduce

Implement SetupAsync and ConfigureApp as shown in the code snippet.
Run multiple tests that depend on AppDbContext.
Observe that SetupAsync uses the same AppDbContext instance from the last ConfigureApp call.

Expected Behavior
Each test should receive a fresh instance of AppDbContext to ensure data isolation between tests.
Actual Behavior
Tests share the AppDbContext instance from the last ConfigureApp call, leading to potential data leakage and interference.
Suggested Fix
Ensure that each test creates and uses a unique AppDbContext instance. This can be achieved by modifying the configuration setup to guarantee a new instance per test.

have you seen this part in the documentation?

[DisableWafCache] attribute disables the internal caching of WAF instances. that should make each test-class completely isolated from eachother. but it won't be able to give you isolation between each test-method of a test-class, because that's not how xunit test-fixtures work. a test-fixture is meant to be a shared resource for all test-methods of a test-class. if you don't want shared state, don't use a test-fixture.
if you want total isolation betweeen each test-method, you're on your own with the vanilla WebApplicationFactory. because the design goal of the AppFixture<TProgram> is not to provide individual test isolation (which is mostly a unit testing concept). the appfixture is designed specifically for integration-testing to prevent booting up WAFs unnecessarily per test-methods/test-classes to speed up test runs. in a typical integration testing scenario, you only need a few derived AppFixture<TProgram> classes each representing a uniquely configured system-under-test instance (which you issue requests against). you just need to rethink your data strategy for integration testing in a way that resembles a real web server, where there is only one database and all data goes in there. see this example project.
if you do the typical vanilla WAF instance/database per each test-method, your tests will be running a lot slower and use more computing resources.
hope that clears things up...

you should really look into using a real sql database via testcontainers package because the EF inmemory database doesn't behave exactly as a real db server and your tests might be less valueable because of that. here's an example. basically, you'll have a fresh database per each test-class that way. this seems to be the most popular way to do integration testing (with a real database) nowadays.


Reaction emoji doesn’t update in split view

If your in split view and set a new default reaction in settings the emoji icon won’t update in the main view (settings).
This is because the cells only update on viewDidAppear.
Sent with GitHawk

Ah! Didn't notice that when I added the reaction vc to split view. I'll take care of this
Sent with GitHawk


Iterate over map and dynamically assign key as label value

Hi, i'm wondering if this is doable? Given this json:
{
    "input": {
        "a": {
            "foo": 2,
            "bar": 0
        }
    },
    "output": {
        "a": {
            "status": {
                "code": 0
            },
            "meta": {
                "foo": 0,
                "bar": 0
            }
        },
	"b": {
            "status": {
                "code": 1
            },
            "meta": {
                "foo": 0,
                "bar": 0
            }
        }
    }
}

I would like to iterate over the output map, and get something like:
output_status_code{name="a"} 0
output_status_code{name="b"} 1

Is it just because of JSONPath limitation? If it's possible, what would be the config like?

I do have same problem. Can't figure out, how actually get key values itself, in order to use them as label or whatever later reference all of them later in values/labels.

Any news about this issue ?


Major Refactoring of Where/Having Support

"Where" and "Having" are essentially the same. This PR brings the code to parity, ensures similar capabilities for both clauses, and simplifies the implementation somewhat.
There is a breaking change in this PR, but I think it is in a rarely used capability (reusable and composable where clauses). The change is well documented and there is a relatively simple fix.

Coverage: 100.0%. Remained the same when pulling 955b519e2aacebdae41f36e849df5785619090f3 on jeffgbutler:reusable-where-clauses-v2 into 45e35e7cab0fe8136a91683a2dcb2c99b4a04477 on mybatis:master.


Feature Support priority for Broker Load job.

Proposed changes
Support priority for Broker Load job.
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run buildall

run buildall

run buildall

LGTM


Imperceptible colors on Buffer Names

I think this is the correct place to open the issue, but please let me know if is not.
I love the simplicity of this colorscheme, kudos on that.
My only gripe with it is that the unselected buffer name/titles have the same background and the foreground color.
Buffer named style.scss is present but not visible:

Buffer named style.scss after terminal selection:

If this is not related to walh, kindly point me to where I should check to get it fixed.
Thanks !

@WantGuns thanks for raising this!  What is the plugin you are using for the tab bar?

Im using barbar -- https://github.com/romgrk/barbar.nvim

@WantGuns I made an update.  I think barbar.nvim is mainstream enough to just include the color highlights is uses into walh.  Please have a look and see if it meets your expectations.
Cheers!

Thanks for the quick fix.
I can see the buffer names now !

I think this comes to personal preference, but after your commit, the selected buffer has a darker foreground than the unselected buffer hehe.

Ah I missed that.  I have corrected this with the last commit.  Give it a look and lets see if it looks fine on your side.
Thanks

@WantGuns can you also share your terminal colorscheme thanks!

Hey casonadams, I am still facing the same colors on my end after updating the plugin :(
tmux.conf is the selected buffer and its foreground is dimmer than the cpu.tmux buffer title:

For the colorscheme, I wanted to go towards something like Hyper (I really like Vercel's color palette), but it seemed over-contrasted and so I went with Tomorrow Night Bright.
The colors for Alacritty are:
https://github.com/eendroroy/alacritty-theme/blob/master/themes/tomorrow_night_bright.yaml

Thanks for sharing the colorscheme.  So that colorscheme has color15 which is normally light colors (for a dark scheme).  However the white is a darkgrey.  Update the following and it will work for you.
    # Default colors
    primary:
      background: '0x000000'
      foreground: '0xeaeaea'

    # Normal colors
    normal:
      black:   '0x000000'
      red:     '0xd54e53'
      green:   '0xb9ca4a'
      yellow:  '0xe6c547'
      blue:    '0x7aa6da'
      magenta: '0xc397d8'
      cyan:    '0x70c0ba'
      white:   '0x424242'

    # Bright colors
    bright:
      black:   '0x666666'
      red:     '0xff3334'
      green:   '0x9ec400'
      yellow:  '0xe7c547'
      blue:    '0x7aa6da'
      magenta: '0xb77ee0'
      cyan:    '0x54ced6'
      white:   '0xeaeaea'

Before change:

After change:

I have some other colorschemes that seems to be really nice with walh https://github.com/casonadams/alacritty-shell/wiki/Themes

Thanks a lot for the input and quick fixes, im off to checking out your colorschemes for Alacritty !


[C# API] Possible type mismatch of property Node.AnchorsPreset

Godot version
4.0 beta 10
System information
Irrevalent
Issue description
The property Node.AnchorsPreset should be of type LayoutPreset just like what Node.SetAnchorsPreset takes, but it is now of type int. This is difficult for users to figure out which preset each int value stands for, which makes the property unusable. Its type should be changed to LayoutPreset instead.
Steps to reproduce
N/A
Minimal reproduction project
N/A

The C# API is generated from ClassDB, this property is defined in Core as taking an INT which is why C# also defines it that way:
https://github.com/godotengine/godot/blob/fcba87e696d58912838d8a4a6987b10efa28e78f/scene/gui/control.h#L285-L286

Additionally, setting the Node.AnchorsPreset property may result in unexpected results, while the SetAnchorsPreset function always gives the correct result. Maybe make Node.AnchorsPreset a readonly property and guide users to the SetAnchorsPreset function for modifying?

The Control.AnchorsPreset property uses the _SetAnchorsLayoutPreset method internally, it's not the same as calling SetAnchorsPreset, they're two different APIs. Someone from the @godotengine/gui-nodes team can explain better how these APIs work but basically _SetAnchorsLayoutPreset sets the anchors preset (like SetAnchorsPreset) but also sets the offset preset and the grow direction preset, maybe renaming the property to anchors_layout_preset would make the difference clearer.

The Control.AnchorsPreset property uses the _SetAnchorsLayoutPreset method internally, it's not the same as calling SetAnchorsPreset, they're two different APIs. Someone from the @godotengine/gui-nodes team can explain better how these APIs work but basically _SetAnchorsLayoutPreset sets the anchors preset (like SetAnchorsPreset) but also sets the offset preset and the grow direction preset, maybe renaming the property to anchors_layout_preset would make the difference clearer.

Thanks for your explanation!

Both anchors_preset and layout_mode are not supposed to be used in scripting. They are marked as PROPERTY_USAGE_INTERNAL, and are only relevant for the editor. They exist as properties because that's the only way to have them in the inspector and in a specific order.
Now, GDScript doesn't hide those properties either, because PROPERTY_USAGE_INTERNAL is for the most part only used for documentation. So it's not an issue with the C# generator, but it's probably something we should address across the board.
The collision is accidental, and the property and the method have no relation. The set_anchors_preset method is just a helper method to set all anchors together according to a LayoutPreset value. The anchors_preset property accepts different values than LayoutPreset (we just happen to reuse that enum internally for validation to reduce duplication of such a large enumeration). They are not compatible for each other.

Hi! Interested in working on this issue. After reading the discussion, I'm unsure of what needs to be done here. It sounds as though things are behaving as intended? Or does there need to be a type change in ClassDB to fix the C# type?

Hey, sorry, there is no consensus yet on what should be done here. And this shouldn't be marked as a good first issue, at least not until we reach one.


data augmentation transforms: RandomResizedCrop not available

Hi Dusty, I'm trying to do data augmentation using pytorch's RandomResizedCrop. However it does not appear to be included in the ssd vision transforms used in jetson-inference.
I was wondering why the data augmentation transforms offered in the jetson-inference torchvision here vs. the original are not the same?
Thanks for your help!

I was wondering why the data augmentation transforms offered in the jetson-inference torchvision here vs. the original are not the same?

Hi @silent-code, I'm not exactly sure, if think the original upstream author wanted to use cv2 to do the transformations so they could more easily do realtime cv demo?   https://github.com/qfgaohao/pytorch-ssd
Not sure - some of them seem custom for bounding box data / ect.

Ah I see. Thanks for the quick response Dusty.


Reordering work items within a tree level

For a tree list in excel, I need to reorder the new work items that are adding. Is there a way to use excel to reorder all tasks under a user story for example and publish it back ?

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

ID: 505849d3-01c8-a2cd-dc6c-271dd1120e2d
Version Independent ID: 577fc252-60e4-f670-bb92-6bdbcb7d5c69
Content: Bulk modify work items using Microsoft Excel - Azure Boards
Content Source: docs/boards/backlogs/office/bulk-add-modify-work-items-excel.md
Product: devops
Technology: devops-agile
GitHub Login: @KathrynEE
Microsoft Alias: kaelli

@randomITperson - Thank you for your question. Could you say more about why the order of work items is important?  That said, in general:

If you are working with a flat list, you can use Excel features to sort your work items
If you are working with a tree list, you don't want to sort your work items as that will change your tree structure and change your hierarchical links
if you want to use Excel to manage the order of your work items as they appear in a team backlog, you can do that (not recommended, but supported) by using the Stack Rank or Backlog Priority field (Agile or Scrum process). You can set values in these fields and that will reorder the work items based on lowest to highest number.  However, the next time the backlog is reordered, these values are subject to change.
If you want to maintain a certain order of work items, you might consider adding a custom field to manage the sort order


Add features to physical servers pages

Depends on ui #970

Adds operations for physical server buttons.
Adds modify actions for physical server buttons.

Gabriel this overlaps with https://github.com/ManageIQ/manageiq/pull/14028

@GabrielSVinha : does this really depend on the UI PR? I guess it's the other way round, isn't it?We need this one to merge first, right?
I think that we can merge this exept for the extra space.
@rodneyhbrown7 : do you agree or should #14028 be merged first?

@martinpovolny you're right. UI depends on this one, thnks.

Agreed this one can be merged first.


Updating nested objects in the GraphQL cache

Here's a pretty common scenario. I have a post object with comments subfield. The code could look something like this:
post.comments ? <PostComments comments={post.comments}/> :
Leave the first comment!

Notice that I'm not loading comments through their own useMulti/withMulti hook/HoC but as part of the query that loads the post.
Because of that, when I add a comment the post.comments nested field will not update until I either reload the page or the poll interval kicks in and the whole post is loaded again.
As discussed with @doronrk, the way around this would be to set up some kind of system that recursively traverses the GraphQL tree and calls each resolver (in this case post.comments) again to update it on the client based on the new comment in cache.
This could be pretty complex, and also would not necessarily work for resolvers that are server-only (such as anything that directly makes a database or API call). But maybe we could at least implement it for the standard hasOne, hasMany, etc. relations, since those represent most of such use cases?

Not sure where I listed the relevant issues in Apollo, maybe here: https://github.com/VulcanJS/Vulcan/issues/2381
Basically the cache update system is architectured so that mutations are responsible to update queries. But that's weird, because a comment creation shouldn't be responsible to updating posts for example (they may not even know the posts they are related to).
Someone proposed to reverse this thinking, and have queries to subscribe mutations.
So here, your post query could subscribe to both comment creation and post creation for example. Sadly that's not the case so a solution to this might be complex/hackish.


Add allow_direct_linking to UIExtension schema

Part 2 of https://github.com/Shopify/core-issues/issues/74004
Vault project: https://vault.shopify.io/gsd/projects/39317
WHY are these changes introduced?
Following the decision on the tech design to introduce a property called allow_direct_linking for an extension point's configurable capabilities. This allows app developers to configure whether the app extension can be used inside navigations (do they render a root url)
https://github.com/Shopify/shopify/pull/524406 is for the core side validations and updates the UiExtension API schema to incorporate target-level capabilities. This pull request allows allow_direct_linking to be configured for a ui_extension target.
WHAT is this pull request doing?
Add an optional capabilities hash which includes an optional allow_direct_linking boolean to the NewExtensionPointSchema.
How to test your changes?
Make sure you have a spin instance with partners constellation, can use customer-accounts-dev-ui. Check out https://github.com/Shopify/shopify/pull/524406 there and enable the api client flag bin/rails g verdict:configure_flag enable_ui_extension_allow_direct_linking --subject_type "api_client" --percent 100

Clone the CLI tool dev clone cli locally
Checkout this branch. From the cli directory, install pnpm pnpm install, then create an app SHOPIFY_CLI_1P_DEV=1 SPIN_INSTANCE={spin_name} SHOPIFY_SERVICE_ENV=spin pnpm create @shopify/app
Create an extension, SPIN_INSTANCE={spin_name} SHOPIFY_SERVICE_ENV=spin pnpm shopify app generate extension from the app's directory
Add allow_direct_linking to the shopify.extension.toml shopify.extension.toml configuration found inside your extension directory

[[extensions.targeting]]
module = "./src/FullPageExtension.tsx"
target = "customer-account.page.render"

  [[extensions.targeting.capabilities]]
  allow_direct_linking = true

Change directory back to main cli directory and deploy SPIN_INSTANCE={spin_name} SHOPIFY_SERVICE_ENV=spin pnpm shopify app deploy --path /Users/path_to_your_app

Post-release steps
We will need to have docs updated when Configure header menu for customer accounts is released tracked in this issue https://github.com/Shopify/core-issues/issues/75186
Measuring impact
How do we know this change was effective? Please choose one:

[x] n/a - this doesn't need measurement, e.g. a linting rule or a bug-fix
[x] Existing analytics will cater for this addition
[ ] PR includes analytics changes to measure impact

Checklist

[x] I've considered possible cross-platform impacts (Mac, Linux, Windows)
[x] I've considered possible documentation changes

Wooohooo! 🙇♀🚢🏄🏻♀


[HOW-TO] How to pin the picamera2 installation to a specific tag

How can we pin the installation to a specific version of picamera2?
I am using sudo apt install -y python3-picamera2, which is the suggested way to do the installation, but this seems to be installing the latest version. Recently some new tags have been introduced, which broke the existing functionality of my app.
Is there a way to pin this installation to 3.0.10?

@XECDesign Serge, do you have a suggestion here?

Look at the man page for 'apt-mark hold' to prevent a package from updating.

Debian wasn't really designed with that kind of use case in mind. But yeah, apt-mark hold  can be used to prevent updates and you can download old versions manually from the archive.
It doesn't look like we ever released a 3.0.10 apt package, but if you wanted to stick to 3.0.9, you could do something like this:
wget https://archive.raspberrypi.org/debian/pool/main/p/picamera2/python3-picamera2_0.3.9-1_all.deb
sudo apt install ./python3-picamera2_0.3.9-1_all.deb
sudo apt-mark hold python3-picamera2

Again, this is completly unsupported and not recommended.

Thanks!
Versions 0.3.10 and 0.3.11 had a couple of small bugs in some corner cases, so we actually released 0.3.12 instead. There should be very little difference between them so I'd be interested to know if there's something that works in 0.3.10 but not in 0.3.12.


chore(main): release 3.1.2

:robot: I have created a release beep boop
3.1.2 (2023-10-11)
What's Changed

build(deps-dev): bump postcss from 8.4.27 to 8.4.31 by @dependabot in https://github.com/johnhwhite/ng-keyboard-sort/pull/89
ci: fix linting step by @johnhwhite in https://github.com/johnhwhite/ng-keyboard-sort/pull/91

Full Changelog: https://github.com/johnhwhite/ng-keyboard-sort/compare/v3.1.1...v3.1.2

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/johnhwhite/ng-keyboard-sort/releases/tag/v3.1.2 :sunflower:


Replace a failed OSD Drive procedure.

Hello ceph people!
Help required here.
It's not clear at all to me how to replace a failed Disk Drive ( luminus ).
In Other Words, I Cannot locate a clear  procedure or a how-to  stating the steps to  replace  failed drives  using ansible   from a CEPH storage system.
Am I the only one with this problem ?
Regards,
Harry.

Hi @grharry
I use ceph-ansible on an almost weekly basis to replace one of our thousands of drives.
I'm currently running pacific, but started of the cluster om luminous so it was basically the same.
You will have issues if your inventory's disk placement is not 100% configured correctly, in my instance I used manual disk placements and not osd_autodiscovery from osds.yml (since osd_autodiscovery is not recommended).

If you're sure everything is correct you can just run:
ansible-playbook -i /opt/ceph-ansible/inventory -e 'ansible_python_interpreter=/usr/bin/python3' infrastructure-playbooks/shrink-osd.yml -e osd_to_kill=1592
Where osd_to_kill is your corresponding OSD that is faulty. This will format the disk, and remove it from ceph crush/ceph completely, no other action required.
If the disk is still readable from time to time, i.e. not completely dead, I generally weigh it out manually, ceph osd reweight osd.x 0 which is the equivalent of setting the osd out. It will still try to read data from the disk while draining it for quicker backfilling. Once it is drained, I run shrink-osd.yml. In the even that the disk is completely dead just run the shrink-osd.yml as above.
If you don't know the faulty OSD and only the harddrive serial number from IPMI/IDRAC/ILO you can use ceph device ls | grep 'serial number' to get the corresponding OSD value, or ceph-volume inventory from within one of your osds/mds/mon/mgr docker containers exec -it ... /bin/bash
Once the disk is replaced with a new one you can run
ansible-playbook -i /opt/ceph-ansible/inventory -e 'ansible_python_interpreter=/usr/bin/python3' site-container.yml --limit=HOSTwithNEWdisk , --limit= it to the ceph osd node with the new/replaced disk.
If you run site-container.yml and it completes successfully but you don't see your new(replacement OSD) added to ceph, I generally ssh to the host with the new disk and run an lsblk -p , find the new disk /dev value and run, dd if=/dev/zero of=/dev/sdX bs=1M #replace X with the target drive letter.
Then proceed to run: ansible-playbook -i /opt/ceph-ansible/inventory -e 'ansible_python_interpreter=/usr/bin/python3' site-container.yml --limit=HOSTwithNEWdisk, again.
ansible_python_interpreter is not required, depending on your environment

Wow !!!
At LAST!
Thank you so much !!!!
I owe U at least a BEER !!!
Regards,
Harry!

This 1:1 relation disappeared after we dropped ceph-disk support (stable-4.0).
If you take a look at this task in the role ceph-facts you will see that we use the filter | unique
https://github.com/ceph/ceph-ansible/blob/f288364c5c268e61d49165704900e8e01ca643c8/roles/ceph-facts/tasks/devices.yml#L45-L50

@jeevadotnet very useful feedback. I'm considering writing a documentation ouf of it 🙂
Thanks!

 This 1:1 relation disappeared after we dropped ceph-disk support (stable-4.0). If you take a look at this task in the role ceph-facts you will see that we use the filter | unique
https://github.com/ceph/ceph-ansible/blob/f288364c5c268e61d49165704900e8e01ca643c8/roles/ceph-facts/tasks/devices.yml#L45-L50

Will try it today when rebuilding testbed with your recommendation as per #7283 , however maybe I did it previously wrong, but tested luminous and octopus with the many:1 relation but then it only created a partition for my /dev/sda.

@jeevadotnet very useful feedback. I'm considering writing a documentation ouf of it 🙂 Thanks!
Haha, pleasure, you always great in helping out here and teaching me the 'way of the ceph', so I eventually over time I've learned enough to be able to reply to other people's issues.

Hello Again!
I need some clarification.
On my ceph instalation ( luminous ).
I've got 4 OSDs DOWN ( waiting for replacement drives )
and my current pg status shows
5 pgs inactive, 3 pgs down, 2 pgs peering, 12 pgs stale
querying the down pg's they seem to be stuck  by the dead OSD's
Do I proceed with .....  shrink-osd.yml -e osd_to_kill=xxx  first?  or   ceph pg force_create_pg   with the id's of down pgs ??
Thank's again for your help!
Harry.


[Documentation Issue] Resource Record Sets

Page Name: dns-usage
Release: 0.11.0
From the documentation, the accepted usage for the dns.Client#zone method is similar to this:
zone = client.zone('acme-co')

I tried same and I got this error:
...
TypeError: zone() takes exactly 3 arguments (2 given)

On introspection, I observed that the signature for the dns.Client#zone method is def zone(self, name, dns_name):. It seems so far that a lot of the API docs/lib is broken...
Any reasons why, please?

Apologies for the discrepancy.  The API implementation got stalled by the difficulty of getting system testing wired up (it requires having an actual DNS zone to work with).  Cleaning up the docs to match the implementation as it evolved got stalled, I'm afraid.


Is calling the library zlib-ng2.dll necessary?

https://github.com/zlib-ng/zlib-ng/blob/92107a92d3cfb11cc464c24ef2a73c2d7d05d2ba/CMakeLists.txt#L1022
I'm packaging this library for conda forge and trying to use it in a downstream project.
The challenge i find is that the naming convention is different between windows and Linux.
While i understand this is often true for historic libraries,
I'm wondering if we should just stick to the name zlib-ng and not add an other suffix
Let me know what name you think is most appropriate.
i kinda wish I had
zlib-ng.dll
and a matching .lib file

This is actually related to #1045.
Having the so-version in the name is especially important on windows, where AFAIK there are little other protections available when you have an updated library and an outdated application, or if you wanted to have two different versions of the dll in the same folder due to two different but related applications using two different versions.

hmm. alright. I'll keep things as is.
do you have FindZLIB_NG files for cmake?

I guess the second question is
zlib-ng2.dll
zlib-ng.lib

is what came out after I compiled with the following options
cmake -G "NMake Makefiles" ^
      -DCMAKE_BUILD_TYPE:STRING="Release" ^
      -DCMAKE_INSTALL_PREFIX:PATH="%LIBRARY_PREFIX%" ^
      -DCMAKE_PREFIX_PATH:PATH="%LIBRARY_PREFIX%" ^
      -DBUILD_SHARED_LIBS=1 ^
      ..

https://github.com/conda-forge/zlib-ng-feedstock/blob/master/recipe/bld.bat
Is that intentional?

zlib-ng.lib is the import library used by linker to create import table that reference entries in the export table of the dll file... It's specific to Windows and follows same naming convention as stock zlib.

do you have FindZLIB_NG files for cmake?

https://github.com/zlib-ng/minizip-ng/pull/574/files#diff-9513dde0d4d2e07d9e0363caa44d4c9d9ac1624c4603bbb33cb5fb75be376b3b


Build Contribution guidelines

We need to build the Contribution guidelines to share our work and let people know how the method works, how they can contribute, and how they can test to give us feedback

Done! https://github.com/felipez3r0/openclasses/commit/4d1e5dd5960ba75f409d0603b13c5d4b82f89b29


[WIP] Create Mock API

This PR is to create a simple mock api server that can be used by e2e testing or by local frontend development

Out of curiosity, had you reviewed any of these projects?

Nock-VCR (JavaScript/Node)
Sepia (JavaScript/Node)
VCR.js (JavaScript)
yakbak (JavaScript/Node)

cc @chriskacerguis

I hadn't seen those before.  They look interesting.  Node has a sea of modules and these didn't come up when looking for something to mock an existing API.   I could potentially look at them and see if they fit well for what we are doing.

@chriskacerguis I'm surprised this was merged with cloudforms in the data still.  Is that a bug in ManageIQ itself in that it's returning cloudforms, or was the mock data generated from cloudforms as opposed to manageiq.

@Fryguy the data I put in the mocks came directly from me just running manageiq locally.

@chalettu Can you please open an issue or make a PR on manageiq?  That data should not be in there.

Oh I see...it's actually the name of a service that is in the sample data that you used.


Update the development dependencies in the README

Development now depends on python 3.5 instead of 3.4.
Also, the python development libraries are now required to build the cryptography package, so these should be listed too.

#878 fixes the version in the README

cryptography isn't a development dependency, It's part of the cli setup. Those requirements are external requirements from the cli needed only if you are developing. The wording is a bit confusing.


How to remove spacing between list item parent and children?

Markwon version: 4.6.2

There is an enter character between list item parent and children like below.
- [x] checked
- [ ] unchecked
- jj
- parent
  - child1
  - child2
- bottom

Actual

Expected

I can remove the space like the expected screenshot by using FirstLineSpacingSpan, but text selection is weird when using the hack.

How can I remove the space? Thanks for your awesome plugin.

Hello @jja08111 ,
I'm not sure there is something we can do here. Text selection is unpredictable and works differently between Android versions and each individual device.
But still, what does it copy in your last screenshot?


fix for SLF4J-429

It is not possible to log at the OFF level. So the "mapping" of the OFF level does not matter. Please create a unit test illustrating the problem.

It is possible to log at any level by calling java.util.logging.Logger.log, which is what actually Jersey or any other application that configures the level at runtime does.
See your test SLF4JBridgeHandlerTest.testLevels failing here https://github.com/pa314159/slf4j/tree/SLF4J-429/failure


FIX/ENHANCE: Remove old files re-created upon merge with updated root repo; add MS Word and RTF formats

Two main changes:

I removed files that were accidentally re-created when I pulled from @lwalen's upstream for my forked repo and were based on rather old content;

Added the LBCF in Microsoft Word and Rich Text Formats
(I know RTF is a bit legacy these days but I still like the format for certain reasons).

BTW: the main benefit of the Word doc is that it provides appropriately tagged heading and in-doc hyperlinked navigation that is highly accessible to screen-reading technologies for blind and vision impaired end-users.


[OOBE]Fix not shortcut not updating

Fix for issue #20953.
Activation key update in the OOBE window when the user changes it in the settings window.
Summary of the Pull Request

PR Checklist

[X] Closes: #20953
[ ] Communication: I've discussed this with core contributors already. If work hasn't been agreed, this work might be rejected
[ ] Tests: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Dev docs: Added/updated
[ ] New binaries: Added on the required places

[ ] JSON for signing for new binaries
[ ] WXS for installer for new binaries and localization folder
[ ] YML for CI pipeline for new test projects
[ ] YML for signed pipeline

[ ] Documentation updated: If checked, please file a pull request on our docs repo and link it here: #xxx

Detailed Description of the Pull Request / Additional comments
Added settings repository reference to the ViewModel constructor to use the repository settings object. Without the reference the OOBE window class created a second instance of the settings object. That caused the problem.
Same problem detected also in the Color Picker, PowerToys Run and Video Conference Mute modules. All fixed.

Tested manually.

@microsoft-github-policy-service agree company="Janea Systems"

Hi Andrey,
I don’t know, whether the API could be deleted, sorry, I have very little experience about this complex code.
Thanks,
 Laszlo

From: Andrey @.>
Sent: 2022. október 12., szerda 10:30
To: @.>
Cc: Laszlo @.>; @.>
Subject: Re: [microsoft/PowerToys] [OOBE]Fix not shortcut not updating (PR #21175)
@yuyoyuppe approved this pull request.
Verified locally, nice catch!
In src/settings-ui/Settings.UI.Library/ViewModels/ColorPickerViewModel.cshttps://github.com/microsoft/PowerToys/pull/21175#discussion_r993152221:

@@ -62,15 +66,14 @@ public ColorPickerViewModel(ISettingsUtils settingsUtils, ISettingsRepository<Ge

         GeneralSettingsConfig = settingsRepository.SettingsConfig;

         _settingsUtils = settingsUtils ?? throw new ArgumentNullException(nameof(settingsUtils));

       if (_settingsUtils.SettingsExists(ColorPickerSettings.ModuleName))

       {

           _colorPickerSettings = _settingsUtils.GetSettingsOrDefault<ColorPickerSettings>(ColorPickerSettings.ModuleName);

so, if this API is error-prone, perhaps we should delete it altogether?
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/PowerToys/pull/21175#pullrequestreview-1138660650, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ANVPUS2FCADBGEWTXGGXOATWCZZLTANCNFSM6AAAAAARCJ5ENE.
You are receiving this because you were mentioned.Message ID: @.***>

@yuyoyuppe any reason to not merge?


Error at src/attributes.c:425: vertex attribute hash key is not a string, Invalid value

I wrote a test code but it throws an error. My environment is python2.7 with anaconda and igraph

Error at src/attributes.c:425: vertex attribute hash key is not a string, Invalid value

X = pd.DataFrame()
X['col1'] = [1,2,3]
y = pd.DataFrame()
y['col1'] = [1,4,3]
mir = MultiIsotonicRegressor()
mir.fit(X, y)

seems that line 59
graph_part.add_vertices(2)

fails because vertex has a attribute setted ysort

sorry from __future__ import unicode_literals causes this problem. thanks again for your code.


CatalogClient.ts fix to have identical return type for get earliest version and get later version methods

Added fix to have identical return type forEarliestVersion and getLatestVersion method

Discarding this request.


Create input validation

-implement a simple validation:

The content of the email field has to be in lower case.
implement the following interactions:
When the user submits the form, check if the email is in lower case.
If the validation is OK, the form is sent.
If the validation is not OK, show an error message to the user near the submit button informing them of the error and the form is not sent

Hi @willococa,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.

Invalid Code Review Request
You submitted a project that was meant to be a collaborative effort, but you completed it on your own. Kindly reach out to your partner and work on the project together.
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear. Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.
Invalid Code Review Request does not count into the code reviews limit.
@Strangeal
Please review once more: https://github.com/willococa/portfolio/commit/5d0493cca1c5456e7850f0617a878004f1262255 this commit was added by my partner and is part of this feature.


Limit how often agents deliberate about the ChangeRoute action

Currently, whenever agents deliberate, they also deliberate about taking the ChangeRoute action. This means that every agent runs the A*-algorithm every time. It would be better to

only deliberate when there are multiple roads to choose from at the end of this road.
only deliberate about changing route on a certain point on the road (e.g. when you are removed 100m from the end of the current road)
after the agent made the choice to change route, he sticks to it, until he changes onto the next road.

References #24

This is a specific version of #26, but even if we don't implement #26, it's good to do this ticket, since it will provide a large speedup in larger scenarios (running A* 1000 times per tick isn't very fast).

Alternative routes are not calculated when driving on the last edge of your route, or on the first 80% of the road. This is a temporary solution, but it's now easily adaptable in the Action.isRelevant() method.


Generated respec does not use custom templates from documentation configuration

Generated respec does not use custom templates from documentation configuration.
E.g. in this spec the customized templates are ignored in the generated respec output.

Fixed in 913b65a79706ba0741927e4f5bd6b1dbbc4e57ed


BUG: of scipy.interpolate.CubicSpline when bc_type' is set to periodic'

in line 701 of file: https://github.com/scipy/scipy/blob/c1372d8aa90a73d8a52f135529293ff4edb98fc8/scipy/interpolate/_cubic.py#L701
            a_m2_m1 = dx[-2]

should be
            a_m2_m1 = dx[-3]

This typo could make the result of some interpolations quite different.

@CastleStar14654 do you have an example where this causes a problem? would be quite useful both to demonstrate the bug and to use as a regression test

Code following would interpolate and extrapolate x and y periodically, and plot second derivative of y to x.  Ideally, function itself, first derivative & second derivative should be periodic and continuous.
import scipy.interpolate as ipt
import numpy as np
import matplotlib.pyplot as plt

x = [0.9, 1.3, 1.9, 2.1, 2.6, 3.0, 3.9, 4.4, 4.7, 5.0, 6.0,
     7.0, 8.0, 9.2, 10.5, 11.3, 11.6, 12.0, 12.6, 13.0, 13.3]
y = [1.3, 1.5, 1.85, 2.1, 2.6, 2.7, 2.4, 2.15, 2.05, 2.1,
     2.25, 2.3, 2.25, 1.95, 1.4, 0.9, 0.7, 0.6, 0.5, 0.4, 1.3]
csp = ipt.CubicSpline(x, y, bc_type='periodic', extrapolate='periodic')
newx = np.linspace(0, 15, 2000)
plt.plot(newx, csp.derivative(2)(newx))
plt.show()

Current version of scipy would give

which shows that the second derivative is not continuous at x = 0.9 and x = 13.3.
When _cubic.py is modified as I suggested, the plot is

and second derivative is continuous at  x = 0.9 and x = 13.3.

Hi! If no one is against it, I will fix the issue.

So I tried to change a_m2_m1 = dx[-2] into a_m2_m1 = dx[-3]. However, I get an IndexError when the number of pair points (x_i, y_i) is n=3. This is because dx is literally np.diff(x), which has shape x.shape[0] - 1.
Looking at the code, the case of n=3 is treated in a specific way when bc_type="not-a-knot":
https://github.com/scipy/scipy/blob/b77138f1f41a0d1190a02e7ba6c2258820a4bf05/scipy/interpolate/_cubic.py#L657-L674
Could it be that also bc_type="periodic" has a specific treatment when n=3? I am honestly no expert in cubic splines.

After reading about cubic spline interpolation, I believe that the code I linked just above can be relaxed for the periodic condition when three points are given. Something like:
if n == 3 and bc[0] in ['not-a-knot', 'periodic' and bc[1] in ['not-a-knot', 'periodic'] and bc[0] == bc[1]:
However I am unsure on how to proceed. The change directly related to the issue would imply another indirect change. @rgommers, should I directly send a pull request with the changes I wrote about, or wait for further discussion here?

Could it be that also bc_type="periodic" has a specific treatment when n=3? I am honestly no expert in cubic splines.
EDIT: I actually realized that bc_type="periodic" and n=3 always implies a parabolic curve.

Sorry for my late reply.
Actually, bc_type="periodic" and n=3 doesn't mean a parabolic curve. But it does worth a specific treatment.
An Example
For instance, let's interpolate these data periodically:

x
y

-x1
0

0
H

x2
0

Let

y = A x^3 + B x^2 + C x + H when -x1 < x < 0;
y = a x^3 + b x^2 + c x + H when 0 < x < x2.

The first and second derivatives should be continuous, so at x = 0

C = c
B = b

At x = -x1 or x2

-A x1^3 + B x1^2 - C x1 + H = 0
a x2^3 + b x2^2 + c x2 + H = 0
3 A x1^2 - 2 B x1 + C = 3 a x2^2 + 2 b x2 + c
-6 A x1 + 2 B = 6 a x2 + 2 b

Finally, we get

Variable
Value

A
-2H / x1^2 x2

a
-A x1 / x2 = -2H / x1 x2^2

B = b
3 A x1 / 2 = -3H / x1 x2

C = c
A x1(x1 - x2) = -H(x1 - x2) / x1 x2

They are not paralobic.

The unique piecewise cubic passing through n=3 points (x, y) with periodic boundary conditions (which implies y[0]==y[2]) is
def per_spl(x, y):
    assert len(x) == len(y) == 3
    y[-1] = y[0]   # shortcut

    x, y = map(np.asarray, (x, y))

    h = x[1:] - x[:-1]
    m = (y[1:] - y[:-1]) / h

    s = (m/h).sum() / (1./h).sum()
    return CubicHermiteSpline(x, y, [s, s, s])


Original checksum returned None

I'm loading a larger number of images and videos via Elodie and I am getting this message frequently, roughly every 3rd or 4th file:
Original checksum returned None
I understand this means the checksum of the file cannot be calculated, and it therefore is most likely skipped completely.
Anything I can do to prevent this from happening ? Any reason why it shouldn't be possible to calculate the checksum of a file that is there (I checked some of them, they open as normal images).
I've linked one of the images that produced an error.
Source File System is Mac OS Extended (Journaled)

User misunderstanding...

User misunderstanding...