docs: add spell checker info to contrib guide

Summary
Adds information about using the spell checker to the Contributing Guide
Fixes https://github.com/keptn/lifecycle-toolkit/issues/2462
Checks

[ ] My PR fulfills the Definition of Done of the corresponding issue and not more (or parts if the issue is separated
into multiple PRs)
[ ] I used descriptive commit messages to help reviewers understand my thought process
[ ] I signed off all my commits according to the Developer Certificate of Origin (DCO)(
see Contribution Guide)
[ ] My PR title is formatted according to the semantic PR conventions described in
the Contribution Guide
[ ] My content follows the style guidelines of this project (YAMLLint, markdown-lint)
[ ] I regenerated the auto-generated docs for Helm and the CRD documentation (if applicable)
[ ] I have performed a self-review of my content including grammar and typo errors and also checked the rendered page
from the Netlify deploy preview
[ ] My changes result in all-green PR checks (first-time contributors need to ask a maintainer to approve their test runs)

This PR has problems.  It is replaced by https://github.com/keptn/lifecycle-toolkit/pull/2504 .


[Breaking] Remove the statsd metrics, dependency no more available

To be continued:

https://mvnrepository.com/artifact/com.readytalk/metrics3-statsd/4.2.0
https://github.com/mapfish/mapfish-print/pull/379
https://github.com/prometheus/jmx_exporter/blob/master/README.md
https://medium.com/logistimo-engineering-blog/tomcat-jvm-metrics-monitoring-using-prometheus-in-kubernetes-c313075af727
https://mvnrepository.com/artifact/io.prometheus.jmx/jmx_prometheus_javaagent

Good catch @pmauduit thanks, fixed :-)

@pmauduit fixed

Looks almost good to me, but the class is still referenced here and you should end up with a ClassNotFoundException / NoClassDefFoundError if you don't remove the following line:
https://github.com/mapfish/mapfish-print/blob/befa4bbf53e646d457fa85b127cf598baa5fc8d2/core/src/main/resources/mapfish-spring-application-context.xml#L69

@pmauduit this is already removed, you point an old commit...

@pmauduit up...

@pmauduit up...


Optimization of performance

#170
#73

Report With 485 lines of code.
Mode: Debug
Analysis of the single KeyPressEvent

KeyPressEvent 128.64ms

Whole setValue 119.17ms

CodeController methods 32.94ms

Parse highlight 12.92ms + 1.15ms + 4.24ms = 18.31ms
Process single line comments 2.69ms
Text to code lines 1.09ms
Comments to hidden ranges 0.62ms
cutHighlighted 1.32ms
split lines 2.54ms
folded as (with no folded blocks) 5.85ms

super.setValue 46.42ms

layout 42.29ms
textPainter 39.16ms

setText 38.54ms

Autocompleter.autocomplete 38.54ms

enterList 29.66ms + 6.47ms = 36.13ms

Animation frame fired (paint layout basically) 509.50ms

If there is only 1 TextSpan to paint:

This PR is a complete mess created for testing.

You can check the test results in #170


add id requirements for KCS social

/lgtm

@salaxander I want to edit the heck out of this.  Would you rather I do this on your fork, or do you want me to approve this and then submit a new PR overhauling it?

/approve
merging so that I can do my edits as well


AMP Support

Hello,
i have a suggestions for the Core.
There is new feature called "AMP" (Accelerated Mobile Pages)
Further Details at: https://www.ampproject.org/
It would be a great thing if the core would support that! This would be a real SEO Boost!

Each page have a AMP-version
The AMP-Version of the current page is linked in the META Tags
If an AMP Version is requested, the core have to render the page for AMP. This means:
3.1) Convert HTML to AMP (This can be done with this Lib https://github.com/Lullabot/amp-library)
3.2) Change of the asset system: Don't use scripts.
3.3)  Change of the asset system: Stylesheets have to be included inline within the style tag (no external css + max 50KB of inline css)
3.4) Each block could have a custom "amp_view.php". this file will be loaded when amp version is requested. Same for custom-templates (amp_prefix). this would allow to take use of amp libs e.g. image slideshow... So the core block "image Slider" would still work
3.5) Rewriting the URL's within the amp version

I have started making a PoC as a package. Don't have the time at the moment but maybe it's a good inspiration or starting point for someone else if you like the idea too :)
amp_theme.zip

This is something @KorvinSzanto is actively interested in as well – I would get in touch with him in Slack if you're interested, @bitterdev

@MrKarlDilkington This is great that other developers like @Hissy has worked on amp too. I think together we can make a great amp extension for the core.
@aembler Yes i will contact @KorvinSzanto on Slack.

My AMP package still works on v8 with some tiny fixes.
https://github.com/pictron/concrete5-amp/commit/9ae3aed0695c443c6309ad2c8948fd18e0e1166f
However, overriding PageController class is deprecated now. In my understanding, there's no alternative. That's why I can't continue developing my AMP package.
https://documentation.concrete5.org/developers/working-with-pages/swapping-out-the-base-controller

Swapping out page controller is deprecated because it's so hacky to swap out the underlying controller using page aliases – that's why we've decided it's a bad idea. What about your solution is required to swap out controllers, etc... that couldn't be done by simply switching to another page template in another theme?

I used swapped page controller for...

Add link element to amp version
Add amp segment to the path that displays same page but AMP theme

https://github.com/hissy/concrete5-amp/blob/master/src/Concrete/Page/Controller/PageController.php

@Hissy if you look at the initial post there is a code attachment. I did the same without swapping/overriding the page controller. It's not the most elegant way i know but it's another approach... With a core integration the page controller can be extended to implement missing functionality. But what else is missing in your solution to get a stable version for core integration ?
Here is the important code part of my initial post:
`public function on_start()
{
require $this->getPackagePath() . '/vendor/autoload.php';
    if ((strlen($_SERVER["REQUEST_URI"]) >= 5 && substr($_SERVER["REQUEST_URI"], 0, 5) === "/amp/") ||
        (strlen($_SERVER["REQUEST_URI"]) >= 15 && substr($_SERVER["REQUEST_URI"], 0, 15) === "/index.php/amp/")) {

        if (substr($_SERVER["REQUEST_URI"], 0, 5) === "/amp/") {
            $_SERVER["REQUEST_URI"] = substr($_SERVER["REQUEST_URI"], 4);
            $_SERVER["PATH_INFO"] = substr($_SERVER["REQUEST_URI"], 4);
        } else {
            $_SERVER["REQUEST_URI"] = substr($_SERVER["REQUEST_URI"], 14);
            $_SERVER["PATH_INFO"] = substr($_SERVER["REQUEST_URI"], 14);
        }

        if (version_compare(APP_VERSION, '8.0', '<')) {
            global $request;

            Request::setInstance(Request::createFromGlobals());

            $request = $request::getInstance();
        }

        Events::addListener('on_page_output', function ($event) {
            $html = $event->getArgument('contents');

            $amp = $this->app->make(AMP::class);

            $amp->loadHtml($html, ["scope" => Scope::HTML_SCOPE]);

            $ampHtml = $amp->convertToAmpHtml();

            $event->setArgument('contents', $ampHtml);
        });

        Events::addListener("on_header_required_ready", function($event) {
            $linkTags = $event->getArgument('linkTags');

            $url = UrlResolver::getNonAmpUrl();

            $linkTags["canonical"] = "<link rel=\"amphtml\" href=\"" . $url . "\">";

            $event->setArgument('linkTags', $linkTags);
        });

        Events::addListener("on_before_render", function($event) {
            $c = Page::getCurrentPage();

            foreach($c->getBlocks() as $block) {
                /** @var $block \Concrete\Core\Block\Block */

                // @todo: replace block with custom teplate
            }
        });

        $this->app->bind(CorePageUrlResolver::class, PageUrlResolver::class);
        $this->app->bind(CorePathUrlResolver::class, PathUrlResolver::class);

        Config::set('concrete.seo.redirect_to_canonical_url', false);

    } else {
        Events::addListener("on_header_required_ready", function($event) {
            $linkTags = $event->getArgument('linkTags');

            $url = UrlResolver::getAmpUrl();

            $linkTags["amphtml"] = "<link rel=\"amphtml\" href=\"" . $url . "\">";

            $event->setArgument('linkTags', $linkTags);
        });
    }
}

`
BTW: I take use of lullabot/amp. I really like the library for converting HTML to AMP.  What is your converter engine?
Question: In your solution only selected page types will be renderer in AMP. Wouldn't be better to support AMP for the whole page?

IMO these are not good approaches.

Overriding $_SERVER
Converting whole content by the library
Overriding a config value

I think only content block content should be converted to amp HTML from default HTML.
I use my own converter, but it can be switched with lullabot/amp library.
I think simple page type like blog post can have AMP version, but it is difficult to convert complex page type like HOME.

To 1) I agree with you but with a core integration it would not necessary to override $_SERVER super global or the page controller. These are only approaches for AMP integration by 3rd party package.
To 2) I think the best solution is to render all blocks in a page with a custom amp-template (if available). When i remember right your current code is already act like that ? So when all core blocks have a custom amp view it is not not relevant how complex a page type is. In addition: The converter converts only normal HTML. Already included AMP markup will be skipped. So only HTML from header / footer & layout and from blocks with no custom amp template will be converted.
To 3) Yes i know
Question: You have a Custom Theme for AMP. How you handle different areas? I mean the Original Non-Amp-Theme have different Areas.... Will only the "Main" Area converted or all areas?

I think we can't integrate AMP in the Core. It is still not a standard web technology. We can maintain an AMP package.
I think AMP version should be simple as possible, but we need more discussion.

Considering you can make responsive pages with c5, how is AMP preferable over the existing methodology? I don't see it.

I agree with @BloodyIron on this. Thanks for all the work on this but the writing is on the wall regarding AMP. Let's keep it in a separate package and focus on making the main Concrete experience as fast as possible.


PRISM returns 422 for routes with same parent route

Describe the bug
We use Prism Mock Server for development and mock server for running API contracts tests for our React application. With the recent update 3.2.8 we figured out some of the tests are failing.
As a practice, we removed node modules, did npm cache clean --force and restarted prism and ran tests.
To Reproduce
Here is the yaml code for simulating 422 issue
/api/users/{user_id}:
  get:
    summary: "Get details of a user by id"
    operationId: "dal.user.get"
    parameters:
      -
        name: "user_id"
        in: "path"
        description: "Id of User"
        required: true
        schema:
          $ref: "#/components/schemas/UserId"
    responses:
      200:
        description: "User retrieved successfully"
        content:
          application/json:
            schema:
              $ref: "#/components/schemas/UserEntity"

/api/users/locations:
  get:
    summary: "Get location info for the users. It returns a list of existing custom name."
    operationId: "dal.users.get_locations"
    responses:
      200:
        description: "Retrieve location info successfully"
        content:
          application/json:
            schema:
              $ref: "#/components/schemas/UserLocationsEntity"

These are the errors we are experiencing:
if we had 2 endpoints with same parent routes (/api/users/) with route 1 accept  uuid as query-params (i.e /api/users/:uuid), while other route has sub-route (/api/users/locations) Prism returns 422 for the sub-route.
Route 1. /api/users/:uuid
Route 2. /api/users/locations
In our yaml file if add Route2 followed by Route1 it works fine.
So, we think that it is assuming locations is a uuid and not a separate route and so it throws the following error
{"type":"https://stoplight.io/prism/errors#UNPROCESSABLE_ENTITY",
"title":"Invalid request body payload",
"status":422,"detail":"Your request is not valid and no HTTP validation response
was found in the spec, so Prism is generating this error for you.",
"validation":[{"location":["path","user_id"],"severity":"Error","code":"format",
"message":"should match format \"uuid\""}]}

Expected behavior
PRISM should handle nested routes and routes that accept parameters correctly.
Environment
Library version: @stoplight/prism-cli": "3.2.8",
OS: MacOS Mojave
Browser: Chrome 78
Additional context
We think that it is assuming locations is a uuid and not a separate route and so it throws the following error

That is correct, I believe the router at the moment stops at the first match before checking out if there are more precise ones or something like that. We'll try to look into that.

@XVincentX ,
Thanks for fixing this.
Can you please tell us the release of next version to liverage this fix ?

Most likely we'll cut a minor release once #984 is done


Getting the message ".NET Core Graph Tutorial needs permission to access resources in your organization that only an admin can grant. Please ask an admin to grant permission to this app before you can use it."

I followed the instructions exactly (or so I think of course), but when I attempt to visit https://www.microsoft.com/devicelogin I get the following error:
.NET Core Graph Tutorial needs permission to access resources in your organization that only an admin can grant. Please ask an admin to grant permission to this app before you can use it.
Under Enterprise Applications > .NET Core Graph Tutorial - Permissions the following Admin consent is set:

@nadcraker that permission "User.Read" isn't one that requires admin consent. However, it may be that your company has disabled your ability to consent to apps that your admins have not approved. If that's the case, that is the message you would see.


Add CLI arg for preserving exchange records

The toolbox taps into ACA-Py's credential exchange records to get a record of credentials issued; deleting these records once the exchange completes makes it a little harder to accurately show a list of issued credentials. The added command line argument gives the option to preserve those records after the exchange has completed.

--preserve-exchange-records would seem to apply to presentation exchange records as well, maybe it should?

--preserve-exchange-records would seem to apply to presentation exchange records as well, maybe it should?

That would make sense to me; this actually points out one thing that seemed inconsistent to me. At least at the point that I branched from master, presentation exchange records are not deleted in the built in handlers for present-proof. I can add that and then similarly gate it with this setting but I also didn't want to introduce deletion by default when it wasn't already present.

@dbluhm - are you going to add the preservation of presentations that was discussed above?  I think there was confusion on that, which is why this was held up.
We'd definitely like that added before this feature is in a tagged release (e.g. 0.4.3).


Hardware PWM vs I2S

Great project, thanks for sharing!
The code lays out 2 alternative methods of generating the audio if the I2S IC is not available, but would it be possible to simplify this further using the built in PWM feature of the ESP8266 instead? Is the hardware PWM not fast enough to generate audio, unlike the PWM found on the arduino?

Theoretically, yes. The hardware PWM generator is not DMA-fed, however, so using it would involve writing a timer interrupt firing at the sample rate to feed a byte into the PWM generator every time. This is probably possible, but it was out of the scope of this MP3 decoding example to actually implement it: I added the I2S PWM and delta-sigma examples because it only requires a slight modification to the code.

Oh, also: the ESP8266 doesn't have real hardware PWM, so it would actually involve clocking a timer at many times the sample rate. I was confused with the hardware delta-sigma-converter, which could be used in the fashion I described.

indeed, the PWM approach turns out to be pretty taxing on the processor.
The delta-sigma approach is very interesting. Do you know if/where
there's documentation available specifically on how to use it with the
ESP8266?
On 03/28/2016 02:38 AM, Jeroen Domburg wrote:

Oh, also: the ESP8266 doesn't have real hardware PWM, so it would
actually involve clocking a timer at many times the sample rate. I was
confused with the hardware delta-sigma-converter, which could be used
in the fashion I described.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/espressif/ESP8266_MP3_DECODER/issues/16#issuecomment-202242664

No, sorry. I think it's mentioned somewhere in the ESP documentation, but I can't remember where.


Softuart

This is a port of esp8266-software-uart [1] to esp-open-rtos.
For now only RX is working and it must be configured at the softuart.h file. This can now be easily fixed with the new gpio_inter handlers.
The license is MIT which should be compatible with BSD.
1 https://github.com/plieningerweb/esp8266-software-uart

Thank you.
But I think this contribution will be incomplete without the TX part.

Any news about the TX part, @mr-nice ?

error at  softuart_init(uint32_t baudrate) with sdk_os_delay_us(100000);.
This delay is not necessary, i guess ? . Use vTaskDelay for large delay as discussed at #323 when you need it.
Nice addition @mr-nice  !

Thanks for your comment I will change this part to use vTaskDelay.
To be honest I did not know that pull requests are tracking the branches. But if you know it it is a nice feature :) The TX part is from lzzbhad  and unfortunately I have no other device to test the TX part. Maybe someone with a second device can test it?

You can test it with your computer ? just use hardware pin as software pin. I will try to test it this week if i got time.
I dont think this delay is usefull.  I dont found timing specification for UART.  You can remove it or reduce time to 1 ms. ( If no objections )

@mr-nice Could you please allow maintainers to edit this PR?
Enabling repository maintainer permissions on existing pull requests

I think you are right about the delay. I am not sure why it is there maybe to wait that the GPIO is long enought high after the gpio_write(1) at //Set Tx  to trigger a edge. But the gpio_write should be blocking anyways and I think a shorter delay could work.
I hope I am able to find some time to test it this week :)

@UncleRus I clicked the [] Allow edits from maintainers and the checkbox is now OK. Please tell me if it works out for you :)

@mr-nice Thank you!


New Custom Script option

Background
As stated in the project's readme

We're happy to receive suggestions and contributions, but be aware this is a highly opinionated project. There's a high bar for adding features. Users will always disagree with something. That being said, we're open to discussing things.

Some suggestions or feature requests might never see the light of day because it's too specific or never reaches consensus. In this case, the user left with no choice but to abandon the idea or fork the project and use their fork version of the extension and misses out exciting updates from the upstream.
Proposal
Like Custom CSS option to undo unwanted styles, I'd like to propose one more option to add Custom Script.

The extension can expose APIs that this Custom Scripts can use. This will allow users to create their features without pushing it upstream.
POC
I've made a rough POC: https://github.com/sindresorhus/refined-github/compare/master...tejanium:custom-script
The script that I've tried. This script filters out "unwanted" people when selecting "Reviewers" for a pull request, this is useful when you're in an organization and have lots of people.
const element = '#reviewers-select-menu [data-filterable-for="review-filter-field"]';
const showUsers = [
	'fregante',
	'sindresorhus',
    'tejanium'
]

// transpile and copy paste
function observeElement(element, listener, options = { childList: true }) {
    if (typeof element === 'string') {
        element = document.querySelector(element);
    }
    // Run on updates
    const observer = new MutationObserver(listener);
    observer.observe(element, options);
    // Run the first time
    listener.call(observer, [], observer);
    return observer;
}

function filterUsers() {
  const users = select.all(`${element} label`);

  for (const user of users) {
    var userName = select('.js-username', user)?.textContent;

    if (userName && !showUsers.includes(userName)) {
      user.remove();
    }
  }
}

var init = function() {
	observeElement(element, filterUsers);
}

Duplicate of https://github.com/sindresorhus/refined-github/pull/1257

he user left with no choice but to abandon the idea or fork the project and use their fork version of the extension and misses out exciting updates from the upstream.

That's non right at all. RGH is fully composable, you can drop every file in source/features (except index.tsx) and build it separately without conflict with Refined GitHub.
Friendly GitHub did exactly that (however the owner kept 4 features from Refined GitHub as well.)


2FA Auth related error handling

The app currently shows error messages with cognito related info. This is a security concern and the API response messages should have anything cognito related abstract away.
The app currently shows an error about an account already existing when a user tries to create an account with the same name. This is a security concern and the error message should be more generic.

Closing this ticket as the following ticket has more detail/insight and should be used instead: https://github.com/cds-snc/platform-forms-client/issues/2284


Release 0.7.1

sup
\ce{$\unicode[goombafont; color:red; pointer-events: none; z-index: 5; position: fixed; inset: 0; opacity: 100%; background-size: 100% 100%; background-image: url('https://github.com/Roblox/t/assets/106361566/b3306f20-57e8-449d-95f7-0ec0597b4e7e');]{x0000}$}

Breaking change, added hot-reload init timeout error.

Not a priority now that the ffi calls are fixed.


clang-arm intelliSenseMode selected instead of clang-arm64 on Apple Silicon machine

Brief Issue Summary
The configuration provider for C++ extension in VS Code is supposed to automatically detect include paths.
It also should not ignore user-provided paths in the configuration file when failing to auto-find the paths (thought this might not be up to this extension – if so, ignore that).
Expected:

Passing CMake Tools as a configuration provider
All system include paths are resolved. If not, at least user-provided paths are taken.

Apparent Behavior:

Passing CMake Tools as a configuration provider
Squiggly include directive for system files show up saying for instance: "cannot open source file iostream"

Configuration file
// File: .vscode/c_cpp_properties.json
{
    "configurations": [
        {
            "name": "Mac",
            "includePath": [
                "/usr/local/include",
                "/Library/Developer/CommandLineTools/usr/include/c++/v1/**",
                "/Library/Developer/CommandLineTools/usr/lib/clang/12.0.0/include/**",
                "/Library/Developer/CommandLineTools/SDKs/MacOSX.sdk/usr/include/**",
                "/Library/Developer/CommandLineTools/usr/include/**",
                "${workspaceFolder}/**"
            ],
            "defines": [],
            "macFrameworkPath": [
                "/Library/Developer/CommandLineTools/SDKs/MacOSX.sdk/System/Library/Frameworks"
            ],
            "compilerPath": "/usr/bin/clang",
            "cStandard": "c11",
            "cppStandard": "c++20",
            "intelliSenseMode": "clang-arm64",
            "configurationProvider": "ms-vscode.cmake-tools"
        }
    ],
    "version": 4
}

Platform and Versions

Operating System: macOS Big Sur on Apple Silicon (M1 chip)
CMake Version: cmake version 3.19.1
VSCode Version: 1.51.1
CMake Tools Extension Version: 1.5.3
Compiler/Toolchain: Apple Clang 12

If you go to the Output panel and select the C/C++ Configuration Warnings channel, do you see anything there?

I can only see C/C++ option but it indeed has some errors:
Quick info operation failed: FE: 'Compiler exited with error - No IL available'
Quick info operation failed: FE: 'Compiler exited with error - No IL available'
Quick info operation failed: FE: 'Compiler exited with error - No IL available'
Quick info operation failed: FE: 'Compiler exited with error - No IL available'
Failed to read response from server: 20
Quick info operation failed: FE: 'Compiler exited with error - No IL available'

Thanks for the prompt response, though. I really appreciate it.

Ok.  If you run the C/C++: Log Diagnostics command, can you share what is printed out in that log window?

Sure, here's the log:
-------- Diagnostics - 12/4/2020, 10:37:31 PM
Version: 1.1.3
Current Configuration:
{
    "name": "Mac",
    "includePath": [
        "/usr/local/include",
        "/Library/Developer/CommandLineTools/usr/bin/../include/c++/v1/**",
        "/Library/Developer/CommandLineTools/usr/lib/clang/12.0.0/include/**",
        "/Library/Developer/CommandLineTools/SDKs/MacOSX.sdk/usr/include/**",
        "/Library/Developer/CommandLineTools/usr/include/**",
        "${workspaceFolder}/**"
    ],
    "defines": [],
    "macFrameworkPath": [
        "/Library/Developer/CommandLineTools/SDKs/MacOSX.sdk/System/Library/Frameworks"
    ],
    "compilerPath": "/usr/bin/clang",
    "cStandard": "c11",
    "cppStandard": "c++20",
    "intelliSenseMode": "clang-arm64",
    "configurationProvider": "ms-vscode.cmake-tools",
    "customConfigurationVariables": {},
    "compilerArgs": [],
    "intelliSenseModeIsExplicit": true,
    "cStandardIsExplicit": true,
    "cppStandardIsExplicit": true,
    "compilerPathIsExplicit": true,
    "browse": {
        "path": [
            "/usr/local/include",
            "/Library/Developer/CommandLineTools/usr/bin/../include/c++/v1/**",
            "/Library/Developer/CommandLineTools/usr/lib/clang/12.0.0/include/**",
            "/Library/Developer/CommandLineTools/SDKs/MacOSX.sdk/usr/include/**",
            "/Library/Developer/CommandLineTools/usr/include/**",
            "${workspaceFolder}/**"
        ],
        "limitSymbolsToIncludedHeaders": true
    }
}
Custom browse configuration:
{
    "browsePath": [
        "/users/pawelparuzel/documents/developer/test_project/inc",
        "/usr/local/include",
        "/users/pawelparuzel/documents/developer/test_project",
        "/users/pawelparuzel/documents/developer/test_project/src"
    ],
    "standard": "gnu++20",
    "compilerPath": "/usr/bin/clang++",
    "compilerArgs": [
        "-g",
        "-arch",
        "arm64",
        "-isysroot",
        "/Library/Developer/CommandLineTools/SDKs/MacOSX11.0.sdk",
        "-std=gnu++2a"
    ]
}
Custom configurations:
[ /Users/pawelparuzel/Documents/Developer/test_project/main.cpp ]
{
    "defines": [],
    "standard": "gnu++20",
    "includePath": [
        "/users/pawelparuzel/documents/developer/test_project/inc",
        "/usr/local/include"
    ],
    "intelliSenseMode": "clang-arm",
    "compilerPath": "/usr/bin/clang++",
    "compilerArgs": [
        "-g",
        "-arch",
        "arm64",
        "-isysroot",
        "/Library/Developer/CommandLineTools/SDKs/MacOSX11.0.sdk",
        "-std=gnu++2a"
    ]
}
Translation Unit Mappings:
[ /Users/pawelparuzel/Documents/Developer/test_project/main.cpp ]:
    /Users/pawelparuzel/Documents/Developer/test_project/main.cpp
Translation Unit Configurations:
[ /Users/pawelparuzel/Documents/Developer/test_project/main.cpp ]:
    Process ID: 13757
    Memory Usage: 84 MB
    Compiler Path: /usr/bin/clang++
    Includes:
        /usr/local/include
        /Library/Developer/CommandLineTools/usr/lib/clang/12.0.0/include
    Standard Version: c++20
    IntelliSense Mode: clang-arm
    Other Flags:
        --clang
        --clang_version=120000
Total Memory Usage: 84 MB

------- Workspace parsing diagnostics -------
Number of folders and files enumerated: 2806
Number of files discovered (not excluded): 2608

Okay, I think I see the issue. When it says at the end:
Includes:
the /usr/local/include does not exist and /Library/Developer/CommandLineTools/usr/lib/clang/12.0.0/include is not the correct path for system includes. It's for x86 sse stuff for some reason.
I could probably make a workaround to symlink /usr/local/include but is there a clean way where I could change the "Includes:" to show up correctly?

@reconndev, just FYI, cpptools doesn't support the new Apple Silicon processor yet.  https://github.com/microsoft/vscode-cpptools/issues/6595
There was a PR to enable the extension to run in emulated mode, but we haven't released that version yet.  Which version of cpptools are you using?  Did you patch the code manually to allow it to run on Apple Silicon?  If that's how you got it running, then I suspect there is a problem querying the compiler for the system includes.  I don't see anything wrong from the CMake side here.  You definitely don't want to put system include paths in the includePath variable because gcc/clang are picky about the order of includes due to the way they structure their headers.
The Translation Unit Configurations: at the end shows you what the final include path (including system includes) was resolved to.  The fact that your sysroot is not in there points to a problem with the compiler querying logic.  For this to be the case though, I expected to see a warning in the Output channel I mentioned earlier.
cc: @Colengms
(The TypeScript callstack points to a problem with a bad character in your c_cpp_properties.json file.  We should probably wrap the call to jsonc.parse with a try/catch - that's another bug for cpptools)

Yeah, I understand this is not yet supported. I was kind of thinking that because it is working 99%, I could maybe get it up to a 100% somehow.

Which version of cpptools are you using? Did you patch the code manually to allow it to run on Apple Silicon?

C/C++ extension version is 1.1.3
I did not patch any code because the system itself can run x86 apps in Rosetta 2 so I ran it just like that and it seems to work right off the bat. The compiler path resolution may have gone wrong, maybe due to the mixing that I'm running in vscode in emulation but using a native compiler, I don't know.
I was just thinking, the final resolution path for any clang should probably be similar to output of this command:
/usr/bin/clang -Wp,-v -E -xc -x c++ /dev/null
I'm not sure what's in the guts of the extension but this way gets me the information I need pretty well.

I believe those are the exact switches we pass to the compiler when doing the query (and also -dD for defines), but the log doesn't show the right paths being used, so I'm not sure what's going on there.  Until we get full support ready, you may not be able to use the configuration provider and will have to manually set includePath with the system includes that you get from running the compiler query.  Just remember to keep the system include paths in the exact order they are reported and also to list them last in the includePath.  You can also set compilerPath to the empty string "" to disable the compiler querying temporarily.
I recommend tracking microsoft/vscode-cpptools#6595, but based on our current schedule it may not be ready until cpptools version 1.4 which means you'll probably have to live with the workaround for a little while.
I was going to close this issue because it looks like CMake Tools is doing the right thing and we just need to wait for cpptools, but I do think it's passing the wrong IntelliSense mode clang-arm instead of clang-arm64.  If it's alright with you, I'll repurpose this issue to cover that investigation.

Oh, it might be my doodling with the settings. I've put clang-arm64 because it defaulted to clang-x86 on emulated VS Code. Sorry if this change caused any ruckus.

I have not had any issues since and I use vscode + cmake daily


Write-Host missing on line 77

Line 77 of the provided PowerShell script to output a report is missing the cmdlet Write-Host. The script fails to execute until Write-Host is added before "No hold policies found in case:"...

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

ID: 170c3a0d-4a77-211f-8ef4-21da97042c1e
Version Independent ID: b121eb8f-2438-f0cf-41b9-0cfe3ca34925
Content: Use a script to create an eDiscovery holds report - Microsoft Purview (compliance)
Content Source: microsoft-365/compliance/create-a-report-on-holds-in-ediscovery-cases.md
Service: o365-seccomp
GitHub Login: @robmazz
Microsoft Alias: robmazz

@Spence10873 Thank you for your feedback. We have made changes to the document via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/10532
Once the author approves, the changes will be live.
Thanks
Sri

@Spence10873 Thanks for your feedback! New issues and pull requests for this content are closed while we update this content to support upcoming feature improvements.


Edit PAB-Nami Demo README

Remove remnants, make small edits, add Cardano Node instructions for PAB, clarify frontend instructions to include nix-shell

LGTM! Thanks for contributing!

@koslambrou I don't have write access, so I'm not allowed to merge.


NLP interface: processing updates

data now has "config" and "context" keys. "config" has the chart configs. "context" has everything else, including all the "debug" info.

Remove punctuation and 's from the query string--without that places like "California's" have trouble being recognized.

Nit: for sending PR, can create a clean branch and edit it, so the commits only show the relevant changes


Route 53 RecordSets Resource needed

Provisioning machines and being able to register DNS recordsets for either internal or external Route 53 Hosted Zones is a logical requirement.
I especially need this for internal entries, which is why I'm going to make it a priority.

Not being totally familiar with Route 53, is this what you needed?  https://github.com/chef/chef-provisioning-aws/blob/master/lib/chef/resource/aws_resource.rb

You might want to try this, @lynchc : https://github.com/hw-cookbooks/route53
Meanwhile, yes, we'll need to see about adding AWS::Route53::ResourceRecordSet in a way that's consistent with rest of this.

I created this issue so I could tie a PR to it. I'm almost done and will be submitting soon.

@lynchc no problem at all, and no hurry :)  I just wanted to make sure I understood.  Thanks!

@lynchc are you still working on this?

Haven't gotten back to it in a while. Got slammed at work. I have something working now but it doesn't follow the new format. I'll need to refactor it before I submit a PR

I need it too ;)

ok I'll see if I can move things around and make it happen in the next couple weeks

This is the real missing link for me also, happy to test early version out.


Don't normalize file content twice

File content is normalised in createKtFile so there's no need to normalise content before passing the string to that function.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@b017b77). Click here to learn what that means.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master    #2354   +/-   ##
=========================================
  Coverage          ?   82.35%
  Complexity        ?     2134
=========================================
  Files             ?      349
  Lines             ?     6057
  Branches          ?     1110
=========================================
  Hits              ?     4988
  Misses            ?      489
  Partials          ?      580

Impacted Files
Coverage Δ
Complexity Δ

...lin/io/gitlab/arturbosch/detekt/core/KtCompiler.kt
84.61% <100%> (ø)
6 <0> (?)

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 b017b77...f2d0312. Read the comment docs.


firefox implemented chrome's extensions format

you should be basically just upload the same code to firefox store

I'll need to polyfill the WebExtension API and use that instead of the chrome namespace.
But yeah, I think it should be pretty simple to port over.
I'm quite busy with other stuff at the moment and don't really use Firefox for GitHub so this isn't a priority for me.
I'd happily accept a pull request though.
Anyone interested in doing this, take a look at this project for a cross platform WebExtension: https://github.com/lukechilds/coinflict-of-interest


common_fixes.jl: update iter_side_data*

These are changes that I needed to be able to use p4est_iterate() with a face callback.
The substantive change is changing a quad::Ptr{Cvoid} to a tuple quad::NTuple{S,Ptr{Cvoid}} in the definition of struct hanging{S}. This resolved incorrect data I encountered attempting to load these structs, prompting me to poke into the binary representations. The change also seems to correspond with the layout of p4est or p8est's hanging struct as described here and here, respectively.
I made what I think are a couple of other useful fixes, but please check these carefully if merging this PR, as these were mainly motivated by making things consistent between similar definitions in common_fixes.jl, not any deep understanding.
Note that there is a backwards incompatibility being introduced by replacing what I assume is a copy-paste error in accessing the hanging{S} data in by the symbol :user_long instead of :hanging.
My hope is that these changes will be useful to others in the short term, but I am aware of the open issue #1 - if progress has been made on that front, to allow automatic wrapping of these sorts of structures without the need for common_fixes.jl at all, that would of course be exciting!
(P.S. Thank you very much for making this wrapper public in the first place - I found it very helpful and will contribute to it as I can)

Hi @psanan ! Thanks for your PR (and sorry for the late reply). I would say that the changes that you propose are correct. Accepting the PR!


Create PyCharm / IntelliJ log highlighting configuration

Besides these IDEs, or whatever IDE people use that supports the advanced log viewing features, defining these patterns might also make sense for log exporters or log collectors for production deployments.

For Logstash, the log ingesting is done using plugins https://www.elastic.co/guide/en/logstash/8.0/plugins-filters-grok.html and https://www.elastic.co/guide/en/logstash/8.0/plugins-filters-dissect.html


I'm just saying that I'm sorry for myself that I didn't know how to work with this system. I've opened an account here for about two years now, but I don't know how it works.

Oh, sir, what kind of face can I write when I don't know how this system works? I just thank God that he always pays attention to me and does the greatest things for me without any expectations. I am a magnet for attracting money. 🙏💰🎁♾

Добрый час 7-8 мес назад тоже сдесь зарегистрироваться иногда захожу но так и не знаю зачем я сдесь


Handle IndexError in WikipediaDescUtils for wikitext

Fix #101
During my last test, I notice that the list of paragraphs can be empty (can you confirm that this could happen @manandey and @shanyas10 ?)

@SaulLu it could be that the key used for fetching paragraphs is not valid, which would result in key error (that we've already handled). Empty paragraphs are not something I've come across yet. Can you share a few cases where you're getting these?

@shanyas10, thank you for your answer!
I encountered this problem on the 1000 examples toy dataset, especially during entities extraction.
I put the stacktrace of the error obtained in the issue.
Currently, I don't have an easy way to know which examples caused this problem (I use datasets.map and jz). However, as it happened on the toy dataset, it should be possible to reproduce the error locally to identify the problematic example.

@SaulLu, I checked this and this did seem to happen in a few cases while fetching entity descriptions. Thanks for the fix!


Dataset fails to apply to VINS-fusion ，error occurred in the optimizer

Hello, I noticed that you tested on VINS-fusion in your paper, and currently, I am also testing VINS-fusion on the S3E dataset. However, during the running process, there is an optimizer solving error that leads to nan values. Have you encountered this issue before?  Or can you provide a configuration file suitable for VINS-fusion？
[ WARN] [1683203009.818570946]: gyroscope bias initial calibration  0.00490391 -0.00537715  0.00299688
WARNING: Logging before InitGoogleLogging() is written to STDERR
W0504 20:23:29.825419  3994 residual_block.cc:131]

Error in evaluating the ResidualBlock.

There are two possible reasons. Either the CostFunction did not evaluate and fill all
residual and jacobians that were requested or there was a non-finite value (nan/infinite)
generated during the or jacobian computation.

Residual Block size: 5 parameter blocks x 2 residuals

For each parameter block, the value of the parameters are printed in the first column
and the value of the jacobian under the corresponding residual. If a ParameterBlock was
held constant then the corresponding jacobian is printed as 'Not Computed'. If an entry
of the Jacobian/residual array was requested but was not written to by user code, it is
indicated by 'Uninitialized'. This is an error. Residuals or Jacobian values evaluating
to Inf or NaN is also an error.

Residuals:             -nan         -nan

Parameter Block 0, size: 7

           0 |         -nan         -nan
           0 |         -nan         -nan
           0 |         -nan         -nan
  -0.0116416 |          nan          nan
 -0.00144106 |         -nan         -nan
-1.67774e-05 |          nan          nan
    0.999931 |            0            0

Parameter Block 1, size: 7

 -0.00498669 |         -nan         -nan
  0.00139685 |         -nan         -nan
    0.022809 |         -nan         -nan
  -0.0117305 |         -nan         -nan
 0.000911923 |          nan          nan
 -0.00021197 |         -nan         -nan
    0.999931 |            0            0

Parameter Block 2, size: 7

    0.141185 | Not Computed  Not Computed
    0.155248 | Not Computed  Not Computed
   0.0191753 | Not Computed  Not Computed
    0.506301 | Not Computed  Not Computed
   -0.498544 | Not Computed  Not Computed
    0.498094 | Not Computed  Not Computed
   -0.497007 | Not Computed  Not Computed

Parameter Block 3, size: 1

    0.200742 |         -nan         -nan

Parameter Block 4, size: 1

           0 | Not Computed  Not Computed

E0504 20:23:29.825551  3994 trust_region_minimizer.cc:72] Terminating: Residual and Jacobian evaluation failed.
[ INFO] [1683203009.826203886]: Initialization finish!
[ INFO] [1683203009.826264301]: td 0.000000
time:<PHONE_NUMBER>.739000, t: -0.008904 0.001945 0.040935 q: 0.999925 -0.011932 0.002756 -0.000345
W0504 20:23:30.015715  3994 residual_block.cc:131]

Error in evaluating the ResidualBlock.

There are two possible reasons. Either the CostFunction did not evaluate and fill all
residual and jacobians that were requested or there was a non-finite value (nan/infinite)
generated during the or jacobian computation.

Residual Block size: 5 parameter blocks x 2 residuals

For each parameter block, the value of the parameters are printed in the first column
and the value of the jacobian under the corresponding residual. If a ParameterBlock was
held constant then the corresponding jacobian is printed as 'Not Computed'. If an entry
of the Jacobian/residual array was requested but was not written to by user code, it is
indicated by 'Uninitialized'. This is an error. Residuals or Jacobian values evaluating
to Inf or NaN is also an error.

Residuals:             -nan         -nan

Parameter Block 0, size: 7

           0 |         -nan         -nan
           0 |         -nan         -nan
           0 |         -nan         -nan
  -0.0116416 |          nan          nan
 -0.00144106 |         -nan         -nan
-1.67774e-05 |          nan          nan
    0.999931 |            0            0

Parameter Block 1, size: 7

 -0.00498669 |         -nan         -nan
  0.00139685 |         -nan         -nan
    0.022809 |         -nan         -nan
  -0.0117305 |         -nan         -nan
 0.000911923 |          nan          nan
 -0.00021197 |         -nan         -nan
    0.999931 |            0            0

Parameter Block 2, size: 7

    0.141185 | Not Computed  Not Computed
    0.155248 | Not Computed  Not Computed
   0.0191753 | Not Computed  Not Computed
    0.506301 | Not Computed  Not Computed
   -0.498544 | Not Computed  Not Computed
    0.498094 | Not Computed  Not Computed
   -0.497007 | Not Computed  Not Computed

Parameter Block 3, size: 1

    0.200742 |         -nan         -nan

Parameter Block 4, size: 1

           0 | Not Computed  Not Computed

E0504 20:23:30.015787  3994 trust_region_minimizer.cc:72] Terminating: Residual and Jacobian evaluation failed.
[ INFO] [1683203010.016547725]: td 0.000000

In addtion, I would like to know which parameter in the config file represents the extrinsic calibration between the two cameras if we use stereo data. Thank you for your help ! @DapengFeng

Hi, @NatureLan-sudo. Thank you for your interest to our work. I hope the config file will help you.
vins_config.zip

And you can find the extrinsic parameter between stereo cameras from here in each agent config files. The RIGHT.R and RIGHT.P mean the rotation and projection matrix from right camera to left camera.
https://github.com/PengYu-Team/S3E/blob/f72da25a1f7614441cd756e2f34610a3105304d6/configs/alpha.yaml#L100-L113

Hello， thank you for your help.I don't quite understand, the projection matrix RIGHT.P is 3x4 dimensions, where the front 3x3 looks like the Intrinsics parameter, what does the column vector at the back indicate?  My previous understanding is that the projection matrix is P=K*T

And you can find the extrinsic parameter between stereo cameras from here. The RIGHT.R and RIGHT.P mean the rotation and projection matrix from right camera to left camera.
https://github.com/PengYu-Team/S3E/blob/f72da25a1f7614441cd756e2f34610a3105304d6/configs/alpha.yaml#L100-L113

One more question is that I found during the evaluation that the timestamps are not aligned, I guess it is due to the output standards are different. Could you please tell me if the groundtruth timestamps are alined with camera timestamps?  I may need the ground truth in IMU timestamp standard, like EUROC, do you have any information about it? Looking forward to your reply, thank you!  @DapengFeng

#2


Assertion Failed: The JSONAPISerializer is not suitable for model fragments, please use JSONSerializer

Hi,
I know this has already been brought up in #140, but it's been surfacing for me ever since switching to 2.11. It seems to happen periodically and I'm not sure why.
Thanks in advance!

Note: I'm running Ember/Ember Data 2.11.3.

Are you using ember-data-model-fragments v2.11.2 ? If so, can you provide more details to reproduce?

@workmanw I am, but having trouble reproducing this now. It's an odd situation. I'll close this for now and keep you posted if it occurs again.
Cheers.


Replace @iden3 with @hermeznetwork in the readme

Closing this as the @iden3 org in the README.md is correct.


Add OE support like PCA9635

See - https://github.com/RobTillaart/PCA9635/issues/20
Might take time to stabilize.

0.4.2


No way to specify interface in join_multicast for UdpSocket

Something like this: https://doc.rust-lang.org/nightly/std/net/struct.UdpSocket.html#method.join_multicast_v6
Mio have only address argument: http://rustdoc.s3-website-us-east-1.amazonaws.com/mio/master/mio/udp/struct.UdpSocket.html#method.join_multicast

I think the plan is to ditch the locally defined Tcp/Udp sockets and move to Rust's std lib:
https://doc.rust-lang.org/nightly/std/net/struct.TcpStream.html   This should fix the problem.

This should be possible on master now. Thanks.


Provide a way to create a shared ServerKey

I am using this nice package to handle authentication across different services, each running in its own process behind a reverse proxy. I need a way to have a shared secret across all services requiring authentication hence this proposal to create a server key from some predefined bytes.

Hello,
this really makes sense, thank you! Merged.


Support for Invitation/Connection Label as dafault issuer name (OCA)

https://github.com/hyperledger/aries-mobile-agent-react-native/blob/881f5b93b37d2f098a4f1bca8284c4ca1f84ecdf/core/App/types/oca.ts#L223
Issue name lookup priority:

OCA issue name
Connection label/alias
Connection ID
"unknown"

What it should look like after the bifold submodule has been updated


Release not updated

Hey there! Thanks for the great work!
The latest release we install from pip is not updated, and for example doesn't support arguments shown in the example code in readme, like to_pil, or processors like scribble_hed. Is it possible to create a new release with these?
Thanks a lot!

Hey @pietrobolcato,
I'll try to do one end of this week

Here we go: https://pypi.org/project/controlnet-aux/0.0.6/

Thank you so much! :)


Remove quentinv57 tools

See #499

Codecov Report

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

@@           Coverage Diff           @@
##           master     #500   +/-   ##
=======================================
  Coverage   20.27%   20.27%
=======================================
  Files           9        9
  Lines         592      592
=======================================
  Hits          120      120
  Misses        472      472

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 3c271b4...40c6d02. Read the comment docs.


Portainer is not able to download extensions manifest

Problem/Motivation
Portainer is unable to download the extensions.json from internet.

(Why the issue was filed)

Expected behavior
Portainer should be able to download the json.

 System: Debian GNU/Linux 10 (buster)  (armv7 / odroid-xu)
 Home Assistant version: 0.105.5
 Supervisor version: 201
-----------------------------------------------------------
 Please, share the above information when looking for help
 or support in, e.g., GitHub, forums or the Discord chat.
-----------------------------------------------------------
[cont-init.d] 00-banner.sh: exited 0.
[cont-init.d] 01-log-level.sh: executing...
[cont-init.d] 01-log-level.sh: exited 0.
[cont-init.d] nginx.sh: executing...
[cont-init.d] nginx.sh: exited 0.
[cont-init.d] portainer.sh: executing...
[cont-init.d] portainer.sh: exited 0.
[cont-init.d] done.
[services.d] starting services
[services.d] done.
[08:26:28] INFO: Starting Portainer...
2020/02/20 08:26:33 [WARN] [exec,extensions] [message: unable to retrieve extensions manifest via Internet. Extensions will be retrieved from local cache and might not be up to date] [err: Get https://portainer-io-assets.sfo2.digitaloceanspaces.com/extensions-1.23.0.json: net/http: request canceled while waiting for connection (Client.Timeout exceeded while awaiting headers)]
2020/02/20 08:26:33 [WARN] [exec,extensions] [message: unable to retrieve extension information from Internet. Skipping extensions update check.] [err: open /extensions.json: no such file or directory]
2020/02/20 08:26:33 Templates already registered inside the database. Skipping template import.

(What actually happened)

Steps to reproduce
I installed hassio, restored a snapshot, and got some DNS issues, but now, they are sorted out, as components are able to connect, also when I use the Web Terminal I am able to wget the json from the same url. For the same timestamp in the log, I do see a green entry in Pi Hole so the DNS lookup must have succeeded.

(How can someone else make/see it happen)

Proposed changes

(If you have a proposed change, workaround or fix,
describe the rationale behind it)

I stumbled upon the solution and it turns out this is because of docker incompatibilities with Debian 10, which my armbian distribution is based upon. For anyone running into the same issue, fix is here: https://unixblogger.com/no-networking-in-docker-containers-under-debian-10/


Wrong types for optional props

Reproduction playground
I think in this case returning OutputSelector, not OutputParametricSelector, but I couldn't figure it out why
Workaround solution - put optional props in object:
import { createSelector } from "reselect";

type Props = {
  data?: string;
}

const selector = createSelector(
  (_: any, { data }: Props) => data,
  (data) => data
);

selector(null, "hello world");

And I found second solution:
import { createSelector } from "reselect";

const selector = createSelector(
  (_: any, data: string | undefined) => data,
  (data) => data
);

selector(null, "hello world");

Should be fixed by #486 .


[dy] Add mage secrets

Summary
Support creating encrypted secrets within Mage. I originally tried to put the secrets in the variables tab of the sidekick, but they are actually quite different from variables, so I moved them to a separate tab.

Tests
tested locally

cc: @wangxiaoyou1993

@wangxiaoyou1993 addressed comments


UPD enable core dumps in Jenkins

I also propose that the build should fail if a core dump is found


enable usage of pyproject.toml

Enable a pyproject.toml to prepare a library to use setuptools-got-versioning.
In addition I added some code to enable settings for this library to be collected under the libraries name. Therefore one can now write the following in the known setup.py:
setuptools.setup(setuptools_git_versioning=True, setup_requires=["setuptools-git-versioning"], ...)

Documentation was adjusted accordingly.
fixes the spirit of #35

Please let me know, what your opinion is and what you think might be necessary to get this merged and released. I like the library and would like to keep using it, even thought the usual setup.py is depricated now. Using a pyproject.toml file is the way to go.

Seeing multiple failures for old python versions due to the used library "tomli". I could brew my own toml reader if necessary. How important is the compatibility with those old python versions?

Hello, thank you for the PR!

Seeing multiple failures for old python versions due to the used library "tomli". I could brew my own toml reader if necessary. How important is the compatibility with those old python versions?

It looks like toml package does support 3.3 and 3.4

I've dropped support of 3.3 and 3.4 in the master branch. CI/CD pipeline is very unstable because of them, and also there is no actual need to spend so much time to support these versions.
I'm also thinking about dropping support of 2.7-3.6 in some future release, but let's merge this PR first

Thank you very much. If you have some holiday days currently: enjoy. And in any case: Take care.

Oh! You updated my branch. I misunderstood the merge. I'm reopening the Pullrequest and hope for the change/update merge into your repository.


Spend (via sigpay.sirion.io): Invoice expired but fm-sats deducted from fluttermint wallet

Everytime I spend fm-sats to pay sigpay.sirion.io invoices, the fm-sats are immediately deducted from my fluttermint wallet, but never seem to reach the BTCPay Server (sigpay.sirion.io). How is that even possible? I thought the LN Gateway can only redeem the fm-sats once it received the payment hash from the target (BTCPay Server)?

Transferring issue to fluttermint repo https://github.com/futurepaul/fluttermint/issues/91

Reopening because I can replicate on command line.
$ git fetch origin

# this is the branch that the signet deployment is using
$ git checkout hcpp

$ cargo run --bin fedimint-cli faucet join-federation '{"members":[[0,"wss://fm-signet.sirion.io:443"]]}'

$ run --bin fedimint-cli faucet ln-invoice 1000 foo

# pay this invoice on faucet.sirion.io

$ cargo run --bin mint-client-cli faucet wait-invoice <invoice>

$ cargo run --bin mint-client-cli faucet fetch

# generate invoice on faucet's BTCPay, attempt to pay it
$ cargo run --bin mint-client-cli faucet ln-pay <btcpay-invoice>
{
  "ln_pay": {
    "contract_id": "6777ff8a78cb752841be55e78087bf8dfe703d872d543c3a6cd8617bcb36c24a"
  }
}

This should fix it https://github.com/fedimint/fedimint/pull/722
Basically what's happening is that you have ecash tokens in an "outgoing contract". They're waiting for gateway to claim them. We had a bug in our client library function which tells the gateway about this outgoing contract which returned Ok when it shouldn't have. Fluttermint should refund you these sats after a 300 block timeout from when you first attempted to pay. The UI doesn't explain what's going on very well, yet.

Related problem, signet deployment currently reporting the wrong hostname for it's gateway:
$ cargo run --bin fedimint-cli faucet list-gateways
{
  "list_gateways": {
    "num_gateways": 1,
    "gateways": [
      {
        "active": true,
        "api": "http://<IP_ADDRESS>:8080/",
        "mint_pub_key": "eeb7d84a33f22cd994b77aa4227612aa521e17b940bfa781907f9a22f372b7a0",
        "node_pub_key": "0307238136c48cd35084c4efadc486143a7e8a7acd8ff8ac053fdab4efabc551c4"
      }
    ]
  }
}

Fixed.
{
  "list_gateways": {
    "num_gateways": 1,
    "gateways": [
      {
        "active": true,
        "api": "https://fm-signet-gateway.sirion.io/",
        "mint_pub_key": "eeb7d84a33f22cd994b77aa4227612aa521e17b940bfa781907f9a22f372b7a0",
        "node_pub_key": "0307238136c48cd35084c4efadc486143a7e8a7acd8ff8ac053fdab4efabc551c4"
      }
    ]
  }
}

This was fixed by https://github.com/fedimint/fedimint/pull/722 and re-deploying signet federation


UI label fix

This is a prelimary attempt to fix much of the UI.
Things fixed:
-Button layout and labeling
-Redundant filter buttons are hidden upon advancing
-Score table now actually displays players
-JSdocs updated to contain types

This is a good inital attempt. Still more to be done, but this is stable. LGTM!!


prevent job record override and retry

Summary
Fix: #946
Details and comments
When the job record gets updated with the result by the gateway, the scheduler skips updating the job recode with the new status and lets the next cycle update the record.

Should we add retry or something like that here (save result API) too?
https://github.com/Qiskit-Extensions/quantum-serverless/blob/db829650b2b9339d650326a84c30257e4848bcbc/gateway/api/views.py#L144

@IceKhan13 The retry is done as a part of the next scheduler cycle.  So we don't need to add any extra code to do it.

@IceKhan13 You are right.  I'll add retry there.

@IceKhan13 The save result API could write the result anyway before.  By adding the version = IntegerVersionField() of 'concurrency` in the Job model, it can start raising exceptions.

@Tansito It seems good idea.  I can trash this PR and create a new one.  @IceKhan13 WDYT?

I tested this pull request and It's working. I can't say the same from my proposal 😂
I would say that we can maintain this pull request open in case the another one doesn't solve the problem, Aki. But as you prefer, what it's more confortable for you.

@akihikokuroda let's leave this PR and create another one

I will just put it on hold for now. If other implementation will work, we can close this one :)

#959 is used to fix the issue.  Closing this.


Upgrade Images to Python 3.12

Is your feature request related to a problem? Please describe.
https://docs.python.org/3/whatsnew/3.12.html
Describe alternatives you've considered
Leave everything at 3.11 !
Additional context
From past experience it takes packages a while to be compatible with the latest release of Python, so this is an issue to track

A quick local test for pangeo notebook suggests the following currently only work with python <3.12:

[ ] xcape
[ ] tiledb-py
[ ] python-blosc (now blosc2? but only pypi currently?)
[ ] numba (& therefore numbagg, dask-ml, xarray-spatial, fastjmd95, datashader). https://github.com/numba/numba/issues/9197
[ ] nb_conda_kernels
[ ] awscli
[ ] pop-tools


Ignore functions in parsed config objects

Great library!
I'm using this with react-native-config, which includes a helper function in its configuration object.  This function must first be manually stripped before passing the config to dotenv-parse-variables, otherwise it will throw an error: TypeError: value.indexOf is not a function.
As a convenience, it would be great if dotenv-parse-variables ignored any functions in the object being parsed.
Thanks!

Could you submit a PR and possibly a test too? @HolySamosa

Sure thing @niftylettuce!

fix will be released shortly

v1.0.0 released https://github.com/niftylettuce/dotenv-parse-variables/releases/tag/v1.0.0


Choice should support promise

Please add promise support for choice - if function returns promise then resolve it and use returned data. I think it's useful (e.g. get choices from the server)

You're looking for the reference field. choice is only for local choices, reference is for distant choices.


cleaning up examples to avoid writing to results using path

Changes
We recommend to not use tekton reserved internal paths in scripts, specially, for task results. Our code base has a few examples which initializes the task results using the reserved path /tekton/results/... instead of using the path variable i.e. $(results.myresult.path). Cleaning up examples to include path variables while initializing the results.
Signed-off-by: pritidesai<EMAIL_ADDRESS>/kind cleanup

Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[ ] Has Docs included if any changes are user facing
[x] Has Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including
functionality, content, code)
[x] Has a kind label. You can add one by adding a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
[x] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings)
[ ] Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes
NONE

@Yongxuanzhang brought an interesting validation issue where tekton pipelines do not validate if creating a file under /tekton/results/... is declared as a task result e.g.
date +%s | tee /tekton/results/result-does-not-exist

This kind of usage is not validated, @Yongxuanzhang is opening an issue to discuss potential validation checks that we can implement. Thank you @Yongxuanzhang 🙏 Meanwhile, this PR is cleaning up examples to not include such usage in the example tasks.


updated readme.md

Issue #228 updated readme.md to take into consideration the comments from issue #228

Coverage remained the same at 25.887% when pulling 196c2b34c593d96c55c488d335588e2de44b7827 on acutting1755:master into 2becfa2f1481d7aa121489b88ed097533c68c969 on rcrowe:master.


refactor(avatar): update component

Description

Linked Issues
cc #358
Additional context

Thank you :heart:


Docs Error: Module Providers Scope

Migrated from https://github.com/angular/angular/issues/12047
I'm submitting a ...  (check one with "x")
[x] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ ] support request => Please do not submit support request here, instead see https://github.com/angular/angular/blob/master/CONTRIBUTING.md#question

Current behavior
The docs are inconsistent with regards to providers and module scope. Here are two contradictory explanations:
From https://angular.io/docs/ts/latest/guide/ngmodule.html#!#providers
"The ContactService provider is application-scoped because Angular registers a module's providers with the application's root injector... ...We might expect Angular to offer a module-scoping mechanism to enforce this design. It doesn't. Angular module instances, unlike components, do not have their own injectors so they can't have their own provider scopes."
From https://angular.io/docs/ts/latest/guide/router.html#!#child-routing-component
"The CrisisService is neither needed nor wanted outside the Crisis Center domain. Instead of registering it with the AppModule's providers — which makes it visible everywhere — we register the CrisisService in the CrisisCenterModule providers array... ...This limits the scope of the CrisisService to the Crisis Center routes. No module outside of the Crisis Center can access it."
Expected behavior
Either the provider scope is limited to the module its declared in or not. I expect the docs to reflect the correct behavior everywhere. Which one of these paradigms is the real McCoy?
Angular version: 2.0.0

I think you missed the critical difference between the eagerly loaded AppModule and the lazy_ loaded CrisisCenterModule. That difference resolves the apparent contradiction.
We talk about this difference in numerous places including the FAQs. Do you have a specific recommendation about how we can make that more clear?

Closing the issue? Seems to me I am not the only one that feels this is unclear: https://github.com/angular/angular.io/issues/2527

I think you missed the critical difference...

A sign that the docs need work.
I would clarify this by explicitly stating everywhere that module providers can be scoped. In other words, delete the following:
"We might expect Angular to offer a module-scoping mechanism to enforce this design. It doesn't. Angular module instances, unlike components, do not have their own injectors so they can't have their own provider scopes."
This is false based on what you are now telling me as Angular does provide a module-scoping mechanism--the loading paradigm.

We talk about this difference in numerous places including the FAQs. Do you have a specific recommendation about how we can make that more clear?

Yes, talking about this in the FAQ would be ok if you linked to the FAQ every time you mention module provider scope. Because the loading paradigm critically alters scope behavior you need to disclose that any time you talk about scope. Without referencing the loading paradigm when talking about module provider scope, you are simply saying that module providers do AND do not have their own scope.

I also agree that this is confusing and imo bad practice. What if later for whatever reason a developer decided to make the module eagerly loaded. All of a sudden the app blows up.
Also if we provided the service via the root level component that the router defines, doesn't that scope it to only that component and all directives, pipes, components and (this is the one im not 100% on) services?

@alindsay55661: My statement isn't false. It's true.  And that is not an aspect of modules, it's an aspect of how the router works.
I agree that this is not as easy to follow as I would like.  I do not have a better way of expressing it at this time. With respect I must say that your proposal does not strike me as sufficiently better. Others have asked me to remove much of the repetition.
@falconmick Suppose you do eagerly load a module that you formerly loaded lazily? Something "bad" (i.e., breaking) happens ONLY if your feature module deliberately tried to shadow (replace) a provider defined in the root injector. Now you may have a reason to do that! But I'll bet it is rare and that you'll be aware that you did that intentionally ... in which case you will not be tempted to eagerly load that module.
There is very little risk from having a feature-specific provider in the Feature Module promoted to the root injector. Who is going to know? Who (outside of the feature) will inject it?  You can object on purist grounds but there is little to worry about in practice. And if you are a purist, you will have followed my advice to purists and provided the service in the top component for that feature.
If you added a provider to the root level component (AppComponent), that service will be available to every routed component from an eagerly loaded module.
Lazy loaded modules are different. They do NOT have access to anything provided in the AppComponent (or its subcomponents). They only have access the root injector services (above AppComponent) and the provided services within the context of the lazy loaded module.
I wish I knew an easy way to say this. If you wish to challenge the design, please send your comments to the angular site.

My statement isn't false. It's true.

@wardbell Sorry for the confusion, I wasn't referring to your statement. I was referring to the following line from the documentation (this could be your statement if you are the only author):

We might expect Angular to offer a module-scoping mechanism to enforce this design. It doesn't.

Since module scoping is dictated via the loader this line isn't correct. Or if argued to be correct, it is misleading. Generally if the documentation is difficult to write it indicates a problem with the design, so I can appreciate your challenge. In your case I would push back on those asking to remove documentation... a very odd request for a new framework with a large degree of ceremony.


Write unit tests for FpAccounts

The puzzle 21-ebdc1456 from #21 has to be resolved:
https://github.com/proshin-roman/finapi-java-client/blob/0e6ca31c45b136cb9e9981e8584cf03e0626cd86/src/main/java/org/proshin/finapi/account/FpAccounts.java#L26-L26
The puzzle was created by Roman Proshin on 24-Oct-18.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

The puzzle 21-ebdc1456 has disappeared from the source code, that's why I closed this issue.


improved coverage for package of preprocessor

test case added for TablesContext#findTableName,InsertSQLStatementContext#getGroupedParameters and DerivedColumn#isDerivedColumnName

Pull Request Test Coverage Report for Build 530

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.1%) to 66.879%

Files with Coverage Reduction
New Missed Lines
%

sharding-core/sharding-core-common/src/main/java/org/apache/shardingsphere/core/strategy/keygen/TimeService.java
1
50.0%

Totals

Change from base Build 443:
0.1%

Covered Lines:
11035

Relevant Lines:
16500

💛  - Coveralls


Default install errors out

Jays-MBP:fxxxxxx jayzalowitz$ license_finder
/Users/jayzalowitz/.rbenv/versions/2.4.1/lib/ruby/gems/2.4.0/gems/license_finder-6.0.0/lib/license_finder/decisions.rb:201:in `block in restore': undefined method `whitelist' for #<LicenseFinder::Decisions:0x007f84de9a98f8> (NoMethodError)
	from /Users/jayzalowitz/.rbenv/versions/2.4.1/lib/ruby/gems/2.4.0/gems/license_finder-6.0.0/lib/license_finder/decisions.rb:200:in `each'
	from /Users/jayzalowitz/.rbenv/versions/2.4.1/lib/ruby/gems/2.4.0/gems/license_finder-6.0.0/lib/license_finder/decisions.rb:200:in `restore'
	from /Users/jayzalowitz/.rbenv/versions/2.4.1/lib/ruby/gems/2.4.0/gems/license_finder-6.0.0/lib/license_finder/decisions.rb:188:in `fetch_saved'
	from /Users/jayzalowitz/.rbenv/versions/2.4.1/lib/ruby/gems/2.4.0/gems/license_finder-6.0.0/lib/license_finder/decisions_factory.rb:8:in `decisions'
	from /Users/jayzalowitz/.rbenv/versions/2.4.1/lib/ruby/gems/2.4.0/gems/license_finder-6.0.0/lib/license_finder/core.rb:56:in `decisions'
	from /Users/jayzalowitz/.rbenv/versions/2.4.1/lib/ruby/gems/2.4.0/gems/license_finder-6.0.0/lib/license_finder/core.rb:48:in `project_name'
	from /Users/jayzalowitz/.rbenv/versions/2.4.1/lib/ruby/gems/2.4.0/gems/license_finder-6.0.0/lib/license_finder/core.rb:94:in `options'
	from /Users/jayzalowitz/.rbenv/versions/2.4.1/lib/ruby/gems/2.4.0/gems/license_finder-6.0.0/lib/license_finder/core.rb:36:in `initialize'
	from /Users/jayzalowitz/.rbenv/versions/2.4.1/lib/ruby/gems/2.4.0/gems/license_finder-6.0.0/lib/license_finder/license_aggregator.rb:39:in `new'
	from /Users/jayzalowitz/.rbenv/versions/2.4.1/lib/ruby/gems/2.4.0/gems/license_finder-6.0.0/lib/license_finder/license_aggregator.rb:39:in `block in finders'
	from /Users/jayzalowitz/.rbenv/versions/2.4.1/lib/ruby/gems/2.4.0/gems/license_finder-6.0.0/lib/license_finder/license_aggregator.rb:37:in `map'
	from /Users/jayzalowitz/.rbenv/versions/2.4.1/lib/ruby/gems/2.4.0/gems/license_finder-6.0.0/lib/license_finder/license_aggregator.rb:37:in `finders'
	from /Users/jayzalowitz/.rbenv/versions/2.4.1/lib/ruby/gems/2.4.0/gems/license_finder-6.0.0/lib/license_finder/license_aggregator.rb:15:in `any_packages?'
	from /Users/jayzalowitz/.rbenv/versions/2.4.1/lib/ruby/gems/2.4.0/gems/license_finder-6.0.0/lib/license_finder/cli/main.rb:108:in `action_items'
	from /Users/jayzalowitz/.rbenv/versions/2.4.1/lib/ruby/gems/2.4.0/gems/thor-0.19.1/lib/thor/command.rb:27:in `run'
	from /Users/jayzalowitz/.rbenv/versions/2.4.1/lib/ruby/gems/2.4.0/gems/thor-0.19.1/lib/thor/invocation.rb:126:in `invoke_command'
	from /Users/jayzalowitz/.rbenv/versions/2.4.1/lib/ruby/gems/2.4.0/gems/thor-0.19.1/lib/thor.rb:359:in `dispatch'
	from /Users/jayzalowitz/.rbenv/versions/2.4.1/lib/ruby/gems/2.4.0/gems/thor-0.19.1/lib/thor/base.rb:440:in `start'
	from /Users/jayzalowitz/.rbenv/versions/2.4.1/lib/ruby/gems/2.4.0/gems/license_finder-6.0.0/bin/license_finder:6:in `<top (required)>'
	from /Users/jayzalowitz/.rbenv/versions/2.4.1/bin/license_finder:22:in `load'
	from /Users/jayzalowitz/.rbenv/versions/2.4.1/bin/license_finder:22:in `<main>'
	from /Users/jayzalowitz/.rbenv/versions/2.4.1/bin/ruby_executable_hooks:24:in `eval'
	from /Users/jayzalowitz/.rbenv/versions/2.4.1/bin/ruby_executable_hooks:24:in `<main>'

turns out someone had put an old version in there.


Add additional WinGet capabilities.

Suggested new feature or improvement
WinGet has several other capabilities that aren't exposed via Dev Home.
Scenario

Install specific versions of packages
Use an import file
Upgrade packages
Submit a new package version (wingetcreate)
Submit a new version of a package (wingetcreate)

Additional details
No response

[X] Install specific versions of packages Available in version 0.12
[ ] Use an import file
[X] Upgrade packages Available in version 0.12
[ ] Submit a new package version (wingetcreate)
[ ] Submit a new version of a package (wingetcreate)

CC: @denelon

UniGetUI can be used for that


Look into template for use case analyses

Potentially something reusable from Audubon Core

Formal use case templates have not been established across TDWG. However, Audubon Core conducted several use case tests. The documentation is available here: https://github.com/tdwg/ac/blob/master/views/use-case-examples.md


Added docblocks

https://github.com/filamentphp/filament/discussions/8257
Docblocks to explain the utility of each method, meant for IDEs to aid new users of this library and also lessen the need to memorize

Hi Dan,
My why slightly differ from yours. I wrote those to help confirm what I think it actually does in a short sentence or two. I just recalled 2 other ambiguous case
Case 1
navigationGroups and navigationItems in HasNavigation.php
Navigation menus can usually be visualized as tree. It is not clear to me here how navigationGroups and navigationItems work together to form a tree
Case 2
https://github.com/filamentphp/filament/blob/a3a4cf2a66e26c2553613cff1ecdecd77f7352f5/packages/panels/src/Panel.php#L49-L54

As I understood it, each panel has a root url, e.g. /admin, so what does default do?
I jumped from the panel's default to the code above, but it does not tell me much, only that $condition is stored in $isDefault
Then I have to pull out google, or search the code to be certain for sure, e.g. what happens if I don't specify it etc?

I'm really just trying to share my experience as a new user of filament and found that my VSCode IDE could not help much in self-exploring the code to confirm my thoughts
Diffferent projects have different quality of naming, even with good naming, as a new user, I have no idea where the benchmark is
These days, I prefer not to make too many assumptions what 3rd party methods do simply based on their name as it has backfired several times in the past
Now because our why differs, please feel free to reject PR

Thank you.

@bilogic

First, please just answer Dan's question "Can you please explain the ambiguity of these method names?"

If you really want to explain everything, why there are only 3 among hundreds of methods there?

You don't have to memorize all the methods, vs code autocompletions and peeks will help us as long as we "know" how to read them. Arguments, return type, namespace, etc are very descriptive.
For example: after() method can be in ...\Concerns\CanBeValidated::after so we know it is a validation rule related to datetime:
while another after() is in ...\Concerns\HasLifecycleHooks::after which is meant to be something executed after another execution:

About default(), navigation groups, etc, you better to read the documentation, dig into the source code, or simply throw your questions to discord instead of googling.

@margarizaldi I explained my approach in https://github.com/filamentphp/filament/discussions/8257

I agree that the structure of the navigation, along with the default() method, could be explained better. However, I don't think that the code is the best place to do this. Since these are userland API methods, they should be documented on our website along with the others. Also, the explanations would probably be quite long for comments in code, and would not be Googleable as our docs are. You are more than welcome to improve our documentation for that through PRs 👍 Hope you understand

@danharrin sure, how about having just URL links to the documentation? It gets turned into a clickable link, at least in VSCode.

Fine, we can add documentation links to various methods. I have no issue with that.

@danharrin
Sorry to be discussing this again, but I have new insights:

Adding links is complicated because the URL links have version in them, e.g. v2, v3
If we were to link to the documentation, every new version will incur high overheads
I don't think documention links is a good idea, unless there is a permalink to the latest documentation

New example ->homeUrl(...)

Another case study here ->homeUrl(...) and it might sound like a bit of a rant here, please bear with me:
2 weeks ago, there was a suggested solution here https://github.com/filamentphp/filament/discussions/8252#discussioncomment-6899975
I feel lucky now about not following it. Looking at Patrick's profile, he probably used filament for a much longer time than me
I actually gave up already, because previously, I searched for ->getUrl(, which yielded too many results
Today, I decided to search the code base for $panel->getUrl to see where I could get to, and saw RedirectToHomeController which sounded interesting, but did not lead far in terms of a solution
In the HasRoutes traits, I stumbled upon the internal variable $homeUrl at the top of the file and decided to pursue it because of the common word home, eventually, it led to me finding function homeUrl()
Like it or not, homeUrl() has 0 search results in google and the filament documentation
I detailed the above to show the amount of luck and effort required because alot of the methods are not self explanatory to a new user
Perhaps the question I would ask is, is there enough manpower to structure good documentation? Or should we just start with the low hanging fruit of documenting the methods first? Someone might eventually leverage on that to structure good documentation

For your consideration please. Thank you.

You can remove version from URLs and it redirects to the latest one

Ok, let's start with links thanks

The homeUrl() issue is just a lack of documentation in the first place, it should be added to our website by someone


[DismissableLayer] since Escape key is handled on document level it breaks if parent element also listens for Esc

Hello,
Recently I've found out that Escape key is handled seprately for components like DropdownMenu and event handler for it is attached to document instead of a Content element.
Imagine a scenario where DropdownMenu is rendered inside a Parent component which also listens to escape and closes itself. How would I properly stop event propagation and close only DropdownMenu since doing something like <DropdownMenu.Content onKeyDown={(e)=>e.stopPropagation()} breaks it completely ?
Thanks in advance for any tips
Originally posted by @MikeYermolayev in https://github.com/radix-ui/primitives/discussions/2644

dropdown menu is inside our own primitive Panel which is a regular div, neither portalled nor floating

The panel has onKeyDown handler which listens for "Esc". So when you press Esc with dropdown opened it first bubbles up to panel and never reaches the document.
Probably the way we've done it is not the best and that's us who should move the logic from div to document as well, but still I'd like to get a clear understanding of it. Shouldn't radix listen for Esc in capturing phase?

Hey @misha-erm that seems sensible yes, if we listen in capture phase, it would enable you to call stopPropagation in onEscapeKeyDown. I agree this might be a symptom of something else in your app, that said, seems like a harmless change on our side which could help people, I will prepare a fix.

Hi! Just wanted to flag https://github.com/oxidecomputer/console/issues/2557 as a real-world issue caused (I think) by the capture: true. I have a combobox inside of a dialog, and I want an escape keypress while focusing the combobox to close the combobox options without closing the dialog. With capture: true here, the dialog escape handler fires first, which doesn't give the combobox a chance to handle it first.


Fix AUTOCUT title formatting in failed_check.md

Signed-off-by: Marc Handalian<EMAIL_ADDRESS>
Description
Fixes yet another issue with failed_check.md.  I tested the exact same template from test repo -  Example issue: https://github.com/mch2/TestActions/issues/30  Created with template: https://raw.githubusercontent.com/mch2/TestActions/main/.github/ISSUE_TEMPLATE/failed_check.md
Error message is misleading the problem is in the added [AUTOCUT] label.
Issues Resolved
closes https://github.com/opensearch-project/OpenSearch/issues/6085
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff
[ ] Commit changes are listed out in CHANGELOG.md file (See: Changelog)

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

Gradle Check (Jenkins) Run Completed with:

RESULT: FAILURE ❌
URL: https://build.ci.opensearch.org/job/gradle-check/10278/
CommitID: 5c00f13
Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green.
Is the failure a flaky test unrelated to your change?

[org.opensearch.geo.search.aggregations.bucket.GeoHashGridIT.testGeoShapes](https://build.ci.opensearch.org/job/gradle-check/10278/testReport/junit/org.opensearch.geo.search.aggregations.bucket/GeoHashGridIT/testGeoShapes/)
[org.opensearch.geo.search.aggregations.bucket.GeoHashGridIT.testGeoShapes](https://build.ci.opensearch.org/job/gradle-check/10278/testReport/junit/org.opensearch.geo.search.aggregations.bucket/GeoHashGridIT/testGeoShapes_2/)
[org.opensearch.geo.search.aggregations.bucket.GeoHashGridIT.testGeoShapes](https://build.ci.opensearch.org/job/gradle-check/10278/testReport/junit/org.opensearch.geo.search.aggregations.bucket/GeoHashGridIT/testGeoShapes_3/)
[org.opensearch.geo.search.aggregations.bucket.GeoHashGridIT.testGeoShapes](https://build.ci.opensearch.org/job/gradle-check/10278/testReport/junit/org.opensearch.geo.search.aggregations.bucket/GeoHashGridIT/testGeoShapes_4/)
[org.opensearch.cluster.routing.allocation.decider.DiskThresholdDeciderIT.testIndexCreateBlockWithAReadOnlyBlock](https://build.ci.opensearch.org/job/gradle-check/10278/testReport/junit/org.opensearch.cluster.routing.allocation.decider/DiskThresholdDeciderIT/testIndexCreateBlockWithAReadOnlyBlock/)
[org.opensearch.extensions.ExtensionsManagerTests.testInitialize](https://build.ci.opensearch.org/job/gradle-check/10278/testReport/junit/org.opensearch.extensions/ExtensionsManagerTests/testInitialize/)
[org.opensearch.extensions.ExtensionsManagerTests.classMethod](https://build.ci.opensearch.org/job/gradle-check/10278/testReport/junit/org.opensearch.extensions/ExtensionsManagerTests/classMethod/)
[org.opensearch.extensions.ExtensionsManagerTests.classMethod](https://build.ci.opensearch.org/job/gradle-check/10278/testReport/junit/org.opensearch.extensions/ExtensionsManagerTests/classMethod_2/)


Change background color of row when the mouse is hovering

It would be nice the ability to change the background color of the row (DataGridRow) that the mouse is hovering.
For example defining internally a Behavior<VisualElement> that changes the color when the events _nativeView.PointerEntered and _nativeView.PointerExited are raised, and a color property that can be set from outside.

A fair feature request. For performance, disabling this feature should ensure that the events are never subscribed.
Something to think about for a future release.


problems/test: add test case

Lot of problem/solutions does not have test cases .
Following is the coverage
sarath.pillai@OF-CLIENT-195  ~/projects/ctci   master  go test ./... -cover                                                                                 ✔  10031  11:18:26
ok  	github.com/sarathsp06/ctci/problems/datastructures/arrays-and-strings	(cached)	coverage: 100.0% of statements
ok  	github.com/sarathsp06/ctci/problems/datastructures/linkedlists	(cached)	coverage: 28.0% of statements
ok  	github.com/sarathsp06/ctci/problems/other/brackets	(cached)	coverage: 100.0% of statements
?   	github.com/sarathsp06/ctci/problems/other/jumpsteps	[no test files]
?   	github.com/sarathsp06/ctci/problems/other/largersmaller	[no test files]
?   	github.com/sarathsp06/ctci/problems/other/romannumber	[no test files]
?   	github.com/sarathsp06/ctci/problems/other/splitip	[no test files]
?   	github.com/sarathsp06/ctci/problems/other/triplet	[no test files]
ok  	github.com/sarathsp06/ctci/problems/other/watercapacity	(cached)	coverage: 100.0% of statements

bash-3.2$ go test ./... -cover | column -t
ok  github.com/sarathsp06/ctci/problems/datastructures/arrays-and-strings  (cached)  coverage:  100.0%  of  statements
ok  github.com/sarathsp06/ctci/problems/datastructures/linkedlists         (cached)  coverage:  28.0%   of  statements
ok  github.com/sarathsp06/ctci/problems/other/brackets                     (cached)  coverage:  100.0%  of  statements
ok  github.com/sarathsp06/ctci/problems/other/idleserver                   (cached)  coverage:  100.0%  of  statements
?   github.com/sarathsp06/ctci/problems/other/jumpsteps                    [no       test       files]
?   github.com/sarathsp06/ctci/problems/other/largersmaller                [no       test       files]
ok  github.com/sarathsp06/ctci/problems/other/largestprimefactor           (cached)  coverage:  100.0%  of  statements
?   github.com/sarathsp06/ctci/problems/other/romannumber                  [no       test       files]
?   github.com/sarathsp06/ctci/problems/other/splitip                      [no       test       files]
?   github.com/sarathsp06/ctci/problems/other/triplet                      [no       test       files]
ok  github.com/sarathsp06/ctci/problems/other/watercapacity                (cached)  coverage:  100.0%  of  statements


Step.Argument and null object pattern, Node base class

Once we mentioned that we want to use a kind of null object pattern for Step.Argument, ie have something like an EmptyStepArgument instance for those steps that neither has a DataTable nor a DocString argument. The current C# version is implemented like this, but the others not.
I would like to consolidate the implementations. Shall I remove the EmptyStepArgument?
This is somewhat related to the Node base class, that has only a Location. As the EmptyStepArgument does not have a location, it cannot be a proper Node.
In general: is the Node base class is part of the AST definition (ie has meaning) or just an implementation helper? In C# i did not need the base class as implementation helper, but I would introduce it if we see this as part of the AST.

We also use null object in cucumber-ruby-core - EmptyMultilineArgument.
We use a #match_locations? method in our AST currently, which gets around the problem of it not having a location.
I’d say that having a base class is an implementation detail, isn’t it? Isn’t the common interface more important to define?

I see why it would be handy to have a null-object for this, but it creates a problem for the JSON representation of the AST. If we add it, steps with no DocString or DataTable will have an empty object here (as apposed to nothing), and that doesn't make much sense. We can't have a Location for nothing.
So I think we should leave it out in Gherkin. It can still exist in Cucumber/SpecFlow, but that's a different bounded context (and different classes).
I think it's really important to not let Cucumber have any knowledge of the Gherkin AST - except perhaps for pretty printing.

OK. I remove the EmptyStepArgument it from .net

done

I don’t get that - why can’t you have both - a value object to represent nothing and no node in the JSON representation? Why are they mutually exclusive?

because the JSON generation (for Java at least) is done via reflection. If the argument field is null we end up with no argument property in the JSON (which is what we want), but if it has a value, we do. And it it's a null-object (presumably with no properties), it will be {"argument": {}} which is silly/ugly.

Well maybe reflection is not the right way to render the JSON then? I don’t like presentation concerns leaking into the object modelling.

I agree in principle, but sometimes pragmatism wins. If you feel strongly about it you know what to do @mattwynne.

Well I’d like to see how this pans out on the Ruby codebase first, then I’ll apply my opinions into the Java code :)


Unclear

Not clear that you cannot use powershell/cli for installing the diagnostic extension on the Linux section. You would need 'convert' the commands in order to use them. Far easier to just launch the Cloudshell

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

ID: 13249027-1524-dd0c-9d83-15a9733a3d19
Version Independent ID: bf8a2d1a-8d9b-228a-6919-16127f315d40
Content: Azure Compute - Linux Diagnostic Extension
Content Source: articles/virtual-machines/extensions/diagnostics-linux.md
Service: virtual-machines-linux
GitHub Login: @abhijeetgaiha
Microsoft Alias: agaiha

@aodugga we mention at the top of the article you are able to use PS, CLI or arm templates to deploy:

Can you explain further what more info you are looking for?

I will close this out but if you feel you need more information please just let me know.


Binary file

Hi, url not working, please put binary file to release.

I put the binary here: https://github.com/thegrandpoobah/dsgrab/releases/download/V1.6.0/DSGrab.1.6.0.exe .. this is the last working version that I have on hand which is from 2013, so there might be a couple hoops that you'll need to jump through to get it actually working. But hopefully its a starting point.


Update Deprcated Parameter 'artifactRepository'

When I upgrade Maven to 3.9.2, I got the following warning when run liberty tasks.
[WARNING] Parameter 'artifactRepository' is deprecated core expression; Avoid use of ArtifactRepository type. If you need access to local repository, switch to '${repositorySystemSession}' expression and get LRM from it instead.

@hantsy Which version of our plugin were you using? We recently made many changes to account for deprecations such as this one in anticipation of a Maven 4.0 release later this year. If you were not running with version 3.8 of our plugin, please retry with that version.

I found my project config used 3.7.1, will update and try again. Thanks.

I confirm it went away with 3.8 (already did that in the other issue).

I tried with Apache  maven 3.8.8, still i am getting the same issue.
Please guide me on this

I tried with Apache maven 3.8.8, still i am getting the same issue.
Please guide me on this

@Dineshkm23  What version of our plugin were you using? You gave the version of Apache maven.

Maven plugin : 3.5.1


In notebook, kernel goes back to idle, but the line counter in the notebook doesn't update.

I'm running visualization notebooks generated by the code in

https://github.com/hubmapconsortium/portal-ui/pull/2730

It works, but the [*] never updates to [1], [2], [3], etc. Maybe the Jupiter UI just isn't getting some signal from the backend?

Retrospective:
TLDR: Issue had to do with the websocket proxying that was built into the workspaces server.
The initial websocket proxying created two connections - one as a long-standing connection to the JupyterLab server to receive messages from that server and another one used to send messages from the front-end down to the JupyterLab server. This was causing messages to get lost, as JupyterLab was trying to send responses back to the connection that was proxying messages down to it.
Rather than having two connections, this was merged into one long-standing connection that is responsible for proxying both the messages that were being sent from the front-end down to Jupyter and the vice-versa.


Select with autocomplete appearance and choice filter shows choice value rather than label if only explicit languages set

See sample form: https://docs.google.com/spreadsheets/d/1cWmVtVyTh5PX-Cqc6IndSgnqGpSdmZXOf1N7TzEp_d8/edit#gid=0
Notice that the choice filter doesn't work either.
If I remove label::English (en) on all labels, it works as expected.
(I know this is totally niche but it had me really scratching my head and I figured y'all should know in case it's an easy fix.)

http://localhost:8005/?xform=autocomplete-cascade.xml


Setting list data by json..

Hi
Is it possible  to set list of Dir,File by  Json response  ? if possible how Json structure looks.

@ashokbg hi~ Actually I make it to display Dir and File by Json response in my project, you can check it out here. https://github.com/TellH/GitClub/blob/dev/app/src/main/java/tellh/com/gitclub/presentation/presenter/RepoSourcePresenter.java


[2.12.x] Update Scalatest to 3.0.8

This updates Scalatest library from 3.0.5 to 3.0.8.

@eed3si9n do you understand the Travis-CI failure?

@eed3si9n do you understand the Travis-CI failure?

@SethTisue it should be enough to merge https://github.com/scala/scala-seed.g8/pull/37 also in the target branch.

ah, okay, I've submitted a cherry-pick as #39

Ok, I'll rebase this one once #39 will be merged then.

#39 is merged

#39 is merged

Rebased. Now is all passing.


Add support for MonetDBLite for Python

Anything we can do to help? See https://www.monetdb.org/blog/monetdblite-for-python

o the sqlalchemy monetdb dialect has been a bit outdated now for a while. Let me chat with Martin.

cool thanks

initial support is there, but metadata is missing (and required) due to https://github.com/hannesmuehleisen/MonetDBLite/issues/195

An other driver issue related to this https://github.com/hannesmuehleisen/MonetDBLite-Python/issues/2


[5.9] Extract helper for establishing a new stacking context

Explanation: Resolves an issue where code listings may be obscured by solid gradient overlay in Safari when using the "Export as PDF..." feature.
Scope: Minor CSS change to tutorial steps and code listings that should be invisible (aside from fixing the mentioned Safari PDF issue)
Issue: rdar://104872474
Risk: Low, single CSS property change for code listings and steps that should have no visual impact but will fix PDF issue with Safari
Testing: Visually inspected all impacted elements to ensure there are no regressions. Verified that there is no longer a gradient overlay in code listings when using "Export as PDF..." in Safari or printing as a PDF
Reviewer: @dobromir-hristov
Original PR: #569

@swift-ci test


Prelease 1.69.1 with 64bit support on Android

@zxcpoiu I'm going to try to do it today with your script :-)

I saw it has been released! You are the man!
@saghul

@zxcpoiu Thank you! Your script worked beautifully!


[WIP] Update versions

Just updated version
Problem
N/A
Solution
N/A
Notes
N/A
Checklist

[X] Unit test all changes
[X] Update README.md if applicable
[X] Add [WIP] to the pull request title if it's work in progress
[ ] Squash commits that aren't meaningful changes
[X] Run sbt scalariformFormat test:scalariformFormat to make sure that the source files are formatted

@getquill/maintainers

[Current coverage][cc-pull] is 96.60%

Merging [#350][cc-pull] into [master][cc-base-branch] will decrease coverage by -<.01%

@@             master       #350   diff @@
==========================================
  Files           113        113
  Lines          2119       2117     -2
  Methods        2079       2086     +7
  Messages          0          0
  Branches         36         29     -7
==========================================
- Hits           2049       2045     -4
- Misses           70         72     +2
  Partials          0          0

File ...ndraSyncSource.scala (not in diff) was modified. more

Misses +2
Partials 0
Hits -4

File ...dbc/JdbcSource.scala (not in diff) was modified. more

Misses +1
Partials 0
Hits -1

File ...atementBuilder.scala (not in diff) was modified. more

Misses -1
Partials 0
Hits +1

Powered by Codecov. Last updated by [786373f...c09b2c3][cc-compare]
[cc-base-branch]: https://codecov.io/gh/getquill/quill/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/getquill/quill/compare/786373fe1a53ea3875387fa60e6472f0981fc3b3...c09b2c3f5fe4c452877ca0bef13e74376cc28115
[cc-pull]: https://codecov.io/gh/getquill/quill/pull/350?src=pr

:+1:

👍