Introduce Follower Replication

Relate to https://github.com/tikv/raft-rs/pull/249
Signed-off-by: Fullstop000<EMAIL_ADDRESS>

can you paste your previous benchmark results here, so others can see the benefit intuitively.

@siddontang I'll make a more concrete benchmark for this. Some previous results are hard to explain why. But I'm ok to post them here :).

@abbccdda  A node in a group will send its group_id in msg and the receiver will update the sender's group info based on it. The leader only picks a delegate when the group info is enough. In a rolling-upgrade/downgrade situation, This can introduce several cases:
Upgrade

If only the leader uses follower replication, it can only know the group info until followers finish upgrading and send their group_id so that the leader uses origin log replication at this point
If only a follower or part of them use follower replication, the leader will just ignore the group_id in the msg so no delegate will be picked and origin log replication keeps processing

Downgrade

If only the leader use origin log replication, the case is just like common raft cluster and nothing special happens (pick a delegate, broadcast appends)
If a follower is downgraded and stop informing the leader its group_id, the leader will remove it from the group system and send entries directly to it

By such a design, the compatibility can be guaranteed when nodes in the cluster use either origin log replication or follower replication
It seems this feature description is missing in the RFC. I'll add it soon.

@Fullstop000 Thanks for the reply. It would be good to add such details to RFC for sure :)


[BUG] Tab color incorrect when hovering

after updating catppuccin pugin to v3.8.0, the color comes to incorrect that it displays red but it was light blue before
plugin settings:
{
  "catppuccin.accentColor": "blue",
  "catppuccin.boldKeywords": false,
  "catppuccin.bracketMode": "neovim",
  "catppuccin.colorOverrides": {
    "mocha": {
      "base": "#000000",
      "mantle": "#000000",
      "crust": "#000000"
    }
  },
  "catppuccin.extraBordersEnabled": false,
  "catppuccin.italicKeywords": false,
  "catppuccin.italicComments": false,
}

screenshot:

False alarm? 😄

@nekowinston there is another similar situation that tab.unfocusedInactiveBackground also meets the problem
screenshot:

And I think it also due to the shade()

Yeah, I confirmed the bug yesterday, but haven't managed to finish the PR yet, I'll probably do so in a few hours.

thank you for the hardworking 😊


You cant use the combination feature /g/ with files that include other files because it breaks the path.
This was always the case.
You must load the css files that include the fonts and gif as separate files. Dont combine them.

Not exactly the same but definitely related. I had the same issue (404s) when using .NET MVC with bundling. Because the relative paths are set with './...', it uses the current directory and searches from there... As with bundling, the current directory changes resulting is a 404. Because of that, I had to change the relative paths to '/content/slick/...' (without the . so starting from the root directory) in the 'theme.css' file.
Keep in mind. '/content' is a path used in MVC in combination with NuGet so you might have to take a different route.


Make Signup form translateable

Currently the Signup form is defined in the folder src/signup-form/de/. For other websites, the de can be exchanged with for example a us, at or it.

@janosh
The new signupform can be translated and is shown for each of the languages.
The submit part of the implementation is still missing.
Current issues, are when the signup form is loaded directly without being redirected from another URL, the site won't load.
The reason is, that the microcopy signup store is not filled yet.
This leads to an undefined country entry:
import { microcopy } from '$lib/stores'
import { get } from 'svelte/store'
var country = get(microcopy).country
console.log('country', country)

Which leads to error messages when the form is loaded. We need to wait until the signup store is filled until, we can access the country and continue importing the  yml files for the form


Lesson 1.1 explains the wrong ARM instruction lrd vs. adr

Under the Booting the Kernel header, the ARM instruction lrd is explained when the adr instruction is used instead. This one was confusing for me as someone familiar with asm but not with ARM.
adr (PC-relative) seems to be the correct instruction.
It might be nice to link every instruction to the proper URL as well. I'm happy to fork this and fix it!

Thanks for the PR @andrewstockwell it is merged now, closing this issue.


[test] stdlib/StringIndex: Spin off O(n^4)  substring replacement test into a standalone long test

Also trim down its input a bit so that this doesn’t take 20 minutes.
rdar://89482809

@swift-ci test


change kernel filename to kernel8

I also add an explanation to the name thing.
btw, what does it really mean boot into 32-bit mode. The cpu disable half of its address wires to act like a 32-bit cpu?

Looks good, thanks @evopen !
32-bit mode means that the CPU uses aarch32 instruction set. In this mode you use r0-r15 32-bit registers instead of x0-x30. You can switch to this mode from EL3. Refer to "Part E The AArch32 Application Level Architecture" of the AArhch64-ReferenceManual for more information.


Project down

Hey, why the project is down from showcase? If there is an error, please let me know, because I am gonna build my project with the help of your project.

There is no issue with the theme, you could continue using it for your project.
The Vercel account under which the project demo was hosted is having some issues, that's why it showing that error.  Will be rectified soon.


Suggestion: NMZ Utilities, Autoretaliate

Add a checkbox that warns when the player is in NMZ and auto-retaliate is turned off.
I'd make the PR myself but I can't figure out how to contribute to your repo lol

I think you'll know if you don't have auto retaliate on pretty quickly. This seems silly.


CUDA 10

Test

TODO: get new CUDA capable machine that does not require me to reboot twice everytime I want to use CUDA

My other computer it the cloud! (terraform and ec2)
Steps: create manually:

an EBS with a tag "mountPoint:home"
a security group with a name (I am using "SSHMaison").
an ssh keyfile (MyKeyName)
have aws configured on you host for your account (aws configure or a key file)

I also have a dns on route 53 (that holds myzone.com), but it is not mandatory, you will connect via the IP; you simply need to comment the blocks data "aws_route53_zone" "myzone" , and resource "aws_route53_record" "dev" in the tffiles (see below).
Get those files and create aterraform.tfvars that contains:
zoneName = "myzone.com"
keyName = "MyKeyName"

Run terraform apply -var 'instance_type=t3.xlarge'.
Thenmosh<EMAIL_ADDRESS>and deploy your code in the home (install go, your vimrc, .tmux.conf, go get etc...).
If you need a GPU, change the AMI in the file topology.tf:
resource "aws_instance" "dev" {
- ami              = data.aws_ami.ubuntu.id
+ ami              = var.deepLearningAMI
  instance_type     = var.instance_type
  key_name          = var.keyName
  availability_zone = "us-east-1d"
  user_data         = file("postInstall.sh")

  tags = {
    Name = "Dev"
  }
}

then terraform apply -var 'instance_type=g3s.xlarge' does the magic:

unmount the EBS
destroy the host
create a new host with a GPU and the AMI that have cuda installed
mount the home

mosh<EMAIL_ADDRESS>et voilà you have your files, a GPU and cuda installed!

Test

Test

Closed due to lack of activity, and has also been superseded by other issue reports, such as #297


Break up paragraphs in IPFS topic

[X] I followed the contributing guidelines: https://github.com/github/explore/blob/master/CONTRIBUTING.md

I am:

[X] Suggesting edits to an existing topic or collection
[ ] Curating a new topic or collection
[ ] Unaffiliated with the project (not self-promoting as e.g. a maintainer, creator, contractor or employee)

EDITING AN EXISTING TOPIC OR COLLECTION
I'm suggesting these edits to an existing topic or collection:

[ ] Image (and my file is *.png, square, dimensions 288x288)
[X] Content (and my changes are in index.md)

Please explain why these changes are necessary:
Makes things look better. Preview:

Think you I hope this will not make it harder for me as it is<EMAIL_ADDRESS>


Added Link to Labeling Service

Added link to LightTag.io: service for in house labeled data software

Maybe we should separate out API stacks and annotation tools? Add other annotation tools that you consider to be great?

Hey @talolard just following up, just following up on this - are you keen on taking this to finish?

Closing this without merging due to lack of response in last 6 days. Happy to look into this again at a later time.


High concurrency writes leads to data loss

testing script
$ cat winston.js
var winston = require('winston');

var logger = new (winston.Logger)({
        transports: [
          new (winston.transports.Console)(),
          new (winston.transports.File)({ filename: 'win.log' })
        ]
      });

var http=require('http');

var server = http.createServer(function(request, response){
    response.writeHead(200, {'Content-Type': 'text/plain'});
    var rd = Math.random() * 50;
    logger.info("hello " + rd);
    response.write('hello ');
    if (Math.floor(rd) == 30){
        process.exit(1);
    }
    response.end();
}).listen(4444);

ab testing command
ab -c 10 -n 100 localhost:4444/

console log result all request
$ node winston.js
info: hello 40.635715366806835
info: hello 47.55378740373999
info: hello 3.97098318208009
info: hello 21.402712271083146
info: hello 13.45938507001847
info: hello 34.73868641303852
info: hello 35.19469986204058
info: hello 17.50683527207002
info: hello 46.6102683218196
info: hello 16.208414710126817
info: hello 18.39921014616266
info: hello 23.423498892225325
info: hello 44.21681684907526
info: hello 35.54235898191109
info: hello 9.904313215520233
info: hello 27.047289244364947
info: hello 32.58601982379332
info: hello 36.48257440654561
info: hello 42.66765915090218
info: hello 9.47293157223612
info: hello 38.31890656147152
info: hello 40.63100240891799
info: hello 26.544194167945534
info: hello 17.846915975678712
info: hello 43.33941290387884
info: hello 26.012209930922836
info: hello 30.679881805554032

But file log only has 3 result:
$ cat win.log
{"level":"info","message":"hello 40.635715366806835","timestamp":"2013-07-30T03:03:43.441Z"}
{"level":"info","message":"hello 47.55378740373999","timestamp":"2013-07-30T03:03:43.446Z"}
{"level":"info","message":"hello 3.97098318208009","timestamp":"2013-07-30T03:03:43.446Z"}

I've checked the same with node 0.12 and it seems working, at least I see all 100 records in log file.
Looks like this ticket affected stream flushes: https://github.com/joyent/node/issues/6118

This seems like a pretty significant issue and one that is till present in 1.0.0.

The best we could hope for is configuration to enable flushing the streams on uncaught exception or exit since the library probably shouldn't do this out of the box.
Here's how to setup hooks to flush the streams manually on exit uncaught exception:
http://stackoverflow.com/questions/18771707/how-to-flush-winston-logs
https://github.com/flatiron/winston/blob/master/lib/winston/logger.js#L594-L637

Are there plans to have Winston flush the logs by itself on uncaught exceptions?

Was there any resolution to this issue?

The resolution to this issue is going to be<EMAIL_ADDRESS>This work is actively underway using native streams and will be much more performant. see https://github.com/winstonjs/winston/compare/v2.x.x

Prospective Winston user, and came across this issue early in my research which gave me pause. Is this issue now resolved in the latest master branch, given it's 2.2.0?

You are lucky to learn about this before having jumped in I guess..
On Wed, Mar 30, 2016 at 7:17 AM, Chris Horsley<EMAIL_ADDRESS>wrote:

Prospective Winston user, and came across this issue early in my research
which gave me pause. Is this issue now resolved in the latest master
branch, given it's 2.2.0?
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/winstonjs/winston/issues/288#issuecomment-203238768

This was finally resolved in 3.0.0

@indexzero how sure are you about that?
is there any better or at-par node.js logging library by now? this issue is like four years old I think.

Very. We have appropriate test coverage ensuring this. See: https://github.com/winstonjs/winston/blob/master/test/transports/00-file-stress.test.js.
Locking issue due to age.


Builder::triggerFakerOnAttributes is calling random functions

The line
$attribute = is_callable($attribute) ? $attribute() : $attribute;

in specified method can call random functions and throw exceptions or even do pretty nasty things,
eg. if I make and use something like provider for random font-awesome icon name, sometimes I get
[ErrorException]
  Missing argument 1 for unlink(), called in /vendor/laracasts/testdummy/src/Laracasts/TestDummy/Builder.php on line 192/200 and defined

I suggest, changing the line to:
$attribute = $attribute instanceof \Closure ? $attribute() : $attribute;

("all Faker properties are wrapped in closures."... so check for them and not for all callables)

PR?


virt-api: dialer should wrap ipv6 address with square brackets to portfoward ipv6 VM

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #9850
Special notes for your reviewer:
Release note:

virt-api: portfowrad can handle IPv6 VM

/sig network

Thank you for the PR @lxs137.
Could you please update the commit message to include more information about the problem this PR solves?

@orelmisan I have updated the commit message :-)

@lxs137 In the network e2e tests there is a comment about port forwarding with IPv6 not being currently supported, with a link to a tracking issue.
I've asked @AlonaKaplan (the network maintainer) to take a look at this PR.

/retest

/ok-to-test

/retest


widgets: Make the todo and poll widgets look consistent / similar.

Earlier the todo and poll widgets' UI differed a lot, especially in the
font size and weight, and the color and shape of the checkbox and vote
count box.
Now the font weights and sizes are consistent in both the widgets and
the todo widget's checkboxes and poll widget's vote count boxes are
styled similarly.
This is the rebased and cleaned version of #21006, and is visually the
same. (The commit history has been tidied and unnecessary naming
changes have been discarded, so we only have the minimal necessary
CSS only changes.)
PR #22551 also makes some related UI (though mostly functional) changes.
Fixes: #20283.
Screenshots and screen captures:

GIFs can be seen in the original PR (#21006)
Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[x] Calls out remaining decisions and concerns.
[x] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Responsiveness and internationalization.
[ ] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
[x] Corner cases, error conditions, and easily imagined bugs.

@YashRE42 , since you reviewed #21006 earlier, would you be up for doing a round of review here?

@YashRE42 any feedback on this one? Thanks!

Just commenting to bump this up since @YashRE42 seems to have been inactive for a while now

@N-Shar-ma thanks for working on this! Left some comments!

Noticed that visual indicators for hover state were missing in the todo widget for dark mode, so added those:
https://user-images.githubusercontent.com/68962290/205689850-bcc0cf0d-2032-4ce3-b473-681a2742dc65.mp4

@amanagr Any more feedback from your side? I'll re-review once the PR is ready from your perspective. Thanks!

@N-Shar-ma thanks for the update! Can you split the changes to poll widget and todo widget into independent commits. Also, it would be helpful if you can split css changes that are independent of other changes like conversion of pt to px values for the poll widget.

@amanagr I've separated a commit for conversion of pt to px. Since the issue involves making both the widgets looks similar, and many of the CSS changes apply to both widgets, I'm not sure if we should separate these changes into 2 commits when everything is so closely related... I hope that makes sense?

@N-Shar-ma LGTM! Would it make sense to make the size of checkbox in todo and numbers in poll widget the same?

Would it make sense to make the size of checkbox in todo and numbers in poll widget the same?

I think we should keep the checkboxes square, but it would be very reasonable to try making the poll boxes square rather than rectangular too. I'm not sure how to easily test polls with >9 votes for an option, but we should make sure multi-digit vote totals continue to work fine.

@alya, I tweaked the height and width for the checkboxes just a lil so that single and double digit votes fit comfortably in a square shape, while checkboxes with >=3 digits votes retain their initial height but widen to the right, to allow the vote count to fit horizontally:

Does this look okay?

Looks great to me!

@timabbott This is a major refresh on the appearance of polls and todo lists that I'd love to get integrated. It's been reviewed by @amanagr and me.

I posted a couple comments; otherwise the code looks reasonable to me. I'll do manual testing and hopefully integrate once the above are resolved. Thanks @N-Shar-ma!

@timabbott Have addressed your comments, and also removed duplication in the styling of .add-task and .poll-option buttons across the 2 widgets, which I'd somehow missed till now.

Merged, thanks @N-Shar-ma! I think we're going to want to do a proper redesign pass on these widgets, but that doesn't mean this isn't a significant visual improvement here :).


Not able to use Button Panel in the AMG(AWS managed Grafana) dashboard

Gets below error when uses Button Panel in the dashboard when used in AMG(AWS managed Grafana)
{message: "Internal server error", messageld: "plugin unavailable", statusCode: 500, traceID: "'} message: "Internal server error" messageld: "plugin.unavailable" statusCode: 500
Please find the attachment:

Need help on this concern

same issue


Project doesn't install anything

works here as expected.
You may give a bit more information to find out why it didn't work for you.
For example, when you try to build a distribution package you may involve the INSTALL_DIR make flag to overwrite the default.
Also the DESTDIR env variable is respected by the makefile.

It is incompatible with package build as it is.
        ifeq ($(user),root)
        INSTALL_DIR ?= /usr/lib/lv2
        else
        INSTALL_DIR ?= ~/.lv2
        endif

Package builders now work as user, not as root, so normally project is expected to be installed into $DESTDIR$PREFIX.
/usr should be $PREFIX.

@yurivict
with lv2 plugins we've a game changer, as those could be easily installed on ~./lv2 as user.  Means users could just build and install as a user and expect  it to work.
So, as packager you may overwrite the  INSTALL_DIR parameter which could be done in your build script.


Gfr test fix sha256 regex

Status
Ready/In Progress/In Hold (Reason for hold)
Related Content Pull Request
Related PR: link to the PR at demisto/content
Related Issues
Related: link to the issue
Description
A few sentences describing the overall goals of the pull request's commits.

Docker Image Ready - Dev
Docker automatic build at CircleCI has deployed your docker image: devdemisto/btfl-soup:<IP_ADDRESS>70
It is available now on docker hub at: https://hub.docker.com/r/devdemisto/btfl-soup/tags
Get started by pulling the image:
docker pull devdemisto/btfl-soup:<IP_ADDRESS>70

Docker Metadata

Image Size: 26.35 MB

Image ID: sha256:6173c42e66c0c6a2654d6f11222551e3d40817fb68865ae1473e54e4449470be

Created: 2021-08-08T10:10:31.28286311Z

Arch: linux/amd64

Command: ["python3"]

Environment:

PATH=/usr/local/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
LANG=C.UTF-8
GPG_KEY=E3FF2839C048B25C084DEBE9B26995E310250568
PYTHON_VERSION=3.9.5
PYTHON_PIP_VERSION=21.1.2
PYTHON_GET_PIP_URL=https://github.com/pypa/get-pip/raw/936e08ce004d0b2fae8952c50f7ccce1bc578ce5/public/get-pip.py
PYTHON_GET_PIP_SHA256=8890955d56a8262348470a76dc432825f61a84a54e2985a86cd520f656a6e220
DOCKER_IMAGE=devdemisto/btfl-soup:<IP_ADDRESS>70

Labels:

org.opencontainers.image.authors:Demisto<EMAIL_ADDRESS>org.opencontainers.image.revision:36d61fdb667232de8ce40db9a4b2af6ecd18dc78
org.opencontainers.image.version:<IP_ADDRESS>70


Why doesn't EmulatorTrace use the ledger rules?

Describe the feature you'd like
Sorry for my ignorance, but it's not clear to me why EmulatorTrace reimplements the ledger rather than using the real ledger code from https://github.com/input-output-hk/cardano-ledger.
Describe alternatives you've considered
No response

As I understand, it's historically. And sometimes, to my taste, it might be too tedious to work directly with cardano-api types.
The plan is to switch fully to cardano-ledger (and cardano-api).

To continue's @ak3n 's answer, it's because the Emulator (with the ledger rules) was implemented before cardano-ledger integrated Alonzo era. We have to do that in order to be able write the plutus-apps framework for writing Plutus contracts. Now that the cardano-ledger implemented the rules, we should definitely switch to using it instead of our own.
By the way, our Emulator is using the real code in addition to our reimplementation. So, basically, the emulator does use the real ledger validation rules.

I meant that we're using the cardano-ledger rules for validating transactions that are submitted in the emulator. But you're right, for constructing the ScriptContext, we're not. We plan on doing it soon.

Fixed in #642.


Script hanging at PlexMetaManager splash.

Running the script on unRaid via this docker config, but my script just hangs at the initial splash: https://pastebin.com/UmQYSjUH
Here's my config: https://pastebin.com/xZHX95Eq
Running unRaid 6.8.3, PMM 1.4.0. Docker.

Turns out I'm just dumb. I scheduled it to run every 12 hours and it was doing what I told it. Added --run to post arguments, which fixed it. My bad.


Mojo -helloworld.ml

Pull Request Type
What kind of change does this PR introduce?
[ ] Solved an issue/task
added hello_mojo.ml

Merged

This is an automated message from Fork, Commit, Merge [BOT].

Thank you for your contribution! Your pull request has been merged. The files have been reset for the next contributor.
What's next?
If you're looking for more ways to contribute, I invite you to check out our forkcommitmerge.io website repository. The repository contain real issues that you can help to resolve or create some of your own if you see any bugs or want to add new features to the site, etc. You can also check out the Influences section in the README to find more projects similar to this one.
Also please leave a star to this project if you feel it helped you, i would really appreciate it.
I look forward to seeing your contributions!


Major enhancements to tests

Now you can test configuration methods and getter methods independently.
Configuration methods can't be mocked. Main reason is because I think it's interesting to test how the code interacts with the internals of a real device.
Getter methods can be run against a live device or against a mocked device. Just set the mock flag in the driver test to whatever you want to do.
To mock a device call is as simple as just creating a file inside mock_data where filename is the call you are mocking and the content is whatever the real device would respond.
Juniper can be equally mocked and it doesn't matter if you are using RPC calls or the junos_views functionality in your code. You just have to mock the RPC call you are making and that will work for both raw RPC calls or for the junos_views wrapper.

Great idea ! I think this approach will speed up work on getters part :)


i3-ratiosplit: init at 0.1.0

Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 118094 1

1 package built:

i3-ratiosplit

Result of nixpkgs-review pr 118094 at 18afe4ca run on x86_64-linux 1

1 package failed to build:

i3-ratiosplit

Note that build failures may predate this PR, and could be nondeterministic or hardware dependent.
Please exercise your independent judgement.

Not sure why I didn't see that failure on my side. In any event, I've updated, thanks @SuperSandro2000 !


vscode: eamodio.gitlens can't load icons

Describe the bug
In VSCode (and Codium), the extension eamodio.gitlens is unable to load icons, and uses generic rectangles instead.
Opening VSCode/Codium with --verbose shows:
vscode-file: Refused to load resource /nix/store/wls3y80nfld9hln166zw0b1rhssj5wyk-vscode-extension-eamodio-gitlens-2022.12.604/share/vscode/extensions/eamodio.gitlens/dist/glicons.woff2 from vscode-file: protocol (original URL: vscode-file://vscode-app/nix/store/wls3y80nfld9hln166zw0b1rhssj5wyk-vscode-extension-eamodio-gitlens-2022.12.604/share/vscode/extensions/eamodio.gitlens/dist/glicons.woff2)

Steps To Reproduce
Steps to reproduce the behavior:

nix-shell -p "vscode-with-extensions.override {vscodeExtensions = [vscode-extensions.eamodio.gitlens];}"
Open code.
You can see that what are supposed to be GitLens icons are rectangles instead.

Expected behavior
GitLens should load the icons correctly.
Screenshots
EDIT: Instead of rectangles, the icons are invisible now.

Additional context
Bug also happens with the extension fetched from https://github.com/nix-community/nix-vscode-extensions (both from vscode-marketplace and open-vsx).
The bug doesn't reproduce with vscode-fhs.
Notify maintainers

Metadata
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.1.12-zen1, NixOS, 23.05 (Stoat), 23.05.20230223.03fb722`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.13.2`
 - channels(root): `"nixos"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

VScode only allows files to be loaded from certain root folders (source).
// Define an initial set of roots we allow loading from
// - appRoot	: all files installed as part of the app
// - extensions : all files shipped from extensions
// - storage    : all files in global and workspace storage (https://github.com/microsoft/vscode/issues/116735)

On NixOS, the extensionsDir is in the Nix store with links to the different extensions (also in the Nix store).
This issue is that the extensions tries to load the file from the extension's folder directly (/nix/store/wls3y80nfld9hln166zw0b1rhssj5wyk-vscode-extension-eamodio-gitlens-2022.12.604) and not from the extensions folder (something like /nix/store/xxxxx-vscode-extensions).
If the path of the font is /nix/store/xxxxx-vscode-extensions/share/vscode/extensions/eamodio.gitlens/dist/glicons.woff2, it should work (/nix/store/xxxxx-vscode-extensions must be in the authorised folders whereas /nix/store/wls3y80nfld9hln166zw0b1rhssj5wyk-vscode-extension-eamodio-gitlens-2022.12.604 isn't).

Facing this until today. Anyone has workaround for this issue?

@CedricFinance thanks for the research! Do you think this is something we can patch in our packaging? Or does it require an upstream fix?

I researched into this as well. I am not really sure how @CedricFinance came to his conclusion, that the problem stems from vscode/vscodium not having permissions to load icons from specific places, if I understood him correctly. Neither do I know how we could test this hypothesis.
What I noticed, however, is that the github.copilot extension has the same problem:

Figure 1

Missing GitHub-Copilot Icon

Furthermore, if I go into my /nix/store/XXXXX-vscode-extensions/share/vscode/extensions/ folder and issues the command rg "\"icons\": \{" --max-columns=100 --max-columns-preview=10 ./*, I get the following output:
./vscjava.vscode-spring-boot-dashboard/package.json
30:    "icons": {

./github.copilot/package.json
304:		"icons": {

./eamodio.gitlens/package.json
7665:		"icons": {

Not sure about the spring-boot-dashboard extension, but it seems like extensions which define this attribute in their package.json cannot load their icons.


When no horizontal scrolling is needed, white bar shows in place of scroll bar

Chrome 68.0.3440.106, macOS 10.13.4.
See the white bar at the bottom when all questions fit horizontally:

When the window width is reduced, the scroll bar appears as expected:

To clarify, this bug will only surface if you are crazy like LN or I and use a mac but manually went to system preferences and set scrollbars to "always display".

🤔 :think: I definitely don't remember doing that! Would be good to get another check on a Mac. If it is only a thing that happens under very specific conditions then I agree we can close.
Apologies, I couldn't find a record of our prior conversation about this.

My settings:

Interesting. I think the default (which eg Matt has) is "When scrolling". This "Automatically" thing is new, I'm not sure how it works.

(And in either case, when you last pointed this out my main problem was that the bar appeared in a nonsensical spot; now that it appears just below the table attached to it I don't really have a problem with this visual artifact at all personally)

I think the default (which eg Matt has) is "When scrolling".

That's right, I remember, now. And the slight challenge with that is that a user needs to know about gestures.
Since it's platform and settings specific, I'm going to close. At least I'll have a record of the conversation next time I get puzzled by it.
I also think removing the gradient will help because then it won't be so high contrast.


stop running bot

Rasa version:0.13.0a3
Python version: 3.7
Operating system (windows, osx, ...):ubuntu
Issue: i want to stop the running bot via flask api, so i use common shutdown "werkzeug.server.shutdown" method in channels/facebook.py code, it was not shutdown the server, any other method to shutdown the server

Please ask these kind of questions in our forum

I am already posted in forum,but no one reply
On Thu, May 9, 2019, 8:10 PM Akela Drissner-Schmid<EMAIL_ADDRESS>wrote:

Closed #3406 https://github.com/RasaHQ/rasa/issues/3406.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/RasaHQ/rasa/issues/3406#event-2330458084, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AHGBH4SU2WWMXXD6Z6PESPTPUQZV7ANCNFSM4HLR4BOA
.


framework: refactor kmod for improved AMD support

Description of changes
Fixes #1028
Changes several aspects of framework-laptop-kmod support:

default value has been changed to true when NixOS >= 24.05 and linux kernel >=6.10; false otherwise
adds required patch for framework AMD enablement when kernel <6.10
add README section describing usage

Also: this may be an upstream nixpkgs issue, but the lack of a good lore.kernel.org fetcher is kind of surprising. I tried using this sample code with fetchpatch initially, but got weird errors with it. I implemented a fetchb4 fetcher for my own personal usage, which uses b4 to download and format patches from https://lore.kernel.org and the skeleton of that is present in this PR using runCommand for simplicity
Things done

[x] Tested the changes in your own NixOS Configuration
[x] Tested the changes end-to-end by using your fork of nixos-hardware and
importing it via <nixos-hardware> or Flake input

Edit: I've been running this on my own framework 16 for a bit now and everything seems to be working fine

Bump :p

kmod is not needed anymore in the latest kernels btw

The kernel now fails to build with 6.11.

The kernel now fails to build with 6.11.

There was a patch yesterday that landed on stable and unstable that makes it work on 6.11. Should be fine now, it's working on my machine.

Awesome I will give it another go today. Thanks!
On September 21, 2024 8:34:16 AM PDT, quatquatt @.***> wrote:

The kernel now fails to build with 6.11.

There was a patch yesterday that landed on stable and unstable that makes it work on 6.11. Should be fine now, it's working on my machine.
--
Reply to this email directly or view it on GitHub:
https://github.com/NixOS/nixos-hardware/pull/1029#issuecomment-2365229083
You are receiving this because you commented.
Message ID: @.***>


discussion(stream_pk): what should DynamicFilter's distribution key be?

If it can follow the upstream, we save a data shuffle.
If it is strictly the PK, we do not need to add anything to stream_pk.
If it is the comparison/join key, we do not get any benefit. This is the current behaviour so we should change it.
I think choosing the dist_key = PK is best. Any thoughts on this?

If it is strictly the PK, we do not need to add anything to stream_pk.

Didn't get it... If it is strictly the PK, what do we need to add into stream_pk?

Didn't get it... If it is not strictly the PK, what do we need to add into stream_pk?

For instance, if we use something other than PK, e.g. upstream key, then we will have to add that key to the stream_pk. So adds additional length to the key.

After some discussion, we decided to use upstream's distribution key as DynamicFilter's distribution key.

After offline discussion, we determined that its best for DynamicFilter's distribution to follow the upstream distribution. We should fix this in PR.

After offline discussion, we determined that its best for DynamicFilter's distribution to follow the upstream distribution. We should fix this in a PR.

Hi, @jon-chuang is this done yet?


Show one record per row in console tab

In before versions I could see one record per row, as it shows this image:

but now the view isn't understandable

Hi @erdusa ,
Other users reported this undesired behaviour. I'm looking into this.
Thank you for contributing.
Best regards


TableDataSet, or migration from 2.0.x to 2.1.x

Hi Allen,
My app uses TableDataSet to update the table with data from other parts of my application (forms, database sync etc) using .setData(..) but I've noticed it has been removed in 2.1.x
What is the new, recommended way of achieving the same kind of update?
Actually (not sure this is the right thing to do) I've tried to directly use TableDataStore:
import {TableDataStore} from 'react-bootstrap-table';

But it doesn't seem to work (TableDataStore is not exported on the package level), so I guess this is on purpose, because we are not supposed to play with TableDataStore directly?
EDIT: I will wait for your feedback to have an official answser, but after some digging around, it looks like there is something in progress that I could use for my table updates: https://github.com/AllenFang/react-bootstrap-table/pull/369  :)
Thanks!

@jbilcke, sorry for removingTableDataSet since v2.1.0. Because some reason and really few people use it. BTW, without TableDataSet, you still can update data just by updating react state. Following is a simple example:
 class BasicTable extends React.Component {

  constructor(props) {
     super(props);
     this.state = {
        products: YOUR_DATA
     };
  }

  handleFetchDataBtnClick = () => {
      //get your data and set data to state...
      this.setState({
          products: DATA_FROM_FETCH
      });
  }

 handleAddDataBtnClick= () => {
      //here just create a fake data and update state...
      const product = { .... };
      this.setState({
          products: [ ...this.state.products, product  ]
      });
  }

  render() {
    return (
      <div>
          <button onClick={ this.handleFetchDataBtnClick }>Fetch data</button>
          <button onClick={ this.handleAddDataBtnClick}>Add data</button>
          <BootstrapTable data={ this.state.products }>
              <TableHeaderColumn dataField='id' isKey={ true }>Product ID</TableHeaderColumn>
              <TableHeaderColumn dataField='name'>Product Name</TableHeaderColumn>
              <TableHeaderColumn dataField='price'>Product Price</TableHeaderColumn>
          </BootstrapTable>
      </div>
    );
  }
}

Anyway, #369 is just a remote example(meteor-like), but without this PR, you still can table data by state.
Thanks and feel free to discuss with me.
Sorry for this API change


Allow 'DEFAULT_ACTIONS_URL' to Specify 'none'

Feature Description
Hello,
I just read through the following discussions/changes regarding DEFAULT_ACTIONS_URL.

https://github.com/go-gitea/gitea/issues/24789
https://github.com/go-gitea/gitea/pull/25604

My understanding is that this variable will default to github and can be set to self (though that is not recommended). The strongly recommended path forward seems to be to encourage use of fully-qualified action names (FQANs) and eliminate ambiguity, especially with a desire to push GitHub to support them.
This makes me wonder if adding a none or disabled option for DEFAULT_ACTIONS_URL would be useful. This setting would require that all action names be fully qualified, and avoid assuming anything in the first place. The default setting github could be retained for backwards compatibility on existing instances, but the docs could strongly recommend none and perhaps someday future releases of Gitea could remove the option altogether (an easier sell once GitHub supports FQANs). Ultimately, this would make it easy for entire instances (especially self-hosted ones) to enforce the better, less-ambiguous practice.
Thanks for your consideration,
Nick
Screenshots
No response

I think it makes sense to have a none when you want to strictly enforce FQAN. It would make the actions incompatible with GitHub until they support it, but if one is only concerned about running on Gitea, it will work and eliminate any amiguity in the action names.


Container is not building

I got the following changes:

stretch -> bookworm. because stretch repos are gone.
python3.7 -> 3.9. because 3.7 no longer compiles. There is one python module that doesn't work anything about 3.9.
gcloud sdk from 288 -> 488. Again it doesn't install.
python-pkg-resources is not installed with apt as python3-pkg-resources, and python-smbus is python3-smbus.

I got two questions:

Do you forsee updating gcloud sdk from 288 to 455 would be problematic?
bw2/hail-utils repo is pretty much empty now. Is that still a requirement? hail-utils is no longer a module and can't be installed.

It builds after this patch. I can create a pull request if changes are okay.
diff --git a/docker/Dockerfile b/docker/Dockerfile
index 97d9526f..c08e73af 100644
--- a/docker/Dockerfile
+++ b/docker/Dockerfile
@@ -1,4 +1,4 @@
-FROM bitnami/minideb:stretch
+FROM bitnami/minideb:bookworm

 LABEL maintainer="Broad TGG"

@@ -27,14 +27,14 @@ RUN install_packages \
 # install java-8    (adds 340Mb)
 RUN install_packages default-jdk

-# install python3.7 (adds 800Mb)
+# install python3.9 (adds 800Mb)
 RUN apt-get update \
        && apt-get dist-upgrade -y \
-       && apt-get install -y --no-install-recommends python-smbus libncursesw5-dev libgdbm-dev libc6-dev zlib1g-dev libsqlite3-dev libssl-dev openssl libffi-dev \
+       && apt-get install -y --no-install-recommends python3-smbus python3-pkg-resources libncursesw5-dev libgdbm-dev libc6-dev zlib1g-dev libsqlite3-dev libssl-dev openssl libffi-dev \
        && apt-get clean \
     && rm -rf /var/lib/apt/lists/*

-ENV PYTHON_VERSION="3.7.1"
+ENV PYTHON_VERSION="3.9.17"
 RUN wget https://www.python.org/ftp/python/${PYTHON_VERSION}/Python-${PYTHON_VERSION}.tar.xz \
        && tar xvf Python-${PYTHON_VERSION}.tar.xz \
        && rm Python-${PYTHON_VERSION}.tar.xz
@@ -44,16 +44,14 @@ RUN ./configure --enable-optimizations \
     && make clean
 WORKDIR /

-RUN install_packages python-pkg-resources
-

 # install gcloud tools
 RUN python3 -m pip install --upgrade pip
 RUN python3 -m pip install --no-cache-dir -U crcmod

 ENV CLOUDSDK_PYTHON=python3
-RUN wget https://dl.google.com/dl/cloudsdk/channels/rapid/downloads/google-cloud-sdk-286.0.0-linux-x86_64.tar.gz
-RUN tar xzf google-cloud-sdk-286.0.0-linux-x86_64.tar.gz && rm google-cloud-sdk-286.0.0-linux-x86_64.tar.gz
+RUN wget https://dl.google.com/dl/cloudsdk/channels/rapid/downloads/google-cloud-sdk-455.0.0-linux-x86_64.tar.gz
+RUN tar xzf google-cloud-sdk-455.0.0-linux-x86_64.tar.gz && rm google-cloud-sdk-455.0.0-linux-x86_64.tar.gz
 RUN /google-cloud-sdk/install.sh -q

 # python packages
@@ -62,8 +60,8 @@ RUN python3 -m pip install --upgrade gsutil
 # install GCS connector using the same steps as in https://raw.githubusercontent.com/broadinstitute/install-gcs-connector/master/install_gcs_connector.py
 # assumes ~/.config/gcloud/application_default_credentials.json will be mounted into the docker container
 RUN python3 -m pip install hail
-COPY docker/spark-defaults.conf /usr/local/lib/python3.7/site-packages/pyspark/conf/spark-defaults.conf
-RUN wget https://repo1.maven.org/maven2/com/google/cloud/bigdataoss/gcs-connector/hadoop2-1.9.17/gcs-connector-hadoop2-1.9.17-shaded.jar -O /usr/local/lib/python3.7/site-packages/pyspark/jars/gcs-connector-hadoop2-1.9.17-shaded.jar
+COPY docker/spark-defaults.conf /usr/local/lib/python3.9/site-packages/pyspark/conf/spark-defaults.conf
+RUN wget https://repo1.maven.org/maven2/com/google/cloud/bigdataoss/gcs-connector/hadoop2-1.9.17/gcs-connector-hadoop2-1.9.17-shaded.jar -O /usr/local/lib/python3.9/site-packages/pyspark/jars/gcs-connector-hadoop2-1.9.17-shaded.jar

 # install htslib
 ENV SAMTOOLS_VERSION="1.9"
@@ -100,9 +98,6 @@ WORKDIR /ensembl-vep-release-${VEP_VERSION}
 RUN perl INSTALL.pl -a ap -n -l -g all
 RUN ln -s /ensembl-vep-release-${VEP_VERSION}/vep /vep

-# install hail utils
-RUN python3 -m pip install git+https://github.com/bw2/hail-utils.git
-
 WORKDIR /seqr-loading-pipelines
 COPY requirements.txt .
 RUN python3 -m pip install -r ./requirements.txt
@@ -118,7 +113,7 @@ COPY docker/bashrc /root/.bashrc
 COPY docker/gitconfig /root/.gitconfig
 COPY docker/bin/*.sh /usr/local/bin/

-ENV PATH=/usr/local/lib/python3.7/site-packages/pyspark/bin:/google-cloud-sdk/bin:$PATH
+ENV PATH=/usr/local/lib/python3.9/site-packages/pyspark/bin:/google-cloud-sdk/bin:$PATH
 ENV PYTHONPATH=".:/seqr-loading-pipelines:/seqr-loading-pipelines/luigi_pipeline"

 COPY docker/entrypoint.sh /

Taking a look...

Another point:
default-jdk now install java17 in bookworm but hail requires java 8 or 11

Looking at the hail install instruction, the seqr-pipeline-runner Dockerfile is missing dependencies libopenblas-base liblapack3 g++.
I think libopenblas-base is now named libopenblas0.

@barslmn I've got a branch here which includes your changes plus a few additional ones to mirror our infrastructure (pinning hail to the version "0.2.122" that we would like, bumping python to 3.10).  Feel free to pull that branch and test (and also comment on the PR!)

Thanks a lot, I will try it out.
Another thing I noticed is that my image is  almost double the size the one on gcr:
seqr@gendebian:~$ docker images
REPOSITORY                                      TAG               IMAGE ID       CREATED         SIZE
seqr-project/pipeline-runner             20231117_163008   7b9e1c6c2dc1   16 hours ago    6.93GB
gcr.io/seqr-project/pipeline-runner             gcloud-prod       6dfe8699a825   11 months ago   3.43GB

pip cache takes about 1.7Gb that we can clean with pip cache purge


ReadLightNovel down

Source information
ReadlightNovel
Steps to reproduce
No steps click on the source no novels show up
Expected behavior
Novels showing up
Actual behavior
Error no novels showing up
And saved novels no longer work
LNReader version
Latets
Android version
Android 11
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
[X] I have written a short but informative title.
[X] I have updated the app to version 1.1.12.
[X] I have updated all installed extensions.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I will fill out all of the requested information in this form.

Closing since the source is working as expected. Reopen if the issue still persists.


Memory plugin needs a refresh method

Currently there's no way of refreshing the memory information once it's been retrieved

Added in PR #53


Bug 62870 / Templates : Add ability to provide parameters

Description
This is the implementation of a feature in enhancement list:
"Templates : Add ability to provide parameters:"

https://bz.apache.org/bugzilla/show_bug.cgi?id=62870

It has been implemented by members of UbikLoadPack (https://ubikloadpack.com) team (Thomas, Florent and Philippe) and is donated to Apache JMeter project since we have signed a CCLA few years ago.
Motivation and Context
This feature will allow providing a kind of wizard for template allowing to specify parameters when creating a new Plan:
https://bz.apache.org/bugzilla/show_bug.cgi?id=62870
How Has This Been Tested?
The PR contains JUnit tests for part of the code.
And we have performed manual testing.
Screenshots (if appropriate):
The feature is fully documented in the PR with screenshots as long as existing one (template).

Types of changes

New feature (non-breaking change which adds functionality)
No Breaking change as parameters are optional

Checklist:

[X] My code follows the code style of this project.
[X] I have updated the documentation accordingly.

Codecov Report

Merging #432 into trunk will decrease coverage by 0.03%.
The diff coverage is 28.43%.

@@             Coverage Diff             @@
##             trunk     #432      +/-   ##
===========================================
- Coverage     58.6%   58.57%   -0.04%
- Complexity   10638    10667      +29
===========================================
  Files         1196     1198       +2
  Lines        76087    76331     +244
  Branches      7361     7399      +38
===========================================
+ Hits         44594    44712     +118
- Misses       28986    29106     +120
- Partials      2507     2513       +6

Impacted Files
Coverage Δ
Complexity Δ

...pache/jmeter/gui/action/SelectTemplatesDialog.java
1.58% <0%> (-1.42%)
1 <0> (ø)

src/core/org/apache/jmeter/util/TemplateUtil.java
0% <0%> (ø)
0 <0> (?)

...rg/apache/jmeter/gui/action/template/Template.java
20.93% <4.28%> (+20.93%)
12 <2> (+12)
:arrow_up:

...he/jmeter/gui/action/template/TemplateManager.java
74.41% <84.21%> (+74.41%)
13 <10> (+13)
:arrow_up:

...meter/gui/action/template/TestTemplateManager.java
87.8% <87.8%> (ø)
4 <4> (?)

Continue to review full report at Codecov.

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

Hello Team,
The PR looks good to me, tested on Mac OSX.
Anybody willing to commit it ?
I can take in charge , but I would prefer somebody else reviewing it.
If no feedback by tomorrow evening, I'll commit it.
Thanks

This seems to be quite a big patch. Do you think two days are enough to have a good look at it?

Hi Felix,
You’re right.
Take your time
Sorry, I am always impatient guy :)
Regards

Hello Felix,
I have committed the fixes following your review.
Let me know if you will commit the PR (which I would prefer), otherwise I'll do it.
Thanks
Regards

Hello,
I'll be merging it today.
Regards


searching for a friend on STEAM！🤪🤪🤪

I want to searching for a friend on STEAM！！！🤪
My email<EMAIL_ADDRESS>My id ：“晨哥&iuoiang”

glad to be steam friend


vue-test-utils doesn't track object props

vue-test-utils doesn't track object props
Context
i have a User object which is the data model for a vue.js UserComponent. When the component is mounted the user has undefined fields. Then inside the mounted() method the user is fetched from the server and all its fields are setted.
Problem
So i found that vue-test-utils doesn't redraw component when user's fields changed.
Note
On the other hand the same test works good if i change object prop user to scalar.
Steps to reproduce
https://github.com/alexey2baranov/vue-test-utils-getting-started
Expected behaviour
Test should be done
Actual behaviour
Test fails
Possible Solution
Unfortunately i don't know any possible solution. I tried to reset user prop by wrapper.setProp({user: user}) after its fields was fetchad. Unfortunatelly it is impossible as vue-test-utils prevent to set the same object to prop twice.

I've run into a similar issue today. I believe it comes down to the fact that initialising props using mount({ propsData }) will not place reactivity on any objects within propsData. But using wrapper.setProps(...) will place reactivity. This means that if I have the prop myProp: { a: 12, b: 'hello' }, and I have a button that goes myProp.a += 1, I will not trigger computed or watcher updates if I provided myProp to mounted; but I will if I originally provided it via setProps.
And you can't say "well just use setProps", because like in the OP's case, we have things happening on created and mounted, by which point it's too late to be using setProps; it wouldn't match how the component functions in the actual application.

The example you showed mutates the prop, has a promise that you dont await... This is not a VTU issue.

@dobromir-hristov I was very dissapointed that you closed my issue. I suppose this issue clearaly shows VTU bug, I think it must be reopened.  Maybe you have to review it again or I will open it in another one.
I will try to explain this test. You say i don't await mounted  promise, but it is not clearaly true. If you see in the code again you can see  await waitForExpect(). waitForExpect checks all expectations every 50ms during 5 seconds, which is the default Jest timeout. 5seconds-long waitForExpect guarantes that the 1second-long mounted  promise resolves before last waitForExpect  iteration checks the expectations.
So this test clearaly shows that VTU mounts a component, then the component mutates its object property inside mounted  hoock in 1 second, and then the test checks every 50 ms that the VTU redraws component. During this 5 seconds VTU doesn't redraw the component and the test fails.
Maybe you can show me the clearer way of awaiting the mounted promise. Then I can refactor my code. But I don't think it changes the test's result.


how to get svg file of each layer of the cell from gds file

Hi Heitzmann,
Thank you for this great module.
I have a gds file for the library. I am able to obtain the svg files of all cells.  But, is there a way to get a svg file for each layer of the cells separately?
svg.zip
Thanks

This should be a good start:
import sys
import copy
import gdspy

if len(sys.argv) < 2:
    sys.stderr.write(f"Missing file name!\nUSAGE: {sys.argv[0]} filename\n")
    sys.exit(1)

for fname in sys.argv[1:]:
    print(f"Processing {fname} layers:", end="", flush=True)

    lib = gdspy.GdsLibrary(infile=sys.argv[1])

    all_layers = set()
    for cell in lib.top_level():
        all_layers.update(cell.get_layers())

    for keep in all_layers:
        print(f" {keep}", end="", flush=True)
        lib_copy = copy.deepcopy(lib)
        for cell in lib_copy.cells.values():
            cell.remove_polygons(lambda pts, layer, datatype: layer != keep)
            # Checking only the first item in path.layers works because the library was
            # loaded from a file, so it neve creates paths with more than 1 element.
            cell.remove_paths(lambda path: path.layers[0] != keep)
            cell.remove_labels(lambda label: label.layer != keep)
        for cell in lib_copy.top_level():
            # Make sure all cell names are valid file names
            cell.write_svg(f"{cell.name}-{keep}.svg")

    print("\n")

The bounding boxes will be different for each layer saved, since many elements are removed for each layer. If that's a problem, another option is to keep all elements, but make them transparent in the SVG output:
import sys
import copy
import gdspy

if len(sys.argv) < 2:
    sys.stderr.write(f"Missing file name!\nUSAGE: {sys.argv[0]} filename\n")
    sys.exit(1)

for fname in sys.argv[1:]:
    print(f"Processing {fname} cells:", end="", flush=True)

    lib = gdspy.GdsLibrary(infile=sys.argv[1])
    for cell in lib.top_level():
        print(f" {cell.name}", end="", flush=True)

        ld, lt = cell.get_svg_classes()
        all_layers = set(layer for layer, typ in ld.union(lt))
        for keep in all_layers:
            style = {
                (layer, typ): {"stroke-opacity": "0", "fill-opacity": "0"}
                for layer, typ in ld
                if layer != keep
            }
            fontstyle = {
                (layer, typ): {"stroke-opacity": "0", "fill-opacity": "0"}
                for layer, typ in lt
                if layer != keep
            }
            cell.write_svg(f"{cell.name}-{keep}.svg", style=style, fontstyle=style)

    print("\n")

This should be a good start:
import sys
import copy
import gdspy

if len(sys.argv) < 2:
    sys.stderr.write(f"Missing file name!\nUSAGE: {sys.argv[0]} filename\n")
    sys.exit(1)

for fname in sys.argv[1:]:
    print(f"Processing {fname} layers:", end="", flush=True)

    lib = gdspy.GdsLibrary(infile=sys.argv[1])

    all_layers = set()
    for cell in lib.top_level():
        all_layers.update(cell.get_layers())

    for keep in all_layers:
        print(f" {keep}", end="", flush=True)
        lib_copy = copy.deepcopy(lib)
        for cell in lib_copy.cells.values():
            cell.remove_polygons(lambda pts, layer, datatype: layer != keep)
            # Checking only the first item in path.layers works because the library was
            # loaded from a file, so it neve creates paths with more than 1 element.
            cell.remove_paths(lambda path: path.layers[0] != keep)
            cell.remove_labels(lambda label: label.layer != keep)
        for cell in lib_copy.top_level():
            # Make sure all cell names are valid file names
            cell.write_svg(f"{cell.name}-{keep}.svg")

    print("\n")

The bounding boxes will be different for each layer saved, since many elements are removed for each layer. If that's a problem, another option is to keep all elements, but make them transparent in the SVG output:
import sys
import copy
import gdspy

if len(sys.argv) < 2:
    sys.stderr.write(f"Missing file name!\nUSAGE: {sys.argv[0]} filename\n")
    sys.exit(1)

for fname in sys.argv[1:]:
    print(f"Processing {fname} cells:", end="", flush=True)

    lib = gdspy.GdsLibrary(infile=sys.argv[1])
    for cell in lib.top_level():
        print(f" {cell.name}", end="", flush=True)

        ld, lt = cell.get_svg_classes()
        all_layers = set(layer for layer, typ in ld.union(lt))
        for keep in all_layers:
            style = {
                (layer, typ): {"stroke-opacity": "0", "fill-opacity": "0"}
                for layer, typ in ld
                if layer != keep
            }
            fontstyle = {
                (layer, typ): {"stroke-opacity": "0", "fill-opacity": "0"}
                for layer, typ in lt
                if layer != keep
            }
            cell.write_svg(f"{cell.name}-{keep}.svg", style=style, fontstyle=style)

    print("\n")

This works great! I am getting files like these.
AND2_X1.zip
Is there a way to remove the text/label which appears at the bottom of the images.
Also, is it possible to get the layer names corresponding to the layer numbers.

This works great! I am getting files like these.
AND2_X1.zip
Is there a way to remove the text/label which appears at the bottom of the images.
Also, is it possible to get the layer names corresponding to the layer numbers.

@mo-nu The layer names come from a typo in my script: it was fontstye=style, instead of fontstyle=fontstyle. It's fixed now.
Regarding layer names: GDS files have no notion of layer names. That information is not present in the file, it is a template or technology specification used by your software. You'd have to manually assign names to layer number (based on a dictionary, for example):
layer_names = {
    0: "layer 0",
    1: "layer 1",
    # etc.
}

Then, when saving the file inside the loop, use, for example:
layer_name = layer_names[keep]  # Make sure all you layers have a name, or this will raise an error.
cell.write_svg(f"{cell.name}-{layer_name}.svg", style=style, fontstyle=fontstyle)

@mo-nu The layer names come from a typo in my script: it was fontstye=style, instead of fontstyle=fontstyle. It's fixed now.
Regarding layer names: GDS files have no notion of layer names. That information is not present in the file, it is a template or technology specification used by your software. You'd have to manually assign names to layer number (based on a dictionary, for example):
layer_names = {
    0: "layer 0",
    1: "layer 1",
    # etc.
}

Then, when saving the file inside the loop, use, for example:
layer_name = layer_names[keep]  # Make sure all you layers have a name, or this will raise an error.
cell.write_svg(f"{cell.name}-{layer_name}.svg", style=style, fontstyle=fontstyle)

Thank you, that change fixed the layers part.
import gdspy
import numpy
import sys
filename=''
del sys.argv[0]
try:
filename=str(sys.argv[0])
except:
print(f"Must provide a filename")
exit()
for file in sys.argv:
filename=str(file)
print(f"Generating .svg files for {filename}")
lib = gdspy.GdsLibrary(infile=filename)
cellList = lib.cells
for val in cellList:
	cell = cellList[val]
	print(f"Processing {cell}")
	cell.write_svg(f"{val}.svg")

If I run this code, I get the svg file of the cell(with all layers). I am attaching an example file.
AND2_X1.zip
How do I change this code, so that I don't get the text/label which appears at the bottom of the image.

Thank you, that change fixed the layers part.
import gdspy
import numpy
import sys
filename=''
del sys.argv[0]
try:
filename=str(sys.argv[0])
except:
print(f"Must provide a filename")
exit()
for file in sys.argv:
filename=str(file)
print(f"Generating .svg files for {filename}")
lib = gdspy.GdsLibrary(infile=filename)
cellList = lib.cells
for val in cellList:
	cell = cellList[val]
	print(f"Processing {cell}")
	cell.write_svg(f"{val}.svg")

If I run this code, I get the svg file of the cell(with all layers). I am attaching an example file.
AND2_X1.zip
How do I change this code, so that I don't get the text/label which appears at the bottom of the image.

Thank you, that change fixed the layers part.

import gdspy
import numpy
import sys
filename=''
del sys.argv[0]
try:
filename=str(sys.argv[0])
except:
print(f"Must provide a filename")
exit()
for file in sys.argv:
filename=str(file)
print(f"Generating .svg files for {filename}")
lib = gdspy.GdsLibrary(infile=filename)
cellList = lib.cells
for val in cellList:
cell = cellList[val]
print(f"Processing {cell}")
cell.write_svg(f"{val}.svg")

If I run this code, I get the svg file of the cell(with all layers). I am attaching an example file.
AND2_X1.zip
How do I change this code, so that I don't get the text/label which appears at the bottom of the image.

Thank you, that change fixed the layers part.

import gdspy
import numpy
import sys
filename=''
del sys.argv[0]
try:
filename=str(sys.argv[0])
except:
print(f"Must provide a filename")
exit()
for file in sys.argv:
filename=str(file)
print(f"Generating .svg files for {filename}")
lib = gdspy.GdsLibrary(infile=filename)
cellList = lib.cells
for val in cellList:
cell = cellList[val]
print(f"Processing {cell}")
cell.write_svg(f"{val}.svg")

If I run this code, I get the svg file of the cell(with all layers). I am attaching an example file.
AND2_X1.zip
How do I change this code, so that I don't get the text/label which appears at the bottom of the image.

Thank you, that change fixed the layers part.
import gdspy
import numpy
import sys

filename=''

del sys.argv[0]

try:
	filename=str(sys.argv[0])
except:
	print(f"Must provide a filename")
	print(f"Usage example: gdstosvg ../path_to/gds_files/*/*latest.gds")
	exit()

for file in sys.argv:
	filename=str(file)
	print(f"Generating .svg files for {filename}")
	lib = gdspy.GdsLibrary(infile=filename)

	cellList = lib.cells
	for val in cellList:
		cell = cellList[val]
		print(f"Processing {cell}")
		cell.write_svg(f"{val}.svg")

If I run this code, I get the svg file of the cell(with all layers). I am attaching an example file.
AND2_X1.zip
How do I change this code, so that I don't get the text/label which appears at the bottom of the image.

Thank you, that change fixed the layers part.
import gdspy
import numpy
import sys

filename=''

del sys.argv[0]

try:
	filename=str(sys.argv[0])
except:
	print(f"Must provide a filename")
	print(f"Usage example: gdstosvg ../path_to/gds_files/*/*latest.gds")
	exit()

for file in sys.argv:
	filename=str(file)
	print(f"Generating .svg files for {filename}")
	lib = gdspy.GdsLibrary(infile=filename)

	cellList = lib.cells
	for val in cellList:
		cell = cellList[val]
		print(f"Processing {cell}")
		cell.write_svg(f"{val}.svg")

If I run this code, I get the svg file of the cell(with all layers). I am attaching an example file.
AND2_X1.zip
How do I change this code, so that I don't get the text/label which appears at the bottom of the image.

@mo-nu In this specific case, it looks like all undesired labels are in layer 63, text type 63, so removing those from the library before starting the conversion should do the trick:
lib = ...
for cell in lib.cells.values():
    cell.remove_labels(lambda label: label.layer == 63 and label.texttype == 63)
...

@mo-nu In this specific case, it looks like all undesired labels are in layer 63, text type 63, so removing those from the library before starting the conversion should do the trick:
lib = ...
for cell in lib.cells.values():
    cell.remove_labels(lambda label: label.layer == 63 and label.texttype == 63)
...

Thank you so much!


ntr metal ion transmembrane transport

I tried to do this by TG, but there is no chebi  term for metal ion
(current term metal ion transport uses "metal cation" in logical def)

Do you know of any metal anions that occur in a physiological system? I'm not sure they do.

good point. So should  "metal ion transport" really be "metal cation transport"

I don't think so, because it wouldn't be the term most commonly used.

Just call the term as 'metal ion transport' and use the cation term from ChEBI in the logical def.   I was confused!


"Return all robots to your hand" returns cards when it sounds like it should return objects

duplicate


New check: napostrophe - suggest that we check for the presence of this glyph which is depreciated and warn to remove it.

What needs to be checked?
(napostrophe - suggest that we check for the presence of this glyph which is depreciated and warn to remove it.)
Detailed description of the problem
napostrophe is a glyph in old glyph sets and even has a unicode ( 0149 ) but it is depreciated as a mistake in unicode now.
https://en.wikipedia.org/wiki/N-apostrophe
Resources and steps needed to reproduce the problem
Just check for the presence of the unicode code point in the font I think.
Suggested profile
Suggest which profile the check should be added to. The most common are:
Note sure but: ( ? )

[X] Vendor-specific: Google Fonts
[ ] Vendor-specific: Adobe Fonts
[ ] OpenType (requirements imposed by the OpenType specification)
[X] Universal (broadly accepted best practices on the type design community)
[ ] Other:

Suggested result
Which log result level should the check have:

[ ] 🔥 FAIL (An issue that must be corrected for the font to function properly)
[X] ⚠ WARN (A potential issues that may need to be addressed)

Severity assessment
(Classify the problem on a scale of 1 (minor) to 5 (major). How "buggy" would the font be considered if it had the problem described?)
3?

From that wikipedia article:

The Unicode standard recommends that a sequence of an apostrophe followed by n (’n) be used to encode this diagraph.[1] (A precomposed character form was included in Unicode for legacy ISO/IEC 6937 and CP853 document compatibility, as U+0149 ŉ LATIN SMALL LETTER N PRECEDED BY APOSTROPHE, but its use is deprecated.[2] The use of deprecated characters such as ŉ is "strongly discouraged".[3] However it continues to be used in the Afrikaans versions of Facebook and other publications, probably to avoid the tendency of auto-correction software (designed for English quotation marks) to turn a typed 'n (straight apostrophe, n) into ‘n (left single quotation mark, n), which is incorrect but common (rather than the correct form, ’n). The code point has been removed from some computer fonts, such as Charis SIL and Doulos SIL.)

The upper case, or majuscule form has never been included in any international keyboards and is not encoded as a precomposed character. It may be generated by combining (U+02BC ʼ MODIFIER LETTER APOSTROPHE) and N to create ʼN.

@EbenSorkin, I'm a bit concerned about the part that says "However it continues to be used in the Afrikaans versions of Facebook and other publications,"

Some of it being used probably has to do with really old keyboards IBM made
still being used. And there are " roughly 46 million people who speak
Afrikaans as a first or second language worldwide. "
46 Million is not that trivial a number. 😅
But since you can make it "by combining (U+02BC ʼ MODIFIER LETTER APOSTROPHE
https://en.wikipedia.org/wiki/ʼ) and N
https://en.wikipedia.org/wiki/N" maybe we should encourage this to
conform to Unicode?
Perhaps we should include it but perform a calt transformation in the font
to the now orthodox unicode model for shaping as standard in SSA and PRI
level fonts?
On Tue, Oct 15, 2024 at 6:18 PM Felipe Corrêa da Silva Sanches <
@.***> wrote:

@EbenSorkin https://github.com/EbenSorkin, I'm a bit concerned about
the part that says "However it continues to be used in the Afrikaans
versions of Facebook and other publications,"
—
Reply to this email directly, view it on GitHub
https://github.com/fonttools/fontbakery/issues/4861#issuecomment-2415268099,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAQUQXPJCTMHEJCNLYJVGWLZ3WIE7AVCNFSM6AAAAABPZFWVW6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMJVGI3DQMBZHE
.
You are receiving this because you were mentioned.Message ID:
@.***>

A suggestion that a character should be removed from a font would sound quite weird, I think. Is the goal of such suggestion to actively discourage the users from using it by making it somewhat inaccessible?


Upgrade: disable xDS translation during the upgrade process to avoid client request failures

Description:
The upgrade of EG may cause clients to experience request failures or a temporary loss of connectivity to the Envoy
For example, while upgrading from v1.0.2 to v1.1.0 following steps in the upgrade guide, the EG fails to reconcile the BackendTLSPolicy CRs as the CRD is deleted and recreated during the upgrade process.
Before the upgrade, a BackendTLSPolicy CRD is created to configure the TLS settings for the Backend service, following the steps in the Backend TLS: Gateway to Backend task.
Using egctl to get the generanted xDS cluster, you can see the TLS configuration for the Backend Cluster:
       {
          "cluster": {
             ... omitted for brevity
            "name": "httproute/default/backend/rule/0",
            "outlierDetection": {},
            "perConnectionBufferLimitBytes": 32768,
            "transportSocketMatches": [
              {
                "match": {
                  "name": "httproute/default/backend/rule/0/tls/0"
                },
                "name": "httproute/default/backend/rule/0/tls/0",
                "transportSocket": {
                  "name": "envoy.transport_sockets.tls",
                  "typedConfig": {
                    "@type": "type.googleapis.com/envoy.extensions.transport_sockets.tls.v3.UpstreamTlsContext",
                    "commonTlsContext": {
                      "combinedValidationContext": {
                        "defaultValidationContext": {
                          "matchTypedSubjectAltNames": [
                            {
                              "matcher": {
                                "exact": "www.example.com"
                              },
                              "sanType": "DNS"
                            }
                          ]
                        },
                        "validationContextSdsSecretConfig": {
                          "name": "enable-backend-tls/default-ca",
                          "sdsConfig": {
                            "ads": {},
                            "resourceApiVersion": "V3"
                          }
                        }
                      }
                    },
                    "sni": "www.example.com"
                  }
                }
              }
            ],
            "type": "EDS"
          },
        }

Delete the BackendTLSPolicy CRD, as the upgrade guide suggests:
kubectl delete crd backendtlspolicies.gateway.networking.k8s.io

EG reports the following error message in the logs:
E0802 09:30:12.542305       1 reflector.go:147] pkg/mod/k8s.io/client-go@v0.29.2/tools/cache/reflector.go:229: Failed to watch *v1alpha2.BackendTLSPolicy: failed to list *v1alpha2.BackendTLSPolicy: the server could not find the requested resource (get backendtlspolicies.gateway.networking.k8s.io)

The generated xDS does not contain the TLS configuration for the Backend Cluster, and the client requests fails.
Failed request example, as the error message suggests, the envoy sends an HTTP request to the backend service, which is an HTTPS server:
 curl -v -HHost:www.example.com --resolve "www.example.com:80:<IP_ADDRESS>" \
http://www.example.com:80/get
* Added www.example.com:80:<IP_ADDRESS> to DNS cache
* Hostname www.example.com was found in DNS cache
*   Trying <IP_ADDRESS>:80...
* Connected to www.example.com (<IP_ADDRESS>) port 80
> GET /get HTTP/1.1
> Host:www.example.com
> User-Agent: curl/8.8.0-DEV
> Accept: */*
>
* Request completely sent off
< HTTP/1.1 400 Bad Request
< date: Fri, 02 Aug 2024 09:48:47 GMT
< transfer-encoding: chunked
<
Client sent an HTTP request to an HTTPS server.

The xDS cluster does not contain the TLS configuration for the Backend Cluster:
        {
          "cluster": {
            "@type": "type.googleapis.com/envoy.config.cluster.v3.Cluster",
            "circuitBreakers": {
              "thresholds": [
                {
                  "maxRetries": 1024
                }
              ]
            },
            "commonLbConfig": {
              "localityWeightedLbConfig": {}
            },
            "connectTimeout": "10s",
            "dnsLookupFamily": "V4_ONLY",
            "edsClusterConfig": {
              "edsConfig": {
                "ads": {},
                "resourceApiVersion": "V3"
              },
              "serviceName": "httproute/default/backend/rule/0"
            },
            "lbPolicy": "LEAST_REQUEST",
            "name": "httproute/default/backend/rule/0",
            "outlierDetection": {},
            "perConnectionBufferLimitBytes": 32768,
            "type": "EDS"
          },
          "lastUpdated": "2024-08-02T08:50:06.487Z",
          "versionInfo": "3519b134757f0bea49bbf0397de781693ab58f27b315c5ca77d642ee24a03b26"
        }

Similarly, the client request will also fail if any of the Gateway CRDs or EG CRDs have breaking changes and need to be deleted and recreated during the upgrade process. If any of the CRs need to be recreated/modified due to breaking changes, the same issue will occur as well.
Disable xDS translation during the upgrade process
One way to avoid this issue is to disable the xDS translation during the upgrade process. This way, the EG will not generate the xDS configuration with the broken CRs or any temporary middle state during the upgrade process, and the Envoy will continue to use the existing xDS configuration generated with the correct version of CRDs before the upgrade. Once the upgrade is complete, the xDS translation can be enabled, and the EG will generate the xDS configuration with the updated CRDs
Alternative:
Disable xDS server while upgrading.
[optional Relevant Links:]

Any extra documentation required to understand the issue.

cc @envoyproxy/gateway-maintainers

thanks for raising this issue @zhaohuabing
the issue here is specific to the fact that the below operations are not atomic
kubectl delete crd backendtlspolicies.gateway.networking.k8s.io # delete v1alpha2 BackendTLSPolicy
kubectl apply -f ./gateway-helm/crds/gatewayapi-crds.yaml # install v1alpha3 BackendTLSPolicy

which may cause downtime for a short duration.
rather than adding complexity into control plane for this, I'd prefer if we able to instrument envoy to used a stale xds contents (disable connecting to the xds server) for the short duration while the CRDs and control planes were upgraded

Other breaking changes that require modification on existing CRs will also cause similar issues. For example, the xPolicy reconciling will fail because the namesapece of the targetRef is not supported anymore.


Add OnPrivBufferStarting/Ending signals, similar to Chan buffers

Allows hooking into query buffer playback before and after all the
lines of the query buffer are replayed. The EModRet return value
has no effect at the moment, but may be used in the future to e.g.
prevent playback. The Chan version of these signals use EModRet to
skip emitting the status message, but not the whole playback.

Coverage increased (+0.008%) to 40.095% when pulling 4d5724eb0aa567503c4ec830012e389f2d4b5cd5 on torarnv:priv-buffer-playback-signals into 56620af107921707062b3eb8089274979d78f278 on znc:master.

CI build failures seem unrelated.

I restarted the failed travis build. Perhaps "leader" will this time wait successfully for his "minions", whatever that means...

Kicking Travis seems to have fixed it 😄


Fix 293: ORDER BY pushes unnecessary projections

Looking at the first test failure: it appears I'll have to teach OrderBinder to resolve columnrefs itself, so it can do that after matching against aliases, instead of pre-resolving via SelectBinder.

Hi, sorry for the delay. Thank you very much for your PR. I am looking into it today,

Would you please so nice to add a test case?


[bug]: shadcn dialog with react hook forms and zod validation close button issue

Describe the bug
I have a shadcn dialog, there is a form inside it that uses react-hook-forms and zod validation, the problem is that when I have mode: all selected like this, useForm({mode: 'all'}) , on clicking the close dialog button, I get the validation error messages displayed since I am using formState errors property to display validation errors. I have to click the close button twice to close the dialog. Is this expected behavior?

Affected component/components
Dialog
How to reproduce

Create a simple form with react-hooks-form useForm() hook, and zod validation schema.
Add mode all in useForm({mode: 'all'})
Put the form inside the dialog and try to close the dialog with the close button,
On clicking the first. time, the dialog won't close, if there are any validation errors,
On clicking the 2nd time, it will close

Codesandbox/StackBlitz link
No response
Logs
No response
System Info
MacOs 14.5 , Firefox, Brave

Before submitting

[X] I've made research efforts and searched the documentation
[X] I've searched for existing issues

I'm not sure, but I guess since every button has a type of "submit" by default. Setting the button type of the close button as "button" should fix your problem, in you dialog.tsx/jsx

Scroll to the DialogContent
Inside it find the DialogPrimitive.Close component
add the type="button"  to the DialogPrimitive.Close component like this:

   <DialogPrimitive.Close
       type="button"
       className="absolute right-4 top-4 rounded-sm opacity-70 ring-offset-background transition-opacity hover:opacity-100 focus:outline-none focus:ring-2 focus:ring-ring focus:ring-offset-2 disabled:pointer-events-none data-[state=open]:bg-accent data-[state=open]:text-muted-foreground">
        <X className="h-4 w-4" />
        Close
   </DialogPrimitive.Close>

Let me know if it works with you!

I'm not sure, but I guess since every button has a type of "submit" by default. Setting the button type of the close button as "button" should fix your problem, in you dialog.tsx/jsx
* Scroll to the **DialogContent**

* Inside it find the **DialogPrimitive.Close** component

* add the `type="button" ` to the **DialogPrimitive.Close** component like this:

   <DialogPrimitive.Close
       type="button"
       className="absolute right-4 top-4 rounded-sm opacity-70 ring-offset-background transition-opacity hover:opacity-100 focus:outline-none focus:ring-2 focus:ring-ring focus:ring-offset-2 disabled:pointer-events-none data-[state=open]:bg-accent data-[state=open]:text-muted-foreground">
        <X className="h-4 w-4" />
        Close
   </DialogPrimitive.Close>

Let me know if it works with you!

I tried that, however, it didn't work - same behavior. Nevertheless, for the time being I am going to use mode: 'onSubmit', that works well and I think it's enough for my current requirement. Thanks for you prompt reply.

I have the same problem，there is a form inside the dialog, input some words in the Input element，the words will selected when i click other place

it will selected

Hey i wanted to ask, does your select component work properly inside the dialog form? Mine is closed automatically when the selection is clicked


enable self-diagnostics in example app

The self-diagnostics feature has been invaluable while testing.
I would like to enable this in our example app.
I confirmed that the *.log files are excluded from git.

LGTM.
Do we have asp.net sample apps and worker/console as well?

@cijothomas not yet, but this is something I want to add! :)


Add postgres support for specifying schema when defining enum type

Hello!
Since we also ran into the need for specifying the schema name when defining a postgres enum to avoid collision when the enum types name appears in multiple schemas (https://github.com/launchbadge/sqlx/issues/1171), here is an attempt to implement this feature.
This PR contains the following changes:

extending the enum type macro to accept a schema attribute
update postgres implementation to treat the enum types name as schema.enum_name when schema is specified
when querying the type info by type id from the pg_catalog, the schema's name + the count of type that have the same name as the type that is being queried are joined to the fetched data. In case there are multiple types with the same name, use the schema.enum_name format (unless the schema is the default public schema, in that case we just use the enum name)
when querying the type by name and if the schema attribute is provided, include it in the fetch query so that only the desired type gets returned.
add a test for an enum type defined with an already existing name color_upper but in a different schema test which verified that there is no collision with these 2 enums

This is a quite minimalistic implementation, therefore it's possible that some things are missing (some edge cases that need to be covered/tested, better code design, ...), if so, let me know and I'll be happy to add the requested changes.

@lazareviczoran sorry that it took so long to look at this, in the intervening time it looks like some merge conflicts were introduced.
I'm closing this due to inactivity, feel free to reopen if you want to address the conflicts.

@lazareviczoran sorry that it took so long to look at this, in the intervening time it looks like some merge conflicts were introduced.
I'm closing this due to inactivity, feel free to reopen if you want to address the conflicts.

No worries, I rebased and opened #2638 instead.


color change (per minute) not working

Describe the bug
If you set the color change function to "every minute", the color won't change every minute.
To Reproduce
Steps to reproduce the behavior:

Go to 'Einstellungen -> Uhr'
Click on 'Front - Farbwechsel'
Choose 'jede minute'
See led, they won't change color every minute

Expected behavior
A color change every minute
Additional context
Add any other context about the problem here.

changed the code from resetting the respective flags in the loop to resetting the flags in the respective mode...
added the following flags:
bool clockMinPassed      = true;    // flag for the color change of the clock face (1 min)
bool clockHourPassed     = true;    // flag for the color change of the clock face (1 hour)
bool clockDayPassed      = true;    // flag for the color change of the clock face (1 day)
bool downMinPassed       = true;    // flag for the color change of the downlights (1 min)
bool downHourPassed      = true;    // flag for the color change of the downlights (1 hour)
bool downDayPassed       = true;    // flag for the color change of the downlights (1 day)

clearing the flag of the edge lights (in current active mode)
downMinPassed  = false;     // reset the color change flag (1 min)
downHourPassed = false;     // reset the color change flag (1 hour)
downDayPassed  = false;     // reset the color change flag (1 day)

clearing the flag of the downlights
downMinPassed  = false;      // reset the color change flag (1 min)
downHourPassed = false;      // reset the color change flag (1 hour)
downDayPassed  = false;      // reset the color change flag (1 day)

Seems to work now -> adding it to the next release


Is loss_t2t in RAM necessary? Can I just remove it?

Can I remove this part from the source code? tag-text-generation

Hi, sorry for the delay reply due to my busy schedule.
Yes, Tag-Text-Generation is unnecessary, if you only have image-tag dataset.
By the way, recently I found a miss in the released code, if you train from scratch, don't forget to load the pertained Swin backbone.
            if stage == 'train_from_scratch':
                # download from https://github.com/microsoft/Swin-Transformer
                state_dict = torch.load(vision_config['ckpt'], map_location="cpu")['model']

                for k in list(state_dict.keys()):
                    if 'relative_position_bias_table' in k:
                        dst_num_pos = (2 * vision_config['window_size'] - 1) ** 2
                        state_dict[k] = interpolate_relative_pos_embed(state_dict[k], dst_num_pos, param_name=k)
                    elif ('relative_position_index' in k) or ('attn_mask' in k):
                        del state_dict[k]
                print("### Load Vision BackboneL", vit)
                msg = self.visual_encoder.load_state_dict(state_dict, strict = False)
                print("missing_keys: ", msg.missing_keys)
                print("unexpected_keys: ", msg.unexpected_keys)

Thanks for the reply.

@xinyu1205
batch_size=32, 8*a100, loss_tag is very large
Train Epoch: [0]  [   50/14844]  eta: 5:44:45  lr: 0.000017  loss_tag: 437.2372  loss_dis: 0.4514  time: 1.1722  data: 0.1298  max mem: 24075 Train Epoch: [0]  [  100/14844]  eta: 5:16:42  lr: 0.000034  loss_tag: 492.6901  loss_dis: 0.4581  time: 1.1805  data: 0.1312  max mem: 24075 Train Epoch: [0]  [  150/14844]  eta: 5:06:15  lr: 0.000050  loss_tag: 458.0364  loss_dis: 0.4138  time: 1.1658  data: 0.1228  max mem: 24075 Train Epoch: [0]  [  200/14844]  eta: 4:59:15  lr: 0.000067  loss_tag: 398.1259  loss_dis: 0.3213  time: 1.1435  data: 0.1240  max mem: 24075 Train Epoch: [0]  [  250/14844]  eta: 4:53:46  lr: 0.000083  loss_tag: 440.8609  loss_dis: 0.3175  time: 1.1279  data: 0.1260  max mem: 24075 Train Epoch: [0]  [  300/14844]  eta: 4:49:10  lr: 0.000100  loss_tag: 388.8815  loss_dis: 0.2827  time: 1.1128  data: 0.1257  max mem: 24075 Train Epoch: [0]  [  350/14844]  eta: 4:45:09  lr: 0.000100  loss_tag: 360.2780  loss_dis: 0.2842  time: 1.0958  data: 0.1268  max mem: 24075 Train Epoch: [0]  [  400/14844]  eta: 4:41:33  lr: 0.000100  loss_tag: 429.6051  loss_dis: 0.2782  time: 1.0945  data: 0.1249  max mem: 24075 Train Epoch: [0]  [  450/14844]  eta: 4:38:28  lr: 0.000100  loss_tag: 360.5910  loss_dis: 0.2774  time: 1.0925  data: 0.1237  max mem: 24075 Train Epoch: [0]  [  500/14844]  eta: 4:36:15  lr: 0.000100  loss_tag: 388.7999  loss_dis: 0.2806  time: 1.1121  data: 0.1259  max mem: 24075 Train Epoch: [0]  [  550/14844]  eta: 4:34:38  lr: 0.000100  loss_tag: 336.6460  loss_dis: 0.2823  time: 1.1294  data: 0.1218  max mem: 24075 Train Epoch: [0]  [  600/14844]  eta: 4:33:22  lr: 0.000100  loss_tag: 335.7622  loss_dis: 0.2787  time: 1.1403  data: 0.1255  max mem: 24075 Train Epoch: [0]  [  650/14844]  eta: 4:32:21  lr: 0.000100  loss_tag: 497.8206  loss_dis: 0.2815  time: 1.1540  data: 0.1272  max mem: 24075 Train Epoch: [0]  [  700/14844]  eta: 4:31:29  lr: 0.000100  loss_tag: 1359.2052  loss_dis: 0.4506  time: 1.1637  data: 0.1278  max mem: 24075 Train Epoch: [0]  [  750/14844]  eta: 4:30:50  lr: 0.000100  loss_tag: 496.2915  loss_dis: 0.4714  time: 1.1740  data: 0.1257  max mem: 24075 Train Epoch: [0]  [  800/14844]  eta: 4:30:14  lr: 0.000100  loss_tag: 468.4954  loss_dis: 0.4744  time: 1.1810  data: 0.1311  max mem: 24075 Train Epoch: [0]  [  850/14844]  eta: 4:29:37  lr: 0.000100  loss_tag: 435.1667  loss_dis: 0.4665  time: 1.1759  data: 0.1290  max mem: 24075 Train Epoch: [0]  [  900/14844]  eta: 4:28:58  lr: 0.000100  loss_tag: 460.7809  loss_dis: 0.4758  time: 1.1790  data: 0.1286  max mem: 24075

It's common, since the reduction of tagging loss is sum().
https://github.com/xinyu1205/recognize-anything/blob/main/ram/models/utils.py#L326
Different loss scale is balanced with detach().
https://github.com/xinyu1205/recognize-anything/blob/main/pretrain.py#L121
Moreover, since RAM converges quickly, you can evaluate performance after training fewer epochs.

https://github.com/xinyu1205/recognize-anything/blob/main/pretrain.py#L121
@xinyu1205
I only have loss_tag and loss_dis. Do I need to balance these two losses?

If you want to highlight the open set tagging ability, I think balance is better.

@xinyu1205
Could you please provide the training log?  loss_t2t , loss_tag and loss_dis


support cert by ip for aws

:nut_and_bolt: Description: What code changed, and why?
need to support cert by ip on aws deployment
:chains: Related Resources
https://chefio.atlassian.net/browse/CHEF-2297
:+1: Definition of Done
:athletic_shoe: How to Build and Test the Change
following test is done on Aws deployment

[x] upgrade version
[x] cert rotate
[x] add cert
[ ] delete cert

:white_check_mark: Checklist
All PRs must tick these:

[ ] I have read the CONTRIBUTING document.
[ ] All commits signed-off for the Developer Certification of Origin.

With occasional exceptions, all PRs from Progress employees must tick these:

[ ] Is the code clear? (complicated code or lots of comments--subdivide and use well-named methods, meaningful variable names, etc.)
[ ] Consistency checked? (user notifications, user prompts, visual patterns, code patterns, variable names)
[ ] Repeated code blocks eliminated? (adapt and reuse existing components, blocks, functions, etc.)
[ ] Spelling, grammar, typos checked? (at a minimum use make spell in any component directory)
[ ] Code well-formatted? (indents, line breaks, etc. improve rather than hinder readability)

All PRs from Progress employees should tick these if appropriate:

[ ] Tests added/updated? (all new code needs new tests)
[ ] Docs added/updated? (all customer-facing changes)

Please add a note next to any checkbox above if you are NOT ticking it.
:camera: Screenshots, if applicable

Fix verify pipeline


STATISTICS_INFO event in mpegts.js, when the playback starts, the speed attribute is much smaller than the video frame size

The trigger time of STATISTICS_INFO has always been a mystery. When I play the live stream of http/flv, the trigger event is roughly once every 0.6s, and I have never understood what the speed value means. If it is bytes/s, it is too small, and even if it is kb/s, it is not large enough. My requirement is to monitor the real-time download volume of the URL, but it seems that the speed is not enough. How should I deal with it? Please help me, it will be very grateful!

By reading the source code, I found that the unit of the speed attribute is KB/S, and the interval for triggering the STATISTICS_INFO event can be configured through the statisticsInfoReportInterval attribute in the configuration, and the unit is milliseconds.


Ignore warnings on persist call in test_setitem_extended_API_2d_mask

Follow up to https://github.com/dask/dask/pull/9828; it looks like the numpy warning in question also gets emitted on the dx.persist() call (relevant test failure), so should probably also catch it there.
cc @jrbourbeau

[ ] Closes #xxxx
[ ] Tests added / passed
[ ] Passes pre-commit run --all-files

Hrm, it looks like the macOS Python 3.11 CI build is stalling for some reason. I think it's safe to ignore for now given the macOS Python 3.9 build passed and it would certainly be unrelated to the changes in this PR. I'll merge this in for now and keep an eye out to see if we see this happening elsewhere (I've not seen it anywhere else so far).
Also xref https://github.com/dask/dask/pull/9844


added: links translation

a elements were not translated

hello @Barbarararara, did you try version 1.3.6? As it should have been fixed by commit 7ab5c32fb3246c7a1869f3aa7e231ebd448ef848


config.json error

I put all the secrets according to the constants someone help me with this error

Hi!
@4mlsx Can you please share your config.json file with all the sensitive tokens removed so I can see what's wrong?

@4mlsx We'll close this out since this is old, please feel free to reopen if still experiencing this, but this is not locally reproducible for me.


Detect all keys of Logitech Y-BF37

Description of the new feature / enhancement
I came about a Logitech Internet Navigator keyboard (Y-BF37). A few of the special keys are not detected by Keyboard Manager.
The scan codes of such keys are:
0x90001 Messenger/SMS
0x90002 Webcam
0x90003 iTouch
0x90004 Shopping
F key functions with 'F-lock' key OFF
0xc0201 New (F1)
0xc0289 Reply mail (F2)
0xc028b Forward mail (F3)
0xc028c Send (F4)
0xc021a Undo (F5)
0xc0279 Redo (F6)
0xc0208 Print (F7)
0xc0207 Save (F8)
0xc0194 My Computer (F9)
0xc01a7 My Documents (F10)
0xc01b6 My Pictures (F11) ??
0xc01b7 My Music (F12) ??
Does Keyboard Manager need support at Windows driver level? Currently these keys do not register either as individual keys or as key combinations
Scenario when this would be used?
This is a weird vintage keyboard and Logitech never released its application/driver for 64bit Windows.
I would love to make full use of the Internet Navigator keyboard and being able to map ALL the special keys.
Supporting information
It's an USB keyboard, VID 046d, PID c309
Here is the patch to Gnome in 2013:
https://cgit.freedesktop.org/systemd/systemd/commit/?id=792d6163

we detect vkeys, if another app sends inputs and registers after us, we will see the bubble through. We currently don't know who sent what.
/dup #14120 and #14055 are the R&D items to see if we can't support this scenario easily.  based on that, we'll create an actual work item


trust-region optimization

Hi all,
Thanks for the awesome library. It would be fantastic to eventually see some options for trust-region optimization. For smaller dimensional problems in my field (e.g., variance components, or their generalization), it provides a much more robust approach for inference compared with 2nd order counterparts.

Indeed, trust-region methods would be nice to have but I didn't get the 'compared with 2nd order counterparts' comment because in my view trust region is mainly an alternative to line search. For instance, it would be great to have trustncg (Newton-CG + Trust region), which is an alternative to ncg (Newton-CG + Line search).

That's great to hear!
Re my comparison: I'm far from an expert, and my limited understanding was that Newton-CG + line search provides a direction first (Hessian- or approximate Hessian-modified gradient) and then tries to determine the magnitude using line search (or some fixed rate), whereas trust region first limits the magnitude based on the currently accepted trust radius, and then finds an appropriate direction which may or may not coincide with ncg (depending if optimal step is inside the ball). If things are convex then they should ultimately find the same optima, but for instances like variance components, which (I believe) are generally non-convex, trust-based approaches seem to be a bit better behaved.

This may be dead but through a minor object-oriented modification of OSQP you can support arbitrary convex spaces for which the support function as well as (projection onto) the recession and normal cones can be easily computed, which is most simple convex sets of interest (see Banjac et al. (2019) "Infeasibility Detection in the Alternating Direction Method
of Multipliers for Convex Optimization"). Then the Newton-CG algorithm can just call this general OSQP to solve the subproblem to the desired accuracy. This method can also be used to solve quadratically constrained QPs quite easily as well, with $A_i^T A_i = P_i$, etc.  I wrote an ADMM-based QP solver in Golang that I was using as part of several projects last year, though I lost the files around a month ago when my laptop was stolen. But a ConvexSet class shouldn't be too difficult to recreate.


Buggy border at some window sizes

I noticed while playing with the example that it would flicker noticeable while resizing.
Looking further it seems that at some sizes the border disappears / gets hidden.
It happens at all window sizes (big or small).
Any idea what's going on?
Buggy:

OK:

Hi @tritao!
I'm already seeing this, and other tasks at the same time :see_no_evil:  :smile: .
It's a bug in the multi_split_view used by this package. I'll resolve it there and update the dependency.

Hi @tritao!
Version 1.7.0 released.
If you need anything, let me know.


improvement: Add scroll lock to bottom of console output

Closes https://github.com/marimo-team/marimo/issues/826
// Keep scroll at the bottom if it is within 120px of the bottom,
// so when we add new content, it will lock to the bottom
// This won't handle large jumps in the scroll position
// if there is a lot of content added at once.

Oh that's a much better solution. I can actually get it to work with that. Thanks for the pointer


Add dynamically branch and commit number to post receive URLs

Is it possible to define post receive URL, like this :
/webhook/git/xxx?branch=<branch>&commit=<commit>

And let redmine git hosting plugin replace branch and commit keywords with the current branch and commit ?
For your information I try to use Redmine + Redmine Git Hosting + PHPCI .
The PHPCI code which handle web hook request is here :
https://github.com/Block8/PHPCI/blob/1385379f1a63e7bbf228e51f32470b94fb5c7dcb/PHPCI/Controller/WebhookController.php#L119
PHPCI can also handle message and committer.

Is it possible to define post receive URL, like this :

What for? Why not use PostReceivedUrls to handle this?

I want to use PostReceivedUrls, but how can I define branch/commit number keyword in URL to be replaced by git hosting plugin when the link is called ?

PostReceivedUrls send a payload like this one to the called url.


Please update Krailos items database

Krailos miscellaneous and items are without info (no price, no npc that buy it)
Thanks

@vinilodeon Tibialyzer uses the data from TibiaWiki. Please edit the TibiaWiki pages for the items you’re talking about and it will automatically be fixed in Tibialyzer on the next intake.

How can I do it, Mathias?

Which items do you mean exactly? I can do it for you.

Tibialyzer is not recognizing all new items and new creatures...

For example if Ogre Klubba information is incorrect or missing go to http://tibia.wikia.com/wiki/Ogre_Klubba, click edit, and fix it! Everyone can contribute to the wiki.

@mathiasbynens But this information is correct (example)
http://tibia.wikia.com/wiki/Ogre_Nose_Ring
however it's not showing in Tibialyzer who buys this item... just dropped from...

Older items are working well...

Thanks..

I will perform a new scan and release a new version with the updated database next weekend. If you want, you can manually update your database by running the python scripts in the "Database Scan" folder.


Add VpcAssociationPolicy doc

What type of PR is this?

Which issue does this PR fix:
What does this PR do / Why do we need it:
If an issue # is not available please add repro steps and logs from aws-gateway-controller showing the issue:
Testing done on this change:

Automation added to e2e:

Will this PR introduce any new dependencies?:

Will this break upgrades or downgrades. Has updating a running cluster been tested?:
Does this PR introduce any user-facing change?:

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

Haven't had a chance to review the doc yet, but please make sure any new doc change is included in mkdocs.yml to add them on the website.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
23 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.2%) to 46.204%

Files with Coverage Reduction
New Missed Lines
%

pkg/gateway/model_build_servicenetwork.go
3
86.96%

pkg/deploy/lattice/service_network_manager.go
20
88.31%

Totals

Change from base Build<PHONE_NUMBER>:
0.2%

Covered Lines:
3931

Relevant Lines:
8508

💛  - Coveralls

@xWink Thanks for your detailed comments! I think I should have addressed all of them, please review again when you have a time, thank you!


[RFR] Feature/examples ts typings

3 new commit based on PR #105 to be merge just after

@divyenduz I can rebase if you want once #105 is merged