Update lib types to mark Annex B as deprecated

Fixes #43709
Fixes #32191

You probably have to update baselines.
If we do this, the first thing you'll see every time you request completions on a string is a bunch of crossed-out properties. @sandersn any idea how easy it would be to give them a lower sort order based on JSDoc tags?

@deprecated has its own NodeFlags entry right now, so you just have to use the rules from the checker to decide if the signature as a whole should be considered deprecated (basically, whether all the signature's declarations are deprecated). I don't know the sort order code but I assume it would be easy.

If we do this, the first thing you'll see every time you request completions on a string is a bunch of crossed-out properties.

A quick grep of the baselines for "big" doesn't look like there's a test for this, so you should add one, something like
/// <reference path='fourslash.ts' />
//// var s = "foo"./*1*/
verify.baselineCompletions()

Which should make it obvious what order "big" et al have.

I think before this goes in, we need to add the sorting logic first. I'd like to discuss that at an upcoming editor sync with @mjbvz and others first.

#43796 tracks this.

CC @bterlson

@sandersn verify.baselineCompletions method is broken. Your code emits baseline as a global completion such that
[
  {
    "marker": {
      "fileName": "/tests/cases/fourslash/completionsStringMethods.ts",
      "position": 6,
      "name": ""
    },
    "completionList": {
      "isGlobalCompletion": true,
      "isMemberCompletion": false,
      "isNewIdentifierLocation": false,
      "entries": [
        {
          "name": "globalThis",
          "kind": "module",
          "kindModifiers": "",
          "sortText": "5"
        },
        {
          "name": "eval",
          "kind": "function",
          "kindModifiers": "declare",
          "sortText": "5"
        },
        // ...,
      ]
    }
  }
}

I also found the following broken test.
https://github.com/microsoft/TypeScript/blob/a14b22718ab1bc7d395c981b0a8f95f4d6e361b6/tests/cases/fourslash/completionsJSDocTags.ts#L60-L62
https://github.com/microsoft/TypeScript/blob/a14b22718ab1bc7d395c981b0a8f95f4d6e361b6/tests/baselines/reference/completionsJSDocTags.baseline#L3-L11

I'm a fan of this, and I think the sorting would make it shine 👍🏻

Sorting PR: https://github.com/microsoft/TypeScript/pull/43880

Blocked by #45615

I found that RegExp.$1 to RegExp.$9 should also be deprecated.

This pull request can be reviewed. Thank you for your time.


Adjust style and behavior of Photo Gallery component

Summary of changes
Asana Ticket:  Bug | Photo gallery: 6+ photos break padding, 7th breaks previous/next

Fix: spread thumbnails equally across available space using %
Fix: use TOTAL thumbnail count instead of page thumbnail count to hide nav

Reviewed on Adamo's machine. 👍 Looks good! Thank you!!


pod returns 500 error on first github ping

This was really confusing for me at first, I followed the instructions exactly for adding a GitHub hook but then I got the red error symbol.

The problem is that before GitHub sends any push events, it sends a "ping" event to test if the connection is working. It could be great to have pod respond to that with a 200 OK status instead of throwing the error TypeError: Cannot read property "length" of undefined, to avoid confusion.
I added a note about this to the wiki for now https://github.com/yyx990803/pod/wiki/Using-a-remote-repo#3-setup-github-remote-hook

Good catch. I've definitely run into this when using pod and just ignored it and pushed again.
This should be relatively easy to implement. Let me see what I can do about this.


Require at least one path in inconsistent try test

This testcase gives inconsistent results. It mostly passes, but sometimes fails with only 2 paths found. I was unable to reproduce the issue in the joern console, so believe it is an issue specific to the test. As such, I've updated the test to only require that some path is found. In the meantime, I've opened https://github.com/joernio/joern/issues/689 to keep track of this in case the same issue appears elsewhere.

Closed in favour of https://github.com/joernio/joern/pull/693


Segmentation fault happened at step3

Hi Heng,
I have a problem while running the miniasm to assemble a plant genome. My genome is nearly 400MB. It was sequenced by PacBio with depth of 85x. Before running miniasm, I used the minimap to do the overlap.  Do you have some suggestions to solve this problem?
Thanks,
Wen-Biao
[M::main] ===> Step 1: reading read mappings <===
[M::ma_hit_read::10096.666_1.00] read<PHONE_NUMBER> hits; stored<PHONE_NUMBER> hits and 3335749 sequences<PHONE_NUMBER>7 bp)
[M::main] ===> Step 2: 1-pass (crude) read selection <===
[M::ma_hit_sub::11325.809_1.00] 3223555 query sequences remain after sub
[M::ma_hit_cut::11845.499_1.00]<PHONE_NUMBER> hits remain after cut
[M::ma_hit_flt::12142.079_1.00]<PHONE_NUMBER> hits remain after filtering; crude coverage after filtering: 465.37
[M::main] ===> Step 3: 2-pass (fine) read selection <===
[M::ma_hit_sub::12272.969_1.00] 3203496 query sequences remain after sub
[M::ma_hit_cut::12411.309_1.00]<PHONE_NUMBER> hits remain after cut
Segmentation fault

Could you check if you have enough RAM? Thanks.

We have 512Gb RAM, but I submit my jobs only requiring 48Gb using bsub. And here are some log information
Exited with exit code 139.
Resource usage summary:
CPU time   : 725882.19 sec.
Max Memory :    126026 MB
Max Swap   :    131639 MB
Max Processes  :         5
Max Threads    :        48
I will rerun it with more RAM.
Thanks.

The Segmentation fault happened again even though I allocated 256Gb memory. Maybe another cause?

I don't know. I have to reproduce the segfault to fix it. Is this a public data set or can I debug on it?

Hi Heng,
thanks. Unfortunately, the data is not public now. I tested the miniasm using the public pacbio reads from Arabidopsis thaliana(Ler). It showed good performance and was greatly fast. Actually, I have already run the assembly successfully by using PBcR pipeline.  So I am very curious whether miniasm can get better assembly results.

I have encountered a similar problem. My genome is 620Mb, I have 84x coverage. I have run
./minimap/minimap -Sw5 -L100 -m0 -t32 reads.fasta.gz reads.fasta.gz | gzip -1 > reads.paf.gz
and obtained a file reads.paf.gz which is 148,091,787,711 bytes. Then I have run
./miniasm/miniasm -f reads.fasta.gz reads.paf.gz > reads.gfa and it fails with the following message
[M::main] ===> Step 1: reading read mappings <===
[M::ma_hit_read::13678.392*0.99] read<PHONE_NUMBER> hits; stored<PHONE_NUMBER> hits and 4359024 sequences<PHONE_NUMBER>4 bp)
[M::main] ===> Step 2: 1-pass (crude) read selection <===
[M::ma_hit_sub::20164.461*0.99] 3981562 query sequences remain after sub
[M::ma_hit_cut::21492.442*0.99]<PHONE_NUMBER> hits remain after cut
[M::ma_hit_flt::23199.391*0.99]<PHONE_NUMBER> hits remain after filtering; crude coverage after filtering: 410.91
[M::main] ===> Step 3: 2-pass (fine) read selection <===
[M::ma_hit_sub::23309.968*0.99] 3910410 query sequences remain after sub
[M::ma_hit_cut::23765.825*0.99]<PHONE_NUMBER> hits remain after cut
[M::ma_hit_chimeric::24052.557*0.99] identified 390841 chimeric reads
Segmentation fault (core dumped)

My server has 512Gb of RAM, of which I can use it entirely. I also checked for duplicate reads with the command zcat reads.fasta.gz | perl -ne 'print "$1\n" if />(\S+)/' | sort | uniq -d, but there is none. Any suggestion?

Hi Heng
Hope you are doing well.
We are trying to assemble our latest nanopore human genome set with miniasm, but encountering a coredump at the point of chimeric read detection:
It's about 30X coverage (~100Gb FASTA) but some of the reads are very long. The server has 1TB of RAM.
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/lib/x86_64-linux-gnu/libthread_db.so.1".
Core was generated by `/mnt/human/bin/miniasm/miniasm -f rel4a.fastq rel4a.paf.gz'.
Program terminated with signal SIGSEGV, Segmentation fault.
#0  ma_hit_mark_unused (d=d@entry=0x1a43010, n=n@entry=-1524641225, a=a@entry=0x7f5b928b5010) at hit.c:30
30                      d->seq[a[i].qns>>32].aux = d->seq[a[i].tn].aux = 1;
(gdb) bt
#0  ma_hit_mark_unused (d=d@entry=0x1a43010, n=n@entry=-1524641225, a=a@entry=0x7f5b928b5010) at hit.c:30
#1  0x00000000004081ac in ma_hit_contained (opt=opt@entry=0x7fff0d1567d0, d=d@entry=0x1a43010, sub=sub@entry=0x7fdb8da20010, n=, a=a@entry=0x7f5b928b5010) at hit.c:292
#2  0x00000000004018bc in main (argc=4, argv=0x7fff0d156928) at main.c:138
Let me know if you would like the specific input files uploaded?
Best
Nick

Can confirm that proposed patch https://github.com/voutcn/miniasm/commit/b39d7577d30bca8f368370945995e576c00c358d solves this issue. Thanks!

Thanks a million, @nickloman and @voutcn. Fix committed.


feat(deposit): add ignore check flag

To allow sending failing transactions, which is useful for demo purposes.
We're using it here:
https://www.zetachain.com/docs/developers/omnichain/transactions/#usdc-to-erc-20-custody-contract-with-a-message
To show an example of a transaction that's failing:
npx hardhat deposit --amount 1 --network sepolia_testnet --erc20 0x1c7D4B196Cb0C7B01d743Fbc6116a902379C7238 --message '[["string"], ["hello"]]' --recipient 0x4955a3F38ff86ae92A914445099caa8eA2B9bA32 --ignore-checks

(just realized that I shouldn't have used it before it's shipped, but alright 😁, it's supposed to fail anyways)

Let's merge this and ship Toolkit v8.


DNN-6437 Restore deprecated GetModule methods

These are simple overloads of a more complex method call.

See JIRA issue at https://dnntracker.atlassian.net/browse/DNN-6437

I did not alter either one of the defaults in these methods because it was unclear what the legacy behavior of these methods was prior to 7.3.  I presume that they had different semantics which is why they use different cache settings.
I'll investigate the history and make changes if appropriate.

After further research, the caching behavior is correct. DNN caches moduleinfo objects using TabID as part of the cachekey.  When you don't have a TabID, then you don't have the info needed to pull the object from the cache, thus GetModule(int moduleId) defaults to ignoring the cache since it won't find anything in the cache anyway.

rebased and merged


ImageTools.resize + performInPlace: false issue on iOS

Fuse 1.2.1 build 13974, macOS.
Affected: iOS
Not affected: Android
Reproduction:
<App>
    <JavaScript>
    var Observable = require("FuseJS/Observable");
    var Camera = require("FuseJS/Camera");
    var ImageTools = require("FuseJS/ImageTools");

    var image = Observable("");

    function displayImage(x) {
        image.value = x.path;
    }

    function takePic() {
        Camera.takePicture()
        .then(function(image) {
            var args = { desiredWidth:1280, desiredHeight:720 , mode:ImageTools.KEEP_ASPECT, performInPlace:false};
            ImageTools.resize(image, args).then(
              function(image2) {
               // CameraRoll.publishImage(image);
                displayImage(image2);
              }
            ).catch(
              function(reason) {
                console.log("Couldn't resize image: " + reason);
              }
            );
        }).catch(
            function(reason){
              console.log("Couldn't take picture: " + reason);
        });
    }

    module.exports = {
        takePic: takePic,
        image: image
    };
    </JavaScript>
    <Panel Width="128" Height="48" Clicked="{takePic}">
        <Text Value="Take Pic" TextColor="#fff" Alignment="Center" />
        <Rectangle Color="#18f" CornerRadius="2" />
    </Panel>
    <WhileString Value="{image}" Equals="" Invert="true">
        <Image File="{image}" />
    </WhileString>
</App>

With performInPlace:false set, iOS does not show the image after its taken. All we get is: ImageSource error: 'BundleFileImageSource-failed-conversion' in the Problems tab.
Android works just fine with the same code. iOS works fine when performInPlace is set to true or omitted altogether.

@bondehagen please assign this to me. Somehow I don't have access to assign it to myself

The forum thread Image Resize Problem iOS posted by Ahmed Diab was linked to this issue.


[CHORE] Disable the Paloma LightNode Purchase Page

Disable https://www.palomachain.com/purchase/

Shipped.


Evaluate mistralai/mamba-codestral-7B-v0.1

API results look good, going to leave this open to try local weights.


Territory protocols - new warnings proposal

User might want to get notified in some specific situations:

when some of the T campaign attributes are not filled in
when some T campaign attributes are different from the closest-parent values

@davidzbiral when you have time, would you be able to collect some thoughts here?

@adammertel : Let's call it "Data collection protocol". "Data collection campaign" is what the protocol describes, not the description.
"Data collection protocol not filled in"-type of notifications should concern only first-level Ts - i.e. it is not normally expected that subTs will have anything filled in. All fields should be filled in. When to display the notification: in the Detail of the T (anywhere else?).
subTs different from parent: this concerns the suggester-based only (of course not the textual ones). Yes, good to implement. Not high priority, but good, so if you are working on this and want to have the whole thing at once, then yes, check whether subT clashes with the level above.
As part of this, too, allow - as discussed earlier, setting Data collection protocol for the root (root T). But normally there will be nothing, because one root will contain several different data collection campaigns with different guidelines. This is why we don't want to make the validation too intrusive. Also it shows that all suggester fields of the protocol should accept multiple values, and the clash is only when parent does not have one of subT values among its multiples.
One correction. I change the idea of URL (text field) for Data collection guidelines. Please rather construct them as an R entity, not free url. I.e. suggester accepting R. The URL will hang on that R instead. Move this where you need it.

Probably not so needed. Closing.


obs-vkcapture: install 32bit wrapper

obs-vkcapture supports OGL but one must preload the so. Upstream provides
wrappers for that which you're supposed to use but it's bound to a single
bitness. Install the 32bit wrappers with a suffix to provide access to both.
Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

vkcapture and glcapture are just wrappers for gamecapture and take it from PATH.

Ignore the artefacts, that's with my current mesa build, this is with yours obs-glcatpure32

Do you know if this would also assist with flatpak OpenGL capturing? Here's an example:
Does not capture (OpenGL game flatpak, using nix package)
┌───────────────────>
│keenan in ~ 3s
❄ zsh ›
└─>obs-gamecapture io.openrct2.OpenRCT2
ERROR: ld.so: object '/nix/store/hrymyxiizlvpny7kqjhmasaqk9ni68f4-obs-vkcapture-1.5.0/$LIB/obs_glcapture/libobs_glcapture.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored.
ERROR: ld.so: object '/nix/store/hrymyxiizlvpny7kqjhmasaqk9ni68f4-obs-vkcapture-1.5.0/$LIB/obs_glcapture/libobs_glcapture.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored.
ERROR: ld.so: object '/nix/store/hrymyxiizlvpny7kqjhmasaqk9ni68f4-obs-vkcapture-1.5.0/$LIB/obs_glcapture/libobs_glcapture.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored.
ERROR: ld.so: object '/nix/store/hrymyxiizlvpny7kqjhmasaqk9ni68f4-obs-vkcapture-1.5.0/$LIB/obs_glcapture/libobs_glcapture.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored.
ERROR: ld.so: object '/nix/store/hrymyxiizlvpny7kqjhmasaqk9ni68f4-obs-vkcapture-1.5.0/$LIB/obs_glcapture/libobs_glcapture.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored.
ERROR: ld.so: object '/nix/store/hrymyxiizlvpny7kqjhmasaqk9ni68f4-obs-vkcapture-1.5.0/$LIB/obs_glcapture/libobs_glcapture.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored.
ERROR: ld.so: object '/nix/store/hrymyxiizlvpny7kqjhmasaqk9ni68f4-obs-vkcapture-1.5.0/$LIB/obs_glcapture/libobs_glcapture.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored.
ERROR: ld.so: object '/nix/store/hrymyxiizlvpny7kqjhmasaqk9ni68f4-obs-vkcapture-1.5.0/$LIB/obs_glcapture/libobs_glcapture.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored.
ERROR: ld.so: object '/nix/store/hrymyxiizlvpny7kqjhmasaqk9ni68f4-obs-vkcapture-1.5.0/$LIB/obs_glcapture/libobs_glcapture.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored.
ERROR: ld.so: object '/nix/store/hrymyxiizlvpny7kqjhmasaqk9ni68f4-obs-vkcapture-1.5.0/$LIB/obs_glcapture/libobs_glcapture.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored.
ERROR: ld.so: object '/nix/store/hrymyxiizlvpny7kqjhmasaqk9ni68f4-obs-vkcapture-1.5.0/$LIB/obs_glcapture/libobs_glcapture.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored.
ERROR: ld.so: object '/nix/store/hrymyxiizlvpny7kqjhmasaqk9ni68f4-obs-vkcapture-1.5.0/$LIB/obs_glcapture/libobs_glcapture.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored.
ERROR: ld.so: object '/nix/store/hrymyxiizlvpny7kqjhmasaqk9ni68f4-obs-vkcapture-1.5.0/$LIB/obs_glcapture/libobs_glcapture.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored.

Does capture (OpenGL game flatpak, using the flatpak obs-vkcapture)
┌───────────────────>
│keenan in ~
❄ zsh ›
└─>flatpak run --command=/usr/lib/extensions/vulkan/OBSVkCapture/bin/obs-gamecapture io.openrct2.OpenRCT2 openrct2

Does capture (Vulkan game flatpak, using nix package)
┌───────────────────>
│keenan in ~
❄ zsh ›
└─>obs-gamecapture net.pcsx2.PCSX2
ERROR: ld.so: object '/nix/store/hrymyxiizlvpny7kqjhmasaqk9ni68f4-obs-vkcapture-1.5.0/$LIB/obs_glcapture/libobs_glcapture.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored.

@keenanweaver Some notes:

FlatPak libs are isolated from the system, you'll need to install and use the FlatPak version of the layer: https://github.com/flathub/org.freedesktop.Platform.VulkanLayer.OBSVkCapture
Both examples that you used are built for 64-bits and not related to this PR (we're testing obs-gamecapture32 instead of obs-gamecapture).

@PedroHLC odd, glcapture32 shouldn't be working for 32bit here, only gamecapture32. Are you sure it's 32bit?

The non-gamecapture 32bit links are now simply symlinks to gamecapture32 and don't print warnings to gamecapture but anything other than that would be unreasonably complex.


buy_pct on original value?

buy_pct  keeps taking parts of remaining balance rather than the percentage of the original balance. EG if start balance is $50 and buy_pct is set to 10%. It will take 5$ then, 4.5$ then 4.05$ etc. It should take 5$ from 50 each time, and then stop once there is no balance remaining.

duplicate https://github.com/carlos8f/zenbot/issues/174

no, I should correct myself, the question here is to keep the original value in mind. Sounds like a potentially unwanted enhancement.

any news on this?

I think some functionality like this landed recently, please take another look and let me know if this issue is still accurate.


[DOC] - Update Readme with GIF of new UI.

Preliminary Checks

[X] This issue is not a question, feature request, RFC, or anything other than a bug report. Please post those things in GitHub Discussions: https://github.com/nebari-dev/nebari/discussions

Summary
The new UI is aesthetically and functionally nicer than the POC version from a month or two ago.
We should replace the existing gif in the readme with an updated version
Steps to Resolve this Issue

Make new video/gif demoing the updated UI & functionality
Replace existing GIF in REadme.
...

The readme has been updated. This can be closed


Issue with training at 8000Hz

Hello Everyone,
I am trying to train on 8000Hz using Piper. But the voices are not clear after 4353 epochs. It looks like it's mumbling.
I have my own dataset recorded at 8000Hz.
Here is the sample of original recording
Also the generated audio sound like this
The preprocessing was done like this
python3 -m piper_train.preprocess \
  --language en-us \
  --input-dir path/to/data \
  --output-dir path/to/data/out-dir \
  --dataset-format ljspeech \
  --single-speaker \
  --sample-rate 8000

And for training, I m running this script
python3 -m piper_train \
    --dataset-dir path/to/data/out-dir \
    --accelerator 'gpu' \
    --devices 1 \
    --batch-size 32 \
    --validation-split 0.05 \
    --num-test-examples 0 \
    --max_epochs 10000 \
    --checkpoint-epochs 1 \
    --precision 32 \
    --quality high

Please remember I m not finetuning a new model, but training from scratch. Should use a pre-trained model at 22050 for my model to be at 8000?
Thank you for your help.

Hiya, I do some work in TTS and signal processing- my experience has been like yours. I tried training 8kHz models on many architectures and they have a very, very hard time learning a good correspondence between text and spec or text and wav. I can't really say why for sure but my personal theory is that the human voice has many resonant/turbulent frequencies above 4kHz which are important to phone identity. For example, the difference between "s" "th" and "f" is almost entirely high-frequency spectra (see https://home.cc.umanitoba.ca/~robh/howto.html). When we use data sampled at 8kHz, we lose everything above 4kHz (Nyquist frequency). I think without these "clues" from the high frequencies, the model is sort of lost, scared and confused... :) You can see in this paper on LibriTTS (https://www.arxiv-vanity.com/papers/1904.02882/) they discuss why they use a higher sampling rate (24 versus 16 for LibriSpeech) and they say that 16kHz is "too low" to achieve "high quality" TTS. I assume this is a qualitative discussion (they go on to measure WER performance on 16 versus 24 kHz TTS) rather than a discussion about whether or not its literally possible (we know 16 kHz is definitely possible), but still seems to be another clue to be that 8kHz is "too low".

Thank you @rachel-beeson-connex and @synesthesiam for the reply. I believe if this is not a possible way other than downsampling higher Hz output. But this will increase the latency. Any other suggestions?

I think even if you downsample the parameter size of the model remains the same - maybe synesthesiam can confirm? However, if you want a smaller parameter size model, you can use the --quality flag when you're training and set it to 'x-low'. I did try this setting once for training and the results were decent but not perfect. I think it benefits from a longer training time.

I think even if you downsample the parameter size of the model remains the same - maybe synesthesiam can confirm? However, if you want a smaller parameter size model, you can use the --quality flag when you're training and set it to 'x-low'. I did try this setting once for training and the results were decent but not perfect. I think it benefits from a longer training time.

Thanks @rachel-beeson-connex for giving it a thought. Yes i did tried that and it doesn't work like that.


Bump Base

This PR advances dependency on Base to newest snapshot version, which removed validating builders for standard Protobuf classes, and introduced a number of other changes. See #252 for details.
The main work does in this PR is migrating to specific entity types (instead of standard ones that can no longer be used because of absence of validating builders).
Other notable changes:

Affected test environment classes were pushed to package level.
Error Prone warnings BadImport and EqualsGetClass were addressed.
Naming for exception factories was simplified.

@armiol, it builds in local environment. Once you review this PR, let me know and I'll merge Base first to make it the build of this PR pass. PTAL.


panel disappears when pluggin in external monitor

Hyprland Version
 System/Version info
<Paste the output of the command here>

Hyprland, built from branch HEAD at commit 03ebbe18ed8517ee22591eac82cd54322f42cb7d  (props: bump ver to 0.34.0).
Date: Mon Jan 1 12:03:15 2024
Tag: v0.34.0
flags: (if any)

Bug or Regression?
Bug
Description
I'm using nwg-panel on a Laptop running hyprland on Arch. When plugging in an external monitor via HDMI the panel disappears. ps shows the process as still running.
How to reproduce
plugin monitor on HDMI port
run ps | grep nwg-panel to show it's still running
Crash reports, logs, images, videos
No response

Are you sure this is a Hyprland bug and not a nwg-panel one? For me and many other users this does not happen with other panels/bars.

Am Donnerstag, dem 01.02.2024 um 07:34 -0800 schrieb Mihai Fufezan:

Are you sure this is a Hyprland bug and not a nwg-panel one? For me
and many other users this does not happen with other panels/bars.

just checked with waybar and looks like a nwg-panel issue.
Sorry


Is it possible to convert a long 1-line array into a multiline array?

Would it be possible to split a 1 line array like the below one
var tempName = ['temp',process.pid,(Math.random() * 0x1000000000).toString(36),new Date().getTime()].join('-');

to be beautified into something like this?
var tempName = [
    'temp',
    process.pid,
    (Math.random() * 0x1000000000).toString(36),
    new Date().getTime()
].join('-');

@brupelo - this feature does not exist at this time.

+1 here
intellij does this by default for long arrays, would be great to have here


Adhere to XDG Base Directory Specification

The cli should be a good citizen and adhere to the "XDG Base Directory Specification":
https://specifications.freedesktop.org/basedir-spec/basedir-spec-latest.html

Done


Add SecretsManager, issue #105

This is an attempt at #105.

Released under 1.3.0. Thanks again for your contribution!

I've raised #166 and #167 as follow-ups. Feel free to contribute if you're interested!


Add clock_nanosleep for linux and solaris

Add clock_nanosleep.
Support is a bit scarce but it's the only sleep call with options for monotonic clock and absolute time.

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @alexcrichton (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

Thanks!


Develop an API to update a question

Develop an API to update a question

Created an api to update the question with different title and body


[FLINK-32141][checkpointing] turn down a log level in the SharedState…

…Registry
What is the purpose of the change
Turn down a log level in the SharedStateRegistry, described in  FLINK-32141.

@zoltar9264 Thanks for the PR, +1 for "FsStateChangelogStorage will directly use the FileStateHandle of the previous checkpoint instead of PlaceholderStreamStateHandle."  When changelog is enabled, SharedStateRegistry would print too much info log.

Thanks @reswqa for helping merge to master.

Can you open a PR and backport it to release-1.17.

Sure, but I just checked the 1.17 code and found no conflicts (this commit can be merge into 1.17 directly), am I missing something?

but I just checked the 1.17 code and found no conflicts (this commit can be merge into 1.17 directly), am I missing something?

I just found the fix version on the ticket is 1.17.2, so I would like you to confirm if this need to pick it to 1.17.
Thank you for your confirmation, I will manually merge it :)


Closes #351

Pulls mu and sigma from the db instead of using the ones returned from our workers. Includes some related refactoring.

Lgtm and works in my local tests.
@Janzert, want to take a look?


Set custom route for knex migrate:make

Currently, I have this project structure:
src
-entities
--users
---model.js
---migrations
----20190303042400_create_users_table
What I'm trying to accomplish is to have my config setted up to my entities folder, so I can do like:
npx knex migrate:make create_users_table --migrations-directory users
But I can only accomplish this by doing:
npx knex migrate:make create_users_table1 --cwd="./src/entities" --migrations-directory users/migrations
That I find very repetitive given the fact that my entities folder will never change, I tried to set a base folder route in my knex file but I didn't get to make it work, so, is it possible?

I don't quite understand what are you after... why do you have to give --cwd there and not only --migrations-directory users/migrations.
Closing as this sounds like knex usage question and github issue tracker should not be used for that. Can be reopened if there is a knex bug involved.

I don't quite understand what are you after... why do you have to give --cwd there and not only --migrations-directory users/migrations.
Closing as this sounds like knex usage question and github issue tracker should not be used for that. Can be reopened if there is a knex bug involved.

@elhigu So this is a bug, cause when I do just the command that you suggest, the migration is been created at the same level as src in the usual migration folder, not even creating the folder users inside it
I expect that command to at least create something like:
src
migrations
-users
--my_new_migration.js
Even when I try: npx knex migrate:make create_users_table1 --migrations-directory src/entities/users/migrations
It still create the migration in the migration folder, that's the reason why I use --cwd

Sounds like a bug I suppose... should write jake test to verify this.

I specified multiple migration directories, and after run knex migrate:make create_table_name, it always create file in the last folder in directories(migrations/DML/201901010101_create_table_name.js).
// knex.js
module.exports = {
  migrations: {
    directory: ['migrations/DCL', 'migrations/DDL', 'migrations/DML']
  }
}

after debugging, I find below line in knex source migrationGenerator.js
// line 65 of MigrationGenerator.js in knex source codes
_writeNewMigration(name, tmpl) {
    const config = this.config;

    const dirs = this._absoluteConfigDirs();

   // below line will only fetch the last specified directory
    const dir = dirs.slice(-1)[0]; // Get last specified directory
  }

@idododu it is expected behavior that migration is created to the last of the configured migration directories. Please add feature request if you need it to work in a different manner.

facing same problem, waiting for solution

Possible solution:
require('dotenv').config()

module.exports = {
  development: {
    client: "pg",
    connection: {
        database : process.env.DB_NAME,
        host : process.env.DB_HOST,
        user : process.env.DB_USERNAME,
        password : process.env.DB_PASSWORD,
    },
    migrations: {
      directory: "src/migrations"
    }
    // debug: true
  }
};

You can do this:
require('dotenv').config()

module.exports = {
  development: {
    client: "pg",
    connection: {
        database : process.env.DB_NAME,
        host : process.env.DB_HOST,
        user : process.env.DB_USERNAME,
        password : process.env.DB_PASSWORD,
    },
    migrations: {
      directory: "src/migrations"
    }
    // debug: true
  }
};

In my case I mistakenly named migrations key as migration in knexfile.ts.
like
Expected:
migrations: {
      directory: './database/migrations'
    },

What I had written:
migration: {
      directory: './database/migrations'
    },


⚠ SceneTime has degraded performance

In 6f84e7f, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 2869 ms

Resolved: SceneTime performance has improved in fa02609 after 6 minutes.


Fold intro

Write an introductory description that describes in 2-4 sentences what you do, what you enjoy doing, what you want to do better, what is important to do, and why you are here

Hello! I am a college student studying software with a focus on front-end development. I enjoy the creative process of creating beautiful web interfaces and am constantly looking to learn new technologies. My goal is to develop my skills in JavaScript and its application. It is important for me to have a friendly team where we can solve problems together and grow professionally. I'm here to learn and get better at everything JavaScript related.


newClientRequest() now accepts an initial header

This allows newClientRequest() to set an Authorization header from either an SSH response or link relation without calling git credentials. Fixes #533
/cc @jatoben

@sinbad/@rubyist: I'd appreciate a review, since you two have worked in this part of the codebase. It's a pretty trivial patch, but I still feel weird about merging it by myself.

:+1:

Looks good to me, not sure why Travis is upset after that latest commit though, tests are green here.

Intermittent test failures are my favorite :trollface: I've started putting some more details, but it hasn't been helping. I'll probably try a PR that changes them all to bash today.


Tags

Any chance we could get some fresh tags added pointing to a more recent commit? I'd like to use this library but don't want to use dev-master, and the latest tag is a year old and broken.
Thanks

Is v0.31.1 really broke?
@knasher can you look into this?

Sorry 'broken' is over exaggerating it a bit, but 0.31 returns some incorrect bank holidays. When I loaded in dev-master, the expected bank holidays are loaded

Which ones are incorrect may I ask?
@eddiejaoude I think we can still tag but not sure of dependancies being
used by the team
On Wed, Jul 1, 2015 at 12:00 AM, Dan Brooks<EMAIL_ADDRESS>wrote:

Sorry 'broken' is over exaggerating it a bit, but 0.31 returns some
incorrect bank holidays. When I loaded in dev-master, the expected bank
holidays are loaded
—
Reply to this email directly or view it on GitHub
https://github.com/ministryofjustice/date-utils/issues/15#issuecomment-117367393
.

@danbroooks released tag for you 0.4.0

Which ones are incorrect may I ask?

@danbroooks Good to log this against the previous tag to make people aware (thanks @brettminnie )
Nice one @jeremyquinton

Sorry I wasn't sure how much tags were actually being used seeing as there were only three made in fairly quick succession this time last year.
Just off the top of my head I think 0.31.0 was coming back with Boxing Day this year being on the 26th (a saturday), whereas for my purposes I wanted it to be the next working day, the 28th. master returns Boxing Day as being on the 28th.

Thanks for the tag, however perhaps give this a read. The tag added was 0.4.0, however previous tags have been 0.18.1, 0.21.1 and 0.31.1.
To composer the latest tag is actually the oldest version of the four. Those putting "ministryofjustice/date-utils": "~0.4" in their composer.json will end up getting 0.31, which is actually older than this most recent tag.

Well noticed @danbroooks .
0.4 is old than 0.31.
0.31.* could be deleted, but I remember packagist didn't delete old tags before, which also caused issues in the past.
Is it worth bumping the major @jeremyquinton and starting again? eg. 1.0.0 and go from there

Yes I would keep existing tags in place (however perhaps drop 0.4.0 before people pick it up), then either bump to 1.0.0 or increment 0.31.* until you are ready to commit to a 1.0.0 version.

The old tags where pre-release.sprint number.revision. So yes could be
considered a release now
@jeremyquinton :+1: for picking this up, I'll look at the fix for the
reported problem for the 0.31.x branch as well seeing as I am on whoreliday
:)
B
On Wed, Jul 1, 2015 at 12:42 PM, Dan Brooks<EMAIL_ADDRESS>wrote:

Yes I would keep existing tags in place (however perhaps drop 0.4.0
before people pick it up), then either bump to 1.0.0 or increment 0.31.*
until you are ready to commit to a 1.0.0 version.
—
Reply to this email directly or view it on GitHub
https://github.com/ministryofjustice/date-utils/issues/15#issuecomment-117625228
.

@danbroooks well spotted. I have released tag 1.0.0. This library is stable enough to be version 1.0.0.
Should I delete the 0.4.0 tag I created or is it going to create issues?

I would delete 0.4 as it is newer than 0.31 which is more confusing.
Everyone will probably use 1.0 now anyway as it is the latest stable.

Agree with Eddie here
On Thu, 02 Jul 2015, 12:31 Eddie Abou-Jaoude<EMAIL_ADDRESS>wrote:

I would delete 0.4 as it is newer than 0.31 which is more confusing.
Everyone will probably use 1.0 now anyway as it is the latest stable.
—
Reply to this email directly or view it on GitHub
https://github.com/ministryofjustice/date-utils/issues/15#issuecomment-117993299
.

I deleted 0.4.0 and created a 1.0.0 tag


Add a new install method 'none' that doesn't install.

Source: https://github.com/edelight/chef-mongodb/pull/367

:+1:

:+1: LGTM.
would be nice to add this to the readme

:+1: LGTM.


Add control flow functions from p5js

If drawing a static sketch (i.e. with no moving parts), then it seems wasteful to repeatedly call the (draw) function. Personally, I find it annoying when my laptop fan continues going crazy even after all the drawing has supposedly been completed.
p5js provides loop(), noLoop() and redraw() functions to alter the control flow. After drawing a static picture, for example, noLoop() can be called and no unnecessary CPU churn will take place afterwards. Another example: the user could click their mouse to start and stop a drawing loop.
So, my suggestion is to add the equivalent of these functions to sketch. I'm not sure how they would interact with redefinitions - I suppose that the loop would be started again once the sketch is redefined?
References:

https://p5js.org/learn/program-flow.html
https://p5js.org/examples/structure-no-loop.html

As a workaround right now you can do that with kit.sdl2:
;;; test should name a sketch
* (make-instance 'test)
; => #<TEST {1006764313}>
* (setf (kit.sdl2:idle-render *) nil)  ; stops redrawing the sketch automatically
; => T
* (kit.sdl2:render (kit.sdl2:last-window))  ; redraws the last window
; => NIL
* (defmethod kit.sdl2:mousebutton-event ((window test) st ts but x y)
    ;; redraw the window on each mouse button click
    (when (eq st :mousebuttondown)
      (kit.sdl2:render window)))
; => #<STANDARD-METHOD KIT.SDL2:MOUSEBUTTON-EVENT (TEST T T T T T) {1005E3AF33}>

Done! We could consider adding the function to trigger redraws, but I can't think of any application that can't be achieved with start & stop functions.


'Array(T)#count' is expected to be invoked with a block, but no block was given

Just updated my Crystal version to 0.8.0 and my specs broke.
in ./src/cryload/stats.cr:26: 'Array(Cryload::Request)#count' is expected to be invoked with a block, but no block was given

      total_request_time / @requests.count

The related code: https://github.com/sdogruyol/cryload/blob/master/src/cryload/stats.cr#L25-L27
Do i have to specifiy a block for count even if i don't have to?

Use size

So this is intentional?

Yes

OK. Thanks @jhass :+1:

Btw i forgot to ask what's the motivation for this?

https://github.com/manastech/crystal/issues/1363

Thanks @asterite


Update iOS simulators to iOS 11

Had to comment out a couple of tests related to QualityOfService. There are some relevant changes in the releases notes of iOS 11:

NSOperation

NSOperation now responds to KVO and KVC for representing the finished, cancelled, executing and ready states as the strings @“finished”, @“cancelled”, @“executing” and @“ready” in addition to the older versions like @“isReady”.

NSOperation completionBlock now uniformly is set to nil upon completion of an operation to avoid circular references to operations.

NSOperationQueue

NSOperationQueue will no longer create a serial backing dispatch queue for each operation. Instead it now shares a single concurrent dispatch queue.

Quality of service for NSOperations when enqueued to an operation will no longer be calculated in advance, instead the quality of service for an operation is only applied if it is requested upon the execution of an operation. If a NSOperationQueue specifies a quality of service, the backing dispatch queue will have that specified quality of service. Any operations run upon that queue will inherit the effective quality of service if they do not have a quality of service specified. However if an NSOperation has a quality of service specified it will enforce that quality of service when running upon the NSOperationQueue’s backing dispatch queue.

Quality of service overrides now only apply when waiting for operations to finish. This means that the work done to calculate the effective quality of service is not paid up-front but instead calculated when waiting is invoked.

NSOperationQueueDefaultMaxConcurrentOperationCount should no longer hit thread exhaustion states.


[UI] Ping email server should be blocked utill get the reponse.

Enhanced controlling.

close it as fixed.


Set User Agent for CCM

Description
This PR sets user agent for CCM.

We can actually remove the vendor directory. Talked with Vish about it.

Github workflow modification?


Could we improve the limits fixed to dependency injection in v1

I frequently find myself in a situation where I would need some value or some service in a property of a ContentPart, but autofac can't be used in v1 to initialize properties, even if this is in its possibilities.
Yes I know, lot of people will give me a lesson of good software usage and practice and so and so...you know what I mean.

You can already do that with a content handler during the Initializing or Loading events.

It is a trick and not identical to what we get with injection.
Will the content events persist as NH is gone ?

The content events have nothing to do with NH, just the content management triggering them. And they are still there. And we'll never implement dependency injection for data object, for the exact reason you mentioned: "lot of people will give me a lesson of good software usage and practice and so and so"

Ok, Great. We should call this the Orchard Academy (seems very ecolo).
So I am ready to learn why.


Add tfBurnable logic to accounts for issued currencies

Rather than adding a clawback transaction type, if the Issuer has the flag set at the time the trustline was created then the issuer can unilaterally make a "negative payment" to the holder of the currency. A negative payment bypasses strong transactional stakeholder status on the destination, only allowing their hooks to be weakly executed so a clawback can't be blocked. In addition, the Issuer is a strong transactional stakeholder on all dealings.

A strong TSH for the issuer could e.g. also enforce what pairs a currency can be traded in.


Support to install the SapMachine Version for the aarch64 (Apple Silicon) architecture

Hello SapMachine Manager for MacOS Team,
is it planned to add support for the the aarch64 (Apple Silicon) architecture Version of SapMachine?
Best Regards
Gregor

This is still the case. SapMachine Manager downloads and installs only the versions of SapMachine for the architecture the Mac is running.

Hi @mthielemann
then I'm wondering why I run into this issue with the aarch64 version of the SAP Cloud Connector (https://tools.hana.ondemand.com/#cloud):
 sapcc-2.17.1-macosx-aarch64 which java
/usr/bin/java
➜  sapcc-2.17.1-macosx-aarch64 java --version
openjdk 21.0.4 2024-07-16 LTS
OpenJDK Runtime Environment SapMachine (build 21.0.4+7-LTS)
OpenJDK 64-Bit Server VM SapMachine (build 21.0.4+7-LTS, mixed mode, sharing)
➜  sapcc-2.17.1-macosx-aarch64 ./go.sh
using java /Library/Java/JavaVirtualMachines/sapmachine-21.jdk/Contents/Home/bin/java

Detected mismatch between jdk binaries for x86_64 and scc binaries for arm64
Stopping the process. Ensure to use the corresponding binaries either for x86_64 or for arm64
Set env variable NO_CHECK=1 to overcome this check
➜  sapcc-2.17.1-macosx-aarch64 /Library/Java/JavaVirtualMachines/sapmachine-21.jdk/Contents/Home/bin/java --version
openjdk 21.0.4 2024-07-16 LTS
OpenJDK Runtime Environment SapMachine (build 21.0.4+7-LTS)
OpenJDK 64-Bit Server VM SapMachine (build 21.0.4+7-LTS, mixed mode, sharing

Best Regards
Gregor

No idea. SapMachine Manger just downloads and installs the archive for the current architecture. If you see any issues with SapMachine you probably should reach out to the guys who are developing it. I just developed this tool and don't know much about Java or SapMachine in particular.
Regards,
Marc


Cannot switch to release channel

Thank you for developing this tool.
I'm using Xunit and as far as I understand I need to run on the release branch.
I cannot switch to release as the option is not present.

I'm using Visual Studio 2017 Community edition

Hope you can help.

I install it manually via the release tab on Github and it works!


[WIP: hacky workaround] Retain the value typed in the target namespace field when clicking away

WIP. May resolve https://github.com/konveyor/forklift-ui/issues/463 (https://bugzilla.redhat.com/show_bug.cgi?id=1943373).
This is a rough one. We may want to consider pushing this fix to post-GA, or removing the ability to create a new namespace from the plan wizard.
The PatternFly Select component does not support retaining the search text for its typeahead variant when the user leaves the field. In order to get this behavior, we have to use the onTypeaheadInputChanged prop introduced in https://github.com/patternfly/patternfly-react/pull/5375, which means upgrading to the latest version of PatternFly. There are a few problems with this:

The upgrade makes it possible to fix this bug, but it involves a hacky workaround for a PF regression I found that is introduced by the upgrade: https://github.com/patternfly/patternfly-react/issues/5625
Upgrading PatternFly also breaks a few other things in our UI, particularly where we had to work around other PF bugs that are now fixed. Removing these workarounds is non-trivial.
It would be better to implement the desired behavior as part of the PatternFly Select component itself, instead of relying on onTypeaheadInputChanged and working around the issues it presents. We can't get that done before GA.

In its current state this PR fixes #463, but it's hacky and not ideal, and it does not yet address other issues caused by upgrading PF.
I would prefer to remove the ability to create a new namespace in the wizard for GA and reintroduce it later. This would alleviate the confusion (and users can still create a namespace outside our UI and have it automatically appear here). It would also avoid the need to address #507 right away.

Closing this to revisit post-GA.


Please start respecting semver

The golang versioning/numbering scheme is intended to work like this:
https://semver.org
So for example v2.3.4 means vMAJOR.MINOR.PATCH:

MAJOR version when you make incompatible API changes
MINOR version when you add functionality in a backward compatible manner
PATCH version when you make backward compatible bug fixes

However with this project you sometimes make changes to the code that break the API but only the minor version number.
So when a project does a go get -u changes come in from your project that break everything that depends on your project. So for example, in a project we depend on, that project depends on excelize, and we are seeing:
/Users/myproject/go/pkg/mod/gitlab.com/myrepo/security@v0.139.1/page_accounts_export.go:371:3: unknown field Panes in struct literal of type excelize.Panes
/Users/myproject/go/pkg/mod/gitlab.com/myrepo/security@v0.139.1/page_accounts_export.go:371:21: undefined: excelize.PaneOptions

I know how annoying it is to have to do a major version bump, but it would be helpful if you could restrict API breaking changes to major version updates.  If this is not possible I understand, your project is awesome and all your work is much appreciated.

Thanks for your kind notice. This related with issues #231, #1079 and #1516. I will follow that in the future release.


How to add resources to context?

I want to add drizzle orm handler in request context.
import { RouterHandler, Router } from "@tsndr/cloudflare-worker-router"
import { NodePgDatabase } from "drizzle-orm/node-postgres"
import type * as schema from "../db/schema.js"
export type Var<T = string> = T
export type Secret<T = string> = T
export type Env = {}
export type ExtCtx = {
	db_client: NodePgDatabase<typeof schema>
} & ExecutionContext                     // --------------- Without this it will error in fetch
export type ExtReq = {
}
export type Handler = RouterHandler<Env, ExtCtx, ExtReq>

Here is my default fetch
export default {
	async fetch(request: Request, env: Env, ctx: ExtCtx) {
		let resp = applyHttpAuth(request)
		ctx.db_client = await db_client()
		if (resp === true) {
			return router.handle(request, env, ctx)
		} else {
			return resp
		}
	},
}

Now i am creating handler
router.get("/users", async (req) => {
		let users = await req.ctx!.db_client.query.users.findMany({})
		console.log({ users })
		return new Response(JSON.stringify(users))
	})

The problem is typescript says ctx might be undefined. I added ! (bang) for that. But it gives error that db_client cannot be found in ExecutionContext. It should be searching from ExtCtx instead of ExecutionContext?
Property 'db_client' does not exist on type 'ExecutionContext'.

Also if u don't mind whats the role of ExtReq?
Thank You
Shirshak

Hey, I will create an example repository, in the meantime here's a little starter where I put sentry on the ctx:
import { Router, RouterHandler } from '@tsndr/cloudflare-worker-router'
import { Toucan } from 'toucan-js'
import { User } from './models/user.model'

import authMiddleware from './middlewares/auth.middleware'

import * as authController from './controllers/auth.controller'
import * as userController from './controllers/user.controller'

export type Var<T = string> = T
export type Secret<T = string> = T

export type Env = {
    ENV: Var<'dev' | 'prod'>
}

// Request Extension
export type ExtReq = {
    user?: User
}

// Context Extension
export type ExtCtx = {
    sentry?: Toucan
}

// Handler Type
export type Handler = RouterHandler<Env, ExtCtx, ExtReq>

// Initialize Router
const router = new Router<Env, ExtCtx, ExtReq>()

// Enable cors
router.cors()

// Debug
//router.get('/debug/env', ({ env }) => Response.json(env))

// Auth
router.post('/auth/sign-up', authController.signUp)
router.post('/auth/sign-in', authController.signIn)
router.get('/auth/profile', authMiddleware, authController.profile)
router.post('/auth/profile', authMiddleware, authController.updateProfile)

export default {
    async fetch(request: Request, env: Env, ctx: ExecutionContext): Promise<Response> {
        const sentry: Toucan = initSentry()
        try {
            return await router.handle(request, env, ctx, { sentry })
        } catch(err) {
            console.error(err)
            sentry?.captureException(err)
            return new Response(null, { status: 500 })
        }
    }
}

As seen in the example above, ExtReq can be used to extend req.

@tsndr
Hi, thank you for the details.
Could u please tell me whats the difference between ext context and ext req.
Thank you.

router.get('/user', ({ env, req, res, sentry }) => {
//                                    ^
//                                    This is where sentry would be available
    return Response.json({
        id: 1,
        name: 'John Doe'
    })
})

thank you.


Ignore missing submodule artifacts in buildall init [skip ci]

Prevents buildall from failing on empty m2 cache
Tested with:
rm -rf ~/.m2/repository/com/nvidia ; ./build/buildall

Follow-up to #7840
Signed-off-by: Gera Shegalov<EMAIL_ADDRESS>

Thanks @tgravescs for reporting the issue

build

A little curious why we call fail-never instead of just ignoring the result?

A little curious why we call fail-never instead of just ignoring the result? And possibly outputting a message saying that it is okay if this failed.

Good question. Without fail-never all submodules downstream from sql-plugin may fail.
But since the issue is largely about spark-rapids-jni which is a sql-pliugin dependency I think we can just  initialize sql-plugin without worrying about missing submodule dependencies,

build


Schemaspy6 error

Hi,
I am running the following command:
java -jar schemaspy-6.0.0.jar -dp force-metadata-jdbc-driver-2.3.jar -debug -t force -u MyUser -p MyPassword -db MyDB -norows -o doc -connprops url=https://test.salesforce.com/services/Soap/u/23.0 -cat % -s force
and I get ERROR - null. I have activated the debug to find out the line where it crashes.
INFO  - Starting Main v6.0.0 on LT18202 with PID 11484 (C:\Users\mtejeiro\Documents\sw\schemaspy6\schemaspy-6.0.0.jar started by mtejeiro in C:\Users\mtejeiro\Documents\sw\schemaspy6)
INFO  - The following profiles are active: default
INFO  - Started Main in 2.177 seconds (JVM running for 3.072)
DEBUG - Debug enabled
INFO  - Configuration file not found
INFO  - Starting schema analysis
DEBUG - Resolving dbType: force ->
/C:/Users/mtejeiro/Documents/sw/schemaspy6/force.properties
ForceMetaDataDriver: Force.com connection url https://test.salesforce.com/services/Soap/u/23.0
ForceMetaDataDriver: Filter [exclusionNames=[user], inclusionNames=[], keepCustom=true, keepStandard=false]
ERROR - null
java.lang.NullPointerException: null
at org.schemaspy.service.DbmsService.lambda$fetchDbmsMeta$5(DbmsService.java:107)
at org.schemaspy.service.DbmsService.onlyLogException(DbmsService.java:123)
at org.schemaspy.service.DbmsService.fetchDbmsMeta(DbmsService.java:107)
at org.schemaspy.service.SqlService.connect(SqlService.java:71)
at org.schemaspy.SchemaAnalyzer.analyze(SchemaAnalyzer.java:186)
With schemaspy 5, it works well but I would like to migrate to 6. Have you faced this issue or any clue about why it could fail?
Thanks,
Manuel

Hey,
Not sure what I did but now it works.
I download your code instead of grabbing the .jar and compiled.
I am using now the following command:
java -jar schemaspy-6.0.0.jar -dp force-metadata-jdbc-driver-2.3.jar -debug -t force -u MyUser -p MyPassword -db MyDatabase -norows -s force -o doc -connprops url=https://test.salesforce.com/services/Soap/u/23.0 -cat % -s force


Error Handling

We have some errors result on SendTextMessage which make throw error exceptions and make slow processes. Is there any solution we safe receive these errors.
https://core.telegram.org/method/messages.sendMessage#return-errors
Exception Message: [Error]: PEER_ID_INVALID

We have 403 FORBIDDEN exception when we SendTextMessage to user which blocked the bot.
https://core.telegram.org/api/errors
Exception Message: Response status code does not indicate success: 403 (Forbidden).

Hi,
the links are specific to the normal telegram api, not the bot api.
I'll need to test the case when a bot is blocked, stay tuned...

Scenario 1
We SendTextMessage from a bot to a group, right after removing from group.
{
  "ok": false,
  "error_code": 403,
  "description": "[Error]: Forbidden: bot was kicked from the group chat"
}

Scenario 2
We SendTextMessage from a bot to a unknown channel
{
  "ok": false,
  "error_code": 400,
  "description": "[Error]: Bad Request: channel not found"
}

Scenario 3
We SendTextMessage from a bot to a user, right after stop and blocking bot by this user
{
  "ok": false,
  "error_code": 403,
  "description": "[Error]: Bot was blocked by the user"
}

These errors should be thrown as an ApiException.
Can you catch these?

Exception Handling is a heavy and expensive operation as far as performance is concerned  [ref]

We need receive these errors normally (not with an exception) and make decision based on these received messages. Anyway you can find out more respectful documents about error handling with Condition Checking is so better than Exception Handling
I think SendTextMessage need an output from ResultMessage class like
public class ResultMessage
{
    bool IsOk {get; set;}
    Message? Result {get; set;}
    Error? Error {get; set;}
}

public class Error
{
    int Code {get; set;}
    string Description {get; set;}
}

ok, i need to think about this

These errors should be thrown as an ApiException.
Can you catch these?

i also test it. i can catch any exemptions with ApiException. if there was a exception we catch it like this:
[(Message: System.AggregateException: One or more errors occurred. ---> System.Net.Http.HttpRequestException: Response status code does not indicate success: 403 (Forbidden).
   at System.Net.Http.HttpResponseMessage.EnsureSuccessStatusCode()
   at Telegram.Bot.Api.<SendWebRequest>d__78`1.MoveNext()
   --- End of inner exception stack trace ---
   at System.Threading.Tasks.Task`1.GetResultCore(Boolean waitCompletionNotification)
   at TelegramGateWay.ViaTelegram.SendText() in C:\Users\Admin\Desktop\Projects\SimSend_SVN\trunk\TelegramGateWay\TelegramGateWay\Intermediate\ViaTelegram.cs:line 58
---> (Inner Exception #0) System.Net.Http.HttpRequestException: Response status code does not indicate success: 403 (Forbidden).
   at System.Net.Http.HttpResponseMessage.EnsureSuccessStatusCode()
   at Telegram.Bot.Api.<SendWebRequest>d__78`1.MoveNext()<---
)]
funny thing is in previous releases you return the error description but in new version you just return the error code. i hope you can fix it soon. thank you.

bot.sendMessage('@medve_dev', 'test BOt');
send to username response
Unhandled rejection Error: 400 {"ok":false,"error_code":400,"description":"[Error]: Bad Request: channel not found"}

I think that the implementation without throwing exceptions would result in a silent fail for most users. As this is undesirable, I am against changing to condition checking instead of exception handling.
On the other hand, it would be a good idea to include the error description in the ApiException class.

When async methods like SendTextMessageAsync() are used standalone (without user request) then bot will be frozen if he was stopped or blocked by user. Sync methods like SendTextMessage() works correct in this case.
How can I resolve this?

Dear a-a-k,
First of all, for a new issue, open a new issue.
Secondly, the two functions you are referring to are literally the same. The non Async methods call the Async methods without doing anything. This is probably an issue in your own code.
If you make a new issue and share your code there, we might be able to help you.
Kind regards,
Martijn Verbakel

Dear Matijn, thank you for your response, but what does it mean?

The non Async methods call the Async methods without doing anything.

And I think that this is not new issue conversely this is exactly not handled error problem. Do you not think so? I'm really don't understand what you meant.

@a-a-k this might be a bit late, but the normal method just looks like this:
`public void SendTextMessage(params)
{
SendTextMessageAsync(params);
}
so its not a difference at all.
and the difference is, if you don't wait for the async method to finish with .Wait(), the exception won't be thrown, so i assume you probably DO wait for them and thus get the errors they fire.

And @amastaneh : i don't think changing the errors to statuses that have to be checked first is an option. After all, this is just a c# implementation for the API and not like a whole control panel for a bot that takes over everything for you.

Dear @Olfi01 & @mhverbakel
Thank you for your attention and try to help. The issue is about Condition Checking versus Exception Handling
Google: Condition Checking vs Exception Handling
1- I think logic and performance come with condition checking.
2- Also, In sending mass messages in a loop (or in other similar scenarios) exception after a simple SendMessage make stop the routine process. But return a simple result when the bot is blocked from some users (or groups) is better than throw. Some of programs need continue to sending without exception interrupts.
3- The blocking result or any other situations are simple conditions. Did you think we need throw exception when we face with some conditions or some situations at all?

@amastaneh i think we should indeed throw exeptions. if you really want conditions, just catch those by yourself. Making it conditions would, as @mhverbakel said, result in a silent fail for those user who arent capable of bulding themselves a workaround. Don't you think it would be better to leave them able to determine the cause of whats happening than you not having to code a single try-catch-loop? if youre sending mass messages, just do it like for example
while(condition) { try { //send Message } catch { //handle Exception } }
Since I myself used this library even back since I was a little noob, I prefer simplicity before logical correctness.


Figma updates following Button update

Following the release of Button updates (inclusion of appearance and color options) the following needs to be updated:
Next

Overlay – layers refer to secondary button
Segmented Button – component descriptions mention CTA, Primary, Secondary
Segmented Button – component descriptions mention CTA, Primary, Secondary
Form Field – CTA button mentioned in adornment playground
Toast – QA Playground
System Status – can now be unblocked

Legacy (light and dark)

Pill – needs updating to use new actionable styles
Toggle Button – needs updating to use new actionable styles
Data Grid – QA Playground mentions CTA, Primary and Secondary
Segmented Button – references CTA, Primary, Secondary
Toast – mentions CTA in QA Playground
Split Button pattern – multiple references
Dialog – QA Playground mentions primary button

FYI @bhoppers2008

@bhoppers2008 probably needs splitting up ?

Espresso Goal to be released by September 13th

@bhoppers2008 can you mark the ones you've resolved please!

Frappe Goal: update and sync with code by EOS
@navkaur76 to update above list to tick off the completed work at SOS

status about needs updating @navkaur76

This issue is closed. Updates complete and merged. Ready to publish.


feat: Add support for disabling connection termination for unhealthy targets and AZ DNS affinity

Description
Add dns_record_client_routing_policy to aws_lb and target_health_state to aws_lb_target_group.
Motivation and Context
https://github.com/hashicorp/terraform-provider-aws/pull/34070
https://github.com/hashicorp/terraform-provider-aws/issues/33992
Breaking Changes
No.
How Has This Been Tested?

[x] I have updated at least one of the examples/* to demonstrate and validate my change(s)
[x] I have tested and validated these changes using one or more of the provided examples/* projects

[x] I have executed pre-commit run -a on my pull request

This PR is included in version 9.1.0 :tada:


Augmenting the Documentation

Configure your environment, add:

For windows, Ensure that C:\path-to-anaconda3 is in your PATH environment.
For windows, ensure that you allow scripts to be executed.
a. Set-Execution-Policy -ExecutionPolicy RemoteSigned could be useful. You would need to run powershell as administrator.

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

ID: 542c2bdc-5868-71d5-1d1a-a2c6d61744b0
Version Independent ID: 1bba15a0-227a-8137-a023-90b8df19881b
Content: Create a Python function using Visual Studio Code - Azure Functions
Content Source: articles/azure-functions/create-first-function-vs-code-python.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

@joshkyh Thank you for your feedback! We will review and update as appropriate.

@ggailey777 do you approve this change?

I can no longer repro this issue. #please-close


Can't paste Color Links in prefabs

Sorry to make another issue so soon after the last haha. It's basically the same issue is last time but with pasting color links between ui elements. If you need more info, let me know

No worries, that was a silly oversight on my side. Should work properly now!

It works, but only on one object at a time. Is that intended?

Not sure what do you mean by this, can you elaborate please?

Say I want to copy the color links for a button. If I select 5 buttons and paste the color link, it only pastes to one.

Hm, that wouldn't work, yeah. Now that you mentioned it, you also can't manually link a property for multiple selected objects. I'll see what can be done (probably tomorrow) about it.

Cool, wasn't sure if it were a bug or not. Appreciate the quick responses!

Should be working now! Thank you too for helping making the tool better C:

It's still only applying to one at a time 😅

Guess I misunderstood your problem and fixed a different big bug instead lol. Nooooow I think it works though...

Sweet, pasting to multiple ui objects works now! Thanks :)


Update accelerated-networking-overview.md

Removing unnecessary abbreviation used once in article - As per Microsoft Writing Style guidelines:
https://learn.microsoft.com/en-us/style-guide/acronyms#dont-introduce-acronyms-that-are-used-just-once

Thanks, @huffmatt!


Regenerate EventGrid data plane SDK to include event Schemas for new …

…event types from IotHub, Media Services, Container Registry, Maps, and AppConfiguration services. (#5232)
Merge into a local branch for CI

(message created by the CI based on PR content)

Codecov Report

Merging #5346 into master will increase coverage by 0.01%.
The diff coverage is 56.33%.

@@            Coverage Diff             @@
##           master    #5346      +/-   ##
==========================================
+ Coverage   48.33%   48.34%   +0.01%
==========================================
  Files        6557     6597      +40
  Lines      188652   189091     +439
==========================================
+ Hits        91178    91424     +246
- Misses      97474    97667     +193

Impacted Files
Coverage Δ

...grid/models/storage_blob_created_event_data_py3.py
26.66% <ø> (ø)
:arrow_up:

azure-eventgrid/azure/eventgrid/version.py
100% <100%> (ø)
:arrow_up:

.../azure/eventgrid/models/event_grid_client_enums.py
100% <100%> (ø)
:arrow_up:

...edia_live_event_ingest_heartbeat_event_data_py3.py
27.77% <27.77%> (ø)

...ls/container_registry_artifact_event_target_py3.py
33.33% <33.33%> (ø)

...e_event_incoming_stream_received_event_data_py3.py
33.33% <33.33%> (ø)

...ure/eventgrid/models/maps_geofence_geometry_py3.py
36.36% <36.36%> (ø)

...ent_track_discontinuity_detected_event_data_py3.py
38.46% <38.46%> (ø)

...vent_incoming_data_chunk_dropped_event_data_py3.py
41.66% <41.66%> (ø)

...ent_incoming_streams_out_of_sync_event_data_py3.py
41.66% <41.66%> (ø)

... and 75 more

Continue to review full report at Codecov.

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


Kpi tracker parallelize

Purpose
Decrease the number of network round trips when inserting transactions.
In particular making a query for ID and then inserting two values, waiting for each of those to complete is inefficient. It takes at least 2 * network latency per transaction to insert a transaction. Multiplied by the number of transactions as it used to be.
The changes are

Insert account statistics in a single query without looking up transaction id first.
Execute all transaction insertions at once to remove the network overhead.

Checklist

[ ] My code follows the style of this project.
[ ] The code compiles without warnings.
[ ] I have performed a self-review of the changes.
[ ] I have documented my code, in particular the intent of the
hard-to-understand areas.
[ ] (If necessary) I have updated the CHANGELOG.

Looks nice. We talked about the obsolete overhead of maintaining an index on transaction type for the transactions table. Could we include the removal of this in this PR as well?

Yes, I have some changes after these that I did not push yet. Sorry. Will request another review once I did.


Error Management Improvements

Description:
Errors such as incomplete setups are not clearly flagged, leading to confusion. Skore needs better error handling and guidance when users miss critical steps like setting up views.
User Story:
As a user, I want clear guidance when I make a mistake or skip a setup step, so I can correct my actions without confusion.
Acceptance Criteria:

[ ] Users are blocked from proceeding until views are properly set up.
[ ] Implement default input
[ ] Detailed error messages explain what is missing and how to resolve it.
[ ] Visual indicators for incomplete setups or missing views are added.

Is this epic aimed at improving the score once, or is it an ever-desirable property?
What concrete issues have you found right now about UX oriented error management?

Finally this issue is diluted in other tickets, so it's no longer needed here.

Given Anass's last comment, I'm closing this ticket.


Where can i get access to sparql-gremlin jar file ?

Sorry just a beginner in this . So lot of confusion .

If I am not wrong, you will have do maven build of the source-code, JARs will them be created.

I have clone of the sparql-gremlin , according to you i have to maven build it . Right ?

Yes, *harsh9t[at]gmail[dot]com is my mail. Lets connect there.


[Bug report] Update js package to the latest version

Version
2.1.0
Platforms
dart, Web
Device Model
Any
flutter info
Doesn't matter

How to reproduce?
A lot of packages uses js ^0.7.0, so conflicts with this package
Logs
No response
Example code (optional)
No response
Contact
No response

@CaiJingLong

@CaiJingLong

@CaiJingLong please it's important for us

Yes this is needed

@CaiJingLong

@AlexV525

@CaiJingLong @AlexV525


Codecov works now when action success

Example

example link

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@7629d35). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #18   +/-   ##
=======================================
  Coverage        ?   62.74%
=======================================
  Files           ?        3
  Lines           ?       51
  Branches        ?        8
=======================================
  Hits            ?       32
  Misses          ?       16
  Partials        ?        3

Continue to review full report at Codecov.

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


[CWS] add windows support in the CWS server

What does this PR do?
This is the second part of the PRs for the Windows support. It brings :+1:

Windows events serialization
gRPC server now support Windows
Fixes some model issues
CWS on Windows can now start sending event thanks to the direct sender

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes
On Linux, test that the policy are reloaded with a kill HUP

Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[ ] If applicable, the config template has been updated.

@modernplumbing There is nothing special to QA on Windows with this PR as the model is not ready yet. We only need to validate the linux part.


Create a cache filter

A cache filter would be awesome. See https://github.com/nodejitsu/node-http-proxy/issues/853, https://github.com/tcs-de/nodecache

A cache filter can be easily implemented as a prefilter. When requests arrive if the cache-filter has the data returns it otherwise pass the request to the next step.
What we are thinking is on creating a more intelligent cache filter, that works on that situations where some providers can be down. The scenario we want to follow is:

Request arrive requesting some resource.
If cache has got the resource and its TTL is valid then return it.
If cache hasn't got the resource (or it has gut its TTL is not valid) then pass the request to the next step in the chain.
The request if finally redirected to the provider:
** If data is returned by the provider, it is stored in the cache and returned to the client.
** If data is not returned by the provider (for example, because it fails with 500 error) then return the data stored in the cache filter, although its TTL is invalid. It is better to return something old than nothing. This also required to have a "grace" time, that is, a time while the last stored data will be returned although is TTL is invalid.

This is a bit tricky to implement with current Clyde philosophy, where a filter can make actions before or after a request, but not in both sides.

See filter repository: https://github.com/acanimal/clyde-memory-cache


Tech Arch Optimizations

What are there deliverables?
What are they working on?
How could they automate their processes?

[ ] Meet with the Technical Architecture group
[ ] Brief team

Subject to talk to them:

what are the benefits of standards to the dept?
what should be standardized and what should not?
bi-modal (some technology may be standardized, like large complex and expensive systems, but others not like development tools and stacks since they change too often; moving to micro-services may not have that much impact on skillsets and maintenance, etc.)

Was there a meeting setup? Did we follow up on this? Will leave open but worth a review.


Renderer setprops 20rc1

Start with a description of this PR. Then edit the list below to the items that make sense for your PR scope, and check off the boxes as you go!
Contributor Checklist

[ ] I have broken down my PR scope into the following TODO tasks

[ ] task 1
[ ] task 2

[ ] I have run the tests locally and they passed. (refer to testing section in contributing)
[ ] I have added tests, or extended existing tests, to cover any new features or bugs fixed in this PR

optionals

[ ] I have added entry in the CHANGELOG.md
[ ] If this PR needs a follow-up in dash docs, community thread, I have mentioned the relevant URLS as follow

[ ] this github #PR number updates the dash docs
[ ] here is the show and tell thread in plotly dash community

test


[BUG] common subroutine cannot be called from subroutines having different scopes

Please fill all required sections
Kind of proposals

[]Parser
[x]Linter
[]Simulator
[]Testing

Describe the problem
It is not possible to include/call a same custom subroutine from two subroutines having a different scope. This is because falco always defines a scope for a parsed subroutine. But I am not sure it is supposed to be the case in Fastly.
VCL code that cause the problem / reproduceable
Let's say I have:
$ cat common.vcl

sub mytest BOOL {
  return true;
}

And another "main" vcl file:
$ cat main.vcl

include "common";

sub vcl_recv {
    declare local var.myvar BOOL;
    set var.myvar = mytest();
}

sub vcl_deliver {
    declare local var.myvar BOOL;
    set var.myvar = mytest();
}

And so when I lint my main.vcl file, I receive the following error.
[ERROR] Function "mytest" is not available in scope DELIVER\nSee reference documentation

https://fiddle.fastly.dev/fiddle/f5243e1d
Expected behavior
The above works in Fastly and does not raise errors.
I see that the reason is that falco always assigns a scope to a subroutine. So if it cannot recognize the scope from a suffix or an annotation, it will assign a recv by default.
https://github.com/ysugimoto/falco/blob/c3ea5168b4f2cff0c4388b09954dd7ec6c378c76/linter/helper.go#L369
In Fastly, there is this "init" scope which seems to be more of an "unscoped" value and will support having subroutines callable from anywhere.
If I relate to the Fastly documentation:
https://developer.fastly.com/reference/vcl/subroutines/
This special mention on that page implies that subroutines are callable from any scope while it uses compatible objects.

IMPORTANT: The validity of a subroutine to the compiler depends on where it is called from. A subroutine called from vcl_recv may contain only code that would be valid within the vcl_recv subroutine.

Desktop (please complete the following information)
required

OS: Ubuntu 22.04
Shell bash

Thanks a lot for your help, and congrats on this great work that helps a lot when working with Fastly locally.

Hi @lobre , thank you for reaching out to our project 👋
It seems to be documented about user subroutine scope recognition here: https://github.com/ysugimoto/falco/blob/main/docs/linter.md#user-defined-subroutine
So you can put an annotation onto your custom subroutine like:
$ cat common.vcl

# @scope: recv,deliver
sub mytest BOOL {
  return true;
}

Then falco could recognize custom subroutine scope as you want 👍


[core] Batch small changes

Used this PR to push small changes to a few things, most notably hiding the Designer job ad while it isn't open 😅


[Visualize Bug] AssetLoader can not be properly initialized and fail to load "box.bam"

Hey Metadrive team,

The version of MetaDrive: <IP_ADDRESS>. (May 8).
Try to use off-screen render ("top-down view")

I am trying to visualize the checkpoints (Nodes) in your marl_envs.
While simply turning on both self._show_navi_info and show_navi_mark flags in BaseNavigation Class,  the following line never works.
# in the `metadrive/component/vehicle_navigation_module/base_navigation.py`
if show_navi_mark:
    navi_point_model = AssetLoader.loader.loadModel(AssetLoader.file_path("models", "box.bam"))
    navi_point_model.reparentTo(self._goal_node_path)

# Error Message:
# AttributeError: 'NoneType' object has no attribute 'loadModel'

I tried to add AssetLoader.init_loader(self.engine) before calling env.engine.reset(), but the error message is still there.
Meanwhile, I found that so many visualizing functions might not be maintained in the current MetaDrive? (or at least, the declaration of those functions can not being found...)
# Can not found the implementation of .setPos() and .setH() functions
self._goal_node_path.setPos(panda_vector(pos_of_goal[0], pos_of_goal[1], 1.8))
self._goal_node_path.setH(self._goal_node_path.getH() + 3)

Hope to hear from you guys soon!

Hi, Zi-ang,
Sorry for the late reply. Are you using the 2D top-down renderer? The Navi mark is only available using the 3D renderer.
Quanyi

Hey Quanyi,
Yes, you are right. I am using the 2D top-down renderer. Thanks for your reply!

Hey Quanyi,
Yes, you are right. I am using the 2D top-down renderer. Thanks for your reply!
Btw, do you have any suggestion for adding the Navi mark in the 2D top-down render?

Hi, Zi-ang,
Sorry for the late reply. Are you using the 2D top-down renderer? The Navi mark is only available using the 3D renderer.
Quanyi

Hey Quanyi,
Yes, you are right. I am using the 2D top-down renderer. Thanks for your reply!
Btw, do you have any suggestion for adding the Navi mark in the 2D top-down render?

Yes. I believe you can do some customization in class TopDownRenderer. Just refer to the code for drawing the vehicle, which tells you how to draw a box in the scene. Then every frame you can access the checkpoint position from vehicle.navigation and draw a box representing the checkpoint.
To be specific, you should modify this function:
https://github.com/metadriverse/metadrive/blob/4d77e619b2bc53397c6836e26a17dfc43b5d8cd8/metadrive/obs/top_down_renderer.py#L339
Just write a new for loop or modify an existing one to traverse all vehicles so that you can get the navigation point position for each vehicle and use a similar way to draw it on the 2D renderer.


mrsiproc and wftfi -matlab path fix

matlab path fixed and suggested changes for more flexible matlabVersion usage

I wanted for this to appear in the pull request mrsiproc and wftfi #169

@stebo85 @astewartau @aswinnarayanan Could someone please merge @korbinian90 's updates? :)


Update ProcessFullDirectory.py to accept automator_random.sh

Update ProcessFullDirectory.py to accept automator_random.sh

Just created a pull request for the update!


Adds environment variable to replace host on the Share Secret Link page.

Updated main.py and README.rst. HOST environment variable can be used to replace the host on the Share Secret Link page.

isn't this solved in 143 with the HOST_OVERRIDE variable?


Mount user provided CA bundle to the container

ADO may fail when calling OCM when a cluster-wide proxy is enabled with a user-provided trust bundle.
Updated the configuration to mount trusted CA bundle to the addon-operator-manager container
JIRA: MTSRE-1236

@thetechnick Could you please review thanks

/retest

/retest


Try but fall

Hi!
Your project looks interesting.
I followed your instructions to create a docker, but unfortunately I didn't end up with anything that works:
I don't have any files in my /media folder, even if I changed the cron data or added a new youtube channel.
P.S
I have a gui in localhost:5005

Go to yout ytdlp2STRM folder and run:
python cli.py --media youtube --params direct
and paste the output here.

Hi, this the output :
/opt/ytdlp2STRM$ sudo python cli.py --media youtube --params direct
Traceback (most recent call last):
File "/opt/ytdlp2STRM/cli.py", line 3, in
import config.plugins as  plugins
File "/opt/ytdlp2STRM/config/plugins.py", line 2, in
from plugins.youtube import youtube
File "/opt/ytdlp2STRM/plugins/youtube/youtube.py", line 1, in
from flask import stream_with_context, Response, send_file, redirect
ModuleNotFoundError: No module named 'flask'

@VLiulian  Did you try installing the requirements again? Make sure you use sudo or maybe install requirements manually.

Go to yout ytdlp2STRM folder and run:
python cli.py --media youtube --params direct
and paste the output here.

I have a similar issue and this is what I get
Running youtube with ['direct'] params
Preparing channel https://www.youtube.com/user/xbox
/user/xbox
Working channel URL: https://www.youtube.com/user/xbox
Getting channel ID...
Traceback (most recent call last):
  File "C:\ProgramData\ytdlp2STRM\cli.py", line 64, in <module>
    main()
  File "C:\ProgramData\ytdlp2STRM\cli.py", line 61, in main
    r = eval("{}.{}.{}".format("plugins",method,"to_strm"))(*params)
        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\ProgramData\ytdlp2STRM\plugins\youtube\youtube.py", line 618, in to_strm
    channel_strm(
  File "C:\ProgramData\ytdlp2STRM\plugins\youtube\youtube.py", line 395, in channel_strm
    yt = Youtube(
         ^^^^^^^^
  File "C:\ProgramData\ytdlp2STRM\plugins\youtube\youtube.py", line 25, in __init__
    self.channel_id = self.get_id()
                      ^^^^^^^^^^^^^
  File "C:\ProgramData\ytdlp2STRM\plugins\youtube\youtube.py", line 81, in get_id
    self.get_id(
TypeError: Youtube.get_id() takes 1 positional argument but 2 were given

I've made a small fix in the YouTube class,. It's not typical, but please try it with the latest commit may be works. Additionally, it would be helpful if you could execute the following yt-dlp command in your terminal and paste the output here:
yt-dlp --compat-options no-youtube-channel-redirect --compat-options no-youtube-unavailable-videos --restrict-filenames --ignore-errors --no-warnings --playlist-start 1 --playlist-end 1 --print channel_url https://www.youtube.com/user/xbox
Regarding the Jellyfin media library setup, you only need to add the strm_output_folder that you have configured in the plugin. For example, on my Windows machine, I have set the strm_output_folder in the YouTube plugin to D:/Youtube. This means that all Youtube STRM files will be created inside this directory. Therefore, D:/Youtube is the folder I need to add to Jellyfin as a media library.


How to do something bigger with connections?

Currently, there is forceFetch on router, so it keeps all the connections updated, but how to do subscription without forceFetch? With mutations, user can have fatQuery to mark what can change, but what with subscription?

After I slept on that I understand that it's not good to use fatQuery in subscribes, but it's better to have more little suscribes used :) Closing this and thanks for your work 👍


belindas-closet-android_3_152_placeholder-images

Resolves a number of smaller issues
#147
Removes name field on product card and name field on product edit page, (Though we might consider replacing it later with a productType dropdown so admins can change the category a product fits into. As it was, it was just a text editable field which doesn't make sense because productType/name was an Enum
#148
Product cards with a description too long could end up being too wide. This could look off putting  when set beside another, narrower product card. Now there is a max width for the cards, However, I'm not sure if the method I used will hold up across devices with different pixel densities, Have to test it more extensively.
#149
On the products detail page, descriptions are now limited to 2 lines max before they're cut off. This can be tweaked obviously. The full description is still visible on the individual product's page and when editing
#150
included an id field into our mock Datasource to bring it further in line with what our final DB is going to look like. Also, edited the navigation to individual product pages to now use these IDs so products in the same category don't all go to the same individual product page like before.
#151
Refactored the capitalization function on the Category title on the product detail page to use a different method of capitalization. The old method was deprecated. Also removed the category name above the product card in the individual product page as it seemed unnecessary when only viewing one item as opposed to all the items in a specific category.
#152
Added a placeholder image and set categories and individual products to use this image as a default if no other image is provided. We can consider this a placeholder placeholder image and can change it later.

Hello, Nickolas! Thanks for squashing those bugs. I will do an actual review of this later but in the future can we make sure not to resolve so many issues in a single PR? Just makes it easier to suss everything out when it's separate. Same idea behind unit testing--keeping things separate makes it easier to figure out what went wrong if something goes wonky.
But I totally get that it can be annoying/it's easy to forget to push/pull every time a single task has been completed. It's been helpful for me to practice doing that even when I'm working on solo personal projects.

Looks great!


fix local multiplayer test example

.connect() has been renamed to .start() in 2f86d92

@googlebot I signed it!


Upgrade org.apache.maven.plugins:maven-surefire-plugin 3.0.0 -> 3.1.0

This PR contains the following updates:

Package
Type
Update
Change

org.apache.maven.plugins:maven-surefire-plugin
build
minor
3.0.0 -> 3.1.0

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

Suggested commit message:
Upgrade Surefire 3.0.0 -> 3.1.0

See https://github.com/apache/maven-surefire/compare/surefire-3.0.0...surefire-3.1.0

IMHO yes :)


Enhance Rails/TimeZone to accept methods with the :in timezone option

Closes #821.
Time.{now,at,new} were extended with the supported :in timezone option.

Updated.

Thanks!


TestRTC - Improve bug reporting mechanism

Currently working on this.

The main work is tackled! Thanks @KaptenJansson
Will mark as closed and we open other bugs for other small improvements.


Fails when ran with 'use_strict'

I run node with the --use_strict option. This plugin fails with the message:

SyntaxError: Unexpected eval or arguments in strict mode
at thenify (/Users/dev/work/node_modules/thenify/index.js:17:15)

This prevents it and packages that depend on it from being used in certain contexts, such as in the Atom editor. Why is eval needed?

I personally use es6-promisify now. It doesnt use eval and is use_strict compatible. Ive even issued a PR to a package that I needed to use but couldnt due to the use_strict issue.


SHA-256 hash of over 4GB file is incorrect (doesn't match openssl)

I think a 32 bit integer is overflowing.  If I had to guess I would say it is the variable called TOTAL.  I will try to get a standalone test for this posted to this ticket when I get time.
SHA-256 is supposed to be able to handle messages of almost 2^64 bits.

It's definitely overflow of the TOTAL variable. Thanks, I'll fix it ASAP.

Hey! Could you please verify the bug is fixed now?

I tried 1GB, 5GB, and 10GB random files and they all have correct hashes now.  Good job!
Can you make a release of this so I can bower install this guy?


Graph generation stuff

will add longer description soon
basically cleaned up a lot of primaite stuff to make it easier to generate networks and serialize them as laydowns
for now there is no graph generator in place but there will be one with the next pr
but the infrastructure is all there now

@IntelliumSamwel is this your active branch still? just checkin


Update AWS IoT section to include new features from re:Invent 2017

I can start working on this, would anyone else be interested in joining me in this effort?

@lynnlangit It would be great if you could work on this.

I can - but not until Dec due to current work obligations


Adds csvkit 0.9.1 along with all dependencies.

csvkit is a suite of utilities for converting to and working with CSV, the king of tabular file formats.
This PR also includes python modules: dbf, dbfread, agate, agate-dbf, agate-excel, pytimeparse
Further, since csvkit uses pinned versions of dateutil 2.2 and openpyxl 2.2.0-b1, these are also included.
Things done

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

[x] NixOS
[ ] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

I am not quite sure what the failure in travis is. The error I get is error: opening directory ‘/tmp/nix-build-python2.7-pytest-2.8.6.drv-0/pytest-of-travis/pytest-0/testdir/test_cache_writefail_permissions0/.cache’: Permission denied but this does not seem related to my changes, and I am unable to reproduce this error in my machine. Any help would be appreciated.


Shorten README and add obvious links to installation and configuration

The README was made before the documentation was even started. Rewrite the whole document and provide links to the documentation, especially to installation and configuration. Ideally create a nice gif that showcases what the connector is all about.
Most connectors are very short on documentation (or lacking any documentation whatsoever), and users, including myself, are accustomed to look for a docker-compose.yml file as a reference. This project doesn't have one, so help users navigate to the docker-compose example files and configuration reference.

I added an "Installation" section, referring to the documentation. I'll leave it at that for now.


Stop using ApMon in WMAgent jobs

Impact of the bug
WMAgent
Describe the bug
Service has been retired, even though we still try to send job information (via ApMon) to cms-jobmon.cern.ch. Error in the condor.out log is something like:
======== WMAgent Run the job starting at Sat Nov  2 11:39:54 GMT 2019 ========
Sat Nov  2 11:39:55 2019: ApMon[ERROR]: Error resolving cms-jobmon.cern.ch: [Errno -2] Name or service not known
Sat Nov  2 11:39:55 2019: ApMon[WARNING]: Not sending parameters since no destination is defined.
Sat Nov  2 11:39:55 2019: ApMon[DEBUG]: Closing UDP socket on ApMon object destroy.
Sat Nov  2 11:39:55 2019: ApMon[ERROR]: Error resolving cms-jobmon.cern.ch: [Errno -2] Name or service not known
....

How to reproduce it
none
Expected behavior
If we want to keep using this implementation, we might just pass/skip any job update.
Additional context and error message
AFAIK the service has been retired in October 2019.

Do you need any help on this issue? We have seen this issue when we resume paused jobs. I don't know where are we supposed to send the job information instead of ApMon.

I think the initial idea is to keep that code around. However, it would be better to just skip any of those job updates in my opinion. Anyways, whatever we decide to do, that will require changes to the source code.
If you feel like making those changes, please go ahead and make a pull request!

Starting with ReqMgr2 HG2002 (1.3.0.pre6), the PerformanceMonitor plugin has been disabled, so production jobs will no longer report to the old dashboard, see:
https://github.com/dmwm/WMCore/pull/9510

New issue created: https://github.com/dmwm/WMCore/issues/9536
closing this one


Port action related tests to the new module smart-actions

The related tests should be moved to the new place.

This was already done.


Use capitalized ID in the doc

As per https://github.com/godotengine/godot/pull/69695#pullrequestreview-1208020708
The remaining lowercase ids in the documentation are part of the sample code.

Thanks!


Show data as percentage growth

It'd be good to be able to show the data in terms of daily change, rather than absolute accumulation.

Added confirmed_delta in https://github.com/zbraniecki/covid19-timeseries/commit/e8cd784ba7a842d6ef78273db7a20bef3bbfe4b3 and active_delta in https://github.com/zbraniecki/covid19-timeseries/commit/e4a9c20acd778d4d95e4025d94557d1ab6f0c08d


Update autorest version to prerelease 3.0.0-alpha.20240417.2.x

Triggered from pull request: https://github.com/Azure/autorest.csharp/pull/4185

Close in favor of #43872


unclear input field

in registration form of beneficiary the input field 'choose file' is unclear. it does not tell which/what file to choose leading to confusion and unclear objective of that field.
solution-
We can add a label to field which will tell what file to choose.

please assign this issue to me under gssoc'23

what is the web route to this page. can you specify.


Simplify consumer class by removing exchanges

RabbitMQ exchanges are only relevant to the publisher of the messages, so its a
bit weird for the consumer stuff to create these.
They're only used in one place for creating test infrastructure.
Simplifying the consumer makes it easier to change the search infrastructure.
At the moment this is difficult because there is a lot of logic for
constructing permission strings.
The ultimate goal is to separate queue creation so we can set up multiple
queues per consumer.
Trello: https://trello.com/c/k2L67c4a/272-cleanup-puppet-module-for-declaring-message-queue-consumers

Closing in favour of https://github.com/alphagov/govuk-puppet/pull/6445


deps: Don't expose backon to users directly

Let's better to keep backon as internal implementation details. Don't expose to user directly.
By introducing this change, our users don't need to depend on backon or use backon::Backoff anymore.

Implemented.