Improve development and deployment using Containerisation (Docker Images)

Summary:
I am proposing the introduction of Docker containerization to the slideio project to simplify the development and deployment processes, especially regarding dependency management and compiler standardization.
Background:
slideio works flawlessly via the python bindings, which are easily installed. However, setting up the environment for C++ usage and managing dependencies can be challenging due to the library's complex dependencies and the use of specific versions of external libraries. Additionally, the library aims to support a wide range of Linux distributions dating back to 2014, which introduces further complexity in terms of compiler standardization.
Issue:
Complex dependencies and outdated documentation makes it difficult for new users to setup for development of the library. Because of the requirement of C++14, Boost is needed to supply functionality which has been adopted in recent C++ standards (e.g. boost::filesystem), adding an unnecessary dependency.
Proposed Solution:
Implementing Docker containerization for slideio would address these challenges by:

Streamlining the Development Environment:
Docker would allow us to create a consistent development environment across different systems. This eliminates the issues arising from dependency management and varying OS/compiler configurations.

Facilitating Adoption of Newer C++ Standards:
By containerizing the development environment, we can include specific compiler versions that support newer C++ standards, enabling us to modernize the codebase without worrying about compatibility with older Linux distributions.

Ease of Deployment:
Docker containers can simplify the process of deploying slideio, making it more accessible to new users and contributors.

Conclusion:
I am convinced that Docker containerization could be a substantial improvement to the slideio project, both from a development and a user standpoint. It aligns with @Booritas previous mention of the challenges faced due to dependencies and having to support many linux distributions.
Thank you for considering this proposal.

Thank you for your proposal; it makes a lot of sense. We need to take into account that the following operating systems are supported by slideio:

Mac OS x64
Mac OS arm
Windows x64
Linux x64 distributions starting from 2014.
Linux arm is planned.

Providing a Docker container will facilitate the development of the library. Originally, Docker containers were Linux-based. Later, Windows containers appeared. Recently, I've seen news about the availability of OSX containers. I had a bad experience with Windows containers, and I would expect that OSX containers are still not in a good state. Please correct me if I'm wrong. For Linux, we have a broad range of available containers. For creating deployments, I use manylinux2014_x86_64. Unfortunately, the container is not suitable for development due to some missing features that prevent VS Code/CLion from connecting to the container for debugging purposes. Therefore, we can select any Linux container suitable for development, ensuring that the code is compilable by manylinux_2014. It makes sense to create two containers - one for active development (my preference would be Ubuntu) and another for checking with manylinux_2014. Both containers should have Conan installed with all dependencies. This shouldn't involve much effort.
The use of the latest standards is currently restricted by manylinux_2014, which uses GCC 10 toolkit. As far as I know, GCC 10 has good coverage of C++14. In the previous versions of the library, I used manylinux_2010 with GCC 8, which had quite poor support for C++ standards, necessitating the use of Boost.
By the way, the filesystem is a C++17 feature, which is why I haven't used it yet.
So, my questions to you are:

Which Linux distribution should we use for the development Docker container?
Should we also try Windows and OSX containers?
Would you find time to work on this feature (with my support, of course)?

It looks like manylinux2014 will "reach End of Life (EOL) on June 30th, 2024" as it is based on CentOS7. The newest manylinux_2_28 images are available for x86_64, aarch64, ppc64le and s390x. It is based on GCC 12, which has basically complete coverage of C++20.

The development container should be basically the same as the build image (manylinux_2_28)
Windows and macOS(Intel and arm) both support running linux docker containers via Docker Desktop
Sure!

For this to work, we should also be compiling the custom dependencies when compiling the source. This would also make your private conan server obsolete. If the compile times become prohibitively long, we may consider a modified docker image where they are already compiled.
In general, it'd be great to get more information on these @slideio/stable dependencies.
Why is it necessary to host these specific versions? What makes other versions, hosted on conan center, unsuitable?

I just realized that continuing support for Windows obviously requires a native Windows environment. I am not familiar with developing for Windows, so unfortunately can't help with that. As for macOS, a well documented dockerfile should suffice for anyone to setup the repo for development.

It looks like manylinux2014 will "reach End of Life (EOL) on June 30th, 2024" as it is based on CentOS7. The newest manylinux_2_28 images are available for x86_64, aarch64, ppc64le and s390x. It is based on GCC 12, which has basically complete coverage of C++20.

The development container should be basically the same as the build image (manylinux_2_28)
Windows and macOS(Intel and arm) both support running linux docker containers via Docker Desktop
Sure!

For this to work, we should also be compiling the custom dependencies when compiling the source. This would also make your private conan server obsolete. If the compile times become prohibitively long, we may consider a modified docker image where they are already compiled.
In general, it'd be great to get more information on these @slideio/stable dependencies. Why is it necessary to host these specific versions? What makes other versions, hosted on conan center, unsuitable?

It is good to know that multilinux_2014 has reached the end of its life, and there is a better replacement. Let us use manylinux_2_28 for future releases.
We don't have to compile all dependencies for the new container. It would be enough to download them from a Conan server (private or central). You're right; we don't need any Conan server after building such a container.
Regarding slideio@stable dependencies, there is a small set of private packages located in the repository https://github.com/Booritas/conan-recipes that do not exist in Conan-Central or have some modifications to the original sources. Note: not all recipes in the repository are used, and I need to clear it up. The most important one is jxrLib - a recipe for the JXR codec. There were also some conflicts between dependencies that I resolved by making small changes in the build scripts.
Another reason for creating a copy of the packages is that some packages on the central server do not contain builds for the required OS and compiler, especially for the Mac ARM processor. Creating a copy allows compiling the recipe and saving binaries to avoid recompilation multiple times.
The rest of the packages are copies from the central Conan server. JFrog often rebuilds the packages. I've encountered the problem that the same build does not work one week later because a package was recompiled with another GLib. This happened every time I prepared a new version. Fixing such a problem may take a week or two, taking into account that I have my main job and can only work evenings and weekends. Creating the copies saved me a lot of time.

I forked the mentioned recipes repo and started a dockerfile.

You don't need this repo. You can download compiled files from the canon server. Here is my old code for the manilinux docker file. Now I would do it a bit differently. I would clone slideio repo and run
python install.py -a conan . It would install all dependency in the container.

In other words, I would use beginning of the Dockerfile (updated to the new base):
FROM quay.io/pypa/manylinux2014_x86_64:latest AS base
ARG CONAN_REMOTE
ENV CONAN_REMOTE ${CONAN_REMOTE}
ENV CONAN_REVISIONS_ENABLED 1
RUN yum install -y wget
RUN yum install -y vim
RUN yum install -y gtk2-devel
RUN yum install -y libva-devel
RUN yum install -y soci-sqlite3-devel.x86_64
RUN yum info m4
RUN yum -y update m4
RUN yum info m4
RUN update-alternatives --install /usr/bin/python3 python3 /opt/python/cp38-cp38/bin/python3 10
RUN update-alternatives --install /usr/bin/pip3 pip3 /opt/python/cp38-cp38/bin/pip3 10
RUN yes | pip3 install numpy
# install cmake
RUN yum remove cmake -y
RUN wget -qO- "https://github.com/Kitware/CMake/releases/download/v3.22.3/cmake-3.22.3-linux-x86_64.tar.gz" | tar --strip-components=1 -xz -C /usr/local
RUN update-alternatives --install /usr/bin/cmake cmake /usr/local/bin/cmake 10
RUN update-alternatives --install /usr/bin/ccmake ccmake /usr/local/bin/ccmake 10
#install conan
RUN yes | pip3 install conan
RUN update-alternatives --install /usr/bin/conan conan /opt/python/cp38-cp38/bin/conan 10

Add
clone of the slideio repo
python install.py -a conan
remove slideio directory

This should do the trick.

I see what you mean, but I'd like to actually compile every dependency specific to slideio, i.e. not pull anything from your slideio/stable conan server but build those dependencies in the image. That not only enables the close inspection of those dependencies but is also necessary to use a compiler of my choosing. Currently these dependencies prevent using some compilers due to ABI mismatches etc.

You can force conan to rebuild all packages that come from the server. It is very easy to do with a small change in the install.py script. You need to use parameter --build Here modified function in the install.py:
def process_conan_profile(profile, trg_dir, conan_file):
    generator = "cmake_multi"
    command = ['conan','install',
        '-pr',profile,
        '-if',trg_dir,
        '-g', generator,
       '--build'
        ]
    command.append(conan_file)
    print(command)
    subprocess.check_call(command)

In this case, conan will take only recipes from the server and recompile all dependencies. You can as well set compiler of your choice in the conan profile:
[settings]
os=Linux
arch=x86_64
compiler=gcc
compiler.version=10
compiler.libcxx=libstdc++
build_type=Release
[options]
tinyxml2:shared=False
gdal:fPIC=True
[build_requires]
[env]

Conan supports quite a large range of the compilers.
After you run install.py all dependencies will be recompiled with a compiler you set and copied to the local conan repository and ready to use. You don't have to recompile them next time nor you have to pull anything from the server. You can work offline.


(Dev) Dependencies?

Your quick start guide tells me to install all packages as a dependency:
npm install formsnap sveltekit-superforms zod

But Superforms tells me to install as dev dependency:
npm i -D sveltekit-superforms zod

I feel like installing all these packages as normal dependencies and shipping them to production makes the site performance very worse. Do we actually need to install them this way?

When using SvelteKit/Vite, it doesn't matter anymore: https://github.com/vitejs/vite/discussions/1803#discussioncomment-322107


Dungeon Raid can gen uncompletable

Mod Version:  0.0.8
Forge Version: 2772
What happens:
Currently, if the Lava parkour room generates as the first room from the entrance, it's impossible to get out of the original room, as the stairs for the lava room block the door.
https://i.imgur.com/uGUSOZf.png

oops.
If you happen to still be stuck (or anyone else finds this issue), the best way to get out would likely be to put yourself into spectator mode to fly though.
I'll get this fixed up for the next release.

Yea that was the solution we went with at the time

Finally got this fixed for the next release


Gradle's JVM probing logic fails when using JDK with wrong page size on ARM

Due to this open JDK bug  Gradle JVM probing fails as there's unexpected output with this additional line:
[0.015s][error][cds] Unable to map CDS archive -- os::vm_allocation_granularity() expected: 4096 actual: 65536

The probing logic in Gradle should be more resilient since this "error" isn't actually fatal.
Expected Behavior
Gradle JVM probing does not fail in the described scenario
Current Behavior
Gradle JVM probing fails with the following output:

Unable to determine version for JDK located at /var/lib/jenkins/.gradle/caches/transforms-3/741c1b0a94f5a1be265ba151b1841ee6/transformed/linux-15.0.1-aarch64.tar.gz. Reason: Unexpected command output:
[0.003s][error][cds] Unable to map CDS archive -- os::vm_allocation_granularity() expected: 4096 actual: 65536
/var/lib/jenkins/.gradle/caches/transforms-3/741c1b0a94f5a1be265ba151b1841ee6/transformed/linux-15.0.1-aarch64.tar.gz
15.0.1
AdoptOpenJDK
aarch64
OpenJDK 64-Bit Server VM
15.0.1+9
OpenJDK Runtime Environment

at org.gradle.internal.jvm.inspection.DefaultJvmVersionDetector.validate(DefaultJvmVersionDetector.java:59)
at org.gradle.internal.jvm.inspection.DefaultJvmVersionDetector.getVersionFromJavaHome(DefaultJvmVersionDetector.java:52)
at org.gradle.internal.jvm.inspection.DefaultJvmVersionDetector.getJavaVersion(DefaultJvmVersionDetector.java:48) |
at org.gradle.api.tasks.testing.Test.getJavaVersion(Test.java:273)

Context
We run into this when running our builds on aarch64 systems.
Steps to Reproduce

Setup linux aarch64 System
Install AdoptJDK 15.0.1+9 AARCH64
`EXPORT RUNTIME_JAVA_HOME=<PATH TO ADOPTJDK 15>
clone elasticsearch
checkout master
run ./gradlew  :x-pack:plugin:ml:qa:native-multi-node-tests:javaRestTest

Environment
Operating system | Linux 4.18.0-240.1.1.el8_3.aarch64 |
-- | -- | --
CPU cores | 32 cores |
Max Gradle workers | 32 workers |
Java runtime | AdoptOpenJDK OpenJDK Runtime Environment 15.0.2+7 |
Java VM | AdoptOpenJDK OpenJDK 64-Bit Server VM 15.0.2+7 (mixed mode) |
Max JVM memory heap size | 29.9 GiB |
Locale | English (United States) |
Default charset | UTF-8 |
Username | jenkins |

One extra piece of information if you try to reproduce this is that the error only gets printed if the linux-aarch64 machine you reproduce on has a different page size to the machine that your JDK was built on.
On linux-aarch64 RHEL/CentOS distributions are built with a page size of 64KB and Ubuntu/Debian/SLES distributions are built with a page size of 4KB.
We see the problem with AdoptJDK 15.0.1+9 AARCH64 on CentOS 8, which means that AdoptJDK 15.0.1+9 AARCH64 must have been built on a distribution that was built with a 4KB page size.
So if you try to reproduce with AdoptJDK 15.0.1+9 AARCH64 you will need to do this on RHEL or CentOS.  If you try to reproduce with AdoptJDK 15.0.1+9 AARCH64 on Ubuntu, Debian or SLES then you will not see the problem.

Thanks for all the details on this, very appreciated!

Are there plans to fix this for 7.0?


Rebase of PR #794 (fixes #793)

https://github.com/benmosher/eslint-plugin-import/pull/794 appeared stalled, so I just updated it.  LMK what to do RE changelog and squashing.
@Ephys, LMK if you rebase #794 and I can just close this (thank you for creating this, I just didn't want to see it languish :) ).
Closes #793

This change is

Coverage increased (+0.002%) to 95.967% when pulling db253b78680e7ce3a1e4675627fc8fbcf81dd1ce on SpainTrain:pr-794 into dd281306089ba15394aeb2b85241509286442e34 on benmosher:master.

It's not a good idea to open a replacement PR; if the collaborators want to continue it, we can push directly to that PR. Opening up a new one just clutters up the repo. A comment on the original PR would have been better.

@ljharb
In general, yes I agree.  However there was a comment 23 days ago and a comment similar to your suggestion 15 days ago in that PR,which neither have been responded to.  I understand maintainers are busy, so this was an attempt to minimize work for others.  For organizational purposes, I thought this was pretty well titled to indicate that it is an intentional dupe.
I am happy to follow house rules an any repo, but tend towards taking (even a small bit) of work off others' plate rather than add noise to a discussion.
Thanks!

My apologies for my lack of response, what should I do to help sort this out? :)
I'm happy to close my PR in favor to this one, or merge your PR with mine. Whichever solution works best for you

@Ephys the rebase is easy, so probably best all around to rebase yours if you can?  Thanks again.


Add query to staking contracts to list stakers.

The ability to list all stakers is important for visualizing membership and the distribution of voting power in a DAO.
Specifically, we'll need to add a ListStakers { start_after: String, limit: u32 } query to cw20-stake and cw-native-staked-balances-voting. For an example of doing this, cw721-stake already has this implemented here:
https://github.com/DA0-DA0/dao-contracts/blob/87f94960413f53d8f49176ce9330372766655ffc/contracts/cw721-stake/src/contract.rs#L470-L491
I suspect that the query for the other contracts will look pretty similar to this.

@borgbob should we take a stab at this? We definitely could use this functionality for our own DAO!

@bekauz is taking this on. :)


Support HEEx files. Works with PhoenixFramework plugin.

Some small changes to get this to work with HEEx files. Note that I also use the Phoenix Framework plugin for syntax highlighting. https://github.com/phoenixframework/vscode-phoenix

👌 Found this from https://elixirforum.com/t/vscode-phoenix-heex/42461/5 and would love to see it added if possible.
Currently trying to get formatting working for HEEx and haven't found any good solutions. this looks promising :)


ENH: use analytic formula to fit log-Laplace distribution

Is your feature request related to a problem? Please describe.
As mentioned in this comment, the link between log-Laplace and Laplace distribution could be exploited to derive an analytical solution to the MLE of log-Laplace, which is not done currently. gh-17832 also mentions the desirability to have analytical fit method where applicable.
Describe the solution you'd like.
The improvement could go in three steps:

update the documentation to mention the link between Laplace distribution and log-laplace distribution.

Implement MLE using analytic formula. For known floc this is straightforward; for unknown floc a univariate solver may be needed.

For completeness, implement MM using closed-form formula or efficient solver.

Describe alternatives you've considered.
No response
Additional context (e.g. screenshots, GIFs)
No response

The link between log Laplace and Laplace parametrization is as follows.
Let $Z\sim \textrm{LogLaplace}(c,L,S)$ according to scipy’s parameterization, where c, L, S are respectively the shape, location, and scale parameter. The density on the doc page is equivalent to
$$
f_Z(z) = \frac{c}{2(z-L)} e^{-c \left| \ln\frac{z-L}{S}\right|}
$$
Consider random variables $X$ and $Y$ such that $X=h(Y)$ for some strictly monotone function h. It can be shown that their densities are linked by
$$
f_Y(y)=f_X(h(y)) h’(y)
$$
Now let $X\sim \textrm{Laplace}(a,b)$ where a is the location and b is the scale. It’s density is
$$
f_X(x)=\frac{1}{2b}e^{-\frac{|x-a|}{b}}
$$
Let $h(y)=\ln y$. Then
$$
f_Y(y)=\frac{1}{2by} e^{-\frac{1}{b} \left| \ln\frac{y}{e^a}\right|}
$$
By matching coefficients, one can see $f_Y$ is equivalent to $f_Z$ with $c=1/b$, $L=0$, and $S=e^a$.

The MLE for Laplace distribution is well known, but is shown below because it will be used later.
Let $\mathbf{x}=(x_1, \cdots, x_n)$ be a sample of n i.i.d. observations drawn from $\textrm{Laplace}(a,b)$. Let $\sum\equiv\sum_{i=1}^n$. The log likelihood is
$$
\mathcal{l}(a,b)=-n\ln(2b)-\frac{1}{b}\sum |x_i-a|
$$
The log likelihood is maximized at
$$
\hat{a} \in \mathrm{med}(\mathbf{x}), \hat{b}=\frac{1}{n} \sum |x_i-\hat{a}|
$$

The MLE for log-Laplace proceeds as follows. Let $\mathbf{y}=(y_1,\cdots,y_n)$ be a sample of n independent observations drawn from $\textrm{LogLaplace}(1/b, m, e^a)$, i.e. using the alternative parameterization. Denote $x_i:=h(y_i)=\ln(y_i-m)$. The log likelihood of the sample can be shown be to
$$
\mathcal{l}(m,a,b)=-n\ln(2b)-\frac{1}{b}\sum |x_i -a| -\sum x_i
$$
If $m$ is known, $x_i$ may be treated as constants and the MLE is exactly the same as the Laplace case.
If $m$ is not known but a and b are both known, let $y_i’=y_i e^{-a}$ and $m’=m e^{-a}$, and write
$$
\mathcal{l}(m’)=-\frac{1}{b}\sum |\ln(y’_i-m’) | -\sum \ln(y’_i-m’)+\textrm{const}
$$
where $m’\le y’_i$. We proceed to maximize $\mathcal{l}(m’)$. To simplify notations, drop the prime.

If $c=1$, the first term vanishes and we’re left with
$$
\mathcal{l}(m)=-2\sum_{y_i-m\ge 1}\ln(y_i-m)+\textrm{const}
$$
If there are no terms in the summation, i.e. if $y_n-1\le m\le y_1$, the function is constant for any choice of $m$ (on this interval).
If $y_n-1>y_1$, then some terms will be summed. For any fixed number of terms in the summation, the function is increasing, and the function is continuous at the “breakpoints”. Therefore the function is increasing, and attains maximum at $y_1$.
Putting these together, for the case $c=1$, a possible choice is $\hat{m}=y_1$. Note that the log likelihood function is guaranteed to be finite in this case.

If $c<1$, let $m \uparrow y_1$, then the first summand $\ln(y_1-m)$ is guaranteed to be included and grows without bound, while the other summands $\ln(y_i-m)$ are finite where $y_i>y_1$. The MLE is thus $\hat{m}=y_1$, with likelihood value $+\infty$. The estimator $\hat{m}$ is consistent.
Note, however, that if $c$ is unknown and to be estimated, then as $\hat{m} \uparrow y_1$, $\hat{c} =n\left(\sum |\ln (y_i-\hat{m})|\right)^{-1} \downarrow 0$ and the likelihood value tends to $+\infty$. The MLE for $c$ would be $\hat{c}=0$, which is inconsistent. This means $c$ cannot be estimated consistently by MLE if $m$ is unknown.

If $c>1$, consider first the case $m < y_1 - 1$. Then the first summation vanishes and all terms are in the second summation. The resulting log likelihood is increasing in m; this shows the maximizer $\hat{m} \ge y_1 - 1$.
Next, consider $m \uparrow y_1$. Then the first term tends to $-\infty$ while the other terms are finite; this shows $\hat{m} < y_1$.
Therefore the log likelihood is maximized somewhere in $[y_1 - 1, y_1)$, and may be located by a bracketed maximizer. I’m unable to show that there is a unique (local) maxima. If there are in fact more than one local minima, any one of them might be returned. This behavior is similar to that of numerical MLE.

For completeness, the moment estimator for shape and/or scale parameter, with known location parameter, is given below.
Let $\tilde{X}$ be log Laplace variable with shape $\sqrt{d}$, location 0 and scale 1, where $d > 4$. Its moments are
$$
\mathbb{E}[\tilde{X}] = \frac{d}{d - 1},
\mathbb{E}[\tilde{X}^2] = \frac{d}{d - 4},
\mathrm{Var}[\tilde{X}] = \frac{d(2d+1)}{(d-4)(d-1)^2}
$$
Let $X$ be a log Laplace variable with shape $\sqrt{d}$, location $L$, and scale $S$, where $d > 4$. Given a sample, denote the sample mean by $m_1$, and the (biased) sample variance by $m_2$.
If only one parameter is unknown, the MM equation is
$$
\mathbb{E}[X] = \frac{Sd}{d-1}+L=m_1
$$
it’s easy to solve any one unknown.


[FLINK-13003][table-planner-blink] Support Temporal TableFunction Join in blink planner

What is the purpose of the change
This is a feature port from flink-planner to blink-planner to support temporal TableFunction join (or called versioned joins).
Brief change log

Introduce some rules to convert a UDTF correlate to temporal table join.
Introduce TemporalProcessTimeJoinOperator and TemporalRowTimeJoinOperator for support temporal join in runtime.

Verifying this change
This change added tests and can be verified as follows:

Added integration tests TemporalUdtfJoinITCase
Added plan test TemporalUdtfJoinTest

The above cases are based on SQL, some TableAPI cases are not ported.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

Can you describe the rule process of LookupJoin and TemporalTableJoin separately?

Can you describe the rule process of LookupJoin and TemporalTableJoin separately?
And I strongly suggest you change class name of them, because them really make me confused...

FOR SYSTEM_TIME AS OF:

Logical: LogicalCorrelate with LogicalSnapshot
LogicalCorrelateToJoinFromTemporalTableRule
FlinkLogical: LogicalJoin with LogicalSnapshot
StreamExecLookupJoinRule
Physical: StreamExecLookupJoin

LATERAL udtf(left.proctime):

Logical: LogicalCorrelate with a TemporalTableFunction
LogicalCorrelateToJoinFromTemporalTableFunctionRule
FlinkLogical: LogicalJoin with a special join condition TEMPORAL_JOIN_CONDITION
StreamExecTemporalJoinRule
Physical: StreamExecTemporalJoin


Investigate/fix the mouse disappearing when viewing the equipment hideout area

Steps:
Make dev usec account
Go to hideout
click equipment area/enter
We likely need a live dump of a profile with the area crafted to figure this out.

Originally written by chomp

EXCEPTION: System.NullReferenceException: Object reference not set to an instance of an object
  at EFT.UI.EquipmentTab.Show (ItemContextAbstractClass equipmentContext, EFT.InventoryLogic.InventoryEquipment equipment, EFT.InventoryLogic.InventoryController inventoryController, EFT.SkillManager skills, InsuranceCompanyClass insurance, System.Boolean inRaid) [0x0004a] in <d5ca384bce9649a092f6409484aa53c4>:0
  at EFT.UI.HideoutMannequinEquipmentScreen.UpdateEquipment (EFT.InventoryLogic.InventoryEquipment inventoryEquipment) [0x0006a] in <d5ca384bce9649a092f6409484aa53c4>:0
  at EFT.UI.HideoutMannequinEquipmentScreen.UpdateAreaStashView (EFT.Hideout.AreaData areaData, EFT.InventoryLogic.CompoundItem stash) [0x0000c] in <d5ca384bce9649a092f6409484aa53c4>:0
  at UI.Hideout.BaseHideoutAreaTransferItemsScreen`2[TController,TScreen].Show (EFT.InventoryLogic.CompoundItem stashToShow, System.Int32 selectedTabIndex) [0x0004a] in <d5ca384bce9649a092f6409484aa53c4>:0
  at EFT.UI.HideoutMannequinEquipmentScreen.Show (EFT.Hideout.AreaData areaData, EFT.InventoryLogic.InventoryController inventoryController, EFT.HealthSystem.IHealthController healthController, ISession session) [0x000fa] in <d5ca384bce9649a092f6409484aa53c4>:0
  at EFT.UI.HideoutMannequinEquipmentScreen.Show (EFT.UI.HideoutMannequinEquipmentScreen+GClass3417 controller) [0x00007] in <d5ca384bce9649a092f6409484aa53c4>:0
  at UserInterfaceClass+GClass3377`2[T,U,V].DisplayScreen () [0x00000] in <d5ca384bce9649a092f6409484aa53c4>:0
  at UserInterfaceClass+GClass3377`2+Struct1085[T,U,V].MoveNext () [0x000b6] in <d5ca384bce9649a092f6409484aa53c4>:0

Originally written by chomp

Had a look at the client code, if i interpret the offsets correctly then it's getting a NullReferenceException at line 85 here:
Most likely it's not finding the correct slots for the hideout Inventory

Attachments

image.png (73 KiB)
image.png (86 KiB)

Originally written by TetrisGG

I have a suspicion the inventory 'stash' item we added needs additional properties:

Slots or something similar perhaps

Attachments

image.png (8 KiB)

Originally written by chomp

the stash we add needs a child, this is the mannequin

Attachments

image.png (21 KiB)

Originally written by chomp

https://github.com/sp-tarkov/server/commit/557dc55a33bcb99717c88c81c7075696d634d726

Originally written by chomp


HSEARCH-5122 Add compatibility with OpenSearch 2.13.0

https://hibernate.atlassian.net/browse/HSEARCH-5122

https://ci.hibernate.org/job/hibernate-search/job/PR-4091/3/


New libbi mode - 'export'

Changes in this PR:

added a new mode 'export', which forces the libbi to discharge at full speed to the grid
the chargefromgrid command now accepts 'enable' and 'disable' (true and false still works)
updated README to include/update the libbi section
updated the libbi status codes as I found the full list in the app source

Updated change list

added a new mode 'export', which forces the libbi to discharge at full speed to the grid
updated README to include/update the libbi section
updated the libbi status codes as I found the full list in the app source
added new command 'chargetarget' to the libbi module to set the charge from grid target

@mattburns would you mind merging this, so we could get all these changes into CJNE's pymyenergi?

awesome, thanks. And thanks for the nudge!


helmfiles key does not respect environments inside helmfiles

Operating system
Arch Linux
Helmfile Version
0.149.0
Helm Version
v3.10.2
Bug description
In top level helmfile.yaml, I have:
helmfiles:
   - path: a/helmfile.yaml
   - path: b/helmfile.yaml

and inside a/helmfile.yaml, I have:
environments:
  default:
    values:
    - dev.yaml
  prod:
    values:
    - prod.yaml

releases:
  - chart: .
    name: a
    namespace: "{{ .Values.namespace }}"
    createNamespace: true
    values:
      - {{ .Values.environment }}.yaml

now when I call:
helmfile diff/apply

everything works fine, however
helmfile diff/apply -e prod

causes
err: no releases found that matches specified selector() and environment(prod), in any helmfile

when I modify top level helmfile.yaml to have environments key:
environments:
  default:
    values:
    - namespace: dev
  prod:
    values:
    - namespace: prod

I am getting
in ./helmfile.yaml: in .helmfiles[0]: in bag/helmfile.yaml: error during helmfile.yaml.part.0 parsing: template: stringTemplate:16:26: executing "stringTemplate" at <.Values.namespace>: map has no entry for key "namespace

regardless whether top level helmfile.yaml has namespace or not

@dolohow Hey! It's because your top-level helmfile.yaml does not have the specified environment defined.

And see also https://github.com/helmfile/helmfile/blob/main/docs/proposals/towards-1.0.md#the-changes-in-10. We're going to forbid defining environments and releases within the same part, and enforce the use of ---.

Ok, it does work, however it's not documented therefore I added this issue.
Thanks a lot.  Otherwise I would need to have like two top level helmfiles.yaml which would be suboptimal

@dolohow Thanks for confirming!
What kind of documentation could have helped you?
We do feature the separator in notable places like

https://github.com/helmfile/helmfile/blob/490bb5d1474fa3a79bcffabf2d8984a77e3862cc/docs/index.md#environment
https://github.com/helmfile/helmfile/blob/490bb5d1474fa3a79bcffabf2d8984a77e3862cc/docs/advanced-features.md#lockfile-per-environment

and probably several other examples that cover environments.

Yeah, right. Somehow I did not notice this when reading.  I was under impression that it's not needed, and also I mainly relied on this giant yaml that serves as reference and this one does not use --- separator.


Fixed bigint insertion bug in x86

This change is

Coverage remained the same at 74.697% when pulling 13e69b54bc1ec4dbcc6da4c24a4be0b39abf0fe0 on david-puglielli:bigint-x86-test-fix into e0f9afae2f92cabe8670204a6261ee0cadc44a2a on Microsoft:dev.

Codecov Report

Merging #777 into dev will increase coverage by 0.06%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##              dev     #777      +/-   ##
==========================================
+ Coverage   79.99%   80.06%   +0.06%
==========================================
  Files          25       25
  Lines        7323     7323
==========================================
+ Hits         5858     5863       +5
+ Misses       1465     1460       -5

Impacted Files
Coverage Δ

...php-7.1.17-src/ext/pdo_sqlsrv/shared/core_conn.cpp
87.06% <0%> (+1.43%)
:arrow_up:

Continue to review full report at Codecov.

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


Updating Modules and Activities Response

This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[x] The title of the PR is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[x] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[ ] If applicable, the PR references the bug/issue that it fixes.
[x] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[x] I have read the contribution guidelines.
[x] My spec meets the review criteria:

[x] The spec conforms to the Swagger 2.0 specification.
[x] The spec follows the guidelines described in the Swagger checklist document.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

@vrdmr Please take a look.

Automation for azure-libraries-for-java

Automation for azure-sdk-for-node

Automation for azure-sdk-for-python

Automation for azure-sdk-for-go

Are these changes deployed to the server and ready for me to merge the swagger?

gentle ping, @avkaur

I'm going to close this PR as stale. Feel free to comment on this thread again and I'll open it right up, or open a new PR.

Hi @marstr,
The changes are now deployed. Could you please reopen the PR and merge the changes.
Thanks,
Varad
@avkaur

@vrdmr,
It looks like there are some conflicts. Please fix them before I merge.

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-java

Closing as per @vrdmr's recommendation above.


WIP: Develop

Pull Request Test Coverage Report for Build 772

16 of 36 (44.44%)  changed or added relevant lines in 8 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.1%) to 75.643%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/data_access/dataAccess.service.ts
0
1
0.0%

src/data_access/providers/fileAccess.ts
0
1
0.0%

src/rules/rules.service.ts
1
2
50.0%

src/app.controller.ts
2
6
33.33%

src/data_access/providers/databaseAccess.ts
0
5
0.0%

src/remote-config/utils.ts
7
15
46.67%

Files with Coverage Reduction
New Missed Lines
%

src/remote-config/utils.ts
1
77.22%

Totals

Change from base Build 720:
-0.1%

Covered Lines:
1287

Relevant Lines:
1686

💛  - Coveralls


Integrate new translated content

This issue is for tracking the work to integrate the translations of the site provided by IBM.

The new translations are on this branch: https://github.com/strongloop/expressjs.com/tree/ibm-translations.
There are some small issues that I'd like to resolve before we merge this.
@hacksparrow Let's discuss this next week.

Open comments from Sue:

Menu bar
In some languages the menu bar is very crowded (see Germany). We had some comments raised as suggestions - surprisingly not defects. You might want to try and address this somehow. Maybe insert some
 ?

In some languages the drop downs from the menu bar are too narrow and cause strings to be wrapped multiple times (Korean Guide menu). You might want to increase the width of these drop downs, if possible?

There was a comment about the text in the top right of the menu bar being too small / almost unreadable. You might increase?

There was a comment about the text in the top right of the menu bar being too small / almost unreadable. You might increase?

Fixed in 8554ccfed1fc6d545ca34fad569591d65a3e12b1

👍

The only menu issue I see is in German (ko looks fine):

Might be nice to fix this, but not critical.

Here are a couple more minor display issues.
1. Extra padding at the bottom of code samples on localized pages

Compare with the same page in English:

This may be because the translated pages have:

...

While the en pages have

...

If there's any way to fix this via CSS without changing this in ALL the returned translated pages, we should do so.
2. API ref. code blocks don't have syntax highlighting
This occurs in all langs, including en; for example:

This is probably because we replaced ~~~js with the equivalent HTML tags everywhere except the API doc.  However, it's not happening on the live site, for which the same is true.   But again, if we can fix with CSS, that would be best solution.

Merged the PR.  I will open a new issue for the unresolved layout issues above.


Add dayAriaLabelFormat into more places

This PR is a follow-up for #842.
I've added dayAriaLabelFormat prop into DayPickerRangeController and DayPickerSingleDateController to pass the prop to DayPicker.

Coverage remained the same at 86.571% when pulling 6ff22f2dfa5e6ce777b9eb234d4415848eb98cbd on koba04:add-dayarialabel-into-daypicker-controller into b2a48746c9ea94408241e4244c408f0ea99d8d75 on airbnb:master.

Thanks!


remove more stuff

Coverage remained the same at 100.0% when pulling 372f8dced5f9bd313567d37f48cf25159cc007ff on clean-up into c9e5557d28af8d3adf2e71b0e0e09b2b924556df on main.


Failed to install 'phonegap-plugin-push':undefined undefined

Expected Behaviour
iOS builds on PGB with phonegap-plugin-push version 2.0.0-rc5 installed and using the latest cli: 7.0.1
Actual Behaviour
Builds fails with the following log.
https://pastebin.com/raw/fSHXJXFY
Reproduce Scenario (including but not limited to)
Set the cli to the newest 7.0.1 in config.xml:
<preference name='phonegap-version' value='cli-7.0.1' />
Install phonegap-plugin-push with the following line in config.xml:
<plugin name="phonegap-plugin-push" spec="~2.0.0-rc5">
        <variable name="SENDER_ID" value="***" />
</plugin>

Steps to Reproduce
Zip the project folder and upload it to PGB.
Platform and Version (eg. Android 5.0 or iOS 9.2.1)
N/A
(Android) What device vendor (e.g. Samsung, HTC, Sony...)
N/A
Cordova CLI version and cordova platform version
cli-7.0.1 (4.4.0 / 6.2.3 / 5.0.0)
Plugin version
2.0.0-rc5
Sample Push Data Payload
Sample Code that illustrates the problem
Logs taken while reproducing problem
https://pastebin.com/raw/fSHXJXFY

@edoreld you don't provide the SENDER_ID when using 2.x or greater. You need to include your Google Services file. https://github.com/phonegap/phonegap-plugin-push/blob/master/docs/INSTALLATION.md#installation-requirements
Also, the PGB team has asked for people with issues regarding phonegap build please start an issue on the support forum: https://forums.adobe.com/community/phonegap/build
Please include an app id of an app that can replicate this issue.

To install push you must first install cocoapods. Follow these steps on your terminal in the Ionic project directory.
First remove what you tried to install
ionic plugin remove phonegap-plugin-push
Next install cocoapods
sudo gem install cocoapods
Then you need to sync the cocoapods repo
pod setup
This may take a while
Finally install push
ionic plugin add phonegap-plugin-push --variable SENDER_ID="XXXXXXX"


cephfs: add file open/close and read/write/seek and unlink

Add cephfs functions related to regular files including:

open
close
read
write
seek

Additionally, we add unlink to the paths.go file so that the files tests can clean up after themselves.
Checklist

[x] Added tests for features and functional changes
[x] Public functions and types are documented
[x] Standard formatting is applied to Go code

@nixpanic I think my most recent patches should cover the cases that concerned you. PTAL, thanks.

@ansiwen any additional feedback/discussion?
@nixpanic any thoughts on the current version?


Rpg dialogue exercise

@BrooklinJazz

Looks great! Feel free to merge this so your other PRs include only new changes.


Cannot start dev server with error (node:21264) UnhandledPromiseRejectionWarning: Error

Describe the bug
Error occured while running yarn dev command
Reproduction
$ yarn create vite-app <project-name>
$ cd <project-name>
$ yarn
$ yarn dev
yarn run v1.22.4
warning package.json: No license field
$ vite
vite v1.0.0-rc.3
(node:21264) UnhandledPromiseRejectionWarning: Error: Package exports for 'F:\project\demo\vitedemo\node_modules\koa' do not define a valid '.' target
    at resolveExportsTarget (internal/modules/cjs/loader.js:545:13)
    at applyExports (internal/modules/cjs/loader.js:459:14)
    at resolveExports (internal/modules/cjs/loader.js:508:12)
    at Function.Module._findPath (internal/modules/cjs/loader.js:577:20)
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:879:27)
    at Function.Module._load (internal/modules/cjs/loader.js:785:27)
    at Module.require (internal/modules/cjs/loader.js:956:19)
    at require (internal/modules/cjs/helpers.js:74:18)
    at Object.<anonymous> (F:\project\demo\vitedemo\node_modules\vite\dist\node\server\index.js:9:31)
    at Module._compile (internal/modules/cjs/loader.js:1063:30)
(node:21264) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:21264) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.Done in 0.28s.

System Info

required vite version: v1.0.0-rc.3
required Operating System: Windows 10 (1909)
required Node version: v13.1.0
Optional:

yarn version 1.22.4
Installed vue version (from yarn.lock) 3.0.0-rc.4
Installed @vue/compiler-sfc version 3.0.0-rc.4

Logs
F:\project\demo>yarn create vite-app vitedemo
yarn create v1.22.4
[1/4] Resolving packages...
[2/4] Fetching packages...
[3/4] Linking dependencies...
[4/4] Building fresh packages...
success Installed<EMAIL_ADDRESS>with binaries:
      - create-vite-app
      - cva
[########] 8/8Scaffolding project in F:\project\demo\vitedemo...

Done. Now run:

  cd vitedemo
  npm install (or `yarn`)
  npm run dev (or `yarn dev`)

Done in 0.73s.

F:\project\demo>cd vitedemo

F:\project\demo\vitedemo>yarn
warning package.json: No license field
info No lockfile found.
warning<EMAIL_ADDRESS>No license field
[1/4] Resolving packages...
[2/4] Fetching packages...
info<EMAIL_ADDRESS>The platform "win32" is incompatible with this module.
info<EMAIL_ADDRESS>is an optional dependency and failed compatibility check. Excluding it from installation.
[3/4] Linking dependencies...
[4/4] Building fresh packages...
success Saved lockfile.
Done in 32.17s.

F:\project\demo\vitedemo>yarn dev
yarn run v1.22.4
warning package.json: No license field
$ vite
vite v1.0.0-rc.3
(node:21264) UnhandledPromiseRejectionWarning: Error: Package exports for 'F:\project\demo\vitedemo\node_modules\koa' do not define a valid '.' target
    at resolveExportsTarget (internal/modules/cjs/loader.js:545:13)
    at applyExports (internal/modules/cjs/loader.js:459:14)
    at resolveExports (internal/modules/cjs/loader.js:508:12)
    at Function.Module._findPath (internal/modules/cjs/loader.js:577:20)
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:879:27)
    at Function.Module._load (internal/modules/cjs/loader.js:785:27)
    at Module.require (internal/modules/cjs/loader.js:956:19)
    at require (internal/modules/cjs/helpers.js:74:18)
    at Object.<anonymous> (F:\project\demo\vitedemo\node_modules\vite\dist\node\server\index.js:9:31)
    at Module._compile (internal/modules/cjs/loader.js:1063:30)
(node:21264) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:21264) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.Done in 0.28s.

Related with https://github.com/nodejs/modules/issues/446.You can upgrade your node.js versions >= 13.2.0


Add a "comment" field in the submission form.

It is already in the database scheme, just add the field in the form and show it on the score graph.

(and make it clear that it's a public comment, please)


fix(docs): remove invalid character from id.

When I try to click on any element with the dolar sign I get the following error:

This is happening because the id attribute has an invalid character ($).
https://pinia.vuejs.org/api/interfaces/pinia._StoreWithState.html

@posva ohh, should have taken more time to see it carefully. Can I still make the changes or is better to open an issue?

Make the changes but don’t open an issue about this

Doesn't packages/docs/typedoc-markdown.js generate the id's that are failing?

Oh sorry, I didn't see on the phone. I'm starting to think this should actually be handled by vitepress 🤔 It should probably use document.getElementById() instead to avoid this specific problem. Vue Router does a similar thing

@posva  I'm not that good to understand your solution, but I think $ is not a valid character for an id. When I link you https://pinia.vuejs.org/api/interfaces/pinia._StoreWithState.html#Methods-$patch you can see that the page don't scroll down to $patch section.
So using document.getElementById() would fix the click, but not when opening the link.

oa


[webview_flutter] fixing crash select tag

flutter/flutter#25767

Thanks again for the contribution. It looks like this PR has been sitting for a little while since the last round of feedback, do you still have time to work on it?

Hi @mklim , this PR fixes the issue that an app is crashing as soon as you click on a select box on a web page.
I've setup a minimal project to demo this crash.
https://github.com/tlueder/flutter_show_webview_crash
As soon as you click on the select box the app crashes with:
W/System.err( 6524): java.lang.ClassCastException: $Proxy0 cannot be cast to android.view.WindowManagerImpl
W/System.err( 6524):    at android.view.Window.setWindowManager(Window.java:778)
W/System.err( 6524):    at android.view.Window.setWindowManager(Window.java:760)
W/System.err( 6524):    at android.app.Dialog.(Dialog.java:207)
W/System.err( 6524):    at android.app.AlertDialog.(AlertDialog.java:204)
W/System.err( 6524):    at android.app.AlertDialog$Builder.create(AlertDialog.java:1105)
W/System.err( 6524):    at JB.(PG:6)
W/System.err( 6524):    at org.chromium.content.browser.input.SelectPopup.show(PG:41)
W/System.err( 6524):    at android.os.MessageQueue.nativePollOnce(Native Method)
W/System.err( 6524):    at android.os.MessageQueue.next(MessageQueue.java:336)
W/System.err( 6524):    at android.os.Looper.loop(Looper.java:174)
W/System.err( 6524):    at android.app.ActivityThread.main(ActivityThread.java:7356)
W/System.err( 6524):    at java.lang.reflect.Method.invoke(Native Method)
W/System.err( 6524):    at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:492)
W/System.err( 6524):    at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:930)
F/chromium( 6524): [FATAL:jni_android.cc(249)] Please include Java exception stack in crash report
F/libc    ( 6524): Fatal signal 5 (SIGTRAP), code 128 (SI_KERNEL), fault addr 0x0 in tid 6524 (w_webview_crash), pid 6524 (w_webview_crash)

Build fingerprint: 'google/sdk_gphone_x86/generic_x86:10/QSR1.191030.002/5978551:userdebug/dev-keys'
Revision: '0'
ABI: 'x86'
Timestamp: 2020-01-07 11:33:30+0800
pid: 6524, tid: 6524, name: w_webview_crash  >>> de.nets.flutter_show_webview_crash <<<
uid: 10133
signal 5 (SIGTRAP), code 128 (SI_KERNEL), fault addr 0x0
Abort message: '[FATAL:jni_android.cc(249)] Please include Java exception stack in crash report
'
eax 00000000  ebx cae07864  ecx e9d7f700  edx 00000400
edi e6227980  esi ff9179c0
ebp e9d7f700  esp ff917550  eip c83eec30
backtrace:
#00 pc 01cfdc30  /system/product/app/TrichromeLibrary/TrichromeLibrary.apk!libmonochrome.so (offset 0x66c000) (BuildId: ffe50d4377c82928759c1f40f1b35ba9f411454a)
Lost connection to device.
Hope this helps.

This PR is related to this issue:
https://github.com/flutter/flutter/issues/25767

+1 this issue has been open for over a year: https://github.com/flutter/flutter/issues/34248

The issue seems to be fixed in various PRs opened by @amirh
The fix should be available on master. I'm closing this PR now. @ametsuramet if you found this issue not fixed, please feel empowered to tag me and reopen the PR

Latest version of webview_flutter 0.3.22 still crashes when selecting a dropdown. Was this fix pushed to flutter itself, or the package? If flutter, which version resolves this issue

Hi @cyanglaz , I believe this is a mixup with another issue.
https://github.com/flutter/flutter/issues/54164
which was fixed by @amirh in PR
https://github.com/flutter/plugins/pull/2663
As far as I know there was no patch to flutter or the engine to fix this.
This issue is still alive.
I've created a sample app to show this in action
https://github.com/tlueder/flutter_show_webview_crash
For this issue we have multiple PRs
https://github.com/flutter/plugins/pull/2535
https://github.com/flutter/plugins/pull/2330
https://github.com/flutter/plugins/pull/2081
https://github.com/flutter/plugins/pull/2071
https://github.com/flutter/plugins/pull/1861
Please correct me when i'm wrong and missed something.
Thanks

@tlueder was there an issue created for this crash?

@cyanglaz  This PR and all the other I linked are related to this issue:
https://github.com/flutter/flutter/issues/25767

I'm going to close the PR because the CLA is not signed. Feel free to ping me when the CLA is signed and I'll re-open the PR. Or you can open a new PR after signing the CLA.

I'm going to close the PR because the CLA is not signed. Feel free to ping me when the CLA is signed and I'll re-open the PR. Or you can open a new PR after signing the CLA.


added support for "path" cookie parameter

Hi, thanks, there is a bug as session kill still clear on /. A test would be welcome :)

I think you could use that as a model https://github.com/encode/starlette/pull/1147 regards

superseded by c2bd54634a62c03f81a3eb9dabf8845939476eb7 , thank you!


Add a more detailed README.md

Content
I'm opening this PR early for any additional conversations or ideas related to the README.md file.
Follow-up suggestions
A quick list of follow-up suggestions, will be updated as the conversation continues (if it doesn't get updated then mention me and/or read through the conversation).

We could also consider adding a CONTRIBUTORS.md file for any new contributors.

This is good, thanks!

This is good, thanks!

Eh? There's a big TODO block in the README tho... You're welcome though! I'll work more on it later, have to do some errands for around 2 hours. In the meantime if you have any suggestions feel free to share them!

All that really needs to be done is adding the contributors, but it really depends on what you'd define as a "contributor" to the mod - do you mean contributors to the repository or contributors to the modpack as a whole?

I'd say contributers as a whole but order them first as contributers to the repo and then just general contributors (or maybe split them as maintainers/contributors for more clarity).

Agreed.

Alright, adding the CONTRIBUTORS.md file should be in a separate PR so I'll leave this PR like this. If you have any suggestions or corrections feel free to do them and/or say them here. I'll make sure to look into it!


Correction of B1

This pull request corrects the calibration for B1, improvements are still needed, but results are still satisfactory.
The current results of B1 are as follows:

Qubit
Readout Fidelity
Assignment Fidelity
T1 (µs)
T2 (µs)
Gate infidelity (e-3)

B1
0.919
0.96
24.9 ± 0.2
2.9 ± 0.1
1.46 ± 0.09

Report Links:
1- Qubit_Spectroscopy: http://login.qrccluster.com:9000/Qz2x1ECAS7CWwV10ffbd6Q==/
2- Rabi: http://login.qrccluster.com:9000/0JqxiS8yQ6WDE8rz6Jmv7g==/
3- Classification: http://login.qrccluster.com:9000/JVYePaRXQB2PVXwO5LpsiQ==/
4- Ramsey_T1: http://login.qrccluster.com:9000/Sy_6dodRQh6B1cy1VhiluA==/
5- T2: http://login.qrccluster.com:9000/xHWgX8RuQLeAgLhIK-0SGA==/
6- rb: http://login.qrccluster.com:9000/7djyzA6CRi-86iAwFXFznw==/

The current results of B1 are as follows:

Qubit
Readout Fidelity
Assignment Fidelity
T1 (µs)
T2 (µs)
Gate infidelity (e-3)

0.919
0.96
24.9 ± 0.2
2.9 ± 0.1
1.46 ± 0.09

Report Links:
1-Qubit_Spectroscopy: http://login.qrccluster.com:9000/Qz2x1ECAS7CWwV10ffbd6Q==/
2-Rabi: http://login.qrccluster.com:9000/0JqxiS8yQ6WDE8rz6Jmv7g==/
3-Classification: http://login.qrccluster.com:9000/JVYePaRXQB2PVXwO5LpsiQ==/
4- Ramsey_T1: http://login.qrccluster.com:9000/Sy_6dodRQh6B1cy1VhiluA==/
5- T2: http://login.qrccluster.com:9000/xHWgX8RuQLeAgLhIK-0SGA==/

Given #191 is this PR still relevant? @HishamKazim @Edoardo-Pedicillo

@alecandido No, this pull request will be closed.


[Broken] Pornhub playlist only download first 36 videos

Checklist

[ ✅] I'm reporting a broken site support
[ ✅] I've verified that I'm running yt-dlp version 2021.08.10
[ ✅] I've checked that all provided URLs are alive and playable in a browser
[ ✅] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[ ✅] I've searched the bugtracker for similar issues including closed ones

Verbose log

yt-dlp -v https://www.pornhub.com/playlist/95816161
[debug] Command-line config: ['-v', 'https://www.pornhub.com/playlist/95816161']
[debug] Encodings: locale cp1252, fs utf-8, out utf-8, pref cp1252
[debug] yt-dlp version 2021.08.10 (exe)
[debug] Python version 3.8.10 (CPython 64bit) - Windows-10-10.0.19042-SP0
[debug] exe versions: ffmpeg N-102564-g2261cc6d8a, ffprobe N-102564-g2261cc6d8a
[debug] Optional libraries: mutagen, pycryptodome, sqlite, websockets
[debug] Proxy map: {}
[debug] [PornHubPagedVideoList] Extracting URL: https://www.pornhub.com/playlist/95816161
[download] Downloading playlist: playlist/95816161
[PornHubPagedVideoList] playlist/95816161: Downloading page 1
[PornHubPagedVideoList] playlist playlist/95816161: Downloading 36 videos
[download] Downloading video 1 of 36
[debug] [PornHub] Extracting URL: http://www.pornhub.com/view_video.php?viewkey=ph5c0a6ab9a71b1&pkey=95816161
[PornHub] ph5c0a6ab9a71b1: Downloading pc webpage
[PornHub] ph5c0a6ab9a71b1: Downloading JSON metadata
[PornHub] ph5c0a6ab9a71b1: Downloading m3u8 information
[PornHub] ph5c0a6ab9a71b1: Downloading m3u8 information
[PornHub] ph5c0a6ab9a71b1: Downloading m3u8 information
[PornHub] ph5c0a6ab9a71b1: Downloading m3u8 information
[PornHub] ph5c0a6ab9a71b1: Downloading m3u8 information

Description
yt-dlp scrap only the first 36 videos from pornhub playlists, this issue on the youtube-dl as well, let's hope someone fix it on this fork.
Thanks

Isn't 1 video enough? haha JK

1 video Isn't enough? haha JK

Lots of videos and channels goes down everyday so it's better to backup them for educational purposes 😜

@snwefly On a different Note, do you have made experiences with ytdlp and 1080p downloads from PornHub? I always used yt-dl and in the past 6 months YouTube-dl often downloaded only the 720p version of videos that were available in 1080p on PornHub. I had to first use -F and find the specific version which I then had to use with the -f flag, something like: -f hls-5321-1080p   But for another video it then would be something like hls-8913-1080p

To add to my original comment: I am not a programmer and can’t code but i assume this particular ytdlp commit fixed the 720p/1080p resolution issue with the PornHub extractor? https://github.com/yt-dlp/yt-dlp/blob/master/yt_dlp/extractor/pornhub.py


add tests for extraGas on relays

Codecov Report

Merging #148 (679f34f) into develop (0895588) will increase coverage by 17.79%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##           develop     #148       +/-   ##
============================================
+ Coverage    71.73%   89.52%   +17.79%
============================================
  Files           68       33       -35
  Lines         2257     1079     -1178
  Branches       335      150      -185
============================================
- Hits          1619      966      -653
+ Misses         638      113      -525

Flag
Coverage Δ

batch-submitter
?

contracts
89.52% <ø> (ø)

core-utils
?

data-transport-layer
?

message-relayer
?

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

Impacted Files
Coverage Δ

packages/data-transport-layer/src/utils/eth-tx.ts

...ckages/data-transport-layer/src/utils/constants.ts

...h-submitter/src/batch-submitter/batch-submitter.ts

packages/core-utils/src/common/index.ts

...ubmitter/src/batch-submitter/tx-batch-submitter.ts

...ices/l1-ingestion/handlers/transaction-enqueued.ts

packages/core-utils/src/alias.ts

packages/core-utils/src/batches.ts

.../src/services/l2-ingestion/handlers/transaction.ts

packages/core-utils/src/index.ts

... and 25 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 0895588...679f34f. Read the comment docs.


Is it Possible to load all tab pages at start?

Like the older version of this?

prerenderingSiblingsNumber={Infinity} should work


Transuranus-Sciantix coupling

Dear all,
as Transuranus users , we noticed that in the v1m2j23 version of TU there are some parts referring to a coupling between TU and Sciantix.
However, in the official version of Sciantix there are no references (either in the code or in the manual) to this coupling.
Furthermore, if we create a static library from the official version of Sciantix, it cannot be linked to Transuranus.
We guess that the code needs some modifications to make this functionality possible, so what are your plans?
Will a future release allow this coupling?
Thanks in advance

In the 019_coupling_TU branch, we have extended the code so that it can be coupled with the v1m1j24 version of the TRANSURANUS software package distributed by JRC-EC Karlsruhe.


Deprecate the ruff alias to ruff check

We should display a warning when this legacy interface is used and, eventually, remove the alias.

Let's deprecate this as part of v0.3.0 (and remove in v0.4.0).


How to use JSON strings instead of JSON locations when checking

Please provide examples as soon as possible. Thank you.

Hello @tygithub1.
Do you mean you would like to use JSON pointers instead of JSON locations when checking the reported problems in your program?

I am sorry but I cannot understand your question. What do you mean by JSON strings in your question? Please provide me with more details about what you really want to achieve.
Thank you.

Hello big man
I am a rookie ,Maybe my description is not clear,now ,I want to validate a network resource that may be in JSON format. I don't think I should write this JSON data into the file every time, but should process it through the corresponding JSON api. So I think you should provide an interface to validate whether a string in JSON format conforms to my definition of JSON schema. Um.Refer to this address
Https://github.com/java-json-tools/json-schema-validator,
thanks!

Hello @tygithub1.
It's easy to achive.
Please see the following code sample shown in README.md with small modification.
JsonValidationService service = JsonValidationService.newInstance();

// Reads the JSON schema from file
JsonSchema schema = service.readSchema(Paths.get("your.schema.json"));

// Problem handler which will print problems found.
ProblemHandler handler = service.createProblemPrinter(System.out::println);

String json = " ... your JSON to be validated here ... ";
// Reads the JSON instance from the string above
try (JsonReader reader = service.createReader(new StringReader(json), schema, handler)) {
    JsonValue value = reader.readValue();
    // Do something useful here
}

Your can use StringReader as a reader of JSON instance in a string. The instance of StringReader can be passed to JsonValidationService#createReader() or JsonValidationService#createParser()as the first parameter.
Thank you.

Wow, I just tried it. Very good. Thank you very much.


Logic App Connector - No support for uint256[] and string[] in ABI

This issue is for a: (mark with an x)
- [ ] bug report -> please search issues before submitting
- [x] feature request
- [ ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

Use the Ethereum Connector, and under the abi put in a parameter with uint256[] data type such as below:

"abi": "[\n{\n    "constant": false,\n    "inputs": [\n      { "internalType": "address", "name": "to", "type": "address" },\n      { "internalType": "uint256[]", "name": "tokenIds", "type": "uint256[]" },\n      { "internalType": "uint256[]", "name": "amounts", "type": "uint256[]" },\n      { "internalType": "string[]", "name": "metaDataList", "type": "string[]" }\n    ],\n    "name": "batchMintUniqueTokenTo",\n    "outputs": [],\n    "payable": false,\n    "stateMutability": "nonpayable",\n    "type": "function"\n  }\n]",
Any log messages given by the failure

Error executing the api '/contractSchema/functions/batchMintUniqueTokenTo/metadata'
{
"status": 400,
"message": "The datatype 'uint256[]' is not supported\r\nclientRequestId: d3e4a0c7-0906-4ac6-9ea2-1acb2b5e8d0b",
"source": "blockchainethereum-ae.azconn-ae.p.azurewebsites.net"
}

Expected/desired behavior

Can pass in an array as an input parameter / argument

OS and Version?

Windows 10

This is a known limitation of the existing version of the connector and support for the feature will be considered in future versions.


Keyboard cursor resets to offset 0 during wrong command

For example,
When i type "add", which is a wrong command,
The cursor will reset to the starting position e.g. "|add" instead of "add|"

The entire incorrect command will be selected. User can choose to delete it or the reach the end position by pressing the right arrow key


Ministro

Description of Change
Characteristic notification support on UWP
Issues Resolved

API Changes

None
Platforms Affected

UWP

Behavioral Changes

None
Testing Procedure

PR Checklist

[x] Rebased on top of the target branch at time of PR
[x] Changes adhere to coding standard

@dministro I can't take these changes against master.  I also can't change the way notifications work as there is a lot of special cases to deal with disconnecting on UWP.  I will look into uncommenting this on the dev branch though


Getting list of connected users in chat

How would I be able to get the list of users in chat? I know this may be outside of the scope of the module but if there are any tips on the best way to accomplish this it would be greatly appreciated!

moved to the forum


Mingw/5.2.3

initial support of MinGW build
https://github.com/bincrafters/community/issues/42

outdated and no longer needed (commits are in testing/stable already btw)


V0.12.1.x govobj sync

disable fCached values
use two maps for storing votes, by hash and parent-hash/type
disable part of flatdb.dump (still overwriting)
fixed govobj/votes relay and sync

some nits but utACK

ACK squash merge 732a8a3 :)
@schinzelh

@UdjinM6 @schinzelh : I noted "squash merge" in a couple of PRs recently. Does this mean I don't have to squash all my commits into one myself anymore?

@crowning- Yep, github introduced this wonderfull "squash and merge" option


Fix description of Observable example code.

The descriptions for 2 of the references were switched.

Merged, thanks! Nice catch.


DotProgressBar again becomes visible when fragment or activity resumes

I am experiencing the same thing.  I am in activity A. I set the visibility of the DotProgressBar to Visibility.GONE before going to activity B. Then when i click back to go back to activity A, in onResume() i event tried setting the visibility of the DotProgressBar to Visibility.GONE.  But some how the visibility toggles to visible after that point.

I solved this issue by removimg visibilityListener.
On 2 Dec 2016 00:14, "Etienne Lawlor"<EMAIL_ADDRESS>wrote:

I am experiencing the same thing. I am in activity A. I set the
visibility of the DotProgressBar to Visibility.GONE before going to
activity B. Then when i click back to go back to activity A, in onResume()
i event tried setting the visibility of the DotProgressBar to
Visibility.GONE. But some how the visibility toggles to visible after that
point.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Is the issue here
@Override
protected void onAttachedToWindow() {
super.onAttachedToWindow();
startAnimation();
}

@silvestrpredko  Is the issue here
  @Override
  protected void onAttachedToWindow() {
    super.onAttachedToWindow();
    startAnimation();
  }

@lawloretienne Hey, I will fix it).

@silvestrpredko I'm looking forward to the fix

@silvestrpredko Any updates on this?

 @Override     protected void onVisibilityChanged(@NonNull View changedView, int visibility) {         super.onVisibilityChanged(changedView, visibility);         /*if (visibility == GONE || visibility == INVISIBLE) {             stopAnimation();         } else {             startAnimation();         }*/     }
Just comment these lines. It will work fine.

@lawloretienne @embydextrous fixed, please check.


about myself

Hello Shriramu2002!
Sorry for the late review and great job on finishing Session 1's assignment :partying_face:

Hello Shriramu2002!
Sorry for the late review and great job on finishing Session 1's assignment :partying_face:


Fixing wording used for "VS Code"

As far i can see the convention for the naming should be "VS Code" instead of just "Code", doesn't it?

Thanks for the PR, but in this context we use Code intentionally to refer to the Code OSS project.


Clock and issue

I am seeing a periodical issue with the clock, it is the 3rd time that the clock losses an hour but minutes are accurate.
I set time from App loader , for many hour the time is correct but at certain time time is an hour less.

Have you set the time zone in settings? In the UK, it needs to be set to 1 for British Summer Time.

I haven't set, but it has the value 1.
In any event, the same behavior has happen in different days, so I can't find relation with summer time.
I will try to identify if the behavior happens always at the same time or after same number of hours

Partial good new, the clock has been working fine for several days BUT I don't know what is causing a different behaviour.
Because before ALWAYS after a day it had lost an hour.


Feature/skip deleted

Add harvest behavior objects to harvester. This makes it possible to introduce new record processing behavior by injecting them into the harvesters.
Create skip deleted behavior for OAI, make it the default behavior for OAI harvesters.

@no-reply and I have discussed this, and I think it looks good to merge.
I am about to make some compatible changes related to indexing and mapping activities, and will have a PR ready soon.
We might change where the behavior classes are located, but we're going to defer doing so until both of our branches are merged.

I'm :+1: on this; just curious if we're planning to retain the commented method. Fix up and :squash: as appropriate.

I can fix up and squash once I'm in the air.

:airplane: :jack_o_lantern:

I think this is ready. :rocket:

:100:


Dependency failure hydrolysis

Due to project rename of Hydrolysis to Polymer-Analyzer the dependency path has to be changed to:
"hydrolysis": "git://github.com/Polymer/polymer-analyzer#v1.22.0",
With the newest version (1.24.1) the generator does not work.

I have almost ported the generator to use polymer-analyzer, unfortunatelly the java output is not alright because the lack of type info in annotations.
I'll postpone this until https://github.com/Polymer/polymer-analyzer/issues/305 is resolved.


Refactors settings::RuleStrategy.rule as Rule enum

Description

Fix https://github.com/kubewarden/user-group-psp-policy/issues/16

Test

To test this pull request, you can run the following commands:
make test
All test should pass

Additional Information
Tradeoff

Potential improvement

After consideration, I think that doing the checks via the deserializer makes everything needlessly complicated. The card did indeed ask to try to do it via de Deserializer, which I know think it's a wrong approach. I'm sorry for that.
I would be happy just having
pub(crate) struct RuleStrategy {
    pub rule: Rule,
    pub ranges: Vec<IDRange>,
    pub overwrite: bool,
}

Where Rule is an enum with variants MustRunAs, RunAsAny, MustRunAsNonRoot, MayRunAs (with the correct serde attributes for it to deserialize correctly). And having the validation of the settings being performed on runtime at validate().

tagged in main as v0.4.8, will be released shortly.


Mods used in Scenarios/Themes should take priority

It's possible to choose a theme (say, Cobblemon), and then that mod gets dropped during the pruning phase.

Fixed in 381b4f6c6c21766087a6c758f85baa607de8a6fa

Reopened because the solution should take EVERYTHING into account, not just the scenario.


Refactor the code related to loss computation

Fixes #854
Proposed Changes

create a common interface for the loss functions
refactor the codebase to use new loss implementations

Checklist

[ ] CONTRIBUTING guide has been followed.
[ ] PR is based on the current GaNDLF master .
[ ] Non-breaking change (does not break existing functionality): provide as many details as possible for any breaking change.
[ ] Function/class source code documentation added/updated (ensure typing is used to provide type hints, including and not limited to using Optional if a variable has a pre-defined value).
[ ] Code has been blacked for style consistency and linting.
[ ] If applicable, version information has been updated in GANDLF/version.py.
[ ] If adding a git submodule, add to list of exceptions for black styling in pyproject.toml file.
[ ] Usage documentation has been updated, if appropriate.
[ ] Tests added or modified to cover the changes; if coverage is reduced, please give explanation.
[ ] If customized dependency installation is required (i.e., a separate pip install step is needed for PR to be functional), please ensure it is reflected in all the files that control the CI, namely: python-test.yml, and all docker files [1,2,3].
[ ] The logging library is being used and no print statements are left.

@sarthakpati What is your opinion on the implementation of interface and example loss? When we agree on that I will proceed to port the rest.

Should this PR not be going into https://github.com/szmazurek/GaNDLF/tree/dev ?

Should this PR not be going into https://github.com/szmazurek/GaNDLF/tree/dev ?

Could be, but as it is directly related to existing core code I tought that we can update directly here.

This commit was done without proper git initialization and hence is getting flagged by the CLA bot. Can you rebase this?

This commit was done without proper git initialization and hence is getting flagged by the CLA bot. Can you rebase this?

Done


Possible redundant files in the submodule source

Discussed in https://github.com/sumneko/lua-language-server/discussions/1590

Originally posted by icp1994 September 25, 2022
This is the current set of files inside the zipped submodule source
3rd/
ci_staging/
doc/
locale/
log/
luamake/
make/
meta/
script/
test/
tools/
changelog.md
debugger.lua
LICENSE
main.lua
make.lua
README.md
test.lua
theme-tokens.md

I think 3 of these - ci_staging, log, and luamake are unnecessary for building from source but I'm not totally sure.

log is now removed in https://github.com/sumneko/lua-language-server/commit/e28d88a96995c617427534e93cf7affcd202f8e2


Update all the urls to new http4s-dom site

I case you missed it, it has a live example of http4s in the browser! 😁
https://http4s.github.io/http4s-dom

Do you want it here, or do you want a subdomain on http4s.org?

Sure, a subdomain would be cool, but this is fine too.

I'm good with this until the subdomain is set up.  Is the breakage that other issue you fixed?

Yeah, this inherited a bad main but should be fixed now.


[Fix] fix Concrete_Data instead of Concrete_data

homogenize the file name Concrete_Data instead of  Concrete_data for the reading of the file in the notebook.

You are absolutely right. Thanks for the PR. I ended up correcting together with some other stuff.


[suggestion] Support release profile in test_env.py

Description
test_env.py builds and uses several binaries in debug mode (kagami, iroha, iroha_client_cli). In some cases I find it more practical to setup a testing environment using release binaries. When I do not change binaries themselves, but want to run tests again and again, tests running with release Iroha build might complete much faster, although I haven't checked.
So, my request is to extend test_env.py CLI with an argument to specify using binaries with release profile:
./test_env.py setup --release
# or
./test_env.py setup --profile=release

import argparse
def setup_environment(profile):
# Logic to set up the environment using specified profile
if profile == 'release':
# Use release binaries
print("Setting up the environment with release binaries...")
# Your code to use release binaries goes here
else:
# Use debug binaries by default
print("Setting up the environment with debug binaries...")
# Your code to use debug binaries goes here
if name == "main":
parser = argparse.ArgumentParser(description='Setup environment for testing')
parser.add_argument('--profile', choices=['debug', 'release'], default='debug',
help='Specify the profile for binaries (default: debug)')
args = parser.parse_args()

setup_environment(args.profile)

import argparse
def setup_environment(profile):
# Logic to set up the environment using specified profile
if profile == 'release':
# Use release binaries
print("Setting up the environment with release binaries...")
# Your code to use release binaries goes here
else:
# Use debug binaries by default
print("Setting up the environment with debug binaries...")
# Your code to use debug binaries goes here
if name == "main":
parser = argparse.ArgumentParser(description='Setup environment for testing')
parser.add_argument('--profile', choices=['debug', 'release'], default='debug',
help='Specify the profile for binaries (default: debug)')
args = parser.parse_args()

setup_environment(args.profile)


New package: Porta v0.1.0

Registering package: Porta
Repository: https://github.com/iamazadi/Porta
Created by: @iamazadi
Version: v0.1.0
Commit: ce63ab649497e7c1cd63070cc54d57b16ab735c1
Git reference: master
Release notes:

Added London Tsai, Flower and Planet Hopf to the example gallery.

[noblock] The issues raised by AutoMerge have been addressed including adding compatibilities and changing the repo url in order to end with Porta.jl.git.
Should we make a new pull request?

[noblock]
Retrigger Registrator. This will update the pull request


Install NASM under Windows runners

This PR is to test what dependencies are needed to support the vendored copy of BoringSSL within aws-lc-sys.

Seems that was all that was necessary - good news!


Remove a stray // user: from a FileCheck line to make test more robust.

NFC.

@swift-ci smoke test and merge


Add a basic healthcheck endpoint

Just need some endpoint to healthcheck against so the instance can be put into service by the load balancer. Obviously we want a more complete healthcheck whenever possible.

here is my take with quick DB ping: https://github.com/bluesky-social/indigo/pull/32

See #32


ArrayOutOfBoundsException in GuiMatterTransmitter.delPlayer

Yampst reported this crash to me. I do not have any background information.
If you look at your code, you can probably see why this is happening.

Should be fixed in 1.10.2

:+1:


Sfint 4118

Link to design : Case-Assist-User-Experience
Demo:
https://user-images.githubusercontent.com/86681870/144076242-d165f3cb-0286-4510-8711-50a873818658.mov
When the value is missing:
https://user-images.githubusercontent.com/86681870/144076388-84b6c4b7-25d6-4699-bc2e-7160d3872eb8.mov
When the number of suggestions is set to zero:

When the number of suggestions is equal to the number of options we have:

The component should be prefixed with Quantic.

To implement end to end tests, you can refer to this topic to get some hints.


add DuckHunt game

PR Description 📜
Add DuckHunt Game
Fixes #741

Mark the task you have completed ✅

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

Thank you soo much for contributing to our repository 💗

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

@kunjgit Changes made & issues resolved, kindly merge it.

Thank you  @MisterMickey , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀

@daccdacc @kunjgit Kindly put label of gssoc23 and level-3


php74: 7.4.11 -> 7.4.12

Motivation for this change
Things done

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

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

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

Result of nixpkgs-review pr 102715 1

12 packages built:

adminer
arcanist
engelsystem
icingaweb2
matomo
matomo-beta
nextcloud-news-updater
phoronix-test-suite
php (php74)
qcachegrind
wp-cli
yle-dl


hook speeddial-shortcuts.js breaks Vivaldi 3.6.2165.3

Just updated Vivaldi 3.6.2165.3 snapshot, speeddial-shortcuts.js causes speed dial on new tab to fail, appears
"
An error occurred
Please consider reporting the issue to help us fix it
"

sorry, doble posted due to a 404 page after submitting the first report, please ignore

sorry, doble posted due to a 404 page after submitting the first report, please ignore

Can you just close it?

Can you just close it?


test: add a few func tests on the autocomplete

clear button
move to on click on result

rebased (to fix conflict on readme)

rebased again, so travis can check it.


Update Java SDK

Update Java SDK to latest version : 7u71-b14

Is there a particular reason to update the java version?

Mostly picking up the most recent security vulnerabilities fixes


Fix Sphinx python docstring formatting error.

Description
Fixes #13021
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[X] Changes are complete (i.e. I finished coding on this PR)
[X] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[X] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[X] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

@mxnet-label-bot add [pr-awaiting-review, doc]

@mxnet-label-bot add [pr-awaiting-merge]

@mxnet-label-bot update [pr-awaiting-merge]

@mxnet-label-bot add [Doc]


linux-40 lost external connection phone device.

Is there an existing issue for this?

[X] I have searched the existing infra issues

Type of Request
bug
Infrastructure Environment
LUCI, Github, Cocoon scheduler, Autosubmit, etc...
What is happening?
linux-40 lost external connection phone device.

Steps to reproduce
Step 1:
Step 2:
..
Step n:
Expected results
I expect to see X when Y is finished.

External connection has been restored by rebooting phone device.


File not listed in recently opened files

Issue Type: Bug
I created a new file, added some text, then saved it as a new name to my Documents folder.  I then closed it. After that I went to reopen it in the same Visual Studio Code session (I had not restarted VSC) and the file was not listed in my recently opened files. I tried opening a new VSC session (without closing my current one) and it was not listed there either. I also tried shuttind down VSC completely then reopening and it is still not listed. I have seen this (or similar) behavior at least once before.
VS Code version: Code 1.50.0 (93c2f0fbf16c5a4b10e4d5f89737d9c2c25488a3, 2020-10-07T06:10:52.432Z)
OS version: Windows_NT x64 10.0.18363

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-6820HQ CPU @ 2.70GHz (8 x 2712)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offopengl: enabled_onprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
31.86GB (21.78GB free)

Process Argv
--crash-reporter-id f33085d1-fe45-4a14-9ef7-725c2f833c89

Screen Reader
no

VM
0%

Extensions (2)

Extension
Author (truncated)
Version

remote-wsl
ms-
0.50.1

powershell
ms-
2020.6.0

This may be related, but I just ran into an issue where after I opened a project via terminal, 2 files from another open workspace were showing up in my newly opened workspace.

Recently opened are only filled from actions such as File > Open. Otherwise use the recently editor list in Cmd+P.

Title of this report was changed to an incorrect description. File was not saved outside current project. This will reproduce the behavior:

Create a new file in VSC (VS Code)
Type some text
File->Save As within VSC and save the file somewhere
Close the file in VSC
File->Open Recent in VSC and file is not listed

Are you saying that Save As will not update recently opened files? I would expect it to do so; that is normal behavior on every other application I have seen (For example, Word). None of the steps I described involve opening or saving the file outside VS Code.


Add expo test app

Description
This PR adds an Expo test app. This app was created using expo-cli, and has been integrated (partially) into the monorepo. So far, dependencies are integrated but build and bundling are not.
Why add an Expo app? The bare workflow is an interesting alternative to ejecting. It lets you keep current more easily with Expo updates, while still building and controlling the source code for iOS and Android.
Further, the Expo tools are very nice, and if we plan to interop with them, we need a test app to ensure we aren't breaking anything.
I have plans to turn this into a more interesting app in a future PR, but I wanted to commit the code-generated baseline app first with the absolute minimum set of modifications needed for working in our monorepo.

Test plan

Why does this PR look familiar? I re-created it from the one you already reviewed... It turns out that I fouled up my fork pretty badly, and had to hard-reset main and origin/main to match upstream/main. So I no longer trusted git to do the right thing re: merging. To be safe, after the hard reset, I re-created this PR.


AWS Serverless Lambda - synchronous logs

Hello
I was using this sink to log to Splunk from my C# Serverless Lambda. We found that the platform kills all background threads after the API response is sent from the controller. This was causing some logs to go missing. Would be great if you could add an additional parameter that makes all the calls to Splunk synchronous in place of the periodic batching.
Thanks

@bhavishyachandra you most likely need to call Log.CloseAndFlush() before exiting.
https://merbla.com/2016/07/06/serilog-log-closeandflush/

Did you find a solution?

@oguzhancagliyan I don't have a solution. we moved away from Lambda and hosted our API in a docker container.

This is proving to be a big problem for us too. When sending logs directly to Splunk from a C# lambda (it's an aspnetcore app) they get dropped if the lambda executes quickly.  Has anyone thought about ways to solve this in a clean way?  Some sort of middleware that flushes on each request?  Is there a way to just "flush" and not close?

Closing as a part of larger Serilog contrib reorg
Checkout https://github.com/serilog/serilog/issues/1627

Hi @lorenh ,
wrote a snippets here


Update Razor Components and JSInterop to use System.Text.Json

We need to integrate with System.Text.Json and replace our use of SimpleJSON. We don't have to expose the System.Text.Json APIs, but we need to have a default serializer to use with JS Interop.
Since we don't want to use JSON.NET in Blazor (code size), and we don't want to have breaking changes, using System.Text.Json is the most logical choice.

Dup of https://github.com/aspnet/AspNetCore-Internal/issues/1678?

external > internal

Moving this to preview5 as the required .Net Core feature is going to land then.


Project is being build more than once

I have created a custom project that does not use the .Net Sdk. I am calling it through an MSBuild task from a number of C# projects. The call to the MSBuild task is identical (identical values for the Target, Properties) for all C# projects.
The specified project+target is built more than once (especially if using parallel built).
A sample solution that reproduces this:
ReproDoubleBuild.zip
I am using the msbuild included with Visual Studio 15.3.0 Preview 3.0

I have identified the reason for the double build to be due to a new global property when building for more than one target platform. The value for the TargetFramework global property would be different and trigger a new build even though the custom project does not consumes the value of that property.
Using the RemoveProperties attribute of the MSBuild task with the values equal to the names of the properties that are not used by the target project seems to make it work.
Ideally I would like to have a way to disable global properties being passed over to the build of that project (e.g. RemoveAllProperties="true").


Pandas 1 updates

Please note, these are two logical commits.

Ignoring travis ci test failures - tested locally, all is good. Busywork should sort this out.


Basic user creation

Ability to create account
Ability to login/logout
Linking a created poll to account logged in

Some notes related to this:

Use bcrypt to hash the password on the backend

https://www.npmjs.com/package/bcrypt

We should decide between using the Bearer scheme and cookies

Cookies

Set JWT to http-only cookies with like 1-hour expiration
Set JWT on every request -> middleware is good for this
Frontend automatically sends cookies
Backend middleware checks request cookies

Bearer

During authentication return the generated JWT token to frontend
Frontend saves the token to local storage etc
With every request, the frontend sends the token in the Authorization header
Backend reads it from the header and acts accordingly


[BUG] OB Freeze / Auto STOPs

i logged in to my rdp
OB was running with no problem
bots was on 200
OB CPU usage was on 99%
OB Ram usage was on 1GB
i reduced bots, then OB stats and progress froze (except time counter)
RAM and CPU Reduced after OB froze
this all happened 2 more times before this (one time bot was on 150 and i reduced to 100 then...)
i don't remember old times that what config i was using, but this time my config was appending users in a file
note {i was using remote proxies every time that OB froze}

Same on last ver
i did not reduced bots this time, but OB get freeze (all things, except timer)
maybe it's because of using remote proxy source

After stop and reStarting runner :

my proxy settings (URLs are localhost and there is no problem with the remote source or proxies)

also i enabled logs, and there was nothing weird and there was no errors

After Clicking on OK, Runner STOPPED :
Runner Bots Display Mode = Everything
Proxies Source = Remote (Localhost)
Maybe a supernatural thing making this errors to stop me, lol

First problem that made OB freeze was because of #639
my proxies api had problem that a solved with a script and sharing file on localhost (because i could not add them all as file source #657)
and the second problem that made OB Automatic stop was Runner Bots Display Mode
i always disable it because of CPU usage, but this ability made that problem
i turned it ON and problem solved
but another problem is still there, that i posted #664