Add Tip to Global Network Policy Section

Description
When I first started working with Calico, I ran into an issue where new network policies I was creating were not behaving as they should. I could not find any network policy that was causing this, but later found out that there was a lingering Global Network Policy that was behind the issue. This tip could have saved me the time I spent trying to address this issue.
Related issues/PRs
Todos
Release Note

Deploy preview for calico ready!
Built with commit 311f668d2f5ea4b8619b1e886530a1ff5e0448b8
https://deploy-preview-3460--calico.netlify.app

Thanks @PythonSyntax1 !


Date & Time controls: Overview cut off along with broken format in API Reference

OpenUI5 version
Since https://github.com/SAP/openui5/commit/49782b00fbec7394deb32254681e708d129589ff
URL (minimal example if possible)

https://sdk.openui5.org/nightly/#/api/sap.m.DatePicker
https://sdk.openui5.org/nightly/#/api/sap.m.DateRangeSelection
https://sdk.openui5.org/nightly/#/api/sap.m.DateTimePicker
https://sdk.openui5.org/nightly/#/api/sap.m.TimePicker

Steps to reproduce the problem

Open above API Reference pages.
Compare each page with the actual JSDoc comments of the respective control in the source code.
Notice how the Overview section in the control's API Reference page is incomplete.

The rest of the description with examples is mentioned in the Constructor section with broken format.
Analysis
It looks like the JSDoc tag @example is not compatible within

 tags even with
.
For example:

Given the following
...
 that contains @example: https://github.com/SAP/openui5/blob/7660bd4b7220dcc34aa44248c8a3ff2dd1232df9/src/sap.m/src/sap/m/DatePicker.js#L112-L151
The generated https://sdk.openui5.org/nightly/test-resources/sap/m/designtime/apiref/api.json contains the following invalid "description" HTML node which ends with
 despite the open
:

Use the value property if you want to bind the DatePicker to a model using the sap.ui.model.type.Date

Any other information? (attach screenshot if possible)
Rather than having specific code snippets with @example, it would be better if the documentation topic "Dates, Times, Timestamps, and Time Zones" could be promoted and highlighted in the API reference, which I personally consider crucial information for every UI5 dev dealing with date & time related controls.

These are wrong usages of the @example tag. @example in JSDoc is a so called block tag which means, when it appears, it automatically ends the previous block tag (here: the implicit @description tag).
In other words: @example tags cannot be nested into the description of an API.
The remainder of the description therefore becomes part of the content of the last @example tag. The SDK then shows all examples as part of the constructor.

Hello @boghyon,
Thank you for sharing this finding. I've created an internal incident<PHONE_NUMBER>. The status of the issue will be updated here in GitHub.
Regards,
Siyana


Primer Sequences

Thank you for an awesome tool. Works fantastically (using the docker container)!
Are the primer sequences that are being used publicly available? I'm currently using Thermofisher primers which cover a slightly smaller area.

Sure, we're using primers from the ARCTIC set. The protocol is published at https://www.protocols.io/view/sanger-sequencing-of-a-part-of-the-sars-cov-2-spik-bsbdnai6. We have submitted a preprint with some validation data as well, but I don't think that's live yet.

the preprint is online now: https://www.medrxiv.org/content/10.1101/2<IP_ADDRESS>52266v1
the protocol can be found here: https://www.protocols.io/view/sanger-sequencing-of-a-part-of-the-sars-cov-2-spik-bsbdnai6
best,
Tue

Closing the issue now, but let us know if there's any questions about either the lab or the software side of this.


[DatePicker] Scroll jump to top when opening with selected value

As I have seen in changelog of v5 ALPHA 26 the scroll jump issue has been fixed but its not completed fixed yet.
Ya, Its fixed for component [ DesktopDatePicker ] but not for component [DatePicker]
Current Behavior 😯

Expected Behavior 🤔
Should not jump.
Steps to Reproduce 🕹
Steps:

Visit https://next.material-ui.com/components/date-picker/
Scroll down then on Responsive datepicker click on datepicker icon button.

Reference PR #25010 by Mr. @eps1lon

@eps1lon have we get any solutions ?

@eps1lon have we get any solutions ?

No. Once this is fixed, you'll be notified if you've subscribed to this issue. Otherwise see Issue Bumping.


Parsing in streaming-like way

Hi. As my idea for solving #16 I though that I could listen the Matched event of line break terminal and keep track of current line and last line end's position in stream. Unfortunately I learned that events are fired only after whole text has been parsed. Is there an easy way to monitor the parsing process as it goes?
I was thinking about this also because that way it souldn't be necessary to wait until all text has been parsed before handling the matches. Or is that not such a good idea :question:

To be involved in the parsing cycle, you would derive from the Parser (or a subclass) and override the InnerParse() method. It is done this way since adding an event for the parser at this stage would considerably degrade performance.
However, the reason why you wouldn't want to handle the matches while parsing/tokenizing (typically) is when there is a failure at the end of a group.. for example, to do as you suggest, you could implement it this way:
using System;
using Eto.Parse;
using Eto.Parse.Parsers;

namespace TestEolParsing
{
	static class Program
	{
		class MyEolTerminal : EolTerminal
		{
			public int LineCount {get;set;}
			protected override int InnerParse(ParseArgs args)
			{
				var ret = base.InnerParse(args);
				if (ret > 0)
				{
					// found a newline!
					LineCount++;
				}
				return ret;
			}
		}

		public static void Main(string[] args)
		{
			var eol = new MyEolTerminal();
			var grammar = new Grammar(+("some text" & ~((eol & "second text") | (eol & "third text"))));

			var match = grammar.Match(@"some text
second textsome text
third textsome text
text that doesn't match");

			Console.WriteLine("Line Count: {0}, Result: {1}", eol.LineCount, match.ErrorMessage);

			return;
		}
	}
}

The Problem is, this will return a line count of 5, where the error is actually on line 3.  This is because the eol is successfully parsed multiple times, but then the group the eol is in fails (the second text group and twice for the last line) so they can't be counted.
Therefore, it is still best to handle the matches after the parsing stage to ensure you are only counting valid matches.


Removed unnecessary properties

Coverage decreased (-0.3%) to 51.774% when pulling 3858e4a93c3ccad23e71b456f4f442a7cf4f9534 on compae:bug/configuration-files into 39d217a1d27c475a64963a2aec8c8aac501f09fa on Stratio:master.

Coverage decreased (-0.3%) to 51.774% when pulling 278c4279d0d1364aec9e28d3b49f4f05b4be4931 on compae:bug/configuration-files into 185fa9b8c93dfd1399efd1360d3ab21b8e9e7da6 on Stratio:master.


Update case-study-capturing-and-creating-content-for-holotour.md

about the author adjustments

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/case-study-capturing-and-creating-content-for-holotour.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Update SCSS grammar

I'm not sure if this is the right repository, but there are several issues with how vscode handles linting and syntax highlighting of scss files e.g.:

https://github.com/microsoft/vscode/issues/81943
https://github.com/microsoft/vscode/issues/78974

I did see a reference to atom/language-sass  in the source code (for vscode), but that repository has been dead for over a year.

Since there are already issues for this (microsoft/vscode#81943, microsoft/vscode#78974) those are the best ones to up-vote. If you have other specific issues, please feel free to open them in the microsoft/vscode repository. If there is a better supported grammar than atom/language-sass then please do mention in! I am happy to consider switching to a better maintained grammar.


snowflake_object_parameter resource import has missing fields

Terraform CLI and Provider Versions
Terraform v1.7.1
on darwin_arm64
+ provider registry.terraform.io/hashicorp/aws v5.31.0
+ provider registry.terraform.io/hashicorp/local v2.1.0
+ provider registry.terraform.io/snowflake-labs/snowflake v0.84.1

Terraform Configuration
terraform {
  required_version = "1.7.1"
  required_providers {
    aws = {
      source  = "hashicorp/aws"
      version = "5.31.0"
    }
    local = {
      source  = "hashicorp/local"
      version = "2.1.0"
    }
    snowflake = {
      source  = "Snowflake-Labs/snowflake"
      version = "0.84.1"
    }
  }
}

provider "aws" {
  region              = ""
  allowed_account_ids = []
  profile             = ""
}

provider "snowflake" {
  role    = ""
  account = ""
}

Expected Behavior

Documentation for importing snowflake_object_parameter should have information what <object_identifier> means.
terraform plan should not show snowflake_object_parameter.test must be replaced after doing `terraform import.

Actual Behavior
terraform state show snowflake_object_parameter.test:
resource "snowflake_object_parameter" "test" {
    id    = "DATA_RETENTION_TIME_IN_DAYS❄TABLE❄dbName.schemaName.tableName"
    value = "1"
}

Steps to Reproduce

terraform import snowflake_object_parameter.test DATA_RETENTION_TIME_IN_DAYS❄TABLE❄dbName.schemaName.tableName
terraform plan should show nothing

How much impact is this issue causing?
High
Logs
No response
Additional Information
No response

Hey @afterdesign. Thanks for reaching out to us.
We have the redesign of the existing resources on our roadmap (https://github.com/Snowflake-Labs/terraform-provider-snowflake/blob/main/ROADMAP.md#19012024-roadmap-overview). I will bump snowflake_object_parameter's priority.


Gitlab renames Master to Maintainer

In gitlab release 11 gitlab will rename the Master access level to Maintainer.

https://gitlab.com/gitlab-com/www-gitlab-com/blob/release-11-0/source/images/11_0/master-to-maintainer.png
See the discussion here:
https://gitlab.com/gitlab-org/gitlab-ce/issues/42751
Can we provide a PR for this?

@jgoldhammer
MAINTAINER access level has been added and MASTER has been deprecated. The changes have been merged with the master branch but an official release has not been done,  will do a release when there are additional fixes/updates to release.

This has been released in 4.8.25.


Test coverage: neuropeptide test

We should have a test that verifies that we get "True" out of this.  cc: @gsarma @travs
Related to issue #105
  >>> n = P.Neuron()
  >>> n.neuropeptide("TH")
  neuropeptide=`TH'

  >>> s = set(x.name() for x in n.load())
  >>> s == set(['CEPDR', 'PDER', 'CEPDL', 'PDEL', 'CEPVR', 'CEPVL'])
  True

Added test with @travs.  Will commit as soon as I have commit privileges.

great!  please use the pull request mechanism for now


android-studio-{canary,dev}: properly fix interpreter paths

Motivation for this change
This makes the native debugger on android-studio work again.
I have no idea how this functioned beforehand, but it seems I messed up the scope of the dontPatchShebangs line. The fact that this gets triggered conditionally, and for example not on the beta or stable branches of AS it really confusing to me. Either way, it now works again on all branches.
Fixes: #117985
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase notes)

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

[X] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 132423 at 0d315b7f run on x86_64-linux 1

4 packages built successfully:

android-studio (androidStudioPackages.stable)
androidStudioPackages.beta
androidStudioPackages.canary
androidStudioPackages.dev

/marvin opt-in
/status needs_reviewer


Testcafe Browser Tools.app tries to execute even when there is no interaction with a local browser

What is your Scenario?
Hi,
I am trying to run my testcafe tests inside of AWS Device Farm so that I can run my tests on real mobile devices, specifically iOS Safari.
AWS Device Farm uses Appium to handle the communication and send instructions to the remote browser.
I have a test working on my Mac where I am running Appium and using this selenium browser provider my test gets executed and in iOS Simulator I can see the test run and behave correctly.
This is not the case inside of AWS Device Farm.
This problem might be related to this one about System Integrity Protection (SIP) on Mac OS but I can't seem to figure out if this is just a limitation of testcafe and I am out of luck or if there is some workaround. I saw this answer in Stack overflow but I can't use this since the devices in Device farm only have access to the public internet and can't communicate with the server running Appium or a remote testcafe session.
What is the Current behavior?
In AWS Device Farm, when calling createTestCafe I see the following exception being thrown
NativeBinaryHasFailedError: The find-window process failed with the 1 exit code.
The application /Users/device-farm/.testcafe-browser-tools/TestCafe Browser Tools.app/Contents/MacOS/testcafe-browser-tools cannot be opened for an unexpected reason, error=Error Domain=NSOSStatusErrorDomain Code=-10826 "kLSNoLaunchPermissionErr: User doesn't have permission to launch the app (managed networks)" UserInfo={_LSFunction=_LSLaunchWithRunningboard, _LSLine=2508, NSUnderlyingError=0x7f9d7ed2c260 {Error Domain=RBSRequestErrorDomain Code=5 "Launch failed." UserInfo={NSLocalizedFailureReason=Launch failed., NSUnderlyingError=0x7f9d7ed2af50 {Error Domain=OSLaunchdErrorDomain Code=125 "Domain does not support specified action" UserInfo={NSLocalizedFailureReason=Domain does not support specified action}}}}}
    at ChildProcess.<anonymous> (/private/tmp/scratch0ockPu.scratch/test-package2ya78f/node_modules/MangaWebReaderTestCafe/node_modules/testcafe-browser-tools/src/utils/exec.js:74:24)
    at ChildProcess.emit (events.js:314:20)
    at ChildProcess.EventEmitter.emit (domain.js:483:12)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:276:12)
    at Process.callbackTrampoline (internal/async_hooks.js:126:14)

What is the Expected behavior?
I would expect the test to attempt to execute since it shouldn't need to interact with any local browser. The docs on npm for testcafe-browser-tools give me the impression that the application is only needed when trying to interact with a browser installed locally on the platform testcafe is executing on.
In the case of a remote browser, I would expect this application to not run at all
What is your public website URL? (or attach your complete example)
https://www.google.com
What is your TestCafe test code?
import { Selector } from 'testcafe';

fixture`Test Google`.page("https://www.google.com");
test.only('Check Title', async (t) => {
  await t.expect(Selector('#main').visible).eql(true);
});

Your complete configuration file
// Detailed explanation of settings can be found at
// https://devexpress.github.io/testcafe/documentation/using-testcafe/configuration-file.html
{
  "concurrency": 1,
  "skipJsErrors": true,
  "skipUncaughtErrors": true,
  "pageLoadTimeout": 15000,
  "selectorTimeout": 5000,
  "assertionTimeout": 3000,
  "quarantineMode": true,
  "reporter": [
    {
      "name": "spec"
    }
  ],
  "src": "src/tests/**/*.ts"
}

Your complete test report
npm run device-farm-test
> testcafe selenium:safari:iOS

Try to use selenium browser
BrowserProfile: [safari]
browser: [safari]
version: []
platform: [iOS]
proxy: []
platform_name: [undefined]
device_name: [undefined]
os_version: [undefined]
 Running tests in:
 - Safari 15.4 / iOS 15.4

 Test Google
 ✓ Check Title

 1 passed (3s)
BUILD SUCCEEDED

Screenshots
N/A
Steps to Reproduce

Install XCode
Install Appium npm install -g appium
Run Appium on port 4444 appium -p 4444
Execute testcafe using selenium browser testcafe selenium:safari:iOS
Validate test runs and passes on your local mac
Taking the same tests and deploying it to AWS Device Farm fails with the above errors

Excluding steps involved in setting up device farm since I don't think it is necessary. I can add detailed instructions though if you need them.
TestCafe version
1.18.5
Node.js version
v12.20.0
Command-line arguments
testcafe selenium:safari:iOS
Browser name(s) and version(s)
Safari
Platform(s) and version(s)
iOS 15.02
Other
No response

Not sure if this is also related to https://github.com/DevExpress/testcafe/pull/7028

Skipping the permissions check like what is done in this other issue seems to resolve the error.
The tests still fail but that seems to be because the devices can't connect to the mac that is orchestrating the tests. Looking at the video provided by device farm I can see it try to connect to <IP_ADDRESS> and then fail.
I will close this task since the root cause is the same as  #7028. The remaining issue of actually getting tests running on DeviceFarm Mobile at this time isn't due to any issue in TestCafe


Update release_tests.py

This commit modifies release_tests.py to invoke each of the tests in the list of tests.
Resolves #996
Signed-off-by: Trang Dang<EMAIL_ADDRESS>

@trangttdang can you update your commit message such that it has a body and a Signed-off-by like your previous commit?

This commit modifies release_tests.py to invoke each of the tests in the list of tests.
Resolves #996
Signed-off-by: Trang Dang<EMAIL_ADDRESS>
Do git commit --amend and then update the commit message with this:
Update release_tests.py

This commit modifies release_tests.py to invoke each of the tests
in the list of tests.

Resolves #996

Signed-off-by: Trang Dang<EMAIL_ADDRESS>
Then git push -f origin <your dev branch>

Closing this as #1017 was opened to replace it.


The browser stays on the testing page when the test is finished

import { Selector, ClientFunction } from 'testcafe';

fixture `Google`.page `http://google.com`;

test('success', async t => {
    await t
    .typeText('#lst-ib', 'devexpress')
    .pressKey('enter');

    const getDxLink = Selector('a').with({ text: 'DevExpress' });
    const dxLink = await getDxLink();

    await t.click(dxLink);
});

@AlexanderMoskovkin Is this caused by https://github.com/DevExpress/testcafe-hammerhead/issues/838? Or it's a separate issue?

it's another problem


Remove I2C0 initialization code from the EE-02 BSP.

This code assumed that pins 26 and 28 would be used for I2C, but this is not dictated by the EE-02 design.  It does not belong in the generic BSP.

Furthermore, prior to removing it herein, the call to hal_i2c_init seems to have had no effect.  It was still necessary to call hal_i2c_init from application code.  Out of curiosity, can this be explained?


RC > Remove runtime modules

Codecov Report

Merging #483 into rc-2.0.0 will increase coverage by 0.57%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           rc-2.0.0   #483      +/-   ##
==========================================
+ Coverage     99.42%   100%   +0.57%
==========================================
  Files            72     54      -18
  Lines          1922   1670     -252
  Branches        302    244      -58
==========================================
- Hits           1911   1670     -241
+ Misses           11      0      -11

Flag
Coverage Δ

#builder
100% <100%> (ø)
:arrow_up:

#runtime
100% <100%> (+3.92%)
:arrow_up:

#srcapp
100% <100%> (ø)
:arrow_up:

Impacted Files
Coverage Δ

config/axe/axe.config.js
100% <ø> (ø)
:arrow_up:

config/karma/shared.karma.conf.js
100% <ø> (ø)
:arrow_up:

lib/sky-pages-route-generator.js
100% <ø> (ø)
:arrow_up:

config/webpack/common.webpack.config.js
100% <ø> (ø)
:arrow_up:

cli/generate.js
100% <ø> (ø)
:arrow_up:

src/app/app.component.ts
100% <100%> (ø)
:arrow_up:

cli/build-public-library.js
100% <100%> (ø)
:arrow_up:

runtime/bootstrapper.ts
100% <100%> (ø)
:arrow_up:

cli/e2e.js
100% <100%> (ø)
:arrow_up:

cli/utils/ts-linter.js
100% <100%> (ø)
:arrow_up:

... and 5 more

Continue to review full report at Codecov.

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


fix: remove redundant updateTransform in SpineSystem

Please check if the PR fulfills these requirements

[ ] The commit message follows our guidelines
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Bug fix
What is the current behavior? (You can also link to an open issue here)
Spine's updateTransform will be called twice in one frame.
What is the new behavior (if this is a feature change)?
Spine's updateTransform will only be called in container's children update.
Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
No
Other information:

When I debugger these codes, I didn't find that updateTransform would be called twice.


MWPW-159137-Sitemap for batch 27

Set up sitemap for batch 27

Resolves: MWPW-159137
Test URLs:

Before: https://stage--cc--adobecom.hlx.live/careers?martech=off
After: https://sitemap--cc--adobecom.hlx.live/careers?martech=off

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 89.72%. Comparing base (4e8b5df) to head (1f8f9fb).
Report is 92 commits behind head on stage.

Additional details and impacted files
@@            Coverage Diff             @@
##            stage     #436      +/-   ##
==========================================
- Coverage   90.26%   89.72%   -0.55%
==========================================
  Files          17       23       +6
  Lines        2301     4349    +2048
==========================================
+ Hits         2077     3902    +1825
- Misses        224      447     +223

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

Pr can be verified once its merged to production and pages are published


Fix parenting for clocking

Fix parenting for clocking
For AllXXX, use the clocking information available in the definition,
but for the elaborated side, clone the same so parenting works
correctly.

@hs-apotell please fix conflicts

@hs-apotell , fixed version.


Can't enable SPO feature

Set-SPOTenant : Setting the property EnableAzureADB2BIntegration is not supported by your version of the service.

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

ID: 71747405-a38b-c475-7915-bb10242b689f
Version Independent ID: ccf67a68-63d7-9d04-69c7-728fabb3be24
Content: SharePoint and OneDrive integration with Azure AD B2B (Preview)
Content Source: SharePoint/SharePointOnline/sharepoint-azureb2b-integration-preview.md
Service: sharepoint-online
GitHub Login: @MikePlumleyMSFT
Microsoft Alias: mikeplum

Same here. On Targeted release, in Europe.

The feature is currently rolling out. You should see it available on your tenants within the next ten days.

Thanks for the clarification @MikePlumleyMSFT

Thanks, Mike. I read this page before I saw the announcement in the message center. A notification header on the page that it's still rolling out would be nice (but not necessary).

@Thank you for submitting feedback. We understand that this issue has been resolved.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.

@officedocsbot close

Would it be possible to standardise the message that is returned when the feature isn't available? For other SPO features, we've previously seen messages about a given feature is experimental which immediately called out it wasn't available in that particular tenant.


Bindgen generates different padding when cross-compiling based on host system

As previously discussed on IRC (https://mozilla.logbot.info/servo/20180821#c15200737-c15201117), building rust-mozjs on macOS when cross-compiling to Android (using NDK 12) causes the ForOfIterator to contain a padding value that is not present on our Linux->Android builds.

Unless https://github.com/servo/mozjs/pull/148 is not a fix for this, I think this is invalid. The problem is that the toolchain includes were not right, because the flags from:
https://searchfox.org/mozilla-central/rev/c3fef66a5b211ea8038c1c132706d02db408093a/build/moz.configure/android-ndk.configure#324
Weren't being accounted for.

Closing for now, but let's reopen if there's a deeper issue here.


Breast motion off menu item does not work

Something about the back_quote key code? Keyboard layouts?

works fine now?!


Add support for sending per-statement labels

The driver will now accept per-statement labels in addition to the per-connection labels. It checks to make sure it doesn't send more than the max of 64 labels per query, but doesn't check anything else about label validity. I have an accompanying PR for Looker to make use of this. This introduces an auxiliary API that is not typical of JDBC drivers and would not normally be used by "typical" users of this driver. As such, it is end-to-end tested in helltool, but not here.

Ya the idea was to use that method in an end-to-end test in helltool. Anyway, I've added a test, which also tests to make sure that the URL labels are correctly parsed and added to the query as well, which is not covered in the helltool test.


[meta] Implement all styles that are being dropped from JSHint

As of JSHint 3.0, all style conformance rules are being dropped: https://github.com/jshint/jshint/issues/1339 . JSCS is the perfect tool to reimplement these checks. Hopefully, all can be implemented, as this will help a lot with adoption of JSCS as the defacto style checker.
Here's all the options being dropped:

[x] gcl: not necessary as it was a relaxing option for "white"
[x] white: already covered by the various JSCS options
[x] onevar: already covered by existing JSCS options
[x] PR https://github.com/mdevils/node-jscs/pull/115 sub: http://www.jshint.com/docs/options/#sub
[x] PR https://github.com/mdevils/node-jscs/pull/114 laxbreak: http://www.jshint.com/docs/options/#laxbreak https://github.com/mdevils/node-jscs/issues/104
[x] PR https://github.com/mdevils/node-jscs/pull/113 laxcomma: http://www.jshint.com/docs/options/#laxcomma https://github.com/mdevils/node-jscs/issues/103 && https://github.com/mdevils/node-jscs/issues/66
[x] PR https://github.com/mdevils/node-jscs/pull/110 trailing: http://www.jshint.com/docs/options/#trailing https://github.com/mdevils/node-jscs/issues/105
[x] PR https://github.com/mdevils/node-jscs/pull/118 quotmark: http://www.jshint.com/docs/options/#quotmark https://github.com/mdevils/node-jscs/issues/106
[x] PR https://github.com/mdevils/node-jscs/pull/119 newcap: http://www.jshint.com/docs/options/#newcap
[x] PR https://github.com/mdevils/node-jscs/pull/120 nomen: http://www.jshint.com/docs/options/#nomen
[x] PR https://github.com/mdevils/node-jscs/pull/121 camelcase: http://www.jshint.com/docs/options/#camelcase
[x] PR https://github.com/mdevils/node-jscs/pull/122 noempty: http://www.jshint.com/docs/options/#noempty
[x] PR https://github.com/mdevils/node-jscs/pull/123 immed: http://www.jshint.com/docs/options/#immed
[x] PR https://github.com/mdevils/node-jscs/pull/134 smarttabs: https://github.com/mdevils/node-jscs/issues/60 http://www.jshint.com/docs/options/#smarttabs
[x] PR https://github.com/mdevils/node-jscs/pull/135 maxlen: http://www.jshint.com/docs/options/#maxlen
[x] PR https://github.com/mdevils/node-jscs/pull/137 indent: https://github.com/mdevils/node-jscs/issues/60 http://www.jshint.com/docs/options/#indent
[x] PR https://github.com/mdevils/node-jscs/pull/142 multistr: https://github.com/mdevils/node-jscs/issues/141 http://www.jshint.com/docs/options/#multistr

...

Great!


EHN: Cut duplicate codes via the factory design pattern

I saw there have a lot of duplicate codes. And they both have the same structure.
Maybe we could extract the same codes via the factory design pattern?
Duplicate codes:

https://github.com/Zsailer/pandas_flavor/blob/f9308140d559ef1cb80587dedf6a7e32ca1f0b67/pandas_flavor/register.py#L19-L26
https://github.com/Zsailer/pandas_flavor/blob/f9308140d559ef1cb80587dedf6a7e32ca1f0b67/pandas_flavor/register.py#L38-L47
https://github.com/Zsailer/pandas_flavor/blob/f9308140d559ef1cb80587dedf6a7e32ca1f0b67/pandas_flavor/xarray.py#L13-L21

The prototype of this idea.
It could work for pandas-like object, such as pandas.DataFrame, pandas.Series, pandas.Index, geopandas.GeoDataFrame, and geopandas.GeoSeries.
def register_method_factory(register_accessor):
    @wraps(register_accessor)
    def decorator(method):
        def method_accessor(pd_obj):
            @wraps(method)
            def wrapper(*args, **kwargs):
                return method(pd_obj, *args, **kwargs)

            return wrapper

        # Register method as pandas object inner method.
        register_accessor(method.__name__)(method_accessor)

        # Must return method itself, otherwise would get None.
        return method

    return decorator

# or register_dataframe_method = register_method_factory(register_dataframe_accessor)
@register_method_factory
def register_dataframe_method(method):
    """Docstring"""

    return register_dataframe_accessor(method)

@register_method_factory
def register_dataarray_method(method):
    """Docstring"""

    return register_dataarray_accessor(method)

@Zeroto521 this feels like a great way to enable arbitrary dataframe extensions, perhaps even to other up-and-coming dataframe libraries. How would you like to move forward here?

This feature already finished in https://github.com/Zeroto521/my-data-toolkit/blob/main/dtoolkit/accessor/register.py
And use this wrapper to decorate geopandas.GeoDataFrame (https://github.com/Zeroto521/my-data-toolkit/blob/main/dtoolkit/geoaccessor/register.py)
PS: I need to focus on my work so that any open-source activities will get late.


Add a "share" button to the active tab

This PR replaces #32 which had got a bit muddled with merges and so on. See that PR for discussion (mainly re IE8 support).

Currently deployed to staging.

Gorgeous ❤


More Iris compatibility + update to Loom 0.11

The big change is changing the beginFrame start from setupRender to compileChunks.

major changes needed


(feature) display response time for successful requests

Display the response time in milliseconds in the ResponseTab similarly to the Postman
Address some linting issues

Screenshot:

closes #7

Thanks for contributing @thuoe! Merging this - will push a new release soon 😄


fix: missing nil check in open_at_point()

Without the check, attempting to open a link like [[file:./file.org::*Heading]] outside of any heading would cause a Lua error.
I'm not super proficient in Lua dev, so if there's any unit test I should add for this, I'd need some guidance.
In any case, thank you for working on this great plugin! :slightly_smiling_face:

Looks good to me, thanks!


Multi configuration tests via compose

This contains a compose script which will start up a v1 and v2 registry with an nginx configuration for multiple tls and configuration scenarios. Currently there is no automated framework based on this configuration but it is intended to build client automation tests using these configurations. This also currently uses localhost which makes testing secure scenarios within the Docker engine a challenge without a code change (disabling <IP_ADDRESS> as insecure).
To test simply tag an image with the appropriate port number of the configuration which is desired to be tested, the currently available tests are...
# 5000  No tls or auth, v2 + v1
# 5001 No tls or auth, v1 only

# TLS Configuration chart
# Username/Password: testuser/passpassword
#      | ca  | client | basic | notes
# 5440 | yes | no     | no    | Tests CA certificate
# 5441 | yes | no     | yes   | Tests basic auth over TLS
# 5442 | yes | yes    | no    | Tests client auth with client CA
# 5443 | yes | yes    | no    | Tests client auth without client CA
# 5444 | yes | yes    | yes   | Tests using basic auth + tls auth
# 5445 | no  | no     | no    | Tests insecure using TLS
# 5446 | no  | no     | yes   | Tests sending credentials to server with insecure TLS
# 5447 | no  | yes    | no    | Tests client auth to insecure
# 5448 | yes | no     | no    | Bad SSL version

Usage within Docker
$ docker pull hello-world
$ docker tag hello-world localhost:5000/hello-world
$ docker push localhost:5000/hello-world

disabling <IP_ADDRESS> as insecure

I think it would make sense to disable <IP_ADDRESS> as insecure as soon as any --insecure-registry flag is added (i.e., don't use the defaults if a user specified another value); this would allow you to override this behavior for the tests.
Pehaps create a feature request for that in docker?

@dmcgowan @stevvooe I would really love it if we could merge this before you cut the RC.  This isn't documentation per se but Derek's test illustrate configurations that I don't have the bandwidth to document this go round.  It would be useful to tell users --- look here for now and have this available.

I'm okay merging it.
@dmcgowan Could you add a README with the content of the issue description?

Yes I can add a readme
Also I added it to contrib/docker-integration for lack of a better name. My primary use case for this is integration testing with Docker, so the name makes sense to me, but that is not an accurate description of what this is.

@dmcgowan That is fine. We'll figure out where this goes in the future.

Added README, removing WIP label as well.  @moxiegirl will you review the README and let me know if there is something else you want added. Also feel free to add later or request additional test cases.

LGTM!

Thanks guys! @dmcgowan I'll give the README a review too.


fix image name for dogstatsd

What does this PR do?
fix document
Motivation
I pulled deprecated image

Thank you Himura-san!


Discord

Can you also tell how to add it to discord

Not under the scope of this project, sorry. Feel free to take a look at the forks of this project in the Insights tab.


Bump icons to 1.8.1

🙏 Thanks for this.
I'd like to check the docs once more before releasing, but should be ready to package up 5.3.1 soon.

The only docs suggestion I'd have for the docs is to switch the default from list-group-full to collapse  since the number of topics is now so large that the bottom of the menu now goes below the fold.

Using collapse looks closer to the Bootstrap style and ensures that all open topics can be displayed without scrolling.

Also this line  is incorrect.  The link doesn't work until the docs are merge to master, but the name of the file is wrong.
<xref
-   href="https://github.com/infotexture/dita-bootstrap/blob/master/includes/bootstrap.icons.hdr.xml"
+   href="https://github.com/infotexture/dita-bootstrap/blob/master/includes/bootstrap.icons.hdf.xml"


"Invalid or expired CSRF token" in HTTP tests

Package version
8.1.0
Describe the bug
When using the withCsrfToken() method the CSRF token does not seem to actually get set (correctly).
During the test run the following output is shown:
[14:42:06.195] WARN (9766): Invalid or expired CSRF token
    request_id: "i53xqlaaus7ylpclo56moity"
    x-request-id: "i53xqlaaus7ylpclo56moity"

It seems like a CSRF token is set in the request headers, but possible it's an invalid one?

In the reproduction repo I show a minimal example that shows the issue.
Thank you for looking into this.
Reproduction repo
https://github.com/casperboone/adonis-csrf-test-bug/commit/02850bd586e42df7f15eabd6028ee476839fc081

Fixed in https://github.com/adonisjs/shield/releases/tag/v8.1.1
Please also consult the updated docs. https://github.com/adonisjs/v6-docs/commit/43ba644919382aa8bf2aec7404204bdef6623067


Added generation GraphQLClient.fsi

@Zaid-Ajaj can I use FusionTasks NuGet to simplify code?
task computation expression does not require Async.AwaitTask, async computation expression requires it.
With FusionTasks we can use the same code and simplify generation

@xperiandri task is already included when asyncReturnType: "task" in which case the Ply package is included

I need https://www.nuget.org/packages/FSharp.Control.FusionTasks/2.2.0

You mean you want to use FusionTasks to simplify the async implementation? I don't really think adding a dependency just for Async.AwaitTask is worth it.
If you need FusionTasks in your application, why not simply add it in your project that references the generated Snowflaqe client?

It worse because now we have 2 generator functions everywhere async/task appear which complecates the code a lot and adds more chances for mistake

I agree it is a bit complicated but it makes the result nicer for users who want Task<'T> as the output of the client function, not just for simplifying the implementation details.
I think implementing error handling option from #29 will be the last hurdle and it's still doable IMO
Do you have any other features might be impacted because of the task/option?

I don't know why could people not use https://www.nuget.org/packages/FSharp.Control.FusionTasks/2.2.0
It is one of the first NuGet packages I install to any F# project. Adding Async.AwaitTask to every async .NET BCL call is crazy...

The same as manually calling OnPropertyChanged instead of installing PropertyChanged.Fody and make it done automatically

I don't know why could people not use https://www.nuget.org/packages/FSharp.Control.FusionTasks/2.2.0
It is one of the first NuGet packages I install to any F# project. Adding Async.AwaitTask to every async .NET BCL call is crazy...

Users of Snowflaqe don't have to call Async.AwaitTask, it is just the generated code. You can always install FusionTasks into your own projects whenever you need it.
Sticking with Ply for now as a dependency of the generated projects because it gives a nice task CE to work with when tasks are preferred.

Sticking with Ply for now as a dependency of the generated projects because it gives a nice task CE to work with when tasks are preferred.

One of the reasons I made the task change was that 'fits' more when the actual 'async' functionality is using tasks anyway (i.e. HttpClient and the Json libs are task based), but then I was also having a go at inserting the generated lib into an existing C# stack and the tasks returned from Ply just work directly there which is nice for interop.
(I'm a massive beginner with F# and had never heard of FusionTasks, but it looks to me like task { } is going to end up part of FSharp Core at some point, at which point the issue and the requirement for 3rd party libs might just go away? (or maybe i'm being hopeful looking at how long that question has been running for!))

Hi @anthony-mi what's the status of this PR? Any plans to continue working on?
@xperiandri if there are issues or discussions outside of this PR, please consider opening a separate issue to discuss it there

@Zaid-Ajaj Anthony decided to exit from the project. The other developer will pick up the work in several weeks.

Thanks for the update @xperiandri does this mean this PR can closed to start all over later on or should I keep it open for now?

Keep it open for now. I will let you know

@Zaid-Ajaj we have discovered that in order to have FSI we need to declare all public members within it too

So that we think of removing FSI and rewriting the client class without the default constructor

So that we think of removing FSI and rewriting the client class without the default constructor

Sounds like a good idea for now. Proper FSI support can be added in another PR

We will have a look closer soon. Removing FSI looks easier to implement an maintain for now


Error Opening Project created by old version in the new version

This issue was imported from CodePlex
jirikadlec2 wrote 2011-12-16 at 21:17
I have tried opening a project that was created by an older DotSpatial version (May 2011) with the new version 1.0 RC 2. Opening the project throws an unhandled exception when trying to open the jacobs.dspx file in the attachment. The exception occurs
in DotSpatial.Serialization when it is deserializing a Featureset.
It cannot find the constructor of FeatureSet that has the string Filename parameter --> resulting in the unhandled exception.

Not actual


Update package.json

Snyk has scanned your PR to find newly introduced vulnerabilities
Snyk detected changes in the following dependency files:

package.json
package-lock.json

These are the new vulnerabilities introduced via those changes:
package.json

Severity
Issue
Package
Remediation

Arbitrary Code Injection SNYK-JS-MORGAN-72579
morgan 1.9.1
Upgrade to 3.2.4

Arbitrary File Write via Archive Extraction (Zip Slip) npm:adm-zip:20180415
adm-zip 1.2.3
Not available

Uninitialized Memory Exposure npm:base64-url:20180512
base64-url 1.2.3
Upgrade to 9.1.0 (breaking change)

Regular Expression Denial of Service (ReDoS) npm:fresh:20170908
fresh 3.4.5
Not available

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to run tests on your PRs.
For more information:
🧐 View latest PR Tests report
🛠 Adjust project PR Tests settings
📚 Read more about Snyk's PR Tests logic

Snyk commands and options
Signed in users, can quickly trigger actions by clicking on the above links:

Send a fix commit to this PR
Retest PR
Mark status check to successful (only allowed to Snyk admins)

`


testv2: Fix waitMachinesHasNodes, increase timeouts, and label nodes to skip eviction

What this PR does / why we need it:
This PR introduces several changes to the new testing framework:

waitMachinesHasNodes has been fixed to properly wait for all Machines to get rolled-out

Currently, the function is not working properly if there's one MachineDeployment with multiple replicas. That's the case for all non-AWS providers where we use one MachineDeployment with 2 replicas
It might happen that waitMachinesHasNodes proceed right after the first Machine is rolled out without waiting for the second Machine to get rolled out. This can cause tests to flake
This is fixed by ensuring that we have the expected number of Machines and that no Machine has a deletion timestamp

Timeout for waiting Machines and nodes to become ready is increased to 20 minutes

Some providers are very slow (e.g. Equinix Metal and Azure) and 10 minutes is not enough, so it's causing tests to flake

Label nodes managed by machine-controller to skip eviction

The purpose of this change is to speed up tests on providers that take a lot of time to rotate nodes (e.g. Equinix Metal, Azure...)

Which issue(s) this PR fixes:
Fixes #2321
What type of PR is this?
/kind bug
Does this PR introduce a user-facing change? Then add your Release Note here:
NONE

Documentation:
NONE

/test pull-kubeone-e2e-aws-default-upgrade-containerd-from-v1.23.9-to-v1.24.3

/lgtm
/approve


undefined symbols

Went to try this out in our 4.2.1 iRODS environment and am getting the following in the irods log
Aug 15 16:44:50 pid:114854 remote addresses: <IP_ADDRESS>, <IP_ADDRESS> ERROR: [-]	/home/irodsbuild/irods/server/core/src/irods_resource_manager.cpp:669:irods::error irods::resource_manager::process_init_results(genQueryOut_t *) :  status [PLUGIN_ERROR]  errno [] -- message []
	[-]	/home/irodsbuild/irods/server/core/src/irods_resource_manager.cpp:122:irods::error irods::resource_manager::load_resource_plugin(resource_ptr &, const std::string, const std::string, const std::string) :  status [PLUGIN_ERROR]  errno [] -- message []
		[-]	/home/irodsbuild/irods/lib/core/include/irods_load_plugin.hpp:176:irods::error irods::load_plugin(PluginType *&, const std::string &, const std::string &, const std::string &, const std::string &) [PluginType = irods::resource] :  status [PLUGIN_ERROR]  errno [] -- message [failed to open shared object file [/usr/lib/irods/plugins/resources/libirados.so] :: dlerror: is [/usr/lib/irods/plugins/resources/libirados.so: undefined symbol: _ZN5irods8resource9add_childERKSsS2_N5boost10shared_ptrIS0_EE]]

The machine is Ubuntu 14.04 with packages from 4.2.1 of iRODS and 10.2.9 of Ceph.
The irados/Makefile needed to be changed, since the static libirods_client.a file doesn't exist.
#EXTRALIBS = /usr/lib/irods/libirods_client.a -lrados
EXTRALIBS = -lirods_client  -lrados

Also had to add -std=gnu++11 to GCC in order to allow compilation, due to warning from /usr/include/c++/4.8/bits/c++0x_warning.h about ISO C++ 2011 standards.
Copying the resulting libirados.so to /usr/lib/irods/plugins/resources/libirados.so is fine, but when restarting the irods server or attempting to iput to the resource fails with the above log entry.

Hi @robnaccy - we're actively preparing this codebase for compatibility with iRODS 4.2+.  There were a great number of things that moved in the overhaul of our buildchain with 4.2.


🐛 BUG: Problems with run language server

Describe the Bug
Hello everyone,
I have a problem with running the language server in the Helix editor. Interdimietly when I run the Vue file it exit. In the Helix log file, I have an output like the below:
2023-06-10T11:30:03.593 helix_view::theme [WARN] Theme: malformed hexcode: statusline1
2023-06-10T11:30:03.593 helix_view::theme [WARN] Theme: malformed hexcode: statusline2
2023-06-10T11:30:03.594 helix_view::theme [WARN] Theme: malformed hexcode: statusline1
2023-06-10T11:30:20.640 helix_term::application [WARN] Ignoring a client/registerCapability request because dynamic capability registration is not enabled. Please report this upstream to the language server
2023-06-10T11:30:20.892 helix_term::application [ERROR] Language Server: Method workspace/inlayHint/refresh not found in request 3
2023-06-10T11:30:23.811 helix_lsp::transport [ERROR] err <- "/nix/store/r1b289b7m3vjhkfy7sr1w0gpm407xa1f-node-dependencies-js-packages-0.0.1/lib/node_modules/vscode-jsonrpc/lib/common/connection.js:565\n"
2023-06-10T11:30:23.811 helix_lsp::transport [ERROR] err <- "                        responsePromise.reject(new messages_1.ResponseError(error.code, error.message, error.data));\n"
2023-06-10T11:30:23.811 helix_lsp::transport [ERROR] err <- "                                               ^\n"
2023-06-10T11:30:23.811 helix_lsp::transport [ERROR] err <- "\n"
2023-06-10T11:30:23.811 helix_lsp::transport [ERROR] err <- "ResponseError: Method not found: workspace/inlayHint/refresh\n"
2023-06-10T11:30:23.811 helix_lsp::transport [ERROR] err <- "    at handleResponse (/nix/store/r1b289b7m3vjhkfy7sr1w0gpm407xa1f-node-dependencies-js-packages-0.0.1/lib/node_modules/vscode-jsonrpc/lib/common/connection.js:565:48)\n"
2023-06-10T11:30:23.811 helix_lsp::transport [ERROR] err <- "    at handleMessage (/nix/store/r1b289b7m3vjhkfy7sr1w0gpm407xa1f-node-dependencies-js-packages-0.0.1/lib/node_modules/vscode-jsonrpc/lib/common/connection.js:345:13)\n"
2023-06-10T11:30:23.812 helix_lsp::transport [ERROR] err <- "    at processMessageQueue (/nix/store/r1b289b7m3vjhkfy7sr1w0gpm407xa1f-node-dependencies-js-packages-0.0.1/lib/node_modules/vscode-jsonrpc/lib/common/connection.js:362:17)\n"
2023-06-10T11:30:23.812 helix_lsp::transport [ERROR] err <- "    at Immediate.<anonymous> (/nix/store/r1b289b7m3vjhkfy7sr1w0gpm407xa1f-node-dependencies-js-packages-0.0.1/lib/node_modules/vscode-jsonrpc/lib/common/connection.js:334:13)\n"
2023-06-10T11:30:23.812 helix_lsp::transport [ERROR] err <- "    at process.processImmediate (node:internal/timers:476:21) {\n"
2023-06-10T11:30:23.812 helix_lsp::transport [ERROR] err <- "  code: -32601,\n"
2023-06-10T11:30:23.812 helix_lsp::transport [ERROR] err <- "  data: undefined\n"
2023-06-10T11:30:23.812 helix_lsp::transport [ERROR] err <- "}\n"
2023-06-10T11:30:23.812 helix_lsp::transport [ERROR] err <- "\n"
2023-06-10T11:30:23.812 helix_lsp::transport [ERROR] err <- "Node.js v18.16.0\n"
2023-06-10T11:30:23.840 helix_lsp::transport [ERROR] err: <- StreamClosed

Have someone had a problem like the above?
Steps to Reproduce
I'm working on NixOS so I configure it in .nix file like below:
        {
          name = "astro";
          scope = "source.astro";
          injection-regex = "astro";
          file-types = ["astro"];
          roots = ["package.json" "astro.config.js"];
          auto-format = true;
          formatter = {
            command = "prettier";
            args = ["--config" ".prettierrc"];
          };
          language-server = {
            command = "astro-ls";
            args = ["--stdio"];
          };
          config = {
            "typescript" = {
              tsdk = "${nodeDeps}/lib/node_modules/typescript/lib";
            };
            "environment" = "node";
          };
        }

The output from the above snippet looks like the below (language.toml):
[[language]]
auto-format = true
file-types = ["astro"]
injection-regex = "astro"
name = "astro"
roots = ["package.json", "astro.config.js"]
scope = "source.astro"

[language.config]
environment = "node"

[language.config.typescript]
tsdk = "/nix/store/r1b289b7m3vjhkfy7sr1w0gpm407xa1f-node-dependencies-js-packages-0.0.1/lib/node_modules/typescript/lib"

[language.formatter]
args = ["--config", ".prettierrc"]
command = "prettier"

[language.language-server]
args = ["--stdio"]
command = "astro-ls"

I've got the same problem. Have you perhaps found a workaround?

It's a upstream issue, fixed by https://github.com/volarjs/volar.js/commit/eb8f8d93fef4eab4d006feb50d8952863a05fdd1


Replace filter interface

What this PR does:
Which issue(s) this PR fixes:

Fixes #108
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

Codecov Report

Merging #109 (b434c39) into develop (a654db8) will decrease coverage by 0.47%.
The diff coverage is 68.75%.

@@             Coverage Diff             @@
##           develop     #109      +/-   ##
===========================================
- Coverage    41.84%   41.36%   -0.48%
===========================================
  Files           33       33
  Lines         1410     1378      -32
===========================================
- Hits           590      570      -20
+ Misses         736      730       -6
+ Partials        84       78       -6

Impacted Files
Coverage Δ

pkg/client/dubbo/dubbo.go
22.01% <ø> (ø)

pkg/client/mapper.go
78.94% <ø> (ø)

pkg/client/request.go
0.00% <ø> (ø)

pkg/filter/accesslog/access_log.go
4.08% <ø> (ø)

pkg/filter/recovery/recovery.go
77.77% <ø> (ø)

pkg/filter/replacepath/replace_path.go
61.11% <ø> (ø)

pkg/filter/response/response.go
5.38% <0.00%> (ø)

pkg/registry/util.go
0.00% <ø> (ø)

pkg/router/api.go
100.00% <ø> (ø)

pkg/client/http/http.go
60.31% <33.33%> (ø)

... and 9 more

Continue to review full report at Codecov.

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

Codecov Report

Merging #109 (b434c39) into develop (a654db8) will decrease coverage by 0.47%.
The diff coverage is 68.75%.

@@             Coverage Diff             @@
##           develop     #109      +/-   ##
===========================================
- Coverage    41.84%   41.36%   -0.48%
===========================================
  Files           33       33
  Lines         1410     1378      -32
===========================================
- Hits           590      570      -20
+ Misses         736      730       -6
+ Partials        84       78       -6

Impacted Files
Coverage Δ

pkg/client/dubbo/dubbo.go
22.01% <ø> (ø)

pkg/client/mapper.go
78.94% <ø> (ø)

pkg/client/request.go
0.00% <ø> (ø)

pkg/filter/accesslog/access_log.go
4.08% <ø> (ø)

pkg/filter/recovery/recovery.go
77.77% <ø> (ø)

pkg/filter/replacepath/replace_path.go
61.11% <ø> (ø)

pkg/filter/response/response.go
5.38% <0.00%> (ø)

pkg/registry/util.go
0.00% <ø> (ø)

pkg/router/api.go
100.00% <ø> (ø)

pkg/client/http/http.go
60.31% <33.33%> (ø)

... and 9 more

Continue to review full report at Codecov.

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


Update Andromeda description and add centralized dev instructions

Added a more full description of Andromeda (the package in the repo and the tool itself). Also added the directions to run Andromeda locally to the main repo README so they're not split between two README's in separate folders.

Added a CITATION.cff file with references to the original and a more recent Andromeda papers that Chris North pointed us to in April. Also references the source repo and another that I referenced in re-working the code.


lint: make sure to bubble up errors

make sure to bubble up errors when fail to update
manifests. This is so unlikely we want to know.

/cherry-pick release-4.14

/cherry-pick release-4.14


Fix github url ghr to gchr and updated docker syntax

Description
Updated link to github from ghr.io to gchr.io
Pull Request Type

[X] Documentation content changes
[ ] Bugfix / Missing Redirects
[ ] Docusaurus site code changes

PR Checklist

[ ] The commit message follows our guidelines included in the CONTRIBUTING.md
[ ] All tests of Lint, cspell and check-content pass. (How to launch them?)
[X] No sensitive content has been committed

ok perfect, I opened a request on the other repo


Sdk tools GitHub release

Added script to verify agent OS to repo
Build creates an archive publishing tools as a Github release.

@Azure/azure-sdk-eng

@chidozieononiwu the commits look a little off while view the changes in this PR. What steps did you take to rebase?


add display size of registry catalog url - [merged]

In GitLab by @sokril on Nov 6, 2020, 06:26
Merges reg -> dev

In GitLab by @wknet123 on Nov 6, 2020, 07:14
mentioned in commit 1536f5d87f47778cbc0010c6a3db8428ddd301fe

In GitLab by @wknet123 on Nov 6, 2020, 07:14
merged


Guide on Tracing Reactive Apps w/ Sleuth

Small Guide on Reactive Tracing with Spring Cloud Sleuth... more to come.

Hey @Mario5Gray! Just wanted to check to see if this was ready to merge or is still in progress. Just wanted to make sure you weren't waiting around on me :)

Hi Brian!  I got all the kinks and bugs out, adn believe this is complete,
and waiting for merge.
Thanks!
Mario
On Fri, Aug 26, 2022 at 10:21 AM Brian McClain @.***>
wrote:

Hey @Mario5Gray https://github.com/Mario5Gray! Just wanted to check to
see if this was ready to merge or is still in progress. Just wanted to make
sure you weren't waiting around on me :)
—
Reply to this email directly, view it on GitHub
https://github.com/vmware-tanzu/tanzu-dev-portal/pull/2201#issuecomment-1228742013,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADM66HSS7HCNEAKB2COBOK3V3D4LNANCNFSM56JASSDA
.
You are receiving this because you were mentioned.Message ID:
@.***>

@Mario5Gray Live! https://tanzu.vmware.com/developer/guides/distributed-tracing-reactive-kotlin-pt1/


Service mesh integration with k8s

Show how service mesh integrate with k8s on AWS. The couple of possibilities:

Linkerd
Istio/Envoy

assign to @omarlari

assign this to dalbhanj@, spoke to omarlari and he is okay with moving this to me

PR #92 adds Istio. We could probably do with one or two other linkerd examples before we close this issue. Aim is to show how useful the mesh is for enterprises running microservices.

PR #94 means this issue can be closed


I'm looking for a relative layout. Have an example? Try the LMLayerView and then have the error.

I'm looking for a relative layout. Have an example? Try the LMLayerView and then have the error.
-> MarkupKit 4.5

LMLayerView no longer exists, but I think LMAnchorView may be what you are looking for:
https://github.com/gk-brown/MarkupKit/blob/master/README.md#lmanchorview
https://github.com/gk-brown/MarkupKit/wiki/Examples#anchor-view

Thanks


Run Build Task (CTRL + Shift +B) Slow to open drop down list

It takes between 5 to 10 seconds to open the dialog.
Prior to the latest update it was near instantaneous.
Version: 1.29.1 (user setup)
Commit: bc24f98b5f70467bc689abf41cc5550ca637088e
Date: 2018-11-15T19:13:36.375Z
Electron: 2.0.12
Chrome: 61.0.3163.100
Node.js: 8.9.3
V8: 6.1.534.41
Architecture: x64
Windows 10 Enterprise 10.0.15063

Duplicate of #63078


sample: net: fix incorrect statistics of zperf

The sequence id from iperf starts with 1. The commit changes the initial value of "next_id" in zperf from 0 to 1. In addition, the error output of "error" and "out of order" packets is corrected.
Signed-off-by: Yanqin Wei<EMAIL_ADDRESS>

The change looks ok, but the commit title prefix is not correct, it's not modifying a sample, please use net: zperf:  for the prefix.

Yes. I did the work in the previous release, in which the zperf is still located in the 'sample' directory.  The title is updated. Thanks.


Change result from getauxblock to BIP22 validation state

This should probably include https://github.com/dogecoin/dogecoin/commit/e2334c99e

Actually with https://github.com/dogecoin/dogecoin/commit/e2334c99e it's basically no-change isn't it? This adds a more complex return which e2334c99e then turns back into a simple one?
This is probably a "Does not need to be applied" in hindsight.

https://github.com/dogecoin/dogecoin/commit/e2334c99e also adds a compatibility method though. I do not know what current pool implementations use so I'd rather have us offer both variants. Or am I missing something here?

I'll check, but it doesn't look like getauxblockbip22 is actually exposed anywhere

getauxblock decides based on the parameters what to return. So it is kinda exposed.

Ah, got it! Yup will get that glued on in.


Bsapp 1300

As Ligaleiter I can not change my email when creating a new event.
As admin I can select another mail than my own one

new bugs emerged


Too many Reindeers

Please help me with settings of Reindeers.

I need:

Reduce their amount.
Reduce Reindeer amount in one horde.
Make them keep distance from players.

Thank you very much for your beautiful mode!

Animal spawning has seen some changes in the last update. Not enough long term testing has been done yet to say this is fixed, so I'll leave this open for now.


Add build.cwd option

WHAT
(Write what you need)
How about add build.cwd option to change cwd during building?
- name: nkf
  description: Netword Kanji Filter
  url: https://free.nchc.org.tw/osdn//nkf/70406/nkf-2.1.5.tar.gz
  command:
    build:
      cwd: nkf-2.1.5
      steps:
        - make
        - make install prefix=~/.local
    link:
      - from: nkf

WHY
(Write the background of this issue)
Sometimes build.steps requires changing cwd, but cd command is not working.
- name: nkf
  description: Netword Kanji Filter
  url: https://free.nchc.org.tw/osdn//nkf/70406/nkf-2.1.5.tar.gz
  command:
    build:
      steps:
        - cd nkf-2.1.5 # not working
        - make
        - make install prefix=~/.local
    link:
      - from: nkf

Thank you for the suggestion. I added build.directory param in new version. Please try it out.


Replace usage of cirq.Circuit.from_ops(...) with cirq.Circuit(...) in circuits,work,dev_tools

Part of #1787
This should be nothing but basically a search-replace of Circuit.from_ops( with Circuit(, followed by formatting, followed by tiny manual tweaks.

Is it worth leaving one usage of from_ops in to check that the deprecated behavior is still functional?

I left some tests in circuit_tests.py. A future change renames the test to include deprecated.

@mpharrigan I take it back, I accidentally replaced those from_ops also. Fixed it now.


pyenv shell reports Error: did not expect more than one argument.

If I install pyenv-virtualenv, it will report error every time I switch python env.
$ pyenv shell anaconda3-2.1.0
pyenv-virtualenv: activate anaconda3-2.1.0
Error: did not expect more than one argument.

Want to back this issue? Place a bounty on it! We accept bounties via Bountysource.

Weird :disappointed:
The anaconda3-2.1.0 must not be a virtualenv and it shouldn't be activate basically. The pyenv-virtualenv should check if the activating version is a virtualenv or not with using pyenv-virtualenv-prefix. Please gist the output from PYENV_DEBUG=1 pyenv virtualenv-prefix anaconda3-2.1.0 to help investigation.
It seems there is something going wrong in auto-activation feature of pyenv-virtualenv. If nothing helps, just removing eval "$(pyenv virtualenv-init -)" from your shell configuration may help.

I confirmed there is activate script in the prefix of anaconda3-2.1.0. This leads pyenv-virtualenv-prefix to recognize anaconda3-2.1.0 as a virtualenv.
Until I or someone else prepare a workaround for this, I'd recommend you to disable eval "$(pyenv virtualenv-init -)" if you're using anaconda :(

This is the gist: https://gist.github.com/xu-cheng/9cec6e81760167ba4275
I will disable it then. Thanks for helping.

I pushed 0fc9441f091051cb482b47ce01455ecd0d9dcca3 to master to avoid try activating Anaconda.

Thanks, it fixes the problem. Hope you can tag a new version for pyenv-virtualenv. I can help to push it to homebrew.


NPM package

Hi there,
it would be helpful to publish this library as a NPM package, so it can be imported and updated easily.

Thanks for the recommendation @Spone.  I've been meaning to get stuff onto NPM for years.... for whatever reason, your issue helped me finally do it :)


Additional empty-name backend might result in unknown backend state

For some reason I'm getting an additional backend, with an empty name :
bash-4.4# curl -s -S localhost:8000/trident/v1/backend
{"backends":["ontap-economy","cluster1",""]}

I'm thinking it might be causing an issue with the output of my regular backend listing with tridentctl, and the status of one of them (unknown) :
yann$ tridentctl get backend
+---------------+-------------------+---------+---------+
|     NAME      |  STORAGE DRIVER   |  STATE  | VOLUMES |
+---------------+-------------------+---------+---------+
| cluster1      | ontap-nas         | unknown |       3 |
| ontap-economy | ontap-nas-economy | online  |       1 |
+---------------+-------------------+---------+---------+

What version are you using? Can you try it with 19.04.1 or later? We think this is fixed.

It has been a while I don’t think I have that instance anymore. Thanks !


Embedded binaries in blpapi wheels since 3.16.3

Per https://www.bloomberg.com/professional/support/api-library/ from version 3.16.3
Bloomberg ships the C++ binaries within the wheels. Just doing the good old
python -m pip install --index-url=https://bcms.bloomberg.com/pip/simple/ blpapi should be sufficient for most platforms.
Python3.9 is not yet supported, probably in the next version.

https://bloomberg.bintray.com is EOL,
BCMS (Bloomberg Content Management System) is the new public file host for the libraries: https://bcms.bloomberg.com/BLPAPI-Generic/ and https://bcms.bloomberg.com/pip/simple/blpapi/

Great thanks for pointing this out. Would you be able to submit a PR to update the endpoints? There's a discussion of the process at https://github.com/conda-forge/blpapi-feedstock#updating-blpapi-feedstock. For this I think we would need to

Update the url section recipe/meta.yaml
Bump the build number in recipe/meta.yaml


Problem with deserializeUser when using the mongodb driver

passport.deserializeUser(function (id, next) {
  User.findOneById(id, next); // this works
  // User.findOne(id, next); // this does not work
});

Closing as the project is no longer maintained.


translate: change the help info to english

Codecov Report

Merging #2563 into master will increase coverage by -0.07%.

@@            Coverage Diff             @@
##           master    #2563      +/-   ##
==========================================
- Coverage   40.18%   40.11%   -0.07%
==========================================
  Files        1725     1725
  Lines       60225    60225
  Branches     8388     8388
==========================================
- Hits        24201    24161      -40
- Misses      36024    36064      +40

Impacted Files
Coverage Δ

.../java/com/navercorp/pinpoint/rpc/cluster/Role.java
0% <ø> (-71.43%)
:x:

...a/com/navercorp/pinpoint/rpc/util/StringUtils.java
0% <ø> (-28.58%)
:x:

.../com/navercorp/pinpoint/rpc/packet/PongPacket.java
50% <ø> (-21.43%)
:x:

...npoint/rpc/server/SimpleServerMessageListener.java
80% <ø> (-20%)
:x:

...rp/pinpoint/rpc/control/ControlMessageDecoder.java
66.07% <ø> (-14.29%)
:x:

...vercorp/pinpoint/rpc/packet/ClientClosePacket.java
72.22% <ø> (-11.12%)
:x:

...om/navercorp/pinpoint/rpc/codec/PacketDecoder.java
71.08% <ø> (-7.23%)
:x:

.../pinpoint/rpc/client/PinpointClientHandshaker.java
67.36% <ø> (-6.95%)
:x:

...m/navercorp/pinpoint/rpc/packet/RequestPacket.java
89.74% <ø> (-5.13%)
:x:

.../profiler/sender/StandbySpanStreamDataStorage.java
79.66% <ø> (-1.7%)
:x:

... and 3 more

Continue to review full report at Codecov.

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

Thank you for your contribution.


Stress tests

Integrate the Track 2 SDK with the service team stress tests. We should talk to @jsquire @kinelski to see how this was done in EH and if they have thoughts on what we should do for SB.

Marking as 10 initially as there is a good deal of ambiguity as to what needs to be done here and how long it will take. From EH experience, this ended up taking a good chunk of time.

The service team mentioned that we didn't need to integrate with their stress harness. We should just do simple console apps that will run over several days to verify correctness.


Convenience methods for setting Stylistic Set labels?

Following @tiroj comment in https://github.com/googlefonts/fontbakery/issues/3155#issuecomment-845522476 and https://github.com/googlefonts/glyphsLib/pull/615 I wonder if its worth adding a convenience feature to fontTools to make it easy to read and write these values, since GlyphsApp, VOLT and perhaps other compilers (FontForge?) don't support them, so post-processing scripts to fix them will be widely useful.

can't they just add featureNames blocks to the AFDKO feature files?
https://adobe-type-tools.github.io/afdko/OpenTypeFeatureFileSpecification.html#8.c
fontTools.feaLib (and thus fontmake) support those already. I'd be surprised if other font editors did not. Don't they?

We’re not using AFDKO.
AFDKO still won’t compile subtable structures for features other than kern, and none of the common AFDKO-based font editing tools have proper UI for contextual GPOS or mark filtering at the lookup level, especially not for variable fonts since they presume a single .fea source for GSUB and that GPOS will be magically compiled from native kerning and anchor data. In effect, they don’t actually have GPOS editing at all. VOLT and FontCreator are currently the only shipping tools that I would consider using for most of our projects. Some of what @simoncozens is working on looks promising.

Here's a snippet to read the descriptive names for stylistic sets (adapted from FontGoggles). It should be expanded to also handle character variations.
_stylisticSets = {f"ss{i:02}" for i in range(1, 21)}

def getStylisticSetNames(font):
    gsubTable = font.get("GSUB")
    nameTable = font.get("name")
    if gsubTable is None or nameTable is None:
        return {}
    gsubTable = gsubTable.table
    names = {}
    for feature in gsubTable.FeatureList.FeatureRecord:
        tag = feature.FeatureTag
        if tag in _stylisticSets and tag not in names:
            feaParams = feature.Feature.FeatureParams
            if feaParams is not None:
                nameRecord = nameTable.getName(feaParams.UINameID, 3, 1)
                if nameRecord is not None:
                    names[tag] = nameRecord.toUnicode()
    return names

Building FeatureParams is a bit more complex, and the code in fontTools.feaLib.builder isn't directly reusable, but can still be a reference.

Dear future me returning to this topic, use names[str(tag)] = nameRecord.toUnicode() for easier working with the index ;)


Tailwind doesn't work correctly

For some reason, the styles are wrong (prefers color scheme doesn't work, daisyui classes don't work) when I configure Tailwind support like this:
tailwind-input-file = "ui/app.css"
tailwind-config-file = "ui/tailwind.config.ts"

But if I use this instead and generate the stylesheet directly with the CLI, it works correctly
$ tailwindcss -i ./app.css -o ./target/style.css

style-file = "target/style.css"

Here's my Tailwind config
export default {
  content: {
    files: ["*.html", "./src/**/*.rs"],
  },
  theme: {
    extend: {},
  },
  plugins: [require("@tailwindcss/typography"), require("daisyui")],
  daisyui: {
    logs: false,
    themes: [...],
  },
};

Nevermind I made a mistake in my tailwind config.


The same bug as in autocomplete

I can understand if you can't fix it but anyways...
If you write a property and want to set its getter or setter then xCode suggests a list of methods and all of them are usually incorrect. Your plugin doesn't suggest a correct list too. But in this case the standard behaviour is even better than your one because in this case it at least allows to see the exact keyboard output and your plugin always tries to replace the current word with another one from list.
I could suggest one of the following solutions:

to disable your pugin for setting of getter/setter params;
to show a really complete list of word comletions. Note: a programmer can set any method as setter/getter even if this method is private for current object (doesn't exist in the current .h file but is declared in .m file)

FYI the plugin already allows for seeing exact keyboard output in two ways:

if you enable "list header" in the plugin options
if you disable "inline preview" in the plugin options (or "type over completions" in Xcode options)

Also, there are several options allowing you to tweak the SNR of completions to your liking, e.g.:

you can filter completions by score more aggressively
you can choose to require at least a few letters to be the prefix of the match
you can tweak the scoring function to your liking

The plugin is context independent. It neither removes nor adds any completions to the list that Xcode generates for given context.


Support Python 3 without the 2to3 tool

I'm trying to add a Glue Python 3 package for Fedora and using the 2to3 tool is a little bit clunky on packaging land.
This PR includes fairly simple changes to add Python 3 native support for glue.

I run the tests and they pass in both Python 2.7 and Python 3.4

Coverage increased (+0.4%) to 92.927% when pulling 188ce87d07f6fd18cfe2db2713d86f30de052ded on lorenzogil:master into 5c9cf65e9db80c47d3947255dfcf3b314cfe8c25 on jorgebastida:master.


19 config assets array sort

This fixes the sorting issue, but made me think about merging install_folder field into AssetInfo again...
Additionally, toml-edit might be an alternative to allow for a more compact asset_info field.

Thoughts above will not be implemented now. I suspect the config API might be under frequent changes, before I fully stabilize whats the best way forward.


[Feat] Social Icons

Description
I would like to add social icons to the login page and the signup page for a better way to login and signup , i will make sure the buttons has the same theme as the site has , please assign this issue to me under gssoc.
Screenshots
No response
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I'm a GSSoC'24-Extd contributor
[ ] I'm a Hacktoberfest'24 contributor
[X] I am willing to work on this issue (optional)

Hey @Dipeshdahiya, that's a good suggestion, but have you noticed that major e-commerce websites mainly use phone number verification with SMS OTP? This is not feasible for us because it's a paid service, which is why we use email OTP verification for easy login and signup. I hope you understand why we’re not implementing this feature.

Okay understood. Is their any other issue from your side which is pending and you like someone to work on , is anything then please tell me i would be very happy to work on that issue instead.


Fix XCT fails build in F# projects

Description of Change
Include PreserveXamarinCommunityToolkit.cs only in C# projects
Tested it with C# and F# projects:

It's still included in C# projects
It's no longer included in F# projects (and build no longer fails)

Bugs Fixed

Fixes #1124

API Changes
None
Behavioral Changes
None
PR Checklist

[ ] Has tests (if omitted, state reason in description)
[ ] Has samples (if omitted, state reason in description)
[x] Rebased on top of main at time of PR
[x] Changes adhere to coding standard

[ ] Updated documentation

Hi @maxkoshevoi and thanks for this contribution)
Can you please explain how did you test it? Did you create nuget packages locally?

@AndreiMisiukevich Turns out our CI pipelines create NuGet for each successful run! (you can find it in artifacts)
Here's how I did the testing:

Created F# project and included public NuGet into it.

Build failed as described in the issue.

The issue is that we are publishing .targets in our NuGet that are including PreserveXamarinCommunityToolkit.cs into every project that NuGet is added to. By the way, do you know, why is this needed?
Downloaded NuGet from pipeline artifacts for this PR
2. Included new NuGet in F# project
2) Build succeeded
3. Included new NuGet in C# project
3) Build succeeded
4. Deleted PreserveXamarinCommunityToolkit.cs from the NuGet's build folder
4) F# project still built successfully, but C# project failed with "Unable to find PreserveXamarinCommunityToolkit.cs error".

Perhaps we should add a new file PreserveXamarinCommunityToolkit.fs
// <auto-generated/>

open Xamarin.CommunityToolkit.Helpers

[<assembly: PreserveToolkit>]

And amend targets file as
<Project>
  <ItemGroup>
    <Compile Condition="'$(MSBuildProjectExtension)' == '.csproj'"
             Include="$(MSBuildThisFileDirectory)PreserveXamarinCommunityToolkit.cs"
             Visible="false" />
    <Compile Condition="'$(MSBuildProjectExtension)' == '.fsproj'"
             Include="$(MSBuildThisFileDirectory)PreserveXamarinCommunityToolkit.fs"
             Visible="false" />
  </ItemGroup>
</Project>

And csrpoj
<None Include="build\**\*.cs;build\**\*.fs;build\**\*.targets" Pack="true" PackagePath="build" />

Good point! Added PreserveXamarinCommunityToolkit.fs, but I still don't get why do we need to added it into projects that depend on the NuGet.
Testing: I'll update this comment after the testing

Need to update Xamarin.CommunityToolkit.csproj file (at the bottom)
<None Include="build\**\*.cs;build\**\*.fs;build\**\*.targets" Pack="true" PackagePath="build" />

Need to include a new file there

Good point! Added PreserveXamarinCommunityToolkit.fs, but I still don't get why do we need to added it into projects that depend on the NuGet.

Frankly speaking I don't know too)
Maybe @jfversluis @pictos know better

Need to update Xamarin.CommunityToolkit.csproj file

XCT builds just fine. The problem is when I try to build F# project that includes XCT NuGet.
Those files are not included in XCT assembly anyway (since they don't have .shared. in them).

I mean, we need to add .fs file to csproj for copying it to the NuGet bundle.

// <auto-generated/>
namespace Xamarin.CommunityToolkit.Helpers

[<assembly: PreserveToolkit>]
()

Please go ahead with this one

@AndreiMisiukevich Also we need add PreserveXamarinCommunityToolkit.fs at the top somehow, since file that contains entry point must be the last in F#, and currently PreserveXamarinCommunityToolkit.fs is being added last, so this is another issue

@maxkoshevoi seems it's working for me. Is it working for you now as well?

Ah, okay, it is not... I see.
Need to find how to fix it

@maxkoshevoi seems it's working for me. Is it working for you now as well?

Got the same exception (Files in libraries or multiple-file applications must begin with a namespace or module declaration...). After adding module Hi to Program.fs, got this one:

Manually shifting this to the top of the fsproj file solves the problem
  <Import Project="..\packages\Xamarin.CommunityToolkit.1.1.0-preview1982\build\Xamarin.CommunityToolkit.targets" Condition="Exists('..\packages\Xamarin.CommunityToolkit.1.1.0-preview1982\build\Xamarin.CommunityToolkit.targets')" />

Manually shifting this to the top of the fsproj file solves the problem
  <Import Project="..\packages\Xamarin.CommunityToolkit.1.1.0-preview1982\build\Xamarin.CommunityToolkit.targets" Condition="Exists('..\packages\Xamarin.CommunityToolkit.1.1.0-preview1982\build\Xamarin.CommunityToolkit.targets')" />

It doesn't for me. Are you using SDK-style fsproj file?

SDK-style

Nope, it's not SDK-style in my case

It worked for me too. But location for targets file is different (there is no packages folder anywhere near the project).
Here's the path where they are located for me: C:\Users\Mi\.nuget\packages\xamarin.communitytoolkit\1.1.0-preview1982\build\Xamarin.CommunityToolkit.targets

Well, let's invite more people to this conversation
CC @jfversluis @pictos @jsuarezruiz @sthewissen @brminnick

I don't know, maybe we can skip adding this attribute for F# projects for now. Let's see what others think)

maybe we can skip adding this attribute for F# projects for now

That's what I initially thought 😄
I tried to add it, faced couple of issues and decided that it's not worth it, or someone would point out, that it needs to be done (if this is the case).
Currently I'm not even sure that we need to add it to C# projects, and I completely don't understand what it's purpose is.
So I'm totally fine not adding it to F# projects =)

@jfversluis @brminnick @jsuarezruiz do you know someone on MS side that can help us with this issue?

Confirmed this fix with the F# team internally 👍

FYI - I cherry-picked this to develop branch as well https://github.com/xamarin/XamarinCommunityToolkit/commit/4ee78fc9741412afc8f83546e9866a7475f01e30


feat(protocol): interpret bridge message.fee as the max fee

BEGIN_COMMIT_OVERRIDE
feat(protocol): interpret bridge message.fee as the max fee and refund unused fee
refactor(protocol): deprecate bridge message's 'refundTo' field
END_COMMIT_OVERRIDE

In this PR, we measure the actual gas used by message invocation x, calculate an actual fee per gas p, then allow the relayer to charge  actual fee  of (x+o)*p from message.fee, the rest are refunded to message.destOwner. Note o is the message processing overheads, whose value depends on whether the message is processed in one step or two.
We consider message.fee/message.gasLimit the max fee per gas, lets call it m, we ensure that:

if m < block.basefee, p = m
else p = (m + block.basefee)/2

@KorbinianK On our app, this is how the message.fee shall be calculated:

estimate the max gas fee used on the destination chain to invoke the message call, use 0 if there is no message call at all (only Ether transfer), call it X
add Bridge.TWO_STEP_PROCESSING_GAS_OVERHEAD() to X to get Y
multiple the user-selected base fee per gas B with Y to get Z. We can also suggest a value = 2*current dest chain base fee.
Z is the fee.

This PR is ready to be reviewed again for merge.

@dantaik I think we need to record the basefee of the first step of the two step invocation so the relayer can be profitable considering both steps.

@dantaik I think we need to record the basefee of the first step of the two step invocation so the relayer can be profitable considering both steps. However this still doesn't let us know whether it's worth it for the relayer to even send the first tx because we don't know if the second tx baseFee at that point.
is there no way to do both steps at once and block proposers check for messages to be released/executed similiar to how eth deposits worked before, and proposers get a small% of fee?
Or when a bridge message is sent, they pay to finalize previous messages? IDK just spitballing here but it seems this still doesn't fully solve the "how to make two transactions guaranteed to be profitable so I know I should send the first tx"
Or perhaps best of all, one step mode can be activated with a signature from the watchdog, which could be verified onchain and the relayer can collect this before submitting.

I‘m afraid this current implementation is the best we can have for 2-step message execution. Hopefully we can disable the 2-step execution months after the genesis if there is no bug identified.

@dantaik I think we need to record the basefee of the first step of the two step invocation so the relayer can be profitable considering both steps. However this still doesn't let us know whether it's worth it for the relayer to even send the first tx because we don't know if the second tx baseFee at that point.
is there no way to do both steps at once and block proposers check for messages to be released/executed similiar to how eth deposits worked before, and proposers get a small% of fee?
Or when a bridge message is sent, they pay to finalize previous messages? IDK just spitballing here but it seems this still doesn't fully solve the "how to make two transactions guaranteed to be profitable so I know I should send the first tx"
Or perhaps best of all, one step mode can be activated with a signature from the watchdog, which could be verified onchain and the relayer can collect this before submitting.

I think we can pay the two steps separately:

step 1:  https://github.com/taikoxyz/taiko-mono/pull/16722
step 2: still working on it


Mouse click on VRTK UI Canvas along with UIPointer possible?

Using the vrtk UI canvas, is there a feature or property that could be used where, Both Oculus touch's pointer and the mouse interaction can be used?
For now, The canvas is made World space and interactable with Oculus UI pointer. How can make the UI elements be interactable with mouse as well?

Unity's UI system only allows using one input module and ours is just a VR input module.
I think we only allow mouse stuff on non-VRTK_UICanvases, so if you want to use one with both you're out of luck.
Maybe there's an easy way to support mice by just emitting pointer events from the mouse to our UI stuff? I don't really know 😄

A more important question is what the purpose of supporting both is. Is it because you want to click in the editor game window, click in the spectator view, or because you want to also support playing your game with a mouse and keyboard?

Any of the screen space canvases should work for mouse interaction. The example scenes use the SDK Setup Switcher prefab that offers such a GUI which isn't visible in the HMD.

@dantman  I was going for an application in which the UI could be controlled by the user using the Oculus, and the user outside the oculus (in front of the system).

@bddckr Mouse interaction works fine when I make the canvas mode to Screen space. But this disrupts the control from the Oculus touch.

I have a similar situation where I have a non-VR camera (rendered on monitor) that needs the ability to click on VRTK_UICanvas items alongside someone using the Vive.

As I said: Unity's event system only allows one active input module. If you find time to basically merge our VR one with theirs (which does mouse, keyboard and touch) go for it!

Y'all play nice please, just make a live stream example for the peeps who
don't know how to integrate the. Script. It'll take what 5-10minutes tops
On Nov 3, 2017 5:23 AM, "Christopher-Marcel Böddecker" <
<EMAIL_ADDRESS>wrote:

As I said: Unity's event system only allows one active input module. If
you find time to basically merge our VR one with theirs (which does mouse,
keyboard and touch) go for it!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/thestonefox/VRTK/issues/1554#issuecomment-341666065,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AfRctsBZWWHRq3eoAFjr6VhaC2iNRhhbks5syum5gaJpZM4P0rZR
.

@FreakinKat Not sure if bot, spam, sarcasm or dumb.
The technical issue was explained and hints have been given to add support for mouse interactions. The actual UI VR support is working just fine and is explained by an example scene and the documentation.

 It's
unauthorized use and not me.
On Nov 3, 2017 6:24 AM, "Christopher-Marcel Böddecker" <
<EMAIL_ADDRESS>wrote:

@FreakinKat https://github.com/freakinkat Not sure if bot, spam,
sarcasm or dumb.
The technical issue was explained and hints have been given to add support
for mouse interactions. The actual UI VR support is working just fine and
is explained by an example scene and the documentation.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/thestonefox/VRTK/issues/1554#issuecomment-341678261,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AfRctvPK5DoB8cTEv3xkdNjJlfpPoy0Vks5syvf8gaJpZM4P0rZR
.

@Eraile I tried your solution, but my mouse clicks still have no response on the menu.


[Style] Support backslashy method modifiers (the python decorator way)

I'd like to use the following syntax (possible since ruby 2.2), which is not compatible with rubocop
# memoize is a method modifier
memoize \
def testme
  "Hello"
end

# instead of
def testme
  "Hello"
end
memoize :testme

Upside:

def stays aligned with all the other def
DRY (method name is already returned by def in ruby 2.2)
feels like python decorators

Is there any way to use the existing rules, to allow it (without disabling rules)?
Will you support that syntax at some point?
(see http://thepugautomatic.com/2015/01/backslashy-ruby/)

I think that's a bad idea, as it's high unidiomatic (and abuses /). A common approach for this is:
memoize def testme
  "Hello"
end

I know but it's not applicable multiple time (or really ugly), and it hides the def part (personal taste).
ex
def testme
  "Hello"
end
memoize :testme
wrapme :testme
somehelper :testme

# vs

somehelper wrapme memoize def testme
  "Hello"
end

# vs

somehelper \
wrapme \
memoize \
def testme
  "Hello"
end

But anyway, even with the syntax you're proposing, rubocop is making warnings. Will this (the syntax you proposed) be supported ?

 Here are the contents of my test file
# encoding: utf-8
memoize \
  def testme
    'Hello'
  end

# instead of
def testme
  'Hello'
end
memoize :testme

# this is documentation
class Test
  memoize \
    def slash
      'Hello'
    end

  # instead of
  def testme
    'Hello'
  end
  memoize :testme

  private_class_method \
    def self.foo
    end

  helper_method \
    def current_user
    end
end

@vmeyet are you asking if RuboCop can suggest to use this style, or are you asking for RuboCop to not produce offenses/errors when processing this style?
In my opinion, since this is valid syntax, RuboCop should be able to process this code without error. That being said, RuboCop produces the following output when I run it against the example file.
$ b rubocop -D test.rb
Inspecting 1 file
An error occurred while Style/IndentationWidth cop was inspecting /Users/rrosenblum/work/rubocop/test.rb.
To see the complete backtrace run rubocop -d.
W

Offenses:

test.rb:28:5: W: Lint/DefEndAlignment: end at 28, 4 is not aligned with `private_class_method \
    def` at 27, 4
    end
    ^^^

1 file inspected, 1 offense detected

1 error occurred:
An error occurred while Style/IndentationWidth cop was inspecting /Users/rrosenblum/work/rubocop/test.rb.
Errors are usually caused by RuboCop bugs.
Please, report your problems to RuboCop's issue tracker.
Mention the following information in the issue report:
0.31.0 (using Parser <IP_ADDRESS>, running on ruby 2.2.2 x86_64-darwin14)

I will defer to @bbatsov and @jonas054 for any decisions being made, about the style and whether or not to have a cop for it. It does seem though, that we may want to look into Style/IndentationWidth and Lint/DefEndAlignment

@vmeyet are you asking if RuboCop can suggest to use this style, or are you asking for RuboCop to not produce offenses/errors when processing this style?

I'm asking to not produce offenses/errors or (better) having a rule where you can set an EnforcedStyle options.
(this could be an offense by default, but with the EnforcedStyle: backslash_method_modifier option, it will enforce that style).

Closing due to lack of activity.


[enhancement]: JUST serve as an on/off

Describe the solution you'd like
I use homekit for primary link to eufy cam. I want to be able to turn camera off via homekit. Can you make it so it can just toggle video on and off?

You could try the latest beta. I have made some changes that should enable you to do just what you want.
Please let me know, if this is what you meant.

Awesome! Thank you. Mind helping with the command? Not the most tech saavy

Awesome! Thank you. Mind helping with the command? Not the most tech saavy

To install beta? Here are the instructions

I tried turning the camera on/off and it seems broken to me.
When I click it off it goes back on, but the camera does turn off.. I just can't turn it back on? Anyway here is the logs I got:::
[7/14/2022, 10:36:49 PM] [homebridge-eufy-security] This plugin threw an error from the characteristic 'On': Unhandled error thrown inside write handler for characteristic: this.CameraService.getCharacteristic is not a function. See https://homebridge.io/w/JtMGR for more info.
[7/14/2022, 10:36:49 PM] [EufySecurity-2.1.0-beta.8] DEBUG: Dining Room Camera Property Changes: enabled false
[7/14/2022, 10:36:56 PM] [EufySecurity-2.1.0-beta.8] DEBUG: Dining Room Camera SET DeviceEnabled: false
[7/14/2022, 10:36:56 PM] [homebridge-eufy-security] This plugin threw an error from the characteristic 'On': Unhandled error thrown inside write handler for characteristic: this.CameraService.getCharacteristic is not a function. See https://homebridge.io/w/JtMGR for more info.
[7/14/2022, 10:37:01 PM] [EufySecurity-2.1.0-beta.8] INFO:  Looking for old cached accessories that seem to be outdated...
[7/14/2022, 10:37:01 PM] [EufySecurity-2.1.0-beta.8] INFO:  No outdated cached accessories found.

Found the bug. Will fix it in the next release. Stay tuned.

Sry, forgot to fix this in beta.9 will hopefully find it's way in beta.10 ;-)


Simpler icon getter

Presently need to specify player and profile twice

Rename current input_binding_get_icon() to input_binding_get_icon_ext(), and replace with input_binding_get_icon(verb, [playerIndex], [alternate], [profileName])

https://github.com/JujuAdams/Input/commit/7b43cf4f3abf5cd4f03d1590094572a634beb4e7


Detection model results

Hello, I want to get the model results FCViT-B24 for detection, this one is on google cloud drive, but I don't have access, thank you.

Just tested, it is publicly accessible.

Just tested, it is publicly accessible.

Sorry, I don't know what the problem is, the FCViT-B12 results for detection I have access to, but when I access FCViT-B24 Google Drive shows that I don't have permission.

@C-C-Y Thanks for the issue. Updated. Please double-check again.


[Docker Desktop] Desktop Images UI

Tell us about your request
Provide a UI with Visibility into Hub and local images in Docker Desktop, with the ability to manage local images (prune, update etc)
Which service(s) is this request for?
Desktop
Tell us about the problem you're trying to solve. What are you trying to do, and why is it hard?

N/A
Additional context
N/A
Attachments

In the initial desktop release of images, the container launch panel supports port and mount setting.

It would also be used to allow the user to enter one or more environment variable name-vale pairs, replicating the docker run -e flag.

@psychemedia thank you for the feedback! I am going to mark this ticket as closed for the initial ship but we do have plans to add some more items including env and command for starting the container :)


Ubuntu for IoT failed to find memory cgroup

Version:
k3s version v0.9.1
Describe the bug
I am not running a Raspberry Pi4 ubunut for iot.
config cgroup not working
$ vim /boot/cmdline.txt
# text
dwc_otg.fiq_fix_enable=2 console=ttyAMA0,115200 kgdboc=ttyAMA0,115200 console=tty1 root=/dev/mmcblk0p2 rootfstype=ext4 rootwait rootflags=noload net.ifnames=0 cgroup_memory=1 cgroup_enable=memory

console output
INFO[2019-11-14T18:15:28.309895304Z] Wrote kubeconfig /etc/rancher/k3s/k3s.yaml
INFO[2019-11-14T18:15:28.310050786Z] Run: k3s kubectl
INFO[2019-11-14T18:15:28.310106363Z] k3s is up and running
ERRO[2019-11-14T18:15:28.310530982Z] Failed to find memory cgroup, you may need to add "cgroup_memory=1 cgroup_enable=memory" to your linux cmdline (/boot/cmdline.txt on a Raspberry Pi)
FATA[2019-11-14T18:15:28.310661410Z] failed to find memory cgroup, you may need to add "cgroup_memory=1 cgroup_enable=memory" to your linux cmdline (/boot/cmdline.txt on a Raspberry Pi)

To Reproduce
Run sudo k3s server on ubunut for iot
Expected behavior
k3s starts successfully and continues to run
Additional context
Running ubunut for iot

Thanks for the info!
Would you mine sharing the output of:
curl -sfL https://raw.githubusercontent.com/rancher/k3s/master/contrib/util/check-config.sh | sh -

It you're running ubuntu server (ubuntu-19.10-preinstalled-server-<armhf/arm64>+raspi3) I believe the correct file to update the cmdline is either nobtcmd.txt (default) or btcmd.txt (if you changed the config to enabled bluetooth).

You can run cat /proc/cmdline to see the cmdline of the running system to verify that cgroup_memory stuff is actually used.

@erikwilson Thank you

Thanks @zimme!!
the path for (ubuntu-19.10-preinstalled-server-<armhf/arm64>+raspi3) is
/boot/firmware/nobtcmd.txt

Can this be closed now or is there anything else needed?

I have to say this answer was hard to find. Had issues with RKE and k3s installs on pi's ubuntu 18.04 64 bit arm builds.

unable to do it with ubuntu 18.04 on rpi 4. What file to be edited??

For raspberry pi 4 the file to be edited is /boot/firmware/nobtcmd.txt

For the record, I think you're actually supposed to edit /etc/default/flash-kernel.

Thanks @zimme!!
the path for (ubuntu-19.10-preinstalled-server-<armhf/arm64>+raspi3) is
/boot/firmware/nobtcmd.txt

thnx you both
the path for  (ubuntu-20.04-preinstalled-server-arm64+raspi) is
'/boot/firmware/cmdline.txt'

The stuff in /boot/firmware/cmdline.txt is merged with the contents of /etc/default/flash-kernel. I don't know what the point of that is other than letting you easily edit the boot options for a broken system by sticking the card in a Windows PC.

on ubuntu 20.04, /etc/default/flash-kernel didn't seem to work for me. Adding to boot/firmware/cmdline.txt did.


Redirect URI incorrect when used with Express

I'm using Authom like this:
app.use('/auth/:service', authom.app);

I'm trying to authentication with Google, which uses OAuth2.  The redirect_uri passed to Google contains / instead of /auth/google.
I think the problem is in oauth2.js where req.url is used:
this.code.query.redirect_uri = url.format(req.url)

If I add these two lines above this, the problem is resolved:
  req.url.pathname = req.baseUrl;
  delete req.url.href;

Similar changes still need to be made later on for onCode.
In any case, I think that originalUrl needs to come into play here because /auth/google isn't in req.url.  Is this the correct fix?  I would submit a pull request, but since this is an as-documented usage of Authom, I wanted to make sure I wasn't doing anything wrong to begin with.  Any insight is welcomed!  Thanks.

I made some changes in a fork:  https://github.com/bradisbell/authom/commit/894fc773bed45160cfbe2de999574c807a038bac  If that change looks good, I will put in a pull request.  But, please let me know if this commit doesn't make any sense.

Is this going to be merged in? i'm still getting this issue


Conditional workflow - disable condition on job props

Currently there is no common use case for condition on job props. So we will disable it in phase one. If needed in the future, we can add the support back.
Conditional workflow design - https://github.com/azkaban/azkaban/issues/1826

Pull Request Test Coverage Report for Build 3974

0 of 0   changed or added relevant lines in 0 files are covered.
4 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.01%) to 35.961%

Files with Coverage Reduction
New Missed Lines
%

azkaban-exec-server/src/main/java/azkaban/execapp/FlowRunner.java
1
80.46%

azkaban-common/src/main/java/azkaban/jobExecutor/AbstractProcessJob.java
3
63.1%

Totals

Change from base Build 3965:
-0.01%

Covered Lines:
10398

Relevant Lines:
28915

💛  - Coveralls