ClassAttributesSeparationFixer - add option for no new lines between properties

Implementation for https://github.com/FriendsOfPHP/PHP-CS-Fixer/issues/4755
Changes the class_attributes_separation option to take options for each type (const, method or property. The options can be either one or none, indicating one or no spaces between the types.
Example
A configuration like this:
'class_attributes_separation' => ['elements' => ['property' => 'none']],

Would change this example code:
<?php
class A
{
    private $a = null;

    public $b = 1;

    function A() {}
}

to this:
<?php
class A
{
    private $a = null;
    public $b = 1;

    function A() {}
}

I get this error when running the new test I added. What does it mean?

This is because the unit tests for fixers first check that the input code is changed to the expected code then it checks that the expected code is not changed again when used as input. It means the fixer changes code that is not expected to change.

@juliendufresne  @kubawerlos Thanks for checking this PR, with the help of @ruudk this PR is now ready for review.

@kubawerlos Thanks! Fixed

@keradus @GrahamCampbell What do you think of this? Would love to be able to remove all those empty newlines from our project 🙏

@adri Before we merge this PR, could you please rebase and squash commits to keep one commit per authorship?

Thanks @adri and @ruudk , great to have this in :)
Hope you guys @ TicketSwap are holding up and get through the lockdown in NL, love the service you provide!

Sorry missed your comment @julienfalque and already compiled the list of PR te merge. Sadly the merge script didn't squash the commit, but we can do without it.

Thanks for merging this. I see this is merged to master. Will this be released as 2.17.x?

Hope you guys @ TicketSwap are holding up and get through the lockdown in NL, love the service you provide!

@SpacePossum Thanks, hope it too 😊

yeah this will be in 2.17.0 :)

@SpacePossum What's the ETA for 2.17.0?

My hope is within two months, however it depends on what we find during regression testing it and the time the maintainer has, maybe not ideal but we shouldn't rush it. People can help out a lot by using the master version for now and report any bugs they find on their repo's, as I think 2.17 is mostly feature-complete by this point.

When properties have comments, the newlines are also removed. I don't think it should behave like that.
'class_attributes_separation' => ['elements' => ['property' => 'none']],

class Entity
{
    /**
     * @ORM\Column(name="id", type="uuid")
     * @ORM\Id
     */
    private UuidInterface $id;

    /**
     * @ORM\Column(name="description", type="text")
     */
    private string $description = '';
}

gets converted to
class Entity
{
    /**
     * @ORM\Column(name="id", type="uuid")
     * @ORM\Id
     */
    private UuidInterface $id;
    /**
     * @ORM\Column(name="description", type="text")
     */
    private string $description = '';
}

What do you think? How should this behave?

I would prefer keeping the empty line when there is a DocBlock comment. Same with constants and methods.

Indeed, that's also what I expected, so maybe it's a bug in this PR? I don't know enough from the internals to know where this responsibility resides.

I would expect it to be removed as the configuration chosen is none - maybe we need new config option like none_where_there_is_no_phpdoc_before_and_one_where_is_phpdoc_before.

I agree with @kubawerlos here, for none I expect no exceptions. The new option is fine by me, although maybe a shorter name for it can be found.

@SpacePossum @kubawerlos @julienfalque Implemented in https://github.com/FriendsOfPHP/PHP-CS-Fixer/pull/5060. What do you think?


Add interactive sample

This is effectively what .NET does for our samples: we write samples as tests and extract from them snippets we include in our packages' samples/ directory. This makes sure that our samples actually compile and run - something they didn't always do when I first started a few years ago.
Alternatively, I long ago prototype an idea about using Jupyter Notebooks, but you had to install additional kernels (think: language processors). It wasn't a good experience for anything other than Python examples. However, the Polyglot Notebooks (formerly .NET Interactive Notebooks) extension for VSCode makes this easier, and GitHub's rendering has improved as well. See https://github.com/Azure/azure-sdk-for-net/blob/56530de2487b10ea2cb0d6a0ccda56b2b3ab1636/sdk/keyvault/Azure.Security.KeyVault.Secrets/samples/InteractiveSample.ipynb for the file herein displayed as users might see it online. Combining this with VSCode for the Web (or perhaps a .devcontainer), our samples could be interactive while explaining, and this could be done across multiple languages.
/cc @jongio


Switch to #pragma once

Hey guys,
should we switch from using #ifdef include guards to using "#pragma once"?
#pragma once is widely supported (https://en.wikipedia.org/wiki/Pragma_once#Portability)
and less error prone (as you don't have to manually change the include guard when renaming files).
For some compilers it improves performance (although not for gcc and clang as they have ifdef guard optimizations).

#pragma once is not part of the C++ standard, isn't?
Anyway, it is fine to me,  I am not particularly against it.
+1

+1


Path Based rate limit still rate limiting other paths

helm.sh/chart=kong-2.10.2
kong/kubernetes-ingress-controller:2.3
KongPlugin
apiVersion: configuration.konghq.com/v1 kind: KongPlugin metadata:   name: rate-limiting-mobile config:   minute: 3   limit_by: path   path: /v3/services/register   hour: 10000   fault_tolerant: true   hide_client_headers: false plugin: rate-limiting
route ingress
annotations:
cert-manager.io/cluster-issuer: letsencrypt-integration
external-dns.alpha.kubernetes.io/skip: 'true'
konghq.com/plugins: rate-limiting-mobile
kubernetes.io/ingress.class: kong-ingress-controller
status:
loadBalancer:
ingress:
- ip: x.x.x.x
spec:
rules:
- host: example-host.com
http:
paths:
- path: /
pathType: ImplementationSpecific
backend:
service:
name: example-host
port:
number: 9142
- host: example-host-1.com
http:
paths:
- path: /
pathType: ImplementationSpecific
backend:
service:
name: example-host-1
port:
number: 9142
- host: example-host-2.com
http:
paths:
- path: /
pathType: ImplementationSpecific
backend:
service:
name: example-host-2
port:
number: 9142
Problem is I want to rate limit everything going to  /v3/services/register,  But every path gets rate limited. Any suggestions?

Duplicate of https://github.com/Kong/kubernetes-ingress-controller/issues/2669


Insert Silently

Is your feature request related to a problem? Please describe.
Our app downloads data from a private api where users can download their data and the data the system requires them to have, it has a main 'loading' screen to load this data in using the main connection to the sqlite database. It also has a 'sync' connection to the database that downloads extra items in the background while the user is using the app, there are some fairly complex queries with joins when they are looking at certain 'show' pages that bring in all the related data. This is using a stream so that when the user has completed certain things on these pages the page updates automatically without having to refresh the whole screen.
The main issue we're having is when looking at one of these 'show' pages (which is a watchSingle with joins). There are also futures and other streams on these pages to load in extra data.
This is causing the page to jump/flicker while data is being loaded in the background.
Our example of this is you have a jobs table with a venue id.
The user looks at job 1 which joins in venue 1.
Venue 1 is already in the database as it was downloaded alongside job 1, however when looking at job 1, venue 45 has been downloaded and has caused the page to refresh as the 'venues' table has updated.
Describe the solution you'd like
Is there a way to only listen to certain rows or insert silently when we know the data won't being used on a frontend?

Venue 1 is already in the database as it was downloaded alongside job 1, however when looking at job 1, venue 45 has been downloaded and has caused the page to refresh as the 'venues' table has updated.

Is it really the queries which are slow here (they should run on a background isolate with NativeDatabase.createInBackground)? If the only problem is refreshing the UI when the stream updates, you can use .distinct() with a ListEquality() on the stream or the part of the UI responsible for showing a single venue. If refreshing the UI in response to new data causes the UI glitches, that should hopefully solve that.

Is there a way to only listen to certain rows or insert silently when we know the data won't being used on a frontend?

This feels like solving a problem that shouldn't exist :D In my mind, this should be solved by the subscriber ignoring duplicate rows where possible. If it turns out that this is still hugely inefficient in your case, I can take a look at ways to inhibit some stream updates though.

Thanks for the reply.
Our main issue is that the page refreshes at all, the query is really fast.
As an example our page is setup like this.
StreamBuilder(
           stream: database.venueRepository.get(426),
           builder: (context, snapshot) {
           return  Page Widgets
          }
);

Which is calling
(select(db.venues)..where((tbl) => tbl.id.equals(id))..limit(1)).watchSingle();
Within this stream builder we have other futures/streams active and if anything is added to the venues table while the user is looking at the single venue page it causes a rebuild on the stream and the page to flicker.

I see, can you see if adding a distinct to the stream helpers? E.g. query.watchSingle().distinct(). That will make the resulting stream not emit an item if it's equal to the previous one. You may have to do that on multiple methods with streams causing rebuilds, but I think that should fix the problem.

@simolus3 Why don't we add distinct automatically?
People can use db.tableUpdates if they just want to listen for any changes.


Style injection conflicts with SSR hydration on document

Describe the bug
tl;dr: vite dev injected


Add support for returning all the platforms to the Maven client

Add support for returning all the platforms to the Maven client by accepting 'all' as the Quarkus version classifer

Alright, I'm squashing this and deploying once the build finishes. Thank you!


Add Ctrl + S as a shortcut for saving in Registry Preview

Description of the new feature / enhancement
When pressing Ctrl + S, it should do the same as clicking the "Save file" button
Scenario when this would be used?
To quickly save the file after making changes to it.
Supporting information
No response

Agreed.  It's currently F2 for Save and Shift+F2 for Save as, but I like Ctrl+S and Shift+Ctrl+S better.
Added it as part of #25266


Creating Wireframes

5 hours
Sketching out the basic layout and structure of the app's screens, and identifying the key user flows and interactions

I used, "countdown to anything", "Visual timer" and "event Countdown" for the initial blue print, which is based in addition, remove and edit the timer


Update debiased estimation loss function to accommodate V-pred

This PR:

Updates debiased estimation loss function for V-pred. The previous function was intended only for epsilon. For reference:
https://github.com/kohya-ss/sd-scripts/issues/1058#issuecomment-1916893278
Adds a deprecation notice for scale_v_pred_loss_like_noise_pred.
For reference:
https://github.com/kohya-ss/sd-scripts/pull/934
https://github.com/TiankaiHang/Min-SNR-Diffusion-Training/blob/main/guided_diffusion/gaussian_diffusion.py#L864

cc @feffy380 @sdbds, let me know if I'm missing something.

At the current time I believe you never actually want to use scale_v_prediction_loss_like_noise_prediction, as it was originally a "fix" for an already outdated and wrongly implemented version of MinSNR. Tying it to ZSNR doesn't really make any sense.

You're right. I guess a warning only will have to suffice.

is this correct?

Difficult for me to say affirmatively. I'd prefer waiting for a second or third opinion on it.

Based on this chart, I think there may be some use for scale_v_pred_loss_like_noise_pred as well.

I can remove the deprecation notice if you would like.

Your plots are a bit confusing, I assume "Debiased estimation" is the one in this PR with v_prediction=False, "Debiased estimation vpred" is with v_prediction=True, so far good, but with "SNR" it gets unclear. If you use "Scale vpred like noise vpred" with "SNR weighted loss vpred", then "Scale vpred like noise vpred" definitely doesn't look right to me. The weighting shouldn't decrease with timesteps increasing.
The purpose of this change is to clamp debiased weighting between (0,1) to accomodate for v-prediction loss.

Some A/B/C tests from bluvoll trying to convert eps-based checkpoint to vpred:
MinSNR | broken Debiased | fixed Debiased

Tried to do some math and plot this as well.

I think it's important to see how each weighting strategy changes the effective SNR. However, in practice no one trains v-pred without ZSNR, it would make more sense to plot using ZSNR schedule. Notice that Debiased estimation vpred weighting * SNR(t) overlaps with vpred-like loss:

Debiased estimation vpred weighting: apply_debiased_estimation , v_pred=True
vpred-like loss: scale_v_prediction_loss_like_noise_prediction
MinSNR(5) vpred: apply_snr_weight, min_snr_gamma=5, vpred=True
True Debiased estimation weighting: discussed in #1058, basically it's 1/SNR(t)

If there's anything of note here, it's that Debiased+vpred AND vpred-like loss look suspiciously alike to MinSNR+vpred at higher timesteps, which can suggest that scale_v_prediction_loss_like_noise_prediction may (or may not) be useful for Debiased+vpred. Didn't include some options since the readability is already really poor. Either way, discarding scale_v_prediction_loss_like_noise_prediction right away was a poor choice.

Additionally, here's a plot comparing variants of debiased estimation.

Non-vpred variant is kinda useless with that spike at the beginning.
What I'd suggest is to leave scale_v_pred_loss_like_noise_pred as is. V-prediction variant of Debiased estimation seems useful.

Some A/B/C tests from bluvoll trying to convert eps-based checkpoint to vpred on a tiny dataset: MinSNR | broken Debiased | fixed Debiased

It looks good from the results.
For some reason, it seems that debias estimation loss function late in training cause color contamination.

Considering that the original paper was published a long time ago, I thought it would be a good idea to refer to the cited paper for updates.
https://scholar.google.com/scholar?cites=6450976606823846518&as_sdt=2005&sciodt=0,5&hl=zh-CN&oi=gsb

@sdbds

For some reason, it seems that debias estimation loss function late in training cause color contamination.

I'm not exactly sure what do you mean by "color contamination", but if it's about weird color splotches, I do think this is rather strange. My theory is that they appear because of the new prediction target and schedule being wonky at first, and this will eventually go away with sufficient training. In fact, this model was trained on 300k samples using 1/(snr+1) and I don't see any splotches there. It doesn't happen with MinSNR on the test examples likely because MinSNR doesn't rescale neither the mid nor high timesteps, and mid timesteps recieve less training with debiased. Notice how "grey" MinSNR results look. This is only a speculation though, and it may not be true.

I thought it would be a good idea to refer to the cited paper for updates.

I only found these loosely related papers, but looks like they don't focus on what this PR attempts to do at all.

https://www.ecva.net/papers/eccv_2024/papers_ECCV/papers/00328.pdf
https://arxiv.org/pdf/2312.00210


events: Ensure spectators don't have streams and subs information.

Followup on #21995
Relevant conversation - https://github.com/zulip/zulip/pull/21995#issuecomment-1119097931
@timabbott would this work better? I am not sure having an assert is better here vs just enforcing these variables are false for spectators.

Well what I had in mind is having the view function raise a JsonableError if a request is made with these invalid settings, and then an assert in this code path just for readability. (And of course we'd then actually use the values passed in rather than hardcoding).

Well what I had in mind is having the view function raise a JsonableError if a request is made with these invalid settings, and then an assert in this code path just for readability. (And of course we'd then actually use the values passed in rather than hardcoding).

@timabbott should we change the client_gravatar default to False to make it easier for developers?

@timabbott question for you above; marking for potential integration review.

should we change the client_gravatar default to False to make it easier for developers?

This is a question for the "api design" stream. I think the answer is that True is likely the correct default for the preexisting authenticated use case, so I don't really want to change that. (Also, I think we're likely moving in the direction of no longer using gravatar at all, rather than doing more customizations related to it).

I tweaked the strings to save a bit of translation work, and documented the API change (it's minor, but still a change); merging with those changes.
Thanks @amanagr!


Add H5105 Hygrometer Thermometer to Supported Devices

Is your feature request related to a problem? Please describe.
The Govee H5105 is not on the list of supported devices.
Describe the solution you'd like
List Govee H5105 as supported.
Additional context
I just ordered 6 times the Govee H5105 and they seem to work fine. Putted in the battery and the appeared in Auto-Discover and sending data. They are showing up as H5101 / H5102 / H5177

https://github.com/Bluetooth-Devices/govee-ble/releases/tag/v0.29.0


Plugin is disabled on multiselection even for indicators that don't depend on a selection

Encoding/line ending etc. should still be shown even when there are multiple selections
The last check is the reason
https://github.com/jfcherng-sublime/ST-CustomStatusMessage/blob/bcad7311f96174604708ed15a8595e388e9e6103/plugin/listener.py#L41
Think it'd be better to add that check the indicators that should fail on multiselection and then instead of erasing the whole message just make those indicators ""?

This seems to be a difficult issue. What should be shown if I have a template "{line_endings}, Ln {row} Col {col}"?

the only way I can figure out is add another setting like template_multi_selection.


[Bug] [SWCK Helm Chart] release full name doesn't respect the one in values.yaml file (or --set option)

Search before asking

[X] I had searched in the issues and found no similar issues.

Apache SkyWalking Component
Helm Chart (apache/skywalking-kubernetes)
What happened
We provide the nameOverride and fullnameOverride for users to customize the service/Pod names
https://github.com/apache/skywalking-kubernetes/blob/670d7053aab8dfbed9e325f9f3345dc6696e6352/chart/operator/values.yaml#L55-L56
but we also add a very long suffix (like -controller-manager-metrics-service) to the fullnameOverride in some resources
https://github.com/apache/skywalking-kubernetes/blob/670d7053aab8dfbed9e325f9f3345dc6696e6352/chart/operator/templates/service.yaml#L22
This can easily make the final name exceed the maximum allowed length (usually 64) in Kubernetes and cause the deployment fail.
We should trim the final name to 64 after we add those suffices, or remove those suffices wherever possible, for example, I don't think we need suffix -mutating-webhook-configuration in the MutatingWebhookConfiguration here
https://github.com/apache/skywalking-kubernetes/blob/670d7053aab8dfbed9e325f9f3345dc6696e6352/chart/operator/templates/webhook.yaml#L22
What you expected to happen
no matter how long the release name is given, the deployment should be in success
How to reproduce
Example:
helm -n adapter upgrade --install test-a-very-long-name-that-should-fail ./chart/operator

Anything else
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

@kezhenxu94 Thanks for pointing out the bug, I will fix it later.


Import error when running recently added 'discrete_cql_online.py' from examples/cql

Hi, when attempting to run the example on the Discrete CQL algorithm (online) discrete_cql_online.py I experienced an import issue for the DiscreteCQLLoss class from torchrl/objectives/utils.py.
Issue can be reproduced both running directly the script or isolating the import in a jupyter notebook as below:
 from torchrl.objectives import CQLLoss, DiscreteCQLLoss, SoftUpdate

ImportError                               Traceback (most recent call last)
Cell In[1], line 1
----> 1 from torchrl.objectives import CQLLoss, DiscreteCQLLoss, SoftUpdate
ImportError: cannot import name 'DiscreteCQLLoss' from 'torchrl.objectives'

I tried installing both torchrl and torchrl-nighly; other than this tiny issue the library is working perfectly for me.
The CQLLoss class is correctly imported.
I found that the Discrete CQL had been recently added:
https://github.com/pytorch/rl/pull/1666
Thank you!

can you print this
$ pip list | grep torchrl

You may have both torchrl and torchrl-nightly in your environment

Second likely explanation: you're running the code from the torchrl root of the repo, but your git clone isn't updated, and when you run the script it tries to get a module that is not in the local directory.
If you're running it from /path/to/rl try to move somewhere else (eg, /path/to/rl/examples/cql)
I tried to reprod locally but couldn't. I also checked and the class is present and accessible in the nightlies on pypi...

Classification: Confidential
Actually I downloaded both utils.py and discrete_cql_online.py from git online and put them in a local folder prior to attempt running the example. I did not clone the whole repo.
Anyway I now tried to clone the rl repo and run the script as you instructed - keeping the pypi torchrl-nightly installed.
Result is the same, here is the printout (just pruned some paths):
@.:~/TORCHRL/GITCLONE/rl/examples/cql$ python discrete_cql_online.py
UserWarning: Failed to import torchrl C++ binaries. Some modules (eg, prioritized replay buffers) may not work with your installation. If you installed TorchRL from PyPI, please report the bug on TorchRL github. If you installed TorchRL locally and/or in development mode, check that you have all the required compiling packages.
warnings.warn(EXTENSION_WARNING)
Traceback (most recent call last):
File ".../TORCHRL/GITCLONE/rl/examples/cql/discrete_cql_online.py", line 25, in
from utils import (
File ".../TORCHRL/GITCLONE/rl/examples/cql/utils.py", line 35, in
from torchrl.objectives import CQLLoss, DiscreteCQLLoss, SoftUpdate
ImportError: cannot import name 'DiscreteCQLLoss' from 'torchrl.objectives' (.../lib/python3.11/site-packages/torchrl/objectives/init.py)
@.:~/TORCHRL/GITCLONE/rl/examples/cql
The highlighted user warning was also there before
From: Vincent Moens @.>
Sent: venerdì 1 dicembre 2023 16:06
To: pytorch/rl @.>
Cc: Paolo Spallaccini @.>; Author @.>
Subject: Re: [pytorch/rl] Import error when running recently added 'discrete_cql_online.py' from examples/cql (Issue #1725)
[CAUTION: This Email is from outside the Organization. Unless you trust the sender, Don't click links or open attachments as it may be a Phishing email, which can steal your Information and compromise your Computer.]
Second likely explanation: you're running the code from the torchrl root of the repo, but your git clone isn't updated, and when you run the script it tries to get a module that is not in the local directory.
If you're running it from /path/to/rl try to move somewhere else (eg, /path/to/rl/examples/cql)
I tried to reprod locally but couldn't. I also checked and the class is present and accessible in the nightlies on pypi...

Reply to this email directly, view it on GitHubhttps://github.com/pytorch/rl/issues/1725#issuecomment-1836272937, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AENWEPWPZS6FJHHMXMJ3XNDYHHW3VAVCNFSM6AAAAABAC5P6BWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMZWGI3TEOJTG4.
You are receiving this because you authored the thread.Message ID: @.***>
::DISCLAIMER::

The contents of this e-mail and any attachment(s) are confidential and intended for the named recipient(s) only. E-mail transmission is not guaranteed to be secure or error-free as information could be intercepted, corrupted, lost, destroyed, arrive late or incomplete, or may contain viruses in transmission. The e mail and its contents (with or without referred errors) shall therefore not attach any liability on the originator or HCL or its affiliates. Views or opinions, if any, presented in this email are solely those of the author and may not necessarily reflect the views or opinions of HCL or its affiliates. Any form of reproduction, dissemination, copying, disclosure, modification, distribution and / or publication of this message without the prior written consent of authorized representative of HCL is strictly prohibited. If you have received this email in error please delete it and notify the sender immediately. Before opening any email and/or attachments, please check them for viruses and other defects.


Updates to support layout views

Found some minor things to correct from your PR and thought I'd just do them as I was experimenting with your changes. If you merge these into this branch, they'll get added to your PR and we can continue discussing/reviewing.
Thanks again!

@jedireza thanks for the replies. I've merged the PR, please proceed as you best see fit.


onioncircuits: 0.7 -> 0.8.1

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
[x] 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.

I don't have a darwin machine available, so somebody else would need to do that.

@nagy Thank you! Could you try removing broken = stdenv.isDarwin; and check if OfBorg successfully builds this package on darwin-*?

darwin has now built successfully.


Update locale-ru.json

Could you create another pull request to merge it to develop branch?

@valzav, @reilag can just change this pull request by clocking the Edit button at the top to target it at develop branch.


⚠ Bofrost.fr has degraded performance

In 31fc3fd, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 801 ms

Resolved: Bofrost.fr performance has improved in 70182b2.


Breaking change to d3.forceSimulation in Version 4.12.1

Hey!
I'm new to the d3 library and might have discovered a bug with the forceSimulation in the latest version.
For me some of the examples aren't working with the latest Version of d3 js.
Here is a list of them:

https://bl.ocks.org/mbostock/4600693
https://bl.ocks.org/HarryStevens/bc938c8d45008d99faed47039fbe5d49
https://bl.ocks.org/mbostock/aba1a8d1a484f5c5f294eebd353842da

I tested these with the following Browsers:
Firefox 57.0.2
Google Chrome 63
They seem use the d3 version from https://d3js.org/d3.v4.min.js which currently is 4.12.1. Using that source produces the following error:

Uncaught TypeError: Cannot read property 'force' of undefined at (index):28

The issue seems to be with function calls like these:
d3.forceSimulation().force(...)
When I use version 4.12.0 from https://cdnjs.cloudflare.com/ajax/libs/d3/4.12.0/d3.min.js everything runs smoothly.
Unfortunately I don't (yet) have sufficient knowledge of the library to narrow it down further, but something broke between version 4.12.0 to 4.12.1.

This appears to be a bug in UglifyJS, which was updated in the 4.12.1 release. It is rewriting the return statement of d3.forceSimulation and dropping the assignment to simulation = .

Okay - is downgrading that library an option or do we have to wait for them to fix the issues?

Filed as mishoo/UglifyJS2#2666. I have pushed 4.12.2 which reverts to an earlier version of UglifyJS.


Use Gzip compression.

Remove assetsUrl
Bump version to scallop 1.0

@aquamatthias looks good. Just a small question, can we remove welcomeFile here: Line 40


'add_slot' failing in Windows 11 installation

Here is my output. I would appreciate any help in getting the tool working.
'_UserObject' object has no attribute 'add_slot'
Traceback (most recent call last):
  File "F:\Users\Paul\AppData\Local\Programs\Python\Python311\Lib\site-packages\basic_pitch\predict.py", line 132, in main
    predict_and_save(
  File "F:\Users\Paul\AppData\Local\Programs\Python\Python311\Lib\site-packages\basic_pitch\inference.py", line 383, in predict_and_save
    model = saved_model.load(str(model_path))
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "F:\Users\Paul\AppData\Local\Programs\Python\Python311\Lib\site-packages\tensorflow\python\saved_model\load.py", line 912, in load
    result = load_partial(export_dir, None, tags, options)["root"]
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "F:\Users\Paul\AppData\Local\Programs\Python\Python311\Lib\site-packages\tensorflow\python\saved_model\load.py", line 1042, in load_partial
    loader = Loader(object_graph_proto, saved_model_proto, export_dir,
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "F:\Users\Paul\AppData\Local\Programs\Python\Python311\Lib\site-packages\tensorflow\python\saved_model\load.py", line 223, in __init__
    self._load_all()
  File "F:\Users\Paul\AppData\Local\Programs\Python\Python311\Lib\site-packages\tensorflow\python\saved_model\load.py", line 320, in _load_all
    self._load_nodes()
  File "F:\Users\Paul\AppData\Local\Programs\Python\Python311\Lib\site-packages\tensorflow\python\saved_model\load.py", line 529, in _load_nodes
    slot_variable = optimizer_object.add_slot(
                    ^^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: '_UserObject' object has no attribute 'add_slot'

Thanks

Hi. This is a problem we discovered recently as well. It's due to the tensorflow 2.16 release where keras 3 is now used for the saved model format. Currently the main branch has tensorflow pinned to <2.16 which should work until we push the new minor release (once we're sure there are no bugs in main).

Hi @pdduggan ,
In the future, it is better to provide at least the following specifications as this can be a few possible issues. A few examples of what should be provided is (but exclusively)

System Specifications (CPU + Architecture, GPU + Architecture, RAM, and possibly other requirements depending)
Operating System (and terminal environment Win10 CMD or MacOS TERM, etc)
Python Environment + Pip Environment (VirtualEnv or Pyenv recommended)
Related pip module versions (Tensorflow, basic-audio)
Development Environment (Jupyter Lab, VSCode, Visual Studio Community/Professional, vim, ...)
Reproduction Steps
Screenshots / Terminal logs
Other related information

A few things that I have previously considered when setting this (or other Machine Learning projects is the following)
OS: Windows 11 - Command Line Terminal VIA VSCode
CPU & GPU: AMD Ryzen 5 PRO 7540U + AMD Radeon 740M Graphics (CPU-only, GPU disabled)
Python Version: 3.10.11 through Pyenv
PIP Versions:
(tensorflow-cpu<2.11 | basic-pitch | setuptools) -> installations required
Package                      Version

absl-py                      2.1.0
asttokens                    2.4.1
astunparse                   1.6.3
audioread                    3.0.1
basic-pitch                  0.2.6
cachetools                   5.3.3
certifi                      2024.2.2
cffi                         1.16.0
charset-normalizer           3.3.2
colorama                     0.4.6
comm                         0.2.2
debugpy                      1.8.1
decorator                    5.1.1
dm-tree                      0.1.8
exceptiongroup               1.2.0
executing                    2.0.1
flatbuffers                  24.3.7
future                       1.0.0
gast                         0.4.0
google-auth                  2.28.2
google-auth-oauthlib         0.4.6
google-pasta                 0.2.0
grpcio                       1.62.1
h5py                         3.10.0
idna                         3.6
ipykernel                    6.29.3
ipython                      8.22.2
jedi                         0.19.1
joblib                       1.3.2
jupyter_client               8.6.1
jupyter_core                 5.7.2
keras                        2.10.0
keras-nightly                3.1.0.dev2024031503
Keras-Preprocessing          1.1.2
lazy_loader                  0.3
libclang                     16.0.6
librosa                      0.10.1
llvmlite                     0.42.0
Markdown                     3.6
markdown-it-py               3.0.0
MarkupSafe                   2.1.5
matplotlib-inline            0.1.6
mdurl                        0.1.2
mido                         1.3.2
mir-eval                     0.7
ml-dtypes                    0.3.2
msgpack                      1.0.8
namex                        0.0.7
nest-asyncio                 1.6.0
numba                        0.59.0
numpy                        1.23.5
oauthlib                     3.2.2
opt-einsum                   3.3.0
packaging                    23.2
parso                        0.8.3
pip                          23.0.1
platformdirs                 4.2.0
pooch                        1.8.1
pretty-midi                  0.2.10
prompt-toolkit               3.0.43
protobuf                     3.19.6
psutil                       5.9.8
pure-eval                    0.2.2
pyasn1                       0.5.1
pyasn1-modules               0.3.0
pycparser                    2.21
Pygments                     2.17.2
python-dateutil              2.9.0.post0
pywin32                      306
pyzmq                        25.1.2
requests                     2.31.0
requests-oauthlib            1.4.0
resampy                      0.4.3
rich                         13.7.1
rsa                          4.9
scikit-learn                 1.4.1.post1
scipy                        1.12.0
setuptools                   65.5.0
six                          1.16.0
soundfile                    0.12.1
soxr                         0.3.7
stack-data                   0.6.3
tb-nightly                   2.17.0a20240315
tensorboard                  2.10.1
tensorboard-data-server      0.6.1
tensorboard-plugin-wit       1.8.1
tensorflow-cpu               2.10.1
tensorflow-directml-plugin   0.2.0.dev221020
tensorflow-estimator         2.10.0
tensorflow-intel             2.10.1
tensorflow-io-gcs-filesystem 0.31.0
termcolor                    2.4.0
tf-nightly                   2.17.0.dev20240306
tf_nightly_intel             2.17.0.dev20240306
threadpoolctl                3.3.0
tornado                      6.4
traitlets                    5.14.2
typing_extensions            4.10.0
urllib3                      2.2.1
wcwidth                      0.2.13
Werkzeug                     3.0.1
wheel                        0.43.0
wrapt                        1.16.0
Other information: Jupyter on Visual Studio Code Insiders (Daily build)
Here are a few steps I took to solve it:

Unsure that your python version is within the supported set (3.7...3.10)
Use pyenv to perform the previous step
Try using this if you run into issues https://github.com/microsoft/tensorflow-directml-plugin
If you are running into an error related to Multiple OpKernels, keep deleting Tensorflow until all have been deleted (there may be multiple versions) and then reinstall (tensorflow-cpu XOR tensorflow XOR tf-nightly)
Reference: https://stackoverflow.com/questions/72994470/error-when-using-directml-multiple-opkernel-registrations-match-nodedef-at-the
Ensure you are using the correct python kernel for jupyter, or for your terminal/IDE
If using jupyter, ensure that the kernel is reset after EVERY PACKAGE CHANGE (CTRL-SHIFT-P -> Restart Kernel) or ensure that you reactivate your virtual environment (WSL, MacOS, Ubuntu) or any other steps related to your OS
If you are unable to install from pip, I would recommend installing "setuptools" package which uses a deprecated package called "distutils" which is a requirement for numpy as I recall..
Disable the GPU if you are running into issues, and run cpu-bound to get started, which I added in the attached code.
Load the model directly through TensorFlow instead of through this library as you can add additional configuration easier
If you run into a CUDART_64 error try this https://stackoverflow.com/questions/66206835/importerror-could-not-load-dynamic-library-cudart64-110-dll
Try this if you want to only use CPU https://stackoverflow.com/questions/37660312/how-to-run-tensorflow-on-cpu

# Uses Spotify Open-Source Basic Pitch audio-to-midi conversion
from basic_pitch.inference import predict
from basic_pitch import ICASSP_2022_MODEL_PATH
from os.path import join as pathjoin
from os import sep
import tensorflow as tf

# Set variables
input_file_path = pathjoin("C:" + sep, "repo", "JupyterNotebooks", "cached_data", "blob_data", "nuvoleBianche.mp3")
output_file_path = pathjoin("C:" + sep, "repo", "JupyterNotebooks", "cached_data", "processed_data", "")

# Check version and load model (used as workaround in Windows)
print(tf.__version__)
tf.config.set_visible_devices([], "GPU")
basic_pitch_model = tf.saved_model.load(ICASSP_2022_MODEL_PATH)

# Perform prediction
(model_output, midi_data, note_events) = predict(
    input_file_path,
    basic_pitch_model,
)

# Do something with the data
print(note_events)

@drubinstein I would recommend that this issue is set to closed based on the initial description and may be re-opened once the author provides more information or goes through the necessary steps based on input I provided.

Hi @Dragonfore . I'm gonna keep the issue open. I'm fairly confident the issue is with tensorflow 2.16.1 as I ran into this error earlier this week when working on and merging #100.
Regardless, there's no reason to be a stickler here and close and reopen the issue. He provided enough detail to get started and I could always ask for more e.g. how he's invoking it. What could be useful is an issue template and I can make one or base one off of what you wrote. Thanks for your comments.
FYI your packages may have a slight conflict given that they install tensorflow-cpu<2.11 and tf-nightly 2.17.

Hi @drubinstein Thanks for the FYI on the package conflicts! I do think that an issue template would be beneficial if you have some time or I might be able to create a PR to do this sometime in the next week or so.
Sorry about the recommendation to close, I should have waited until after a response was provided as there was a reasonable amount of detail. After re-reading, I did come across as a stickler, my goal only goal was trying to help out for the reporting of possible issues. I'll keep that in mind for the future.
Additionally, @pdduggan if you have any other questions please feel free to provide more info and sorry for one of my previous comment about closing prematurely.

I had the same issue and rolling back from tensorflow 2.16 to 2.15 fixed the problem for me.
I'm on OSX using python 3.11

Until I can find a way to reserialize the model or find what transitive dependency is breaking model loading, I've pinned tensorflow to <2.15.1. I hope this doesn't impede any of your work.


[WFLY-5351] allow to query cancelled persistent timer

allow to query wildfy model even for cancelled persistent timer (see https://issues.jboss.org/browse/WFLY-5351)
With this PR, the query ends with ModelNode empty/undefined values instead of a failure.
Test included as integration/smoke test.

Can one of the admins verify this patch?

this is ok to test

Also, we require that all contributions be made under the terms of the MIT License, http://www.opensource.org/licenses/mit-license.php. Do you agree to the these terms?

Yes I do. I modified test header files.
If you want I can squash everything in one commit.

@McFoggy Thanks. Yes, please squash this to one commit.

@bstansberry squashed & rebased for convenience.

Thanks!


Allow user to cycle through list

echo "c\nb\na" | walker -d

Select last item
Press Arrow Down or Tab
Nothing happens

Would be great to have an option to cycle through list so on step 3 selection goes back to item 1. Not necessary just for dmenu mode could be great when used with single module defined from plugin.

added cycle boolean to list config.


Make ListTagsOptions work correctly with ListOptions

The ListTagsOptions was not defined correctly with an invalid documentation reference. Fixed this and added a test suite for tags (like the product test)

@Bouwdie what makes you think ListTagOptions was defined incorrectly? Seem perfectly valid and correct to me...

@svanharmelen You are absolutely right! i reverted that change and updated the test. Maybe the test is still of some use?

Sure thing! Tests are always welcome 👍🏻 Thanks!


slice geometry

Create a THREEJS like slice geometry to slice a cube easily.

http://fnndsc.github.io/vjs/#geometry_slice


✨ Add support for a wildcard pattern _.

I think this would be a simple one to add:

Add a Wildcard variant to AST.Pattern
Create a parser that consumes a _ and potentially any valid name after it so _ and _foo would both parse as Wildcards.

Letting the parser work with _foo-style wildcards would be handy for "commenting out" a variable in a function arg, or otherwise labelling something that isn't being used.


Review packaging metadata

THOUGHTS:

In particular, the "Development Status" classifier, currently as "1 - Planning".
I guess we're past that point!

Closing after having merged #88.


V1.2.1

To be completely honest after three weeks and much intervening development I don't remember if this was a stable version or not. I am closing this request for now and will revisit updating main soon(tm)


Create a third_party/sqlite repository on flutter.googlesource.com

Flutter will use an SQLite database file to write a description of a sequence of scenes that can be read to play them back for further debugging and profiling (only present in debug & profile modes).
The SQLite sources are derived from the amalgamated sources generated from either the upstream Fossil or Git repositories. Pulling in sources from the repository itself is not recommended both for performance (as outlined in the section on amalgamation) and size reasons (the entire repo is 140 MB at --depth=1)
A one-click mechanism to update and amalgamated sources, version information, and an optimized set of GN rules is present in https://github.com/chinmaygarde/sqlite. The new repo can either be initialized to https://github.com/chinmaygarde/sqlite/commit/ef1f9f4172161e176ba68a0bd401e15ddc2ea25c or left bare and I can fill it with the contents of that repo.
The LICENSE has been verified to be part of the "unencumbered" list here. cc @Hixie in case he wants to verify the repo layout.

The repo is ready: https://flutter.googlesource.com/third_party/sqlite/, with initialization commit: https://github.com/chinmaygarde/sqlite/commit/ef1f9f4172161e176ba68a0bd401e15ddc2ea25c.


[DELETED] Update license scan to include CMake dependencies

It looks like fossa license scan does not support CMake dependencies :crying_cat_face:


Add random quantum Tanner codes

Also:

allow passing a random number seed to ClassicalCode.random
remove unreasonable defaults in a few quantum code constructors

Tagging @mittaltushant for visibility

Looks good


Publish updated tutorials from this repo

Is your feature request related to a problem? Please describe.
I reluctantly followed the Application setup tutorial because it mentioned Parcel as the bundler. I have no intention to deal with yet another system just to get a peak into some example code.
However, being courageous and ready to run rm -rf later on, I ran the command npx create-ol-app and found myself with a Vite installation, which is entirely different from what's on the page -- and a good thing FWIW.
The README in the then generated project mentions yet another command to install the demo application using Vite, that's neither mentioned in the online tutorial nor the documentation section of this repo: npx create-ol-app my-app --template vite.
Describe the solution you'd like
Although not on par with reality of "create-ol-app", please publish the updated documentation sources of this repo that match the current setup people get when they run the scripts so they know what to expect.
Thank you.

The docs are versioned, and by the time the latest release (v6.9.0) was updated, the tutorial was still for Parcel. https://openlayers.org/en/main/doc/tutorials/bundle.html has the updated tutorial already, and so will the docs for the next release.

ok, thanks ... so this is already in the making?
'cos there is no link to that "main" folder or the "bundle.html" page in

https://openlayers.org/en/latest/doc/tutorials/
That's what the site's main menu points to, and it has the word "latest" in it ;-) so does the "Quick Start" 🤔

Yes, this is in the making, and will show up under https://openlayers.org/en/latest/doc/tutorials/ automatically when we publish the next release.
The docs under https://openlayers.org/en/latest/doc/tutorials/ are for the latest published version, which is the same as https://openlayers.org/en/v6.9.0/doc/tutorials/ at the time of writing.


Verify the functioning of friendly urls over lighttpd

It's giving 404 errors

What configuration for lighttpd have you used?

I was not the person who contributed with the lighttpd integration :(


Question: Is there a way to retain the fingerprint between browsers?

Title says it all 👀

between browsers

On the same machine?
If so you could use setOption to exclude fingerprint components that are browser specific leaving only machine specific components. Still this is a maybe and would depend on the browser API and how it returns results exactly. The smallest change in results for the same API call would throw this off.
If this does not work you would need to fork it get have more granular with the components and possibly adding new ones or normalize certain browser API return results to get a reliable "machine only" fingerprint from any browser on that machine.
You could check out hw-fingerprint if that is truly what you need.


Fix Typings

The TypeScript typings in 2.14.0 are incorrect and breaking our builds. This should fix the issue.
Thanks for all the hard work ❤

@todd thanks! Is there a linter for this so we can run it before release?

And do you need a new release or just merging this in is enough?

moment.d.ts is published in the npm package and we download it from npm. It needs new version.

It's going to have to be a new release, unfortunately 😭
You can use tslint for style-related things. Otherwise, you'll have to rely on the actual TypeScript compiler (tsc) to catch these sort of issues.

Thanks guys! I think a new release is required. Until one ships moving back to 2.13.0 should do.

Have you considered making use of a test usage file such as is used on Definitely Typed? Essentially you have a file which demonstrates usage of moment. If the Typings change such that usage is broken then running TSC against the Typing and the test file will result in a failing build.

@johnnyreilly  Can we just run all our tests under this?

I left a comment for @ichernev on Gitter to more or less the same effect (minus the part about the test case). It would require adding TypeScript as a development dependency, though.

Oh and thanks for all the hard work x2 :-)

It's a different sort of test; compilation as opposed to execution. That said, it's probably possible to combine them; though would have to look at moments setup. (On a phone right now so can't😞)

The more extreme step (though worth it I would suggest) is switching to writing moment in TypeScript (I don't think you do that right now?) Doing that would make this sort of error much harder to ship 😃

Just had a look at the code. The lowest friction way to to this would probably be to drop in the moment-tests.ts file from DT. Then make the tests part of your package.json this: "test": "grunt test && tsc". This would compile the definition tests file as part of the tests. Problems would fail the build.
As @todd says, you'd need to add TypeScript as a dev dependency.

tsc automatically tests a file that ends with *.ts? Can we put the tests file in another folder? Also we'd need to update the ts tests file when we make changes?

Tsc triggers compilation. If the Typing file (moment.d.ts) no longer satisfied the usage file (moment-tests.ts) then compilation would fail which should fail the build.

Can we put the tests file in another folder?

Yes

Also we'd need to update the ts tests file when we make changes?

Yes

If I get a moment (ha!) I'll see if I can do a quick fork and see if I can implement this for you

@johnnyreilly well, it won't be just tsc it will be tsc path/to/tests.ts and possibly also add the moment.d.ts file in there too. That is why I was a bit surprised.

Yes it will probably be tsc -proj typing-tests or something. That folder could be made to reference moment.d.ts where it is now or you could move moment.d.ts into the typing-tests folder and change the Typings reference in package.json

Fwiw it could just be tsc if we had an accompanying tsconfig.json (which might be a good idea anyway)


Move non-assignment scalar array-array ops to scalar util module

Straightforward move. No changes to the function bodies, only the method signatures.
One more refactoring after this one, then SIMD PRs start.

Build failed:

ci (stable)

Network failure. We'll be green again after my next PR. Closed.

Oh, bors did not, in fact, merge. Reopened.


kboot/gpu: Add "no-map" property for reserved memory nodes

Without this property u-boot will add reserved memory regions to the EFI memory map with the wrong type. This results in Linux mapping it and thus breaking the intended use in the asahi driver.

I think it's fine to just have this in the DT, but I guess being defensive won't hurt.


Target Geolocation [Target Analysis]

Geolocate individual pixeltargets (converting error from pixels to degrees of latitude/longitude) and create a Target to wrap each pixeltarget. Each target should only contain one pixeltarget and duplicates will later be merged.

Working on it


assignment-1

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
Adding assignment 1
What did you learn from the changes you have made?
How to work with git bash
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
Struggling with commit/pull/push to github. Need to refresh
How were these changes tested?
locally in VS code
A reference to a related issue in your repository (if applicable)
@mentions of the person or team responsible for reviewing proposed changes (At least 2 people)

Checklist

[x ] I can confirm that my changes are working as intended

@dtxe , @michaeladrouillard , can you please review this pull request? Thank you.

Hi there, glad to see you have began so early and soon on your assignment! That being said, you have unfortunately made a pull request to the wrong repo. You shouldn't be making a pull request to uoft-dsi/shell, but rather your own forked shell repo.
Please read the instructions on your assignment (Submission Information) more carefully (specifically the Assignment Submission Guide)


API to access gradle build cache key of task

Expected Behavior
When a task execute, I would like to be able to access the build cache key that gradle computed
Current Behavior
There is no API to do that so it can be seen only within log files
Context
We are building docker images and it would be very inefficient to have to save them and copy them to the gradle cache. We would like to use the docker registry (local or remote) like a gradle cache but we need access to the cache key so we can validate all inputs are unchanged and go with upToDateWhen closures.

The cache key Gradle calculates is an implementation detail that we don't want users to rely on. This is why there is no public API to access it. Its value can and will change due to factors that might not be suitable for your usage. Calculating your own key using inputs specific to your use case is recommended instead.

I would use the cache key the same way gradle uses it. If the implementation changes, it invalidates all the gradle build cache and this is fine with me. It would also be fine with me if it invalidated my docker cache implementation.
I think if it was just documented as unreliable then developers would be able to make their own choices.
The fact is that it is not always possible or efficient to put stuff in the gradle cache but I think the idea of caching goes beyond that.
Why force people to recode something that works fine?


chore: added asterisks to required registration fields

What issues does this PR close?
Closes #7
### Checklist
- [x] The PR title is short and expressive.
- [x] I have updated the CHANGELOG.md. See [changelog_update.md](https://github.com/sovity/authority-portal/tree/main/docs/dev/changelog_updates.md) for more information.
- [ ] I have updated the Deployment Migration Notes Section in the CHANGELOG.md for any configuration / external API changes.
- [x] I have performed a **self-review**

Preview

Industry is a required field too


OTA stopped working with weather and websocket draw example, does not work after ftdi upload

Hello, I bought 5 Badgys on Tindie.
I uploaded some tests OTA on the badgies, When I tried the weather, 2 of them froze, and when I restarted a 3rd one, it froze also (it had the websocketdraw on it).
Now, I am trying to upload using FTDI, it uploads well (Blinking led, and 100% status), but still stays frozen to the last state when it stopped working.
I jumped 0 and gnd as per your chart. and left rest in the air as per your rst => NC instruction.
What do you think goes wrong? I am uploading the Badgy, and hello INO, but I cannot get it to show on the screen when i plug it back
The only one that keeps on working is the hourly quote, but I do not dare changing anything in the code, nor developing with Badgy until I can get it to work without OTA
Thank you for your help

Hi @vidax!
Try using esptool to erase the flash and try again
esptool.py --port <serial-port-of-FTDI> erase_flash
or in Arduino

Let us know how it goes, thanks.


Migrate outlook add in page

Migrate the Outlook Add-in page to new designed landing page. Refer to Dev Task 231649.
Here is the guidance of new designed landing page.
https://aka.ms/docslanding

Docs Build status updates of commit f5c7a88:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/index.md
:warning:Warning

Details

docs/toc.yml
:warning:Warning

Details

docs/outlook/index.md
:white_check_mark:Succeeded

docs/outlook/index.yml
:white_check_mark:Succeeded

docs/index.md

Line 0: [Warning-InvalidFileLink] Invalid file link:(~/outlook/index.md).

docs/toc.yml

Line 0: [Warning-InvalidFileLink] Invalid file link:(~/outlook/index.md).

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit c7982e2:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/index.md
:warning:Warning

Details

docs/outlook/index.md
:white_check_mark:Succeeded

docs/outlook/index.yml
:white_check_mark:Succeeded

docs/toc.yml
:white_check_mark:Succeeded

docs/index.md

Line 0: [Warning-InvalidFileLink] Invalid file link:(~/outlook/index.md).

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit f1a0808:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/index.md
:warning:Warning

Details

docs/outlook/index.md
:white_check_mark:Succeeded

docs/outlook/index.yml
:white_check_mark:Succeeded

docs/toc.yml
:white_check_mark:Succeeded

docs/index.md

Line 0: [Warning-InvalidFileLink] Invalid file link:(~/outlook/index.md).

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Closing and reopening to generate preview site.

Docs Build status updates of commit f1a0808:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/index.md
:warning:Warning
View
Details

docs/outlook/index.md
:white_check_mark:Succeeded
View

docs/outlook/index.yml
:white_check_mark:Succeeded
View

docs/toc.yml
:white_check_mark:Succeeded
View

docs/index.md

Line 0: [Warning-InvalidFileLink] Invalid file link:(~/outlook/index.md).

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Update components to support React 17.x

componentWillReceiveProps will be removed in React 17.x.

https://github.com/missive/emoji-mart/pull/372


Clearly show ticket purchases in Overview and History

Right now a ticket purchase shows up as a red -0 DCR in the Overview and History tabs. When I click on the transaction, the transaction Type is Ticket_purchase. It should thus be possible to make these types of transaction more clear to the user. Maybe it can say "Ticket" just like the votes say "Vote"

Decrediton also uses a blue color for the Ticket purchase. That might be good for the mobile wallet as well (since your DCR are only locked up and remain linked to your wallet)


pycobertura diff should fail when uncovered lines went up

Right now, pycobertura diff will return a non-zero exit status if the line rate worsened:
https://github.com/SurveyMonkey/pycobertura/blob/6cf7b14d11a7f7cc13270dbae2e5db6eb184ecf2/pycobertura/cli.py#L96
It should rather check if the number of uncovered lines got worse, not the line rate.

Fixed by #36


Change to use map for Sum and Product for auto simplification.

Codecov Report

Merging #7 (e073e9a) into main (c6f9174) will decrease coverage by 1.29%.
The diff coverage is 25.00%.

@@            Coverage Diff             @@
##             main       #7      +/-   ##
==========================================
- Coverage   34.04%   32.74%   -1.30%
==========================================
  Files           1        1
  Lines         282      342      +60
  Branches      183      227      +44
==========================================
+ Hits           96      112      +16
- Misses          9       16       +7
- Partials      177      214      +37

Impacted Files
Coverage Δ

include/mathia.h
32.74% <25.00%> (-1.30%)
:arrow_down:

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 c6f9174...e073e9a. Read the comment docs.


Update quest-modding.md

Added additional details to QuestPatcher instructions.

I just saw that there was a guide on making submissions. I understand if this gets rejected due to not following a workflow.


GitHub 7865 haspropwithvaluequery leaks

Fixes #7865

I agree that Dict::Pair was probably the wrong choice here as it really shouldn't be used out of a dictionary.  I'll make a thin wrapper that cleans it up migrate external usage to that and then run a valgrind test for leaks.

I'll fix the swig builds this evening.  Need to ignore the PairHolder funcs.

This didn't show any leaks in valgrind at least.

asan also doesn't report any mem errors.

@greglandrum this one is finally ready as well.

@bp-kelley since this changes the public API, there should be an entry in the "Backwards Incompatible Changes" part of the release notes.

@greglandrum I think I addressed all the comments.


fixed timeout config for zms java client

I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.

Fixes: https://github.com/yahoo/athenz/issues/897.
This might fix https://github.com/yahoo/athenz/issues/894 also. Need to be verified.


single apostrophe (') incorrectly parsed as option link

Sample text (from api.txt):
Example:

Let's set an extmark at the first row (row=0) and third column (column=2).
|api-indexing| Passing id=0 creates a new mark and returns the id: >

      01 2345678
    0 ex|ample..
<        ^ extmark position
>
    let g:mark_ns = nvim_create_namespace('myplugin')
    let g:mark_id = nvim_buf_set_extmark(0, g:mark_ns, 0, 2, {})
<
We can get the mark by its id: >

The ' char in Let's confuses the parser, and lots of errors follow.

Kind of expected, I'll see what I can do!

Could you confirm the issue is fixed on your end too ?

Nice! I checked all single-quote cases in api.txt and couldn't find any issues. Thanks @vigoux !


[capi] add docs for unarchive exp

#1578  👈
main

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

Join @brian-statsig and the rest of your teammates on  Graphite


SPICE is no longer available in RHEL9/CentOS9 Stream

System Information
Linux distribution
Centos9 Stream/ RHEL 9
Provider and libvirt versions
terraform/providers/libvirt/go.mod:require github.com/dmacvicar/terraform-provider-libvirt v0.6.12

Checklist

[ ] Is it a bug or something that does not work as expected? Please make sure you fill the version information below:

Description of Issue/Question
terraform-provider-libvirt uses SPICE  by default for its libvirtxml.DomainGraphic, but this has been deprecated/removed in RHEL9/CentOS9 Stream, see https://github.com/openshift/installer/pull/5622
[DEBUG] libvirt_domain.bootstrap: apply errored, but we're indicating that via the Error pointer rather than returning it: Error defining libvirt domain: virError(Code=67, Domain=10, Message='unsupported configuration: spice graphics are not supported with this QEMU')

terraform-libvirt-provider could parse the equivalent of virsh domcapabilities to make a better decision regarding which graphics element to add by default:
$ virsh domcapabilities
<domainCapabilities>
<!-- [...] -->
  <devices>
    <!-- [...] -->
    <graphics supported='yes'>
      <enum name='type'>
        <value>sdl</value>
        <value>vnc</value>
        <value>spice</value>
        <value>egl-headless</value>
      </enum>
    </graphics>
  </devices>
</domainCapabilities>

Or it could switch to VNC by default instead of SPICE.

I do not think this is actionable - default setting of one Linux distribution should not dictate defaults here. As a matter of fact, it's just a default that's easily changeable.
I believe this issue should be closed.


Parallel installation of extensions may interfere with versioning

Issue Type: Bug
Our corporate policy means that we've got a requirement to use a custom build of the liveshare extension with additional auditing built in. We also have to use the liveshare gatekeeper extension. I'm finding that if we try to install both of these in a container, I can specify the vsix for liveshare and it installs the correct version, but if I add gatekeeper, I end up with the latest build of liveshare, not our custom one.
What I believe is happening is that Remote-Containers is calling code --install-extension <liveshare vsix> --install-extension <gatekeeper>. Seeing that liveshare installation has not (yet) completed, it appears to decide to install the latest version of that as a dependency of gatekeeper, even though I'd specifically asked for a different one. If the installations are done as separate steps, I get the versions I want (but the extension doesn't make this easy to do).
Would it be possible to have VSCode check the list of extensions being installed to see whether they satisfy any dependencies needed, and only add extensions which are not already being provided?
VS Code version: Code 1.55.2 (3c4e3df9e89829dce27b7b5c24508306b151f30d, 2021-04-13T09:35:57.887Z)
OS version: Windows_NT x64 10.0.19041
Remote OS version: Linux x64 4.19.128-microsoft-standard
Remote OS version: Linux x64 4.19.128-microsoft-standard

Sorry, should have provided a way to reproduce.
Current behaviour
code --install-extension<EMAIL_ADDRESS>--install-extension vsls-contrib.gatekeeper
installs liveshare version 1.0.4131. Though until restart, it may show as the earlier version - there appears to be a race condition with it trying to install both versions at once, but the end result is always that I get the newer one.
Expected behaviour:
liveshare version 1.0.2048


fix(qwik-city): service worker over-fetching

What is it?

[ ] Feature / enhancement
[x] Bug
[ ] Docs / tests / types / typos
[ ] Infra

Description
Fixing over-prefetching by the service worker
Fixes #6795
Checklist:

[ ] My code follows the developer guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have ran pnpm change and documented my changes
[ ] I have made corresponding changes to the Qwik docs
[ ] Added new tests to cover the fix / functionality

Thanks @gioboa !
not ready yet, still have more work to do 😊


Bump build/release workflow to JDK 17

See https://github.com/quarkiverse/quarkiverse-devops/issues/218

Why was this PR closed?

Ah I see it is already changed. Nevermind ;)


[pw reset] - remove username from field label

Remove username from the label of https://www.dosomething.org/user/password because a. username doesn't work and b. majority of members don't know their username

this is now showing the word email!


Inspector tooltip is blank

In the past I used this tooltip to see the loaded UI5 version on the page, however, from some time it appears blank.
Steps to reproduce the issue:

Open https://ui5.sap.com/
Click on the inspector icon

OS: Windows 11
Browser: Version 125.0.6422.113 (Official Build) (64-bit)
UI5 Inspector: 1.6.3

Hi Petar,
the issue has been fixed in version 1.6.4
Please, update your extension and retest.
Thank you!
Best regards,
Petya Markova.
(UI5 Inspector Developer)

@PetyaMarkovaBogdanova thanks :)
Tested and it works in 1.6.4.


Add Persepolis Download Manager

It's innosetup installer
Change it to "innosetup": true
You can use $underscoreVersion inside autoupdate
I would replace that spaces inside checkver regex with \\s+

{
    "homepage": "https://persepolisdm.github.io/",
    "description": "A Free as libre download manager & a GUI for aria2, written in Python",
    "version": "3.1.0",
    "license": "GPL-3.0-only",
    "architecture": {
        "64bit": {
            "url": "https://github.com/persepolisdm/persepolis/releases/download/3.1.0/persepolis_3_1_0_windows_64bit.exe",
            "hash": "fe1fc842b49daa555af3d1ffce4f8f5a2b664620d5004b96cbb6f177ed7be29f"
        },
        "32bit": {
            "url": "https://github.com/persepolisdm/persepolis/releases/download/3.1.0/persepolis_3_1_0)_windows_32bit.exe",
            "hash": "dde3b8b72dc903b369b95af641454f6fe74e8961cd2008fdc5a1427e88b39809"
        }
    },
    "innosetup": true,
    "bin": [
        [
            "Persepolis Download Manager.exe",
            "persepolis"
        ]
    ],
    "shortcuts": [
        [
            "Persepolis Download Manager.exe",
            "Persepolis Download Manager"
        ]
    ],
    "checkver": {
        "github": "https://github.com/persepolisdm/persepolis"
    },
    "autoupdate": {
        "architecture": {
            "64bit": {
                "url": "https://github.com/persepolisdm/persepolis/releases/download/$version/persepolis_$underscoreVersion_windows_64bit.exe",
                "hash": {
                    "url": "https://github.com/AdmiringWorm/chocolatey-packages/raw/master/automatic/persepolis/legal/VERIFICATION.txt",
                    "find": "\\s+checksum64:\\s+([A-f\\d]{64})"
                }
            },
            "32bit": {
                "url": "https://github.com/persepolisdm/persepolis/releases/download/$version/persepolis_$underscoreVersion)_windows_32bit.exe",
                "hash": {
                    "url": "https://github.com/AdmiringWorm/chocolatey-packages/raw/master/automatic/persepolis/legal/VERIFICATION.txt",
                    "find": "\\s+checksum32:\\s+([A-f\\d]{64})"
                }
            }
        }
    }
}

Thanks for your help. I think we'd better not purposely make dependence for Windows app, they intends to be redundant. At least it would be updated with the installer package.


powertoys keyboard manager does not open

Microsoft PowerToys version
0.79.0
Installation method
GitHub
Running as admin
Yes
Area(s) with issue?
Keyboard Manager
Steps to reproduce
PowerToysReport_2024-03-10-21-22-16.zip
✔ Expected Behavior
I was waiting for the keyboard organizer to open when I pressed it.
❌ Actual Behavior
When I press to open the keyboard manager, nothing happens.
Other Software
No response

@niels9001 From the logs I see some RTL error, but the user doensn't use a RTL locale, any idea what happens here?

/dup #31708
@RektSemih , can you please add to that original issue?
Also please try the local builds and report there if you're able to? https://github.com/microsoft/PowerToys/issues/31708#issuecomment-1986515217


JSX does not mark React as used when imported via import React = require("react");

I have react option enabled for "no-unused-variable"
 "no-unused-variable": [true, "react"],

Still React is reported as unused when imported like this:
 import React = require("react);

but this import statement works
 import * as React from "react";

Example
/// <reference path="typings.d.ts" />
import React = require("react"); // TSLint reports "unused variable: 'React'"
const result: JSX.Element =
Test
;
export = result;

It seems that the problem is caused by react-specific import detection code is only in visitNamespaceImport() method (noUnusedVariableRule.ts file), but not in visitImportEqualsDeclaration() method.

Good catch @cezary-biernacki, thanks! Is using import * as React from "react"; an acceptable workaround for you until this gets fixed?

@cezary-biernacki @JKillian - according to the spec the two are equivalent. So whilst the rule should be fixed for completeness, @cezary-biernacki you should be safe to use the import variation instead of the require(..) variation.

@JKillian: Yes, the workaround is acceptable, just it is confusing. I had to debug TSLint itself to discover what kind of syntax works, as the documentation does not provide necessary details.

Another variant of this: TSLint does not mark React as used if you import it like this:
import React from 'react'
The d.ts file from DefinitelyTyped does not contain a default export, but it's perfectly fine to import React like that if you use Babel (since Babel has interop between ES6 modules and CJS modules).

Babel has interop between ES6 modules and CJS modules

well it's a bit more than that; Babel automatically creates synthetic default exports. but I agree that the import React from 'react' syntax should also be supported along with import React = require("react"). as indicated by the labels, this is an easy fix that anyone is welcome to contribute.

There is another issue related to the original problem #689 and it affects imports in jest unit-test cases. The linter also doesn't mark react as used when calling TestUtils.renderIntoDocument. I am using tslint 3.8.1.
import * as React from 'react';
import * as ReactDOM from 'react-dom';
import * as TestUtils from 'react-addons-test-utils';

describe('Input', () => {
    it('just an input test', () => {
       const input = TestUtils.renderIntoDocument(
            <input type = 'text' onKeyPress = {keyPressAction.bind(this)}/>
        );
        const inputNode: Element = ReactDOM.findDOMNode(input);
        inputNode.setAttribute('value', 'test');
        expect(input.getAttribute('value')).toEqual('test');
    });
});

We are porting from eslint + babel and have the following use case:

We do all imports in index.js files.
We dependency inject all imports into the other files as named object keys.

So in our render.js file we will have the following snippet:
export default function create({
  React,
  StyleSheet,
  css,
  styles
}) {
// snip with tsx stuff
}

But setting "no-unused-variable": [true, "react"] is not working.

at this point I would recommend switching to the compiler options and disabling no-unused-variable, see https://github.com/palantir/tslint/issues/1481 and linked threads.

Should be fixed by #2235, which uses TypeScript's implementation for no-unused-variable.


Add support for replacing an existing kv prefix with a new set

This PR adds a new method ReplaceConfig that will remove any keys (for the source's prefix) that are not present in the new config. This is primarily to solve https://github.com/containous/traefik/issues/2093 – by adding this, we can make Traefik use ReplaceConfig instead.
This will have the desired effect of removing any stale keys (that can lead to Traefik picking up config that the user thinks is not present), while maintaining any existing keys.

Coverage decreased (-0.7%) to 82.66% when pulling f6de4c5a9ec74de40774ac745354531228602781 on phroggyy:feat-config-replace into f987d01c3d97d01fffb73a57fa35a659c944396a on containous:master.

@ldez any progress on getting this merged?


L515: Depth sensor frames don't arrive

Required Info

Camera Model
L515

Firmware Version
<IP_ADDRESS>

Operating System & Version
Ubuntu Mate 20.04.1

Kernel Version (Linux Only)
5.4.0

Platform
Raspberry Pi 4

SDK Version
2.49.0

Language
N/A

Segment
others

Issue Description
Depth sensor previously worked on this system.
Attempting to enable the depth sensor fails either with a frame timeout or fatal error message.
Plugging the sensor into a different system doesn't resolve the issue, nor does swapping out the cable.
$ rs-hello-realsense
RealSense error calling rs2_pipeline_wait_for_frames(pipe:0xaaaab28cba40):
    Frame didn't arrive within 15000

Enabling the depth sensor in realsense-viewer outputs notifications.cpp:514 - Fatal error occured (16).
However the RGB camera and motion sensor can be enabled in the viewer and work fine.
dmesg output
[102473.739903] uvcvideo: Failed to query (GET_CUR) UVC control 1 on unit 3: -32 (exp. 1024).
[102473.834610] uvcvideo: Failed to query (GET_CUR) UVC control 1 on unit 3: -32 (exp. 1024).
[102473.889788] uvcvideo: Failed to query (GET_CUR) UVC control 1 on unit 3: -32 (exp. 1024).
[102473.945402] uvcvideo: Failed to query (GET_CUR) UVC control 1 on unit 3: -32 (exp. 1024).
[102474.177094] uvcvideo: Failed to query (GET_CUR) UVC control 1 on unit 3: -32 (exp. 1024).
[102474.227699] uvcvideo: Failed to query (GET_CUR) UVC control 1 on unit 3: -32 (exp. 1024).
[102474.232125] uvcvideo: Non-zero status (-71) in video completion handler.
[102474.279071] uvcvideo: Failed to query (GET_CUR) UVC control 1 on unit 3: -32 (exp. 1024).
[102474.329620] uvcvideo: Failed to query (GET_CUR) UVC control 1 on unit 3: -32 (exp. 1024).
[102474.419785] uvcvideo: Failed to query (GET_CUR) UVC control 1 on unit 3: -32 (exp. 1024).
[102474.485905] hid-sensor-hub 0003:8086:0B64.0004: No report with id 0xffffffff found
[102474.485908] hid-sensor-hub 0003:8086:0B64.0004: No report with id 0xffffffff found
[102475.616594] uvcvideo: Failed to query (GET_CUR) UVC control 1 on unit 3: -32 (exp. 1024).
[102475.675546] uvcvideo: Failed to query (GET_CUR) UVC control 1 on unit 3: -32 (exp. 1024).
[102476.931092] uvcvideo: Failed to query (GET_CUR) UVC control 1 on unit 3: -32 (exp. 1024).
[102484.230743] uvcvideo: Failed to query (GET_CUR) UVC control 1 on unit 3: -32 (exp. 1024).
[102485.490983] uvcvideo: Failed to query (GET_CUR) UVC control 1 on unit 3: -32 (exp. 1024).
[102486.743140] uvcvideo: Failed to query (GET_CUR) UVC control 1 on unit 3: -32 (exp. 1024).
[102488.000075] uvcvideo: Failed to query (GET_CUR) UVC control 1 on unit 3: -32 (exp. 1024).
[102489.262998] uvcvideo: Failed to query (GET_CUR) UVC control 1 on unit 3: -32 (exp. 1024).

Hi @Scratchcat1   There have been a few past reports of this L515 error on Ubuntu, Windows and Mac in https://github.com/IntelRealSense/librealsense/issues/9329 https://github.com/IntelRealSense/librealsense/issues/9333 and https://github.com/IntelRealSense/librealsense/issues/9822
Intel's advice in those cases was to return the camera to the retailer that the camera was purchased from, using the Return Material Authorization (RMA) procedure of that retailer.

Hi @Scratchcat1  Bearing in mind Intel's recommendation above to return the camera to the retailer that it was purchased from, do you require further assistance with this case please?  Thanks!

I've returned the camera for replacement so I'll close this issue. Thanks for your help.

You are very welcome, @Scratchcat1 - thanks very much for the update!


issue when installing

pip error log:
Collecting git+https://github.com/jnsougata/discohook
  Cloning https://github.com/jnsougata/discohook to c:\users\nyxical\appdata\local\temp\pip-req-build-zg3l20ui
  Running command git clone --filter=blob:none --quiet https://github.com/jnsougata/discohook 'C:\Users\nyxical\AppData\Local\Temp\pip-req-build-zg3l20ui'
  Resolved https://github.com/jnsougata/discohook to commit c71483aeb472af51f54de79a930ff2e331da2b03
  Installing build dependencies ... done
  Getting requirements to build wheel ... done
  Preparing metadata (pyproject.toml) ... done
Collecting aiohttp (from discohook==0.0.6a518+gc71483a)
  Using cached aiohttp-3.9.1-cp312-cp312-win_amd64.whl.metadata (7.6 kB)
Collecting ed25519 (from discohook==0.0.6a518+gc71483a)
  Using cached ed25519-1.5.tar.gz (868 kB)
  Installing build dependencies ... done
  Getting requirements to build wheel ... error
  error: subprocess-exited-with-error

  × Getting requirements to build wheel did not run successfully.
  │ exit code: 1
  ╰─> [31 lines of output]
      C:\Users\nyxical\AppData\Local\Temp\pip-install-0rkcke9p\ed25519_87d27c588b44494a9af0a2445a666e2b\versioneer.py:467: SyntaxWarning: invalid escape sequence '\s'
        LONG_VERSION_PY['git'] = '''
      Traceback (most recent call last):
        File "C:\Users\nyxical\AppData\Local\Programs\Python\Python312\Lib\site-packages\pip\_vendor\pyproject_hooks\_in_process\_in_process.py", line 353, in <module>
          main()
        File "C:\Users\nyxical\AppData\Local\Programs\Python\Python312\Lib\site-packages\pip\_vendor\pyproject_hooks\_in_process\_in_process.py", line 335, in main
          json_out['return_val'] = hook(**hook_input['kwargs'])
                                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
        File "C:\Users\nyxical\AppData\Local\Programs\Python\Python312\Lib\site-packages\pip\_vendor\pyproject_hooks\_in_process\_in_process.py", line 118, in get_requires_for_build_wheel
          return hook(config_settings)
                 ^^^^^^^^^^^^^^^^^^^^^
        File "C:\Users\nyxical\AppData\Local\Temp\pip-build-env-_01pxbv9\overlay\Lib\site-packages\setuptools\build_meta.py", line 325, in get_requires_for_build_wheel
          return self._get_build_requires(config_settings, requirements=['wheel'])
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
        File "C:\Users\nyxical\AppData\Local\Temp\pip-build-env-_01pxbv9\overlay\Lib\site-packages\setuptools\build_meta.py", line 295, in _get_build_requires
          self.run_setup()
        File "C:\Users\nyxical\AppData\Local\Temp\pip-build-env-_01pxbv9\overlay\Lib\site-packages\setuptools\build_meta.py", line 480, in run_setup
          super(_BuildMetaLegacyBackend, self).run_setup(setup_script=setup_script)
        File "C:\Users\nyxical\AppData\Local\Temp\pip-build-env-_01pxbv9\overlay\Lib\site-packages\setuptools\build_meta.py", line 311, in run_setup
          exec(code, locals())
        File "<string>", line 115, in <module>
        File "C:\Users\nyxical\AppData\Local\Temp\pip-install-0rkcke9p\ed25519_87d27c588b44494a9af0a2445a666e2b\versioneer.py", line 1405, in get_version
          return get_versions()["version"]
                 ^^^^^^^^^^^^^^
        File "C:\Users\nyxical\AppData\Local\Temp\pip-install-0rkcke9p\ed25519_87d27c588b44494a9af0a2445a666e2b\versioneer.py", line 1339, in get_versions
          cfg = get_config_from_root(root)
                ^^^^^^^^^^^^^^^^^^^^^^^^^^
        File "C:\Users\nyxical\AppData\Local\Temp\pip-install-0rkcke9p\ed25519_87d27c588b44494a9af0a2445a666e2b\versioneer.py", line 399, in get_config_from_root
          parser = configparser.SafeConfigParser()
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
      AttributeError: module 'configparser' has no attribute 'SafeConfigParser'. Did you mean: 'RawConfigParser'?
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: subprocess-exited-with-error

× Getting requirements to build wheel did not run successfully.
│ exit code: 1
╰─> See above for output.

note: This error originates from a subprocess, and is likely not a problem with
pip.

python version: Python 3.12.0

I guess you have to install python3-pip to your system

I guess you have to install python3-pip to your system

First of all, it seems like the user has a Windows system and pip is either installed along with Python 3 (via .exe) or using the command line (after Pytnon is installed). Second, the error shows that something went wrong while building the wheel for your project and it most likely is not a problem with pip. I'm not sure what causes this, but it clearly is not with the setup of @nyxical420.

It was most likely an issue from the library ed25519 which have been replaced with pynacl recently.

In that case, Discohook probably should switch to Pynacle?

Already switched. Thanks.


Create multiple secrets if BASE_IMAGES_REPO and REPO_REGISTRY are different

The tests needs to pass multiple secrets if the base image and auxilary images are coming from different repos, this PR creates secrets for OCR as well as OCIR registries if they are different.
Also this PR fixes the Dockerfile used in ItMiiAuxiliaryImageCluster.java is not substituted with BUSYBOX_IMAGE:TAG locations
BUSYBOX issue fixed
https://build.weblogick8s.org:8443/job/weblogic-kubernetes-operator-kind-new/10355/
All tests in Kind job
https://build.weblogick8s.org:8443/job/weblogic-kubernetes-operator-kind-new/10345/

Talked to Sankar and confirmed that he tested multiple repositories/secrets scenario on OKD. Approving this PR.


Cannot call a namespace ('dayjs')

Is it a duplicate question?
No
Describe the bug
Warnings during frontend build
To Reproduce
Steps to reproduce the behavior:

npm run build

Expected behavior
No warnings during build
Info

Uptime Kuma Version: https://github.com/louislam/uptime-kuma/commit/440c178403bb1086b4caa1c868568a6f191928c1
Using Docker?: ---
OS: Windows
Browser: ---

Screenshots
Error Log
vite v2.4.4 building for production...
transforming (589) node_modules\vue-chart-3\node_modules\lodash\_hashClear.jsCannot call a namespace ('dayjs')
Cannot call a namespace ('dayjs')
✓ 606 modules transformed.

I also have the same issue with vite v3.2.5
vite v3.2.5 building for production... transforming... Cannot call a namespace ('dayjs')

@Hartha-aloufi
Uptime Kuma uses Vite 3.1.8
Are you sure you are writing in correct repository?

I also have the same issue with vite v3.2.5 vite v3.2.5 building for production... transforming... Cannot call a namespace ('dayjs')

Expexted message during building, but there are no negative impact. Just ignore it.

Same issues as well; how do you know there are not issues about it? @louislam

import * as _dayjs from 'dayjs';

const dayjs = _dayjs;

this works for me.

import dayjs from 'dayjs'

I also have the same issue with vite v3.2.5 vite v3.2.5 building for production... transforming... Cannot call a namespace ('dayjs')

Expexted message during building, but there are no negative impact. Just ignore it.

it's a lie, it changes the build process in vite sim. Install date-fns to solve this!

I also have the same issue with vite v3.2.5 vite v3.2.5 building for production... transforming... Cannot call a namespace ('dayjs')

Expexted message during building, but there are no negative impact. Just ignore it.

For me it had a negative impact on the build
anyone solved this ?

For me it had a negative impact on the build
anyone solved this ?

Instead of using this import,
import * as dayjs from 'dayjs';

use this:
import dayjs from 'dayjs';

import * as _dayjs from 'dayjs';
const dayjs = _dayjs;
worked for me too.

For me it had a negative impact on the build
anyone solved this ?

Instead of using this import,
import * as dayjs from 'dayjs';

use this:
import dayjs from 'dayjs';

Does not exist for me

For me it had a negative impact on the build
anyone solved this ?

Instead of using this import,
import * as dayjs from 'dayjs';

use this:
import dayjs from 'dayjs';

thanks you，it working


fix: Consider changed success message for importer / exporter

The '1 projects imports succeeded' message was changed to '1 project import succeeded' in TeamForCapella 7.0.0

Imported was successfully tested in the staging environment. I'll test the exporter after my vacation.


Webhooks not generated as services with OpenAPI 3.1 Specification

Description
With OpenAPI Specification 3.1, webhooks are not generated as services, resulting in an empty file.
// services.gen.ts
// This file is auto-generated by @hey-api/openapi-ts

import { createClient, createConfig, type Options } from '@hey-api/client-axios';

export const client = createClient(createConfig());

// types.gen.ts
// This file is auto-generated by @hey-api/openapi-ts

export type Pet = {
    id: number;
    name: string;
    tag?: string;
};

Reproducible example or configuration
import { defineConfig } from '@hey-api/openapi-ts';

export default defineConfig({
    client: '@hey-api/client-axios',
    input: './webhook.yaml',
    output: './src/generated-client',
    schemas: false,
});

OpenAPI specification (optional)
openapi: 3.1.0
info:
  title: Webhook Example
  version: 1.0.0
  description: >
    Easy example of documentation with a single webhook,
    provided by OpenAPI Initiative.

webhooks:
  newPet:
    post:
      description: A new pet is born, let's come and discover it in Petstore.
      requestBody:
        description: Information about a new pet in the system
        content:
          application/json:
            schema:
              $ref: "#/components/schemas/Pet"
      responses:
        "200":
          description: Return a 200 status to indicate that the data was received successfully

components:
  schemas:
    Pet:
      required:
        - id
        - name
      properties:
        id:
          type: integer
          format: int64
        name:
          type: string
        tag:
          type: string

System information (optional)
@hey-api/client-axios: 0.2.7
@hey-api/openapi-ts": 0.53.5

Hi @markkkkas, yep, that needs to be (and will be) added https://github.com/hey-api/openapi-ts/blob/main/packages/openapi-ts/src/openApi/3.1.x/parser/index.ts#L204

@markkkkas can you provide more details about your use case? How are you using webhooks? Do you use or expect to use any of the third party plugins?


How to get RSSI of my wifi network. (IDFGH-2356)

Is there any function from which I can get my wifi RSSI directly without scanning the wifi?
Actually I am using the -
static wifi_ap_record_t *ap_records;
esp_wifi_scan_start(&scan_config, true);
esp_wifi_scan_get_ap_num(&ap_num);
ap_records = (wifi_ap_record_t *) malloc(sizeof(wifi_ap_record_t) * ap_num);
esp_wifi_scan_get_ap_records(&ap_num, ap_records);
after this I am priting the wifi ssid and rssi and auth.
strcpy(ssid_list[ap_index+1],(char *)ap_records[ap_index].ssid);
strcpy(wifi_temp,(char *)ap_records[ap_index].ssid);
//wifi_temp[14] = '.';
wifi_temp[13] = '~';
wifi_temp[14] = '\0';
sprintf(ret_val, "%s<%d<%d", wifi_temp, ap_records[ap_index].rssi, ap_records[ap_index].authmode);
ap_index++;
Till here I am able to do everything and able to get RSSI also and then I am writing my ssid and password in NVS.
But what I want to know later on if I need to/print my wifi rssi how can I do it.
I used this function:-
static wifi_ap_record_t curr_wifi;
esp_wifi_scan_get_ap_records(1, curr_wifi);
// here curr_wifi is my stored SSID in NVS which I chosed to connect.
But this did not work.

wifi_ap_record_t ap;
esp_wifi_sta_get_ap_info(&ap);
printf("%d\n", ap.rssi);

See https://docs.espressif.com/projects/esp-idf/en/stable/api-reference/network/esp_wifi.html#_CPPv424esp_wifi_sta_get_ap_infoP16wifi_ap_record_t

Thanks, It worked. 👍


[Runtime] Fix PrebuiltStringMap find with non-terminated keys.

Don't use strcmp to compare the candidate key with the search key, as the search key may not be NUL terminated. Use strncmp and a length check on the candidate key.

@swift-ci please test


Update flex columns to use new ag-Grid functionality

ag-Grid now supports flex columns. The documentation can be found here:
https://www.ag-grid.com/javascript-grid-resizing/#column-flex
Our flex columns should use the built-in ag-Grid flex functionality.

to be clear -- this ticket and PR is being blocked by the following ag-ticket
AG-3852 | react: cellRenderer being remounted when updating
which seems to be causing real performance problems.


EthGetTransactionByBlockNumberAndIndex

Summary

Completion Criteria

Additional Links & Resources

@elmattic Let's have more fleshed-out descriptions, at least one that makes it easier to reason about a year from now or so.


samples and generator updates for 4.4 release

update all samples, all languages, all platforms
update VSIX and yeoman code generators
add .NET Templates
update master README.md
update SPEC.md
update INSTALLING_CLI_TOOLS.md

@edjez @homezcx, samples 12 didn't have much conceptual difference with what we show in sample 13, the CoreBot sample.  So to simply the number of samples and attempt to provide samples that are differentiated we removed it.  If we evolve CoreBot at some point where it doesn't have LUIS (which we don't plan to do at this time), then we'll have a specific sample focused on NLP with LUIS.  To find the specific sample you mention, you can find that here:  https://github.com/microsoft/BotBuilder-Samples/tree/1b93328a6f86f3d8e40712f438fed65c07605f80/samples/csharp_dotnetcore/12.nlp-with-luis.

👍 thx we'll see if we can adapt to sample 13 instead.
@homzcx fyi


Fix typo

.Net -> .NET

@changeworld : Thanks for your contribution! The author, @CelesteDG, has been notified to review your proposed change.

Verified editorial correction.


Make ?module use latest dependency version when many are given

Attempting to acquire https://unpkg.com/react-table@6.8.6/es/index.js?module yields:
import React, { Component } from<EMAIL_ADDRESS>|| ^16.x.x?module";

Ah, good catch. This is because we pull the version straight from the peerDependencies and dependencies in the package.json file. react-table version 6.8.6 has:

In this case, since it's an ||, we should probably just default to using the latest version of all the options listed.

Hmm, this might be a separate issue but right now I'm not even seeing the peerDependencies value get used?
If I fetch https://unpkg.com/preact@10.0.0-alpha.2/hooks/dist/hooks.mjs?module, the preact import gets rewritten to https://unpkg.com/preact@latest?module even though<EMAIL_ADDRESS>specifies:
"peerDependencies": {
    "preact": "^10.0.0-alpha.0"
 }

Since preact@latest redirects to<EMAIL_ADDRESS>the re-written import won't be compatible.


Add vector data upload log

Hi,
While uploading a book onto vector db, it would be good to have an option to turn on status log saying how many docs processed/left to process as at some point i just thought nothing really happens, but in fact the program was still processing 100 pages pdf

So adding a debug log inside the loop here?

I've added some logging, up to a max of 10 log statements in the PDF reader.  Reopen if this is not what you were expecting.
See 7b893410573184df4a1f8ff4f28525670d0b11ef


Cross Compile Help

Question
Hello @Spasi I am working on getting cross compilation working on my ubuntu machine, and I am running into an issue (some step that I am missing).
I setup my packages the same as the lwjgl-ci does, and am attempting to compile in the same way:
 set LWJGL_BUILD_TYPE=nightly
 set LWJGL_BUILD_ARCH=arm64
 ant -emacs hydrate-kotlinc clean-generated generate
 ant -emacs compile
 ant -emacs compile-native -Dgcc.libpath.opengl=/usr/lib/aarch64-linux-gnu/mesa

But it still produces x64 binaries.
Can you please tell me what I am doing wrong?
Thank you,
Trevor

I found my mistake. After spending hours running builds to troubleshoot this, I remembered there is a difference between set and export! I guess I have spent too much time doing Windows development lately.


StreamBridge does not allow manual Partition for Producer

Hello,
I am using a functional interface to consume a message from one cluster and then a StreamBridge to produce the result to another Cluster. We formerly used a StreamPartitioner while being in the same Cluster to select only part of our Key for partitioning in Kafka. This worked quite well but now we need to get the data from a different Cluster and thus cannot use the Spring-Cloud-Stream Kafka-Streams approach anymore.
public Consumer<Message<?>> processor() {
 [...]
 return message -> {
streamBridge.send(OUTPUT_BINDING, MessageBuilder.withPayload(event != null ? event : KafkaNull.INSTANCE)
                .setHeader(KafkaHeaders.MESSAGE_KEY, identifier)
                .setHeader("partitionKey", partition(identifier.getKeyForPartitioning(), PARTITION_COUNT))
                .build())
 [...]
}
}

When using the StreamBridge, the configuration concerning the "partition-key-expression" does not seem to have any effect. Partitioning is not attempted when Debugging the "PartitionAwareFunctionWrapper" since the "outputMessageEnricher" is empty. Also setting the "scst_partition" manually as a header did not have the desired results.
the configuration seems correct for me:
spring.cloud.stream.bindings.<binding>.producer.partition-key-expression: headers['partitionKey']
spring.cloud.stream.bindings.<binding>.producer.partition-count: 32

Maybe some configuration is missing here or this feature is not supported by the StreamBridge somehow? The partition-key-expression seems to be completely ignored.
I would appreciate if somebody could tell me how to configure the Partition in the producer while using a StreamBridge in a Functional Consumer.

First of all StreamBridge is not a component of spring-cloud-function. SO I will transfer the issue to spring-cloud-stream.
My other question is about "...we need to get the data from a different Cluster and thus cannot use the Spring-Cloud-Stream Kafka-Streams approach anymore....". What do you mean by that? You can have multiple binders connected to different clusters


Add documentation on how to use sqlalchemy

Add mkdoc documentation on sqlalchemy as a developer tool

Omg amazing :)

any clue where the header buttons for navigating to the pr/diff in slack went?

bookmarks. it was better before :disappointed:

oh it works on hover? maybe not

this is pretty clean

@abjjabjj take notes

lol on the header design?

yeah, it's even clickable when the screen becomes smol

noted

on hover works for me btw

yeah it's working for me too, I didn't like it til i discovered that