Apply for a library membership

Description
Before we can support requesting physical items and access to restricted digitised content, library users need to be able to sign in on wellcomecollection.org. We will support sign in using existing library accounts, as well offer online pre-registration that can be upgraded to a full library account in-person. We will also provide an administration tool for library staff to manage accounts.
Objective
Researchers engage with a broader range of our collections
Measures

Existing library members can sign in to their account

New library members can register for an account

Up-to-date information available about membership

To do

Sign in to an existing account

Manage an existing account

Sign up for a new account

Account admin tool for staff

Link to feature: https://wellcomecollection.productboard.com/feature-board/planning/features/4870169

Closing as done – we've implemented sign-in for existing members and sign-up for new members, and administration occurs through Sierra, as per the previous workflows.


added new notation

added new eval notation proposed for latex-math

 Add new eval notation to latex-math & check eval/tuning cheatsheet


cli skeleton

Let's implement https://github.com/yegor256/cam/issues/227 in this repository, by packing project as Python command-line application, that can be installed using pip from PyPI.

@h1alexbel the puzzle #9 is still not solved.

@h1alexbel 3 puzzles #10, #11, #9 are still not solved.

@h1alexbel 4 puzzles #10, #11, #14, #9 are still not solved.

@rultor release, tag is 0.0.1

@rultor release, tag is 0.0.1

@h1alexbel OK, I will release it now. Please check the progress here

@rultor release, tag is 0.0.1

@h1alexbel Oops, I failed. You can see the full log here (spent 7min)

@rultor release, tag is 0.0.1

@rultor release, tag is 0.0.1

@h1alexbel OK, I will release it now. Please check the progress here

@rultor release, tag is 0.0.1

@h1alexbel Oops, I failed. You can see the full log here (spent 7min)

@rultor release, tag is 0.0.1

@rultor release, tag is 0.0.1

@h1alexbel OK, I will release it now. Please check the progress here

@rultor release, tag is 0.0.1

@h1alexbel Oops, I failed. You can see the full log here (spent 6min)

@rultor release, tag is 0.0.01

@rultor release, tag is 0.0.1

@h1alexbel OK, I will release it now. Please check the progress here

@rultor release, tag is 0.0.1

@h1alexbel Oops, I failed. You can see the full log here (spent 7min)

@rultor release, tag is 0.0.1

@rultor release, tag is 0.0.1

@h1alexbel OK, I will release it now. Please check the progress here

@rultor release, tag is 0.0.1

@h1alexbel Done! FYI, the full log is here (took me 7min)

@h1alexbel 3 puzzles #10, #11, #9 are still not solved; solved: #14.

@h1alexbel 3 puzzles #18, #19, #9 are still not solved; solved: #10, #11, #14.

@h1alexbel 5 puzzles #18, #22, #23, #24, #9 are still not solved; solved: #10, #11, #14, #19.

@h1alexbel 4 puzzles #18, #22, #24, #9 are still not solved; solved: #10, #11, #14, #19, #23.

@h1alexbel 3 puzzles #18, #22, #9 are still not solved; solved: #10, #11, #14, #19, #23, #24.

@h1alexbel 4 puzzles #18, #22, #43, #9 are still not solved; solved: #10, #11, #14, #19, #23, #24.

@h1alexbel 3 puzzles #18, #22, #43 are still not solved; solved: #10, #11, #14, #19, #23, #24, #9.

@h1alexbel 2 puzzles #62, #63 are still not solved; solved: #10, #11, #14, #18, #19, #22, #23, #24, #43, #9.

@h1alexbel the puzzle #63 is still not solved; solved: #10, #11, #14, #18, #19, #22, #23, #24, #43, #62, #9.


Sass Lint doesn't apply subrules beyond the first subrule.

Sass Lint: 1.4.0 to 1.12.1
For rules like class-name-format and clean-import-paths that have more than one subrule excluding severity, Sass Lint won't apply subrules beyond the first subrule.
Example 1
SassLint-Config.yaml
rules:
  class-name-format:
    - 1
    - allow-leading-underscore: false
    - convention: ^[a-zA-Z0-9]+(_[a-zA-Z0-9]+)*$
    - convention-explanation: "Omae wa mou shindeiru"

Sass Lint will apply

- 1
- allow-leading-underscore: false

Sass Lint won't apply

- convention: ^[a-zA-Z0-9]+(_[a-zA-Z0-9]+)*$
- convention-explanation: "Omae wa mou shindeiru"

Example 2
I've created a project that demonstrates the issue as well as a bug fix.
Project: https://github.com/M1ras/SassLint-Issues/tree/rules
Bug Fix: https://github.com/M1ras/sass-lint/tree/rules
Project Structure

config/sass-lint.yaml contains class-name-format and clean-import-paths rules.
src/class-name-format.scss contains code that should violate class-name-format rules but doesn't.
src/clean-import-paths.scss contains code that should violate clean-import-paths rules but doesn't.
scripts/build.sh tests violations for Sass Lint 1.4.0 to 1.12.1 as well as a bug fix. Logs are stored in build/ as build/sass-lint@SASS_LINT_VERSION with the bug fix as build/sass-lint@OmaewWaMouShindeiru.

Bug Fix
The bug fix follows contribution guidelines, passes all linting and tests, and is ready for a pull request.

Please see the last comment of #1175


[Bug]: NPE returned when trying to access MethodSymbol from context.typeOf(node) in semantic model

Description
Get the following error when building https://github.com/tharindu-nw/ballerina-registry/tree/graphql/projects/package_api_graphql
Compiling source
    registry/package.api.graphql:0.1.0
ballerina/sql:1.6.2 [central.ballerina.io -> home repo]  100% [============================================] 364/364 KB (0:00:01 / 0:00:00)
    ballerina/sql:1.6.2 pulled from central successfully
error: compilation failed: The compiler extension in package 'ballerina:sql:1.6.2' failed to complete. Cannot read field "pos" because "funcNode.symbol" is null

Ballerina version: 2201.3.0
Got the same error after downgrading the sql version to 1.6.1
Steps to Reproduce

do a bal build with https://github.com/tharindu-nw/ballerina-registry/tree/graphql/projects/package_api_graphql

Affected Version(s)
No response
OS, DB, other environment details and versions
No response
Related area
-> Compilation
Related issue(s) (optional)
No response
Suggested label(s) (optional)
No response
Suggested assignee(s) (optional)
No response

This is due to compiler throwing NPE when we try to access a method symbol from BallerinaSemanticModel.
Code Line: https://github.com/ballerina-platform/module-ballerina-sql/blob/v1.6.2/compiler-plugin/src/main/java/io/ballerina/stdlib/sql/compiler/analyzer/MethodAnalyzer.java#L68

After checking the code, found a syntax error (unnecessary new keyword) here: https://github.com/tharindu-nw/ballerina-registry/blob/5221c8241ffbd473230d00df767f9ecf3012159f/projects/package_api_graphql/package_api_graphql_service.bal#L56
Tried building after removing that and got the following error:
ERROR [ballerina/log/2.5.1::natives.bal:(267:1,270:12)] {ballerina/jballerina.java}METHOD_SIGNATURE_DOES_NOT_MATCH 'Incompatible param type for method 'replace' in class 'java.lang.String': Java type 'char' will not be matched to ballerina type 'handle''

Had to debug the original code shared here against ballerina-lang 2201.3.0 and 2201.3.x, It turns out to be the original problem comes from:
https://github.com/ballerina-platform/ballerina-lang/blob/0dbeaf81dbb2a8cec554b43ffb5044ae229aa169/compiler/ballerina-lang/src/main/java/io/ballerina/compiler/api/impl/NodeFinder.java#L288
Here, funcNode.symbol is getting null

Issue seems to go away after removing the extra new keyword here: https://github.com/tharindu-nw/ballerina-registry/blob/5221c8241ffbd473230d00df767f9ecf3012159f/projects/package_api_graphql/package_api_graphql_service.bal#L56

Due to this https://github.com/ballerina-platform/ballerina-lang/issues/39612#issuecomment-1434099052

Due to this https://github.com/ballerina-platform/ballerina-lang/issues/39612#issuecomment-1434099052


Speed up rclpy loggers

This patch introduces an alternative implementation of the rclpy.logging.RcutilsLogger that will cache log call configurations. Not ready yet but soon enough.

Thanks for proposing a quick alternative solution! Would you mind elaborating on what the issues in the original code were and how the new change addresses them?

Thanks for proposing a quick alternative solution! Would you mind elaborating on what the issues in the original code were and how the new change addresses them?

I updated the PR description. I think it's a combination of factors that can be summarized as "too much work on every log call". This changes addresses this by "doing less work once".

I am looking forward to seeing this change live from the bdai container soon! I had to suppress a lot of log messages currently to get around the performance issue.

I wonder if we would like to contribute this change back to ROS2 as well?

I'm onboard.

I am looking forward to seeing this change live from the bdai container soon! I had to suppress a lot of log messages currently to get around the performance issue.

Note you will have to wrap your logger, like logger = as_memoizing_logger(node.get_logger()). We could inject it for our bdai_ros2_wrappers.node.Node subclass, but I suspect that may create some confusion when people start mixing up our nodes with regular nodes (and ask why the logger in their regular node has awful performance).


Matrix operation

HI! thanks for your work! 'I have some questions about the Matrix opearation.
When the input size in example.py is set to (1,512,1024), it seems impossible to compute "x2 = x2@z" in model.py (line 91).

Same here, I have encountered the same issue here using the given example.


Show matching paren highlights in the floating window

Hi folks! Currently the MatchParen highlight is not applied in the floating window. I'm happy to open a PR if anyone has any guidance as to how this might be possible.

Unlikely to happen, we'd need a hook into matchparen. Besides, with the latest PR:s, I think we don't even show the parenthesises anymore.

Wouldn't be easy to implement, but should be possible with searchpair / searchpairpos

Given the level of complexity, I think we will pass on this.


PVnet backtest analysis

Detailed Description
We have run a backtest for PVNet and we shoudl do some analysis on this
Context

good to be able to send to people, to show how good our forecast is

Possible Implementation
Merge national all together from each init forecast
Metrics:

MAE
RMSE
R2
p10,p25,p75,p90 absolute errors using pinball score
MBE?
Ramp rate

We should compare
A: PVnet to PVLive update for the whole of 2022
B:

PVnet to PVLive update for the whole of 2022-07-01 to 2023-01-01
PVLive initial to PVLive update for the whole of 2022-07-01 to 2023-01-01

Extra

look at different forecast horzons
std distirbution of MAE - maybe T test
bonus: National_xg -

Run Pinball loss on each quantile forecast for PVnet

I'd definitely like to see step 8, as it's interesting, and they are looking for seasonal / time-of-day analysis.
For probabilistic, I would add just a simple count of exceedences.
General principle - let's play around a bit, but then narrow down on the metrics we wish to see repeated when we do this analysis again, as I'm 100% sure we will want to re-run this in another few months as we improve PVNet2.

Yep, exceedence and pinball, agreed.

I'm going to close this because I think it has been completed


Set the Animation Type with parameters through XIB

Set the Animation Type with parameters through XIB, for example: slide(way: way, direction: direction)

@JqyModi sorry, I didn't see your message, please have a look at https://github.com/IBAnimatable/IBAnimatable/blob/master/Documentation/IBAnimatable 3.0 Migration Guide.md#changes-for-animationtype thanks


update many docs to point at new locations and have better instructions

these docs are also pushed to be live at the /v/0.9.0-SNAPSHOT/ endpoint

merging as they'll be much better than the old ones.  CTR request!  @rdowner you wrote the original so I'd esp like to know if this looks good to you, esp the way we now instruct for multi-repos, and how the org names here are changed

I'm happy with the content.
However I think that this information should be in the "Developers" section of the website, and not part of the user guide (same comment applies to PR #1 on this project)
Reasons:

The first place a would-be developer would look when wanting to get started is under the "Developers" heading on the main website. This information is not directly linked.
Putting it in the user guide freezes it in time once a release is made. If the procedures change again in the future, the old version will permanently remain. Whereas by putting it in the website, always the latest information is permalinked and obsolete information removed.
It currently only exists as a /v/0.9.0-SNAPSHOT/* URL which is an obscure URL (i.e. starting from brooklyn.apache.org it is many clicks to get there)

After trying out the script a couple of times, IMO trying to script it is the wrong way to go about things. There are assumptions and edge cases which can trip it up. It also unnecessarily imposes a naming convention on our developers. Can we simply come up with a couple of one-liner commands which handles the basic case and let the power users figure out the rest themselves?
e.g. the basic clone-and-get-submodules in the "Get the code" page does a very effective job. Its only issue is that the remote is named origin when upstream would be more accurate. So maybe suggest that the user runs:
git remote rename origin upstream; git submodule foreach 'git remote rename origin upstream'

To fork everything, you just run this command:
hub fork; git submodule foreach 'hub fork'

This leaves you with a new remote that is your GH user ID, which is sensible. A power user who prefers a different name will know how to rename it, if that is what they want.
And then, default to pushing to your own repo:
git config remote.pushDefault mygithubid; git submodule foreach 'git config remote.pushDefault mygithubid'

I think this handful of one-liners will cover 75% of developers; it means I can do this:
# git branch my-new-feature-branch upstream/master
# git checkout my-new-feature-branch
# [make some commits]
# git push
To https://github.com/rdowner/brooklyn.git
 * [new branch]      my-new-feature-branch -> my-new-feature-branch

Note how my branch is tracking upstream/master for the purpose of git pull, but a git push goes to my own fork. (The remaining 25% of developers will be power users who know what they are doing!)
All the stuff about the GitHub PR branches and adding the Apache pure-git repo is unnecessary for almost everyone so can be omitted. Committers do need to know it, but that is what the Committer Guide is for :smile:

@rdowner i strongly agree with almost everything.  i will:

move github instructions to the unversioned "Developers" section
provide line-by-line instructions rather than a brittle magic script

the only thing i'm unsure of is PR's -- people other than committers may want to review PR's.  however i'll look for a good way to handle that.

@richardcloudsoft I am testing out your notes, one minor issue - hub fork requires the presence of an origin remote, so should be executed before the rename.

@neykov @richardcloudsoft see https://github.com/apache/brooklyn-docs/pull/11 -- i do the hub fork first


KAFKA-2327

Added unit tests as well. These fail without the fix, but pass with the fix.

LGTM. Thanks for the catch. I'm running tests locally to validate and if all passes I'll (try to) merge.

In fact, those new test methods came from testAdvertiseConfigured, right? So it would make sense to change props.put(KafkaConfig.AdvertisedPortProp, advertisedPort.toString) there too. ;)

@eribeiro Good point :)
@gwenshap Thanks for the quick review - have you seen this? @ijuma posted merge instructions for github here: https://cwiki.apache.org/confluence/display/KAFKA/Patch+submission+and+review#Patchsubmissionandreview-MergingGitHubPullRequests
And also this:
https://cwiki.apache.org/confluence/display/KAFKA/Contributing+Code+Changes

Note: the merge script is attached to the JIRA issue, but it's still waiting for someone to merge it.

Issue link https://issues.apache.org/jira/browse/KAFKA-2187

@granders, can you change the PR name to something that matches our standard? It looks like the merge script automatically uses the PR name as the commit comment, so we need something like:
"KAFKA-2327; broker doesn't start if config defines advertised.host but not advertised.port"
Or @ijuma can modify the script to allow editing the commit message :)

@gwenshap Sounds good - I changed the PR title - does that do the trick?

Yes! Should be merged now.

Awesome, thanks!


Ability to render math equations on Project Euler Feed

Prerequisites

[X] Put an X between the brackets on this line if you have done all of the following:

Checked that your issue isn't already filed: https://github.com/GetStream/Winds/issues

Description
Problems from the Project euler feed, such as this do not display equations correctly (in the web viewer at least), is there a way of showing these equations?
Steps to Reproduce

Open example page, or other Project Euler question

Expected behavior: [Parse LaTeX equations and render in human readable format.]
Actual behavior: [Shows equations as plain text, e.g. $x+1$.]
Reproduces how often: [Always.]
Versions
all versions up to 3.1.6
Additional Information
none.

Will be happy if parsing this requires enabling a config, I don't see a need to automatically parse every article from every feed, but I would like to enable parsing for all articles for a specific feed

@micsthepick I see that formulas are parsed and passed the filter of Winds sanitizer, so it's the matter of styling right?
Winds uses react-html-parser library to parse the article content, feel free to create a PR to address this issue.


ItchySats 0.4.1 "got stuck" after startup

Behavior:

Start the daemon after upgrading
The usser interface is reachable everything seems fine
Run into the Not connected to daemon error in the UI. The wallet balance goes to 0
Refresh does not help.
Check the logs, the logs are stale, no new logs are printed anymore (for more then a minute); unfortunately don't have the logs, but there were no errors or warnings. The last message I saw was the one saying that the Heartbeat pulse measurements fell behind more than heartbeat interval (10), likely missing a heartbeat from the maker. Diff between pulses: 10 - but we see that one quite often at the moment, so not sure if that is relevant
Restarted daemon and did not run into issues

Might have been a glitch with Electrum not being available (yet). My guess goes to Electrum because the wallet balance was 0. Not sure we can make that a better experience?
The fact that we did not see any more log messages points more towards a deadlock somewhere, hard to say. I thought I'll report it and see if I can ever reproduce it.
I don't think this should be related to the changes between 0.4.0 and 0.4.1.

Closing because this version is no longer running in production.


add 0.22.0 client deregister feature

See: https://github.com/sensu/sensu/blob/master/CHANGELOG.md#0220---2016-01-29

+1

I like it! :wink:

Actually cherry picked to develop in https://github.com/sensu/sensu-chef/commit/9ff7841eedb63744bee2c38db803de4a3c214576


Go back to main branch for dora

Fixes #532
The new ruby buildpack has been released. This contains the lax version matching for bundler and is therefore supposed to work also with the main branch of cf-acceptance-test that references an older version of bundler.

/lgtm
/approve

/lgtm
/approve


bug: fix org scoped kvm import issue #220

Fixed the org scoped kvm import for issue #220.

@g-greatdevaks - can you please fix the lint issue?

I have addressed the lint issue. Kindly review the PR.

I've verified the bug and tested the fix, LGTM!


change type cast method

fix bug of intsalling on windows with vs2022

Can you please accept the DCO (git commit --amend --no-edit --signoff) :pray:


EXT: ParticleWeighting

Description
This moves parts of the ED-PIC extension into its own extension for reuse in other domains.
We already conditionally honor weightings in tools like openPMD-viewer and its not specific to PIC simulations.
Implements issue: #237
Affected Components

[x] base
[x] EXT:  ED-PIC
[x] EXT:  ParticleWeighting

Logic Changes
The definition of the weighting record and the macroWeighted and weightingPower attributes is moved from ED-PIC to its own extension.
ED-PIC now depends on the new extension, so there is no change to the ED-PIC extension on a logical level.
Writer Changes

[x] openPMD-api: https://github.com/openPMD/openPMD-api: not affected, because added by users manually

Reader Changes

[ ] openPMD-validator: https://github.com/openPMD/openPMD-validator can be generalized to check weighting if either ED-PIC or ParticleWeighting (or both) are declared
[x] openPMD-viewer: https://github.com/openPMD/openPMD-viewer already applies weighting conditionally if found; can be generalized to check the extension first
[x] yt: https://github.com/yt-project/yt not affected
[x] VisIt: https://github.com/openPMD/openPMD-visit-plugin not affected
[x] converter: https://github.com/openPMD/openPMD-converter not affected
[x] openPMD-api: https://github.com/openPMD/openPMD-api not affected, because read by users manually

Data Updater

[x] https://github.com/openPMD/openPMD-updater no update needed

@AngelFP exactly - so if you want to use this as a writer, you add the ParticleWeighting string in the semicolon-separated list of the openPMDextension attribute and then declare the things written in this extension.
For readers, they check if the openPMDextension contains the ParticleWeighting string (or the ED-PIC) and if it's present, the conventions for weighting were used by the writer.


The live webcam does not exit and produce out on keypress

Windows 10
Python 3.7.7
In the video_demo.py file after I run it and it is recording from my webcam I cannot close the window.  From code I think it is cv2.waitKey(1).  I tried various version such as specifying a specific key pres for cv.waitKey but I still had to shut down from killing the command line and no output was produced.

It seems to be the opencv problem, not the repo. You can check the code logic and find if there exists some dead-loop.
Honestly, the graphic system like opencv or other renders often crash on my macOS : (


bug v3: auto_close and auto_open broken

Did you check docs and existing issues?

[X] I have read all the trouble.nvim docs
[X] I have searched the existing issues of trouble.nvim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
NVIM v0.10.0-dev-2949+g206475d79-Homebrew
Operating system/version
MacOS 14.4.1
Describe the bug
I can't get auto_open and auto_close to work even in the simplest configuration. Am I missing something?
Steps To Reproduce

nvim -u repro.lua
<leader>a, <leader>r
<leader>xx
<leader>a, <leader>r

Expected Behavior
I would expect the Trouble window to open after first <leader>a, and close after last <leader>r.
Actual behavior:
https://github.com/folke/trouble.nvim/assets/16707112/1bb164f9-54e9-434a-af77-e874e06cb0f3
Thanks for the plugin!
Repro
Btw, the issue template for v3 doesn't contain branch = "dev", this may lead some people into installing v2 when creating MWEs by mistakes.
-- DO NOT change the paths and don't remove the colorscheme
local root = vim.fn.fnamemodify("./.repro", ":p")

-- set stdpaths to use .repro
for _, name in ipairs({ "config", "data", "state", "cache" }) do
  vim.env[("XDG_%s_HOME"):format(name:upper())] = root .. "/" .. name
end

-- bootstrap lazy
local lazypath = root .. "/plugins/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
  vim.fn.system({ "git", "clone", "--filter=blob:none", "https://github.com/folke/lazy.nvim.git", lazypath, })
end
vim.opt.runtimepath:prepend(lazypath)

-- install plugins
local plugins = {
    "folke/tokyonight.nvim",
    {
        "folke/trouble.nvim",
        branch = "dev",
        keys = {
            {
                "<leader>xx",
                "<cmd>Trouble diagnostics toggle<cr>",
                desc = "Diagnostics (Trouble)",
            },
        },
        opts = {
            auto_close = true,
            auto_open = true,
        }
    },
    {
        "nvim-treesitter/nvim-treesitter",
        build = ":TSUpdate",
        config = function ()
            require("nvim-treesitter.configs").setup {
                ensure_installed = { "markdown", "markdown_inline" },
            }
        end
    },
}

require("lazy").setup(plugins, {
    root = root .. "/plugins",
})

vim.cmd.colorscheme("tokyonight")
-- add anything else here

local ns = vim.api.nvim_create_namespace("test")

vim.keymap.set("n", "<leader>a", function()
    print("Adding diagnostics")
    vim.diagnostic.set(ns, 0, {{
        lnum=0,
        col=0,
        message="test",
        severity = vim.diagnostic.severity.ERROR,
    }})
end)

vim.keymap.set("n", "<leader>r", function()
    print("Removing diagnostics")
    vim.diagnostic.reset()
end)

I have also noticed that the latest version of trouble auto closes too early (when resolving the current error, not all errors).

Should be fixed now. Thank you for the detailed bug report!


chore: add README table of contents

This PR introduces markers on the README file for table of contents to be used by an application to generate it, alongside an action that checks whether it is up to date or not. CI will prompt the user to run the command if the table of contents is out dated.

Example error: https://github.com/konflux-ci/konflux-ci/actions/runs/9190224546/job/25273975445?pr=90


travis-ci flooded by builds

Hi, I would like to discuss the possibility to limit how many things gets built by travis-ci for this repo.
All other repos under Ouranosinc are stalled by the massive queue of xclim builds generated by various commits and/or forks each time anything is pushed.
Is it possible to improve the if rules of the travis config?
example below spans from build#2280 upto build#2295  (ie 15 builds of 10+ jobs each, spawned within seconds)

@Zeitsperre We can remove the documentation builds since it's now done on RtD.
Can we run the "Allowed Failures" on releases only ?
Can linting go with the black, flake8 stuff that does not require installation ?
The doctests could be run with one of the normal test suites.

@fmigneault @huard
Yes, absolutely, can clean this up a bit. Those suggestions are pretty great/no-nonsense. I can start on this right now. The one thing I would do differently is run the allowed_fail builds on master branch only. I'd like some warning before tagging a release that breaks on Windows/macOS.


Out of scope question - any plans on mirroring whole github like ghtorrent?

I see this repo tries to get all public github events and archive them for analysis. But do you have any plans for archiving the code from all the commits for further analysis based on code, languages and such?

Nope, explicitly out of scope.. and GHTorrent already solves that. :)


Add Declared License

Type: Incomplete
Summary:
Add Declared License
Details:
Adding MIT
Resolution:
Only reference to license found is on this page:
https://mvnrepository.com/artifact/args4j/args4j/2.0.12
Affected definitions:

args4j 2.0.12

Confirming source repo license is currently MIT: https://github.com/kohsuke/args4j/blob/master/LICENSE


fix: disable cancel button on a disabled select widget

Description
Disable the cancel button on a disabled select widget
Fixes #13254
Type of change

Bugfix (non-breaking change which fixes an issue)

How Has This Been Tested?

Unit Test

Checklist:

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

Test coverage results :test_tube:
:green_circle: Total coverage has increased
// Code coverage diff between base branch:release and head branch: fix/disable-cancel-button-on-a-disabled-select-widget

Status
File
% Stmts
% Branch
% Funcs
% Lines

:green_circle:
total
56.41 (0)
37.85 (0.02)
36.01 (0)
56.63 (0.01)

:green_circle:
app/client/src/components/ads/Icon.tsx
99.44 (0.01)
85 (0.79)
100 (0)
99.43 (0)

:green_circle:
app/client/src/selectors/commentsSelectors.ts
85.25 (1.64)
64.71 (2.95)
73.33 (0)
90.59 (2.35)

:green_circle:
app/client/src/utils/WorkerUtil.ts
89.76 (0)
72.55 (1.96)
100 (0)
93.33 (0)

:green_circle:
app/client/src/widgets/SelectWidget/component/SelectButton.tsx
100 (0)
83.33 (33.33)
100 (0)
100 (0)

/ok-to-test sha=2d57840

/ok-to-test sha=47398e2

/ok-to-test sha=70c475f

/ok-to-test sha=76b2e21

/ok-to-test sha=fa48a02

Tested this PR and working as expected

Disable the cancel button on a disabled select widget - Fixed

/ok-to-test sha=de8aebf

/ok-to-test sha=fdc3143

/ok-to-test sha=2f8c178


Feature/dockstore 263 fix tests

Adding a few constraints to make tests more deterministic.
Also includes a database update.

Coverage remained the same at 71.402% when pulling b6b4d2ddd8ab7eeecc73a0684ec1b87b3ed14cca on feature/dockstore_263_more_refresh_file into dbad2167f6b48b323ebf6b23a60084bab3be78b1 on develop.


MVP spec mismatch when client calling withdraw()

https://github.com/omisego/plasma-mvp/blob/1683070b28bb64b0193f7dd4220957849fd6d317/plasma/client/client.py#L47
According to the Minimal Viable Plasma,
priority is <blknum *<PHONE_NUMBER> + txindex * 10000 + oindex>.
So, <txPos[0] *<PHONE_NUMBER> + txPos[1] * 10000 * txPos[2] + 0> should be changed to
txPos[0] *<PHONE_NUMBER> + txPos[1] * 10000 + txPos[2] * 0

Yep this looks like a bug, should be
utxoPos = txPos[0] *<PHONE_NUMBER> + txPos[1] * 10000 + txPos[2] * 1


Update build.gradle

Updated gradle version, compileSdkVersion with the corresponding buildToolsVersion. Using implementation instead of compile as of gradle 3.0.1.

Could this be merged ?

Partially duplicate of #692

@sesam does it mean the PR #692 shall be closed in favour of this one, or both should be merged?

I'd prefer to wait with this until react-native itself adopts gradle 3. Don't want to break existing installations unnecessarily.

as of this PR all looks good except Gradle 3.0.1 classpath 'com.android.tools.build:gradle:3.0.1' currently as by latest react-native 0.56.0 they upgraded there gradle to com.android.tools.build:gradle:3.1.2 would that create issue?

@yeomann What do you mean? React Native is still on Gradle 2: https://github.com/facebook/react-native/blob/18e9ea211296d2ce553d3fb37411f18e31d00181/build.gradle#L16

As per Change log
Upgrade gradle-plugin to 2.3.3, gradle to 3.5.1, gradle-download-task to 3.4.3
@oblador  so I guess its Gradle 3

AFAIK you need gradle-plugin 3 to use implementation, but might be wrong. What I do know though is that this breaks a fresh project generated with react-native init and that’s not something I can accept. If you have a solution for this, I’m happy to merge.

@yeomann: I'm aware, but you don't seem to be getting the implications of breaking tens of thousands projects. Working out of the box for a fresh project is a strict requirement from my end. Steps to reproduce:
react-native init RNVITest
cd RNVITest
yarn add mahomahoxd/react-native-vector-icons # <-this PR
react-native link react-native-vector-icons
react-native run-android

Fails with:
FAILURE: Build failed with an exception.

* Where:
Build file '/Users/joel.arvidsson/Code/RNVITest/node_modules/react-native-vector-icons/android/build.gradle' line: 4

* What went wrong:
A problem occurred evaluating project ':react-native-vector-icons'.
> Could not find method google() for arguments [] on repository container of type org.gradle.api.internal.artifacts.dsl.DefaultRepositoryHandler.

Just changing the implementation gives:
> Could not find method implementation() for arguments [com.facebook.react:react-native:+] on object of type org.gradle.api.internal.artifacts.dsl.dependencies.DefaultDependencyHandler.

@oblador

but you don't seem to be getting the implications of breaking tens of thousands projects.

I am very aware! my point is just this By the end of 2018, Compile has to be "implementation" by Google
For those who decided NOT to do, will be left out. Google is ENFORCING that this new changes should be implemented, all Older apps would be require to update!
regarding

react-native init RNVITest
cd RNVITest
yarn add mahomahoxd/react-native-vector-icons # <-this PR
react-native link react-native-vector-icons
react-native run-android

yup I tried and same error. Error is because of
dependencies {
    classpath 'com.android.tools.build:gradle:3.1.0' // or even can be 3.1.2
}

I felt like blaming this issue to React-Native because right now, We are forced to use "implementation" That's why by default react-native should configure with Gradle 3+ instead they are having classpath 'com.android.tools.build:gradle:2.3.3'
According to what I think is that, developers are forced right now to use "implementation" thus, everyone is going to manually upgrade gradle and edit each npm module with gradle setup until react-native 0.57.0 will come because I know in RN 0.57.0 they are working on Gradle 3+

Next version of react native will most likely fix this and it’ll be released before the end of this year (according to the release cycle in a few weeks, but it hasn’t been followed with the latest release). My current plan is to wait for react native core to at least release an RC with support before I start migrating my packages.

in that case, I think would be nice if we can add a little note in readme, regarding "obsolete" error.


git-machete: 3.3.0 -> 3.4.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for git-machete is: "Git repository organizer and rebase/merge workflow automation tool"
meta.homepage for git-machete is: "https://github.com/VirtusLab/git-machete"
meta.changelog for git-machete is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

Warning: a test defined in passthru.tests did not pass

/nix/store/jy8q77d7bmvp542wi29y6rna5jncj456-git-machete-3.4.1/bin/git-machete passed the binary check.

/nix/store/jy8q77d7bmvp542wi29y6rna5jncj456-git-machete-3.4.1/bin/.git-machete-wrapped passed the binary check.

0 of 2 passed binary check by having a zero exit code.

2 of 2 passed binary check by having the new version present in output.

found 3.4.1 with grep in /nix/store/jy8q77d7bmvp542wi29y6rna5jncj456-git-machete-3.4.1

found 3.4.1 in filename of file in /nix/store/jy8q77d7bmvp542wi29y6rna5jncj456-git-machete-3.4.1

directory tree listing: https://gist.github.com/e8b14168bbc0baeab970d12d088e25d0

du listing: https://gist.github.com/469fb821d276d522f3d719ed7014bfe0

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

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
git-machete

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/jy8q77d7bmvp542wi29y6rna5jncj456-git-machete-3.4.1 \
  --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 git-machete https://github.com/r-ryantm/nixpkgs/archive/302d33f4d1cfb89ad9cc75cfa9d4247d79899dbb.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/jy8q77d7bmvp542wi29y6rna5jncj456-git-machete-3.4.1
ls -la /nix/store/jy8q77d7bmvp542wi29y6rna5jncj456-git-machete-3.4.1/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 run on x86_64-linux 1

1 package built:

git-machete

Maintainer pings
cc @blitz for testing.

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

1 package built:

git-machete


feat(payment): PAYPAL-1208 bump checkout sdk version

What?
Bump checkout-sdk-js version
Why?
According to task https://jira.bigcommerce.com/browse/PAYPAL-1208
Checkout-sdk-js PR: https://github.com/bigcommerce/checkout-sdk-js/pull/1313
Testing / Proof
Tested on dev and int
@bigcommerce/checkout

We have updated Circle Ci configration today.
Could you please fetch and merge the lastest master, and then all CircleCi checks would be good.

Hi there, it is already on v1.209.0 now.
https://github.com/bigcommerce/checkout-js/blob/bc17f27e5ddf418784bb33953df994c0fb264344/package.json#L34


pageserver: make local timeline deletion infallible

Problem
In https://github.com/neondatabase/neon/pull/9589, timeline offload code is modified to return an explicit error type rather than propagating anyhow::Error.  One of the 'Other' cases there is I/O errors from local timeline deletion, which shouldn't need to exist, because our policy is not to try and continue running if the local disk gives us errors.
Summary of changes

Make delete_local_timeline_directory and use .fatal_err( on I/O errors

Checklist before requesting a review

[ ] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.
[ ] Do we need to implement analytics? if so did you add the relevant metrics to the dashboard?
[ ] If this PR requires public announcement, mark it with /release-notes label and add several sentences in this section.

Checklist before merging

[ ] Do not forget to reformat commit message to not include the above checklist

@erikgrinaker for context, our internal note on infallible local I/O is https://docs.neon.build/storage/handling_io_and_logical_errors.html (not all code respects this yet)


Added Deploy To Bluemix button to the README

After adding support to deploy MEANJS to Cloud Foundry, this pull request adds support for the "Deploy To Bluemix" button.  With a single click this button will deploy a working copy of MEANJS to Bluemix.  In addition to the running instance of MEANJS in the cloud, it also creates a copy of the code in a Git repo in Bluemix and sets up a build and deploy pipeline to the repo.  This allows the user to deploy changes made to MEANJS to Bluemix just by doing a git commit.
To see this functionality in action you can checkout the README in my fork.  Just click the Deploy To Bluemix button to test it out.

@ryanjbaxter I signed up for Bluemix today to try this out and I'm still waiting on my confirmation email to finish signing up to try this out.

@codydaig great let me know how it goes

@ryanjbaxter How long does it normally take to get the confirmation email?

@codydaig generally not too long, but give it 24 hrs if it does not arrive by tomorrow let me know

@ryanjbaxter This worked! It just took almost 12 hours to get the email to be able to signup for an account.
@lirantal LGTM as long as we're okay with putting a IBM Bluemix button in the README.

@codydaig great thanks.  Not sure why it took so long to complete the registration.  Usually it is almost instantaneously.  It is so much easier to use the button (as long as the registration happens quickly) than having to do it using the command line.

@ryanjbaxter I agree. It is a lot easier. I'm not opposed, but being an open project, I'm always afraid of being too opinionated. But if other people in the project are cool with it then I say let's merge it.

@codydaig sure I generally don't mind.

@lirantal LGTM :-D

Thanks!
-Ryan

On Aug 21, 2015, at 5:40 PM, Liran Tal<EMAIL_ADDRESS>wrote:
Merged #790.
—
Reply to this email directly or view it on GitHub.


Updates the URL to a linkless label with "spo navigation node set". Closes #5228

Updates the URL to a linkless label with "spo navigation node set". Closes #5228

Thanks for your PR @Saurabh7019! We'll review it asap.

Merged manually, thank you for this fix! 👏


https://validkube.com/ CORS Issue

Hi
It appears the tool hasn't been working for some time now.
For example when pressing  the clean button on the web UI it says
Cross-Origin Request Blocked: The Same Origin Policy disallows reading the remote resource at https://gtgmn58dh9.execute-api.us-east-1.amazonaws.com/production/kubeneat. (Reason: CORS header ‘Access-Control-Allow-Origin’ missing). Status code: 502.

Thanks for the feedback, I believe it's a new regression, will fix asap :)

hey @bio-boris, I see this is happening only in the clean button. Is that occurred also in other endpoints ?

Validate may have had the same error I don't remember anymore

Hi @bio-boris, we solved the issue, you can take a look at the solution at that commit b195bb26ed6ec42ac429e9e0ea8e87b44040576d

LGTM Thanks!


WIP: 1.0.0

Closes #10
Closes #17
Resolves #23
Resolves #28

@mbaumgartenbr Is this release contains move to last visible child on opening a node?
example
https://github.com/embraceitmobile/animated_tree_view/blob/d7fb5bd6f0c760b3a7e8e430c2695b26dd555605/lib/tree_view/animated_list_controller.dart#L142
Or should I open an issue?

Hey @amorenew!
I did have it auto scrolling previous to #4f7cf190ec3f1f07a77da569f79e786c0044b344 but I had to remove it as it was causing some issues when asynchronously expanding nodes (i.e. a lazy tree view that loads the children when expanding the parent).
I'm not sure I'll be implementing auto scrolling with this version since it can be easily done outside the package using packages like scroll_to_index. Auto scrolling is very specific for each use of treeviews, be it a filesystem view, a catalog, database overview, etc. I may work on it in the future.
I think once I get the API done I'll write a simple example using scroll_to_index.
Feel free to open the issue!


IndieWeb "Metal" swag

From an idea brainstormed in a HWC event, I commissioned an alternative design for a tee shirt/sticker.

I have a SVG and EPS I can share to get a design in the official store.

I love it!


Cannot enter ~ symbol in simple dialog

Issue Type: Bug
When the simple dialog is enabled, the address cannot be entered with the ~ symbol, and every time you enter ~, the entire path is replaced with the home directory.
VS Code version: Code 1.45.1 (5763d909d5f12fe19f215cbfdd29a91c0fa9208a, 2020-05-14T08:27:22.494Z)
OS version: Linux x64 5.3.0-3-amd64

System Info

Item
Value

CPUs
AMD Ryzen 7 3750H with Radeon Vega Mobile Gfx (8 x 1582)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: disabled_softwareskia_renderer: disabled_off_okvideo_decode: unavailable_offviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
1, 2, 2

Memory (System)
14.66GB (1.37GB free)

Process Argv
--no-sandbox --unity-launch

Screen Reader
no

VM
0%

DESKTOP_SESSION
deepin

XDG_CURRENT_DESKTOP
Deepin

XDG_SESSION_DESKTOP
deepin

XDG_SESSION_TYPE
x11

Extensions (4)

Extension
Author (truncated)
Version

doxdocgen
csc
0.6.0

githistory
don
0.6.5

vscode-language-pack-zh-hans
MS-
1.45.1

cpptools
ms-
0.28.1

This is intentional and was a feature request. The idea was to provide a quick way to go to user home.

I know that, but I think it should be done only when the path begins with the ~ character, not when you type the ~ in the path.


translation keys prefix

Update TerriaMap to match changes from https://github.com/TerriaJS/terriajs/pull/5611

[ ] Release terriajs 8.0.0-alpha.88
[ ] Upgrade TerriaMap@next to use new version
[ ] Merge this


Impossible to upload a file through Phoenix / Absinthe

Describe the bug
My backend doesn't recognize the file upload (Elixir/Phoenix with Absinthe)
Expected behavior
It should upload it without issue.
Details of the problem
If I use cURL it'll work perfectly. For instance, when trying to upload a CSV file I do like this
Client query
curl -X POST -F query="mutation { importTeamFromCsv(file: \"csv\") { id } }" -F<EMAIL_ADDRESS>localhost:4000/graphql

Server response
iex(13)> [info] POST /graphql
[debug] Processing with Absinthe.Plug.GraphiQL
  Parameters: %{"csv" => %Plug.Upload{content_type: "application/octet-stream", filename: "team-members-list.csv", path: "/var/folders/j8/2v58d41s1tl3q716mb9rs3mm0000gn/T//plug-1614/multipart-1614961290-359254200507814-8"}, "query" => "mutation { importTeamFromCsv(file: \"csv\") { id } }"}
  Pipelines: [:graphql, :authenticated]
[debug] ABSINTHE schema=Bigseat.Schema variables=%{}
---
mutation { importTeamFromCsv(file: "csv") { id } }

This is working and will even touch some breakpoint placed afterwards in the code. On the other end, when going through Altair GraphQL this will happen
Client query
mutation importTeamFromCsv(
  $file: Upload!
) {
  importTeamFromCsv(
    file: $file
  ) {
    email
    firstName
    lastName
  }
}

Server response (in Altair GraphQL)
{
  "errors": [
    {
      "message": "No query document supplied"
    }
  ]
}

Server response (in the console)
iex(14)> [info] POST /graphql
[debug] Processing with Absinthe.Plug.GraphiQL
  Parameters: %{"0" => %Plug.Upload{content_type: "text/csv", filename: "team-members-list.csv", path: "/var/folders/j8/2v58d41s1tl3q716mb9rs3mm0000gn/T//plug-1614/multipart-1614961388-269174870404028-8"}, "map" => "{\"0\":[\"variables.file.0\"]}", "operations" => "{\"query\":\"mutation importTeamFromCsv(\\n  $file: Upload!\\n) {\\n  importTeamFromCsv(\\n    file: $file\\n  ) {\\n    email\\n    firstName\\n    lastName\\n  }\\n}\",\"variables\":{\"file\":[null]},\"operationName\":\"importTeamFromCsv\"}"}
  Pipelines: [:graphql, :authenticated]
[debug] QUERY OK source="people" db=2.7ms idle=1997.8ms
SELECT p0."id", p0."organization_id", p0."email", p0."encrypted_password", p0."first_name", p0."group", p0."type", p0."origin", p0."is_admin", p0."last_name", p0."api_key", p0."inserted_at", p0."updated_at" FROM "people" AS p0 WHERE (p0."api_key" = $1) ["pSixdnqTFjYof6P_uofYkDTgcmVMpBPfmC75QYdwO6KKEt7Qw8YP9nR7r7bspklDJjixbhzeem9Y7kit1gBJKw=="]
[info] Sent 400 in 7ms

I've tried changing the headers from Altair but setting it to multipart or octet-stream will crash in the same kind of way, only with a parsing error. I don't believe I should add any headers but I can say I've tried
With headers changes
{
  "errors": [
    {
      "locations": [
        {
          "column": 1,
          "line": 1
        }
      ],
      "message": "Parsing failed at `------WebK`"
    }
  ]
}

Screenshots

Working breakpoint

Non working query from Altair GraphQL

Hello @Loschcode I'm not sure what file upload mechanism exists in phoenix/absinthe but Altair uses the GraphQL multipart request spec, and if you look at the contents of its readme, you'd see that the cURL command equivalent of what Altair does is different from the one you have used. So that makes me think that the upload mechanism is using a different specification which Altair doesn't recognize/support.


Use linuxserver base image

Addresses #16 by adopting linuxserver base image and adding unifi-protect-backup run process.

Made some minor changes and added a potential fix for the backwards compatibility issue. I will test this out tomorrow and if it works out I will squash the commits and merge. Thanks for your help!

Thank you for this work, I learnt some new things about docker in the process which is awesome.

No problem! thank you for building this utility! It allows me to expand my Unifi Cloud Key beyond 5TB lol


Convertion of OemText to Text fails

Used this version with the current DevPreview of NAV:

My app.json contains:
{

  "target": "Internal"
}

I have reduced the problem to the following code snipped:
pageextension 50100 CustomerListExt extends "Customer List"
{
    trigger OnOpenPage();
    var Object:      Record Object;
        name:        Text;
        companyName: Text;
    begin
      if Object.FindSet then
      begin
          // *** This convertion fails: OemText to Text ***
          name        := Object.Name;
          companyName := Object."Company Name";
      end;

      Message('First Object: %1 (company: %2)', name, companyName);
    end;
}

When I open the Customer List this message is displayed:

Is there another way to convert the OemText into a valid string?

We'll investigate and get back to you on this one.


Remove outliers in trade aggregation

This problem arises for trade dust in markets where the price is relatively* close to 1.
For example, let's take a real market (currently the most liquid market on the stellar DEX) XLM/CNY.
The price for this pair had been within an order of magnitude of 1 for a while now. Suppose the pair is trading in the 1.9 XLM/CNY and I enter a an ask for 0.0000001 CNY at that price.
Now, the outcome of course depends on how stellar core handles rounding in these kinds of scenarios. I actually haven't really ever dug into this issue but always assumed that stellar core just rounds against the maker. So, in this case, it would round down (in the case of a bid it would round up).
In this case, I cannot receive 0.00000019 XLM for my 0.0000001 CNY because there are only 7 decimal places available. Instead, when my offer gets taken I receive 0.0000001 XLM. Of course, the amounts here are miniscule and I might not care.
However, these dust trades have a significant impact on charts. They can corrupt highs and lows for an entire day of aggregated trades. Additionally, they force the chart to scale out to such an extent that the actual price action is difficult to see.
In the past, I have simply hacked a solution by disallowing trades with amount < 0.000001. However, after thinking about it I think perhaps a better solution would be something like:
Disallow a trade where: amount A is < 0.000001 and amount B is < 1 (or something to this effect). This would ensure that for really extreme prices (i.e. BTC/DOGE), trades with BTC < 0.000001 would still be considered.
I hope my explanation is clear.

@jedmccaleb Oh, woops I totally forgot about this issue. My bad.  However, buried in there is a real issue that I have had to deal with in constructing lupoex (I have been aggregating trades on the stellar DEX for a while now). I have updated my OP with what I think is a more rational explanation.


Fixed mistakes

@yihezkel : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Install-PSResource not adding Scripts install path to $env:PATH and allowing to invoke installed script

PR Summary
Installing a script with Install-PSResource happens successfully except that the script installation path (%USERPROFILE%\Documents\PowerShell\Scripts) isn't added to the environment PATH variable, so the script can't be invoked without prepending install folder path.
PR Context
Resolves #709 and was working in V2.
PR Checklist

[ ] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[ ] Summarized changes
[ ] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[ ] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[ ] None
OR
[ ] Documentation needed

[ ] Issue filed:

User-facing changes

[ ] Not Applicable
OR
[ ] Documentation needed

[ ] Issue filed:

Testing - New and feature

[ ] N/A or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[ ] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and enumerated concerns in the summary.

V2 prompts for user to accept/confirm the action before setting the environment variable.  I'm not a fan of prompts but since it's changing the env var, I think it might be important to include (see: https://github.com/PowerShell/PowerShellGetv2/blob/5aba7531f526b6e60b2757c36799995597b6dcec/src/PowerShellGet/PSGet.Resource.psd1#L222 for the v2 prompt)

This looks good to me!  Should be good once the tests are resolved.


Path to registry repositories fails when registry name has invalid URL characters

We should not use the registry name to construct a URI. A uuid should be best.

Fixed in #62


warning: problem when requiring fullcalendar/fullcalendar

Currently you require fullcalendar/fullcalendar. This may cause problems in certain Contao 4 installations. Since fullcalendar/fullcalendar also requires components/jquery and since Contao 4 previously used
"config": {
    "component-dir": "assets"
}

in the root composer.json, components/jquery will also get installed into assets/ and thus may delete and overwrite contao-components/jquery, breaking the JavaScript of the Contao installation, if jQuery is enabled in the page layout, since the path to jQuery will not be in the expected one (and the version will automatically bump from 1.x to 3.x).
To fix this, users have to delete
"config": {
    "component-dir": "assets"
}

from their root composer.json and instead add
"extra": {
    "contao-component-dir": "assets"
}

This will install contao-components/jquery into assets/ and components/jquery into components/.
See also https://community.contao.org/de/showthread.php?71651-Nach-Update-auf-4-4-21-wird-jquery-nicht-mehr-lokal-geladen-nur-CDN

Problem ist behoben. Neue Version steht zur Verfügung. Danke für das Feedback.


fix(iroh-relay): do not use spawn_blocking in stun handler

Description
This resulted in memory accumulation , due to blocking threads not getting shutdown, once started. According to the docs from tokio, there is no way to cleanup these tasks/threads, once they have been started.
From the docs: https://docs.rs/tokio/latest/tokio/task/fn.spawn_blocking.html

Be aware that tasks spawned using spawn_blocking cannot be aborted because they are not async. If you call abort on a spawn_blocking task, then this will not have any effect, and the task will continue running normally. The exception is if the task has not started running yet; in that case, calling abort may prevent the task from starting.
When you shut down the executor, it will wait indefinitely for all blocking operations to finish. You can use shutdown_timeout to stop waiting for them after a certain timeout. Be aware that this will still not cancel the tasks — they are simply allowed to keep running after the method returns. It is possible for a blocking task to be cancelled if it has not yet started running, but this is not guaranteed.

The referred to limit is 512 by default, meaning with the default thread stack size of 2MiB, we are looking at 1GiB of memory, simply for keeping around these threads.
Looking at https://docs.rs/tokio/latest/tokio/runtime/struct.Builder.html#method.thread_keep_alive these threads should get cleaned up after 10s, but this does not seem to happen. So there seems to be some unintended lingering to happen, in either case.
Breaking Changes
None
Notes & open questions
This is unfortuante, but the parsing of stun packets should be fast enough.
Change checklist

[x] Self-review.
[x] Documentation updates following the style guide, if relevant.
[x] Tests if relevant.
[x] All breaking changes documented.

the spawn blocking was introduced in #1091, do you or @flub who approved it remember why you thought it was necessary?

From my memory spawn_blocking was introduced as a knee-jerk reaction for some behaviour we didn't understand.  I recall looking several times at this and wondering if the threads overhead really was better than this tiny bit of computation... but didn't dare to touch it as it was a response to an incident.  I'm all for removing it, we can do better now (I mean both evaluating and responding to incidents as well as writing a scalable server).

Quoting dig from the depths of the discord archive, #1091 was introduced because:

I also added some potential improvements to the stun server to make it scale better here
not totally sure how much sense it makes though to spawn a task for each incoming req

Flub:

no way that scaling is the issue right now

Dig:

yes, but I wanted to make sure it wasn't

merging, if this turns out to be a problem, we have better ways looking at them now as @flub pointed out


Reserialize MRTK changes from 2.6 cycle

Overview
Some serialized fields have been added / removed / changed, so this brings some assets and scenes up to date with those changes.

/azp run


Remove duplicated Access Policy structs

When I was first putting Access support together, I made a boo-boo and
duplicated the Access Group structs into the Access Policy itself. This
shouldn't have been the case as they are the same structs, but they
should be tied to the groups and only referenced in the policies
instead.

Thanks, there is some inconsistency with our struct naming scheme, but possibly nothing we can clean up for now.


Updating to<EMAIL_ADDRESS>removes External Links from Navigations

I just updated to the latest version it looks like External Links within Navigations are not rendering.

This does appear to be a regression in 0.4.13. Links are missing in navigations in 0.4.13, but downgrading to 0.4.12 restores them. Possibly something missed in a37f111057c974e95b52e6a11c2dee97014dc95a ?

Yeah, I missed something in there. Sorry guys, been crazy busy so I haven't been able to get around to some of these issues yet :/

Looks like my check for enabled links broke this since externalLinks get their enabled attribute set to false. Pushed a fix. New version: +<EMAIL_ADDRESS>


General Journal batch posting locking out all users during delete journal deleteall

🕮 Describe the issue
Posting a journal batch takes about 5% of the total time.
My solution is to separate the posting of the journals from the deletion after posting of the journals.
To do this, I would update a new flag on the journal batch when the journals have posted.
Then execute a separate routine that deletes all the journals which will result in the Ledger tables not being locked. This will avoid the lockouts that occur when users post a very large journal batch.
Recently had the problem where a user posted 47,000 journal lines and locked the system for over 2 hours.
By separating the posting (using an event), then deleting the journals in a separate process, I reduced the lockout to only about 5 minutes.
🔧 To Reproduce
Steps to reproduce the behavior:

Creaate a large G/L Journal batch (at least 10,000 lines).
Click the post.
The post will show 100% complete, then take much longer to finish (this is the Deleteall statement when it deletes the posted journal lines.)
No error - just very, very slow.

📣 Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

This is probably good idea, but we cannot process it here. Should you create this idea on aka.ms/bcideas, it will be processed for the scope of next release.


feature: prometheus histogram metrics

Describe the Feature Request
Add Histogram metrics support https://prometheus.io/docs/practices/histograms/
Describe Preferred Solution
Annotation-based approach.

Hi @sbilello, we are using Micrometer as the mechanism to capture the metrics and they meters can be configured to export percentiles. Its a matter of just setting management.metrics.distribution.percentiles-histogram.gql=true.

@sbilello I think unless there is a specific feature that you suggest for the DGS framework, this looks more like a Q&A discussion.

This is cool! Thanks, @berngp!
Yes, we can move this as Q&A but I was wondering if it would make sense to specify the percentiles as part of the configuration to reduce the number of buckets. See something like https://github.com/micrometer-metrics/micrometer/blob/main/micrometer-core/src/main/java/io/micrometer/core/annotation/Timed.java#L54
I was also wondering why we have different labels for the metrics:
gql_query_seconds_bucket{gql_queryComplexity="25",outcome="success",le="0.001",} 0.0
gql_resolver_seconds_bucket{gql_field="Query.getUserByUserIdInput",outcome="success",le="7.158278826",} 20.0

I will work on a custom tag as described in the doc unless we can not make consistent the tags between these 2 metrics.
I would like to show the duration in percentiles by query name for the gql_resolver* and the gql_query* metrics

Not sure I understand which labels are you referring to on gql.query vs gql.resolver.
The gql.query captures the entire scope of the GraphQL Query. Since each GraphQL Query can have multiple fields we can't capture them as part of the gql.query tag, I don't believe there is a way to express an array of values associated to a tag puls the cardinality will be problematic. The gql.resolver on the other hand can capture the GraphQL Field and therefore is able to include it.


[Bug] Error in parsing time with time zone from postgresql to time.Time (Related to #225)

I am having exactly the same issue as presented at https://github.com/uptrace/bun/issues/225
I know this should been fixed, but for some reason I am having the same error
	sqldb := sql.OpenDB(pgdriver.NewConnector(pgdriver.WithDSN(dsn)))
	db := bun.NewDB(sqldb, pgdialect.New())

	...
	type VerificationCode struct {
	bun.BaseModel `bun:"table:verification_codes"`

	Destinatary string    `bun:",pk" json:"destinatary"`
	Method      string    `json:"method"`
	Expiration  time.Time `json:"expiration"`
	Hash        []byte
        }

        ...
       vc.Expiration = time.Now()
       _, err = db.NewInsert().Model(&vc).Exec(context.Background())

	...

        existingVC := new(models.VerificationCode)
	err := db.NewSelect().Model(existingVC).Where("destinatary = ?", request.Destinatary).Scan(context.Background())

Error:
parsing time "\xff\xff\xff\x00-12-31 17:23:24-06:36:36 BC": extra text: "-06:36:36 BC"

I'm having the same issue. The column type is timestamptz.

Please provide the program that reproduces the problem.

Apologies, I was unable to reproduce this issue outside my code base.
So I will close this and reopen if I manage to isolate it


[Bug]: The empty space is clickable when the user edits the App name or the organisation name

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
It is observed that form the homepage when the user tries to edit the application name or the organization name by clicking on the pencil icon and after editing the name when the user clicks on the empty space(where the pencil icon was displayed initially) the changes gets saved.
Expected result:
Clicking on pencil icon should show save icon and cancel icon so that user may click on save icon to make the changes or cancel icon to un-save changes
Actual result:
Currently empty space is displayed and which is clickable

Steps To Reproduce
1.Launch appmsith app in release or prod environment
2.From the homepage click on the three dots from any existing app or the organization name
3.click on the pencil icon
4.Edit the name and click on the empty space - where the pencil icon was displayed initially and observe
Environment
Production
Version
Cloud

Hi @YogeshJayaseelan, @Nikhil-Nandagopal, @riteshkew
I would like to work on this issue, I am planning to add tick icon for save and cross icon for cancel. Is it fine or do you give any suggestions or refer any icons or buttons i could use in place of save and cancel.

@Sai6347 This does not seem like an icon issue but more of how the \div containing the icon was defined.

@Sai6347 This does not seem like an icon issue but more of how the \div containing the icon was defined. Please go ahead if you are planning to look into this issue. Do reference the PR to this issue for better tracking.

@riteshkew ok thanks for confirmation, i will work on it.

Hi @riteshkew, can you please review this PR or assign someone to review it.


Invalid type for FormApplication.object

I guess this error affects every class that inherits from FormApplication.
Currently, FormApplication is typed so (pseudo-code):
class FormApplication<Data> {
  constructor(object?: O);
  object: O;
}

while the real code is so (pseude-code):
class FormApplication {
  constructor(object = {}) {
    this.object = object.
  }
}

So there are 3 options:

Let the user deal with this and keep the current state. This would lead to runtime errors: new UserConfig().render(true);.
Adding at least {} as type to FormApplication.object. This would lead to many type assertions being necessary.
Mark the first attribute as required. This would make the following use case unusable (which I do not know why anyone should do it, but who knows).

const config = new UserConfig();
config.user = game.user;

I would make it required. For some reason, it seems to be common in foundry that parameters have default values, even if it doesn’t make any sense to not provide the parameter


Add 2 new Armors

#Add 2 new types of armour
PREM KUMAR S
PES2UG21CS397

if u still want to work on the issue, kindly sync your and redo the changes so that there are no merge conflicts


This FAQ was a bit miss-leading

Have updated as to what I believe it should be, as it refers to the - within the enrollment practice only.

@marckean : Thanks for your contribution! The author, @genlin, has been notified to review your proposed change.

@marckean，Thanks for submitting the pull request.  This article is for the subscriptions types such as pay as you go. So I think we do not have to mention "within an enrollment".  Let me know if there is any question.


Build is failing on macOS (FLEX_INCLUDE_DIR is set to NOTFOUND)

mac os, error when typing make -j2. It gives me the following and then eventually a "make*** [all] error 2"
CMake Error at /Users/x/Downloads/retdec-master 3/build/external/src/yaramod-project-stamp/yaramod-project-configure-Release.cmake:16 (message):
Command failed: 1
'/usr/local/Cellar/cmake/3.12.1/bin/cmake' '-DCMAKE_BUILD_TYPE=Release' '-DCMAKE_C_COMPILER=/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/cc' '-DCMAKE_CXX_COMPILER=/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/c++' '-GUnix Makefiles' '/Users/x/Downloads/retdec-master 3/build/external/src/yaramod-project'

Could you attach the file /Users/x/Downloads/retdec-master 3/build/external/src/yaramod-project-stamp/yaramod-project-configure-Release.cmake?

And also the following two log files please:

/Users/x/Downloads/retdec-master 3/build/external/src/yaramod-project-stamp/yaramod-project-configure-err.log
/Users/x/Downloads/retdec-master 3/build/external/src/yaramod-project-stamp/yaramod-project-configure-out.log

how do I attach the file
/Users/x/Downloads/retdec-master 3/build/external/src/yaramod-project-stamp/yaramod-project-configure-Release.cmake?

here is the yaramod-project-configure-release:
set(command "/usr/local/Cellar/cmake/3.12.1/bin/cmake;-DCMAKE_BUILD_TYPE=Release;-DCMAKE_C_COMPILER=/Applications/Xcode.app/Co$
execute_process(
COMMAND ${command}
RESULT_VARIABLE result
OUTPUT_FILE "/Users/x/Downloads/retdec-master 3/build/external/src/yaramod-project-stamp/yaramod-project-configure-out.lo$
ERROR_FILE "/Users/x/Downloads/retdec-master 3/build/external/src/yaramod-project-stamp/yaramod-project-configure-err.log"
)
if(result)
set(msg "Command failed: ${result}\n")
foreach(arg IN LISTS command)
set(msg "${msg} '${arg}'")
endforeach()
set(msg "${msg}\nSee also\n  /Users/x/Downloads/retdec-master 3/build/external/src/yaramod-project-stamp/yaramod-project-$
message(FATAL_ERROR "${msg}")
else()
set(msg "yaramod-project configure command succeeded.  See also /Users/x/Downloads/retdec-master 3/build/external/src/yar$
message(STATUS "${msg}")

Have you installed Flex and Bison via Homebrew and set the following variables before running cmake, as requested in our build instructions for macOS?
# Apple ships old Flex & Bison, so Homebrew versions should be used.
export CMAKE_INCLUDE_PATH="/usr/local/opt/flex/include"
export CMAKE_LIBRARY_PATH="/usr/local/opt/flex/lib;/usr/local/opt/bison/lib"
export PATH="/usr/local/opt/flex/bin:/usr/local/opt/bison/bin:$PATH"

@massive44939 Based on this comment, the above solution should fix the problem. Please verify. If this solution fixes the problem for you, you can close the issue.

I am closing this issue as we have not received any feedback in the last three weeks. Feel free to reopen it if the above solution does not work for you.


Fix openvino main export

Fix main_export for decoder models (which were exported twice  : with /without past) as well as the resulting model file name which where wrongfully set for single models (SD and encoder/decoder models were not concerned).  Also now the different OVModelForXxx classes all use main_export to export the model when needed

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Thanks Ella!  I noticed one issue with CausalLM export: when loading the model, I got the error "use_cachewas set toTruebut the loaded model only supportsuse_cache=False". During export I saw Overriding 2 configuration item(s) - use_cache -> True but it appears the model is exported without cache.

Thanks Ella! I noticed one issue with CausalLM export: when loading the model, I got the error "use_cachewas set toTruebut the loaded model only supportsuse_cache=False". During export I saw Overriding 2 configuration item(s) - use_cache -> True but it appears the model is exported without cache.

Thanks a lot for reporting it @helena-intel, could you share a code snippet for me to reproduce this ?

cc @eaidova

Thanks a lot for reporting it @helena-intel, could you share a code snippet for me to reproduce this ?

When I tried to reproduce it it did not occur, so I suppose it was a strange thing in my environment. Sorry for the false alarm!

When I tried to reproduce it it did not occur, so I suppose it was a strange thing in my environment. Sorry for the false alarm!

Thanks a lot for checking @helena-intel


README.md shows comments it shouldn't

e.g.
<!-- [START logging_install_with_bom] -->

Fixed in #88


Feature request: nginx request_id

Since Nginx version 1.11 $unique_id variable is included by default.
I think it is useful when you want to share your unique id between your reverse proxy and your server and you can't send modsecurity's unique id in the headers because it's not available as a nginx variable.
I managed to patch my ngx_http_modsecurity.c ngx_http_modsecurity_create_ctx  changing the random salted string generated with this:
    ngx_str_t variable_name = ngx_string("request_id");
    ngx_int_t hash = ngx_hash_key (variable_name.data, variable_name.len);
    ngx_http_variable_value_t * uid = ngx_http_get_variable(r, &variable_name, hash);

I guess it would be a great feature.
Thank you

Hi @asdfuken
You're more than welcome to submit a pull request for features to get evaluated by the community. Maybe we could think about adding this as a configuration or compile time (ifdef) directive.
But I strongly encourage suggesting/adding new features to libModSecurity as although support for 2.x is still ongoing, 3.x is the main development focus now.

This feature is now available in libModSecurity: https://github.com/SpiderLabs/ModSecurity/commit/4dd28127577eca0bf854db5c134a4134db6ff1ec


Export data

Hi, thank you for the effort!
I think sushi is a great minimal finance app.
Can you implement a way to export data via csv file for further personal analytics?

Hey @zeratosk, this is a feature i was planning to implement as well. But I might need the calendar filters feature first before this one is implemented.

It would also be cool to have a backup/restore feature for when someone changes his phone.
Also thank you for the app ;)

This has been implemented, but still needs some polishing. Feel free to open new issues if you found anything.


Add call home to Artifactory

Call home request does not contain any data from the users. It sends only the product ID (e.g. vault-plugin-secrets-artifactory/0.2.18) and the Vault actions (e.g. pathConfigRotateUpdate) to Artifactory call home API endpoint.
Also fixes #90

I would strongly recommend making this able to be opt-out, many secure environments won't have the ability to reach random sites on the Internet. Additionally, some people may have privacy concerns with any sort of "callHome" regardless of  what data is included.


chrony: 3.2 -> 3.3

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools.
This update was made based on information from https://repology.org/metapackage/chrony/versions.
These checks were done:

built on NixOS
ran ‘/nix/store/d5v3k2h8rdhxzyg4my66nrr0zhdhzvaw-chrony-3.3/bin/chronyc --help’ got 0 exit code
ran ‘/nix/store/d5v3k2h8rdhxzyg4my66nrr0zhdhzvaw-chrony-3.3/bin/chronyc help’ got 0 exit code
ran ‘/nix/store/d5v3k2h8rdhxzyg4my66nrr0zhdhzvaw-chrony-3.3/bin/chronyd -h’ got 0 exit code
ran ‘/nix/store/d5v3k2h8rdhxzyg4my66nrr0zhdhzvaw-chrony-3.3/bin/chronyd --help’ got 0 exit code
found 3.3 with grep in /nix/store/d5v3k2h8rdhxzyg4my66nrr0zhdhzvaw-chrony-3.3
directory tree listing: https://gist.github.com/60dede830d6efc2bf29be2b4983e4c97

cc @rickynils @fpletz for review

Success on x86_64-linux (full log)
Attempted: chrony
Partial log (click to expand)
shrinking RPATHs of ELF executables and libraries in /nix/store/d5v3k2h8rdhxzyg4my66nrr0zhdhzvaw-chrony-3.3
shrinking /nix/store/d5v3k2h8rdhxzyg4my66nrr0zhdhzvaw-chrony-3.3/bin/chronyc
shrinking /nix/store/d5v3k2h8rdhxzyg4my66nrr0zhdhzvaw-chrony-3.3/sbin/chronyd
gzipping man pages under /nix/store/d5v3k2h8rdhxzyg4my66nrr0zhdhzvaw-chrony-3.3/share/man/
strip is /nix/store/fzcs0fn6bb04m82frhlb78nc03ny3w55-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/d5v3k2h8rdhxzyg4my66nrr0zhdhzvaw-chrony-3.3/bin  /nix/store/d5v3k2h8rdhxzyg4my66nrr0zhdhzvaw-chrony-3.3/sbin
patching script interpreter paths in /nix/store/d5v3k2h8rdhxzyg4my66nrr0zhdhzvaw-chrony-3.3
checking for references to /tmp/nix-build-chrony-3.3.drv-0 in /nix/store/d5v3k2h8rdhxzyg4my66nrr0zhdhzvaw-chrony-3.3...
moving /nix/store/d5v3k2h8rdhxzyg4my66nrr0zhdhzvaw-chrony-3.3/sbin/* to /nix/store/d5v3k2h8rdhxzyg4my66nrr0zhdhzvaw-chrony-3.3/bin
/nix/store/d5v3k2h8rdhxzyg4my66nrr0zhdhzvaw-chrony-3.3

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: chrony
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowBroken = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowBroken = true; }
to ~/.config/nixpkgs/config.nix.

Success on aarch64-linux (full log)
Attempted: chrony
Partial log (click to expand)
shrinking RPATHs of ELF executables and libraries in /nix/store/5qg02clkbwapjibprhqgdfvbjixa89z3-chrony-3.3
shrinking /nix/store/5qg02clkbwapjibprhqgdfvbjixa89z3-chrony-3.3/bin/chronyc
shrinking /nix/store/5qg02clkbwapjibprhqgdfvbjixa89z3-chrony-3.3/sbin/chronyd
gzipping man pages under /nix/store/5qg02clkbwapjibprhqgdfvbjixa89z3-chrony-3.3/share/man/
strip is /nix/store/3zq400fri5dv7d30lpxlqm2v9y1iis6j-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/5qg02clkbwapjibprhqgdfvbjixa89z3-chrony-3.3/bin  /nix/store/5qg02clkbwapjibprhqgdfvbjixa89z3-chrony-3.3/sbin
patching script interpreter paths in /nix/store/5qg02clkbwapjibprhqgdfvbjixa89z3-chrony-3.3
checking for references to /build in /nix/store/5qg02clkbwapjibprhqgdfvbjixa89z3-chrony-3.3...
moving /nix/store/5qg02clkbwapjibprhqgdfvbjixa89z3-chrony-3.3/sbin/* to /nix/store/5qg02clkbwapjibprhqgdfvbjixa89z3-chrony-3.3/bin
/nix/store/5qg02clkbwapjibprhqgdfvbjixa89z3-chrony-3.3


feat: CU-86934hnmg rest api skipping fields

Ticket
https://github.com/VirtusLab-Open-Source/strapi-plugin-comments/issues/239
Summary
What does this PR do/solve?

fields of comment entity can be skipped

Test Plan

start the server
make a API call like http://localhost:1337/api/comments/<id>?omit[]=blocked&omit[]=related&omit[]=author
response should not contain specified fields

Task linked: CU-86934hnmg Make related not mandatory population


Explore Document Level Validation for Authors

Explore ways to implement validation for fields added in Word document.

Not in the requriements as of now, closing this one.


Save Button

When making a new class roster, clicking "save" is not clear that it is saving. You can't see the confirmation message. Can the save button just save and close the editor?

This has been fixed by making the confirmation modal a larger window containing more specific information, including whether the student emails submitted were valid or not.


QuadraticMinimizer bug fixes

After experimentation inside mllib ALS I found out two bugs

QuadraticMinimizer Quasi Definite System was not reinitializing correctly:
[H A']
[A 0]
I did not re-initialize 0 correctly and so when state was re-used results were incorrect. This issue will only show up in EQUALITY formulation. Added a test-case to guard against it.

QuadraticMinimizer used to return the primal variable (x) but changed it to return dual variable (z) which gives a projection style result. The algorithm converges only if x and z are within absTol.
Basically in place of getting numbers like 1e-5 for POSITIVE constraint, now we will get 0.0

Improved convergence of ProbabilitySimplex using L1 rho formula

@dlwh give it one week or so...I have started testing the convergence and runtime of NonlinearMinimizer as well inside Spark through Alternating Minimization (Matrix factorization for large ranks and loss functions like loglikelihood and kl divergence)...next week I will find out more convergence related issues from NonlinearMinimizer/SPG based flow as well...

@dlwh you can merge it...I did not get time to test NonlinearMinimizer variants last few weeks...


xcmPallet.limitedTeleportAssets method not supported

I would like to accept RMRK tokens on Statemine but cannot because it is not possible to 'teleport' KSM over to initialize the Statemine account. Can devs do something please?

Hi! We released a new version of the app v15.9310.0 that also bundles that methods from XcmPallet.
Due to hardware limitations in the Ledger Nano S, it's only going to be available for Nano S+ and Nano X.
Thanks for the report!


Having trouble entering API keys

How do you enter the API key?

Via preferences → plugins and choose live-code-runner. :)


Is this milter used in production

Hello Kitani, i was browsing this project, and was wondering if you are currently using this milter in production.  It looks very interesting.

Hello amit777, i'm using this milter in production.

Excellent. Could you provide any details on the number of emails per day or the type of email traffic being processed?

I'm sorry. For the part of the system that are provided to my customer, I can't provide any details.


two bugs: deletedAt do not respect nowSQL and .asDate shows type error

bug 1 is in this repo: https://github.com/bingtsingw/orchid-reproduction

How to reproduce bug 2:

go to  src/utils/base-table.ts
replace all parse((v: any): Date => (v ? new Date(v) : v)) with asDate()
go back to test file.

You'll see .create() need to pass createdAt, updatedAt and deletedAt, which nullable() and default do not work

Ready, I publised a fix: orchid-orm 1.34.2
Both problems are fixed, asDate() should now be working fine


Incorrect sort for age breakdown on small multiples view

To replicate, view uninsured people by age, then view breakdown by all groups. It's using alpha sort.

Marking this as closed as this is included as part of issue #592


How I can change input mode of code editor?

I want to change input mode of code editor from replace to insert.

For future reference, this is done by pressing the Insert key.


generalization of docker-compose file and .env file

docker-compose and env files generalized for more convenience.
I personally prefer to modify the .env files of the deployment recipe rather than hard-coding the version or any other stuff in the docker-compose.yml itself.

Thank you for your PR 👍
This is true for templates that are used as templates ( such as the Websoft9 repository, which you also got your idea from in your PR )
My intention with this repository is to be able to set up a fast and simple cluster that only versions, ports, and login information need to be configured. If you look at the rest of my repositories, there are categories for Docker and Kubernetes, each with different programs for the user to easily deploy.


'target' option is ignored if args contains --

I'm using maturin-action in the following way:
- uses<EMAIL_ADDRESS>  with:
    maturin-version: 'v0.13.7'
    target: x86
    command: publish
    args: "-- -Ctarget-feature=+crt-static"

My build fails with the output:
  --- stderr
  error: your Rust target architecture (64-bit) does not match your python interpreter (32-bit)
warning: build failed, waiting for other jobs to finish...
💥 maturin failed
  Caused by: Failed to build a native library through cargo
  Caused by: Cargo build finished with "exit code: 101": `"cargo" "rustc" "--release" "--message-format" "json" "--lib" "--crate-type" "cdylib" "--" "-Ctarget-feature=+crt-static" "--target" "i686-pc-windows-msvc"`

I think the reason is that maturin is being called in this way:
maturin.exe publish -- -Ctarget-feature=+crt-static --target i686-pc-windows-msvc
...

The --target option is ignored because it's specified after all the arguments that may contain -- delimiter. When I changed my config to this one:
- uses<EMAIL_ADDRESS>  with:
    maturin-version: 'v0.13.7'
    command: publish
    args: "--target i686-pc-windows-msvc -- -Ctarget-feature=+crt-static"

I've got what I expected:
maturin.exe publish --target i686-pc-windows-msvc -- -Ctarget-feature=+crt-static
...

Interesting, thanks for the report! I think change from Array.push to Array.unshift should fix it: https://github.com/PyO3/maturin-action/pull/140.

Please try v1.35.2.

It works like a charm! Thank you for such a great support.


Error when constructing "sealed" (by private constructor) class that matches generic type

TypeScript Version:  3.2.0-dev.201xxxxx

Search Terms:
generic, narrowing, Conversion of type <> to type <> may be a mistake because neither type sufficiently overlaps with the other. If this was intentional, convert the expression to 'unknown' first, sealed, final
Code
// A *self-contained* demonstration of the problem follows...
// Test this by running `tsc` on the command-line, rather than through another build tool such as Gulp, Webpack, etc.

export abstract class Foo { abstract doSomething(): number; }

export class Bar extends Foo {

    private constructor() {
        super();
    }

    public static factory() {
        return new Bar();
    }

    doSomething(): number{
        return 0;
    }

    somethingElse() {
        return '';
    }
}

// should work because the type T is gauranteed to be Bar and not a sub class of Bar
function a<T extends Foo>(thing: T): T {
    if (thing instanceof Bar) {
        return Bar.factory();
    }
    return thing;
}

// should work because the type T is gauranteed to be Bar and not a sub class of Bar
function b<T extends Foo>(thing: T): T {
    if (thing instanceof Bar) {
        return Bar.factory() as T;
    }
    return thing;
}

Expected behavior:
Since you can make it impossible for a class to be extended you can gaurantee that a class "sufficiently overlaps" with a generic type. This code should compile.
Actual behavior:
The code does not compile.
Playground Link:
http://www.typescriptlang.org/play/#src=export abstract class Foo { abstract doSomething()%3A number%3B }

export class Bar extends Foo {

    private constructor() {
        super()%3B
    }

    public static factory() {
        return new Bar()%3B
    }

    doSomething()%3A number{
        return 0%3B
    }

    somethingElse() {
        return ''%3B%0D%0A%20%20%20%20%7D%0D%0A%7D%0D%0A%0D%0A%2F%2F%20should%20work%20because%20the%20type%20T%20is%20gauranteed%20to%20be%20Bar%20and%20not%20a%20sub%20class%20of%20Bar%0D%0Afunction%20a%3CT%20extends%20Foo%3E(thing%3A%20T)%3A%20T%20%7B%0D%0A%20%20%20%20if%20(thing%20instanceof%20Bar)%20%7B%0D%0A%20%20%20%20%20%20%20%20return%20Bar.factory()%3B%0D%0A%20%20%20%20%7D%0D%0A%20%20%20%20return%20thing%3B%0D%0A%7D%0D%0A%0D%0A%2F%2F%20should%20work%20because%20the%20type%20T%20is%20gauranteed%20to%20be%20Bar%20and%20not%20a%20sub%20class%20of%20Bar%0D%0Afunction%20b%3CT%20extends%20Foo%3E(thing%3A%20T)%3A%20T%20%7B%0D%0A%20%20%20%20if%20(thing%20instanceof%20Bar)%20%7B%0D%0A%20%20%20%20%20%20%20%20return%20Bar.factory()%20as%20T%3B%0D%0A%20%20%20%20%7D%0D%0A%20%20%20%20return%20thing%3B%0D%0A%7D
Related Issues:
https://github.com/Microsoft/TypeScript/issues/8306
https://github.com/Microsoft/TypeScript/issues/8306#issuecomment-378434818
https://github.com/Microsoft/TypeScript/issues/28819

This is working as intended. We don't have the concept of "sealed" or "final" types (that discussion is already taking place in #8306), and we never consider a zero order type (such as Bar in your example) to be assignable to a higher order type (such as T in your example).

Can I change this to a feature request then?


Implement google analytics.

Reopen to implement with react. Dont know why it is closed anyways..

this is done by cloudflare outomatically


feature: auto formatting

In response to #33
Summary:

Added elkjs to create a directed graph using layered ELK algorithm
Increased zoom out to 0.05 to be able to better locate & navigate large, dispersed graph.
Added control icon to disperse the nodes

Points for improvement:

the layout is hard set - if we move a node it'll get moved back once we change the schema. It would be great to keep the position set by the user and use the ELK compiled graph as a base but it would probably require reworking of how the nodes are stored - not sure how yet

Took me long enough to open this PR 😅   Let me know what you think about this feature!

Thanks for this, sorry it took so long to get around to checking it.
I have a few ideas around keeping the user's placement around on top of the dispersement, but needs locations in general to be stored, so will be pretty big.


OpenCVSharp Version 4 depends on mfplat.dll

Summary of your issue
When deploying our application that was upgraded to latest OpenCVSharp all worked fine in desktop environment, but when I tried to use OpenCVSharp 4.0.0 on Win 2012 R2 environment I got:

System.TypeInitializationException : The type initializer for 'OpenCvSharp.NativeMethods' threw an exception.
---- System.DllNotFoundException : Unable to load DLL 'OpenCvSharpExtern' or one of its dependencies: The specified module could not be found. (Exception from HRESULT: 0x8007007E)
at OpenCvSharp.NativeMethods.core_Mat_new2(Int32 rows, Int32 cols, Int32 type)
at OpenCvSharp.Mat..ctor(Size size, MatType type) in C:\projects\opencvsharp\src\OpenCvSharp\Modules\core\Mat\Mat.cs:line 121

After some digging and running Procmon, I noticed that the code tries to load  C:\Windows\System32\mfplat.dll
I installed Windows Server Essentials Media Pack (https://www.microsoft.com/en-us/download/details.aspx?id=40837), and got the dependency satisfied, but I believe the code should run in server environment since no device interaction is performed.
Environment
Windows Server 2012 R2
What did you do when you faced the problem?
Just run any code referencing OpenCVSharp on a server without mfplat.dll available.

It actually seems that OpenCVSharp 3.4.4 has this issue as well, when 3.4.1 does not.

It may be because I built OpenCV with WITH_MSMF flag enabled. I will reconsider the flag in development of 4.0.1 or later. https://github.com/opencv/opencv/blob/7fb70e170154d064ef12d8fec61c0ae70812ce3d/CMakeLists.txt#L344

Thanks, when can we expect 4.0.1 release?

Thank you for waiting https://www.nuget.org/packages/OpenCvSharp4/<IP_ADDRESS>90326

I still get the mfplat.dll error even with the latest version on nuget

Hi, could it be that the issue is back? I now test it with 4.1.1 and see it again...

Hi, I see the dependency upon mfplat.dll is back again on newer versions of the library (4.4, 4.5). Is this intentional?


Add SwinV2 in TorchVision

🚀 The feature
Swin Transforms were added to TorchVision on #5491 thanks to the work of @xiaohu2015 and @jdsgomes. At that point, the official code of SwinV2 was not released so we decided to postpone its addition. Later @YosuaMichael has refactored the class to make it compatible with 3d use-cases at #6088. With the code of V2 now available, we should consider adding it to TorchVision.
Here is what needs to be done:

Examine if it's possible to extend the existing Swin Transformer class to support both V1 and V2 models. An example of such an extension was recently done for EfficientNets. On the other hand, if the implementations are substantially different, we can follow the approach we took for MobileNets v2 and v3 and use separate classes.
Provide an implementation that uses TorchVision building blocks and idioms. See #5319 for pointers.
Provide proof in form of pre-trained weights for the smallest variant that reproduces the accuracy of the paper.

Motivation, pitch
We should keep TorchVision fresh by offering the latest influential architectures to support the work of ML researchers and practitioners.
Alternatives
No response
Additional context
No response
cc @datumbox

@ain-soph thanks for the PR! Please comment on this ticket to be able to assign it to you. Github won't let me do it. :(

Sure!


fix missing always block inside post

Signed-off-by: Tianle Huang<EMAIL_ADDRESS>Description
During testing of the Jenkins job, I find this following error.
Started by user [opensearch](http://jenki-jenki-fpgmrv2ryxko-1366042710.us-east-1.elb.amazonaws.com/user/admin)
Rebuilds build #[17](http://jenki-jenki-fpgmrv2ryxko-1366042710.us-east-1.elb.amazonaws.com/job/Playground/job/tianleh-test/job/tianleh-os-yum-4-12/17)
Obtained jenkins/opensearch/distribution-build.jenkinsfile from git https://github.com/tianleh/opensearch-build.git
Running in Durability level: MAX_SURVIVABILITY
org.codehaus.groovy.control.MultipleCompilationErrorsException: startup failed:
WorkflowScript: 397: Expected a symbol @ line 397, column 29.
                               lib.jenkins.Messages.new(this).add(
                               ^

WorkflowScript: 396: Invalid condition "script" - valid conditions are [always, changed, fixed, regression, aborted, success, unsuccessful, unstable, failure, notBuilt, cleanup] @ line 396, column 25.
                           script {
                           ^

WorkflowScript: 397: Arguments to "error" must be explicitly named. @ line 397, column 29.
                               lib.jenkins.Messages.new(this).add(
                               ^

3 errors

	at org.codehaus.groovy.control.ErrorCollector.failIfErrors(ErrorCollector.java:310)
	at org.codehaus.groovy.control.CompilationUnit.applyToPrimaryClassNodes(CompilationUnit.java:1085)
	at org.codehaus.groovy.control.CompilationUnit.doPhaseOperation(CompilationUnit.java:603)
	at org.codehaus.groovy.control.CompilationUnit.processPhaseOperations(CompilationUnit.java:581)
	at org.codehaus.groovy.control.CompilationUnit.compile(CompilationUnit.java:558)
	at groovy.lang.GroovyClassLoader.doParseClass(GroovyClassLoader.java:298)
	at groovy.lang.GroovyClassLoader.parseClass(GroovyClassLoader.java:268)
	at groovy.lang.GroovyShell.parseClass(GroovyShell.java:688)
	at groovy.lang.GroovyShell.parse(GroovyShell.java:700)
	at org.jenkinsci.plugins.workflow.cps.CpsGroovyShell.doParse(CpsGroovyShell.java:142)
	at org.jenkinsci.plugins.workflow.cps.CpsGroovyShell.reparse(CpsGroovyShell.java:127)
	at org.jenkinsci.plugins.workflow.cps.CpsFlowExecution.parseScript(CpsFlowExecution.java:571)
	at org.jenkinsci.plugins.workflow.cps.CpsFlowExecution.start(CpsFlowExecution.java:523)
	at org.jenkinsci.plugins.workflow.job.WorkflowRun.run(WorkflowRun.java:337)
	at hudson.model.ResourceController.execute(ResourceController.java:97)
	at hudson.model.Executor.run(Executor.java:429)
Finished: FAILURE

Issues Resolved
Check List

[x] Commits are signed per the DCO using --signoff

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

Codecov Report

Merging #2029 (60cf5a8) into main (7012178) will increase coverage by 0.33%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main    #2029      +/-   ##
============================================
+ Coverage     94.17%   94.50%   +0.33%
============================================
  Files           195      152      -43
  Lines          3860     3732     -128
  Branches         29       19      -10
============================================
- Hits           3635     3527     -108
+ Misses          219      205      -14
+ Partials          6        0       -6

Impacted Files
Coverage Δ

src/system/temporary_directory.py
83.87% <0.00%> (-12.91%)
:arrow_down:

src/system/os.py
93.75% <0.00%> (-6.25%)
:arrow_down:

src/test_workflow/perf_test/perf_test_cluster.py
80.00% <0.00%> (-2.86%)
:arrow_down:

...sts/jenkins/jobs/DetectTestDockerAgent_Jenkinsfile

tests/jenkins/jobs/RunIntegTestScript_Jenkinsfile

...ests/jenkins/jobs/AssembleManifest_rpm_Jenkinsfile

tests/jenkins/jobs/BuildAssembleUpload_Jenkinsfile

tests/jenkins/jobs/PromoteArtifacts_Jenkinsfile

tests/jenkins/jobs/BuildDockerImage_Jenkinsfile

tests/jenkins/jobs/UploadTestResults_Jenkinsfile

... and 44 more

Continue to review full report at Codecov.

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


Search for components: Searching with a text that doesn't match any results breaks the site

Steps to reproduce:

Go to https://sveltesociety.dev/components/
In the 'Search for components...' field, type a text that doesn't match any results (e.g. 'hello')
The site stops working at this point

the filtration of components stops working
links to other pages like Tools stop working
following error is reported in the console logs

Uncaught (in promise) TypeError: Cannot read properties of undefined (reading 'length')     at Object.p (sort-582dd570.js:3:2453)     at Sl (vendor-d78fc752.js:4:1769)     at Hn (vendor-d78fc752.js:4:1438)

Same problem here: https://sveltesociety.dev/tools/
Preparing a fix.


Making dependencies optional for easier OSGi deployment

Marking javax.cache api and reactor-stream dependencies as optional in pom makes it easier to deploy redisson in an OSGi environment when not using these modules. This change is in continuation to #793

Done


Store build dependencies separately

Right now all dependencies are stored together. We could split out build dependencies and store them separately.
A common case where this distinction would be useful is RUSTSEC-2018-0006: this is used by the current version of clap as a build dependency and poses no security risk in that context; however, its uses as a runtime dependency would be problematic.
We need to keep information about build dependencies because a bug in a code generator such as protobuf or cap'n proto only included as a build dependency may still pose a security risk at runtime.

Implemented in f80c2a756f5dcf5f3be8be340428275805636337


[Discuss] Null handling

Recently there have been quite a few PRs and issues opened relating to null/empty string handling.
I have gathered links to these PRs/issues here, if we would like to have a wider discussion on this topic.
#2715 - stringFormat extractionFn should be able to return null on null values (Fix for #2706) #2715
#2737 - Fix predicate-based ValueMatcher behavior for IncrementalIndex on missing columns.
#2758 - handle null values in In Filter
#2765 - Null string is encoded as "null" in incremental index
#2771 - Various ExtractionFn null handling fixes.
#2772 - DimensionPredicateFilter and JavaScriptFilter don't handle null columns properly
#2776 - Inconsistent null/empty str handling for multi-value/single-value dimensions in QueryableIndexStorageAdapter

Duplicate of #4349


After Authenticating Patient Number Please wait progress bar appear and It Never show the appointment and hospital activities!

After Phone number is authenticated.....Progress bar keeps loading and no details is displayed.

After Phone number is authenticated.....Progress bar keeps loading and no details is displayed.

can u find any thing about it?


Feature/expose cli tool as prop

Update test project so tests get run on Jenkins.
Expose the obfuscar CLI tool via MSBuild props so they can easily be used in consuming projects.

It seems the tests are flaky. I don't think whoever wrote them considered they are run in parallel so sometimes more than one process will try to access the same file. We could probably fix this in CI/CD but I wonder if this is the correct approach?