The result is not good

I used your pretrained result and "eval.py". I tried both default settings and changing some parameters. However the generated speech is confusing.
model_epoch_36_gs_36000_0.zip

Can you tell me who did you run eval.py?


compatibility with https extension

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
using  the https extension, the error SSL certifcation verify failed, self signed certifcate
to be fair, browsers give this warning too, but there's an option to ignore it
Describe the solution you'd like
A clear and concise description of what you want to happen.
make it allowable?
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Is this the extension you are using? https://github.com/papuSpartan/stable-diffusion-webui-auto-tls-https. Try and add the SSL certificate to the system cert store as mentioned on their README.

Is this the extension you are using? https://github.com/papuSpartan/stable-diffusion-webui-auto-tls-https. Try and add the SSL certificate to the system cert store as mentioned on their README.

Sweet, that worked. Do you mind updating the troubleshooting section to include this tip? I ran into this as well.

Okay I added that, closing this issue.

Is there a way for you to detect the URI scheme and set the plug-in default to http or https automatically? The way it went for me was:

Follow install steps
Plugin gave error in Krita, nowhere else.
Figured it was TLS related as I'm not new at computers. Checked plugin settings, saw it was http://<IP_ADDRESS>:8000, added an "s".
Didn't help, checked troubleshooting docs.
Didn't see anything, came here.
Fixed with your tip, but did have to restart Krita once or twice .

Thanks, cool extension and plug-in!

Is there a way for you to detect the URI scheme and set the plug-in default to http or https automatically? The way it went for me was:
1. Follow install steps

2. Plugin gave error in Krita, nowhere else.

3. Figured it was TLS related as I'm not new at computers. Checked plugin settings, saw it was http://<IP_ADDRESS>:8000, added an "s".

4. Didn't help, checked troubleshooting docs.

5. Didn't see anything, came here.

6. Fixed with your tip, but did have to restart Krita once or twice .

Thanks, cool extension and plug-in!

For most browsers (& builtin libraries), http & https are treated as separate URLs. So the way auto-detect would work is attempt to connect to either http or https first, fail, then try the other one.


Updating import statement for new version of django-polymorphic.

An import statement in models_bases/init.py was preventing us from
upgrading to django-polymorphic v0.8.0 or later. This import statement
has been fixed and the django-polymorphic requirement in setup.py has
been upgraded.

 (3) Update dependencies as noted by "make piprot"


Issue with putio config init

Hi,
I did a new install on two different Ubuntu 18.04.5 LTS boxes and on both I get the following error when running the putio config init command.
Traceback (most recent call last):
File "/home/zerith/.local/bin/putio", line 5, in
from putio_automator.cli import main
File "/home/zerith/.local/lib/python3.6/site-packages/putio_automator/cli.py", line 34, in
from . import commands
File "/home/zerith/.local/lib/python3.6/site-packages/putio_automator/commands/init.py", line 8, in
import putio_automator.commands.torrents
File "/home/zerith/.local/lib/python3.6/site-packages/putio_automator/commands/torrents.py", line 12, in
from fswatch import Monitor
File "/home/zerith/.local/lib/python3.6/site-packages/fswatch/init.py", line 2, in
from .fswatch import Monitor
File "/home/zerith/.local/lib/python3.6/site-packages/fswatch/fswatch.py", line 4, in
from fswatch import libfswatch
File "/home/zerith/.local/lib/python3.6/site-packages/fswatch/libfswatch.py", line 9, in
fsw_init_library = lib.fsw_init_library
File "/usr/lib/python3.6/ctypes/init.py", line 361, in getattr
func = self.getitem(name)
File "/usr/lib/python3.6/ctypes/init.py", line 366, in getitem
func = self._FuncPtr((name_or_ordinal, self))
AttributeError: /usr/bin/python3: undefined symbol: fsw_init_library
Any idea why this will happen?

I'll need to rethink this choice - it looks like you need an OS package in addition to the python one. Try apt install fswatch, then try again. If that works, awesome. I will look for an alternative solution that doesn't require an OS package to support it. 🤞🏼

Hi, I initially did manually try to install both fs-watch and fswatch to see if either worked but neither did. I also tried via apt and made no difference.
nuc:/mnt/files/downloads/putio$ sudo apt install fswatch
Reading package lists... Done
Building dependency tree
Reading state information... Done
The following additional packages will be installed:
  libfswatch9
The following NEW packages will be installed:
  fswatch libfswatch9
0 upgraded, 2 newly installed, 0 to remove and 0 not upgraded.
Need to get 166 kB of archives.
After this operation, 526 kB of additional disk space will be used.
Do you want to continue? [Y/n]
Get:1 http://za.archive.ubuntu.com/ubuntu bionic-updates/universe amd64 libfswatch9 amd64 1.11.2+repack-10ubuntu0.1 [93.7 kB]
Get:2 http://za.archive.ubuntu.com/ubuntu bionic-updates/universe amd64 fswatch amd64 1.11.2+repack-10ubuntu0.1 [72.0 kB]
Fetched 166 kB in 1s (280 kB/s)
Selecting previously unselected package libfswatch9.
(Reading database ... 188863 files and directories currently installed.)
Preparing to unpack .../libfswatch9_1.11.2+repack-10ubuntu0.1_amd64.deb ...
Unpacking libfswatch9 (1.11.2+repack-10ubuntu0.1) ...
Selecting previously unselected package fswatch.
Preparing to unpack .../fswatch_1.11.2+repack-10ubuntu0.1_amd64.deb ...
Unpacking fswatch (1.11.2+repack-10ubuntu0.1) ...
Setting up libfswatch9 (1.11.2+repack-10ubuntu0.1) ...
Setting up fswatch (1.11.2+repack-10ubuntu0.1) ...
Processing triggers for man-db (2.8.3-2ubuntu0.1) ...
Processing triggers for install-info (6.5.0.dfsg.1-2) ...
Processing triggers for libc-bin (2.27-3ubuntu1.4) ...
nuc:/mnt/files/downloads/putio$ pip2.7 install --user putio-automator
/usr/lib/python2.7/dist-packages/secretstorage/dhcrypto.py:15: CryptographyDeprecationWarning: Python 2 is no longer supported by the Python core team. Support for it is now deprecated in cryptography, and will be removed in the next release.
  from cryptography.utils import int_from_bytes
DEPRECATION: Python 2.7 reached the end of its life on January 1st, 2020. Please upgrade your Python as Python 2.7 is no longer maintained. pip 21.0 will drop support for Python 2.7 in January 2021. More details about Python 2 support in pip can be found at https://pip.pypa.io/en/latest/development/release-process/#python-2-support pip 21.0 will remove support for this functionality.
Collecting putio-automator
  Using cached putio-automator-2.0.0.tar.gz (12 kB)
Collecting appdirs
  Using cached appdirs-1.4.4-py2.py3-none-any.whl (9.6 kB)
Requirement already satisfied: click in /usr/local/lib/python2.7/dist-packages (from putio-automator) (7.1.2)
Collecting click-log
  Using cached click_log-0.3.2-py2.py3-none-any.whl (4.6 kB)
ERROR: Could not find a version that satisfies the requirement fswatch (from putio-automator) (from versions: none)
ERROR: No matching distribution found for fswatch (from putio-automator)

I tried with pip 21.2.4 (python 3.6) and get the following.
@nuc:~$ putio config init Traceback (most recent call last):   File "/home/**/.local/bin/putio", line 5, in <module>     from putio_automator.cli import main   File "/home/**/.local/lib/python3.6/site-packages/putio_automator/cli.py", line 34, in <module>     from . import commands   File "/home/**/.local/lib/python3.6/site-packages/putio_automator/commands/__init__.py", line 8, in <module>     import putio_automator.commands.torrents   File "/home/**/.local/lib/python3.6/site-packages/putio_automator/commands/torrents.py", line 12, in <module>     from fswatch import Monitor   File "/home/**/.local/lib/python3.6/site-packages/fswatch/__init__.py", line 2, in <module>     from .fswatch import Monitor   File "/home/**/.local/lib/python3.6/site-packages/fswatch/fswatch.py", line 4, in <module>     from fswatch import libfswatch   File "/home/**/.local/lib/python3.6/site-packages/fswatch/libfswatch.py", line 9, in <module>     fsw_init_library = lib.fsw_init_library   File "/usr/lib/python3.6/ctypes/__init__.py", line 361, in __getattr__     func = self.__getitem__(name)   File "/usr/lib/python3.6/ctypes/__init__.py", line 366, in __getitem__     func = self._FuncPtr((name_or_ordinal, self)) AttributeError: /usr/bin/python3: undefined symbol: fsw_init_library

can you try this with the more recent LTS?

I'll have a hotfix out by the end of the day for this. I'll use some ducktyping and mix the two approaches depending on OS.

Awesome, thanks.

yes, you must use --user option when installing.

find which config file is being used with:
putio config show

then verify the contents of the file are correct.
if you are always executing this from a specific folder, you can override the config with a .env file with the appropriate values in it.

use which putio to figure out which binary you are executing. it should be in ~/.local/bin for ubuntu i think.


Add missing Security tab entries

Questions
Answers

Description?
Since the new Security page has been added in the develop branch, it was not available after an upgrade from previous versions. This PR fixes it by adding the missing tab entries in the DB during the upgrade.

Type?
bug fix

BC breaks?
no

Deprecations?
no

Fixed ticket?
Fixes PrestaShop/PrestaShop#27735.

How to test?
Try to upgrade from <IP_ADDRESS> to develop, the menu security should be available in the menu.

Possible impacts?

Thanks @marwachelly


When using this tool to generate a CJK font library, I encountered an issue that causes an error when loading the font.

In the code, the ids_size variable is of type uint8_t. However, the font library generated by the tool has a list_length of 256, causing ids_size to wrap around to 0. This results in lv_mem_alloc failing to allocate RAM.
[
HSBR26.txt
](url)

In the cmap_split function, can these two conditions be changed to less than 255? Since path.end is incremented by 1, if all_codepoints[i] - all_codepoints[j] equals 255, it satisfies the condition of being less than 256. After path.end + 1, it becomes 256. Could this be the reason why list_length exceeds 255?

I'm not the developer of the font converter, but it seems like an oversight to me. Can you send a pull request to fix it?


Add a forward-only task to sbibm

this PR paves the way to allow users to add simulations without a reference posterior. This way, (I hope) it becomes easier for users to test drive and benchmark sbi for their use case even.
Closes #19

I am not yet 100% happy with the naming scheme. I also observed that no tasks appear to have unit tests.

Thanks for starting off a PR! Perhaps it would be best to have a general unit tests that checks all tasks in the same way and consider more specific ones on a per-task basis?

[x] added a very general test suite for two_moons (a starting point for more perhaps)
[x] finished norefposterior implementation for the time being
[ ] add unit tests to illustrate usage limitations

The code is pretty much done at this point. the simulate step for norefposterior turns out to be quite slow. I am not a pyro expert and hence need to check at some later point in time, what the root cause of this might be.
@jan-matthis please review.

alright, this PR is ready to go from my side. Studying the other tasks, I see that my application tends to be somewhat special as I simulate an entire dense representation. So I am happy to receive any feedback on that.
I'll check now, how to obtain benchmark results on this task.

Great, I will review it tomorrow

Could you rebase this on main for the next round of reviews?

huh, I thought I did so last night. Let me check.

Done, I hope.

Sorry folks, I hand-cuffed myself by using git rebase in a wrong fashion. I undid my rebase from earlier now (using cherry-pick) and repeated the exercise.

I hope I have addressed all comments now. Sorry for the mess with the accidental rebase.

No worries at all -- unfortunately, I think something is not quite right yet: Currently the PR still contains changes are on the main branch already

Not sure what you mean. github says this PR has no conflicts. I just did a manual merge upstream main ... what am I missing?

I think you resolved the issue with commit (b876c91). All looks good now!
I assume this is ready to be merged then? 🎉

Let's wait for #29 to be merged first. Then I will close this PR, split the code according to #29 and resubmit a new PR. One merit of this is also, that this very long conversation gets squashed into a handful of commits. Feel free to disagree.
Quite frankly: I'd love to get on with stuff and start running experiments.

#29 is merged

Let's wait for #29 to be merged first. Then I will close this PR, split the code according to #29 and resubmit a new PR. One merit of this is also, that this very long conversation gets squashed into a handful of commits. Feel free to disagree.

Sounds good to me!

closed in favor of #34
Thanks to @jan-matthis and @janfb for all your valuable feedback


Require should be added to ImportSections to allow recursive requirement checks

Is your feature request related to a problem?
Yes. I'm trying to create a library of reusable devspace components that can be used for different project across the company. My goal is for example to create a common setup for all golang projects and import all the required components to keep the devspace.yaml within the projects as small as possible. This is to avoid to have to apply changes to similar projects one by one. While I was setting up things, I noticed that require is not imported recursive. To me this is some kind of important, as I want to define the requirements in the different components, that are using different commands and plugins.
Which solution do you suggest?
I suggest to add require to the imported fields list here: https://github.com/loft-sh/devspace/blob/main/pkg/devspace/config/loader/imports.go#L17
I already prepared a Pull-Request for that. Which I will reference to this issue soon.
Which alternative solutions exist?
Additional context

/kind feature

@dtomasi thanks for creating this issue! Yes we can do that and thanks for the PR!


conda plugin should create a noarch directory

I installed the conda user plugin.
I successfully deployed a conda package. I now have a linux-64 and osx-64 directory under the repository.
When I run a command with the artifactory channel, I get the following warning:
Fetching package metadata ....
WARNING: The remote server could not find the noarch directory for the
requested channel with url: https://artifactory.xxx.xxx:443/artifactory/ics-conda

It is possible you have given conda an invalid channel. Please double-check
your conda configuration using `conda config --show`.

If the requested url is in fact a valid conda channel, please request that the
channel administrator create `noarch/repodata.json` and associated
`noarch/repodata.json.bz2` files, even if `noarch/repodata.json` is empty.
$ mkdir noarch
$ echo '{}' > noarch/repodata.json
$ bzip2 -k noarch/repodata.json
.........

I tried creating manually the noarch directory and the repodata.json files but that resulted in the conda plugin to throw an exception. Full trace here:
https://gist.github.com/beenje/ac5342e9730f2bc7f386575978519542
I guess to avoid the warning I have to upload a noarch package. But it would be nice if the conda plugin would handle that itself.
Or am I missing something?

@beenje Can you share the content of the repodata.json file you are trying to uploade under noarch folder?
I was able to do that and got no errors from conda plugin using the following content:
{
    "info": {},
    "packages": {}
}

As mentioned in the warning, I tried to upload an empty object:
{}
I uploaded a noarch package and then deleted it and my repodata.json is now identical to yours:
{
    "info": {},
    "packages": {}
}

That's indeed what should be used. It would be nice if the plugin was initializing that automatically.

@beenje can you confirm that the warning message is gone after uploading the repodata.json file with the changed content?

Yes, no more warning from conda commands with this repodata.json under noarch.

conda support has been officially added to artifactory from version 6.3: https://www.jfrog.com/confluence/display/RTF/Conda+Repositories
This issue can be closed.


Fix Flaky TaskBatcherTests Tests

Signed-off-by: kkewwei<EMAIL_ADDRESS>Description
org.opensearch.cluster.service.TaskBatcherTests.testNoTasksAreDroppedInParallelSubmission is flaky
Related Issues
Resolves #12815
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Failing checks are inspected and point to the corresponding known issue(s) (See: Troubleshooting Failing Builds)
[ ] Commits are signed per the DCO using --signoff
[ ] Commit changes are listed out in CHANGELOG.md file (See: Changelog)
[ ] Public documentation issue/PR created

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

❌ Gradle check result for 5c301ab: FAILURE
Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

org.opensearch.indices.store.IndicesStoreIntegrationIT.testShardActiveElsewhereDoesNotDeleteAnother {p0={"cluster.remote_store.state.enabled":"true"}}
https://github.com/opensearch-project/OpenSearch/issues/12835

Thanks @kkewwei. See above for what we typically do for other flakes (review, open new issues if needed, force push to re-kick gradle check).

org.opensearch.cluster.MinimumClusterManagerNodesIT.testThreeNodesNoClusterManagerBlock

org.opensearch.cluster.MinimumClusterManagerNodesIT.testThreeNodesNoClusterManagerBlock #10006


Updated bit.py Bit._repr when no register is assigned

Summary
bit.py Bit._repr with or without a provided register should be consistent.
Details and comments
When a Bit is created without providing a register and index, Bit._repr is not set and Bit.__repr__(self) returns the default string representation of the object. This seems inconsistent to me. I suggest that in this case, Bit._repr is set to f"{self.__class__.__name__}(no register assigned)" (or something similar).

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

I appreciate the detailed response! You are right, I just wanted prettier printing, but I guess being able to tell the instances apart this way makes sense. Feel free to close this PR. :)


Update embedded containerd to 1.4.9-k3s1

https://github.com/containerd/containerd/releases/tag/v1.4.9
https://github.com/containerd/containerd/pull/5747

This was validated as part of 1.22 validations, so I am going to close this out. We are bumping the embedded containerd version today anyway again for 1.22.2+k3s2


Simplify the use of S3 bucket for work directory

Here is the config I used in order to use S3 as the work directory:
workDir = "$HOME/nextflow-ci-dev/work"

process.executor = 'float'

float {
    address = '...'
    username = '...'
    password = '...'
    nfs = '[mode=rw]s3://nextflow-ci-dev/work'
}

I also had to mount s3://nextflow-ci-dev to my launch environment.
It would be nice to simplify the S3 usage in two ways:

If the user specifies the workDir as an S3 path, set the nfs option automatically. That way they don't have to set it twice or think about how things are mounted or wonder why the setting is called nfs when they are using it for S3 😄

Leverage Nextflow's ability to transparently operate on S3 paths through the Path API. In other words, your executor should be able to interact with the work directory without it being mounted locally. That way, the user doesn't need to mount an S3 bucket, they can just run their pipeline.

I initially tried to do this with workDir = 's3://nextflow-ci-dev/work', but it failed with an UnsupportedOperationException. No error trace unfortunately, but it is somewhere in GridTaskHandler::submit(). This exception likely means that the executor tried to do a file operation that isn't supported by S3.

These are very good suggestions.  At least we could do item one.  But we will also try item two.  Thanks!

For number two, I would be happy to help you out since it might be tricky. But it would be a huge win for you guys in terms of attracting Nextflow users, who are quite spoiled and used to not having to mount their S3 bucket. Maybe I can review your code and try to outline the changes you would need to make?

That would be very helpful.  Will let you know if we have any problem.

Since you have a REST API as you said in #24, probably the most straightforward approach would be to model the float executor after the AWS Batch executor in Nextflow rather than the grid executor, using the REST API directly instead of the CLI. The grid executor isn't really the right abstraction for float, because float is running in the cloud and using object storage as the work directory.
If that's too much work, you might be able to get away with customizing the grid task handler for float.

Cedric, I'm going to try a few things today to see if I can get the S3 paths to work.
I see that the float CLI can accept an S3 path for the job script. Can it accept the job script via stdin?

Did you change the build system at all? I'm trying to build the plugin following the README, but I when I run it, I get the following error:
$ ./gradlew compileGroovy

BUILD SUCCESSFUL in 3s
21 actionable tasks: 1 executed, 20 up-to-date

$ ./launch.sh run hello -plugins nf-float
Missing '.launch.classpath' file -- create it by running: ./gradlew exportClasspath

$ ./gradlew exportClasspath

FAILURE: Build failed with an exception.

Task 'exportClasspath' not found in root project 'nf-float'.

Never mind the build issue, I got it to work. I will submit a PR with some general improvements separately.
Let me know if the float CLI can accept stdin, but we should be able to make it work either way.

Cedric, sorry to spam you with comments. Although I was able to build locally, I'm getting this error when I try to run a pipeline:
$ ./launch.sh run hello -plugins nf-float
N E X T F L O W  ~  version 23.06.0-edge
Launching `https://github.com/nextflow-io/hello` [determined_wright] DSL2 - revision: 1d71f857bb [master]
ERROR ~ Unknown executor name: float

I am just following the README instructions. Please let me know if there is anything else I need to do to make it work.

Looks like the plugin is not loaded.  You may need to check .nextflow.log to see if nf-float is loaded.

Unfortunately, the float command command does not support stdin scripts.  The work around I can think of is to download the script and put it to some temp directory.

The nf-lfoat plugin is being loaded, but it's not registering the executor. That's all I can tell for now. I will keep digging, maybe there is a bug in Nextflow. Which version of Nextflow are you building against?
As for the job script, it says the float CLI accepts an S3 path so we can just do that.

The source code I am building against on my dev machine is 23.02.1-edge.
The version I am test with is version 23.04.2 build 5870

Is it possible for you to create a PR in the repo?  Maybe I could take a look and debug the code.

No code changes yet, this is just the base plugin. Here is a summary of what I'm doing:

clone nf-float plugin
clone nextflow to ../nextflow at revision v23.02.1-edge
add includeBuild('../nextflow') to settings.gradle
build plugin with ./gradlew compileGroovy
test plugin: ./launch.sh run hello -plugins nf-float

At this point I get this error:
Missing '.launch.classpath' file -- create it by running: ./gradlew exportClasspath

This was the first issue I asked about. It looks like the Makefile is missing a build step, compared to nf-hello:
compile:
	./gradlew :nextflow:exportClasspath compileGroovy
	@echo "DONE `date`"

So I run ./gradlew :nextflow:exportClasspath to generate the classpath file.
When I run the plugin again, I get the missing executor error:
 ./launch.sh run hello -plugins nf-float
N E X T F L O W  ~  version 23.02.1-edge
Launching `https://github.com/nextflow-io/hello` [desperate_noyce] DSL2 - revision: 1d71f857bb [master]
Unknown executor name: float

So, how you are building and running this plugin differently from what I have done?

Oh, I compile and generate the zip file with ./gradlew makeZip to create the zip file and them deploy it to $NXF_HOME/plugins folder.
I will also create a configure file include the float scope.  Then I can run it with nextflow run <script> -c float.conf

Okay, I see how that might be more reliable. I'm not sure why the source build isn't working, could be a bug in Nextflow. I will look into it when I have time.
Thanks Cedric. I think that should be enough to get me into testing. I'm going to make a PR with some general suggestions around configuration, and I will look into the improved S3 support.

Can you add me as a collaborator so that I can push a branch? Or would you prefer that I create a fork?

I don't have the privilege to add cooperator.  Please feel free to create a fork.

Fixed by #44


Fails when reading masked superstrings of length $> 2^{31} =<PHONE_NUMBER>$

On a FASTA file with a single entry, a masked superstring, of length above $2^{31}$, FMSI v0.2 fails with The file 'large_input.fa' is in incorrect format. It is supposed to be a fasta file with a single entry, the masked superstring

@OndrejSladky I've put such a file into /nfs/archive/Vesely-public/escherichia_coli.k63.r_0.1.S_local.k_23.d_2.fa

Thanks for catching! The kseq library did not by default support this, so I have rewritten the functionality. It should now work even for large masked superstrings.

What's the corresponding commit? If kseq didn't support this, it might be worth creating a PR for klib.

I see the commit. This is actually something that should be very thoroughly tested – to me this seems as a potentially very slow and dangerous code, and I would be much more inclined towards using the old one with mentioning this as a potential technical limitation.
Potential issues:

Pref - Many comparisons instead of arrays, etc.
Perf - Likely many C++ reallocations while reading strings
Correctness - Own parsing of the format that doesn't anticipate all the weird versions of FASTA

Correctness - Own parsing of the FASTA format, which doesn't anticipate all the weird versions of FASTA (these are always some of the worst errors in bioinformatics)

In our case, we deal with very specific FASTA files, where sequences are not allowed to have anything else than ACGTacgt and it is not expected to encounter anything than a single entry with a masked superstrings.
Hence, I do not see this as an issue. Furthermore, according to all the documentations of the fasta format, I've seen, the parsing should be done correctly. We can discuss this more if you disagree.
Furthermore, if there is incorrectness in the parser, it'd be good to spot it as it is also used by kmercamel and Prophasm.
Regarding the performance. The code was taken from kmercamel, where this was actually more efficient than loading the sequences. This is a bit different case, but in general, it would surprise me if this became the bottleneck.
Especially considering the fact that the creation of the index currently requires to build the inverse suffix array, peak memory consumption should not be increased by this. I believe that the time should be also dominated by this, but can check this a bit more.

In the end I found a version of kseq from seqtk that supported very long input files. Therefore I reverted the commit and changed to this parser.
Thanks for pointing this out!


Tag the current release at 0.8.0 when Gorgonia's 0.8.0 is ready

see this: https://github.com/chewxy/gorgonia/pull/156
and this https://github.com/chewxy/gorgonia/issues/140

Done


pranjalT24.md

I wanted to add my readme to the repo

@pranjalT24 Go for it. Brother don't forget to ⭐

Thanks bro I did it. But why its not showing in Hactoberfest Id of mine the
pull request u had merged
On Thu, Oct 13, 2022 at 11:39 PM Aaquib Ali @.***>
wrote:

@pranjalT24 https://github.com/pranjalT24 Go for it. Brother don't
forget to ⭐
—
Reply to this email directly, view it on GitHub
https://github.com/imaaquibali/Awesome-GitHub-README-Profiles/issues/22#issuecomment-1277990899,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASJ23BBKLNM3CRGL3T6JEX3WDBF45ANCNFSM6AAAAAAREQG4PQ
.
You are receiving this because you were assigned.Message ID:
@.***
com>


Unable to open file: Os { code: 2, kind: NotFound, message: "No such file or directory" }

Hi,
I try to use Angora but I face some problems.

thread 'main' panicked at 'Unable to open file: Os { code: 2, kind: NotFound, message: "No such file or directory" }', fuzzer/src/check_dep.rs:34:35

stack backtrace:
0: rust_begin_unwind
at /build/rustc-h1hlaa/rustc-1.51.0+dfsg1+llvm/library/std/src/panicking.rs:493:5
1: core::panicking::panic_fmt
at /build/rustc-h1hlaa/rustc-1.51.0+dfsg1+llvm/library/core/src/panicking.rs:92:14
2: core::result::unwrap_failed
at /build/rustc-h1hlaa/rustc-1.51.0+dfsg1+llvm/library/core/src/result.rs:1284:5
3: angora::check_dep::mmap_file
4: angora::command::CommandOpt::new
5: angora::fuzz_main::fuzz_main
6: fuzzer::main
note: Some details are omitted, run with RUST_BACKTRACE=full for a verbose backtrace.
Can you give me some advice?

I had the same problem！！！

From what I see, it is most likely your path to fast or track is incorrect.

got it,thank u !!!
------------------ 原始邮件 ------------------
发件人:                                                                                "AngoraFuzzer/Angora"                                                                                @.>;
发送时间: 2021年11月4日(星期四) 中午12:53
@.>;
<EMAIL_ADDRESS>主题: Re: [AngoraFuzzer/Angora] Unable to open file: Os { code: 2, kind: NotFound, message: "No such file or directory" } (Issue #112)
From what I see, it is most likely your path to fast or track is incorrect.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.


Removing forEach and ToSlice() from unordered set

Removed forEach method from unordered set

@rheaj99 Can you also add tests for Union and Intersection functions ?


update

[ok] Do only one thing
[ok] Non breaking API changes
[ok] Tested

What did this pull request do?
Void use const Variable.

User Case Description

我也不太会描述

If we don't provide an API to change the prefix, it is not necessary to make this change?


Include Kubernetes abbreviation k8s

I was at Kubecon last week, and a few times I encountered people who did not know that k8s == Kubernetes. Therefore, we should mention this abbreviation on the front page.

/lgtm
/approve


Identity-based Connections - no examples??????  really?

The section Identity-based Connections really needs some examples.  The lingo in this section is frankly gibberish/buzz-words. I want to use Managed Identities in Azure and DefaultAzureCredential to allow developers to debug locally, but this section was no help in my efforts to create a connection string.
Examples are crucial.  Function Apps are gaining a horrible reputation in my company because of the large and cryptic amount of configuration for the comparatively little code you are trying to run. I cannot tell you how many times I've had to read and re-read the Function app documentation and look through examples to figure out why my Function Apps are non-functional yet again.
Why do you need examples???  Database connection strings are so different and complicated, and there are entire websites and apps to help you build them. They have been around for decades to help overcome this cryptic and difficult thing for programmers.

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

ID: 1d50399c-0873-e903-35ca-1845c84b6c6c
Version Independent ID: 1a3150b4-5eb9-0ac5-ee2b-d1bb6b6fd63e
Content: Azure Service Bus trigger for Azure Functions
Content Source: articles/azure-functions/functions-bindings-service-bus-trigger.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

@rsnet Thanks for your feedback and sorry to hear about the experience. We have assigned this issue to the author to review further and take the next course of action.


add createActionAsync

This PR adds the function createActionAsync   which creates an asynchronous action. This avoids writing a lot a boilerplate code.
There is definitely room for improvements regarding testing and doc but I would like to make sure you are willing to integrate this change before.

I'm not sure about this pull request. There are two main concerns. First is that it's rely on redux-thunk middleware and I would rather not have any dependency on this project. Second is that it doesn't add any core feature, it's just a wrapper on top of the existing ones, so I would rather have people write their own (there is a sample of how to do it in the README) rather than imposing one.

Do I have to create a new npm package just to add 1 function ? I'd rather not.

That's up to you. The redux-thunk package is just one function with something like 10 lines of code, and that's fine. Otherwise, you can just have the code in your own project, other people who need it will do the same.

Adding this code to many projects is a clear sign a code duplication, I avoid cut & paste like the plague.

It is, for sure. Publish it on NPM then, I still see no good reason to add / impose a dependency to redux-thunk in this project. Otherwise, someone will ask the same thing for redux-promise or any async middleware tomorrow.

https://github.com/FredericHeem/redux-act-async

Nicely done!


Policy as Code for All Clouds

Per Cameron's request - PaC examples for AWS/Azure/GCP/Kubernetes

Ah, looks like this was closed and a commit was made at around the same time, which resulted in one cleaned-up bucket and one newly created one. (Anytime a commit is made, we make a new bucket.) Looks like I have an edge case to cover!

I deleted the branch and pushed it again, I wasn't sure what was going on
On Thu, Oct 22, 2020 at 1:38 PM Christian Nunciato<EMAIL_ADDRESS>wrote:

Ah, looks like this was closed and a commit was made at around the same
time, which resulted in one cleaned-up bucket and one newly created one.
(Anytime a commit is made, we make a new bucket.) Looks like I have an edge
case to cover!
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/pulumi/docs/pull/4415#issuecomment-714683779, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAE35UAYFJY254535ASIRKTSMB3YRANCNFSM4S3Q5PGQ
.


Fix Weather.razor page for fluentblazor project template

Pull Request
📖 Description
Fix extraneous quotation mark.
An extraneous quotation mark makes the razor file invalid.
🎫 Issues
dotnet new fluentblazor

creates a project that will no build.
👩💻 Reviewer Notes
📑 Test Plan
✅ Checklist
General

[ ] I have added tests for my changes.
[X] I have tested my changes.
[ ] I have updated the project documentation to reflect my changes.
[X] I have read the CONTRIBUTING documentation and followed the standards for this project.

Component-specific

[ ] I have added a new component
[ ] I have added Unit Tests for my new compontent
[ ] I have modified an existing component
[ ] I have validate Unit Tests for an existing component

⏭ Next Steps

Hi Gary, Thanks again!
Already found that one too (albeit just after publishing package...)! Already fixed it in latest commit. Closing this one.


Change minutes interval in day view

Hello, is it possible to change minutes interval in day view from 30 minutes to 15? If it's not currently possible, do you plan this feature in future?
Thank you

Yes, it's possible, but not currently:-), I'll add it into the to-do list.
------------------ 原始邮件 ------------------
发件人<EMAIL_ADDRESS>发送时间: 2018年4月19日(星期四) 下午3:21
收件人<EMAIL_ADDRESS>抄送<EMAIL_ADDRESS>主题: [StephenChou1017/react-big-scheduler] Change minutes interval in dayview (#4)
Hello, is it possible to change minutes interval in day view from 30 minutes to 15? If it's not currently possible, do you plan this feature in future?
Thank you
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Ok, I have basic functionality working, Could you please take a look on the latest version on my github and write me some comments (if you want a different way to implement it)? There is new configuration property minuteStep, which indicates one "span" time in minutes. So far I've tested just 15 and 10 minutes and it seems to be working.

@henryroach
I've downloaded your code and had a try.  I agree with you on adding a new property "minuteStep" to the SchedulerData.config object,  but I think we can merge three head cells (two if set to 15) into one(one head cell per hour), just like the fullcalendar-scheduler does, what do you think:-)

Ok, sounds good, i will adjust the solution and let you know ;-)


Use a fixed field length

Fix the length of the field so that it's more appropriate for the input that it requires.

You can preview this change here:
Built with commit c2cd5b36d463ef084319b8f1e4f0ef0b8d7a5c08
https://deploy-preview-239--govuk-design-system-preview.netlify.com


Gmd laueretal2020 revisions

Changes in font size and addition of correlation coefficients to plots created by ecs_scatter.ncl applied during the revsion of Lauer et al. (Geosci. Model Dev. Discuss.), in which these diagnostics are described.

Looks good to me, but maybe @schlunma should test it.


Storage of encrypted content encryption keys in an EDV

In some instances a group of documents inside an EDV may have a common level of access meaning they could be encrypted with a common encryption key and that encryption key is then encrypted once (via key agreement and key wrapping) to each of the valid recipients. This optimisation can be useful to bypass the need to perform the key agreement and key wrapping step on each document inside the group.
How this would look for instance is an EDV persists a document of the following structure.
{
  "id":"z19x9iFMnfo4YLsShKAvnJk4L",
  "sequence":0,
  "jwe":{
    "protected": {
        "enc":"C20P",
        "kid": "https://edv.example.com/document/4287248957624"
    }, // base64 decoded
    "iv":"FoJ5uPIR6HDPFCtD",
    "ciphertext":"tIupQ-9MeYLdkAc1Us0Mdlp1kZ5Dbavq0No-eJ91cF0R0hE",
    "tag":"TMRcEPc74knOIbXhLDJA_w"
  }
}

Where the kid references another document in the EDV of the following structure
{
  "id":"z19x9iFMnfo4YLsShKAvnJk4L",
  "jwe":{
    "protected": {
        "enc":"A256KW"
    }, // base64 decoded
    "recipients":[
      {
        "header":{
          "kid":"urn:123",
          "alg":"ECDH-ES",
          "epk":{
            "kty":"OKP",
            "crv":"X25519",
            "x":"d7rIddZWblHmCc0mYZJw39SGteink_afiLraUb-qwgs"
          },
          "apu":"d7rIddZWblHmCc0mYZJw39SGteink_afiLraUb-qwgs",
          "apv":"dXJuOjEyMw"
        }
      }
    ],
    "iv":"FoJ5uPIR6HDPFCtD",
    "ciphertext":"tIupQ-9MeYLdkAc1Us0Mdlp1kZ5Dbavq0No-eJ91cF0R0hE", < - encrypted content encryption key
    "tag":"TMRcEPc74knOIbXhLDJA_w"
  }
}

Where the decryption of the above document yields the content encryption/decryption key for the first document.
Questions

The above approach can be implemented with EDV's today where an encrypted content encryption key is stored as an encrypted document, however if we wanted greater semantic separation we could elect to store encrypted content encryption keys under a new endpoint /keys instead of storing these as encrypted documents.

Should this pattern feature anywhere in the spec or is it more of a candidate for an implementation guide?

We are implementing something like this for the self-sovereign health
record (vaccine credentials too) use-case and would really appreciate the
community's input and participation.
Our architecture assumes that the actual health records are NOT in EDVs
because the hospitals, labs, and payers have to have the data in the clear.
We treat them as normal resource servers (Issuers and verifiers in
SSI-speak). From a (GDPR) role perspective, they are "processors" (not
"controllers") although they are very jealous of the controllers. I'm
overstating this a bit for clarity.
The authorization server (AS) that represents the "controller" technology
is modeled as an EDV that holds the policies for turning requests into
access tokens at the resource servers (RS) as processors. In our model, as
in @Tobias Looker @.> case above, each patient's
policies are encrypted with a separate patient-level key.  Unlike @Tobias
Looker @.> use-case, we have no reason to wrap
keys because the policies are never shared. As a matter of fact, for
obvious security reasons, the AS can only read policies in the EDV.
Policies are written to the EDV only by the user agent of the patient as
resource owner (RO).
The interesting question then becomes, where are the keys to the policy EDV
stored so they're accessible to the AS on a need-to-know basis when a
request comes in to a particular patient? We don't assume that the RO's
mobile wallet is online when a request comes to the AS. The encryption key
will ideally be in a system that is separate from the EDV and is designed
explicitly as a (Web) KMS. Like the EDV, the AS, and the RSs, the KMS
should be a standards-based delegate of the RO. @Tobias Looker
@.***> seems to be asking whether the EDV and the KMS
are one and the same? I have the same question.
The best-practice for our use-case will demonstrate zero-trust architecture
built around DIDs, VC, and, as much as possible, capabilities. The design
should also demonstrate how audits will mitigate the risk of a compromised
AS or RS.

Adrian

On Mon, May 24, 2021 at 8:38 PM Tobias Looker @.***>
wrote:

In some instances a group of documents inside an EDV may have a common
level of access meaning they could be encrypted with a common encryption
key and that encryption key is then encrypted once (via key agreement and
key wrapping) to each of the valid recipients. This optimisation can be
useful to bypass the need to perform the key agreement and key wrapping
step on each document inside the group.
How this would look for instance is an EDV persists a document of the
following structure.
{
"id":"z19x9iFMnfo4YLsShKAvnJk4L",
"sequence":0,
"jwe":{
"protected": {
"enc":"C20P",
"kid": "https://edv.example.com/document/4287248957624"
}, // base64 decoded
"iv":"FoJ5uPIR6HDPFCtD",
"ciphertext":"tIupQ-9MeYLdkAc1Us0Mdlp1kZ5Dbavq0No-eJ91cF0R0hE",
"tag":"TMRcEPc74knOIbXhLDJA_w"
}
}
Where the kid references another document in the EDV of the following
structure
{
"id":"z19x9iFMnfo4YLsShKAvnJk4L",
"jwe":{
"protected": {
"enc":"A256KW"
}, // base64 decoded
"recipients":[
{
"header":{
"kid":"urn:123",
"alg":"ECDH-ES",
"epk":{
"kty":"OKP",
"crv":"X25519",
"x":"d7rIddZWblHmCc0mYZJw39SGteink_afiLraUb-qwgs"
},
"apu":"d7rIddZWblHmCc0mYZJw39SGteink_afiLraUb-qwgs",
"apv":"dXJuOjEyMw"
}
}
],
"iv":"FoJ5uPIR6HDPFCtD",
"ciphertext":"tIupQ-9MeYLdkAc1Us0Mdlp1kZ5Dbavq0No-eJ91cF0R0hE", < - encrypted content encryption key
"tag":"TMRcEPc74knOIbXhLDJA_w"
}
}
Where the decryption of the above document yields the content
encryption/decryption key for the first document.
Questions

The above approach can be implemented with EDV's today where an
encrypted content encryption key is stored as an encrypted document,
however if we wanted greater semantic separation we could elect to store
encrypted content encryption keys under a new endpoint /keys instead
of storing these as encrypted documents.
2.
Should this pattern feature anywhere in the spec or is it more of a
candidate for an implementation guide?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/decentralized-identity/edv-spec/issues/45, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AABB4YLJZ7KDYRMGL57Z7L3TPLWQLANCNFSM45OGPJ7A
.

Someone who has edit access to third-party comments could perhaps edit @agropper's latest comment above, to change all instances of @tobias Looker<EMAIL_ADDRESS>(no codefence) to `@tobias Looker<EMAIL_ADDRESS>(with codefence) or @tplooker (who is actually on the issue), so we stop pinging @tobias about this issue?

Discussed on WG call 5th of August, no resolution taken about the future of the issue. Had a brief discussion about other potential usecases for the capability such as integration with a web KMS.

Discussed on Aug 2, 2021 call.
See comment on https://github.com/decentralized-identity/edv-spec/issues/42#issuecomment-912046683 -- this should be covered in the Sharing section.

[@dmitrizagidulin] Discussed on Aug 2, 2021 call.

s/Aug 2/Sept 2/


Add Pod Security Policy

In order to ensure pod security is being applied to all projects in rancher, we need a consistent way of propagating those settings to the k8s clusters managed by rancher.
Right now in rancher we have pod security policy templates (PSPT) the are associated with a given Rancher project. However, these are not being applied in any way to the Kubernetes clusters our Rancher projects are managing.
To solve this problem we will be creating 4 sync handlers.
PSPT sync handler

Check if the Pod Security Policy (PSP) exists in the current namespace and cluster
If the PSP does not exist then create it
If the PSP exists and matches the current PSPT do nothing
If the PSP exists and does not match the current PSPT than update the PSP

Project sync handler

Retrieve the PSPT based on the PSPT ID
Compare the PSPT to the PSP for the current cluster and namespace
If it does not match create/update the PSP for the current cluster and namespace

Cluster sync handler

Retrieve the PSPT based on the PSPT ID
Compare the PSPT to the PSP for the current cluster and namespace
If it does not match create/update the PSP for the current cluster and namespace

Service Account sync handler

Check if a cluster role exists authorizing the PSP for the current namespace and cluster
If a PSP cluster role does not exist, create it
Check if a role binding exists between the PSP cluster role and the Service Account
If a role binding does not exist, then create a Role Binding between the PSP cluster role and the Service Account

In order for this to work we need to specify a way for the user to enable PSP admissions controller on RKE clusters in the rancher UI.  Then when kontainer-engine is invoked, it can pass the appropriate parameters to it for the cluster (will need to update container-engine as well).

Tested with latest 2.0 builds.
PSP can be now be set at cluster level / project level . Issues found when testing this feature is tracked as different issues.


How to work with RDLC or Word Reports in VsCode?

I have transfered existing Reports to VsCode with txt2al.exe.
When I change the DataSet in the .al-File, how is the connected .rdlc-File supposed to know about that change in the XML-DataSet? In addition, how do I edit exists Report Layouts or create new Reports?

I create a word document and link it like this

Then i publish and go to Custom Report Layouts in dynamics, i create a new record

Then I export the layout. It will download your original file with the data linked. Then once you are happy with the layout file, just overwrite your original layout in your project with the new one and republish.
If you change the dataset, you will have to repeat this process. ensure that you use a new custom report layout when you need to export a change as the Default layout file is linked to the record at the time of creation of the custom layout
http://thinkaboutit.be/2017/08/how-do-i-create-a-report-and-layout-in-vscode/


Bug with arrays

Hi! Thank you for goqu library, it is very helpful.
But I have found a big problem working with arrays.
I have a struct which contains array:
type Hero struct {
	Id int64 `db:"id" json:"id" goqu:"skipinsert"`
	Skills []string `db:"skills" json:"skills"`
}

I have a function which creates a Hero (INSERT with transaction):
func (r Repo) InsertOneTransaction(tx *goqu.TxDatabase, h *Hero) error {
	dialect := goqu.Dialect("postgres")

	q, _, _ := dialect. // Look result query below
		Insert("heroes").
		Rows(h).
		Returning(h).
		ToSQL()

	if _, err := tx.ScanStruct(h, q); err != nil {
		return err
	}

	return nil
}

Result query is:
INSERT INTO "heroes" ("skills") VALUES (()) RETURNING "id", "skills"
I have tested arrays with other types (like int64, float64, etc...) and goqu always transform arrays to '()'
Looks like it is not possible to use arrays in goqu.
If am wrong, please correct me.

I want to clarify the situation:
Example 1 (empty array):
hero := Hero{Skills: []string{}}
Goqu transforms it to INSERT INTO "heroes" ("skills") VALUES (()) RETURNING "id", "skills"
Example 2 (array is not empty):
hero := Hero{Skills: []string{"flying"}}
Goqu transforms it to INSERT INTO "heroes" ("skills") VALUES (('flying')) RETURNING "id", "skills"

Hi @KlimSidorov, by default goqu will serialize slices with parens because the most common use case for using them is in an IN clause.
If you want the postgres array syntax II would recommend using the drivers array datatypes (https://godoc.org/github.com/lib/pq#Array).

In your example above I would use https://godoc.org/github.com/lib/pq#StringArray

Thank you for your response.
In my case I have a big struct, so when I do queries I provide whole struct as an argument for Rows() and Returning().
It is a problem for me to use pg.Array only for special fields as I have said earlier I provide whole struct (but not several fields of the struct). And if I set pg.Array as a field type, then it will be very uncomfortable to use it next in my project.
Also I use pgx instead of pq and working with types there is not so great.
I think I will use hand made sql queries for structs which contain arrays + goqu for others

@KlimSidorov I set up this workaround, since I had the same problem.
https://gist.github.com/rgalanakis/a2d5b8b61ee9540981bb43a103bf59c7
// PGArrayExpr takes a slice of items and returns a literal expression
// with a Postgres array that can be used by goqu.
// It does this by using goqu natively and then manipulating the string
// to surround it in an array literal.
//
// To use this in a struct, you can do something like:
//
//     type MyModel struct {
//       Tags   []string        `json:"tags" db:"-"`
//       DbTags goqu.Expression `json:"-" db:"tags"`
//     }
//     body := `{"tags":["x", "y"]}`
//     m := MyModel{}
//     _ = json.Unmarshal([]byte(body), &m)
//     m.DbTags = goqux.PGArrayExpr(m.Tags)
//     sql, _, _ := goqu.Insert("modeltable").Rows(m).ToSQL()
func PGArrayExpr[T any](arr []T) goqu.Expression {
  if len(arr) == 0 {
    return goqu.L("'{}'")
  }
  lit := goqu.V(arr)
  selectSql, _, err := goqu.From(lit).ToSQL()
  if err != nil {
    panic(err)
  }
  valuesSql := strings.TrimPrefix(selectSql, "SELECT * FROM ")
  if valuesSql == selectSql {
    panic("expected go to output an (invalid) 'SELECT * FROM (x, y, z)' for the slice")
  }
  if valuesSql[0] != '(' || valuesSql[len(valuesSql)-1] != ')' {
    panic("expected goqu to output '(x, y, z)' but is missing parens")
  }
  arraySql := fmt.Sprintf("ARRAY[%s]", valuesSql[1:len(valuesSql)-1])
  return goqu.L(arraySql)
}


Missing documentation on how logging appenders are selected for ECS reformatting

Describe the bug
Missing documentation on how ECS reformatting works: how does it know which appenders to apply to. This would allow users to be confident that they aren't overriding e.g. their Sentry logger at the same time as their normal STDOUT stream.
Or my expectations for the reformatter might be too high and it just reformats ALL appenders, in which case this should also be documented as a warning to users :)
Expected behavior
A list of conditions that must be met for the ECS reformatter to be applied to an appender (on top of enabling the reformatting option, which is already documented).

Thanks for reporting this.
Currently we reformat only FileAppender types (hardcoded) and plan to apply that to console appenders as well (see #2570).
We had a related discussion before, and we ended up only with an option for fine tuning is based on formatter types, but I don't think this is what you need.
@felixbarny I'd be happy to hear your opinion on this. To me it sounds reasonable to allow filtering based on appender type/name or even logger name pattern.

Thanks for reporting this. Currently we reformat only FileAppender types (hardcoded) and plan to apply that to console appenders as well (see #2570).

There is actually this bit of the docs on which appenders are affected:
[...] while SHADE and REPLACE options are only relevant to file log appenders, the OVERRIDE option is also valid for other appenders, like System out and console

The "other appenders" part had me concerned about stuff like the Sentry appender which would not be a file appender I guess, probably has an in-memory buffer and makes a bunch of network calls.

We had a related discussion before, and we ended up only with an option for fine tuning is based on formatter types, but I don't think this is what you need.

No, this is actually what I was looking for. I missed it in the docs because the doc for log_ecs_reformatting, which is basically the entry point for this feature's documentation, doesn't mention it. Since the docs I was complaining about do exist you can close this issue if you wish.
The default value for this option does explain why my normal STDOUT appender is being reformatted (it normally uses a PatternLayout formatter) and does reassure me that my Sentry appender should be "safe" from reformatting.

No, this is actually what I was looking for

Awesome, then I will indeed close this issue and keep waiting for feedback on actually required enhancements.

The default value for this option does explain why my normal STDOUT appender is being reformatted (it normally uses a PatternLayout formatter)

That's right! I forgot I added this different handling for OVERRIDE. I'll change #2570 accordingly.
Thanks!


Improve sorting networks API, Add support for up to 32 elements

Improve API by adding inheritance and nullability
Add support for up to 32 elements sorting using bose-nelson algorithm

Codecov Report

Merging #4 into master will increase coverage by 0.04%.
The diff coverage is 99.09%.

@@            Coverage Diff             @@
##           master       #4      +/-   ##
==========================================
+ Coverage   99.94%   99.98%   +0.04%
==========================================
  Files          16       13       -3
  Lines        3618    24427   +20809
  Branches       44       44
==========================================
+ Hits         3616    24424   +20808
- Misses          2        3       +1

Impacted Files
Coverage Δ

src/SortingNetworks/SNBase.cs
96.77% <0.00%> (-3.23%)
:arrow_down:

src/SortingNetworks/SNBestKnown.cs
0.00% <0.00%> (ø)

src/SortingNetworks/SNBoseNelson.Comparable.cs
100.00% <ø> (ø)

src/SortingNetworks/SNBoseNelson.Comparison.cs
100.00% <ø> (ø)

.../SortingNetworks/SNBoseNelson.ComparisonPointer.cs
100.00% <ø> (ø)

src/SortingNetworks/SNBoseNelson.cs
0.00% <0.00%> (ø)

src/SortingNetworks/SN.cs
100.00% <100.00%> (ø)

...ortingNetworks/SNBestKnown.Comparable.Ascending.cs
100.00% <100.00%> (ø)

...rtingNetworks/SNBestKnown.Comparable.Descending.cs
100.00% <100.00%> (ø)

...ortingNetworks/SNBestKnown.Comparison.Ascending.cs
100.00% <100.00%> (ø)

... and 6 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 013b9d6...834467d. Read the comment docs.


Handle the JSON Response

Handle the JSON Response correctly . Sometimes it crashes up the whole application

Hi, I would love to work on this issue.


Update strings.xml

Fixed typos

Hi @gwenl,
I need to ask you to sign off your PR as per of https://github.com/vector-im/riot-android/blob/master/CONTRIBUTING.rst#sign-off.
Do you know that you can also do translation on https://translate.riot.im/projects/riot-android/ ?

Signed-off-by: Gwenaël Longo<EMAIL_ADDRESS>


More simple container/viewer checks

Fixes #12385

Sure thing! :)

/botio-linux preview

Looks good to me. Thanks!


Updates the Fedora port used in executors

The Fedora docker image mounts the application at /, rather than /fcrepo/, so the port used by the executors with Fedora was wrong.

Failures before this change were all Ldp::HttpError: Status 405: https://app.circleci.com/pipelines/github/samvera/hyrax/3602/workflows/ecd1c1da-331d-41d0-8067-f4bbb10965b3/jobs/23471
Versus failures after are not: https://app.circleci.com/pipelines/github/samvera/hyrax/3718/workflows/e032bfbc-1c26-4add-83e4-1975bd88a336/jobs/24758


Fix status code for rebuildBrokerResourceFromHelixTags

This PR fixes the status code rebuildBrokerResourceFromHelixTags.
Previously no matter what the response was, the response code was always 200.
Now it strictly follows the instruction shown on the method above(200/404/500).

Codecov Report

Merging #2751 into master will increase coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #2751      +/-   ##
==========================================
+ Coverage   68.53%   68.53%   +<.01%
==========================================
  Files         876      876
  Lines       42288    42292       +4
  Branches     5748     5750       +2
==========================================
+ Hits        28980    28984       +4
- Misses      11445    11447       +2
+ Partials     1863     1861       -2

Impacted Files
Coverage Δ

...troller/api/resources/PinotTableTenantConfigs.java
0% <0%> (ø)
:arrow_up:

...impl/dictionary/FloatOffHeapMutableDictionary.java
74.54% <0%> (-12.73%)
:arrow_down:

...e/impl/dictionary/LongOnHeapMutableDictionary.java
82.22% <0%> (-6.67%)
:arrow_down:

...impl/dictionary/DoubleOnHeapMutableDictionary.java
68.88% <0%> (-6.67%)
:arrow_down:

.../impl/dictionary/FloatOnHeapMutableDictionary.java
86.66% <0%> (-6.67%)
:arrow_down:

...ot/core/query/pruner/ColumnValueSegmentPruner.java
84.74% <0%> (-5.09%)
:arrow_down:

...not/broker/broker/helix/ClusterChangeMediator.java
62.71% <0%> (-3.39%)
:arrow_down:

...rEqualitiesToInClauseFilterQueryTreeOptimizer.java
93.33% <0%> (-2.67%)
:arrow_down:

...not/broker/requesthandler/RangeMergeOptimizer.java
89.7% <0%> (-1.48%)
:arrow_down:

.../pinot/core/common/datatable/DataTableBuilder.java
78.82% <0%> (-1.18%)
:arrow_down:

... and 14 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 7d57001...9e76d65. Read the comment docs.


[Query] Why use build pod instead of k8s job?

Hey! I am new to kpack. I notice that kpack will create a build job with several InitContainers to do the build staff one by one. I am out of curiosity that is k8s native job more suitable to do it? Any background? Thanks!

@tomkennedy513 Would you mind taking a look? Thanks!


ListItem doesn't lose focus when mouse out after click

[x] This is not a v0.x issue.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior 🤔

ListItem should lose focus after mouse out.
Current Behavior 😯

Still focused after mouse out
Steps to Reproduce 🕹

Link:
Compare from v3.9.0 to v3.9.2.
https://imgur.com/a/r0jpijj
Context 🔦

Your Environment 🌎

Tech
Version

Material-UI
v3.9.2

React
16.8.3

Browser
Chrome Version 72.0.3626.109

TypeScript
/

etc.
MacOS Version 10.14.3

@xs9627 Thank you for the report, the regression was introduced in #14212. We should be able to fix it by removing the unterlated focus style changes of the pull request, we should have keep it isolated to the key handling:
--- a/packages/material-ui/src/ListItem/ListItem.js
+++ b/packages/material-ui/src/ListItem/ListItem.js
@@ -20,7 +20,7 @@ export const styles = theme => ({
     textAlign: 'left',
     paddingTop: 11, // To use 10px in v4
     paddingBottom: 11, // To use 10px in v4
-    '&$selected, &$selected:hover, &$selected:focus': {
+    '&$selected, &$selected:hover': {
       backgroundColor: theme.palette.action.selected,
     },
   },
@@ -28,9 +28,10 @@ export const styles = theme => ({
   container: {
     position: 'relative',
   },
-  // To remove in v4
   /* Styles applied to the `component`'s `focusVisibleClassName` property if `button={true}`. */
-  focusVisible: {},
+  focusVisible: {
+    backgroundColor: theme.palette.action.selected,
+  },
   /* Legacy styles applied to the root element. Use `root` instead. */
   default: {},
   /* Styles applied to the `component` element if `dense={true}` or `children` includes `Avatar`. */
@@ -69,9 +70,6 @@ export const styles = theme => ({
         backgroundColor: 'transparent',
       },
     },
-    '&:focus': {
-      backgroundColor: theme.palette.action.hover,
-    },
   },
   /* Styles applied to the `component` element if `children` includes `ListItemSecondaryAction`. */
   secondaryAction: {

Do you want to submit a pull request? :)

It's now only fixed in 4.x.x, shouldn't it be fixed in 3.x.x too, considering it is broken in 3.x.x too?

@leroydev We would be happy to apply the same bug fix on v3.9. We hope to release the first beta release of v4 this weekend.


Feature request: Add/Import full Lyrics to be displayed as track is streamed

Is this possible?
Best regards

This is not supported by Spotify's Web API, so it can't unfortunately be fetched.


Bridge is not publishing in application topic

Is this a bug or a feature request?
BUG
What did you expect?
Bridge publish in application topic
What happened?
Events status up are publishing with no erros, just seen below. But phyPayload is crypted and we dont know how decrypted
'''
{"phyPayload":"QAEA/wOAAAABTzUhn6mgd0J2oncpWoa5XQ8=","txInfo":{"frequency":903900000,"modulation":"LORA","loRaModulationInfo":{"bandwidth":125,"spreadingFactor":7,"codeRate":"4/5","polarizationInversion":false}},"rxInfo":{"gatewayID":"uCfr//5Qoik=","time":null,"timeSinceGPSEpoch":null,"rssi":-65,"loRaSNR":10.8,"channel":0,"rfChain":0,"board":0,"antenna":0,"location":null,"fineTimestampType":"NONE","context":"HHMJuw=="}}
'''
Bridge has publised just once on application topic with decrypted data and no more publishing.
'''
{"applicationID":"1","applicationName":"Invision","deviceName":"No","devEUI":"0000000000000000","rxInfo":[{"gatewayID":"b827ebfffe50a229","name":"Campos","time":"2019-06-11T14:28:11.512039902Z","rssi":-63,"loRaSNR":8.8,"location":{"latitude":-21.752006316798564,"longitude":-41.22344724933007,"altitude":0}}],"txInfo":{"frequency":903900000,"dr":3},"adr":true,"fCnt":0,"fPort":1,"data":"SGVsbG8sIHdvcmxkIQ=="}
''''

The LoRa Gateway Bridge does not publish to the application topic. This is the responsibility of the LoRa App Server and depends on the configured integrations. Please see https://www.loraserver.io/guides/troubleshooting/ for troubleshooting steps. As this might be related to encryption keys, I also suggest to search on https://forum.loraserver.io which contains a lot of questions / answers related encryption and / or MIC related issues.


Moonbeam WalletConnect  click connect button show Webpage not available

How to reproduce errors？
NovaWallet -> Dapps -> moonbeam -> WalletConnect -> OK button -> Connect button

Hey @iceqing ! Wallet Connect support is not available via Nova DApp browser. However, soon we will start integrating Metamask support for EVM-based chains


Resolve issue with latest openpyxl

See https://github.com/samwebster/times-excel-reader/commit/8adc5b0e3f61858f4b271fa4a939264e394d1949

Seems to be working with v3.1.1
Maybe a bug got fixed? https://openpyxl.readthedocs.io/en/stable/changes.html

@samwebster

Problem still seems to be there - https://github.com/samwebster/times-excel-reader/actions/runs/4276711710/jobs/7445005244

Seems to be this issue: https://foss.heptapod.net/openpyxl/openpyxl/-/issues/1967


I am commiting my photography logo.

Here is my pull request.

@hjl5363 I will wait until you do another push of a file on your own to your branch before I merge this pull request.

@hjl5363 You didn't make another push to your branch, but you did do this correctly with some help after class. Rather than waiting for another push,  I've merged in your branch and deleted it from the remote repo, so you should go ahead and delete your local branch. *You'll find instructions on how to do that in my slide deck (or video) on the git commands.
For the next homework assignment you'll create a new branch with a new purpose.
For the next homework assignment you'll create a new branch with a new purpose.


Remove dependency on external docker images

This "call home" (docker pull in https://github.com/elyra-ai/examples/blob/master/pipelines/dax_noaa_weather_data/analyze_NOAA_weather_data.pipeline#L17 of the "Pandas" docker image (https://hub.docker.com/r/amancevice/pandas/) from your Docker Hub account) will be unacceptable in any corporate security-restricted environment, where only security-approved docker containers and image registries are permitted (even if not running on completely air-gapped servers). In fact one should avoid any docker pull operations whatsoever (as this is what was assumed to work here), as these require root-level priviledges and will be unlikely to work (unless the notebook is run as root).
I might have missed it from reading the docs - please advise if there is a truly local option - running in the same docker container (supplied and approved by the corporation) where Jupyter Lab is being executed? I mean it is a false premise that this environment must be resource-poor. Jupyter client stays in a browser on a thin client machine, correct, but python kernel is nearly always run on the server-side, on a large compute node. No need to improve on this client-server arch that already works fine for individual Notebooks. Just let the user scripts (pipeline code payload) run on the same machine (but dedicated python kernel!) as the controlling notebook. This is how papermill works by the way (no dependency on external docker images).

@ptitzler thank you for addressing both our concerns!
So if the docker image running Jupyter and Elyra can be completely user-defined and generic (i.e. without any custom runtime applications) and can be scanned for vulnerabilities and perhaps even hosted in an internal image registry (like Red Hat Container Registry), then it looks like "problem solved" and just a documentation issue:) I will give it a solid test drive in our on-prem Openshift installation when the time permits (which is most likely next weekend:)

Absolutely! We do publish an official Elyra container image based on the files in https://github.com/elyra-ai/elyra/tree/master/etc/docker/elyra that you could use as a baseline. Feel free to open an issue if you do run into trouble getting an image to work or reach out on https://gitter.im/elyra-ai/community. @lresende, @akchinSTC  fyi

Opened https://github.com/elyra-ai/elyra/issues/1350 to improve the content for custom Elyra container images

Opened #1350 to improve the content for custom Elyra container images

Perfect, so I will report any problems I might encounter with the Elyra container under Openshift in this new issue.

I should probably mention that Elyra can be installed as part of Open Data Hub on Red Hat OpenShift: https://elyra.readthedocs.io/en/latest/recipes/deploying-elyra-with-opendatahub.html

I should probably mention that Elyra can be installed as part of Open Data Hub on Red Hat OpenShift: https://elyra.readthedocs.io/en/latest/recipes/deploying-elyra-with-opendatahub.html

Yes, I've heard about it from Red Hat people, but the only problem (as with Kubeflow on its own) is our obsolete Openshift installation (still running 3.11).

I turns out (my apologies) that container images built using https://github.com/elyra-ai/elyra/blob/master/etc/docker/elyra/Dockerfile won't work on RHOS. We've still got some work to do documenting how we are building the image for Open Data Hub.

how we are building the image for Open Data Hub.
I understand you completely, it's never been easy to port containerized apps from Docker to Openshift due to additional security considerations (Jupyter Notebook was I thing the only one working out of the box:)

I can't seem to find the "local" runtime option. Is it at all possible to use Elyra, but disable all containerization options and just use the available (Python) kernels on the host that runs JupyterLab?

I can't seem to find the "local" runtime option.

To use the local runtime you must create pipelines using the generic pipeline editor:

When you click "run" the option is displayed:

This option is unavailable in the Kubeflow Pipelines and Airflow pipeline editor.

Is it at all possible to use Elyra, but disable all containerization options and just use the available (Python) kernels on the host that runs JupyterLab?

Not currently. The options for Airflow and Kubeflow Pipelines are always displayed by the UI.


Is there a gitter or way to ask questions?

Hello!
Was giving this a go and wanted to ask some things, but found I couldn't access the gitter

Feel free to ask questions here.


Support skipsdist and usedevelop, pep8 cleanup

This PR makes changes to support skipsdist and usedevelop. In addition, it makes some updates to clean up the code using flake8.

@kendallc - sorry that this PR got completely stale, I am quickfixing some breaking bugs and make a new release, which will make things even worse.
If you are still using detox and are interested to pick up development (and cherry pick the features from here), please get in touch and we'll invite you to the org :heart:

I started extracting the non flake8 relevant bits into a new PR here: https://github.com/tox-dev/detox/pull/31 not quite done, but should be almost everything. Thanks for your work!

Closing this now. All relevant fixes have been ported and are now in master. Hope to get this fixed up and released with 0.16 soon. Thanks again @kendallc!


Multiple instances of SDK with different API Keys

Hello!
Stumbled upon an issue where if two (or more) instances of the OpenTok class are created with different api_keys set, requests from any instance use the api_key from the latest-instantiated object.
To reproduce, try out this snippet which instantiates two objects with different credentials (example credentials spoofed) and switches creating sessions between the two:
OTSDK     = OpenTok::OpenTok.new("1234", "1234")
OTSDK_TWO = OpenTok::OpenTok.new("9999", "9999")

def decode_opentok_session_id(tok)
  Base64.urlsafe_decode64(tok[2..tok.length])
end

sessions = []
5.times do |i|
  if i % 2 != 0
    session = OTSDK.create_session
  else
    session = OTSDK_TWO.create_session
  end

  sessions << session
end

sessions.each { |s| puts s.api_key }

sessions.each do |sesh|
  puts decode_opentok_session_id(sesh.session_id)
end

The output would be along the lines of:
1234
9999
1234
9999
1234

1~1234~xyz...
1~9999~zyx...
1~9999~zyx...
1~9999~zyx...
1~9999~zyx...

As you can see, the session objects returned have the correct api_key but the session_id from the actual API response doesn't -- this can/will result in the dreaded "API key-session mismatch" when generating a token.
At any rate, this is resolved in the latest (unreleased) version of the gem but is present in 2.3.4; I believe the issue lives in client.rb where the headers value is a class-variable rather than an instance variable. Since the latest code uses the instance's api_key variable when generating headers, it's no longer present. Given it's resolved (and we've found a workaround) I don't think a PR is necessary. Thanks for fixing in the latest version!

@Fishz No problem. We'll try to get this released as soon as we can.


Duplicate footerTemplate on ui-grid 3.0-RC21

Hi,
I found this closed issue #1956, but it's still duplicate when selection functionality is activated.
Here's a plunker with the bug:
http://plnkr.co/edit/AX7eKu4S54FqiJzCk00H?p=preview

Any news ?

It's showing once per render container, which is expected behaviour.  If you want the template to only show on the body render container, then you'd need to write some specific code to achieve that.
Something like: http://plnkr.co/edit/FPQonUnkpZgxsdsxXefl?p=preview

Okay, sorry for the misunderstanding of this behaviour.
Your hack is working fine for me, thanks!


implement normalize_token

See https://github.com/pydata/xarray/pull/3276#discussion_r332113438

How should this be implemented?

https://docs.dask.org/en/latest/custom-collections.html#implementing-deterministic-hashing
@normalize_token.register(Dataset)
def tokenize_dataset(ds):
    return (Dataset, ds._variables, ds._coord_names, ds._attrs)

@normalize_token.register(DataArray)
def tokenize_dataarray(da):
    return (DataArray, ds._variable, ds._coords, ds._name)

@normalize_token.register(Variable)
def tokenize_variable(v):
    # Note: it's v.data, not v._data, in order to cope with the
    # wrappers around NetCDF and the like
    return (Variable, v._dims, v.data, v._attrs)

# This is only a minor optimization: don't convert pd.Index to np.ndarray
@normalize_token.register(IndexVariable)
def tokenize_indexvariable(v):
    return (IndexVariable, v._dims, v._data.array, v._attrs)

You'll need to write a dummy normalize_token for when dask is not installed.
Unit tests:

running tokenize() twice on the same object returns the same result
changing the content of a data_var (or the variable, for DataArray) changes the output
changing the content of a coord changes the output
attrs, name, and dimension names change the output
whether a variable is a data_var or a coord changes the output
dask arrays aren't computed
non-numpy, non-dask NEP18 data is not converted to numpy
works with xarray's fancy wrappers around NetCDF and the like


chore: Rename Finset.inter_sdiff to Finset.inter_sdiff_assoc

... and turn it around. This matches Set.inter_diff_assoc.

This matches Set.inter_sdiff_assoc

No, it doesn't, since Set.inter_sdiff_assoc does not exist; it's called Set.inter_diff_assoc (without the s). Is there a good reason for not making the Finset lemma actually match the Set one?

Note that the operator is called sdiff but all Set lemmas call it diff. So, yes, it matches the Set lemma, up to a naming issue that's out of scope for this PR.

So, yes, it matches the Set lemma, up to a naming issue that's out of scope for this PR

This seems like a slightly awkward argument to justify. There was a naming inconsistency about this lemma before; there'll still be a naming inconsistency afterwards; and it's not obvious to me that the new inconsistency is strictly less severe than the old one – they're both pretty trivial imho. So does the PR actually achieve anything?

No, the new lemma name is perfectly consistent with the existing (imperfect) naming convention, namely that the sdiff lemmas for Set call it diff while the Finset lemmas call it sdiff. See eg Set.diff_self vs Finset.sdiff_self.


chore(billing): Add materialized views that will make it easier to do billing calculations

Description
Add materialized views that provide usage stats for our major tables.
Testing
How was this PR tested?
It hasn't, yet :)

@tssweeney Finally got around to testing this -- all working ok.


Vulnerable to Infinite Loop via malformed MKV file through file-type package

The following report is provided by NPM audit when using the latest version of decompress package (v4.2.1):
file-type  <16.5.4
Severity: moderate
file-type vulnerable to Infinite Loop via malformed MKV file - https://github.com/advisories/GHSA-mhxj-85r3-2x55

node_modules/decompress-tar/node_modules/file-type
node_modules/decompress-tarbz2/node_modules/file-type
node_modules/decompress-targz/node_modules/file-type
node_modules/decompress-unzip/node_modules/file-type
  decompress-tar  >=4.0.0

  Depends on vulnerable versions of file-type
  node_modules/decompress-tar
  decompress-tarbz2  >=4.0.0

  Depends on vulnerable versions of file-type
  node_modules/decompress-tarbz2
    decompress  >=4.0.0
    Depends on vulnerable versions of decompress-tarbz2
    node_modules/decompress

  decompress-targz  >=4.0.0
  Depends on vulnerable versions of file-type
  node_modules/decompress-targz

  decompress-unzip  >=4.0.1
  Depends on vulnerable versions of file-type
  node_modules/decompress-unzip

More description about the problem is provided here: https://github.com/advisories/GHSA-mhxj-85r3-2x55
The fix was implemented in file-type v16.5.4, so probably file-type should just be bumped in sub-packages: decompress-tar, decompress-tarbz2, decompress-targz, decompress-unzip.

@sindresorhus any chance you could give some security love to this package? <3

@sindresorhus @kevva : When can we expect the file-type vulnerability fix?
https://github.com/advisories/GHSA-mhxj-85r3-2x55.
https://nvd.nist.gov/vuln/detail/CVE-2022-36313
<EMAIL_ADDRESS>└─┬<EMAIL_ADDRESS>└─┬<EMAIL_ADDRESS>└──<EMAIL_ADDRESS>

@qooban @alfaproject : How do you resolve this issue?

@UdayKumarNettem I don't have a solution for that. I provided my findings in the issue description.


epochs.py raises error with argument proj=False

Hello,
On epochs.py line 682, inside class Epochs:
proj = proj or raw.proj  # proj is on when applied in Raw
Here, proj will become None if raw.proj==None.
It's because in Python, (None or False) == False, whereas (False or None) == None.
I believe it's confusing anyway to compare a None type value with a boolean value.
Sometimes, I have raw arrays having raw.proj==None, and it crashes the program by raising an error after the following line:
if proj not in [True, 'delayed', False]:
So for my code, I just added the following line to fix:
if proj == None: proj= False
after each line of "proj = proj or raw.proj". There were only 2 such lines affected.

can you share full code snippet to reproduce the error?
is it still failing on current master?
thx

Hello, after re-installing, I could confirm that this error does not happen in the master.
I think I was using an old version (0.8.x). Sorry for that.

No problem, thanks for checking! Let us know if you hit any other problems.


Add scanrLazy for lazy lists

Thanks!

Thanks for merging @garyb. I was unsure if I should be exposing a new function named scanrLazy or be removing the re-exported scanr from Data.Traversable and implementing this as scanr.
The file seems inconsistent (e.g. filter vs. foldrLazy). Thoughts?

Yeah, I think it's a bit of a mess in there at the moment, perhaps next time we make a breaking release we can clean things up a bit!


[Examples] update core input mouse

Added list of colours and the mouse button on-screen for people to easier try

@CrackedPixel I prefer to avoid showing that much text...


Stateless Cover open/close issue in Google Assistant Cloud Component

Home Assistant release with the issue:
0.91.0b4
Last working Home Assistant release (if known):
N/A
Operating environment (Hass.io/Docker/Windows/etc.):
Debian Stretch
Component/platform:
Cloud (Covers) - Google Assistant
Description of problem:
Covers without state tracking do not work with the new Cover Voice Commands; (Integrated into Google Assistant - Cloud Component).
I can control my Garage Door which has state tracking by simply stating "open garage door" and "close garage door". However I am unable to control my curtain which does not track state.
I can close the curtain my stating "close curtain", however after it is closed, requesting "open curtain" does not work.
I believe this is because google thinks the curtain is already open. As asking Google "Is the curtain open?" (when it is closed) responds with "Yes, the curtain is open". Thus it does not submit the open curtain command.
One way to overcome this would be to submit some pseudo state, but as the curtain can be operated manually, this will have sync issues. So ideally there needs to a way to both open and close a cover in a stateless manner. Is this possible with the API Google Provides?
Problem-relevant configuration.yaml entries:

- platform: mqtt
    name: "Bedroom Curtain"
    command_topic: "CurtainControl"
    state_topic: "CurtainState"
    qos: 0
    retain: false
    payload_open: "UP"
    payload_close: "DOWN"
    optimistic: false

Additional information:
The cover works in Lovelace interface as expected without requiring state tracking.

This is not an issue that the Google Assistant integration should solve. This is why there is a state topic to tell Home Assistant when the state changes, or if that's not possible, use optimistic so Home Assistant will assume the command was successful and adjust the state accordingly.

(for reference quoted via discord disccussion)

This doesn't solve the issue though, although it now optimistically tracks the state received from Home Assistant.

It doesn't work if I manually activate the blinds and the state changes independent of Home Assistant.

I want 'open curtains' to always submit "open" mqtt command and 'close curtains' to always submit "close" mqtt command. Basically I don't want any state to exist at all.

After some more Discord discussion, the issue seems to be that Google won't issue an open command if it queries the state and the state of the device is already open, even if that state is no longer the real device state.


[Feature]: Helm chart wish list

Is there an existing issue for this?

[X] I have searched the existing issues

Summary
Helm charts should not assume "high availability" is desired.  We deploy to kubernetes for dev and production.  For dev, we keep it simple, whereas for production, we add replicas, etc.  Kubernetes is just a deployment platform that has flexibility and elasticity.
We had to switch back to deploying the docker-compose all-in-one on ec2, as the helm chart was not ready for prime time.
Here are my wish lists for the helm chart:

The helm chart should ideally deploy a Deployment, not a StatefulSet.  There's rarely a need to store data using local store when we have databases.  We run our databases/redis on ec2, external to kubernetes.  StatefulSets add complexity to managing kubernetes.
Do not assume that the customer wants to run a HorizontalPodAutoscaler just because they want to run as a Deployment.  Give the option to turn autoscaling on and configure the autoscaler separately from running as a Deployment.
Make sure that it works with external database / redis.  There's no need to have multiple databases (mongo and postgres), that adds complexity to backup and maintenance routines.  Ideally, use redis for session management, database for storage, and these are run externally.  (As a side note, the app should also not generally care about the version of database or redis, otherwise the customer has to run multiple database servers and redis servers to support their multiple apps.)
Either the credentials are in the database URL or they are not.  Don't have a username/password config and also expect username password to be in the URL.  Stick to one or the other.  Don't try to parse a database URL to get credentials!
Don't force the database to be run in a specific "high availability" mode (i.e. MongoDB replicaset as an example).
Allow extraVolumes and extraVolumeMounts.  This allows for things like "secret store csi driver" to work, even if the mounted files aren't used for anything.
Allow the configuration to come from existing (external to helm-chart) secrets and config maps using the same flexibility as envFrom on a Deployment.  I should be able to get some settings from one secret, others from another, choose the key to get it from, and also set some of the settings directly in values.yaml.    We use GitOps and do not wish to store secrets in the values.yaml, and we use the secrets store csi driver to get other secrets.
Do not assume that the user uses a specific technology like "ExternalSecrets".
Make sure that technologies like KeyCloak do not add "yet another database technology" to the stack and it just "works" once you setup the optional SSO options in values.yaml
Make sure error messages are easy to figure out.  It isn't obvious that something couldn't log into the database.  I see a lot of noise and Temporal messages that are confusing, then the pod just shuts down.

Why should this be worked on?
The helm chart is basically unusable by Enterprises

+1


Translate content/ru/docs/concepts/architecture/controller.md to russian

Hello
Translate
content/ru/docs/concepts/architecture/cloud-controller.md
content/ru/docs/concepts/architecture/controller.md
to russian

/assign @msheldyakov

/retest

/retest
/lgtm
/approve

/lgtm

Hello. After edit my translate files, run checks  and show error in the cloud-controller.md file. How i can fix my error ?
2:53:43 PM: Error: Error building site: "/opt/build/repo/content/ru/docs/concepts/architecture/cloud-controller.md:13:1": failed to render shortcode "glossary_definition": failed to process shortcode: "/opt/build/repo/layouts/shortcodes/glossary_definition.html:18:19": execute of template failed: template: shortcodes/glossary_definition.html:18:19: executing "shortcodes/glossary_definition.html" at <$term_info.Content>: nil pointer evaluating resource.Resource.Content 2:53:43 PM: Makefile:27: recipe for target 'build' failed

@mshalmanov There are two errors
5:06:09 PM: ERROR 2021/05/25 11:36:09 [ru] "docs/concepts/architecture/cloud-controller.md": "cloud-controller-manager" is not a valid glossary term_id, see ./docs/reference/glossary/* for a full list
ERROR 2021/05/25 11:36:09 [ru] no Resource or Page matching "/static/community-values.md*".

The first -
5:06:09 PM: ERROR 2021/05/25 11:36:09 [ru] "docs/concepts/architecture/cloud-controller.md": "cloud-controller-manager" is not a valid glossary term_id, see ./docs/reference/glossary/* for a full list

This is related to this failure -
5:06:09 PM: Error: Error building site: "/opt/build/repo/content/ru/docs/concepts/architecture/cloud-controller.md:13:1": failed to render shortcode "glossary_definition": failed to process shortcode: "/opt/build/repo/layouts/shortcodes/glossary_definition.html:18:19": execute of template failed: template: shortcodes/glossary_definition.html:18:19: executing "shortcodes/glossary_definition.html" at <$term_info.Content>: nil pointer evaluating resource.Resource.Content

This is because you are trying to refer to a glossary definition for the term cloud-controller-manager but there's no definition for that.
Example - https://kubernetes.io/docs/reference/glossary/?fundamental=true - look for Controller definition in English. This definition is located at content/en/docs/reference/glossary/controller.md
For cloud-controller-manager glossary definition, you need to define content/ru/docs/reference/glossary/cloud-controller-manager.md and for an example you can look at content/en/docs/reference/glossary/cloud-controller-manager.md
The second error -
ERROR 2021/05/25 11:36:09 [ru] no Resource or Page matching "/static/community-values.md*".

This is related to the community values page as the error says. In content/ru/community/values.md, you refer to /static/community-values.md but there's no such page for Russian language. Example for English language is - content/en/community/values.md and the /static/community-values.md page is -> content/en/community/static/community-values.md. So, for Russian language, you have to create something at content/ru/community/static/community-values.md
With that, all the errors should be gone :)
So, in short, you basically need to define these two markdown files -
content/ru/community/static/community-values.md
content/ru/docs/reference/glossary/cloud-controller-manager.md
Start by creating the above two files as simply empty files and see if git submodule update --init --recursive --depth 1 && make production-build works with no errors and then start writing content :)

/lgtm
/approve

/label tide/merge-method-squash

According to @karuppiah7890 's comment,
@mshalmanov you added content/ru/docs/reference/glossary/cloud-controller-manager.md file in the commit e37b083,
but the file content/ru/community/static/community-values.md is not added yet.
@mshalmanov Could you PTAL again?
Thanks!

/assign @msheldyakov

спасибо!
/approve


use new v6 URL for rss

We can probably craft these manually again, will try that later (vs calling get)

The V6 RSS feeds seem to only work about 10% of the time in VLC and any other media player.
I have old playlists with the V3 URLs and they work perfectly.
The streams with the V6 URLS (https://player.odycdn.com/v6/streams/...) either drop out while playing or don't work at all.
The streams with the V3 URLs (https://player.odycdn.com/api/v3/streams/free/...) work perfectly 100% of the time even when seeking the videos to random positions.
I tried several podcast players, GPodder, Grover, VLC and the RSS based videos either don't play or disconnect.
After the video stops playing, it can't be restarted again.
Why bother with the change to V6?

Hmm, we haven't heard other complaints about this yet. They're using the same exact service to provide the data, the only difference is the syntax of the URL. A common v6 URL between how we stream directly on Odysee and via other services like RSS gives us the best caching scenario - otherwise v3 would need to get cached separately by the CDN.
We'll give this a test...

Appreciate the heads up!


Sprout processing

Need to clarify how source files will be added to the working directory if using github workflows.

Will resolve #6 when merged

Closes #6


updates to daprdocs, concepts, getting-started index pages

Thank you for helping make the Dapr documentation better!
Please follow this checklist before submitting:

[X] Read the contribution guide
[X] Commands include options for Linux, MacOS, and Windows within codetabs
[X] New file and folder names are globally unique
[X] Page references use shortcodes instead of markdown or URL links
[X] Images use HTML style and have alternative text
[X] Places where multiple code/command options are given have codetabs

In addition, please fill out the following to help reviewers understand this pull request:
Description
updated daprdocs, concepts, getting-started index pages to improve readability:

https://docs.dapr.io/ - added a curated landing page experience, i.e. organize links into reasonable sections that flow for the reader.
https://docs.dapr.io/concepts/ - added alert style messaging to cross link into getting-started to direct readers to hands on learning
https://docs.dapr.io/getting-started/ - included an action button in the alert
Added new Java logo to ensure text alignment with other SDK cards

Issue reference
https://github.com/dapr/docs/issues/1805

built and verified website locally already


Finance: Check is now Mint Bills

In December 2014, Check was acquired by Intuit and renamed Mint Bills. This PR updates the site to reflect that change:

Check Inc. is now "Mint Bills (formerly Check)" and reflects the new Twitter and URL
Logos for both "Mint" and "Mint Bills (formerly Check)" have been changed to reflect the new Mint logo after their recent rebranding

It's not critical, but Mint and Mint Bills have different logos. :)
Mint's logo has blue and green stripes while Mint Bill's logo has blue and yellow stripe.


Bug: Front end small issues

Feedback from the presentation:

Responsive for IPhone: When looking at your own timetable program, the website showed not right
Pleas no zoom on hover effect