Optional arguments should be in square brackets, required arguments should be in angle brackets.

Resolves #14
Modify formatter: optional arguments should be in square brackets, required arguments should be in angle brackets.
Fixed help for remove.
Example help for init:
$ ./antler-proj init --help
Initialize a new project creating the directory tree and a `project.yaml` file.
Usage: antler-proj init [OPTIONS] path project_name [version]

Positionals:
  path <TEXT>                 The root path to create the project in.
  project_name <TEXT>         The name of the project.
  version [TEXT], default: 0.0.0
                              The version to store in the project file.

Options:
  -h,--help                   Print this help message and exit
  -p <TEXT>                   The root path to create the project in.
  -n <TEXT>                   The name of the project.
  -v [TEXT], default: 0.0.0   The version to store in the project file.

Examples:
	antler-proj init MyProjectName 1.0.0

Ohh. This solution looks a little bit as an overkill IMHO. When I wrote #14 I didn't know abilities of the CLI11 so it was my wish to follow BNF notation if possible. But if the CLI library doesn't support BNF IMHO it's OK to use it "as is" instead of adding our own support. Anyway there are many popular CLI programs which don't support it too.
IMHO the bigger problem is duplication of the options. IMHO we should leave only one way to declare arguments: options or positionals, because both of them make the interface confusing and difficult to understand. I prefer options instead of positionals because they are explicit and make less problems with parsing. It is easily to recognize an option and its parameter than correctly parse positional argument and don't confuse them.
And anyway the new help text is incorrect IMHO.
The positionals should look like:
Positionals:
  <path>                    The root path to create the project in.
  <project_name>            The name of the project.
  [version]                 The version to store in the project file. Default: 0.0.0

Because they are not options which have names but positional arguments which recognizing by their position in the command line.
Options:
  -v <version>, default: 0.0.0   The version to store in the project file.

Because parameter  must be declared with -v option. -v can't be used without a parameter so <> instead of []

I'm fine with cancelling this PR together with issue, as I do not feel it gives benefit.

IMHO the bigger problem is duplication of the options. IMHO we should leave only one way to declare arguments: options or positionals, because both of them make the interface confusing and difficult to understand. I prefer options instead of positionals because they are explicit and make less problems with parsing. It is easily to recognize an option and its parameter than correctly parse positional argument and don't confuse them.

Will https://github.com/AntelopeIO/antler-proj/issues/13 cover it fully or partially? If partially we might want to create a seperate issue.

Not necessarily in this PR: we may want to add type_name_fn() to options.

Created issue: https://github.com/AntelopeIO/antler-proj/issues/51

I'm fine with cancelling this PR together with issue, as I do not feel it gives benefit.

IMHO the bigger problem is duplication of the options. IMHO we should leave only one way to declare arguments: options or positionals, because both of them make the interface confusing and difficult to understand. I prefer options instead of positionals because they are explicit and make less problems with parsing. It is easily to recognize an option and its parameter than correctly parse positional argument and don't confuse them.

Will #13 cover it fully or partially? If partially we might want to create a seperate issue.

Not necessarily in this PR: we may want to add type_name_fn() to options.

Created issue: #51

IMHO #13 should be completely rewritten because the parser was significantly changed since I wrote it.
IMHO we should make a meeting where agree how the parser should look in details based on all the registered bugs, write this agreement in a file and then update the parser as close to the agreement as possible but without overengineering.
Or we can review and retest all the  issues which takes the parser, then write a description how the parser should looks to cover all these issues, we discuss this description and then update the parser as agreed.
Or at least collect in one text all requirements, wishes and problems of the CLI to have a base for discussion.
It looks that fixing of the bugs one by one doesn't allow to find a general solution and we will forced to rewrite the same  several times.
Well IMHO it would be better to talk about this problem together and then decide what to do.

Cancelling this PR, as the description of original issue has changed.
I will raise a new PR to fix the description of remove


Invalid code in input string for cyrillic label

Hello!
An attempt to set a label in Russian gives an error:
imagettfbbox(): any2eucjp(): invalid code in input string (2)

There are no mistakes in English.
Also, there are no errors if you encode the text in Russian with UTF-8 encoding. The error only occurs with a label.
$encodedText = 'Hello!';
$qrCode = new QrCode($encodedText);
$qrCode->setLabel('Текст на русском', 16, __DIR__ . '/../../vendor/endroid/qr-code/assets/fonts/noto_sans.otf');

Hi @erkinsergey unfortunately I can not reproduce the error on this end. If I Google the error message it appears to have something to do with your GD installation, see https://www.programmersought.com/article/80162245565/#_php___enablegdjisconv__13. Maybe you can try that or try the provided to_entities workaround and report back if that helped. Or you could try running in a different environment / PHP installation to rule this out.
Good luck!

Thanks for the quick response!
Yes, I saw this article when I was looking for an answer.
I really didn't get the error on another PHP installation. This is not your library's fault.


cli: mud test should use top level test dir (rather than src/test)

Or really, it probably ought to figure this out via foundry config so we can consolidate on a single source of truth for configuring this?
This somewhat blocks moving towards checked-in Solidity artifacts (e.g. ABIs)

yep, this is actually what I do in #309. The config part gets a bit lost in the other changes tho

I could extract the config/test changes as a separate PR if you want?
but it would still be a breaking change (everyone has test by default in their foundry.toml, but actual tests are in src/test according to templates etc)

I could extract the config/test changes as a separate PR if you want? but it would still be a breaking change (everyone has test by default in their foundry.toml, but actual tests are in src/test according to templates etc)

I think separating it out makes sense (I like well contained PRs) and I personally don't mind a breaking change like this (it's an easy fix for users, we should just document it).

https://github.com/latticexyz/mud/pull/392


py312 build

py312 build

Linter check found the following problems:
ERROR conda.cli.main_run:execute(124): `conda run conda-lint /tmp/abs_299yukvd32/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:56: missing_license_file: The recipe is missing the about/license_file or about/license_url key.

===== ERRORS =====

clone/recipe/meta.yaml:56: missing_license_family: The recipe is missing the about/license_family key.
clone/recipe/meta.yaml:48: incorrect_license: The recipe's about/license key is not an SPDX compliant license or license exception, reference https://spdx.org/licenses/
===== Final Report: =====
2 Errors and 1 Warning were found


Update README.md

Please add a note for Typescript in the README. I've spent one day to figure out a fix.

Hi @thednp, I don't think putting the types in vite.config.mjs is a good place.
Why didn't the types property in the tsconfig work for you? Do you have a vite-env.d.ts file? Because that would be a better place to put the types.
/// <reference types="vite-plugin-solid-svg/types-component-solid" />
/// <reference types="vite/client" />

I think the recommended place is whatever the "vite/client" config is.

The types didn't work for me in SolidStart 0.4.2 for some reason, probably because vinxi is a wrapper around vite, but setting vite-env.d.ts indeed solves the problem.
Thank you!


Playstore - add instructions how to connect if scan doesn't work

Scan doesn't find the instance.
When I manually enter some URL (local network, duckdns, with or without port) nothing will be found, I get only error messages . This normally will result in a bad rating.
I suggest to add some more instructions how to use the app, what are the requirements for a successful scan, what to try if scan doesn't work.

https://github.com/home-assistant/home-assistant-android/issues/6

manual entering the URL also will not find it. But it looks that some users are able to manual enter.
That's why I suggest to add some information about the requirements and the format how to enter the URL manually. I would like to test the app, but I can't if I can't connect.

Some help to enter the url : https://github.com/home-assistant/home-assistant-android/pull/66

Here's what I tried:

automatic scan - failed
manually enter hassio.local:8123 - failed, couldn't click button
manually enter http://hassio.local:8123 - failed with error message
manually enter eternal url (hosted via nabu casa) - worked!

Android does not support zeroconf/mdns, so hassio.local does not work. Once we add #6, we will find it.

http://<IP_ADDRESS>:8123 doesn't work in the app. It works from my desktop's browser.

Might be the same issue as mine: https://community-home-assistant-assets.s3.dualstack.us-west-2.amazonaws.com/original/3X/9/8/9806d0a94945806a242e148f0b0fde6a0257053b.jpeg

Example has been added and will be in next release.


FIX: Use numpy.linspace to calculate mosaic plots' cutting planes

References: #281.

Reportlets generated by tests look good to me.


Fix broken link to "Required" section

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

Thanks @dchristensen.


SAI-4526 : Race condition in CoordinatorHttpSolrCall during synthetic collection/replica creation/lookup

Description
From a unit test case that issue concurrent select queries to coordinator nodes, it’s found that there could be 3 race condition issues:

If multiple concurrent requests find the synthetic collection is not yet created, they might all attempt to create the synthetic collection. This could trigger SolrException on collection already exists

Similarly, if multiple concurrent requests find there’s no replica of the synthetic collection for current node (multiple coordinator node scenario), then CoordinatorHttpSolrCall#addReplica could be invoked multiple times. This should not trigger any exception, but would create multiple replicas for the same node in the synthetic collection

The existing logic here assumes if syntheticColl.getReplicas(solrCall.cores.getZkController().getNodeName()) returns non empty result, then the following call in here should return a core. Unfortunately, the first call can return a non empty list but with a DOWN replica if another request is in the progress of creating such replica. In this case, the solrCall.getCoreByCollection(syntheticCollectionName, isPreferLeader) would call super.getCoreByCollection at here which would return a null (since super impl only returns active replica). So CoordinatorHttpSolrCall#getCoreByCollection would end up calling CoordinatorHttpSolrCall#getCore , introducing an infinite loop and cause stackoverflow

Take note that these race condition should be pretty rare, considering that it only happens if there are multiple concurrent requests when the synthetic collection/replica was first created.
Solution

For collection creation exception, check again if the collection exists, if so, ignore the exception and proceed
For replica, if the replica for such node already found in the DocCollection, then ensure that it's active using zkStateReader.waitForState. This avoids the infinite loop caused by the presence of down replica.

Take note that this does NOT avoid the 2nd issue above, concurrent requests can still create multiple replica for the same node in the synthetic collection, though it's probably benign (and unlikely)
Remarks
First attempt was actually provide proper locking to avoid race condition. However, it's quite tricky to get it right - might need to force refresh the zkReader and do multiple extra reads. The extra cost and complexity probably does not justify the gain.

Closing as it's addressed in upstream in https://issues.apache.org/jira/browse/SOLR-16871


WIP: Missile Launcher new component

Description:
In the spirit of hacktoberfest, I'm am submitting a custom component I've had implemented for a while.
It allows integration of a USB Missile Launcher (the foam ones) into HASS.
Since the missile launch can control the movements along with having a camera on it, I tried to figure out where to best implement it (as I didn't find any example of integrating a camera that you can also control.) So I implemented it as a media_player, which can show the image and the select source has the commands so that you can control it.
(I added a couple @TODO statements of areas that I need feedback on.)
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#<home-assistant.github.io PR number goes here>
Example entry for configuration.yaml (if applicable):
media_player:
platform: missile_launcher
Checklist:
If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] New dependencies have been added to the REQUIREMENTS variable (example).
[ ] New dependencies are only imported inside functions that use them (example).
[ ] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

Hello @ABOTlegacy,
When attempting to inspect the commits of your pull request for CLA signature status among all authors we encountered commit(s) which were not linked to a GitHub account, thus not allowing us to determine their status(es).
The commits that are missing a linked GitHub account are the following:

b98a5c2f31c117cb2a3e759f290e7644cce1eb31 - This commit has something that looks like an email address (abottom@wennsoft.com). Maybe try linking that to GitHub?.

Unfortunately, we are unable to accept this pull request until this situation is corrected.
Here are your options:

If you had an email address set for the commit that simply wasn't linked to your GitHub account you can link that email now and it will retroactively apply to your commits. The simplest way to do this is to click the link to one of the above commits and look for a blue question mark in a blue circle in the top left. Hovering over that bubble will show you what email address you used. Clicking on that button will take you to your email address settings on GitHub. Just add the email address on that page and you're all set. GitHub has more information about this option in their help center.

If you didn't use an email address at all, it was an invalid email, or it's one you can't link to your GitHub, you will need to change the authorship information of the commit and your global Git settings so this doesn't happen again going forward. GitHub provides some great instructions on how to change your authorship information in their help center.

If you only made a single commit you should be able to rungit commit --amend --author="Author Name<EMAIL_ADDRESS>
(substituting Author Name and<EMAIL_ADDRESS>for your actual information) to set the authorship information.
If you made more than one commit and the commit with the missing authorship information is not the most recent one you have two options:

You can re-create all commits missing authorship information. This is going to be the easiest solution for developers that aren't extremely confident in their Git and command line skills.
You can use this script that GitHub provides to rewrite history. Please note: this should be used only if you are very confident in your abilities and understand its impacts.

Whichever method you choose, I will come by to re-check the pull request once you push the fixes to this branch.

We apologize for this inconvenience, especially since it usually bites new contributors to Home Assistant. We hope you understand the need for us to protect ourselves and the great community we all have built legally. The best thing to come out of this is that you only need to fix this once and it benefits the entire Home Assistant and GitHub community.
Thanks, I look forward to checking this PR again soon! :heart:

Hello @ABOTlegacy,
When attempting to inspect the commits of your pull request for CLA signature status among all authors we encountered commit(s) which were not linked to a GitHub account, thus not allowing us to determine their status(es).
The commits that are missing a linked GitHub account are the following:

76c0245817d2932d1d00ff6bc0e2edf95f0197f5 - This commit has something that looks like an email address (abottom@wennsoft.com). Maybe try linking that to GitHub?.
b98a5c2f31c117cb2a3e759f290e7644cce1eb31 - This commit has something that looks like an email address (abottom@wennsoft.com). Maybe try linking that to GitHub?.

Unfortunately, we are unable to accept this pull request until this situation is corrected.
Here are your options:

If you had an email address set for the commit that simply wasn't linked to your GitHub account you can link that email now and it will retroactively apply to your commits. The simplest way to do this is to click the link to one of the above commits and look for a blue question mark in a blue circle in the top left. Hovering over that bubble will show you what email address you used. Clicking on that button will take you to your email address settings on GitHub. Just add the email address on that page and you're all set. GitHub has more information about this option in their help center.

If you didn't use an email address at all, it was an invalid email, or it's one you can't link to your GitHub, you will need to change the authorship information of the commit and your global Git settings so this doesn't happen again going forward. GitHub provides some great instructions on how to change your authorship information in their help center.

If you only made a single commit you should be able to rungit commit --amend --author="Author Name<EMAIL_ADDRESS>
(substituting Author Name and<EMAIL_ADDRESS>for your actual information) to set the authorship information.
If you made more than one commit and the commit with the missing authorship information is not the most recent one you have two options:

You can re-create all commits missing authorship information. This is going to be the easiest solution for developers that aren't extremely confident in their Git and command line skills.
You can use this script that GitHub provides to rewrite history. Please note: this should be used only if you are very confident in your abilities and understand its impacts.

Whichever method you choose, I will come by to re-check the pull request once you push the fixes to this branch.

We apologize for this inconvenience, especially since it usually bites new contributors to Home Assistant. We hope you understand the need for us to protect ourselves and the great community we all have built legally. The best thing to come out of this is that you only need to fix this once and it benefits the entire Home Assistant and GitHub community.
Thanks, I look forward to checking this PR again soon! :heart:

@homeassistant Way to out a guys email address lol, also this looks pretty cool, just to note.


Perforce update

This is the latest from //tinyprods/iknow-github/latest

This is our version 1.0, fully functional, with up-to-date language models, thanks to everybody who helped making this a reality !!!


Files and documents from the SANBI CWL tutorial session (March 2017).

Some simple tools and workflows as discussed on gitter.

The build error is due to https://github.com/common-workflow-language/cwltool/commit/ab9e86e6f9c4cb25381f047a449c098c9aa4ff02#diff-2eeaed663bd0d25b7e608891384b7298L50

I made an issue for us to think about the html5lib problem: https://github.com/common-workflow-language/cwltool/issues/347

Thank you @pvanheus !


"Warning: control reaches end of non-void function." - Should I worry about that?

While compiling in Ubuntu 18.10 I get:
/fc/src/log/log_message.cpp:165:4: warning: control reaches end of non-void function [-Wreturn-type]
}

Nothing to worry about. Just a warning.


Microsoft Stream on sharepoint doesn't work with the custom help pane

Microsoft Stream on sharepoint doesn't work with the custom help pane

Learn Build status updates of commit 24129f1:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

powerapps-docs/maker/data-platform/create-custom-help-pages.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


upcoming: [M3-7580] - Add search field to Create Linode UIs

Description 📝
Adds a search field to the Clone Linode and Create Linode from Backup flows to allow users to filter Linodes based on their labels.
These changes are gated behind the linodeCloneUIChanges feature flag.
Changes  🔄

Add search field to Create Linode Clone and Backup flows
When a user selects "Clone" on an existing Linode, navigate to Clone Linode UI, filter on the Linode's label and pre-select that Linode
Refactored DebouncedSearchTextField to allow field state to be externally managed

This was necessary to allow automatically filtering from the query param

Preview 📷

Before
After

How to test 🧪

Navigate to the Create Linode page
Select the "Clone Linode"/"Backups" tab
Ensure you can filter Linodes based on their label
Click on a Linode's action menu and select "Clone Linode"
Verify the Linode is selected in the list and the search field is pre-filled with the Linode's label
Verify you can clear and edit the contents of the search field after it is pre-filled
Verify existing usages of DebouncedSearchTextField continue to work as expected

As an Author I have considered 🤔
Check all that apply

[x] 👀 Doing a self review
[x] ❔ Our contribution guidelines
[x] 🤏 Splitting feature into small PRs
[x] ➕ Adding a changeset
[ ] 🧪 Providing/Improving test coverage
[ ] 🔐 Removing all sensitive information from the code and PR description
[x] 🚩 Using a feature flag to protect the release
[x] 👣 Providing comprehensive reproduction steps
[ ] 📑 Providing or updating our documentation
[ ] 🕛 Scheduling a pair reviewing session
[ ] 📱 Providing mobile support
[ ] ♿  Providing accessibility support

I know you mentioned that the size of the search bar came from the mock up, however, I think it would align better with the other input fields if we match the width accordingly.

I know you mentioned that the size of the search bar came from the mock up, however, I think it would align better with the other input fields if we match the width accordingly.

Thanks for the suggestions. I will check with UX.


fix: updated the docs

Description
Fixed spacing issues

What type of PR is this? (check all applicable)

[ ] 🍕 Feature
[ ] 🐛 Bug Fix
[x] 📝 Documentation Update
[ ] 🎨 Style
[ ] 🧑💻 Code Refactor
[ ] 🔥 Performance Improvements
[ ] ✅ Test
[ ] 🤖 Build
[ ] 🔁 CI
[ ] 📦 Chore (Release)
[ ] ⏩ Revert

Related Tickets & Documents

Fixes #145
Mobile & Desktop Screenshots/Recordings

Added tests?

[ ] 👍 yes
[x] 🙅 no, because they aren't needed
[ ] 🙋 no, because I need help

Added to documentation?

[ ] 📜 README.md
[ ] 📓 docs.opensauced.pizza
[ ] 🍕 dev.to/opensauced
[ ] 📕 storybook
[x] 🙅 no documentation needed

[optional] Are there any post-deployment tasks we need to perform?
[optional] What gif best describes this PR or how it makes you feel?

Hey @dev-phantom! :) There are some merge conflicts on your branch, so you need to fix them.

Hey @dev-phantom the highlights page has been updated and changed, so this is no longer needed.


Updated composer json to support php8.1

No major change added, only bumped PHP version to support ^8.1 and ran composer test to ensure current functionality are working as expected

@cviebrock can you please review and share feedback

This push is not required, the current ^8.0 already supports the recent PHP 8.1 release update


Fix EVM connect wallet button on solana pages

📝 Changed routes:

/solana/[...paths]

(view full size)

Commit bf738d64bf168128ee99b98d1113fb0447d4770b (https://thirdweb-9xbrjuxws.thirdweb-preview.com).


fix: 5087 carousel card layout

Thank you for your contribution to the KodaDot NFT gallery.
👇 _ Let's make a quick check before the contribution.
PR Type

[x] Bugfix
[ ] Feature
[ ] Refactoring

Context

[x] Closes #5087
[ ] Requires deployment <>

Before submitting pull request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[x] I've didn't break any original functionality
[x] I've posted a screenshot of demonstrated change in this PR

Optional

[x] I've tested it at </bsx/collection>
[x] I've tested PR on mobile
[ ] I've written unit tests 🧪
[ ] I've found edge cases

Had issue bounty label?

[x] Fill up your KSM address: Payout

Community participation

[ ] Are you at KodaDot Discord?

Screenshot 📸

[x] My fix has changed something on UI; a screenshot is best to understand changes for others.

pay 10 usd

😍 Perfect, I’ve sent the payout
💵 $10 @ 38.98 USD/KSM ~ 0.257 $KSM
🧗 JHJGu2aWyAFBfcso8bR7ma82p15YAavyN6vQfJyWuhniNGa
🔗 0x1b21522dc87b40dda66c395f1a233c03ca0bd2e29103b92a666c33be465a686c
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


Implement Release Notes Gift Icon in Desktop App

We should keep a changelog here. Maybe VS Code style with GIFs, might help especially in relation to #32 and #39.

On launch, we should open a pop-up in front. Perhaps the pop-up should automatically be de-focused if a line is searched from mobile etc

Add a gift icon that glows/highlighted when there's new release notes etc, should open a completely separate window

Using GitHub API we can get html description. Assuming that we can upload gifs and all to the release notes section. I suggest instead of having changelog there (as we do right now) and move to a big CHANGELOG.md file. This will allow us to remove release-notes dir from the repo and we don’t have to worry about LFS


work with Chinese video website

Dear Sir,
I'm from China. Sorry for any inconvenience with my poor English.
I really like your extension but I just found it cannot work on Chinese video website like bilibili.com or douyin.com. I try to download your source code and change "host_permissions" and "matches" into "https:///".  And then it works very fine on some Chinese video webistes like bilibili.com, douyin.com and so on.
I just want to share this information to you or someone also need use this extension on Chinese webiste. Thanks for your work and your extension.

Hi @dean840513 and thanks for the feedback! I know it works on every site with that setting, but it's hard to get it approved to the Chrome Web Store like that. So that is why the scope is limited.
Sorry for this! But as you said, you can download the extension from here and change it to make it work on sites you need. Thanks for pointing that out! I will leave this issue open for reference!


Dockerfile now installs perl for gitsubmodules; closes #128.

closes #128

please format to for readability - see lines 20-24


k3s_1_27: 1.27.13+k3s1 -> 1.27.14+k3s1

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for k3s_1_27 is: A lightweight Kubernetes distribution
meta.homepage for k3s_1_27 is: https://k3s.io
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.27.14+k3s1 with grep in /nix/store/8xfc6z4p6vcqdsyiqv6sdadbcx7hzw3m-k3s-1.27.14+k3s1
found 1.27.14+k3s1 in filename of file in /nix/store/8xfc6z4p6vcqdsyiqv6sdadbcx7hzw3m-k3s-1.27.14+k3s1

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

k3s_1_27

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/8xfc6z4p6vcqdsyiqv6sdadbcx7hzw3m-k3s-1.27.14+k3s1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A k3s_1_27 https://github.com/r-ryantm/nixpkgs/archive/3e38946f078437ed24618b5fb7b5badde211664c.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/3e38946f078437ed24618b5fb7b5badde211664c#k3s_1_27

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/8xfc6z4p6vcqdsyiqv6sdadbcx7hzw3m-k3s-1.27.14+k3s1
ls -la /nix/store/8xfc6z4p6vcqdsyiqv6sdadbcx7hzw3m-k3s-1.27.14+k3s1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

1 package built:

k3s_1_27

Maintainer pings
cc @euank @Mic92 @superherointj @wrmilling @yajo for testing.

@ofborg build k3s_1_27.passthru.tests.etcd k3s_1_27.passthru.tests.single-node k3s_1_27.passthru.tests.multi-node


Feature request: implement syslog backend

Act as a syslogd remote (UDP, TCP, TCP+TLS) so logvoyage can easily used by everyone for nearly everything by just adding:
*.*                                         @<logvoyage-host>

to ryslogd.conf

Great. Will investigate this and update documentation.

awesome.
Ideally logvoyage should also work as server/target for https://github.com/papertrail/remote_syslog2


Make saving ttrt artifacts optional

I think we can make saving these disabled by default:
File 'out.ttm' copied to '/home/nsmith/src/tt-mlir/ttrt-artifacts/out' successfully.
File 'out/input_0.pt' saved to '/home/nsmith/src/tt-mlir/ttrt-artifacts/out/input_0.pt' successfully.
File 'out/output_0.pt' saved to '/home/nsmith/src/tt-mlir/ttrt-artifacts/out/output_0.pt' successfully.
File 'out/system_desc.ttsys' saved to '/home/nsmith/src/tt-mlir/ttrt-artifacts/out/system_desc.ttsys' successfully.

Let's add a new flag like --save-artifacts or --save-inputs or --save-inputs-on-failure.

merged: 4ab584532d2ea6c169f4492193e0b50d672267eb


SL Review: Send email to ADMs after final submission

User Story
As an ADM i want to get an email after final submission so that I have a record of what was submitted
AC

[ ]  ADMs to get an email with each submission after final submission
[ ] Email should contain the PDF of the form

Copy for email-
The   Senior Leadership Review (SLR) process is now over. Thank you for completing the form(s) for your senior leader(s). Shortly, you will receive an email with a PDF copy of the form(s) you have completed l for your records. we encourage you to share it with the senior leader(s).
The information you’ve provided will be shared with Executive Talent Management, Corporate Workforce Strategies in the Public Service Agency, ministry strategic human resources (ministry-specific data), and DMC/DMCPSR to inform executive succession and development strategies, programs and activities.
Now that the SLR process is complete, the senior leader can target their learning and development plan, goals, and actions in support of their aspirations and readiness to compete for an executive role.
Since the Senior Leadership Review is a biennial process, there will be opportunities in the future for other senior leaders in your organization who aspire to executive roles to participate in the SLR process.
If you have any questions, please contact Jodi Little, Leadership Development and Succession Lead.

@HasanSifat  Some questions:

When you say "ADM to get 1 email with pdf attachments of all submissions made by the ADM." By "All the submissions" do you mean each time ADM clicks "Submit for Validation", we should attach a copy of that submission in the email? We currently update the submission each time ADM clicks "Submit for validation" and then clicks "Final submission". So we only have the final version to attache wants to change that.

For deletion of "all drafts", do you mean SL Review's drafts that were never submitted? Currently, we delete the ones that ADM actually submits.

"ADMs cannot come back and edit their submissions", do you want the submissions will be kept in the "submitted forms" tab just not editable thereafter?

What do we do about forms that were in draft, after Jan 27 deadline?
What do we do about submitted forms after Jan 27 deadline?

make custom group for late submitters and make form accessible to that group after Jan 27 deadline

@JodiLittle Jodi to give PSA team a few week's (or more) notice to add names into Key Cloak - two weeks minimum but three weeks is better.

UPdated wording from Jodi:

Follow up email after final submission:

The Senior Leadership Review (SLR) process is now over. Thank you for completing the form(s) for your senior leader(s). Shortly, you will receive an email with a PDF copy of the form(s) you have completed for your records. We encourage you to share it with the senior leader(s).
The information you’ve provided will be shared with Executive Talent Management and Corporate Workforce Strategies in the Public Service Agency, ministry SLR key contact (ministry-specific data), and DMC/DMCPSR to inform executive succession and development strategies, programs and activities.
Now that the SLR process is complete, the senior leader can target their learning and development plan, goals and actions in support of their aspirations and readiness to compete for an executive role.
The Senior Leadership Review is expected to occur every two years and there will be other opportunities for other senior leaders to come forward in future processes.
Expect us to contact you with a survey about your experience with this SLR process.
If you have any questions, please contact the SLR Team

@Andrew-Vargas-bcgov how do we test the final submission email and message?
In TEST nothing happens, and in DEV I get the "Almost Done" message with last years date and no email

@Stella-Archer is this one that Iman could pick up? or is this one not needed?

The final email message has been updated:
[Insert ADMs name],
Attached is a PDF copy of the Senior Leadership Review submission for [insert senior leaders name]. Please save it for your records and share it with [insert senior leaders name]. Note that the file will be deleted from the system on November 10, 2023.
The senior leader can utilize this information to shape their learning and development plan, inform goals, and actions to support their career aspirations, and build readiness to compete for an executive role.
Thank you for participating in the Senior Leadership Review process. If you have any questions, please contact the SLR Team.
Link "SLR Team to :
<EMAIL_ADDRESS>

Late breaking update to the above:
,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,
[Insert ADMs name],
Attached is a PDF copy of the Senior Leadership Review submission for [insert senior leaders name]. Please save it for your records and share it with [insert senior leaders name]. Note that the file will be deleted from the system on December 5, 2023.
The senior leader can utilize this information to shape their learning and development plan, inform goals, and actions to support their career aspirations, and build readiness to compete for an executive role. For their development, the senior leader can consider:
•	targeted experiential and exposure opportunities
•	formal ministry learning opportunities (e.g., in-house leadership development programs)
•	formal or experiential corporate learning opportunities such as senior leadership courses, confidential coaching services and career support services.
Thank you for participating in the Senior Leadership Review process. If you have any questions, please contact the SLR Team.
,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,
*note new bullets added
Link "SLR Team to :
<EMAIL_ADDRESS>

added to dev. will move to test in the morning.

moved to test

Tested and verified working!

@Andrew-Vargas-bcgov
we need to add some links as highlighted below

[ ] senior leadership courses
Link > https://learningcentre.gww.gov.bc.ca/learninghub/?s=Senior+leader&post_type=courses
[ ] confidential coaching services
Link > https://www2.gov.bc.ca/gov/content/careers-myhr/all-employees/career-development/coaching-services#services
[ ] career support services
Link > https://www2.gov.bc.ca/gov/content/careers-myhr/all-employees/safety-health-well-being/health/efas/career-services

Tested and verified updated ✔

added links works

Brilliant! I've tested all the 3 new links - love the name differentiation too
PO Approved & Closing this tix


fix: install nvm before node

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

CircleCI Orb
GitHub release

Your semantic-release bot :package::rocket:


#2705 - Removed 2021-2022 E2E Tests and E2E Files Renamed

Removed 2021-2022 E2E tests, and removed the program year from E2E files already in program year folders.
Deleted 2021-2022 E2E Tests as requested in the ticket.

Remove the program year from the file names.

Thanks for doing the cleanup. Looks like in 2023-2024 alone has some files still having py in file names.


[Question / Bug]: Cancelling GPG passphrase freezes page

Hey,
I'm not sure if this is a bug, or unavoidable behavior by GPG, but it definitely has something to do with Obsidian.
A few days ago I created a note, test.md. I had opted in the settings for a note to be .gpg instead of .md, so test.md became test.gpg.
I let that note sit for a few days and decided to mess around with this plugin today, and just for testing purposes, I wanted to see what would happen if I clicked on the note to open it, but then cancelled the GPG prompt to enter my passphrase.
Once I clicked cancel, Obsidian will not allow me to click on any other note at all. All it does is just bring up a Obsidian notification which is the typical GPG error about cancelling the operation.
It will let me re-click on the note I originally tried to open and it prompts me for my passphrase again, but clicking cancel starts to loop all over. I can't click on any other note to open it until I actually click on that note and enter the passphrase.
Keeping in mind that the note I clicked cancel on is the only note in the vault encrypted. The rest are not and are just standard .md files.
I can however, open a new tab and start navigating around again, but I'd have to close the original tab because it's now useless unless I actually enter that passphrase.

hi @Aetherinox did you log this in the wrong plugin repo?

Jesus..... I have your plugin installed and the GPG encryption plugin.
Yeah, that was my bad. I only have yours starred. Not the other. The holidays are starting to mess with my brain already.
Sorry lol

Ha, all good mate 😊👍


Renaming qstudio.jar - odd behavior

I downloaded qStudio for Linux/Mac (3.03) and moved it to /Applications/qStudio on my Mac.
I renamed it to qStudio-3.03.jar then issued:
java -jar qStudio-3.03.jar
and I was unable to open a SQLite database that was in the same directory. Is this expected? Is qStudio sensitive to the name of its .jar file? Or could I be doing something wrong? Thanks.

There should be no change in behaviour based on name.
I've now added logging as you suggested to capture the current directory to allow debugging thi sbetter in future.


Add another shop to Merrymellow Marsh

It's a little strange if the entire town only has two businesses, both restaurants. Add a third and fourth business; perhaps a rug store and a shipping store.

Presumably the town has a gym, or at least that's what Bones says.


json5 markdown support

Added json5 as an identifier in JSON language, to add support for json5 in markdown
Closes Microsoft/vscode(#59567)

Thanks. Yes this looks like the correct change but you need to run the build script to rebuild the grammar too

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

@mjbvz do we need package-lock.json also to be committed in the repo?
I have build and added the build files

No, since  you have not updated any dependencies you should not commit the package-lock. Please remove it from this change as it is blocking merging this PR

@mjbvz Done with the change, reverted package.json file to previous commit

Thanks! Will be in the first VS Code 1.29 insiders build


No usable version of libssl found

Description

I'm trying to start using this but I can't add a dependency because no usable version of libssl found.
Expected Behavior

As a "standalone executable" it would work on a typical ubuntu system.
Actual Behavior

Returned error: No usable version of the libssl was found
Steps to Reproduce

buckaroo init
buckaroo add buckaroo add<EMAIL_ADDRESS>

Context

I am trying to take an old cmake project out of the stone age but I couldn't get dependencies to resolve on Windows (I may log a bug for that later) and I can't seem to get it to work on my linux system either.
Your Environment

Version used: 2.2.0
Operating System and Architecture: Ubuntu 19.10, x64

Thanks for opening your issue.
First of all: buckaroo needs a buckaroo.toml to be present in the project's repository.
It doesn't look like wxWidget has not been ported yet.
Some libraries have been ported to buckaroo and are available here:
https://github.com/buckaroo-pm
Regarding the error message, I think we should provide an explanation why no version could be found.

I'm having this issue too now.
$ buckaroo add github.com/buckaroo-pm/SFML
info Adding dependency on github.com/buckaroo-pm/sfml at *
[git] info Fetching refs from https://github.com/buckaroo-pm/sfml.git
No usable version of the libssl was found

hmm...  As far as I remember sfml doesn't have libssl as a dependency.
Could you post your buckaroo.toml and buckaroo.lock.toml (if available)?

Nevermind, I don't know what happened but it seems to work fine now.

I made a new project using buckaroo quickstart and am having the same problem. buckaroo.toml is empty

I ran into this on Arch, and was able to work around it by installing openssl-1.0. The libssl in question doesn't seem to be a dependency of any package, but of buckaroo itself. From what I can tell, it's being loaded via libdl in System.Security.Cryptography.Native.OpenSsl.so.

Is there a fix for this yet? I'm running into the same issue.

Having the same issue I'm afraid


Suggested edits for README

This is a repo for the Machine Learning Immunogenicity Team in the August 2016 NCBI Hackathon
This project looks into the application of Machine Learning (ML) techniques in the prediction of Immunogenicity (Categorical; Positive or Negative) based on a peptide and its associated amino acid properties. This study uses peptide data from the Immune Epitode Database (IEDB).  The R package "Peptides" has been used to compute the amino acid properties and mashup with peptide data to enable the use of ML algorithms for immunogenicity analysis, particularly, the algorithms that are more efficient with numeric and categorical data instead of string sequence.
Tensorflow is an open source software library ML that provides linear regression and classification algorithms (open sourced by Google in Nov 2015) for multi-dimensional arrays (aka “Tensors”). K-fold cross-validation as well as hold-out of test data was used to train and test the generated models.
Initial application of Logistic Regression (LR) and Neural Networks (NN) looks promising with approximately 82% and 90% predictive accuracy respectively. Various other ML algorithms such as variants of Neural Networks such as Convoluted NN, RESNET, MUST-CNN as well as Random Forest, Bayesian Networks should be considered as part of future work.
The following are provided:
R scripts for data wrangling of IEDB data and mashup with Amino Acid properties
Python Notebook for application of Logistic Regression and Neural Networks using Tensorflow

Comments addressed.


Proguard configuration not exported

Lib documentation states:

If you use proguard, add the following line to your proguard rules: -keep class no.nordicsemi.android.log.** { *; }

This manual step is easy to forget (especially if the logger is a transient dependency) and can be automated. Reference: https://developer.android.com/studio/projects/android-library, the section "A library module may include its own ProGuard configuration file".
In essence, change the build.gradle to say something like
    defaultConfig {
        consumerProguardFiles 'lib-proguard-rules.txt'
    }

And include the rule in the lib-proguard-rules.txt file.

Fixed in 2.1.2


Run CI in python3 as well

Run dual_quaternions tests with Python3 in Travis CI. This requires a few modifications:

In setup.py, from distutils.core import setup --> from setuptools import setup although this old article says it doesn't like it but I think that can be disregarded.
Code needs to become python3 compatible. Just need to fix one test!
Modify Travis config to run python3. Not sure how that would work with ROS installation.

on 3: you might just want to have a seperate job so that it tries it with clean python 3, assuming ROS isn't really needed. Then you can have tests that isolate the different use cases.
Also: just started using github actions, pretty cool

That's the plan. I saw github actions, might try that for the next project!


tf.signal.stft does not work in eager execution mode

Please make sure that this is a bug. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Manjaro
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): 1.14.0
Python version: 3.7
CUDA/cuDNN version: 10.1/7.6
GPU model and memory: GTX 1060

Describe the current behavior
tf.signal.stft does not work in eager execution mode, producing the error:
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/lib/python3.7/site-packages/tensorflow/python/ops/signal/spectral_ops.py", line 83, in stft
    signals, frame_length, frame_step, pad_end=pad_end)
  File "/usr/lib/python3.7/site-packages/tensorflow/python/ops/signal/shape_ops.py", line 120, in frame
    num_outer_dimensions = array_ops.size(outer_dimensions)
  File "/usr/lib/python3.7/site-packages/tensorflow/python/util/dispatch.py", line 180, in wrapper
    return target(*args, **kwargs)
  File "/usr/lib/python3.7/site-packages/tensorflow/python/ops/array_ops.py", line 416, in size
    return size_internal(input, name, optimize=True, out_type=out_type)
  File "/usr/lib/python3.7/site-packages/tensorflow/python/ops/array_ops.py", line 437, in size_internal
    num_elements = np.prod(input._shape_tuple(), dtype=np_out_type)  # pylint: disable=protected-access
  File "/usr/lib/python3.7/site-packages/numpy/core/fromnumeric.py", line 2772, in prod
    initial=initial)
  File "/usr/lib/python3.7/site-packages/numpy/core/fromnumeric.py", line 86, in _wrapreduction
    return ufunc.reduce(obj, axis, dtype, out, **passkwargs)
TypeError: int() argument must be a string, a bytes-like object or a number, not 'NoneType'

Describe the expected behavior
tf.signal.stft should work as it does without eager execution
Code to reproduce the issue
import tensorflow as tf
tf.enable_eager_execution()
input = tf.keras.Input([None])
spec = tf.signal.stft(input, 400, 160)

Was able to reproduce the error in TF 2.0 Beta and also in TF1.14.

@pplantinga Hi, I think you just missed the lambda layer. Because the functional API is used to building graph, so the tensor operations have to be wrapped in tf.keras.layers.Layer instance. So, a workable example should be
import tensorflow as tf

tf.enable_eager_execution()

input = tf.keras.Input([None])
spec = tf.keras.layers.Lambda(lambda x: tf.signal.stft(x, 400, 160))(input)
model = tf.keras.Model(inputs=input, outputs=spec)

y = model(tf.random.uniform(minval=-100, maxval=100, shape=(100, 10000), dtype=tf.float32))
print(y)

If you don't really want to build graph via functional API, the following example can manipulate stft result eagerly:
import tensorflow as tf

tf.enable_eager_execution()

x = tf.random.uniform(minval=-100, maxval=100, shape=(100, 10000), dtype=tf.float32)
print(tf.signal.stft(x, 400, 160))

See more details about functional API. Thanks!


Hour and Minute Arrows Not Visible

The hour and minute up/down arrows are not visible for me (using version 5.0.0-beta.9).
This appears to be because node_modules/ng-pick-datetime/assets/style/picker.min.css doesn't have the icon-up-open:before and icon-down-open:before styles that I see on the demo site.
I was able to work around this by making my own copy of picker.min.css and adding the additional styles at the top and adding the font assets.

@jlenthe2
Yes, I made a silly mistake. Forgot to add the font style.
Update to the version 5.0.0-beta.10 should be all right
Thanks.


[Bug]: Jailbreak to enable Sydney; the "main" branch of LibreChat does not contain the path client/defaultSystemMessage.md

Contact Details
No response
What happened?
When using Bing AI and turning on Sydney, if you click on the button "System message", (it's blue and right below the checkbox to enable Sydney) it returns to a 404 page of the LibreChat repository with the error "the "main" branch of LibreChat does not contain the path client/defaultSystemMessage.md"
Steps to Reproduce

Select Bing AI as your Chatbot
Go to session settings
Turn on Sydney (click on the checkbox "Jailbreak to enable Sydney")
Click on "System Message" (a blue button right below the checkbox)
You're done reproducing.

What browsers are you seeing the problem on?
No response
Relevant log output
No response
Screenshots

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Thanks for letting us know, the documentation has changed, this was an oversight.
You can find the relevent doc here:
https://docs.librechat.ai/features/bing_jailbreak.html
or here:
https://github.com/danny-avila/LibreChat/blob/main/docs/features/bing_jailbreak.md#default-system-message-for-jailbreak-mode-sydney


refactor(MappingType): replace NONE and MAPPED with ORDER_OF_KEYS and NO_MAPPING respectively (GitHub #197)

Codecov Report
Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (551474f) 90.12% compared to head (8edc689) 90.04%.
Report is 1 commits behind head on dev.

Files
Patch %
Lines

...pport/container/MethodInvokerContainerCreator.java
50.00%
0 Missing and 2 partials :warning:

Additional details and impacted files
@@             Coverage Diff              @@
##                dev     #198      +/-   ##
============================================
- Coverage     90.12%   90.04%   -0.09%
+ Complexity     1520     1519       -1
============================================
  Files           148      148
  Lines          3735     3735
  Branches        365      365
============================================
- Hits           3366     3363       -3
  Misses          203      203
- Partials        166      169       +3

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


added getINP to web-vitals exports

#72 this is a pull request to add getINP to web-vitals exports

Please merge this feature!


Fix delete failure after master0 execution

Describe what this PR does / why we need it
fix delete failure after master0 execution
Does this pull request fix one issue?
#263

Describe how you did it
Describe how to verify it
The CleanFS method will do this
Special notes for reviews

seems like this PR does not solve this problem.


Binja can not handle more than 100 sections

Version and Platform (required):
3.2.3835-dev Personal (Build ID e414dfad)
Windows 11
Bug Description:
If an ELF (in this case a coredump) contains more than 100 sections binja appears to just stop parsing them at 100
Steps To Reproduce:
Please provide all steps required to reproduce the behavior:
Create an ELF with more than 100 sections
load it into binja
Expected Behavior:
binja should parse out the number of sections listed in the elf header
Screenshots:

Additional Information:

Fixed in 3.2.3878-dev


Create Drivetrain

Before issuing a pull request, please see the contributing page.

Closed because stale (check #2)


Update postgrex to ~> 0.17

Background
I got the error
> mix deps.get
Resolving Hex dependencies...

Failed to use "postgrex" because
  mix.exs requires ~> 0.17
  ecto_sql (version 3.10.0) requires ~> 0.16.0 or ~> 1.0

** (Mix) Hex dependency resolution failed, change the version requirements of your dependencies or unlock them (by using mix deps.update or mix deps.unlock). If you are unable to resolve the conflicts you can try overriding with {:dependency, "~> 1.0", override: true}

What I did
I just updated the postgrex version to ~> 0.17 to resolve the error.

Thanks, pushed a new version!

Ah, We also need to support ~> 0.16.0.
https://github.com/elixir-ecto/ecto_sql/commit/d8eaabcf34d52fad1d286dab653b2f200ba99333


Error thrown when closing WebviewPanel

As discussed in #9 , an error is thrown when a webviewpanel is closed.
I've been able to confirm you only need to create and register a panel to recreate this:
    this.messenger = new Messenger();

    ...

    const panel = vscode.window.createWebviewPanel('type, 'title', vscode.ViewColumn.Two, {});
    this.messenger.registerWebviewPanel(panel);

Closing the panel throws this error in the console:
mainThreadExtensionService.ts:80 Error: Webview is disposed

Should be fixed now


finding Replaces header Call-ID in dialog map

I am trying to figure the best way to find the Call-ID in the srf._dialogs map when I get an INVITE or REFER with a Replaces header. In an attended transfer scenario, the endpoint sets up a dialog with the transferTarget and then sends a REFER to the transferee with a  Replaces header containing the Call-ID of the dialog with the transferTarget.
In this scenario, drachtio is acting as a B2BUA between all the endpoints (and media is just peer to peer for now), so I actually need to find the dialog of the other side of the call (UAS dialog if received on the UAC side, and vice versa).
I have a gist of a latter diagram and sip trace of a successful attended transfer through drachtio here.
I am wondering if there is a better way to find the dialog associated with the Call-ID in the replaces header than what I am doing below by looping through the map of dialogs:
// referParams has all the parsed elements of a refer header
findReplacesDialog(srf._dialogs, referParams.replacesCallId);

const findReplacesDialog = (dialogMap, replacesCallId) => {
  var replacesDialog;

  const dialogCallIds = Array.from(dialogMap.keys());
  debug('Dialog Keys');
  debug(dialogCallIds);
  dialogCallIds.every((dialog, i) => {
    const currentDialog = dialogMap.get(dialog);
    const previousDialog = dialogMap.get(dialogCallIds[i-1]);
    const nextDialog = dialogMap.get(dialogCallIds[i+1]);

    debug('replacesCallId');
    debug(replacesCallId);
    debug('currentDialog.sip.callId');
    debug(currentDialog.sip.callId);
    if (nextDialog) { debug('nextDialog.sip.callId'); debug(nextDialog.sip.callId); }
    if (previousDialog) { debug('previousDialog.sip.callId'); debug(previousDialog.sip.callId) }

    if (currentDialog.sip.callId == replacesCallId) {
      debug(`FOUND A MATCH!!!! ${dialog}`);
      // drachtio adds ';uas' when it's the UAS of the dialog
      if (dialog == `${replacesCallId};uas`) {
        debug('UAS dialog: setting repalcesDialog equal to previousDialog');
        replacesDialog = previousDialog;
        return false;
      } else {
        debug('UAC dialog: setting repalcesDialog equal to nextDialog');
        replacesDialog = nextDialog;
        return false;
      }
    }

    return true;
  });

  return replacesDialog;
};

Thanks very much for the gist with the ladder diagram, that illustrates the problem clearly.
I'm not sure I totally follow the logic above (I think I'm confused by what seems the reverse use of 'every' instead of simply 'find'), but it seems like the issue is that we need to retrieve an existing dialog (the transferree dialog) from the internal map of dialogs that srf keeps.  And we need to do that by Call-ID, is that correct?
Should we just add a public method to Srf to return a dialog by call-id?
Unfortunately, it is the case that a call-id could be used on two different dialogs, though that is undesirable and somewhat unlikely (the app would have to force the call-id on different legs).
Perhaps we could also add some standard implementation for attended transfer to https://github.com/davehorton/drachtio-fn-b2b-sugar ?

also, let's try not to assume that the format of the dialog id is the calling =+[";uas"|";uac"] quite yet.  I'm looking into changing that to be more correctly some combination of the callid and from tag

I'm testing a new branch in drachtio-server calling dialog-naming to test a change in assigning dialog ids -- they will now be ${callid};from-tag=${fromTag}, which will be more strictly correct.  So hold off on any hardcoding in your app of dialog id construction, and once I merge this branch I will add a public method in Srf to return a dialog given the call-id and the from tag, which I think will support the attended transfer scenario nicely

Yeah, I think that will work since the Replaces headers includes the from-tag and to-tag. But, the hard part is that we actually need the dialog on the other side of the B2BUA call, since that is the dialog we need to update for the transfer.
Here is an example of the current dialog map:
srf._dialogs = [
121314, # call B
891011;uas, # call B
4567, # call A
1234;uas # call A
]

So, currently, if the Replaces header has the Call-ID 4567, I actually need to update the 1234;uas dialog.

OK, just published<EMAIL_ADDRESS>which adds
findDialogById(stackDialogId), and
findDialogByCallIDAndFromTag(callId, tag)

Note: these functions require<EMAIL_ADDRESS>or above (currently the head of the 'develop' branch), which formats dialog ids as {Call-ID};from-tag={from-tag}.
additionally, if you have a reference to a dialog (uas or uac) that was created by Srf#createB2BUA then dialog.other will be a reference to the paired dialog.
So handling attended transfer should be a matter of:

parsing the Refer-To header (I think I need to add a utility function somewhere to make that easier, probably in drachtio-sip), then
calling findDialogByCallIDAndFromTag(callId, tag) to retrieve the dialog being replaced, then
reinviting the paired dialog by calling Dialog#modify on dialog.other.

I'll leave this issue open for now until you have time to test.

I was able to test. This worked great! Thank you so much


southxchange.hasWithdraw False, but API supports withdrawals

Sorry to submit multiple issues at once. When I initialize southxchange from an older version of ccxt (since I'm experiencing the issue described in #331), I'm seeing that southxchange.hasWithdraw == False
But southxchange API seems to support withdraw:
https://www.southxchange.com/Home/Api#withdraw
Can we get the unified method implemented for southxchange withdrawals?

Sure, will add it shortly. As you may know, we don't have them all unified, so there may be hasWithdraw = false, where it is not implemented yet. But we will implement it quickly.

Here's a quick overview of the current state:

Beautiful! That image is very helpful.
I'm going to be going through each and every exchange today determining viability for my project. I'll be noting which ones do not have withdrawal set up in ccxt while their withdrawal capability is, in fact, available through their API.
I think I'd like to try and learn how I can contribute Pull requests to implement some of these if I find myself capable... it would be an honor to contribute to ccxt in that manner. I've got to read up on the process first, but hopefully I can get it working.
Thanks!

I'll be noting which ones do not have withdrawal set up in ccxt while their withdrawal capability is, in fact, available through their API.

That info itself would be really helpful in our work ;)

I think I'd like to try and learn how I can contribute Pull requests to implement some of these if I find myself capable...

And I think it would be awesome to see some PRs from you! )

I've got to read up on the process first, but hopefully I can get it working.

Sure, no problem, feel free to ask, we actually welcome all pull requests, and any help is appreciated! If you feel like going for it, read previous pull requests and issues by other people, there's a lot of useful info you may find interesting for understanding the inner workings of the library better (to have better control over your finance).
Also, this: https://github.com/ccxt-dev/ccxt/blob/master/CONTRIBUTING.md
In the meantime we will try to upload the withdraw() for southxchange... ;)

Thanks, Igor! I'll read up and get back to you. I'd appreciate the addition of southxchange in the meantime, and I'll let you know if I have any difficulties.
One question before I get started... does it matter that I only know Python 2.7? Can I submit a fix in just the one language and still be helpful?

One question before I get started... does it matter that I only know Python 2.7? Can I submit a fix in just the one language and still be helpful?

We actually thought on answering that question beforehand, in the CONTRIBUTING doc ;)
https://github.com/ccxt-dev/ccxt/blob/master/CONTRIBUTING.md#multilanguage-support

The ccxt library is available in three different languages (more to come). We encourage developers to design portable code, so that a single-language user can read code in other languages and understand it easily. This helps the adoption of the library. The main goal is to provide a generalized, unified, consistent and robust interface to as many existing cryptocurrency exchanges as possible.

At first, all language-specific versions were developed in parallel, but separately from each other. But when it became too hard to maintain and keep the code consistent among all supported languages we decided to switch to what we call a master/slave process. There is now a single master version in one language, that is JavaScript. Other language-specific versions are syntactically derived (transpiled, generated) from the master version. But it doesn't mean that you have to be a JS coder to contribute. The portability principle allows Python and PHP devs to effectively participate in developing the master version as well.

The support for withdraw has been added to SouthXchange implementation in ccxt as of version 1.9.168+. Please report if you have any difficulties with it, though, it should work normally. Thx again!

I would close this issue for now, if you don't mind. The implemented should work anyway. Let us know, please, whether it does or does not work for you. Thx again! Feel free to reopen this if needed or just ask questions, if any. Cheers!

Yes, it's fine to close. So sorry for the delay... I've been swimming in other tasks trying to get this automation functional. Right now the Ethereum I sent to test southxchange withdraw with is stuck in limbo. It's on the blockchain, but I can't use it yet. Will report back soon.

Thx!


Same-directory grouping

It'd make, say, pulling different branches easy if all the entries under the same directory were clustered together.

Honestly, sorting is still a big open issue that needs to be figured out much better than it is now.


Removed Knative Buffering

solution for https://github.com/pivotal/skenario/issues/61

@juliababkina looks like the units aren't passing on this change yet.  They need to be fixed before I will merge this PR.

Looks like there is one more unit test failure: https://gist.githubusercontent.com/josephburnett/13f39b9bd1d3ca7c8104e79b3e21f533/raw/09104608232311b434200dbecae1a2f07952f6b1/gistfile1.txt


WIP - devsecops update

Updated devsecops applications:
cosign:

updated the script that cosign uses to generate the public key to sign objects
created service account, and rbac to support cosign job

spring-petclinic:

The CI was failing bc the spring-petclinic application was pointing to the deployed quay registry.
converted the helm chart to kustomize to align with original application and to support pipeline promotion

LGTM.


Preset loading seems broken

Required Info

Camera Model
D435

Firmware Version
lastest

Operating System & Version
Ubuntu  20.04.01

Kernel Version (Linux Only)
5.8

Platform
x86

SDK Version
v2.41

Language

Segment
Robot

Issue Description
Loading of json preset files seems broken for HighAccuracy Preset.
Steps to reproduce:

Open realsense viewer
Activate Stereo module and see that the output is behaving like it is on default
Switch preset in the dropdown menu to High Accuracy and see that the outputs switches indeed to High Accuracy (much less points)
Use Save current stereo modules settings to file to save the configuration file to HighAccuracySaved.json
Close and restart realsense viewer
Use Load pre-configured stereo modules settings button to load the HighAccuracySaved.json configuration file that you saved before
Start the Stereo module and see on the output that the settings are not applied, confirm by selecting Default, then High Accuracy in the dropdown (the output will then properly switch)

This seems to afect json loading from the SDK and ros wrapper too

Continuing, High Accuracy settings (without save/load json) seems to not be applied with the following scenario:

Open realsense viewer
Select High accuray with Stereo Module to off
Start stereo module and see High Accuracy settings not applied in the output

Continuing, High Accuracy settings (without save/load json) seems to not be applied with the following scenario:

Open realsense viewer
Select High accuray with Stereo Module to off
Start stereo module and see High Accuracy settings not applied in the output

To add to the confusion, I believe the difference in outputs are due to the laser emitter behind switched off when selecting High Accuracy in some conditions

To add to the confusion, I believe the difference in outputs are due to the laser emitter behind switched off when selecting High Accuracy in some conditions

Hi @doisyg   I was able to replicate your findings by following your procedure using a D435i.  I found that a workaround was to:

Load in the HighAccuracySaved.json preset whilst the camera is inactive, so that it is added to the drop-down lit of presets.
Activate the stream with one of the existing builtin SDK presets applied, such as Default.
Select the HighAccuracySaved custom preset from the list whilst the stream is active.

The stream then correctly changed to the typical data conditions present when High Accuracy is used.
Whilst this workaround is okay for the Viewer, it would be problematic to implement when loading the preset in ROS if you had to set Default as the json first and then the HighAccuracySaved..json preset file.  I wonder if you could set the Default preset initially in the roslaunch instruction, and then put an arg in the launch file to overwrite that setting with the path to HighAccuracySaved..json.
https://github.com/IntelRealSense/realsense-ros/issues/753
https://github.com/IntelRealSense/realsense-ros/blob/development/realsense2_camera/launch/rs_camera.launch#L5

Hi @doisyg   I was able to replicate your findings by following your procedure using a D435i.  I found that a workaround was to:

Load in the HighAccuracySaved.json preset whilst the camera is inactive, so that it is added to the drop-down lit of presets.
Activate the stream with one of the existing builtin SDK presets applied, such as Default.
Select the HighAccuracySaved custom preset from the list whilst the stream is active.

The stream then correctly changed to the typical data conditions present when High Accuracy is used.
Whilst this workaround is okay for the Viewer, it would be problematic to implement when loading the preset in ROS if you had to set Default as the json first and then the HighAccuracySaved..json preset file.  I wonder if you could set the Default preset initially in the roslaunch instruction, and then put an arg in the launch file to overwrite that setting with the path to HighAccuracySaved..json.
https://github.com/IntelRealSense/realsense-ros/issues/753
https://github.com/IntelRealSense/realsense-ros/blob/development/realsense2_camera/launch/rs_camera.launch#L5

The stream then correctly changed to the typical data conditions present when High Accuracy is used.

I am not sure anymore if the difference in the output in these conditions is due to improper application  of Advanced Settings from or just a bug turning off the laser emitter (try to toggle the emitter off/on to see if it impacts the output)

The stream then correctly changed to the typical data conditions present when High Accuracy is used.

I am not sure anymore if the difference in the output in these conditions is due to improper application  of Advanced Settings from or just a bug turning off the laser emitter (try to toggle the emitter off/on to see if it impacts the output)

I re-ran the testing and did not experience any change in the displayed Laser status of the emitter in the options side-panel.
It is normal though for the depth image to potentially degrade when the emitter staus is changed from Laser (On) to Off, or if the value of the Laser Power setting is reduced.  So if the depth quality degraded if the emitter was manually toggled off, that would be an expected behavior.
If you are referring to how the depth image loses some depth detail when High Accuracy settings are used, this is also normal because pixels that have a high confidence of measurement accuracy are prioritized.  If you wanted a detailed image that also had accuracy, the Medium Density preset provides a balance between these two priorities.

I re-ran the testing and did not experience any change in the displayed Laser status of the emitter in the options side-panel.
It is normal though for the depth image to potentially degrade when the emitter staus is changed from Laser (On) to Off, or if the value of the Laser Power setting is reduced.  So if the depth quality degraded if the emitter was manually toggled off, that would be an expected behavior.
If you are referring to how the depth image loses some depth detail when High Accuracy settings are used, this is also normal because pixels that have a high confidence of measurement accuracy are prioritized.  If you wanted a detailed image that also had accuracy, the Medium Density preset provides a balance between these two priorities.

Hi @doisyg   Do you require further assistance with this case, please?  Thanks!

Hi @doisyg   Do you require further assistance with this case, please?  Thanks!

I believe the bug which is switching off the laser emitter without updating the status when changing the preset in realsense-viewer should be fixed to avoid confusion

At this point, it may be worth waiting for the next SDK version and coming back to the issue if it is still present.
I note that custom presets are addressed in a PR commited in the past 24 hours at the time of writing this.
https://github.com/IntelRealSense/librealsense/commit/7e72d615bf4c02156c182ad1f44665453830d83d

Hi @doisyg   The new librealsene version 2.42.0 is now available.  Would it be possible please to check whether your problem is corrected in this version, please?   Thanks!

Issue indeed solved with the last lib version. Thx, closing.

Excellent news - thanks very much for the update!


refactor(Bicategory/Functor): refactor prelaxfunctors

This PR suggests a refactor (and renaming) of prelax functors. This is to provide more reusability of code between oplax functors, pseudofunctors (and soon) lax functors.
This is done in the following way:

A new file is created called Bicategory.Functor.Prelax.
What (before this suggested refactor) was called PrelaxFunctor, is renamed to LaxPrefunctor. Recall that this is a prefunctor that is also equipped with a map map₂ on 2-morphisms.
A new structure is added, taking over the name PrelaxFunctor, which additionally requires that the associated prefunctors between hom types are functors. In other words, map₂ should respect identities and compositions.
Both Pseudofunctor and OplaxFunctor are updated to extend the new notion of PrelaxFunctor.

Finally:
Coercions have also been removed between these various structures. This is to prevent the following behaviour in the current library: given a pseudofunctor F the infoview will print F.map as (↑F.toPrelaxfunctor).map which makes long expressions unecessarily hard to read. The change is just like how Functor does not have a coercion to PreFunctor.
This refactor will allow us to define mapFunctor and map₂Iso on the level of PrelaxFunctors, which allows it to be reused for both Pseudofunctors and OplaxFunctors instead of having to be defined separately.
A new constructors on this level has also been added, PrelaxFunctor.mkOfHomFunctors, that constructs a PrelaxFunctor given a map on types and functors between the corresponding hom types. This could for example be used to construct the pseudofunctor in the bicategorical yoneda lemma, by using the collection of functors precomposing (see Bicategory.Basic).
The file CategoryTheory/WithTerminal has been edited just to fix the proof after these new changes.

[x] depends on: #13900

I am not fussed about the specific names I've chosen, and I am very happy to hear other suggestions. Also, I was not there for the mathport, so I am unsure exactly what to do with many of the porting notes and #align now that PrelaxFunctor has changed. I have removed some of them that didn't feel relevant were anymore, but I'm not sure if I was actually supposed to do that.

Thanks for doing this! This is definitely something I left unfinished.
I feel LaxPreFunctor/PrelaxFunctor might have better names as it's hard to see which is the parent. How about PrelaxFunctorStruct/PrelaxFunctor or PrePrelaxFunctor/PrelaxFunctor?

Thanks for writing this library in the first place! It has been very helpful for the development of fibered categories :)

Thanks!
bors merge


Using JS string interpolation inside @raw in template content throws "Unexpected end of template expression"

Hey,
I was trying to @raw to include a script in my page, which was part of a template (a basic root template with  tags, etc). Inside the script block I used JS string interpolation e.g. console.log(`hello ${name}`). Unfortunately this threw an exception, detailed below.
Minimal reproducable example:
testroot.jte:
@import gg.jte.Content

@param Content content

    <title>Test</title>

    ${content}

testraw.jte:
@template.testroot(
content = @`

Hello World!

    @raw

    @endraw
`
)

(In the real example, I had included the script as a separate parameter passed to the template, which also failed)
Would you happen to know what's going on here?
Thanks!
Exception encountered:
gg.jte.TemplateException: Failed to compile testraw.jte, error at line 7: Unexpected end of template expression
	at gg.jte.compiler.java.JavaCodeGenerator.onError(JavaCodeGenerator.java:221)
	at gg.jte.compiler.TemplateParser.extractTextPart(TemplateParser.java:446)
	at gg.jte.compiler.TemplateParser.doParse(TemplateParser.java:359)
	at gg.jte.compiler.TemplateParser.parse(TemplateParser.java:94)
	at gg.jte.compiler.java.JavaCodeGenerator$ContentProcessor.writeJavaCode(JavaCodeGenerator.java:573)
	at gg.jte.compiler.java.JavaCodeGenerator$ContentProcessor.process(JavaCodeGenerator.java:547)
	at gg.jte.compiler.java.JavaCodeGenerator.writeJavaCodeWithContentSupport(JavaCodeGenerator.java:429)
	at gg.jte.compiler.java.JavaCodeGenerator.appendParam(JavaCodeGenerator.java:482)
	at gg.jte.compiler.java.JavaCodeGenerator.appendParams(JavaCodeGenerator.java:470)
	at gg.jte.compiler.java.JavaCodeGenerator.onTemplateCall(JavaCodeGenerator.java:388)
	at gg.jte.compiler.TemplateParser.lambda$doParse$3(TemplateParser.java:257)
	at gg.jte.compiler.TemplateParser.extract(TemplateParser.java:947)
	at gg.jte.compiler.TemplateParser.doParse(TemplateParser.java:257)
	at gg.jte.compiler.TemplateParser.parse(TemplateParser.java:94)
	at gg.jte.compiler.TemplateParser.parse(TemplateParser.java:89)
	at gg.jte.compiler.TemplateCompiler.generateTemplateCall(TemplateCompiler.java:299)
	at gg.jte.compiler.TemplateCompiler.generate(TemplateCompiler.java:218)
	at gg.jte.compiler.TemplateCompiler.precompile(TemplateCompiler.java:142)
	at gg.jte.compiler.TemplateCompiler.load(TemplateCompiler.java:45)
	...

I just pushed a fix for the @raw example you provided. In case you would like to give it a try, you could clone this repository and run a maven clean install on the main branch. Then temporarily set 2.1.2-SNAPSHOT as jte version in your project.
I'm afraid, the other example you were talking about would still fail. Without @raw, the first backtick of the js interpolated string will close the jte content block. Also, ${} will be interpreted as jte keywords and the Java compiler will complain about the missing variable.
Would it be possible for you to have the JS in a separate script file and reference that from jte? This should also avoid all sorts of security problems when mixing jte with JS. This is the policy we're following at work in jte templates.

Hey @casid, thanks for the prompt fix! I have tested it out locally and the fix works a charm.
I might have explained poorly - the other example I was still using @raw, just passing it as another param in the template instead of content. E.g.
@param gg.jte.Content title
@param gg.jte.Content content
@param gg.jte.Content script = @``

<!doctype html>

    <title>${title}</title>
    <meta charset="utf-8">

    ${content}

${script}

Using @raw here with your fix works a charm. Having a separate file would definitely work - but the script is still quite small and having it next to the HTML is kinda nice (and I'm lazy :p)
Thanks for all your work on JTE - it has been an absolute joy to use!

Thanks for confirming the fix and for the heads up! Glad you enjoy working with jte :-)

I just released version 2.1.2 to maven central, including this fix :-)


SerializeAsV2 not respected

Hi,
I have recently update this package (from NuGet version v5.0.0-RC5) to v5.0.0, and the command  SerializeAsV2 was not respected.
For example, in the swagger.json the keyword "allOf" was added, but this is not compliant with swagger 2.0 specs.

Picture: both with SerializeAsV2 set as true
Left -> generated with 5.0.0 -> OpenApi 3 keyword (wrong)
Right -> generated whit 5.0.0-RC5 ->Swagger 2.0 specs (correct)
How can I generate a swagger.json compliant with swagger 2.0 specs, using the v5.0.0 NuGet of this library?
Thanks

There's an update on the way that will make allOf behavior opt-in. This should solve your problem so stay tuned ...

As of 5.1.0, the allOf behavior has been made opt-in. This should resolve your issue.


Add Spectrum badge

Links to the newly created channel for this project on Spectrum, spectrum.chat/styled-components/stylelint-processor

Coverage remained the same at 100.0% when pulling ebd00143412cf92f25fc1003a6660d4d93d746f0 on spectrum-badge into 36b4b437b1f6339f93792bdb60f0acfc5db4357a on master.


Feature request: Autocomplete Update

As seen here: https://www.google.com/design/spec/components/text-fields.html#text-fields-auto-complete-text-field a autocomplete could have icons.
I came up with the following:

Still todo:

fix bugs in highlighting (

)

add support for images with highlighted text

Autocomplete is working fine. Highlighted text option could be removed

How can I use this with the CDN? what should I add to the css and js?

@lastnoob this hasn't been merged (yet) so you can't use it (yet)

Any ETA on merging?

@kristoftorfs at least I don't know anything about an ETA and because of the lack of merges and bug fixes lately I started my own

@Dogfalo @acburst Could this please be merged because people start forking it from me which could ultimately result in people not getting other updates of materialize

@SuperDJ ETA on the fixes? I really really hope this is going to be merged soon. We have a project that actually needs autocomplete and we just realised, 5 days before beta, that this is not in yet. I will be tagging myself to know the progress on this... we will have to use select2 meanwhile :(

@SiliticX I can tell you that I'm not going to create a PR again for this any time soon. I also wished they had merged this already but sadly they haven't. In the mean time you can add the following code.
In forms.js. Note that I deleted the functions for highlighting text because it didn't work
           /**************************
	 * Auto complete plugin  *
	 *************************/
	$(input_selector).each(function() {
		var $input = $(this);

			if( $input.hasClass('autocomplete') ) {
				var $array = $input.data('array'),
						$inputDiv = $input.closest('.input-field'); // Div to append on

				// Check if "data-array" isn't empty
				if( $array !== '' ) {
					// Create html element
					var $html = '
';

					for( var i = 0; i < $array.length; i++ ) {
						// If path and class aren't empty add image to auto complete else create normal element
						if( $array[i]['path'] !== '' && $array[i]['class'] !== '' ) {
							$html += '
'+$array[i]['value']+'
';
						} else {
							$html += '
'+$array[i]['value']+'
';
						}
					}

					$html += '
';
					$inputDiv.append($html); // Set ul in body
					// End create html element

					// Perform search
					$(document).on('keyup', $input, function () {
						var $val = $input.val().trim(),
								$select = $('.autocomplete-content');
						// Check if the input isn't empty
						if ($val != '') {
							$select.children('li').addClass('hide');
							$select.children('li').filter(function() {
								$select.removeClass('hide'); // Show results

								return $(this).text().indexOf($val) !== -1;
							}).removeClass('hide');
						} else {
							$select.children().addClass('hide');
						}
					});

					// Set input value
					$('.autocomplete-option').click(function() {
						$input.val($(this).text().trim());
					});
				} else {
					return false;
				}
			}
		});

For styling you can simply add the scss below in your own stylesheet:
// Auto complete
.autocomplete-content {
margin-top: -1rem;
@extend .z-depth-1;

	li {
		clear: both;
		color: rgba(0, 0, 0, 0.87);
		cursor: pointer;
		line-height: 1.5rem;
		width: 100%;
		text-align: left;
		text-transform: none;

		> span {
			color: $input-focus-color;
			font-size: 1.2rem;
			padding: 1rem;
			display: block;

			.highlight {
				color: #000000;
			}
		}

		img {
			height: 52px;
			width: 52px;
			padding: 5px;
			margin: 0 15px;
		}

		&:hover {
			background: #eee;
			cursor: pointer;
		}
	}
 }

@lastnoob @kristoftorfs @SiliticX Maybe you can make an issue of this if you need it

@SuperDJ Thanks!

Hi, any update on this?

Hey guys, this is a good feature, and now days very useless, any updates when it can be available??

Thanks again for this work SuperDJ. I simplified and cleaned up the code a bit

when will it be released?

+1

+1
Anyone can update documentation ? A little How-to in form would be great


(chocolatey-core.extension) Get-AppInstallLocation returns wrong results for paint.net

Expected Behavior
Given the following UninstallString value in the registry:
MsiExec.exe /X{guid}

Get-AppInstallLocation should not return "MsiExec.exe ", but ignore this value (as it does not contain path information) and proceed to check subsequent locations (Program Files, App Paths etc.).
Current Behavior
Get-AppInstallLocation returns "MsiExec.exe ", which is wrong on many levels:

has no path information
points to an executable, not to a directory
contains a trailing space

Possible Solution
Enhance registry value parsing code so that it

handles values containing command line arguments (Windows Installer packages use it, "X:\Path\y.exe /Uninstall" is also common) trims the extracted path,
tests the extracted path to verify it really exists, is absolute and is a directory.

Steps to Reproduce (for bugs)
cinst -y paint.net

Context
The paint.net package passes the bogus value obtained from Get-AppInstallLocation to Register-Application, which throws an error and the entire package installation fails.
Also reported on Disqus: https://chocolatey.org/packages/paint.net#comment-3558201902
Your Environment

Package Version used:
chocolatey-core.extension 1.3.1
paint.net 4.0.19
Operating System and version: Windows 10 1607 Pro x64 PL
Chocolatey version: 0.10.5
Install/uninstall gist: https://gist.github.com/jberezanski/f36431d463e9b23d25c9096ee86d044a

Hm.... works on my machine, and the value IS tested:
https://github.com/chocolatey/chocolatey-coreteampackages/blob/master/extensions/chocolatey-core.extension/extensions/Get-AppInstallLocation.ps1#L45
> import-module C:\ProgramData\chocolatey\helpers\chocolateyInstaller.psm1

> Get-AppInstallLocation 'Paint.NET*' -Verbose
VERBOSE: Trying local and machine (x32 & x64) Uninstall keys
VERBOSE: Retrieving all uninstall registry keys
VERBOSE: Trying Uninstall key property 'InstallLocation'
VERBOSE: Trying Uninstall key property 'UninstallString'
VERBOSE: Trying Uninstall key property 'DisplayIcon'
VERBOSE: Trying Program Files with 2 levels depth: C:\Program Files C:\Program Files\*\* C:\Program Files (x86) C:\Program Files (x86)\*\*
C:\Program Files\paint.net

Try cd'ing to $Env:SystemRoot\System32:
PS C:\> Import-Module C:\ProgramData\chocolatey\helpers\chocolateyInstaller.psm1
PS C:\> Get-AppInstallLocation 'Paint.NET*' -Verbose -OutVariable x | % { "[$_]" }
VERBOSE: Trying local and machine (x32 & x64) Uninstall keys
VERBOSE: Retrieving all uninstall registry keys
VERBOSE: Trying Uninstall key property 'InstallLocation'
VERBOSE: Trying Uninstall key property 'UninstallString'
VERBOSE: Trying Uninstall key property 'DisplayIcon'
VERBOSE: Trying Program Files with 2 levels depth: C:\Program Files C:\Program Files\*\* C:\Program Files (x86)
C:\Program Files (x86)\*\*
[C:\Program Files\paint.net]
PS C:\> Test-Path $x
True
PS C:\> cd $Env:SystemRoot\System32
PS C:\WINDOWS\System32> Get-AppInstallLocation 'Paint.NET*' -Verbose -OutVariable x | % { "[$_]" }
VERBOSE: Trying local and machine (x32 & x64) Uninstall keys
VERBOSE: Retrieving all uninstall registry keys
VERBOSE: Trying Uninstall key property 'InstallLocation'
VERBOSE: Trying Uninstall key property 'UninstallString'
[MsiExec.exe ]
PS C:\WINDOWS\System32> Test-Path $x
True

Relative paths can be detected and rejected by calling e.g. [IO.Path]::IsPathRooted($path).
Apart from that, a simple Test-Path is not sufficient, because it does not check whether the object is a directory. It would require something like:
function Test-Directory($path)
{
    $i = Get-Item -Path $path -ErrorAction SilentlyContinue
    return $i -ne $null -and $i.PSIsContainer
}

There is also the matter of UninstallStrings with arguments containing path separators, which the current simplistic parsing method will not handle correctly (the trailing space is a symptom of it):
PS C:\> $location = 'MsiExec.exe /X{F10AAD91-58DF-44EC-A647-810197141667}'
PS C:\> $location.Replace('"', '') | Split-Path | % { "[$_]" }
[MsiExec.exe ]
PS C:\> $location = '"C:\Program Files (x86)\Steam\steam.exe" steam://uninstall/7760'
PS C:\> $location.Replace('"', '') | Split-Path | % { "[$_]" }
[C:\Program Files (x86)\Steam\steam.exe steam:\\uninstall]
PS C:\> $location = 'C:\Windows\IsUninst.exe -fE:\Gry\XComCE\Uninst.isu'
PS C:\> $location.Replace('"', '') | Split-Path | % { "[$_]" }
[C:\Windows\IsUninst.exe -fE:\Gry\XComCE]

Here is one better parser that can be put instead of the current one:
iex ". { `$args[0] } $location"`

https://github.com/chocolatey/chocolatey-coreteampackages/commit/555925cb5f0d3ea3ded52b9b22fb8779feae105f is enough to fix this problem with Paint.Net.

There is also the matter of UninstallStrings with arguments containing path separators, which the current simplistic parsing method will not handle correctly (the trailing space is a symptom of it).

This was made only to handle quoted single path. I made this parsing function instead, that should return unquoted 'first argument'. It utilizes powershell parsing of arguments with some special posh chars replaced prior to iex and returned later:
function parse($s){
    $escape = ";'><"
    $escape -split '' | ? { $_ } | % {$i=0} { $i++; $s = $s -replace $_, ([char]$i) }
    $s = iex ". { `$args[0] } $s"
    $s -split '' |  ? { $_ } | % {$r=''} { $a=[byte][char]$_; $r += if ( $a -le $escape.Length) { $escape[$a-1] } else {$_} }
    $r
}

If nobody has objections I can make it into module.

@majkinetor if it's compatible with POSHv2, then there is no objection from me

I am hesitant to introduce any change to the extension since it can affect big number of packages.. The new parsing method even if better can change behavior of existing packages so all would need to be tested for it.
Since this is not actually an improvement to any existing package and I don't have time to be purist, I will just leave it be for now. If anybody else wants to give it a go feel free to do so.


Multiple tutorial messages

Diagnose and fix these multiple tutorial messages.
Perhaps the issue must be closed as completed before the contributor's comments are taken into account? I'm not sure.
Originally posted by @ubiquibot in https://github.com/ubiquity/telegram-ubiquibot/pull/17#issuecomment-1705716566

This is because verifyFirstCheck function is not correct.
In that function, it checks if count of issues that a specific user commented is one. But I think we also need to count the prs that the user commented.
I want to start this issue but I exceeded maximum limit...
Could anyone assign me?

This is a test


Order of MAC and MTU fields inconsistent between Add Interfaces and Edit Interface pages

Issue type
[ ] Feature request
[X] Bug report
[ ] Documentation

Environment

Python version:  2.7.9
NetBox version:  2.2.9

Description
Order of MAC and MTU fields inconsistent between Add Interfaces and Edit Interface pages. Affects both Devices and Virtual Machines.
On Add Interface Pages (eg https://hostname/dcim/devices/PK/interfaces/add/ and https://hostname/virtualization/virtual-machines/PK/interfaces/add/) the order of the fields is
Name
MTU
MAC Address
Description
On Edit Interface Pages (eg https://hostname/dcim/interfaces/PK/edit/ and  https://hostname/virtualization/vm-interfaces/PK/edit/) the order of the fields is
Name
MAC Address
MTU
Description
It's a small subtle problem but I notice it when I'm creating say two or more interfaces on a device/vm using numeric ranges for bulk creating (including MAC address). I then go to edit the the 2nd, 3rd interface etc to correct the MAC address. I often end up entering the MAC in to the MTU field.

Hi,
The change you've made has corrected the issue with Devices, but the problem appears to persist with Virtual Machines. I believe a similar change is required in /netbox/virtualization/forms.py.
Thanks
Steve


Set ubuntu default directory to /etc/default

Currently the path for Jenkins slave config on Ubuntu is /etc/sysconfig which is the default which doesn't exist in the base installation, to work around this I created a file resource which the class depended on;
file { '/etc/sysconfig':
  ensure => 'directory'
}

class { '::jenkins::slave':
  require => File['/etc/sysconfig']
}

It would be nice if it used the same as the Debian installation path.

Thank you for a pull request! Please check this document for how the Jenkins project handles pull requests

I'm not sure this change is necessary
tyler@artichoke:~$ facter osfamily
Debian
tyler@artichoke:~$ cat /etc/issue
Ubuntu 12.04.4 LTS \n \l

tyler@artichoke:~$

Under both ruby facter and cfacter, Ubuntu is part of osfamily Debian.  Eg., https://github.com/puppetlabs/facter/blob/ececbe6b3cfbad162400bbd58f81ba84b0e4bafb/lib/src/facts/resolvers/operating_system_resolver.cc#L271

Yes, this certainly seems the case I think I was getting it confused with operatingsystem fact, but in any case the reason I did this change was it wasn't installing the daemon package with puppet 3.7.x with future parser and Ubuntu 12.04, I will revisit this and see if I can narrow it down.

I believe the comments are right so I will be closing this, I think it was related to issues with my facts.


pc - make csv uploading user experience better

Added instructions for format of CSV file
Moved sample CSV file into sample-data subdirectory of main repo

Rebasing from main to potentially fix Codecov


[Back end] Follower count on profile doesnt update without page refresh

Not perfect, but I can push up a change that just does a window.reload() at each click


Keep existing id_token if not in token result

Returning an ID token from requests to the token endpoint using the
refresh_token grant is optional. Thus also applying the ID token to
the user data structure must be optional if no new id_token value is
returned. This change implements just that in a very similiar way as how
the existing refresh_token value is kept.
Fixes: #1236

@brockallen any ETA when you would consider to merge this?

Thanks.


Angular1: Setting the content with ng-repeat, events are undefined #2354

issue: https://github.com/froala-labs/froala-editor-js-2/issues/2354
Description: Angular1-Setting the content with ng-repeat, events are undefined #2354
Fix: provided undefined check for the events

Not merging this, Though the console error is resolved but the events are not working in case of ng-repeat,

Hi @kapil2704  the events are working in case of ng-repeat -
please find the screenshot :
two events are defined blur and keyup


bug: Line number is not displayed correctly for minphony

Expected behaviour
Suggest correct line 155 for conflicting minphony
Actual behaviour
Conflicting line is 155 where line number 0 is suggested:
Output of checkmake --version
gitpod /workspace/Kreyrock $ checkmake --version
checkmake  built at  by  with

Using commit https://github.com/mrtazz/checkmake/commit/03dd76b964dd2cf535a7e1f0efe5731c183a3071
Output of checkmake --debug <your makefile>
https://gist.githubusercontent.com/Kreyren/03329ef4dd8c6b35152ca49cb93a27f7/raw/66f97f8c0acc21798a81b98f68f0e3a8e37ca4b2/gistfile1.txt
Output of make --version
gitpod /workspace/Kreyrock $ make --version
GNU Make 4.2.1
Built for x86_64-pc-linux-gnu
Copyright (C) 1988-2016 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>
This is free software: you are free to change and redist

Sample Makefile to reproduce issue
https://gist.githubusercontent.com/Kreyren/788b99fc6a926fe1f7503b034c2b40f8/raw/3f8ebccd9557b348600a0987e8dc86292167157d/gistfile1.txt

If I run it on the Makefile from the docker branch it also seems to report the incorrect line number, it gives the line number from the next command instead of the one that had a bug.
      RULE                 DESCRIPTION             LINE NUMBER

  phonydeclared   Target "binaries" should be      58
                  declared PHONY.
  maxbodylength   Target body for "coverage"       72
                  exceeds allowed length of 5
                  (8).


Destroy dup overlays

Looking at the PR, I also found overlays.rake which seems to do the same thing as what you've written.
Anyway, my comment still stands, with the amount of data we have now, it will be better to use a smarter query.

🚀


Large response body fails

When returning a response over 65k bytes, Ace's HTTP2 service fails, with a timeout. We've verified locally with Mojito and Gun clients. I've added a reproducer here:
https://github.com/varnerac/Ace/tree/large_response_reproducer/

Should clarify that the reproducing test is https://github.com/varnerac/Ace/blob/large_response_reproducer/test/ace/http2_test.exs#L277-L292
It relies on a change in the simple app here: https://github.com/varnerac/Ace/blob/large_response_reproducer/test/support.exs#L110

Could you open a PR to add this test, even without the fix.
However instead of modifying simple app just make the greeting be an enormous binary https://github.com/varnerac/Ace/blob/large_response_reproducer/test/ace/http2_test.exs#L280

Hi @varnerac could you eventually solve the large body issue?
I seem to be having same/similar issues downloading large files via ssl, same code worked fine on clear-text

I did not


When entering the seed phrase the word should be red until it's entered correctly

'Restore wallet' and 'confirm seed phrase' secreens: when entering the seed phrase the word should be red until it's entered correctly. As it's done on the android and UI wallets

Checked