Corrupted world from bclib 1.2.5 to 1.3.4 update

Hello, creator of Another Quality Modpack 2 here.
Recently, we updated the modpack to v2.0.5, including in this update was Better End (1.0.2 from 1.0.1), Better Nether (6.0.10 from 6.0.9), and BCLib (1.3.4 from 1.2.5).
When I released this, every player was crashing as soon as they explored Nether for more than 2 seconds. players from single player, on their own servers, our dedicated servers, and even locally when I tried to reproduce. We ran binary searches, removing mod by mod by mod, and nothing did it. The crash log said it was xaero minimap (was not updated), and then removing that, it was reporting classes when crashing. Xaero's Minimap wasn't touched in weeks, so I know its not this mod.
Long story short, I tried to figure out which mods and ultimately discovered it. I went on my world locally from the last AQM2 version, where it used 6.0.9 BN, 1.0.1 BE, 1.2.5 BCLib, and went into the nether to make sure there  wasn't a crash with these versions, and there wasn't. I then updated all three of those mods to their latest, and then I opened up the game, joined the world, and BCLib told me it needed to run patch fixes, so I let it do its thing.
When it finished, I joined the Nether, and I crashed. The same crash other people experienced from my 2.0.5 modpack update, but I wasn't on 2.0.5, I was on <IP_ADDRESS>, with just the three mods updated.
As requested, here is my world folder, and logs.
Note: please ignore the stoneholm message, I had removed stoneholm in my local instance, it works just fine without the structure mod, as I tested before I updated BN,BCLib,BE and had no issues with stoneholm being removed.
Latest Log (prior to updating the three mods, and playing just fine. Teleported to Nether, flew around for a minute or two and exited): https://gist.github.com/itsdinkd/82e987c1bac83f6f5a5dac732343f458
Latest Log (when updating the three mods to latest and being forced to apply fixes and crashing in nether): https://gist.github.com/itsdinkd/e40c9d06f38d32bac608697cb9560332
Latest Log (From downgrading the three mods back to what they were at before, after letting BCLib apply patches, and crashing in nether): https://gist.github.com/itsdinkd/802be0ccea03c36bfb7e018c17ae9fc3
World save: https://www118.zippyshare.com/v/0smZbiaT/file.html

Steps to reproduce

Download AQM2 <IP_ADDRESS> on Curseforge
Create world, join nether
Exit, Update BN + BE + BCLib to latest
Join Nether
fly around for 5 seconds
Crash

Cause of this issue was disabling ore in 1 file, blocks.json, and not the generator.json file. Wasn't aware that you needed to disable ore in both files. Also, having duplicate entries b being added to configs over time by updating, where the only difference is comments. I disabled ore on duplicated lines, and once cleared, and enabled ore, crash stopped.

As this is a BN issue and is not related to BClib I'm closing it


Replace physical proto folder by git submodule link

By linking folder to https://github.com/apache/incubator-skywalking-data-collect-protocol , we can share the unique proto definition files. If you are not familiar with git submodule, you should learn, because, you need to udpate the link by submodule update command, when the protocol updated for some reasons.

protos need to add package SkyWalking.NetworkProtocol; to generate namespace for C# code. Just like ApplicationRegisterService.proto#L8

Yes. I have used git submodule to update the links. Then run generate_protos.bat to re-generate the C # files , and run dotnet build to re-build the project ,an error occurred :
DiscoveryServiceGrpc.cs(13,12): error CS1001: Identifier expected [D:\skywalking-csharp\src\SkyWalking.NetworkProtocol\SkyWalking.NetworkProtocol.csproj]

So we  need to add package SkyWalking.NetworkProtocol; into proto definition files to fix this issue . e.g.
syntax = "proto3";

option java_multiple_files = true;
option java_package = "org.apache.skywalking.apm.network.proto";

import "KeyWithIntegerValue.proto";

package SkyWalking.NetworkProtocol;

...

Get your point. But by following grpc offical examples: https://github.com/grpc/grpc/blob/master/examples/protos/helloworld.proto#L22 ,  I prefer the package name stays in lower case. Such as skywalking.network.protocol.
What do you think?

And I check the generated source codes: https://github.com/grpc/grpc/blob/master/examples/csharp/helloworld-from-cli/Greeter/Helloworld.cs#L10
Look like the first letter of each package name will be upper.

The better way is explicitly provide an option csharp_namespace in proto files.  https://developers.google.com/protocol-buffers/docs/proto3#packages


[style]: GlassMorphism Navbar

What's the style idea?

Right now the navbar is not looking good
I will add glass morphic effect with some backdrop filter so that logo is also visible properly
Also the the border hover effect on navbar

Add screenshots
Sample (Its just for sample i can change according to you )
https://github.com/agamjotsingh18/codesetgo/assets/101205226/ca02321b-93f2-4c08-a612-09a60f221d88
Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I have read the Contributing Guidelines
[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSoC'23 contributor
[X] I want to work on this issue

Hey @agamjotsingh18 Please review it

Also isssue #595 is similar and not completed

Sure @parasss19


openssl: better check that withPerl=false prevents perl reference

Motivation for this change
This is a PR to fix #19965, making it possible to remove a large 50MB perl dependency from the closure of openssl.bin when using withPerl=false. The checks in openssl/default.nix weren't quite right before, and there was an unnecessarily restrictive assert preventing you from setting withPerl to false when not cross-compiling.
I wanted to do this so I could deploy nodejs-slim in a Docker container, without bringing in perl. I'll open a separate PR for that.
Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[X] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[X] Fits CONTRIBUTING.md.

So at this point I see
# head $(nix-build -E '((import ./. {}).openssl.override { withPerl = false; }).bin')/bin/c_rehash
#!/usr/bin/env perl

So there is no bug, other than the pesky assert (the withPerl option introduced in ef24a2815ed949c86110339e048cef2b33b84297 and immediately locked out for no apparent reason). We could just remove disallowedReferences and merge this to master.

Why not keep the disallowedReferences to keep this from regressing again? Looking at #19965 it seems like it has in the past.

Why not keep the disallowedReferences to keep this from regressing again? Looking at #19965 it seems like it has in the past.

If the goal is to prevent dependency on buildPackages.perl spliced from the nativeBuildInputs, then it's probably need to be spelled out in disallowedReferences explicitly (not just perl) and unconditionally with respect to withPerl. At that point we might as well set strictDeps = true;.

Also can we organize the two changes into separate commits (or even better, separate PRs)?

I only found a single passing reference to strictDeps in the nixpkgs manual which I don't fully understand, but happy to switch to it if you prefer. For now I just updated the disallowedReferences.

I just rebased on master, squashed and force pushed since the other PR landed. Sorry if I stepped on what you were doing with staging @veprbl. Also sorry for the noise other reviewers, I don't seem to have the permissions to remove the pending reviewer requests.

@ofborg eval

This change breaks native compilation of openssl, since in the case of withPerl and native compilation, it will still have the buildPackages.perl path in the result as that is the same as perl:
dev/nixpkgs - [604c9732c75●] » nix build --print-out-paths .#perl
warning: Git tree '/home/jade/dev/nixpkgs' is dirty
/nix/store/76a9zh8vvgcm4szrwvgvg0bfpnj23p7p-perl-5.34.1-man
/nix/store/6qkn235ap89fpq0w1nr7x2ipf5a0f9cy-perl-5.34.1
dev/nixpkgs - [604c9732c75●] » nix build --print-out-paths .#buildPackages.perl
warning: Git tree '/home/jade/dev/nixpkgs' is dirty
/nix/store/76a9zh8vvgcm4szrwvgvg0bfpnj23p7p-perl-5.34.1-man
/nix/store/6qkn235ap89fpq0w1nr7x2ipf5a0f9cy-perl-5.34.1

The question this brings up is whether we should set withPerl to false always to avoid causing unnecessary closure size in native-built docker containers for instance. One potential unfortunate ramification of this is that on not nix-darwin/nixos systems, /usr/bin/env perl is probably someone else's perl. Is this catastrophic? Probably not; I don't think that script does anything terribly advanced anyway.

Anything left to do on this PR? Would be great to have this merged.

A rebase would be a good start.

Rebase done!
I think the solution in https://github.com/NixOS/nixpkgs/issues/19965#issuecomment-1242707773 is better though.

Rebase done!
I think the solution in #19965 (comment) is better though.

Great, cause I already implemented it 😁
Will open a new PR soon.
Should I drop the withPerl flag completely, or set it to false by default and use the replacement script and then allow people to set it to true if they do want the Perl based script for some reason?

Should I drop the withPerl flag completely

AFAICT from Googling around, the native version is preferred for modern OpenSSL, so I'd vote drop it.

Should I drop the withPerl flag completely

AFAICT from Googling around, the native version is preferred for modern OpenSSL, so I'd vote drop it.

only noticed it now, I went for leaving it in with false as default: https://github.com/NixOS/nixpkgs/pull/219323


InfluxDb rc30 returns only columns and no values

Hi guys,
I get a response without any values when I do select * from localBankingEvent
{
    "results": [
        {
            "series": [
                {
                    "name": "localBankingEvent",
                    "columns": [
                        "time",
                        "Amount",
                        "BaseAmount",
                        "BaseBingo_BonusBalance",
                        "BaseBingo_BronzeBalance",
                        "BaseBingo_GoldBalance",
                        "BaseBingo_PointsBalance",
                        "BaseBingo_RechargePointsBalance",
                        "BaseBingo_SilverBalance",
                        "BaseCashBalance",
                        "BaseCasino_BonusBalance",
                        "Bingo_BonusBalance",
                        "Bingo_BronzeBalance",
                        "Bingo_GoldBalance",
                        "Bingo_PointsBalance",
                        "Bingo_RechargePointsBalance",
                        "Bingo_SilverBalance",
                        "BonusPromoCode",
                        "CashBalance",
                        "Casino_BonusBalance",
                        "CompletedAtUtc",
                        "CorrelationToken",
                        "Count",
                        "Currency",
                        "CurrencyCode",
                        "Description",
                        "Environment",
                        "InitiatedByAdmin",
                        "IsCompleted",
                        "LicenseHolder",
                        "Partner",
                        "RemoteRef",
                        "StartedAtUtc",
                        "TransactionId",
                        "TransactionType",
                        "UserName",
                        "Wallet",
                        "WalletVersion",
                        "site",
                        "timestamp",
                        "userId"
                    ]
                }
            ]
        }
    ]
}

I send the following JSON:
{
   "database":"testing",
   "points":[
      {
         "tags":{
            "bedeUserId":"1",
            "siteCode":"test"
         },
         "fields":{
            "Amount":100.00,
            "BaseAmount":100.00,
            "BonusPromoCode":"",
            "CompletedAtUtc":"2015-05-21T15:05:41Z",
            "Count":314,
            "CurrencyCode":"GBP",
            "Description":"",
            "InitiatedByAdmin":"False",
            "IsCompleted":"True",
            "RemoteRef":"",
            "StartedAtUtc":"2015-05-21T15:05:41Z",
            "TransactionType":"Deposit",
            "Wallet":"Cash",
            "UserName":"testplayer1",
            "TransactionId":"2148294039",
            "userId":"271661",
            "CorrelationToken":"GEN18c4adba-11f5-4404-846e-29235616a598",
            "timestamp":1432220741,
            "site":"test",
            "LicenseHolder":"IP&S",
            "Partner":"IP&S",
            "Currency":"GBP",
            "Environment":"local",
            "WalletVersion":2,
            "CashBalance":893230.51,
            "BaseCashBalance":893230.51,
            "Casino_BonusBalance":2761.12,
            "BaseCasino_BonusBalance":2761.12,
            "Bingo_BonusBalance":100.00,
            "BaseBingo_BonusBalance":100.00,
            "Bingo_PointsBalance":333.00,
            "BaseBingo_PointsBalance":333.00,
            "Bingo_GoldBalance":45.00,
            "BaseBingo_GoldBalance":45.00,
            "Bingo_SilverBalance":15.00,
            "BaseBingo_SilverBalance":15.00,
            "Bingo_BronzeBalance":0.00,
            "BaseBingo_BronzeBalance":0.00,
            "Bingo_RechargePointsBalance":0.00,
            "BaseBingo_RechargePointsBalance":0.00
         },
         "time":1432220741000,
         "name":"localBankingEvent"
      }]
}

When I do the same with another series it returns values.
What am I doing wrong?

Could this be because I have another series with the same tags?
"tags":{
"bedeUserId":"271661",
"siteCode":"test"
},

I can see tags when using show series.

@abeninskibede this is a dupe of #1997 - we should have this fixed in the next week or so.

@toddboom are you sure it is the same issue because apart from not getting tags in not getting column values - just column names?

Aggretages like sum(Amount) returns null:
{
    "results": [
        {
            "series": [
                {
                    "name": "localBankingEvent",
                    "columns": [
                        "time",
                        "count"
                    ],
                    "values": [
                        [
                            "1970-01-01T00:00:00Z",
                            null
                        ]
                    ]
                }
            ]
        }
    ]
}

Ah found the problem: was supplying time as milliseconds without specifying a precision.


Fix bug in np settings entry

Task/Issue URL: https://app.asana.com/0/488551667048375/1205310661579618/f
Description
See attached task description
Steps to test this PR
See attached task description

Current dependencies on/for this PR:

develop

PR #3482   👈

This comment was auto-generated by Graphite.


error parsing fragment

Hi,
I kept getting this during Jekyll Build and I couldn't see what is wrong
Remote Theme: Cleaning up /tmp/jekyll-remote-theme-20210603-1-1m256il
/usr/gem/gems/nokogiri-1.11.1-x86_64-linux/lib/nokogiri/xml/node.rb:840:in `in_context': error parsing fragment (1) (RuntimeError)

Not sure if this is a error or not because it's NOT in red. GH-page status is fine as well as first commit with _config.yml
Please help. Thanks

I cannot replicate this issue.  Also not enough information is given to reproduce this error.  Thanks


[zh-cn] update network-policies.md

名字空间 -> 命名空间

This change is not necessary.
This page has dozens of 名字空间 and you only changed one.
In fact, 名字空间 is preferred. Please refer to glossary in https://kubernetes.io/zh-cn/docs/contribute/localization_zh/
/hold

The 命名空间 that everyone has been calling，
Can you tell us why you want to change to 名字空间?
@tengqm
https://github.com/kubernetes/website/commit/d38315f27b32b0b356c6b5ce7c2837118cfadccb


Added report types to readme

stolen from "istanbul help reports"

Changes Unknown when pulling d4eb40e94aa53bab4a16974977dd94d59d77ae50 on 4uto:master into ** on istanbuljs:master**.

I escaped a , thanks Anna Henningsen

Changes Unknown when pulling 996707b1a237bfa0e8a2b760b10a664f81fd2d94 on 4uto:master into ** on istanbuljs:master**.

@4uto @addaleax sorry to be that nit-picky OSS maintainer, but I think I would rather see this new content in this README.md:
https://github.com/istanbuljs/istanbul-reports
With a section added to the nyc README that says, see a full list of reports here.
I think this would make it more likely that we keep documentation up to date as we add additional reports. What do you think?

@4uto just checking in, are you alright with this change?

@bcoe Sorry for the delay. My new baby is keeping me away from the computer.
Your thinking is sound. I was just under the impression that nyc was the new black and that istanbul wasn't "in the front" anymore. I'll go make the change over there and then come back here with an update like you suggested.

@4uto thanks, and babies are more important that JavaScript test coverage! 👶
My personal goal, is to keep the individual READMEs fairly concise, but to gradually flesh out http://istanbul.js.org/ further:
https://github.com/istanbuljs/istanbuljs.github.io


Connection stuck if first broker on the list is down

Issue:
let config = { brokers: ['broker1:9092', 'broker2:9092', 'broker3:9092'] }
assuming broker1:9092 is down, kafkajs will fail to connect to the cluster. But if the order
is :
let config = { brokers: ['broker2:9092', 'broker1:9092', 'broker3:9092'] }
or
let config = { brokers: ['broker3:9092', 'broker2:9092', 'broker1:9092'] }
The connection will work fine.
I'd expect kafkajs to be able to connect to the cluster in case one broker is down, and that broker is the first on the list, WDYT?

Update: it seems that it does manage to connect https://github.com/tulios/kafkajs/blob/master/src/broker/index.js#L79 but it get stuck on this call https://github.com/tulios/kafkajs/blob/master/src/broker/index.js#L82
which is related to the issue I currently have with this broker, so perhaps adding a timeout to this call will make the client more robust to this edge case

Right, we should enforce a timeout for the API versions call.
Thanks for investigation

I can try to make a PR to enhance this, do you have a rough example of how you'd like it to act?

You can pass a request timeout to any request
https://github.com/tulios/kafkajs/blob/326403fb096a097e821e03a5c780bb6e73064ba9/src/network/connection.js#L252
make apiVersions accept an optional requestTimeout
https://github.com/tulios/kafkajs/blob/326403fb096a097e821e03a5c780bb6e73064ba9/src/broker/index.js#L130
then always pass the connectionTimeout to apiVersions in the connect method
https://github.com/tulios/kafkajs/blob/326403fb096a097e821e03a5c780bb6e73064ba9/src/broker/index.js#L82
I am thinking about using the connectionTimeout because this is a fast request and it is part of the connection flow, WDYT?

sounds good, let me have a go at it

https://github.com/tulios/kafkajs/pull/369

Confirmed my side

Fixed by #369


fix(workflows): add ts dt

Changes

#943  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @MasterPtato and the rest of your teammates on  Graphite


Performance of Compreess side for AMD GPUs

I am using ZFP for 4MB buffer to compress on AMD GPUs and compared to the numbers that I have for NVIDIA there is a huge delay for compression side for compression rate of 8 and 1 dimension data. I was wondering if these numbers sound OK to you and if there is anything could be done to improve the poor compression side. Thanks.

Compression   Section
time in seconds

Decompress
time in seocnds

compress set-parameters
0.000024398

decompress set-parameters
0.000015471

opening zfp stream
0.000010577

opening zfp stream
0.000002414

pure compression time
1.539804423

decompress bitstream setting
0.000017298

compression dev sync
0.000595486

pure decpmpress time
0.188093374

clean up compress
0.000006698

clean up decpmpress
0.000002639

That does seem awfully slow (4 MB in over a second).  We are aware that our HIP implementation is quite a bit slower than the CUDA implementation, though we're still observing close to 20 GB/s compression throughput at a rate of 8 bits/value on an AMD MI60 when compressing 3D double-precision data.  By comparison, an NVIDIA V100 achieves 120 GB/s.
Are you able to share the data and (preferably) the code so we can take a closer look?  Does the uncompressed and compressed data both reside on the GPU?
@cjy7117 Do you have anything to add?

@pkoosha A brief update: We've done some work to improve the performance of the HIP implementation.  In particular, the high compression (but not decompression) overhead you're seeing is likely due to a GPU cold start, where the ROCM runtime performs lazy initialization during the first kernel launch.  We're now explicitly warming the GPU during zfp_stream_set_execution().  We also improved the performance of memory copies between host and device by pinning memory pages.  As a result, HIP and CUDA throughput should be roughly similar for comparable graphics cards.
The double-precision performance is still disappointing.  We're currently investigating the reasons for this and will provide an update once we know more.

Sweet! Thanks for the update! Let me know once you have the numbers and I am excited to see how it goes!
Best,
Pouya Kousha

From: Peter Lindstrom @.>
Sent: Thursday, February 17, 2022 12:44:37 PM
To: LLNL/zfp @.>
Cc: pkoosha @.>; Mention @.>
Subject: Re: [LLNL/zfp] Performance of Compreess side for AMD GPUs (#144)
@pkooshahttps://urldefense.com/v3/__https://github.com/pkoosha__;!!KGKeukY!hslyLcviO6XhfBy_X8qy2N9CL1JnFLeFboayDQGgr1pykrcBeIs-Qqfc1O9S_vpq$ A brief update: We've done some work to improve the performance of the HIP implementation. In particular, the high compression (but not decompression) overhead you're seeing is likely due to a GPU cold start, where the ROCM runtime performs lazy initialization during the first kernel launch. We're now explicitly warming the GPU during zfp_stream_set_execution(). We also improved the performance of memory copies between host and device by pinning memory pages. As a result, HIP and CUDA throughput should be roughly similar for comparable graphics cards.
The double-precision performance is still disappointing. We're currently investigating the reasons for this and will provide an update once we know more.
—
Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https://github.com/LLNL/zfp/issues/144*issuecomment-1043233536__;Iw!!KGKeukY!hslyLcviO6XhfBy_X8qy2N9CL1JnFLeFboayDQGgr1pykrcBeIs-Qqfc1DlT4bwE$, or unsubscribehttps://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AKOA4LHCV2DGCCYK3Z5S2VTU3UXYLANCNFSM5EH55ZDA__;!!KGKeukY!hslyLcviO6XhfBy_X8qy2N9CL1JnFLeFboayDQGgr1pykrcBeIs-Qqfc1L_BaD45$.
Triage notifications on the go with GitHub Mobile for iOShttps://urldefense.com/v3/__https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675__;!!KGKeukY!hslyLcviO6XhfBy_X8qy2N9CL1JnFLeFboayDQGgr1pykrcBeIs-Qqfc1MhFKj6_$ or Androidhttps://urldefense.com/v3/__https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign*3Dnotification-email*26utm_medium*3Demail*26utm_source*3Dgithub__;JSUlJSU!!KGKeukY!hslyLcviO6XhfBy_X8qy2N9CL1JnFLeFboayDQGgr1pykrcBeIs-Qqfc1Ho1-0jl$.
You are receiving this because you were mentioned.Message ID: @.***>

@lindstro thanks for these numbers, they look promising, could you please let me know which branch contains these and how can I ensure I have the best set of practices when running on AMD GPUs? Thank you very much!

This is on the staging branch.  The results are based on 3D fields of doubles (I believe one of the Miranda fields from SDRBench).  You should not have to do anything in particular on AMD hardware other than enable the HIP execution policy via zfp_stream_set_execution(stream, zfp_exec_hip).

@lindstro Thank you for the instructions on running ZFP with AMD support! We have collected some throughput numbers on mi100s and a100s with different thread block size.

We chose the Hurricane ISABEL single-precision dataset by issuing run commands: zfp -f -i Pf48.bin.f32 -3 500 500 100 -z Pf48.bin.f32.zfp -o Pf48.bin.f32.zfp.out -s
We also enabled ZFP internal profiling flag ZFP_WITH_CUDA_PROFILE for the numbers

We observed that

A100 doesn't have a noticeable advantage over MI100 either in encode (~180 GB/s vs ~170 GB/s) or decode (~130GB/s vs ~145GB/s)
Changing thread block size doesn't seem to impact performance a lot.

Do you have any comments? Is there any other direction that we should explore?

@R0n12 Thanks for sharing these results.  I assume you're using the latest staging branch?  I just tried to repeat this experiment using the default settings (without messing with the thread block size), and I'm recording 200 GB/s compression throughput on an older V100 at a rate of 1.  At a rate of 16, compression throughput is 70 GB/s.  Decompression throughput varies from 140 GB/s (rate = 1) to 62 GB/s (rate = 16).  That's quite a bit faster than what you're seeing, and on an older generation GPU.
I also have some more recent changes to the staging branch that have not been pushed yet, and those suggest even higher throughput at high rates, e.g., at rate = 16, compression throughput is 140 GB/s; decompression throughput is 82 GB/s.  These changes will eventually be merged into develop.
Regarding MI100, I could repeat those experiments with some effort, but I believe a more appropriate comparison would be with MI250.
I don't know if it matters, but I'm not using the -z or -o options as there's no need to write the (de)compressed data to file for these experiments.  I am using -s to force decompression, however.  And, of course, I'm running with -x cuda, which I'm sure you are, too.
Finally, throughput should be roughly twice as high if the input data were doubles.  Moreover, the 2D encoder (using -2 500 50000) on V100 is much faster than the 3D encoder; at a rate of 1, throughput is over 500 GB/s.  This likely has to do with register spillage.  Decode throughput, however, suffers.  We need to investigate these issues further before the next release.


Assistance for drawing trendlines

It would be good to have a tool that could assist with drawing trend lines. Something similar to that:
http://bl.ocks.org/andredumas/69f49097e9bb5c0c6e4d

we do but no documentation
Measure example in visual tests

I would say that a measure component is normally distinct from a trend line annotation although I do agree that they have similar functionality. The whole of the tool package is pretty badly documented but I think that reflects at whether we think they are appropriate implementations.

The measure example no longer exists but wasn't much more than an SVG line hooked up to mousedown/mouseup events. The linked example above is a more feature-rich version.


Simple Typo

"The following table lists the Windows editions that support Modern device management through (MDM):"
Should be:
The following table lists the Windows editions that support Modern device management  (MDM) through:

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

ID: 4d4dce4b-8a6a-a7e7-3fd7-1a86c2723010
Version Independent ID: 4d4dce4b-8a6a-a7e7-3fd7-1a86c2723010
Content: Mobile Device Management overview - Windows Client Management
Content Source: windows/client-management/mdm-overview.md
Product: windows-client
Technology: itpro-manage
GitHub Login: @vinaypamnani-msft
Microsoft Alias: vinpa

@vinaypamnani-msft , Created new PR https://github.com/MicrosoftDocs/windows-itpro-docs/pull/11754

Thanks @karlbo for catching that, and thanks @RAJU2529 (again 😄) for submitting the edit. The issue auto-closed when the PR merged, but the change won't be live until later today around 11:00 Pacific.


Kate - PROBLEMS WITH SONG-to-CULTURE LINKS IN SONG METADATA

Kate's notes:
I started looking at liner notes on the Smithsonian webpage when trying to understand why Shipibo-Conibo were considered different cultures. I did this by just clicking on the links in the "Sources" column of the SongMetadata sheet. I was surprised to find that the song with ID 897 that was assigned to "Shipibo-Conibo" in my dataset was assigned to "Cocama" on the Smithsonian page. I checked a few more surrounding songs, and found a handful of mismatches in the culture assigned to a song in my spreadsheet vs. the culture assigned to the song on the Smithsonian page. This is obviously a big concern, because it introduces a new level of error: coded data for a given song could be assigned to the wrong culture, making it seem like that culture has different types of songs than it does. I suggest someone go through and double check the culture for all songs for which we have a hyperlink - I only got through about 45. (Or, maybe the Smithsonian site is wrong?!?)
I suggest checking all those songs I haven't yet checked that have electronic liner notes -- it shouldn't take too long and would eliminate easy-to-correct mistakes re: attribution of cantometrics data to correct culture ID

CURRENT soc_id
song_id
CURRENT CULTURE
Sources
SHOULD BE ( ACCORDING TO SMITHSONIAN PAGE LINKED IN "SOURCES")
SHOULD HAVE CULTURE ID

13135
896
Asháninka
Notes by Harry Tschopik, Jr.,American Museum of Natural History; Willard Rhodes, Columbia University; http://www.folkways.si.edu/TrackDetails.aspx?itemid=11291
Shipibo-Conibo
14026

14026
897
Shipibo-Conibo
Notes by Harry Tschopik, Jr.,American Museum of Natural History; Willard Rhodes, Columbia University; http://www.folkways.si.edu/TrackDetails.aspx?itemid=11272
Cocama
13927

24091
2499
S Peruvian Quechua
This record is a two LP set.; http://www.folkways.si.edu/TrackDetails.aspx?itemid=28665
Q'ero
62301

24102
3378
Ch'unchu
http://www.folkways.si.edu/TrackDetails.aspx?itemid=14464
Mestizo
??62454

13927
9445
Cocama
Notes by Harry Tschopik, Jr.,American Museum of Natural History; Willard Rhodes, Columbia University; http://www.folkways.si.edu/TrackDetails.aspx?itemid=11288
Asháninka
13135

Additional note on sources: The hyperlinks to songs on the Folkways website that are in the "Sources" column of the metadata sheet don't seem to work anymore. But, all Cantometrics songs are linked to their sources through the source tags in the "Source Tag" column, which have hyperlinks in the GJB Data Sources spreadsheet . I didn't create the Sources column so I can't speak to its accuracy, but I created the Source Tags. In most cases they link to the whole album, not a specific song, but I feel pretty confident about their accuracy. (The specific song can be located via the archival source number, which gives the track number on the album). See my notes below about each of the specific examples, but it looks like in several cases someone put the link to the wrong song in the "Sources" column because multiple songs on the album from different cultures have the same name (ex. "Social Dance"). Some cases were genuine errors, which I have corrected. I would suggest deleting the hyperlinks in the Sources column because they don't work anymore and some of them are incorrect. Could also be nice to hyperlink the Source Tags to their URLs (when there is a URL). --Stella

Stella's notes:

|   | CURRENT soc_id | song_id
-- | -- | -- | --
Confirmed our society info is correct, no changes need to be made. Wrong song with same name was linked; I have removed hyperlink. --Stella | Issue36 | 13135 | 896
Confirmed our society info is correct, no changes need to be made. Wrong song with same name was linked; I have removed hyperlink. --Stella | Issue37 | 14026 | 897
Corrected to Q'ero (62301) and corrected song lat lon based on liner notes. --Stella | Issue38 | 24091 | 2499
Liner notes confirm this is indeed performed by Peruvian mestizos from Arequipa. I suggest either of two options: 1) renaming the society currently called "Lima" (62454) to "Peruvian Mestizo," (one of the 3 recordings currently associated with that society is already by Peruvian mestizos from Apurimac, not Lima), and assigning song 3378 to that society, or 2) if it is important to keep Lima mestizos separate, create a new Peruvian Mestizo society and assign 3378 and NC91c (the song from Apurimac) to it. --Stella | Issue39 | 24102 | 3378
Confirmed our society info is correct, no changes need to be made. Wrong song with same name was linked; I have removed hyperlink. --Stella | Issue40 | 13927 | 9445

Additional note on sources: The hyperlinks to songs on the Folkways website that are in the "Sources" column of the metadata sheet don't seem to work anymore. But, all Cantometrics songs are linked to their sources through the source tags in the "Source Tag" column, which have hyperlinks in the GJB Data Sources spreadsheet . I didn't create the Sources column so I can't speak to its accuracy, but I created the Source Tags. In most cases they link to the whole album, not a specific song, but I feel pretty confident about their accuracy. (The specific song can be located via the archival source number, which gives the track number on the album). See my notes below about each of the specific examples, but it looks like in several cases someone put the link to the wrong song in the "Sources" column because multiple songs on the album from different cultures have the same name (ex. "Social Dance"). Some cases were genuine errors, which I have corrected. I would suggest deleting the hyperlinks in the Sources column because they don't work anymore and some of them are incorrect. Could also be nice to hyperlink the Source Tags to their URLs (when there is a URL). --Stella

Re: song 3378, I decided to create new Arequipa society and assign to that, because it seems to be a distinct subculture from Lima.
All other song-specific issues here have been corrected.
Broken links in Sources field have been removed across all song metadata.

Yoingu as I recall was a designation for a blend of different groups, or
for them when they got together.  There should be a note on this in the
metadata, and there are sources
www.culturalequity.org
https://archive.culturalequity.org
www.theglobaljukebox.org
“The precise role of the artist is to illuminate that darkness, blaze
roads through that vast forest, so that we will not, in all our doing, lose
sight of its purpose, which is to make the world a more human dwelling
place."  --- James Baldwin
On Mon, Jan 10, 2022 at 9:10 AM stellasilbert @.***>
wrote:

Closed #63 https://github.com/theglobaljukebox/cantometrics/issues/63.
—
Reply to this email directly, view it on GitHub
https://github.com/theglobaljukebox/cantometrics/issues/63#event-5866682680,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUILTLMLGLBFUS4644KKNTLUVLZHDANCNFSM5E2ZISUQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were assigned.Message ID:
@.***>

Think maybe you meant to put your most recent comment on this thread:  theglobaljukebox/cantometrics#62. If possible, try to keep your comments on the relevant thread, so that they are easier to find and understand later should someone need to look back. But your suggestion sounds good! It looks like the note about Yolngu being an amalgamated group is already in the culture description in the metadata.


[Containerapp] az containerapp env update: Support to update --dapr-connection-string. Hide --dapr-instrumentation-key

Related command

Description

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Thank you for your contribution! We will review the pull request and get back to you soon.

History Notes
[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

Since there are multiple changes in this PR, please split the changelog into individual items and list them in the History Notes section

History Notes
[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

Since there are multiple changes in this PR, please split the changelog into individual items and list them in the History Notes section

Addressed comment


Link to actual latest ntttcp release

The link to download ntttcp points to what was the latest release at the time, but fixes have been made to ntttcp since then. Use github's special "latest" link so that the link doesn't become stale.
There is also an irrelevant link to the toplevel github page, which I'm sure people will be able to find from the release page if they are interested. I've removed it.

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

#label:"aq-pr-triaged"

#sign-off

Due to the upcoming Ignite conference, all merging and publishing is on hold until the morning of October 12.
This PR is ready for merge on 10/12.


Protein Embedding with last activation layers?

Is it possible to obtain the last activation values using AlphaFold?
Something like ESM allows with the model.forward method.

I would precise the question:
How can we execute AF2 pipeline to get fixed-length numeric vector which will represent single AA sequence?
If it is possible, could you add, should we expect that AA sequence length is limited?

Hi,
Although the ability to return the final representations/embeddings is not currently exposed in the RunModel container, it should be possible to enable it by adding a return_representations=True key-word argument here:
https://github.com/deepmind/alphafold/blob/d26287ea57e1c5a71372f42bf16f486bb9203068/alphafold/model/model.py#L64

I didn't run the actual model but I was using the jupyter notebook provided by @sokrypton. He suggested to edit class AlphaFold (located inalphafold/model/modules.py ) set return_representations=True.
In the jupyter notebook he provided,
prediction_result = model_runner.predict(processed_feature_dict)
gives 'prediction_result' as a dictionary with a key as 'representations'
prediction_result.keys()
dict_keys(['distogram', 'experimentally_resolved', 'masked_msa', 'predicted_lddt', 'representations', 'structure_module', 'plddt'])
this returns a nested dictionary and then
prediction_result['representations'].keys() outputs
dict_keys(['msa', 'msa_first_row', 'pair', 'single', 'structure_module'])
it contains the learned representations, although I am not sure which one to use. Hope it helps

@tfgg  Could you suggest which representation would be a good choice as an protein embedding for downstream tasks? since i get 5 different representations from the prediction result?

@tfgg
Is there any reason why this thread was closed? @xinformatics shared some tips but the main questions still haven't answered.
Thank you for considering.

@xinformatics The first section of the article The AlphaFold2 Method Paper: A Fount of Good Ideas suggests that s_i is the embedding you want to use. This would correspond to the single key in the prediction_result['representations'] dict.

At every step of the process, {s_i} is kept updated, communicating back and forth with {z_{ij}}, so that whatever is built up in {z_{ij}} is made accessible to {s_i}. As a result {s_i} is front and center in all the major modules. And at the end, in the structure module, it is ultimately {s_i}, not {z_{ij}}, that encodes the structure (where the quaternions get extracted to generate the structure). This avoids the awkwardness of having to project the 2D representation onto 3D space.

@rmeinl Thank you so much. I was thinking on the similar lines. Actually, the problem is my case is that I only need the representations (not the final PDB product) and somehow I am unable to figure out how to run AF2 prediction in a loop. I have 964 sequences and I wish to avoid running AF2 manually on each sequence.  The embedding extraction is available on my Github Alphafold

Ah interesting! I'm looking at a similar task. Two things I'll look at is 1) "turning off" the recycling step (doing a one-pass only) and 2) using only 1 of the models (instead of all 7 and then select the best scoring as they do in the provided AlphaFold.ipynb).
[...]
model_names = ['model_1', 'model_2', 'model_3', 'model_4', 'model_5', 'model_2_ptm']

[...]
for model_name in model_names:
   [...]

[...]
# Find the best model according to the mean pLDDT
best_model_name = max(plddts.keys(), key=lambda x: plddts[x].mean())

[...]

Hi @xinformatics,
I set return_representations=True  within alphafold/model/modules.py, relaunched the docker container, and ran the same experiment again. However, the feature.pkl is still the same. Could you please point out which jupyter notebook ColabFold use to generate the protein embedding?
Best,
Po-Yu


Extra/typo {% endif %}  breaking template.

https://github.com/Shopify/starter-theme/blob/cf7c2ad7ab830f4cfa441d2af19ef6afa4b9291c/src/sections/header.liquid#L34

Thanks @dturton for finding it !
It is a side-effect of https://github.com/Shopify/starter-theme/pull/70/files#diff-6d538607628fb9b7b84234d302df7e27L34


function create accepts random string as image

amp fn create command accepts a random string value as image and returns a function ID.
$ amp function create test foobar
2453ddc69201b35a447373a58fdd572c6841513923cd1f49f81a67f73f0e987b

Obsoleted by #1138


Fix check to exclude UDFs from being marked as async if binding does not exist

There was a mistake in following PR: https://github.com/microsoft/Power-Fx/commit/752efe46c476975a5433d9aab127f0b66808c187

✅ No public API change.


Can you please publish some example modules or a repo of modules?

Feel free to check out ours:
https://github.com/P5vc/ServerConfigurations
However, keep in mind that we use a slightly altered/custom implementation of aviary.sh that may not strictly stick to the model intended by team-video. that being said, our modules should still serve as decent examples.


not able to write to hostpath /etc/ambient-config/config.json.tmp in Bottlerocket os in EKS

Is this the right place to submit this?

[X] This is not a security vulnerability or a crashing bug
[X] This is not a question about how to use Istio

Bug Description
this issue is related to bug #45304 the Bottlerocket OS that we are using in AWS EKS  is quite secure and doesn't allow access to write in /etc dir but allows access for /var and /opt according to OS-Doc ,  but in istio-cni helm chart  the hostpath is hard-coded (line 185 in   CNI-daemonset.yaml  ) .
please could we add hostpath variable in CNI chart values to replace this hostpath value.
Version
$ istioctl version
1.18.0-rc.0
$ kubectl version --short
Client Version: v1.27.1
Kustomize Version: v5.0.1
Server Version: v1.25.9-eks-0a21954

Additional Information
No response
Affected product area

[X] Ambient
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
[ ] Upgrade
[ ] Multi Cluster
[ ] Virtual Machine
[ ] Control Plane Revisions

I see that #45404 was merged; can we close this issue?

Sure, #45405 should solve the file access issue , still facing the GitRepository failure may be istio was not able to configure iptables on host with Bottlerocket os in EKS. Will test and try to identify the exact issue.


error map？

Could you please explain how you generated the reconstruction images and error map in your paper?


Feat: Support for deeply nested relation filters + Issue: GraphQL input <Model>WhereUniqueInput shouldn’t include Relation fields

Fix WhereUniqueInput GraphQL input types to match Prisma's types
Fix detection of relation kinds
I haven't tested everything thoroughly but it seemed to work from my testing. You gave me the go-ahead to make a pull request so feel free to play around with these changes!

Seems to be working fine, so I've just merged it! Will be released as part of 1.0.0-rc.5. Thanks!


Migrate from maven to gradle

opendistro-for-elasticsearch/security pull request intake form
Please provide as much details as possible to get feedback/acceptance on your PR quickly

Category: (Enhancement, New feature, Bug fix, Test fix, Refactoring, Maintenance, Documentation)
Maintenance

Github Issue # or road-map entry, if available:

Description of changes:

Why these changes are required?
Security plugin has 2 build systems: Maven and Gradle(building .rpm and .deb packages). It's necessary to combine them into a single one(Gradle).

What is the old behavior before changes and new behavior after changes? (Please add any example/logs/screen-shot if available)

Testing done: (Please provide details of testing done: Unit testing, integration testing and manual testing)

Unit Tests
Manual Testing:

Ran ./gradlew clean build --no-daemon -Dbuild.snapshot=false -x test, artifacts are generated in gradle-build/distributions/
The following artifacts are generated:

opendistro_security-<IP_ADDRESS>.jar
opendistro-security-<IP_ADDRESS>.zip
opendistro-security-<IP_ADDRESS>-securityadmin-standalone.zip
opendistro-security-<IP_ADDRESS>-securityadmin-standalone.tar.gz

Installed security plugin generated by Gradle on ES. Securify configurations are loaded into elasticsearch-7.10.2/plugins/opendistro_security/securityconfig
Ran install_demo_configuration.sh, security settings are added into elasticsearch.yml file.
The plugin folder name in elasticsearch-7.10.2/plugins is opendistro_security

TO-DOs, if any: (Please describe pending items and provide Github issues# for each of them)

Is it backport from main branch? (If yes, please add backport PR # and commits #)

By making a contribution to this project, I certify that:
(a) The contribution was created in whole or in part by me and I
have the right to submit it under the open source license
indicated in the file; or
(b) The contribution is based upon previous work that, to the best
of my knowledge, is covered under an appropriate open source
license and I have the right under that license to submit that
work with modifications, whether created in whole or in part
by me, under the same open source license (unless I am
permitted to submit under a different license), as indicated
in the file; or
(c) The contribution was provided directly to me by some other
person who certified (a), (b) or (c) and I have not modified
it.
(d) I understand and agree that this project and the contribution
are public and that a record of the contribution (including all
personal information I submit with it, including my sign-off) is
maintained indefinitely and may be redistributed consistent with
this project or the open source license(s) involved.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Will re-open later if needed.


rgh-netiquette - Support new views

closes #8012

Test URLs
https://github.com/download-directory/download-directory.github.io/issues/74
Screenshot

I have tried bleow code and add some class for TimeLineItem, but it doesn't work.
export default function TimelineItem({
	className,
}: {
	className?: string;
}): JSX.Element {}

And it thorw an error: TypeError: Cannot read properties of undefined

I also check the dom-chef lib, I don't find anything useful for me.

Unlike React components, any attributes you pass to dom-chef JSX will automatically be added. For classes, they will be appended. So <TimelineItem className='something'> will just work.
However you most likely just want to change the TimelineItem itself since we do want it to stay compatible across all the features that use it (e.g. closing-remarks)
Wherever possible, when you make changes to components or classes, search the codebase for the keyword and you might be able to fix multiple features at once. e.g. like you did for color-fg-muted in https://github.com/refined-github/refined-github/pull/7313

Unlike React components, any attributes you pass to dom-chef JSX will automatically be added. For classes, they will be appended. So  will just work.

As you can see I add ml-md-0 to remove the margin, but it be overrode by ml-md-6. This is the reason why I want to have a customize classname here.

However you most likely just want to change the TimelineItem itself since we do want it to stay compatible across all the features that use it (e.g. closing-remarks)
Wherever possible, when you make changes to components or classes, search the codebase for the keyword and you might be able to fix multiple features at once. e.g. like you did for color-fg-muted in https://github.com/refined-github/refined-github/pull/7313

What's your opinion fro this way? It's a liitle bit wired for me.

As you can see I add ml-md-0 to remove the margin, but it be overrode by ml-md-6

Since the component needs to work in both layouts, then the right class needs to be applied depending on context. TimelineItem is very simple so you can just duplicate it and export a new version just for the new timeline, e.g.
export function TimelineItemOld(): JSX.Element {
	// Classes copied from #issuecomment-new + mt-3 added
	return
;
}

!! adjust/remove all classes as needed
export function TimelineItem(): JSX.Element {
	return
;
}

And then in the features you can use:
const Wrapper = yourLogicToDetectOldView ? TimelineItemOld : TimelineItem;

const banner = (
	<Wrapper>{createBanner({etc})}</Wrapper>
)

You can alternatively use the createBanner style instead of JSX, like:
const banner = createTimelineItem(
	yourLogicToDetectOldView ? 'old' : 'react',
	createBanner({etc})
)

Up to you

Wait, to me it looks like it's in the wrong container. The banner should not go under the sidebar for example, it should be part of the main thread column

Wait, to me it looks like it's in the wrong container. The banner should not go under the sidebar for example, it should be part of the main thread column

No, it just on the front of comment box.


Add missing GNOME core apps

Add 7 missing GNOME core apps from the official list of GNOME core apps: https://apps.gnome.org/ (first section)
These commits add:

GNOME Calculator
GNOME Clocks
GNOME Contacts
GNOME Disks
GNOME Fonts (Font Viewer)
GNOME Software
GNOME System Monitor

Thanks!


transactionName is undefined when finished on the useNotifications().notifications

I am using useNotifications() in order to get updates about transactions in my app. and i use the transactionName and it is undefined as shown

Hi @leviyehonatan, I've tried to reproduce your problem, but everything seems to work for me.
Do you remember to change the transactionName like this:
const { state, send } = useContractFunction(contract, 'deposit', { transactionName: 'Wrap' })


Error on running sudo sg_raw -s 40 -i password.bin /dev/sda c1 e1 00 00 00 00 00 00 28 00 command

Hi there
When i try to run this command i receive the following error:
SCSI Status: Check Condition -
Sense Information: Fixed format, current; Sense key: Illegal Request- Additional sense: Invalid command operation code".
I´m sure that dev is sda. Any idea?
Thanks.

I followed the procedure today, it worked just fine for me.

@cclcyber I have the same problem. What kernel version was your system? ie output of uname -a ? I have found that on older linux systems where dmesg | grep -i "scsi" or dmesg | grep -i "Type" shows the drive detected as "Type 0" will not work.
But on newer Linux kernels where it is detected as "Type 13", it will work.


Not Found - GET https://registry.npmjs.org/@types%2Fvue-grid-layout - Not found

vue3.0
下载@types/vue-grid-layout一直报错，更改了registry也没有用，这个是什么原因呢？
error：
npm ERR! code E404
npm ERR! 404 Not Found - GET https://registry.npmjs.org/@types%2Fvue-grid-layout - Not found
npm ERR! 404
npm ERR! 404  '@types/vue-grid-layout@*' is not in this registry.
npm ERR! 404
npm ERR! 404 Note that you can also install from a
npm ERR! 404 tarball, folder, http url, or git url.
npm ERR! A complete log of this run can be found in: C:\Program Files\nodejs\node_cache_logs\2023-07-07T02_19_52_211Z-debug-0.log

I think you are trying to download type definition for this library, but I don't think there is one in DefinitelyTyped.
So probably you have to write it by yourself.

ok，thinks
---- Replied Message ----
| From | @.> |
| Date | 07/10/2023 14:00 |
| To | jbaysolutions/vue-grid-layout @.> |
| Cc | 958264712 @.>,
Author @.> |
| Subject | Re: [jbaysolutions/vue-grid-layout] Not Found - GET @.***%2Fvue-grid-layout - Not found (Issue #764) |
I think you are trying to download type definition for this library, but I don't think there is one in DefinitelyTyped.
So probably you have to write it by yourself.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


HTTP/2 Prevent modification of activeStreams while iterating

Motivation:
The Http2Connection interface exposes an activeStreams() method which allows direct iteration over the underlying collection. There are a few places that make copies of this collection to avoid modification while iterating, and a few places that do not make copies. The copy operation can be expensive on hot code paths and also we are not consistently iterating over the activeStreams collection.
Modifications:

The Http2Connection interface should reduce the exposure of the underlying collection and just expose what is necessary for the interface to function.  This is just a means to iterate over the collection.
The DefaultHttp2Connection should use this new interface and protect it's internal state while iteration is occurring.

Result:
Reduction in surface area of the Http2Connection interface.  Consistent iteration of the set of active streams.  Concurrent modification exceptions are handled in 1 encapsulated spot.

@nmittler @buchgr - FYI.

Fixes https://github.com/netty/netty/issues/3550.

@Scottmitch looks fantastic, great job!  Just a couple minor nits.

@buchgr - Did you want to take a look...I think your PR https://github.com/netty/netty/pull/3516 has some iteration in it.  I can wait for you to finish that PR and absorb the changes if you want?

@Scottmitch that would be great. thanks! I ll update my PR later today. I ll have a look at your PR tonight as well. Does that work for you?

@buchgr works for me!

@Scottmitch great job! I like it.

@nmittler @buchgr - Care to take another look?

@Scottmitch just one comment. LGTM.

@Scottmitch looks awesome! just one minor question ... otherwise LGTM. Feel free to cherry-pick.

@buchgr @nmittler - WDYT about removing the interface ActiveStream?

@Scottmitch you could also just pass in the listeners right?

@buchgr - And the removal policy I guess...

@Scottmitch still LGTM

@Scottmitch awww fine :D LGTM

Alright I'll cherry-pick after I pull in @buchgr's https://github.com/netty/netty/pull/3516 PR.

Thanks for the review everyone!
Cherry-picked 4.1 (83ce8a9) master (28bd5a5).


Momentum Panning

Hi @timmywil.
Firstly, thank you for saving my life with this library.
Unfortunately, it's not "there" yet. On mobile, users are used to "overscroll bounce" and "momentum scrolling" effects. Wondering if we can figure out a way to implement this in this library?
Thanks so much again.
Daryl

Second request for this. I'll look into it.

The way I implemented it was to use another library called PEP which gives you a nice elastic option with drag easing then just disable panning
Pep homepage
img.panzoom({ 	disablePan: true });
Example here DEMO

Thanks, but that's not quite what we're after. There's no overscroll bounce.


Add api to allow deletion of items older than a provided timestamp

To allow newsfeed cleanup, e.g. on a scheduled basis select users and remove feed items > 30 days old.

@dmorgantini can you take a look?

@cliftonc I'm not certain that this provides the value we are looking for.  Presuming that we want to be able to reduce the number of posts that we are storing, we need to be able to do this across millions of users rather than for an individual user.

Thats the next step - I'm taking this in small steps.


Regression on 0.6.3: could not find schema due to Draft 4 being used

Hi, trying 0.6.3 I noticed a regression, caused by #221
with 0.6.3 it breaks
❯ ~/Downloads/kubeconform-063/kubeconform -debug -strict -schema-location target/kubernetes-json-schema/master target/test.yaml
2023/07/18 20:04:30 using schema found at target/kubernetes-json-schema/master/master-standalone-strict/verticalpodautoscaler-autoscaling-v1beta2.json
target/test.yaml - VerticalPodAutoscaler toggleservice-vpa failed validation: could not find schema for VerticalPodAutoscaler

with 0.6.2 it worked fine
❯ ~/Downloads/kubeconform-062/kubeconform -debug -strict -schema-location target/kubernetes-json-schema/master target/test.yaml
2023/07/18 20:04:34 using schema found at target/kubernetes-json-schema/master/master-standalone-strict/verticalpodautoscaler-autoscaling-v1beta2.json

what happens is that the schema is not valid in draft 4 but it is in the default 2020 schema
2023/07/18 20:28:18 error jsonschema file:///.../target/kubernetes-json-schema/master/master-standalone-strict/verticalpodautoscaler-autoscaling-v1beta2.json compilation failed: '/properties/spec/required' does not validate with http://json-schema.org/draft-04/schema#/properties/properties/additionalProperties/$ref/properties/required/$ref/minItems: minimum 1 items required, but found 0 items

from https://github.com/yannh/kubeconform/issues/217#issuecomment-1635675208 I assume we need to fix our schemas to be Draft 4 ?

if that's the case it would be good to push this to a 0.7.0 version at least and have a changelog warning

i guess it should use the draft version specified in the schema and only fallback to draft 4 🤔

I've tried with some public schemas and it works, so I'll change ours.
It would be good to have more debug entries inside downloadSchema. I see that the debug option is not available there to conditionally print logs
❯ ~/dev/yannh/kubeconform/bin/kubeconform -debug -strict -schema-location 'https://raw.githubusercontent.com/zapier/kubechecks/main/schemas/v1.23.0/{{ .ResourceKind }}{{ .KindSuffix }}.json' target/test.yaml
2023/07/18 20:44:08 using schema found at https://raw.githubusercontent.com/zapier/kubechecks/main/schemas/v1.23.0/verticalpodautoscaler-autoscaling-v1beta2.json

@yannh I cannot find many sources of autoscaler or its swagger definition, do you know where to get them? maybe to contribute them to https://github.com/yannh/kubernetes-json-schema/

I agree it would be good to have a way to have debug info in downloadschema, but I ve found it can be really verbose since well, by architecture, when you use multiple schema repositories it will try to find every file in every repository until it finds it...

@carlossg any chance you could share the schema so I could reproduce the original issue? I feel like Kubeconform should complain if it downloads an invalid schema!

it was this one I think
{
  "type": "object",
  "properties": {
    "spec": {
      "type": "object",
      "required": [],
      "properties": {
        "targetRef": {
          "type": "object"
        },
        "updatePolicy": {
          "type": "object",
          "properties": {
            "minReplicas": {
              "type": "integer"
            },
            "updateMode": {
              "type": "string"
            }
          },
          "additionalProperties": false
        },
        "resourcePolicy": {
          "type": "object",
          "properties": {
            "containerPolicies": {
              "type": "array",
              "items": {
                "type": "object",
                "properties": {
                  "containerName": {
                    "type": "string"
                  },
                  "controlledValues": {
                    "type": "string",
                    "enum": [
                      "RequestsAndLimits",
                      "RequestsOnly"
                    ]
                  },
                  "mode": {
                    "type": "string",
                    "enum": [
                      "Auto",
                      "Off"
                    ]
                  },
                  "minAllowed": {
                    "type": "object"
                  },
                  "maxAllowed": {
                    "type": "object"
                  },
                  "controlledResources": {
                    "type": "array",
                    "items": {
                      "type": "string",
                      "enum": [
                        "cpu",
                        "memory"
                      ]
                    }
                  }
                },
                "additionalProperties": false
              }
            }
          },
          "additionalProperties": false
        }
      },
      "additionalProperties": false
    }
  }
}

Here's the VPA CRD that will validate with kubeconform v0.6.3+ : https://github.com/kubernetes/autoscaler/blob/master/vertical-pod-autoscaler/deploy/vpa-v1-crd-gen.yaml

:wave:  My understanding is that the schemas should be in draft 4... Am I mistaken?  See https://github.com/yannh/kubeconform/issues/217#issuecomment-1635675208  - I would close this ticket unless a lot of public CRDs aren't in that format?

Hey guys, I'd like to report something that is somewhat related. It might help other users who struggled with this like we did.
Starting with v0.6.3 we got the "could not find schema" for an RDSInstance resource:
RDSInstance dcpc-commerce-markets failed validation: could not find schema for RDSInstance

I checked with:

v0.6.2 - ok
v0.6.3 - not found
v0.6.4 - not found
v0.6.5 - not found
v0.6.6 - not found

In our case this is a crossplane resource and the CRD is managed by a different team. I converted the CRD spect to json-schema format and added the validation fields:
  {
+   "id": "http://json-schema.org/draft-04/schema#",
+   "$schema": "http://json-schema.org/draft-04/schema#",
    "properties": {
      "apiVersion": {
        "type": "string"
      },
      "kind": {
        "type": "string"
      },
  [...]

As it turns out, there was an enum list with a duplicate in it:

As soon as the duplicate was removed the could not find schema error disappeared and the validation started working on all recent versions:
Before
KUBECONFORM [0.6.2]
2024/06/10 16:22:47 using schema found at /Users/megamorf/projects/myapp/app-k8s-crds-jsonschemas/caas-v3/schemas/aws.contoso.com/rdsinstance_v1alpha1.json
/Users/megamorf/projects/myapp/app-infrastructure/clusters/app-dev/rdsinstance-dcpc-commerce-markets.yaml - RDSInstance dcpc-commerce-markets is valid
Summary: 3 resources found in 1 file - Valid: 3, Invalid: 0, Errors: 0, Skipped: 0

KUBECONFORM [0.6.3]
2024/06/10 16:22:47 using schema found at /Users/megamorf/projects/myapp/app-k8s-crds-jsonschemas/caas-v3/schemas/aws.contoso.com/rdsinstance_v1alpha1.json
/Users/megamorf/projects/myapp/app-infrastructure/clusters/app-dev/rdsinstance-dcpc-commerce-markets.yaml - RDSInstance dcpc-commerce-markets failed validation: could not find schema for RDSInstance
Summary: 3 resources found in 1 file - Valid: 2, Invalid: 0, Errors: 1, Skipped: 0

KUBECONFORM [0.6.4]
2024/06/10 16:22:47 using schema found at /Users/megamorf/projects/myapp/app-k8s-crds-jsonschemas/caas-v3/schemas/aws.contoso.com/rdsinstance_v1alpha1.json
/Users/megamorf/projects/myapp/app-infrastructure/clusters/app-dev/rdsinstance-dcpc-commerce-markets.yaml - RDSInstance dcpc-commerce-markets failed validation: could not find schema for RDSInstance
Summary: 3 resources found in 1 file - Valid: 2, Invalid: 0, Errors: 1, Skipped: 0

KUBECONFORM [0.6.5]
2024/06/10 16:22:47 using schema found at /Users/megamorf/projects/myapp/app-k8s-crds-jsonschemas/caas-v3/schemas/aws.contoso.com/rdsinstance_v1alpha1.json
/Users/megamorf/projects/myapp/app-infrastructure/clusters/app-dev/rdsinstance-dcpc-commerce-markets.yaml - RDSInstance dcpc-commerce-markets failed validation: could not find schema for RDSInstance
Summary: 3 resources found in 1 file - Valid: 2, Invalid: 0, Errors: 1, Skipped: 0

KUBECONFORM [0.6.6]
2024/06/10 16:22:47 using schema found at /Users/megamorf/projects/myapp/app-k8s-crds-jsonschemas/caas-v3/schemas/aws.contoso.com/rdsinstance_v1alpha1.json
/Users/megamorf/projects/myapp/app-infrastructure/clusters/app-dev/rdsinstance-dcpc-commerce-markets.yaml - RDSInstance dcpc-commerce-markets failed validation: could not find schema for RDSInstance
Summary: 3 resources found in 1 file - Valid: 2, Invalid: 0, Errors: 1, Skipped: 0

After
KUBECONFORM [0.6.2]
2024/06/10 16:22:47 using schema found at /Users/megamorf/projects/myapp/app-k8s-crds-jsonschemas/caas-v3/schemas/aws.contoso.com/rdsinstance_v1alpha1.json
/Users/megamorf/projects/myapp/app-infrastructure/clusters/app-dev/rdsinstance-dcpc-commerce-markets.yaml - RDSInstance dcpc-commerce-markets is valid
Summary: 3 resources found in 1 file - Valid: 3, Invalid: 0, Errors: 0, Skipped: 0

KUBECONFORM [0.6.3]
2024/06/10 16:22:47 using schema found at /Users/megamorf/projects/myapp/app-k8s-crds-jsonschemas/caas-v3/schemas/aws.contoso.com/rdsinstance_v1alpha1.json
/Users/megamorf/projects/myapp/app-infrastructure/clusters/app-dev/rdsinstance-dcpc-commerce-markets.yaml - RDSInstance dcpc-commerce-markets is valid
Summary: 3 resources found in 1 file - Valid: 3, Invalid: 0, Errors: 0, Skipped: 0

KUBECONFORM [0.6.4]
2024/06/10 16:22:47 using schema found at /Users/megamorf/projects/myapp/app-k8s-crds-jsonschemas/caas-v3/schemas/aws.contoso.com/rdsinstance_v1alpha1.json
/Users/megamorf/projects/myapp/app-infrastructure/clusters/app-dev/rdsinstance-dcpc-commerce-markets.yaml - RDSInstance dcpc-commerce-markets is valid
Summary: 3 resources found in 1 file - Valid: 3, Invalid: 0, Errors: 0, Skipped: 0

KUBECONFORM [0.6.5]
2024/06/10 16:22:47 using schema found at /Users/megamorf/projects/myapp/app-k8s-crds-jsonschemas/caas-v3/schemas/aws.contoso.com/rdsinstance_v1alpha1.json
/Users/megamorf/projects/myapp/app-infrastructure/clusters/app-dev/rdsinstance-dcpc-commerce-markets.yaml - RDSInstance dcpc-commerce-markets is valid
Summary: 3 resources found in 1 file - Valid: 3, Invalid: 0, Errors: 0, Skipped: 0

KUBECONFORM [0.6.6]
2024/06/10 16:22:47 using schema found at /Users/megamorf/projects/myapp/app-k8s-crds-jsonschemas/caas-v3/schemas/aws.contoso.com/rdsinstance_v1alpha1.json
/Users/megamorf/projects/myapp/app-infrastructure/clusters/app-dev/rdsinstance-dcpc-commerce-markets.yaml - RDSInstance dcpc-commerce-markets is valid
Summary: 3 resources found in 1 file - Valid: 3, Invalid: 0, Errors: 0, Skipped: 0


MAT-6864: Bump up madie-design-system

MADiE PR
Jira Ticket: MAT-6864
(Optional) Related Tickets:
Summary
All Submissions

[ ] This PR has the JIRA linked.
[ ] Required tests are included.
[ ] No extemporaneous files are included (i.e Complied files or testing results).
[ ] This PR is merging into the correct branch.
[ ] All Documentation needed for this PR is Complete (or noted in a TODO or other Ticket).
[ ] Any breaking changes or failing automations are noted by placing a comment on this PR.

DevSecOps
If there is a question if this PR has a security or infrastructure impact, please contact the Security or DevOps engineer assigned to this project to discuss it further.

[ ] This PR has NO significant security impact (i.e Changing auth methods, Adding a new user type, Adding a required but vulnerable package).
[ ] All CDN/Web dependencies are hosted internally (i.e MADiE-Root Repo).

Reviewers
By Approving this PR you are attesting to the following:

Code is maintainable and reusable, reuses existing code and infrastructure where appropriate, and accomplishes the task’s purpose.
The tests appropriately test the new code, including edge cases.
If you have any concerns they are brought up either to the developer assigned, security engineer, or leads.

closed as the needed changes for the JIRA appear to have been picked up with another PR


Discord showing non-rich-presenced "Atom Editor" as game even after closing Atom

So, when having Atom open, everything works fine and Discord shows the rich presence perfectly (It looks pretty awesome, by the way, thanks for making this :D)
But once Atom is closed, and atom-discord updates, Discord shows "playing Atom Editor", with no rich presence, even though I'm not having Atom open.
Sometimes, after having closed Atom, there's a brief moment Discord shows that I'm not playing anything at the moment (like it should), but then it comes back with no rich presence, like described above.
I should also add that after restarting Discord, everything is normal again (until I start Atom again but I'm repeating myself)
Here are some pics:

Having Atom open

No Atom open whatsoever
Eh, don't worry about my profile picture by the way, everything's fine

#11

Woups, I'm sorry <3

That's okay :D


Problem save CPU profile in vs code 1.6.1

VSCode Version: 1.6.1
OS Version: Windows 8.1 Enterprise 64-bit

Steps to Reproduce:
New VS Code now is update but I have problem to save CPU profile on version 1.6.0 and 1.6.1.
When I Start CPU profile and stop, profile is created but when I save this, see zero (empty) size file saved!
I have only 1 extension (beautify).

This is a chrome bug which should hopefully get fixed soon and which we would get with an electron update in the future.
@bpasero can provide more details if needed

Yup, got fixed in newer electron releases. We can keep this open to track the issue.

electron/electron#7403


[BUG]: Results Page - Improper State

Description
Problem
Improper state usage. Conflict with Tooltip's content rendering and main compoent rendering.

(optional) What browsers are you seeing the problem on?
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

assigning to myself


⚠ Bofrost.at has degraded performance

In fcefa08, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1390 ms

Resolved: Bofrost.at performance has improved in 9e5b37a.


CLI doesn't seem to create TODO.x.yml

I used this project many times.
Since why, I am creating this issue.
I had forgotten what extensions were used for the YAML, .yml or .yaml.
So I installed the CLI and tried both with always only one of the list of available locales, but never did it create the appropriate TODO file, and I still wonder why.
Here are the files I created.
locales/app.yml
_version: 2
introduction:
  en: |
    Welcome! It appears to be your first time using our services.

    It is a pleasure to meet you.

    Before we can collaborate and grant you access to our services, please take the time to review our terms and conditions and privacy policy.

    Before accepting, we recommend reading our policies.

Cargo.toml
[package]
name = "purpura_handler"
version = "0.1.0"
edition = "2021"

[package.metadata.i18n]
available-locales = ["en", "de", "es", "fr", "it", "ja", "ko", "pt", "ru", "zh"]
default-locale = "en"

[dependencies]
chrono = { version = "0.4.32", features = ["serde"] }
lapin = { version = "2.3.1", features = ["codegen", "vendored-openssl"] }
rust-i18n = "3.0.1"
scylla = { version = "0.11.1", features = ["chrono", "time"] }
serde = { version = "1.0.195", features = ["derive"] }
serde_yaml = "0.9.30"
serenity = { version = "0.12.0", features = ["full"] }
totp-rs = { version = "5.5.1", features = ["qr", "serde", "zeroize", "rand"] }
tracing = { version = "0.1.40", features = ["async-await"] }

Thank you for having read this.
Have a great day.

The TODO file is extracted from the Rust source code by searching t! method.

Thanks a lot.


Add parentheses to function definitions

The revision adds parentheses to the 'now' function in heartbeat.ex and the 'submit_memory' function in system.ex files. This is to comply with the Elixir style guide which recommends defining functions with parentheses, even when they don't have arguments.

@radwo @hamir-suspect I'm having issues running front with the latest versions of elixir because of this functions without parentheses, so I've made this change, thoughts?


'errno.h' and 'assert.h' files not found when building on OS X

I get errors when running:
npm install
script/bootstrap
script/buid
Ex, for script/bootstrap
Node: v0.12.0
npm: v1.4.28
Installing build modules...
child_process: customFds option is deprecated, use stdio instead.
In file included from ../src/main.cc:1:
In file included from ../node_modules/nan/nan.h:177:
In file included from /Users/claudio/.node-gyp/0.12.0/deps/uv/include/uv.h:48:
/Users/claudio/.node-gyp/0.12.0/deps/uv/include/uv-errno.h:25:10: fatal error: 'errno.h' file not found
#include <errno.h>
...

or npm install
...
><EMAIL_ADDRESS>install /Users/claudio/Sites/test/atom-shell-starter/node_modules/runas
> node-gyp rebuild

child_process: customFds option is deprecated, use stdio instead.
  CXX(target) Release/obj.target/runas/src/main.o
In file included from ../src/main.cc:1:
In file included from ../node_modules/nan/nan.h:23:
In file included from /Users/claudio/.node-gyp/0.12.0/deps/uv/include/uv.h:48:
/Users/claudio/.node-gyp/0.12.0/deps/uv/include/uv-errno.h:25:10: fatal error: 'errno.h' file not found
#include <errno.h>
...

Do you have Xcode installed?

I didn't.
May I suggest that you add a section on the Readme named "Requirements" listing what one should have installed before attempting to use any of the builder scripts?

I have install Xcode, and now I get another error.
I run npm install and it succeeds.
Then I run script/bootstrap and it fails:
Node: v0.12.0
npm: v1.4.28
Installing build modules...
Installing apm...
dyld: lazy symbol binding failed: Symbol not found: _node_module_register
  Referenced from: /Users/claudio/Sites/tmp/atom-shell-starter/apm/node_modules/atom-package-manager/node_modules/keytar/build/Release/keytar.node
  Expected in: dynamic lookup

dyld: Symbol not found: _node_module_register
  Referenced from: /Users/claudio/Sites/tmp/atom-shell-starter/apm/node_modules/atom-package-manager/node_modules/keytar/build/Release/keytar.node
  Expected in: dynamic lookup

/Users/claudio/Sites/tmp/atom-shell-starter/apm/node_modules/atom-package-manager/bin/apm: line 40: 29689 Trace/BPT trap: 5       "$binDir/$nodeBin" --harmony_collections "$binDir/../lib/cli.js" "$@"
i

I also tried to script/clean and repeat script/bootstrap, but to no avail.

Sorry for all the trouble - I think your node.js version needs to be 0.10.x, not the one that was just released

Thanks, I'll try again with node 0.10.x.
On 16/02/15 23:01, Paul Betts wrote:

Sorry for all the trouble - I think your node.js version needs to be
0.10.x, not the one that was just released
—
Reply to this email directly or view it on GitHub
https://github.com/atom/atom-shell-starter/issues/43#issuecomment-74585918.

Ok, I'm using node 0.10.36 now, and Pyton 2.7.
script/bootstrapruns fine.
But script/build fails at the grunt task:
Running "build-atom-shell" task
>> Traceback (most recent call last):
>>   File "script/bootstrap.py", line 127, in <module>
>>     sys.exit(main())
>>   File "script/bootstrap.py", line 35, in main
>>     update_atom_modules('spec')
>>   File "script/bootstrap.py", line 77, in update_atom_modules
>>     execute_stdout([apm, 'install'])
>>   File "/private/var/folders/4j/knpk5tyj5kd5vs8hqkg62wwr0000gn/T/eightoheight-build/atom-shell/script/lib/util.py", line 150, in execute_stdout
>>     execute(argv)
>>   File "/private/var/folders/4j/knpk5tyj5kd5vs8hqkg62wwr0000gn/T/eightoheight-build/atom-shell/script/lib/util.py", line 132, in execute
>>     output = subprocess.check_output(argv, stderr=subprocess.STDOUT)
>>   File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/subprocess.py", line 566, in check_output
>>     process = Popen(stdout=PIPE, *popenargs, **kwargs)
>>   File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/subprocess.py", line 709, in __init__
>>     errread, errwrite)
>>   File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/subprocess.py", line 1326, in _execute_child
>>     raise child_exception
>> OSError: [Errno 2] No such file or directory

What now?

@claudio-silva I think you got a half-baked eightoheight-build directory, run bootstrap again

Yes, that fixed it.
I've successfully built atom-shell.
Thank you very much for your help.
Just a couple of additional questions:

To install the application on a user's computer, which files must be installed?
For ex. in the demo case, is it:

EightOhEight.app
EightOhEight Helper.app

and/or anything more?

Is it really 108Mb just for an empty app? I'm shocked...

@claudio-silva You don't have to think about the helper, you just copy EightOhEight.app over

Is it really 108Mb just for an empty app? I'm shocked...

Apple routinely sends down 1GB+ updates, let's keep it in perspective :)  Zipped it will come to ~40MB

Indeed! :-)
I just wished there was a lighter solution for developing desktop apps with web technologies.
 I'll try to work on making Atom Shell easier to build.

Great!
I'm still evaluating my options to find out what is the easiest, simplest, lightest, fastest way to create apps using web tech.
I like (and use) Atom and I'm looking forward to using atom-shell-start on my projects.
Too bad it doesn't work for mobile. I guess I'll have to use Cordova for that, but it may be difficult to make the same application run on both platforms seamlessly.
What are your thoughts on that?

@claudio-silva Atom Shell probably won't ever end up running on Mobile because of iOS, but you could probably pretty easily reimplement the Cordova APIs you need in Atom Shell so you wouldn't have to have as many #IFs in your code

@paulcbetts Thanks for the tip! It could be a very viable solution, if I just map calls from one API to the other.


bazel run with --run_under can hang

I've noticed that bazel run --run_under will never return under some circumstances.  The test I run has two genrule dependencies.  When these do non-trivial things, or if there's a large delay (overnight) the bazel command line program ('bazel run') will not return after the --run_under executable has finished.  strace shows it blocked reading from file descriptor 4 or 5.  The bazel daemon itself is sleeping on a futex.

It appears the issue may be caused by SSH's ControlPersist feature, which persists SSH connections for faster reconnect time.  If the script executed by --run_under creates the initial SSH connection, it will hang, very likely because of a bug in OpenSSH which does not close stderr:
https://bugzilla.mindrot.org/show_bug.cgi?id=1988
The workaround is to close stderr explicitly before calling ssh (e.g. use 2>&-).  That appears to fix the problem.
I'm not sure Bazel should be in the business of working around fd bugs in child processes, so I'm closing this issue.


Support for stacked progress bars

It would be nice to have support for stacked progress bars
Suggestion for API
data-bind="progress: { stacked: arrayOfProgressBinding }"
And arrayOfProgressBinding would be something like
arrayOfProgressBinding = [
{value: valueSuccess, type: typeSuccess, text: text, textHidden: hidden, animated: animated, striped: striped},
{value: valueWarning, type: typeWarning, text: text, textHidden: hidden, animated: animated, striped: striped},
{value: valueError, type: typeError, text: text, textHidden: hidden, animated: animated, striped: striped}
];

Implemented


1abbdbeff610417b46f4a7b47c477adfb451a

Expected Behavior
Link to the documentation or explain the expected outcome of following the documentation.
Actual Behavior
Explain what actually happens when you follow the documentation.
Section that requires update
Provide a link to the page that needs an update and be specific about which section requires the update.
Additional information
Any other details or screenshots you think are relevant.

1abbdbeff610417b46f4a7b47c477adfb451a


Re-enable setApplicationBadgeIconNumber.  Fixes #1201.  Works very well.

This change is

Do we know what caused this to no longer be an issue?

"Works very well."
Should we not make subjective assertions about the code in the commit message? Seems like either you are trying to convince someone of it which seems wrong in a commit message or that you are extra proud of yourself which is fine but also probably not appropriate for the commit message. I'd prefer if we had a concrete metric+comparison point to back up any such claims.

No.  Quite likely resolved by new projections drop with RS1 SDK?

In reply to: 271708686 [](ancestors = 271708686)

I'll drop that fragment when i merge.

In reply to: 271709239 [](ancestors = 271709239)


[bug] BottomSheet with inputs not working on native

This is a follow up for issue #292  , which was opened because it was not working on web.
As I stated in that issue, the problem is solved, but unfortunately  it's now broken on native side, android side only apparently (tried on both device and simulator, on iOS I could have only tested on simulator and it's working).
For the purpose of this issue, I created a brand new repository with the latest create<EMAIL_ADDRESS>To Reproduce
Run on android (device or emulator) the code in this repo https://github.com/apperside/tamagui-bottom-sheet-bug-reproduction, open the bottom sheet and focus one oh the inputs

Hello @natew ,
any news on this?
I just tried to clone the reproduction repo, run upgrade:tamagui and try it again, but the problem still persists with tamagui 1.2.12 ( the issue was first opened for tamagui 1.0.1).
⚠ **the bug reproduction have been moved to the branch bottomsheet-bug**⚠

Sheet now has a prop to avoid keyboard


how to use ?

hello how to use this player , please tell me the instruction

As the title suggests, this is an exclusive player for Bilibili. It cannot be used anywhere else. There is a UserScript for Bilibili based on that you can have a look.

Very unfortunate, such a good media player can only be used by Bilibili.

I am sorry...

you have discord ? i need ask something to you

We can discuss here, instead. I think.


About page-turning judgment

Hi, I've encountered a habitual issue in usage. Currently, dragForwardInteraction/dragBackwardInteraction  is recognized through setting Start and End Rect for page flipping. However, in practical use, many users have provided feedback that they would prefer recognition of the previous or next page based on the direction of the gesture swipe, rather than being restricted within a fixed area. Is there any good way to set this up?

Hi 👋
Yea, it perfectly makes sense, but unfortunately it is not possible with the current API. Moreover current implementation relies on where tap has started to determine whether next & current or prev & current pages should be rendered. Maybe it could be refactored to wait for the gesture to happen and then take correct pages to render, I will try to look at it 👀

@choristery I have managed to come up with some implementation, which as for me works pretty decent 👀
Could you please try a behavior in the demo app on feature/new-drag-gesture-interaction branch? The new configuration could be set via dragInteraction param in rememberPageCurlConfig:
PageCurl(
    count = pages.size,
    config = rememberPageCurlConfig(
        dragInteraction = PageCurlConfig.GestureDragInteraction()
    )
) { index ->
    HowToPage(index, pages[index])
}

I opened a PR #25, but I would like to hear (or better say read 😅) your opinions before merging it to main branch.
Also I have to change API of rememberPageCurlConfig a little bit to support another drag interaction.

I think the new version of gesture interaction is very well-executed, aligning with user expectations during usage. However, there are areas for improvement in the UI interaction. Currently, the edges of the rolled-up pages do not seem to follow the finger's position on the screen. This gives users a sense of being out of control. When the finger is pressed and begins to move, the raised edge of the page should always follow the finger's position. This would provide a better sense of control.

Thx for checking!
However I am not sure that I fully understand what do you mean by "following the finger's position on the screen". Could you please record a video (possibly with visible touch positions)?

Hi, I uploaded a video on YouTube, hoping it can be helpful to you.
Flip Style

Ah, I see 🤔
If I am not mistaken the PageCurl never behave like on that video. User's finger was always anchored to the page's flip (the edge between back-page of the current page and the content of next page).
I will think about how that behavior could be modified and will try to make it configurable. Could I ask you to open another issue to support such finger-on-page-edge behavior?

Released in v1.5.0 🎉


wip/resolve issue with link hovered state not showing

An event listener needs to be added to the click trigger for the cursor which sets setLinkHovered to false afterwards.

An event listener has been added to ensure the class is removed after a link has been clicked.


proxy-status diff of what Pilot has not what it has sent to Envoy

After further discussions with users they are interested in what Pilot would send not what Pilot has sent. This is far more useful as it enables them to see if Pilot has propagated their Istio config changes to Envoy yet.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: liamawhite
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: costinm
Assign the PR to them by writing /assign @costinm in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Is this something that is going to make it into 1.0?

@zachgersh unless anyone has any objections, yes! 🙂

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: liamawhite, nmittler
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS [nmittler]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Kustomize Plugins Support

This project seems quite useful. Before integrating in my workflow, I was wondering if the project supports Kustomize plugins (like in the kustomize build --enable_alpha_plugins example_path/ command?
My use-case is that I would like to automate a Kustomize deployment using Terraform with this library, and to use a Kustomize plugin with my configuration.
Thanks!

I don't think so. The latest release fixed the built-in plugins. But there is no support for external ones.

I don't think so. The latest release fixed the built-in plugins. But there is no support for external ones.

Ok, thank you for the follow-up. I would be interested in contributing to the project and probably integrating such a feature.
Would it be possible to do so?

Upstream makes it very clear it's alpha, so as long as that's clear here as well it's fine with me.
You'd probably have to add an attribute to the kustomization_build data source and depending on that attribute change the default options returned by krusty.MakeDefaultOptions() to allow external plugins.

I'm closing this because there has not been activity in quite a while.


minor gateway improvements

Move structs to model.go and rename some of them
unexport few variables and structs
No functional change

[ ] Ambient
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

/test release-notes


Restrict Certain HTML Tags (e.g., li, span) in Web Crawling

While using crawl4ai for web scraping, we have encountered the need to restrict specific HTML tags such as  and  during the crawling process. These tags often introduce noise or unnecessary elements that do not contribute valuable data for certain scraping tasks, especially when extracting content-rich elements like paragraphs or headers.

@roshan-sinha-dev Hi sorry for my delay,  you can achieve this using the css_selector. For example if you want to ignore nav and its children you can pass *:not(nav):not(nav *), look at the following code:

# List of tags to exclude from crawling
excluded_tags = [
    "nav",
    "aside",
    "footer",
    "header",
    "form",
    "noscript",
    "iframe",
    "script",
    "style",
    "link",
    "input",
    "button"
]

# Function to convert the list of excluded tags to a CSS selector
def generate_css_selector(tags):
    # Join each tag with :not() and chain them
    return '*{}'.format(''.join(f':not({tag})' for tag in tags))

# Generate the CSS selector
css_selector = generate_css_selector(excluded_tags)

result = crawler.run(
    url="https://www.nbcnews.com/business",
    word_count_threshold=0,
    bypass_cache=True,
    css_selector=css_selector
)

print(len(result.markdown))

This code targets around 1600 html tags while without this, will be 1800+.

Thank you @unclecode But it sometimes seems to be not working I tried for a few tags like "a" or "li" but it is still crawling those tags those tags

I see @roshan-sinha-dev can you send me your code script with the website you're trying to crawl, I can test for you and see what is wrong.

from crawl4ai import WebCrawler
from langchain_text_splitters import RecursiveCharacterTextSplitter
Create an instance of WebCrawler
crawler = WebCrawler()
Warm up the crawler (load necessary models)
crawler.warmup()
excluded_tags = [
"li",
"span",
"a",
"ul"
]
def generate_css_selector(tags):
# Join each tag with :not() and chain them
return '*{}'.format(''.join(f':not({tag})' for tag in tags))
css_selector = generate_css_selector(excluded_tags)
print(css_selector)
Run the crawler on a URL
result = crawler.run(url="https://iimamritsar.ac.in/p/director", css_selector=css_selector)
print(result.markdown)

@roshan-sinha-dev You are right, try this approach plz:
def after_get_url(driver):
        print("[HOOK] after_get_url")
        driver.execute_script("""document.querySelectorAll('li, ul, span, a').forEach(el => el.remove());""")
        return driver

crawler_strategy = LocalSeleniumCrawlerStrategy(verbose=True)
crawler_strategy.set_hook('after_get_url', after_get_url)
new_crawler = WebCrawler(verbose=True, crawler_strategy=crawler_strategy)
new_crawler.warmup()
result = new_crawler.run(url="https://iimamritsar.ac.in/p/director", bypass_cache=True)

I tested, the final markdown doesn't have any of those tags.

Thank you @unclecode  it is working now!!!!

@roshan-sinha-dev in new release, we will add an extra parameter excluded_tags and handle this internally.


Fix an error when dumping a TOC

Pull request
This closes #560.
How Has This Been Tested?
Tested by using the PDF file in #560 .
Checklist

[ ] I have added tests that prove my fix is effective or that my feature
works
[ ] I have added docstrings to newly created methods and classes
[x] I have optimized the code at least one time after creating the initial
version
[x] I have updated the README.md or I am verified that this
is not necessary
[x] I have updated the readthedocs documentation or I
verified that this is not necessary
[x] I have added a consice human-readable description of the change to
CHANGELOG.md

@estshorter thanks for fixing this!


merge feat/type2's smt_trie into develop

This is a recipe for dependency hell:
https://github.com/0xPolygonZero/zk_evm/blob/daf294ac863006f1d9f19370148581ef210b0d1b/Cargo.toml#L94C1-L94C9
Let's have only one copy of smt_trie, synchronised between develop and feat/type2. Eventually we'll have just one codebase

Yeah as long as there are no dependencies on the modified types common types (eg. GenerationInputs) in feat/type2. From a quick look, I think it's ok, but @wborgeaud would be the one to know.

mpt-trie / smt-trie don't depend on any local crates so we're safe in bringing smt-trie into develop :)


The metal material does not self-shadowing when using roughness texture.

I have such material.
metal = Material("brick")
metal.set_metallic(1)
metal.set_roughness(0.45)

This gives the following:

It can be seen that the sphere has artifacts in self-shadowing. I also found out that the problem is related to the roughness map, maybe I'm using it incorrectly.
roughness_tex = loader.loadTexture("tex/metal/roughness.png")
roughness_tex.set_magfilter(SamplerState.FT_linear_mipmap_linear)
roughness_tex.set_minfilter(SamplerState.FT_linear_mipmap_linear)
roughness_tex.set_wrap_u(Texture.WM_repeat)
roughness_tex.set_wrap_v(Texture.WM_repeat)

metal_stage = TextureStage('metal_stage')
metal_stage.set_mode(TextureStage.M_selector)

It looks like this without ambient lighting. Which seems normal to me.

It seems that I was mistaken in the fact that the metal flag affects this problem.
This is just a duplicate of this problem. 19
My problem is applying a roughness texture.

Closing as duplicate of #19


Improved ssh config support in Paramiko backend

The include statement in the ssh config is now supported when using paramiko as the backend
This is a relatively simple change, but it allows my overly complicated ssh setup to work out of the box.  Please let me know if there is anything I need to change so this can be merged.

Merged, thanks!


Feature/add-spatial-units

Added a float spatialUnitFactorMeters field to the TrajectoryInfo block of output data specifying the conversion factor to meters (i.e. for nanometers, spatialUnitFactorMeters = 1e-9).
Added this data for the supported simulation engines in different ways:

CytoSim is always in microns, so just hardcoded it into conversion
PhysiCell units are saved as a user inputted string in the MultiCellDS files, so I'm extracting it and using pint to parse it and get the conversion factor
ReaDDy doesn't save units information in trajectory files, so it must be provided by the user
same for custom data, provided by user

Pull request recommendations:

[ ] Name your pull request your-development-type/short-description. Ex: feature/read-tiff-files
[ ] Link to any relevant issue in the PR description. Ex: Resolves [gh-12], adds tiff file format support
[ ] Provide relevant tests for your feature or bug fix.
[ ] Provide or update documentation for any feature added by your pull request.

Thanks for contributing!

The content of of the changes looks good, but it is concerning that the CI is failing. Do you know why it is failing @blairlyons?

it's failing to set up miniconda? yeah I don't know how to resolve that. I was hoping I could wait a bit and try again. Everything's passing in my dev environment

@JacksonMaxfield do you know what to do about the CI failures? It looks like it's failing to set up conda?

@JacksonMaxfield do you know what to do about the CI failures? It looks like it's failing to set up conda?

Yea. Looks like GitHub changes their action spec but no worries because setup-miniconda has already patched. Go through the .github/workflows dir and replace anything with setup-miniconda@v1 with setup-miniconda@v2


Undesired model remove event triggered

Description: When model is removed from subcollection also model remove event is triggered.
Expected: when model does not satisfy filter condition anymore, only subcollection remove event should be triggered
Actual: also model remove event is triggered
Following test will fail:
  describe 'updating model', ->
    it 'should not trigger remove on the model', ->
      filter = ->
         model.get('prop') == 'val'
      parent = new Collection
      child = new Collection
      model = new Model({id:1})
      parent.add(model)
      spy = sinon.spy()
      model.on 'remove',spy
      subset = new Subset(parent:parent,child:child,filter:filter)
      expect(child.length).to.equal 0
      model.set({prop: 'val'})
      expect(child.length).to.equal 1
      model.set({prop: null})
      expect(spy.called).to.be.false

How to reproduce:
http://jsfiddle.net/mauron85/8axcsvot/3/

Actually, after bit thinking I realized that it's not an issue, but my wrong assumptions.
I should not listen to model remove event, without checking which collection it was removed from.
Always do the check (probably in your view), like this:
var parent = new Collection;
var child = new Collection;
var model = new Model;
var subset = new Subset({
  parent: parent,
  child: child,
  filter: //some filter here
});

model.on('remove', function(model, collection, options) {
  if (collection === child) {
    //now you can be sure model was removed from child subcollection and from parent
  }
 });

And full test:
    it 'should not trigger remove on the model', ->
      filter = ->
        model.get('prop') == 'val'
      parent = new Collection
      child = new Collection
      model = new Model({id:1})
      parent.add(model)
      model.on 'remove', (model, collection, options)->
        expect(collection).to.equal child
      subset = new Subset(parent:parent,child:child,filter:filter)
      expect(child.length).to.equal 0
      model.set({prop: 'val'})
      expect(child.length).to.equal 1
      model.set({prop: null})

I will add remark in README.md to be aware of this, to not making same mistakes.