test: sleep 30s after restarting mgt server in test_kubernetes_supported_versions.py

Description
This PR fixes test failure with test_secondary_storage.py
mgmt server is restarted in cleanup of test_kubernetes_supported_versions.py as last step.
in some smoke tests, test_secondary_storage.py is run behind test_kubernetes_supported_versions.py.
but in  test_secondary_storage.py it checks if all hosts are Up as first step.
sometimes, the mgmt server is Up, but hosts are still in Connecting state. this causes the test failure with test_secondary_storage.py

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[ ] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[x] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?

@blueorangutan package

@DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

@sureshanaparti i think we don't need testing of these changes, leaving it up to you.

@blueorangutan test

@weizhouapache a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian Build Failed (tid-3238)

@blueorangutan test

@blueorangutan test

@rohityadavcloud a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-3251)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 26811 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr5962-t3251-kvm-centos7.zip
Intermittent failure detected: /marvin/tests/smoke/test_router_dns.py

Trillian test result (tid-3242)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 35552 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr5962-t3242-kvm-centos7.zip
Smoke tests completed. 80 look OK, 0 have errors
Only failed tests results shown below:

Test
Result
Time (s)
Test File


Solved failure of buildup when Cgo is disabled

Applied a dummy implementation of daemon/service when Cgo is disabled.
Signed-off-by: Xuan Chen<EMAIL_ADDRESS>

@xiaq Please feel free to let me know what you think of it. Thanks for the patience.


Add raw data to data-raw folder

@raulalexb, please add the data to this folder:
https://github.com/Global-Health-Engineering/datadurbanplasticwaste22/tree/main/data-raw
and then commit to finalise.

Hi @larnsce ! I am currently preparing my data for the transfer to a .csv file and I stumbled upon following question:
Can I leave empty cells empty (without any data), in hopes that they will store the number "0" when transfered as a .csv file? Or do I explicitly have to write 0 in those cells? Thanks ;)

Hi @raulalexb. Thanks for reaching out. The question is whether your empty cells represent:

missing data, in which case you should put NA
an actual zero (as in 0 kg), in which case you should put 0

That said, you can leave the cells empty, and once I know the answer to the question, I can do this in the final tidying process with R.

Thanks @larnsce ! They should represent 0, as in 0 amounts (in numbers) collected, so your second example. In this case I will leave them empty, is that okay? Thanks again for your help!

Okay, great. Yes, leave them empty and I will reformat them to 0s.
Do you have empty cells for other variables?

No I do not, just the the cells that had 0 amounts collected on a given day ;)

Hi @larnsce !
I just uploaded both my data and Chiara's data. I added a litte readme, but this will obviously get updated with better information in the future.
Important for the data: I formatted my data so that it is all one big chunk of information that can be converted (hopefully easily) into a .csv file. Chiara's data is split into two, the beach data and the riverboom data. Since her riverboom data has no dates, I put it into a seperate section. Should I just add her litterboom data into the main sectionig block, and just put that the date is N/A?
Thanks again for your time and help ;)
Best,
Raúl

Hi @raulalexb, thank you for writing up the README and adding the data.
I will start with your data for now, and will then follow up with Chiara's. When there is no date, then you can add NA, but leave it as is now.
Questions:

In your data, what does the first untitled column represent? It's a running id from 1 to 14, but what are the groups?

Another question for the columns "Weight of PET" and "Weight of HDPE/PP":
You have dates where you have 0 for "Weight of PET", but you indicate that you collected items categorised as PET for that date.
For example, on 14th October (first 10 rows of 336 rows):

date
brand
plastic
amount
weight_pet

2022-10-14
Sunshine D
PET
2
0

2022-10-14
d'Lite
HDPE/PP
10
0

2022-10-14
NOLA
HDPE/PP
3
0

2022-10-14
YUM YUM
HDPE/PP
2
0

2022-10-14
Checker's House Brand
HDPE/PP
1
0

2022-10-14
Gentle Magic
PET
2
0

2022-10-14
Woolworths Food
HDPE/PP
9
0

2022-10-14
UnID Medical
HDPE/PP
66
0

2022-10-14
DEO
HDPE/PP
35
0

2022-10-14
OTHER
HDPE/PP
183
0

What happened in these cases? Did you not actually weigh the PET on those days?

Hi @larnsce!
Thank you for looking into my data :)
To your questions:

I forgot to title the first column, sorry! It’s supposed to be the collection day. In example, all the data with “1” was collected on the first day of collection, the data with a “4” on the fourth, and so on…

Yes, 06.09 represents September 6th

The weights are in kg. The way the weights work is that all the PET and HDPE/PP weight is weighed before and after collection (as a check). Since this only yields us with two data points per collection, I decided to write those two data points for every row on the same collection date. This part was tricky for me, but I hope it is understandable now.

Of course! I will do that right now.

Thanks once more!
Let me know if this clears up your questions.
Saludos,
Raúl

@larnsce To your last question concerning the PET and HDPE/PP weights.
Yes there are days with 0 PET weight where we still did collect PET, that is because there were only single bottles on that day, usually less than 10, which didn't even weigh 1kg all together. Since the scale's accuracy was lower than the actual weight of a handful of bottles, it wasn't even worth writing them down. I hope that clarifies it ;)

@larnsce
I finished the name changes and uploaded the coordinates for the locations.

@raulalexb: Is that something you could do?

I finished the name changes and uploaded the coordinates for the locations.

Great work on the name changes. I should have specified the coordinates better. Please update them and provide them in decimal format (e.g. 34.576482).


IPMIView: 2.17.0 -> 2.18.0

Motivation for this change
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

It's okay if you don't know how to fix the libIKVM patching issue by yourself, I can help you if you'd like.

help me, please)
Running desktop-file validation
building '/nix/store/xawpdj7id36y11vzqp2pid2f599wqawy-IPMIView_2.18.0_build.201007_bundleJRE_Linux_x64.tar.gz.drv'...

trying https://www.supermicro.com/wftp/utility/IPMIView/Linux/IPMIView_2.18.0_build.201007_bundleJRE_Linux_x64.tar.gz
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  103M  100  103M    0     0  7699k      0  0:00:13  0:00:13 --:--:-- 7679k
copying path '/nix/store/a1hpnpjrxdllbfg179vzj7hnh581sz3c-gcc-wrapper-9.3.0' from 'https://cache.nixos.org'...
copying path '/nix/store/qkgbmfnkxw51sfzrxlxsbj6kf2zi7ab2-stdenv-linux' from 'https://cache.nixos.org'...
building '/nix/store/vrc3a2bp9v4np94gjazwwr98bwxbw7lz-IPMIView-2.18.0.drv'...
unpacking sources
unpacking source archive /nix/store/4mx1wp32ijww8w33znjxnvrv33wr1kz9-IPMIView_2.18.0_build.201007_bundleJRE_Linux_x64.tar.gz
source root is IPMIView_2.18.0_build.201007_bundleJRE_Linux_x64
setting SOURCE_DATE_EPOCH to timestamp<PHONE_NUMBER> of file IPMIView_2.18.0_build.201007_bundleJRE_Linux_x64/libiKVM_v11_64.so
patching sources
configuring
no configure script, doing nothing
building
patchelf: no section headers. The input file is probably a statically linked, self-decompressing binary
builder for '/nix/store/vrc3a2bp9v4np94gjazwwr98bwxbw7lz-IPMIView-2.18.0.drv' failed with exit code 1
error: build of '/nix/store/vrc3a2bp9v4np94gjazwwr98bwxbw7lz-IPMIView-2.18.0.drv' failed

So the issue is with the patchElf call at https://github.com/NixOS/nixpkgs/blob/2d89106346fdf2801cd0419b7f52c47d1db18212/pkgs/applications/misc/ipmiview/default.nix#L35
I don't know what changed in IPMIView which casues it to fail but have a workaround suggestion:

Remove this and the immediately above calls. This makes the derivation to build but the KVM console functionality is broken.
To fix the KVM console via an other method, you can extend the LD_LIBRARY_PATH environment variable with the path to gcc-unwrapped.lib when IPMIView is started. To do that, we need to have a makeWrapper call. Fortunately there is an existing makeWrapper invocation here: https://github.com/NixOS/nixpkgs/blob/2d89106346fdf2801cd0419b7f52c47d1db18212/pkgs/applications/misc/ipmiview/default.nix#L58-L59
Adding gcc-unwrapped.lib after fontconfig seems to make KVM console working again.

Here is the complete diff I suggest (click to expand)
@@ -31,8 +31,6 @@ stdenv.mkDerivation rec {
   ''
     patchelf --set-rpath "${stdenv.lib.makeLibraryPath [ libX11 libXext libXrender libXtst libXi ]}" ./jre/lib/amd64/libawt_xawt.so
     patchelf --set-rpath "${stdenv.lib.makeLibraryPath [ freetype ]}" ./jre/lib/amd64/libfontmanager.so
-    patchelf --set-rpath "${gcc-unwrapped.lib}/lib" ./libiKVM64.so
-    patchelf --set-rpath "${gcc-unwrapped.lib}/lib" ./libiKVM_v11_64.so
     patchelf --set-rpath "${gcc.cc}/lib:$out/jre/lib/amd64/jli" --set-interpreter "$(cat $NIX_CC/nix-support/dynamic-linker)" ./jre/bin/java
     patchelf --set-interpreter "$(cat $NIX_CC/nix-support/dynamic-linker)" ./BMCSecurity/${stunnelBinary}
   '';
@@ -56,7 +54,7 @@ stdenv.mkDerivation rec {
     # WORK_DIR: unfortunately the ikvm related binaries are loaded from
     #           and user configuration is written to files in the CWD
     makeWrapper $out/jre/bin/java $out/bin/IPMIView \
-      --set LD_LIBRARY_PATH "${stdenv.lib.makeLibraryPath [ fontconfig ]}" \
+      --set LD_LIBRARY_PATH "${stdenv.lib.makeLibraryPath [ fontconfig gcc-unwrapped.lib ]}" \
       --prefix PATH : "$out/jre/bin:${iputils}/bin:${psmisc}/bin" \
       --add-flags "-jar $out/IPMIView20.jar" \
       --run 'WORK_DIR=''${XDG_DATA_HOME:-~/.local/share}/ipmiview

@vlaci thx man! build succesed

Just one nitpick, please keep the original commit message header IPMIView: 2.17.0 -> 2.18.0.
The contribution guide contains further information on commit messages https://github.com/NixOS/nixpkgs/blob/master/.github/CONTRIBUTING.md#submitting-changes

Just one nitpick, please keep the original commit message header IPMIView: 2.17.0 -> 2.18.0.
The contribution guide contains further information on commit messages https://github.com/NixOS/nixpkgs/blob/master/.github/CONTRIBUTING.md#submitting-changes

fixed

LGTM

LGTM
Thank you, for the contribution!

thank you to you!

Result of nixpkgs-review pr 106428 run on x86_64-linux 1

1 package built:

ipmiview

@vlaci  Hello!
when will this task be merged to master?

As soon as a maintainer with write access picks it up. I can review packages I maintain but I am not able to merge.

As soon as a maintainer with write access picks it up. I can review packages I maintain but I am not able to merge.

got it

@mweinelt pls, approve

@mweinelt Hello!
When merge this commit to nixpkgs-unstable?
I have to create PR for merge?

The nixpkgs manual contains some information how and when the channels are updated. TL; DR nixpkgs-unstable is usually updated in every few days. No need for additional PR.

The nixpkgs manual contains some information how and when the channels are updated. TL; DR nixpkgs-unstable is usually updated in every few days. No need for additional PR.

thx!


Remove comment in command substitution in completion.zsh

After 22adb6494f4957d0bcc29a821b738b4e5d5796a5, the Zsh completion using ** fails.
$ cat **__fzf_generic_path_completion:23: bad pattern: #

The cause of this bug is that a comment is in the command substitution.

Thanks for testing it before the release. I had interactive_comments enabled, which prevented the error from occurring when I tested it.
Comments within a command substitution inside a function are not automatically removed when parsed, unlike other comments in a function. Therefore, moving the comment outside of the substitution should not trigger any error.

Options:

Remove the comment (preferred)
setopt localoptions interactive_comments
Move the comment outside the command substitution

Option 1 seems safe, and one can use git blame to check why the line was written that way.
Option 2 should work, but who knows what kind of pitfalls this might unleash.
Option 3 makes the comment one line separated from the actual affected code line, which looks odd.

Thanks!


Time dimension from multiple NetCDF4 files mis-handled by MultiZarrToZarr

I'm trying to do some small local tests of kerchunk to make sure it behaves as I expect. I tried to concatenate some Earth System model (GEOS-FP) NetCDF4 output from here: https://portal.nccs.nasa.gov/datashare/gmao/geos-fp/das/Y2022/M09/D01/
In this particular case, I'm grabbing just 6 hours of tavg1_2d_flx output; you can download them with a simple command like:
wget -nc https://portal.nccs.nasa.gov/datashare/gmao/geos-fp/das/Y2022/M09/D01/GEOS.fp.asm.tavg1_2d_flx_Nx.20220901_0{0,1,2,3,4,5,6}30.V01.nc4

Xarray's open_mfdataset works swimmingly here:
import kerchunk.hdf
import kerchunk.combine
import fsspec
import ujson
import xarray as xr

fs = fsspec.filesystem('file')

ncfiles = sorted(fs.glob("raw-data/*.nc4"))

xr.open_mfdataset(ncfiles)
# <xarray.Dataset>
# Dimensions:   (lon: 1152, lat: 721, time: 7)
# Coordinates:
#   * lon       (lon) float64 -180.0 -179.7 -179.4 -179.1 ... 179.1 179.4 179.7
#   * lat       (lat) float64 -90.0 -89.75 -89.5 -89.25 ... 89.25 89.5 89.75 90.0
#   * time      (time) datetime64[ns] 2022-09-01T00:30:00 ... 2022-09-01T06:30:00
# Data variables: (12/43)
#     BSTAR     (time, lat, lon) float32 dask.array<chunksize=(1, 721, 1152), meta=np.ndarray>
#     CDH       (time, lat, lon) float32 dask.array<chunksize=(1, 721, 1152), meta=np.ndarray>
#     CDM       (time, lat, lon) float32 dask.array<chunksize=(1, 721, 1152), meta=np.ndarray>
# ...

(I know this isn't a "real" use case for Kerchunk because I can just as (if not more) easily process these files locally, but I wanted to make sure I understand how Kerchunk works before expanding to more complex use cases with larger datasets in remote storage).
Using kerchunk with individual files works fine as well:
for ncf in ncfiles:
    with fs.open(ncf, "rb") as f:
        chunks = kerchunk.hdf.SingleHdf5ToZarr(f, ncf)
        outfile = ncf.replace(".nc4", ".json")
        with fs.open(outfile, "wb") as of:
            of.write(ujson.dumps(chunks.translate()).encode())

jfiles = sorted(fs.glob("raw-data/*.json"))

xr.open_dataset(
    "reference://", engine = "zarr",
    backend_kwargs = {
        "storage_options": {"fo": jfiles[0]},
        "consolidated": False
    }
)
# <xarray.Dataset>
# Dimensions:   (time: 1, lat: 721, lon: 1152)
# Coordinates:
#   * lat       (lat) float64 -90.0 -89.75 -89.5 -89.25 ... 89.25 89.5 89.75 90.0
#   * lon       (lon) float64 -180.0 -179.7 -179.4 -179.1 ... 179.1 179.4 179.7
#   * time      (time) datetime64[ns] 2022-09-01T00:30:00
# Data variables: (12/43)
#     BSTAR     (time, lat, lon) float32 ...
#     CDH       (time, lat, lon) float32 ...
#     CDM       (time, lat, lon) float32 ...
# ...

However, when I go to combine these using MultiZarrToZarr, something goes wrong with the time index. Specifically:

Time dimension is length 1, not 7 (as expected)
Time values are parsed as NaT.

multi = kerchunk.combine.MultiZarrToZarr(
    jfiles,
    concat_dims = ["time"],
    identical_dims= ["lat", "lon"]
)
combined = "test.json"
with fs.open(combined, "wb") as of:
    of.write(ujson.dumps(multi.translate()).encode())

# Try to read the result
xr.open_dataset(
    "reference://", engine = "zarr",
    backend_kwargs = {
        "storage_options": {
            "fo": combined
        },
        "consolidated": False
    }
)

# <xarray.Dataset>
# Dimensions:   (time: 1, lat: 721, lon: 1152)    # NOTE: Only 1 timestep.
# Coordinates:
#   * lat       (lat) float64 -90.0 -89.75 -89.5 -89.25 ... 89.25 89.5 89.75 90.0
#   * lon       (lon) float64 -180.0 -179.7 -179.4 -179.1 ... 179.1 179.4 179.7
#   * time      (time) datetime64[ns] NaT
# Data variables: (12/43)
#     BSTAR     (time, lat, lon) float32 ...
#     CDH       (time, lat, lon) float32 ...
#     CDM       (time, lat, lon) float32 ...
# ...

Here's a relevant chunk of the test.json file. Something does look fishy with the time dimension:
        "time/.zarray": "{\n    \"chunks\": [\n        1\n    ],\n    \"compressor\": null,\n    \"dtype\": \"<i4\",\n    \"fill_value\": 0,
\n    \"filters\": null,\n    \"order\": \"C\",\n    \"shape\": [\n        1\n    ],\n    \"zarr_format\": 2\n}",
        "time/0": "\u0000\u0000\u0000\u0000",
        "time/.zattrs": "{\n    \"_ARRAY_DIMENSIONS\": [\n        \"time\"\n    ],\n    \"begin_date\": 20220901,\n    \"begin_time\": 3000,
\n    \"long_name\": \"time\",\n    \"time_increment\": 10000,\n    \"units\": \"minutes since 2022-09-01 00:30:00\"\n}",

By comparison, here's an analogous chunk from one of the JSON files generated for an individual NetCDF file:
        "time/.zarray": "{\n    \"chunks\": [\n        1\n    ],\n    \"compressor\": {\n        \"id\": \"zlib\",\n        \"level\": 2\n
  },\n    \"dtype\": \"<i4\",\n    \"fill_value\": null,\n    \"filters\": [\n        {\n            \"elementsize\": 4,\n            \"id\"
: \"shuffle\"\n        }\n    ],\n    \"order\": \"C\",\n    \"shape\": [\n        1\n    ],\n    \"zarr_format\": 2\n}",
        "time/.zattrs": "{\n    \"_ARRAY_DIMENSIONS\": [\n        \"time\"\n    ],\n    \"begin_date\": 20220901,\n    \"begin_time\": 3000,
\n    \"long_name\": \"time\",\n    \"time_increment\": 10000,\n    \"units\": \"minutes since 2022-09-01 00:30:00\"\n}",
        "time/0": "x^c```\u0000\u0000\u0000\u0004\u0000\u0001"

A few observations:

For the single NetCDF, the compressor is (correctly) set to zlib, but is converted to null in the combined file.
The fill_value is null in the single NetCDF but 0 in the combined file.
The single NetCDF has values for elementsize and shuffle, but the combined file does not.

I wonder if this is because of nonstandard compression for these NetCDF4 files...? I wonder if this may be related to this (https://github.com/nasa/zarr-eosdis-store/issues/5), which I encountered while trying to generate DMR++ files for a similar dataset.

Awesome, that did the trick — thanks!

This helped me just now as well!
I wonder if this setting coo_map={"time": "cf:time"} should be a default if xarray uses cftime by default?
Also, is there a reason it is coo_map and not coordinate_map or at least coord_map ? It was confusing what this parameter was for (same goes for coo_dtypes)
Thanks!

I think maybe "coo" is an astro thing? I'd be happy to allow "coord", but we should probably keep the alias for now.


Update to Unexpected 11

... And announce compatibility with it in peerDependencies.
Turns out no code changes are actually necessary. I just dived in because we started seeing errors when updating to the latest version and assumed that it was due to Unexpected 11.

Hey, thanks again for this PR. Do you know if they changed something in particular from 10 to 11. As it seems to work I guess at least they didn't touch the assertions we're using ;)
About the frozen top-level expect: Since this is only testing the immutable plugin I was ok with just polluting unexpected. However I see this solution is way cleaner (and I think also advised in the docs) 👍

The changes are summarized here: http://unexpected.js.org/releases/ -- also see the linked migration guide :)
We didn't make any changes that affect the plugin code itself, so a lazier version of this PR would be to stay on Unexpected 10 and just pull in the peerDependencies change, then publish a new version. But it should be good practice to update the dev setup of plugins to at least the latest major version.

I merged this, as the discussion was done, and the tests pass neatly.

Can I get an npm publish, @erikmueller?

Hey, thanks for stepping in. I honestly totally forgot. Should be published as 0.4.0

NP.


Loading the local super long graph will have a short black screen problem. What is the reason for this?

When opening an issue, please complete the template. You have provided no information.

Have you solved it?


Update form fix

Codecov Report

Merging #244 (ef98c76) into main (63b0cfb) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #244   +/-   ##
=======================================
  Coverage   85.71%   85.71%
=======================================
  Files          19       19
  Lines         427      427
  Branches       54       54
=======================================
  Hits          366      366
  Misses         56       56
  Partials        5        5

Impacted Files
Coverage Δ

backend/controllers/logs.js
55.55% <ø> (ø)

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 63b0cfb...ef98c76. Read the comment docs.


[WebToolsE2E][Aspire] Building the eShop solution fails with some errors: "Unable to find package xxx with version. Versions from dotnet-tools were not considered."

REGRESSION INFO: Worked on Aspire 8.0.0-preview.4.24156.9
INSTALL STEPS

Clean machine: Win11 x64 23h2 ENU
Install VS 17.10 Preview 3 [ Includes Aspire 8.0.0-preview.5.24178.3 and SDK 8.0.300-preview.24170.4 ]
Install latest Aspire 8.0 P5 [8.0.0-preview.5.24179.3]
Apply NuGet dotnet-tools Feed
Install Docker Desktop for Windows

REPRO STEPS

Clone eShop repo
Checkout the aspire-preview5 branch
Update sdk version to 8.0.300-preview.24170.4 in global.json
Open eShop.Web.slnf
Build

ACTUAL

EXPECTED
Building the eShop.Web.slnf without any errors.

@eerhardt I wasn't hitting this, can you confirm that you're not either?

@eerhardt I wasn't hitting this, can you confirm that you're not either?

I fixed this with https://github.com/dotnet/eShop/pull/257/commits/de648e78e0fb0b86ff5bf436855f4324206f706f

Not reproduce on VS17.10 Preview 3 [Include 8.0.0-preview.5.24201.12].


Unable To find this Plugin In The Community Plugins

Hi!
Has there been any Obsidian update that is making this plugin defunct?
I couldn't find in the Obsidian Community Plugins.
I manually download the source code and pasted into the plugins folder but when I try activating it says "failed to load the plugin"
Can you guide me if there's something wrong?
Thanks!

Hi,
I know this issue is closed, but can you write those information on the main page ? And where do we have to put those files ? (not 100% familiar with Obsidian for now)
Thanks

I put the files into the folder containing the vault into .obsidian/plugins/obsidian-emoji-shortcuts

Exactly!


Further CI support improvements.

Fix MSVC build failures
Robustify conda environment setup

(This isn't entirely ready for review / merge yet. I expect to add another commit once a new conda build package is released.)

[Current coverage][cc-pull] is 87.46%

Merging [#1882][cc-pull] into [master][cc-base-branch] will not change coverage

@@             master      #1882   diff @@
==========================================
  Files           292        292
  Lines         51981      51981
  Methods           0          0
  Messages          0          0
  Branches       5368       5368
==========================================
  Hits          45461      45461
  Misses         5603       5603
  Partials        917        917

Powered by Codecov. Last updated by [1c6d133...ac8ca2a][cc-compare]
[cc-base-branch]: https://codecov.io/gh/numba/numba/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/numba/numba/compare/1c6d1338da4635646ca769f24802fff442922b98...ac8ca2afa1c9855d7764fcfb6bb5c87b870ca293
[cc-pull]: https://codecov.io/gh/numba/numba/pull/1882?src=pr

Do you think the AppVeyor test script can use this as well? (note it uses --tags important to shorten the test time)

That would be ideal indeed. Perhaps we can try setting it up to work from the branch, to test it.

Is the Windows CI down at the moment? The last Windows build appears to be 0.26.0.dev np110py35_53 whereas  the latest build number for linux is 185.
Will this PR fix that? I'm very keen to try out both the new linalg support and the cfunc support!

Perhaps we can try setting it up to work from the branch, to test it

AppVeyor is supposed to run on every PR, perhaps something wrong went with the latest changesets.

"Error parsing appveyor.yml: (Line: 14, Col: 21, Idx: 557) - (Line: 14, Col: 25, Idx: 561): While parsing a block mapping, did not find expected key."

I tried to get the .appveyor.yml syntax more in line with .travis.yml. Obviously I failed, so I have reverted that bit of environment configuration logic.

@dhirschfeld: That issue was due to a problem in the Win64 builder that was fixed yesterday.  I think the latest Win64 packages are now available on anaconda.org.

Thanks, I've updated and have been playing around with the latest functionality!


Create soisyearbook.is-cool.dev.json

Requirements

[x] You have completed your website.
[x] The website is reachable.
[x] The CNAME record doesn't contain https:// or /.
[x] There is sufficient information at the owner field.

Description
For my project use.
Link to Website
https://soisyearbook.github.io/home/

CLA Assistant Lite bot: 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 can sign the CLA by just posting a Pull Request Comment same as the below format.

I have read the CLA Document and I hereby sign the CLA

You can retrigger this bot by commenting recheck in this Pull Request

I have read the CLA Document and I hereby sign the CLA

Can a maintainer approve the workflow

will close this pull request and will open a new one


fix the legacy card style issue

the card.css was shared by the new radioCardGroup component and the deprecated card component. but recently Aasim and Amir have made changes to it for the new style of the radio card group component. the ML extension is still using the card component.
the fix is to copy the content of previous version of card.css and create a new css file that is only used by card component.
after September release, Udeesha will replace the radio component with radio card group component in ML extension and I will remove the card component.
before:

after:

https://github.com/microsoft/azuredatastudio/issues/12420

Pull Request Test Coverage Report for Build 260266470

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 38.365%

Totals

Change from base Build 260263215:
0.0%

Covered Lines:
19128

Relevant Lines:
45708

💛  - Coveralls


feat: add avatar-click event in contact info

:hammer_and_wrench: Type Of Change

[ ] Fix
[x] Feature
[ ] Refactoring
[ ] Documentation

:book: Description
This pull request adds avatar-click event in contact info component.
:bulb: Context
When a user clicks on any of the avatars in the callbar when on a call with multiple other participants (i.e. a group call) it opens the “Manage participants” modal. To implement that, we need to add avatar-click event in contact info component.
:pencil: Checklist

[x] I have reviewed my changes
[ ] I have added tests
[x] I have added all relevant documentation
[x] I have validated components with a screen reader
[x] I have validated components keyboard navigation
[x] I have considered the performance impact of my change
[x] I have checked that my change did not significantly increase bundle size
[ ] I am exporting any new components or constants in the index.js in the component directory
[ ] I am exporting any new components or constants in the index.js in the root

:crystal_ball: Next Steps
Make a new release for dialtone-vue
:camera: Screenshots / GIFs
No UI update
:link: Sources
JIRA: https://dialpad.atlassian.net/browse/DP-71224
Vue3 PR: https://github.com/dialpad/dialtone-vue/pull/1165

Cool!, could you please add the event on storybook so we can see the event emitted and also some JSDoc, please?
At this moment in the deploy preview I cannot see any event on the Actions section, and the documentation is null.

Seems like there is a11y failing test
FAIL recipes/list_items/contact_info/contact_info.stories.js

I still cannot see the click event on action tab in storybook.
You need to add in contact_info.stories.js
export const argsData = {
  **onAvatarClick: action('avatar-click'),**
  presence: null,
};

and inside argTypesData
onAvatarClick: {
   table: {
     disable: true,
   },
 },

you should add  @avatar-click="onAvatarClick"  in contact_info_default.story.vue also.

Approving it to not block the PR, will check it out the a11y issue with dt-list-item later. Jira ticket DLT-1224


Implement fstest.TestFS

Nice library!
Go provides a testing function for fs.FS: https://tip.golang.org/pkg/testing/fstest/#TestFS
I added a test for this and fixed the revealed issues.

Cool, I didn't know that test TestFS existed. It looks like the code change has some issues as it doesn't compile right now:
$ go test -v ./...
# github.com/psanford/memfs [github.com/psanford/memfs.test]
./memfs.go:302:7: d.dirOffset undefined (type *fhDir has no field or method dirOffset)
FAIL    github.com/psanford/memfs [build failed]
FAIL

Ups, I fixed it.

I merged this. Thank!


[Design] Convert Carbon Aware SDK into a Library

Description
As a Carbon Aware Developer, I need to use the Carbon Aware SDK as a library so that I can access the SDK without opening and HTTP connection.
Acceptance Criteria

[x] Design needs to consider how to support the same .NET versions as Azure Functions 4.0
[ ] Design needs to consider how to expose the SDK API routes as a library and while keeping the Aggregator functionality hidden to the user.
[ ] Design needs to satisfy supporting multiple languages for error messages within the package
[x] Design needs to consider how to work with Config Files within the code as well as ensuring the config files are installed and uninstalled with the package
[ ] Designed needs to consider allowing consumers to step into the code while debugging
[x] Design needs to consider package versioning
[ ] Final architectural decisions proposed in an Architectural Decision Record log

Out of Scope
We will be providing guidance on how to publish the SDK as a package, but we will not be publishing the package.
Dependencies
The customer is considering using the library as part of an Azure Function, so there is a dependency to meet the requirements of Azure Functions.
Sprint-Ready Checklist

[x] Acceptance criteria defined
[x] Engineering team understands acceptance criteria
[x] External / 3rd Party dependencies identified

Tasks

[x] Add Library section

[x] Goals
[x] Non Goals
[x] Proposed design

[x] Short term
[x] Long term

[x] Add Packaging section

[x] Goals
[x] Non Goals
[x] Proposed Design

[x] Write ADR

Hi @esgraham and @gfmatthews . Design doc has been updated with the estimates (see page 2 - Effort comment)


For a 0.1.0 release

This is really a pre-release, to make the KindOfProxy subtypes available to some other packages in development. I figure these types are stable enough for now.

Codecov Report

Merging #22 (b460b71) into master (e170e2e) will increase coverage by 30.83%.
The diff coverage is 34.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@             Coverage Diff             @@
##           master      #22       +/-   ##
===========================================
+ Coverage   27.50%   58.33%   +30.83%
===========================================
  Files           5        6        +1
  Lines          80       84        +4
===========================================
+ Hits           22       49       +27
+ Misses         58       35       -23

Impacted Files
Coverage Δ

src/LearnAPI.jl
100.00% <ø> (ø)

src/data_interface.jl
0.00% <0.00%> (ø)

src/fit_update_ingest.jl
0.00% <0.00%> (ø)

src/algorithm_traits.jl
25.00% <25.00%> (ø)

src/operations.jl
100.00% <100.00%> (+100.00%)
:arrow_up:

src/tools.jl
97.05% <100.00%> (+15.57%)
:arrow_up:


Site Header Source Code

Currently, the Site Header component includes a button that links to this repo. I understand why this button is in for the demo, documentation, etc., however, it has been added to the repo's component code and thus as a developer trying to use this repository for my own projects it is very hard for me to easily personalise your Site Header.
I am not sure if I'm missing something, but this does not seem like a good idea to have been done.

@Thomas-Smyth I agree with you, we'll take care of this soon

@AaronCoplan you got this or am I ok to go ahead and fix it while working on bits atm?

Also on another slightly related note. I assume that the Site.Footer is not yet complete?
That has the same issue, but there is also no way of editing the links, etc.

@Thomas-Smyth I've opened #177 for the site footer so we can track the two separately

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add support for AWS amd64 image build

This PR adds the packer scripts required to build, package and push an AWS ubuntu-focal amd64 image to AWS.
The images are kept up to date as the script uses the latest official release of the image version.
For a working set up the following environment variables need to be set
AWS_ACCESS_KEY_ID=YOUR_ACCESS_KEY
AWS_SECRET_ACCESS_KEY=YOUR_SECRET_KEY

fixes #9

@obasekiosa i would also suggest removing NVIDIA drivers variable, and also, create a new script, that will just download and install the runner. Then, you can reference that script in the packer file. The goal is to get the image just with the runner, and as light as possible, for starters, we can add drivers later

@obasekiosa i would also suggest removing NVIDIA drivers variable, and also, create a new script, that will just download and install the runner. Then, you can reference that script in the packer file. The goal is to get the image just with the runner, and as light as possible, for starters, we can add drivers later

Ok, I will do this.
I'll also be releasing a PR for the AWS pulumi by today or Monday morning.


i18n(zh-cn): Update api-reference.mdx

Description (required)
Update api-reference.mdx
Related issues & labels (optional)

#8923
Suggested label: i18n

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

zh-cn
reference/api-reference.mdx
Localization changed, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


Add profile kyocera-printer

What does this PR do?

Add profile kyocera-printer
Motivation

Additional Notes

Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached
[ ] If the PR doesn't need to be tested during QA, please add a qa/skip-qa label.

@NouhaManai96 Can I have another review ? I had to fix a formatting issue.


Add more information about usage and setup.

Does this support webrtc? (maybe some package replacement shinanigans?)
This seems like an awesome library, but I am not sure how to get started using it, and what environments I can use it in.

This only works with udp/tcp bittorrent since there is no dht and thus no bep44 support in webtorrent yet: https://github.com/feross/webtorrent/issues/288

It seems like you added bep44 support to webtorrent/bittorrent-dht


IRC -> discord mentions don't work

Irc users should be able to use @ nickname syntax to generate a real discord mention for matching discord names.

This is likely related to the discord api change regarding the allowed mentions change

This is likely related to the discord api change regarding the allowed mentions change

Fixed in cd5ccdee643cb935232c51f54361cf1755a43470

Fixed in cd5ccdee643cb935232c51f54361cf1755a43470


Grant Packagist App access to repository

Hi,
The Packagist is using GitHub Hooks for integration but they'll stop working in 2019. @chobie , please allow Packagist app to access this repository.
More Info: https://packagist.org/about#how-to-update-packages

Done.


Quick tour Colab notebook not working

Hi, I'm getting the following error in the cell where get_form_by_ticker is called.
HTTPError: 403 Client Error: Forbidden for url: http://www.sec.gov/cgi-bin/browse-edgar?CIK=rgld&Find=Search&owner=exclude&action=getcompany
Could you please help me how to solve this issue?
Thanks.

Changing the "company" and "email" params in the get_form_by_ticker function worked for me.
    'rgld',
    '10-K',
    company='XYZ Technologies',
    email='new email'
)```


I have just started with open source. So here is what I want to know that how can I make quality pull requests for #oktoberfest and am I supposed to submit it somewhere for oktoberfest?

Description
A brief description of the question or issue:
You can also join the discord community here
Feel free to check out other cool repositories of Eddie Jaoude Community here

I will close this because you already started a Discussion here too https://github.com/EddieJaoudeCommunity/support/discussions/295
Please do not spam in multiple places, your questions will get answered by the community when they have time 🤓


Posix shared memory across containers in pods

Every Pod needs this added to support Posix Shared Memory across containers in pods
volumeMounts:
  - mountPath: /dev/shm
    name: dshm

volumes:
 - name: dshm
      emptyDir:
        medium: Memory
        sizeLimit: "256Mi"

Note that sizeLimit is optional and depends on: https://github.com/kubernetes/kubernetes/pull/63641

Addressed in https://github.com/onepanelio/core/pull/61


Add data types to integration tests

We will cover the data types for different sources/sinks as much as possible in the integration tests. We'll cover all the zero, null, maximum, and minimum values in the test. For overflow values, we'll test it depending on the testing result and the test implementation

For zero value testing, we have two issues just fixed for array. https://github.com/risingwavelabs/risingwave/issues/11351 https://github.com/risingwavelabs/risingwave/issues/11352
And for the record, we don't plan to support the zero date time like '0000-00-00' for MySQL and dates Before Christ for PostgreSQL

So far we've added the following source/sinks

PG cdc source https://github.com/risingwavelabs/risingwave/pull/13176
MySQL cdc source https://github.com/risingwavelabs/risingwave/pull/13546
redis sink https://github.com/risingwavelabs/risingwave/pull/13772
kafka cdc sink https://github.com/risingwavelabs/risingwave/pull/13607


Headers cli flag

Added option to set headers through cli tool

Can you add doc to readme? also would it be better to do Authorization: Bear abcd instead. Better for commandline.

I forgot you can use the same flag multiple times with commander, I've adjusted this now to stack multiple headers:
... -H 'Authorization: Bearer abcd' -H 'Another: header'
Also added the header option to the README. I left the short flag as -H instead of -h though to avoid a collision with the -h help flag, but feel free to change this

@helios1138 plz take a look. 😄


Instructions for gh-pages regeneration

#84

@jjjordanmsft is this how you did it last time? I was thinking - maybe it will be easier to use /docs folder in master branch instead?

re: docs vs gh-pages.  Maybe it's better -- I don't have a strong opinion either way.


BAT91 not working

for a large file more than 3mb, it said:

Neither works for win7 or win10.
but for BAT85, it works.
I need to paste to somewhere do not accept %*, so how do I fix it?
Thank you all!
`Unable to find type [BAT91]: make sure that the assembly containing this type is loaded.
At line:8 char:56

'@; cd -Lit($env:CD); function X([int]$x=1){[BAT91] <<<< ::Dec([ref]$f,$x+1,$x,$k); expand -R $x -F:* .; del $x -
force}

CategoryInfo          : InvalidOperation: (BAT91:String) [], RuntimeException
FullyQualifiedErrorId : TypeNotFound

Microsoft (R) File Expansion Utility  Version 6.1.7600.16385
Copyright (c) Microsoft Corporation. All rights reserved.
Can't open input file: 1.
Remove-Item : Cannot find path 'C:_\7z\1' because it does not exist.
At line:8 char:107

'@; cd -Lit($env:CD); function X([int]$x=1){[BAT91]::Dec([ref]$f,$x+1,$x,$k); expand -R $x -F:* .; del <<<<  $x -
force}

CategoryInfo          : ObjectNotFound: (C:_\7z\1:String) [Remove-Item], ItemNotFoundException
FullyQualifiedErrorId : PathNotFound,Microsoft.PowerShell.Commands.RemoveItemCommand
`

update: 6.1 works fine for file size 500k, while 6.3 does not, I did not know why and how to debug it....

Another bug, if compressed target does not exit, this script may dell this user all files listed in C:\users , it exits in 6.1.
I did not test for 6.3. Be cautions.

some files could be txted, some not...

There must be something wrong with the configuration you're using,
because none of the issues you've mentioned have ever been reported,
and the script has been working for huge files just fine.
Maybe it's a copy-paste error? did you use Code - Download ZIP button in github?
In any case, in v6.4 I've changed a few things to hopefully prevent your issues

Thanks a lot for replying and updating!
As it crashes randomly, I think it is the encode reason. I all use BAT85 to prevent char like (*%.
What I try to encode file is a PDF:
https://support.industry.siemens.com/cs/attachments/109479728/109479728_LGF_TIAV16_DOC_V5_0_0_en.pdf
refer to : https://support.industry.siemens.com/cs/ww/en/view/109479728
I tried when region and language setting to Non-English like below and it still crashes...

In 6.4 the problem exists...

Thanks a lot for updating!

v6.5 at least fixed the issue with BAT85 (because BAT91 worked fine, that's what I personally use a lot)

v6.5 at least fixed the issue with BAT85 (because BAT91 worked fine, that's what I personally use a lot)

Thanks again for updating!
In windows 7 there is a also warning, but the BAT85 is working now.
I will test some other file as far as I could.

Yeah, no idea why that warning, I can't replicate it in my windows 7 vm

Yeah, no idea why that warning, I can't replicate it in my windows 7 vm It's related to the cleanup part, so not critical (if you get leftover random-named folders you can clean them up manually)

Thanks for everything, please consider some protection for NULL input, it was dangerous, delete all C:\Users[UserName].
This issue could be closed by now.

the way SendTo works the input can't be null, isn't it?
but I do test anyway:
if (!(test-path -lit $($arg[0].Value.Trim('"')))) {write-host "ERROR! path not found" -fore Yellow; timeout -1; return}


Ignore changes on attributes fails using OctocatalogDiff::API::V1.catalog_diff

Description of problem

What did you do?

I've basically used the last example of https://github.com/github/octocatalog-diff/blob/master/doc/dev/api/v1/calls/catalog-diff.md#controlling-diffs and extended it a little bit.
...
catalog_diff_result = OctocatalogDiff::API::V1.catalog_diff(
  {
    :to_catalog   => catalog_new.path,
    :from_catalog => catalog_old.path,
    :ignore       => [
      {
        type: Regexp.new('\AFile\z'),
        attr: Regexp.new("\Aparameters\f(owner|group|ensure|alias)\z"),
      },
    ],
  }
)
...

What happened?

mbaur@example:~ $ ./diffenvs -e pp_4635 -n server666 --no-puppet
I, [2018-02-12T13:39:06.453395 #29060]  INFO -- : Running diffenvs for server123 (production <=> pp_4635)
/home/mbaur/.rvm/gems/ruby-2.4.0/gems/octocatalog-diff-1.5.2/lib/octocatalog-diff/catalog-diff/differ.rb:398:in `=~': no implicit conversion of Regexp into String (TypeError)
	from /home/mbaur/.rvm/gems/ruby-2.4.0/gems/octocatalog-diff-1.5.2/lib/octocatalog-diff/catalog-diff/differ.rb:398:in `ignore_match?'
	from /home/mbaur/.rvm/gems/ruby-2.4.0/gems/octocatalog-diff-1.5.2/lib/octocatalog-diff/catalog-diff/differ.rb:442:in `block in ignored?'
	from /home/mbaur/.rvm/rubies/ruby-2.4.0/lib/ruby/2.4.0/set.rb:324:in `each_key'
	from /home/mbaur/.rvm/rubies/ruby-2.4.0/lib/ruby/2.4.0/set.rb:324:in `each'
	from /home/mbaur/.rvm/gems/ruby-2.4.0/gems/octocatalog-diff-1.5.2/lib/octocatalog-diff/catalog-diff/differ.rb:441:in `ignored?'
	from /home/mbaur/.rvm/gems/ruby-2.4.0/gems/octocatalog-diff-1.5.2/lib/octocatalog-diff/catalog-diff/differ.rb:222:in `block in filter_diffs_for_ignored_items'
	from /home/mbaur/.rvm/gems/ruby-2.4.0/gems/octocatalog-diff-1.5.2/lib/octocatalog-diff/catalog-diff/differ.rb:222:in `reject!'
	from /home/mbaur/.rvm/gems/ruby-2.4.0/gems/octocatalog-diff-1.5.2/lib/octocatalog-diff/catalog-diff/differ.rb:222:in `filter_diffs_for_ignored_items'
	from /home/mbaur/.rvm/gems/ruby-2.4.0/gems/octocatalog-diff-1.5.2/lib/octocatalog-diff/catalog-diff/differ.rb:191:in `catdiff'
	from /home/mbaur/.rvm/gems/ruby-2.4.0/gems/octocatalog-diff-1.5.2/lib/octocatalog-diff/catalog-diff/differ.rb:80:in `diff'
	from /home/mbaur/.rvm/gems/ruby-2.4.0/gems/octocatalog-diff-1.5.2/lib/octocatalog-diff/cli/diffs.rb:34:in `diffs'
	from /home/mbaur/.rvm/gems/ruby-2.4.0/gems/octocatalog-diff-1.5.2/lib/octocatalog-diff/api/v1/catalog-diff.rb:54:in `catalog_diff'
	from /home/mbaur/.rvm/gems/ruby-2.4.0/gems/octocatalog-diff-1.5.2/lib/octocatalog-diff/api/v1.rb:19:in `catalog_diff'
	from diffenvs:140:in `<main>'

What did you expect to happen?
Changes on owner/group/ensure/alias parameters for file resources get ignored.

How can someone reproduce the problem?
The code above should reliably fail.

Command used and debugging output
See above

Platform and version information

Your OS: Ubuntu 14.04.5rzuvb
Your Ruby version: 2.4.0
Your version of Puppet: 5.3.4
Your version of octocatalog-diff: 1.5.2

Do the tests pass from a clean checkout?
Using octocatalog-diff from gem.

Anything else to add that you think will be helpful?

Workaround:
catalog_diff_result = OctocatalogDiff::API::V1.catalog_diff(
  {
    :to_catalog   => catalog_new.path,
    :from_catalog => catalog_old.path,
    :ignore       => [
      {
        type: Regexp.new('\AFile\z'),
        attr: "parameters\fensure"
      },
      {
        type: Regexp.new('\AFile\z'),
        attr: "parameters\falias"
      },
      {
        type: Regexp.new('\AFile\z'),
        attr: "parameters\fowner"
      },
      {
        type: Regexp.new('\AFile\z'),
        attr: "parameters\fgroup"
      },
    ],
  }
)


Rails 5.1 ActionView::Template::Error: undefined method `map' for nil:NilClass

After upgrading to Rails 5.1, I'm getting the following error originating from preloader_extension.rb:49
ActionView::Template::Error: undefined method `map' for nil:NilClass

Adding the following to line 49 seems to do the trick but I'm not sure what is causing this behavior.
return {} if @preloaded_records.empty?

Does anyone have any insight on why this might be happening? Below is the full stack trace
"/Users/youkib/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/activerecord-precount-0.6.3/lib/active_record/precount/preloader_extension.rb:50:in `load_records'"
"/Users/youkib/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/activerecord-5.1.0/lib/active_record/associations/preloader/association.rb:61:in `associated_records_by_owner'"
"/Users/youkib/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/activerecord-precount-0.6.3/lib/active_record/precount/preloader_extension.rb:21:in `preload'"
"/Users/youkib/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/activerecord-5.1.0/lib/active_record/associations/preloader/association.rb:19:in `run'"
"/Users/youkib/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/activerecord-5.1.0/lib/active_record/associations/preloader.rb:151:in `block (2 levels) in preloaders_for_one'"
"/Users/youkib/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/activerecord-5.1.0/lib/active_record/associations/preloader.rb:149:in `each'"
"/Users/youkib/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/activerecord-5.1.0/lib/active_record/associations/preloader.rb:149:in `map'"
"/Users/youkib/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/activerecord-5.1.0/lib/active_record/associations/preloader.rb:149:in `block in preloaders_for_one'"
"/Users/youkib/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/activerecord-5.1.0/lib/active_record/associations/preloader.rb:148:in `each'"
"/Users/youkib/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/activerecord-5.1.0/lib/active_record/associations/preloader.rb:148:in `flat_map'"
"/Users/youkib/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/activerecord-5.1.0/lib/active_record/associations/preloader.rb:148:in `preloaders_for_one'"
"/Users/youkib/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/activerecord-5.1.0/lib/active_record/associations/preloader.rb:115:in `preloaders_on'"
"/Users/youkib/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/activerecord-5.1.0/lib/active_record/associations/preloader.rb:102:in `block in preload'"
"/Users/youkib/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/activerecord-5.1.0/lib/active_record/associations/preloader.rb:101:in `each'"
"/Users/youkib/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/activerecord-5.1.0/lib/active_record/associations/preloader.rb:101:in `flat_map'"
"/Users/youkib/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/activerecord-5.1.0/lib/active_record/associations/preloader.rb:101:in `preload'"
"/Users/youkib/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/activerecord-5.1.0/lib/active_record/relation.rb:667:in `block in exec_queries'"
"/Users/youkib/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/activerecord-5.1.0/lib/active_record/relation.rb:665:in `each'"
"/Users/youkib/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/activerecord-5.1.0/lib/active_record/relation.rb:665:in `exec_queries'"
"/Users/youkib/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/activerecord-5.1.0/lib/active_record/relation.rb:547:in `load'"
"/Users/youkib/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/activerecord-5.1.0/lib/active_record/relation.rb:255:in `records'"
"/Users/youkib/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/activerecord-5.1.0/lib/active_record/relation.rb:251:in `to_a'"
"/Users/youkib/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/will_paginate-3.1.5/lib/will_paginate/active_record.rb:126:in `block in to_a'"
"/Users/youkib/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/will_paginate-3.1.5/lib/will_paginate/collection.rb:96:in `create'"
"/Users/youkib/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/will_paginate-3.1.5/lib/will_paginate/active_record.rb:125:in `to_a'"
"/Users/youkib/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/activerecord-5.1.0/lib/active_record/relation/finder_methods.rb:524:in `find_take_with_limit'"
"/Users/youkib/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/activerecord-5.1.0/lib/active_record/relation/finder_methods.rb:100:in `take'"
"/Users/youkib/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/activerecord-5.1.0/lib/active_record/relation.rb:639:in `inspect'"
...

Thanks in advance for any assistance and I'd be more than happy to make a pull request if this is the correct fix to the problem.

I'm not sure what's the correct fix. But surely your PR would be appreciated if all tests pass.
The current maintenance policy is this. I'll add anyone who can fix new issues to repository collaborators and gem owners.

I have encountered same error and succeeded to fix this problem, but I'm not sure my approach is appropiate too and still can't send pull request.
In ActiveRecord5.1 with activerecord-precount, there are two problems.
Firstly, ActiveRecord::Associations::CollectionAssociation#count has removed in AR 5.1, but activerecord-precount uses this method. (ref: https://github.com/rails/rails/pull/26383)
In my approach, simply extending the above method defined as extension.
Secondly, I cannot specified the cause of root not yet. Howerver, I have confirmed that it existes the difference of return type of records_for method between using AR 5.0,2 and AR 5.1 .
I have succeeded to pass all test by transforming the return value of records_for to the appropriate type against this problem, but I think this is not the fundamental solution.
I hope this will help.

@ysdanielkim
I have faced with the same problem, and fixed with https://github.com/k0kubun/activerecord-precount/pull/25 . Please retry after this patch is released.
@naoki-k
Thanks for your insightful comment. Though the deletion of ActiveRecord::Associations::CollectionAssociation#count is not this issues's root cause, this may
Have a look at the current CI with Rails v5.1.x build,

FYI @naoki-k

Howerver, I have confirmed that it existes the difference of return type of records_for method between using AR 5.0.2 and AR 5.1 .

This is because the records_for was removed at https://github.com/rails/rails/pull/26379

FYI @naoki-k
NoMethodError: undefined method count'` error was resolved at https://github.com/k0kubun/activerecord-precount/pull/27 , and your inspection was great helpful. Thanks!


Webpublisher: Fix use ftrack username in burnins

Brief description
Burnins were using default pypeclub username
Description
If ftrack username is found based on provided user email, it is used in burnins (left bottom corner)
Testing notes:

Publish via Webpublish with user email that doesn't map to default pypeclub username
Check bottome left corner of published burnin

Task linked: OP-2726 Webpublisher: Ftrack review burnin displaying pypeclub


SerDeTest.scala: update tests to match values for mem, cpu and disk

mem, cpu and disk is set to 128.0, 0.1 and 256.0 here:
src/main/scala/org/apache/mesos/chronos/scheduler/config/SchedulerConfiguration.scala

mem, cpu and disk is set to 0.0, 0.0 and 0.0 here:
src/main/scala/org/apache/mesos/chronos/scheduler/jobs/Jobs.scala

I suspect the non-zero defaults are only being set if ScheduleBasedJob is parsed from a strnig but I can't explain why we're seeing these failures now or for how long they have been present.

Could you please rebase against master?


booster = 'dart' doesn't utilize GPU to speed up?

I did some simple test in R with lasted master compilation, and I found when I set booster='dart' although GPU device was initialized, Volatile GPU-Util showed that GPU didn't run. booster='gbtree' run successfully.
R code
library(xgboost)
set.seed(111)
N <- 800000
p <- 50
X <- matrix(runif(N * p, 0, 1), ncol = p)
colnames(X) <- paste0("x", 1:p)
beta <- runif(p)
y <- X %*% beta #+ rnorm(N, mean = 0, sd  = 0.1)
tr <- sample.int(N, N * 0.75)

param1 <- list(nrounds = 20, eta = 0.3, subsample = 0.63, nthread = 1,
  lambda = 0, lambda_bias = 0, alpha = 0, gamma = 0, max_depth = 30,
  colsample_bytree = 0.4, num_parallel_tree = 1,
  min_child_weight = 10, predictor = "gpu_predictor",
  tree_method = "gpu_hist", grow_policy = "lossguide",
  booster = "dart", sample_type = "uniform",
  normalize_type = "tree", rate_drop = 0.2, skip_drop = 0.5,
  max_leaves = 1e4L, max_bin = 256, verbose = TRUE, gpu_id = 0)

param1$data <- X[tr,]
param1$label <- y[tr]

set.seed(2019)
bst1 <- do.call(xgboost::xgboost, param1)
test_pred1 <- predict(bst1, newdata = X[-tr,])

R info
> sessionInfo()
R version 3.5.1 (2018-07-02)
Platform: x86_64-pc-linux-gnu (64-bit)
Running under: Ubuntu 16.04.5 LTS

Matrix products: default
BLAS: /usr/lib/openblas-base/libblas.so.3
LAPACK: /usr/lib/libopenblasp-r0.2.18.so

locale:
 [1] LC_CTYPE=en_US.UTF-8       LC_NUMERIC=C               LC_TIME=en_US.UTF-8        LC_COLLATE=en_US.UTF-8     LC_MONETARY=en_US.UTF-8    LC_MESSAGES=en_US.UTF-8    LC_PAPER=en_US.UTF-8
 [8] LC_NAME=C                  LC_ADDRESS=C               LC_TELEPHONE=C             LC_MEASUREMENT=en_US.UTF-8 LC_IDENTIFICATION=C

attached base packages:
[1] stats     graphics  grDevices utils     datasets  parallel  methods   base

other attached packages:
[1] DBI_1.0.0      data.table_1.11.8

loaded via a namespace (and not attached):
[1] compiler_3.5.1       zoo_1.8-4            yaml_2.2.0           Rcpp_1.0.0           grid_3.5.

Will prepare a fix soon, thanks.


feat: improve block content ux

Codecov Report

Merging #83 (c2539c4) into master (68c72f3) will increase coverage by 1.87%.
The diff coverage is 89.66%.

@@            Coverage Diff             @@
##           master      #83      +/-   ##
==========================================
+ Coverage   67.20%   69.06%   +1.87%
==========================================
  Files          76       78       +2
  Lines         948     1031      +83
  Branches        1        1
==========================================
+ Hits          637      712      +75
- Misses        311      319       +8

Impacted Files
Coverage Δ

...otion/model/common/richtext/RichTextFragment.scala
28.58% <ø> (ø)

.../src/main/scala/zio/notion/dsl/PatchedColumn.scala
90.00% <50.00%> (ø)

...rc/main/scala/zio/notion/dsl/BlockContentDSL.scala
90.25% <90.25%> (ø)

...core/src/main/scala/zio/notion/dsl/ColumnDSL.scala
100.00% <100.00%> (ø)

...la/zio/notion/model/common/richtext/RichText.scala
100.00% <100.00%> (ø)


Cannot use union result as a subquery

I've tried to use result of abc and efg tables as a subquery to calculate the average value from those union tables
 abc, efg = Tables('abc', 'efg')
 hij = Query.from_(abc).select(abc.t).union(Query.from_(efg).select(efg.t))
 q = Query.from_(hij).select(an.Avg(hij.t))

but actual query I get is
'SELECT AVG("sq0"."t") FROM (SELECT "t" FROM "abc") "sq0"'

expected query should be
'SELECT AVG("sq0"."t") FROM ((SELECT "t" FROM "abc") UNION (SELECT "t" FROM "efg")) "sq0"'

Am I doing something wrong? or there is any other ways to achieve this.
Thanks

Hi @eddyzhow, thanks for reporting this. It seem like a bug in pypika. I have labelled this issue as a a bug for now.

Thanks for the detailed issue ticket. A PR has been raised.

This is now in version 0.9.1


Disable color transition when loading pages in dark mode

Ideally, the transition should occur only when someone clicks on the toggle theme button, not when someone with dark mode enabled loads a page.
See: https://css-tricks.com/transitions-only-after-page-load

+1


Add documentation for Set/Remove-PnPAdaptiveScopeProperty

Before creating a pull request, make sure that you have read the contribution file located at
https://github.com/pnp/powerShell/blob/dev/CONTRIBUTING.md
Type

[X] Bug Fix
[X] New Feature
[ ] Sample

Related Issues?
Documentation updates
What is in this Pull Request ?

Added documentation for Set-PnPAdaptiveScopeProperty
Added documentation for Remove-PnPAdaptiveScopeProperty
Removed parameters that do not exist in Remove-PnPPropertyBagValue from documentation

Tagging @erwinvanhunen for review

Thanks @brenle !


zuul: remove sqlreporter from the pipeline

This reporter is no longer supported.

I fast-tracked this change to fix the tenant, otherwise the config would not load.

Thanks!


Qt eventFilters only work when "all" Qt plugins are selected

I'm trying to change the background of a program using eventFilter(). The background change should take place after hovering the mouse over the buttons. But after compiling with Nuitka, this function does not work. I wrote a small example and added a video so that if you want you can reproduce the error.
Exemple link: https://drive.google.com/file/d/1e9SUbP0nnDTJ6H2_wA_8-l9PGAC1WbhY/view?usp=sharing
Those were the arguments that I used on Windows:
nuitka --mingw64 --onefile   --plugin-enable=pylint-warnings --plugin-enable=qt-plugins --include-data-file=images\*=images\ --windows-onefile-tempdir main.py
https://doc.qt.io/archives/qtforpython-5.12/PySide2/QtCore/QObject.html#PySide2.QtCore.PySide2.QtCore.QObject.installEventFilter
https://srinikom.github.io/pyside-docs/PySide/QtCore/QObject.html#PySide.QtCore.PySide.QtCore.QObject.installEventFilter

My config:
Windows 10 21H1 (Build 19043)
Python 3.8.8
Nuitka==<IP_ADDRESS>
PyQt5==5.15.4
PyQt5-Qt5==5.15.2
PyQt5-sip==12.8.1

I can confirm this is not working, and it's not even working with the private PySide2 compile that has my PySide 6.1 changes contained.

Ah, with "all" qt plugins, it's working for me with my private PySide2 compile. Maybe also will for your PyQt5. I will have to investigate, which missing plugin causes this, and if it shouldn't be part of the "sensible" default.

As an aside, Nuitka commercial 0.6.14 will be able to include the Qt images inside the standalone binary already, fully protected.
The normal version has a --include-data-dir=images=images that includes the whole folder as of 0.6.14 that you can use if you are not commercial, and it's a bit simpler.
And I don't quite get, why you needed to replace the slashes in the path. Not having a working URL that is relative to __file__ is a bug, as you require execution inside the dist folder, or inside the python file folder, which is unfriendly.
And of course, once released, PySide6.1 will also work.

With --include-qt-plugins=sensible,styles it works for me. I am adding that for 0.6.14 release to be included in sensible, as without it no hover events are generated.

With --include-qt-plugins=sensible,styles it works for me. I am adding that for 0.6.14 release to be included in sensible, as without it no hover events are generated.

This worked very well for me! Thank you so much for the quick return

I'm looking forward to version 0.6.14.
Will --include-data-dir= work recursively? this is something that might be good to implement in future versions of Nuitka

And I don't quite get, why you needed to replace the slashes in the path. Not having a working URL that is relative to __file__ is a bug, as you require execution inside the dist folder, or inside the python file folder, which is unfriendly.

Apparently the setStyleSheet method only accepts this bar /
But I'm not sure, maybe I'm using it the wrong way

Again I thank the Nuitka project and its quick return, I think I can finish the issue now.


Deploy to PyPI on master

Changes proposed in this pull request:

Deploy non-tagged builds to Test PyPI
Deploy tagged builds to live PyPI
Like https://github.com/hugovk/pypistats/pull/45, but only for a single $TOXENV value, in case the lint build job also uses Python 3.7 at some point

Re-encrypted the password from the pylast directory, committed directly to master in https://github.com/pylast/pylast/commit/69b54e8c450086c6b564c0ee8aaea51d8512ba92.
✅ Non-tagged master build:

3.6 skipped, as expected:

https://travis-ci.org/pylast/pylast/jobs/488218321#L887

3.7 deployed to Test PyPI, as expected:

https://travis-ci.org/pylast/pylast/jobs/488218320#L641
https://test.pypi.org/project/pylast/3.1.0.dev0/#files

✅ Tagged master build:

3.6 skipped, as expected:

https://travis-ci.org/pylast/pylast/jobs/502961893#L911

3.7 deployed to PyPI, as expected:

https://travis-ci.org/pylast/pylast/jobs/502961892#L649

https://pypi.org/project/pylast/3.1.0/#files

✅ Also updated the release checklist:

https://github.com/pylast/pylast/commit/a5c53dbf5402535ca79c66cebe1ca1e1a3dc7baa#diff-60cd2f42437b5cb3c2c09391a6f4c54a


Support host_inventory['platform'] and host_inventory['platform_version']

You can get host_inventory['platform'] and host_inventory['platform_version'] with this fix.
host_inventory['platform'] is not compatible with Ohai, for example, it returns redhat with CentOS or other Red Hat based Linux.

Released as v2.12.1.


Doesn't work for second time for Collection view

Hi,
First of all i would like to thank to share this wonderful Code.
I have implemented the same on Collection view for first scroll it works proper but when i do the scroll for second time it doesn't call the function again.
I just have written in my code is
__weak MyClassName *weakSelf = self;
[self.collectionView addInfiniteScrollingWithActionHandler:^{
[weakSelf parseAfterScroll];
}];

You need to call [self.collectionView.infiniteScrollingView stopAnimating];
The problem is the state of InfitieScorllView is not being updated to stopped.  It will not trigger the action handler unless the state is set to stopped.  Calling stopAnimating after your action handler is completed resets the state.


⚠ Roblox Site has degraded performance

In 676d0f7, Roblox Site (https://www.roblox.com) experienced degraded performance:

HTTP code: 200
Response time: 3580 ms

Resolved: Roblox Site performance has improved in b879cc3 after 8 minutes.


Add a metadata field to the Response

Add a metadata field to the Response that can be populated by model providers. (#698)
DashScope callings fill the requestId as the response metadata.

Summary by CodeRabbit

New Features

Enhanced response objects to include metadata, improving traceability and context of responses.
Introduced metadata handling in various models, enabling enriched data processing and response generation.

Refactor

Updated methods across models to support metadata inclusion, ensuring consistency and reliability in data handling.

Tests

Added comprehensive tests to validate the presence and correctness of metadata in responses, ensuring robustness and accuracy.

@langchain4j please take a look. Then I can continue to promote the implementation of features such as function-call.

@jiangsier-xyz sorry for the late reply. I foresee that the API will be changed in the near future, so ChatLanguageModel will not return Result<T> any more. We can still merge this PR if you think it is required (at least as a temporary solution).


An error in bash script

Hello Dear All,
there is an error in bash script https://docs.microsoft.com/en-us/azure/devops/repos/git/set-up-credential-managers?view=azure-devops
Instead
git-credential-manager install
supposed to be
git credential-manager install
without "-" between git and credential.
King regards,
Yelena.

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

ID: 6c2f260d-9321-d669-172d-01289c233263
Version Independent ID: 14e9ae18-df9d-7555-902c-3e8b4c2b2f22
Content: Connect to your Git repos using credential managers - Azure Repos
Content Source: docs/repos/git/set-up-credential-managers.md
Product: devops
Technology: devops-code-git
GitHub Login: @apawast
Microsoft Alias: sdanie

@yelenapakumirzakova -- Yelena, thank you for bringing this to my attention.  It has been fixed in the documentation and should be live by tomorrow.


Add customPaymentMethods option for stripe.elements

Summary & motivation

Add customPaymentMethods option for stripe.elements
Add selectedPaymentMethod in elements.submit resolved promise

Testing & documentation

Added valid tests.

Hi, I found this issue because our production Stripe.JS integration today crashed in an unexpected way.
Actually we're importing this library with a script inside the head of our website in this way:
<script src="https://js.stripe.com/v3/">
After some debugging we found that now elements.submit resolved promise returns an object containing this new key selectedPaymentMethod, and I found this issue about it that has not been merged yet.
Until now we were matching for an empty object (with Objects.keys) to check for a failure in the submit operation but from today all of our payments started to fail because of the new key.
We fixed this switching to result.hasOwnProperty("error") but I'm wondering why it has been changed in this way and how we can avoid this kind of unexpected upstream updates from your library?
I'm almost sure that the relative documentation changed in the last days too: where can I found the docs changelog?

Until now we were matching for an empty object (with Objects.keys) to check for a failure in the submit operation but from today all of our payments started to fail because of the new key.
We fixed this switching to result.hasOwnProperty("error") but I'm wondering why it has been changed in this way and how we can avoid this kind of unexpected upstream updates from your library?
I'm almost sure that the relative documentation changed in the last days too: where can I found the docs changelog?

Thank you for writing in. You're right that there was a recent update to the elements.submit method, which now resolves with a new key, selectedPaymentMethod. Our documentation was indeed updated a couple of days ago to reflect this change.
We recommend coding defensively when handling resolved promises by explicitly checking for an error object, as you've done with result.hasOwnProperty("error"). This approach can help prevent similar issues arising from unexpected updates.
If you have further questions or need additional assistance, please feel free to reach out to Stripe Support.


Editor scrolls to bottom when pressing enter in the middle of a big paragraph

same issue https://github.com/facebook/draft-js/issues/304

fix it after next version


There is no Input Data Providers Type, Help me.

Describe the bug

I installed MRTK(Foundation, Example) in Unity.
I went into the HandInteractionExamples Scene.
I saw the bug, Console message is

'Unable to register IMixedRealityInputDeviceManager data provider (Windows Mixed Reality Device Manager) because the value of concreteType is null.
This may be caused by code being stripped during linking. The link.xml file in the MixedRealityToolkit.Generated folder is used to control code preservation.'
'Unable to register IMixedRealityInputDeviceManager data provider (Windows Mixed Reality Eye Gaze Provider) because the value of concreteType is null.
This may be caused by code being stripped during linking. The link.xml file in the MixedRealityToolkit.Generated folder is used to control code preservation.'

I took the route below to investigate the problem.
Hierarchy > MixedRealityToolkit > MixedRealityToolkit Script > Input > Input Data Providers > Windows Mixed Reality Device Manager > Type

The type item is marked in red, and there is an error.

There are two things marked in red ones.

Windows Mixed Reality Device Manager > Type
Windows Mixed Reality Eye Gaze Provider > Type

To reproduce
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots

If applicable, add screenshots to help explain your problem.
Your setup (please complete the following information)

Unity Version 2020.2.7
I tested Unity 2019.4.8  version, but the same problem occurred.
MRTK Version 2.6.1
I tested MRTK 2.5.3  version, but the same problem occurred.

Target platform (please complete the following information)

HoloLens 2

Additional context
Add any other context about the problem here.

我遇到和你一样的问题，请问这个bug解决了吗？如何解决的呢

Please see the solution in #9528.


As a dev, I'd like to add SSO to my app using MSAL Angular so that users in Teams don't need to sign in one more time

Depends on #5

Install the @microsoft/teams-js dependency. Add TeamsGuard and TeamsService. Extend AAD app with:

identifier URI (the URI must be built as follows: api://domain.from.tunnel/appId

OAuth2 permission

reply URL pointing to the tunnel URL

preauthorized applications (appIds are fixed values as shown in the snippet, permissionIds match the ID of the OAuth2 permission defined above):
 "preAuthorizedApplications": [
 	{
 		"appId": "5e3ce6c0-2b1f-4285-8d4b-75ee78787346",
 		"permissionIds": [
 			"cf38eb5b-8fcd-4697-9bd5-d80b7f98dfc5"
 		]
 	},
 	{
 		"appId": "1fec8e78-bce4-4aaf-ab1b-5451cc387264",
 		"permissionIds": [
 			"cf38eb5b-8fcd-4697-9bd5-d80b7f98dfc5"
 		]
 	}
 ],

This is included in v0.1


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-k8s-29-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 5002

Questions ?
Please refer to the Backport tool documentation


Add databse dump to use laravel latest feature.

I noticed huge speed improvements on running migrations when I switched from tradition migration to schema dumps.
It will be great to add support for schema dumps.

See my reply on the PR.


README

Hi, perhaps worth adding to README.md that the extensions are under: Menu --> File --> Extensions to make it easier for new users to find. Thanks for the extension.

I would also like to know what are the "Node and Pod" extensions. As a newer Kubernetes and Lens user, I don't know what the extensions are or why I would want them beyond a vanilla OpenLens installation.

I linked here because it was suggested that this added log viewing back in. I can't find the interface to open logs! 😄 . Thus, i'm not clear what the extension does. Would be super helpful to drop a quick note about the provided functionality


backport: jsonrpc http server: expose health endpoint (#11395)

Backport of #11395

Merging is blocked
The base branch requires all commits to be signed.


CI: pybind11 adjustments

pybind11 2.9.0 release -- bump the vfx2022 and "latest" CI tests to
use it.
Adjust other CI test pybind11 versions to get good coverage across all
the pybind11 releases, and to synchronize the vfx platfom tests with
the pybind11 release that was current for its respective year.
Sync build_scripts/build_pybind11.bash with the latest in OIIO, including
bumping the default pybind11 version we download to 2.8.1.
Opportunistic touch-up of warnings about llvm version to bring
into sync with our actual current requirements.

Signed-off-by: Larry Gritz<EMAIL_ADDRESS>

Any objections?


[Feature Request ]Expose / lock seed after generated?

It would be great if I can have the ability to lock the seed when I like to keep the overall look of the image.

I know I can modify the seed in json file. But I hope that when I use a random (-1) seed to draw, whenever an effect is good to me, I hope to know its seed value, so that I can reproduce with the same seed in the next drawing to produce images. This seed cannot be determined from the beginning.


rip should be marked as archived and/or indicate users should look at uv

I've written many posts that have pointed to rip, as well as seen blog posts linked around that prefix-dev wrote, it would be good if people finding this project anew didn't think it was being actively maintained.

As reference: https://github.com/astral-sh/uv/issues/1572#issuecomment-1957318567


Bump controller-gen version from 0.9.2 to 0.13.0

Description
This PR update the controller-gen version from 0.9.2 to 0.13.0

/rebase test=true

/merge


WFLY-9477: avoid registering ROOT.war as default web module more than…

… once
Issue: WFLY-9477
default-web-module is an optional attribute with default value "ROOT.war" but because the module names have to be unique per server it essentially becomes required. Since you can control which host to deploy to in other ways the simplest solution would be to remove the default value. However some users may depend on the default value so I propose to register "ROOT.war" once and then ignore it if it's used by other hosts.

@fl4via Please review.

Retest this please

@bstansberry done


Update Gradle to 7.6 and use v3 GitHub Actions

I updated the gradle wrapper to use the latest version so CI can be done against JDK 19
Also noticed that the Github Actions were old versions so updated to the v3 ones
During local build noticed exceptions due to tests being run with java.util.logging.manager set to use log4j but the log4j jars were not available to the test classpath so have fixed that as well

@kallestenflo rebased the PR so that there's no conflicts

closing as this is superseded by #993 and #995


error: the trait bound time::OffsetDateTime: OutputType is not satisfied

I'm trying to use this struct as SimpleObject:
use async_graphql::{CSimpleObject};

#[derive(SimpleObject)]
struct Player {
    id: String,
    created_at: time::OffsetDateTime,
    email: String,
}

but I'm getting this error:
error[E0277]: the trait bound `time::OffsetDateTime: OutputType` is not satisfied
   |
50 | #[derive(SimpleObject)]
   |          ^^^^^^^^^^^^ the trait `OutputType` is not implemented for `time::OffsetDateTime`
   |
   = help: the following other types implement trait `OutputType`:
             &'a [T]
             &'impl0 T
             Arc<T>
             Arc<[T]>
             BTreeMap<K, V>
             BTreeSet<T>
             Cow<'a, T>
             Edge<Cursor, Node, EdgeFields, Name>
           and 56 others
   = note: required because of the requirements on the impl of `OutputType` for `&time::OffsetDateTime`
   = note: this error originates in the derive macro `SimpleObject` (in Nightly builds, run with -Z macro-backtrace for more info)

What can I do to fix this?
I tried to search but I cannot understand what to do.

I saw https://github.com/async-graphql/async-graphql/pull/791. I'm trying...

It works.

You must enable time feature.


Add PropertyChanged support

Currently propertychanged is fired for the card object when it gets updated, but if you bind to properties on the card they do not fire a propertychanged event. For instance if you bind an itemscontrol to card.attachments the application is not notified that a change has occurred.
A simple way to solve this is to use an observable collection, this will notify if an item is added or deleted, but wont notify if a property on an object in the collection has changed, not sure if that is needed though.
This is an issue with all the collections on a card I am binding too, I'm not sure if this is true for other properties on Card.

I currently have a work around for this by wrapping the Card class and exposing its properties through my view model. In the view model I listen for the updated event and just fire propertychanged for everything I'm bound to, however this is not desirable as it causes everything to re-evaluate its bindings instead of what has changed.

I like the idea, but I need to investigate whether INotifyPropertyChanged is a concept used outside of WPF (the only place I've used it).  If it's a more general idea, then I don't have a problem making those cards conform.  However if it's something that's explicitly tied to WPF, I'd prefer that three entities in Manatee.Trello be treated as domain- or data-access-level classes and you create specific view models to represent them (as you've done in your workaround).

SO has a good post with several decent opinions on using IPropertyChanged.  The general consensus is that individual events should be created.
That said, the Updated event as it's currently implemented is effectively a more strongly typed version of PropertyChanged anyway.
I think this would be a good change.  I'll also implement INotifyCollectionChanged on the collections.  Finally, Updated will be marked obsolete.

I understand, yes property changed events are for wpf, I don't see them being used elsewhere, but they can be. It's just a software pattern that uses events to let other software know a property has changed. Wpf uses it for databinding.

In reworking this for a bit, I just realized that Updated is not the same as PropertyChanged.  Specifically, a single Updated notifies that one or more properties have changed; PropertyChanged only notifies of a single property.  This means that an entity would have to raise PropertyChanged for each property.
I like the single event approach for this library.

I do however like the idea of having view-model-compatible entities.  It may be beneficial to publish these in a separate library.  @scrubs767 is that something you'd be interested in?  I don't imagine such a project would take too long.

I would have to take a look. Looking at Manatee.Trello as a black box, is there a way to tell what has changed?
Some questions:
From what I understand, the Card requests an update, how does it know a change has occurred?
When a change occurs does it just make a new request for all the information again, or does it just receive the updates?
Worst case scenario, I could wrap Card and listen for updates, then poll each property for changes including every property on every object in a collection.

@scrubs767 have you read through the wiki?  It explains the updating strategy.  And yes, you're correct.
That line is really just a sanity check to ensure that it's not merging a JSON object onto itself.  It has been a while since I wrote that.  It may be worth reviewing that code again to see if I can make it more efficient.

@scrubs767 I'm closing this issue for now.  If you'd like me to, I can work up a view model package that supplies notifying entities.  Otherwise I think your workaround is the best approach.

I think you should close it, and I think you should not work on this because in order for this to work as expected from a wpf developer, any and everything that a developer is listening to would need to automatically refresh on a pretty short timer. This would be a whole bunch of network traffic, most of which probably wont be needed. It would be better to let the developer decide when to update.
A much better way to support this would be to add an event that get fired when a property gets updated on an object, this could be done in the synchronizationcontext (maybe mergeproperties, not sure though, I didnt get a chance to look at it), throw one event listing all the properties updated. This way a developer could call update on something, and their bindings would still work properly. This event should only get fired if the properties are actually different.
Here is an example:
A window is displayed showing the card information, the developer added a timer to refresh the card every 60 seconds. This window has bindings to properties on the card. Someone through the web interface added a label and moved its list. When the timer kicks off and the card is refreshed an event is fired and the property names are added to the eventargs. Now the wpf developers code can listen for that event and fire off the appropriate propertychanged events. The UI updates itself.
A developer could use that architecture in any way because you arent tying yourself to WPF, that type of thing could be used in any application, consol, WPF WinForms droid...(I know nothing about IOS), but its plain c#.

I left that comment in place, but after writing it and thinking about it for a bit, none of its needed really. Its just as easy to run a timer calling updated and then throwing a property changed on that card.
The same thing could be said for a list of cards (or anything) return from a search, just rerun the search again.


fix(adapters): extend adapters

fix issue #201
Description

Related Issue(s)

Screenshots

Checklist

[ ] I've read the CONTRIBUTING guidelines.

Great spot. Thank you


Web and code documentation mismatch?

Laravel Version: 5.5.28

Description:
I am new at Laravel, sorry if I misunderstood something.
Please look at the first comment at the function handleTransactionException (framework/src/Illuminate/Database/Concerns/ManagesTransactions.php:61)
It says following: «On a deadlock, MySQL rolls back the entire transaction so we can't just retry the query. We have to throw this exception all the way out and let the developer handle it in another way. »
Does it comment correct? Official documentation (https://laravel.com/docs/5.5/database#database-transactions) says that Laravel can dealing with deadlocks automatically and I am supposed to do nothing in that case.

That comment is related to nested transactions, thus the check for $this->transactions > 1.
https://github.com/laravel/framework/blob/a926eede7a37b04123c103c398664a6f0389d0e0/src/Illuminate/Database/Concerns/ManagesTransactions.php#L61-L69

@sisve Thank you very much, it was not obvious for me


Custom Layers

Do we want custom layers in here? Soon I will need the NALU as well. Not sure if this belongs in here

Codecov Report

Merging #34 into master will decrease coverage by 2.92%.
The diff coverage is 11.11%.

@@            Coverage Diff             @@
##           master      #34      +/-   ##
==========================================
- Coverage   86.95%   84.03%   -2.93%
==========================================
  Files          11       12       +1
  Lines         230      238       +8
==========================================
  Hits          200      200
- Misses         30       38       +8

Impacted Files
Coverage Δ

src/GenerativeModels.jl
100% <ø> (ø)
:arrow_up:

src/utils/layers.jl
0% <0%> (ø)

src/utils/saveload.jl
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Making gun fire on trigger press instead of release

The gun felt really weird in the demo because it wouldn't fire until you
released the trigger, so I made it work with UseButtonDown instead of
UseButtonUp.
I used InteractingUpdate() directly to keep this change contained in the
gun file, because I wasn't sure if you would want to add UseButtonDown() to NVRInteractable.

Thanks @MattRix, I just added in a UseButtonDown and switched it to use that. You're totally right, that was super weird haha.


[Task] Pages that use grid are slightly differently aligned than ones that use the FormWrapper

Pages that use grid are slightly differently aligned than ones that use the FormWrapper
Current behavior

Provide point of contact's spacing of the content wrapper on the page looks a little off for instance when compared to the other pages (ask @billhimmelsbach to show you)

@billhimmelsbach
Is the issue you're seeing related to this issue I flagged: https://github.com/cfpb/sbl-frontend/issues/529

Duplicate of #529


Subclassed models don't carry through to the Layout

It's easiest to describe this bug via an example...
The following works:
public class Model {}
View.cshtml:
@model Model
@{ _Layout = "Layout"; }
...
Layout.cshtml:
@model Model
...
The following doesn't work:
public class Model {}
public class Model2 : Model {}
View.cshtml:
@model Model2
@{ _Layout = "Layout"; }
...
Layout.cshtml:
@model Model
...
The first example works fine but when I try to parse the 2nd example, I get the following exception:
System.NullReferenceException was unhandled
Message=Object reference not set to an instance of an object.
Source=l1lqrdfo
StackTrace:
at CompiledRazorTemplates.Dynamic.cddfefecbdb.Execute()
at RazorEngine.Templating.TemplateBase.RazorEngine.Templating.ITemplate.Run(ExecuteContext context)
at RazorEngine.Templating.TemplateBase.RazorEngine.Templating.ITemplate.Run(ExecuteContext context)
at RazorEngine.Templating.TemplateService.Run(ITemplate template)
at RazorEngine.Templating.TemplateService.Parse[T](String razorTemplate, T model, String name)
at ConsoleApplication2.Program.Main4() in d:\Temp\ConsoleApplication2\Program.cs:line 62
at ConsoleApplication2.Program.Main(String[] args) in d:\Temp\ConsoleApplication2\Program.cs:line 18
at System.AppDomain._nExecuteAssembly(RuntimeAssembly assembly, String[] args)
at System.AppDomain.ExecuteAssembly(String assemblyFile, Evidence assemblySecurity, String[] args)
at Microsoft.VisualStudio.HostingProcess.HostProc.RunUsersAssembly()
at System.Threading.ThreadHelper.ThreadStart_Context(Object state)
at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean ignoreSyncCtx)
at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state)
at System.Threading.ThreadHelper.ThreadStart()
InnerException:

I see what you are describing. Will investigate.


HTTP error 502.5 in Chrome

I have just followed the instructions in VS 2017 to update installed NuGet dependencies for AspNetCore from 2.1.0-preview1 final to 2.1.1.
Now I can't even debug into the start file

Release 2.1.1 is still being deployed, please use 2.1.0 instead.

@dotNetAthlete if you're still having issues, please file another issue.


Update kubernetes-cluster-on-prem-add-ingress-controller-service-anno…

…s.md
Small changes only; post-migration formatting.
Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the maintainers.
What Type of PR is This?

[ ] Issue
[ ] Feature
[x ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screen Shoot.

Preview environment: https://ci.pr.harness.io/pr-241


Show info after application start

handy with the docker one-line deployer

this is classic only, as that's what the docker container uses at present (and the normal brooklyn dist), and it was easier to get going.
fully accept that when we move to osgi we'll have to change and do this all another way.
BTW i had been thinking the move to osgi would drop these args, most args in fact, in favour of config files and sending additional commands to the server afterwards, and we'd have to rebuild the Dockerfile of course.  however it's worth bearing in mind that that the multi-command approach is a little tedious in docker-land however, as containers prefer the "just-one-process" model ... and many shells like alpine don't have job control to be able to background.  entirely possible but maybe worth a discussion about whether to send startup instructions / duplicate this flag to brooklyn osgi or not.

LGTM in that case, merging.


Dluksza include ref in upload request

Re-applied #969 against the current master, fixing some merge conflicts.

Echoing @sinbad's comments in https://github.com/github/git-lfs/pull/969#issuecomment-187685942:

I'm not crazy about how this makes the LFS push invoked by the pre-push hook semantically different to using git lfs push.

I'm sure it's not intentional. The fetch command sends it unless the --all flag is used. scanAll() only returns a slice of lfs pointers with no ref information.
None of the git lfs push calls because of how uploadPointers() is used.
I think scanAll() would have to change to return LFS pointers along with the ref that they're in. But I don't think git rev-list returns that info. If it did, it could loop through the refs, starting transfer queues for each one, like the lfs pre-push command.

Instead of using git rev-list --all, any thoughts on getting the refs first with git show-ref --heads --tags? At least for pushes. Fetches probably do want the remote refs too (but not stash).

If it did, it could loop through the refs, starting transfer queues for each one, like the lfs pre-push command.

I assume you're talking about git lfs push --all only? The problem I can see with the above is that the same LFS object can be reached by multiple refs, so you're going to get upload requests for the same oid duplicated in different queues. You'd really have to serialise the pushing of each ref to avoid that - it probably would only result in duplicated uploads where the one that finishes last is thrown away (same as if 2 people uploaded simultaneously) but it becomes so much more likely it's worth avoiding.

I assume you're talking about git lfs push --all only?

No, all code paths just build up a list of lfs pointers first, before uploading.
https://github.com/github/git-lfs/blob/master/commands/command_push.go#L55-L74
The pre-push command relies on the transfer queue's Wait() function.
https://github.com/github/git-lfs/blob/6b00ebc996a7cd2b28fdcd027fe527c48975c113/commands/command_pre_push.go#L59-L74
You're right that it does send the OIDs to the batch api multiple times, potentially. But it doesn't transfer the same one multiple times in a single command. We could probably solve this by tracking the successful uploads, and preventing them from being added to another ref's transfer queue.

I made a few code spikes that illustrate how we could refactor those commands. #1038 changes lfs push --all so it runs git show-ref to get the list of local refs. #1040 updates lfs pre-push so it doesn't attempt to upload duplicate objects that have been uploaded in the same process (from another ref).

Wow, this is a major yak shave :/  Redoing this in #1128....


Use Biocontainer images rather than lab-specific images

Currently, the pipeline uses Docker images that are stored in the lab's Docker Hub organization. Maintaining these images is tedious and will likely phase out. Replace images from zavolab with those from Biocontainers. This may require adding missing tools on BioConda.

The images r-zavolab and r-tidyverse  when changed by the official ones or the biocontainers rise an error as they are missing the library optparse.
The same thing happens with the python image and the module pysam.
Should these be added as dependencies?
Also, the image from oligomap cannot be found besides the one from zavolanlab - which make sense as it has bee created here. Should I leave it as it is?

Let's see what we can do:
Python: I guess you can use the official Python image wherever pysam is not required (pretty sure not all scripts need it). For those that do need it, you can try the pysam BioContainers image (try to use the same pysam version we are currently using).
For oligomap and R, I guess you would really need to contribute the images to BioContainers yourself (ideally, for R, you'd even create one image per tool/script!). However, that's a bit of an overkill and would take too much time. Please address what you can (including Python as described above) with a PR, then create two more issues for conributing BioConductor images for (1) oligomap and (2) R and/or tidyverse with optparse. We can leave them as very low priority to fix when we have time. Link the issues here and then we can close this issue when the PR is merged.

Oops, accidentally closed this. Re-opened for a minute before your PR will close it ;-)


[Gemserk TextureBleeder] missing code License

https://github.com/gemserk/imageprocessing/issues/5

Readme.txt contains warning.

Gemserk code deleted: https://github.com/JFixby/TextureBleeder/commit/74cf634527f8bdcc2473a9898f3143b80b58d6da