step-definitions.ts assumes that code outside the project root can't be valid

Current behavior
We have a project which defines an integration folder outside the project root (it is a sibling directory) as ../src. This works great. However, we've been stuck on the older browserify with the original cypress-cucumber-preprocessor. For more explanation: we are co-locating tests that use storybook as the source with the component they are testing. Because we have both e2e and integration tests, we use the --project flag to set the root directory to the location of the specific test boilerplate (either integration or e2e).
I just got things to work with @badeball/cypress-cucumber-preprocessor (yay!) but the check that a step file is within the project root is failing. I don't think this is necessary at all, and can be safely removed.
Desired behavior
The only check is whether the step is within the integration folder, just like Cypress does.
Test code to reproduce
Here's the patch that patch-package made. I can make a PR against the source if you'd like.
diff --git<EMAIL_ADDRESS><EMAIL_ADDRESS>index 808b126..ef12071 100644
---<EMAIL_ADDRESS>+++<EMAIL_ADDRESS>@@ -40,9 +40,6 @@ function getStepDefinitionPatterns(configuration, filepath) {
     if (!(0, is_path_inside_1.default)(filepath, fullIntegrationFolder)) {
         throw new Error(`${filepath} is not inside ${fullIntegrationFolder}`);
     }
-    if (!(0, is_path_inside_1.default)(filepath, configuration.cypress.projectRoot)) {
-        throw new Error(`${filepath} is not inside ${configuration.cypress.projectRoot}`);
-    }
     (0, debug_1.default)(`looking for step definitions using ${util_1.default.inspect(configuration.preprocessor.stepDefinitions)}`);
     const filepathReplacement = trimFeatureExtension(path_1.default.relative(fullIntegrationFolder, filepath));
     (0, debug_1.default)(`replacing [filepath] with ${util_1.default.inspect(filepathReplacement)}`);

Versions

Cypress version: doesn't matter but we are on 9.6.0. This has worked since version 6 I think
Preprocessor version: 9.2.1
Node version: 14.15.4

You're right, this does seem unnecessary. I think it originates as a sanity check for when I initially wrote the code. However, I see no reason for why steps should be required to be within the project root.

I've released v10.0.2 which removes the mentioned fix. Hope that helps.

NICE, thanks! That's one patch-package I can remove 🤣


How to get list of trays from a printer?

I am making a project using 'ezprinting' in which there is a necessity to allow users to select a tray from a list of trays and print to it.
I used the Printer.get_printer_attributes() to get all the printer attributes which also contains list of trays. However, the problem is that the response given by different printers(of different manufacturers) are different.
I would like to know if there is a manufacturer independent way of getting list of trays. If that doesn't exist, atleast is there a set of standard formats that every manufacturer has to follow.
Any help would be greatly appreciated. Thank you.

Sorry for the delay in answering.
You are right each manufacturer spits out different formats. I never took the time to try to find patterns and/or reliable ways of extracting such information.
Are you using it with CUPS server or GCP? GCP is being dicsontinued, I plan to remove support for it soon.


Fix issues with TextEntry hotkey branch

Changes

Revert widget changes not related to the select all hotkey (on_resize will be added in https://github.com/pyglet/pyglet/pull/930)
Merge in recent changes from pyglet's master branch
Revert the improperly placed and platform-specific hotkey handling on TextEntry
Add platform-specific select all support
Add MOTION_SELECT_ALL handling in Caret
Add docstring and type annotation to Caret.select_all

tests list:
Winows env:
WSL 版本： <IP_ADDRESS>
内核版本： <IP_ADDRESS>-1
WSLg 版本： 1.0.57
MSRDC 版本： 1.2.4485
Direct3D 版本： 1.608.2-61064218
DXCore 版本： 10.0.25880.1000-230602-1350.main
Windows 版本： 10.0.19045.3393

Mac env:
MBP 2019 MacOS Ventura 13.2
Darwin shenjack.local 22.3.0 Darwin Kernel Version 22.3.0: Thu Jan  5 20:53:49 PST 2023; root:xnu-8792.81.2~2/RELEASE_X86_64 x86_64

function: cmd/ctrl + a can select all in TextEntry of example/gui/widget.py

[x] Windows 10 22H2 Python 3.8.18 (works)
[x] Windows 10 22H2 Python 3.11.4 (works)
[x] Windows 10 22H2 WSL Python 3.11.0rc1 (works)
[x] Windows 10 22H2 WSL Python 3.10.12 (works)
[ ] Mac Python 3.11.4 (not working)
[ ] Mac Python 3.10.11 (not working)
[ ] Mac Python 3.8.10 (not working)

Thank you! This confirms the reports from another user that Mac is currently broken. I'll see if I can find reference from projects on other platforms.
Howeve, it's good to know that Windows works. I've updated the checklist in my comment on the parent PR.


require function won't compile

Hi guys,
I've been trying to compile the function require(stripe, testkey) function for a while now in order to follow the Stripe tutorials for Node, and I've been getting the same compiler error that goes like this:
Module parse failed: Unexpected token (152:24)
[1] You may need an appropriate loader to handle this file type.
[1] |         opts.auth = args.pop();
[1] |       } else if (utils.isOptionsHash(arg)) {
[1] |         const params = {...args.pop()};
[1] |
[1] |         const extraKeys = Object.keys(params).filter(
I'm on OSX Big Sur, but I also had my friend try it on windows, and he ran into the same error.
The error says something about the loader, so I thought I'd post it here, just in case someone knows what's going on with this.
I've already installed stripe with the following commands
npm install --save stripe
npm install --save @stripe/react-stripe-js @stripe/stripe-js

The stripe package is for node, while @stripe/react-stripe-js and @stripe/stripe-js packages are for the browser. Can you confirm you're only using the stripe package in node?

The compile error is in the stripe package. The other two packages are for other functionality, and I do currently also have them installed because it has them in the tutorial. Before I had those browser packages installed, I was also getting the same compile error from the stripe package, though.

Hi @thebigbadbobby,
Sorry for the late reply. It's hard to tell from your error, but it looks like you might be importing our esm module somehow but your compiler step is expecting es5 javascript. It really depends on how your build is set up.
This issue has been open for a while, so I'm going to go ahead and close. Please chime in if you are still having issues and can provide more information about your build configuration.


Specta should have non-default 'derive'/'macros' feature

I also wonder if Specta should be way more decoupled so breaking changes in the main crate less likely given they are the ecosystem redefining one's. Eg. Have 'specta_ts' or 'specta-utils' crate?

We can't do this now because of the impl in the Tauri crate. 😢


How to import DeformConv?

I follow: mmdetection_env.sh to install this lib
there is no problem when run compile.py and python setup.py install
Then, run import mmdet , it works well
But, when I enter /mmdet/ops/dcn/, and run import import DeformConv or import deform_conv,
It displays:
ModuleNotFoundError: No module named 'DeformConv'
or
ModuleNotFoundError: No module named 'deform_conv'
In the file /mmdet/ops/dcn/, there are two .so files:
deform_conv_cuda.cpython-36m-x86_64-linux-gnu.so
deform_pool_cuda.cpython-36m-x86_64-linux-gnu.so
The evns:
pytorch: 1.0.1.post2
python: Python 3.6.2

You cannot run a python interpreter or python script inside a package. You can just go outside the package and use from mmdet.ops import DeformConv.


update ray version in dependency?

Current dependency is ray version 2.2.0. Ray version 2.3.0 is supposed to have fixed a memory leak bug: see https://discuss.ray.io/t/object-store-spilling-terabytes-of-data/8699/5
I don't know if that particular bug has the potential to impact similarity calculations, but it seems important.
I am using ray version 2.6.3 and working to avoid object spillage. Current version of ray is 2.8.1.

Also there is a supposed security vulnerability in ray older than version 2.8.1. The ray folks say that this is basically baloney; but, this is not a good look for us.
I think that this would force us to upgrade the minimum version of Python to 3.9. However, Python 3.8 is end-of-life this year, so, no loss.
Should update the test grid to include the newest versions of Python (now up to 3.13).


to_dense does not preserve dtype in SparseArray

This isn't a huge deal, but it seems a little odd:
In [1]: import pandas as pd

In [2]: a = pd.SparseArray([True, False, False, False, True], fill_value=False, dtype=bool)

In [3]: a
Out[3]:
[True, False, False, False, True]
Fill: False
IntIndex
Indices: array([0, 4], dtype=int32)

In [4]: a.dtype
Out[4]: dtype('bool')

In [5]: d = a.to_dense()

In [6]: d
Out[6]: array([ 1.,  0.,  0.,  0.,  1.])

In [7]: d.dtype
Out[7]: dtype('float64')

I would have expected d to retain the dtype of bool. I can cast down, but I am still wasting 7 bytes per element in the process.

INSTALLED VERSIONS
commit: None
python: 2.7.6.final.0
python-bits: 64
OS: Linux
OS-release: 3.13.0-57-generic
machine: x86_64
processor: x86_64
byteorder: little
LC_ALL: None
LANG: en_US.UTF-8
pandas: 0.16.2
nose: 1.3.7
Cython: 0.22.1
numpy: 1.9.2
scipy: 0.15.1
statsmodels: None
IPython: 3.2.0
sphinx: 1.2.2
patsy: None
dateutil: 2.4.2
pytz: 2015.4
bottleneck: None
tables: None
numexpr: None
matplotlib: 1.4.3
openpyxl: None
xlrd: None
xlwt: None
xlsxwriter: None
lxml: None
bs4: None
html5lib: None
httplib2: None
apiclient: None
sqlalchemy: None
pymysql: None
psycopg2: None

prob a bug, not really well tested and not really a lot of dev support in sparse
pull requests are welcome

I may be speaking too soon, but after skimming the code for this, it looks pretty simple to fix. I'll work on a PR!


Set JTAG ID using the git hash

This PR resolves both #55 (implementation) and #88 (test update).  Once PR #90 is merged I'll sync this PR with those changes and request a reviewer.

I'll add these changes to PR #90 so there is just one PR that has to be reviewed.  Closing this PR as a result.


Chown command in docker file adds a new layer when used to change the ownership of the copied directory/file

Output of docker version:
$ docker --version
Docker version 1.9.1, build a34a1d5/1.9.1

Output of docker info:
$ docker info
Containers: 4
Images: 180
Server Version: 1.9.1
Storage Driver: devicemapper
 Pool Name: docker-202:48-131076-pool
 Pool Blocksize: 65.54 kB
 Base Device Size: 10.74 GB
 Backing Filesystem:
 Data file: /dev/loop0
 Metadata file: /dev/loop1
 Data Space Used: 5.911 GB
 Data Space Total: 107.4 GB
 Data Space Available: 9.915 GB
 Metadata Space Used: 10.95 MB
 Metadata Space Total: 2.147 GB
 Metadata Space Available: 2.137 GB
 Udev Sync Supported: true
 Deferred Removal Enabled: false
 Deferred Deletion Enabled: false
 Deferred Deleted Device Count: 0
 Data loop file: /mnt/docker/devicemapper/devicemapper/data
 Metadata loop file: /mnt/docker/devicemapper/devicemapper/metadata
 Library Version: 1.02.93-RHEL7 (2015-01-28)
Execution Driver: native-0.2
Logging Driver: json-file
Kernel Version: 4.1.19-24.31.amzn1.x86_64
Operating System: Amazon Linux AMI 2015.09
CPUs: 1
Total Memory: 3.679 GiB
Name: ip-10-10-10-22
ID: V25U:OHKU:HIOQ:AW2A:V2HA:7OIB:EA4V:OTYO:CTAA:2SD4:KMXY:O536

Additional environment details (AWS, VirtualBox, physical, etc.):
Steps to reproduce the issue:

Create a docker file as below
FROM tomcat
RUN useradd -d /home/test test
ADD test123 /home/test/test123
RUN chown -R test:test /home/sdpuser/test123
mkdir test123 && cd test123
cp  test123
docker build -t test .

Describe the results you received:
When you do a docker history add the last you see chown is getting duplicated thus adding up 28.72 MB space. Ideally should run as a command but its a new layer of the same size. I figured this when I was building a custom image for my project and creating non root user within my containers.  Is this an expected behavior ?
I have applied a workaround by changing the ownership within the hostOs.
docker history test
IMAGE               CREATED             CREATED BY                                      SIZE                COMMENT
ec6211788d5f        52 seconds ago      /bin/sh -c chown -R sdpuser:sdpuser /home/sdp   28.72 MB
aa53295d188d        58 seconds ago      /bin/sh -c #(nop) ADD dir:f9bbeebffd367903e94   28.72 MB
10c5e8e82d47        41 minutes ago      /bin/sh -c useradd -d /home/sdpuser sdpuser     329.1 kB
cdc78b94a897        6 months ago        /bin/sh -c #(nop) CMD ["catalina.sh" "run"]     0 B
82f58c961431        6 months ago        /bin/sh -c #(nop) EXPOSE 8080/tcp               0 B
34d8147d3641        6 months ago        /bin/sh -c set -x  && curl -fSL "$TOMCAT_TGZ_   12.91 MB
73deaaca356b        6 months ago        /bin/sh -c #(nop) ENV TOMCAT_TGZ_URL=https://   0 B
a1deecf5c68e        6 months ago        /bin/sh -c #(nop) ENV TOMCAT_VERSION=8.0.28     0 B
8384577d6bb9        6 months ago        /bin/sh -c #(nop) ENV TOMCAT_MAJOR=8            0 B
a99360bba5f9        6 months ago        /bin/sh -c gpg --keyserver pool.sks-keyserver   111 kB
7f0c5db847d7        6 months ago        /bin/sh -c #(nop) WORKDIR /usr/local/tomcat     0 B
92bbf8c9f01c        6 months ago        /bin/sh -c mkdir -p "$CATALINA_HOME"            0 B
7d22540101dc        6 months ago        /bin/sh -c #(nop) ENV PATH=/usr/local/tomcat/   0 B
864b08468887        6 months ago        /bin/sh -c #(nop) ENV CATALINA_HOME=/usr/loca   0 B
30d5a52e17ca        6 months ago        /bin/sh -c apt-get update && apt-get install    173.8 MB
75fc529176f0        6 months ago        /bin/sh -c #(nop) ENV JAVA_DEBIAN_VERSION=7u8   0 B
9ca451982f56        6 months ago        /bin/sh -c #(nop) ENV JAVA_VERSION=7u85         0 B
927bc2c0c1e5        6 months ago        /bin/sh -c #(nop) ENV LANG=C.UTF-8              0 B
b38503f73b41        6 months ago        /bin/sh -c apt-get update && apt-get install    680.7 kB
4c0cc976b7bb        6 months ago        /bin/sh -c apt-get update && apt-get install    44.27 MB
1d6f63d023f5        6 months ago        /bin/sh -c #(nop) CMD ["/bin/bash"]             0 B
ef2704e74ecc        6 months ago        /bin/sh -c #(nop) ADD file:3037fa9e903e9ae533   125.1 MB
Describe the results you expected:
Additional information you deem important (e.g. issue happens only occasionally):

Yes, this is expected.
The filesystem has to copy up the the file into the new layer so that it can be chowned.

Thanks for the info

@ameyrk Afaik, chown commond does not introduce additional disk usage when the storage driver is chosen as devicemapper.


Default to vcpkg instead of pkg-config for easy Windows building

After a couple days of fighting link errors when trying to use a version of opencv installed via vcpkg I hacked the build script to use the vcpkg crate to get the opencv paths including header path and was immediately succesful except for having to edit gen_rust.py to add loop to the reserved keywords.
Properly implemented, this would allow easy MSVC builds via:
vcpkg install opencv4[necessary features here]
cargo build

Hi, will it be possible for you to share the resulting code?

This is the main part. I had to rearrange other things to let the probe return the header path instead of defaulting to the bundled ones.
    #[cfg(target_env = "msvc")]
    fn probe_vcpkg(pkg_name: &str) -> Result<Self> {
        let opencv = vcpkg::Config::new().find_package(pkg_name)?;
        let libs: Vec<String> = opencv
            .found_libs
            .iter()
            .map(|lib| lib.file_name().unwrap().to_string_lossy().to_string())
            .collect();
        let version = opencv
            .include_paths
            .iter()
            .filter_map(get_version_from_headers)
            .next();
        let libdir = opencv
            .link_paths
            .first()
            .map(|x| x.clone())
            .unwrap_or_else(|| PathBuf::from(""));
        let header_path = opencv
            .include_paths
            .first()
            .map(|x| x.clone())
            .unwrap_or_else(|| PathBuf::from(""));

        Ok(Self {
            pkg_name: pkg_name.to_owned(),
            libs,
            link_paths: opencv.link_paths,
            framework_paths: vec![],
            include_paths: opencv.include_paths,
            header_path: Some(header_path),
            version: version.unwrap_or("0.0.0".to_owned()),
            prefix: PathBuf::from(""),
            libdir: libdir,
        })
    }

Thanks! If you could do the pull request with all of the changes I'd really appreciate it too.

I'll try to get it cleaned up

Even in the "messy" form it would be welcome, there is no need to actually make it merge ready, I just want to take a look at the changes in their wholeness. I don't have any experience with vcpkg so I might miss some minor detail if I don't have the whole changeset.

Can you please check if vcpkg branch still works for you for hassle-free build with vcpkg? I removed the OPENCV_HEADER_PATH var btw so if you have set it up in the past it won't have any effect.

I can't figure out why at a quick glance, but your version causes link errors without --release

Can you please share the full build log?

cargo clean
cargo build -vv

Hey, @Threep , did you manage to find out what's wrong in the version from vcpkg branch compared to yours?

My power supply broke down and in the time it took to fix it I forgot about it. Sorry for the delay.
All my builds are with buildtime-bindgen but I've either misunderstood how I'm supposed to use the crate or uncovered a bug because this fails:
opencv = { git = "https://github.com/twistedfall/opencv-rust", branch = "vcpkg", default-features = false, features = [ "opencv-41", "buildtime-bindgen" ] }`

But if I build with cd opencv-rust && cargo --no-default-features --features opencv-41,buildtime-bindgen beforehand, this works
opencv = { path = "opencv-rust", default-features = false, features = [ "opencv-41", "buildtime-bindgen" ] }

Also, I missed it in my patch since I only tested with 4.x versions, but the vcpkg uses opencv3 for 3.4 while opencv is an allias for opencv4
build-fails.txt
build-succeeds1.txt
build-succeeds2.txt

Thanks for the update. And you're saying that with your initial changes you don't get linking failure on the first try, right? Will it be possible to send the build log of that too? So cargo clean and cargo build -vv on the branch that you used initially for the pull request.

I get the exact same linking failures with my own branch when it's clean, I was running build on my local copy while tinkering with it and didn't realize that affected it
Clean copy directly from git, my fork or yours, or cloned in advance = Link errors on use
Already built local copy = Links without issue
build-fork.txt

Well, vcpkg integration works then, I'll release it. Linking issue on Windows is not new, but your additional info about successful link on the already built crate is interesting, I'll check it out.

0.27.0 is released

The linking issue should be fixed in 0.28.0


Allow to set Access-Control-Expose-Headers in Cors filter

Regarding to issue https://github.com/yiisoft/yii2/issues/6492

Thanks!


turbo.can_stream() is always False

Hi, I would love to use Turbo in my Flask app, so thanks for working on this!
I don't know if it should work, but it mostly doesn't for me.
If I check for turbo.can_stream(), I always get False. I I return without checking, sometimes it works, and sometimes it doesn't.
Thanks for your help!

The can_stream method returns True only when the client explicitly indicates that it can accept streams. If you look at their documentation, the only time streams are supported in HTTP is in the response to a form submission via POST request.
The other option for streams is to use the push mode, which works over a WebSocket connection outside of the HTTP request/response cycle. This is not currently supported in this package.

oh, thank you for explanation.
any plan on adding Websocket Stream support? :)

I'm thinking about it. The problem is that the WebSocket options for Flask all introduce many complications, like the need to use gevent.


Validate fields in IPPool objects upon creation and updates

IMPORTANT: Please do not create a Pull Request without creating an issue first.
Problem:

Solution:

If a user tries to configure the embedded DHCP server with a designated IP address, the IPPool validator will validate the input. The create/update requests will be rejected if:

The server IP is not within the defined subnet
The server IP is the same as the network IP or broadcast IP of the subnet
The server IP is the same as the router IP (if exists)
The server IP collides with any already-allocated IPs

Related Issue:
harvester/harvester#5065
Test plan:

Install the vm-dhcp-controller chart with the version containing the fix

Create a VM Network (NAD) before creating any IPPool objects

Create an IPPool object with the server IP outside of the subnet (should be rejected by the webhook)
$ cat <<EOF | kubectl apply -f -
apiVersion: network.harvesterhci.io/v1alpha1
kind: IPPool
metadata:
  namespace: default
  name: test-net
spec:
  ipv4Config:
    serverIP: <IP_ADDRESS>
    cidr: <IP_ADDRESS>/16
    pool:
      start: <IP_ADDRESS>
      end: <IP_ADDRESS>
    router: <IP_ADDRESS>
    leaseTime: 300
  networkName: default/test-net
EOF

Error from server (InternalError): error when creating "STDIN": admission webhook "validator.harvester-system.harvester-vm-dhcp-controller-webhook" denied the request: Internal error occurred: could not create IPPool default/test-net because server ip <IP_ADDRESS> is not within subnet

Modify the .spec.serverIP to the same IP as the network IP, broadcast IP, and router IP, then try to create it with the manifest (all should be rejected by the webhook)

Modify the .spec.serverIP to a sane IP address, i.e., within the subnet, but not collide with the network IP, broadcast IP, or router IP, then create it with the manifest (this should be successful)

Create a VM attaching to the VM Network, which should have the vmnetcfg object created under the hood

Check the IPPool object to see which IP address is allocated to the VM's interface (under .status.ipv4.allocated)

Modify the .spec.serverIP of the IPPool object, updating it to the VM's IP (should be rejected by the webhook)

The recent commits address invalid user inputs not only limited to the serverIP field but also other mandatory fields, using CRD validation marks (thanks to @ibrokethecloud) and some custom logic implemented in the validating webhook.

Merge conflict resolved.


python311Packages.pathlib-abc: 0.1.1 -> 0.2.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python311Packages.pathlib-abc is: Python base classes for rich path objects
meta.homepage for python311Packages.pathlib-abc is: https://github.com/barneygale/pathlib-abc
meta.changelog for python311Packages.pathlib-abc is: https://github.com/barneygale/pathlib-abc/blob/0.2.0/CHANGES.rst
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.2.0 with grep in /nix/store/cinxxdznhk2h7nqv324qs5rmmdsr61rm-python3.11-pathlib-abc-0.2.0
found 0.2.0 in filename of file in /nix/store/cinxxdznhk2h7nqv324qs5rmmdsr61rm-python3.11-pathlib-abc-0.2.0

Rebuild report (if merged into master) (click to expand)

16 total rebuild path(s)

16 package rebuild(s)

First fifty rebuilds by attrpath
python311Packages.fastai
python311Packages.pathlib-abc
python311Packages.pathy
python311Packages.spacy
python311Packages.spacy-lookups-data
python311Packages.spacy-transformers
python311Packages.textacy
python311Packages.textnets
python312Packages.fastai
python312Packages.pathlib-abc
python312Packages.pathy
python312Packages.spacy
python312Packages.spacy-lookups-data
python312Packages.spacy-transformers
python312Packages.textacy
python312Packages.textnets

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/cinxxdznhk2h7nqv324qs5rmmdsr61rm-python3.11-pathlib-abc-0.2.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python311Packages.pathlib-abc https://github.com/r-ryantm/nixpkgs/archive/c0aa438ed1d588a0c730eb92942cf94077fccf5b.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/c0aa438ed1d588a0c730eb92942cf94077fccf5b#python311Packages.pathlib-abc

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/cinxxdznhk2h7nqv324qs5rmmdsr61rm-python3.11-pathlib-abc-0.2.0
ls -la /nix/store/cinxxdznhk2h7nqv324qs5rmmdsr61rm-python3.11-pathlib-abc-0.2.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

28 packages failed to build:

python311Packages.fastai
python311Packages.fastai.dist
python311Packages.pathy
python311Packages.pathy.dist
python311Packages.spacy
python311Packages.spacy-lookups-data
python311Packages.spacy-lookups-data.dist
python311Packages.spacy-transformers
python311Packages.spacy-transformers.dist
python311Packages.spacy.dist
python311Packages.textacy
python311Packages.textacy.dist
python311Packages.textnets
python311Packages.textnets.dist
python312Packages.fastai
python312Packages.fastai.dist
python312Packages.pathy
python312Packages.pathy.dist
python312Packages.spacy
python312Packages.spacy-lookups-data
python312Packages.spacy-lookups-data.dist
python312Packages.spacy-transformers
python312Packages.spacy-transformers.dist
python312Packages.spacy.dist
python312Packages.textacy
python312Packages.textacy.dist
python312Packages.textnets
python312Packages.textnets.dist

4 packages built:

python311Packages.pathlib-abc
python311Packages.pathlib-abc.dist
python312Packages.pathlib-abc
python312Packages.pathlib-abc.dist

Maintainer pings

Change looks good to me.

How does this look good, when it regresses 28 packages?

How does this look good, when it regresses up to 28 packages?

And how many of them are new? spacy is consuming pathy which got broken and and spacy is already notoriously broken.

https://hydra.nixos.org/eval/1805225?filter=textacy&compare=1805217&full=
https://hydra.nixos.org/eval/1805225?filter=spacy&compare=1805217&full=
https://hydra.nixos.org/eval/1805225?filter=fastai&compare=1805217&full=

Also, see https://github.com/NixOS/nixpkgs/pull/299791#issuecomment-2025670910 for the difference if pathlib-abc will be reverted. According to nix-review not that many after all.


Add simple BMP encoder.

Using the Wikipedia page on BMP files and the JPEGEncoder as a guide, I've implemented simple RGB BMP encoding.  RGBA images are simply stripped of their alpha channel as transparent bitmaps aren't widely supported.
In addition to the two unit tests, I've also manually validated the fractal example from the README works when Luma -> Rgb, ".png" -> ".bmp", and image::PNG -> image::BMP.

Shameless ping to @nwin and/or @ccgn for a review.

Looks good, thank you!


[Feature] Add Transaction Record

Adds the ability to record a new "transaction" entity
Misc.
Simplify imports with a jsconfig.json
UI
Style example I'm aiming to match

Adds routing for new "transactions" page,
Adds RouterConfig with ROUTER_PATH_CONSTANTS,
Adds 404 redirect to Home behaviour,
fleshes out the basics of the Icarus UI form design and components,

Form prototype created for adding entity,

Form with intended fields and mock text replaced,

Potential idea to use AngularJS for DI and services,
https://codeburst.io/how-to-hook-reactjs-to-your-existing-angularjs-1-x-app-5ab1ac59c0c1

Updated form designs,

Routing and nav bar added
https://user-images.githubusercontent.com/26985949/194760615-0924baf2-f31c-4a06-ad9d-b9c223902ffd.mp4
Server Data Interaction for Form and List views added
https://user-images.githubusercontent.com/26985949/194760528-51784f38-0663-4683-8d4c-59217ee6ceb9.mp4


about com.google.android.gms.permission.AD_ID Permission

on google console i am being told that if my app uses advertising id and targets android 13+ i need to declare the permission
com.google.android.gms.permission.AD_ID
i am assuming if i am using this admob plugin, this applies to me too right? if so, does this plugin implement the permission for me automatically, or should i add it manually?

i found out there was already an issue about this, apologies!


Create README.md

Create a proper README.md, give proper description, instructions on how to use a local copy of this repo, contributors, etc.

@deepesh611  Go ahead and work on it.

Sure, however I will need a bit of your help.
It would be helpful if u could give a list of points that sum you your project. It may be the best features, the target audience, anything that you can think of, that you would like the public to see.
As a programmer, I could list some things by reading the repo contents, however there is a quite high chance that I may miss some point, and so I turn to the repo owner for help.

Yeah you can find a pdf file in repo most of data can be found there if you still need something do tell me.

Yeah you can find a pdf file in repo most of data can be found there if you still need something do tell me

I have submitted a PR, pls have a look and make sure to verify the information adn give a feedback


Fix/gh 176 keep default kafka message binding when null

Closes https://github.com/springwolf/springwolf-core/issues/176


Add admin.js Script to JSHint Task in Gruntfile

The admin.js script is not listed in the JSHint task. This pull request adds it.

+1


Put packer log in expected artifacts path

Putting packer log under ova folder.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/override pbnj-tooling-presubmit
/override cluster-api-provider-cloudstack-tooling-presubmit
/override cloudstack-cloudmonkey-tooling-presubmit


[Feature]: e2e test with kubernetes for xline cluster of 3 node.

Description about the feature
/assignme
Code of Conduct

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

/assignme

/help


add: gallery app in react

This PR adds a Gallery app written in React .

@vaishsabu Please do another pull request

should i make any changes?


add postgres

[x] add PostgreSQL

Good job! I started merge you changes! :D

Thanks and thank you that extension is awesome! :)

I done. Please download my change from branch
https://github.com/Bajdzis/vscode-database/tree/dev-postgres and testing postgres server.

@Bajdzis LGTM! I found some issues and fixed (PR #6.)

@Bajdzis Postgres server test, all green.


Align on treatment of global variables (feedback 1-20)

[id: 1-20], [CONCEPT], [FUND] The current approach to define workflow inputs is to use global variables
as shown in the Parameters help section, right? Personally, I was a bit surprised by this approach. Before
that I had played around with dewret and the way I intuitively expected to define workflow input was to
define global variables and pass them as arguments to tasks, e.g.
import sys
import yaml
from dewret.tasks import task, run
from dewret.renderers.cwl import render
some_number = 3
@task()
def increment(num: int) -> int:
  return num + 1
result = increment(num=some_number)
workflow = run(result)
cwl = render(workflow)
yaml.dump(cwl, sys.stdout, indent=2)

Which obviously does not work, i.e. the produced output reads
class: Workflow
cwlVersion: 1.2
inputs: {}
outputs:
out:
label: outoutputSource: increment-012ef3b3ffb9d15c3f2837aa4bb20a8d/out
type: int
steps:
increment-012ef3b3ffb9d15c3f2837aa4bb20a8d:
in:
num:
default: 3
out:
- out
run: increment

Hence, it seems that currently the only intended way of defining workflow input is using global
variables which sneak into a function (as context). I know this is very subjective (so, please no offence!)
but I strongly reject the concept of context aware functions because they are violating the
encapsulation and the very idea behind a function, i.e. that a function should be a stateless and entirely
encapsulated entity whose result should only depend on the explicit input.

Start discussion #20

The desired behavior in the example above is now implemented with #27


Please merge, These small changes will solve many confusions

Please

@CodeWithSwastik 🙏🏻

@FalseDev ^

I do not have write access to the repository, I've only contributed 😅


improvement: support generating sign in tokens on register

Supports: https://github.com/team-alembic/ash_authentication_phoenix/pull/274

Thanks :) Once this releases we should depend on it for the other one.


⚠ Bofrost.be has degraded performance

In 2df83a2, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1785 ms

Resolved: Bofrost.be performance has improved in 2a66ff0.


Couldn't move back to middle while in options mode

Might couse of this line, if I change it to:
    .candlestick-wrapper.candlestick-contents .candlestick-bg .candlestick-nc {
        left: 100px;
    }

could fixed this, I would like to send a PR if this is acceptable to you.

You use the 'options' mode, and you can't move to middle with click ?

Yes, because this three line overwrite the above default size left property at this line, so the candlestick-nc is out of this box while in default options mode.
could fix it by replace origin one:
.candlestick-wrapper .candlestick-bg .candlestick-nc {
    left: 100px;
}

with:
.candlestick-wrapper.candlestick-contents .candlestick-bg .candlestick-nc {
    left: 100px;
}

which I think is just a typo.

Thanks for contributions


Can i add whole docker files into docker folder

@erikspears could you assign this task to me

@Durgesh4993 assigned...what is it you want to accomplish with this task?  Just curious as to the benefits of your approach, if you could say something about that.  Thanks!

@erikspears Thank you, for giving me opportunity to do this task

Closing this out.


License

Thank you very much for making the dataset and evaluation code available!
As there is no license file, I was wondering under which license the evaluation code is published?

Thanks for pointing that out, I have added the License.

Thanks for pointing that out, I have added the License.


Feature/status page

First version of status page

@feikesteenbergen can you post a screenshot? I'm curious to see how it looks like!

@feikesteenbergen I was wondering why this is closed?  Are you not interested in pursuing it?


Compile fail at 100% on Ubuntu MATE

Compiler fails at 100%
Ubuntu MATE 15.10
Pretty OS fresh install - one week ago
Am I missing development libraries?
[ 96%] Building CXX object es-app/CMakeFiles/emulationstation.dir/src/views/gamelist/DetailedGameListView.cpp.o
[ 97%] Building CXX object es-app/CMakeFiles/emulationstation.dir/src/views/gamelist/IGameListView.cpp.o
[ 98%] Building CXX object es-app/CMakeFiles/emulationstation.dir/src/views/gamelist/ISimpleGameListView.cpp.o
[ 99%] Building CXX object es-app/CMakeFiles/emulationstation.dir/src/views/gamelist/GridGameListView.cpp.o
[100%] Building CXX object es-app/CMakeFiles/emulationstation.dir/src/views/SystemView.cpp.o
[100%] Building CXX object es-app/CMakeFiles/emulationstation.dir/src/views/ViewController.cpp.o
Linking CXX executable ../emulationstation
CMakeFiles/emulationstation.dir/src/GamelistDB.cpp.o: In function populate_recursive(boost::filesystem::path const&, std::vector<std::string, std::allocator<std::string> > const&, boost::filesystem::path const&, SystemData const*, sqlite3*, sqlite3_stmt*)': GamelistDB.cpp:(.text+0x1530): undefined reference toboost::filesystem::path_traits::dispatch(boost::filesystem::directory_entry const&, std::string&)'
GamelistDB.cpp:(.text+0x156c): undefined reference to boost::filesystem::path_traits::dispatch(boost::filesystem::directory_entry const&, std::string&)' GamelistDB.cpp:(.text+0x159a): undefined reference toboost::filesystem::path_traits::dispatch(boost::filesystem::directory_entry const&, std::string&)'
CMakeFiles/emulationstation.dir/src/main.cpp.o: In function main': main.cpp:(.text.startup+0x30): undefined reference toboost::locale::generator::generate(std::string const&) const'
../libes-core.a(ThemeData.cpp.o): In function ThemeData::getThemeSets()': ThemeData.cpp:(.text+0x6ae): undefined reference toboost::filesystem::path_traits::dispatch(boost::filesystem::directory_entry const&, std::string&)'
ThemeData.cpp:(.text+0x6d6): undefined reference to `boost::filesystem::path_traits::dispatch(boost::filesystem::directory_entry const&, std::string&)'
collect2: error: ld returned 1 exit status
es-app/CMakeFiles/emulationstation.dir/build.make:872: recipe for target 'emulationstation' failed
make[2]: *** [emulationstation] Error 1
CMakeFiles/Makefile2:301: recipe for target 'es-app/CMakeFiles/emulationstation.dir/all' failed
make[1]: *** [es-app/CMakeFiles/emulationstation.dir/all] Error 2
Makefile:136: recipe for target 'all' failed
make: *** [all] Error 2
marc@Pi:~/EmulationStation$

Resolved.
sudo apt-get libboost-all-dev

I am getting this exact error.  I ran sudo apt-get install libboost-all-dev but the error is still occurring.

Hello, I'm having this issue too. Surely installing all dev libraries won't help. The error message is very clear. Did anyone manage to solve this?

I backed out of Ubuntu Mate and went with Raspbian. Sucks though, liked Ubuntu personally. Nice interface, even if a small performance penalty.
Marc

On Jul 28, 2017, at 6:05 AM, Luca<EMAIL_ADDRESS>wrote:
Hello, I'm having this issue too. Surely installing all dev libraries won't help. The error message is very clear. Did anyone manage to solve this?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or mute the thread.


Optimize dockerfile

Switching these two commands will allow us to take advantage of cached
cpanm command when deploy.pl script changes

:+1:


[Feature]: API can get process disk usage stats

Blockers/Dependencies
Immediately available solution depends on disk usage logic from metric-proxy. See the disk usage fetcher.
Otherwise this is blocked on an exploration for updating controller logic to surface this information on the process/statefulset itself to avoid auth complexity in the api.
Background
As a API client
I want to view stats for given cf processes
So that I can observe my applications
Acceptance Criteria
Happy Path
GIVEN cluster with cf-k8s-controllers, api shim, and metrics server installed
WHEN I make a request that looks like
curl "https://api.example.org/v3/processes/[guid]/stats" \
-X GET \
-H "Authorization: bearer [token]"

THEN I get a response that looks like
HTTP/1.1 200 OK
Content-Type: application/json

{
  "resources": [
    {
      "type": "web",
      "index": 0,
      "state": "RUNNING",
      "usage": { "time": "2016-03-23T23:17:30.476314154Z", "cpu": 0.00038711029163348665, "mem": 19177472, "disk": 69705728 },
      "host": "<IP_ADDRESS>",
      "instance_ports": [],
      "uptime": 9042,
      "mem_quota": 268435456,
      "disk_quota":<PHONE_NUMBER>,
      "fds_quota": 0,
      "isolation_segment": null,
      "details": null
    }
  ]
}

Dev Notes
API docs: https://v3-apidocs.cloudfoundry.org/version/3.113.0/index.html#get-stats-for-a-process
Note on RBAC:

The metrics api group is separate from corev1, so the existing process/pod RBAC is likely insufficient, we will have to determine the necessary RBAC on the cluster node for access to disk usage
For direct node access requirements, we would need a privileged client in the api
For propagated information on the process/statefulsets, we can use a user client in the api

Fallback:

If metrics server is not available, continue to return the empty usage block
Do we also include a runtime log for debugging as well?

Pretty sure this was fixed in https://github.com/cloudfoundry/korifi/pull/2081!


Fix for bug in Python TLVWriter

-- Fixed a bug in the Python TLVWriter implementation caused all instances of the class to use the same output buffer.

This change addresses issue 357.


Imports do not appear to be finding functions

We are attempting to convert an old flex app to royale.  We have the newest nightly build however there are still build errors which seem to indicate either we are not accessing the right directories or have something set wrong.  Examples of errors.
Type was not found or was not a compile-time constant: URLRequest.
:
: 		 	 var u:URLRequest = new URLRequest("http
Error: Call to a possibly undefined method navigateToURL.
:
: 		 	 navigateToURL(u,"_blank");
Newer to this type of software so some guidance is appreciated.

Can you provide more info? I understand you downloaded latest nightly build Royale SDK, so you're up to date. That's good. Then how are you building your app? ANT? Maven? IDE?. Also a sample of the code you're trying to compile will help.
Thanks

Using moonshine latest version.  Doing a build as javascript.  Top headers are as follows

<s:Application xmlns:fx="http://ns.adobe.com/mxml/2009" xmlns:mx="library://ns.adobe.com/flex/mx"
creationComplete="initApp()"
xmlns:s="library://ns.apache.org/royale/spark"
xmlns:js="library://ns.apache.org/royale/basic"
xmlns:veh="*">
Area of one of the code areas is as follows.
public function openPLTWindow(event:MouseEvent):void {
var u:URLRequest = new URLRequest("http://                                               ");
navigateToURL(u,"_blank");
}  Sorry cant include url but you get the idea  I have the royale version defined to the project and in the general settings for moonshine.  I know under mxroyale urlrequest is there but not sure how to make sure its is reference when the compile occurs.

It is not going to work. Royale doesn't provide you straigh forward compilation. Please read about Emulation Component you can find all information on Wiki in this respository. - This is the closest thing which you can help you went to path which you are trying to.

I will take a look.


fix Template not found exception

[secuirty]  index.html file can not be found

Kindly explain on how you intend to solve this issue by this changes. Currently for production build, the build of react app is served through flask frontend, due to which it contains this route. But as the build is not there during development, so it renders this issue. Kindly submit an issue before opening a pr. :)


Implement all "Subgardens" for Jena

Today everything worked well for the two subgardens "Ökologie" and "System". For next week it would be great to have all subgardens included:
"Alpinum" -> Aegopodium podagraria - Primula vulgaris (44-59)
"Wasser/Goethe/Sand/Haus" -> Acorus calamus - Silphium integrifolium (120-144) and Arnica longifolia - Hepatica nobilis  (1-5).... Please all in alphabetical order

The new subgardens and specieslists have been added. I also made the specieslist alphabetical for all gardens. If you prefer the alphabetical order for only the new subgardens, please let me know.

Hey, Robert. I now, undo my changes and the species list will not be retrieved alphabetical by default. But I am adding a button for the collector to choose if they want to view the species in alphabetical order or numerical order (based on order).

By default, the plant list will be displayed in numerical order. But if you click the button next to "Plant" label, the plants will be listed in alphabetical order.

After clicking on alphabetical order, the button will be changed and you can go back to numerical order by clicking on it again.

Thanks!:)

Right now I type in the collections from January for the "Alpinum" and the "Wasser/Goethe/Sand/Haus" . For the second one everything works fine but for the "Alpinum" there is no connection to the cloud possible.

Robert, thanks again for reporting this issue. The problem is that the old code was relying on (sub)garden names to be unique and therefore was working fine. But now as we have 2 System and 2 Alpinum gardens; one for Tübingen and one for Jena, I had to change the code a bit. To avoid any problems, you need delete the older collections from the devices (because the data structures do not match anymore).
However, you will not have this issue anymore. I am sorry for the inconvenience.

Hello, now everythin worked!:)

That's great!


Proof of concept to support deep parsing of query string params

Need to add tests but this is a basic proof of concept for a fix for https://github.com/zeit/next.js/issues/2530
Before i finish this off completely, is this an acceptable solution? i wasn't sure about bringing in a new dependency?

I'm not sure if we want to do this inside of next, since qs is a pretty big library

Lets close this. qs is too big to pull in for this feature. Users can parse deep-nested queries themselves using the full url. Which we do provide.


feat(curriculum): Add links for the MP3 files

Had to go through the trouble of finding audio clips in the source code of the sample project to finish mine.
Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #XXXXX

Hi @acdavit. Congrats on your first pull request (PR)! 🎉
Thank you for your contribution to the page! 👍
We are happy to accept these changes and look forward to future contributions. 📝


App to Record, Playback, and View Sound

#5

Record sound.
Play back sound.
View details of the sound wave.

[ ] Support sound recording - https://developer.mozilla.org/en-US/docs/Web/API/MediaStream_Recording_API/Using_the_MediaStream_Recording_API#basic_app_setup

[ ] Begin taking on the following model in the form json file:

{
  "project": "react-js",
  "components": [
    {
      "type": "button",
      "actions": [
        {
          "event": "onclick"
        }
      ]
    }
  ]
}


Better tap detection

Improves the false positive rate on tap detection and exposes a debugging parameter on the GUI in dev mode (number of trackers over threshold).
this PR needs https://github.com/SlimeVR/SolarXR-Protocol/pull/118 merged first

Improves the false positive, I read it as makes more false positives happens XD


Render search results on map

When trying to search on the map, I want to be able to actually see the search results.
Pre-Context:

Search results are already available (a search query has been entered) in the code, in a similar format as to how they are available to the search result page
There is already code for rendering points of interests as markers and for rendering a single room-result with the preview popup

Post-Context:

Linking to Detailpages is already possible and the user will want to do it to make sense of search results.
In the future there might be automatic focussing of the map or filtering of the visible pins for a better visual result.

Acceptance Criteria:

[ ] For each search result a pin and the room name are rendered

adjusted acceptance

[ ] pins / labels are rendered for all results that have location data and omitted for all others


[bug] tauri build modifies Cargo.toml?

Describe the bug
Running stat src-tauri/Cargo.toml && pnpm tauri build --bundles none > /dev/null 2> /dev/null && stat src-tauri/Cargo.toml
I see the mtime for my Cargo.toml changes after pnpm tauri build. If I do the same with only cargo build --release, the mtime doesn't change, so I think it must be in Tauri.

My beforeDevCommand and beforeBuildCommand are empty strings, so I don't think anything else in my build pipeline should be modifying the Cargo.toml manifest.
Reproduction
I can go back and make a repro repository, I'm just curious if this is intended behavior and if there's anything I can do to disable it.
Expected behavior
I expect Cargo.toml to not change during the build process.
I'm trying to embed the Git version in my final exe and for some possibly Windows-specific reason, this modification is causing the Git version to show up as dirty / modified
Full tauri info output
[✔] Environment
    - OS: Windows 10.0.22621 X64
    ✔ WebView2: 120.0.2210.144
    ✔ MSVC:
        - Visual Studio Build Tools 2019
        - Visual Studio Community 2022
    ✔ rustc: 1.75.0 (82e1608df 2023-12-21)
    ✔ cargo: 1.75.0 (1d8b05cdd 2023-11-20)
    ✔ rustup: 1.26.0 (5af9b9484 2023-04-05)
    ✔ Rust toolchain: 1.75.0-x86_64-pc-windows-msvc (overridden by 'C:\cygwin64\home\User\projects\firezone\rust\rust-toolchain.toml')
    - node: 20.10.0
    - pnpm: 8.14.3
    - npm: 10.2.3

[-] Packages
    - tauri [RUST]: 1.5.4
    - tauri-build [RUST]: 1.5.1
    - wry [RUST]: 0.24.7
    - tao [RUST]: 0.16.5
    - @tauri-apps/api [NPM]: 1.5.3
    - @tauri-apps/cli [NPM]: 1.5.9

[-] App
    - build-type: bundle
    - CSP: unset
    - distDir: ../src
    - devPath: ../src

Stack trace
N/A

Additional context
I searched in issues and docs but of course "Cargo.toml" and even "manifest" returned a lot of false positives, so maybe I missed something.

I think it might be for this feature injection thing: https://github.com/tauri-apps/tauri/blob/00e1567584721644797b587205187f9cbe4e5cd1/tooling/cli/src/interface/rust/manifest.rs#L299-L310

yep, that's it. You can disable it by using workspace inheritance for the tauri dependency as a workaround.
We should add a check for this though, i think we have that for the plist files too already.

For the Tauri dep itself, e.g. tauri = { workspace = true }?
For anyone else who runs into this, it ultimately turned out to be a problem with line endings, which I solved (assuming the PR is merged) by adding a .gitattributes file with * -text: https://github.com/firezone/firezone/pull/3432/files
I still don't love rewriting Cargo.toml, but the rewrite was actually correct, it's just that Github had it out on our Windows runner as CRLF and then the toml_edit crate must have rewritten it as LF.
I wish CRLF would just disappear, I'm pretty sure modern Windows tools all handle LF just fine :/

For the Tauri dep itself, e.g. tauri = { workspace = true }?

yup
Okay, the check would probably fail on line endings (or maybe we have that check already and this is exactly what's happening 🤔)

I wish CRLF would just disappear, I'm pretty sure modern Windows tools all handle LF just fine :/

Same :(

The rewriting Cargo.toml with LF on Windows is annoying. Is there a way to disable the rewriting feature?

https://github.com/tauri-apps/tauri/issues/8711#issuecomment-1917855313

In my Cargo.toml, I have the following:
[build-dependencies]
tauri-build = { version = '2.0.0-beta', default-features = false, features = ['codegen', 'config-toml'] }

[dependencies]
tauri = { workspace = true }

When build, it is still changing the file to LF. Am I doing it wrong?

maybe try the same with tauri-build 🤔

Having this similar issue now: https://github.com/firezone/firezone/pull/6597
I'd like to use the notification feature on Linux but disable it on Windows, since I use tauri-winrt-notification there.
Is that possible?


Support Equivalence failure cache by template-uuid

Is this a BUG REPORT or FEATURE REQUEST?:
/kind feature
Description:
For batch workload, there're several "similar" tasks in one job; if one task failed to fit one node, the other tasks will have the same result (except pod affinity/anti-affinity). In kube-batch, we don-t know which tasks are similar, but we can support customized indexed, e.g. an annotation of task template uuid.

xref https://github.com/kubernetes/kubernetes/issues/72322

NOTE: the result of pod affinity/ant-affinity can not be re-used, let's handle other predicates in v0.4 .

I would like to work on this issue. Per my understanding, we have to wait for the upstream issue kubernetes/kubernetes#72322 to be fixed, right？
And the change will be adding a map to the session.  The map will cache the result that if a template-uuid fits a node.   The map will be used in actions like allocate to improve the performance.

It's not necessary to wait for kubernetes/kubernetes#72322 ; we can have that feature here, and propose it to upstream :)

And the change will be adding a map to the session. The map will cache the result that if a template-uuid fits a node. The map will be used in actions like allocate to improve the performance.

Yes, we need to add a map in session; and this map is used by PredicatesFn directly, we do not need to change any actions :)

we can have that feature here, and propose it to upstream

So we will calculate the pod-template-hash for the pod.  How do we store the hash?  I can come up with two approaches:

Set it as the pod's annotation, and call API to update the pod, maybe in SchedulerCache.AddPod. So that we don't have to re-calculate the hash everytime.
Add a new field in TaskInfo, calculate and set the field in NewTaskInfo.

Which one do you prefer? Or any better approach?

So we will calculate the pod-template-hash for the pod. How do we store the hash? I can come up with two approaches:

Calculating hash is really heavy, prefer to let operation/controller set an annotation for it. In kube-batch, only the pods from the same PodGroup will check eCache.

And we need to pay attention on pod affinity/anti-affinity.

@zionwu , thanks very much! Please continue your work :)

I think it is a good solution to Add a new field in TaskInfo, calculate and set the field in NewTaskInfo. Although Calculating hash is really heavy, we only calculates it once when we put it to TaskInfo.

regarding "hash", how to resolve conflict?

@k82cn In my option the conflict is very rare, even it happens, kubelet could deal with it well.

kubelet could deal with it well.

if hash conflict, scheduler may not bind pod.

kubelet could deal with it well.

if hash conflict, scheduler may not bind pod.

The hash is stored in TaskInfo, it is just used for equivalence cache. We do not need to add it to pod.

/sig apps

@k82cn @hex108 Do we have a conclusion on how to get the hash?  do we let controller to set it as annotation or calculate it by kube-batch?
I also checked the latest implementation  of eCache in default scheduler,  it is using the UID of pod's controller as hash :
func GetEquivHash(pod *v1.Pod) types.UID {
	ownerReferences := pod.GetOwnerReferences()
	if ownerReferences != nil {
		return ownerReferences[0].UID
	}

	// If pod's UID of controllerRef is nil, return nil.
	return ""
}

In my opinion this is also a good approach, even for controller with multiple pod specs, like tf-operator. For TFJob, if one of its pod specs is unschedulable, the whole job is unschedulable and we don't have to schedule other pod spec of this job.
What do you think of this approach?


fix: remove encoding from password file reads

The password files are opened in binary mode so an encoding argument isn't necessary and causes the script to crash.
Fixes #2

Hi @brycelowe , sorry for the delay here. If you could rebase this branch of of the current head of devel and resign your commit with your GPG key then I can get this merged no problem!

Hey @enpaul, I've gone ahead and rebased against devel and signed it with my GPG key.  Please let me know if you need anything else!


Application Review

As
applicant
I want to
see the important information for my application before I confirm it
in order to
double check the information
Tasks
Acceptance criteria

After submitting the application, the applicant sees his / her personal profile information and below the current application

The age is replaced by the birthday

@tobiduer @hpi-swt2/scrum-master-2016 is this already contained in another user story?

@bjrne: I do not think so. Did you find one?

@tobiduer still the case?

has been done in #257

@tobiduer @bjrne Please close issues not until PR is merged and approve by PO

@Baschdl: This issue is a duplicate ;)


declaration sorter missing in official documentation

it would be good to list this module in documentation. It was breaking svg fallback rules by reordering background position rule. Found mention only in changelog.
cssDeclarationSorter: false
helped.

Done http://cssnano.co/guides/optimisations/ and http://cssnano.co/guides/optimisations/
Also can you create new issue around bugs in cssDeclarationSorter?


Wrong identification of data

The examples (documentation is old) and the class use the word "username" BUT sometimes return de Display Name and others the Nick Name and it should be using only the Twitch ID. I know that is an irc protocol problem but at least the vars should be named properly to be able to use de Twitch API to retrieve the missing data (im done all this and there is data conflicts)

In some places the display name has been used when the login name is available anyways. This is all old code and I don't know if it's right to change it to the login (if even available) except for a major update. You're free to make a PR to tmijs/docs to reflect the current values.


Add LtiAdvantage submodule, update LineItems view

Includes changes from the following downstream PRs:

Add LtiAdvantage submodule by willwm · Pull Request #1 · learningcom/LtiAdvantageTool
Fix error when platform doesn't return line items. Add a way to submi… by steelheaddigital · Pull Request #3 · learningcom/LtiAdvantageTool
Retrieve and display any context members on load by steelheaddigital · Pull Request #4 · learningcom/LtiAdvantageTool
Retrieve line item data on load by steelheaddigital · Pull Request #6 · learningcom/LtiAdvantageTool

Hey learningcom team, I am new to LTI and edtech was wondering if this is actually up to date and up to standards of the LTI 2.0/LTI advantage given that the readme of both this and your fork say its out of date.

           var resultsUrl = $"{LtiRequest.AssignmentGradeServices.LineItemUrl}/{Constants.ServiceEndpoints.Ags.ResultsService}";             var resultsResponse = await lineItemClient.GetAsync(resultsUrl);             var resultsContent = await resultsResponse.Content.ReadAsStringAsync();             var results = JsonConvert.DeserializeObject<ResultContainer>(resultsContent);             Results = results;
in Tool.cshtml.cs, this seems to be throwing a json conversion error, anyone else have this issue?


FIX-#4570: Replace np.bool -> np.bool_

I was getting:
  DeprecationWarning: `np.bool` is a deprecated alias for the builtin `bool`. To silence this warning, use `bool` by itself. Doing this will not modify any behavior and is safe. If you specifically wanted the numpy scalar type, use `np.bool_` here.
  Deprecated in NumPy 1.20; for more details and guidance: https://numpy.org/devdocs/release/1.20.0-notes.html#deprecations

I assume that this is the correct substitution, instead of changing to a simple bool, but I don't understand what this code does here, so I may be wrong.
If this looks good I can do the final TODOs in this checklist
What do these changes do?

[x] commit message follows format outlined here
[x] passes flake8 modin/ asv_bench/benchmarks scripts/doc_checker.py
[x] passes black --check modin/ asv_bench/benchmarks scripts/doc_checker.py
[x] signed commit with git commit -s
[x] Resolves #4570
[x] tests added and passing
[x] module layout described at docs/development/architecture.rst is up-to-date
[x] added (Issue Number: PR title (PR Number)) and github username to release notes for next major release

@devin-petersohn Thanks, I think I updated the branch correctly. I didn't do the task:

added (Issue Number: PR title (PR Number)) and github username to release notes for next major release

because you are saying that is autogenerated? Or I need to do that?

@NickCrews, we should probably replace all entries of np.bool in the codebase (search for np.bool -> replace all to np.bool_). Would you be able to handle that?

@YarShev That followup commit replaced all the other instances of np.bool. But I think in those cases they should get replaced with just bool. Double check me.

because you are saying that is autogenerated? Or I need to do that?

Not exactly, sorry for not being clear. We are working toward that, but for now the message also needs to be manually added here: https://github.com/modin-project/modin/blame/master/docs/release_notes/release_notes-0.16.0.rst#L9

Added entry to release notes! That last CI run failed, though it looked unrelated to this PR? IDK we will see this new run again.
I will note, as a new contributor the number of hoops I have to jump through, as compared to contributing to say pandas, is annoying. Autogenerated release notes, and looser requirements on how to format commit messages and issue numbers etc would probably make me more likely to contribute in the future. Just my two cents, I'm sure there are reasons for the structure!

Added entry to release notes! That last CI run failed, though it looked unrelated to this PR? IDK we will see this new run again.
I will note, as a new contributor the number of hoops I have to jump through, as compared to contributing to say pandas, is annoying. Autogenerated release notes, and looser requirements on how to format commit messages and issue numbers etc would probably make me more likely to contribute in the future. Just my two cents, I'm sure there are reasons for the structure!

Hi @NickCrews! That makes a lot of sense, thank you - we'll be sure to keep that in mind moving forward! One of the reasons we like to lint commits is so that the commit history on the main branch remains clear + descriptive!
I've approved the CI to run, and look forward to merging when everything is green!

@RehanSD I just had to force-push again to fix a error in the docs build, thanks!

@YarShev That followup commit replaced all the other instances of np.bool. But I think in those cases they should get replaced with just bool. Double check me.

Hi @NickCrews quick question - why are we replacing those instances with bool instead of np.bool_? I did some checking, and it seems that bool and np.bool_ behave similarly when doing dtypes checking (i.e. df.dtypes == bool and df.dtypes == np.bool_ return the same result), but np.bool_ != bool - so unless there's a compelling reason to swap to bool, I'd prefer sticking with np.bool_, in case there are some edge cases we're missing!
(I've also approved the run again)!

@RehanSD, thanks, that is a great question! Some info to be considered - https://stackoverflow.com/questions/55905690/how-exactly-does-the-behavior-of-python-bool-and-numpy-bool-differ.

I made the choice between going to bool vs np.bool_ based on:

where possible, use bool. I figured anywhere we could be basic python and not doing stuff with numpy the better. numpy recommends using bool, and I figured it would be the one users are more likely to use. These dtype checks seemed like prime use cases where someone would use bool instead of np.bool_. Per your tests you did above (nice, I didn't even bother doing that explicitly, thanks!), I think that is even more evidence this is an OK thing to do. Also, I read numpy's release notes as saying that going to bool in these cases is fine in cases where we are only interacting with the public API of numpy.
In the cases where I kept np.bool_ , it was because it seemed like we were getting into internals, doing some meta-programming to create new types and methods etc. I thought the distinction was more important there so I wanted to let sleeping dogs lie.

Hopefully that reasoning makes sense? Other things I'm missing? I will change everything to np.bool_ if you tell me, but I think I still prefer the way the PR is now.
CI failures still look unrelated? These same workflows are failing on main.

I think that makes sense to me - but that begets the broader question, why use np.bool_ at all? I'm fairly certain that we could just use bool everywhere - the only benefit from using np.bool_ (as far as I can tell) is that it is more memory efficient than bool. If we do decide to use np.bool_ I think it would be good to document why we made that decision (e.g. a comment that goes something like The following APIs use np.bool_ to maintain backwards compatibility/be more memory efficient/etc.
Thank you for pointing out the CI errors - I'll take a closer look at those later today/tomorrow!

Just checked and the errors are due to a dependency bug we are currently trying to resolve. #4568 is one particular solution - once we've got something merged for that, you'll be able to pull from master and have CI actually run!

Yeah that's true, and I think in a nice simple world we DO only use bool. I'm not sure what the implications are. I just started diving into the internals of what isin = Map.register(pandas.DataFrame.isin, dtypes=np.bool_) actually does, but then I stopped. Too complicated for me to put that much effort in, someone who is more familiar will have to speak to that.
I agree a comment would be good. At my current level of understanding I think I would say something like Perhaps we could just use bool here?, I dont want to pretend I know more than I do, or it might keep the next maintainer from actually making a change that's good.

Just ran a quick test, which supports the idea that we could probably just swap np.bool_ with bool everywhere - the following code snippet seems to indicate that pandas treats np.bool_ as bool in the backend:
In [1]: import pandas as pd; import numpy as np

In [2]: df = pd.DataFrame([[True, False, True]]*1000, columns=["A", "B", "C"], dtype=np.bool_)

In [3]: df.dtypes
Out[3]:
A    bool
B    bool
C    bool
dtype: object

In [4]: df.info()
<class 'pandas.core.frame.DataFrame'>
RangeIndex: 1000 entries, 0 to 999
Data columns (total 3 columns):
 #   Column  Non-Null Count  Dtype
---  ------  --------------  -----
 0   A       1000 non-null   bool
 1   B       1000 non-null   bool
 2   C       1000 non-null   bool
dtypes: bool(3)
memory usage: 3.1 KB

In [5]: bool_df = pd.DataFrame([[True, False, True]]*1000, columns=["A", "B", "C"])

In [6]: bool_df.dtypes
Out[6]:
A    bool
B    bool
C    bool
dtype: object

In [7]: bool_df.info()
<class 'pandas.core.frame.DataFrame'>
RangeIndex: 1000 entries, 0 to 999
Data columns (total 3 columns):
 #   Column  Non-Null Count  Dtype
---  ------  --------------  -----
 0   A       1000 non-null   bool
 1   B       1000 non-null   bool
 2   C       1000 non-null   bool
dtypes: bool(3)
memory usage: 3.1 KB

In [8]: np_bool_df = pd.DataFrame([[np.bool_(True), np.bool_(False), np.bool_(True)]]*1000, columns=["A", "B", "C"])

In [9]: np_bool_df.dtypes
Out[9]:
A    bool
B    bool
C    bool
dtype: object

In [10]: np_bool_df.info()
<class 'pandas.core.frame.DataFrame'>
RangeIndex: 1000 entries, 0 to 999
Data columns (total 3 columns):
 #   Column  Non-Null Count  Dtype
---  ------  --------------  -----
 0   A       1000 non-null   bool
 1   B       1000 non-null   bool
 2   C       1000 non-null   bool
dtypes: bool(3)
memory usage: 3.1 KB

@YarShev That followup commit replaced all the other instances of np.bool. But I think in those cases they should get replaced with just bool. Double check me.

Hi @NickCrews quick question - why are we replacing those instances with bool instead of np.bool_? I did some checking, and it seems that bool and np.bool_ behave similarly when doing dtypes checking (i.e. df.dtypes == bool and df.dtypes == np.bool_ return the same result), but np.bool_ != bool - so unless there's a compelling reason to swap to bool, I'd prefer sticking with np.bool_, in case there are some edge cases we're missing!
(I've also approved the run again)!

@RehanSD, I am not sure that is the fair check you did because when comparing df.dtypes == bool or df.dtypes == np.bool_, as well as np.bool_ != bool it seems we are doing different operations.
import pandas as pd; import numpy as np

df = pd.DataFrame([[True, False, True]]*1000, columns=["A", "B", "C"], dtype=np.bool_)
df.dtypes == bool
A    True
B    True
C    True
dtype: bool

df.dtypes == np.bool
DeprecationWarning: `np.bool` is a deprecated alias for the builtin `bool`. To silence this warning, use `bool` by itself. Doing this will not modify any behavior and is safe. If you specifically wanted the numpy scalar type, use `np.bool_` here.
Deprecated in NumPy 1.20; for more details and guidance: https://numpy.org/devdocs/release/1.20.0-notes.html#deprecations
  df.dtypes == np.bool
A    True
B    True
C    True
dtype: bool

df.dtypes == np.bool_
A    True
B    True
C    True
dtype: bool

np.bool_ == bool
False

I will note, as a new contributor the number of hoops I have to jump through, as compared to contributing to say pandas, is annoying. Autogenerated release notes, and looser requirements on how to format commit messages and issue numbers etc would probably make me more likely to contribute in the future. Just my two cents, I'm sure there are reasons for the structure!

Thanks @NickCrews this is good feedback. I think the contribution requirements list has grown and a lot of things have been added to solve one problem or another, but I agree that we should definitely try to soften as many of these requirements. This checklist has grown to be a bit too long:

Most of these requirements were reminders for myself/other maintainers to enforce some good discipline, but don't necessarily need to be so strict.

@RehanSD @YarShev I think the conversation is getting a bit too in the weeds. I think the way @NickCrews has structured this is actually fine: np.bool_ is used within the query compiler and bool is used at the higher layers in Modin (where code should be more readable). I don't think we need to necessarily go so deep.
I am fine with the changes as-is.

@RehanSD @YarShev I think the conversation is getting a bit too in the weeds. I think the way @NickCrews has structured this is actually fine: np.bool_ is used within the query compiler and bool is used at the higher layers in Modin (where code should be more readable). I don't think we need to necessarily go so deep.
I am fine with the changes as-is.

@devin-petersohn, a Modin developer might be confused with this behavior in future and he will have a question why np.bool_ is used in some places, but Python bool is used in others. I am rather for the consistent behavior everywhere if there is no any implication or side effect of using a concrete boolean dtype.

To be honest, I think @devin-petersohn is right - I'm happy to merge this PR as is and open a new PR to do a refactor.
@YarShev I don't know if np.bool_ is actually more memory efficient when used with pandas. The np.bool_ type seems to take up about 3 bits less than bool, so in the dfs I created above, we'd expect the np.bool_ ones to be about a KB less in memory, but both seem to be the same amount of memory. Obviously this could be due to some caching that Python/pandas are doing (since my tests aren't very rigorous, and the whole df seems to take less memory than I'd expect), but it seems to me that pandas is just converting np.bool_ to bool under the hood, negating any memory savings we would expect.

@devin-petersohn, makes sense to me, thanks! @NickCrews, please rebase on current master to fix failures in CI.

Suggestions applied. Thank you all! That was easy to apply those suggestions. I haven't used that feature before but that was slick!

I'm nervous that adding a oneline fixer of or (inc in [bool, np.bool_] and d in [bool, np.bool_]) is a weird exceptional case that separates bool/np.bool_ from all the other dtypes. Seems like a code smell, and we should solve this in a more generalized way.
Looking at how pandas does this, their describe() uses select_dtypes() so that those two methods are guaranteed to have the same behavior and always be in sync. Do we want to / can we do the same thing? Or somehow merge that logic together so they both use the same helper functions?
IDK, could just do this tweak, merge, and then fixup later, but then it runs the risk of being a TODO that never is fixed 😆

That makes sense - but I think there may be an easier way to handle this - we can just use pandas.core.dtypes.common.pandas_dtype which converts a type to a pandas or numpy dtype. The result of calling this on np.bool_ is bool, which works well, so we can just map this across include and exclude in both select_dtypes and describe.

@RehanSD OK that seems like a nice simple alternative. Do you think you could actually implement it though? I am getting blocked from being able to work on this locaclly by https://github.com/modin-project/modin/issues/4579

Of course - happy to do so (unless you'd prefer to do it, since it seems that Mahesh may have unblocked you)?

@RehanSD thank you, that commit looks great! @YarShev I think this is ready for review again.

Thank you guys, LGTM!

Thank you @NickCrews! I've gone ahead and merged this!


Regular zombie children since my latest containerd update

Description
Since updating (via apt full-upgrade) to the latest available version of containerd-shim-runc-v2, I'm seeing a lot of previously happy health check exec scripts locking up due to "defunct" processes not being reaped by the parent shim process. Across approx 100 containers running check scripts every 10 seconds I see an average of 5 defunct scripts an hour and have to clear them out by hand.
Steps to reproduce the issue

Start a php-fpm container which includes the php fpm health check script from here: https://github.com/renatomefi/php-fpm-healthcheck
Setup a regular exec that runs the php-fpm-healthcheck script every few seconds
Wait. Eventually it will get stuck.

Describe the results you received and expected
I'd expect the containerd-shim-runc-v2 process to correctly handle SIGCHLD signals and reap the child exec process. What actually happens is that the containerd-shim-runc-v2 fails to respond to the SIGCHLD signal - even when I send it by hand - and I'm forced to kill it using a '-9' signal.
What version of containerd are you using?
1.6.10
Any other relevant information
containerd-shim-runc-v2:
Version:  1.6.10
Revision: 770bd0108c32f3fb5c73ae1264f7e503fe7b2661
Go version: go1.18.8
runc version 1.1.4
commit: v1.1.4-0-g5fd4c4d
spec: 1.0.2-dev
go: go1.18.8
libseccomp: 2.5.1
Linux foo 5.4.0-132-generic #148-Ubuntu SMP Mon Oct 17 16:02:06 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux
Show configuration if it is related to CRI plugin.
No response

Hi,
Not sure is related to last containerd update.
It seem, that I have the same issue since I upgrade kernel to 5.4.0-132.
I got lot of messages related to "io.containerd.ttrpc.v1.task"
~$  sudo journalctl --since today | grep "io.containerd.runc.v2" | grep "io.containerd.ttrpc.v1.task" | wc -l
1298

I have got errors with 2 "docker/containerd" release :

docker 20.10.18, runc 1.1.4, containerd 1.6.8
docker 20.10.21, runc 1.1.4, containerd 1.6.10

When I downgrade kernel to 5.4.0-126, I got no more error

I got also lot of error "failed to remove runc container"
sudo journalctl --since today | grep -i error | grep  "failed to remove runc container" | wc l
97

I use rke1.
I saw lot or error in the kubelet logs
$ docker logs  kubelet 2>&1 | grep E1126 | grep DeadlineExceeded
E1126 17:29:38.193412    2098 kuberuntime_manager.go:1003] "PodSandboxStatus of sandbox for pod" err="rpc error: code = DeadlineExceeded desc = context deadline exceeded" podSandboxID="30d52e4fb7a0ec8f5e96bb888c5ae9119999830e85c114ca5999394f4fc7c112" pod="cattle-system/cattle-cluster-agent-8454887c89-6b5hb"
E1126 17:31:39.372906    2098 kuberuntime_manager.go:1003] "PodSandboxStatus of sandbox for pod" err="rpc error: code = DeadlineExceeded desc = context deadline exceeded" podSandboxID="30d52e4fb7a0ec8f5e96bb888c5ae9119999830e85c114ca5999394f4fc7c112" pod="cattle-system/cattle-cluster-agent-8454887c89-6b5hb"
E1126 17:31:49.900799    2098 remote_runtime.go:144] "StopPodSandbox from runtime service failed" err="rpc error: code = DeadlineExceeded desc = context deadline exceeded" podSandboxID="30d52e4fb7a0ec8f5e96bb888c5ae9119999830e85c114ca5999394f4fc7c112"
E1126 17:31:49.900851    2098 kuberuntime_gc.go:176] "Failed to stop sandbox before removing" err="rpc error: code = DeadlineExceeded desc = context deadline exceeded" sandboxID="30d52e4fb7a0ec8f5e96bb888c5ae9119999830e85c114ca5999394f4fc7c112"
E1126 17:33:39.395975    2098 kuberuntime_manager.go:1003] "PodSandboxStatus of sandbox for pod" err="rpc error: code = DeadlineExceeded desc = context deadline exceeded" podSandboxID="30d52e4fb7a0ec8f5e96bb888c5ae9119999830e85c114ca5999394f4fc7c112" pod="cattle-system/cattle-cluster-agent-8454887c89-6b5hb"
E1126 17:34:50.392052    2098 remote_runtime.go:144] "StopPodSandbox from runtime service failed" err="rpc error: code = DeadlineExceeded desc = context deadline exceeded" podSandboxID="30d52e4fb7a0ec8f5e96bb888c5ae9119999830e85c114ca5999394f4fc7c112"
E1126 17:34:50.392079    2098 kuberuntime_gc.go:176] "Failed to stop sandbox before removing" err="rpc error: code = DeadlineExceeded desc = context deadline exceeded" sandboxID="30d52e4fb7a0ec8f5e96bb888c5ae9119999830e85c114ca5999394f4fc7c112"`

may be related to this ? https://github.com/containerd/containerd/issues/4604

I confirm that i have no more issues with kernel 5.4.0-126.
I have not tested kernel.4.0-128 or 5.4.0-131

I'm seeing the same here with rke1 nodes, however I believe this is just a biproduct of the zombie processes. The kubelet is trying to collect metrics etc from the zombie container and does not get a response, which causes it to not be able to properly report back to rancher.

Reverting to kernel 5.4.0-126 stopped the zombies from spawning and persisting. This behavior was seen on two separate clusters both matching the same versions listed above. We need to be able to stay up to date on kernel security patches though so this is far from a long term solution. Containerd is the technical owner of the zombie processes and should be the one to reap them, regardless of kernel version.

I think this can be closed, it's a kernel bug:
https://bugs.launchpad.net/ubuntu/+source/containerd/+bug/1996678
Fix is in 5.4.0-135, changelog entry:
 * containerd sporadic timeouts (LP: #1996678)
    - epoll: call final ep_events_available() check under the lock
    - epoll: check for events when removing a timed out thread from the wait queue
    - Revert "fs: check FMODE_LSEEK to control internal pipe splicing"

Agreed this is most likely due to that kernel bug, so I'll go ahead and close for housekeeping purposes, but if you update to the latest distro kernel and are still having an issue please ping us here.

Hi, same here,
k8s v1.21.10, kernel 5.4.0-131-generic + container.d 1.6.9-1 works
k8s v1.21.10, kernel 5.4.0-132-generic + container.d 1.6.10-1 fails
k8s v1.21.10, kernel 5.4.0-135-generic + container.d 1.6.10-1 fixed


ExtractorError: no conn, hlsvp or url_encoded_fmt_stream_map information found

> sudo youtube-dl -U
youtube-dl is up-to-date (2015.04.17)

>  youtube-dl --verbose --no-playlist https://www.youtube.com/watch\?v\=NvF-GZI20L4\&index\=9\&list\=PLZdCLR02grLrKAOj8FJ1GGmNM5l7Okz0a
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'--verbose', u'--no-playlist', u'https://www.youtube.com/watch?v=NvF-GZI20L4&index=9&list=PLZdCLR02grLrKAOj8FJ1GGmNM5l7Okz0a']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2015.04.17
[debug] Python version 2.7.6 - Darwin-13.4.0-x86_64-i386-64bit
[debug] exe versions: ffmpeg 2.2.2, ffprobe 2.2.2
[debug] Proxy map: {}
[youtube:playlist] Downloading just video NvF-GZI20L4 because of --no-playlist
[youtube] NvF-GZI20L4: Downloading webpage
[youtube] NvF-GZI20L4: Downloading video info webpage
[youtube] NvF-GZI20L4: Extracting video information
ERROR: no conn, hlsvp or url_encoded_fmt_stream_map information found in video info; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/youtube.py", line 876, in _real_extract
    raise ValueError('Could not find ytplayer.config')  # caught below
ValueError: Could not find ytplayer.config
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 651, in extract_info
    ie_result = ie.extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 275, in extract
    return self._real_extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/youtube.py", line 1111, in _real_extract
    raise ExtractorError('no conn, hlsvp or url_encoded_fmt_stream_map information found in video info')
ExtractorError: no conn, hlsvp or url_encoded_fmt_stream_map information found in video info; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

I was able to download the video now, so this must have been a youtube hiccup.


chore(main): release 2.84.0

:robot: I have created a release beep boop
2.84.0 (2023-11-27)
Features

Feature Versioning V2 (#2382) (bcfb10e)
Rely on Flagsmith Engine for segment evaluation (#2865) (322eb08)
ui: hide API keys from integrations list (#3019) (b02a524)

Bug Fixes

WIP Move groups of tests to proper location (#3027) (1592919)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/Flagsmith/flagsmith/releases/tag/v2.84.0 :sunflower:


New function needed:transfer with different character encoding standards

There are many character encoding standards such as UNICODE\UTF-8\UTF-16\UTF-32\UCS\ANSI\GBK\GB2312 and so on. The default that in golang is utf-8. But many times we received others, or we need to send with others , and it's hard for us to deal with. I wonder if golang can support official way to deal with the problem, that means, we can translate the character from one encoding way to another.

https://godoc.org/golang.org/x/text/encoding

Thanks @defia
That works, but I think there should be a offical way instead of third-part way to solve this problem.

golang.org/x/text/encoding is not third-party. It is written by the same contributors who write the standard library. It's the right answer, it's the official way, and it's well supported.


⚠ Bofrost.es has degraded performance

In 7768959, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1550 ms

Resolved: Bofrost.es performance has improved in 46fdada after 8 minutes.


Created Hangman.py

this will run with words.py please merge that as well
-- Hangman game in python --
What it does?

Take letter inputs from the user
Check for the words
Check for the duplicate letter and words
Make a visual as per the tries of the inputs entered
Menu driven for multiple plays
Fully interactive game

Hey @tomriddle77 pleace accept this pull request

Please create a issue and come back here and then tag the issue so that I can assign it to you

Hey I made a pull request please merge this

I have updated more info about the game!

Please create a issue and come back here and then tag the issue so that I can assign it to you

Hey @tomriddle77 thanks ,I forgot to mention the issue for this pull request #69 . Please assign this to me. It was great contributing to the repo..


Fix Texture prepare

Description of change

Fixing some mistakes on Texture preparation.
Bug rep:
https://www.pixiplayground.com/#/edit/8bFs2kvJ1gaJddAP7MRek
Fixed:
https://www.pixiplayground.com/#/edit/aIRNjeeax2Aj83HwPix5a
Pre-Merge Checklist

[x] Tests and/or benchmarks are included
[x] Documentation is changed or added
[x] Lint process passed (npm run lint)
[x] Tests passed (npm run test)

Codecov Report

Merging #6041 into dev will decrease coverage by <.01%.
The diff coverage is 25%.

@@            Coverage Diff             @@
##              dev    #6041      +/-   ##
==========================================
- Coverage   71.05%   71.04%   -0.01%
==========================================
  Files         208      208
  Lines       10561    10562       +1
==========================================
  Hits         7504     7504
- Misses       3057     3058       +1

Impacted Files
Coverage Δ

packages/prepare/src/Prepare.js
5% <ø> (ø)
:arrow_up:

packages/prepare/src/BasePrepare.js
71.87% <25%> (-0.57%)
:arrow_down:

Continue to review full report at Codecov.

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

@eXponenta
I still don't get this PR. Your example of the "bug" doesn't work. Can you please fix or update that?

Yep, we need playground with read-only links, because someone edit example.


Allow HTTP client timeout to be configurable on plugin enable.

- What I did
Make HTTP client timeout configurable for plugins. This is important to not hang the plugin client when the plugin's HTTP server is unresponsive. The default timeout is 0, which implies no timeout.
- How I did it
Added a timeout option to plugin enable
- How to verify it
Write an unresponsive plugin HTTP server and ensure that the requests to the plugin timeout within what's been configured.
Signed-off-by: Anusha Ragunathan<EMAIL_ADDRESS>

cc @vieux @tiborvass : PTAL. Once we agree on the design, I'll add doc updates and test.

LGTM if green

LGTM

Looks like this adds a new flag, so updates to docs, completion scripts and man pages are needed?


LoadError: no such file to load -- sidekiq/actor in sidekiq v4.0.0.pre1

I got this error when upgrade to sidekiq v4.0.0.pre1

Same error with Sidekiq 4.0.0.pre2. Also appears that the Job and Poller classes depend on sidekiq/actor which no longer exists in Sidekiq 4.

@disbelief  @muhammet I have release sidekiq-cron version 0.4.0 where it should be fixed

Worked for me with Sidekiq 4.0.1.

Having the same one ! Here's my stacktrace if that can help !
http://pastebin.com/N0SU6vMM

@dfabreguette-ap pls use sidekiq-cron version 0.4.0


[CDS-100456] Added GitOps Auto Updater Job

Thanks for contributing to the Harness Developer Hub! Our code owners will review your submission.
Description

Please describe your changes: __________________________________
Jira/GitHub Issue numbers (if any): ______________________________
Preview links/images (Internal contributors only): __________________

PR lifecycle
We aim to merge PRs within one week or less, but delays happen sometimes.
If your PR is open longer than two weeks without any human activity, please tag a code owner in a comment.
PRs must meet these requirements to be merged:

[ ] Successful preview build.
[ ] Code owner review.
[ ] No merge conflicts.
[ ] Release notes/new features docs: Feature/version released to at least one prod environment.

 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.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://67212849223bfa42b811db75--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6721f7186f1f194ff44e3860--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://67220a9d7a714a654b5872c0--harness-developer.netlify.app


build element naming

The names for the build element should change as follow (the yaml and CLI are should follow suit)
dockerFile -> dockerfile : This should include the full path of the Dockerfile.  Empty mean ${context} + "/Dockerfile"
dockerFilePath -> remove
buildContext -> context
buildArgs -> args
githubSecretName -> webhookSecret
gitSecretName -> pullSecret
pushRegistrySecretName -> pushRegistrySecret
buildImageName -> imageName
enablePr -> pr

What is push for?

To be discussed. Need CLI or API review to decide what should be renamed exactly.

Done as part of your API refactoring @ibuildthecloud

Confirmed nothing to QA here from a functionality perspective. 👍


Custom IDs in sidebar rendering as Markdown.Links

again, probably needs an overload in the sidebar code...

Same as #69?

Ah good point!  Yes it would be the same issue.


Download crashes when trying to download all images: "isic image download images/ -l 0"

I can download smaller amountsm, but it alwasys crashes when downloading all images, this time with 51 seconds to go.
File "C:\Users\x\PycharmProjects\HCI_Home5\isic-cli\lib\site-packages\requests\models.py", line 760, in generate
for chunk in self.raw.stream(chunk_size, decode_content=True):
File "C:\Users\x\PycharmProjects\HCI_Home5\isic-cli\lib\site-packages\urllib3\response.py", line 579, in stream
data = self.read(amt=amt, decode_content=decode_content)
File "C:\Users\x\PycharmProjects\HCI_Home5\isic-cli\lib\site-packages\urllib3\response.py", line 544, in read
raise IncompleteRead(self._fp_bytes_read, self.length_remaining)
File "C:\Users\x\AppData\Local\Programs\Python\Python39\lib\contextlib.py", line 137, in exit
self.gen.throw(typ, value, traceback)
File "C:\Users\x\PycharmProjects\HCI_Home5\isic-cli\lib\site-packages\urllib3\response.py", line 446, in _error_catcher
raise ReadTimeoutError(self._pool, None, "Read timed out.")
urllib3.exceptions.ReadTimeoutError: HTTPSConnectionPool(host='content.isic-archive.com', port=443): Read timed out.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "c:\users\x\pycharmprojects\isic-cli\isic_cli\cli_init_.py", line 132, in main
cli()
File "C:\Users\x\PycharmProjects\HCI_Home5\isic-cli\lib\site-packages\click\core.py", line 1130, in call
return self.main(*args, **kwargs)
File "C:\Users\Pascal\PycharmProjects\HCI_Home5\isic-cli\lib\site-packages\click\core.py", line 1055, in main
rv = self.invoke(ctx)
File "C:\Users\x\PycharmProjects\HCI_Home5\isic-cli\lib\site-packages\click\core.py", line 1657, in invoke
return _process_result(sub_ctx.command.invoke(sub_ctx))
File "C:\Users\x\PycharmProjects\HCI_Home5\isic-cli\lib\site-packages\click\core.py", line 1657, in invoke
return _process_result(sub_ctx.command.invoke(sub_ctx))
File "C:\Users\x\PycharmProjects\HCI_Home5\isic-cli\lib\site-packages\click\core.py", line 1404, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "C:\Users\x\PycharmProjects\HCI_Home5\isic-cli\lib\site-packages\click\core.py", line 760, in invoke
return __callback(*args, **kwargs)
File "C:\Users\x\PycharmProjects\HCI_Home5\isic-cli\lib\site-packages\click\decorators.py", line 38, in new_func
return f(get_current_context().obj, *args, **kwargs)
File "c:\users\x\pycharmprojects\isic-cli\isic_cli\cli\utils.py", line 17, in decorator
f(ctx, **kwargs)
File "c:\users\x\pycharmprojects\isic-cli\isic_cli\cli\image.py", line 99, in download
Parallel()(delayed(download_image)(image, outdir, progress, task2) for image in images)
File "C:\Users\x\PycharmProjects\HCI_Home5\isic-cli\lib\site-packages\joblib\parallel.py", line 1056, in call
self.retrieve()
File "C:\Users\x\PycharmProjects\HCI_Home5\isic-cli\lib\site-packages\joblib\parallel.py", line 935, in retrieve
self._output.extend(job.get(timeout=self.timeout))
File "C:\Users\x\AppData\Local\Programs\Python\Python39\lib\multiprocessing\pool.py", line 771, in get
raise self._value
File "C:\Users\x\AppData\Local\Programs\Python\Python39\lib\multiprocessing\pool.py", line 125, in worker
result = (True, func(*args, **kwds))
File "C:\Users\x\PycharmProjects\HCI_Home5\isic-cli\lib\site-packages\joblib_parallel_backends.py", line 595, in call
return self.func(*args, **kwargs)
File "C:\Users\x\PycharmProjects\HCI_Home5\isic-cli\lib\site-packages\joblib\parallel.py", line 262, in call
return [func(*args, **kwargs)
File "C:\Users\x\PycharmProjects\HCI_Home5\isic-cli\lib\site-packages\joblib\parallel.py", line 262, in
return [func(*args, **kwargs)
File "c:\users\pascal\pycharmprojects\isic-cli\isic_cli\io\http.py", line 112, in download_image
for chunk in r.iter_content(1024 * 1024 * 5):
File "C:\Users\x\PycharmProjects\HCI_Home5\isic-cli\lib\site-packages\requests\models.py", line 767, in generate
raise ConnectionError(e)
requests.exceptions.ConnectionError: HTTPSConnectionPool(host='content.isic-archive.com', port=443): Read timed out.
isic-cli: v2.0.1.dev1+g2dd86f9
python:   v3.9.12
time:     2022-06-05T09:15:26.758860
os:       Windows-10-10.0.19043-SP0
env:      -
user:     -
command:  isic image download images3/ -l 0

Sorry for this, when making tens of thousands of requests there tend to be a few failures. We perform automatic retries in a number of cases but it appears we missed one.
A new version is out with significantly better retry logic. You can pip install --upgrade isic-cli to get it. Let me know if it works for you.
In the next couple of days I'll push a new release that also allows a download command to pick up where it left off, which should also help if this occurs again.

In the next couple of days I'll push a new release that also allows a download command to pick up where it left off, which should also help if this occurs again.

This feature should be available now in v3.0.0.

Thank you for the fast and extensive reply, I already found another repo that can download the dataset, but I'm gonna give this a try

Which repo was it if you don't mind me asking? Most of these projects use the old ISIC API which is being decommissioned soon.


feat(Chat): Possible to get files from storage to send on chats

What this PR does 📖

Possible to get files from storage to send on chats
Attachments now are persistent, we can move to other page, other chat, and attachments will keep same for that specific chat

Which issue(s) this PR fixes 🔨

Resolve #559
Resolve #398

Special notes for reviewers 🗒
Additional comments 🎤

I am having friend request issues in this branch - with automation and manual testing

I am getting a small bug, when picking the file to send I chose the file, hit send and it does not show the file above the chatbar. Once I refresh the page however the file exists.

@lgmarchi There is an issue when picking up more than one file from local machine to send on chat.
https://github.com/user-attachments/assets/e4f10d63-486d-4eda-983e-0c9bb72468d1

Go to chat
Select upload files
Choose upload from local files
Pick up more than one file on your local machine
Preview is not loaded (bug)
Refresh screen and you will see that sometimes the message is sent and even appears only on remote or local side instead of both sides


Kindred Segregation

text: 1. Only requires the pool cost to be paid in order to keep the ally. Allies with blood costs have no pool cost to recruit, meaning they can be kept by paying 0 pool.
link: https://groups.google.com/d/msg/rec.games.trading-cards.jyhad/GsI1UyH54jU/x9LfDZX1vR4J

Ruling already present. Please check before submitting.


[release-4.18] OCPBUGS-46081: update a2 gpu detection logic to be dynamic

this change updates the way that a2 instance types are validated for quota. gcp has added a new type of a2 instance category named "ultragpu". the ultragpu series has a different type of gpu, "nvidia-a100-80gb", which requires a new constant be applied to the quota.
to make this workflow more modular in the future, it is also being adjusted to dynamically discover the gpu type during the instance quota validation.

/label qe-approved

/label cherry-pick-approved

/approve
/lgtm
/label backport-risk-assessed

/jira refresh


RF Scanner Hold Button

Is there a way to scan the RF code without having to hold the button on the device down? I am trying to learn the RF hex code for my ceiling fan remote and the act of holding the ‘on’ button down provides a different action from the remote than just pressing the button once without holding it. I believe this is giving me incorrect hex codes. Any ideas on what to do? TIA.

Unfortunately not, sorry!


Query folding support

Description
Through this pull request query folding support is being added
Remaining Work

[x] Confirm the navigator loads each table
[X] Finish refactoring each function
[x] Modify interactions with Microsoft Graph to use v1.0 over beta when possible
[x] Modify the way the device compliance policies are requested
[X] Modify the way the device configuration profiles are requested
[x] Perform regression testing
[x] Verify the network request are not being duplicated

Related issue
#45 #62 #65 #67 #79 #82 #85 #89 #92 #94 #96 #97

Refactoring is almost complete and the goal is to merge these changes early next week

Triage - Templates will happen when we are ready to release this version. That way there will not be an impact to customers and partners who are looking to use the current release.


Added paranoid option to the cache key

Hello,
I added the paranoid option to the cache key so consecutive calls with different paranoid settings can show the correct results :
User.findAll({paranoid: false});
User.findAll({paranoid: true});

Thank you !


Code Selection List Issues with Html and CSS Performance Inside Visual Studio Code

Hello,
The following codes are shown on Visual Studio Code's screen when opened my old/new html, and the html5 's content looked a little different than ever before, for instance, now the html5 selection list has html 5boilerplate and another html5 contains of "i frame tag" in it, as well as CSS isn't able to function at all. Thank you so much for helping me  out to such a problem.

Can you add screenshots that show the problem (maybe before and after)