CLI extension: create --help: --name argument: should say "Name of the Azure Spring Cloud Service instance".

Consider this command
az spring-cloud create --name "ejb0831ascs01svc" -g "ejb0831ascs01"

Consider this corresponding help output.
az spring-cloud create --help
This command is from the following extension: spring-cloud

Command
    az spring-cloud create : Create an Azure Spring Cloud.

Arguments
    --name -n                               [Required] : Name of Azure Spring Cloud.

The above sub-command creates a service instance.
Now we have this command to create an app, like this:
az spring-cloud app create --name "ejb0831ascs01app" -s "ejb0831ascs01svc" -g "ejb0831ascs01" --is-public true --runtime-version Java_11 --assign-identity

The help for this sub-command is
az spring-cloud app create --help
This command is from the following extension: spring-cloud

Command
    az spring-cloud app create : Create a new app with a default deployment in the Azure Spring
    Cloud.

Arguments
    --name -n        [Required] : Name of app.

Note that both spring-cloud create and spring-cloud app create have
a --name option, but its meaning is different between the two
contexts.  In the former, --name means "The name of the Azure Spring
Cloud Service Instance".  In the latter, --name means "The name of the
Azure Spring Cloud App within the service specified with the --service
option.
I am not suggesting we change this.  It is too late for such a change now.  But we should at least change the
help.  Here is what I think the help for these commands should look
like, regarding the --name option.
az spring-cloud create --help
This command is from the following extension: spring-cloud

Command
    az spring-cloud create : Create an Azure Spring Cloud.

Arguments
    --name -n                               [Required] : The name of the Azure Spring Cloud Service Instance

az spring-cloud app create --help
This command is from the following extension: spring-cloud

Command
    az spring-cloud app create : Create a new app with a default deployment in the Azure Spring
    Cloud.

Arguments
    --name -n        [Required] : The name of app running within the specified service.

I'm good with the suggested change in the description.

Agree with the point, a little more specifying helper text would be useful.
A little more consistency could be:
az spring-cloud create --help
This command is from the following extension: spring-cloud

Command
    az spring-cloud create : Create an Azure Spring Cloud instance.

Arguments
    --name -n                               [Required] : The name of Azure Spring Cloud instance

az spring-cloud app create --help
This command is from the following extension: spring-cloud

Command
    az spring-cloud app create : Create a new app with a default deployment in the Azure Spring
    Cloud instance.

Arguments
    --name -n        [Required] : The name of app running within the specified Azure Spring Cloud instance.

But in general, i agree with @edburns it is confusing the command between 'az spring-cloud create' and 'az spring-cloud app create'. Is there any better word or any other way to improve the clarity in general?

The command helper text has been refined. PR has been linked above.
az spring create --help
This command is from the following extension: spring

Command
    az spring create : Create an Azure Spring Apps instance.

Arguments
    --name -n                                        [Required] : The name of Azure Spring Apps
                                                                  instance.

az spring app create --help
This command is from the following extension: spring

Command
    az spring app create : Create a new app with a default deployment in the Azure Spring Apps
    instance.

Arguments
    --name -n                                        [Required] : The name of app running in the
                                                                  specified Azure Spring Apps
                                                                  instance.

Will close the issue. Please let me know if there if any other question. Thanks a lot!


bcbio upgrade error

Hi,
I found the following error during bcbio-nextgen upgrade. Can you help me to solve it?
Thanks.
Creating manifest of installed packages in ./bcbio/manifest
Third party tools upgrade complete.
Installing additional tools
Installing gatk
Upgrading bcbio-nextgen data files
Setting up virtual machine
[localhost] local: echo $HOME
[localhost] local: uname -m
Running GGD recipe: dbnsfp
Traceback (most recent call last):
File "./bcbio/bin/bcbio_nextgen.py", line 4, in
import('pkg_resources').run_script('bcbio-nextgen==0.9.9', 'bcbio_nextgen.py')
File "build/bdist.linux-x86_64/egg/pkg_resources/init.py", line 726, in run_script
File "build/bdist.linux-x86_64/egg/pkg_resources/init.py", line 1484, in run_script
File "./bcbio/anaconda/lib/python2.7/site-packages/bcbio_nextgen-0.9.9-py2.7.egg-info/scripts/bcbio_nextgen.py", line 207, in
install.upgrade_bcbio(kwargs["args"])
File "./bcbio/anaconda/lib/python2.7/site-packages/bcbio/install.py", line 93, in upgrade_bcbio
upgrade_bcbio_data(args, REMOTES)
File "./bcbio/anaconda/lib/python2.7/site-packages/bcbio/install.py", line 283, in upgrade_bcbio_data
cbl_deploy.deploy(s)
File "./bcbio/tmpbcbio-install/cloudbiolinux/cloudbio/deploy/init.py", line 65, in deploy
_setup_vm(options, vm_launcher, actions)
File "./bcbio/tmpbcbio-install/cloudbiolinux/cloudbio/deploy/init.py", line 110, in _setup_vm
configure_instance(options, actions)
File "./bcbio/tmpbcbio-install/cloudbiolinux/cloudbio/deploy/init.py", line 268, in configure_instance
setup_biodata(options)
File "./bcbio/tmpbcbio-install/cloudbiolinux/cloudbio/deploy/init.py", line 250, in setup_biodata
install_proc(options["genomes"], ["ggd", "s3", "raw"])
File "./bcbio/tmpbcbio-install/cloudbiolinux/cloudbio/biodata/genomes.py", line 344, in install_data
_prep_genomes(env, genomes, genome_indexes, ready_approaches)
File "./bcbio/tmpbcbio-install/cloudbiolinux/cloudbio/biodata/genomes.py", line 473, in _prep_genomes
retrieve_fn(env, manager, gid, idx)
File "./bcbio/tmpbcbio-install/cloudbiolinux/cloudbio/biodata/genomes.py", line 812, in _install_with_ggd
ggd.install_recipe(env.cwd, recipe_file)
File "./bcbio/tmpbcbio-install/cloudbiolinux/cloudbio/biodata/ggd.py", line 30, in install_recipe
recipe["recipe"]["full"]["recipe_type"])
File "./bcbio/tmpbcbio-install/cloudbiolinux/cloudbio/biodata/ggd.py", line 62, in _run_recipe
subprocess.check_output(["bash", run_file])
File "./bcbio/anaconda/lib/python2.7/subprocess.py", line 574, in check_output
raise CalledProcessError(retcode, cmd, output=output)
subprocess.CalledProcessError: Command '['bash', './bcbio/genomes/Hsapiens/GRCh37/txtmp/ggd-run.sh']' returned non-zero exit status 1

Can anybody help me to solve the problem? Thanks!

Thanks, it looks like the dbnsfp recipe isn't working; sometimes external dependencies are down which causes errors like these. If you rerun it does it give the same error? If so is there more information in the error output anywhere?

Sorry about the issue. This is a problem with dbNSFP and I believe is the problem described here:
https://github.com/chapmanb/cloudbiolinux/issues/231#issuecomment-256566663
This is due to some fixes we tried to put in for the new version on hg19. I'm trying to fix that now and will report back as soon as we've resolved the issue.

Hi Brad,
Should I get the older version dbNSFPv2.9.1 or wait for your fixing? If I need to get the older version dbNSFPv2.9.1, what are the specific steps to get the upgrade successful.
Thanks!

Thanks for the patience and sorry again about the issue. I pushed a fix that I hope will resolve this. If you rm -rf tmpbcbio-install and re-run it should get the latest recipe and hopefully work cleanly for you.

Brad,
I ran rm -rf tmpbcbio-install and the folder and files are removed but when I ran the upgrade command as follows,
bcbio_nextgen.py upgrade -u stable --tools --data --datatarget gemini --datatarget cadd --datatarget vep --datatarget dbnsfp --datatarget battenberg --toolplus gatk=/user/GenomeAnalysisTK-3.6.tar/GenomeAnalysisTK.jar
I found the same error. Any suggestion?
Thanks.
Running GGD recipe: dbnsfp
% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
100   383    0   383    0     0     71      0 --:--:--  0:00:05 --:--:--  1040
100 12.4G    0 12.4G    0     0  31.6M      0 --:--:--  0:06:43 --:--:-- 50.0M
[E::hts_idx_push] unsorted positions
tbx_index_build failed: dbNSFP.txt.gz

Sorry about the issue, you probably need to remove the older problematic build from the temporary directory. You could do with: rm /path/to/bcbio/genomes/Hsapiens/GRCh37/txtmp/variation/dbNSFP.txt.gz which should allow it to be rebuilt with the new version. Hope this fixes it for you.


[Windows] Update miniconda download link

Description
In 2018 repo.continuum.io was replaced with repo.anaconda.com (https://github.com/conda/conda/issues/6886). Recently old repo have been finally shut down and it causes image generation to fail. This updates repo URL in Miniconda installation script.
Related issue: -
Check list

[ ] Related issue / work item is attached
[ ] Tests are written (if applicable)
[ ] Documentation is updated (if applicable)
[ ] Changes are tested and related VM images are successfully generated

/azp run windows2019,windows2022


[Protocol] MVP implementation off the on-chain SubmitProof message handling

Objective
MVP Implementation of SubmitProof to enable Shannon TestNet.
Origin Document

#44

Goals

Enable E2E validation of the entire protocol
Unblock iterative development of the claim & proof lifecycle
Unblock iterative development of on-chain tokenomics

Deliverables

[ ] A single PR that introduces the business logic for the on-chain SubmitProof message (verification, etc...)
[x] A single PR that updates the CLI to submit SubmitProof messages
[ ] Introduce make targets to enable development (visibility, triggering, debugging , etc...) of on-chain proofs
[ ] Add extensive unit tests

Non-goals / Non-deliverables

Account Management (i.e. Burn & Mint) business logic related to the Claim & Proof lifecycle (leave TODOs)
Governance Params necessary for the Claim & Proof lifecycle (leave TODOs)

General deliverables

[ ] Comments: Add/update TODOs and comments alongside the source code so it is easier to follow.
[ ] Testing: Add new tests (unit and/or E2E) to the test suite.
[ ] Makefile: Add new targets to the Makefile to make the new functionality easier to use.
[ ] Documentation: Update architectural or development READMEs; use mermaid diagrams where appropriate.

Creator: @Olshansk
Co-Owners: @bryanchriswhite

@Olshansk I'm starting on; got a happy path unit test already.

A single PR that introduces the business logic for the on-chain SubmitProof message (verification, etc...)

@bryanchriswhite Have you (planning to, considering, open-to) implemented the equivalent of #148 and #151 for proofs?
I created #243 for the work we discussed today and will drive that myself but wanted to know if the above is on your radar.

@bryanchriswhite Have you (planning to, considering, open-to) implemented the equivalent of #148 and #151 for proofs?

Yessir! The starting point I was referring to is analogous to #148 + #236 - testutils (which I will endeavor to isolate as reasonable). I think this is the best place to start as it's the most "outside", allowing me to work outside-in. Parts of the #151 analogue may be driven out along the way but otherwise I think it will be quite natural to lay out more isolated PRs for the various pieces.

I created #243 for the work we discussed today and will drive that myself but wanted to know if the above is on your radar.

:+1: :pray:


Make sure profile header URL has http:// in href

We don't want to show http:// in the UI, but the href needs
it. The bug was caused by the assumption that http:// would
always be there. The fix is just to remove http:// and
then add it back in the href so it will always be there.

Added a parser and validator for the URL.

Thought about this, but ... If someone adds a link to say, their personal website, and it's not https and they don't have SSL, not sure I should be forcing https.

Ahhh ok that's fine then.


Add _includes directory for quickly adding automatic snippets

The way I'm thinking is to have a directory where you can just put partials to get them automatically included. For example, want to add some stuff to the <head> section? Just create a file _includes/_head.html/md/blade.php. The contents will be rendered and/or injected before the end of the closing </head> tag.

https://github.com/hydephp/framework/issues/442#issuecomment-1133936888

Am also considering handling the creation of required directories in the app bootstrapping file
Edit: Or we just skip this all together and handle it with .gitkeeps

https://github.com/hydephp/framework/issues/442#issuecomment-1149030435

I do like this idea, but I also don't want a bunch of bloated directories. Could be something to integrate with data collections? Shelving for now, would love to get some feedback on this.

The complexity this adds should be weighed against the relative ease of publishing or modifying components in the resources/views directory

Created this class, which works pretty nicely, but I'm not sure if it adds enough value to be justified. Going to close this for now.
class RendersIncludedFile
{
    public static string $includesPath = '_includes';

    public static function render(string $file, ?string $default = null): string
    {
        $filepath = Hyde::path(static::$includesPath . '/' . $file);

        if (! file_exists($filepath)) {
            if ($default === null) {
                throw new \Exception("Included file '$filepath' does not exist, and no default value was provided.");
            }
            return $default;
        }

        $extension = pathinfo($filepath, PATHINFO_EXTENSION);

        return match ($extension) {
            'md' => MarkdownConverter::parse(file_get_contents($filepath)),
            'blade.php' => Blade::render($filepath),
            'php' => include $filepath,
            'html' => file_get_contents($filepath),
            'css' => '<style>' . file_get_contents($filepath) . '</style>',
            'js' => '<script>' . file_get_contents($filepath) . '</script>',
            default => throw new \Exception("Unsupported file extension for includes: $extension"),
        };
    }
}

Which can then be used in Blade views through the Hyde facade
class Hyde
{
    public static function include(string $file, ?string $default = null): string
    {
        return RendersIncludedFile::render($file, $default);
    }
}

Thinking about adding sometime simple in the resources directory. Starting small, and we can expand from there.

Okay, I think this contract could be nice. It's rather simple and doesn't have any of the magic the previous implementation had. A Hyde::include() helper could use the automatic filetype detection.
interface IncludeFacadeContract
{
    /**
     * Get the raw contents of a partial file in the includes directory.
     *
     * @param string $partial The name of the partial file, including the extension.
     * @param string|null $default The default value to return if the partial is not found.
     * @return string|null The contents of the partial file, or the default value if not found.
     */
    public static function get(string $partial, ?string $default = null): ?string;

    /**
     * Get the rendered Markdown of a partial file in the includes directory.
     *
     * @param string $partial The name of the partial file, without the extension.
     * @param string|null $default The default value to return if the partial is not found.
     * @return string|null The contents of the partial file, or the default value if not found.
     */
    public static function markdown(string $partial, ?string $default = null): ?string;

    /**
     * Get the rendered Blade of a partial file in the includes directory.
     *
     * @param string $partial The name of the partial file, without the extension.
     * @param string|null $default The default value to return if the partial is not found.
     * @return string|null The contents of the partial file, or the default value if not found.
     */
    public static function blade(string $partial, ?string $default = null): ?string;
}

Could be neat with an Includes::code() helper to render Torchlight blocks?


Question about removing a broken node from group

A group's peers remain unchanged after initialization.(So consistenthash package doesn't provide Remove method) But node may become unavailable because of network, its hardware or something else.
Why groupcache doesn't support removing a broken node?
I guess the reason is that another node could reuse broken one's network address(ip:port), and replace it. But I figured I would ask just in case I'm wrong.
Thanks in advance.

I'm pretty sure it does. If you're using the HTTPPool you can call Set(peers...) to update the peer list.
https://godoc.org/github.com/golang/groupcache#HTTPPool.Set

@d4l3k HTTPPool.Set creates a new hash ring.  It's the reason why consistenthash package doesn't provide Remove methond. Thanks a lot.

Ah, interesting, I didn't realize that. I'm not affiliated with the groupcache package at all just saw the issue.


chore(ci): Switch to Coveralls

Coveralls has better support for recording and aggregating parallel test runs.
Also adds a filter to leave out generated code, test support code and, non-critical and untestable utility code like logging and metrics from test coverage reporting to reduce the noise.
Signed-off-by: Charith Ellawala<EMAIL_ADDRESS>

Yeah, I noticed that too. I am hoping that the E2E test run would fix that tonight.

Oh, actually, E2E doesn't run all the unit tests so the reported number would be quite low 😞


REQUEST: @salaboy as approver of dapr/java-sdk

GitHub Username
@salaboy as an approver in dapr/java-sdk
List of contributions
TestContainers and SpringBoot integration added to Java SDK.

PRs:
https://github.com/dapr/java-sdk/pulls?q=is%3Apr+author%3Asalaboy+is%3Aclosed
Reviews:
https://github.com/dapr/java-sdk/pulls?q=is%3Apr+-author%3Asalaboy+salaboy

Nomination
Nominated by @artursouza. @salaboy  has authored and reviewed some relevant PRs for Java-SDK, and has been a core contributor to the Java SDK overall quality improvements.

+1 binding

+1 Non-binding

+1

Completed and closing


Optional Clear Button

I like how lightweight your multi select component is, the only additional functionality I would like to see is the option to remove the clear button that appears when selections are made. It seems like a simple capability to add, I would be happy to update the project with this feature.

I must have missed this, I see now that setting "ClearSelectedIcon={<></>}" will do just this.

Using ClearSelectedIcon={<></>} still results in a "Clear Selected" button being added to the DOM, but with no child elements. This might look fine, but it's no good for keyboard accessibility. Users can still tab to the button but will get no visual indication of it.
It'd be better if we could completely opt out of the clear button.

@nickmcmillan I guess you are right I'll see if we can just remove it completely

@nickmcmillan README.md is updated new documentation will be available on NPM in few minutes

@nickmcmillan have pushed new update updates will be available in few minutes


Building Rancher server image when "http://gondor.apana.org.au" cannot be reached

As of the time of this issue being posted, I've attempted to build the rancher server image by running the build-image.sh script inside the server directory of this repo, in order to build a custom version of the rancher server. Unfortunately the build always stops with this error:
curl -R -L -O http://gondor.apana.org.au/~herbert/dash/files/dash-0.5.8.tar.gz
curl: (7) Failed to connect to gondor.apana.org.au port 80: Connection refused
This is probably due to the site in question being currently offline. As a result, the server image cannot be built this way at this time.

We can't do anything on a host being down, maybe change the source where we built from. I'm still curious what commit/branch you were trying to build here which came up with this error?


all homebridge devices "no response" - one of two tokens wrong

Hi @afharo,
you know I'm not a programmer. Unfortunately, since you made the great changes, I no longer understand the code and cannot solve any problems. :-D
I have two roborocks in my config.json.
One of two tokens is wrong - since I reset the robot, due to a faulty firmware update. But I only get to read the token again the day after tomorrow.
The problem is, however, that all other devices on the homebridge are no longer accessible.

Probably a bug?
[2020-1-13 9:41:28] [Staubsauger EG Flur] ERR connect | miio.device, next try in 2 minutes | Error: Could not connect to device, token might be wrong
 (node:1597) UnhandledPromiseRejectionWarning: Error: Could not connect to device, token might be wrong
  at /usr/lib/node_modules/homebridge-xiaomi-roborock-vacuum/node_modules/miio/lib/network.js:329:16
  at async XiaomiRoborockVacuum.initializeDevice (/usr/lib/node_modules/homebridge-xiaomi-roborock-vacuum/index.js:336:20)
  at async XiaomiRoborockVacuum.connect (/usr/lib/node_modules/homebridge-xiaomi-roborock-vacuum/index.js:404:7)
(node:1597) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 3)

Thank you

Let me have a look at it this evening or tomorrow (I'll be home alone) :)

So to be clear, in order to reproduce it. If I set a second instance of the robot in config.json with a wrong token, I will see all the devices connected via homebridge failing (not just the faulty device)?

So to be clear, in order to reproduce it. If I set a second instance of the robot in config.json with a wrong token, I will see all the devices connected via homebridge failing (not just the faulty device)?

Exactly!


Event fixes

We get lots of reports about the editor working slowly in Angular and after checking it further it appears the problem comes from the angular-froala binding to each keyup event to get the editor content and update model. Instead, it should do that only on contentChanged event.

The events should be set before initializing the editor to make sure we catch the initialized event all the time. This would also fix the issue #59.

Update README for calling events. #64.

@cgallarno @andrewattellwise @Whyves would love to hear your thoughts before merging this.

@stefanneculai The reason behind the keyup event is that I believe many angular dev expect the model update in 'realtime', however with the editor I find that's not case as I'm not editing while also displaying the content, so I see no issue with this. However it seems that inserting a table doesn't fire the contentChanged event, which means the model isn't updated with the table until another action is performed.

That table insert issue of not calling contentChanged events was fixed recently. I will merge this for now and see what's the feedback from developers.

I think that switching to contentChanged event is a good thing. However, the way you implemented it introduces a breaking change in the behavior of the application. It may have side effects.
A more appropriate way for doing the change would have been to keep the "keyup" event but wrap it in an activation option. This way people can still have the same behavior as before but for optimization purpose you can turn the "keyup" event off via the option. In the next major version of angular-froala, you could have set the option to false by default but at least you warn people that breaking changes have been introduced.

@Whyves I believe no one (or should not) uses the master version but they are using the Bower one. Bower will be updated only when we tag a new release.

Effectively. But I assume that when you will update bower you will still be in V2 so this will nonetheless cause a breaking change ... unless it is modified as proposed. When you release, you should mention this breaking change so that people are warned before updating.

Could you please make a PR with that change?

@stefanneculai, Ok, I will take a look at this.


:new: use ssf-parent-pom version 9

Codecov Report

Merging #120 into dev will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev     #120   +/-   ##
=======================================
  Coverage   87.84%   87.84%
=======================================
  Files         131      131
  Lines        2362     2362
  Branches      275      275
=======================================
  Hits         2075     2075
  Misses        172      172
  Partials      115      115

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 5f217f8...2dfddc7. Read the comment docs.


Pin JS dependency & verify it

It would be nice to pin the DarkReader package to a specific version, hard-code the SHA-512 hash, then subsequently verify the download. Then we'll have greater reproducibility and security.

Indeed. While I'm not paranoid enough to care much about Dark Reader ever becoming unstable/unreliable, I can understand your reasoning here.
I thought about using a Dark Reader submodule, but they don't provide pre-compiled scripts there, as far as I know. Compiling it manually is too much of a work for someone installing the extension. Thus, the perfect scenario would be to checkout a specific submodule version when ASDF-loading the extension. Still, can be quite demanding of a process...
I'm open to ideas there, as it's a non-trivial problem, I guess :D

Compiling it manually is too much of work

I was thinking merely hard-coding the version number in the URL at https://github.com/aartaka/nx-dark-reader/blob/master/nx-dark-reader.lisp#L51 and then comparing the downloaded file with a hard-coded hash. Like SRI.

Well, now that we've apparently switched to the UserScript / WebExtensions approach, I'm not sure how to proceed, as it does not seem the metadata block has ANY support for any sort of integrity check whatsoever. Horrifying!

Yeah, that's a tough one. We can extend the syntax of @require with signature checking, though :)

See https://github.com/violentmonkey/violentmonkey/issues/1558#issuecomment-1231947929 for the issue I started there. Since I assume we're (going to be) using our own "userscript manager," using TamperMonkey's syntax would be a good idea.


Issue with User Library List and Object Browser

The User Library List and Object browser aren't working.
The Library is list saying *** Not found *** and my filter in the object browser isn't showing any files or objects.Any help is greatly appreciated!! Logging into a different machine does NOT reproduce the issue. Obviously, I've tried reinstalling Code for i.
Screenshot attached.
UPDATE: logs attached:
DB2ForIComponent.txt
CodeForIBMiLog.txt
CodeForIBMi (1).pdf

|Context|Version|
|-|-|
|Code for IBM i version|2.6.5|
|Visual Studio Code version|1.86.0|
|Operating System|linux_x64|

Active extensions

Beautify (beautify): 1.5.0

COBOL (cobol): 10.2.5

Code Spell Checker (code-spell-checker): 3.0.1

Code for IBM i Walkthroughs (vscode-ibmi-walkthroughs): 0.4.0

Comment Anchors (comment-anchors): 1.10.4

Db2 for IBM i (vscode-db2i): 0.6.1

ESLint (vscode-eslint): 2.4.4

Emmet (emmet): 1.0.0

Eradani Extension (ec-generators): 0.11.6

Extension Authoring (extension-editing): 1.0.0

Git (git): 1.0.0

Git Base (git-base): 1.0.0

Git Graph (git-graph): 1.30.0

GitHub (github): 0.0.1

GitHub Copilot (copilot): 1.157.0

GitHub Copilot Chat (copilot-chat): 0.12.0

GitLens — Git supercharged (gitlens): 14.7.0

IBM i Debug (ibmidebug): 1.0.0

JSON Language Features (json-language-features): 1.0.0

JavaScript Debugger (js-debug): 1.86.1

Local Tunnel Port Forwarding (tunnel-forwarding): 1.0.0

Markdown All in One (markdown-all-in-one): 3.6.2

Markdown Math (markdown-math): 1.0.0

Merge Conflict (merge-conflict): 1.0.0

NPM support for VS Code (npm): 1.0.1

Node Debug Auto-attach (debug-auto-launch): 1.0.0

Server Ready Action (debug-server-ready): 1.0.0

TypeScript and JavaScript Language Features (typescript-language-features): 1.0.0

Remote system
|Setting|Value|
|-|-|
|IBM i OS|n/a|
|Tech Refresh|n/a|
|CCSID|?|
|SQL|Enabled
|Source dates|Disabled
Enabled features
|/QOpenSys/pkgs/bin|/usr/bin|/QSYS.lib/ILEDITOR.lib|/QSYS.LIB|/QIBM/ProdData/IBMiDebugService/bin|
|-|-|-|-|-|
|bash|attr|GENCMDXML.PGM|QZDFMDB2.PGM|startDebugService.sh|
|git|iconv|GETNEWLIBL.PGM|||
|grep|setccsid||||
|ls|||||
|md5sum|||||
|sort|||||
|stat|||||
|tar|||||
|tn5250|||||

Shell env

BUILDLIB=RAMIRO

CURLIB=RAMIRO

HOME=/home/RAMIRO

HOST=blue.*******.com

LIBLS=

LOGIN=ramiro

LOGNAME=ramiro

MAIL=/var/spool/mail/ramiro

OLDPWD=/home/ramiro

PASE_USRGRP_LIMITED=N

PATH=/QOpenSys/usr/bin:/usr/ccs/bin:/QOpenSys/usr/bin/X11:/usr/sbin:.:/usr/bin

PWD=/home/RAMIRO

SHELL=/QOpenSys/pkgs/bin/bash

SHLVL=1

SSH_CLIENT=**************

SSH_CONNECTION=******************

TZ=<EST>5<EDT>,M3.2.0,M11.1.0

USER=ramiro

USERNAME=ramiro

_=/QOpenSys/usr/bin/env

Variants

{

  "american": "#@$",

  "local": "#@$"

}

Errors

[

  {

    "command": "ls -p /QIBM/ProdData/IBMiDebugService/bin/certs/debug_service.pfx",

    "code": 2,

    "stderr": "CPF2103: Library ECNCT37 already exists in library list.\n/QIBM/ProdData/IBMiDebugService/bin/certs/debug_service.pfx not found",

    "cwd": "/home/RAMIRO"

  }

]

Please share the contents of your Code for IBM i output channel after the list doesn't appear or error appears.

CodeForIBMiLog.txt
DB2ForIComponent.txt

@zalmang Something is very odd indeed.
I am seeing Generated from Blue a lot in your output. Do you know where that would come from?
/home/RAMIRO: LC_ALL=EN_US.UTF-8 system "call QSYS/QZDFMDB2 PARM('-d' '-i' '-t')"

        select os.OBJNAME as ODOBNM
             , coalesce(os.OBJTEXT, '') as ODOBTX
             , os.OBJATTRIBUTE as ODOBAT
          from table( SYSTOOLS.SPLIT( INPUT_LIST => 'RAMIRO', DELIMITER => ',' ) ) libs
             , table( QSYS2.OBJECT_STATISTICS( OBJECT_SCHEMA => 'QSYS', OBJTYPELIST => '*LIB', OBJECT_NAME => libs.ELEMENT ) ) os

{
    "code": 0,
    "signal": null,
    "stdout": "DB2>\n                                                   **** Generated from Blue *****",
    "stderr": ""
}

@zalmang Something is very odd indeed.
I am seeing Generated from Blue a lot in your output. Do you know where that would come from?
/home/RAMIRO: LC_ALL=EN_US.UTF-8 system "call QSYS/QZDFMDB2 PARM('-d' '-i' '-t')"

        select os.OBJNAME as ODOBNM
             , coalesce(os.OBJTEXT, '') as ODOBTX
             , os.OBJATTRIBUTE as ODOBAT
          from table( SYSTOOLS.SPLIT( INPUT_LIST => 'RAMIRO', DELIMITER => ',' ) ) libs
             , table( QSYS2.OBJECT_STATISTICS( OBJECT_SCHEMA => 'QSYS', OBJTYPELIST => '*LIB', OBJECT_NAME => libs.ELEMENT ) ) os

{
    "code": 0,
    "signal": null,
    "stdout": "DB2>\n                                                   **** Generated from Blue *****",
    "stderr": ""
}

That's the name of the machine.

@worksofliam any ideas? Thanks

@zalmang It could be an issue with the library list defined in your connection... Please try and create a new connection for the same system and see if the new connection works for the same system...

@zalmang Looks like it could be:
'No results from SQL execution': https://codefori.github.io/docs/#/pages/tips/setup?id=no-results-from-sql-execution

@worksofliam that's it! I'm trying to mark your last comment as an answer, but it's not letting me. All the best!


Add detail to collide_and_get_contacts

Resolves  godotengine/godot-docs#5406 by expanding on the format of the output array for collide_and_get_contacts and collide_with_motion_and_get_contacts, and describing how the contact point pairs can be used to calculate collision normals and depths.
I didn't add anything about the suggested "mention that shapes can be separated using a variety of techniques" for the sake of brevity, but additions are welcome.

I think you linked a wrong issue/PR number

Sorry, forgot the issue was in godot-docs. It should be linked correctly now.

@snailrhymer Can you squash commits and amend the commit message to something more descriptive?

Cherry-picked for 3.4.5.


Merging and moving to R5 (sequence of actions)

Let's go through this in the meeting to make sure we're not forgetting something:
template-work branch to be merged to master
master to be merged to mpd-r4b (will be conserved, needs a special introduction/disclaimer and maybe link from master IG)
master to be merged/converted to mpd-r5 or similar, which will be the new branch for continuous work.
mpd-r5 back to master when we see it works well in R5.
Decide which branches we could get rid of.

template-work branch to be merged to master

validate if master needs to be merged into mpd-r4b (will be conserved, needs a special introduction/disclaimer and maybe link from master IG) or just template-work branch into mpd-r4b

merge master into mpd-r5 to validate which will be the new branch for continuous work.

after validation (and possible corrections) merge mpd-r5 back to master when we see it works well in R5.

MPD-R4B done and validated (I will do the disclaimers and stuff)
MPD-R5 done and validated. Shows 0 errors in web build qa, so I guess we're ready to merge mpd-r5 back to master.
(So, back to you, @joofio )

build in master from r5 ok:
https://build.fhir.org/ig/hl7-eu/unicom-ig/qa.html
Seems good to close the issue


generator file reformat code

https://github.com/shuntakeuch1/kotlin-mybatis-entity-generator/pull/21


Discuss tools setup for CI

Currently the tools we need for CI have no defined install policy. For some tools we install specific versions and make sure that version is used. For other tools we install latest or similar and sometimes check that the version we want is actually executed. I.e. for some tools the version is never actually checked.
Currently the main purpose of the versions defined in Makefile.tools is to cache tools for github actions and update the cache if necessary. This topic came up in #239.
We should agree on a policy to either pin specific version or use latest.

My 2 cents: I'm usually in favor of using pinned versions as it guarantees better reproducibility and avoids unattended breakages.

I agree with Simon on this one 👍

Agree with Simon. Pinning allows reproducibility and consistency across all envs (dev, ci, ..). The additional benefit is that we cache the tools and thereby close to zero transient errors because of network issues.
We do need a periodic task to upgrade the tools.


Release v0.2.2

Release v0.2.2 to pub.dev.

Update CHANGELOG.md, README.md & pubspec.yaml

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 85.52%. Comparing base (5fb9360) to head (d4c30b5).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #86   +/-   ##
=======================================
  Coverage   85.52%   85.52%
=======================================
  Files          58       58
  Lines         898      898
=======================================
  Hits          768      768
  Misses        130      130

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


enable application when user update an app

OCD run https://ocd.appscale.com:8080/job/Daily Build/2896/

OCD tool test https://ocd.appscale.com:8080/job/Tools-test/1589

OCD run https://ocd.appscale.com:8080/job/Daily Build/2900/

OCD run https://ocd.appscale.com:8080/job/Daily Build/2904/

Got few extra runs with migration, since it stresses more the restore of the application:
https://ocd.appscale.com:8080/job/Migration/1151/
https://ocd.appscale.com:8080/job/Migration/1150/
https://ocd.appscale.com:8080/job/Migration/1149/

OCD run https://ocd.appscale.com:8080/job/Daily Build/2911/

OCD run https://ocd.appscale.com:8080/job/Daily Build/2915/


Remove link types

Codecov Report

Merging #943 (e86c6f3) into master (79aa6d6) will increase coverage by 0.57%.
The diff coverage is 60.00%.

@@            Coverage Diff             @@
##           master     #943      +/-   ##
==========================================
+ Coverage   30.93%   31.51%   +0.57%
==========================================
  Files          92       93       +1
  Lines        8876     8751     -125
==========================================
+ Hits         2746     2758      +12
+ Misses       5838     5699     -139
- Partials      292      294       +2

Flag
Coverage Δ

unittests
31.51% <60.00%> (+0.57%)
:arrow_up:

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

Impacted Files
Coverage Δ

database/gormdb_submission.go
67.33% <0.00%> (ø)

qf/course_submissions.go
0.00% <0.00%> (ø)

qf/ids.go
0.00% <ø> (ø)

qf/quickfeed.pb.go
87.50% <ø> (ø)

qf/remove_remote_id.go
35.89% <ø> (+4.78%)
:arrow_up:

qf/requests.pb.go
4.75% <0.00%> (+0.27%)
:arrow_up:

qf/validation.go
0.00% <ø> (ø)

web/quickfeed_service.go
34.22% <50.00%> (-0.46%)
:arrow_down:

database/gormdb_assignment.go
60.74% <63.15%> (-0.29%)
:arrow_down:

web/courses.go
51.37% <79.36%> (+0.52%)
:arrow_up:

... and 5 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

I've studied the SubmissionOwner thing, and it adds unnecessary complexity, IMO. Will it help to solve our problems if we add the following:
    uint64 EnrollmentID                    = 3;

To the Submission message type?
That way, we can do things like:
        if (isGroup(submission)) {
            const clone = new Map(this.groupSubmissions)
            this.groupSubmissions = clone.set(submission.groupID, new Submissions({ submissions }))
        } else {
            const clone = new Map(this.userSubmissions)
            this.userSubmissions = clone.set(submission.EnrollmentID, new Submissions({ submissions }))
        }

Moreover, would it help to add a field to the following:
message CourseSubmissions {
    SubmissionRequest.SubmissionType Type = 1;
    map<uint64, Submissions> submissions  = 1;
}


[improvement] Dark mode

As title.

This will be the last feature I add (in the current list of issues), because I use light mode myself 🤗.

Wait actually, this seems to be the easiest feature I can add. Decision changed. 🤗

Feature added.


Reset password url no host

Hi, I'm facing problem in sending reset password emails.
My client site use Angular. receive url in email:
http://:/auth/password/edit?config=default&redirect_url=http%3A%2F%2Flocalhost%3A4200%2F&reset_password_token=srdQMPRCunjsvHmLsKCJ
devise_token_auth 0.1.42
ruby 2.4.0p0
Rails 5.1.2

How is host defined in your config/environments/development.rb & ../production.rb file?
This is my development.rb file which renders the correct host in reset links if it helps:
config.action_mailer.raise_delivery_errors = true
  config.action_mailer.delivery_method = :test
  host = 'localhost:3333'
  config.action_mailer.default_url_options = { host: host, protocol: 'http' }

  config.action_mailer.perform_caching = false

@nicholasshirley Thank you!! This work for me.
I want to dynamic generate my host and port.
So this is my original code.
host = "#{Rails::Server.new.options[:Host]}:#{Rails::Server.new.options[:Port]}
But it doesn't work.

and now how its the best way to configure the "production.rb" file???

@ZhanZero glad you were able to get to the bottom of this! Closing for now.


[OTBN] Produce random instruction generation framework

We require a method to generate random instructions for OTBN Pre-DV
Issue comprises the following tasks:

[x] Basic instruction generation
[x] Instruction generation for all of RV32I except branch
[ ] Instruction generation for all remaining instructions (branch, loop)

estimate 24

remaining 2020-08-21 8

Basic instruction generation was merged as #3140.

Generation for all of RV32I except for the CSR instructions will have been implemented once #3181 is merged.

Update: Everything is now done (and merged) except for LOOP/LOOPI support. I've got that working locally and it should be merged early next week.

Update: Everything is now done (and merged) except for LOOP/LOOPI support. I've got that working locally and it should be merged early next week.


perf: don't fix cursors on 0.10+

no worries :-) I've actually also tried to switch to conform.nvim. Can't remember exactly why I've ended up not doing it. Maybe I should try again.


Media: Implement MVP for optimizing an existing video

Feature Description
For the pre-publish video optimization we need to implement the foundation for actually supporting this kind of functionality. Specifically, to allow optimizing existing videos in a story and uploading that optimized video to WordPress.
For testing purposes only we can add a button somewhere in the design panel to trigger this. Perhaps in the "Video Settings" panel:

This will not be the final UI.
Alternatives Considered

Additional Context

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria

This is behind a new feature flag
Videos that are already optimized can't be optimized again
Original video needs to be preserved and somehow linked to the optimized one in the database (in case we want to offer a "revert" functionality)
Video is updated on canvas after optimization
Display spinner / loading indicator while in progress (not a real accurate progress bar, see #5982 for that)

Implementation Brief

@spacedmonkey please let me know if I forgot to add anything here.
Given the requirements I'd say it's easily 8pts.


Merge refs/heads/refactor/connectors into alpha

Merge refs/heads/refactor/connectors into alpha

This PR has been merged into alpha and tagged as 1.7.0-alpha-202305071339. Thanks for your contribution!


Excel export option missing in dashboard

New excel feature #22006 was added in 2.1.0, but the export option is not available in dashboards.
How to reproduce the bug

Go to Dashboard
Click on '....'
Open "Download
See that there is no "Export to Excel" option

Screenshots
Download menu in dashboards:

Download menu in charts:

Environment

superset version: 2.1.0

Checklist
Make sure to follow these steps before submitting your issue - thank you!

[x] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[x] I have reproduced the issue with at least the latest released version of superset.
[x] I have checked the issue tracker for the same issue and I haven't found one similar.

Pinging @igorsinc here in case you have the wherewithal to take this on as an add-on to your prior work 🤞
I'm on the fence as to whether this qualifies as a bug (and should stay open) or is a feature request (and should be moved to DIscussions). ¯\_(ツ)_/¯


Bug: Broken links to same page heading

Hi!
Today I suddenly find that links to the same page heading (ala ToC) are broken.
Example:
In the top of this page you can see ToC links,
that points to the same page. For example, "Nginx" one.
behaviour:
expected: scroll to Nginx heading
now it is: open main page
In MarkDown its look like [Nginx](#nginx) - for compatibility with GitHub (and other generating libs)
Generated url:
must be: /#/quick?id=nginx
now it is: /#/?id=nginx
Possible mistake coomit
That links were worked a couple days ago, but now did not.
So, I think this happens after this commit ?

Thanks! :smile:


LIN-578 Fix broken script pipeline Dockerfile

Description
The CMD command in the script dag Dockerfile was ["standalone"], which is apparently not going to work(if you really try to build the docker image) since it is a airflow command.
Fixes # (issue)
LIN-578
Type of change

[x] Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
Pass all existing tests and local docker build/run success.

Confirmed that, with @mingjerli's fix, Dockerfile successfully builds for the SCRIPT framework. However, I encountered several other issues along the way of testing the original fix, so ended up adding more commits to address them. These additional issues include:

Use correct variable name in airflow_dockerfile.jinja; update relevant tests accordingly
Change to official pip package name if an alias exists
Add type check for mypy as ImportNodeORM's package_name is set as nullable while our code assumes package_name is always string

Approving this PR as it involves relatively straightforward fixes. But, for objectivity, it would be good if you can give your look @lionsardesai.
Also, this PR raise questions around test coverage, i.e., we may need to add tests for checking if the generated Dockerfiles build successfully. But this is out of this PR's scope, so just leaving a record here.


Changes for ability to use the plugin for AWS China.

More : https://issues.jenkins-ci.org/browse/JENKINS-45964

Thanks, looks good


MacOS reduce disk space

Problem Description
I would like to use mitmdump only on MacOS.
If I remove mitmweb and mitmproxy the OS says that the mitmdump is damaged...
Steps to reproduce the behavior:

Download the macos tar.gz
extract the downloaded file
Remove the mitmweb and mitmproxy
Try launch mitmdump

System Information
10.3.0

First advice is to install mitmproxy via brew:
brew install --cask mitmproxy
Second advice is that you don't have to physically delete mitmproxy and mitmweb, just use mitmdump

Thank you for your input.
But, I need only the mitmdump executable... so again...
How can I keep the mitmdump executable and remove the mitmweb and mitmproxy from the downloaded archive?

This works, but it's nothing we would ever recommend and something we don't provide support for:
> tar xzf mitmproxy-10.3.0-macos-arm64.tar.gz
> cd mitmproxy.app/Contents/MacOS
> ll
total 51744
-rwxr-xr-x@ 1 mhils  primarygroup   8.4M Apr 17 13:32 mitmdump*
-rwxr-xr-x@ 1 mhils  primarygroup   8.4M Apr 17 13:32 mitmproxy*
-rwxr-xr-x@ 1 mhils  primarygroup   8.4M Apr 17 13:32 mitmweb*
> rm mitmproxy mitmweb
> ./mitmdump
[11:10:25.696] HTTP(S) proxy listening at *:8080.

I did exactly the same and got message that mitmdump is corrupted.
Maybe it's related to signing/notarize ?


Just a Request

While what is there is what some would say proper English, it doesn't make as much sense as something more understandable
D:\pocketnet.gui-master>grep -r -i "You can enter less than 5 tags" *
-localization/en.js:____loclib.e13162 = "You can enter less than 5 tags"
+localization/en.js:____loclib.e13162 = "You can only enter up to 5 tags"

Proper English is, "You can enter fewer than 5 tags." Fewer is the right word because 'tag' is a countable noun. This statement means you can enter 1-4 tags; you cannot enter 5 tags.
Proper English for entering 5 or fewer tags is, "You can enter a maximum of 5 tags."

"You can enter 5 tags or less."
"You can enter no more than 5 tags."
"Ye can verily recount no greater than 5 tags here."
All of these are fine.


CITATION.cff

Guys, check the CITATION.cff in the 'develop' branch. Check your names and add your orcid ID.
If you don't have an Orcid ID, please create one here:
https://orcid.org/

I updated my citation

@shandlermason, please update this with your orcid ID.

Ok will do in morning


chore: run yarn on main

Description

After running yarn on main, I got a different yarn.lock file

#nochangelog

Mergings this as I see it also occurring in other PRs like this https://github.com/artsy/eigen/pull/8142 cc @rajsam003


Does not package correctly on Windows

I was trying to build the npm package on Windows 10 using Git bash by doing the following:
./tool/make-package.sh

The output structure of dist/package was like this
./
../
CHANGELOG.md
LICENSE
README.md
bundles/
cjs/
package.json
src/
typings/

The cjs and typings folders were copied to a subfolder instead of being copied directly to package. In order to fix this I had to change the following lines in the make-package.sh file.
cp -R dist/cjs/* dist/package
cp -R dist/typings/* dist/package

This should probably be changed in the repo.

Hey @codemile, you were right, it was broken on Windows.


Failed to pass "ensure controller files directory exists"

Getting error for "ansible-playbook prereq.yml"
TASK [ensure controller files directory exists] ********************************
Monday 17 August 2020  20:51:20 +0000 (0:00:00.076)       0:00:09.423 *********
fatal: [localhost]: FAILED! => {"msg": "The conditional check 'controller.protocol == 'https'' failed. The error was: An unhandled exception occurred while templating '{u'protocol': u"{{ controller_protocol | default('https') }}", u'extraEnv': u'{{ controller_extraEnv | default({}) }}', u'dir': {u'become': u'{{ controller_dir_become | default(false) }}'}, u'ssl': {u'keystore': {u'password': u'openwhisk', u'name': u'{{ __controller_ssl_keyPrefix }}openwhisk-keystore.p12'}, u'cert': u'{{ __controller_ssl_keyPrefix }}openwhisk-server-cert.pem', u'cn': u'openwhisk-controllers', u'key': u'{{ __controller_ssl_keyPrefix }}openwhisk-server-key.pem', u'keyPrefix': u'{{ __controller_ssl_keyPrefix }}', u'clientAuth': u"{{ controller_client_auth | default('true') }}", u'storeFlavor': u'PKCS12'}, u'heap': u"{{ controller_heap | default('2g') }}", u'managedFraction': u'{{ controller_managed_fraction | default(1.0 - (controller_blackbox_fraction | default(__controller_blackbox_fraction))) }}', u'entitlement': {u'spi': u"{{ controller_entitlement_spi | default('') }}"}, u'instances': u"{{ groups['controllers'] | length }}", u'loglevel': u"{{ controller_loglevel | default(whisk_loglevel) | default('INFO') }}", u'timeoutFactor': u'{{ controller_timeout_factor | default(2) }}', u'blackboxFraction': u'{{ controller_blackbox_fraction | default(__controller_blackbox_fraction) }}', u'confdir': u'{{ config_root_dir }}/controller', u'authentication': {u'spi': u"{{ controller_authentication_spi | default('') }}"}, u'arguments': u"{{ controller_arguments | default('') }}", u'timeoutAddon': u"{{ controller_timeout_addon | default('1 m') }}", u'basePort': 10001, u'akka': {u'cluster': {u'bindPort': 2551, u'basePort': 8000, u'host': u"{{ groups['controllers'] | map('extract', hostvars, 'ansible_host') | list }}", u'seedNodes': u"{{ groups['controllers'] | map('extract', hostvars, 'ansible_host') | list }}"}, u'provider': u'cluster'}, u'loadbalancer': {u'spi': u"{{ controller_loadbalancer_spi | default('') }}"}}'. Error was a <class 'ansible.errors.AnsibleError'>, original message: An unhandled exception occurred while running the lookup plugin 'ini'. Error was a <class 'ConfigParser.NoSectionError'>, original message: No section: u'controller'\n\nThe error appears to have been in '/home/amit/openwhisk/ansible/setup.yml': line 89, column 5, but may\nbe elsewhere in the file depending on the exact syntax problem.\n\nThe offending line appears to be:\n\n  # Generate Controller certificates\n  - name: ensure controller files directory exists\n    ^ here\n"}
The conditional check 'controller.protocol == 'https'' failed. The error
was: An unhandled exception occurred while templating '{u'protocol': u"{{
controller_protocol | default('https') }}", u'extraEnv': u'{{
controller_extraEnv | default({}) }}', u'dir': {u'become': u'{{
controller_dir_become | default(false) }}'}, u'ssl': {u'keystore': {u'password':
u'openwhisk', u'name': u'{{ __controller_ssl_keyPrefix }}openwhisk-
keystore.p12'}, u'cert': u'{{ __controller_ssl_keyPrefix }}openwhisk-server-
cert.pem', u'cn': u'openwhisk-controllers', u'key': u'{{
__controller_ssl_keyPrefix }}openwhisk-server-key.pem', u'keyPrefix': u'{{
__controller_ssl_keyPrefix }}', u'clientAuth': u"{{ controller_client_auth |
default('true') }}", u'storeFlavor': u'PKCS12'}, u'heap': u"{{ controller_heap |
default('2g') }}", u'managedFraction': u'{{ controller_managed_fraction |
default(1.0 - (controller_blackbox_fraction |
default(__controller_blackbox_fraction))) }}', u'entitlement': {u'spi': u"{{
controller_entitlement_spi | default('') }}"}, u'instances': u"{{
groups['controllers'] | length }}", u'loglevel': u"{{ controller_loglevel |
default(whisk_loglevel) | default('INFO') }}", u'timeoutFactor': u'{{
controller_timeout_factor | default(2) }}', u'blackboxFraction': u'{{
controller_blackbox_fraction | default(__controller_blackbox_fraction) }}',
u'confdir': u'{{ config_root_dir }}/controller', u'authentication': {u'spi':
u"{{ controller_authentication_spi | default('') }}"}, u'arguments': u"{{
controller_arguments | default('') }}", u'timeoutAddon': u"{{
controller_timeout_addon | default('1 m') }}", u'basePort': 10001, u'akka':
{u'cluster': {u'bindPort': 2551, u'basePort': 8000, u'host': u"{{
groups['controllers'] | map('extract', hostvars, 'ansible_host') | list }}",
u'seedNodes': u"{{ groups['controllers'] | map('extract', hostvars,
'ansible_host') | list }}"}, u'provider': u'cluster'}, u'loadbalancer': {u'spi':
u"{{ controller_loadbalancer_spi | default('') }}"}}'. Error was a <class
'ansible.errors.AnsibleError'>, original message: An unhandled exception
occurred while running the lookup plugin 'ini'. Error was a <class
'ConfigParser.NoSectionError'>, original message: No section: u'controller'  The
error appears to have been in '/home/amit/openwhisk/ansible/setup.yml': line 89,
column 5, but may be elsewhere in the file depending on the exact syntax
problem.  The offending line appears to be:    # Generate Controller
certificates   - name: ensure controller files directory exists     ^ here
PLAY RECAP *********************************************************************
localhost                  : ok=5    changed=2    unreachable=0    failed=1

@samanta-amit
Could you share the details about the issue and how to reproduce it?

I was trying to build OpenWhisk on Ubuntu 18.04. So after installing docker and all the prerequisites following "/all.sh" script.
Then, I configured the CouchDB database parameters, then I ran "ansible-playbook setup.yml
But, I am getting aforementioned error that it failed to "ensure controller files directory exists".

I was trying to build OpenWhisk on Ubuntu 18.04. So after installing docker and all the prerequisites following "/all.sh" script.
Then, I configured the CouchDB database parameters, then I ran "ansible-playbook setup.yml
But, I am getting aforementioned error that it failed to "ensure controller files directory exists".

Could you share your python and ansible version as well?

ansible 2.5.2 and Python 2.7.17

Thank you for sharing.
Hm.. that ansible task is just to ensure the directory for the controller certificates.
Since it does nothing but check the existence and create it if not exist, I feel there would be no issue in most cases.
Also it is seemingly the ansible task issue but it also says about the missing section for controller and the offending line.

If you didn't touch any configuration, there will be the section.
I observed a similar issue when I used a different version of Ansible or Python, so I asked them.
Could you check once if controller section and controller.protocol configuration exist in ansible/group_vars/all?

@style95 i couldn't reproduce this simply by changing the controller protocol :/
my ansible version is 2.7.7

Yes, the controller section exists. Without it also, I can build the OpenWhisk. But, I am facing another problem related to invoking a "hello" message following the command "wsk action invoke /whisk.system/utils/echo -p message hello --result".
error: Unable to invoke action 'utils/echo': Post https://<IP_ADDRESS>/api/v1/namespaces/whisk.system/actions/utils/echo?blocking=true&result=true: dial tcp <IP_ADDRESS>:443: getsockopt: connection refused.
Do you have any idea about this error?

@rabbah
When I faced the similar issue, I used older ansible than 2.5.2.
And that was because the ansible syntax had changed.
@samanta-amit
So the above controller directory issue is resolved?
Regarding the connection refused, need to check a couple of things.

Haven't you faced any error or issue during all ansible tasks?
Are Nginx and Controller containers running?
Could you access the controller directly with the controller IP/port?

Thanks for replying. I am really sorry for pestering you will all the questions.
No, the controller directory issue is there. But, with that issue also, I can build Openwhisk.
About the connection issue, I checked the things you asked.

I did not face any error during all the listed ansible tasks, only one aforementioned error.
Both Nginx and Container are running.
I do not know,  how to check it? Would you please elaborate on this a little?

In addition to that, I tried to enable the Docker API following the instructions in
https://www.ivankrizsan.se/2016/05/18/enabling-docker-remote-api-on-ubuntu-16-04/

Sorry, I did not meant to close this issue. I put this wrongly. Please ignore it.

Then I suspect your controller is not running well because of the above issue.
If you don't need to secure the connection between nginx and controller, I suggest changing controller protocol to http as @rabbah mentioned above.
You can add this line to environments/local/group_vars/all or your environment configuration.
controller_protocol: http

Regarding the direct access to a controller, you can check the controller port using docker ps.
$ docker ps
CONTAINER ID        IMAGE                                                      COMMAND                  CREATED             STATUS              PORTS                                                                                NAMES
4b01049c888d        whisk/controller:latest   "/bin/sh -c 'exec /i…"   44 hours ago        Up 44 hours         <IP_ADDRESS>:15000->15000/tcp, <IP_ADDRESS>:16000->16000/tcp, <IP_ADDRESS>:8000->2551/tcp, <IP_ADDRESS>:10001->8080/tcp   controller0

You can access to http://localhost:10001/ping and the controller would return just pong.
$ curl http://localhost:10001/ping
pong

You can also check whether your controller is running well or not via logs.
The default location of the logs is /tmp/wsklogs/controller0/.
Even you can configure your wsk cli to directly connect to the controller as well.
$ wsk property set --apihost http://localhost:10001

In this case, the cli will directly communicate with a controller. So the Nginx would do nothing here.
BTW, did you just clone the openwhisk repo and use it or change anything?
You anyway would need to solve the controller directory issue so I want to make sure the codes are intact.

For the avoidance of doubt - did you try the "quickstart" which runs a standalone openwhisk stack as a Java process?  It is simpler to install and get running which may be less prone to the errors you're encountering.
cd openwhisk
./gradlew core:standalone:bootRun

See https://github.com/apache/openwhisk#quick-start.

@rabbah I tried to do that, but the building is stuck at "96% EXECUTING". Will try to do it from scratch again and get back to you.
@style95 I tried to add this "controller_protocol: http", but still the error is there. May be, I will try to build it again and come back to you.

That's expected - the Java process remains the foreground. If you run the wsk init command at the start you can start using openwhisk.

@rabbah I configured wsk by the following command.
wsk property set
--apihost 'http://localhost:3233'
--auth '23bc46b1-71f6-4ed5-8c54-816aa4f8c502:123zO3xZCLrMN6v2BKK1dXYFpXlPkccOFqm12CdAsMgRU4VrNZ9lyGVCGuMDGIwP'
Then, I rebuild OpenWhisk again. But, still, it stuck at "96% EXECUTING". Do I need to do anything else?

@rabbah This is the executing state, it is going on for more than 1.30 hour. It seems like in a loop and checking the things again and again. Is there anything wrong in it or is this the correct way?

It's not stuck. Maybe "building is misleading". What you're seeing are the server logs. Run a wsk command from another terminal like wsk action list and you'll see logs scroll buy.

Thanks. After doing "wsk action list", I am getting this error.
error: Unable to obtain the list of actions for namespace 'default': Get http://localhost:3232/api/v1/namespaces/_/actions?limit=30&skip=0: dial tcp <IP_ADDRESS>:3232: getsockopt: connection refused


OCM-10677 | feat: Adding extra validation to verify user is logged in

Adding extra validation to check the valid fields are present in the ocm.json file verifying the user is logged in.
OCM-10677

/lgtm

/cherrypick release_1.2.48


python3Packages.whirlpool-sixth-sense: init at 0.15.1

Motivation for this change
Python library for Whirlpool 6th Sense appliances
https://github.com/abmantis/whirlpool-sixth-sense
This is a Home Assistant dependency.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

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

2 packages built:

python38Packages.whirlpool-sixth-sense
python39Packages.whirlpool-sixth-sense

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

2 packages built:

python38Packages.whirlpool-sixth-sense
python39Packages.whirlpool-sixth-sense


import .csv in c#

Hi, I'd like to do something like this in c# :
LOAD CSV WITH HEADERS FROM 'file:/Users/Stage01/Desktop/csv/rels.csv' AS rels
MATCH (from {id: rels.From}), (to {id: rels.To})
CREATE from-[:REL {type: rels.'Relationship'}]->to
how can I translate this in c# using neo4jclient?

var client = new GraphClient(URI);
client.Cypher
    .LoadCsv(new Uri("file:/Users/Stage01/Desktop/csv/rels.csv"), "rels", withHeaders:true)
    .Match("(from {id: rels.From}), (to {id: rels.To})")
    .Create("from-[:REL {type: rels.'Relationship'}]->to


You should be able to pass a list to loop.cycle

Currently when you pass a list to loop.cycle, it will just set the item to the entire list. It would be great if there was a way to be able to use loop.cycle to cycle through a list instead of just being given the entire list.

loop.cycle(*values)

I swear I tried to figure out how to do this for hours. Thanks for the solution. Just one of those days.


$submit-data operation parameter should be camelCase

According to the spec for the operation, the measureReport parameter should be camelCase.
In the operation definition below, the parameter is named measurereport. This causes a null pointer exception when the operation is called with a camelCase measureReport parameter:
https://github.com/DBCG/cqf-ruler/blob/29e5a37fad5ec1890d74c0b087920049e76ef30d/r4/src/main/java/org/opencds/cqf/r4/providers/MeasureOperationsProvider.java#L590

Resolved by #284


Made some enhancements on the Scala Options' code

The main goal of this Pull Request is to make some code enhancements on the different Options used in this project.
Some pattern matching snippets were unnecessary since we could have treated options as collections.
for example :

foreach can replace the following code:

option match {
  case None => {}
  case Some(x) => foo(x)
}

forall can replace the following code:

option match {
  case None => true
  case Some(x) => foo(x)
}

exists can replace the following code:

option match {
  case None => false
  case Some(x) => foo(x)
}

I also removed some unnecessary pattern matching conditions which could have been anticipated using a getOrElse on the Scala Option.
These modifications were tested by running the existant tests.

this is good!

Hello @tdas ,
Thanks a lot for your feedback. I made the requested changes.
Otherwise, it seems that there's some issues on Circle CI with the  Prepare Python env step (not only on my pipeline, but also on other pipelines). Can you please re-run my pipeline once the problem is fixed ? (I don't have the necessary privileges).
Thanks a lot,

Hello @rxin , Thanks for your feedback. I made the requested changes 👍

looks good to me, I will merge it soon.


net: gptp: Set stack size correctly for handler thread

The gptp stack size was not properly set, one must use
the K_THREAD_STACK_SIZEOF() macro to calculate stack size.
Signed-off-by: Jukka Rissanen<EMAIL_ADDRESS>

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@52f82fa). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #9824   +/-   ##
=========================================
  Coverage          ?   52.24%
=========================================
  Files             ?      212
  Lines             ?    25949
  Branches          ?     5593
=========================================
  Hits              ?    13558
  Misses            ?    10155
  Partials          ?     2236

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 52f82fa...730b151. Read the comment docs.


release downgrade failed

I think it looks like that the latest 1.0.0 release PHP downgrade failed
https://github.com/rectorphp/type-perfect/actions/runs/11128835212

hmm maybe this was repaired in the meantime. I am not sure, just stumbled over this error in the github actions tab

It looks like duplicated process, as it tries to re-downgrade already released version. The downgrade release already there, so should be no longer another downgrade process, it maybe due to downgrade process target on same repository.


flexget fails to build with python-3.13

Whilst building flexget I ran into the following error:
      gcc -fno-strict-overflow -Wsign-compare -DDYNAMIC_ANNOTATIONS_ENABLED=1 -DNDEBUG -fcf-protection -fexceptions -fcf-protection -fexceptions -fcf-protection -fexceptions -O3 -fPIC -I/home/han/.virtualenv/flexget/include -I/usr/include/python3.13 -c src/greenlet/greenlet.cpp -o build/temp.linux-x86_64-cpython-313/src/greenlet/greenlet.o
      In file included from src/greenlet/greenlet_greenlet.hpp:26,
                       from src/greenlet/greenlet_internal.hpp:20,
                       from src/greenlet/greenlet.cpp:19:
      /usr/include/python3.13/internal/pycore_frame.h:8:4: error: #error "this header requires Py_BUILD_CORE define"
          8 | #  error "this header requires Py_BUILD_CORE define"
            |    ^~~~~

And then a lot more of the same. Whilst googling I ran into this report, which describes exactly the same problem:
https://src.fedoraproject.org/rpms/python-greenlet/c/d426d412697fad13bd735496ac1a934e29ad2a72?branch=rawhide
So please someone examine building flexget with python-3.13 and try to reproduce and then fix this problem.

project is dead


ToggleSwitchElement: Uncaught DOMException

I'm using the library @primer/view-components as an npm package and compiling it into the rails 7 application with esbuild. After the upgrade from 0.0.92, I receive the current error in the javascript console.
toggle_switch.js:141 Uncaught DOMException: Failed to execute 'define' on 'CustomElementRegistry': this constructor has already been used with this registry
    at http://localhost:3000/assets/application-5fdd23dba4c210059bbefff8367af8dc8ea85c1da52231cab49e88648fb0561b.js:6076:27
    at http://localhost:3000/assets/application-5fdd23dba4c210059bbefff8367af8dc8ea85c1da52231cab49e88648fb0561b.js:8129:3
(anonymous) @ toggle_switch.js:141
(anonymous) @ index.ts:140

I tried to investigate and I discovered if I remove the export from alpha/toggle_switch.js everything works as expected.
ToggleSwitchElement = __decorate([
    controller
], ToggleSwitchElement);
//export { ToggleSwitchElement };
if (!window.customElements.get('toggle-switch')) {
    window.ToggleSwitchElement = ToggleSwitchElement;
    window.customElements.define('toggle-switch', ToggleSwitchElement);
}

@camertron since you're the author of #1321 can you tell me if it's a library bug and point me in the right direction on how to fix it?
Thank you for your support.

Hi @safeforge, sorry to hear you're running into this issue!
I'm adding this to our team project board so we can triage and get back to you if @camertron hasn't weighed in by then. Thanks!


Clarification on steps for adding library to project?

I'm trying to add the lwESP library to a project and I was wondering if I could have clarification on the process. Do I need to complete the porting guide steps before adding the library to the project itself?
I am following the steps from the Getting Started guide, but I am unable to build the project. I am not attempting to actually implement any of the lwESP functions yet in my application, just compile the library alongside my already-working project.

Unless you call lwesp_init in your application part, library should compile with no LL port, but you do need to prepare system port, at least header file -> there is a definition of some types used across the lib.

Thanks! So there are 3 main files to configure:

The LL port
lwesp_sys_template.c
lwesp_sys_port.h

Is there a template file for the LL port, like there is for the other components?

Correct

Thanks!


"My Groups" Tab

create a tab (between home and profile) that will display the groups your created, and the groups you have joined

Acceptance Criteria:
Add a "My Groups" tab that displays a user's created study groups, along with its information such as name, location, description


Different BPM-Values for GrandMA3

Hello,
I am trying to use your code to set the BPM via OSC to GrandMA3. Do you know that console/software?
The problem I ran into is, that GrandMA3 does not expect the absolute value for BPM.
Instead it handles the BPM based on percent of the SpeedMaster-Fader, where 100% are 240 BPM.
Could you imagine to add another argument, like "-sGMA3 or --serverGMA3" which modifies the measured BPM with this formula "sqrt(bpm/240)*100" and sends it to given IP/port/address?
The idea about the additional argument instead of modifying just the output-value in general is that I could send the BPM to GMA3 (in the format it expects) and furthermore as an absolute value to another OSC-target, like MadMapper.
Thanks in advance,
Micha

hi, will take a look

Added -c option for custom calculation, so for you -c GMA3 should be ok now.

in case of problem, just open another one


Effect Stacking on Disabled Effects

Describe the bug
Now that effects are disabled on rightclick, I came to a situation where I had multiple disabled effects of the same type.
To Reproduce
Steps to reproduce the behavior:

Apply a effect let's say Cantrip - Light
Right click to disable
Apply effect again

Expected behavior
Activate the disabled Effect
Screenshots

Additional context
It even adds another active effect on an already active effect, if you apply the CE with a spell. That may be a problem with midi-qol and not within your scope.

Yeah, neither midi nor CE checks if there is an existing disabled effect before applying a new one. It's probably out of scope to do so... if you prefer to delete effects, just do that instead.


UX improvements

Add and "Click Away Listener" for some popup or dailog modal who need it for a better ux. For example, once the link has been copied, it would be interesting if the popup closed directly, I also find it important that the user closes this popup by clicking elsewhere

Here, we have successfully modified his code but the user still has to click on "Cancel" to close the form, I think it should close automatically after the success of the operation.

Here, we have successfully added a code but the form is still present, I think it should close automatically. If the idea is to allow multiple codes to be added, I suggest adding a checkbox to ask the user if he wants to leave the form open.

Done !


Can we deploy this example application to azure cluster instead of local cluster?

Simply speaking, I deployed this example to Azure but couldn't see anything except the manager explorer page.
I found a closed question: https://github.com/MicrosoftDocs/azure-docs/issues/17032
In that discussion thread, seems like this page used to contain this section #deploy-the-application-to-azure
But it's gone now...feel like it would be helpful.
Can someone suggest what kind of configurations do I have to make if I just simply want to deploy to Azure and see the simple voting page?
Thanks in advance.

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

ID: 3aa58af5-02ea-e425-1fc5-9f588d394016
Version Independent ID: 86440381-783a-6e49-bacb-59b1c1581d81
Content: Create a .NET app on Service Fabric in Azure
Content Source: articles/service-fabric/service-fabric-quickstart-dotnet.md
Service: service-fabric
GitHub Login: @MikkelHegn
Microsoft Alias: mikhegn

Never mind, I found the page that answers my question. I am gonna close this question.
Guess it would be nice to include this link in this page.
https://docs.microsoft.com/en-us/azure/service-fabric/service-fabric-tutorial-deploy-app-to-party-cluster

Thanks for reaching out. I will reopen and look into adding a link to the doc :)

Based on the layout of other docs I think we will keep this one as is. However this comment will remain on this doc and if others have issues they can comment and we can always revisit.


chore: Update k8s debian package repositories to pkgs.k8s.io, bump critools to 1.28

What problem does this PR solve?:

Update debian package repositories to use pkgs.k8s.io
Bump critools to v1.28

Which issue(s) does this PR fix?:

https://jira.d2iq.com/browse/D2IQ-99586
https://jira.d2iq.com/browse/D2IQ-99953

Special notes for your reviewer:

There was an attempt to template kubernetes_version via
{{ kubernetes_major }}.{{ kubernetes_minor }}.{{ kubernetes_patch}} but this fails many tests as the templated version is referenced by the Magefile here.

Does this PR introduce a user-facing change?:

added a commit to use 1.28.6 so its consistent with https://github.com/mesosphere/konvoy-image-builder/pull/988

Thanks for this. I fixed the deb revision in the dkp-release repo, but not here. I'll want to backport this to release-2.5


origin_access_control_id not implemented

Description
We can only use the legacy origin_access_identity like below:
origin = {
  default = {
    domain_name      = module.web_app.s3_bucket_bucket_regional_domain_name
    s3_origin_config = {
      origin_access_identity = "web_app"
    }
  }
}

origin_access_control does not work:
  origin = {
    default = {
      domain_name              = module.web_app.s3_bucket_bucket_regional_domain_name
      origin_id                = module.web_app.s3_bucket_id
      origin_access_control_id = aws_cloudfront_origin_access_control.test.id
    }
  }

I can also confirm that its not working as the code states. had to use the next config
origin = {
s3_one = {
  domain_name = "testbucket.s3.eu-west-1.amazonaws.com"
  origin_id   = "s3-my-private-bucket"
  origin_access_control = "oac"
}

}

+1 This is not working as intended.
As a workaround (I am not proud of this and it should be thrown in a corner somewhere), you can have the module refer to itself, and manually pass in the origin_access_control_id.
      origin_access_control_id = module.this.cloudfront_origin_access_controls_ids[0]

Any updates on this? Just started using this and wanted not to implement legacy config in a new project, but I am experiencing the same issue.

There are two ways to achieve the same result:

origin_access_control_id - expects ID of the resource created externally from the module (e.g. E345SXM82MIOSU)
origin_access_control - key in the origin_access_control argument created by the module.

See this example for more details: https://github.com/terraform-aws-modules/terraform-aws-cloudfront/blob/master/examples/complete/main.tf#L96-L101
The workaround by @Duceswild99 can be like:
     origin_access_control = "key_from_origin_access_control" # created by this module


Linux?

Hi,
Have you been able to use it under Linux?
I tried google chrome / chromium and I can select the device and click the "connect" button but then the control doesn't work. I'm using a Jabra 510 usb.
Also I don't know which logs to check in order to debug this.
Do you have more information?
Thanks :)

I have the same issue with Fedora 37 and Jabra Evolve 40. I connect and it still say Not connected, and the next time it states "Paired" but will not connect.

Are you guys using (native) BlueTooth connections between computer and the device?
It sounds a bit like the behaviour you get when using the BT device profile, rather than the USB HID protocol.
The Jabra SDK libraries only work over USB. And connected via BT, there is no mute command defined in the protocol...

I connect via the USB where the buttons are. I have the same experience in Fedora 38.

Hi, I'm using USB too

Same here, Linux Fedora 38 with Google Chrome Stable, using Jabra 410 USB

In my case the issue was not related to this extension (also the extension was not required to make it work).
The issue was on the permissions of the hidraw device.
I fixed it like this:
$ groups
[...] input [...]

$ lsusb | grep -i jabra
Bus 003 Device 034: ID 0b0e:0420 GN Netcom Jabra SPEAK 510

$ cat /etc/udev/rules.d/50-jabra.rules
SUBSYSTEM=="hidraw", ATTRS{idProduct}=="0420", ATTRS{idVendor}=="0b0e", MODE="0660", GROUP="input"

# unplug, replug usb device

$ ls -la /dev/hidraw*
[...]
crw-rw---- 1 root input 240, 6 Oct 30 11:53 /dev/hidraw6
[...]

Notes:

My user belongs to the input group, you'll probably want to use plugdev in ubuntu.
Adapt product & vendor ids to your device

Related: https://developer.chrome.com/articles/hid/#dev-tips


ol.style.text - gap between two line text

[x] I am submitting a bug or feature request, not a usage question. Go to https://stackoverflow.com/questions/tagged/openlayers for questions.
[x] I have searched GitHub to see if a similar bug or feature request has already been reported.
[x] I have verified that the issue is present in the latest version of OpenLayers (see 'LATEST' on https://openlayers.org/).
[ ] If reporting a bug, I have created a CodePen or prepared a stack trace (using the latest version and unminified code, so e.g. ol-debug.js, not ol.js) that shows the issue.

Is there any oportunity to remove space between two line text in label feature?
In API I don't find any oportunity, maybe in library code?
In OpenLayers v2.13 there wasn't any blank space in OL v4.6.4 (and lower) it is. Styling text in label look better for me in previous version OpenLayers.
Maybe is better option no space between two line and when I need it I just put it... ?

text: new ol.style.Text({
            text: 3.5 +'\nbar'
        })

Starting with v6, you'll be able to control the line height. You can set a line-height in the font of the style:
import TextStyle from 'ol/style/Text';
new TextStyle({
  font: '12px/1 sans-serif'
});

The /1 sets a line-height of 1, which is lower than the default of 1.2.
See #9307.


How to serialize as a commented out line?

Hi,
I'm still pretty new to serde's concepts, so it might be that there is a very easy solution to my issue.
I want to be able to serialize some lines in my ini files as comments, for example if they contain a default value.
I.e. I might have some setting=100 which should be serialized as ;setting=100, if 100 happens to be the default value for this setting. These "commented out" default values are pretty common in configuration files as a form of self-documentation.
I see that the parser already has a Item::Comment variant that the writer seems to be able to serialize, but I can't find any serialize_comment (or similar) method anywhere and the writer's API is not public.
Any hints how to achieve my goal would bei highly appreciated. I don't mind if it involves a custom serialize_with attribute and function or a manually impl Serialize/r for T.

This seems tricky, since I don't believe serde has any built-in concept of comments when serializing, and doesn't have a good way of interweaving metadata like a "this value is unchanged from default" flag to cross abstraction boundaries. serde_ini::Writer also isn't a trait so you can't really proxy it easily without buffering (serializing to vec/str, then parsing back out with serde_ini::Parser, then write back out again).
It's not pretty but your best bet might be something like:
use std::collections::HashMap;

trait DefaultValue<T: PartialEq> {
    const VALUE: T;
    const NAME: &'static str;
}

fn serialize_or_default<D: DefaultValue<T>, T: PartialEq + std::string::ToString, S: serde::Serializer>(value: &T, mut s: S) -> Result<S::Ok, S::Error> {
    let name = if value == &D::VALUE {
        format!(";{}", D::NAME)
    } else {
        D::NAME.to_owned()
    };
    serde::Serialize::serialize(
        &std::iter::once((name, value.to_string())).collect::<HashMap<_, _>>(),
        s
    )
}

struct DefaultSetting; impl DefaultValue<u32> for DefaultSetting { const NAME: &'static str = "setting"; const VALUE: u32 = 100; }
#[derive(Deserialize, Serialize, Debug)]
struct Data {
    #[serde(flatten, serialize_with = "serialize_or_default::<DefaultSetting, _, _>")]
    setting: u32,
}

assert_eq!(serde_ini::to_string(&Data {
    setting: 100
}).unwrap(), ";setting=100\r\n");
assert_eq!(serde_ini::to_string(&Data {
    setting: 99
}).unwrap(), "setting=99\r\n");

Not very optimal because flatten introduces buffering (plus ToString is needed due to #6, and HashMap isn't the best choice of an interim serializeable map) but it's probably the quickest and least invasive approach, just throw a macro around the DefaultValue impls to make them less verbose.

Very interesting approach, thanks @arcnmx! It seems to be working good when serializing. Yet, when deserializing, the flatten attribute seems to be causing trouble:
Unwrapping creates a "can only flatten structs and maps" panic in assert_eq!(serde_ini::from_str::<Data>("setting=100\r\n").unwrap(), Data { setting: 100 }); to your example code.
I have not yet fully grasped struct flattening in serde, but from quickly skimming through the docs, it seems to me as if I'd need some custom deserialize_with function as well which would be doing the inverse of what you did in serialize_or_default. Am I guessing right?

Oh, sorry, I assumed flatten was a serialization-only attribute... You'd maybe just be better off using #[serde(into = ...)], there are a bunch of ways to do it but I guess something like this could work:
#[derive(Serialize, Deserialize, Debug, Clone, PartialEq)]
#[serde(into = "DataWithDefaults")]
struct Data {
    setting: u32,
}

impl Default for Data {
    fn default() -> Self {
        Data {
            setting: 100,
        }
    }
}

#[derive(Serialize, Deserialize)]
#[serde(transparent)]
struct DataWithDefaults(HashMap<String, String>);

fn convert(d: &Data) -> HashMap<&'static str, String> {
    // this could possibly be done lazy/messy using serde::de::IntoDeserializer or even like: serde_ini::from_str(&serde_ini::to_string(&d).unwrap()).unwrap()
    // consider an ordered map if that's important?
    std::iter::once(
        ("setting", d.setting.to_string())
    ).collect()
}

impl From<Data> for DataWithDefaults {
    fn from(d: Data) -> Self {
        let d = convert(&d);
        let defaults = convert(&Data::default());
        DataWithDefaults(
            d.into_iter()
                .map(|(k, v)| (
                    if defaults.get(&k) == Some(&v) { format!(";{}", k) } else { k.into() },
                    v
                )).collect()
        )
    }
}

assert_eq!(serde_ini::to_string(&Data {
    setting: 100
}).unwrap(), ";setting=100\r\n");
assert_eq!(serde_ini::to_string(&Data {
    setting: 99
}).unwrap(), "setting=99\r\n");
assert_eq!(serde_ini::from_str::<Data>("setting=100\r\n").unwrap(), Data {
    setting: 100
});

Hi @arcnmx ,
thanks again for your kind help and all the example code!
Actually, I went for the #[serde(into = "...")] way now for my project. Yes, it is a little verbose, and it might not be the most efficient solution due to all the .to_string() allocations and the large HashMap (I actually used BTreeMap to have more control about the ordering of the serialization), but hey: it works! My convert function now has a large
[
    ("setting1", d.setting1.to_string()),
    ("setting2", d.setting2.to_string()),
    // ...
    ("settingn", d.settingn.to_string()),
]
.iter()
.cloned()
.collect()

block at the end, as I was not able to turn your idea to use serde::de::IntoDeserializer into code that the compiler accepted. I might try to reduce all the boilerplate with some custom macro_rules! later, but for now I'm kind of happy that it is possible to do what I want with what is there right now on stable and in the current version of serde_ini.
I admit that having a more powerful/generic serde_ini::Writer with the additions that you suggest above would be helpful in the long run, but for this small project your suggestions and hints solved my issue.
Feel free to close this issue unless you want to keep it open to work on the Writer improvements.


Support Maven Wrapper

Nowadays Maven Wrapper looks like a the facto standard for including maven in your project so you do not need to install maven separately.
Tools like Spring Initializr scaffolds your project with it.
Dependency Analytics should support using the mvnw command out of the box if it is present in the project's folder.
Anyway I tried to set it up manually by setting maven.executable.path like:
"maven.executable.path": "${fileWorkspaceFolder}/mvnw"
But looks like ${fileWorkspaceFolder} variable replacement is not supported.
Then I tried:
"maven.executable.path": "/workspaces/project-folder/mvnw"
but got an error.  This is the log:
 CMD :"/workspaces/project-folder/mvnw" --quiet clean -f "/workspaces/project-folder/pom.xml" && "/workspaces/project-folder/mvnw" --quiet org.apache.maven.plugins:maven-dependency-plugin:3.0.2:tree -f "/workspaces/project-folder/pom.xml" -DoutputFile="/workspaces/project-folder/target/dependencies.txt" -DoutputType=dot -DappendOutput=true
 STDOUT :
 STDERR : /workspaces/project-folder/mvnw: 219: cannot open /vscode/vscode-server/bin/x64/3c4e3df9e89829dce27b7b5c24508306b151f30d/.mvn/wrapper/maven-wrapper.properties: No such file
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed

  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0Warning: Failed to create the file
Warning: /vscode/vscode-server/bin/x64/3c4e3df9e89829dce27b7b5c24508306b151f30d
Warning: /.mvn/wrapper/maven-wrapper.jar: No such file or directory

  2 50710    2  1039    0     0  14842      0  0:00:03 --:--:--  0:00:03 14842
curl: (23) Failed writing body (0 != 1039)
Error: Could not find or load main class org.apache.maven.wrapper.MavenWrapperMain

If I run those commands in the terminal they are executed without any problem and the dependencies.txtfile is generated in the target folder.

Related issue: https://github.com/fabric8-analytics/fabric8-analytics-vscode-extension/issues/541

Also related to: https://github.com/fabric8-analytics/fabric8-analytics-vscode-extension/issues/503

As a workaround I have created a custom mvnw wrapper file for mvnw itself that first changes directory to the workspace folder like (in the example /workspaces/project-folder):
#!/bin/sh
# Hack needed by redhat.fabric8-analytics extension that runs maven commands from vscode-server binary directory (not from workspace directory)
cd /workspaces/project-folder
/workspaces/project-folder/mvnw "$@"

and set the path to that wrapper in maven.executable.path setting.
NOTE: Also take into account that in latest version (0.7.3) maven.executable.path is not respected.  See: https://github.com/fabric8-analytics/fabric8-analytics-vscode-extension/issues/665

I do not have Maven globally installed and I'm getting this error:
Command failed: mvn --version 'mvn' is not recognized as an internal or external command, operable program or batch file.

So as a workaround I added did this in settings.json:
{
  "redHatDependencyAnalytics.mvn.executable.path": "./mvnw"
}

But then I got this error:
Command failed: ./mvnw --version '.' is not recognized as an internal or external command, operable program or batch file.

So as a second workaround I did this:
{
  "redHatDependencyAnalytics.mvn.executable.path": "%USERPROFILE%\\.m2\\wrapper\\dists\\apache-maven-3.9.5-bin\\32db9c34\\apache-maven-3.9.5\\bin\\mvn"
}

Which worked well but now this doesn't work on linux, so as a third workaround I did this:
{
  "redHatDependencyAnalytics.mvn.executable.path": "$HOME/.m2/wrapper/dists/apache-maven-3.9.5-bin/32db9c34/apache-maven-3.9.5/bin/mvn"
}

Which worked well on linux but now this doesn't work on windows.
And then I gave up and removed RedHat Dependency Analytics extension. I just have two questions for the project maintainers:

Why there is no default support for Maven Wrapper?
Why are you making life difficult then it already is?

@harshrathod50 The problem here is that the mvn commands are not run from the VSCode workspace folder (that is needed by mvnw to find the .mvn folder).
As I said here I just created a bash wrapper for the mvnw that before calling the "real" mvnw command it sets the current working directory to the workspace folder and set that wraper as the redHatDependencyAnalytics.mvn.executable.path.
You can try to create a "compatible" wrapper for both bash and windows batch like this.


Add table anchors in About billing for GitHub Actions

What article on docs.github.com is affected?
https://docs.github.com/en/billing/managing-billing-for-github-actions/about-billing-for-github-actions
What part(s) of the article would you like to see updated?
Each of these should have a section so that one could easily link to it:
https://github.com/github/docs/blob/77e4e47c7460acbc0ff6a41897fd2854cc1bf5e3/content/billing/managing-billing-for-github-actions/about-billing-for-github-actions.md#L26-L32
https://github.com/github/docs/blob/77e4e47c7460acbc0ff6a41897fd2854cc1bf5e3/content/billing/managing-billing-for-github-actions/about-billing-for-github-actions.md#L36-L40
https://github.com/github/docs/blob/77e4e47c7460acbc0ff6a41897fd2854cc1bf5e3/content/billing/managing-billing-for-github-actions/about-billing-for-github-actions.md#L52-L56
https://github.com/github/docs/blob/77e4e47c7460acbc0ff6a41897fd2854cc1bf5e3/content/billing/managing-billing-for-github-actions/about-billing-for-github-actions.md#L64-L68
https://github.com/github/docs/blob/77e4e47c7460acbc0ff6a41897fd2854cc1bf5e3/content/billing/managing-billing-for-github-actions/about-billing-for-github-actions.md#L70-L75
Additional information
I want to be able to link to the section that identifies macos runners as considerably more expensive than linux runners, but I can't because there are no sections for the tables.

@jsoref you want the updated doc to have the link that you have mentioned above with each section?

The above snip has one section with its link.

@raunak6662: I have no idea what you're saying.
Here's what I want:
https://github.com/jsoref/github-docs/blob/billing-anchors/content/billing/managing-billing-for-github-actions/about-billing-for-github-actions.md#included-storage-and-minutes-per-month
https://github.com/jsoref/github-docs/blob/billing-anchors/content/billing/managing-billing-for-github-actions/about-billing-for-github-actions.md#operating-system-minute-multiplier
https://github.com/jsoref/github-docs/blob/billing-anchors/content/billing/managing-billing-for-github-actions/about-billing-for-github-actions.md#operating-system-per-minute-rate
https://github.com/jsoref/github-docs/blob/billing-anchors/content/billing/managing-billing-for-github-actions/about-billing-for-github-actions.md#sample-minutes-cost-calculation
https://github.com/jsoref/github-docs/blob/billing-anchors/content/billing/managing-billing-for-github-actions/about-billing-for-github-actions.md#sample-storage-cost-calculation

@jsoref am new to open source and just need a bit more help.
In the attached pic, you want to update the text to a link. right?

If not, cayou please explain me a little more so that I can contribute to it.

@jsoref Thanks so much for opening an issue! I'll triage this for the team to take a look :eyes:

I agree that this would be helpful for users and also for our own support team! You or anyone else is welcome to open a PR to add headers to those sections.


Update of axios to >=0.21.1

Hi!
Would it be possible to update vuetables dependency axios to >=0.21.1?
As can be read at https://www.npmjs.com/advisories/1594, lower versions contain strong security vulnerabilities
Best regards

Yes! That is a great suggestion, will update later today.

Yes! That is a great suggestion, will update later today.

v3.3.3 has been updated with axios v0.21.1.


haskellPackages.tmp-postgres: Add pginit dependency

Motivation for this change
Want to use this package but it didn't pass tests due to missing programs.
Things done

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

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

@peti Are you aware of some way I can propagate the pginit program (necessary for this library to function) onto the PATH of a nix-shell? Basically, it would be nicer for users if the following program were to work without adding -p postgresql to the command line:
#! /usr/bin/env nix-shell
#! nix-shell -i runghc --pure -p "pkgs.haskellPackages.ghcWithPackages (h: [h.tmp-postgres])"
import Database.Postgres.Temp
import Control.Exception
main = startAndLogToTmp [] >>= either throwIO (putStrLn . connectionString)

Do you know some way I can propagate the pginit program (necessary for this library to function) onto the PATH of a nix-shell?

I don't think there's an easy way to accomplish that. The proper solution for this problem is for tmp-postgres to compile the full path to pginit into it's code rather than relying on $PATH.

Thanks, yes that makes sense.


SlimScroll issue

Hi there !
There is an issue with the SlimScroll plugin which do not prevent the entire page to be scrolled when used on any div.
Does anybody know how to prevent the entire page to be scrooled when using slimscroll ?
Many thanks

SlimScroll in sidebar?
If yes, the SlimScroll plugin is only used in fixed layout or in navbar dropdowns.

not in sidebar but in navbar dropdowns. check it on the live preview https://almsaeedstudio.com/themes/AdminLTE/index2.html, in the "message dropdown", if you scroll the message list, the whole page is also scrolling but should not...

If i'm scroll in the dropdown the whole page doesn't move. (FF v50)
With which browser you have that issue?

Also FF50, but I'm on MacOSX. I've never tested it on windows. Are you on windows ?

Yep Windows 10, but wait i test it on macOS Sierra

I doesn't have this issue on macOS Sierra with FF50 or Safari 10

It stay on position even if im on end of the scroll content

I'm still on yosemite. Maybe a yosemite issue. Just tested on chrome on my system and no problem... strange

yep thats strange, other question do you get some console errors?

no, absolutely no error. don't know how to fix it...

maybe i'll close this issue that appears to be related to my environment.

@REJack
If you have any idea on how to fix this, please just let me know


Make QueryResult.row and QueryResultRow public for helping users define their custom types and impl TryGetable

Summary
Make QueryResult.row and QueryResultRow public for helping users define their custom types and impl TryGetable
Motivation
I am using PostgreSQL and I have made some composite types. For ORM, I want to create my custom types and their value parser. When I tried to implement TryGetable, I found it impossible to get information inside QueryResult.
Additional Information

Hey @GreenYun, I think you don't need to make QueryResult.row and QueryResultRow public to implement TryGetable.
Checkout some of the related discussion and PR:

#139
#400


Add Umbraco 13 support?

Feature summary
We are upgrading one of our client projects to Umbraco 13. We use this great package to hide some properties for users, but it seems this package is not ready for Umbraco 13 yet.

@LottePitcher Sorry, I think I was a bit too quick. 😅 I see that a PR has already been created. I will wait patiently until it is merged.

Hey @erikjanwestendorp, yes thanks to that pull request with a fix for v13, I have been able to get a new release out. I've tested v1.1.4 on an upgrade and a new v13 install so 🤞 you can finish upgrading your client's project now (or more likely after Christmas!)
https://www.nuget.org/packages/Our.Umbraco.Community.AdminOnlyProperty/1.1.4
I'd love to hear confirmation that this is all ok as it's the first time I've implemented multi-targeting. So I'll leave this issue open and hope to see it closed when you've had a chance to try again

@LottePitcher Thank you very much for the new version! 😄 I can confirm that this works as expected! Merry Christmas and a Happy New Year! 🎄🎅

@erikjanwestendorp Hurray, thanks for confirming so soon. Happy Christmas and New Year to you too!


FIX mobile view

Just some small props fixes
Issue: https://github.com/storybooks/storybook/issues/5544

@tmeasday please resolve conflicts and update snapshots

@shilman should be good now