Merge CustomFolderIcons with Customization properties page

Resolved / Related Issues
Items resolved / related issues by this PR.

Closes #7431

Changes

[x] Merge CustomFolderIcons with Customization properties page
[x] Tweaked UI of Customization page
[ ] Add ViewModel of Customization page

Validation
How did you test these changes?

[x] Built and ran the app
[x] Tested the changes for accessibility

Screenshots (optional)
Add screenshots here.

Tweaked UI of Customization page

This wasn't on the list, I'll review the changes but make sure to open an issue and get approval in the future.

Add ViewModel of Customization page

Do you want to do that in this PR?

I'm doing that.

Tweaked UI of Customization page

This wasn't on the list, I'll review the changes but make sure to open an issue and get approval in the future.
What changes were made?

I have should written like: Fixed margin of GridView

This looks a lot better! Is it just the theme or is the separator line missing?

This looks a lot better! Is it just the theme or is the separator line missing?

Yeah, But you know, Main property page hosting NavigationView and OK/Cancel buttons has ScrollViewer wrapping Frame. So I set MaxHeight and if you change window height:
To fix this, remove ScrollViewer from Main property page

I changed color with: (whole separator should be this, I believe. But what do you think?)

Let's restore to what it was before, it's not visible like this

To fix this, I need to remove ScrollViewer from Main property page

Let's hold off on this change, I have some redesign work for the properties window (let me know if you're interested). I'd prefer if we limited this PR to code quality.

Done. @yaira2
No UI changed


Better deploy method

Currently the deploy scheme creates a bijillion commits in a branch. Force pushing it would be nicer.

Make sure website uses git-lfs was a check thing too

lfs is gonna be skipped, doesn't seem like its worth the effort


Remove duplicated NLTK requirement

The current requirements.txt file requires two different versions of NLTK which will stop the installation.

Duplicate of #36


Protobuf serialize for cr

Ⅰ. Describe what this PR did
Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #973 into feature_protubuf will decrease coverage by 2.18%.
The diff coverage is 25.81%.

@@                  Coverage Diff                   @@
##             feature_protubuf     #973      +/-   ##
======================================================
- Coverage               44.15%   41.97%   -2.19%
+ Complexity               1171     1160      -11
======================================================
  Files                     250      254       +4
  Lines                    9484     9754     +270
  Branches                 1065     1095      +30
======================================================
- Hits                     4188     4094      -94
- Misses                   4865     5264     +399
+ Partials                  431      396      -35

Impacted Files
Coverage Δ
Complexity Δ

...a/io/seata/core/rpc/netty/AbstractRpcRemoting.java
9.32% <ø> (-1.5%)
4 <0> (-1)

...ava/io/seata/core/constants/ConfigurationKeys.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

.../src/main/java/io/seata/core/protocol/Version.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...in/java/io/seata/tm/DefaultTransactionManager.java
4.76% <ø> (-11.91%)
1 <0> (-1)

...rc/main/java/io/seata/core/model/GlobalStatus.java
100% <ø> (ø)
7 <0> (ø)
:arrow_down:

...o/seata/core/protocol/serialize/SerializeType.java
0% <0%> (ø)
0 <0> (?)

...covery/registery/etcd/EtcdRegistryServiceImpl.java
0% <0%> (ø)
0 <0> (?)

...ta/rm/datasource/exec/SelectForUpdateExecutor.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...o/seata/server/coordinator/DefaultCoordinator.java
47.46% <0%> (-6.01%)
17 <0> (ø)

...a/io/seata/core/rpc/netty/MessageCodecHandler.java
0% <0%> (-44.16%)
0 <0> (-7)

... and 35 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 ea4facc...3cdda35. Read the comment docs.


Replace rust-sdk bindings with ones that work

Things required from rust-sdk:

[x] https://github.com/matrix-org/matrix-rust-sdk/issues/796
[x] https://github.com/matrix-org/matrix-rust-sdk/issues/797
[x] https://github.com/matrix-org/matrix-rust-sdk/issues/800
[x] https://github.com/matrix-org/matrix-rust-sdk/issues/801
[x] https://github.com/matrix-org/matrix-rust-sdk/issues/802
[x] NPM Package to use as a dependency
[x] https://github.com/matrix-org/matrix-rust-sdk/issues/842

Blocking TODO for this PR:

[x] Update unit tests
[x] Remove old bindings from dependencies
[x] Add new bindings as a dependency
[x] Add locks on sync processing and room encryption

Nice to haves:

[ ] https://github.com/matrix-org/matrix-rust-sdk/issues/798
[ ] https://github.com/matrix-org/matrix-rust-sdk/issues/799
[ ] https://github.com/matrix-org/matrix-rust-sdk/issues/814

Fixes https://github.com/turt2live/matrix-bot-sdk/issues/209
Fixes https://github.com/turt2live/matrix-bot-sdk/issues/196
Fixes https://github.com/turt2live/matrix-bot-sdk/issues/208

\o/


I got and idea

Add a command that shows how many giveaways are active in that server pls

This repo will not handle any feature requests, only bug updates, please make feature requests in the new repo


Escape enum variants to avoid collision with generated Unknown variant

This fixes enum variant collisions in #386.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Generated code diff: https://github.com/jdisanti2019/smithy-rs/compare/main-generated...fix-unknown-enum-name-collision-generated


pilot: cluster use select all endpoints if label is empty

Signed-off-by: Yuchen Dai<EMAIL_ADDRESS>Thanks @howardjohn
Fix #23821

See if existing tests passes

This should also be put into master once we get this merged.

This should also be put into master once we get this merged.

With additional tests :)

/cherrypick master

@ericvn: #23898 failed to apply on top of branch "master":
error: Failed to merge in the changes.
Using index info to reconstruct a base tree...
M	pilot/pkg/networking/core/v1alpha3/cluster_builder.go
Falling back to patching base and 3-way merge...
Auto-merging pilot/pkg/networking/core/v1alpha3/cluster_builder.go
CONFLICT (content): Merge conflict in pilot/pkg/networking/core/v1alpha3/cluster_builder.go
Patch failed at 0001 pilot: cluster use select all endpoints if label is empty

In response to this:

/cherrypick master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@ericvn: new issue created for failed cherrypick: #24467

In response to this:

/cherrypick master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Manual cherrypick to master in https://github.com/istio/istio/pull/24468


Need to allow for custom origins for On-Premise requests to the local connector

Currently only select Plotly domains are allows:
https://github.com/plotly/plotly-database-connector/blob/abf60d3cf0da4b520737f85002751a0d538c3342/backend/settings.js#L26-L30
This issue is currently being hit by matthews@porch
cc @chriddyp @scjody

Apparently this can be configured via the Replicated YAML -- currently investigating!

Modifying ~/.plotly/connector/settings.yaml to include the line below solves the issue.
CORS_ALLOWED_ORIGINS: ['https://buildly.plot.ly']

closed in the latest release


New version: Mintty.WSLtty version 3.7.4

Created by 🥟 Dumplings in workflow run #6326.

Log
Updated: 3.7.3 → 3.7.4
Submitting WinGet manifests
Creating manifests
Uploading manifests and making commits
Creating a pull request

Microsoft Reviewers: Open in CodeFlow

/azp run

https://github.com/microsoft/winget-pkgs/issues/142854


Port swift/remote to Windows and MSVC

Don't rely on dlfcn.h, use Win32 APIs instead. This requires adding a CPP file to avoid importing <windows.h> in a header.
Fix Error C2248 'anonymous-namespace'::RemoteASTContextImpl::fail': cannot access private member declared in class 'anonymous-namespace'::RemoteASTContextImpl' by making a private method protected.
Fix error C2872: 'ReflectionContext': ambiguous symbol by qualifying the namespace of ReflectionContext
Unhandled control path warnings - have to use std::cerr and std::abort as we get linker errors when we use llvm_unreachable

Thanks @compnerd. I introduced a swift_unreachable API. I made it a static method as there's no real benefit to it being a macro, I think. Also, I stuck to C-only APIs (assert and abort) to avoid Swift compilation errors - we still get a customized message and a stack trace so that's fine.

@swift-ci please test and merge

@compnerd sorry for the churn, tests passed but this didnt merge

@swift-ci Please smoke test OS X platform

@hughbe going to be a bit more paranoid and run the smoke tests on macOS before merging.


[Question] Question about serverless driver over HTTP

Now, neon provides serverless driver over Websocket in this repo.
Recently, I noticed that Neon may release http-based driver in this blog https://neon.tech/blog/postgres-ai-playground
My question is, why a  HTTP-based driver is needed for neon?

We hope to announce something on this soon. Stay tuned!


Support for Error class in NativeSerializer (feature request)

Description
Consider bugsnag code like below that tries to leverage metadata to track details of error.
const err = new Error('this is cause');
const params = { foo: 1 };
bugsnag.notify(new Error('This is an example'), (report) => {
  report.metadata.account = { cause: err, params };
});

It will create meatadata like:
{
  cause: {},
  params: { foo: 1 }
}

Note cause is logged as empty object.
Describe the solution you'd like
Implement handling for Error type within https://github.com/bugsnag/bugsnag-react-native/blob/master/src/NativeSerializer.js so it'll log relevant info (eg., error message)

Hey @daisy1754, thanks for raising this. I've confirmed that this is an issue and have scheduled some time for us to work on adding support for Error objects in the native serializer.


Thumb bugs when device orientation is changed

@hadukin
Thumb Decoration becomes offset from the appropriate position when a selection is made and device screen is rotated.

@DreuDavids hi, what version are you using? I am not able to reproduce this problem, please attach the code and I will try to fix it

I re ran the code today and seems like the problem might have been with the emulator.
cant reproduce the glitch anymore.

not relevant


mcstrans package should not be installed by default

Multi Category Security (MCS) is not used on most of systems where SElinux is enabled.
This package is recomended to be removed by CIS compliance profile.
Need an option not to install 'mcstrans' pacakge.

Fixed in version 3.1.0 of the cookbook, which is up on the Supermarket now.


TSVN errors when working on a solution that does not use SVN

TSVN Extension Version: 5.0.707
Tortoise SVN Version: 1.14.1
Some projects that I work on use SVN for source control, and others use Git.
Since updating to TSVN 5.0.686, I get TortoiseSVN error dialogs popping up in Visual Studio whenever I work on a solution that uses Git and I perform an action like adding or removing a file to a project:
Subversion reported an error:

'<PATH OF FILE> is not a working copy'

I still see this error in TSVN 5.0.707. I also updated to Tortoise SVN 1.14.1 recently, so this may also be the culprit. Disabling the extension while working on Git-based projects is a workaround, but it's a bit of a pain since I frequently have to work on both types of projects at the same time.

I confirm. More exactly, opening any file that is not part of the SVN repository causes an error box to popup (eg. just openeing a header file from a third-party library).  Workflow killer :(
I have to revert to version 4.

Should be fixed in the new release 🚀
Be sure to get the right version because I have splitted the extension in a VS2022 version and a VS2019 and prior version.
Will be on holiday from tomorrow, so new builds will have to wait for a bit 🌞

Fixed confirmed :) Have great vacations :)

Fixed for me too!


Conflicting 'id' value set by mapping for Challenge entity

Error when processing the first event:
 Oct 11 23:39:37.853 INFO Process events from block, subgraph_id: QmdCZUbn7GmGkJcGqd8YTF5GPvZKsZ4ovpPRyYiy14jdN9, subgraph_name: adchain, block_hash: 0xb070524b5c997592eb62468263c5668348c1267dbdcf7244211a29e274d03062, block_number: 5495282
 Oct 11 23:39:37.853 INFO 1 events are relevant for this subgraph, subgraph_id: QmdCZUbn7GmGkJcGqd8YTF5GPvZKsZ4ovpPRyYiy14jdN9, subgraph_name: adchain
component: RuntimeHostBuilder
 component: RuntimeHost
  subgraph_name: adchain
   subgraph_id: QmdCZUbn7GmGkJcGqd8YTF5GPvZKsZ4ovpPRyYiy14jdN9
    data_source: AdNetworks
     Oct 11 23:39:37.853 DEBG Process Ethereum event, handler: challenge, signature: _Challenge(bytes32,uint256,string,uint256,uint256,address), block_number: 5495282
component: SubgraphInstanceManager
 Oct 11 23:39:37.857 ERRO Subgraph instance failed to run: Failed to process event: Failed to handle Ethereum event with handler "challenge": Trap: Trap { kind: Host(HostExternalsError("Conflicting \'id\' value set by mapping for Challenge entity: 1 != 0x0100000000000000000000000000000000000000000000000000000000000000")) }, id: QmdCZUbn7GmGkJcGqd8YTF5GPvZKsZ4ovpPRyYiy14jdN9

@nenadjaja

I'll try to reproduce @timmclean . Are you using Infura?

still seeing this on block 5567777

test


add contact box to my portfolio

I probably should have explained this a little better during our chat last week, sorry for requesting changes to this branch again!
When we have stacked PRs/branches like we did here, we usually want to merge the one that's closest to main first. In other words, what would have been best would be to merge newbranch -> main, then merge week-3-datastore -> main once newbranch has been merged. What happened here was week-3-datasore -> newbranch, then once this PR is completed, newbranch -> main.
It's preferred to do the first way for a couple of reasons:

It's easier to keep track of features and when they are included. Here, instead of this PR only adding the contact box, this PR also adds the datastore stuff. So when you are looking back at PRs, it would be more difficult to know that this merge into main added both features.
It's easier to fix issues. When merging the week-3-datasore branch into this one, it caused some differences with some files that need to be fixed. But it shouldn't be the responsibility for this branch (newbranch) to fix that. Ideally, adding the contact box should be its own separate feature, and not responsible for fixing changes that it didn't cause in the first place  (like differences caused by adding datastore stuff)
It's easier for reviewing. If you merge another PR into an existing one, it makes this PR have all the changes associated with both mixed together, so it is hard to see which changes originally came from which PR.

Since unmerging can be very messy, especially after adding commits, we can just make the changes here (hopefully for the last time), then merge this branch into main. I pointed out some of the cleanups like spacing, temp files, merge conflicts, but feel free to fix anything else wonky you see while doing these fixes.
If you have any questions, feel free to send me a chat or ask me during our check in this week!


Add a question mark icon tooltip close to the search box

Search tips:

search by catalog number is search first if you type numbers and letters
....

@bandrow developed a quick idea on hover, which looks like this:

Can preview on https://www.areg.dev.metacell.us/. Please let me know id the idea without the icon works for you and review on the text

To add:

if you are having trouble, please check rrid.site -> https://scicrunch.org/resources


Getting zero exit code on deploy failure

Describe the bug
There is an upload failure, but CLI returns 0 exit code.
To Reproduce
Steps to reproduce the behavior:

Given issue with SWA Azure - like this one => https://github.com/Azure/static-web-apps/issues/1097
When running the command 'swa deploy --env production'
Then I can see error in log

✖ Deployment Failed :(
✖ Deployment Failure Reason: Failed to deploy the Azure Functions.
and exit code is "0"

Expected behavior
Have non-zero exit code, to get failures in pipelines
Screenshots

Desktop (please complete the following information):

OS: Debian GNU/Linux 11 (bullseye)
Version 1.0.6

We are getting the same behaviour when trying to deploy a new env without any free slots available.
Our workaround for pipelines is a shell script that greps the output for a keyword:
swa deploy -O build/your-app --env "full_env"  2>&1 | tee run.log
if grep -q BadRequest run.log; then exit 1; else exit 0; fi

the always 0 exit code makes it really hard to use this in any CI/CD scenario. I want to fail my GitHub Actions workflow if deploy fails, but since the swa exit code is is 0, GitHub Actions assumes it succeeded (as will most/all CI systems).

yea it is anoying. We run into this from first try because 1st we thought "nice that was easy" but then "oh no, it errors bur it does not fail".
It needs to return an error code, else we need to look into the job everytime.


Feature/memcached cach/sdk 130

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Visual Studio very slow on file operations

When adding or removing a file to Visual Studio project with Saltarelle target, VS gets very unresponsive compared to a normal C# project. Having this issue since i started using Saltarelle. I suspected Saltarelle is compiling project behind the scenes after every change to the project. After some research, and taking reference from Script# .targets file, i came up with following solution that seems to work, but as i'm not a msbuild script expert, not sure if it has any side effects:
I removed below lines:
	<Target Name="AfterCompile" DependsOnTargets="CoreCompile"
          Inputs="$(MSBuildAllProjects);
                $(AssemblyOriginatorKeyFile);
                @(Compile);
                @(ReferencePath);
                @(EmbeddedResource);
                @(CustomAdditionalCompileInputs)"
          Outputs="@(DocFileItem);
                   @(IntermediateScript);
                   @(IntermediateAssembly);
                   @(CustomAdditionalCompileOutputs)"
          Returns="">

Replaced it with:
<Target Name="BuildScript">

And at the end of file added this below
  <PropertyGroup>
    <BuildDependsOn>$(BuildDependsOn);BuildScript</BuildDependsOn>
  </PropertyGroup>
</Project>

So, now it seems like BuildScript (Saltarelle compiler step) step is only run after build and VS is much more responsive.
What is your opinion on this solution, would it cause any problems?

FYI, I was able to narrow the issue down to having resharper enable/disable, If i start the same solution with VS in safemode (resharper disabled) then the slowness of the file operations go away.
I been trying to turn off things in resharper to see what exactly is affecting it, if anything. Can anyone confirm if they are using resharper too when they notice the slowness ?

No, I don't use resharper but I notice the slowness (VS2013).


[#7803] update cell style in statistics table

add cell link style to agent name column & small refactoring.

@binDongKim PTAL.

I've tested it and it's displayed like below:

There seems to be 2 issues:

You applied the link-style and click event to the Agent Name field. What I'm curious is that the new tab which is opened through the Agent Name field uses the same url with the Agent Id field. Is this your intention?

I think you need to modify the cell-click logic part because now, if you click any field except the Application field, the new tab is always opened with the Agent Id. So I suggest you specify the clicked cell field here other than just else:

What I'm curious is that the new tab which is opened through the Agent Name field uses the same url with the Agent Id field. Is this your intention?
Yes. It's my intention.
So I suggest you specify the clicked cell field here other than just.
Other than the application field, the others are both link to the agent inspector page. If possible, i would like to make the agent version and agent jvm version cell clickable to make additional filter, but i haven't figure out how. Before that i intend to  make the others field link to the same.

I got your point. Thank you.


Don't add -O flags to [C/CXX/F]FLAGS outside of conda-build

Solution to issue cannot be found in the documentation.

[X] I checked the documentation.

Issue
We are using the conda-forge GNU compiler toolchain for building our packages. We would like to transition to also using the conda-forge GNU compilers for development and debugging. Unfortunately, it looks like when we activate an environment containing the GNU compilers, the activation scripts add the -O2 optimization flag to [C/CXX/F]FLAGS, which ends up being picked up by the make toolchain, and thus removes debug info automatically from our builds:
prm-login:~> conda activate test3
prm-login:~> echo $CXXFLAGS
-fvisibility-inlines-hidden -std=c++17 -fmessage-length=0 -march=nocona -mtune=haswell -ftree-vectorize -fPIC -fstack-protector-strong -fno-plt -O2 -ffunction-sections -pipe -isystem /gpfs/fs1/mpapadakis/miniconda3/envs/test3/include

If we wanted to use the compilers, we would have to externally remove this flag from [C/CXX/F]FLAGS after conda activation has completed.
The relevant snippet from the activation script:
if [ "${CONDA_BUILD:-0}" = "1" ]; then
  CXXFLAGS_USED="-fvisibility-inlines-hidden -std=c++17 -fmessage-length=0 -march=nocona -mtune=haswell -ftree-vectorize -fPIC -fstack-protector-strong -fno-plt -O2 -ffunction-sections -pipe -isystem ${PREFIX}/include -fdebug-prefix-map=${SRC_DIR}=/usr/local/src/conda/${PKG_NAME}-${PKG_VERSION} -fdebug-prefix-map=${PREFIX}=/usr/local/src/conda-prefix"
  DEBUG_CXXFLAGS_USED="-fvisibility-inlines-hidden -std=c++17 -fmessage-length=0 -march=nocona -mtune=haswell -ftree-vectorize -fPIC -fstack-protector-all -fno-plt -Og -g -Wall -Wextra -fvar-tracking-assignments -ffunction-sections -pipe -isystem ${PREFIX}/include -fdebug-prefix-map=${SRC_DIR}=/usr/local/src/conda/${PKG_NAME}-${PKG_VERSION} -fdebug-prefix-map=${PREFIX}=/usr/local/src/conda-prefix"
else
  CXXFLAGS_USED="-fvisibility-inlines-hidden -std=c++17 -fmessage-length=0 -march=nocona -mtune=haswell -ftree-vectorize -fPIC -fstack-protector-strong -fno-plt -O2 -ffunction-sections -pipe -isystem ${CONDA_PREFIX}/include"
  DEBUG_CXXFLAGS_USED="-fvisibility-inlines-hidden -std=c++17 -fmessage-length=0 -march=nocona -mtune=haswell -ftree-vectorize -fPIC -fstack-protector-all -fno-plt -Og -g -Wall -Wextra -fvar-tracking-assignments -ffunction-sections -pipe -isystem ${CONDA_PREFIX}/include"
fi

comes from the following lines in the feedstock:
https://github.com/conda-forge/ctng-compiler-activation-feedstock/blob/main/recipe/activate-g%2B%2B.sh#L85-L93
IIUC, the reason for setting these flags outside of conda-build is to improve the chances that a codebase built with these compilers will be ABI-compatible with the pre-built packages in the same conda environment (that were built with similar flags).
I don't believe optimization flags are relevant for this goal. Please correct me if I'm wrong, but I don't think the level of optimization used when compiling a new binary affects whether it is ABI-compatible with a set of existing libraries. Therefore, assuming that the use of the conda-forge compilers outside of conda-build is an expected use case, I think you should not force a level of optimization on users.
I assume a potential fix for this would be to remove -O2 from @CXXFLAGS@ and instead add it explicitly on this line: https://github.com/conda-forge/ctng-compiler-activation-feedstock/blob/main/recipe/activate-g%2B%2B.sh#L88 (and similarly for other flags).
I would note that optimization flags are the most immediately problematic, but the same argument as above could be made for other flags that are set even outside of conda-build, but don't seem relevant to guaranteeing ABI compatibility, e.g. -fmessage-length=0.
Installed packages
# packages in environment at /gpfs/fs1/mpapadakis/miniconda3/envs/test3:
#
# Name                    Version                   Build  Channel
_libgcc_mutex             0.1                 conda_forge    conda-forge
_openmp_mutex             4.5                       2_gnu    conda-forge
binutils                  2.36.1               hdd6e379_2    conda-forge
binutils_impl_linux-64    2.36.1               h193b22a_2    conda-forge
binutils_linux-64         2.36                hf3e587d_10    conda-forge
c-compiler                1.4.2                h166bdaf_0    conda-forge
compilers                 1.4.2                ha770c72_0    conda-forge
cxx-compiler              1.4.2                h924138e_0    conda-forge
fortran-compiler          1.4.2                h2a4ca65_0    conda-forge
gcc                       10.3.0              he2824d0_10    conda-forge
gcc_impl_linux-64         10.3.0              hf2f2afa_16    conda-forge
gcc_linux-64              10.3.0              hc39de41_10    conda-forge
gfortran                  10.3.0              h18518b4_10    conda-forge
gfortran_impl_linux-64    10.3.0              h73f4979_16    conda-forge
gfortran_linux-64         10.3.0              hb09a455_10    conda-forge
gxx                       10.3.0              he2824d0_10    conda-forge
gxx_impl_linux-64         10.3.0              hf2f2afa_16    conda-forge
gxx_linux-64              10.3.0              h2593f52_10    conda-forge
kernel-headers_linux-64   2.6.32              he073ed8_15    conda-forge
ld_impl_linux-64          2.36.1               hea4e1c9_2    conda-forge
libgcc-devel_linux-64     10.3.0              he6cfe16_16    conda-forge
libgcc-ng                 12.1.0              h8d9b700_16    conda-forge
libgfortran5              12.1.0              hdcd56e2_16    conda-forge
libgomp                   12.1.0              h8d9b700_16    conda-forge
libsanitizer              10.3.0              h26c7422_16    conda-forge
libstdcxx-devel_linux-64  10.3.0              he6cfe16_16    conda-forge
libstdcxx-ng              12.1.0              ha89aaad_16    conda-forge
sysroot_linux-64          2.12                he073ed8_15    conda-forge

Environment info
active environment : test3
    active env location : /gpfs/fs1/mpapadakis/miniconda3/envs/test3
            shell level : 2
       user config file : /home/mpapadakis/.condarc
 populated config files : /home/mpapadakis/.condarc
          conda version : 4.10.3
    conda-build version : not installed
         python version : 3.9.5.final.0
       virtual packages : __linux=4.15.0=0
                          __glibc=2.27=0
                          __unix=0=0
                          __archspec=1=x86_64
       base environment : /gpfs/fs1/mpapadakis/miniconda3  (writable)
      conda av data dir : /gpfs/fs1/mpapadakis/miniconda3/etc/conda
  conda av metadata url : None
           channel URLs : https://repo.anaconda.com/pkgs/main/linux-64
                          https://repo.anaconda.com/pkgs/main/noarch
                          https://repo.anaconda.com/pkgs/r/linux-64
                          https://repo.anaconda.com/pkgs/r/noarch
          package cache : /gpfs/fs1/mpapadakis/miniconda3/pkgs
                          /home/mpapadakis/.conda/pkgs
       envs directories : /gpfs/fs1/mpapadakis/miniconda3/envs
                          /home/mpapadakis/.conda/envs
               platform : linux-64
             user-agent : conda/4.10.3 requests/2.25.1 CPython/3.9.5 Linux/4.15.0-167-generic ubuntu/18.04.3 glibc/2.27
                UID:GID : 38092:30
             netrc file : None
           offline mode : False

Please see https://docs.conda.io/projects/conda-build/en/latest/resources/compiler-tools.html#customizing-the-compilers about customizing compilers.

IIUC, the reason for setting these flags outside of conda-build is to improve the chances that a codebase built with these compilers will be ABI-compatible with the pre-built packages in the same conda environment (that were built with similar flags).

That's not the only goal. The other goal is to provide compilers outside of conda-build that behave mostly like the compilers in a conda-build setting.

@manopapad , you might also find https://conda-forge.org/docs/user/faq.html#faq-compiler-required-options useful.  It is possible to use the conda-forge toolchain in an environment without using the activation scripts.

There is also the DEBUG_CXXFLAGS env variable which you can use for debug builds I believe.

I currently have this in the root of my CMakeLists.txt.
# In a Conda environment the compiler activation scripts by default specify optimization flags. Even in
# debug mode. These flags make proper debugging very hard. To combat this issue, the code below removes the
# optimization flags (specified with `-Ox`) from the C(XX)_FLAGS if the build type is Debug. The compiler
# will then default to -O0, which should allow for proper debugging.
# This assumes that a multi-config generator is not used.
if (CMAKE_COMPILER_IS_GNUCXX AND CMAKE_BUILD_TYPE STREQUAL "Debug")
	# Remove the -Ox flag from the compiler flags
	string(REGEX REPLACE "-O[g0-9][ ]*" "" CMAKE_CXX_FLAGS ${CMAKE_CXX_FLAGS})
	string(REGEX REPLACE "-O[g0-9][ ]*" "" CMAKE_C_FLAGS ${CMAKE_C_FLAGS})
endif()

The FAQ doesn't seem to discuss running the toolchain without the activation scripts, perhaps the link was meant to go to a specific revision?
Instead of creating an entirely separate compiler package, would it be possible to have an override environment variable for the optimization flags? (e.g. something like CONDA_OVERRIDE_CUDA)

The FAQ doesn't seem to discuss running the toolchain without the activation scripts, perhaps the link was meant to go to a specific revision?

You're right.  I saw gcc mentioned in https://conda-forge.org/docs/user/faq.html#faq-compiler-required-options and didn't read it carefully enough.  If you don't want the activation scripts, install (gcc, gxx and binutils).  Those will give you all of the short-named symlinks.
If you don't want to have to specify $CONDA_PREFIX/include and $CONDA_PREFIX/lib via command-line arguments, you might also want the conda-gcc-specs package.
If I get a chance, I'll improve the FAQ entry.

Just to add my +1 to this.
I'd also like to see all the other superfluous/opinionated compiler switches removed. I'd rather manage them myself (e.g. via cmake/conan). Though the forced optimisation flag is the most frustrating.
I take the point that consistency between conda-build/packages and user environments is a goal. Perhaps forking+customising is the only reasonable solution?


Custom icon entities ignore actual state when pressed

If you create a button of type "Entity (custom icons)", then pressing the button toggles between on and off regardless of the actual state of the entity. For example, if the appearance entity is a switch, if you set the action to "turn on", you would expect repeated presses to leave the visual state always on, but instead the visual state toggles back and forth. Or if you set an action that does not affect the switch state (or does not affect it immediately), you would expect the visual state to remain unchanged, but instead it toggles on and off.
This does not happen with switches created from "Entity (generic)", those correctly reflect the entity state. This only affects custom icon entities. The visual state does correctly update on its own if the state changes in HASS, the problem only seems to be that the state changes on button presses instead of waiting for HASS to say the state has changed.
This bug occurs regardless of what type of entities are used, pressing the physical button always toggles the visual appearance between on and off even if the appearance entity has a binary state that differs from this.

Looking at the Stream Deck SDK, it looks like it's the Stream Deck that will automatically toggle between states when the key is released. A workaround to this may be for the plugin to call setState every time it receives a key release event, to update the state to the HASS state.

Can confirm, setting "DisableAutomaticStates" to true in the plugin's manifest.json file fixes this bug completely. This solution only works for two-state actions, but that's what this plugin uses, so it's perfect.

Hi @Guspaz,
that is a nice finding! From the beginning the automatic switch behaviour was a problem.
I am quite sure i did not find anything like "DisableAutomaticStates" that when i initially added the feature. Maybe the option has been added to the API meanwhile
I will add this to the manifest.

You can find the documentation for it here: https://docs.elgato.com/sdk/plugins/manifest#actions
It was added in 6.4: https://docs.elgato.com/sdk/plugins/changelog
So it's probably very recent. It's also bizarre that it only works on actions with two states, and not three or more, but since this plugin only has either one or two states, that isn't a problem.

I'm facing this too. I have an icon setup for my front door state (binary door/window sensor). On key press it opens the door lock (sensor.***). When I click the butto, the doorlock opens, but the door is still closed. The icon changes to the open state however. Is this the fix?


Deprecate mixins

This PR deprecates the mixins and introduces a new API that is exposed completely via the session service.
We discussed a few options for replacing the current mixins with a new mechanism in #2185 (thanks to everyone who chimed in!) but focussed mostly on concepts that would replace the mixins directly. Since we have dropped support for older versions of Ember though and now have new capabilities that we can leverage that were not available when ESA's current mixin mechanism was first introduced, I realized we could build a completely new API.
Historically, transitioning between routes was only really possible from route classes (or controllers) since there was no routing service. For that reason, we had to establish a connection between the session and its change state on one side and the routes on the other side. With the routing service that is no longer necessary though as we can transition between routes from the session directly using the service. That means we can just drop the ApplicationRouteMixin that implemented the above described connection between the routing layer and the session and handle all of that in the session itself. We still maintain customizability of course since any app can simply extend the session service and override the respective methods, similar to how you would do the same thing in the application route currently.
The AuthenticatedRouteMixin's and UnauthenticatedRouteMixin's main responsibilities were adding custom logic into the route classes' beforeModel methods that would check for the session to be authenticated or not and transition elsewhere etc. A mixin is a nice way to share that code and automatically put it in the right place. While exposing that API via the session service now will require a bit more code, it still doesn't put significant additional cognitive load on the developer and it's easy to extract the code into a base class if one wanted to do that:
export default ProtectedRoute extends Route {
  @service session;

  beforeModel(transition) {
    this.session.requireAuthentication(transition, 'login');
  }
}

The opposite of this would be this.session.prohibitAuthentication('index') for disallowing a route to be visited when the session is authenticated already.
The OAuth2ImplicitGrantCallbackRouteMixin and DataAdapterMixin mixins are deprecated without 1:1 replacements. They provide very little value anyway and would usually only be used once per application. Thus, adding guidance in the docs for achieving the respective behavior should be sufficient.
TODO

[x] update README and guides
[x] deprecate DataAdapterMixin
[x] deprecate OAuth2ImplicitGrantCallbackRouteMixin
[x] update the API docs template to show the deprecation status of mixins

closes #2185

hmm any ideas on why the specs are failing?

@patsy-issa I think we're running into this: ember-cli/babel-plugin-ember-modules-api-polyfill#64

@marcoow -<EMAIL_ADDRESS>was just released, and should fix the underlying issue referenced in https://github.com/ember-cli/babel-plugin-ember-modules-api-polyfill/issues/64.
https://github.com/babel/ember-cli-babel/releases/tag/v7.20.1

Thanks @rwjblue ❤

Unfortunately we're running into ember-cli/babel-plugin-ember-modules-api-polyfill#110 now…

@patsy-issa CI is green now

Great to see this merged. Thanks for the amazing work!
Any idea when we can expect a release?

@pjcarly we released a beta version including this today. 🎉


CI Pipeline not picking up latest build image

Investigated and troubleshooted the code.
Checked if triggers are correctly configured
Checked if the image is correctly uploaded to the image stream

I made some changes to fix. I did some testing at my end to validate that as well. However, I think it would be better if we can test by making changes to the moodle app and see if the changes get reflected after the pipeline completes successfully. Will work with Warren to make the changes and test the pipeline run

I've added the changes and successfully run the pipeline. I've added an ImageChange trigger on the Build Config, but it didn’t seem to be working, and the ConfigChange didn’t seem to trigger a rebuild either. Not quite sure what’s missing or misconfigured yet, but I was able to successfully refresh the environment by deleting the pod.
The Adaptable theme / plugin has been installed as expected.
However, I'm not sure why the Moodle version is not latest. Latest build is 2021-22-03, but we are still on build 2021-09-13, even though it was built today.

@kkapoor-fresh I think it may be cached according to this log message: "imagestream.image.openshift.io/moodle-gamma unchanged"
https://github.com/bcgov/moodle/runs/3795036624?check_suite_focus=true#step:5:24
I've added "--no-cache" to the start-build command, in an attempt to get the latest updates for Moodle.
https://github.com/bcgov/moodle/blob/8f2387a94d8501315a210756d5a3601617d04cdb/.github/workflows/deploy.yml#L66

@warrenchristian1telus
The image change trigger is not required in this case because we do not automatically want to trigger the Deployment config once the build step in the pipeline is complete. The Deploy to Dev step in the GA pipeline is responsible for fetching the latest image from the image stream - "moodle-gamma" defined in the Deployment config template. So, when we run the template for deployment config, it should configure a new deploymentconfig and deploy the app.
The message  "imagestream.image.openshift.io/moodle-gamma unchanged"  is correct as the image stream template inside the buildconfig resource defined in openshift/spp/docker-build doesn't change. The thing that is going to change is the image inside the imagestream tag .
Regarding the --no-cache, as per my understanding, I am creating a new build config resource everytime the pipeline runs. When this happens, the subsequent star-build command should be building the moodle image from scratch.

Well, something has been keeping updates from reaching the pod. Any suggestions as to what else that might be? I can’t see how the pod will get updated if it doesn’t get replaced when the content changes in a build. It seemed to be building the image, but the pod never gets replaced.
Also, not sure if it’s another issue, but I saved a change and the build/action didn’t run. I tried re-running the last job, and it seems to be stuck in the queue:
@.***
Warren Christian
Senior Developer / Analyst, TELUS Health & Payment Solutions
T 1<PHONE_NUMBER>
telus.comhttps://telus.com/  Facebookhttps://www.facebook.com/telus  Twitterhttps://twitter.com/telus  Instagramhttps://www.instagram.com/telus/  YouTubehttps://www.youtube.com/telus  LinkedInhttps://ca.linkedin.com/company/telus
This email, including any attachments, is for the sole use of the intended recipient and may contain confidential information. If you are not the intended recipient, please notify us immediately and destroy this email and any copies.
From: kkapoor-fresh @.>
Sent: Tuesday, October 5, 2021 6:51 AM
To: bcgov/moodle @.>
Cc: Warren Christian @.>; Mention @.>
Subject: Re: [bcgov/moodle] CI Pipeline not picking up latest build image (#44)
@warrenchristian1telushttps://github.com/warrenchristian1telus
The image change trigger is not required in this case because we do not automatically want to trigger the Deployment config once the build step in the pipeline is complete. The Deploy to Dev step in the GA pipeline is responsible for fetching the latest image from the image stream - "moodle-gamma" defined in the Deployment config template. So, when we run the template for deployment config, it should configure a new deploymentconfig and deploy the app.
The message "imagestream.image.openshift.io/moodle-gamma unchanged" is correct as the image stream template inside the buildconfig resource defined in openshift/spp/docker-build doesn't change. The thing that is going to change is the image inside the imagestream tag .
Regarding the --no-cache, as per my understanding, I am creating a new build config resource everytime the pipeline runs. When this happens, the subsequent star-build command should be building the moodle image from scratch.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/bcgov/moodle/issues/44#issuecomment-934434495, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQWCMFDTWDPIZ66VG4G5DGTUFL7FPANCNFSM5EUO66TA.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

I have observed that sometimes the GITHUB ACTIONS Jobs get queued up due to issues with GA on Github side.
On your other question, yes there is something that is preventing the pods from being replaced and recreated. Ideally, when the deployment config resource is created it shouls be recreating the replication controller and pods. I am thinking to do an explicit rollout of the deployment config using oc rollout command in the Actions Deplou to Dev step

@warrenchristian1telus There was a syntactical error while deploying the build when image stream changes were added to the template file. Did you manage to fix it already?

@kkapoor-fresh I believe I fixed the syntax issue. It's re-building now, so we should find out soon if it worked.

The Deployment config is now picking up the latest build via Github actions.
Here's the image created by the build

Here's the deployment config/pod referencing the latest build image


Evaluetion on DanceTrack test

Thank you for your great job.
I successfully run the code on DanceTrack test dataset, but I can not get the test result on https://codalab.lisn.upsaclay.fr/competitions/5832.
I get this error:

How can I get the results of test set?

hi~
This may be a problem of Codalab. Can you get the result now?

Yes. Thank you.

错误怎么解决的呢


Update Modal docs to kill StateRecorder

Modal.doc.js was the only page still using the old StateRecorder component. This change refactors all the examples to use the new live docs which lets us kill the StateRecorder component once and for all.

Deploy preview for gestalt ready!
Built with commit ca7d81dbcea740a98b4cd9951a0ded2b0eabcf66
https://deploy-preview-199--gestalt.netlify.com

Codecov Report

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

@@           Coverage Diff           @@
##           master     #199   +/-   ##
=======================================
  Coverage   61.03%   61.03%
=======================================
  Files          40       40
  Lines         770      770
  Branches      178      178
=======================================
  Hits          470      470
  Misses        207      207
  Partials       93       93

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 bb0db16...ca7d81d. Read the comment docs.

@pfarejowicz It looks like some of content for our code formatting shows above our modals:

@christianvuerings good call out, the fix is to add
color: rgba(197, 200, 198, 0.7);
opacity: 1 !important;

to AtomDark.css which fixes the issue without changing anything else visually. The react-live editor was applying a 0.7 opacity which messes with the Modal overlay. Reseting it to 1 and switching from the opacity CSS attribute to the rgba() color attribute fixes the issue without changing the look and feel.


Less compile

Today, we use less.parse method on building less.
This method is deprecated, we have to change for less.parse method.

PR opened: https://github.com/es6rocks/harmonic/pull/144

Fixed by #144


Change Conditional Tests for Top-Level Variables

Workaround for https://github.com/ansible/ansible/issues/39414

Closed by #31


Allow partial contracts

Change-type: patch
Signed-off-by: Josh Bowling<EMAIL_ADDRESS>
jellyfish-core will sometimes pass partial contracts, namely when querying with additionalProperties: false.

@balena-ci I self certify!


Update README with d3 install info

This addon just uses the installed version of d3.

The blueprint is supposed to do that though (and it runs when you install the addon).

Oh I ran npm install, not ember install. Righto.


in multi languages use case, If the default language does not exist, other languages are not applied

version 2.0.0
I am really grateful for what "template_" has been removed from 1.3.0 to 2.0.0. 🙇♂
It seems to be really good when I go to one of 2.0.0, but it becomes more processing. 🙏
When using multi language,
If the MAIN RES ID does not use $ {app_name}, it will not be reflected in other languages.
Within our app/main/res/values folder, we have the following file:
<!--strings.xml-->
<resources>
    <string name="app_name">Test</string>
    <string name="welcome_message">Welcome!</string>
</resources>

👉   default ID 'welcome_message' is not used ${app_name}
Then, Within our app/main/res/values-es folder, we have the following file:
<!--any_file.xml-->
<resources>
    <string name="welcome_message">Bienvenido a ${app_name}</string>
</resources>

👉   es ID 'welcome_message' is used ${app_name}
👉   In this case, es string expects "Bienvenido a Test", but it is Bienvenido a ${app_name}

Hi @tocgic, thanks for your input.
I'm not sure if this is something that should be fixed, or if it should instead be some configurable feature, because the thing is that, usually the "main" and "language-specific" strings should have the same structure, meaning that if you need to use a placeholder in the language-specific one, it's because that placeholder is in the main one too. This helps finding placeholders quicker by just checking the main strings instead of having to go through all the languages too. However, I know that each project is different and can have its own unique requirements, I'm just trying to figure out if this requirement could be common across different projects, or rare, which I think is more the case here.
So just out of curiosity, do you believe that this issue where the same String in different languages has different structures is something common? - do you think you could bump into this same issue in multiple projects? Or do you think this is a very special exception that only happens in the project you're working on right now due to some extraordinary circumstances that don't allow you to keep the same structure between the "main" and "language-specific" strings?

We just observed the same behaviour in my current project. For the context, my app is a fork of another app, we changed the default language of the app and edited some strings. We added a placeholder only in one translation but not in the default one (we could have changed it but we didn't because it was not really necessary for us).
This is not really a problem but it took some time for us to observe it as the string was on a secondary screen, if you won't fix it, it would be nice to have a note about it 🙂

I see, thanks for the context @Florian14, I just wanted to make sure that there were cases in which there's no other way around but to define templates only on translations and not in the main resources, and taking into account that you're not the first person mentioning it, I think we can safely say that it can happen, although it doesn't seem to be a very common case. So for now I will address it as a configurable option that would make the plugin to search through all string resources across all languages if enabled. Now, this will definitely mean some performance penalties for this plugin, as it will have to search across all languages, as opposed to just the main one as it is now, so this new behavior will be disabled by default.

Hi @tocgic, @Florian14
I just released the version 2.1.0 of Stem, where you'd be able to turn on the config for searching templates not only through your main string resources, but also through your language-specific ones. It is disabled by default so in order to enable it, you'd need to add this block of code into your build.gradle file where you've applied Stem plugin into:
androidStem {
    includeLocalizedOnlyTemplates = true
}

More info on the README file.
Please feel free to reopen this ticket if you're still seeing the same issue after turning the config on.
Cheers!


Sign code for drivers in Windows

how to sign drivers by BuildUtil?

I currently use a third-party software to load a digital signature into an EXE file.

To sign a kernel mode device driver you need a EV code signing certificate (700 USD anualy). To get EV cert you need a registered company with active bank account. so everything is in your hands.


Added Guide to Contribute Examples

Changelog / Overview

Added a Guide

Motivation

Now the contributors can get an idea on how add new examples to the docs
Explanation for Changes

I made sure they get the steps right in the contribution of the examples
Testing Status

Further Comments

Closes #1373
Checklist

[x] I have read the Contributing Guidelines
[x] I have written a descriptive PR title (see top of PR template for examples)
[x] I have written a changelog entry for the PR or deem it unnecessary
[ ] My new functions/classes either have a docstring or are private
[ ] My new functions/classes have tests added and (optional) examples in the docs
[x] My new documentation builds, looks correctly formatted, and adds no additional build warnings

Reviewer Checklist

[ ] The PR title is descriptive enough
[ ] The PR is labeled correctly
[ ] The changelog entry is completed if necessary
[ ] Newly added functions/classes either have a docstring or are private
[ ] Newly added functions/classes have tests added and (optional) examples in the docs
[ ] Newly added documentation builds, looks correctly formatted, and adds no additional build warnings
EDIT:
page is here: https://manimce--1387.org.readthedocs.build/en/1387/contributing/documentation.html

I think there's an issue in spacing for the code block:

Should be just a simple indentation fix.

@behackl For Link to MyPy, I can just add these points from https://mypy.readthedocs.io/en/stable/running_mypy.html right?
But I tried using it and it was a part of manim package though, I had to use pip install mypy for mypy to be working. So if we don't have MyPy as part of the package should I remove it in Missing Sections?


Ideas for the image

Any ideas for the big picture? Not asking for a picture link, only ideas... I think that Mac (I think it's a Mac) it's not good... and doesn't make sense with Worldev! @mikicat @JeDa

Fixed.


Retrieve VPN Server address via Azure CLI

Is there a way to retrieve the server address from the CLI instead of downloading the VPN client configuration file and pulling it from the VpnSettings.xml file?

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

ID: aa7be494-b0c4-1e92-829e-119b05aadcb5
Version Independent ID: e2032468-6c27-a6a6-6194-591ca21a3c55
Content: Connect a computer to an Azure virtual network using Point-to-Site and native Azure certificate authentication: PowerShell
Content Source: articles/vpn-gateway/vpn-gateway-howto-point-to-site-rm-ps.md
Service: vpn-gateway
GitHub Login: @cherylmc
Microsoft Alias: cherylmc

@k12sec which server address are you looking for?

On OSX you have to provide the VpnServer address to get the connection to work, IP address doesn't work. I am looking for a way to pull this value programmatically so I don't have to log into the portal to get the value. It doesn't appear to be in the gateway properties so I don't know if this is a value that can be retrieved.
From the VpnSettings.xml that is bundled with the VPN Client download from the Azure Portal.

azuregateway-GUID.cloudapp.net

Realized the XML broke the post. Here is a screenshot.

@k12sec unfortunately there is not a way to get the VPN server info programmatically via the CLI. If you would like us to add this in the future, please leave your feedback Here.
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


renaming an account resulted in a duplicate account

Renamed an account, logged out immediately and now I still have an account listed using the old account name, but can't login to it anymore. The newly named account is there as well, and with that one I can login.

Seems to be a duplicate of #97:

> Q&A test was renamed to Alter Ego


Add prometheus jvm metrics. Also have erddap start on load (so datase…

…t loading begins immediately).
To see the metrics on a running server the path is:
http://localhost:8080/erddap/metrics?debug=text
Description
Adds basic Prometheus jvm metrics. This can be expanded on with additional metrics to track.
This also indicates to Tomcat/Jetty/others (in the web.xml) that the ERDDAP servlet should be loaded at startup which means dataset loading should now start immediately instead of waiting until the first request is made.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[ X ] New feature (non-breaking change which adds functionality)
[ X ] This change requires a documentation update

Checklist before requesting a review

[ X ] I have performed a self-review of my code
[ X ] My code follows the style guidelines of this project
[ X ] I have commented my code, particularly in hard-to-understand areas
[ X ] I have made corresponding changes to the documentation
[ X ] My changes generate no new warnings
[ X ] I have added tests that prove my fix is effective or that my feature works
[ X ] New and existing unit tests pass locally with my changes

This is great! I'd imagine we probably need a setting to disable the metrics entirely?

The current metrics I don't believe are sensitive. They are just general jvm data.
I added a setup.xml parameter, however just a note this doesn't stop the metrics servlet from running (the servlet is registered outside of java, in the web.xml). If prometheus metrics are disabled with the parameter that prevents the jvm metrics from getting registered. If we register other metrics without checking the parameter, they will be served.
If we want to use prometheus to track (and serve) the same metrics as shown on the status page, I'd prefer to only track them in one spot (prometheus). With the current configuration that would likely mean serving those specific metrics through prometheus even if the use prometheus parameter is off. Since those are already available publicly, maybe that's ok.


Zoom widget is greyed out

First of all, thank you for this awesome widget. It's beautiful and has many options out of the box.
I'm trying to get the Zoom widget to work, but no luck. It's greyed out and no matter what, I can't get it to recognize my Zoom app. Do you have any pointers?

I'm closing this for now, if anyone is still experience this feel free to re open it or create a new issue!


Is this a library for use with webassembly?

Hi,
Can this library be used to directly draw on a HTML5 Canvas when writing a Webassembly program?

Hi, at the moment that is not possible. At some point I would like to try it, but probably not soon.

I'm just new to webassembly and only have read about GO, sorry if don't really understand the languages yet.
Maybe I was a bit strict with 'directly'.
Can you create a Webassembly with this library to create a canvas and then in one JS call draw that entire canvas in the DOM?

No worries. So this library doesn't do anything with HTML or JS at the moment, it is pure Go and uses the system OpenGL driver to render. This is not available in a Webassembly context, so it won't work there. To make it work would require a separate implementation for Webassembly, which would be possible but I just have not started it yet.

Okay clear and thank you. I read into GO a couple of hours yesterday and it looks really cool!
I found that they now have https://golang.org/pkg/syscall/js/
This basically let's you import the DOM objects and directly call the JS functions and therefor also Canvas objects which solves the problem I had with Emscripten.
I would like to check if I understand it correctly now.

If you would've used WebGL instead of OpenGL it did work in Webassembly right?

With this library it only works if you compile it for that specific OS platform. So if I want to run it on Linux I have to use a Linux compiler, Windows a Windows compiler and Android/iOS "The GO Mobile" project?

Thanks in advance

Sure thing.

Yes, that would have worked. Although of course with Webassembly you have access to the 2D Canvas as well, so it could have used that directly. I used OpenGL here only because I didn't really see an alternative, and the whole reason I wanted this library was that I used the canvas a lot but preferred to use Go rather than JS

The library is pure Go, but you will need to supply an OpenGL context. The code that does that will probably have to be compiled on the specific platform, just as you said. But I'm not sure, the go-sdl project for example seems to include precompiled libraries, so it may be possible to cross compile for all platforms using that. I must admit I'm not sure about this though.


remove webUSB

This PR removes "Start (webUSB)" button to make the codes simpler.
It was implemented to support MacOS, which doesn't support communication via WebSerial. But for now, we are focusing on ChromeOS as a workstation and DUT, so it can be removed.
If MacOS is needed for the future, you can reproduce the codes from git log.

Seems that the message needs to be updated, since webUSB is removed.

Turns out after this change it won't work on macOS anymore since it doesn't have a driver that would expose Servo's serial ports in a way that would be visible by Chrome.

The tool aims to support the power consumption measurement of a Chromebook for now, so we deleted this to avoid the code structure becoming too complicated. But it is still future work to support webUSB connection again.

I personally think this should be reverted and reworked so that it does WebSerial in ChromeOS/Linux and WebUSB elsewhere. I suspect the Windows path also needs WebUSB, which is likely what our partners would be using.

Thanks for the feedback everyone! Created #159 to revive the WebUSB support. Please give us some time as our workload is limited. (if you want to work on it, please let me know so that I can re-assign the issue to you!!)


Small grammar correction

"Actions can be run" instead of "Actions can be runned"
Category

[x] Content fix
[ ] New article

Related issues:
What's in this Pull Request?
Small grammar correction, replaced the word "runned" with the word "run".
Guidance

thx @simoncrelling, appreciated.


Kulfy: Watch lyricsboys.com

Kulfy requests the watch of the watch_keyword lyricsboys\.com. See the MS search here and the Stack Exchange search here.

lyricsboys\.com has been seen in 0 true positives, 0 false positives, and 0 NAAs.

Approved by double-beep in Charcoal HQ


[Candidate 1v1 Imminent Victory] Jarmen voice plays at start of map

Issue Description
Jarmen voice plays at start of map.
Expected behavior
No Jarmen voice at start of map.

This will be removed for any of my maps that become ranked, my non-ranked maps have them and since this is currently my non-ranked map, that's the reason it has it. If there are no other issues with my maps then I will rename them to RANK and remove the sounds.

Fixed in [RANK] Imminent Victory ZH v2


Animations not always playing on IOS inside of Modal with presentation style "formSheet"

Description
Hey there!
We're trying to implement a new button with a spinner, as lottie animation inside, when "busy".
We have experience with Lottie from our React stack and so far everything worked like a charm with your RN library.
I've noticed some weird behaviour in RN Modals, when using them in type "formSheet" on IOS.
When the component is rendered directly with the modal, they start autoplaying without problems.
However, if I conditionally render them ({isLoading && <AnimatedLottieView ... />}), they tend not to play on IOS.
On Android they play upon re-render after a state change, on IOS 15.4 they just display the first frame and won't start animation.
Everywhere else in the app, they work without any problems. The described behaviour also happens with all other lottie animations we use in the app, as soon as placed inside a modal with this attribute.
Steps to Reproduce
 <Modal
      visible={isVisible}
      animationType="slide"
      supportedOrientations={['portrait', 'landscape']}
      presentationStyle="formSheet"
      onRequestClose={goBack}>
...
{isLoading &&
   <AnimatedLottieView
          source={require('../../../../resources/lottie/loading_button.json')}
          autoPlay
          autoSize
          loop
    />
}
...
</Modal>

As described above, we use a modal view with the type "formSheet"
If the initial state of ìsLoadingis true`, the animation on IOS works upon the first render but not after updating this state multiple times.
If the initialState of isLoading is `false, thus not rendered initially, it will also only show the first frame upon state change on IOS, still works on Android though.
If I set the presentationStyle to any other (e.g. 'fullscreen') it works as intended

Expected behavior:
The lottie animation should also play in modals with presentationType 'formSheet' upon state change
This gif here shows how it works if the modal around is in fullscreen, or the app is deployed to an Android device:

Actual behavior:
In this gif you can see that the animation doesn't show, run on IOS and after a state change.

As described above the animation works upon the initial render but not after a state change. You can see this behaviour in this gif here:

Versions
lottie-react-native: 5.1.3
react-native: 0.66.2
IOS: 15.4
Thank you very much for looking into this, let me know if you found something!
Cheers
Lucien

The same issue

This doesn't seem to be fixed by #986. The workarounds don't work for me either.

Heya.
I'm going through the repo trying to fix as many issues as i can. I'll take a look at this next weekend to see if it is still happening with V6. We did update our API surface to bring it more in line with other normal React Native views, but i'll check this regardless. If there is still any interest and anyone still has a use case, let me know

Testing on the latest version of lottie the issue seems resolved:

We did have a couple of PRs regarding the autoPlay prop since 5.1.3. I will close this, but if anyone is still running into this issue, feel free to open a new issue with details and we can take it from there.


'plotman dirs' fails on the command line

Bug Description
When executed on the CLI, plotman dirs fails with the following error message:
    File "/home/chia-blockchain/venv/bin/plotman", line 8, in <module>
        sys.exit(main())
    File "/home/chia-blockchain/venv/lib/python3.7/site-packages/plotman/plotman.py", line 205, in main
        print(reporting.dirs_report(jobs, cfg.directories, cfg.scheduling, get_term_width()))
    TypeError: dirs_report() missing 1 required positional argument: 'width'

To Reproduce
Execute plotman dirs on the command line (no interactive mode)
Expected behavior
Show directories info, as outlined in plotman -h
System Setup:

OS: OMV5 / Debian 5.10.24~bpo10+1
Plotman version: 0.4
Method of archiving: None

Config:
http://paste.debian.net/1200942/
Additional context & screenshots
Suspecting a problem with obtaining term information, I've also tried to change to use_stty_size: False in the configuration file, but it failed with the same error.

Thanks for catching this and my apologies for the hassle.  I've already got one hotfix in for v0.4 and I'll add this.  Might release it later today.  I think it is related to a change in the configuration structure.  Also, right at the moment I have been working on adding type hints and checking for them which would have caught this...  ah well, next time.

Alrighty, fixed and committed.  If you wanted to try it out you could use @release/v0.4.1 instead of @main in the installation instructions in the readme.  Thanks again.

Thank YOU, Sir! And please don't apologize for anything. The tool you've created here to make my and many lifes easier have certainly been a completely different level of hassle...   ;-)   Your efforts are massively appreciated!
Tried to test as you've advised but files are not found. I think the release tag has not been pushed yet. But please don't worry. I can test when your hotfix is done, too.

released in v0.4.1
https://github.com/ericaltendorf/plotman/blob/v0.4.1/CHANGELOG.md

Many thanks, Eric! Working as expected now. Closing as solved.


Maybe unsound in RLocal::new

Hello, thank you for your contribution in this project, I am scanning the unsoundness problem in rust project.
I notice the following code:
pub struct RLocal<T: Copy> {
    old_value: T,
    variable: *mut T,
}

impl<T> RLocal<T>
where
    T: Copy,
{
    pub fn new(new_value: T, variable: *mut T) -> RLocal<T> {
        unsafe {
            let old_value = libr::get(variable);
            libr::set(variable, new_value);

            Self {
                old_value,
                variable,
            }
        }
    }
}

Considering that pub mod raii, and new  is also a pub function. I assume that users can directly call this function. This potential situation could result in libr::get being called to  a null pointer, and might trigger undefined behavior (UB). For safety reasons, I felt it necessary to report this issue. If you have performed checks elsewhere that ensure this is safe, please don’t take offense at my raising this issue.
I suggest Several possible fixes:

If there is no external usage for RLocal or new, they should not marked as pub, at least its new should not marked as pub
new method should add additional check for null pointer.
mark new method as unsafe and proper doc to let users know that they should provide valid Pointers.

It's really an internal function from an internal crate (neither are meant to be used in other projects). We can make it pub(crate) for clarity.

Thanks for your reply, I think maybe same problem for https://github.com/posit-dev/ark/blob/a569d6ee15d85723aec5625620c0593a7a8476a7/crates/harp/src/vector/list.rs#L65, the index ptr offset is not been varify.
and
https://github.com/posit-dev/ark/blob/a569d6ee15d85723aec5625620c0593a7a8476a7/crates/ark/src/sys/unix/console.rs#L13
https://github.com/posit-dev/ark/blob/a569d6ee15d85723aec5625620c0593a7a8476a7/crates/ark/src/sys/windows/console.rs#L37
I understand that these functions are not intended for external use, but in that case it might be more appropriate to declare them as pub(crate). This will ensure that future projects on these modules will not cause possible unsound problems, thank you!


[common] allow code-server to run with additional args

Description of the change
Enable code-server addon to run with additional configured runtime arguments
Benefits
Enables additional runtime features, such as:
      --user-data-dir        Path to the user data directory.
      --extensions-dir       Path to the extensions directory.
      --list-extensions      List installed VS Code extensions.
      --force                Avoid prompts when installing VS Code extensions.
      --install-extension    Install or update a VS Code extension by id or vsix. The identifier of an extension is `${publisher}.${name}`.
                             To install a specific version provide `@${version}`. For example<EMAIL_ADDRESS>      --enable-proposed-api  Enable proposed API features for extensions. Can receive one or more extension IDs to enable individually.

As configured currently, the code-server addon will never retain state for code-server extensions, settings, etc as these are all persisted in ~/.local.  This can be overridden with the --user-data-dir argument to reference a persisted location (e.g. /config/.vscode)
Checklist

[x] Chart version bumped in Chart.yaml according to semver.
[x] Title of the PR starts with chart name (e.g. [home-assistant])
[ ] Chart is using our common library as a dependency.
[ ] (optional) Variables are documented in the README.md

@billimek you'll need to bump the version in the common-test chart as well. I ran into this with the 2.0.0 release, hence all the patch bumps lol

@billimek you'll need to bump the version in the common-test chart as well. I ran into this with the 2.0.0 release, hence all the patch bumps lol

Really!? 🤔 That seems undesirable... I thought it would only need to be updated when there's an actual change to that chart (e.g. new or modified tests)

When we publish a new version of common it tries to release common-test too

I tried going through the CI logs and the chart-releaser code, but can't really find a reason why it would do that.
Maybe we try it one more time without bumping common-test, if it still borks we'll fix that for now (yay for common-2.1.1 😆), and start looking at a more permanent fix?

Sounds good to me!

Might want to want for https://github.com/k8s-at-home/charts/pull/415 to be merged first


1.16.x: 2.1: World/BiomeConfig settings (base terraingen / noise)

Hook up all the world/biome config settings related to base terraingen / noise).
WorldConfig:

fracture horizontal/vertical
world height settings
water level settings
bedrock settings
CHC smoothing
improved smoothing

BiomeConfig:

biomeHeight
biomeVolatility
smoothRadius
maxAverageHeight
maxAverageDepth
volatility1
volatility2
volatilityWeight1
volatilityWeight2
disableNotchHeightControl
CHC
CHCSmoothRadius
water level settings

Fixed by @SuperCoder7979 , WorldConfig customHeightControlSmoothing / improvedSmoothing, BiomeConfig disableNotchHeightControl aren't used anymore looks like, can they be removed?

customHeightControlSmoothing / improvedSmoothing are default now, will remove. disableNotchheightControl to be added soon.


nixos/nscd: start in early boot

Services that have dynamic users require nscd to resolve users
via pam_systemd. Those services might not even create
their own dynamic users itself i.e. iptables.
To make sure nscd is always started when this is happening we move
nscd to sysinit.target and make sure that it is always started before
starting/reloading/restarting any other service.
Fixes: https://github.com/NixOS/nixpkgs/issues/106127

Motivation for this change
works on my machine (tm)
Things done

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

[ ] 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/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

cc @NixOS/systemd

TODO: add some test to test upgrading similar to nixos/tests/acme.nix.

any specific reason to close beyond "this is old" ?

I think this one got reverted by accident. Reopened here: https://github.com/NixOS/nixpkgs/pull/154928


Bug fixes

Fixing various errors and warnings of TypeScript and Eslint

The test failure is because of
https://github.com/nteract/node-XMLHttpRequest/issues/2


DSD-571: Notification dismissible button styling

Fixes JIRA ticket DSD-571
This PR does the following:

Fixing the styling of the Notification component's dismissible button.

How has this been tested?

Checklist:

[ ] I have updated the Storybook documentation accordingly.
[ ] All new and existing tests passed.

Front End Review:

[ ] View the example in Storybook

Tugboat has finished building the preview for this pull request!
Link:

https://pr756-lg1vkx6ukrwua5mxflbysodbfusthohv.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/618582d3e36865cdf4369878

Tugboat has finished building the preview for this pull request!
Link:

https://pr756-lg1vkx6ukrwua5mxflbysodbfusthohv.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/618582d3e36865cdf4369878


feature: supported new CONNECT method.

TRACE  method also need to support.

TRACE method also need to support.

supprted now. you can fetch the lastest apisix (master branch), it has supported this feature now.


Fixed grammar in last sentence.

The original form of the sentence was difficult to understand, so I split it into 2 complete sentences.

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Thanks for contributing to the freeCodeCamp repository. Congratulations on your first PR! 🎊
Look forward to more contributions from you! ✈


Message queue changes

Messages are now being saved by day, and room name, within the messageQueue object. There is also a 'bringDownTheHammer' function that will be run every 12 hours that will  clear out messages that are 2 days old, in order to clean up old messages.


Tile placeholder image for Maps API

We need a placeholder image in PNG format (not so important but better if it's in PNG32) in 256x256px and if possible also in 512x512px (for upcoming retina support).
The image will be used as a fallback when a http layer type is used and the urlTemplate provided is not valid.

Thanks :+1:


Allow the content-type to be specified in the cast command

Thank you for your PR! What is the motivation for this? In case we guess the type incorrectly?

The guess works fine in the case of a local file. In the case of a url though, we cannot guess the content type and default to video/mp4. This is the reason why I let the user inform the content type in that situation.
If the content-type is provided in the command line, it sounded sensible that it would override the guessed one.

@Konubinix Do you have a use case, where this would be a problem with current code?

Also, shouldn't we trust the server's mime type in that case?

@theychx. I encountered the problem when wanting to stream an jpg file. Casting a local image works, since the mimetype is found to be image/jpeg, yet when casting from the url to my nas, the default mimetype video/mp4 makes the chromecast unhappy.
Actually, I mostly stream video/webm files. With the default mimetype of video/mp4, everything works fine, but I think it is better to give the exact mime-type to avoid issues in the future.

@skorokithakis. I like this idea. That would definitely do it.

I guess that's the best we can do, short of issuing a HEAD request to the server and actually reading the header. I'm torn between doing that and just guessing based on the extension, what do you think? Of course, we could also have the latter as a fallback for the former.

With the mime-type only being important when casting images, I really think that we should go with the simplest possible solution.

Ah, if video files work regardless of mime type, yes, let's just go with the easy solution, agreed.

The funny thing is that the youtube_dl library already performs a HEAD request to get the content-type and then deduce the extension (see https://github.com/rg3/youtube-dl/blob/master/youtube_dl/extractor/generic.py#L2198). It would be awesome if it returned the content-type as well as the extension. The code of youtube_dl is big and complex enough for me not to try to find out why the content-type is not returned. It probably has good reasons not to do so.
I propose the following:

expose the ext part of _info in StreamInfo in a property with the name ext
make get_mime work when only the extension is given, so that get_mime(".mp4") returns "video/mp4"
use get_mime("." + stream.ext) in the cast command to guess the content-type

Still, I cannot find a way to factorize the calls to get_mime("." + stream.ext) that will be easy to read. I still fall on content_type=get_mime("." + stream.ext).
I will change the pull request accordingly to the above description.

Create a property on stream called mime_type and call get_mime there? I'm not crazy about the "something" + ext part, though, isn't the entire filename available somewhere in the stream?

@skorokithakis I took your comments into account

I propose the following:
In StreamInfo:
@property
def direct_url_mime_type(self):
    if self.is_video and self._info.get("direct"):
        return get_mime(self._info["webpage_url_basename"])
    else:
        return None

And then do:
content_type=stream.direct_url_mime_type or "video/mp4"

is_remote_file is the obvious choice here. Cheers.

I updated my proposal.

I think that makes sense, and I think get_mime belongs in utils (and should probably be called guess_mime). Sounds good to me!

@skorokithakis I updated the proposal again, so that mime stuff for both local and remote files are done in StreamInfo. (:face_with_head_bandage: sorry)

The updated proposal makes the most sense to me. I wanted to suggest it, but I figured you'd have a good reason not to do the guessing in that method, and I'm not near a computer so I can check. I'm glad this is actually doable, the only change is make would be to name the property guessed_content_type or to make it a method called guess_content_type().

With git grep video | wc -l giving 146 matches. Renaming video to media is not an easy task. I suggest to implement the guessed_content_type first, and dedicate another task to the renaming video -> media.
Is that ok with you if I proposed here the guessed_content_type solution, without renaming video into media?

I took into account your comments, with the exception of the renaming of video -> media

That works fine for me, but we were suggesting renaming is_video to is_remote_file, which is only used in four places.

I'm busy for the next hour or so (debugging something unrelated). If you're free in the mean time, I'd appreciate it, otherwise I'll do it in a bit.

Are you merging this, or should I push commits to this PR?

You can push here, I figure we might as well include the rename, since this branch is mildly refactory anyway.

 I meant guessed_content_type not guess_mime.
(having the "video/mp4" fallback in play_media_url makes things simpler)
I'm going to bed now. We should consider replacing all instances of the word video with media.

It doesn't matter about the commit, we're squashing anyway. Agreed about video and media, PyCharm can do that pretty quickly, and we don't need to do it in this PR. Will merge now, thank you all!

By the way, @theychx, will things still work well if we pass None as the content-type instead of mp4 when guess_mime can't guess?

Yes, there's a "video/mp4" fallback in play_media_url.

That will only work when the argument hasn't been passed, though, not when it's None (unless I'm mistaken).

You're right. Let me fix that.

Approved, thanks, everyone. I'll merge as soon as CI passes.

Thanks for the help :-).


Add init schema job to Helm hooks so that schema can be updated when upgrading deployment

…upgrading deployment

Purpose of the pull request

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

(or)
If your pull request contain incompatible change, you should also add it to docs/docs/en/guide/upgrede/incompatible.md

Codecov Report

Merging #13898 (fe5d96e) into dev (61637cc) will decrease coverage by 0.10%.
The diff coverage is n/a.

:exclamation: Current head fe5d96e differs from pull request most recent head b05002a. Consider uploading reports for the commit b05002a to get more accurate results

@@             Coverage Diff              @@
##                dev   #13898      +/-   ##
============================================
- Coverage     38.92%   38.82%   -0.10%
+ Complexity     4460     4440      -20
============================================
  Files          1158     1158
  Lines         42429    42391      -38
  Branches       4776     4751      -25
============================================
- Hits          16514    16459      -55
- Misses        24097    24117      +20
+ Partials       1818     1815       -3

see 35 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Crash - undefined method `each' for nil:NilClass

Please note that no mkv.log file was generated as the tool does not seem to have reached that far before crash.
I am running the latest gem on 64-bit Windows and using hardware NVIDIA encode on a 3080 Ti. I have successfully transcoded mkvs with this tool before, but a particular batch from a dvd series box set seems to be unsupported by the transcoder. Due to the large file size and copyrighted nature of the mkvs I have only provided a mediainfo dump for now, but I can privately send a drive link to an actual test file if requested. I'm fairly confident the input mkvs are good dumps because they play fine in VLC and Window's default video player as well.
Command ran: other-transcode {inputpath} --add-subtitle all --nvenc-recommended --nvenc-multipass fullres --copy-track-names
(Same error when running command with no options.)
other-transcode output:
Verifying "ffprobe" availability...
Verifying "ffmpeg" availability...
Verifying "mkvpropedit" availability...
Finding encoders...
Scanning media...
C:/Ruby31-x64/bin/other-transcode: undefined method `each' for nil:NilClass

      media_info['streams'].each do |stream|
                           ^^^^^

mkv information:
MediaInfo.txt
Thanks.

@hjk321 I'm sorry that you had this problem. And thanks for the dump from MediaInfo! The crash occurred before any transcoding took place which is why there's no .log file being generated.
And it appears that crash occurred during the scanning of the input media.
Which versions of ffmpeg and ffprobe do you have installed?

ffmpeg version 5.1-essentials_build-www.gyan.dev
ffprobe version 5.1-essentials_build-www.gyan.dev

@hjk321 Thanks. I'm sure that's where you got it. But can you please execute ffprobe -version within PowerShell to confirm that. Thanks.

Yes, that is the version string as displayed in powershell. It has the website appended for whatever reason.

@hjk321 Thanks! Just trying to be methodical. :)
OK, next I need you to execute ffprobe directly on one of your problem input files to simulate what other-transcode is doing internally. Please try this within PowerShell:
ffprobe -loglevel quiet -show_streams -show_format -print_format json input.mkv

...and copy the output of that here. Thanks.

Github does not support upload of .json so I'll have to upload it as .txt. Thanks for your help.
ffprobe.json.txt

@hjk321 Thanks! OK, now it looks unlikely that ffprobe itself failed or provided the incorrect data. Let's confirm that by forcing other-transcode to scan only without transcoding. Please try this with PowerShell:
other-transcode --scan input.mkv

...and, again, copy the output of that here. Thanks.

I found out what the issue was - I was running the tool from a script and did not put the input path in quotes. This batch happened to have a space in the folder names. Quoting the path makes everything work fine.
I suppose the only fix here would be a more explicit "file not found" message (as the current message gives the impression of an internal error when it is not). Sorry to waste your time.

@hjk321 No problem, I'm glad your problem is solved.
And, unfortunately, I can't fix the problem that way because the failure occurs within the Ruby runtime before I would know the file is not found.


Tapering off qubits using stabilizer conditions.

This module tapers off qubits given a set of stabilizers.
Additionally, it reduces the number of terms of a Hamiltonian (operator) using stabilizer conditions.
The module allows the user to:

Select which qubit to taper off by a given stabilizer.
Keep the length of the Pauli string after reducing terms the same as the starting one.
Return which qubits have been tapered off.

The module comes with documentation and test module.
Xavier Bonet-Monroig and Mark Steudtner

I have signed the CLA!

I'm ok with my comments being contributed.

Hey @xabomon thanks for the PR.  Looks great.  Can you merge with master so we get a fresh CI build and we can check out why this is failing. I suspect something with the init registration.

@xabomon looks like on of your tests is still attempting to import reduce_number_of_terms from transforms and not utils.

Hi @ncrubin,
Sorry for this, I just fixed the import in the test module.
Best,
Xavi

Hi @ncrubin ,
After yesterday's commit I saw in the CI that one of the test was failing in the python2.7 version because I was defining it as an empty list, after changing the mutable variables to None.
I updated the test module that includes the new possible case, and change the old one. This should now have no problem in the CI.
Let me know if anything else needs to be updated.
Best,
Xavi

I believe the rules for implementing super() are different in python2.7.  I can look this weekend to resolve this if you don't solve by then!

Hi Nick,
Don’t worry about it! I can’t look at it during the weekend, but I will do it next week.
Thanks,
Xavi

@xabomon bump on this?  it would be great to get this in!

Hi Nick,
I'm really sorry for not being active on this lately.
I manage to narrow down that the test fail in python 2.7. I am suspicious that the fail is due to our customize error messages but I have seen code within OpenFermion with customize errors.
Currently I'm building an environment with 2.7 to see if I can reproduce the issue outside the test module.
Please, let me know if you have heard about similar issues before and how to address them.
Best,
Xavi

Hi @ncrubin ,
We have fixed the CI issue but now the coverage is failing.
As far as I understand, the coveralls counts how many new lines are used in the test module. We have tested all possible issue in the main functions, this implies that internal functions are indirectly tested. Do we have to also cover internal functions?
Thanks for the assistance.
Xavi

I finally managed to fulfill CI and coverage.
Hopefully, we can now merge this PR.
Best,
Xavi


Error while setting Disabled as true or false

In the home page of bootstrap-switch (http://www.bootstrap-switch.org), when I try to enable or disable the field, it throws an error:
Command Run in the console:
$("[name=download-version]").bootstrapSwitch('setDisabled',true);
Error thrown in line 486:
TypeError: Cannot read property 'apply' of undefined
Code reference where the error is thrown:
if (!data) {
$this.data("bootstrap-switch", data = new BootstrapSwitch(this, option));
}
if (typeof option === "string") {
return ret = data[option].apply(data, args);
}

I was strugling with this too for the past 30 minutes. Googling brought me to the v2 documentation. And even looking at the v3 documentation it wasn't clear to me that these 'options' can be called like 'methods' .. I thought they were only for initialisation. I know in the menu it already says (v2), but it would be helpful if you could put a 'method' example on the doc page and a clear note on the top of the page for which version the documentation is.


Remove redundant khelpati.com filter

A copy of the filter of khelpati was mistakenly added in nepalpress section.

@rabinadk1 Thank you :)

@rabinadk1 Thank you :)


deal with deprecation of x/crypto/openpgp

Description
Per https://github.com/golang/go/issues/44226 golang.org/x/crypto/openpgp is now frozen and deprecated. We should look to switch to a different implementation to ensure we stay up to date as best we can with this technology.

similar to https://github.com/sigstore/rekor/issues/286

I remember @bobcallaway looked at this for rekor, as considered swapping for the protonmail version but had some issues.


Add short warning "Requires verified Steam account" and "Online status on Friend/Chat"

As explained in the title, please add texts in README if the script required you to have verified account to get API key and Online status to allow the script to get your currently playing game.
This is because steam by default restrict web api portal to be accessed by unverified account.

yeah sure


setup.py syntax error

I think you forgot a comma in your setup.py file after download_url='https://github.com/simonmonk/raspirobotboard3/tarball/0.1' on line 16.

thanks.
On 1 February 2016 at 18:27, marioIncandeza<EMAIL_ADDRESS>wrote:

I think you forgot a comma in your setuppy file after download_url='
https://githubcom/simonmonk/raspirobotboard3/tarball/01' on line 16
—
Reply to this email directly or view it on GitHub
https://github.com/simonmonk/raspirobotboard3/issues/3.

--
http://simonmonk.org
@simonmonk2

I posted this on the adafruit forum but no reply so far, so I thought I'd try here:
When I run test_motors.py, occasionally the motors will draw too much power and cause the Pi to reboot (I get the little rainbow square in the upper right hand corner). I'm using a BK Precision (that's the brand) DC adjustable power supply box to power the RasPiRobot board (for the moment), so I can see the voltage and how much current the bot is drawing. My first mistake was using puny leads to connect to the board. My second mistake was setting the current limit a little too low on the power supply box. Once I corrected these mistakes, I ran the test again. No crash this time, but occasionally that rainbow square would rear its head and make me real nervous. My discomfort led me to test the voltage coming out of the 5v GPIO pins. My measurement is 4.68v which seems a little low. This (http://elinux.org/RPi_Hardware#Power) documentation indicates that the power supply should stay within the bounds of 5v +or- 0.25v. I'm a little concerned that the board still isn't getting enough power (even when things aren't spinning). What should I do? And how would I know if the voltage regulator on the RasPiRobot board is functioning properly or not?

Hi 4.68V is what I would expect for a RRB3, as there is a Shottky diode
between the switched mode 5V supply's output and the 5V GPIO pin (something
recommended in the HAT standard) to allow USB power to be applied to the Pi
at the same time as power from the RRB3. In testing, I found that a Pi2 is
actually happy down to about 3.7V, so you should not find any problem with
it operating at 4.7V. The original Pi is not quite so tolerant, but even
so, 4.7V should be more than enough.
Hope this helps - Simon.
On 3 February 2016 at 23:57, marioIncandeza<EMAIL_ADDRESS>wrote:

I posted this on the adafruit forum but no reply so far, so I thought I'd
try here:
When I run test_motors.py, occasionally the motors will draw too much
power and cause the Pi to reboot (I get the little rainbow square in the
upper right hand corner). I'm using a BK Precision (that's the brand) DC
adjustable power supply box to power the RasPiRobot board (for the moment),
so I can see the voltage and how much current the bot is drawing. My first
mistake was using puny leads to connect to the board. My second mistake was
setting the current limit a little too low on the power supply box. Once I
corrected these mistakes, I ran the test again. No crash this time, but
occasionally that rainbow square would rear its head and make me real
nervous. My discomfort led me to test the voltage coming out of the 5v GPIO
pins. My measurement is 4.68v which seems a little low. This (
http://elinux.org/RPi_Hardware#Power) documentation indicates that the
power supply should stay within the bounds of 5v +or- 0.25v. I'm a lit tle
conc erned that the board still isn't getting enough power (even when
things aren't spinning). What should I do? And how would I know if the
voltage regulator on the RasPiRobot board is functioning properly or not?
—
Reply to this email directly or view it on GitHub
https://github.com/simonmonk/raspirobotboard3/issues/3#issuecomment-179539931
.

--
http://simonmonk.org
@simonmonk2

Update: I just retested with a Pi2 running a WiFi module and got it down to
2.8V before it crashed.
On 5 February 2016 at 14:32, Simon Monk<EMAIL_ADDRESS>wrote:

Hi 4.68V is what I would expect for a RRB3, as there is a Shottky diode
between the switched mode 5V supply's output and the 5V GPIO pin (something
recommended in the HAT standard) to allow USB power to be applied to the Pi
at the same time as power from the RRB3. In testing, I found that a Pi2 is
actually happy down to about 3.7V, so you should not find any problem with
it operating at 4.7V. The original Pi is not quite so tolerant, but even
so, 4.7V should be more than enough.
Hope this helps - Simon.
On 3 February 2016 at 23:57, marioIncandeza<EMAIL_ADDRESS>wrote:

I posted this on the adafruit forum but no reply so far, so I thought I'd
try here:
When I run test_motors.py, occasionally the motors will draw too much
power and cause the Pi to reboot (I get the little rainbow square in the
upper right hand corner). I'm using a BK Precision (that's the brand) DC
adjustable power supply box to power the RasPiRobot board (for the moment),
so I can see the voltage and how much current the bot is drawing. My first
mistake was using puny leads to connect to the board. My second mistake was
setting the current limit a little too low on the power supply box. Once I
corrected these mistakes, I ran the test again. No crash this time, but
occasionally that rainbow square would rear its head and make me real
nervous. My discomfort led me to test the voltage coming out of the 5v GPIO
pins. My measurement is 4.68v which seems a little low. This (
http://elinux.org/RPi_Hardware#Power) documentation indicates that the
power supply should stay within the bounds of 5v +or- 0.25v. I'm a lit tle
conc erned that the board still isn't getting enough power (even when
things aren't spinning). What should I do? And how would I know if the
voltage regulator on the RasPiRobot board is functioning properly or not?
—
Reply to this email directly or view it on GitHub
https://github.com/simonmonk/raspirobotboard3/issues/3#issuecomment-179539931
.

--
http://simonmonk.org
@simonmonk2

--
http://simonmonk.org
@simonmonk2

So, originally I was running this on a Pi B+ v1.2. I switched over to a Pi2 and things are working much better. The only other change is that I am using an ssh connection with the Pi2 instead of the HDMI cable and wireless keyboard/mouse I was using with the B+.
Thanks for taking the time to respond. It's much appreciated. I hope this feedback helps, and I'll also post this on the adafruit forum.


Expert Interview: Gio

Overview
We need to do a review with Gio to find out what is not working prior to launch
Action Items

[ ] Gio will self record and make notes
[ ] Add recordings to Resources below
[ ] use Transcriptions of  recordings to get a todo list
[ ] Make Issues
[ ] Add list of issues to the resources below

Resources/Instructions
Recordings
Issues created

a) Home page, "See how civic tech is used by:" shows 8 contributors but in organizations contributors shows only 6. Furthermore I don't find very intuitive that clicking on these contributor logo/link I go to their home page. I think it's better the link from "Organizations" that goes to their CivicTech Org page.
b) In list of projects by an organization, the "Sort by" doesn't seem to work.
c) In "Organizations" page I select contributors, I click on one contributor, see its projects, hit "back" and contributor flag is not set anymore.
d) In "Organizations" page, below "Want to add your Organization?" the button says "Add Your Project" - Maybe that's the path to add an organization, but it looks contradictory.
e) Modal to add an Organization doesn't do validation (of email or URL), at least not going to "next" step (I'm not trying to submit).
f) Add Project: I select an Organization, next step I enter an invalid URL (I get the message of repo not found). I hit "back" on browser and it brings me to the page for 3rd step (add tags), repo URL is empty.

moving forward, the system selected "code-for-all" org tag but I had selected "Codeando Mexico" organization.

g) In Project Search, I can just search for "a", maybe require to enter at least 3 letters?

@giosce Thank you for the review! We are currently processing your comments now and we will write back on a future comment if we have questions

a) We have made the suggested change
b) The sort by feature seems to be working, except the product team is unclear what the best match is supposed to do and we will follow up with the development team. Is there any other sort that you would recommend besides last updated and stargazer
c)

Codeando Mexico  has children that are not showing up on the organization page