PYBITMESSAGE not installing on NIXOS 21.05

Bug Description
Installation of PYBITMESSAGE aborted with error message:

nix-env -iA nixos.pybitmessage --show-trace
installing 'pybitmessage-<IP_ADDRESS>'
error: while evaluating the attribute 'drvPath' at /nix/store/fyfzbrgqy4ir0nia9ypzkhfskcpwx3j0-nixos-21.05.4132.24528474d2b/nixos/lib/customisation.nix:163:7:
while evaluating the attribute 'propagatedBuildInputs' of the derivation 'pybitmessage-<IP_ADDRESS>' at /nix/store/fyfzbrgqy4ir0nia9ypzkhfskcpwx3j0-nixos-21.05.4132.24528474d2b/nixos/pkgs/stdenv/generic/make-derivation.nix:201:11:
while evaluating the attribute 'out.outPath' at /nix/store/fyfzbrgqy4ir0nia9ypzkhfskcpwx3j0-nixos-21.05.4132.24528474d2b/nixos/lib/customisation.nix:156:13:
while evaluating the attribute 'propagatedBuildInputs' of the derivation 'python2.7-pyopencl-2021.1.4' at /nix/store/fyfzbrgqy4ir0nia9ypzkhfskcpwx3j0-nixos-21.05.4132.24528474d2b/nixos/pkgs/stdenv/generic/make-derivation.nix:201:11:
while evaluating 'getOutput' at /nix/store/fyfzbrgqy4ir0nia9ypzkhfskcpwx3j0-nixos-21.05.4132.24528474d2b/nixos/lib/attrsets.nix:486:23, called from undefined position:
while evaluating anonymous function at /nix/store/fyfzbrgqy4ir0nia9ypzkhfskcpwx3j0-nixos-21.05.4132.24528474d2b/nixos/pkgs/stdenv/generic/make-derivation.nix:166:17, called from undefined position:
while evaluating 'callPackageWith' at /nix/store/fyfzbrgqy4ir0nia9ypzkhfskcpwx3j0-nixos-21.05.4132.24528474d2b/nixos/lib/customisation.nix:117:35, called from /nix/store/fyfzbrgqy4ir0nia9ypzkhfskcpwx3j0-nixos-21.05.4132.24528474d2b/nixos/pkgs/top-level/python-packages.nix:6859:13:
while evaluating 'makeOverridable' at /nix/store/fyfzbrgqy4ir0nia9ypzkhfskcpwx3j0-nixos-21.05.4132.24528474d2b/nixos/lib/customisation.nix:67:24, called from /nix/store/fyfzbrgqy4ir0nia9ypzkhfskcpwx3j0-nixos-21.05.4132.24528474d2b/nixos/lib/customisation.nix:121:8:
while evaluating anonymous function at /nix/store/fyfzbrgqy4ir0nia9ypzkhfskcpwx3j0-nixos-21.05.4132.24528474d2b/nixos/pkgs/development/python-modules/pytools/default.nix:1:1, called from /nix/store/fyfzbrgqy4ir0nia9ypzkhfskcpwx3j0-nixos-21.05.4132.24528474d2b/nixos/lib/customisation.nix:69:16:
while evaluating 'makeOverridablePythonPackage' at /nix/store/fyfzbrgqy4ir0nia9ypzkhfskcpwx3j0-nixos-21.05.4132.24528474d2b/nixos/pkgs/top-level/python-packages.nix:32:37, called from /nix/store/fyfzbrgqy4ir0nia9ypzkhfskcpwx3j0-nixos-21.05.4132.24528474d2b/nixos/pkgs/development/python-modules/pytools/default.nix:12:1:
while evaluating 'makeOverridable' at /nix/store/fyfzbrgqy4ir0nia9ypzkhfskcpwx3j0-nixos-21.05.4132.24528474d2b/nixos/lib/customisation.nix:67:24, called from /nix/store/fyfzbrgqy4ir0nia9ypzkhfskcpwx3j0-nixos-21.05.4132.24528474d2b/nixos/pkgs/top-level/python-packages.nix:34:12:
while evaluating anonymous function at /nix/store/fyfzbrgqy4ir0nia9ypzkhfskcpwx3j0-nixos-21.05.4132.24528474d2b/nixos/pkgs/development/interpreters/python/mk-python-derivation.nix:30:1, called from /nix/store/fyfzbrgqy4ir0nia9ypzkhfskcpwx3j0-nixos-21.05.4132.24528474d2b/nixos/lib/customisation.nix:69:16:
pytools-2021.2.1 not supported for interpreter python2.7

Notify maintainers

@FRidh
@fgaz
@dezgeg
@mguentner
@vcunat
@shak-mar
@prusnak
@OmnipotentEntity
@Profpatsch
@jgillich
Metadata

system: "x86_64-linux"
host os: Linux 5.10.79, NixOS, 21.05.4132.24528474d2b (Okapi)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.3.16
channels(root): "nixos-21.05.4132.24528474d2b"
channels(k): ""
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

BitMessage is not compatible with Python 3 and since Python 2.7 is unsupported for almost 2 years (since January 2020) we will have to wait until the upstream releases new version that fixes that.

Thank you prusnak!


Updates to readme to avoid confusion between tobac 1 and tobac 2

I am too late ... but it looks great ;-)


Warning: Skipping DASH manifest

received the following warning message:
"richard@mx143:/DATL/ABA
$ youtube-dl -k https://www.youtube.com/watch?v=ac75H0SvZ5M
[youtube] ac75H0SvZ5M: Downloading webpage
[youtube] ac75H0SvZ5M: Extracting video information
WARNING: [youtube] ac75H0SvZ5M: Skipping DASH manifest: ExtractorError(u'Cannot decrypt signature without player_url; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.',)
[download] Destination: 24. Presentacion de reforzadores-ac75H0SvZ5M.f133.mp4
[download]  24.6% of 18.28MiB at 125.11KiB/s ETA 01:52"
Timed out after this, but restarted and continued without problem.
Maybe I need to include the "--youtube-skip-dash-manifest" option.
Found this about DASH,
http://lifehacker.com/preload-entire-youtube-videos-by-disabling-dash-playbac-1186454034
Thanks for a most useful app!

Duplicate of #5118

Thanks @TheGr33k, but the error message is different.

Cannot decrypt signature without player_url


Go client does not correctly insert, and gives no error

Bug Report
What version of Rust are you using?
1.29.0-nightly
What operating system and CPU are you using?
3 servers:
Intel(R) Xeon(R) CPU E5-2650 v3 @ 2.30GHz (20-core)
Ubuntu 16.04.5 LTS
client:
Intel(R) Core(TM) i7-7700 CPU @ 3.60GHz (8-core)
Ubuntu 16.04.5 LTS
What did you do?
I want to test the performance of TiKV, for this I create a lot of parallel connections and randomly put and get various key-value sizes. For this the RawKVApi Go client is used, and the parallel connections are made by assigning one Go client to each thread.
There are 3 PDs, one on each server, and 3 TiKV nodes, also one on each server (the same servers on which the PDs are running). Everything is set up using Docker (version 17.03.2-ce).
For 400 connections and a key-value size of 256 and 128 bytes respectively there is no problem: keys can be inserted for hours, and when getting them, everything checks out.
However, for the exact same configuration with a key-value size of 256 and 262144 bytes, and only 100 connections (or even lower), keys would not be inserted anymore, without the Go client returning any errors.
What did you expect to see?
I expected the Go client to simply correctly put the keys, or when a key was not inserted correctly, to return an error, which could be handled.
What did you see instead?
After approximately 7000 keys put, the client logging starts showing these messages:
time="2018-08-17T08:33:00+02:00" level=info msg="drop regions that on the store 1(<IP>:20160) due to send request fail, err: rpc error: code = Unavailable desc = transport is closing"
The Go client does not return any errors, but when getting the keys that should be there, they are not (empty values are returned).
Any help would be great, thank you!

Hi, there is currently an issue in our very recent master because one of our new feature is only partially merged and the remaining part is still under review.
Please try run this and restart TiDB:
pd-ctl -d config set cluster-version 2.1.0

@Connor1996 Are there ways to gate our new feature so that users won't fail because of this?

@breeswish seems no, I will merge it as soon as possible.

I tried this.
I also downgraded to a previous release (v2.0.6) of TiDB, but this also didn't help.
With the RawKVClient in go, I put randomly generated keys (10000) of length 256 bytes, and values equal to these keys, appended with zeros up to 262144 bytes. After each Put, I immediately do a Get and check the value, this is always correct, no empty values are returned.
However if I then do a Get for every key (in the same order), the last ~5000 keys (always the exact same amount) consistently return an empty value, even if the Get is sent to only one client, with a single thread. This causes me to believe the values are not present anymore, and there is a data leak.

hi @thomashlvt
can you show us your test program? do you retry insertion when meeting error?

@thomashlvt Could you try again with our TiKV previous release (2.0.6), or latest (i.e. today's) master? The fix for the issue has been shipped.

I found the problem, it was in the client code. My random keys were generated using rand.Intn, but this is also called in the RawKVClient code, making the random numbers not reproducible... (You might consider changing this and using a new rand.Rand struct for generating these numbers so it won't interfere with the code that drives the client. I solved the problem on my side by using this.)
Thank you for your time and the quick responses!


Use processIdentifier to allow multiple screens

Closes #20
You can have multiple screens by either copying the app or by using open -n /Applications/DeskPad.app
But brings in a new set of problems..

Because the display is no longer the same on each run, macOS probably won't save the screen arrangement anymore (and you'll probably trash your hidden macOS config as it sees a new display on each run, so you accumulate more displays than normally possible).

A temporary solution might be to look for the number of application instances and only attach the PID if there is already another instance.

The index of each display is likely similar, but quite long; so it's hard to differentiate windows easily.

A temporary solution might be to include the bundle filename, so users can have their own names for virtual displays.

I also barely tested this (due to macOS permissions being problematic when running from xcode), but I'll probably have a couple of real-world use-cases in the next couple of months where I'll actually test this.

I'm not a swift coder, so this is just the bare minimum to get the ball rolling.
I've also tried or thought about other strategies for display-names / serial numbers:

Start at serial-number 1, then iterated over NSScreens.screens to find a free name, incrementing the serial-number on collisions: this would likely lead to race-conditions; it also needs more changes for communication between AppDelegate and ScreenViewController.
Use the bundle path as identifier, so users can have named displays and hash the bundle-path to get a unique serial number. Unfortunately, this would still break for multiple instances.

I think the best option would be to allow configuring the amount of displays the user wants for a single instance.
However, I don't know enough swift to actually work on this myself.
Hopefully someone is being nerd-sniped here to actually work on a proper implementation.

I made a similar mod, though it has a static number of screens (4) rather than dynamic.  I originally used a random number for the serial number which resulted in the arrangement not being stored since MacOS viewed each screen as different than it had previously seen.  I ended up statically assigning serial numbers (1, 2, 3 ,4) so that they all go back to the previous arrangement.
Your setup is similar to my original problem with the arrangement due to the serial number being based on PID.  I'd suggest instead of doing it on PID that you instead identify which instance number you have open, something along the lines of (pseudocode):
$serialNumber =  $(listProcesses | find Deskpad | count) + 1
This way the serial numbers will be the same each time and you will retain your screen arrangement.
If you'd prefer a static number of screens, see my fork (which you can tweak to be +/- the 4 I have currently set.  https://github.com/agreenbhm/DeskPad


NodeNameFragment is not public (not exported)

I'm trying to do funk stuff with attribute names, and unfortunately I can't dig my way fully down into a NodeName. NodeNameFragment (at the very least) is missing a re-export I would think.

currently have a pr open about this. https://github.com/rs-tml/rstml/pull/28

Sorry was in the hospital, I have merged pr, and will release new version soon.

thank you @vldm, and i hope that all is well. 🙏🏽

Fixed in 0.11.1


Just some clarifications...

Your work is perfect for a project I'm working on with X-Plane.
I have just a few questions:

in the line below, what does the "25" stand for? Is it arbitrary or does it have a useful meaning?

connector.Subscribe(XPlaneConnector.DataRefs.CockpitRadiosNav1FreqHz, 25, (element, value) => {Log($"{element.DataRef}: {value}");});

I need to subscribe to a rather large subset of Datarefs. Is there a way to subscribe to all of them without having to write a line of code for each dataref? Or, can I subscribe to all of them with just a few lines of code? Or even better, could I subscribe to all the "**OperationFailuresRel *** " datarefs in a few lines of code?

I can't begin to tell you how appreciative I am for your work here. Thank You!
Ray S.
raysot @ Comcast.net

Hello,
thank you, I'm glad you appreciate my library.
About your questions:

the "25" you see on that line is the frequency, it tells the library about how many times per second X-Plane must update the DataRef.
The library handles all the communications, and informs you only when the value actually changes.
For examples, on that line you request X-Plane to send the info about Nav1 Hz 25 times per second, the callback will only be called when the Nav1 Hz actually changes.

Since you must provide a callback to handle the values, there's no massive subscription method by now.
Moreover I didn't find a wildcard option to subscribe to X-Plane, I will check it out in the next versions if it will be implemented and eventually I will update the library to handle it.
I think a workaround you could use is declaring a generic callback that can handle all types of values you need, then defining an array with all the DataRefs you want to subscribe to and Subscribe in a loop.

foreach(var dr in MassiveDataRefArray)
{
   connector.Subscribe(dr, 25, GenericDRHandler);
}

Then
private void GenericDRHandler(DataRefElement element, float value)
{
    switch(element.DataRef)
    {
         case "sim/cockpit/radios/nav1_freq_hz":
          break;
          /* ... */
      }
}

Let me know if that helps, if you have further questions feel free to contact me again.


Dissasemble/Improve data for rule 942280

Motivation
All regexes should be generated using regexp-assembly.
Proposed solution
add prefix and suffix

Fix was merged, closing.


LUCENE-9956: Make getBaseQuery, getDrillDownQueries API from DrillDownQuery public

Description
It would be great if users could access the baseQuery of a DrillDownQuery. I think this can be useful for folks who want to access/test the clauses of a BooleanQuery (for example) after they've already wrapped it into a DrillDownQuery.
Currently the Query getBaseQuery() method is package private by default.
Based on comments from Greg Miller in the JIRA ticket, I decided to expose getDrillDownQueries as well
Solution
Change the method signature to public
Tests
Not added
Checklist
Please review the following and check all that apply:

[x] I have reviewed the guidelines for How to Contribute and my code conforms to the standards described there to the best of my ability.
[x] I have created a Jira issue and added the issue ID to my pull request title.
[x] I have given Lucene maintainers access to contribute to my PR branch. (optional but recommended)
[x] I have developed this patch against the main branch.
[x] I have run ./gradlew check.
[ ] I have added tests for my changes.

Looks great. Thanks @gautamworah96!


[BUG] Private or Hidden Modpaks are always shown

A small oversight, it is possible that one of the API key is "string (0)"
In this case the control:
    if ($key->api_key == Input::get('key')) {
        $this->key = $key;
    }

returns "TRUE"
I have found this bug on APIController, may be extended.
It would have to verify that the Input::get ('key') is not empty.

That is true, but would also require a blank API to be set by the user in Solder.

Yes, but old install have the "Default" key empty.
For example I have my own launcher and I don't need the Platform.

Ah, I didn't realize old installs had an empty key.

No problem :D

Thank you for your fix!

Proud to have helped!


HA for deployment tool

Right now the InfraKit container runs only on one manager and uses the OS mode to manage it's plugins.
InfraKit is able to run on all 3 managers and use the Swarm annotations to share the deployment state across the nodes.
Once implemented in our deployment scripts, we should check that the loss of a manager node leads to the election of a new InfraKit leader (as well as a new Swarm manager leader), and that when the missing manager is reestablished it is correctly incorporated in the cluster (Swarm manager and Infrakit, both non leader).

This is possible to do using infrakit-manager swarm functionality which allows for the synchronization of the infrakit leaders with the docker swarm leaders. Along with this, the election process involves freeing any groups managed by the current swarm leader infrakit node and committing them to the new leader.
However, because we are using terraform to manage the health of the instances within the recommitted groups, the .tfstate files that are generate on the original m1 node are moved over to the new leader in the instance that m1 is terminated, and so the health of the worker group is no longer managed by infrakit.


New release?

Several changes have been made since the last release, when could a new one be published?

Depends on #20 I guess?


Dynamic showing names and so on

When we pop an overlay, IE Commentators, would like a dynamic opening for the overlay instead of just showing it plain.
Also needed the ability to remove the frame from the overlay (Hide overlay on toggle)
Needed for:

Player cam
Commentator cam
....

So basically animations of the elements. That can be done on the overlay itself. This doesn't need any change to the HypestBoard software itself. This would be on a game by game, overlay by overlay basis.
However for the ability off hide/show the overlay, without using OBS/Xsplit needs to be implement on HypestBoard itself.
Good one. Will implement. Should be minor.

Gonna also change the title to make it more clear on what needs to be done.


Namespacing in the test table

Hey :D
Thanks a bunch this has been really useful in the past few days. Have you thought about trying to namespace the different arguments in the table. For example a sample s3 mock:
package main

import "fmt"

type S3Bucket struct {
	Name    string
	Storage map[string][]byte
}

func (s *S3Bucket) Save(name string, body []byte) error {
	if err := validateName(name); err != nil {
		return err
	}

	s.Storage[name] = body
	return nil
}

func validateName(name string) error {
	if len(name) == 0 {
		return fmt.Errorf("name must not be empty")
	}

	return nil
}

will produce this test:
func TestS3BucketSave(t *testing.T) {
	tests := []struct {
		name    string
		fname   string
		storage map[string][]byte
		pname   string
		body    []byte
		wantErr bool
	}{
	// TODO: Add test cases.
	}
	for _, tt := range tests {
		s := &S3Bucket{
			Name:    tt.fname,
			Storage: tt.storage,
		}
		if err := s.Save(tt.pname, tt.body); (err != nil) != tt.wantErr {
			t.Errorf("%q. S3Bucket.Save() error = %v, wantErr %v", tt.name, err, tt.wantErr)
		}
	}
}

but since the tests struct includes both the fields and arguments it can get a bit confusing trying mentally keep track of where things going. Would you consider doing something like this instead?
func TestS3BucketSave(t *testing.T) {
	tests := []struct {
		fields struct {
			name string
			body []byte
		}
		args struct {
			name    string
			storage map[string][]byte
		}
		name    string
		wantErr bool
	}{
	// TODO: Add test cases.
	}
	for _, tt := range tests {
		s := &S3Bucket{
			Name:    tt.fields.name,
			Storage: tt.fields.storage,
		}
		if err := s.Save(tt.args.pname, tt.args.body); (err != nil) != tt.wantErr {
			t.Errorf("%q. S3Bucket.Save() error = %v, wantErr %v", tt.name, err, tt.wantErr)
		}
	}
}

Glad to hear you're finding gotests useful. :)
That's actually a great suggestions! I was considering just putting comments like:
func TestS3BucketSave(t *testing.T) {
    tests := []struct {
        name    string
        // Fields.
        fname   string
        storage map[string][]byte
        // Parameters.
        pname   string
        body    []byte
        // Results.
        wantErr bool
    }{
    // TODO: Add test cases.
    }
    for _, tt := range tests {
        s := &S3Bucket{
            Name:    tt.fname,
            Storage: tt.storage,
        }
        if err := s.Save(tt.pname, tt.body); (err != nil) != tt.wantErr {
            t.Errorf("%q. S3Bucket.Save() error = %v, wantErr %v", tt.name, err, tt.wantErr)
        }
    }
}

But your suggestion would eliminate the namespacing hack I use (pname vs. fname). I'll need to think about the implementation some more, and of any edge cases. At first glance, it sounds reasonable.

On second thought, there are some issues with using structs to namespace different arguments, particularly when writing test cases:

If the developer omits one of the structs, they'll get a nil-dereference runtime error from the test code, not the code under test. No fun.
It requires extra work for each test case to create a new struct.

Instead, gotests could do the following:

Prefix an r to every receiver field
Leave parameter names as they are
Namespace the test name in case of conflict

Consider:
func TestS3BucketSave(t *testing.T) {
    tests := []struct {
        tname    string
        rName    string
        rStorage map[string][]byte
        name     string
        body     []byte
        wantErr  bool
    }{
    // TODO: Add test cases.
    }
    for _, tt := range tests {
        s := &S3Bucket{
            Name:    tt.rName,
            Storage: tt.rStorage,
        }
        if err := s.Save(tt.name, tt.body); (err != nil) != tt.wantErr {
            t.Errorf("%q. S3Bucket.Save() error = %v, wantErr %v", tt.tname, err, tt.wantErr)
        }
    }
}

My goal is just understandability. :D
Prefixing each with something as well as including the comments in the example you had earlier would probably accomplish that goal.

On second thought, I prefer your suggestion with the structs. I'm not a big fan of the comments, and the prefix name-spacing is kind of confusing (rName vs pName).


installation with centos CentOS v7.9 reporting [FATAL] This script is designed to only run on CentOS 7 servers

I have an installation with centos CentOS v7.9.2009. I am following the instructions to use cPanel ELevate Project and to update them to almalinux.
After downloading the tool I ran / scripts / elevate-cpanel --check.
I get this error:
[FATAL] This script is designed to only run on CentOS 7 servers.
This despite my version of centos is 7

Hello,
What version of cPanel is being used? Additionally, can you provide the output of stat /var/cpanel/caches/Cpanel-OS as run in the shell?

hello!
cpanel version: cPanel & WHM v104.0.7 (STANDARD)
[root@server19 ~]# stat /var/cpanel/caches/Cpanel-OS
File: ‘/var/cpanel/caches/Cpanel-OS’ -> ‘linux|centos|7|9|2009’
Size: 21              Blocks: 0          IO Block: 4096   symbolic link
Device: 802h/2050d      Inode: 11797709    Links: 1
Access: (0777/lrwxrwxrwx)  Uid: (    0/    root)   Gid: (    0/    root)
Access: 2021-12-09 09:58:31.921026928 +0100
Modify: 2021-12-09 09:58:31.921026928 +0100
Change: 2021-12-09 09:58:31.921026928 +0100
Birth: -

This is caused by a flaw in the program. Line 603 should be comparing Cpanel::OS::distro() against centos (all lowercase) rather than Centos. #116 will fix this. Alternatively, it should not happen on cPanel version 106 and forward, because those know explicitly whether the distro supports ELevate. Thank you for letting us know about this issue.


Fix stream member list when creating streams.

I am pretty sure this is a regression.  When you create a stream, as you check off each member to add, I believe we had a UI element that showed all the users you had added so far, but it seems to have gone away.  For large realms, the current experience is barely usable.  Hopefully this is just something silly like a selector got renamed or something.

I'm unaware of this mechanism actually. I also don't see any elements that appear to be containers for such a thing. Any thoughts/leads on this?

It's possible that the UI has always been like this.  I'm closing this.  We really need to use the add-subscriber UI here, it's at least a little more friendly, although it will be slightly complicated with copy-from-stream.


add a new version containing srcs in cc_library for the tinyobjloader

This version has to changes compared to the 2.0.0-rc1 version:

removes the warning flag that is problematic on some clang versions.
add srcs to the cc_library.

Hello @Vertexwahn, modules you maintain (tinyobjloader) have been updated in this PR. Please review the changes.

needs the skip-url-stability-check and a maintainer review to pass CI

Hello @sergeykhliustin, modules you maintain (bazelpods) have been updated in this PR. Please review the changes.

@Vertexwahn : sorry was offline the past days ... updated the version string according to the comment


chore(master): release 1.1.0

:robot: I have created a release beep boop
1.1.0 (2023-05-24)
Features

added reame (022b7bd)

This PR was generated with Release Please. See documentation.

Coverage report

St.:grey_question:
Category
Percentage
Covered / Total

🟢
Statements
100%
91/91

🟢
Branches
100%
9/9

🟢
Functions
100%
6/6

🟢
Lines
100%
91/91

Test suite run success
5 tests passing in 1 suite.
Report generated by 🧪jest coverage report action from 5f48d6866e43842864bb42dc2a146e519679f913

:robot: Release is at https://github.com/dworac/logger/releases/tag/v1.1.0 :sunflower:


discoverapi.Discover interface

Move DiscoverNew() and DiscoverDelete() methods into the new interface
Add DatastoreUpdate notification
Now this interface can be implemented by any drivers, not only network drivers

Signed-off-by: Alessandro Boch<EMAIL_ADDRESS>

:+1: LGTM.

LGTM


请问训练和测试都出现RuntimeError: CUDA error: invalid device function错误

你好，请问训练和测试都出现如下错误，是否环境配置原因？
EMD-CD:   0%|                                                                                | 0/19 [00:00<?, ?it/s]
Traceback (most recent call last):
File "train_ae.py", line 203, in
cd_loss = validate_loss(it)
File "train_ae.py", line 169, in validate_loss
metrics = EMD_CD(all_recons, all_refs, batch_size=args.val_batch_size, accelerated_cd=True)
File "/userhome/point_cloud/diffusion-point-cloud-main/evaluation/evaluation_metrics.py", line 58, in EMD_CD
cd_lst.append(dl.mean(dim=1) + dr.mean(dim=1))
RuntimeError: CUDA error: invalid device function
Segmentation fault (core dumped)

What's your GPU model?

What's your GPU model?

RTX2080

Can confirm getting the same error running on RTX2080

I will release a version that doesn't require compiling CUDA extensions soon.

请问有知道这个问题怎么解决吗？

Hi, I have a quick question, did you able to build StructuralLossesBacken ? If yes, then would you please let me know how ? I am getting a g++ error ( created an issues already but no response yet).

Hi all,
EMD_CD functions are for validation purpose only. The training doesn't rely on them, so you may remove this part of codes in the training script.

请问有知道这个问题怎么解决吗？
出现这个问题是evaluation目录下面那个要编译的库没有编译好，在Makefile中有个参数：CUDA_ARCH，这个参数是设置GPU的算力，不同的GPU有不同的算力，可以在NVIDIA官网：https://developer.nvidia.com/zh-cn/cuda-gpus#compute查询，然后就是设置自己gpu对应的算力。当然，可以考虑兼容性的问题，不容易崩，具体设置参考：
https://arnon.dk/matching-sm-architectures-arch-and-gencode-for-various-nvidia-cards/

Hi,
My compilation on CUDA 10.1 succeed and the invalid device function error disappears. I guess this error is caused by a mismatch between 10.0 and 10.1. So you can try to compile the metrics on CUDA 10.1.


Add version tags

The contributing guidelines state that this project follows semver but there are no commits annotated with tags. Can you annotate the commits with tags so Github recognizes them as releases and makes it easier to view the repository at a certain release? (I don't know whether it is possible to open a pull request for that)
Are there any plans to publish a new version to npm since there have been quite a few changes since v0.0.10?

Hi Fabian,
Actually I do not have much knowledge about this. I'd be glad if you would
join the team and help with that! 😃
Cedric Poilly
Freelance Web Developer
Mentor at Codementor.io
https://www.codementor.io/cedpoilly https://www.codementor.io/cedpoilly
On Wed, 4 Sep 2019 at 22:26, Fabian Meyertöns<EMAIL_ADDRESS>wrote:

The contributing guidelines state that this project follows semver but
there are no commits annotated with tags. Can you annotate the commits with
tags so Github recognizes them as releases and makes it easier to view the
repository at a certain release? (I don't know whether it is possible to
open a pull request for that)
Are there any plans to publish a new version to npm since there has been
quite a few changes since v0.0.10?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/cedpoilly/vue-gapi/issues/31?email_source=notifications&email_token=ACT6PK6CR35IJLOU3RVJ7TLQH74WXA5CNFSM4ITVCX52YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HJKO7HQ,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACT6PK6GNUVYNFOACUR6NR3QH74WXANCNFSM4ITVCX5Q
.

I can help you with that but only collaborators can do that (i.e. people with push access). It's actually straightforward and you can even do it on Github under the releases tab of the repo.

I have added you to the list of contributors. Shoot me an email at<EMAIL_ADDRESS>


Light Mask layers seem to be ignored by Light2D

3.2.1 Stable Official
macOS 10.15.4
I have two sprites and two Light2Ds in a scene. Each pair is on a different light mask layer. Only the sprite on Light Mask Layer 1 picks up the Light2D, but both the Light2Ds, regardless of light mask assignment. I was going to use this as a masking trick for two layers of graphics, but hit a brick wall.
Steps to reproduce: Set up a scene as described above. I've not tested every edge case, but a Light2D on any (or no light mask layer) will only be picked up by a Sprite on Light Mask Layer 1. The blending mode doesn't seem to matter, even though my reduction project is specifically testing mix mode.
Light2d reduction.zip

Duplicate of #29612.


PHP version required! ! !

Dude, consider a PHP version! ! ! :-D

I don't like PHP language :(
I have no plans to develop it subsequently, so any contributors are welcome to finish this.

related to https://github.com/zema1/suo5/issues/2


Crashes with "Cannot find module"

Describe the bug
See the below Output and Reproduction.
Perhaps something weird; I have specified vitest version 2.0.5 in package.json and running with CLI works. Still the extension prints out vitest version 1.5.1.
Reproduction
Closed source code, but here are the relevant files (slightly modified)
// The file called findEnv.test.ts
/* eslint-disable no-template-curly-in-string */
import { env } from 'process';
import { findEnv } from './findEnv.js';

const CLIENTID = 'prefix/app/${APP_VERSION}';

describe('config', () => {
  it('should find env', () => {
    env.APP_VERSION = '1.0.0';

    const result = findEnv(CLIENTID);
    expect(result).toBe('prefix/app/1.0.0');
  });

  it('should throw error', () => {
    delete env.APP_VERSION;

    expect(() => findEnv(CLIENTID)).toThrowError(
      /Could not find env APP_VERSION/,
    );
  });
});

// The file called findEnv.ts
import { env } from 'process';

export function findEnv(str: string, strict: boolean = true): string {
  // Simplified code
  if (str === 'prefix/app/1.0.0') {
    return str
  } else {
    throw new Error(`Could not find env APP_VERSION`);
  }
}

Output
[INFO 2:17:42 PM] [v1.0.7] Vitest extension is activated because Vitest is installed or there is a Vite/Vitest config file in the workspace.
[INFO 2:17:43 PM] [API] Running Vitest v1.5.1 (my-frontend-app/vite.config.ts) with Node.js: /home/my-username/.nvm/versions/node/v20.14.0/bin/node
[INFO 2:17:43 PM] [API] Vitest v1.5.1 (my-frontend-app/vite.config.ts) process 1466673 created
[INFO 2:18:00 PM] [API] Collecting tests: src/api/utils/findEnv.test.ts
[Worker]
node:internal/event_target:1099
  process.nextTick(() => { throw err; });
                           ^
Error: Cannot find module '/home/my-username/repos/my-frontend-app/node_modules/tinypool/dist/esm/entry/worker.js'
    at Function._resolveFilename (node:internal/modules/cjs/loader:1145:15)
    at Function._load (node:internal/modules/cjs/loader:986:27)
    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:174:12)
    at MessagePort.<anonymous> (node:internal/main/worker_thread:186:26)
    at [nodejs.internal.kHybridDispatch] (node:internal/event_target:825:20)
    at MessagePort.<anonymous> (node:internal/per_context/messageport:23:28)
Emitted 'error' event on Tinypool instance at:
    at EventEmitterReferencingAsyncResource.runInAsyncScope (node:async_hooks:206:9)
    at Tinypool.emit (file:///home/my-username/repos/my-frontend-app/node_modules/tinypool/dist/esm/index.js:61:31)
    at Worker.<anonymous> (file:///home/my-username/repos/my-frontend-app/node_modules/tinypool/dist/esm/index.js:752:30)
    at Worker.emit (node:events:519:28)
    at [kOnErrorMessage] (node:internal/worker:326:10)
    at [kOnMessage] (node:internal/worker:337:37)
    at MessagePort.<anonymous> (node:internal/worker:232:57)
    at [nodejs.internal.kHybridDispatch] (node:internal/event_target:825:20)
    at MessagePort.<anonymous> (node:internal/per_context/messageport:23:28) {
  code: 'MODULE_NOT_FOUND',
  requireStack: []
}

Node.js v20.14.0

Version
1.0.7
Validations

[X] Check that you are using the latest version of the extension
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Ok then, no help from me.


Multi label classification with binary_crossentropy and class_weight - must contain all classes error

Hello,
I have run into an issue with class_weights and binary_crossentropy for multi label outputs.
In essence:
Output is an ndarray Y with Y.shape[1] the number of label outputs, e.g.:
[[0,0,1,0,1,1,0],
[0,0,1,0,1,1,1],
....
[0,0,0,0,1,0,0]]
I have defined the output layer as follows (method A):
...
out_dense=Dense(Y.shape[1],
                 activation='sigmoid', name='OUTPUT_BINARY')(intermediate_dense)

and defined the model:
model = Model(inputs=[... some inputs...], outputs=out_dense)
model.compile(loss='binary_crossentropy', optimizer='adam', metrics=['accuracy'])

weight dictionary defined as follows:
W = {0.0: 0.54356142358280057, 1.0: 6.2390227278685089}
Now, when I fit:
model.fit([...some inputs...], Y, batch_size=64, epochs=1, validation_split=0.25, class_weight={'OUTPUT_BINARY':W}, sample_weight=None)
I get the error:
/usr/local/lib/python3.5/dist-packages/keras/engine/training.py in fit(self, x, y, batch_size, epochs, verbose, callbacks, validation_split, validation_data, shuffle, class_weight, sample_weight, initial_epoch, steps_per_epoch, validation_steps, **kwargs)
   1521             check_batch_axis=False,
   1522             batch_size=batch_size)
-> 1523         # Prepare validation data.
   1524         do_validation = False
   1525         if validation_data:

/usr/local/lib/python3.5/dist-packages/keras/engine/training.py in _standardize_user_data(self, x, y, sample_weight, class_weight, check_batch_axis, batch_size)
   1388                           for (ref, sw, cw, mode)
   1389                           in zip(y, sample_weights, class_weights, self._feed_sample_weight_modes)]
-> 1390         _check_array_lengths(x, y, sample_weights)
   1391         _check_loss_and_target_compatibility(y,
   1392                                              self._feed_loss_fns,

/usr/local/lib/python3.5/dist-packages/keras/engine/training.py in <listcomp>(.0)
   1387         sample_weights = [_standardize_weights(ref, sw, cw, mode)
   1388                           for (ref, sw, cw, mode)
-> 1389                           in zip(y, sample_weights, class_weights, self._feed_sample_weight_modes)]
   1390         _check_array_lengths(x, y, sample_weights)
   1391         _check_loss_and_target_compatibility(y,

/usr/local/lib/python3.5/dist-packages/keras/engine/training.py in _standardize_weights(y, sample_weight, class_weight, sample_weight_mode)
    556                              '`class_weight`.'
    557                              % (existing_classes - existing_class_weight))
--> 558         return weights
    559     else:
    560         if sample_weight_mode is None:

ValueError: `class_weight` must contain all classes in the data. The classes {2, 3, 4, 5, 6, 7, 8, 9, 10, 12, 13, 14, 15, 17, 19, 20, 21, 28, 29, 31, 34, 36, 37, 38, 41, 43, 44, 45, 47} exist in the data but not in `class_weight`.

I have traced it to :
def _standardize_weights(y, sample_weight=None, class_weight=None,
                         sample_weight_mode=None):
...
        if y.shape[1] > 1:
            y_classes = y.argmax(axis=1)
        elif y.shape[1] == 1:
            y_classes = np.reshape(y, y.shape[0])
        else:
            y_classes = y

in training.py.
It seems that here keras sees the output as a softmax type one hot encoded and returns the index of the maximum for each row.
The only way I saw to fix this was to create multiple single outputs by using a for loop (method B):
outputs=[]
l_weights=[]
for i in range(y_in_service_indicator.shape[1]):
    out_dense=Dense(1, activation='sigmoid', name='O_'+str(1+i))(intermediate_dense)
    outputs.append(out_dense)
    l_weights.append(1.0)

model = Model([...some inputs...], outputs=outputs)
model.compile(loss='binary_crossentropy', loss_weights=l_weights,
              optimizer='adam', metrics=['accuracy'])

(I appreciate may not need the loss weights)
and define the outputs as :
y_s_binary = []
for i in list(range(Y.shape[1])):
    y_s_binary.append(y_in_service_indicator[:,i])

and now fit:
model.fit(X_intervals, y_s_binary, batch_size=64, epochs=1, validation_split=0.25,
          class_weight={'OUTPUT_BINARY':W}, sample_weight=None)

This works, but it feels to me like I am missing somehting - would appreciate input on how to make Keras work with single defined Y ndarray and model output as  in method A.

+1 for fixing this! I'm having the same problem!

@maxim5 would that change the optimization process though? The model may quickly learn to always predict 0 for the first dummy class and 1 for the second. Together, they drag the aggregate loss down too quickly and I wonder how that would impact the optimization.

@Runze Since the total loss is the sum of the constituent losses, the value will be affected initially until the model learns the constants for the first two labels, after which the effect will diminish and the error signal will backpropagate just as before.

@maxim5  Thanks for the workaround. However, adding dummy classes requires changing codes in evaluations and so on, maybe we should think of fixing the problem within keras ?

@sharifza Of course. That's why this bug is open.

+1 for fixing this! I'm also have the same problem!

hi, is there any corresponding new api in keras? i get a similar error when setting the class weight with keras version 2.2.0. And my Y is something like:
[[1,0,0,1], [1,1,0,0] [1,0,1,0]]
so for this multi-class and multi-label problem i set
class_weight={0.0:0.6,1.0:0.4} in the model.fit() function,and i get an error message says:
ValueError: class_weightmust contain all classes in the data. The classes {2, 3} exist in the data but not inclass_weight.

i wonder if there is a certain api like tf.nn.weighted_cross_entropy_with_logits in tensorflow ?

I confirm, that I also have the same error on the newest version installed from github  @alanlisten

@goyidao well, you must give weight to four classes, and you only give weights to the first two categories.
e.g. class_weight={0: 0.1, 1: 0.22, 2: 0.45, 4: 0.35}

@dominik335 I updated the answer, you can look at it.

I am also having the error is there a solution or workaround. Below is the error I get when implementing CNN and multilabel multiclass problem.

ValueError                                Traceback (most recent call last)
 in
32                     validation_data=validation_generator,
33                     validation_steps=STEP_SIZE_VALID,class_weight=class_wts,
---> 34                     epochs=5
35 )
/opt/conda/lib/python3.6/site-packages/keras/legacy/interfaces.py in wrapper(*args, **kwargs)
89                 warnings.warn('Update your ' + object_name + ' call to the ' +
90                               'Keras 2 API: ' + signature, stacklevel=2)
---> 91             return func(*args, **kwargs)
92         wrapper._original_function = func
93         return wrapper
/opt/conda/lib/python3.6/site-packages/keras/engine/training.py in fit_generator(self, generator, steps_per_epoch, epochs, verbose, callbacks, validation_data, validation_steps, class_weight, max_queue_size, workers, use_multiprocessing, shuffle, initial_epoch)
1416             use_multiprocessing=use_multiprocessing,
1417             shuffle=shuffle,
-> 1418             initial_epoch=initial_epoch)
1419
1420     @interfaces.legacy_generator_methods_support
/opt/conda/lib/python3.6/site-packages/keras/engine/training_generator.py in fit_generator(model, generator, steps_per_epoch, epochs, verbose, callbacks, validation_data, validation_steps, class_weight, max_queue_size, workers, use_multiprocessing, shuffle, initial_epoch)
215                 outs = model.train_on_batch(x, y,
216                                             sample_weight=sample_weight,
--> 217                                             class_weight=class_weight)
218
219                 outs = to_list(outs)
/opt/conda/lib/python3.6/site-packages/keras/engine/training.py in train_on_batch(self, x, y, sample_weight, class_weight)
1209             x, y,
1210             sample_weight=sample_weight,
-> 1211             class_weight=class_weight)
1212         if self._uses_dynamic_learning_phase():
1213             ins = x + y + sample_weights + [1.]
/opt/conda/lib/python3.6/site-packages/keras/engine/training.py in _standardize_user_data(self, x, y, sample_weight, class_weight, check_array_lengths, batch_size)
799                 for (ref, sw, cw, mode) in
800                 zip(y, sample_weights, class_weights,
--> 801                     feed_sample_weight_modes)
802             ]
803             # Check that all arrays have the same length.
/opt/conda/lib/python3.6/site-packages/keras/engine/training.py in (.0)
797             sample_weights = [
798                 standardize_weights(ref, sw, cw, mode)
--> 799                 for (ref, sw, cw, mode) in
800                 zip(y, sample_weights, class_weights,
801                     feed_sample_weight_modes)
/opt/conda/lib/python3.6/site-packages/keras/engine/training_utils.py in standardize_weights(y, sample_weight, class_weight, sample_weight_mode)
518                              ' The classes %s exist in the data but not in '
519                              'class_weight.'
--> 520                              % (existing_classes - existing_class_weight))
521         return weights
522     else:
ValueError: class_weight must contain all classes in the data. The classes {0, 1, 2, 3, 4, 5, 6, 8, 9, 10, 11, 12, 14} exist in the data but not in class_weight.


test run for move action

typing 'on it' should move any to do issue to in progress

on it

on it

on it

on it

on it

on it

on it

on it

on it

on it

on it

on it

on it

on it

on it

on it

on it

on it


Implement rules using a trie data structure.

Hi there,
I spent some time this week-end implementing a different representation for the rules: replacing the regex by a more efficient Trie data structure. This changes a lot of code, so I would understand that it might require more work to make "production-ready". Also, comments are pretty sparse at the moment, but I would be glad to provide more where you think it makes sense!
I implemented some benchmark to have an idea of the performance gain, and it looks promising (although I did not register the memory consumption, I don't expect it to be unreasonable). Each run is calling the benchmarked function on 100 random domains and I'm using the benchmark npm package (I can attach the code somewhere as well if you want to have a look):
Current version of the code on master (v2.0.0)
tldjs#tldExists x 3,162 ops/sec ±1.74% (92 runs sampled)
tldjs#getDomain x 35.82 ops/sec ±1.65% (62 runs sampled)
tldjs#getSubdomain x 35.77 ops/sec ±1.60% (62 runs sampled)
tldjs#getPublicSuffix x 37.39 ops/sec ±1.13% (64 runs sampled)

New implementation based on the Trie data-structure
tldjs#tldExists x 3,265 ops/sec ±0.83% (92 runs sampled)
tldjs#getDomain x 2,030 ops/sec ±3.08% (84 runs sampled)
tldjs#getSubdomain x 2,037 ops/sec ±2.85% (85 runs sampled)
tldjs#getPublicSuffix x 2,195 ops/sec ±3.53% (83 runs sampled)

So the speed-up is around x60.
It now seems that cleanHostValue is the bottle-neck, since if we make it a no-op, the results become:
tldjs#tldExists x 57,589 ops/sec ±2.23% (92 runs sampled)
tldjs#getDomain x 8,414 ops/sec ±0.75% (94 runs sampled)
tldjs#getSubdomain x 8,257 ops/sec ±0.77% (94 runs sampled)
tldjs#getPublicSuffix x 10,857 ops/sec ±1.27% (92 runs sampled)

I added a flag (dirtyHost) on each of these methods to allow disabling the cleaning (if you know that your host value is already clean and you want maximum performances).
While implementing this new representation, I stumbled upon some cases for which I'm not really sure of the good behavior. I would love to have your feedback on them.
Result if the public suffix does not exist
The following test-case: expect(tld.getPublicSuffix('www.freedom.nsa')).to.be(null); seems to expect the wrong result. According to the specification (https://publicsuffix.org/list/). When no matching rule is found, then "the prevailing rule is "*".", which from what I understand means that we consider the latest label of the hostname to be the public suffix. The result for this test would then be nsa instead of null. What do you think?
Result of exceptions
The following test-case: expect(tld.getPublicSuffix('www.www.ck')).to.be('www.ck'); should probably expect ck instead of www.ck as a public suffix. According to the aforementioned specification, the following should be done when an exception is matched (here: !www.ck):

If the prevailing rule is a exception rule, modify it by removing the leftmost label.

I guess we should then remove www (the leftmost label), resulting in a public suffix of ck.
Result when not public suffix is found
What should be the result of getDomain when no valid public suffix is found? Should it be null as well, or the value passed as host to the function?
Ignoring trailing and leading dots
The specification says that trailing and leading dots found on rules should be ignored. Although there is no such case at the moment, maybe we should handle them and add a test-case? I left a TODO in the parser.
Thanks for reading, and I hope this work can be useful to the project,
Have a nice evening.

Hey @remusao,
thanks for the hard work! Figures are impressive 🙂
I will try to have a close look at it tomorrow—I am viewing houses in London at the moment and some transcripts take the rest of my time at the moment.

When no matching rule is found, then "the prevailing rule is "*".", which from what I understand means that we consider the latest label of the hostname to be the public suffix. The result for this test would then be nsa instead of null. What do you think?

Yeah, I feel it would be nicer to respect what the public suffix spec says — I was feeling frustrated about the methods, what they were doing and for what in #25 (#54 maybe). It started from "using public-suffix.org to enforce a verification of known TLDs" to "implementing PublicSuffix for cookies and stuff while trying to address the former intention".

Hey @oncletom,
Thanks for the answer. Let me know if I can do anything to ease the review (more comments, a tl;dr about the implementation details, or more direct talk are possible!).
Regarding the issues you mention, I was thinking about using only the ICANN section of the file to get the domain (and use the PRIVATE section only to get the public suffix). Do you think it could work?

@remusao ICANN is a good idea! Or IANA (cf. #40) too – I don't know which one validates gTLD first. It would be useful if the Public Suffix list does not encompass all the gTLDs or with a major delay.
I found a place to hopefully Sunday I should be able to put my head around your proposal. Will let you know if I have any question or else — if other contributor(s) want to have a peak at it, they are obviously welcome to do so 🙂

@oncletom Thanks for your reply! I understand that time is precious and this is a big PR. If I can do anything to make it easier to review, let me know.

@remusao since I'm very interested in this PR getting merged I tried to verify the two cases that you highlight in the issue description.
www.freedom.nsa
I think your reasoning is right, the public suffix should be nsa. To verify the reasoning I simply took the definitions from https://publicsuffix.org/list/ and highlighted the essential parts:
Definitions

[ … ]
A domain is said to match a rule if and only if all of the following conditions are met:

When the domain and rule are split into corresponding labels, that the domain contains as many or more labels than the rule.
Beginning with the right-most labels of both the domain and the rule, and continuing for all labels in the rule, one finds that for every pair, either they are identical, or that the label from the rule is "*".

Algorithm

Match domain against all rules and take note of the matching ones.
If no rules match, the prevailing rule is "*".
If more than one rule matches, the prevailing rule is the one which is an exception rule.
If there is no matching exception rule, the prevailing rule is the one with the most labels.
If the prevailing rule is a exception rule, modify it by removing the leftmost label.
The public suffix is the set of labels from the domain which match the labels of the prevailing rule, using the matching algorithm above.
The registered or registrable domain is the public suffix plus one additional label.

prevailing rule = *

Domain: | www | freedom | nsa |
Rule:   |     |         |  *  |

public suffix = nsa

www.www.ck
I think your reasoning is again right (public suffix should be ck):

Match domain against all rules and take note of the matching ones.

Matching rules:
*.ck
!www.ck

If no rules match, the prevailing rule is "*".

If more than one rule matches, the prevailing rule is the one which is an exception rule.

Matching rule:
!www.ck

If there is no matching exception rule, the prevailing rule is the one with the most labels.

If the prevailing rule is a exception rule, modify it by removing the leftmost label.

Modified matching rule:
ck

The public suffix is the set of labels from the domain which match the labels of the prevailing rule, using the matching algorithm above.

The registered or registrable domain is the public suffix plus one additional label.

prevailing rule = ck

Domain: | www | www | ck |
Rule:   |     |     | ck |

public suffix = ck
registered domain = www.ck

nice work @remusao !

@remusao I have been thinking about the clean host vs. dirty host. I agree there is a need for cleaning if it is relevant only. But by changing the interface we can no longer trust the flag which states if the value is clean or not.
In my opinion, the cleanHost function should return a frozen CleanHostString object. Maybe like this:
class CleanHostString extends String {
  constructor(value) {
    super(value);

    this.hostValue = value instanceof CleanHost ? String(value) : cleanHost(value);

    // we ensure the object becomes immutable
    Object.freeze(this);
  }

  // returns the value when the conversion to String primitive happens, with `String(obj)`
  valueOf() {
    return this.hostValue;
  }
}

What do you think?

@oncletom Yeah the flag might not be the ideal solution. What about just asking users to clean the url before giving it to functions of tldjs? So that we simplify the API and all functions expect an already clean host.
Another option could be to wrap clean host in a different type and check from each function if we are given an already clean host, or if the cleaning needs to happen? But then it's not propagated to the caller and we still might have redundancy. Although I guess it's nice because we hide all the complexity from the user and it's more efficient than what we have now (but then users cannot chose to not have any pre-processing at all, or do it themselves, is it something we want to offer?).
Also, on a side note, I started working on a more efficient version of cleanHost to limit its impact on the overall performance of the library.
Thanks for your answer! If you think that the best option is to wrap the clean host in a different type, I can implement it in this PR.
Also, I see you make use of class in your example, and it made me wonder what version of Javascript this library should support? es5, es6, something else?

@oncletom No problem for the delay, thank you so much for reviewing. I will address your comment and add some changes.
One thing that remains is what to do regarding the two failing tests, should I change them to adopt a behavior as described by the specification?

I have not thought much about it — I realised only after publishing the comment the codebase was in ES5 😄 If the trie implementation does not break anything in the API, I'd rather keep it as is and later do a major bump to move to ES2015, when Node 8 is out for example.

Makes sense, I will keep the code compatible (the trie should not break anything).

One thing that remains is what to do regarding the two failing tests, should I change them to adopt a behavior as described by the specification?

Yes, please do proceed as is 🙂 it is totally fine to fix bugs along the way!

@oncletom Actually, after thinking a bit about this cleanHost preprocessing, and if we're ok making a bold mode but being totally clear about it, it would make some sense to me that we only have functions working on hostnames, and provide a function to extract the hostname of any url (cleanHost). In a way it would be conceptually simpler and function would just do one thing. On the other hand, that puts more responsibility in the hands of the user of the library since they have to make sure they only give hostnames to the different functions.
Another option could be to offer two versions of each function: getDomain + getDomainFromHostname, getSubdomain + getSubdomainFromHost, etc. But it does not look as good as the first option IMHO.
The only thing I'm not really confident with, is that we don't have a good way to tell the caller that what was passed in is not a valid hostname and they should clean it first. Which might lead to a lot of silent failures (we would just return null). But then such check would also incur a cost.
I need to think a bit more about it.

I remember at least one part of a production project I was involved with, where Object.freeze() had a considerable negative performance impact. We ended up removing the frozen objects in the end. So should you consider adding immutability using Object.freeze() I highly recommend to benchmark the results with and without Object.freeze().
References:

https://github.com/nodejs/node/issues/14789
http://benediktmeurer.de/2017/08/10/frozen-prototypes/

On the sanitization topic: As a long-time user of this library I often found the interface a bit confusing with respect to whether it required a semantically valid hostname or not. I would prefer a clear contract here and would tend towards dropping all semantic validation/sanitization and leaving this part to the user (or to a different module) such that tld.js would be exclusively concerned with validating and parsing semantically valid hostnames/domains.

@ctavan I agree with the use of Object.freeze, it's extremely slow and we can do otherwise.
I also agree on your second point regarding the API. What do you think @oncletom?

Yeah, Object.freeze used to be slow but I did not it applied to Object.prototype. It does not have to be used.
I think the trie implementation and an API design update are two separate decisions. They have different impacts and we do not have to try to solve them at once; and especially in one single pull request. I'd rather ship a good performance improvement and break the API later on, for good developer experience reasons.
@ctavan when you write

I often found the interface a bit confusing with respect to whether it required a semantically valid hostname or not.

This is very valuable and I would be happy if we could make emerge the pain points. Maybe it is just a documentation issue. Maybe it is an API design issue. And it is certainly a domain/concept issues to clear out.
If you are okay with it, let's discuss API and concepts improvements in a new issue while we are still around here :-)

if we're ok making a bold mode but being totally clear about it, it would make some sense to me that we only have functions working on hostnames, and provide a function to extract the hostname of any url (cleanHost)

A bold move okay but in another PR then, or in an another issue so as we can talk about the design before implementing it.

@oncletom Makes sense. Let's preserve the API for this PR and discuss a possible evolution in another issue. I will clean-up/comment the code a bit and address the concerns between now and tomorrow. Thanks again for the review and discussion!

@oncletom I just updated the PR:

Addressed PR comments
Cleaned-up a bit
Added extensive comments
Increased test coverage to 100%
Re-introduced the use of cleanHost in all functions (let's discuss how to proceed with this in another issue as you suggested).
Changed the expected result for the 2 tests discussed above (regarding spec compliance)

Let me know if there are other things to change before it's ready for merge.

@remusao great! Will have a look at it more in depth within the next hour or so — and might merge and release in the continuity of it. Great work. Thank you very much for taking the time to do it, to be open about the various thoughts and remarks 👍

@oncletom Thanks a lot! Happy to help and improve this library. Shall I create an issue to start the discussion around host cleaning?

@remusao and I am thankful for your help 🙂

Shall I create an issue to start the discussion around host cleaning?

Yes, please go ahead 👍

@remusao also, could you create a separate PR with benchmark as a devDependency and ./bin/benchmark as a script to run the benchmarks? Would be nice to compare further experimentations 🙂

@oncletom Thank you for merging! :D
I will create the issue soon + detail the different options we discussed already, as a starting point for the brainstorm!

could you create a separate PR with benchmark as a devDependency and ./bin/benchmark as a script to run the benchmarks? Would be nice to compare further experimentations 🙂

Will do! Talking about benchmark, I see you added the numbers to the README, but I think we need to correct them. On this benchmark, each op is actually processing 100 hostnames (clean and no unicode). So the corrected numbers should be (x100):
tldjs#tldExists x 326,500 ops/sec ±0.83% (92 runs sampled)
tldjs#getDomain x 203,000 ops/sec ±3.08% (84 runs sampled)
tldjs#getSubdomain x 203,700 ops/sec ±2.85% (85 runs sampled)
tldjs#getPublicSuffix x 219,500 ops/sec ±3.53% (83 runs sampled)

I will improve the benchmarks to be more representative of real cases when adding the benchmark binary.

Hey, @Nekimola thanks for sharing your concern. I actually thought of doing a major bump. The reasoning for not doing so was the behaviour change is a bug fix, as the past implementation was wrong. You were probably relying on a bug rather than on the expected behaviour.
What were you trying to do when expecting a null result?

Thanks for reply, I was checking if a public suffix of a domain doesn't exist.

I was checking if a public suffix of a domain doesn't exist

Well, if you are checking for a public suffix, according to the spec and the following tests (to be found here), you can see an unlisted TLD actually returns a public suffix.
// Unlisted TLD.
checkPublicSuffix('example', null);
checkPublicSuffix('example.example', 'example.example');
checkPublicSuffix('b.example.example', 'example.example');
checkPublicSuffix('a.b.example.example', 'example.example');

If you want to check if a TLD exists, I'd suggest you use #tldExists instead.
Let us know if more precisions are needed.

Yeah, I'm using tldExists method now, thanks again.


WIP: Migrate to GitLab CI

Based on #57, needs rebase.

@gicmo, @ondrejbudai I believe this needs the secrets.SCHUTZBOT_SSH_KEY to be configured for this repository on GitHub.

I think this is blocked by the failures in #57 for Fedora 34 and RHEL 8.
@ondrejbudai any chance you can finish the work on #57 soon or is it blocked by something else ?


Watch list loading error

Netflix add-on version
1.22.1
Operative systems used
Android
Kodi version used
Kodi 20 (Nexus)
Description of the bug
When I try to open the 'watch' list, I receive the message Error 'Typeerror: Actor: Name Property Must Not Be Empty'
Steps to reproduce the behavior
No response
Debug log - mandatory
Logs show nothing
Possible fix
No response
Additional context
No response
Screenshots

The same problem occurs when searching or loading the next page of the list of saved videos to watch on "My List"

im not able to reproduce
where is the debug log?

The problem is that there is nothing in the logs and the problem occurs, for example, when loading the next page of the movie list or when opening the watch list. I had the same problem when searching for movies. I don't know if it's a regional problem (I'm from Poland) or something else.

Same for me. Android and Windows.
Here's a snippet of the log:
2023-08-16 23:48:38.990 T:20872    info <general>: [plugin.video.netflix (1)] The debug logging is set as DISABLED 2023-08-16 23:48:55.499 T:20872   error <general>: EXCEPTION: Actor: name property must not be empty 2023-08-16 23:48:55.503 T:20872   error <general>: [plugin.video.netflix (1)] Traceback (most recent call last):                                                      File "C:\Users\achyl\AppData\Roaming\Kodi\addons\plugin.video.netflix\resources\lib\run_addon.py", line 29, in wrapper                                                        func(*args, **kwargs)                                                      File "C:\Users\achyl\AppData\Roaming\Kodi\addons\plugin.video.netflix\resources\lib\run_addon.py", line 265, in run                                                        success = route(pathitems)                                                      File "C:\Users\achyl\AppData\Roaming\Kodi\addons\plugin.video.netflix\resources\lib\run_addon.py", line 90, in lazy_login_wrapper                                                        return func(*args, **kwargs)                                                      File "C:\Users\achyl\AppData\Roaming\Kodi\addons\plugin.video.netflix\resources\lib\run_addon.py", line 130, in route                                                        _execute(nav_handler, pathitems[1:], G.REQUEST_PARAMS, root_handler)                                                      File "C:\Users\achyl\AppData\Roaming\Kodi\addons\plugin.video.netflix\resources\lib\run_addon.py", line 159, in _execute                                                        executor(pathitems=pathitems)                                                      File "C:\Users\achyl\AppData\Roaming\Kodi\addons\plugin.video.netflix\resources\lib\utils\logging.py", line 139, in timing_wrapper                                                        return func(*args, **kwargs)                                                      File "C:\Users\achyl\AppData\Roaming\Kodi\addons\plugin.video.netflix\resources\lib\navigation\directory_utils.py", line 28, in set_custom_viewmode                                                        override_content_type = func(*args, **kwargs)                                                      File "C:\Users\achyl\AppData\Roaming\Kodi\addons\plugin.video.netflix\resources\lib\navigation\directory.py", line 185, in video_list_sorted                                                        dir_items, extra_data = common.make_call('get_video_list_sorted', call_args)                                                      File "C:\Users\achyl\AppData\Roaming\Kodi\addons\plugin.video.netflix\resources\lib\utils\logging.py", line 139, in timing_wrapper                                                        return func(*args, **kwargs)                                                      File "C:\Users\achyl\AppData\Roaming\Kodi\addons\plugin.video.netflix\resources\lib\common\ipc.py", line 94, in make_call                                                        return make_http_call(endpoint, func_name, data)                                                      File "C:\Users\achyl\AppData\Roaming\Kodi\addons\plugin.video.netflix\resources\lib\common\ipc.py", line 115, in make_http_call                                                        _data = pickle.loads(received_data)                                                      File "C:\Users\achyl\AppData\Roaming\Kodi\addons\plugin.video.netflix\resources\lib\common\kodi_wrappers.py", line 72, in __setstate__                                                        set_video_info_tag(state['infolabels'], video_info)                                                      File "C:\Users\achyl\AppData\Roaming\Kodi\addons\plugin.video.netflix\resources\lib\common\kodi_wrappers.py", line 170, in set_video_info_tag                                                        video_info_tag.setCast([xbmc.Actor(name) for name in cast_names])                                                      File "C:\Users\achyl\AppData\Roaming\Kodi\addons\plugin.video.netflix\resources\lib\common\kodi_wrappers.py", line 170, in <listcomp>                                                        video_info_tag.setCast([xbmc.Actor(name) for name in cast_names])                                                    TypeError: Actor: name property must not be empty


Add heap.io on Frontend

This PR integrates Heap Analytics into the front-end by adding scripts to index HTML during deployment. In response to these changes, an environment variable HEAPIO_APP_ID is injected into CircleCI to store the heap app ID.

 2.0 Add heap.io on frontend

Is there a way that after the user has authenticated, heap.identify(userId) is called?

Is there a way that after the user has authenticated, heap.identify(userId) is called?

@pylipp I am also still unsure how to handle it because we do not have server-side rendering in the FE, but I think it would be possible to update it when a user logs in, which would be ideal.


Layerlist not updating, need to drag a layer to make it refresh!

🐛 Bug
Layerlist not updating, need to drag a layer in the list to make it refresh!
To Reproduce
Steps to reproduce the behavior:
Complicated as this is a special environment, inside a plugin. When I activate the widget for the first time, the generated layers appear correctly. After the deactivation of the widget (and the deletion of those layers), the reactivation is buggy as the layerlist doesn't display correctly all generated layers. I need to drag a layer up and down in the layerlist to make the layerlist update...
Expected behavior
The layerlist should always be up to date.
Environment
napari: 0.4.15
Platform: Linux-5.17.4-200.fc35.x86_64-x86_64-with-glibc2.34
System: Fedora Linux 35 (Workstation Edition)
Python: 3.10.4 (main, Mar 25 2022, 00:00:00) [GCC 11.2.1 20220127 (Red Hat 11.2.1-9)]
Qt: 5.15.2
PyQt5: 5.15.6
NumPy: 1.21.5
SciPy: 1.8.0
Dask: 2022.04.0
VisPy: 0.9.6
OpenGL:

GL version: 4.6.0 NVIDIA 510.47.03
MAX_TEXTURE_SIZE: 16384

Screens:

screen 1: resolution 1920x1200, scale 1.0

Additional context
The same context as #4421

A related bug, that might have triggered it is the following issue: despite having deleted 2 layers (the same that are buggy in the layerlist when reactivating the widget), they are still visible but "frozen" (they don't move around and aren't displayed in the layerlist)

EDIT: I have found the issue: instead of just doing disconnect(), I need to do disconnect(callback_I_added_in_the_widget). Is this an intended behavior?

I will close it as this

I have found the issue: instead of just doing disconnect(), I need to do disconnect(callback_I_added_in_the_widget). Is this an intended behavior?

seems to be the intended behavior :)


Prevent Right Click Hotkey

Could we get a patch that disables right-clicking hotkeys from removing the shortcut?

Hotkeys are controlled by both a skill ID, written by this first line of code...
66 89 56 10           - mov [esi+10],dx
8D 4E 28              - lea ecx,[esi+28]
89 4D EC              - mov [ebp-14],ecx

And a value (4byte, 1 or 0) determining if they're usable or not, written by this one...
89 86 FC000000        - mov [esi+000000FC],eax
89 65 EC              - mov [ebp-14],esp

Problem is this code is also involved...
B9 0C000000           - mov ecx,0000000C
F3 A5                 - repe movsd
8B 83 FC000000        - mov eax,[ebx+000000FC]

And I'm not sure what it's actually doing or why, and those two addresses alone don't stop the hotkey from being disabled (even though individually they do change the skill run and if it can be used or not) so there's something else going on.


[AirGap: 2/2] User should be able to sign tx inside voting aepp using Airgap

parent #327, parent #594, parent #376, parent #333

best option is if airgap changes code on their side:

more than one destination for wallet sync
implement callback

fallback:

copy and paste for user, but it's confusing because user is choosing "separate device" instead of same.

pinged @AndreasGassmann on dev chat to get a sense of the likelihood of code changes on airgap's side + a potential timeline for these changes.

@j28 copy pasting already works for the sync QR and transaction QR codes, it's a bit hidden as we mostly use this for testing and some integration purposes.
The Sync code can be copied through the overflow menu or simply clicking on the QR code, this also applies to transaction QR codes for both Vault and Wallet.

Pasting can be done through the Settings screen with "Paste from clipboard".

Let's continue the conversation with @AndreasGassmann how we can improve the user experience for this.

Hi @j28. Are you looking for a way to do that when using the base aepp as a native app or as a webapp?
If it is only required to do it in the native app, you could just register the "airgap-wallet://" scheme.

@AndreasGassmann I think we need this for both cases. From the beginning, we were thinking about registering itself as a handler of airgap-wallet protocol, until we have checked it support in mobile browsers.

So in this case we probably have to introduce a new "shareWalletRequest" type that also includes a callback URL.


[FEATURE]: Use searchable select or entity selector for operation selection in groups

Feature Motivation
current dropdown won't work as the list of options would eventually become to long
Solution Description
Use a searchable select with a cap on shown results or use an entity selector.
feature_acceptance_criteria

[ ] searchable select or entity selector added to create/edit group view
[ ] operation selection with them works

Further Information
No response
Related Issues
No response

When changing the select to a searchable select in the group view also provide an empty option to deselect the operation.

implemented in 5dc9dfffa1a1940ec6d088ee5bafec7a3e1795d6.


Can we use Pint with VSCode for instance highlights

Is there a way to highlight the code style issues while coding so I don't need to use terminal to update the code style.

We are not sure. Sorry!

@muath-ye it looks like there's already a few extension out there for VS Code. I'm currently using this extension without any issues so far.


missing postgres stats after upgrade to 6.15.0

Description
The Databases area is no longer showing any stats for postgres queries since upgrading from 6.14.0 to 6.15.0. We also upgraded to rails 6.1.1 from rails <IP_ADDRESS> at the same time.  Everything went silent after the Feb 3rd deploy that introduced both upgrades.

I also noticed that all database queries look different, the table names are missing, screenshot comparison of a typical query below:

vs

The changelog for 6.15.0 doesn't mention any activerecord configuration changes, hope I'm not missing a configuration step.
Expected Behavior
Postgres stats should be shown.
Your Environment
ruby 2.7.2
rails 6.1.1
newrelic_rpm 6.15.0

Hello, @gsar
Thank you for bringing this to our attention.
Looking at this, I suspect this may be related to the rails upgrade to 6.1.
We added rails 6.1 to our test suite and everything is working as expected there, however it's possible that some things maybe changed in 6.1 that are not covered by our current test cases. We will need to spend some time digging deeper on 6.1 and experiment with it to confirm our test cases are accurately reflecting how activerecord 6.1 is working.
I do have a couple questions about how you guys are using 6.1, to help guide us in our investigations.
Are you setting legacy_connection_handling to false?
Are you utilizing multiple connections?
Any other changes or new features of rails that you guys have started using?
Thank would be very helpful for us to track this down.
Apologies for the delayed response, thank you for your patience.

We have the same observation/problem after upgrading to Rails 6.1
Environment:
Rails version:	6.1.3
NewRelic gem: 6.1.5
Ruby version:	2.7.1
legacy_connection_handling is not set.
We use multiple connections trough - ActiveRecord::Base.connected_to for a few specific queries. The behavior is consistent throughout the whole application.
The queries are showing up as ActiveRecord instead of Postgres.
This wouldn't be a big deal in the long run, even though it is inconvenient at the moment.
The problem occurs when guessing the SQL statements.

With Rails 6.0

With Rails 6.1

Let me know if I can help somehow with additional information.

@tannalynn sorry for the delayed response.
We are not setting/changing legacy_connection_handling. We have a rather plain vanilla rails config with not much customization.  However, we use the sqreen and airbrake gems, which I know have resulted in conflicts in the past.
We have the standard config/puma.rb setup to have each worker instance start a new db connection with:
on_worker_boot do
  ActiveSupport.on_load(:active_record) do
    ActiveRecord::Base.establish_connection
  end
end

In config/application.rb we set config.load_defaults 6.0.
Everything else is pretty much out of the box.


Conmon multiple attach

- What I did
added support in conmon for multiple ttys attached at the same time
Closes: #1481
- How I did it
changed conmon to store an array of ttys instead of a single fd
- How to verify it
# podman  run --rm --name foo --interactive fedora cat

and in another terminal:
# podman attach foo

What is written in one terminal is visible to the other.
- Description for the changelog
conmon supports more than one attached tty at the same time.

LGTM other then the fmt errors.
@mheon @mrunalp @runcom PTAL

LGTM other then the fmt errors.
@mheon @mrunalp @runcom PTAL

@giuseppe Can you clean this one up.  We need it in Podman.

/test all

@rhatdan it was a flake, I've just forced a push so to retrigger the tests and they all pass now.

@mheon @mrunalp @runcom @wking @umohnani8 PTAL

LGTM

LGTM
Merging.

I actually wanted to make sure that we are handling leave stdin open correctly.


Update notification will not go away above Spotify module

I tried updating the module and still cannot get the “update available for MMM-Spotify module” above the module to go away. Am I doing something wrong?

Hi there. Well what was the outcome of the update command? maybe it failed? do you have local modifications made to the module (except for the account config file ofc)?

I figured it out it was just a merge issue, updated fine now.


Did it crash? I do not know

Windows Terminal version (or Windows build number)
Windows Terminal Preview 版本: 1.8.1092.0
Other Software
Win10 up to date
Steps to reproduce
This is the second time I tried to install. The first time I installed it through the store, when I opened a new tab during use, it would become inoperable. I had no choice but to uninstall. The second time was just now. After I downloaded the latest installation from GitHub, the picture appears. This is every time I start it.
Expected Behavior
No response
Actual Behavior
He was stuck for at least a few minutes

This is really strange! Can you share the contents of your settings file?
It is here: %LOCALAPPDATA%\Packages\Microsoft.WindowsTerminalPreview_8wekyb3d8bbwe\LocalState\settings.json

year ，thanks
settings.zip

Hmm. Okay, our theory that the color scheme is busted is definitely not right - that's all fine.
I wonder if the problem comes from
                "backgroundImage": "D:\\\u684c\u9762\u80cc\u666f\\mo\\01.jpg",

in your default profile. Could you try removing that, to see if it works fine without that image? It's possible that path might be upsetting the terminal for some reason.
This might be a weird variation on #6164

It is not caused by the background image. The problem exists even if I have not set the background image.
So it doesn't matter

Weird. Can you interact with the Terminal at all? Does typing in the Terminal move the cursor at all? Does clicking on the new tab button work? Do keybindings work, like ctrl+shift+t for new tab, or ctrl+shift+p to open the command palette?


Re-enable aerospike logging tests

Description
Re-enable the tests that we disabled in #882, but keep disabling the tests on SLES because logs are still broken there.
Related issue
b/239240173
How has this been tested?
Ran integration tests locally for SLES-15 and Debian-10. Confirmed that the tests are skpped on SLES and run on Debian.
Checklist:

Unit tests

[x] Unit tests do not apply.
[ ] Unit tests have been added/modified and passed for this PR.

Integration tests

[ ] Integration tests do not apply.
[x] Integration tests have been added/modified and passed for this PR.

Documentation

[x] This PR introduces no user visible changes.
[ ] This PR introduces user visible changes and the corresponding documentation change has been made.

Minor version bump

[x] This PR introduces no new features.
[ ] This PR introduces new features, and there is a separate PR to bump the minor version since the last release already.
[ ] This PR bumps the version.

SLES12 and Windows failures are unrelated; merging.


Add v7 to v8 e2e upgrade test

Description

closes: #4216
see the guidelines for commit messages. (view raw markdown for examples)

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Targeted PR against correct branch (see CONTRIBUTING.md).
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the module structure standards and Go style guide.
[ ] Wrote unit and integration tests.
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/).
[ ] Added relevant godoc comments.
[ ] Provide a commit message to be used for the changelog entry in the PR description for review.
[ ] Re-reviewed Files changed in the Github PR explorer.
[ ] Review Codecov Report in the comment section below once CI passes.

Codecov Report

Merging #4591 (4455333) into main (432db29) will increase coverage by 0.04%.
Report is 5 commits behind head on main.
The diff coverage is n/a.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #4591      +/-   ##
==========================================
+ Coverage   79.20%   79.25%   +0.04%
==========================================
  Files         188      188
  Lines       13063    13064       +1
==========================================
+ Hits        10347    10354       +7
+ Misses       2282     2276       -6
  Partials      434      434

see 2 files with indirect coverage changes

TestV6ToV7ChainUpgrade seems to be failing for an unknown reason, I'll create a follow up issue to investigate.

merged with v7 still failing, this will be fixed in a followup


[nuxtjs:i18n-resource-dynamic] Plugin error - Unable to get file name for unknown file "7dc266f9".

Environment
production
Reproduction
npm run build
Describe the bug
[nuxtjs:i18n-resource-dynamic] Plugin error - Unable to get file name for unknown file "7dc266f9".
ERROR  Plugin error - Unable to get file name for unknown file "7dc266f9".                                                                 7:52:53 PM
at error (/D:/Wamp/www/client/horse-management/node_modules/rollup/dist/es/shared/node-entry.js:2125:30)
at Object.FileEmitter.getFileName (/D:/Wamp/www/client/horse-management/node_modules/rollup/dist/es/shared/node-entry.js:24128:24)
at /D:/Wamp/www/client/horse-management/node_modules/@nuxtjs/i18n/dist/module.mjs:1216:50
at Array.reduce ()
at Object.generateBundle (/D:/Wamp/www/client/horse-management/node_modules/@nuxtjs/i18n/dist/module.mjs:1215:44)
at /D:/Wamp/www/client/horse-management/node_modules/rollup/dist/es/shared/node-entry.js:24551:40
at processTicksAndRejections (node:internal/process/task_queues:96:5)
Additional context
No response
Logs
No response

See https://github.com/nuxt-modules/i18n/issues/2029

Please provide a reproduction of the issue over Stackblitz using https://stackblitz.com/github/nuxt/starter/tree/v3-stackblitz so we can investigate it better. If the issue linked by Etienne is not related to yours, please let me know and I'll reopen the issue. 🙂

This is work for me :
npm install<EMAIL_ADDRESS>


QA - New wallet key line instructions for example section

What :computer:

First thing updated with this PR
Second thing updated with this PR
Third thing updated with this PR

Why :hand:

Reason why first thing was added to PR
Reason why second thing was added to PR
Reason why third thing was added to PR

Evidence :camera:
Include screenshots, screen recordings, or console output here demonstrating that your changes work as intended

Notes :memo:

Any notes/thoughts that the reviewers should know prior to reviewing the code?

@MexicanAce @idea404 can you please review the PR with the already implemented fix for hardcoded wallet?
Thank you in advance!


Column fit content width

This PR allow to set setFitContent method on column => auto width for column.

Can you please use TAB instead of spaces? Thanks.
Also, can you make it a bit simpler? Somethink like this:
public function setFitContent()
{
    $this->addAttributes(['class' => 'fit-content']);
}

Last thing - can you rename the class from 'fit-content' to 'datagrid-fit-content'? Thanks a lot! :)

Jasné, zítra to tam máš :)

Thanks!


Unit tests for SpillingGrouper

Any unit test doesn't exist for SpillingGrouper and should be added.

Still relevant.


Replace AWS Linux/aarch64 perf machines with OSUOSL ones

I need to request a new machine:

New machine operating system (e.g. linux/windows/macos/solaris/aix): Linux
New machine architecture (e.g. x64/aarch32/arm32/ppc64/ppc64le/sparc): aarch64
Provider (leave blank if it does not matter): OSUOSL / Linaro
Desired usage: Performance testing
Any unusual specification/setup required: Not virtualised
How many of them are required: 1/2

Please explain what this machine is needed for: Replace AWS RHEL7 system which is the only one available for the perf test jobs (Also that one is not currently used for anything else)

@smlambert If we do this the performance test results will not be comparable before and after. Will that cause any problems, or are we not looking at the results closely enough at the moment for it to be a problem?

We are only keeping 90 builds worth of perf history at the moment, so I do not think this will have a big impact, as we can build up history on the new set of machines in a short space of time, and as you correctly state, we've not moved into our intended 'phase 2' plan of deeper analytics for our perf test results yet.

Running the following on build-osuosl-ubuntu2204-aarch64-1 with a view to switching over:

sanity.perf
extended.perf

/etc/hosts is getting overwritten on reboot by an entry that doesn't match /etc/hostname. I've adjusted /etc/cloud/cloud.cfg to comment out update_etc_hosts so the hostname resolution doesn't get lost as it won't overwrite /etc/hosts:

extended.perf
sanity.perf

Pass - have enabled ci.role.perf on those machines and marked the AWS on offline.
There may be some argument for testing on Graviton machines, and RHEL on aarch64, but at the moment I'm happy to decommission


Rebase to ipfs v0.4.10

Changes Unknown when pulling e5e8af3a4e4b19ec66033ef8c3e7d801c00a7a3a on rebase into ** on master**.

Think I'm going to merge this. There's nothing to look at other than changes to import statements.


[ACS] The identifier serializer updates in the Shared package

Description
As a part of the new MicrosoftTeamsAppIdentifier introduction and making it GA we are making sure the CommunicationIdentifierSerializer is updated regarding the new identifier
Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

/azp run net - Azure.Communication.Identity - ci

Hi @AikoBB. Thank you for your interest in helping to improve the Azure SDK experience and for your contribution. We've noticed that there hasn't been recent engagement on this pull request. If this is still an active work stream, please let us know by pushing some changes or leaving a comment. Otherwise, we'll close this out in 7 days.

Work in-progress


Not returning RD links

Hi,
I've got an issue with torrentio and real debrid.  It was all working fine but now it's not returning any RD links at all.  I've uninstalled / reinstalled,  checked my RD account but nothing is working.
Other apps & addons are working,  just not the torrentio addon

Hey, it's working on my side. Checking the logs it seems there was temporary problem with RD (timeouts).

Yeah it's working now.  Weird.  Thank you for the swift response

Is there anyway to force the addon to re-scrape? For some reason the movie I tried to watch this morning still shows no links. It appears the addon is not trying to search for that one again, other movies I try find plenty of links... searching for Mulan 2020 using windows app

@gonzalo-hvega Responses from addons are cached on device for configured time by the addon. For Torrentio it's 4hrs. Not sure if there's a way to reset the cached responses. You could open a link on one device, then it will be added to Continue Watching list and then it should be synced to other device and you should be able to start it on another device.


AWS Sender ID

in array 'SenderID' => $message['from'] replace to 'MessageAttributes' => [
'AWS.SNS.SMS.SenderID' => [
'DataType' => 'String',
'StringValue' => $message['from']
]
]

You're going to have to help me out here. I have no context for this.
Is this an error message you're seeing? Or is it a change that needs applying - if so, it should really be a pull request

Closing as I have had no response and this issue doesn't give me anything to go on.


Difference between Microsoft.Extension.Caching.* packages

What advantages does it have over using e.g. Microsoft.Extensions.Caching.StackExchangeRedis and official IDistributedCache and IMemoryCache abstractions?
Some of the disadvantages I see is less control, e.g. forced serialization library https://github.com/MichaCo/CacheManager/issues/278

IDistributedCache doesn't have any features other than saving a byte array as far as I know.
That's pretty much doing nothing and cannot really be compared to CacheManager.
That being said, if you don't need any of the CacheManager specific features and just want to quickly throw your byte arrays into Redis then just use those simple abstractions!
What do you mean with forced serialization? CacheManager has many different serialization implementations which should work for most cases so that you as the user don't have to care about that. That being said, you always have the option to not use a build in serializer and also just cache a byte array and care about serialization somewhere yourself.


Added highlighted color to selected tiles and placed tiles; refilled …

…slots when tiles are being placed
What does this implement/fix? Explain your changes.
implement refilling empty slots after clicking place button
Does this reference any currently open issues?
…
Types of changes
What types of changes does your code introduce?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. This is simply a reminder of what we are going to look for before merging your code.

[ ] I have linted my code and I have run tests locally that pass
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation

Any other comments?
…

This function aims for frontend display instead of replacing backend data generation.


Suggest a project (Acctual)

Project
Acctual
Description
Acctual is accounts payable (AP) and accounts receivable (AR) automation for crypto and fiat. Synced to your accounting system.
Please provide the project scope.
Core SDK Integration
Select a primary category of your project.
Payments
Select secondary categories of your project.
Accounting, Payments
Project logo

Value proposition
Acctual is the first true accounts payable and receivables platform for crypto. We make it easy for businesses to pay bills through their Safe. All bills and invoices are synced to the general ledger (Quickbooks, Xero, etc.) and contacts are mapped to their wallet addresses. Acctual is currently developing a fiat integration with plans to have an all encompassing platform to manage your financial operations.
Contact
<EMAIL_ADDRESS>Project URL
https://acctual.com/
Github / Dev Docs
No response
Twitter
@acctualteam
Primary integration
Core SDK
Packages
Safe Core SDK, Transaction Service API
Modules / Guards
We have only used the Safe protocol kit sdk and transaction service API.
Smart Accounts
The Acctual accounts payable solution takes advantage of the multi-signature scheme and transaction batching. We allow companies to import their bills, leverage the Safe signatures mechanism for approval workflows, and finally execute a number of bill payments in a single batch transaction.
Networks
Ethereum, Polygon, Optimism, Arbitrum
Can you create a Safe via the interface?
No
Can you import an existing Safe via the interface?
Yes

Thanks @IssaMohamudHassan, this will be added to our next db sync

Thanks @IssaMohamudHassan, this will be added to our next db sync

Hey @debbchiasafe, just wanted to check in here!

Hey @IssaMohamudHassan - Acctual is now listed on our Ecosystem page:


add favorite page route(/fav)

@boy-warakorn add back the .env.example then you are good to merge
CC @Pittawat2542


SqlBatch: Add missing overrides and tests

fixes https://github.com/dotnet/SqlClient/issues/2222
Add missing overrides for SqlConnection CanCreateBatch and CreateDbBatch() and tests
Add test for SqlBatchCommand CanCreateParameter and CreateParameter() wrapped in #ifdef
Adding a net8 build is out of scope for this PR. That needs approval at very least from the MS side. When it is added the net8 features and tests related to the sql batch feature should light up automatically.
@David-Engel we'll want to get this in before the next preview.
/cc @mgravell

@Wraith2 I think you also want to override the related methods missing on SqlClientFactory (here are the original specs):
public virtual DbBatch CreateBatch() => throw new NotSupportedException();
public virtual DbBatchCommand CreateBatchCommand() => throw new NotSupportedException();

// Covers both CreateBatch and CreateBatchCommand
public virtual bool CanCreateBatch => false;

Those are in a separate file:
https://github.com/dotnet/SqlClient/blob/d30d9d107eca0f11b735a550aa187e57167858c9/src/Microsoft.Data.SqlClient/netcore/src/Microsoft/Data/SqlClient/SqlClientFactory.NetCoreApp.cs#L9-L18

They should light up as soon as the team add a net8 build.

Do these APIs need to be in the ref assemblies, too?

They're overrides so I wouldn't expect to see them in the ref assembly since they're defined on the base class and we didn't merge our own copies of the base classes.


Add flake8 linter

Apart from the first 3 files, all changes are just making me .. er.. the linter happy
Can look at second-to-last 9a63328 commit for the kind of things it caught.
Could also add e.g. pylint but that one is much more picky.

Can we also somehow run it from inside PyCharm?

Guess so: https://gist.github.com/tossmilestone/23139d870841a3d5cba2aea28da1a895


Deploy to GH Pages using Github Actions on merge to master

Added simple deploy of root folder on master branch to GH Pages.


[UI] order service names

It would make sense to alphabetically order the list of services in the 'Service Name' dropdown.

Could you elaborate more on this? Since the dropdown allows you to search I
am curious about the value being added by sorting the service names. This
could be easy to do if we sort in frontend tho.
José Carlos Chávez
Den tor. 4. okt. 2018 kl. 15:46 skrev Jorg Heymans<EMAIL_ADDRESS>
:

It would make sense to alphabetically order the list of services in the
'Service Name' dropdown.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/openzipkin/zipkin/issues/2204, or mute the thread
https://github.com/notifications/unsubscribe-auth/AC7sArpFdGS6EJzZbcK2DnyUeYrL9CvWks5uhhEdgaJpZM4XIJJ0
.

It is quite easy to do in the front end. I think it makes sense to sort it
alphabetically.
On Thu 4 Oct, 2018, 21:07 José Carlos Chávez<EMAIL_ADDRESS>wrote:

Could you elaborate more on this? Since the dropdown allows you to search I
am curious about the value being added by sorting the service names. This
could be easy to do if we sort in frontend tho.
José Carlos Chávez
Den tor. 4. okt. 2018 kl. 15:46 skrev Jorg Heymans <
<EMAIL_ADDRESS>
:

It would make sense to alphabetically order the list of services in the
'Service Name' dropdown.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/openzipkin/zipkin/issues/2204, or mute the thread
<
https://github.com/notifications/unsubscribe-auth/AC7sArpFdGS6EJzZbcK2DnyUeYrL9CvWks5uhhEdgaJpZM4XIJJ0
.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/openzipkin/zipkin/issues/2204#issuecomment-427032514,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEgSsE77ao6YlMHoYJJCKVhLkpb-Sefgks5uhhY8gaJpZM4XIJJ0
.

Imagine service names like serviceA_dev, serviceA_test, serviceA_uat,
having them in predictable order instead of all over the place makes for a
better user experience.
Op do 4 okt. 2018 18:10 schreef Raja Sundaram<EMAIL_ADDRESS>
It is quite easy to do in the front end. I think it makes sense to sort it
alphabetically.
On Thu 4 Oct, 2018, 21:07 José Carlos Chávez<EMAIL_ADDRESS>wrote:

Could you elaborate more on this? Since the dropdown allows you to
search I
am curious about the value being added by sorting the service names. This
could be easy to do if we sort in frontend tho.
José Carlos Chávez
Den tor. 4. okt. 2018 kl. 15:46 skrev Jorg Heymans <
<EMAIL_ADDRESS>
:

It would make sense to alphabetically order the list of services in the
'Service Name' dropdown.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/openzipkin/zipkin/issues/2204, or mute the thread
<

https://github.com/notifications/unsubscribe-auth/AC7sArpFdGS6EJzZbcK2DnyUeYrL9CvWks5uhhEdgaJpZM4XIJJ0

.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<https://github.com/openzipkin/zipkin/issues/2204#issuecomment-427032514
,
or mute the thread
<
https://github.com/notifications/unsubscribe-auth/AEgSsE77ao6YlMHoYJJCKVhLkpb-Sefgks5uhhY8gaJpZM4XIJJ0
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/openzipkin/zipkin/issues/2204#issuecomment-427077289,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAL1AM_HgTycUwoUWJPnowIrfh1FbgKfks5uhjLjgaJpZM4XIJJ0
.

The api doesn't promise any order, but that doesn't mean we can't
lexicographically sort by default in the UI especially as the data is
small. We can also sort in our own api even if that doesn't imply
alternate backends will.
iotw sounds fine.


Screenshots

NuGet package version
Microsoft.UI.Xaml 2.8.0
Windows app type

[X] UWP
[ ] Win32

Device form factor
No response
Windows version
No response
Additional context
No response

It's just dark green? "Ugly" is a very personal opinion. Got a suggestion for something better?

yes, this is of course a personal opinion, but I have yet to see this color palette in any official Windows app for a successful message.
GitHub green color would be better as an example.

I think that's because of the transparency. Using 100% green icon on a 100% green background is a no-go: It won't be even visible. And the color(s) is/are too bright to match the light/dark theme.

The green has a yellow bias, I brought up this specific colour back when the control first appeared.
https://github.com/microsoft/microsoft-ui-xaml/issues/913#issuecomment-736479707