bug

Bug report
What is the issue?
I am trying to download a playlist using youtube-dl, but I am getting an error that says "Unable to extract uploader id". I am using the latest version of youtube-dl and the command that I am using is:
youtube-dl --verbose -f best -o '%(playlist_index)s-%(title)s.%(ext)s' --yes-playlist --console-title --no-check-certificate 'https://www.youtube.com/playlist?list=PLknwEmKsW8OvMsFbU9zo8oJCprAsgc4LO'
What did you expect to happen?
I expected youtube-dl to download the playlist without any errors.
What actually happened?
youtube-dl is unable to extract uploader id and fails to download the videos in the playlist.
Steps to reproduce the issue

Install youtube-dl.
Run the following command:
youtube-dl --verbose -f best -o '%(playlist_index)s-%(title)s.%(ext)s' --yes-playlist --console-title --no-check-certificate 'https://www.youtube.com/playlist?list=PLknwEmKsW8OvMsFbU9zo8oJCprAsgc4LO'
Observe the error "Unable to extract uploader id".

Error message
ERROR: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug. Make sure you are using the latest version; see https://yt-dl.org/update on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Operating system and version
Please specify your operating system and its version (e.g. Windows 10, Ubuntu 20.04).
Additional information
Please provide any additional information that may be helpful in understanding the issue.

I think I got the issue, when I'm trying to download a video on youtube.
My Operationg system is: Linux 6.0.0-12parrot1-amd64 Debian 6.0.12-1parrot1 (2023-01-12) x86_64 GNU/Linux

Read the pinned issue #30839.

@whisperer256
From #30839:

Known issues, fixed in git master code:

YouTube: Unable to extract uploader id

Since you're a "Python lover", you should be familiar with installing the latest master code, in which this issue is fixed.
pip install 'git+https://github.com/ytdl-org/youtube-dl.git@master'
(note, that this will still say: youtube-dl version 2021.12.17)


CMEK support changes

What this PR does / why we need it:
Changes required to support CMEK Encrypted disks in GCP.
Which issue(s) this PR fixes:
Fixes #78
Special notes for your reviewer:
tests  in progress.
Release note:

Support for CMEK Encrypted Disks as described in: https://github.com/gardener/gardener-extension-provider-gcp/issues/564

[celebrate]     Kumar, Ashwani reacted to your message:

From: Tarun Ramakrishna Elankath @.>
Sent: Thursday, May 25, 2023 6:36:56 AM
To: gardener/machine-controller-manager-provider-gcp @.>
Cc: Kumar, Ashwani @.>; Review requested @.>
Subject: [gardener/machine-controller-manager-provider-gcp] CMEK support changes (PR #84)
What this PR does / why we need it:
Changes required to support CMEK Encrypted disks in GCP.
Which issue(s) this PR fixes:
Fixes #78https://github.com/gardener/machine-controller-manager-provider-gcp/issues/78
Special notes for your reviewer:
tests in progress.
Release note:
Support for CMEK Encrypted Disks as described in: https://github.com/gardener/gardener-extension-provider-gcp/issues/564

You can view, comment on, or merge this pull request online at:
https://github.com/gardener/machine-controller-manager-provider-gcp/pull/84
Commit Summary

984b9e7https://github.com/gardener/machine-controller-manager-provider-gcp/pull/84/commits/984b9e7d142d6ae0e0cd2f659b8ae4725b6d1755 CMEK support changes

File Changes
(5 fileshttps://github.com/gardener/machine-controller-manager-provider-gcp/pull/84/files)

M pkg/api/v1alpha1/provider_spec.gohttps://github.com/gardener/machine-controller-manager-provider-gcp/pull/84/files#diff-bd1b247473fbbde507d51db370b78fb10ed9413acde9f98c73e290eaf2bfee2a (9)
M pkg/gcp/machine_controller.gohttps://github.com/gardener/machine-controller-manager-provider-gcp/pull/84/files#diff-c529338f56b2f91e80e0fdd2962629f1f69b94d0ac9f7faef21c5791ce7a9b01 (17)
M pkg/gcp/machine_controller_test.gohttps://github.com/gardener/machine-controller-manager-provider-gcp/pull/84/files#diff-0219be6431bf8e562f7356c75d8dbeba1f0afcb18f5abfa2243cc4d2b406995f (150)
M pkg/gcp/machine_controller_util.gohttps://github.com/gardener/machine-controller-manager-provider-gcp/pull/84/files#diff-37cdf41d20d791270632d63f8e818d3215cb2ce332bf16c837f3f83d3e1b0f8f (14)
M pkg/gcp/validation/validation.gohttps://github.com/gardener/machine-controller-manager-provider-gcp/pull/84/files#diff-4317df83b48e3723adc9d01622dddff54d3074a2056e258b3ac9a07e41055fb4 (9)

Patch Links:

https://github.com/gardener/machine-controller-manager-provider-gcp/pull/84.patch
https://github.com/gardener/machine-controller-manager-provider-gcp/pull/84.diff

—
Reply to this email directly, view it on GitHubhttps://github.com/gardener/machine-controller-manager-provider-gcp/pull/84, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAF5XNR2XKSK7C7QB2FULNTXH34YRANCNFSM6AAAAAAYOLYVCY.
You are receiving this because your review was requested.Message ID: @.***>

HOW TO MANUALLY TEST FEATURE

Setup a GCP shoot cluster using a service account
Setup Cloud KMS Key and Assign IAM role:

Install gcloud CLI: brew install gcloud
Login into gcloud: gcloud auth login
Create a Cloud KMS keyring: gcloud kms keyrings create cmektest --location=us-east4. (This requires IAM privileges to be assigned to you)
Create a Cloud KMS Key for encryption purpose: gcloud kms keys create alpha --keyring=cmektest --location=us-east4 --purpose encryption
List to get fully qualified path of Cloud KMS Key:

gcloud kms keys list --keyring=cmektest --location=us-east4
NAME                                                                                PURPOSE
projects/projectId/locations/us-east4/keyRings/cmektest/cryptoKeys/alpha    ENCRYPT_DECRYPT

Give the service account used to create the shoot cluster relevant IAM role to use this key

gcloud projects add-iam-policy-binding projectId --member<EMAIL_ADDRESS>--role roles/cloudkms.cryptoKeyEncrypterDecrypter

Checkout the machine-controller-manager and machine-controller-manager-provider-gcp in peer directories under $GOPATH/src/github.com/gardener.
Setup and Start Local MCM:

cd $GOPATH/src/github.com/gardener/machine-controller-manager
make download-kubeconfigs
make start 2>&1 | tee /tmp/mcm.log

Start Local MC

cd $GOPATH/src/github.com/gardener/machine-controller-manager-provider-gcp
make start 2>&1 | tee /tmp/mcm-gcp.log
Wait for any pending reconciles to finish.

Edit the MachineClass yaml

k edit mcc
Modify providerSpec.disks as follows: adding kmsKeyName and kmsKeyServiceAccount as illustrated below (example below makes boot disk CMEK encrypted):

     - autoDelete: true
        boot: true
        kmsKeyName: projects/projectId/locations/us-east4/keyRings/cmektest/cryptoKeys/alpha
        kmsKeyServiceAccount<EMAIL_ADDRESS>

Trigger restart of Machine

List the existing Running machine:

k get machine
shoot--userid--shootName-worker-alpha-z1-9485d-xprld

Add the force-deletion label to the machine. (So restart is faster)   k label mc shoot--userid--shootName-worker-alpha-z1-9485d-xprld force-deletion=true

Delete the machine: k delete mc shoot--userid--shootName-worker-alpha-z1-9485d-xprld
Wait for reconcile to finish and confirm new machine is Running

k get machine
NAME                                                        STATUS    AGE
shoot--userid--shootName-worker-alpha-z1-9485d-96fkl   Running   67m

Confirm Disk Creation/Encryption.

Test to see that corresponding boot disk is created via gcloud CLI. (The boot disk name is same as machine name)   gcloud compute disks list | grep 96fkl
   shoot--userid---shootName-worker-alpha-z1-9485d-96fkl  us-east4-c      zone            50       pd-balanced  READY

Confirm that disk is using CMEK encryption key:

    gcloud compute disks describe shoot--shootName-worker-alpha-z1-9485d-96fkl --zone us-east4-c worker-alpha-z1-9485d-96fkl --zone us-east4-c
    creationTimestamp: '2023-05-25T00:46:41.136-07:00'
    diskEncryptionKey:
        kmsKeyName: projects/projectId/locations/us-east4/keyRings/cmektest/cryptoKeys/alpha/cryptoKeyVersions/2
        kmsKeyServiceAccount<EMAIL_ADDRESS>     id: '843731016874110129'

Review comments from meeting:

Support empty kmsKeyServiceAccount  that defaults to compute engine service agent account. (Google's doc is WRONG here)
Move keys to separate GCPDiskEncryption struct.

A suggestion, shall we move the How to test manually from the https://github.com/gardener/machine-controller-manager-provider-gcp/pull/84#issuecomment-1562553340 to the docs instead as I believe this will be handy later as well.  wdyt?
/hold because of https://github.com/gardener/machine-controller-manager-provider-gcp/pull/84#issuecomment-1562859544

Yes, will be moving this inside the docs and updating with tests, refactoring.


Save Dialog not saving file

VSCode Version: Code 1.10.1 (653f8733dd5a5c43d66d7168b4701f94d72b62e5, 2017-03-02T00:33:15.706Z)
OS Version: Windows_NT ia32 10.0.14393
Extensions:

Extension
Author
Version

mssql
ms-mssql
0.3.0

csharp
ms-vscode
1.7.0

vs-keybindings
ms-vscode
0.1.3

After creating a new SQL file with the new New Query feature in the MSSQL Extension, closing the tab pops up the do you want to save message box, choosing save just closes the dialog and the query but does not save the file.
Steps to Reproduce:

ctrl shift P then choose SQL New Query.
Create Query
Close the tab before manually saving will open the Do you want to save message.
click Save and the dialog closes and the query closes, but is not saved.

@SSDiver2112 can you reproduce the same by opening a new untitled file (Ctrl+N), making some changes and closing the tab?

Ctrl+N works correctly. Clicking Save Opens the SaveAs Dialog and allows the file to be saved. It only does it when I use the New Query option.

Ok I see what is happening, closing as duplicate of https://github.com/Microsoft/vscode-mssql/issues/762


Sending screenshot from clipboard via DiscordMessenger is messed up in the official client

Description
Self-explanatory
Steps to reproduce:

Take a screenshot with Alt+PrtSc
Paste it in the channel you want to send
Send it

Version
1.00-MinGW
Screenshots

Interesting issue.  It occurs to me that some pixels have their alpha value replaced with zero?

It occurs to me that some pixels have their alpha value replaced with zero?

in fact it is
if u process this broken screenshot through the ffmpeg you'll get a normal one
ffmpeg -i unknown.png xp.jpg

ffmpeg just cuts off transparency value to automatically make it 255 and then converts it to jpeg, not as photoshop does this
btw this funny ffmpeg behaviour can reveal some hidden information by alpha channel. photoshop does not delete color info even if transparency is 100%, gimp does

Indeed I recognized this issue because the screenshots were showing fine in Discord Messenger - the image loaders ignore the alpha channel.

Can't reproduce the issue. What Windows version are you using? What is your current bit depth? I use Windows 11 22H2, tried with both alt-prsc & ctrl-prsc but the images come out fine.

I found the issue without having to reproduce it.  This line https://github.com/DiscordMessenger/dm/blob/master/src/windows/ImageLoader.cpp#L189 is supposed say 255 << 24 rather than 255 << 16.

Ok, as it turns out it's quite difficult to actually make sure alpha information is accurate coming from windows... that means for now I'll just force alpha to 255. Not a good solution, though. I encourage someone to come in and take a look at this.


Add user definition to HOMEBREW_CASK_OPTS

This is a feature request. I recently tried to automate some developer setups and wanted to use cask, but couldn''t get around it needing to be run as the end user. I can easily discover and provide that user, but my scripts run as root. It would be immensely useful if a user could be defined in the HOMEBREW_CASK_OPTS variable and then set permissions that way so that it can be provisioned without user interaction.

You probably want to file this at the Caskroom for now. The two projects are working on moving closer together but it's a complicated process and we remain two separate entities sharing a common ecosystem, with differing development teams and processes.
That's where the HOMEBREW_CASK_OPTS environment scheme is defined, rather than by us, at this point in time.

Ok thanks, I will.


The plugin is trying to load ai files

For some reason since including the plugin again, I'm getting an error where it's trying to load .ai files from the directory. Is this intended effect?

Hey @theallseingeye, no, this plugin shouldn't do anything with ai files, it doesn't even know how to handle them. Can you show me the exact error message you are seeing? Thanks!


add footer draft

This is to get things started. People can make suggestions in the PR review or I can update the code and push changes.

Looks like this on my machine:

I'm going to redo this PR.....


[community] bacalhau version question

why display error?

@likhita-8091 Thank you for posting. It looks like you are using the Windows deployment; I just wanted to confirm which version of Bacalhau you are using?

@likhita-8091 Thank you for posting. It looks like you are using the Windows deployment; I just wanted to confirm which version of Bacalhau you are using?

yes,i clone source code, run the windows env。the bacalhau version is v1.3.2

@likhita-8091 are you sure that's<EMAIL_ADDRESS>I've noticed that error when trying to run 1.4.0, can you post the output from bacalhau version?


Add flexibility for working with different FHIR versions (at least DSTU3 and R4).

The OpenMRS FHIR module currently supports both DSTU3 and R4 versions of FHIR. We originally used R4 but because of Bunsen limitations we switched to DSTU3 (see #31). But R4 support is important for some other use cases so we should be able to support that too.

On openmrs, DSTU3 is available at ws/fhir2/R3 and R4 is available at ws/fhir2/R4. (see here)
We can just allow the user to choose between using R3 or R4, with the assumption being that the sink will support the chosen FHIR version.
Since the parquet approach requires R3 for now, we should probably use that version as the default.
with R3 being the default for now. ,

Closed by #68


How can I install latest version for angular 4.4.4 version through Angular cli ?

Bug Report or Feature Request (mark with an x)
- [ ] bug report -> please search issues before submitting
- [ ] feature request

Versions.

Repro steps.

The log given by the failure.

Desired functionality.

Mention any other details that might be useful.

e.g.:
npm install @angular/core@latest

Closing as answered by @Gamester2097 👍


Galactic: the “Interact” tool does not work for “ur_manipulator” when following the moveit example

The “Interact” tool  seems not working when trying to set the goal for the “ur_manipulator”.  The log which seems relevant to that is:
[rviz2-3] [INFO] [1645710748.640870337] [moveit_ros_robot_interaction.robot_interaction]: No active joints or end effectors found for group 'ur_manipulator'. Make sure that kinematics.yaml is loaded in this node's namespace. [rviz2-3] [INFO] [1645710748.646816651] [moveit_ros_visualization.motion_planning_frame]: group ur_manipulator [rviz2-3] [INFO] [1645710748.646855345] [moveit_ros_visualization.motion_planning_frame]: Constructing new MoveGroup connection for group 'ur_manipulator' in namespace ''

I'm using en_US.UTF-8 - since I had  problems with MoveIt tutorial #1049
So this time it must be a different story.

Thanks for pointing that out. There was  a regression in #278 which should be fixed by #308.


Align Stax & Flex max elements per content type (NBGL)

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 60.20%. Comparing base (bf24ae5) to head (5f14663).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #729   +/-   ##
=======================================
  Coverage   60.20%   60.20%
=======================================
  Files          12       12
  Lines        1729     1729
=======================================
  Hits         1041     1041
  Misses        688      688

Flag
Coverage Δ

unittests
60.20% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Useless, solved by:
https://github.com/LedgerHQ/ledger-secure-sdk/pull/730


[core-520] Capture docker image SHA for datums

 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.


Merged the parser-js and parser files.

Thanks the comments. Fixed.

Updated.

LGTM

Landed (b0bdf0ebf4ee89181a55e1c8dbfb49d5b314a633)


Feature Request - Ignore Specific Apps

Thanks for all your hard work on this plugin so far!
Now you've got the sensor in based on playback, I'd love to see a way to exclude apps from the sensor such as Youtube so that when using for music it doesn't affect playback scenes for instance or Plex which has it's own Webhooks plugin for better control over scenes for TV/Movies etc.

Thanks for all your hard work on this plugin so far!
Now you've got the sensor in based on playback, I'd love to see a way to exclude apps from the sensor such as Youtube so that when using for music it doesn't affect playback scenes for instance or Plex which has it's own Webhooks plugin for better control over scenes for TV/Movies etc.

That's good idea. Will implement it in the future release.

I see this is implemented, awesome stuff!


How do I install Windows Terminal on a Windows Server 2019 system?

I don't have access to the Store app. Must I build or can I download an installer to try out Windows Terminal?

+1
Need the appx package so we can use this on Windows Server.

Yup, in some companies, Windows Store is simply disabled, so a download alternative would be excellent...

I'm going to make #1386 the master issue for alternate distribution mechanisms. Thanks!

https://github.com/microsoft/terminal/issues/1386
https://github.com/microsoft/terminal/issues/1405
https://github.com/microsoft/terminal/issues/1480
https://github.com/microsoft/terminal/issues/1483
https://github.com/microsoft/terminal/issues/1541

I have not been able to install Windows Terminal on a Server 2019 machine!

That error message indicates that your build of Windows is too old.

@DHowett Well, can the install requirement be reduced to 1809 instead? Or does it rely on APIs that aren't present in that version?

It does rely on APIs that are only present in 1903 and above.

If we did not have a technical limitation, we would not impose such an otherwise arbitrary restriction.

I just installed the latest Windows Server 2019, the September 2020 release, OS Version 10.0.17763.1457. Apparently that's not sufficient for Windows Terminal which requires 10.0.18362.0 or greater. Where does one get a copy of Windows Server 2019 that meets the requirements imposed by the current Windows Terminal?

@David-Noble-at-work I don't believe there's a Windows Server release at/above version 10.0.18362 yet, sorry 😞

@David-Noble-at-work @zadjii-msft That's not completely true. You are correct that there is not yet a full release (2019, 2016 etc) past that version however Windows Server is also available under the "Semi-Annual" release channel which follows the cadence of Windows 10 releases. You need to have volume licensing and active Software Assurance to use that channel however, and given that Microsoft has had some major QA issues around the Windows 10 feature releases (and Windows Server shares the same codebase) I wouldn't recommend using that for any workload that is something you care about, especially if your only reason for using those releases is for Windows Terminal.

Why not have a simple installer that can be used to install it on windows server 2019 ? Does it require any dependencies that would complicate the installation in windows server 2019 ? I am reasonably sure that much of the IT Admin community would appreciate such an availability. Forget 2019, i am not able to install on 2016 as well, If that helps.

https://github.com/microsoft/terminal/issues/1483#issuecomment-691566254
https://github.com/microsoft/terminal/issues/1483#issuecomment-691566330
Please consider reading a thread before you comment on it.


Fixed #1094 ipv6 cluster failure on internal import

Issue #, if available: #1094
Description of changes: Updated internal import, prepared for patch release
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/do-e2e-tests


JIT enabled doesn't train

My models won't train when compiled by jit, even after registering the nodes as the PyTrees. Also, multi-layer models don't work due to input errors in the abstraction. Please fix this by implementing native module classes and not requiring input shapes.

We have now updated TensorWrap to version <IP_ADDRESS>, which adds jit to class inheritance, fixing this issue and improving performance. Additionally, multi layer models are now working and we will be adding more compiled layer detection, in order to make functionallity fast and quicker.


Add PyDict mode

Hello @kisielk.
Please find attached patches that similarly to StrictUnicode mode
(https://github.com/kisielk/og-rek/pull/68) add new PyDict mode that requests
to decode Python dicts as ogórek.Dict instead of builtin map.
This is needed to fix decoding issues that can be there due to different
behaviour of Python dict and builtin Go map:
Ogórek currently represents unpickled dict via map[any]any, which is
logical, but also exhibits issues because builtin Go map behaviour is
different from Python's dict behaviour. For example:

Python's dict allows tuples to be used in keys, while Go map
does not (https://github.com/kisielk/og-rek/issues/50),

Python's dict allows both long and int to be used interchangeable as
keys, while Go map does not handle *big.Int as key with the same
semantic (https://github.com/kisielk/og-rek/issues/55)

Python's dict allows to use numbers interchangeable in keys - all int
and float, but on Go side int(1) and float64(1.0) are considered by
builtin map as different keys.

In Python world bytestring (str from py2) is considered to be related
to both unicode (str on py3) and bytes, but builtin map considers all
string, Bytes and ByteString as different keys.

etc...

All in all there are many differences in behaviour in builtin Python
dict and Go map that result in generally different semantics when
decoding pickled data. Those differences can be fixed only if we add
custom dict implementation that mirrors what Python does.
-> Do that: add custom Dict that implements key -> value mapping with
mirroring Python behaviour and use that Dict instead of map when decoding
if PyDict mode is activated.
The behaviour of new system is explained by the following doc.go
excerpt:
For dicts there are two modes. In the first, default, mode Python dicts are
decoded into standard Go map. This mode tries to use builtin Go type, but
cannot mirror py behaviour fully because e.g. int(1), big.Int(1) and
float64(1.0) are all treated as different keys by Go, while Python treats
them as being equal. It also does not support decoding dicts with tuple
used in keys:

     dict    ↔  map[any]any                       PyDict=n mode, default
             ←  ogórek.Dict

With PyDict=y mode, however, Python dicts are decoded as ogórek.Dict which
mirrors behaviour of Python dict with respect to keys equality, and with
respect to which types are allowed to be used as keys.

     dict    ↔  ogórek.Dict                       PyDict=y mode
             ←  map[any]any

For the implementation we drop support for Go < 1.18 because we need to use
generic Map[Key, Value] from github.com/aristanetworks/gomap. If dropping
support for Go < 1.18 is an issue there is also a way to avoid using generics
via vendoring some code from said github.com/aristanetworks/gomap. However
personally I think that dropping support for Go < 1.18 should be ok because
that Go 1.18 was released in spring of 2022 - almost 2.5 years ago - and is
already EOL now. The older releases became EOL even earlier compared to Go 1.18
and are generally no longer supported by Go upstream. Dropping support for Go <
1.18 remains support for Go 1.18 - 1.23, i.e. 6 releases 3 of which are already
in end of life state. That's why I think that dropping support for Go < 1.18
should be, hopefully, ok.
ZODB/go and Wendelin.core intend to switch to using PyDict mode
while leaving ogórek to remain 100% backward-compatible in its default
mode for other users.
Thanks beforehand,
Kirill
/cc @perrinjerome, @vnmabus, @levinzimmermannn

Rebased and adjusted last patch after Xloosy fix in https://github.com/kisielk/og-rek/commit/85183af0851693d059e457f7fd40070afdf51c3d.
Interdiff:
--- a/ogorek_test.go
+++ b/ogorek_test.go
@@ -164,9 +164,9 @@ func Xloosy(name string, objectIn, objectOut interface{}, picklev ...TestPickle)
        return x
 }

-// Xloosy_uauto is like Xuauto but for Xloosy.
-func Xloosy_uauto(name string, objectIn, objectOut interface{}, picklev ...TestPickle) TestEntry {
-       x := Xuauto(name, objectIn, picklev...)
+// Xloosy_uauto_dgo is like Xuauto_dgo but for Xloosy.
+func Xloosy_uauto_dgo(name string, objectIn, objectOut interface{}, picklev ...TestPickle) TestEntry {
+       x := Xuauto_dgo(name, objectIn, picklev...)
        x.objectOut = objectOut
        return x
 }
@@ -569,7 +569,7 @@ var tests = []TestEntry{

        // loosy encode: decoding back gives another object.
        // the only case where ogórek encoding is loosy is for Go struct types.
-       Xloosy_uauto("[]ogórek.foo{\"Qux\", 4}", []foo{{"Qux", 4}},
+       Xloosy_uauto_dgo("[]ogórek.foo{\"Qux\", 4}", []foo{{"Qux", 4}},
                []interface{}{map[interface{}]interface{}{"Foo": "Qux", "Bar": int64(4)}},

                // MARK + STRING + INT + DICT + LIST

Kamil, thanks a lot for merging the patches and for kind words. I appreciate it.


Question generator

Add a page prompting the user with questions related to the current courses.

find current course code (e.g. TDT4120)
use Wikipendium to look up codes and scan the document page
as well as NTNU's official website (if needed)
pick a random subject based on the PDF's articles, and look up information somewhere else to provide information/generate questions (maybe hijack stackoverflow)


Create bootstrap cluster task for controller via CLI

Issue #, if available:
Description of changes:
Create bootstrap cluster task for controller via CLI
Testing (if applicable):

unit tests

Documentation added/planned (if applicable):
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve


How to logout user on window close?

Hello.
How to force these codes to logout the user when window closed???

You can't.
It's using a session, which means that the clients needs to be closed completely. It's not enough to close the active tab/window.


Components aren't unmounted when Turbolinks cache is disabled

This issue is related to #610. Listening to turbolinks:before-cache event results in component not being unmounted when the cache is disabled. I propose using turbolinks:before-visit event in addition to it, which would cover all the bases. (However, if both events are fired off consecutively, this approach might cause problems. It's worth investigating further.)

@volkanunsal Please keep researching this. Eventually, we might want to create a customized offshoot of turbolinks that can work with react-router and code-splitting by routes.

https://github.com/shakacode/react_on_rails/pull/425 fixes for me the 'flickering' of component which is rendered on every page when navigating between pages.
So changing turbolinks:before-render to turbolinks:before-cache fixes one problem but has a regression on the other.

@szyablitsky @volkanunsal The community will greatly appreciate any research into this matter, and a PR!

@justin808 I've been meaning to come back to this. I'm going to look into it tomorrow!

I discovered that turbolinks:before-visit is called even earlier than turbolinks:before-cache, and it's never omitted like turbolinks:before-cache is, even when cache is disabled. So It can easily be used to cover all cases. I'm going to submit a PR replacing turbolinks:before-cache with turbolinks:before-visit, unless there are concerns about backward compatibility.

may be we should make this configurable?
in our project we use turbolinks:before-render to minimize the absence time for the component which is rendered on all pages of the application

@szyablitsky What version of react are you using? In 15.2.x I get the error discussed in #610

Right now we're using<EMAIL_ADDRESS>&<EMAIL_ADDRESS>I've seen this error before, and did not know this can be related to turbolinks. I do not see this error in our application anymore.
Ha-ha. As it turns out<EMAIL_ADDRESS>already have merged using turbolinks:before-cache, despite changelog, which says it was merged only in 6.2.1. That's why we do not have "The node you're attempting to unmount was rendered by another copy of React" problem. But we have 'flickering' components problem again.

Disable Turbolinks cache with a meta tag. That should take care of it.
On Sun, Dec 11, 2016 at 11:05 PM Sergey Zyablitsky<EMAIL_ADDRESS>wrote:

Right now we're using<EMAIL_ADDRESS>&<EMAIL_ADDRESS>I've seen this error before, and did not know this can be related to
turbolinks. I do not see this error in our application anymore.
Ha-ha. As it turns out<EMAIL_ADDRESS>already have merged using
turbolinks:before-cache, despite changelog, which says it was merged only
in 6.2.1. That's why we do not have "The node you're attempting to unmount
was rendered by another copy of React" problem. But we have 'flickering'
components problem again.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/shakacode/react_on_rails/issues/629#issuecomment-266338832,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAJQMFkGPnvjSEyMwNuQUOMmqJ6nY4IPks5rHMfxgaJpZM4LAPSY
.

@szyablitsky I mean disable the cache after the latest PR lands. You shouldn't disable it when still using turbolinks:before-cache event hook.

@volkanunsal disabling turbolinks cache doesn't solve "flickering" components problem.
React-rails gem has the same issue https://github.com/reactjs/react-rails/issues/607 and they also don't know how to handle it.

@szyablitsky A potential solution is to use fake content at the mount point. It would look something like the component, and have the same dimensions.

@volkanunsal thank you for the idea. This can be used as a partial solution.

I think the best solution is to modify ReactDOM's unmountComponentAtNode and more specifically unmountComponentFromNode so it can bypass emptying of the container https://github.com/facebook/react/blob/8791325db03725ef4801fc58b35a3bb4486a8904/src/renderers/dom/stack/client/ReactMount.js#L190
May be additional boolean parameter like doNotEmptyContainer. And use this parameter in our Turbolinks handlers.


getDataBy...() methds return None if timeseries has no variable called value

Hi Anders,
All three methods getDataById(), getDataByName() and getDataByDescription() return None in case the timeseries that have been queried for do not contain a variable called value.
This is can be seen in code at lines 610, 711 and 810 in TSIClient.py. The payload contains a hardcoded variable name that will be looked for in the TSI environment: "value": {"tsx": "$event.value"}. I think this works fine most of the time, as by default (with the default type I belief, I am a bit rusty with the TSI jargon now) each timeseries has only one variable with the name value.
But the following situation is a problem:

MyPythonDevice has three variables and the current code returns None for all of them. To return e.g. the temperature, one has to use "value": {"tsx": "$event.temperature"} in the payload.
I try to address this in my next PR, at the moment I am not sure whether there is a way to retrieve the variable names (I just quickly checked the JSON for the time series instance and the type, and none of these contained the variable names).
Best regards,
Rafael

I figured out that this endpoint: https://{environmentFqdn}/eventSchema?api-version=2018-11-01-preview can be used to get an eventSchema  (docs here: https://docs.microsoft.com/en-us/rest/api/time-series-insights/dataaccess(preview)/query/geteventschema). This will return the variable names, but just as a list of properties. It also not only contains the variables, but some other stuff as well, and it does not allow to link timeseries ids to variables. This is a body of a sample response:
{
  "properties": [
    {
      "name": "messageId",
      "type": "Double"
    },
    {
      "name": "deviceId",
      "type": "String"
    },
    {
      "name": "temperature",
      "type": "Double"
    },
    {
      "name": "humidity",
      "type": "Double"
    },
    {
      "name": "iothub-connection-device-id",
      "type": "String"
    },
    {
      "name": "value",
      "type": "Double"
    }
  ]
}

temperature, humidity and value are variables in this case. Not yet sure what to do with this ...

@SiriOvregard your PR #24 solves this right? This issue could be closed in that case.

@rafaelschlatter it solves the problem if and when the type JSON build up is something like
{          "id":"59c7d57d-3440-4858-9013-d9ac78ffe5fd",          "name":"Double",          "variables":{             "Value":{                "kind":"numeric",                "value":{                   "tsx":"$event.[value].Double"                },                "aggregation":{                   "tsx":"avg($value)"                }             }          }       },
meaning if the "build up" is not Type > variables > Value > value > tsx, it will not fix the problem
`types={}
jsonResponse = self.getTypes()
for typeElement in jsonResponse['types']:
try:
typeElement['variables']['Value']['value']['tsx']
types[typeElement['id']] = typeElement['variables']['Value']['value']['tsx']
except:
logging.error('"Value" for type id {type} cannot be extracted'.format(type = typeElement['id']))
pass
return types`

As you see, at the moment an error is logged in the case of the type JSON build up being different and thats it.
In our application of the client it works, but i believe the type JSON build up can vary, so I would recommend to test before closing the issue.
(In any case, I believe this represents a bit of a weakness and should probably be amended at some point...)

Lets keep it open

Hi there - first of all thanks for the library - it as facilitated the entry to the TSI API.
I run into this same error message. After digging a bit into the code I came to the conclusion, that the current architecture allows only for one variable, called Value within a TSI-Type. In my case my IoT device that dumps the data onto TSI has currently 5 variables configured (and more variables are in the data structure and currently unused). I am not able to create multiple Types on TSI for the same Time Series. Therefore my conclusion is, that I will not be able to query for the additional data - is that correct?
Here a screenshot of the type in question for better understanding:


Actually wait for redis (fix logical typo)

So I guess because of this issue waiting for redis is not correct when using a celery executor when configuring it using AIRFLOW__CELERY__BROKER_URL environment variable.

Introduced by #495

I am not using this Docker airflow image anymore, but it seems that the issue is still there

RIP this repo


[acix-24] Notify job status as infra failures using job_failure

What does this PR do?
Adds checks to set job status as infra failure with job_failure attribute of gitlab jobs.
Related to https://datadoghq.atlassian.net/browse/ACIX-24 but job_failure is already used in some checks.

Motivation

Additional Notes

To be merged after https://github.com/DataDog/datadog-agent/pull/24070 and https://github.com/DataDog/datadog-agent/pull/24666

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge

/merge


Sumeragi class cannot use ThreadPool class from external directory.

Hello all,
I build Iroha core source by using  CMakeLists.txt file to create Iroha application. Make command line show an error by using ThreadPool class because it's cannot found #include<thread_pool.hpp> .  I found ThreadPool class is file from external library using name iroha/external/src/warchant_thread_pool.  That's error from iroha/core/consensus/sumeragi.cpp file.  After found root cause from sumeragi.cpp,
I add include directory support ThreadPool  class  to CMakeLists.txt on /iroha/core/consensus/.
include_libraries(../warchant_thread_pool/include/thread_pool/)
Iroha source code compiled again by use make verbose=1 command line. Core source code return an error as
/roha/core/consensus/sumeragi.cpp:68:5: error: no matching function for call to ‘tp::ThreadPoolOptions::ThreadPoolOptions(<brace-enclosed initializer list>)’
     });

I not sure what problem from /iroha/external/src/warchant_thread_pool is not updated version.

This is being refactored for v0.95. This should be somewhat usable next week in the develop branch.

I waiting the new source codes upload to upstream.

Please, check new code base in develop branch. Thank you for your interest to the project!


add pkg/builder with BackupBuilder and ObjectMeta functional options

Signed-off-by: Steve Kriss<EMAIL_ADDRESS>@nrb @prydonius this is what I'm leaning towards based on our discussions yesterday:

consolidated pkg/builder for all of these
builder structs are named e.g. BackupBuilder
constructors are called e.g. ForBackup (so usage outside the package is builder.ForBackup(...)
constructors take namespace (if namespaced) + name, since this is almost always used
builders have an ObjectMeta(...) method that takes metav1.Object functional options
we'd have builders for both Velero and core Kubernetes types here

@nrb @carlisia @prydonius please let me know what you think about this approach for test object construction. I'd like to agree on something so I can make a bunch of updates!

would it make sense to move the example (which it's great having that, btw!) to an actual godoc testable example?

Yeah, works for me!


Data Versioning Support

In addition to the versioning of metadata also a versioning of data might be desirable in the future. Therefor, existing approaches should be evaluated and implemented if possible. Furthermore, in the course of evaluating data versioning approaches, approaches on how to reduce the amount of stored data, e.g. by storing differential information, should be evaluated.

Released in version 1.2.0


is_overworld tag suggestion

Blasted barrens biome is not registered in minecraft:is_overworld. (Sorry if this is not registered to prevent strange interactions with other mods)
Also, forge:no_default_monsters, forge:is_rare, and forge:is_wasteland tags might be appropriate.

uploading a fix now, thanks for the headsup!


fix(ibmcloud): Initialize image registry config on creates and bad config

What this PR does / why we need it:
This updates HCCO to initialize image registry config for IBM Cloud platform. With the move of cluster-image-registry-operator to the control plane, a "bad" config is being initialized by the operator (bug: https://issues.redhat.com/browse/OCPBUGS-6797) causing the operator pods to crashloop.
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes #
Checklist

[ ] Subject and description added to both, commit and PR.
[ ] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

/lgtm
/approve

/cherry-pick release-4.12


Update babel instructions to babel 7

I think you need to have @babel/core before the other packages in these instructions, per the @babel/cli docs; I had to do it myself to get my project to work.
Also, good thing I checked for this PR - I was going to do a fix myself; I just got bit by these out-of-date instructions, probably could have saved me ~20 minutes figuring out why adding @babel/node (relying on v7) was breaking everything.

Corrected to say @babel/preset-flow

Any updates? It's been 2 weeks, and these docs are still out-of-date.

Just in case it wasn't clear in the review feedback, I think @babel/core still needs to be installed according to https://babeljs.io/docs/en/babel-cli.

Sorry, didn't realise anything was missing.
Added @babel/core to the list of pkgs to install.
Let me know if anything else is missing, I haven't used flow in a few weeks now, so not sure anymore..

Was updated form what I see


[build] Update "Working for Android" wiki pages

Hello!
Working for Android, Working for Android (on Linux) and Working for Android (on macOS) wiki pages look really outdated. If it's possible, can someone update them, please?

@malyutinegor feel free to edit them. They are open for editing.

@raysan5 How can I edit them if I don't know how to build my game for Android? :D

@malyutinegor yeah, that's a good point... I'll try to review it myself at some point...

Closing this issue for now.


Server world leak

1.2.7
No other mods, default config, fresh world. No one has logged on. I'll try to narrow it down.
Forge <IP_ADDRESS>2, not dev
[16:31:30] [Server thread/DEBUG] [FML/]: The world f80016 (world) may have leaked: first encounter (5 occurrences).
[16:32:10] [Server thread/DEBUG] [FML/]: Gathering id map for writing to world save world
[16:32:20] [Server thread/DEBUG] [FML/]: The world f80016 (world) may have leaked: seen 10 times.
[16:32:55] [Server thread/DEBUG] [FML/]: Gathering id map for writing to world save world
[16:33:10] [Server thread/DEBUG] [FML/]: The world f80016 (world) may have leaked: seen 15 times.
[16:33:40] [Server thread/DEBUG] [FML/]: Gathering id map for writing to world save world
[16:34:00] [Server thread/DEBUG] [FML/]: The world f80016 (world) may have leaked: seen 20 times.
[16:34:25] [Server thread/DEBUG] [FML/]: Gathering id map for writing to world save world
[16:34:50] [Server thread/DEBUG] [FML/]: The world f80016 (world) may have leaked: seen 25 times.
[16:35:10] [Server thread/DEBUG] [FML/]: Gathering id map for writing to world save world
[16:35:40] [Server thread/DEBUG] [FML/]: The world f80016 (world) may have leaked: seen 30 times.
went up to 100 times.

please post the full log in pastebin

https://pastebin.com/riaUiTA0
I forgot about multicraft... doubt that has anything to do with it but I'm testing w/o now too.

cannot produce :/

Yeah, it's being finnicky for me, seems random whether or not it shows up. I'll let you know if I can pin it down to something.
¯_(ツ)_/¯


PyQt bindings

I would like to use nzmqt in my PyQt application, because in Python applications we have the same problems of integrating 2 event loops: zmq's and Qt's.
I've googled for PyQt bindings and didn't find any. I think it can be achieved with the help of SIP but I'm not sure because threading techniques in Python differs from the C++.

Pretty old thread, but yes SIP would be the way to go. On the other hand, you might just use the zmq module in Python as in general the API is far simpler and easier to use than cppzmq


Animation start & progress event are never fired

The progress event is not fired because the animation is started before the progress listener can be registered by the application (because the animation is auto-started by the animate method). Both clients ignore listen operation issued after the animation start call.

Progress event will likely be removed (#319), but i still have to verify that the start event works.


Where can I find CONFIG_PATH

In "python -m torch.distributed.launch --nproc_per_node=4 ./tools/train.py CONFIG_PATH"
Where or how is the config file created

https://github.com/tianweiy/CenterPoint/tree/master/configs/nusc

find the config of the model you want to use in the above link and the name is linked to a file


MPAS-Seaice exact restart looking for missing variables

The seaice_single_column_exact_restart task is currently failing with the following:
Variable iceAge not in /lcrc/group/acme/ac.dcomeau/scratch/polaris/scratch/compile-bug_010224/seaice/single_column/exact_restart/full_run/restarts/restart.2000-01-02_00.00.00.nc.
Variable iceAge not in /lcrc/group/acme/ac.dcomeau/scratch/polaris/scratch/compile-bug_010224/seaice/single_column/exact_restart/restart_run/restarts/restart.2000-01-02_00.00.00.nc.
Variable firstYearIceArea not in /lcrc/group/acme/ac.dcomeau/scratch/polaris/scratch/compile-bug_010224/seaice/single_column/exact_restart/full_run/restarts/restart.2000-01-02_00.00.00.nc.
Variable firstYearIceArea not in /lcrc/group/acme/ac.dcomeau/scratch/polaris/scratch/compile-bug_010224/seaice/single_column/exact_restart/restart_run/restarts/restart.2000-01-02_00.00.00.nc.

These tracers were recently turned off by default in https://github.com/E3SM-Project/E3SM/pull/6090.
Removing these two variables from the list here https://github.com/E3SM-Project/polaris/blob/a43fa75b5a29118509eff71d479613e3cdefcf75/polaris/seaice/tasks/single_column/exact_restart/init.py#L35
allow the test to pass. I'm putting together a branch for the fix.

Looks like I raised this too soon; I'd been working with an older version of polaris when I got this error. On main, the test passes, and the iceAge and firstYearIceArea don't show up in the seaice_single_column_exact_restart.log.

@darincomeau, it would still be worth removing variables that are not used in restart files anymore.  I would suggest making a PR to remove them when you have time.


Add tutorial for LOJ-1015 - Brush (I)

Problem: http://lightoj.com/volume_showproblem.php?problem=1015

Are you going to work on this?

@faiyaz26 Yes bhaiya. Will soon create a pull request.


Should overwrite header Connection with connection-token close.

This proxy should always return the header Connection: close and not the upstream server Connection header, until it supports keep-alive persistent connections.

I haven't look at the code in a while but isn't that exactly what line 107 does?
fputs($fp, "Connection: close\r\n\r\n");
fputs($fp, $data);

That's for the request headers, I suppose I was referring to the response headers since I thought at the time certain upstream webserver configurations could send forced keep-alive connection headers. (I think I actually saw that) But I'm not sure anymore, so you can close this. Thanks for looking into it though.


Feature request: allow to add funds directly to existing channels using a third party wallet.

Feature request: allow to add more funds directly to existing channels using a third party wallet.

Changing a channel's capacity is not possible with the current specs, but it has been discussed and may be part of the next version of the specs (but there are no concrete proposals on how to implement it yet).


BUG: HeadTailBreaks raise RecursionError

HeadTailBreaks raises RecursionError if the maximum value within data is there twice (or more). Then it simply locks itself in the loop in values[values >= mean] - mean will be always the same and both values will always returned.
Steps to reproduce:
data = np.random.pareto(2, 1000)
data = np.append(data, data.max())

mc.HeadTailBreaks(data)

I assume that once there are only the same values within remaining values, head_tail_breaks should stop:
def head_tail_breaks(values, cuts):
    """
    head tail breaks helper function
    """
    values = np.array(values)
    mean = np.mean(values)
    cuts.append(mean)
    if len(values) > 1:
        if len(set(values)) > 1:  #this seems to fix the issue
            return head_tail_breaks(values[values >= mean], cuts)
    return cuts

However, I am not sure if it is the intended behaviour to stop and keep multiple values in the last bin as it does not reflect the definition of HeadTailBreaks algorithm (but I cannot see another solution). Happy to do a PR if this is how you want to fix that.

Hi @martinfleis, thank you for reporting the bug and suggesting the solution.
I think having multiple values in the last bin is inevitable if these values are identical - it is not possible to allocate several identical values into different bins. The example you gave is a nice illustration - there are essentially two maximum values in the input data:
data = np.random.pareto(2, 1000)
data = np.append(data, data.max()) #now we have two maximum values in the input data

Your solution looks good to me! I think we can safely get rid of if len(values) > 1: and just replace it with  if len(set(values)) > 1: . So it will be like:
def head_tail_breaks(values, cuts):
    """
    head tail breaks helper function
    """
    values = np.array(values)
    mean = np.mean(values)
    cuts.append(mean)
    if len(set(values)) > 1:
            return head_tail_breaks(values[values >= mean], cuts)
    return cuts

After making the changes, I rerun the example you provided:
np.random.seed(0)
data = np.random.pareto(2, 1000)
data = np.append(data, data.max())
mc.HeadTailBreaks(data)

The output would have two values in the last bin.

If I twist the appended maximum value slightly - add a small value (0.00001) to it to break the tie:
np.random.seed(0)
data = np.random.pareto(2, 1000)
data = np.append(data, data.max()+0.00001)
mc.HeadTailBreaks(data)

The only change to the output classification is an additional bin.

So if you'd like to open an PR, I would happy to review and merge it!


add schoolyearId parameter to getKlassen

i tried to fix the tests, but i have no clue how they work :(


template标签与script标签的自导导入识别规则不一致

Describe the bug

以$等特殊符号开头的自动导入API在<script></script>内能正常使用读取，但在<template></template>内则报出异常。
假如有一个自动检测导入的API：export const $obj = { name: "张三" }
sfc单文件中

<template>
 {{ $obj.name }} <!-- 报错 Cannot read properties of undefined (reading 'name') -->
</template>
<script>
  console.log($obj.name) // 将正常输出 "张三"
</script>

这类情况是预期的吗
$开头虽为约定内容中有所限制，但当其不冲突时应该可用

Reproduction
..
System Info
win11  edge

Used Package Manager
pnpm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A, please open a GitHub Discussion instead.
[X] The provided reproduction is a minimal reproducible of the bug.

$ is reserved for global/internal APIs. Vue's template compiler will treat it differently. I'd suggest you to avoid using that convention


support convert i96 values into other time unit of arrow's timestamp

Related to  https://github.com/datafuselabs/databend/issues/6627

Ooof, what a simplification!

Thanks a lot, @sundy-li !


TLS problem from 2024

from 2024 I always get the error:
FetchError: request to https://api.openai.com/v1/chat/completions failed, reason: Client network socket disconnected before secure TLS connection was established
I use clash as the proxy, and set:
HTTPS_PROXY=http:<IP_ADDRESS>:7890 in the .env file
before 2023 there's no problem using it.

https://github.com/Chanzhaoyu/chatgpt-web/issues/2039

This error became constant since last week. Looks like https://api.openai.com/v1/chat/completions requires TLS1.3 now and node-fetch cannot get through:
FetchError: request to https://api.openai.com/v1/chat/completions failed, reason: Client network socket disconnected before secure TLS connection was established
    at ClientRequest.<anonymous> (/root/chatgpt-web/service/node_modules/.pnpm/node-fetch@3.3.0/node_modules/node-fetch/src/index.js:108:11)
    at ClientRequest.emit (node:events:525:35)
    at ClientRequest.emit (node:domain:489:12)
    at TLSSocket.socketErrorListener (node:_http_client:502:9)
    at TLSSocket.emit (node:events:525:35)
    at TLSSocket.emit (node:domain:489:12)
    at emitErrorNT (node:internal/streams/destroy:151:8)
    at emitErrorCloseNT (node:internal/streams/destroy:116:3)
    at process.processTicksAndRejections (node:internal/process/task_queues:82:21) {
  type: 'system',
  errno: 'ECONNRESET',
  code: 'ECONNRESET',
  erroredSysCall: undefined
}

Is there anyone who has the same issue?


Add list of styles in README

A list of available styles, with a rendered resume in PDF format would be more accessible instead of running cvres to see the resulting style.

Solved at 30ddec5.


Create Minimize_the_maximum_diff_between_heights.cpp

I have written a code which is Minimize the maximum difference between heights, please review my code and merge it.

@all-contributors please add @abhianand0110 for his code


How to override default server name provided by Pushy server to the client?

I set up Pushy server on my Chef 12 server and installed Pushy client on one of the Windows machines (all working in AWS). When started Pushy client successfully connects to Chef server via HTTPS but then tries to connect to the same server (via TCP) using a completely different FQDN (see the second the third log records below). I want to make Pushy client connect to the server using the same FQDN i.e. tcp://chef.myorg.com:10002 and tcp://chef.myorg.com:10000.

[2015-09-17T18:01:03+00:00] INFO: [i-77a2f5dc] Retrieving configuration from https://chef.myorg.com/organizations/myorg/pushy/config/i-77a2f5dc ...
[2015-09-17T18:01:03+00:00] INFO: [i-77a2f5dc] Connecting to command channel at tcp://ip-10-5-5-41.ec2.internal:10002
[2015-09-17T18:01:03+00:00] INFO: [i-77a2f5dc] Listening for server heartbeat at tcp://ip-10-5-5-41.ec2.internal:10000

You can override this with zeromq_listen_address and can find information https://docs.chef.io/config_rb_push_jobs_server.html


A11y_VisualStudioCodeClient_Search_Global Search_ Keyboard:Keyboard Focus moving from search to replace instead of moving to “Match case, match whole word and Use regular expression”

GitHub Tags:
#A11yMAS; #A11yTCS; #A11ySev2; #DesktopApp; #Win32; #Visual Studio Code Client; #BM-VisualStudioCodeClient-Win32-Aug2022; #WCAG2.4.3;
Environment and OS details:
Visual Studio Code
Windows 11 Pro version 21H2 (OS Build 22000.856)
Repro Steps:

Open Visual studio code editor
Using keyboard navigation View Menu Bar
Select search option
Verify that keyboard focus from search to Match case, match whole word and use regular expression or not.

Actual Result:
Keyboard Focus moving from search to replace instead of moving to “Match case, match whole word and Use regular expression”.
Expected Result:
Keyboard Focus should move from search to “Match case, match whole word and Use regular expression”.
Attachment:

I disagree - users want to tab to the replace box much more often than they want to tab to those buttons. The buttons are in the tab flow, just a spot later.


Open datepicker only on input click when using input-group

Hi, I want to use datetimepicker input in an input-group but I want to open the calendar on input click
and prevent openning it on button click, because I will use the button for a different purpose .
For example:

                    <input type="text" class="form-control card-search" a2e-datetimepicker placeholder="{{'entity.common.to'|translate}}" name="to"
                        (onChange)="dateFilterChange($event,'to')" [date]="date" [options]="a2eOptions" #toInput />

Hello,

but I want to open the calendar on input click

you can open the calendar on input click by using the option: allowInputToggle

prevent openning it on button click, because I will use the button for a different purpose .

I don't think it's possible.  The bootstrap datetimepicker use this input-group-icon as main element to display the calendar. If we remove the input-group-icon, the calendar will throw an error.

Ok thanks


Maybe review the LibSass section

Or maybe not?


Add an example with the manual service discovery

Currently, all the clients are using the hardcoded values of addresses of the sibling services. It's necessary to add the following examples:

[x] Use the service discovery with the DiscoveryClient
[x] Use the service discovery with the Feign clients

Implemented in #31 so, closing the ticket


-webkit cleanup

[x] Remove all -webkit- prefixes that aren't needed anymore.
[x] Add missing min-width: 0; to prevent #11866

Merging.. CI fails, but to my understanding, unrelated.


unexpected vmap error due to commit c36e1f7

Description
Hi,
@michalk8 and I noticed a bug in our tests here that occurs in the latest JAX version. After doing git-bisect, we found the bad commit to be: c36e1f7c1ad4782060cbc8e8c596d85dfb83986f.

Code to reproduce:
git clone https://github.com/ott-jax/ott/ && cd ott && pip install -e'.[test]'
pip install git+https://github.com/jax-ml/jax.git@c36e1f7c1ad4782060cbc8e8c596d85dfb83986f
pytest -k 'test_hessian_sinkhorn[False-0.93-0.91-1-1e-05]'

System info (python version, jaxlib version, accelerator, etc.)
jax:    0.4.36.dev20241029+c36e1f7c1
jaxlib: 0.4.35
numpy:  2.0.2
python: 3.11.10 | packaged by conda-forge | (main, Oct 16 2024, 01:26:25) [Clang 17.0.6 ]
device info: cpu-1, 1 local devices"
process_count: 1
platform: uname_result(system='Darwin', node='Michals-MacBook-Pro-3.local', release='24.1.0', version='Darwin Kernel Version 24.1.0: Thu Oct 10 21:03:15 PDT 2024; root:xnu-11215.41.3~2/RELEASE_ARM64_T6000', machine='arm64')

Thanks for the clear report!

@marcocuturi could you minimize this? There's a lot going on in this code that we aren't familiar with, and it's much harder for us to minimize unfamiliar code than for you to minimize your own code. Think of the time for us to debug this as exponential in the length of the repro you give us.

It might be that the bug is coming from transformations created by lineax,

Are any of lineax's tests failing?

Are any of lineax's tests failing?

No, all of them are passing with the latest version of JAX/lineax.

Here's a simpler example crafted by @michalk8 and @Algue-Rythme demonstrating the problem, which arises indeed from lineax (tagging @patrick-kidger ), leading to the same vmap error:
import jax
import jax.numpy as jnp
import lineax as lx

@jax.custom_vjp
def f(x):
  return x.sum()

def f_fwd(x):
  return x.sum(), (x,)

def f_bwd(res, g):
  x, = res
  A, b = jnp.eye(x.shape[0]), jnp.ones_like(x)
  # op = lx.FunctionLinearOperator(lambda x: x, input_structure=x)  # fails
  op = lx.MatrixLinearOperator(A)  # fails
  r = lx.linear_solve(op, b).value
  return (r * x * g,)

f.defvjp(f_fwd, f_bwd)

rng = jax.random.key(0)
x = jax.random.normal(rng, (10,))
jax.hessian(f)(x)

Yeah, I've also been seeing widespread failures in Diffrax's test suite, due to what looks like a totally different vmap failure. I've spent most of today digging through this and haven't identified a root cause yet. It might take a while to update the JAX ecosystem to be compatible with this version of JAX.

Okay, I think I've identified the root cause: with the latest changes, the batch interpreter has become a dynamic trace, i.e. it calls batch rules when it previously wouldn't. This meant that a lot of arrays were having their nonbatch dimensions now being turned into batch dimensions!
With that problem identified it's been a relatively simple matter to update a couple of batching rules in Equinox to handle this new calling case appropriately.
@marcocuturi @michalk8 can you try https://github.com/patrick-kidger/equinox/pull/907 on your full example / on your tests? If it passes then I'll do a new release of Equinox that is compatible with latest JAX.

Nice find @patrick-kidger ! That’s right, actually everything is a dynamic tracer now. No more automatic rules-only-called-based-on-data-dependence, though rules themselves can choose to behave based on dependence. I believe it gives rules strictly more power/expressiveness.

Hey @mattjj, can we consolidate this kind of knowledge into an updated version of Autodidax?

@marcocuturi @michalk8 can you try https://github.com/patrick-kidger/equinox/pull/907 on your full example / on your tests? If it passes then I'll do a new release of Equinox that is compatible with latest JAX.

Works great, thanks!

@soraros yes, good idea, that’s our plan! Cc @dougalm


[REQUEST] Look at patches from darkxst for his multipan flasher fork of the Silabs Flasher

FYI, darkxst has a downstream fork of Silabs Flasher add-on with patches that might be interested in looking at for your fork of it:
https://github.com/darkxst/multipan_flasher
https://github.com/darkxst/multipan_flasher/pulls?q=is%3Apr
https://github.com/darkxst/multipan_flasher/blob/main/multipan_flasher/DOCS.md
Thinking that perhaps him and you are also interested in collaborating on some patches from your own Silabs Flasher fork:
https://github.com/mercenaruss/zigstar_addons/tree/main/zigstar-silabs-flasher
https://github.com/mercenaruss/zigstar_addons/blob/main/zigstar-silabs-flasher/DOCS.md

Currently no any work in ongoing for Silabs Flasher,will update once i have time for this.


RC test: Natural Selection 1.2.0-rc.1

@amanda-phet, @kathy-phet, @ariel-phet, @oliver-phet, @KatieWoe natural-selection/1.2.0-rc.1 is ready for RC testing.
This RC is for the deliverable described in https://github.com/phetsims/natural-selection/issues/208.  Deadlines are:

testing of this RC should be completed by 9/23/2020
delivery of an RC version to the client by 10/1/2020

Document issues in https://github.com/phetsims/natural-selection/issues and link to this issue.
This is an RC Lite test.  Test platforms are the same as a dev test:

[ ] Latest macOS, Chrome and Safari
[ ] Latest iOS, Safari
[ ] Windows 10, all browsers (IE dropped)
[ ] Latest Chrome OS, Chrome

Assigning to @ariel-phet for prioritization.

General RC Test

What to Test

Click every single button.
Test all possible forms of input.

Test all mouse/trackpad inputs.
Test all touchscreen inputs.

If there is sound, make sure it works.
Make sure you can't lose anything.
Play with the sim normally.
Try to break the sim.
Test all query parameters on all platforms. (See QA Book for a list of query parameters.)
Download HTML on Chrome and iOS.
Make sure the iFrame version of the simulation is working as intended on all platforms.
Make sure the XHTML version of the simulation is working as intended on all platforms.
Complete the test matrix.
Don't forget to make sure the sim works with Legends of Learning.
Test the Game Up harness on at least one platform.
Check this LoL
spreadsheet and notify AR or AM if it not there.
If this is rc.1 please do a memory test.
When making an issue, check to see if it was in a previously published version
Try to include version numbers for browsers
If there is a console available, check for errors and include them in the Problem Description.
As an RC begins and ends, check the sim repo. If there is a maintenance issue, check it and notify developers if
there is a problem.

Focus and Special Instructions
This RC is for PhET-iO brand only. While PhET-iO should be the major focus, please do fully test the sim.
See the Client Guide for client-specific requirements that should be tested.
No links to the PhET brand are included here because we will not be testing it for this RC.

PhET-iO RC Test

What to Test

Make sure that public files do not have password protection. Use a private browser for this.
Make sure that private files do have password protection. Use a private browser for this.
Make sure standalone sim is working properly.
Make sure the wrapper index is working properly.
Make sure each wrapper is working properly.
Launch the simulation in Studio with ?stringTest=xss and make sure the sim doesn't navigate to youtube
For newer PhET-iO wrapper indices, save the "basic example of a functional wrapper" as a .html file and open it. Make
sure the simulation loads without crashing or throwing errors.
N/A For an update or maintenance release please check the backwards compatibility of the playback wrapper.
Load the login wrapper just to make sure it works. Do so by adding this link from the sim deployed root:/wrappers/login/?wrapper=record&validationRule=validateDigits&&numberOfDigits=5&promptText=ENTER_A_5_DIGIT_NUMBER

Further instructions in QA Book
Conduct a recording test to Metacog, further instructions in the QA Book. Do this for iPadOS + Safari and one other random platform.
Conduct a memory test on the stand alone sim wrapper (rc.1).
Test one platform combination with ?phetioDebug on the Studio and State wrapper.

Focus and Special Instructions
[Provide further instructions here. If you have any further tests you want done or specific platforms you want tested,
list them here. Any behaviors you want QA to pay special attention to should be listed here.]

Issues to Verify

[x] memory test for brand=phet-io (1.2 branch) - since this is the first RC on a new branch (1.2), please confirm the results shown in this issue

These issues should have the "status:ready-for-qa" label. Check these issues off and close them if they are fixed.
Otherwise, post a comment in the issue saying that it wasn't fixed and link back to this issue. If the label is
"status:ready-for-review" or "status:fixed-pending-deploy" then assign back to the developer when done, even if fixed.

Link(s)

Wrapper Index
Test Matrix

FAQs for QA Members

There are multiple tests in this issue... Which test should I do first?
Test in order! Test the first thing first, the second thing second, and so on.

How should I format my issue?

We typically assign the developer who opened the issue in the QA repository.

My question isn't in here... What should I do?
You should:

Consult the QA Book.
Google it.
Ask Katie.
Ask a developer.
Google it again.
Cry.

Playback wrapper seems to be exceedingly slow on Mac 10.15 Chrome for both loading log and starting playback. Having trouble even getting it to start. @zepumph please advise.

Noting a fair amount of slowdown between generations on ChromeOS when looking at a data stream (such as colorized).

@KatieWoe I don't know whether slowdown in the colorized data stream wrapper is an issue.  I think it would be expected with NS, since it's doing a massive amount of writing to the console.  If you think it's significant, please create an issue.

I don't know if it's significant. It is a massive amount of slow down, but I don't know how important performance is in that wrapper.

QA is done.

Thanks @KatieWoe.
Your comments in https://github.com/phetsims/QA/issues/551#issuecomment-696846871 and  https://github.com/phetsims/QA/issues/551#issuecomment-697014587 have been promoted to their own issues.

@KatieWoe The test matrix shows errors for #376 and #377.  There are no such issues in the natural-selection repo. I eventually found them in phet-io-wrappers.  When an issue is in another repo, could you please note in the test matrix as (e.g.) phet-io-wrappers#376 ?

Next RC test created in #557.   Closing.


Indexing on multiple disks

Indexing on mulitple disks using multiple base task directories on Middlemanagers / Indexers

Release note

Multiple base task directories can be assigned using druid.indexer.task.baseTaskDirPaths=[\"PATH1\",\"PATH2\",...] in the runtime properties of the Middlemanager / Indexer. This facilitates the usage of multiple disks for indexing tasks

Key changed/added classes in this PR

TaskStorageDirTracker

The base task directories are configured in TaskConfig and TaskStorageDirTracker handles allocation of these directories to a tasks. These can be useful when multiple disks are mounted and all of them need to be utilized.
CliPeon now takes 3 arguments (Base task directory, taskId, attemptId) instead of two.

This PR has:

[x] been self-reviewed.

[ ] using the concurrency checklist (Remove this item if the PR doesn't have any relation to concurrency.)

[ ] added documentation for new or modified features or behaviors.
[x] a release note entry in the PR description.
[ ] added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
[ ] added or updated version, license, or notice information in licenses.yaml
[ ] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
[x] added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
[ ] added integration tests.
[x] been tested in a test Druid cluster.

@AmatyaAvadhanula : Since we've deprecated the druid.indexer.task.baseTaskDir option and recommend druid.indexer.task.baseTaskDirs now, can you please also update the documentation and template configuration files to use the new config instead of old one?

Thank you for the review @rohangarg!
I've added the release notes
Merging since all the tests have passed on GHA


Adjusting Class Count in Pretrained Model

Hi,
I'm using a pretrained MagFace model (based on iResNet18) from the Model Zoo  and trying to finetune it on my custom dataset with 2000 identities (about 10000 images). The dataset is described as recommended in the train.list file, where each line contains the image name (image path + image name), '0' as Separator and the identity (id). I consider the id as the identity of the person on the image (i.e.: /path_to_img/2_12.png 0 2).
My understanding is that during MagFaces Training Identities in the Dataset are considered as classes (targets). So when attempting to start training with "--last_fc_size 2000," I encounter the following error:
**_> RuntimeError: Error(s) in loading state_dict for SoftmaxBuilder:

size mismatch for fc.weight: copying a param with shape torch.Size([512, 10572]) from checkpoint, the shape in current model is torch.Size([512, 2000])._**

Questions: What is considered a "class" during training? And how can I resolve this issue? How could I properly load parameters from a pretrained modell, so that I am sure, that everything goes well?
Thank you in advance for your assistance!

The fc.weight is of size [512, num_of_classes]. For finetuning, that weight should not be loaded as the num_of_classes are inconsistent.
The code may be helpful.

Hi, thank you for your quick reply:)
I have adopted these functions for loading the weights and set the parameter '--last-fc-size 85742' to 3000 when executing the script 'run.sh', as I have 3000 identities in my training data set. The error message has now disappeared.
Thank you @IrvingMeng

When preparing the images for training, I set the size to 112x112 with 3 channels. Is this sufficient for MagFace to be able to use the images correctly?

Hi, face images should be properly aligned by facial landmarks as indicated in: https://github.com/IrvingMeng/MagFace?tab=readme-ov-file#basic-training.

Hi @IrvingMeng, I guess, I could use the indicated code  for aligning my images. i.e. by calling the function norm_crop(img, landmark, image_size=112, mode='arcface')  ?

yep, you are right.

Hi, I please have a question to the landmarks list to be considered when calling the function 'norm_crop(img, landmark)' . I first make a Landmark detection on my image and get as example the  list of landmarks bellow:

[{'chin': [(21, 54), (23, 63), (24, 72), (26, 81), (29, 89), (34, 96), (41, 102), (48, 107), (57, 109), (65, 107), (72, 101), (78, 95), (83, 87), (85, 79), (86, 70), (88, 62), (89, 53)],
'left_eyebrow': [(27, 49), (31, 44), (37, 43), (43, 43), (50, 45)],
'right_eyebrow': [(62, 45), (68, 43), (74, 43), (80, 44), (84, 48)],
'nose_bridge': [(56, 52), (56, 57), (57, 61), (57, 67)],
'nose_tip': [(51, 71), (54, 72), (57, 73), (60, 72), (62, 71)],
'left_eye': [(36, 53), (40, 51), (44, 51), (48, 53), (44, 54), (40, 54)],
'right_eye': [(63, 54), (67, 51), (71, 51), (75, 53), (72, 54), (67, 54)],
'top_lip': [(43, 82), (48, 79), (53, 78), (57, 79), (60, 79), (66, 80), (70, 82), (68, 83), (61, 81), (57, 81), (53, 80), (45, 82)],
'bottom_lip': [(70, 82), (66, 88), (61, 91), (57, 92), (52, 91), (47, 88), (43, 82), (45, 82), (53, 88), (57, 89), (60, 88), (68, 83)]}]

Since the function estimate_norm from the face_align.py file expects a landmark shape of (5, 2), how could I bring the above landmark list into a suitable (5, 2) form? Which landmarks should be considered, for MagFace to be properly trained on the image?


KIT Dataset Training Setting

Hello,
I have a question about the training settings for the KIT dataset.
In the configs folder, there is only a yaml file for humanml3d and none for KIT.
Therefore, I copied the humanml3d file to create a KIT config file and trained the VAE.
I changed the dataset name in the config file from humanml3d to KIT and trained for 30,000 epochs as mentioned in the supplementary material.
However, the training results show lower performance compared to the supplementary's Table E for motion, action, and specific VAE. For example, the results for the motion level VAE are as follows:
R_TOP_1 3.410e-01   R_TOP_2 5.625e-01   R_TOP_3 6.970e-01
gt_R_TOP_1 4.307e-01   gt_R_TOP_2 6.590e-01   gt_R_TOP_3 7.840e-01
FID 3.099e+00
Is there any different setting required when training the KIT dataset compared to humanml3d? Or are there any other points to be aware of?
I look forward to your response.
Thank you.

You need to load the VAE weights trained on the humanml3d dataset as the initial weights.
In config:
PRETRAINED_VAE: $PATH

In train.py:
line 223: model.vae.load_state_dict(vae_dict, strict=False)

Thank you so much for your prompt response!


training tool

Hi , is there a training tool to train face model such as face_p146.xml?

Sorry training tool is not provided.


Add support for NetBSD

NetBSD supports mmap() and pthread_getschedparam():
--- absl/base/config.h.orig
+++ absl/base/config.h
@@ -365,6 +365,7 @@ static_assert(ABSL_INTERNAL_INLINE_NAMES
 #ifdef ABSL_HAVE_MMAP
 #error ABSL_HAVE_MMAP cannot be directly set
 #elif defined(__linux__) || defined(__APPLE__) || defined(__FreeBSD__) ||   \
+    defined(__NetBSD__) || \
     defined(__ros__) || defined(__native_client__) || defined(__asmjs__) || \
     defined(__wasm__) || defined(__Fuchsia__) || defined(__sun) || \
     defined(__ASYLO__)
@@ -378,6 +379,7 @@ static_assert(ABSL_INTERNAL_INLINE_NAMES
 #ifdef ABSL_HAVE_PTHREAD_GETSCHEDPARAM
 #error ABSL_HAVE_PTHREAD_GETSCHEDPARAM cannot be directly set
 #elif defined(__linux__) || defined(__APPLE__) || defined(__FreeBSD__) || \
+    defined(__NetBSD__) || \
     defined(__ros__)
 #define ABSL_HAVE_PTHREAD_GETSCHEDPARAM 1
 #endif

Also _XOPEN_SOURCE must not be defined, so either apply the following, or exclude defining it only for NetBSD (by the way, Darwin/macOS does not need either).
--- absl/time/internal/cctz/src/time_zone_format.cc.orig
+++ absl/time/internal/cctz/src/time_zone_format.cc
@@ -18,11 +18,6 @@
 #endif
 #endif

-#if defined(HAS_STRPTIME) && HAS_STRPTIME
-#if !defined(_XOPEN_SOURCE)
-#define _XOPEN_SOURCE  // Definedness suffices for strptime.
-#endif
-#endif

 #include "absl/base/config.h"
 #include "absl/time/internal/cctz/include/cctz/time_zone.h"

Thank you for the suggestion. Since we don't actively try to support every platform, we take a passive approach to adopt any "harmless" patches. I think this patch seems fit in the category.
However, please note that we don't have any test coverage for NetBSD and don't intend to add the coverage. That said, it would be ideal for you to test the patch on your platform to make sure it works and send us a PR. Otherwise, I can blindly apply the patch but won't have a way to verify its correctness.
In addition, the second half of the change seems to be in cctz and Abseil regularly syncs with it, so you will need to open a PR in https://github.com/google/cctz.


EAU becomes EU

When playing for Russia, after discovering polarized world, player can form EAU and it can form EU on the next day. Also, democracies should be allowed to join EAU, at least if Populists, Traditionalists or Socialists are in power. I can resolve this issue myself.

Also, enlargement of a union seems to be broken. How is it supposed to work?

Also, change regime in sphered country event doesn't appear more than one time in the whole game. Also, install democracy/etc. justification of war is not avaliable for unknown reason

I will resolve this issues after playing throgh the whole scenario.

EU should have cores on these at 1992 - https://commons.wikimedia.org/wiki/File:EC-EU-enlargement_animation.gif#/media/File:EU12-1992-93_European_Union_map.svg

You report several issues in one. Please report one problem or request per issue. I will break this issue into several - moving the comments there..

On the "install democracy/etc. justification of war is not available for unknown reason" - please create a separate issue with details


HtmlUnit won't retrieve the expected page

I'm trying to access this page from HtmlUnit 2.34.1 to fill a form and save the result. The problem is I don't get the same resulting page I get when loading from a "real" browser. The page I get from HtmlUnit has just one noscript element inside its body with the following message:

Please enable JavaScript to view the page content.
Your support ID is: (A large number that changes on each execution).

I am sure I have enabled JavaScript in my WebClient options and setted 30 seconds as the JavaScript timeout (the hole page load takes less than a second). I also tried using the 2.34.2-SNAPSHOT, but the problem persists.
I have been using HtmlUnit almost a year ago and it has been really useful filling forms and retrieving information from the resulting pages, so that's why this issue surprises me that much.
Thank you so much for all the efford. If you need clarification, just ask.

Sorry, but this page works based on a lot of obfuscated javascript. I do not have the time to debug this. To get this working you have to nail down the problem.

No further feedback will close this.


Move the MX color & vector headers to src/shaders

We've already stripped these headers of any MX-specific bits. They seem
generally useful, so let's add them to src/shaders to anybody can use
them.  It's not "OSL standard library" (in stdosl.h and described in the
OSL Language Specification) but they are the next level of goodness --
distributed with the OSL library as a token of our beneficence toward
all mankind.

LGTM


option to show only unread

the sorting options are very useful, but i'd like to propose adding an 'unread' filter to both options, or perhaps better yet, adding a button on the toolbar that toggles between displaying all articles and displaying unread only

There's one special thing about SmartRSS: it additionally has the state called New, so you can click on article, quickly glance over it so it isn't new anymore but stays unread. Of course you can tuen on auto marking as read too.
I'll probably just add filters for both Unread and New only

hi zakius - the issue i'm looking to solve is unread articles being interspersed among read articles - unless there's something i'm not understanding, i don't think the 'New' state solves that???
why unread stuff sometimes gets mixed with read, i don't know - perhaps there's missing date info in the feed or something

The bold blue titles are new, bold ones are unread. I'm not sure what is your exact issue as to be honest I just remove articles that I'm not longer interested in so I have no read ones at all, but I'll try changing my habit for a few days to look further into in. In the meantime can you elaborate a bit more? So I'll know what to look at exactly. Maybe you're experiencing articles updating, it is intended to let you know something changed, though possibly I could use a new state for that in the future to avoid confusion

The bold blue titles are new, bold ones are unread.

yes, i'm aware of that, the problem is that there may be 10 unread articles, then 3 read ones, then 1 unread, repeat... i don't think all of the read ones that are located in the list between unread are updated articles, however i may be wrong
if you intend to auto-purge read articles, or add a sorting option to sort by unread (this would work great as a last sort option), i think that would work great

I'll add the mentioned filters and consider auto removal
I'm not sure about sorting, I assume it would show reverse chronologically unread items first and then read ones?
And to clarify things: are you experiencing and submitting a bug report or only suggesting enhancement?

I'm not sure about sorting, I assume it would show reverse chronologically unread items first and then read ones?

that's what i had in mind

are you experiencing and submitting a bug report or only suggesting enhancement?

feature :)

Okay, I get it
Currently I have one version to release and few bugs to take a look at, but I'll get to your requests in some time

that's great - no hurry - and sorry for all the issues i've been submitting - i've kinda been pretty frustrated trying to find a good feed reader since Newsfox died as a result of the WebExt API and i think yours is the best candidate :)

i'm going to rescind this feature request as well


Metadata provides wrong size

Metadata does not provide the proper ram size but only the size of the entire buffer after remapping.
It can be easily reproduced via:
# -*- mode: ruby -*-
# vi: set ft=ruby :

Vagrant.configure("2") do |config|
  config.vm.provider :libvirt do |v|
    v.memory = 4096
    v.cpus = 2
    v.qemuargs :value => "-qmp"
    v.qemuargs :value => "unix:/tmp/vagrant-qmp.sock,server,nowait"
  end

  config.vm.box = "generic/ubuntu2010"
end

It fails with and without qmp to provide the proper metadata.

After some discussion with @h33p we think the best approach would be to rename the size field to max_address and eventually introduce a real_size field which could provide the usable size of the VM (which might help when doing compacted dumps)


extend blog webmentions feature to all web sites

...ie not just blogger, tumblr, and wp.com. inspired by IRC today.
(as background, the blog webmention feature currently sends all outgoing webmentions for wordpress.com, tumblr, and blogger blogs by watching their feeds and extracting links in new posts when they're posted. this would extend that feature to any site.)
the vast majority of this is done. we just need a text box next to the blog providers on the front page to enter your domain, maybe indieauth or check for a backlink to prove you own the domain (not strictly necessary since anyone can send a webmention for any source and target), and maybe tweak a few other things that come up. surprisingly little work required.

i like that.

+1

@aaronpk's telegraph now does this! https://telegraph.p3k.io/superfeedr . makes it a bit less of a priority to do it in bridgy too.

@snarfed hmm. I thought that was for being able to receive webmentions from sites that don't technically support them, not for being able to send them if you are on a platform that doesn't have a method for sending.

hmm. I thought that was for being able to receive webmentions from sites that don't technically support them

right! that's what this issue is for.

for being able to send them if you are on a platform that doesn't have a method for sending.

that's what bridgy's existing blog feature is! it sends and receives wms for blogs on tumblr, blogger, and wordpress.com. https://brid.gy/about#blogs

...oh, do you mean extending the blog feature to sites that aren't on tumblr/blogger/wp.com? you're right, we could definitely do that too! at least for sending. i'd probably encourage people to use a native plugin in their CMS, for that though, or telegraph itself, since that's what it's for.

oh. OK :-) I thought it was the opposite. Allow users who aren't on tumblr, blogger or wp.com to be able to  easily send webmentions (i.e. Jekyll sites on GitHub pages.)

@snarfed yes, to follow up, Jekyll sites on GH pages was the specific use case I'm thinking about. There isn't a "simple" way for those users to send webmentions. Inserting some js to receive them isn't an issue. I'm used a convoluted method of a ruby plugin and building via Travis CI and then pushing back to GitHub in order to send. Not for the faint of heart or beginning user.
I'm currently working on a Jekyll "theme" that a new user could clone, edit a few lines of the config and be up & running with an indieweb site fairly quickly and easily. Having a solution for also sending would be 👍
Just a thought.

@miklb definitely! sounds like a good telegraph feature request, especially since it already has some superfeedr integration. cc @aaronpk.

A h-feed to Webmention pionger would be a great extension of Telegraph or similar. It's something I need for my Jekyll blog as well and have been thinking of building myself if nothing else. Consuming a Pubsubhubbub h-feed/rss and sending Webmentions for all relevant links in new/updated posts in there.

@aaronpk's telegraph now does this! https://telegraph.p3k.io/superfeedr

Also webmention.app (docs) and Pushl.


[User Story] A novice user needs help to convert their data into a format that they can use with the available python interface

at lease three situation should be considered:

should provide a basic methods for the one who has multiple files, especially, the feature and target are not in the same database (It is better to do with software, not code).
-- with merge key
-- without merge key

should provide basic methods for the one has the other type of data: txt, excel, json, database(sql,...)

should provide methods to deal with large data set (larger than 10GB)

This user story is far to big! Split it in at least 3 (see above)


Expose overscanIndices getter to make it user-extendable

This PR exposes a new overscanIndicesGetter prop to allow the default behavior to be extended.
Primary goal
By default, React Virtualized will try to optimize the overscan indices based on the scroll direction. This is great for most use cases, but it is something that users may want to implement differently, for a variety of reasons (for example, #554), and it would be challenging to try to support them all out of the box.
I took inspiration from the defaultCellRangeRenderer approach, which I believe is a great approach to give advanced users a more fine-grained control while still providing a sensible default and keeping the API surface that needs to be maintained small.
I'm absolutely open to feedback on this and making changes as needed!

This is a very pragmatic solution. Love it. Thanks @clauderic!


Added kubernetes liveness and readiness probes

Kubernetes checks if a pod is ok and if it can retrieve traffic using probes. This commit add two routes to make a liveness probe and a readiness probe.
Added this two routes makes it possible to start mailpit also in a k8s cluster.
Kubernets can then use these two endpoints to monitor the application.

Thank you for your contribution. I don't use Kubernetes so I'm sure this will be useful to others using it :)

A new release (v1.3.7) is building now, so should be ready in the next 10 minutes or so.

Thanks a lot for the fast merge ;)


Insert empty line between suite and alternative branch after def/class

When there is a function or class definition at the end of a suite followed by the beginning of an alternative block, we have to insert a single empty line between them.
In the if-else-statement example below, we insert an empty line after the foo in the if-block, but none after the else-block foo, since in the latter case the enclosing suite already adds empty lines.
if sys.version_info >= (3, 10):
    def foo():
        return "new"
else:
    def foo():
        return "old"
class Bar:
    pass

To do so, we track whether the current suite is the last one in the current statement with a new option on the suite kind.
Fixes #12199

Not sure if this counts as bugfix or breaking change, i'm fine with merging it before the next breaking release.

Not sure if this counts as bugfix or breaking change, i'm fine with merging it before the next breaking release.

I think we have to gate this behind preview according to our version policy. The version policy only allows formatting style changes that address instabilities or formatting that lead to invalid syntax.
The ecosystem checks look good :)


Added Simon Says game

PR Description 📜
Please include summary related to the issue you have fixed and describe your PR in brief over here by specifying the issue number on which you were working below
Fixes #4732
i have added the Simon Says game,in which players watch as colored buttons light up and play sounds in a sequence. they need to remember and repeat this sequence by pressing the buttons in the same order. the game gets harder as it adds more steps making a mistake ends the game, showing the player's score and allowing them to try again. this game combines elements of memory, pattern recognition, and reflexes, making it both challenging and engaging.

Mark the task you have completed ✅

[ x ] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[ x ] I have performed a self-review of my own code or work.
[ x ] I have commented my code, particularly in hard-to-understand areas.
[ x ] My changes generates no new warnings.
[x  ] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[ x ] I have added screenshot for website preview in assets/images
[ x ] I have added entries for my game in main README.md
[ x  ] I have added README.md in my folder
[  ] I have added working video of the game in README.md (optional)
[ x ] I have specified the respective issue number for which I have requested the new game.

Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

Thank you @Shravanikale ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

hey @ishita-43 !! i have added the game entry in the main Readme.md file and game image in the asset folder of the gamezone.


Minor sprite changes + removed unecessary code

Codecov Report
Base: 79.96% // Head: 79.96% // No change to project coverage :thumbsup:

Coverage data is based on head (f9beef1) compared to base (cad4f33).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##           master      #22   +/-   ##
=======================================
  Coverage   79.96%   79.96%
=======================================
  Files          52       52
  Lines        1687     1687
=======================================
  Hits         1349     1349
  Misses        338      338

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Yo, don't let this hang too much, fix the error and let's merge it.


Text looks different from its original format in Brave latest versions

Attachment or Link to the PDF file here:
PDF testing file (1).pdf
Configuration:

Web browser and its version:  Brave Browser v 1.47+
Operating system and its version: Windows and Mac

Steps to reproduce the problem:

Go to Brave browser (version above 1.47)
render/parse the attached PDF in that browser (PDF testing file (1).pdf
)
You can see the text is not aligned properly (not its original format) (if you render/parse the PDF in Chrome/Firefox/Safari works perfectly)

What is the expected behaviour? Text should aligned properly like in Chrome/Firefox/Safari
What went wrong? The text looks different from its original format in Brave latest versions
Screenshot:

Web browser and its version:  Brave Browser v 1.47+

Sorry, but that's not a browser that we support in the PDF.js project; please see https://github.com/mozilla/pdf.js/wiki/Frequently-Asked-Questions#faq-support
Also, you've not said anything about which PDF.js version you're using.

You can see the text is not aligned properly (if you render/parse the PDF in Chrome/Firefox/Safari works perfectly)

If it works in all other browsers that most unfortunately points to a bug in the "Brave browser" itself, rather than in the PDF.js library and you'll probably need to report this to the browser developers instead.


chore(CHANGELOG): update to v2.2.0

Without types for camunda-bpmn-js itself there's not much benefit for the users (yet). Will look into that.


add dynamically generated types directory tree

Coverage remained the same at 99.062% when pulling c48e7f54678cf89bd621b75b7bc9de1c49e559ff on docs into 07551243aa15fed2040c8a1603889ab181f71159 on master.


[passmanager] Change the verifier analysis to use function names inst…

…ead of SILFunction pointers for its internal state.
This enables us to have state independent of the liveness of the SILFunction's
that we are tracking.
I also changed the verifier to implement only verifyFull instead of verify to
ensure that when we run with sil-verify-all this only runs at the end of pass
manager pipelines.
rdar://42301529

NOTE: This still doesn't enable this change. I am going to do that in a separate PR.

@swift-ci smoke test and merge


Could not resolve styled-component

Hi have sanity v3.7.0 nextjs 13.2. I have installed from yarn this package and now have this error
`9:38:08 PM [vite] page reload sanity.config.ts
✘ [ERROR] Could not resolve "sanity"
../node_modules/@sanity/language-filter/lib/index.esm.js:1:1275:
  1 │ ...om"react/jsx-runtime";import{definePlugin as d}from"sanity";import{createContext as u,useMemo as g,useContext...
    ╵                                                       ~~~~~~~~

You can mark the path "sanity" as external to exclude it from the bundle, which will remove this
error.
✘ [ERROR] Could not resolve "styled-components"
../node_modules/@sanity/language-filter/lib/index.esm.js:1:1544:
  1 │ ... I,Button as L}from"@sanity/ui";import T from"styled-components";const P=(e,n,t)=>!e.name.startsWith("locale"...
    ╵                                                 ~~~~~~~~~~~~~~~~~~~

You can mark the path "styled-components" as external to exclude it from the bundle, which will
remove this error.
✘ [ERROR] Could not resolve "styled-components"
../node_modules/@sanity/ui/dist/index.esm.js:6:97:
  6 │ ...t styled, { ThemeProvider as ThemeProvider$1, useTheme as useTheme$1, css, keyframes } from 'styled-components';
    ╵                                                                                ~~~~~~~~~~~~~~~~~~~

You can mark the path "styled-components" as external to exclude it from the bundle, which will
remove this error.
9:38:11 PM [vite] error while updating dependencies:
Error: Build failed with 3 errors:
../node_modules/@sanity/language-filter/lib/index.esm.js:1:1275: ERROR: Could not resolve "sanity"
../node_modules/@sanity/language-filter/lib/index.esm.js:1:1544: ERROR: Could not resolve "styled-components"
../node_modules/@sanity/ui/dist/index.esm.js:6:97: ERROR: Could not resolve "styled-components"
at failureErrorWithLog (/Users/yaroslavkaplunskyi/nextjs-projects/nextjs_TS/sanity-cms/node_modules/esbuild/lib/main.js:1604:15)
at /Users/yaroslavkaplunskyi/nextjs-projects/nextjs_TS/sanity-cms/node_modules/esbuild/lib/main.js:1056:28
at runOnEndCallbacks (/Users/yaroslavkaplunskyi/nextjs-projects/nextjs_TS/sanity-cms/node_modules/esbuild/lib/main.js:1476:61)
at buildResponseToResult (/Users/yaroslavkaplunskyi/nextjs-projects/nextjs_TS/sanity-cms/node_modules/esbuild/lib/main.js:1054:7)
at /Users/yaroslavkaplunskyi/nextjs-projects/nextjs_TS/sanity-cms/node_modules/esbuild/lib/main.js:1166:14
at responseCallbacks. (/Users/yaroslavkaplunskyi/nextjs-projects/nextjs_TS/sanity-cms/node_modules/esbuild/lib/main.js:701:9)
at handleIncomingPacket (/Users/yaroslavkaplunskyi/nextjs-projects/nextjs_TS/sanity-cms/node_modules/esbuild/lib/main.js:756:9)
at Socket.readFromStdout (/Users/yaroslavkaplunskyi/nextjs-projects/nextjs_TS/sanity-cms/node_modules/esbuild/lib/main.js:677:7)
at Socket.emit (node:events:513:28)
at addChunk (node:internal/streams/readable:324:12)`

updating to 3.8.0 solved the problem


Polish

Rather that defining the 3 annotations to enable configuration, auto-configuration and component scan, we use @SpringBootApplication.
We don't recommend using field injections and we prefer if the SpringApplication is as simple as possible as it is the root of the context. This PR moves the logic of executing the command in a separate class.

Codecov Report

Merging #770 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #770   +/-   ##
=========================================
  Coverage     93.15%   93.15%
  Complexity      385      385
=========================================
  Files             2        2
  Lines          5684     5684
  Branches       1480     1480
=========================================
  Hits           5295     5295
  Misses          186      186
  Partials        203      203

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 8ae4b57...6728432. Read the comment docs.

@snicoll Thanks for improving the example!
Question: How will the SpringApplication.run method know to invoke the MyApplicationRunner.run method, since the MyApplication class has no reference to MyApplicationRunner? Does it scan the classpath for classes implementing CommandLineRunner?

Ah, sorry. No Spring doesn't scan the classpath unless told to. Adding @SpringBootApplication means that classpath occurs on the package of the app (and all sub-packages) but you need a stereotype to indicate it should be handled as a bean.
I forgot that and submitted #771 to fix it. Thanks for the review

I see. Thank you for the clarification!


Add https readme section

@montezdesousa in my case the mkcert --install didn't work and I had to use brew so I think is better to just point to that repo and have users follow instructions there.


string surround should take only one argument for both sides

I think?

https://github.com/samhh/fp-ts-std/commit/415ab3635f3295ed8e09f91423415b46167d7059


fix: Should display as XML predefined entities

These currently display as literals but are intended to show as entities.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/msbuild/msbuild-conditions.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


ci: Do not run cloud-cleanup on fork

Add a check to not run cloud-cleanup on a fork!

Thanks for the reviews folks!