Scores not output to output file when running containerized Gnina

Issue summary
I previously ran Gnina outside of a docker container and scores and minimzed affinity and CNN scores as comments were appended directly below the SDF block for each pose. I have since moved Gnina into a docker container and now I find only a single comment for minimzed affinity at 0 and no CNN scores, i.e.:

0.00000
However, scores are output to the terminal, e.g.:
mode |  affinity  |    CNN     |   CNN
| (kcal/mol) | pose score | affinity
-----+------------+------------+----------
1       -7.11       0.7744      6.065
Gnina otherwise seems to run normally, generating plausible poses. Is there a setting in Gnina which might result in this behavior?
Steps to reproduce
I have run Gnina within a docker container as follows:
docker run --gpus all --name name1 -v /home/roc/dir_name/:/home/src/dir_name/ --shm-size 8gb pardela gnina -r /home/src/dir_name/temp/temp_gnina_receptor.pdb --ligand /home/src/dir_name/temp/temp_gnina_ligand.sdf --autobox_ligand /home/src/dir_name/temp/temp_gnina_receptor.pdb -o /home/src/dir_name/temp/temp_gnina_poses.sdf --exhaustiveness 64 --num_modes 10 --device 0 --cnn_scoring "rescore" --pose_sort_order CNNscore --min_rmsd_filter 1.0
Your system configuration
Operating system: Windows 11
Compiler: 11.4.0
CUDA version (if applicable): 12.3
CUDNN version (if applicable): <IP_ADDRESS>
Python version: 3.11

I have not seen that before. If you can share your docker container I can look into it.

Ok, thanks. The dockerfile is as follows:
`FROM nvidia/cuda:11.7.1-devel-ubuntu22.04
ARG OSVERSION "ubuntu2204"
ARG ARCH "x86_64"
ARG NVIDIA_PUBKEY="3bf863cc.pub"
ARG ENVNAME="gnina"
ENV PROJECT_DIR=${ENVNAME}
ENV IMAGE_NAME=${IMAGE_NAME_ARG}
ENV TZ=Europe/Lisbon
WORKDIR /home/nas
WORKDIR /home/src/${PROJECT_DIR}
RUN ln -snf /usr/share/zoneinfo/$TZ /etc/localtime && echo $TZ > /etc/timezone
RUN apt-key adv --fetch-keys https://developer.download.nvidia.com/compute/cuda/repos/${OSVERSION}/${ARCH}/${NVIDIA_PUBKEY}
RUN apt -y update &&
apt -y upgrade &&
apt-get -y install
wget
sudo
nano
build-essential
git
cmake
wget
libboost-all-dev
libeigen3-dev
libgoogle-glog-dev
libprotobuf-dev
protobuf-compiler
libhdf5-dev
libatlas-base-dev
python3-dev
librdkit-dev
python3-numpy
python3-pip
python3-pytest
openbabel
libopenbabel-dev
python3-openbabel
RUN git clone https://github.com/gnina/gnina.git &&
cd gnina &&
mkdir build &&
cd build &&
cmake -DCUDA_ARCH_NAME=All .. &&
make &&
make install
COPY environment_gnina.yml ../environment.yml
RUN wget https://repo.anaconda.com/miniconda/Miniconda3-py39_4.12.0-Linux-x86_64.sh &&
chmod +x Miniconda3-py39_4.12.0-Linux-x86_64.sh && ./Miniconda3-py39_4.12.0-Linux-x86_64.sh -b -p /opt/conda &&
export PATH=/opt/conda/bin:/opt/conda/condabin:$PATH &&
conda init &&
conda env create -f ../environment.yml &&
echo ". /opt/conda/etc/profile.d/conda.sh" >> ~/.bashrc &&
echo 'export PYTHONPATH="/home/src/${PROJECT_DIR}/:$PYTHONPATH"' >> ~/.bashrc &&
echo "conda activate ${CONDA_ENV}" >> ~/.bashrc &&
rm Miniconda3-py39_4.12.0-Linux-x86_64.sh
CMD bash`
and the environment.yml files is:
`name: env
channels:

conda-forge
anaconda
dependencies:
python=3.10
numpy
pandas
conda-forge::rdkit=2023.3.1
conda-forge::nglview
pip`

Unfortunately, no. Reverting to this release version gives the same result.

I've built a container using your docker file and run it:
dkoes@jedi:~/git/gnina/docker/bug$ sudo docker run --gpus all --name bug -v /home/dkoes/tmp:/home/src/dir_name/ --shm-size 8gb c78ea0ceabf9 gnina -r /home/src/dir_name/rec.pdb --ligand /home/src/dir_name/lig.sdf --autobox_ligand /home/src/dir_name/rec.pdb -o /home/src/dir_name/temp_gnina_poses.sdf --exhaustiveness 64 --num_modes 10 --device 0 --cnn_scoring "rescore" --pose_sort_order CNNscore --min_rmsd_filter 1.0

==========
== CUDA ==
==========

CUDA Version 11.7.1

Container image Copyright (c) 2016-2023, NVIDIA CORPORATION & AFFILIATES. All rights reserved.

This container image and its contents are governed by the NVIDIA Deep Learning Container License.
By pulling and using the container, you accept the terms and conditions of this license:
https://developer.nvidia.com/ngc/nvidia-deep-learning-container-license

A copy of this license is made available in this container at /NGC-DL-CONTAINER-LICENSE for your convenience.

==============================
*** Open Babel Warning  in PerceiveBondOrders
  Failed to kekulize aromatic bonds in OBMol::PerceiveBondOrders

==============================
*** Open Babel Warning  in PerceiveBondOrders
  Failed to kekulize aromatic bonds in OBMol::PerceiveBondOrders

              _
             (_)
   __ _ _ __  _ _ __   __ _
  / _` | '_ \| | '_ \ / _` |
 | (_| | | | | | | | | (_| |
  \__, |_| |_|_|_| |_|\__,_|
   __/ |
  |___/

gnina  master:617ee3b   Built Dec 11 2023.
gnina is based on smina and AutoDock Vina.
Please cite appropriately.

Commandline: gnina -r /home/src/dir_name/rec.pdb --ligand /home/src/dir_name/lig.sdf --autobox_ligand /home/src/dir_name/rec.pdb -o /home/src/dir_name/temp_gnina_poses.sdf --exhaustiveness 64 --num_modes 10 --device 0 --cnn_scoring rescore --pose_sort_order CNNscore --min_rmsd_filter 1.0
Using random seed:<PHONE_NUMBER>

0%   10   20   30   40   50   60   70   80   90   100%
|----|----|----|----|----|----|----|----|----|----|
***************************************************

mode |  affinity  |  intramol  |    CNN     |   CNN
     | (kcal/mol) | (kcal/mol) | pose score | affinity
-----+------------+------------+------------+----------
    1       -8.52       -0.77       0.9558      6.849
    2       -8.49       -0.87       0.9003      6.787
    3       -8.30       -0.72       0.4690      6.483
    4       -6.75       -0.57       0.3723      5.034
    5       -6.72       -0.23       0.3103      5.009
    6       -7.00       -0.08       0.3008      4.996
    7       -6.53       -0.59       0.2917      4.971
    8       -6.69       -0.92       0.2642      5.246
    9       -6.90       -0.91       0.2573      4.993
   10       -6.49       -0.84       0.2535      5.226

The scores are present in the output file:
> <minimizedAffinity>
-8.52006

> <CNNscore>
0.9558389783

> <CNNaffinity>
6.8487119675

> <CNN_VS>
6.5462660789

> <CNNaffinity_variance>
0.1397060603

Can you provide your input files so I can see if that's the problem?


Tag release

Can you please tag the 0.0.5 release in github?
Contributor and consumer can then know which code is in release, and which is not.
For example, does 0.0.5 have all the code in master branch?
Thanks.

the 0.0.5 is almost the newest version, you can directly use this

It seems it doesn't have the take picture height bugfix.
https://github.com/ragnraok/RxCamera/issues/48


Fluid external storage filter doesn't work

Issue description:
Whenever fluids are in external storage, the filter doesn't work in exporters/fluidinterfaces.
What happens:
The first external storage set to fluid will be used instead of filter, so if water comes up before lava, it doesn't matter whether the filter in the exporter is set to lava, it will only export water
What you expected to happen:
The filter working
Steps to reproduce:
Setup system not using fluid storage blocks/disks, but with different fluid types along any axis
Version (make sure you are on the latest version before reporting):

Minecraft: 1.12.2
Forge: <IP_ADDRESS>38
Refined Storage: 1.6.15

Does this issue occur on a server? [yes/no]
yes, and in client
If a (crash)log is relevant for this issue, link it here:
none

Can't get this to happen what am I missing?

You're missing that water is also connected with external storage

I later figured out that the filtered item has to exist in the system, and that it prefers to use external storage fluids over cell storage, but doesn't care that it's not the filtered item that it outputs

Outputs water just like normal as you would expect for me. Regardless of whether it is in disk storage or connected with external storage.
Can you post a minimal setup? Cannot seem to reproduce that in any way I try.

Exporter set to water

I'm having a similar issue with RS 1.6.15 and forge <IP_ADDRESS>44
My setup is set up for autocrafting buckets for recipes plus being able to move liquids around when needed.

I get milk from water.  If I remove the milk from system, I get lava from clicking on water.
So its not just export buses... something seems to be b0rked with internal mapping of liquids.

If all fluids are the same priority, it seems that it picks a source in some order, but I can't figure it out. If I set all to same, then try to pull water, I get lava, lava I get... lava, oil I get water, everything else I seem to get milk.

My conclusion to this one was that some other mod is messing with liquids. I had attempted the exact setup he demonstrated just beforehand without running into any issues.
Same with your issues the filtering and priority functions work just fine and as expected in the tests I just did.

Since it seems to be interfering with internal stuff to RS, could it be the Reborn Storage mod?  I think that is the only mod that I have that is interfacing with RS that might be messing with its internal logic.

Reborn storage doesn't interact with fluids at all. The mod that causes this is most likely not related to RS in any way.

I'm not asking to be a PITA or to be combative, but to see if there is some logical way to determine what the root cause is, and get it resolved.  So please excuse me if I ask things; the only cure for ignorance is to ask.
I may be misunderstanding how java/forge works, and the interdependence of mods, but if the other mods aren't hooking into RS, how would they cause a change to RS's internal storage system?
If you put "oil" in the interface, that might be fluid registry of forge, but it does display and update correctly.  The Priority setting, which is internal to RS, seems to be the culprit from the testing I've done, and I don't know what reason another mod would have to poke around at RS code unless it works with RS's internals, such as reborn core.
Would it be how fluids are registered?  If so, shouldn't other mods also be affected, like ID, xnet, EIO or AE2? Mod load order?  Maybe something is registered, RS loads, and then some mod mucks with liquids? Don't think that'd be it because RS would then store/display/return the incorrect liquids, which doesn't seem to be the case, so long as you only use one liquid with the system.

The priority and filter situation indicates to me that RS is unable to differentiate between 2 different fluids.
I'm not familiar enough with how fluids are stored in 1.12 to guess at what might be going wrong and whether or not other mods would be affected.
However Reborn storage hasn't seen any major code changes since last year and I also had that mod in my testing.
I doubt its related to enderIO conduits as the test case demonstrated by thepsychochicken doesn't use those.
The classic way to determine the mod at fault would be to create backups and start removing mods until the problem goes away.

So interesting find... seems that somehow, eio conduits were doing it in my instance. I went on a conduit murdering rampage after testing and testing some more.  seems that something is going wrong when i use conduits instead of cables.
I'm not sure if the cables just have to be present anywhere, or just in parts of the network that go to fluids (I make no claims to know how routing is done in magic block mods).  Last time I did anything remotely like reading networking code was with LP back in like 1.6.4.


sitemapLoader.resolveUrl("p1.html") has a problem

https://github.com/kazurayam/inspectus/blob/0.7.5/src/test/java/com/kazurayam/inspectus/materialize/discovery/SitemapLoaderTest.java has a problem.
Once it was as this, and failed.
    @Test
    public void test_resolveURl_withoutSlash() throws InspectusException {
        SitemapLoader sitemapLoader =
                new SitemapLoader(Target.builder("http://example.com/pages").build());
        URL url = sitemapLoader.resolveUrl("p1.html");
        assertEquals("http://example.com/pages/p1.html", url.toString());
    }

So I changed to
        assertEquals("http://example.com/p1.html", url.toString());

and this test now passes.
But, this behavior is not what I really want.
I want
        assertEquals("http://example.com/pages/p1.html", url.toString());

to pass.

I have found a fact.
See the following code.
    @Test
    public void test_URL_constructor_withoutSlash() throws MalformedURLException {
        URL baseURL1 = new URL("http://example.com/pages");
        URL derived1 = new URL(baseURL1, "p1.html");
        assertEquals("http://example.com/p1.html", derived1.toExternalForm());

        // the "/" character at the end of baseURL matters
        URL baseURL2 = new URL("http://example.com/pages/");
        URL derived2 = new URL(baseURL2, "p1.html");
        assertEquals("http://example.com/pages/p1.html", derived2.toExternalForm());
    }

The "/" character at the end of baseURL matters.


Issue about Poisson encoding method

I notice the implementation of Poisson-distributed spike as below
rand_num = Variable(torch.rand(input.size(0), input.size(1), input.size(2), input.size(3)).cuda())
Poisson_d_input = (torch.abs(input) > rand_num).type(torch.cuda.FloatTensor)
Poisson_d_input = torch.mul(Poisson_d_input, torch.sign(input))

which means that the encoded spikes are triple-valued {0, ±1}. It's a little different from the well-known encoding style with binary value {0, 1}.
I wonder why you choose this way of Poisson encoding.

Thank you for asking questions. We found that normalizing inputs to represent zero mean helps to achieve better classification results, especially for SVHN and CIFAR-10 datasets. This spike encoding scheme would be efficiently implemented with an address-event-representation (AER) communication protocol that represents the discrete events using a 4-tuple {x, y, t, p}, consisting of the coordinates: x,y; timestamp: t; and polarity : p.  The details of our encoding scheme can be found in section 3.1.4 of our paper (https://www.frontiersin.org/articles/10.3389/fnins.2020.00119/full).

Thanks! I used to be unfamiliar with the AER protocol.
Your work introduced the BP algorithm to deeper SNNs. It's a great success!


New rule for checking single quotes

This rule allows people to have HTMLHint check their html for single quotes instead of double quotes or no quotes at all.

Can you tell me when to use single quotes?

It is mainly a styling preference, I think people who would use single quotes in their JavaScript would also prefer their HTML to use single quotes to keep all files consistent.


[BUG] WinGet install commands always sets the version number even when latest is selected

Please confirm these before moving forward

[X] I have searched for my issue and have not found a work-in-progress/duplicate/resolved issue.
[X] I have tested that this issue has not been fixed in the latest (beta or stable) release.
[X] I have checked the FAQ section for solutions.
[X] This issue is about a bug (if it is not, please use the correct template).

UniGetUI Version
3.1.2-beta2
Windows version, edition, and architecture
Windows 11 Pro 22631.4169 x64
Describe your issue
When loading an old bundle, many packages will reference an old application version.
The package installer options will have "Latest" as the version selected for installation.
However, when you run the package installer, it will always include the old version to install, as described in the bundle JSON file.
I would expect it not to include the --version parameter when executing winget.exe.
Steps to reproduce the issue

Use this as the bundle.

test.ubundle.json

Review the Installation Options

Select the Google Chrome Package and click install selection.

Installation Failed as it specified the version defined in the package version header and didn't use the empty string version defined in the "InstallationOptions" section.

You get the following error.
Starting package install operation for package id=Google.Chrome with Manager name=Winget
Given installation options are <InstallationOptions: SkipHashCheck=False;InteractiveInstallation=False;RunAsAdministrator=True;Version=;Architecture=;InstallationScope=;InstallationScope=;CustomParameters=;RemoveDataOnUninstall=False>
Process Executable     : C:\Program Files\UniGetUI\Assets\Utilities\gsudo.exe
Process Call Arguments : "C:\Users\admin\AppData\Local\Microsoft\WindowsApps\winget.exe"  install --id "Google.Chrome" --exact --source winget --accept-source-agreements --disable-interactivity --version 127.0.6533.100 --silent  --accept-package-agreements --force
Working Directory      : C:\Users\admin
Process Start Time     : 20/09/2024 10:17:38 AM
No version found matching: 127.0.6533.100
Process Exit Code      : -1978335209
Process End Time       : 20/09/2024 10:17:38 AM

I would expect the command to perform the install would not use the --version tag as it should be the latest installed.

UniGetUI Log
[20/09/2024 10:15:46 AM]    __  __      _ ______     __  __  ______
                           / / / /___  (_) ____/__  / /_/ / / /  _/
                          / / / / __ \/ / / __/ _ \/ __/ / / // /
                         / /_/ / / / / / /_/ /  __/ /_/ /_/ // /
                         \____/_/ /_/_/\____/\___/\__/\____/___/
                             Welcome to UniGetUI Version 3.1.2-beta2
[20/09/2024 10:15:46 AM]
[20/09/2024 10:15:46 AM] Version Code:  3.1192
[20/09/2024 10:15:46 AM] Encoding Code Page set to 437
[20/09/2024 10:15:46 AM] Using bundled GSudo at C:\Program Files\UniGetUI\Assets\Utilities\gsudo.exe
[20/09/2024 10:15:46 AM] Loaded language locale: en
[20/09/2024 10:15:47 AM] Randomly-generated background API auth token for the current session: n4uhyk6tfyseksewxhxlja47difli61qebmckjxl77ye56llcx3ipg33anenutmq
[20/09/2024 10:15:47 AM] Api running on http://localhost:7058
[20/09/2024 10:15:47 AM] Lang files were updated successfully from GitHub
[20/09/2024 10:15:48 AM] Command npm was not found on the system
[20/09/2024 10:15:48 AM] ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄
                         █▀▀▀▀▀▀▀▀▀▀▀▀▀ MANAGER LOADED ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
                         █ Name: Npm
                         █ Enabled: False
                         █ THE MANAGER IS DISABLED
                         ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
[20/09/2024 10:15:48 AM] Downloaded new icons and screenshots successfully!
[20/09/2024 10:15:48 AM] Command dotnet.exe was not found on the system
[20/09/2024 10:15:48 AM] ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄
                         █▀▀▀▀▀▀▀▀▀▀▀▀▀ MANAGER LOADED ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
                         █ Name: .NET Tool
                         █ Enabled: False
                         █ THE MANAGER IS DISABLED
                         ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
[20/09/2024 10:15:48 AM] Command cargo was not found on the system
[20/09/2024 10:15:48 AM] Could not initialize Package Manager Cargo
[20/09/2024 10:15:48 AM] System.InvalidOperationException: Cannot start process because a file name has not been provided.
                            at System.Diagnostics.Process.Start()
                            at UniGetUI.PackageEngine.Managers.CargoManager.Cargo.LoadManager() in C:\SomePrograms\WingetUI-Store\src\UniGetUI.PackageEngine.Managers.Cargo\Cargo.cs:line 131
                            at UniGetUI.PackageEngine.ManagerClasses.Manager.PackageManager.InitializeAsync() in C:\SomePrograms\WingetUI-Store\src\UniGetUI.PackageEngine.PackageManagerClasses\Manager\PackageManager.cs:line 88
[20/09/2024 10:15:48 AM] Command pwsh.exe was not found on the system
[20/09/2024 10:15:48 AM] ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄
                         █▀▀▀▀▀▀▀▀▀▀▀▀▀ MANAGER LOADED ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
                         █ Name: PowerShell7
                         █ Enabled: False
                         █ THE MANAGER IS DISABLED
                         ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
[20/09/2024 10:15:48 AM] UniGetUI is up to date
[20/09/2024 10:15:48 AM] ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄
                         █▀▀▀▀▀▀▀▀▀▀▀▀▀ MANAGER LOADED ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
                         █ Name: PowerShell
                         █ Enabled: False
                         █ THE MANAGER IS DISABLED
                         ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
[20/09/2024 10:15:48 AM] Command scoop was not found on the system
[20/09/2024 10:15:48 AM] ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄
                         █▀▀▀▀▀▀▀▀▀▀▀▀▀ MANAGER LOADED ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
                         █ Name: Scoop
                         █ Enabled: False
                         █ THE MANAGER IS DISABLED
                         ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
[20/09/2024 10:15:48 AM] ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄
                         █▀▀▀▀▀▀▀▀▀▀▀▀▀ MANAGER LOADED ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
                         █ Name: Pip
                         █ Enabled: False
                         █ THE MANAGER IS DISABLED
                         ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
[20/09/2024 10:15:48 AM] ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄
                         █▀▀▀▀▀▀▀▀▀▀▀▀▀ MANAGER LOADED ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
                         █ Name: Chocolatey
                         █ Enabled: False
                         █ THE MANAGER IS DISABLED
                         ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
[20/09/2024 10:15:49 AM] ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄
                         █▀▀▀▀▀▀▀▀▀▀▀▀▀ MANAGER LOADED ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
                         █ Name: Winget
                         █ Enabled: True
                         █ Found: True
                         █ Fancy exe name: winget.exe
                         █ Executable path: C:\Users\admin\AppData\Local\Microsoft\WindowsApps\winget.exe
                         █ Call arguments:
                         █ Version:
                         █   System WinGet CLI Version: v1.8.1911
                         █   Microsoft.WinGet.Client PSModule version: ""
                         █   Using Native WinGet helper (COM Api)
                         ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
[20/09/2024 10:15:49 AM] LoadComponentsAsync finished executing. All managers loaded. Proceeding to interface.
[20/09/2024 10:15:49 AM] Resetting administrator rights cache for process id 6584
[20/09/2024 10:15:49 AM] Begin attempt to open the package bundle C:\Users\admin\OneDrive\Documents\UniGetUI_Bundles\test.ubundle
[20/09/2024 10:15:50 AM] Microsoft.Powershell.Client detection has been forcefully skipped as the module is not required on the current context
[20/09/2024 10:15:50 AM] Dependency WinGet PowerShell Module for manager Winget is present
[20/09/2024 10:15:51 AM] Found 95 installed packages from Winget
[20/09/2024 10:15:51 AM] Found 4 available updates from Winget
[20/09/2024 10:16:00 AM] Registering package Google.Chrome from manager Winget: winget
[20/09/2024 10:16:00 AM] Non-MSStore WinGet Native Icons have been forcefully disabled on code
[20/09/2024 10:16:00 AM] Loaded icon with URL=UniGetUI.Core.IconEngine.CacheableIcon for package Id=Google.Chrome
[20/09/2024 10:16:00 AM] Icon for package Google.Chrome stored on C:\Users\admin\AppData\Local\UniGetUI\CachedMedia\Winget\Google.Chrome.png
[20/09/2024 10:16:00 AM] Loaded operation parameters for package id=Google.Chrome on manager Winget and operation Install: install --id "Google.Chrome" --exact --source winget --accept-source-agreements --disable-interactivity --version 127.0.6533.100 --silent  --accept-package-agreements --force
[20/09/2024 10:16:00 AM] Caching admin rights for process id 6584
[20/09/2024 10:16:06 AM] Resetting administrator rights cache for process id 6584

Package Managers Logs
Manager WinGet with version:
System WinGet CLI Version: v1.8.1911
Microsoft.WinGet.Client PSModule version: ""
Using Native WinGet helper (COM Api)

——————————————————————————————————————————

Logged native task on manager Winget. Task type is ListSources
Process start time: 20/09/2024 10:15:49 AM
Process end time:   20/09/2024 10:15:49 AM

-- Task information
 ...

The task reported success

——————————————————————————————————————————

Logged subprocess-based task on manager Winget. Task type is RefreshIndexes
Subprocess executable: "C:\Users\admin\AppData\Local\Microsoft\WindowsApps\winget.exe"
Command-line arguments: " source update --disable-interactivity"
Process start time: 20/09/2024 10:15:49 AM
Process end time:   20/09/2024 10:15:50 AM

-- Process STDOUT
 ...

Return code: SUCCESS (0)

——————————————————————————————————————————

Logged native task on manager Winget. Task type is ListInstalledPackages
Process start time: 20/09/2024 10:15:49 AM
Process end time:   20/09/2024 10:15:51 AM

-- Task information
 ...

The task reported success

——————————————————————————————————————————

Logged native task on manager Winget. Task type is ListSources
Process start time: 20/09/2024 10:15:49 AM
Process end time:   20/09/2024 10:15:50 AM

-- Task information
 ...

The task reported success

——————————————————————————————————————————

Logged native task on manager Winget. Task type is ListUpdates
Process start time: 20/09/2024 10:15:50 AM
Process end time:   20/09/2024 10:15:51 AM

-- Task information
 ...

The task reported success

——————————————————————————————————————————

Relevant information
No response
Screenshots and videos
No response

Hi, adding a comment here because I've found an issue that is potentially linked to this one.
Whilst I am facing this exact issue reinstalling a Package Bundle in a test VM, i am also facing the following:
UnigetUI is unable to reinstall any msstore from a Package Bundle because it is attempting to install a specific version, but msstore does not provide ANY version at all.
Example extracted from the output:
This is Power Toys
winget.exe
Process Call Arguments :  install --id "XP89DCGQ3K6VLD" --exact --source msstore --accept-source-agreements --version "0.84.1" --silent --disable-interactivity --accept-package-agreements

Please let me know if you need a separate issue for this.


Sticky toolbar does not stick in iOS chrome

https://github.com/google/material-design-lite/issues/925#issuecomment-120389982

I have reproduced this issue. AFAICT, this happens in every UIWebView. Will investigate on how to fix this.

To work around this issue some very invasive work needs to be done on the components page. We’ll hold off on this, since an overhaul on the components page is planned anyways.

This should be fixed with current iOS Chrome. Please reopen if it's still an issue.


Security issue: Prototype pollution attack(Introduced by the<EMAIL_ADDRESS>=><EMAIL_ADDRESS>=><EMAIL_ADDRESS>

This is a dup of https://github.com/sass/node-sass/issues/2288 which is marked as fixed but it seems that this commit was overridden. Package.json still has the old version
    "request": "~2.79.0",

The issue is not fixed as package.json still has an old version. The commit with the fix has been lost.

It cannot be fixed without break node < 4 support
On Fri., 27 Apr. 2018, 9:47 pm Alexey Kontsevoy<EMAIL_ADDRESS>wrote:

The issue is not fixed as package.json still has an old version. The
commit with the fix has been lost.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/sass/node-sass/issues/2352#issuecomment-384947153,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAjZWEzeVAuBfhDu4tXcZ4kAcN6AUgblks5tswVZgaJpZM4ToBTC
.


Add content for the I'm Infected Section

Add content for the https://www.coronasafe.in/action/i-am-infected page. The major source is https://www.cdc.gov/coronavirus/2019-ncov/about/steps-when-sick.html

https://github.com/coronasafe/coronasafe.in/commit/f754b8d9727199569b9616b8fb7e3a95e2524b0a

This commit fixes #3 we need to fix this issue too

I'm working on it


Affected versions for flickering on Unity + Hololens 2

[Enter feedback here]
Please change
Please avoid Unity 2020.3.21f and 2020.3.22f,
to
Please avoid Unity 2020.3.21f, 2020.3.22f and 2020.3.23f
The linked bug report
https://github.com/microsoft/MixedRealityToolkit-Unity/issues/10288
indicateds .23f is also affected.

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

ID: 8688c8ff-81f9-705a-24eb-c255e31c54f8
Version Independent ID: 915f7e31-a006-a871-de27-14f9f87c3244
Content: MRTK-Unity Developer Documentation - Mixed Reality Toolkit
Content Source: mrtk-unity/index.md
Product: mixed-reality
GitHub Login: @polar-kev
Microsoft Alias: kesemple

https://github.com/MicrosoftDocs/mixed-reality-pr/pull/3451

https://docs.microsoft.com/en-us/windows/mixed-reality/develop/choosing-an-engine?tabs=unity
The page still has the old content.

@RubenGarcia can you clarify or specify what's out of date?

I just checked it again and now all is good, thanks!
Closing.


INEN

Hello, thank you for your excellent work. The INEN module in the paper is not provided in the code. Could you please share the relevant code? Thank you.

Hi, I'm very interested in your work, I can't find the SmoothWassLoss function in the code you gave, can you provide it? Thanks a lot!


Add codacy badge and configuration

As per title this PR adds codacy badge to project's READMEs and also adda codacy configuration to the root of the project (named codacy-conf.json).
codacy-conf.json is not exactly human readable but can be imported and exported in codacy where you can edit it and export again (and possibly push to this repository).
I already did a pass disabling checks that we don't need and enabling the ones that I believe we should enforce. We could also consider disabling all checks related to CSS and Javascript.

BTW, did you understand how to include coverage data with codacy?
We don't really need it as Coveralls should be enough, but if trivial it would be nice.

This PR relates to #626

I did't look into it as we already have coveralls. Is it worth having two services checking for coverage? Or do you want to assess whether codacy's report is more accurate that Coveralls?
Also as you've seen by yourself codacy can be slow in providing the report to a PR, while coveralls shows the report almost in real time.

Right, it is not important. I was only curious as when you go to the codacy main page for the project you see it mentioned as "not configured". Also, I was curious to see if their report is more helpful in showing the missing coverage"...

I've been in touch with codacy's support today and they told me that the latency we are experiencing in seeing codacy's status after our PRs should be fixed soon. As soon as it becomes more responsive we can give it a try!

Sounds Good!


The language of typeScript error prompt in *.vue file is confused

sometimes English, sometimes Chinese

the version of volar and vue-tsc is v1.0.1

Same

I got more version info:

vue-tsc v1.0.1 + volar v0.40.13 works correctly
vue-tsc v1.0.1 + volar ^1.0.0 works wrong

I have the same issue with the latest version of Volar.
The display language in *.vue file should be English instead of Chinese.

same issue +1

After checked the source code of vscode-languageserver-node, I think the problem is that it's using process.env['VSCODE_NLS_CONFIG'] for language server language, but process.env['VSCODE_NLS_CONFIG'] is system language instead of display language by default.
To fix this problem you can try change VSCode display language to any other language, and change back to en, it should force replace for process.env['VSCODE_NLS_CONFIG'].

It seems to work!

Changing display language works to me

I will make a pitch in vue language server in 1.0.5 to avoid this problem.

thanks a lot!


Add support for setting uid/gid from pkg_attributes

This enables the user to produce identical tarballs to fix https://github.com/bazelbuild/rules_pkg/issues/670 by adding a uid/gid option to pkg_attributes.

@nacl  I think it is blocked on you resolving a comment.

Ping, @nacl , any additional feedback?

Friendly ping.  This should get merged, can we come to closure on the last few nits?

Sorry for the long delay...  I went from having time to not having time really fast.


Vulnerability roundup 85: clickhouse-<IP_ADDRESS>: 2 advisories [9.8]

search, files

[ ] CVE-2019-16535 CVSSv3=9.8 (nixos-20.03)
[ ] CVE-2019-15024 CVSSv3=6.5 (nixos-20.03)

Scanned versions: nixos-20.03: a84b797b28e. May contain false positives.

20.03 is frozen and EOL.
Closing.


[UI] Terminating meshery-operator through UI Terminates all meshery pods

Description
When a user terminates meshery-operator through UI-client, it triggers all meshery namespace pods to be deleted.
Expected Behavior
Only pods related to meshery-operator should be terminated.
Screenshots/Logs
>  kubectl get pods -A
NAMESPACE            NAME                                         READY   STATUS        RESTARTS   AGE
kube-system          coredns-f9fd979d6-2b7fv                      1/1     Running       2          4d13h
kube-system          coredns-f9fd979d6-s8zrw                      1/1     Running       2          4d13h
kube-system          etcd-kind-control-plane                      1/1     Running       3          4d13h
kube-system          kindnet-cns2q                                1/1     Running       4          4d13h
kube-system          kube-apiserver-kind-control-plane            1/1     Running       4          4d13h
kube-system          kube-controller-manager-kind-control-plane   1/1     Running       18         4d13h
kube-system          kube-proxy-5l7jz                             1/1     Running       2          4d13h
kube-system          kube-scheduler-kind-control-plane            1/1     Running       22         4d13h
local-path-storage   local-path-provisioner-78776bfc44-gz4kf      1/1     Running       14         4d13h
meshery              meshery-54fc64ccfd-fmmgf                     1/1     Terminating   0          7m14s
meshery              meshery-broker-0                             2/2     Running       4          4d13h
meshery              meshery-consul-6c77dcd486-m6w6n              0/1     Terminating   0          7m1s
meshery              meshery-cpx-5bc6f8dddf-m82c5                 0/1     Terminating   0          6m45s
meshery              meshery-meshsync-f7678dbc5-5b4xr             1/1     Terminating   9          4d13h
meshery              meshery-nsm-6bb69cc498-pjf6p                 1/1     Terminating   0          6m58s
meshery              meshery-operator-549bf67565-w4ks6            2/2     Terminating   11         4d13h
meshery              meshery-osm-f4f7b47c6-fw9bb                  1/1     Terminating   0          6m37s
meshery              meshery-traefik-mesh-cd95d64b-bhmgb          1/1     Terminating   0          6m33s

Environment:

Meshery Version: v0.5.12

Good eye, @piyushsingariya

I would like to work on this issue. And for a bit of clarity clarity can please someone confirm this issue is propagating from the Meshery-UI?

Yes, this is an issue from Meshery UI.

@DantrazTrev any updates?

Hey @piyushsingariya , I don't think it is an issue with the UI, because, from what i see, in the UI, it is just calling the graphql mutation and all the logic for what has to be deleted and all is handled by the graphql resolver. The only variable we are sending is the final state of the operator. So, i suspect it has something to do with the resolver, but, i didn't find anything wrong with that too 🤔 .
@DantrazTrev , what's your view on this ?

Yup @sudo-NithishKarthik sorry I've not been working on this lately . From what I could find There is no mutation for the meshery operator status solely .

Hey @piyushsingariya , I don't think it is an issue with the UI, because, from what i see, in the UI, it is just calling the graphql mutation and all the logic for what has to be deleted and all is handled by the graphql resolver. The only variable we are sending is the final state of the operator. So, i suspect it has something to do with the resolver, but, i didn't find anything wrong with that too 🤔 .
@DantrazTrev , what's your view on this ?

@dhruv0000  what's do you think ?


[FEATURE] Add NextViT

ByteDance has open-sourced a better version of ViT here:
https://github.com/bytedance/next-vit

For anyone looking into reimplementing this model, the authors use pretty strange implementation of AvgPool as mentioned here: https://github.com/bytedance/Next-ViT/issues/2

Is there any ongoing work on this? What can be a good way to start contributing?

The authors of Next-Vit released their code, so copy-pasting with minor adaptations for Timm could be done quite easily. The problem with their AvgPool still remains, I'm re-running some of their experiments and seeing noticeable improvement with proper pooling. Will comment here when runs finish.


Force removal of files left over by test containers (fixes #1276)

fixes #1276

I cancelled the IT build to save money. The changes affect both the IT and the regular build equally so testing just one seemed sufficient.


Ability to makes calls to SSL endpoing with invalid cert

I realize this is an unsafe thing, but the ability to override or pass in an option to ignore SSL errors when making an https call to a server with invalid cert would be very helpful.
In my case, the server is only ever accessed internally, but for some reason uses HTTPS and has an invalid cert. I found this option in Net::HTTP but saw no similar option in HTTP.rb:
verify_mode: OpenSSL::SSL::VERIFY_NONE
https://stackoverflow.com/a/9837520/4167402

You can pass custom ssl_context. So in your case it will look like:
ctx = OpenSSL::SSL::SSLContext.new
# ...
ctx.verify_mode = OpenSSL::SSL::VERIFY_NONE

HTTP.get("https://www.google.com", :ssl_context => ctx)

Pressed wrong button :D
Please close if the answer solves your issue.

This has come up enough times (#308, #354) I wonder if it would be better to have an obvious, easily greppable way to do this

Do we have something in the docs?
Sent from my iPhone

On Nov 16, 2017, at 09:11, Tony Arcieri<EMAIL_ADDRESS>wrote:
This has come up enough times (#308, #354) I wonder if it would be better to have an obvious, easily greppable way to do this
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@ixti this does resolve my issue, thanks! But I'll leave open to allow the rest of the discussion to finish in case any documentation work is needed.


PHP 8.2 compatibility fixes

Currently framework code in version 4 triggers a lot of deprecation errors, because of setting dynamic properties on some classes. This can be easily fixed by declaring these properties on the classes, and should have no impact on the code otherwise.
The reason for fixing this is to be able to fix deprecation errors with PHP 8.2 in Silverstripe modules without these errors getting buried in all the deprecation errors from framework, not to have version 4 of Silverstripe officially support PHP 8.2.

How did you come to this set of specific properties?
My main concern here is that if it's incomplete then we may have a long and slow trickle of PRs like this over the next two years as people upgrade their projects and want to run pho 8.2 on their CMS 4 projects during the upgrade.
I'd recommend using php8.1 in your CMS 4 projects, then upgrade to CMS 5, then upgrade to PHP 8.2.

These specific properties are the ones that cause deprecation notices on /dev/build, template parsing and basically all requests handled by framework. As I mentioned, the point of this PR is not official support for PHP 8.2, just to make it possible to fix deprecation notices in other modules with CMS 4.

I won't have time to look at this until Monday, but here's a couple of things that need to be confirmed before this can be merged:

If a subclass declared these properties with some other visibility already (e.g. protected), will they now get errors for not matching the parent class signature?
If someone tries to access one of these properties before its value is set, is the same error or exception thrown that gets thrown without this pr?

To answer you questions:

Yes, if a subclass declared these same properties with some other visibility than public, then that would cause an error. However, I do not see any use case where someone would have declared these properties in any subclass, they are quite obscure.
No, accessing these properties before a value is set will not cause a deprecation notice or other error.

I tested these changes both with php 8.2 and with older versions, and have seen no impact except for getting rid of the deprecation notices on php 8.2. I see these changes as very low risk, declaring these properties on the classes is good practise anyway.

I see these changes as very low risk

That may be the case, but these are the sort of changes we would usually not make until a major release. To introduce them in a patch release requires a much higher degree of scrutiny, which is why I'm taking this through the paces. I appreciate your patience during the process.

declaring these properties on the classes is good practise anyway

Agreed, but there are many best practices we can't introduce in CMS 4 patch releases, due to semver concerns. As stated above, we need to make sure we're being very careful with anything like this as it's a wide departure from our normal release practice.

Please swap all public being introduced in this PR with var - this will ensure there are no breaking changes for anyone who has declared these properties on their subclasses with different visibilities and should therefore remove the last BC concern I have.

@GuySartorelli are you saying that var as a synonym to public can also act as private or protected if the same variable is later redeclared in child class with protected?

A property declared with var is public by default, but can be overridden with a specific visibility in subclasses.
It's as close as you can get to a dynamic property while still declaring the property, from what I can tell.

Unfortunately it still seems to error when setting non-public visibility: https://3v4l.org/SJorQ

Can we not use the AllowDynamicProperties annotation here?

Before: https://3v4l.org/KVVFO
After: https://3v4l.org/PVrJu

Good idea, totally forgot about #[\AllowDynamicProperties]! Although I think that annotation is only supported for PHP >= 8.0, but I guess PHP 7 would just ignore it?

We probably should also create (if we don't have one already) a ticket to revisit all these places for CMS 6 and do it properly.

We probably should also create (if we don't have one already) a ticket to revisit all these places for CMS 6 and do it properly.

This is for CMS 4, I’m pretty sure it’s already been fixed in 5,. Though we might need a follow up task to remove the annotation (if it’ll get merged up automatically or might be missed if manually merging up?)

The automatic merge up is on a Cron. I've already made a mental note that once this gets merged it needs to be manually merged up (and reverted during the merge up)

This will be automatically tagged when GitHub Actions has finished running on the branch.
I've manually reverted it in the merge-up to 5.0


refactor: rename func-call-spacing to function-call-spacing

Overview
Closes #39
This pull request renames func-call-spacing function to function-call-spacing for the reasons stated in #39. Generally, it only renames all the files and references of the said rules.
Do note that this change won't be applied on migration since it doesn't touch the migration scripts at all.

I think removing the original rule is a breaking change. I think it's better to leave the original rule as an alias. What do you think?

@ota-meshi, valid point. For now, I'll duplicate the plugin under a new name.

Dropped in favor of #63


Update DYLD_FALLBACK_LIBRARY_PATH

Update README.md with new path for DYLD_FALLBACK_LIBRARY_PATH.
On MacOS catalin 10.15.7 , the command line tools folder has changed. It no longer contains the Toolchains directory

Can you please provide more details? In my case (Catalina 10.15.7, Xcode 12.1) I have both /Applications/Xcode.app/Contents/Developer/usr/lib and /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/lib and they contain different set of files.

I just checked my machine and the path /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/lib does exist on my machine.
I've just realised now that running $(xcode-select --print-path) on my machine takes me to /Library/Developer/CommandLineTools. Can you verify if it is the same for you?

Not quite:
macos@macos ~ % xcode-select --print-path
/Applications/Xcode.app/Contents/Developer

I wonder if this is specific to my mac then. I might close the pull-request then as i though the README.md file was a bit outdated

There was a discussion in the issue #205 so your pull request was perfectly valid, thank you!

There was a discussion in the issue #205 so your pull request was perfectly valid, thank you!


[SPARK-15204][SQL] Nullable is not correct for Aggregator

What changes were proposed in this pull request?
object SimpleSum extends Aggregator[Row, Int, Int] {
  def zero: Int = 0
  def reduce(b: Int, a: Row) = b + a.getInt(1)
  def merge(b1: Int, b2: Int) = b1 + b2
  def finish(b: Int) = b
  def bufferEncoder: Encoder[Int] = Encoders.scalaInt
  def outputEncoder: Encoder[Int] = Encoders.scalaInt
}

val df = List(("a", 1), ("a", 2), ("a", 3)).toDF("k", "v")
val df1 = df.groupBy("k").agg(SimpleSum.toColumn as "v1")
df1.printSchema
df1.show

root
 |-- k: string (nullable = true)
 |-- v1: integer (nullable = true)

+---+---+
|  k| v1|
+---+---+
|  a|  6|
+---+---+

The above example shows v1 as nullable=true which is incorrect.
The aggregate fields should be non-nullable in the schema.
How was this patch tested?
unit tests

Can one of the admins verify this patch?

Test build #2978 has started for PR 13012 at commit c66d36c.

Test build #2978 has finished for PR 13012 at commit c66d36c.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.

Jenkins test this please

Jenkins retest this please

Is this even correct? Certain aggregators might return nulls.

blackbox transformations infer nullable=false when you return a primitive. for example:
scala> sc.parallelize(List(1,2,3)).toDS.map(i => i * 2).schema
res0: org.apache.spark.sql.types.StructType = StructType(StructField(value,IntegerType,false))

why would aggregator be any different?

Your last sentence was what I was saying. It is not correct to just have nullable = false.
Please close this pull request as it is not a proper fix.

Hello @rxin , @koertkuipers ,
I have been following your conversation & feel that the right fix will be
override def nullable: Boolean = Literal.fromObject(aggregator.zero, bufferExternalType).nullable

I'm in the process of updating the PR with the changes. Please let me know what do you think about the changes.

That's not the right fix, because the zero value can be non-null, and yet the result can still be null. The right fix probably requires looking at the encoder.

@sbcd90 please just close this until you figure out the right fix.


Broken / wrong links when no paid plan is configured

Issue Summary
As a new Ghost publisher, I'm not ready to connect to Stripe and configure the paid plans for members. I'm starting out with simple free sign-up.
Several UI bugs occur for site visitors in this state. When a signed-in visitor visits the site and there are no configured paid plans,

In some themes, they get Upgrade buttons which display a broken dialog (Journal v1.0.0).
They still see Sign up links (although signed up and logged in) Casper v5.3.1 and Journal v1.0.0 and probably others.

Here, a signed-in member is incorrectly shown the upgrade button in multiple places leading to a broken dialog (Journal v1.0.0 theme):

Broken dialog:

Here, a signed-up and signed-in free member is incorrectly shown the "Sign up" link (Casper theme):

Steps to Reproduce

Create new site with Journal theme.
Sign up as free tier member.
Browse to About page, see the Upgrade button.
See the Signup links in the footers and elsewhere.

The bugs can be seen live on my Ghost(Pro) site, https://dogweather.dev.
Ghost Version
Ghost(Pro)
Node.js Version
unknown
How did you install Ghost?
Pro
Database type
Other
Browser & OS version
Safari / current
Relevant log / error output
No response
Code of Conduct

[X] I agree to be friendly and polite to people in this repository

It looks like there's an error in index.hbs for this theme - that text comes from lines 88-89, and results if the user is a member but not a paid member. The theme needs to be checking if there are paid tiers available and not rendering code here if there aren't..

It looks like there's an error in index.hbs for this theme - that text comes from lines 88-89...

Yep, here's what a forum user found: https://forum.ghost.org/t/editon-theme-bug-on-home-page/32176/10


fix: compat bug on safari 12 & 13

Probably the same that happened in Edge too, although I couldn't reproduce it.
fix #699
Also improve animation style.

Oups the previous one was merged so quickly that I thought I hadn't open it yet :laughing:


Fetch data from multiple clusters

Added new label for each metric: "cluster_url"

Just noticed the discussion.
https://github.com/justwatchcom/elasticsearch_exporter/pull/203


Support for multitenant

It would be nice to have multitenant support, something like appwrite with multiple projects and multiple databases for project. If it had this feature we would have switched from appwrite because it looks very well and promising.

Hey!
Realize now that I answered you in the discussions as well.
I'm not sure I understand this feature request.
Multi tenancy seems more like a configuration done depending on how you host your server and solutions.
And you are free to host as many Serverpod instances with databases as you would like.
Only limitation there is that each Serverpod instance will be connected to a single database.
Could you share a bit more information around what feature you are requesting from the framework?

Hello,
Well we need something like appwrite has, a dashboard where you can create projects and within that project multiple databases but I think we can make use of the configuration files to specify different ports and database for each tenant.


Reload strings issue

http://www.concrete5.org/developers/bugs/5-7-3-1/translate-site-interface-reload-strings-issue/#696604

Hi @aembler, I think @mlocati has fixed this in one of his many updates to the Multilingual code. I can verify that the Reload Strings tool will now open up my starter .mo file, allow me to edit any missing or errant strings, and then save the file back out to the file system where C5 can use it. As an added bonus, the view also shows the context of the strings that it finds. Great stuff!

Forgot to mention that I verified the above behavior using the official <IP_ADDRESS> release.

Good find – yeah, he did some excellent work on this stuff awhile back.


feat(Dialog): add ShowLiteralConfirmModal method

{PR title}
为DialogService添加一个简易的弹出确认对话框的方法，仅需传入提示信息。
Description
feat(Dialog): add ShowLiteralConfirmModal method
Regression?

[ ] Yes
[√] No

Risk

[ ] High
[ ] Medium
[√] Low

Verification

[√] Manual (required)
[ ] Automated

Packaging changes reviewed?

[ ] Yes
[ ] No
[√] N/A

☑ Self Check before Merge
⚠ Please check all items below before review. ⚠

[√] Doc is updated/provided or not needed
[√] Demo is updated/provided or not needed
[√] Merge the latest code from the main branch

Summary by Sourcery
Add a new method ShowLiteralConfirmModal to DialogService for displaying a simple confirmation dialog with a string message, and update the documentation to reflect this new feature.
New Features:

Introduce the ShowLiteralConfirmModal method in DialogService to display a simple confirmation dialog by passing a string message.

Documentation:

Update user-facing documentation to include information about the new ShowLiteralConfirmModal method, including its usage and parameters.

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

@microsoft-github-policy-service agree


Data ambiguities

For the ground truth mesh models, are there any specific rules for triangulation? For example, a rectangular plane may be represented by different numbers and different ways of triangular faces, there may be ambiguity here?

The main concept behind our MeshXL is that a 3D mesh with a given triangulation can be modeled into a one-and-only ordered coordinates sequence. Therefore, we do not design extra rules for triangulation as long as they are valid 3D meshes. In our project, different triangulations also serve as a potential data augmentation strategy, as our MeshXL treat different topologies as different training samples. We adopt planar decimation (with different threshold) to simplify complex 3D meshes, and rotate 3D meshes (and create ordered coordinate sequences after) to create more training samples.
In short, different topologies for a 3D mesh are not considered ambiguities in our project. It is somewhat similar to ColorJitter, RandomHorizontalFlip, and Resize operations for images.

Get it! Thanks for your reply!


Add "Speed is now a landing page factor for Google Search and Ads."

What's changed, or what was fixed?

Add an updates article on speed as a factor in search + ads

Target Live Date: 2018-07-25

[x] This has been reviewed and approved by (Ilya)
[x] I have run npm test locally and all tests pass.
[x] I have added the appropriate type-something label.
[x] I've staged the site and manually verified that my content displays correctly.

CC: @petele

:+1:

Thanks Addy!

:tada: This has been pushed live to https://developers.google.com/web/


Taskbar Calendar doesn't show up after running tweak

Describe the bug
Taskbar Calendar doesn't show up after running tweak
After running the tweak, when I click the Calendar at the taskbar, it doesn't show. Also I tried to tick Disable Notification and then Undo Tweak but it doesn't work.

You will either need to reboot or restart explorer.exe for the changes to take effect. If you already tried the undo tweak for Disable Notification than it's likely something else that might be causing the issue.

You will either need to reboot or restart explorer.exe for the changes to take effect. If you already tried the undo tweak for Disable Notification than it's likely something else that might be causing the issue.

I tried everything ask you said but it didn't work. When I clicked to the taskbar calendar before tweaking, it was normal and no issue. After running the tweak, when click it doesn't show anymore.

Have you run any other tweaks from different tools? I have tested this and running undo restores the calendar functionality.
you can also change the registry directly to ensure they are set correctly
HKCU:\Software\Policies\Microsoft\Windows\Explorer\DisableNotificationCenter should be 0
HKCU:\Software\Microsoft\Windows\CurrentVersion\PushNotifications\ToastEnabled should be 1

Have you run any other tweaks from different tools? I have tested this and running undo restores the calendar functionality.
you can also change the registry directly to ensure they are set correctly
HKCU:\Software\Policies\Microsoft\Windows\Explorer\DisableNotificationCenter should be 0 HKCU:\Software\Microsoft\Windows\CurrentVersion\PushNotifications\ToastEnabled should be 1

No I haven't use any other tweaks.
But currently everything is fine. I tried to reinstall Windows and tweaked twice times. One with default Desktop profile, and one without Disable Location, Activity History and Wifi-Sense. Of course I made sure that Disable Notification is unticked. And yes, the Taskbar Calendar with the Desktop profile without 3 options above is ok. So I'm not sure if there is any bug or not...

Added an Enable Calendar/Notification tweak


Testing of age-structure code

For help see the packages current test and also https://r-pkgs.org/tests.html

Code (testing) coverage with new age-structured model now increased from 40 to 64.8%

Code (testing) coverage with new age-structured model now increased from 64.8% to 95.64% - local testing using covr::package_coverage()


Add Label in Axis

Can i add label in axis to add more clarity each segment ? Thankyou

Can i add label in axis to add more clarity each segment ? Thankyou

Hi. Have you found a solution to draw labels on the axis?


Replace types with those from the http crate

Request
Response
Method
StatusCode
Version
HeaderMap

This has been done in the 0.12.x branch.


Update code coverage settings

This PR tries to improve the quality and reduce the noise of the code coverage integration by:

Ignoring the Tests folder since we don’t care about coverage of test code,
Running the code coverage on multiple OS versions to capture availability branching, and
Running the code coverage on multiple OSes to capture OS branching.

Codecov Report

Merging #4201 into master will decrease coverage by 7.37%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4201      +/-   ##
==========================================
- Coverage   87.74%   80.36%   -7.38%
==========================================
  Files          46       24      -22
  Lines        5729     3051    -2678
==========================================
- Hits         5027     2452    -2575
+ Misses        702      599     -103

Impacted Files
Coverage Δ

Tests/Tests/AFXMLParserResponseSerializerTests.m

Tests/Tests/AFImageResponseSerializerTests.m

Tests/Tests/AFHTTPSessionManagerTests.m

Tests/Tests/AFPropertyListRequestSerializerTests.m

Tests/Tests/AFUIImageViewTests.m

Tests/Tests/AFUIRefreshControlTests.m

Tests/Tests/AFJSONSerializationTests.m

Tests/Tests/AFAutoPurgingImageCacheTests.m

Tests/Tests/AFHTTPRequestSerializationTests.m

Tests/Tests/AFImageDownloaderTests.m

... and 15 more

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 e08306e...9bcb42e. Read the comment docs.

Hi, I am adding codecov to my pod and was looking around how it's done in other pods.
I just noticed that according to codecov it's not recommended to prefix the config file name with a dot (reference).

Can I name the file .codecov.yml?
No. Starting the file with a period can cause issues detecting and reading it.

I don't know how it might affect AFNetworking codecov in the future. Just warning in case it can help 🙂


Add IoT Ops Extension v0.2.0b2

Notes for this extension release can be found here:
https://github.com/Azure/azure-iot-ops-cli-extension/releases/tag/v0.2.0b2

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

General Guidelines

[ ] Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
[ ] Have you run python scripts/ci/test_index.py -q locally? (pip install wheel==0.30.0 required)
[ ] My extension version conforms to the Extension version schema

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

Thank you for your contribution! We will review the pull request and get back to you soon.


⚠ Bofrost.lu has degraded performance

In 4a8f530, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1821 ms

Resolved: Bofrost.lu performance has improved in d46561a after 45 minutes.


Tracking issue: Facets

This is the tracking issue for the implementation of faceting in Meilisearch.
Context
Filtering  was thought as two part release. Some time ago we released filtering (https://github.com/meilisearch/MeiliSearch/pull/592), which allows users to create filter documents on arbitrary fields. While this is very flexible it has some performance issues, and when possible, it is preferable to use faceting.
Faceting, the subject of this tracking issue, differs from filtering in that instead of filtering results in the outbound set of documents, it restricts the documents to be filtered on upstream. In order to do that, we must declare facets at indexing. Facets are document attributes that indicate which category a document belongs to.
Filtering and faceting aim at being complementary.
Specification
Setting up facets
Facet that can be used are set up thanks to the [POST]/indexes/:index_uid/settings route. We add a new attribute attributesForFaceting that accepts a [String]. It's default value is null.
Any call to settings with a value for the attribute attributesForFaceting overwrites the currently set faceting attributes. Passing an empty array resets all faceting attributes.
A re-indexing when the value for this setting is changed.
A call to  [POST]/indexes/:index_uid/settings should list the currently set facets.
TODO:

[x] update POST route to accept new attribute  attributesForFaceting
[x] update GET route to include facet information in its response

Document addition and deletion
On document addition and deletion, the facet indexation must be updated:
TODO:

[x] add new document references to the facet index on document addition
[ ] remove document references from the facet index on document deletion

Search query
When doing a search query, a user can specify:

facets to filter on with the facet syntax (explained below), thanks to the optional parameter facetFilters.
the facets for which to retrieve the matching count for, with the facets optional parameter.

facetFilters accept arguments of the form ["facetName:facetValue"] or [["FacetName:FacetValue"]] specifying the facets to be used, with the following semantic:
[["color:red", "color:blue"], "kind:t-shirt"] <=> ("color:red" OR "color:blue") AND "kind:t-shirt"

inner arrays elements are ORed together, outer array elements are ANDed together.
Array depth must not be greater than 2, and be at least 1:

[["color:red"]] => correct syntax
["color:red"] => correct syntax
"color:red" => incorrect syntax

Errors are sent back to the user if:

the syntax is invalid
it is asked to do faceting on an unregistered or not existing facet. (we could send the list of existing facets to the user)

The facets accepts a [String] containing all the facets for which we want to retrieve count information. It defaults to null. If it is set to ["*"], counts for all facets set above will be returned. It will return facets based on a best effort to match the user request. If a facet name is not existing it will be ignored.
The result of the search query contains two additional fields if the facets parameter has been set:

facets: returning a count of candidates for each requested facets in the facets parameter, if set to [*] a count for all facets is returned.
exhaustiveFacetsCount telling whether the above count is exhaustive or approximative.

TODO:

[x] update route to accept new parameters
[x] check the correctness of the query
[ ] update response to take new fields into account, error handling

Documentation
TODO:

[ ] Document the new feature, (see #632)

Further improvement
These are improvements we will add to this feature eventually and will not be part of the initial release. They may however be discussed here for a latter release.

support for negative faceting
"faceting" on number
allow mixing facets and filters with the filter syntax

Related issues
#420
#424
Any developments on this issue will be logged here

update: when a user specifies a facet filter in a search that either is an empty array, or contains an empty array:
facetFitler: []

or

facetFitler: ["foo:42", []]

this is unspecified behavior, and an error is thrown.
Our call is that the only reason a user would specify such a query is by mistake, and he should be informed.

right now the parser for "FacetKey:FacetValue"  in facet filter expressions is very basic, and may not be powerful enough. At some point in time it may be necessary to implement a more powerful parser.


Fix [Text] : Corrected the Spell Error of "Architecture"

Great Package, would give it a shot!

@all-contributors please add @MANISH007700 for doc


Support HTML Preprocessors by using a custom PurgeCSS Extractor

Hi,
I'm using nuxt-bulma-slim with Buefy and there're some components without style when combined with v-if.
b-notification(type='is-success' v-if="sended" has-icon)
    | Mensagem enviada com sucesso! Dentro de 48h úteis responderemos você 🤗
b-notification(type='is-danger' v-if="error" has-icon)
    | Vixi, aconteceu algum problema! 😵 Procure-nos no
    a(href="https://fb.com/idepead") Facebook
    |  enquanto isso 😅

Screenshot on production.
Could anyone help me?

That's mainly because PurgeCSS can not handle HTML preprocessors out of the box. It would need a custom extractor (https://www.purgecss.com/extractors.html) that compiles Pug (& others) to HTML first.
I'll tackle this in the future, when I have time. Or if anyone wants to give a hand, I think this is the way to go:

Write a custom extractor for *.vue files
Use https://www.npmjs.com/package/vue-template-compiler to compile the template to HTML
Use https://www.npmjs.com/package/purge-from-html to extract the CSS selectors
Win

Hey, @mustardamus I'm interested in helping with contributing to this. Can I take this on?


Fixed issue 848

I just noticed upon changing NpcID it says not to edit... whoops
Previously, the barbarian fishing spot in Zeah (Mount Qudamortem) wasn't supported by the fishing plugin. It's id matched that of Alkharid_Warrior, and the warriors npcid changed as well, fixed both of them.

I just realized that instead of modifying the the NpcID file (and was told that all changes would be reverted anyway b/c script takes care of this file) I could just pass the id within the FishingSpot file


root directory is C:\ on a Linux container

Azure Data Studio Version:
OS Version:

Steps to Reproduce:

open server properties on SQL Server 2022 in container
"root directory" value is C:, which seems unlikely (unless I'm missing something about the meaning of that field)

Does this issue occur when all extensions are disabled?: Yes/No

Does the same server shows diff value on Windows/Mac?

Duplicate of https://github.com/microsoft/azuredatastudio/issues/23657


How to generate all the data in the "data" Folder from the "simulation_data" Folder?

The images are rendered by Houdini using the simulation data.

and how to generate ”all_data.json“ file? Could you please give some houdini rendering file?

The houdini files will not be public unfortunately. However, you can use any rendering pipelines to generate images and to test our method.


Writing JUnit tests.

Need to write JUnit tests for the downloadmanager package.

Hi.. I would like to work on this

Thanks. I will leave it to you then, The project is still in infancy, so feel free to modify any code, add a issue and request a pull request.

@jgolatkar Thanks. I will leave it to you then, The project is still in infancy, so feel free to modify any code, add a issue and request a pull request.

Hi! I am also interested in working on this project. Is it okay if we collaborate @jgolatkar ?

@mehulumistry Hi. Yes, we can collaborate!

@jgolatkar @mehulumistry Hey guys, the repo size is greater than 30MB(unusual for a small project like this). So, I re-wrote history, removed all binary files and reduced repo size to less than 1 MB. Make sure you get a fresh clone of the repository and make your changes there. Sorry for the hassle, but this had to be done.

Hello guys!
Maybe I could give a hand?
I started to write a couple of unit tests. Please let me know if you have done anything.
Thanks

Closing this for now. Will do after significant refactoring


Simply table function example, add some comments

This is one potential way to simplify https://github.com/apache/arrow-datafusion/pull/8306

@alamb  Thank you!


[RESP-374] Update codeowners

:tada: This PR is included in version 2.1.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


feat(dataplane): unify abstract base class

Right now, dataplane does not default to abstract for base classes of polymorphic sets. We'll apply the same logic for mgmt plane to data plane in https://github.com/Azure/autorest.csharp/pull/2878
For backward compatibility, we use suppress-abstract-base-class directive to keep the existing behavior for officially released data plane SDKs.
Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.IoT.TimeSeriesInsights


persistent visual glitch

Describe the bug
when i open the game with ct i see either nothing or a bunch of disorganized lines where my is character is supposed to be, if i enter any server or survival world i barely see my surroundings (sice it's dificult to put into words i'll provide screenshots)
To Reproduce
i don't know what causes me specifically to have this bug but all i need to do is open the game normally so if you really wanna know:
1 open the minecraft launcer
2 select the forge 1.8.9 version (more specifically: 1.8.9-forge1.8.9-<IP_ADDRESS>8-1.8.9)
3 click play
4 on the right of the screen your character should be either absent or replaced by a bunch of lines
Screenshots
in the minecraft home screeen

the next two were taken in the hypixel hub but the effect is analogus in any server and/or single player world (note: on the second screenshot you can see another player also being disfigured)

Setup (please complete the following information):

OS: windows 10
ChatTriggers Version: 2.2.0
Minecraft Version: 1.8.9
Java Version: 1.8.0_111-b14

Additional context
before resorting to the bug report i tried to reboot the game multiple times, restart the computer, completely remove the "ChatTriggers" folder from C:\Users(my username)\AppData\Roaming.minecraft\config and let it regenerate, re-install the mod file and get rid of the old one and change my skin out of desperation, but the bug is persistent and i deliberately avoided regenerating the .minecraft folder because i don't know how computers work and want to avoid potentially getting myself in a bigger problem.
I hope i gave enough info and made an overall useful report, this is my first time making one so i apologize if i made some mistakes.

update: i decided to also regenerate the .minecraft folder and the problem still persists, then i remembered i didn't log into my essentials account for a year and thought (once again out of desperation) that it may have something to do with my problem, but when i tried to access it told me "connection denied by localhost" and now i'm stuck again

Your Java version is really old, I would try updating that to the latest Java 8 version and see if that fixes it

ok, i'll try it when i have time, but it would be wierd if it was that because ct worked when i started using it , the visual glitch happened randomly a copule days ago but i have been using ct for at least a month.

i updated java to version 1.8.0_391 but still have the glitch unfortunately

i tried to open thegame again without changing anything and the glitch fixed itself.


Call delegate ScrollListener on scrollChangeState

Call delegated scroll listener on scroll state changed.

@oscarsj can you merge this PR and publish a new version with this change to Maven Central?


Integrate EF Core diagnostics information into ApplicationInsights

We can either extend Microsoft Application Insights for ASP.NET Core applications or create our own package that integrate EF Core's DiagnosticSource (or possibly logging) events into Application Insights monitoring.

@anpete @ajcvickers I believe one of you is going to start working on this soon. Feel free to self-assign.

@anpete @divega How should we track this given that it isn't directly related to any EF milestone.

@ajcvickers I think it's fine to just say we want to do the work in 2.0 time frame and track it accordingly.

Since this isn't directly tired to 2.0 and we will still be working on this post 2.0, moving it out of 2.0 for now.

Fair enough. Adding [area-eternal]. I think could choose to track in a milestone that fits the timeframe in which we want this completed.


Windows 11 installer can’t connect to the internet

UTM v1.5
i was tried so many time and i struck there only like in network connection part while setup the windows 11 os in mac m1 chip there snapshot will uploaded in nelow please resolve this issue bec i can not go further steps

The problem is that a Windows VM in UTM/QEMU does not have network drivers installed as long as Spice tools is not installed.
Download these https://github.com/utmapp/qemu/releases/download/v6.2.0-utm/spice-guest-tools-0.164.3.iso and make a virtual CD ROM (removable drive) from it.
Try these for installing Win11:
https://winaero.com/how-to-install-windows-11-without-an-internet-connection/
https://4sysops.com/archives/how-to-set-up-windows-11-home-with-an-offline-account/
I tried and had success, it installed.
After installing offline, go to D: and install Spice tools and reboot the VM and then you have Win11 with internet connection and if you specified a shared folder of the host PC, it will be visible in drive Z:.

Duplicate of #3865


Live Preview does not work with aliases

Hovering over an alias instead of the original link does not activate the preview pop-up.
Perhaps the syntax is not properly supported yet?

Thanks for the report. I will see what can be done about this. Can I ask you a question. The live preview gets in the way for many users. I am thinking of changing the activation behavior. So instead of just hovering over a link, you would need to press CTRL key and whatever you hover over would show the live preview. What do you think of that?

I personally prefer the current approach because it has less friction. I haven't had a problem with it. For me it takes 2 seconds for the preview to show up so, in general, if it appears it's because I want it to appear. Also, I tend to use the keyboard shortcuts whenever possible, so I use the mouse only when I lose the cursor or for the live preview.

I personally prefer the current approach because it has less friction. I haven't had a problem with it. For me it takes 2 seconds for the preview to show up so, in general, if it appears it's because I want it to appear. Also, I tend to use the keyboard shortcuts whenever possible, so I use the mouse only when I lose the cursor or for the live preview.

DM me on twitter if you are interested in testing the new concept.


Select Custom Options - Mapping an array is not working

Description
I am using the 2.8.5 version of Saas-ui. The Select component is not working if we map the select options inside it. Its working fine for hard code options but doesn't work on map items, with multiple prop set as true. The OnChange is not being triggered
const calendar = ["Testing","Holidays","primary"];

<Select
    multiple
    onChange={(value) => {
      alert(value);
    }}
    name="interest"
    placeholder="Select your favourite cat"
  >
    <SelectButton />
    <SelectList>
      {/* {console.log(calendarByUser)} */}
      {calendar.map((cal, index) => (
        <>
          <SelectOption value={cal}>
            <span>{cal}</span>
          </SelectOption>
          {index !== arr.length - 1 && (
            <MenuDivider
              borderBottom={"3px solid rgba(0, 0, 0, 0.08)"}
            />
          )}
        </>
      ))}
    </SelectList>
  </Select>

Link to Reproduction
No response
Steps to reproduce
Copy and paste the code I shared above and select an option you will notice the onChange is not working
Saas UI Version
2.8.5
Chakra UI Version
No response
Browser
No response
Operating System

[ ] macOS
[X] Windows
[ ] Linux

Additional Information
No response

I've just tested this and the problem seems that wrapping the select SelectOption with a fragment <></> breaks the functionality. This is a limitation of Chakra UI, not something we can fix in Saas UI at the moment unfortunately.


Wrap Echo framework logger that could be used in microservices

Closes #34

log "github.com/coopnorge/go-logger"
echo "github.com/labstack/gommon/log"

These aliases are a bit confusing, but it doesn't matter too much

@DylanWatson44 I know, any better ideas? :) It must be in set pckg adapter/echo since your project may not use echo so go mods will not fetch it

log "github.com/coopnorge/go-logger"
echo "github.com/labstack/gommon/log"

These aliases are a bit confusing, but it doesn't matter too much

@DylanWatson44 I know, any better ideas? :) It must be in set pckg adapter/echo since your project may not use echo so go mods will not fetch it

go_logger and gommon_log maybe. But you can merge as is

log "github.com/coopnorge/go-logger"
echo "github.com/labstack/gommon/log"

These aliases are a bit confusing, but it doesn't matter too much

@DylanWatson44 I know, any better ideas? :) It must be in set pckg adapter/echo since your project may not use echo so go mods will not fetch it

go_logger and gommon_log maybe. But you can merge as is

@DylanWatson44 go_logger name crossing with the repository, not cool for adapter then. To merge I need anyway 2 approves, so maybe labstack_logger?

log "github.com/coopnorge/go-logger"
echo "github.com/labstack/gommon/log"

These aliases are a bit confusing, but it doesn't matter too much

@DylanWatson44 I know, any better ideas? :) It must be in set pckg adapter/echo since your project may not use echo so go mods will not fetch it

go_logger and gommon_log maybe. But you can merge as is

@DylanWatson44 go_logger name crossing with the repository, not cool for adapter then. To merge I need anyway 2 approves, so maybe labstack_logger?

Sure that works


Add rejectionHandlers field to LoggingOptions

Fixes #1801

I realised that someone made the exact same PR: #1847

@leonardehrenfried, the https://github.com/winstonjs/winston/pull/1847 is not the same PR. It's adding handleRejections (a boolean), yours is adding rejectionHandlers. I believe both should be added.

@leonardehrenfried, the https://github.com/winstonjs/winston/pull/1847 is not the same PR. It's adding handleRejections (a boolean), yours is adding rejectionHandlers. I believe both should be added.

are  handleRejections and rejectionHandlers added in LoggerOptions in the lastet winston version ?


[REACH-344] feat: Start on the Landing Page Section area in Gamut

Overview
PR Checklist

[x] Related to designs: https://www.figma.com/file/2aBlW1pgFt5PT6soRu1HXy/✏-Landing-Page-Templates---Unification?node-id=1%3A43
[x] Related to JIRA ticket: REACH-344
[x] I have run this code to verify it works
[ ] This PR includes unit tests for the code change

Description
These are just some of the smaller entities that be used in a number of Landing Page Section organisms.

 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.Judah Anthony seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

📬Published Alpha Packages:@codecademy/gamut-labs@4.6.3-alpha.462e37.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5fa438475d566e0a86eeaa91--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-labs@4.6.3-alpha.ad8961.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5fa43a5c23b839009f8de849--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-labs@4.6.3-alpha.3789e0.0<EMAIL_ADDRESS>

Glorious start! Nice to see usage of the new styled components and mdx stuff. I can take a deeper look once it builds 👍

I think I have addressed all your concerns.

🚀 Styleguide deploy preview ready!https://5fa4452e662f63111b4c35c6--gamut-preview.netlify.appDeploy Logs

LGTM!
Per your post in Slack, I do agree that I see these more as atoms than organisms however.

100%
I ultimately would just like a LandingPageSection "gallery" for our marketers, so the shouldn't even see thing like "title" which they cannot directly choose.

📬Published Alpha Packages:@codecademy/gamut-labs@4.6.3-alpha.bb1432.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5fa44855ccaccc104fc90c66--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-labs@4.8.0-alpha.21fc6d.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5fa44a12d9e83810d0af06e0--gamut-preview.netlify.appDeploy Logs


Currency Is Not Lootable

Players are unable to loot any currency displayed in the loot sheet. Neither the Loot All button nor Loot Coins button will transfer the currency.

Please test the latest version.
You can take a look at #222 , I attached a file of the latest unreleased version if by the time you read this
the release is not version 3.4.25 or higher.


[Request] Compat for Advanced Netherite Mod

Mod Link: https://www.curseforge.com/minecraft/mc-mods/advanced-netherite

Started on this, but a little perplexed how to proceed.
Take the Axe for example, it currently requires 6 Netherite to make.
Should the Advanced ones take a further 6?

I think its fine as long as its the same with all the other tools. Maybe tune down the 6 since the mod only upgrades already-built netherite tools and weapons?

Made a start

Still thinking how to do the recipes.

Recipes will require 1 Advanced Netherite Ingot to make the dull part and 1 to make the whetstone.
As well as requiring normal Netherite.
eg..

This is open for discussion, but will be released like this.

Lookin' good already, and the adjustments on the sprites are well-made. Thanks for the update!
P.s. Someone really needs to work on that grindstone xD

Is that an offer? 😉


Experimental GRID layout

Try making grid layout using experimental CSS grid instead of flexbox
more: http://gridbyexample.com/
support: http://caniuse.com/#feat=css-grid

flexbox still wins for now, CSS grid is not even supported without custom browser config


Hide IE's clear button from input control

When the input control has focus and is not empty, an 'X' button appears. This overlaps with the existing trigger button. This issue appears on Internet Explorer 10 and 11.

Thanks for your contribution. We'll check the PR as soon as we can.


ResourceManager HA creates problem in Spring XD

While deploying Spring XD via Ambari, we give
spring.hadoop.resourceManagerHost=
Now when due to HA in ResourceManager, StandBy Resource Manager is Active,
Spring XD Jobs requiring YARN execution fails as it does not have capability to access New Resource Manager

We are also facing same issue. An fix planned for this issue?

Hi @rajitsaha and @sureshcoder: There's a separate Spring XD Ambari repo. This one could be related to spring-projects/spring-xd-ambari#9.
As an FYI: @jvalkeal released Ambari plugin for Spring XD 1.3 GA bits. It'd be better to upgrade to the latest improvements and bug-fixes.
I'll close this issue; but, please feel free to add your comments to follow spring-projects/spring-xd-ambari#9 directly.

Hi @sabbyanandan I upgraded Springxd 1.3 and Ambari plugin for SpringXD 1.3.
I am still facing the same issue
spring-projects/spring-xd-ambari#9. issue is still open.
If this issue is only because of Ambari plugin then could you point out Spring-XD document which explains configuring SpringXD for resource Manager HA?

@sureshcoder did you get any hints on the documentation for RM HA? Regards.


archive list layout default value from int to str

When building the site, without config params archiveGroupByDate set, hugo throws an error as the default type is not correct.
Rebuild failed:

Failed to render pages: render of "section" failed: "/hugo/themes/zzo/layouts/archive/list.html:13:170": execute of template failed: template: archive/list.html:13:170: executing "main" at <2006>: wrong type for value; expected string; got int

Thank you so much!


For 3.4.0: Update Set-AzureRmKeyVaultAccessPolicy for example 7 and B…

…YOK, update New-AzureRmKeyVault for more info on soft-delete, formatting fixes for both

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

azureps-cmdlets-docs/ResourceManager/AzureRM.KeyVault/v3.3.1/New-AzureRmKeyVault.md
:white_check_mark:Succeeded

azureps-cmdlets-docs/ResourceManager/AzureRM.KeyVault/v3.4.0/New-AzureRmKeyVault.md
:white_check_mark:Succeeded

azureps-cmdlets-docs/ResourceManager/AzureRM.KeyVault/v3.4.0/Set-AzureRmKeyVaultAccessPolicy.md
:white_check_mark:Succeeded

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

#sign-off @TaraMeyer Finally finished validation ....ready to merge, please!

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

azureps-cmdlets-docs/ResourceManager/AzureRM.KeyVault/v3.3.1/New-AzureRmKeyVault.md
:white_check_mark:Succeeded

azureps-cmdlets-docs/ResourceManager/AzureRM.KeyVault/v3.4.0/New-AzureRmKeyVault.md
:white_check_mark:Succeeded

azureps-cmdlets-docs/ResourceManager/AzureRM.KeyVault/v3.4.0/Set-AzureRmKeyVaultAccessPolicy.md
:white_check_mark:Succeeded

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


Support deploying as AWS Lambda

Not sure if this is at all in scope or anything you would like help with, but I wanted to file this to get your thoughts.
Does the architecture of this application lend itself to being able to be deployed to AWS Lambda? My company doesn't have much in terms of container infrastructure and instead is pushing things to AWS Lambda when possible. We use CloudFormation templates to do this with our probot apps, but I understand that this is instead a Vue application, so it may not be the same thing.

It's a static web app, so it could be deployed to S3 bucket (https://docs.aws.amazon.com/AmazonS3/latest/userguide/HostingWebsiteOnS3Setup.html)
when executed as static web app - tokens are exposed in the network traffic.
You can also deploy it as docker image, anywhere docker images can run (Kubernetes, container apps etc.)
Lambda supports running custom images https://docs.aws.amazon.com/lambda/latest/dg/images-create.html#images-types
If you can define a route that will serve files, it's going to work.


safari FFMPEG Encoding issue

Wavesurfer.js version(s):
6.1.0
Browser and operating system version(s):
Safari Version 15.0 (176<IP_ADDRESS>)
MacOS 12.0 Beta (21A5506j)
Version 100.0.4896.127 (Official Build) (arm64)
Code needed to reproduce the issue:
ffmpeg().input(stream.Readable.from(src)).duration(15).outputFormat('wav').audioFrequency(44100).audioBitrate('128k').noVideo()

Use behaviour needed to reproduce the issue:
When I encode files, they work in wavesurfer chrome, but not on safari. This may be a me issue, as I'm pretty new to FFMPEG, but I think it could be a wavesurfer bug as its a different behavior on different browsers.
To take a look at whats going on, see here

This is not a wavesurfer.js issue.


Invalid JUNIT file

Running the MIES hardware tests with a ITC1600 hardware results in some test errors. But the generated XML file is also invalid, which should never be the case.
Run_0002.zip

Diagnosis: Looks like memory corruption in the string that is saved occurs or on disk. Could not reproduce locally. It might be that the source of the memory corruption is not the unit testing framework itself but MIES using a specific hardware setup.
This issue needs more input and in best case a reproducible setup for further debugging.
We added some tracking of the issue in MIES in https://github.com/AllenInstitute/MIES/pull/1295 to get better statistics when the issue comes up. (And to prevent Bamboo analyzing partial JUNIT xml files)

The MIES PR is now merged. I'm unassigning you until we have some more clues about it.

Another file which was generated using running the tests locally with ITC1600 from Tim.
In this case the hardware did not crash.
JU_2022_09_01_171633_2022-09-01_18-24-04.zip
Line 6166 in the hex editor following:
00018150: 2061 2068 656c 7090 5306 a839 5600 0090  a help.S..9V...
00018160: 5306 a839 5600 0000 0000 0000 0000 0000  S..9V...........
00018170: 0000 0000 0000 00d0 b5eb a639 5600 0000  ...........9V...
00018180: 0000 0000 0000 00a0 b3eb a639 5600 00f8  ...........9V...
00018190: 5306 a839 5600 0000 0000 0000 0000 00b0  S..9V...........
000181a0: 5306 a839 5600 0000 4000 0000 0000 00b0<EMAIL_ADDRESS>000181b0: 5306 a839 5600 0000 0000 0000 0000 0020  S..9V..........

I suggest we add an allocation of memory to e.g. the XOP at init of ~100 MB. Then we run IP in the Vs debugger and add a conditional breakpoint when a store access happens in the allocated memory.

Closing for now.


lb dl downloads more than it should

Suppose I add the following playlist https://www.youtube.com/playlist?list=PLqxP5EuGxPnfmg5P0_96bz9E__-XrLuFc by doing :
lb tubeadd test.db https://www.youtube.com/playlist?list=PLqxP5EuGxPnfmg5P0_96bz9E__-XrLuFc -vv

How can I download a single video using its relevant url from all those added to the database by lb tubeadd?
Normally, I'd expect the following command to do the trick:
lb dl test.db --video https://www.youtube.com/watch?v=MhlFR2wWqHA -vv

Instead of downloading only this video, lb dl downloads them all, which makes progress polling problematic and leads to a race condition.

Here is how I debugged this:
lb dl test.db --video https://www.youtube.com/watch?v=MhlFR2wWqHA -vvvv -pa

...

{'profile': 'video', 'extractor_config': {}, 'prefix': '/home/xk/github/xk/lb', 'print': 'a', 'sort': 'm.path like "http%"\n        , m.title IS NOT NULL desc\n        , m.path\n        , random()', 'where': [], 'include': [], 'exclude': [], 'retry_delay': '14 days', 'db': <Database <sqlite3.Connection object at 0x7fbacce01e40>>, 'verbose': 4, 'database': '/tmp/tmp.ehhyY7Lx7h/test.db', 'playlists': ['https://www.youtube.com/watch?v=MhlFR2wWqHA'], 'defaults': ['sort', 'limit'], 'unk': [], 'action': 'download'}

...

path         count
---------  -------
Aggregate       15

We can see that the URL is being parsed into args.playlists. And the total number of downloads it has queued up is 15.
lb dl test.db --video -s https://www.youtube.com/watch?v=MhlFR2wWqHA -vvvv -pa

...

{'profile': 'video', 'extractor_config': {}, 'prefix': '/home/xk/github/xk/lb', 'print': 'a', 'sort': 'm.path like "http%"\n        , m.title IS NOT NULL desc\n        , m.path\n        , random()', 'where': [], 'include': ['https://www.youtube.com/watch?v=MhlFR2wWqHA'], 'exclude': [], 'retry_delay': '14 days', 'db': <Database <sqlite3.Connection object at 0x7f2a8246de40>>, 'verbose': 4, 'database': '/tmp/tmp.ehhyY7Lx7h/test.db', 'playlists': [], 'defaults': ['sort', 'limit'], 'unk': [], 'action': 'download'}

...

path         count
---------  -------
Aggregate        1

Now it is using args.include and queuing up only a single video which is what was desired.
According to the documentation args.playlists should be used to limit lb dl to specific playlists but it looks like that functionality was never added so it isn't actually doing anything. But to filter by video URL --include or --search (-s) is the correct way and will be required.
After this bug is fixed the behavior of downloading everything when supplying a video URL should not happen any more but instead it should print "No media found" to stderr and exit with a nonzero code

To download a specific playlist(s) it's possible to do this now:
library fs video.db --to-json --playlists https://www.youtube.com/c/BlenderFoundation/videos | library download --video video.db --from-json -

Note that the - is required in the download command to read from stdin.
The existing functionality should be preserved but now you should be able to download videos without running tube-add (but only for single-video or the first video of a playlist--if you want the full playlist use tube-add first):
library download --video video.db $URL

Or line-delimited files (similar to yt-dlp -a)
library download --video video.db --file file_with_urls1.txt file_with_urls2.txt


change order of testing and saving ckpt

Change the order of performing test and saving checkpoint when the epoch reaches test interval, so that the checkpoint can be safely pre-saved if there exist problems or bugs in the testing process

Great! Merged.


Package fixes and updates (for 2.0 release)

I've prepared some minor fixes and updates:

run license header checks also in tests
added license header checks to composer check script (so it will be checked also with every build on travis)
updated license headers in tests
allowed PHPUnit 6.0
updated PHPDocs - added missing params/fixes/typos
added private properties in tests
CS fixes (removed redundant spaces)
use constants (StatusCode, RequestMethod)
if condition improvements
removed ignoring cs checks sections + cs fixes
updated docs
updated suggest section in composer to reflect latest version of packages
removed unused imports

I've noted also that:

NOTES.md is outdated (there is note about it on the top, maybe will be good to do something with it...)
there is one "todo: remove in 2.0" https://github.com/zendframework/zend-expressive/blob/develop/test/IntegrationTest.php#L82-L85 - should it be removed and maybe also some other functionality then?

@webimpress I'll remove NOTES.md and the test you reference during merge. Thanks!

Thanks, @webimpress!


Parser ignore the description if it's a $ref

As said in the title, latest stable release ignores the description if it's a $ref.
It looks like a regression to me because it was working properly a few versions back (can't be sure when, I found at using a code generator).
[main] INFO  o.o.codegen.DefaultCodegen - debugging Cateogry: class Schema {
    type: null
    format: null
    $ref: #/components/schemas/Category
    description: null
    title: null
    multipleOf: null
    maximum: null
    exclusiveMaximum: null
    minimum: null
    exclusiveMinimum: null
    maxLength: null
    minLength: null
    pattern: null
    maxItems: null
    minItems: null
    uniqueItems: null
    maxProperties: null
    minProperties: null
    required: null
    not: null
    properties: null
    additionalProperties: null
    nullable: null
    readOnly: null
    writeOnly: null
    example: null
    externalDocs: null
    deprecated: null
    discriminator: null
    xml: null
}

description is null even though I put down something as follows:
       category:
         $ref: '#/definitions/Category'
        description: "description testing"

For info: Having any property set next to $ref is not a valid OpenAPI document. See https://github.com/OAI/OpenAPI-Specification/issues/556 for details. (you will see my comments and a lot of other telling that $ref+ description would make sense)

Hi, please check https://github.com/OAI/OpenAPI-Specification/blob/master/versions/3.0.2.md#reference-object
This object cannot be extended with additional properties and any properties added SHALL be ignored.

I have the same problem, how can I get the description?


Add checks for local DOM ready. Fixes #61

Adds some small checks to make sure the this.$ bindings are ready before trying to use it. This should fix #61, however it's not an ideal fix - really the bindings should be setup, so this will need to be replaced with a more robust solution down the track.

@seaneking if this resolves the issue for you, can you LGTM and merge?

👍 LGTM


Weekly guidance week4

One thing that is left is to add link to youtube videos on /week4/

This is great, thank you! I only made a few minor changes.


[FEATURE] Reset map

Hi @Astropilot,
After testing many of the features offered by ValheimTooler V1.0 I think I have found a new idea to implement😉.
Currently, the tool proposes to explore the whole map but it's impossible to go back and a miss click is so quickly arrived 😅...
I think it would be complicated to restore the "legal" exploration but at least reset it to avoid a case of no return and not spoil a character on this world permanently.
Adding a "reset map exploration" button could be relevant what do you think ?

Hello Bastien,
I agree with you, the exploration feature of minimap being irreversible, it would be appropriate to propose to reset minimap so as not to spoil the gaming experience.
I will add it to the roadmap for the next version.


added Login view with facebook and twitter buttons

added Login view with facebook and twitter buttons, added userName and password text fields. also added shape, color and design, DB is down for this moment

 Create Login View and allow user to login


Modularize logging

Right now all logging is done via syslog. While this is good as we move further to a general open source platform it's important to be able to build additional options. With that said it would be good to come up with a scheme that allows for anyone to pop in a module to add additional logging mechanisms like going to redis, rethinkdb, elastisearch or a generic file.

Or syslog forwarding, being able to send log output to Splunk, Logstash, or any other arbitrary syslog endpoint.