[BUG] exception swallowed in RaiseAndSetIfChanged

Describe the bug

Exceptions thrown in an OnNext handler attached to an observable created via WhenAnyValue and RaiseAndSetIfChanged are swallowed and don't propagate, in contrast to the documentation of the default exception handler (https://reactiveui.net/docs/handbook/default-exception-handler/).
Steps To Reproduce

RoslynPad reproduction case:
#r "nuget:ReactiveUI/9.13.1"

using System.Reactive.Linq;
using ReactiveUI;

class ReactiveExample : ReactiveObject
{
    public ReactiveExample()
    {
        var o = this.WhenAnyValue(x => x.S).Skip(1);
        o.Subscribe(s =>
        {
            Console.WriteLine("received value " + s);
            throw new Exception("throw on value " + s);
        });
        //this.ThrownExceptions.Subscribe(e => throw new Exception("uncaught", e));
    }

    public string S
    {
        get => _s;
        set => this.RaiseAndSetIfChanged(ref _s, value);
    }

    private string _s = "";
}

var r = new ReactiveExample();
r.S = "bar";

Expected behavior

The exception thrown in the above code should propagate.
Screenshots

N/A
Environment

OS:       Windows
Version   10
Device:   PC

Additional context

I believe this is caused by ReactiveUI.IReactiveObjectExtensions.NotifyObservable, which doesn't rethrow the exception in case ThrownExceptions isn't subscribed.

As per your stack overflow question some additional details where to track it down.
I believe that this is caused by the try/catch in ReactiveUI.IReactiveObjectExtensions.NotifyObservable, which I would have expected to rethrow the exception in case the ThrownExceptions observable doesn't have a subscriber instead of just logging it (see https://github.com/reactiveui/ReactiveUI/blob/a4ee168dade8b5e3f34337fabd56eca10eca5200/src/ReactiveUI/ReactiveObject/IReactiveObjectExtensions.cs from line 382).

Not sure when we can get to it, hopefully soon. Everyone on the project has a lot of work related pressure at the moemnt.


renderPdf(texDirectory.getAbsolutePath(), texFile) is duplicated

Thank you for your project, I've used it for reference!
Probably found a typo:
LatexRenderJob runInternal method runs twice as long

LaTeX has to be run multiple times, if you're e.g. using table of contents/cross references in your documents. (Latexmk) (instead of pdflatex) is more efficient and taking care of that, but you'll have to install perl on your system as well. We're just running the pdf generation twice in order to take care of that. This will slow down the rendering process, but it's a simple hack to get correct results. If you aren't using references, you can also remove the 2nd invocation in order to be more efficient. The assumption is basically, that there will be more reads than writes so that the duration of the rendering process doesn't matter that much.


Error trying to download Forge

Hi, I'm getting a "-1" error trying to download Forge when I try to run python install.py on my modpack zip. I get a url to download the right version of forge in the terminal, when I open it in the browser the forge installer jar downloads. I've tried running installer.py with --manual and --open-browser, both gave "-1 error trying to download Forge".
Thanks for the great program!

Looks like it's running into a Requests-related error (probably network problems or the Forge website is blocking non-browser connections now), but my excellent error handling code doesn't tell what the error actually is.
I'll do some testing to see if I can reproduce the error.
As a quick test, can you change line 47 in util.py to
except requests.RequestException as e:

and add
print(repr(e))

after it? Then it should print out why it's failing.

im not OP, but i had the same error, heres my output:
`[pack data already unzipped]
[minecraft dir already created]
Installing modloader
Updating user launcher profiles
Downloading "https://maven.minecraftforge.net/net/minecraftforge/forge/1.18.2-40.2.0/forge-1.18.2-40.2.0-installer.jar
InvalidSchema('No connection adapters were found for '"https://maven.minecraftforge.net/net/minecraftforge/forge/1.18.2-40.2.0/forge-1.18.2-40.2.0-installer.jar\'')
Traceback (most recent call last):
File "install.py", line 393, in
main(
File "install.py", line 243, in main
modloaders[modloader].main(manifest, mcver, mlver, packname, user_mcdir, manual)
File "/home/gabe/Desktop/modpack-installer-master/forge_install.py", line 44, in main
resp = download(url, outpath, True)
File "/home/gabe/Desktop/modpack-installer-master/util.py", line 55, in download
return r.status_code
UnboundLocalError: local variable 'r' referenced before assignment`

Oh that's really subtle... looks like something is putting a double quote at the beginning of the URL. Thanks @jayx11

Do you have the repository cloned with git? If so, you should be able to do a git pull to update to the latest commit:
git reset --hard HEAD
git pull

Actually, it looks like you downloaded from github directly; can you clone the repository from the command line?
git clone https://github.com/cdbbnnyCode/modpack-installer

Or you can just download the latest version with the "Code" button on the GitHub page.

Closing since the original issue is presumably solved.


Can't begin a text selection outside the bounds of a RichTextLabel

Tested versions

Reproducible in Godot 4.2.2 stable
Reproducible in Godot 4.3-rc3

System information
Windows 11 Pro - Godot 4.3-rc3
Issue description
If you click and start to select text from the outside of a RichTextLabel it does not begin selecting. You must start selecting within the boundary.

In other GUIs, such as web browsers, selection can begin in the margins, which is useful because otherwise getting the first few characters is very challenging.

One thing I tried is to add an [indent] around the text using BBCode, and it allows you begin selecting outside of the line, but always begins with the beginning of the paragraph, not the current line you're on, which also is confusing to a user:

Steps to reproduce

Add a selectable RichTextLabel with text in it, and position it in a window manually or via MarginContainer
Run the project
Try to select text by clicking outside the rich text label and dragging your mouse into it. Nothing will happen.

Minimal reproduction project (MRP)
drag-example.zip

[indent] bug should be fixed by https://github.com/godotengine/godot/pull/95301 (and should also work when selecting from a content margin added by StyleBox).
Selecting from outside of control is a pretty strange thing to expect. I would not consider it a bug (web page act as a single control, so it's not the same, browser won't start selection if you are dragging from a menu bar for example).
Maybe open a proposal at https://github.com/godotengine/godot-proposals/ to see if it's a desired behavior.

Because #95301 allow this to work with a content margin this issue is no longer necessary, because I can add a content margin to the RichTextLabel and that allows the user to begin selecting before the letter and is very user friendly. Thank you.


Use correct message type for ICMPv6 echo request

ICMPv6 uses different message types compared to ICMPv4. So in order to allow ICMPv6 echo requests we have to use message type 128 instead of 8. See https://de.wikipedia.org/wiki/ICMPv6 for reference.
I also opened a PR fixing this issue. See #374.

Merged, see #374


PackedVector2Array remove_at() not removing from array

Godot version
4.0 rc3
System information
Windows 11
Issue description
I am trying to go through a PackedVector2Array and remove points that are too close together (redundant points).
But, when I try to remove points with remove_at(), it doesn't affect the array at all. I expected it to at least reduce the amount of points, but the array remains the same.
Steps to reproduce
I created a reproduction project which consists of a Line2D and a Polygon2D. At _ready(), it is supposed to remove points, but nothing happens. I output the number of points before and after calling remove_at() and there is the same amount of points.
Minimal reproduction project
bug_remove_at.zip

See #62857
You need to use remove_point()


Package simple-nunjucks-loader didn't indexed

Seems like analyzing of simple-nunjucks-loader stuck
It shows December 14, while latest release was December 23

Looks like latest version was just fetched.


Long strings of characters break the message layout in preview mode

Whether as plain paragraph text, a code block, or in a spoiler, long unbroken strings of characters break the message layout when previewing a message from the edit view:

Are you using desktop application of Zulip?

@git-init-priyanshu Thanks for showing interest in this issue.
Looks like this is your first issue at Zulip -- so I would first recommend you set up the development environment and then you will be all set to go. Also, make sure to read our Contributing Guide it will certainly help you in creating your first PR.  :rocket:

Are you using desktop application of Zulip?

Since our desktop app is built using Electronjs the bug is essentially there in both the desktop as well as the web application.

@sbansal1999 got it.


Fixes#313 Unable to replace track with new file on white_theme

Resolves #313

Upload/replace functionality works on white-theme (confirmed both locally and in specs).
The issue was that javascript was listening for the wrong class and needed an id handle to open the hidden pane. FIxed both the js' class name in querySelectorAll('.slide_open_next')  and added # to the view to be able to locate id= div in document.querySelector(id).

I was accidentally able to reproduce the issue in dark theme but tinkering with white_theme_enabled in the application controller. But since we are not going to keep that around for much longer, I didn't dig in why it might be happening.
"Ready For Review" checklist

[x] PR title accurately summarizes changes
[x] New tests were added for isolated methods or new endpoints
[x] I opened an issue for any logical followups. #722
[x] Update title and description to account for additional changes
[ ] All tests green
[x] Booted up the branch locally, exercised any new code
[ ] Percy changes are purposeful or explained
[ ] Css changes are happy on mobile (via Percy is ok)

Percy was my fault — I changed the timing on the "headway" widget to be 1 second, which makes it suddenly show up in the screenshots. I approved the changes.

@ofsound Would you be willing to take some temporary measure to resolve #722 in this branch?

@ofsound Would you be willing to take some temporary measure to resolve #722 in this branch? 💄 🐷

@sudara
I took some general styles from the upload a new album cover section.
Does the new file get uploaded on save, if so should that button change its label? Maybe a Upload File button would be friendlier?
I've also wondered about a "Cancel Button" on this template, what if you don't want to save your changes?

@ofsound nice!
Good point about the button UX.  Agreed that it should be “Upload New Version” or “Upload and Replace” or something more explicit. We could move the “slide open” pattern to be done in stimulus and add the label change.
Feels like with playlist/edit and now this, it’s time for a little stimulus sprinkle that disables the save button until changes are made...
Also, why is this save button hugging the left side, I wonder. Should it be on the right hand side to be consistent with our other forms?

Hai Faifs all around, as @smoofles would say. Let's merge this and handle save button patterns in #733


Find best way to filter and personalize results

We need to get an efficient way to alter the results by filters etc.

This particular issue is quite generic and there are plenty of issues about filtering already, so I'll close


feat: alcove.binds support comment(fix #1).

alcove.binds support comment and its can umount subdir now!

Test fails, closed.


replace hard-coded minimum for DWATN by runtime flag SEAICEdWatMin

What changes does this PR introduce?
Bug fix and feature
What is the current behaviour?
There's a hard-coded minimum of the linear ice-ocean drag coefficient DWATN that lead to unexpected behaviour (http://mailman.mitgcm.org/pipermail/mitgcm-support/2020-May/012503.html), but may also affect other simulations. This minimum was required for older versions of the LSR solver, which could not handle zero entries on the main diagonal of the system matrix. This conditions is no longer a requirement for LSR.
What is the new behaviour
New code allows to have zero drag by setting both SEAICEdWatMin and SEAICE_waterDrag to zero. The freedrift code still requires non-zero drag coefficients, and S/R SEAICE_READPARMS catches this case.
Does this PR introduce a breaking change?
No, default values remain the same
Other information:
Suggested addition to tag-index
replace hard-coded minimum for DWATN by runtime flag SEAICEdWatMin

allows to have zero drag by setting both SEAICEdWatMin and
SEAICE_waterDrag to zero. Non-zero drag had been implemented because
previously the LSR solver could not handle zero entries on the main
diagonal. This requirement has long been lifted.
this fixes a bug (unexpected behavior) reported at MITgcm-support by
Kaley Shrestha, but also allows to modify the somewhat arbitrary
default minimum value of SEAICEdWatMin = 0.25
freedrift code still requires non-zero drag coefficients,
S/R SEAICE_READPARMS catches this case

@jm-c thanks for the suggestions, I added them.

All look good to me. Going to merge soon (after updating tag-index).


Move swapfile to data volume (/var/lib/scylla)

Related with #491, we want to reduce the image size, but if we do that we will not have enough disk space for swapfile.
To fix the problem, we need to move swapfile to the data volume (/var/lib/scylla) to avoid running out disk space of rootfs.
see: https://github.com/scylladb/scylla-machine-image/pull/491#issuecomment-1842737739

It's not a good idea, wasting fast+expensive NVMe space.
Maybe we can add an EBS volume to the AMI.

bard is nice
but, the issue is that now you need to teach whom is using the image, that he need todo something else
to get swap as we recommend
the whole reason it was done in build time, was to save time and complexity for cloud deployment
the price for it was bigger AMIs.
we can't take a step back, without syncing it with whom is depending on it.

Bard:
To create an AMI that creates a new EBS volume and attaches to it, you can use a combination of user data and cloud-init to automate the process. Here's a general outline of the steps involved:

Create a script to create the EBS volume and attach it to the instance: Write a script that uses the AWS CLI or boto3 library to create a new EBS volume of the desired size and attach it to the instance. This script should be placed in the user data section of the instance configuration.

Create an AMI from the instance: Once the script has successfully created and attached the EBS volume, create an AMI from the instance. This AMI will capture the state of the instance, including the attached EBS volume.

Launch new instances from the AMI: When you launch new instances from the AMI, the EBS volume will be automatically attached to each instance. This ensures that each instance has the same storage capacity and configuration.

Here's an example of a user data script that uses the AWS CLI to create a new EBS volume of 100GB and attach it to the instance:
#!/bin/bash
Set the AWS region
aws_region=us-east-1
Create a new EBS volume of 100GB
aws ec2 create-volume --volume-type gp2 --size 100 --region $aws_region > volume.json
Extract the volume ID from the JSON output
volume_id=$(cat volume.json | jq -r '.VolumeId')
Attach the EBS volume to the instance
aws ec2 attach-volume --instance-id $(curl -s http://<IP_ADDRESS>/latest/meta-data/instance-id) --volume-id $volume_id --region $aws_region
Format and mount the EBS volume
mkfs.ext4 /dev/xvdf
mount /dev/xvdf /mnt/ebs
To create an AMI from the instance, you can use the AWS CLI or the Amazon EC2 console. Once the AMI is created, you can launch new instances from it and the EBS volume will be automatically attached to each instance.

I think bard is crap

but, we can certainly create a swap volume in cloud-init on the first boot.

Why does the root disk need to change?

We can add additional EBS volume with following change on packer configuration file:
diff --git a/packer/scylla.json b/packer/scylla.json
index f06cc7c..012e4b1 100644
--- a/packer/scylla.json
+++ b/packer/scylla.json
@@ -36,6 +36,12 @@
         {
           "device_name": "/dev/sdi",
           "virtual_name": "ephemeral7"
+        },
+        {
+          "device_name": "/dev/sdj",
+          "volume_type": "gp3",
+          "volume_size": 20,
+          "delete_on_termination": "true"
         }
       ],
       "ami_name": "{{user `image_name`| clean_resource_name}}",

With the configuration file, machine-image will have additional disk by default.
However, I haven't find a way to do same on GCE.
Maybe user have to manually attach additional disk.

How is it different from sizing the root disk ? both would be saved and we go back to the sizing, but you'll have two snapshots ? or I'm missing something

I don't understand why we should calculate the swap based on any disk size. Based on https://github.com/scylladb/scylla-machine-image/commit/069318bb48963aa5bede707579e499a8bd1288c4 , there is a simple calculation to the relevant size of the swap, the max value should be 16Gb, how long does it take to create it during first boot as @avikivity suggested?

Closing, since using the data volume for swap seems not good idea, as @avikivity commented on the thread.


[BEAM-3538] Remove (or merge) Java 8 specific tests module into the main one

Since Beam recently moved to Java 8 then we don't need to have dedicated module java8tests for that.
All tests from this module moved to corresponding test classes and module java8tests was deleted.
Also, it was added an annotation @Category(ValidatesRunner.class) or @Category(NeedsRunner.class) for those test where it was needed.

Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue.
[ ] Write a pull request description that is detailed enough to understand:

[x] What the pull request does
[x] Why it does it
[x] How it does it
[ ] Why this approach

[x] Each commit in the pull request should have a meaningful subject line and body.
[x] Run mvn clean verify to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

@jbonofre Thanks! Sure, it's resolved and pushed, let's wait for Jenkins.


Support for credential rotation?

Our data security team mandates that IAM secret keys be rotated every 90 days.  Does the library have support for that, or is that something on the roadmap?

Yes, the library supports updates to the client's IAM credentials (including secret keys, but also access keys and session tokens). The library itself uses standard AWS credential providers which pick up the new credentials automatically. Hope this answers your question.

Please let us know if there are any other questions or concerns.


CVE-2019-17268 report for omniauth-weibo-oauth2

Hey,
I took courtesy of reporting the malicious code injection in 0.4.6 into the CVE database.
https://diff.coditsu.io/gems/omniauth-weibo-oauth2/0.4.3/0.4.6
Just wanted to let you know. They initially assigned the CVE-2019-17268.

Hi Maciej,
thank you, i will take a look at tomorrow.
-Never
Maciej Mensfeld<EMAIL_ADDRESS>于2019年10月9日周三 下午8:17写道：

Hey,
I took courtesy of reporting the malicious code injection in 0.4.6 into
the CVE database.
https://diff.coditsu.io/gems/omniauth-weibo-oauth2/0.4.3/0.4.6
Just wanted to let you know. They initially assigned the CVE-2019-17268.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/beenhero/omniauth-weibo-oauth2/issues/36?email_source=notifications&email_token=AAVK66GIST2QQWQBZM2Y563QNXDW5A5CNFSM4I66SKI2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HQTZIXQ,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAVK66BSIILGTVYZJKNFUULQNXDW5ANCNFSM4I66SKIQ
.

ref: https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2019-17268
I believe that info about this breach should be more exposed.

Hi @mensfeld ,
Thank you for your information, get a CVE number make me feel surprised, I sent the abuse report to ISP for now. more information see the Issue report please:

Thank you! I did id so tools like bundler-audit can catch it.
Too many people use automatic dependencies upgrade tools, that's why I'm super paranoic about stuff like this.
I will close this issue as now it's on the Mitre side to verify.


Don't allow ActionController to deserialize to null; prevents NPE crash

I've been coming against residual NPE crashes where form loading fails because the FormDef's ActionController doesn't fail hard when deserialization fails, but rather to deserializes to null.
This PR makes form deserialization fail when the ActionController can't be deserialized. This forces the form to be re-read from xml and re-serialized.
To reproduce, load a form using an intent on master, then build https://github.com/dimagi/commcare-odk/pull/1128 and load the same form. The change in how IntentCallout is serialized somehow causes the FormDef deserialization to not load the ActionController correctly.
This change makes it so that if we change the serialization scheme of ActionController, or I guess other things as seen in the reproduction steps above, then we won't cause user level crashes when CommCare is updated on their devices and the load a form that had previously been loaded on older version of CommCare.

@phillipm Hey, thanks for flagging this. I have no idea if ExtWrapTagged is the right alternative here, though- none of those ExternalizableWrapper classes have javadocs so it's kind of hard to tell. But if you tested and confirmed that it fixes the crash, then happy to merge.


Use old isNaN

Use isNaN() instead of Number.isNaN(). Resolves #3.

published as<EMAIL_ADDRESS>


Fix for Cannot read property measure of null

We are having random crash for this null exception. Can you please merge this?

Thanks!
It's merged. Please install the newest version now<EMAIL_ADDRESS>


added encryption logic in secret.go

:nut_and_bolt: Description: What code changed, and why?

WIP Added encryption logic in secret-helper
:chains: Related Resources
:+1: Definition of Done
:athletic_shoe: How to Build and Test the Change
:white_check_mark: Checklist

[ ] I have read the CONTRIBUTING document.
[ ] Tests added/updated?
[ ] Docs added/updated?
[ ] All commits have been signed-off for the Developer Certification of Origin.

:camera: Screenshots, if applicable

Closing Pull request as i pushed some merged code that introduced changes which were not done by me


Skip pull if build is specified in docker-compose.rebuild

Merge docker-compose.rebuild-gitlab into docker-compose.rebuild
We don't really need a separate command for rebuilding gitlab... we'll
just use invoke docker-compose.rebuild -s gitlab.

Skip pull if build is specified in docker-compose.rebuild
The houston service has "build" and "image" specified, in the
docker-compose.rebuild script, we are definitely building the image,
so there is no need to download the image (skip pull).

/rebase

/rebase


GUI textures and other text visuals.

I am in the midst of re-texturing everything I can get my hands on, but the few things I cant texture or at least I dont know if I can change, are the power bars and the text around various parts of the GUIs.
Text has a lot of problems, overlaps with themselves or visual textures like power bars in the screen. (Also the screen needs more dimensions). Speaking of screens, for liquids they should at least take the color of the liquid. Also, for most if not ALL guis with power, the written text indicator shows both the cap and the current charge, when not only the cap is somewhat redundant in most cases but the text sits on top of the visual charge bar texture, with black text.
The Charge bar texture is basic and ugly, and there is no where that I can change it physically in the texture files, so I assume it's coded in. Which is pretty bad.
Want to make it like TE's charge bar design, but alas.
Also, FYI, using aliased text in the GUI textures without proper measuring is bad.

The RF bars use DrawGradientRect in code indeed.
What do you mean about aliased text? I'm just using basic text writing in MC with the default font. I certainly don't specify any aliasing (wouldn't even know how)


[Feature request] Keyboard accessibility issues*

What problem does this feature solve? : Keyboard accessibility issues
Hello everyone! Could you please add "tabindex" for each date? For now all buttons are available, but, not a dates :(

Yes. At the moment, it's not possible to use the Vue3 vue-datepicker-next using keyboard only.
I tried vue2-datepicker, and the Vue2 version also does not support this.
It whould be awesome and really helpful if you add this essential accessibility requirement.


[Suggestion] Use macro to reduce duplicate code

Hi @nyuichi,
Should we use macro for impl Serialize to reduce duplicate code? For example:
impl Serialize for u8 {
    fn serialize(&self, serializer: &mut Serializer) {
        serializer.serialize_int(Int::from(*self))
    }
}

impl Serialize for u16 {
    fn serialize(&self, serializer: &mut Serializer) {
        serializer.serialize_int(Int::from(*self))
    }
}

If you agree, I can send a PR.

Thanks for your reply :)
Of topic, do you have any plan to support std::io::Write trait in Serializer?


cocoa pods update?

Can you please release 1.1.0 to cocoa pods? Currently cocoa pods still depends on ReachabilitySwift 4.3.1
Thanks in advance

Hi,
I just got assigned ownership of the project on GitHub. I have to get admin access through CocoaPods though to make any schema changes.
Will work on that, might take a minute unless someone knows a quick way to accesss an admin at cocoapods spec repo.

#37 is a blocker for this isue

Hi,
I'm also looking forward to this update, it's a blocker for a SDK I'm writing.
Thanks! 😊

Hi,
I'm also looking forward to this update, it's a blocker for a SDK I'm writing.
Thanks! 😊

You should be able to override the default podspec and point to 'master'.
It's a stable release branch with an updated podsec. The issue I have is I don't have admin access yet to Cocoapods central repo update system. Actually need to contact them about that still if I knew whom to contact that will actually respond.

Hi,
I'm also looking forward to this update, it's a blocker for a SDK I'm writing.
Thanks! 😊

You should be able to override the default podspec and point to 'master'.
It's a stable release branch with an updated podsec pointing to RxSwift 5.0.
The issue I have is I don't have admin access yet to Cocoapods central repo update system. Actually need to contact them about that still if I knew whom to contact that will actually respond.

Thank you for the very quick reply!
I wish I could, but as far as I know, it's not possible in my situation. My SDK has dependencies from both Carthage (whenever possible) & Cocoapods (for a very few number of libraries that don't work well with Carthage).
I'm using RxReachability@master &<EMAIL_ADDRESS>with Carthage.
My issue is I want to generate documentation with Jazzy, which doesn't integrate well with Carthage.
My solution to that is to write a ${PROJECT_NAME}-Documentation.podspec file, that lists dependencies from both Carthage & Cocoapods.
And unfortunately it seems it's not possible to specify a git source for a s.dependency, hence dependency resolution failure...

I just got publishing rights from CocoaPods team.
It's on my agenda for the next rainy day to push the spec upstream. Winter is coming.

Published to ccpods.


Using DWD with the Admin SDK on GCE

I'm struggling to find the proper way to create credentials that can be used with the Admin SDK (directory api).
TL;DR: How can I use DWD on GCE using default credentials?
Using this API required enabling domain-wide delegation (DWD) and that's been done. The code here works well for local installations of Forseti (with the properly supplied flags).
However, I'm unable to find the proper oauth flow for doing this on GCE. I've properly given the GCE instance the correct scope https://www.googleapis.com/auth/admin.directory.group.readonly. I'd assume I could use the GCE credentials classes, but there seems to be no class that also allows me to call .make_delegated() which is required for the API.

GCE credentials can not sign bytes which is needed in order to generate the assertion necessary to do domain-wide delegation. You'll need to use a service account keyfile.


$20 (white) is being mapped to$10 (grey)

https://github.com/clbr/neslib/blob/d061b0f7f1a449941111c31eee0fc2e85b1826d7/neslib.sinc#L1232
I don't understand why this was done. It seems intentional, but I can't figure out what it's trying to do. The comment says:

;$10 because $20 is the same as $30

I think I understand the intent here now.
The library is asking artists to NOT use $20, and instead use $30 as they are the same visually.
As a result, the fade code can behave a little nicer, by skipping passed $20 to $10 (grey) so that the fade is instant.

That's correct. Closing.


379 implement ide model with age resolution

Changes and Information
Please briefly list the changes (main added features, changed items, or corrected bugs) made:

Added The Possibility to use Age Groups in the IDE SECIR Model
m_transitions is still a Time Series but at each time step we now store a Vector auf size number Transitions * num_agegroups.
In order to find the right Index in this Time series the function get_transition_flat_index was added.
m_populationss is still a Time Series but at each time step we now store a Vector auf size number Infection States * num_agegroups.
In order to find the right Index in this Time series the function get_state_flat_index was added.
m_N, m_forceofinfection, m_total_confirmed_cases are now vectors of size num_agegroups.
TransitionDistributions, TransitionProbabilities, TransmissionProbabilityOnContact, RelativeTransmissionNoSymptoms, RiskOfInfectionFromSymptomatic are now Custom Index Array.
In model.cpp we compute the Transtions and Compartments for each Age Group separately. The compution of the force_of_infection was slightly changed.
Added an Example using two Age Groups.
The test ide_secir.cpp still tests the same things as before and uses only one Age Group.
Added The Test File test_ide_secir_ageres.cpp where a Test to Compare with Previous run is implemented
Changed The Test File test_ide_secir_parameters_io.cpp. We now use the data from cases_all_age_ma7.json, therefore we are using 6 Age Groups.

If need be, add additional information and what the reviewer should look out for in particular:

Merge Request - Guideline Checklist
Please check our git workflow. Use the draft feature if the Pull Request is not yet ready to review.
Checks by code author

[x] Every addressed issue is linked (use the "Closes #ISSUE" keyword below)
[x] New code adheres to coding guidelines
[x] No large data files have been added (files should in sum not exceed 100 KB, avoid PDFs, Word docs, etc.)
[ ] Tests are added for new functionality and a local test run was successful (with and without OpenMP)
[ ] Appropriate documentation for new functionality has been added (Doxygen in the code and Markdown files if necessary)
[ ] Proper attention to licenses, especially no new third-party software with conflicting license has been added
[ ] (For ABM development) Checked benchmark results and ran and posted a local test above from before and after development to ensure performance is monitored.

Checks by code reviewer(s)

[ ] Corresponding issue(s) is/are linked and addressed
[ ] Code is clean of development artifacts (no deactivated or commented code lines, no debugging printouts, etc.)
[ ] Appropriate unit tests have been added, CI passes, code coverage and performance is acceptable (did not decrease)
[ ] No large data files added in the whole history of commits(files should in sum not exceed 100 KB, avoid PDFs, Word docs, etc.)
[ ] On merge, add 2-5 lines with the changes (main added features, changed items, or corrected bugs) to the merge-commit-message. This can be taken from the briefly-list-the-changes above (best case) or the separate commit messages (worst case).

Closes #379

We should prevent that we are dividing by 0 when computing the force of infection if the population in some age groups is 0. For this we can make use of the solution that was implemented in #1104 for the ODE model.


Cannot convert from square millimeter to square feet

As described in the title.

This error is because ft2 isn't a recognized unit. I have updated the library to add it as an alias for square feet.

:tada: This issue has been resolved in version 4.10.0 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Radio station is added in the queue after playing TTS Ultimate

Hi,
Thanks for the great plugin, but I noticed that TTS Ultimate does not restore exactly the previous state. For example when I start listening to an online radio station (from the Juke plugin on Sonos) with the use of the Sonos App. And then when I start TTS Ultimate to play something, afterwards the radio station is added as a queue item instead of just playing the station again.
I guess it's related to the way the previous state is captured and played again. I noticed when using group.create.snap than execute TTS and then do a group.play.snap afterwards the original state is played again (without a queue). Maybe this can be done directly in TTS Ultimate?
Mark

Hi, thank you for the suggestion.
TTS-Ultimate is focused mainly on the TTS and reliability, while node-red-contrib-sonos-plus shines on music.
Currently, i cannot simply change the underlying functions, because it's used by some alarm manufacturers, but i can take a look at how sonos-plus handles the queues, for a future implementation.
;-)


Invalid Link on Links that used to work.

Some Direct Download links simply do not work, and some dont work with ones that used to work.
Also, we need to be able to use https://disk.yandex.com as a modpack host.

I don't use yandex but I would assume file access is via cookie. You can't use that. You need to have direct url file access. Hence why the solder docs specify an apache or ngix webserver

@spannerman79 Is correct, you'll need a direct download link from disk.yandex.com for it to work as a host.


Edit PHP 0001: Remove PHP FPM from built-in server group

Summary

Remove typo in which the FPM buildpack was added to the built-in server order group erroneously.
Use Cases

Checklist

[X] I have viewed, signed, and submitted the Contributor License Agreement.
[ ] I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
[ ] I have added an integration test, if necessary.
[X] I have reviewed the styleguide for guidance on my code quality.
[X] I'm happy with the commit history on this PR (I have rebased/squashed as needed).

@sophiewigmore Isn't the error in this group that both php-fpm and php-builtin-server weren't marked optional?
By removing php-fpm from this group, isn't the use case of standalone FPM being ignored? RFC says Separation of FPM into a separate buildpack lets users run FPM in one container and web server in another container.
I have no strong opinions either way - thought I'd drop my $0.02 as I stumbled upon this :)

@arjun024 I don't think the built-in server should be optional here! I spoke to Dan earlier and he was explaining that there's not really a use case for built-in server + FPM, so I figured that would hold true for the case where they run on one container, as well as in cases where they run separately


autopep8 the code and then flake8

See #384 to understand the why...

run autopep8 on all Python code in the repo.  (TONS OF CHANGES!!)
Fix a few things by hand
Move from Pyflakes8 to flake8 with a short list of IGNORE issues.

The migration from pyflakes8 to flake8 seems reasonable to me. I'm less convinced about the mass reformatting, but perhaps others (@blatinier, @posativ) have stronger opinions on that?

Ok for flake8 too. The mass reformatting makes sense at least for 95% of it, the last 5% could have been formatted better to fit the pep8 but it's better this way than before.
I merge.


BSD and Cybersecurity issues

Hi techies,
I am going through the NetX Duo documentation and source code,

As per the NetX Duo documentation, BSD APIs are not fully compatible, is there any future release planned to support all BSD calls
In source code We had observed critical and cybersecurity issues while running static code analysis tool on NetX Duo firmware. are you having same issue?

Duplicate as support ticket<PHONE_NUMBER>004913. Close this one to unify the discussion.

@TiejunMS Do you know when Microsoft is going to release patches or updates to address those cybersecurity issues?

@hwmaier , just so you know, we haven't received issue report yet.


Chat hallucinate

When I query something using chat, it often provides other generic info

Can you provide some examples?
I guess this is to be expected, although not ideal. The prompts attempt to guide the LLM away from hallucinations but I don't know that we'll ever be able to get away from them entirely.
Perhaps once we get full-on LangChain support (using Docker) things might get better


Refused to connect to 'https://q.stripe.com/?{query}' because it violates the following Content Security Policy directive: "connect-src 'self' https://api.stripe.com https://errors.stripe.com https://js.stripe.com".

Summary
There is an issue with Content-Security-Policy using Stripe. There is a suspicious console error reloading checkout page or moving back from it.
Stripe JS is used (https://github.com/stripe/stripe-js/tree/v1.10.0) on client-side of our project. After using 'stripe.redirectToCheckout' (https://stripe.com/docs/js/checkout/redirect_to_checkout) and saving card we periodically can see following error in console: "refused to connect to https://q.stripe.com/?{query} because it violates the following Content Security Policy directive: connect-src 'self' https://api.stripe.com https://errors.stripe.com https://js.stripe.com".
An investigation was made and it is clear that there is a request GET https://checkout.stripe.com/pay/{cs_test}  with Content-security-policy in response header with the same connect-src directive which is violated as was said in error message above. We have an assumption that https://checkout.stripe.com is trying to connect to https://q.stripe.com (this is defined as report-uri of CSP header) and violates connect-src directive. Content-security policy is not configured in our system itself, so it seems as Stripe internal issue. The question is how this can be fixed?
Other information
Browser: Google Chrome Version 87.0.4280.66 (Official Build) (64-bit) (Ubuntu)
For Firefox there are no errors, but 'q.stripe.com' requests are failed sometimes.

Hi @olyverDev, thanks for the detailed report. Stripe.js and Stripe Checkout use q.stripe.com for internal telemetry. It currently does so using an Image beacon, and all Stripe Elements frames and Stripe Checkout are correctly configured to allow that origin for img-src. However Chrome has a bug we've reported to Google where on page unload, the connect-src rule is used for image fetch instead of img-src, which causes some console spam.
Since these request are purely for internal telemetry, there are absolutely no impact on the behavior of Stripe products. These requests are also sometimes blocked by overzealous ad blocker. We apologize these errors are causing console spam, but they can be safely ignored.


Produce not fetching app_name automatically from Fastlane

On running fastlane deploy app_name:McDonaldsZom version:1.0, the fastlane detects the app_name automatically but the produce command prompts for the app_name again as seen in the screenshot. On explicitly provide app_name inside produce ( ... ), it detects the app_name. So, is it possible to pass the app_name directly via fastlane deploy or is the app_name stored somewhere in an Environment variable?

You didn't share your deploy.sh or Fastfile, but I assume: Try passing the name to the produce action:
produce(app_name: options[:app_name])

Thanks. Adding this to my fastfile worked -
lane :deploy do |options|        #cert     produce(     app_name: options[:app_name]     )

Great to hear :+1:


Xcode 12.4 Module 'FirebaseFirestore' not found

Hi, I'm trying to build my iOS app developed on ionic. When building on Xcode, I get the error.
Screenshots:

As for my Podfile, here is the content:

Also, if I build with $ ionic cordova build ios --prod --release. I get this error
The following build commands failed:
	CompileC /Users/jans/Library/Developer/Xcode/DerivedData/Jan-bbcnrsrdkhulxxgtcbuzbbwmxmej/Build/Intermediates.noindex/Jan.build/Release-iphonesimulator/Jan.build/Objects-normal/arm64/AppDelegate+FirebasePlugin.o Jan/Plugins/cordova-plugin-firebasex/AppDelegate+FirebasePlugin.m normal arm64 objective-c com.apple.compilers.llvm.clang.1_0.compiler
(1 failure)
xcodebuild: Command failed with exit code 65
[ERROR] An error occurred while running subprocess cordova.

Hi there!
No idea why the import isn't working, that's quite strange. Usually when things like that happen it is from opening it via the .xcproject instead of the .xcworkspace but that's just a medium-percentage guess
You have noted the command that failed, but not the actual failure reason (usually up higher in the build log, after the command is issued, there will be details about the problem)
Tangent: I can't imagine using version 6.33 of the firebase-ios-sdk though, is there some reason 7.x.x is not accessible?

@mikehardy Thanks for replying!
Regarding using 7.x.x version, I tried 7.6.0 and I also encountered the same problem. If it helps, I assume this was auto generated from cordova-plugin-firebasex.

I'm not sure we will be much help here unfortunately. There is nothing inherently complicated about what these pre-compiled frameworks offer and in a standard Xcode project they should be a drop-in replacement for FirebaseFirestore - with all header / library paths etc working. I have no Cordova reproduction environment so I won't be able to troubleshoot this, all I can say is that it does work in multiple ecosystems (specifically, Flutter and react-native) so it should be possible, but I can't say farther than that.
Maybe someone else with Cordova experience will be able to help but you may have more luck on stackoverflow if there is an active Cordova + firebase community there - this is the first Cordova traffic we've seen here so it's unlikely to get attention without reaching out to other communication platforms

Hi, I'm trying to build my iOS app developed on ionic. When building on Xcode, I get the error.
Screenshots:

As for my Podfile, here is the content:

Also, if I build with $ ionic cordova build ios --prod --release. I get this error
The following build commands failed:
	CompileC /Users/jans/Library/Developer/Xcode/DerivedData/Jan-bbcnrsrdkhulxxgtcbuzbbwmxmej/Build/Intermediates.noindex/Jan.build/Release-iphonesimulator/Jan.build/Objects-normal/arm64/AppDelegate+FirebasePlugin.o Jan/Plugins/cordova-plugin-firebasex/AppDelegate+FirebasePlugin.m normal arm64 objective-c com.apple.compilers.llvm.clang.1_0.compiler
(1 failure)
xcodebuild: Command failed with exit code 65
[ERROR] An error occurred while running subprocess cordova.

Did you ever solve this issue? How?

I noticed a similar issue in a React Native project:

It happens only for Release targets. When the pod is removed the builds succeed.
Pod version we use:

pod 'FirebaseFirestore', :git => 'https://github.com/invertase/firestore-ios-sdk-frameworks.git', :tag => '6.30.0'

Hi there! Not sure why you would use a version from more than a year ago? It may be that some non-forward-compatible changes have leaked in somewhere? For react-native this is definitely definitely working in modern versions though. I maintain react-native-firebase which I mention because it carries the implication that I manage the CI integration there and merge PRs (so I define the build steps and watch the builds) and we average more than a build a day, plus my own app which uses this framework, and it all builds cleanly, including for release

Thanks @mikehardy, we updated to the newest version and this build issue disappeared indeed! 👏


docs(Advanced guides): React without ES6

Translate Advanced guides - React without ES6

I would like to discuss about the translate of this articles title React Without ES6.
I personally think that 不使用 ES6 開發 React is more fluent than the original translate 沒有 ES6 的 React, how do you think of that?
However, the slight problem about choosing 不使用 ES6 開發 React is that we'll need to unify the translation of React Without JSX as well.

I would like to discuss about the translation of this articles' title: React Without ES6.
I personally think that 不使用 ES6 開發 React is more intuitive than the original translation 沒有 ES6 的 React, what is your thought on this?
However, the slight downside about choosing 不使用 ES6 開發 React is that we may need to unify the translation of React Without JSX as well.

Hi, any thoughts on this?

I would like to discuss about the translation of this articles' title: React Without ES6.
I personally think that 不使用 ES6 開發 React is more intuitive than the original translation 沒有 ES6 的 React, what is your thought on this?
However, the slight downside about choosing 不使用 ES6 開發 React is that we may need to unify the translation of React Without JSX as well.

I think your opinion is great, can you help to update React Without JSX docs?

Of course, I'm glad to help!

I would like to discuss about the translation of this articles' title: React Without ES6.
I personally think that 不使用 ES6 開發 React is more intuitive than the original translation 沒有 ES6 的 React, what is your thought on this?
However, the slight downside about choosing 不使用 ES6 開發 React is that we may need to unify the translation of React Without JSX as well.

I think your opinion is great, can you help to update React Without JSX docs in this PR?

Updated in 78ee1a2


internal/core/adt: add isconcrete builtin func

This PR implements isconcrete buildin func from https://github.com/cue-lang/cue/issues/943 proposal.
isconcrete(expr) :: bool
isconcrete reports whether expr resolves to a concrete value, returning true if it does and false otherwise. It is a fatal error if an expression can never evaluate to true.
Motivation: replaces if a.foo != _|_ { field value check without hiding the underlying error.

Notice that

the proposal for exists exhibit the same behavior
if let u= is replaced by u: then the problem disappear


[EngSys] revert using different version of NodeJS for build and test

The original goal was to workaround the issue of not being able to run pnpm on unsupported NodeJS v8.
Currently pnpm v7 that we are using runs fine on v14, v16, v18, and v20. There's no need to have the workaround.

pnpm v8 dropped nodejs v14


Update machine-specific-install.rst

A few clarifications in the cheyenne section.

Sorry - I'm not sure how to do that.
On Tue, Mar 8, 2022 at 9:21 AM Zachary Moon @.***>
wrote:

@lkemmons https://github.com/lkemmons any chance you could add this to
#70 https://github.com/NOAA-CSL/MELODIES-MONET/pull/70 instead?
—
Reply to this email directly, view it on GitHub
https://github.com/NOAA-CSL/MELODIES-MONET/pull/72#issuecomment-1061956682,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AH5BH7KGCJW7JPPHWY5NS7DU6547RANCNFSM5QGZS4WA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

Sorry - I'm not sure how to do that.

That's ok, I can do it. But you don't mind if I do?

That would be great, thanks.
Louisa
On Tue, Mar 8, 2022 at 9:27 AM Zachary Moon @.***>
wrote:

Sorry - I'm not sure how to do that.
That's ok, I can do it. But you don't mind if I do?
—
Reply to this email directly, view it on GitHub
https://github.com/NOAA-CSL/MELODIES-MONET/pull/72#issuecomment-1061963522,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AH5BH7ML5RHY2JWWSMLXOA3U655YDANCNFSM5QGZS4WA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

Moved these changes to #70


CT-1065 RxAndroidBle Publish

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


ADOPTERS.md: Fix OKD documentation link

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

NONE

/test pull-kubevirt-unit-test


Add a FAQ on contention

I've been seeing this so many times everywhere (on the forum, on gitter, from sales etc) that I am surprised we didn't FAQ this earlier. Here it is.

This change is

Once the copy-editing is done I'll do the propagation to the other version directories

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/bf002dc13a534c04dccb2c5ea3a2223f8c85ec08/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/4a8ee438a663807f7a781f5aa80a7399034173fd/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/a90321916660dcb0f65a71d276625c0754c980be/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/80362761a5f15be7a314c9770715d56e6bd2e145/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/8915bc846db781d2a3fcc39fbcb216e8209a1a06/

The title must stay because we often talk about contention in our explanations to users (and in blog posts, and other things) and it would be good for people to have a definition/explanation to search for if they want to understand our answers better.
However I agree we must provide additional entry points to make this information better discoverable. I'll make some attempt. Will let you know.

Review status: 0 of 1 files reviewed at latest revision, 5 unresolved discussions, all commit checks successful.

Comments from Reviewable

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/85f87a9fc7b8724e7b7886cef39d74565b074f1e/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/1f112e524e3365c3dc5e67c780c93ac22b1f91b8/

Review status: 0 of 2 files reviewed at latest revision, 5 unresolved discussions, all commit checks successful.

Comments from Reviewable

Review status: 0 of 2 files reviewed at latest revision, 11 unresolved discussions.

v2.1/performance-best-practices-overview.md, line 232 at r3 (raw file):
Previously, jseldess (Jesse Seldess) wrote…
nit: hold together > are met.

Done.

v2.1/performance-best-practices-overview.md, line 270 at r3 (raw file):
Previously, jseldess (Jesse Seldess) wrote…
Probably best to link directly to the specific FAQ you have in mind.

There are multiple. How do you suggest to do that?

v2.1/performance-best-practices-overview.md, line 287 at r3 (raw file):
Previously, jseldess (Jesse Seldess) wrote…
nit: place comma before the and (the "oxford comma").

Done.

v2.1/performance-best-practices-overview.md, line 305 at r3 (raw file):
Previously, jseldess (Jesse Seldess) wrote…
Is this paragraph introducing the following bulleted list? If so, use a semicolon instead of a period at the end of the last sentence.

Done.

v2.1/sql-faqs.md, line 47 at r3 (raw file):
Previously, jseldess (Jesse Seldess) wrote…
This doesn't need to be its own paragraph. Please make it the second sentence of the previous paragraph.

Done.

v2.1/sql-faqs.md, line 55 at r3 (raw file):
Previously, jseldess (Jesse Seldess) wrote…
I'd rephrase this a bit: Why would increasing the number of nodes not result in more operations per second?

Done.

Comments from Reviewable

v2.1/performance-best-practices-overview.md, line 270 at r3 (raw file):
Previously, knz (kena) wrote…
There are multiple. How do you suggest to do that?

You're right. You can just leave it as-is. Users will see the toc and jump to the FAQ that interests them.

Comments from Reviewable

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/a4541adeca7f882f8b2e2b8d71c72002f5fa5a6d/


"It works!" but no functionality on macOS 10.14.6

Hello! Thanks for writing and sharing Tabula.
I am a new Tabula user. I've tried to run Tabula by downloading Tabula 1.2.1 from https://github.com/tabulapdf/tabula/releases/download/v1.2.1/tabula-mac-1.2.1.zip as linked from your home page at https://tabula.technology/. I've followed the install instructions and am able to run Tabula, but after firing it up, I just get a plain web page in my browser saying "It works!" instead of a prompt or form to upload a file.

I can't proceed with the instructions in "How to Use Tabula: 1. Upload a PDF...", since there seems to be no upload affordance here.
Any ideas what to do here?
I am running macOS 10.14.6 on a 2019 iMac Pro; Firefox 69.0.1; Tabula 1.2.1.

Hi @apjanke — You have something else (most likely the Apache web server) running on port 8080. Try to shut it down, and then start Tabula again.
Thanks!

That was it. User error. Thanks!


Advanced Layout Bootstrap - wrapping formGroup with custom layout

Hi,
i've been playing a lot with formly trying to obtain an advanced layout like this one

Unfortunately I didn't succeed. The only workaround i've found was to create a custom input template, like in the "nested form" example (http://angular-formly.com/#/example/other/nested-formly-forms), but the problem here is that i need to change my model data wrapping all the fields that i want in a new panel with an index key refers to custom template.
Is there any other way to obtain that?
Thanks

Hi @eliagentili! Thanks for your interest in angular-formly. I'm sure that angular-formly can do this for you. The best way to get help is by following the instructions here: http://help.angular-formly.com
I'm going to go ahead and close this issue because the github issues for this project are reserved for bug reports and feature requests. See you on chat! Thanks!

@eliagentili did you try using several <formly-form> tags in your template?


Implement overview keyboard navigation

Navigation action:

[x] left
[x] right
[x] up
[x] down
[x] enter

Cypress test:

[x] left
[x] right
[x] up
[x] down
[x] enter

Resolve #565

I can't get it working locally. Is it ready?

All shortcut not working or just up an down?
I found a bug of the up and down, fixing now.

@antfu I add some cypress test to make sure it is working for now.
Plz review it if you have free time.


adjust view of locations fot easier QAing

We may need to adjust the locations view (the table( because we would like to use it to aid QAing. Some of the  columns are not useful right now (ex request a call) and we need to be able to more readily see who created the location to give feedback to them.
Here is none of my patented mockups for reference:

@EveCoff Do you have anything to add to this?
Thanks!

This screenshot is so useful!

Here is a related request that will also speed up QAing on locations. (much like this request for adding more info at the top of the page when viewing a report:  #487 )
proposed view:

Currently there is not much info at the top of a locations report, and to see who created the location and when, I need to scroll to the bottom, and open up a hidden bit to get that information. It would be nice if this could live at the top of a location.

For the proposed location list view, should those additional fields be added to the end of the list? And did you want to keep latest_non_skip_report_date: Latest report that is NOT is_pending_review and does NOT have a skip tag

another bump on this. would love to get the 'header' information at the top of the individual location records. this would make things much easier to QA!


About new datasets

Due to the needs of the comparison experiment, I used this model to run my own data set (maintaining the original parameters), but found that the results were not normal. Are there any parameters that need to be paid attention to?

According to the No Free Lunch Rule, no one can ensure that a method fits every dataset well. So, please do parameter tuning.


Adding ContainerResource.MapDockerSocket

Adding ContainerResource.MapDockerSocket to allow turning off docker-n-docker
These are the agent side changes to pay attention to the flag.
There are matching changes to the Server side to allow the yaml to set the flag.
Testing
I manually tested this new setting on Windows. I hope to setup Linux as well for testing.

@jtpetty I didn't see any unit tests around this. There are some for HostContext, but nothing tests JobContainer == true.


Revamp + automate Task Table documentation feature

We used to have a docs page at https://github.com/EleutherAI/lm-evaluation-harness/blob/master/docs/task_table.md that listed out all the tasks callable in lm-eval, along with some basic info about them. Its construction was also automated, so one could just run a Python script to regenerate it with new tasks included.
It'd be great to revitalize this now that we have a bunch more groups / subtasks / variants we've added or intend to add.
New features that would be awesome would be:

[ ] List out all individual tasks, along with a few key config attributes (output_type, in particular)
[ ] as a separate sub-table: list out all task groups that are supported
[ ] (Stretch) allow one to specify a one-sentence description field within a YAML config's metadata field, that just says what the task is e.g. 'Abstract Algebra subject in the popular MMLU benchmark, as implemented by original paper' or 'variant of MMLU as implemented by FLAN 2022 paper.' or 'Benchmark for testing formal logic capabilities' that we could auto-populate a blurb column of this table with.

I could probably set this up if @lintangsutawika can provide me a stub to read all the configs from the task folder.

Would this help?
https://github.com/EleutherAI/lm-evaluation-harness/blob/main/scripts/make_table_tasks.py

yeah this is good. But I was looking through the tasks/__init__ file but couldn't find an easy way to read the configs without initialising the tasks.

closed by #1929 for now


problem with the new api for collectionview

TabrisJs Version 2.0 RC1
Moto E2 Android 5.1
Hello everyone
By following the new API of collectionview and by using the bookstore example of the application tabris js 2.0 rc1 I wrote this collectionview but nothing is displayed.
let drawer = tabris.ui.drawer;
drawer.enabled = true;
let compositeDrawer = new tabris.Composite({ left: 0, top: 0, height:70, right:0, }).appendTo(drawer);
let textViewCompositeIdentifiant = new tabris.TextView({left:15, top:14, text:"bla bla"
 }).appendTo(compositeDrawer);

let textViewCompositeEmail = new tabris.TextView({left:15, top:30,text:"bla bla"
}).appendTo(compositeDrawer);

let itemConfig = [{
    title:"examination",
    image:"icons/android/exams_concours.png"
 },
 {
    title:"Planning",
    image:"icons/android/planning_cours.png"
 }];

let drawerCollectionView = new tabris.CollectionView({
    itemCount:itemConfig.length,
    cellHeight:64,
    createCell: () =>{
      let cell = new tabris.Composite({right:0, bottom:0, top: 100, });
      let imageViewCell = new tabris.ImageView({centerY:0, id:"imageViewCell" }).appendTo(cell);
      let textViewCell = new tabris.TextView({centerX:0,id:"textViewCell" }).appendTo(cell);
       return cell;
     },
   updateCell: (view,index) =>{
     let page = itemConfig[index];
     view.find("#imageViewCell").set("image",page.image);
     view.find("#textViewCell").set("text", page.title);
    }
    }).on("select", ({index}) =>
      {
       let indexItem = itemConfig.index;
       drawer.close();
    }).appendTo(drawer);

Hope that my snippet is clear

Try the Drawer Pages snippet; it sounds like it's exactly what you're looking for.

ok

I think the main problem is that you aren't setting a height or width for the CollectionView which does not have its own intrinsic height/width.

@ishigo1987 Did this advice help to resolve your problem?

yes thank you very much


Subdomain grav web and admin split because invalid security token

Hello everyone again
I've gone through about a dozen troubleshooting steps, but I can't figure out the problem anymore.
Grav is in the /domains/test.domains.com/ folder
The first installation will create a website for me at the URL
https://test.domains.com/domains/test.domains.com/
And everything works, but I used URL editing according to the instructions for:

And changed htaccess, the site is now functional only at
https://test.domains.com/
The problem is that you cannot log in to the administration under the URL
https://test.domains.com/admin
It goes only under the old URL
https://test.domains.com/domains/test.domains.com/admin/
But when I log in, the links in the administration are adjusted to the new address and are therefore non-functional.

I've gone through the configuration files, but I can't split the site to normal and admin to the original URL and% n can't log in because an invalid security token is still written, which is just a change.
Some ideas?
I know that this has already been solved here, but unfortunately no one has solved this yet.

Maybe you should try to copy the config/security.yaml file over to your environment configuration so that you have the same salt in both domains.


test: TopN: reduce Put times in TestTTL

What problem does this PR solve?
Sometimes CI will fail due to the TestTTL of TopN.
What is changed and how it works?
Reduce the times of Put.
Check List
Tests

No code

Codecov Report

Merging #2026 into master will increase coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2026      +/-   ##
==========================================
+ Coverage    77.5%   77.53%   +0.02%
==========================================
  Files         180      180
  Lines       18039    18039
==========================================
+ Hits        13982    13987       +5
+ Misses       3001     2996       -5
  Partials     1056     1056

Impacted Files
Coverage Δ

server/schedulers/shuffle_hot_region.go
58.51% <0%> (-7.45%)
:arrow_down:

pkg/etcdutil/etcdutil.go
82.6% <0%> (-5.8%)
:arrow_down:

server/member/leader.go
76.53% <0%> (-3.58%)
:arrow_down:

server/schedulers/random_merge.go
60.71% <0%> (-3.58%)
:arrow_down:

server/cluster/cluster.go
80.94% <0%> (-0.24%)
:arrow_down:

server/server.go
83.62% <0%> (+0.58%)
:arrow_up:

server/grpc_service.go
59.17% <0%> (+1.29%)
:arrow_up:

server/tso/tso.go
81.02% <0%> (+3.64%)
:arrow_up:

server/region_syncer/client.go
85.52% <0%> (+7.89%)
:arrow_up:

server/kv/etcd_kv.go
84.41% <0%> (+9.09%)
:arrow_up:

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 96e65f7...552dfb2. Read the comment docs.

/rebuild

/rebuild

/rebuild

/rebuild

Tested 5 times in CI, 5 times in IDC, no error occurred.

/merge


Google spreadsheet component limitation

https://github.com/elasticio/support/issues/1503
Freshdesk Ticket ID: 8195
Freshdesk Ticket Agent: Tigran Khanzadyan
Freshdesk Ticket Agent Email<EMAIL_ADDRESS>Ticket Priority: Low
Freshdesk Ticket Description:
Hi Support,
It seems that we hit a hard limitation of the google spreadsheet component.  We have a flow that requires writing more than 1k records to a google spreadsheet. However, it has errors in the execution log as follows" Quota exceeded for quota metric 'Read requests' and limit 'Read requests per minute per user' of service 'sheets.googleapis.com' for consumer 'project_number:988848966814'. The error occurs for the rest of 100+ records which caused no more records added to the spreadsheet.
Is this a platform limitation or Google API limitation? What's the best way to solve the large number of records written in a spreadsheet?
Best regards,
Carrie

Plan

Add a single config level option to the action Number of attempts per second that is optional and defaults to the existing environment variable. We should mention in the helptext for the field that Google in general limits the number of total API calls (across all flows for a credential) to 5 per second.

To be more precise Google limits API calls to 60 per minute per user per project.
5 calls per second is a rate limit that can be configured by the env vars REQUEST_TIMEOUT_PERIOD and REQUEST_TIMEOUT_QUOTA. Then we calculate the delay between calls as:
REQUEST_TIMEOUT_PERIOD / REQUEST_TIMEOUT_QUOTA.
This is the link to the actual Google limits: https://developers.google.com/sheets/api/limits#exponential


Live Playground Link in README.MD broken

Title is self explanatory. Clicking the link leads to a 404 Not Found message from github.

It should point to https://beautifulinteractions.github.io/beautiful-react-diagrams. It looks like the repository was moved.

It points me to the documentation but I still don't see the live playground though.
Perhaps it should point here instead?


[BUG] ignore.count method should not be in bytes

    template<typename T> EVE_FORCEINLINE std::ptrdiff_t offset(eve::as_<T> const&) const
    {
      return count_;
    }
    template<typename T> EVE_FORCEINLINE auto count(eve::as_<T> const&) const
    {
      return sizeof(T) - sizeof(element_type_t<T>) * count_;
    }

This count method makes no sense, it should return elements.
Also I would like offset_from_end or roffset or smth.

It used internally to count how many bytes to memcpy. It was organically factored out and I didn't paid attention.
It can be changed to returning # of element and we do the * sizeof(T) in the load[ignore] code as it's the only place we use this.
roffset seems OK to add too

It used internally to count how many bytes to memcpy. It was organically factored out and I didn't paid attention.
It can be changed to returning # of element and we do the * sizeof(T) in the load[ignore] code as it's the only place we use this.
roffset seems OK to add too


tensorflow.python.framework.errors_impl.NotFoundError: ./renderer/rasterize_triangles_kernel.so: undefined symbol: _ZN10tensorflow8internal21CheckOpMessageBuilder9NewStringEv

tensorflow.python.framework.errors_impl.NotFoundError: ./renderer/rasterize_triangles_kernel.so: undefined symbol: _ZN10tensorflow8internal21CheckOpMessageBuilder9NewStringEv

How can I solve this problem?

Seems that your tensorflow version is incompatible with the tf_mesh_renderer. Did you follow the instruction in the readme to install tf_mesh_renderer?

Seems that your tensorflow version is incompatible with the tf_mesh_renderer. Did you follow the instruction in the readme to install tf_mesh_renderer?

I followed the instructions, but I am not sure if I have installed tf_mesh_renderer, can you teach me how to install it correctly.

Seems that your tensorflow version is incompatible with the tf_mesh_renderer. Did you follow the instruction in the readme to install tf_mesh_renderer?

I followed the instructions, but I am not sure if I have installed tf_mesh_renderer, can you teach me how to install it correctl
Follow the below steps carefully

Getting Started
Testing Requirements
Reconstructions can be done on both Windows and Linux. However, we suggest running on Linux because the rendering process is only supported on Linux.
Python 3.6 (numpy, scipy, pillow, argparse).
Tensorflow 1.12.
Basel Face Model 2009 (BFM09).
Expression Basis (transferred from Facewarehouse by Guo et al.). The original BFM09 model does not handle expression variations so extra expression basis are needed.
tf mesh renderer (an older version). We use the library to render reconstruction images. Note that the rendering tool can only be used on Linux.

tensorflow.python.framework.errors_impl.NotFoundError: ./renderer/rasterize_triangles_kernel.so: undefined symbol: _ZN10tensorflow8internal21CheckOpMessageBuilder9NewStringEv

How can I solve this problem?

Did you solve it ?

I think the only problem is dependency on Tensorflow 1.12. Its a petty that someone is still using TensorFlow==1.12 when we have TensorFlow==2.25.


Expiration time is not send in the oauth token response.

According to the doc an aouth token response should looks like:
{"access_token":"ACCESS_TOKEN","token_type":"bearer","expires_in":2592000,"refresh_token":"REFRESH_TOKEN","scope":"read","info":{"username":"bob"}}

Currently the expires_in is not present.

Indeed, this is missing. Would be better if it is changed to expires_at as well.

I would leave it expires_in as that's what the spec states it should be https://tools.ietf.org/html/rfc6749#section-4.2.2

@lucasvanhalst Good catch!


Simscape udp

This is the half of the SimscapeUDP project that works in the codebase, the other half is in a fork of Simscape (most of the code is in the UDP folder).
I still have problems with the simulation, so in my fork I have removed gravity. This has enabled me to test what I have wrote.
I basically sending servotargets messages over UDP, then from the simulation receiving a darwinSensors/Servos message made from the servo stores.

I think this has been largely superseded by webots. Do you want to keep this PR open @CMurtagh-LGTM?

Without Matt, noone has knowledge of simscape, if someone wants to do matlab stuff they can reopen this and @ me.


How to run test suite against the installed project

I am working on packaging nvme-stas for Debian/Ubuntu (see https://bugs.debian.org/1032650). Debian/Ubuntu has autopkgtest for running test against the installed binary package. Can you add documentation how to run the test cases against the installed nvme-stas?

I'm not familiar with autopkgtest. The tests are part of the source repo and are embedded in meson.build files. Are you saying that autopkgtest allows one to run tests without having access to the source repo? How does that work? Can you point me to a project where this is used.

Some information about autopkgtest:

https://packaging.ubuntu.com/html/auto-pkg-test.html
https://salsa.debian.org/ci-team/autopkgtest/blob/master/doc/README.package-tests.rst

You specify which packages you need installed in your test bed and maybe what kind of restrictions you need in your test bed and what to run for the test. When running the test, the current working directory contains the source package content (which is the git export plus the debian directory).

Hi @bdrung - Hope this will answer your question.
Basic testing is done using meson. From the directory where the source package content is located, run the following commands.
BUILD_DIR=".build"
meson setup ${BUILD_DIR}
meson test -C ${BUILD_DIR}

The above commands have been added to the Makefile for convenience and can be run simply with:
make test

Coverage testing is also available, but it is recommended to run that in a VM as this may leave the host computer in a weird state if something were to go wrong. Additional packages are required to run Coverage testing (see: TESTING.md). The command to run coverage testing is:
make coverage


Pass route parameters to url_for

Slightly cleaner than manually building the query string.
@xwsxethan just as a note, this is against your branch, not master.  Tested fine locally.

I'm dumb; I figured out what I did yesterday that caused the error (cause I did the exact same thing today trying to repro). I had put the parameter on the outside of the flask.url_for() call. I'm just gonna say it was a long day yesterday (and apparently this morning already) haha. I've already recreated the change in my branch so you can close this PR if you like. Thanks for figuring it out!

Review status: 0 of 1 files reviewed at latest revision, all discussions resolved.

Comments from the review on Reviewable.io


value of "label_size?  thanks!

Dear author
thanks for sharing, when using the  function in the img ,I am confused,
If  I only have the 3 classes, 0:backgroud ,1: car ,2 :people ,
I need change the   value of "label_size"  to 3?

This value is set according to the label mapping of semanticKITTI; you can set it based on your dataset.
If you only have three categories, it should be 3;


about removeCol method

Description
when i use removeCol method to remove a col in my excel sheet,i have this scence that i need to remove "M","N","O" cols,so my code like this:
colIndex := []string{"M", "N", "O"}
    for _, index := range colIndex {
        if err = file.RemoveCol(sheetName, index); err != nil {
              // some logic
        }
    }

beyond my expectation that i didn't remove the three's cols which i want ,i think while the for loop fisrt run the RemoveCol method，the cols behind the "M" will go head to "M"  cols,so i had to change my code like this:
``
for i := 0; i < 3; i++ {
if err = file.RemoveCol(sheetName, "M"); err != nil {
return 0, err
}
}
<!--
Briefly describe the problem you are having in a few paragraphs.
-->

**Steps to reproduce the issue:**
1.prepare a excel file
2. use RemoveCol method to remove thre  cols you want
3.you will find that you have to use same col that you can remove  three continuous col

**Describe the results you received:**

**Describe the results you expected:** remove  three continuous col

**Output of `go version`:**
1.18
```text
(paste your output here)

Does this situation meet your expectations，if not i think it's a bug and i want to fix it.
Thanks for your view!!
Excelize version or commit ID:
v2.6.1
(paste here)

Environment details (OS, Microsoft Excel™ version, physical, etc.):

Thanks for your issue. This behavior is expected, just like with your delete column in the spreadsheet application, the cells in column N will be moving into column M after deleting column M. Not like InsertCols, this library doesn't support removing multiple columns at once. So you need to remove the column one by one. This issue needn't be fixed. I closed this, if you have any questions, please let me know, and we can reopen this anytime.

Okay，thank you for your question and answers.May i have another question: i set colWidth for cols first,then  i used RemoveCol method to remove some cols and i found that the colWidth i had set is not remove. Is it possible that while we use  RemoveCol method,the corresponding col with can also be deleted?

Which version of the Excelize library are you using? Could you show us a complete, standalone example program or reproducible demo?

Which version of the Excelize library are you using? Could you show us a complete, standalone example program or reproducible demo?

It seems that the issue doesn't exist in version 2.8.0. We are currently using version 2.6.3 in our company, so you must have already fixed this issue. Thank you again for your guidance！！！


[Feature] Link account, download liked songs and all playlist at a time.

What is the purpose of your issue?

[ ] Bug
[x] Feature Request
[ ] Question
[ ] Other

Description
It would be great to be able to link our Spotify account to spotdl and then be able to use --all-playlists argument to download all playlists from the linked account and --liked to download all liked songs of the account.
Log
Do I really need logs for a Feature Request ? 🤔

Yes, downloading liked songs at least, is very much required.

@ketankr9 For the moment, a workaround would be to copy tracks from liked songs to a new playlist and pass this new playlist for download.

@ritiek yes, I tried it once, but it is tiresome and irritating when you have a long list of liked songs.
Apart from being private, the liked song's html page has the same DOM content as that of a public playlist.  Maybe you can copy cookie from the browser(or get it pasted manually) so that the liked songs can be accessed.
Does it seem doable?

Kind of long and really hacky but just started working on this
You can go to Spotify's API documentation for endpoints for playlists and stuff and if you change the API endpoint in the CURL command below, it will work like a normal API and you can get song IDs, URLs, etc.
Heres what I did:
if you open up Dev tools and select Network tab, then load the Liked Songs playlist on the spotify website, you can see that Spotify will send a request to their API.  Find the url starting with api.spotify.com and select it.
(I will attach a picture of my capture)
Once selected, look at the Request Headers on the right and there will be an authentication header sent to their server and the value will start with "Bearer ".
If you copy that entire string and execute on your terminal:
curl -H "Authorization: {Bearer BQAQ40CBWh2j-_xg0fNnq.....iwv9QhvrSrKeFbo}" https://api.spotify.com/v1/me/tracks?limit=50
it will list the tracks in your Liked Songs playlist (obviously paste your entire Bearer string).  It forces a limit tho and I think 50 is the highest you can go.  So I copied that JSON response and saved it to a JavaScript variable (cuz Devtools is already open).
You can iterate through the variable like
trackuri = "https://open.spotify.com/track/";
firstlist = {paste JSON};
masterlist = [];
for(let i = 0; i < firstlst.items.length; i++ ) {
   masterlist.push(trackuri + firstlst.items[i].track.id);
}

Copy masterlist to a text file and you can download all the songs as normal with spotdl --list {file name}
If your Liked Songs playlist is bigger than 50 songs use that curl command again with an offset parameter like this
curl -H "Authorization: {Bearer BQAQ40CBWh2j-_xg0fNnq.....iwv9QhvrSrKeFbo}" https://api.spotify.com/v1/me/tracks?offset=50&limit=20
Just repeat this for the whole playlist.  It seems like a lot but trust me, way faster than saving each song to a new playlist.
With that Bearer token tho you can use their entire API for the most part. I have been messing around with it.
Here is the network dump to find API youll see the Bearer token circled:

And here is the curl command:

Ill see if I can come up with something easier after I mess with the API a little longer

way faster than saving each song to a new playlist.

Woah! That is indeed going to be painful! I tried this in the past on Spotify Desktop but it should still work (and possibly on Spotify Web too). Select multiple tracks in one playlist by using shift+leftclick (or all with ctrl+a) and then copy them (ctrl+c) to a new playlist (ctrl+v). That way you don't need to save each track to the new playlist manually.

Open a new issue and refrain see it under the new features issue.
If it gathers  traction, will implement it in v3.

is there a follow up issue for this?

is there a follow up issue for this?

To download liked songs run spotdl --saved more info spotdl --help

Thanks! --saved doesnt exist. Its actually spotdl --user-auth .....

Hello, i don't understand how to use the --user-auth saved function, can anyone explain?
Thanks

Hello, i don't understand how to use the --user-auth saved function, can anyone explain?
Thanks

Make sure that you are on the latest version v3.6.3. After running spotdl --user-auth saved web browser should open, after that you should login to your Spotify account

Hello, i don't understand how to use the --user-auth saved function, can anyone explain?
Thanks

Make sure that you are on the latest version v3.6.3. After running spotdl --user-auth saved web browser should open, after that you should login to your Spotify account

I'm using spotdl on a headless system through ssh... browser can't open in mycase, how to use this function headlessly?


[core] Improve error message when get_file_json / shell_safe_json_parse fails.

Closes #7740
Note:
We should discuss sometime whether or not the CLI should accept python dict string syntax. I am not sure how often this syntax is used by CLI users.
Moreover, it complicates the logic in shell_safe_json_parse. if a string uses python dict single quotes (invalid json) and also has a syntactical error common to both json and dict strings (extra comma), it is hard to know which error is more relevant to the user. As shell_safe_json_parse technically supports both syntaxes.

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

[ ] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

[ ] I adhere to the Command Guidelines.

Python dict syntax is permitted because Windows cmd.exe only allows use of double quotes, necessitating nonsense like this: "{\"foo\":\"bar\"}"
It allows "{'foo': 'bar'}" since the Linux equivalent '{"foo":"bar"}' is not possible.

Even if the CLI continues to accept python dict strings, we should at least add the ability to restrict string parsing to json, as some clients of shell_safe_json_parse might expect json strings and not python dict strings.  E.g group deployment create/validate

Python dict syntax is permitted because Windows cmd.exe only allows use of double quotes, necessitating nonsense like this: "{\"foo\":\"bar\"}"
It allows "{'foo': 'bar'}" since the Linux equivalent '{"foo":"bar"}' is not possible.

@tjprescott, that's a fair point. I guess I was a bit concerned because of things like the following:
>>> will_it_parse = "{1: (2,3), 2: {1,4}}"
>>> it_did
{1: (2, 3), 2: {1, 4}}
>>> will_it_parse = "{1: (2,3), 2: {1,4}}"
>>> it_did = shell_safe_json_parse(will_it_parse)
>>> it_did
{1: (2, 3), 2: {1, 4}}
>>> is_it_json = json.loads(will_it_parse)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  ...
json.decoder.JSONDecodeError: Expecting property name enclosed in double quotes: line 1 column 2 (char 1)
>>>

is this good to go


Python 3.11 error on Debian 12: This environment is externally managed

Mentioned in:
https://github.com/raspiblitz/raspiblitz/issues/4154#issuecomment-1716008904
and
https://github.com/raspiblitz/raspiblitz/pull/4153

× This environment is externally managed
╰─> To install Python packages system-wide, try apt install
    python3-xyz, where xyz is the package you are trying to
    install.

    If you wish to install a non-Debian-packaged Python package,
    create a virtual environment using python3 -m venv path/to/venv.
    Then use path/to/venv/bin/python and path/to/venv/bin/pip. Make
    sure you have python3-full installed.

    If you wish to install a non-Debian packaged Python application,
    it may be easiest to use pipx install xyz, which will manage a
    virtual environment for you. Make sure you have pipx installed.

    See /usr/share/doc/python3.11/README.venv for more information.```

Possible workarounds are discussed extensively in:
https://stackoverflow.com/questions/75608323/how-do-i-solve-error-externally-managed-environment-every-time-i-use-pip-3
and in: https://stackoverflow.com/questions/75602063/pip-install-r-requirements-txt-is-failing-this-environment-is-externally-mana
Either use in the pip config:
[global]
break-system-packages = true

or
sudo rm /usr/lib/python3.*/EXTERNALLY-MANAGED

to the same effect.

merged #4153 for v1.10.0 release which shjould address this issue

@rootzoll #4153 did not fix this, but since it is only relevant for Python 3.11 in Debian 12 it should not delay the v1.10 release. Tagged for v0.10.1.

Thank you  problem solved

Thank very much!

@openoms what is the status here on the v1.11.0 release?

o brigado o meu tambem resolvi com sudo rm /usr/lib/python3.*/EXTERNALLY-MANAGED

@openoms not sure if #4400 broke the python lnd scripts .. having problems with lnd after the. merge. Have to investigate further next week.

On a sd card build with the #4400 merged the setup with a fresh lnd failed:
When ssh in and trying ./config.scripts/lnd.initwallet.py I get:
Traceback (most recent call last):
  File "/home/admin/./config.scripts/lnd.initwallet.py", line 6, in <module>
    import grpc
ModuleNotFoundError: No module named 'grpc'

@openoms When checking ´pip list | grep grpc´ there is no grpcio 1.42.0 modules listed.

@rootzoll I think it would be safer to reintroduce the fix at this stage, there are too many python dependencies installed in various places.

@rootzoll I think it would be safer to reintroduce the fix at this stage, there are too many python dependencies installed in various places.

just checked the logs of the builld: https://pipelinesghubeus12.actions.githubusercontent.com/IYvj0uVSXPPHLO7ttdl6qtHJCXGXn6RaqGzAAHVqqz5E8aky5C/_apis/pipelines/1/runs/905/signedlogcontent/2?urlExpires=2024-02-11T18%3A14%3A01.7455631Z&urlSigningMethod=HMACV1&urlSignature=kSPaVnzdl60iRyxjuaYCs9opekNg6nDF3rEavwBjkII%3D
It has the same error:
2024-02-09T15:36:20.4345862Z     arm.raspiblitz-arm64-rpi: *** Python DEFAULT libs & dependencies ***
2024-02-09T15:36:20.4692062Z     arm.raspiblitz-arm64-rpi: update-alternatives: using /usr/bin/python3.11 to provide /usr/bin/python (python) in auto mode
2024-02-09T15:36:20.5177940Z     arm.raspiblitz-arm64-rpi: python calls python3.11
2024-02-09T15:36:20.5518559Z     arm.raspiblitz-arm64-rpi: update-alternatives: using /usr/bin/pip3 to provide /usr/bin/pip (pip) in auto mode
2024-02-09T15:36:20.5525947Z ==> arm.raspiblitz-arm64-rpi: update-alternatives: warning: not replacing /usr/bin/pip with a link
2024-02-09T15:36:20.6964487Z ==> arm.raspiblitz-arm64-rpi: sudo: unable to resolve host 3a42a9873732: Name or service not known
2024-02-09T15:36:22.5829092Z ==> arm.raspiblitz-arm64-rpi: error: externally-managed-environment
2024-02-09T15:36:22.5831925Z ==> arm.raspiblitz-arm64-rpi:
2024-02-09T15:36:22.5836675Z ==> arm.raspiblitz-arm64-rpi: × This environment is externally managed
2024-02-09T15:36:22.5842749Z ==> arm.raspiblitz-arm64-rpi: ╰─> To install Python packages system-wide, try apt install
2024-02-09T15:36:22.5848041Z ==> arm.raspiblitz-arm64-rpi:     python3-xyz, where xyz is the package you are trying to
2024-02-09T15:36:22.5855447Z ==> arm.raspiblitz-arm64-rpi:     install.
2024-02-09T15:36:22.5861418Z ==> arm.raspiblitz-arm64-rpi:
2024-02-09T15:36:22.5867587Z ==> arm.raspiblitz-arm64-rpi:     If you wish to install a non-Debian-packaged Python package,
2024-02-09T15:36:22.5870913Z ==> arm.raspiblitz-arm64-rpi:     create a virtual environment using python3 -m venv path/to/venv.
2024-02-09T15:36:22.5878172Z ==> arm.raspiblitz-arm64-rpi:     Then use path/to/venv/bin/python and path/to/venv/bin/pip. Make
2024-02-09T15:36:22.5883077Z ==> arm.raspiblitz-arm64-rpi:     sure you have python3-full installed.
2024-02-09T15:36:22.5887716Z ==> arm.raspiblitz-arm64-rpi:
2024-02-09T15:36:22.5892821Z ==> arm.raspiblitz-arm64-rpi:     For more information visit http://rptl.io/venv
2024-02-09T15:36:22.5898072Z ==> arm.raspiblitz-arm64-rpi:
2024-02-09T15:36:22.5910430Z ==> arm.raspiblitz-arm64-rpi: note: If you believe this is a mistake, please contact your Python installation or OS distribution provider. You can override this, at the risk of breaking your Python installation or OS, by passing --break-system-packages.
2024-02-09T15:36:22.5912767Z ==> arm.raspiblitz-arm64-rpi: hint: See PEP 668 for the detailed specification.

comment on the topic from cdecker re the CLN dependencies install getting the same error:

https://t.me/lightningd/31417

This issue prevents the new CL wallet generation on raspiblitz v0.11.1 as the mnemonic dependency is not installed.
##################
ERROR in _provision.setup.sh --> cl-wallet-new-noseeds on Sat 10 Aug 10:18:08 BST 2024\nDETAIL --> /home/admin/config.scripts/cl.hsmtool.sh new-force mainnet --> seedwords=''\nDEBUG --> /home/admin/raspiblitz.provision-setup.log
***********************************************************
ERROR - please report to development team
***********************************************************
state(error) message(cl-wallet-new-noseeds)

/home/admin/config.scripts/cl.hsmtool.sh new-force mainnet
# Running 'cl.hsmtool.sh new-force mainnet'
cl.log_old

[notice] A new release of pip is available: 24.0 -> 24.2
[notice] To update, run: python3 -m pip install --upgrade pip
error: externally-managed-environment

× This environment is externally managed
╰─> To install Python packages system-wide, try apt install
    python3-xyz, where xyz is the package you are trying to
    install.

    If you wish to install a non-Debian-packaged Python package,
    create a virtual environment using python3 -m venv path/to/venv.
    Then use path/to/venv/bin/python and path/to/venv/bin/pip. Make
    sure you have python3-full installed.

    For more information visit http://rptl.io/venv

note: If you believe this is a mistake, please contact your Python installation or OS distribution provider. You can override this, at the risk of breaking your Python installation or OS, by passing --break-system-packages.
hint: See PEP 668 for the detailed specification.

[notice] A new release of pip is available: 24.0 -> 24.2
[notice] To update, run: python3 -m pip install --upgrade pip
Traceback (most recent call last):
  File "/home/admin/config.scripts/blitz.mnemonic.py", line 4, in <module>
    from mnemonic import Mnemonic
ModuleNotFoundError: No module named 'mnemonic'
seedwords=''
seedwords6x4=''
# No seedwords - exiting

Merged #4692 to address issue - set to final testing

OK merged the PR and tested. Looking good now for CL install. Movin this issue to v1.11.3 to check again for other places.

I use email notifications to be informed of restarts, etc. I'm on v1.11.1 now and am getting this error when running the blitz.notify.sh on script. Is there something I can do on my Raspiblitz to fix it or can I help with this issue in some way, or is it now sorted and I should wait for v.1.11.3?

I use email notifications to be informed of restarts, etc. I'm on v1.11.1 now and am getting this error when running the blitz.notify.sh on script. Is there something I can do on my Raspiblitz to fix it or can I help with this issue in some way, or is it now sorted and I should wait for v.1.11.3?

this is is solved in 0.11.2.
You can just run:
sudo rm /usr/lib/python3.*/EXTERNALLY-MANAGED

as discussed above

Will need to run
pip config set global.break-system-packages true

in all the remaining places where pip install is being run without a virtual environment.


Restrict making of containers to non-infinite ranges

Model-checking that forbids construction of containers from infinite generators.

Urgh, this is going to take some getting used to.

@MetaLang What do you mean, getting used to?

I'm referring to the new Github review system. This is my first time using it and it's going to take some getting used to ;-)

Aha.

Two has approved...so why no merge?

Auto-merge toggled on

Thanks.

@MetaLang array() seems protected against infinite ranges. Any more functions that need to be qualified?

Not sure, I'm guessing we'll just have to cross those bridges as we come to them. The only other functions I can think of at the moment are writeln et al. They'll happily take an infinite range and print out its elements forever.

All checks have passed
5 successful checks

Seems like auto-tester has still now some troubles with automatically merging (it worked on a other PR), let's have an eye on this. I just deprecated one build which will hopefully trigger the merge soon.

It's possible another PR merged. Github doesn't have great sync with auto tester, it sometimes keeps old status.


reportlab Compatibility: Adapt to New ShowBoundaryValue Import

Short description
Fix compatibility with reportlab>=<IP_ADDRESS>.
Proposed changes

Use new import if possible.
Fallback to old import if required.

Resolved issues
Fixes: #751 (cleanly)

Are there plans to deploy this any time soon? Thanks for the great lib!


brackets match guide line is not rendered correctly

Issue Type: Bug
reproduce
edit a golang or C/CPP source file.
click on a bracket.
the bracket match line is not render between the matching brackets, but the upper level match brackets.

just as the above picture, the current matching brackets are lines 103 and 105, but brackets in lines 99 and 109 is rendered.
VS Code version: Code - Insiders 1.25.0-insider (7dc9539a421c9c33e57e2ce0b718f183bb44c756, 2018-06-15T05:17:41.405Z)
OS version: Windows_NT x64 6.1.7601

System Info

Item
Value

CPUs
Intel(R) Pentium(R) CPU B950 @ 2.10GHz (2 x 2095)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: disabled_offnative_gpu_memory_buffers: disabled_softwarerasterization: unavailable_softwarevideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
5.60GB (3.27GB free)

Process Argv
C:\Program Files\Microsoft VS Code Insiders\Code - Insiders.exe

Screen Reader
no

VM
0%

Extensions (3)

Extension
Author (truncated)
Version

rest-client
hum
0.18.4

vscode-language-pack-zh-hans
MS-
1.24.3

Go
ms-
0.6.82

I think this is as designed. The line is indent guide line not brackets match guide line.

Duplicate of #49342


Unable to find keymap

As I activated the extension this error showed up -
No Sublime settings file found at the default location: C:\Users\RoboRishab\AppData\Roaming\Sublime Text 3\Preferences.sublime-settings

Ok got it in the long run so thanks anyway


Added Live Console!

This update took me about an hour so I'll go through everything I did.

Removed the sleep from the start.sh
Added an option to turn on liveConsole in the config.js
Added the live console!
Had to add a couple timeouts and setintervals for it to work
Added a quick hotfix for the liveConsole when text would stay even after update refresh.
Changed account to ac so it just looks cleaner
Added some more color :+1:

Hope you guys like this update xD

Amazing work, MastaCoder! :+1:

I have a much better version of this without the need of added 100 lines XD... only 2-3 lines


WSL distros suddenly duplicates itself - Multi user Environment

Environment
Windows build number: 10.0.19042.0
Your Distribution version: Whole WSL infrastructure (not distro specific)
Whether the issue is on WSL 2 and/or WSL 1: Both
Last updates installed: KB4566782 on August 8th, 2020 and Feature update to Windows 10, version 20H2 on July 25th, 2020
Windows Insider Build channel: Beta

Steps to reproduce
Upon opening Windows Terminal,

Which translates to:
Errors occurred while loading user settings
Several profiles with the same GUID were found in your settings file - duplicates are ignored. Make sure that the GUID of each profile is unique.
using 'wsl -l -v' on powershell: (not as admin)

It listed 10 distros, 5 are duplicates.
using 'wsl -l -v' on powershell: (as admin)

5 distros are there, even tho no distros were installed before and i believe that WSL is still restricted to within each user space (not allowing cross-user access, like running WSL distro on 1 normal user account, while running docker on admin account). I don't even login to my Microsoft Store account on my admin account too.
Checking registry (HKCU\SOFTWARE\Microsoft\Windows\CurrentVersion\Lxss): (as normal user, not admin)

There are only 5 distros registered, nothing else.
My setup is as follows:
2 user accounts:

1 admin (local account, Microsoft store was not logged in)
1 normal user (using Microsoft account. Original distro (Ubuntu & Debian) was installed via store, exported to a .tar  and then installed/loaded via  'wsl --import' command (resulted in Ubuntu_Build, Ubuntu_Server, and KanboardServer).

Admin has no WSL installation; user has 5 distros. All WSL activities are performed under user, not admin.

WSL logs:
Expected behavior
Only 5 WSL distros listed when performing 'wsl -l -v'

Actual behavior
WSL listed duplicates of each distros, making it 10 distros, and confusing the Windows Terminal on the duplicated distros.

PS: Why I need 2 user account (admin and normal user)? Security. Also, I am trying to simulate some work environment, where access to admin account is strictly controlled or only allowed under a very strict exception (My last workplace needed me to register to 3 different managers (dept. head, country manager, and ASEAN/APAC manager) just to get permission to upgrade my account as admin), everything else (software install, updates, etc.) only performed via SCCM/Remote Deploy.

This belongs in the Windows Terminal repo. There must be an unforeseen issue in the way Windows Terminal detects installed distros that affects the multi-user environment.

@wsluser This bug absolutely does not belong on the Windows Terminal repository.
As the user noted, wsl -l literally prints the same distributions twice. How did Terminal cause that?

Any possible resolution to this? Or is it really unreproducible by anyone but me?

OK, update:
Somehow, 12 days later (without modifying anything, I guarantee you I did not modify anything):

HKCU\SOFTWARE\Microsoft\Windows\CurrentVersion\Lxss suddenly listed 10 distros, full of duplicates. Here are examples:

Duplicated WSL, with 2 different version of WSL

Duplicated WSL with broken path & incomplete

Duplicated WSL with misconfigured default UID

REPEAT: I did not modify anything since I put up this report 12 days ago! WSL manager did all of this in background (most probably, otherwise what else can do this?)
The only suggestion I can put now to WSL team is to look closely again how WSL distros are registered and managed using WSL.exe. Transitioning to file-based DB for WSL system (using JSON file or any non-binary, human-readable file like text file) will solve this issue, PLUS enabling multi-user WSL experience without having to install 2 sets of WSL or creating this unwanted and unknown duplicates because of multi-user environment confusion. The weakness in registry-based DB is that it is too restrictive since each user has their own environment!