Code sample for System.ComponentModel.DesignerAttribute is wrong

Describe the bug
[Designer("System.Windows.Forms.Design.DocumentDesigner, System.Windows.Forms.Design.DLL",
typeof(IRootDesigner)),
DesignerCategory("Form")]
public class MyForm : ContainerControl {
// Insert code here.
}
The designer type should come from System.Design instead, or use Strong Type to reference it.
See: https://social.msdn.microsoft.com/Forums/vstudio/en-US/37d0819d-a3bf-46ad-ad99-cd93cadda81b/how-to-make-event-in-the-class-inherited-from-form?forum=csharpgeneral
To Reproduce

Create a "Windows Form Application" project.
Add a class FormBase inherited from System.Windows.Forms.Form. Use the described attributes from the documentation sample.
Change the default added Form1 to inherit from FormBase instead.
Switch to "Form Designer" in Form1. You should see error message regarding "instance" parameter is null because the referred DLL cannot be found.

Expected behavior
After change to use strong name as I described in MSDN forum (I actually copied the declaration from "Object Browser" for System.Windows.Forms.Form ), you should be able to use "Form Designer" in Form1.cs.
Desktop (please complete the following information):

Platform: .NET Framework v4.X

@cheong00 - is the feedback related to a sample that you saw somewhere on docs.microsoft.com?

Yes.
https://docs.microsoft.com/en-us/dotnet/api/system.componentmodel.designerattribute?view=netframework-4.8
And for all version from 2.0 all the way up to 4.8. There aren't "System.Windows.Forms.Design.DLL" exists at least in all full .NET framework versions.

I can't see it in .NET Core 1.X and 2.X, but that is normal because they don't support WinForm applications.
And I can't find any of the two DLLs in .NET Core 3 folder, not sure whether it is because the windows form designer for .NET Core 3 is not ready yet, or it should be at some other DLL.


[Fix-it] Support defaulted and variadic arguments in renamed fix-it

Explanation: Our diagnostics for renamed functions was bailing out whenever any of the arguments were defaulted at the call site, leaving only a fix-it for the base name. If the base name was the same, this was especially confusing.

Scope: The code path affects all rename diagnostics that include parameters. Correct code should not be affected by this change.

Issue: SR-2264

Reviewed by: @jrose-apple (authored by @rintaro)

Risk: Low.

Testing: Added to existing compiler regression tests.

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    All supported platforms     | @swift-ci Please smoke test and merge
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

A smoke test on macOS does the following:

Builds the compiler incrementally.
Builds the standard library only for macOS. Simulator standard libraries and
device standard libraries are not built.
lldb is not built.
The test and validation-test targets are run only for macOS. The optimized
version of these tests are not run.

A smoke test on Linux does the following:

Builds the compiler incrementally.
Builds the standard library incrementally.
lldb is built incrementally.
The swift test and validation-test targets are run. The optimized version of these
tests are not run.
lldb is tested.

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    All supported platforms     | @swift-ci Please test and merge
    OS X platform               | @swift-ci Please test OS X platform
    OS X platform               | @swift-ci Please benchmark
    Linux platform              | @swift-ci Please test Linux platform

Lint Testing
    Language     | Comment
    ------------ | -------------
    Python       | @swift-ci Please Python lint

Note: Only members of the Apple organization can trigger swift-ci.

@swift-ci Please test

Approved by @tkremenek via email.


Create separate data structure for public inputs of the private/public kernel

In C++, the data structure KernelCircuitPublicInputs is used for BOTH private and public kernel.
In TS, KernelCircuitPublicInputs is also defined as a parent class for both types.
The goal of this task is to have separate self-contained classes for both types of Kernel.
CombinedAccumulatedData will have 2 flavours as well. Some fields like public_data_reads and public_data_update_requests can be removed from the private kernel. Some fields might be removed from the public one as well.
Note that PreviousKernelData will be affected as well.
This task is a follower of #1879.

Not relevant anymore since circuits were migrated to Noir.


Fix double alias of complex export/import/default/namespace combination

Fixes a case where we failed to resolve an aliased symbol when the following combination of features are employed:

An exported namespace declaration that merges with a function and that has an export named default
An import of the exported namespace in (1)
Exporting (2) using export=
Importing (3) using a default import while --esModuleInterop is set.

This seems to be caused by the fact that we weren't calling resolveSymbol for an export resolved from an export= symbol in resolveExportByName, resulting in a Symbol that was an alias to an alias, rather than the actual target.
Fixes #39149

@typescript-bot perf test
@typescript-bot run dt
@typescript-bot test this
@typescript-bot user test this


(#9440) exec resource: set path param to path fact

Can one of the admins verify this patch?

@joshcooper can you take a look again please?


fix: enable gw apis with broken lb svc

What this PR does / why we need it:
This PR makes it possible for Gateway resources to be considered usable even if they are attached to a LoadBalancer type Service which has not yet had any IP addresses (or hostnames) provisioned for it.
Which issue this PR fixes
Resolves #2413
PR Readiness Checklist:

[x] the CHANGELOG.md release notes have been updated
[ ] manual testing

Did some manual testing around this by deploying a ktf environment where the Gateway's Service is LoadBalancer type, but metallb was removed leaving the service in a <pending> state:
$ k -n kong-system get svc
NAME                            TYPE           CLUSTER-IP     EXTERNAL-IP   PORT(S)                                                                   AGE
ingress-controller-kong-admin   ClusterIP      <IP_ADDRESS>    <none>        8001/TCP                                                                  13m
ingress-controller-kong-proxy   LoadBalancer   <IP_ADDRESS>   <pending>     80:31131/TCP,443:30031/TCP,8888:32599/TCP,9999:31448/TCP,8899:31300/TCP   13m
ingress-controller-kong-udp     ClusterIP      <IP_ADDRESS>   <none>        9999/UDP

While in this state with the new short circuit, the Gateway did become Ready with only it's ClusterIP for an address:
$ k get gateway kong
NAME   CLASS   ADDRESS        READY   AGE
kong   kong    <IP_ADDRESS>   True    5m6s

$ k -n kong-system port-forward svc/ingress-controller-kong-proxy 8080:80
Forwarding from <IP_ADDRESS>:8080 -> 8000
Forwarding from [::1]:8080 -> 8000
^Z
[1]+  Stopped                 kubectl -n kong-system port-forward svc/ingress-controller-kong-proxy 8080:80
gaming@gaming:~/Code/kubernetes-ingress-controller$ bg
[1]+ kubectl -n kong-system port-forward svc/ingress-controller-kong-proxy 8080:80 &
$ curl -s localhost:8080/httproute-testing | rg httpbin
Handling connection for 8080
    <title>httpbin.org</title>

Afterwards I installed metallb and got an IP provisioned:
$ k -n kong-system get svc
NAME                            TYPE           CLUSTER-IP     EXTERNAL-IP    PORT(S)                                                                   AGE
ingress-controller-kong-admin   ClusterIP      <IP_ADDRESS>    <none>         8001/TCP                                                                  19m
ingress-controller-kong-proxy   LoadBalancer   <IP_ADDRESS>   <IP_ADDRESS>   80:31131/TCP,443:30031/TCP,8888:32599/TCP,9999:31448/TCP,8899:31300/TCP   19m
ingress-controller-kong-udp     ClusterIP      <IP_ADDRESS>   <none>         9999/UDP

The Gateway's addresses were updated accordingly, and the default address became the LoadBalancer IP address:
$ k get gateway kong -o=go-template='{{.spec.addresses}}{{"\n"}}'
[map[type:IPAddress value:<IP_ADDRESS>] map[type:IPAddress value:<IP_ADDRESS>]]
$ k get gateway kong
NAME   CLASS   ADDRESS        READY   AGE
kong   kong    <IP_ADDRESS>   True    12m

And routes are working with the LB ip:
$ curl -s http://<IP_ADDRESS>/httproute-testing | rg httpbin
    <title>httpbin.org</title>

With that and integration tests I'm satisfied.


Place "stop generating" button instead of "send" button when message generating

Description

Also let's add standard tooltip for the icon
Use case/motivation
No response
Related issues
No response
Confidential information

[X] I confirm that do not share any confidential information

Verified on staging successfully


Support new reply type "Opinionated"

Properly display the replies tagged "opinionated" alongside with existing rumor/non-rumor replies.
String copies of "opinionated" type should align with rumors-site.

Duplicated by #32, close this.


PEP8 Support

I want to know that how can i make it easy in VS2017 to formate my python code by pep8

Sorry for not noticing your question so late.
The functionality currently available is accessed via Edit->Advanced->Format Document and it is configurable in Tools/Options/Text Editor/Python
We have these items that track improvements:
https://github.com/Microsoft/PTVS/issues/4694
https://github.com/Microsoft/PTVS/issues/4698

Thanks


New version: NonlinearSolve v3.12.6

Registering package: NonlinearSolve
Repository: https://github.com/SciML/NonlinearSolve.jl
Created by: @ChrisRackauckas
Version: v3.12.6
Commit: 9118d0219cceb7b2468f7a4e39cbb8f33436c11a
Reviewed by: @ChrisRackauckas
Reference: https://github.com/SciML/NonlinearSolve.jl/issues/17#issuecomment-2165825734
Description: High-performance and differentiation-enabled nonlinear solvers (Newton methods), bracketed rootfinding (bisection, Falsi), with sparsity and Newton-Krylov support.

[noblock]
@ChrisRackauckas See AutoMerge comment above.


Update components.md

Updating prop example to include all fields referenced in previous example

Thanks for the effort! However, this is 1. IMO unnecessary for our purpose here and 2. incorrect, as post.comments would normally be an array and can't be used directly with v-html this way.


Fix for the incoming IPv4 destination IP check

Description
When ipconfigETHERNET_DRIVER_FILTERS_PACKETS is disabled, the stack needs to filter out packets based on their destination IP. In reality it doesn't.
Test Steps

#define ipconfigETHERNET_DRIVER_FILTERS_PACKETS (0)
#define ipconfigETHERNET_DRIVER_FILTERS_FRAME_TYPES (1) Note below.
Modify your network driver to allow all packets ( promiscuous mode ).
Inject an IPv4 packet with any unicast source and unicast destination addresses. Such packets can an will occur in the wild even with modern switches.
Note that prvAllowIPPacketIPv4() returns eProcessBuffer for the injected packet and processing continues as if the packet was destined for the DUT.
Once the change from this PR is applied, the DUT correctly drops the packets for which it does not have an end-point.

Random Thoughts
I don't know where that NULL check came from but as of right now, any received packet will have a non-NULL pxNetworkBuffer->pxEndPoint as it propagates through the stack.
Note on: I understand that in promiscuous mode, ipconfigETHERNET_DRIVER_FILTERS_FRAME_TYPES should probably be disabled, however the default is ipconfigENABLED and the IP layer should be capable of filtering out packets regardless of whether the lower layers did their job or not.
Let me know what you think
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@evpopov Thanks for the PR. I'll get this over to one of our FreeRTOS+TCP experts for review.

@shubnil  and @tony-josi-aws
I'm closing this PR because it modifies a public API function's parameters.
I will wait for #1086 to get merged because it deals with the same part of the source and then I will open a new PR that achieves the same goal as this one here, but without modifying public API functions.

/bot run formatting


fix: bump sc4s/splunk versions

:tada: This PR is included in version 1.8.20 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


EHF Electronic Invoice in Norway for Dynamics BC Web Client

How to set up output path for EHF Electronic Invoice for running Dynamics BC Web Client?

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

ID: e8576fb5-7e8b-7c36-63f2-b5e5273d96a5
Version Independent ID: a56d32fa-38dc-50a5-ce16-12d027411267
Content: How to Create Electronic Documents for EHF - Business Central
Content Source: business-central/LocalFunctionality/Norway/how-to-create-electronic-documents-for-ehf.md
Service: dynamics365-business-central
GitHub Login: @SorenGP
Microsoft Alias: sgroespe

Is this what you are looking for? https://docs.microsoft.com/en-us/dynamics365/business-central/localfunctionality/norway/how-to-set-up-ehf

Closing as the issue seems to be closed with the above answer.


Fails with targets of "iPhone-6s, 9.2" and "iPhone-6s-Plus, 9.2"

I've done ios-sim showdevicetypes and get the following
iPhone-4s, 8.4
iPhone-5, 8.4
iPhone-5s, 8.4
iPhone-6, 8.4
iPhone-6-Plus, 8.4
iPad-2, 8.4
iPad-Retina, 8.4
iPad-Air, 8.4
iPhone-4s, 9.2
iPhone-5, 9.2
iPhone-5s, 9.2
iPhone-6, 9.2
iPhone-6-Plus, 9.2
iPhone-6s, 9.2
iPhone-6s-Plus, 9.2
iPad-2, 9.2
iPad-Retina, 9.2
iPad-Air, 9.2
iPad-Air-2, 9.2
iPad-Pro, 9.2
but if I set the target to iPhone-6s, 9.2 or iPhone-6s-Plus, 9.2 then it fails with
not a valid target for emulator
Other targets work fine.

I believe this is a Cordova iOS issue - this has been fixed https://github.com/apache/cordova-ios/commit/07fd9379872b949ce651efb9c4988b3d0bbb1b94
It should be out in the next platform release (> 4.0.1)


Improve code (notebook) list page

From: https://conf.casbin.com/code

To: https://www.kaggle.com/code

I am working on this


Add class ConvexHull3D

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

61 of 63 (96.83%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.3%) to 97.596%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/plxcontroller/geometry_3d/convex_hull_3d.py
43
45
95.56%

Totals

Change from base Build<PHONE_NUMBER>:
-0.3%

Covered Lines:
203

Relevant Lines:
208

💛  - Coveralls


Remove all unconditionally-trigger-refresh hack, such as dummy++, and create real when-to-refresh logic

Warn: simply removing it can cause some ui not refreshed correctly. need to carefully test and reason about where to add refresh logic.

pending #14

already removed. now wait #19


Mistake in text: Chapter <IP_ADDRESS>, Page 122, lines 40-41 (pdf), Version: 2022-10-16

Text says "since it is more likely that the low grade is explained by the class being hard"
but that is not correct...
The low grade could be explained by the class being hard, but is more likely explained by the student having low intelligence.
This correction is consistent with the text further on, in Section <IP_ADDRESS> where the example is revisited and correctly explained.
The sentence in <IP_ADDRESS> should be revised to match and more clearly contrast low-intelligence vs difficult class, rather than the way it is worded now, as high vs difficult  (as the latter are not competing explanations).

good point, fixed


Help getting CEPHFS working

I am trying to get CEPHFS working. The procedure I am following is the one found here. The procedure follows the section on RBD.

I've struggled with the process, and I have one outstanding issue. I am documenting the process in this issue, hopefully for the benefit of others, but also because I'd like help relative to the final issue.
I already have a CEPH cluster up, and a seperate K8S cluster, and running.

The steps are as follows:

Create the namespace: kubectl create ns cephfs
Create the secret: kubectl create secret generic ceph-secret-admin --from-literal=key="AQDJtspdXMyJLRAAZrRBzSyGR2rG5UqLHuDnAw==" -n cephfs
Create the provisioner: kubectl create -n cephfs -f Ceph-FS-Provisioner.yaml
Create the storage class: kubectl create -f Ceph-FS-StorageClass.yaml
Create a PVC: kubectl create -f Ceph-FS-PVC.yaml

I've updated the Ceph-FS-Provisioner.yaml to be consistent with K8S 1.16 (e.g. Deployment no longer in extensions/v1beta1, and now has a selector tag)
Further details as follows. The key I am using in step 2:

The creation of the provisioner (step 3):

The creation of the storage class (step 4):

Note that this storage class is defined as follows:

Specifically it uses a claimRoot /pvc-volumes
When I try to create the PVC it never binds.

My provisioner pod:

Its description:

I dont quite understand the claimRoot /pvc-volumes. I have not exported this from CEPH. I am guessing that I need to based on the comment I see here

Do I need to export "/pvc-volumes" and if so, does someone know the command for this?
Thanks.
As an aside, the tail end of the log on the provisioner pod

Hi, look at this:
https://github.com/kubernetes-incubator/external-storage/issues/941#issuecomment-418971539

@kkostin Ty. I eventually got CEPH RBD working. If I ever need to revisit CEPH FS then I am sure I'll reference this issue.
For now I'll close it.


Misleading error message when test project is missing test runner

Moved from https://github.com/dotnet/cli/issues/5135 on behalf of @bleroy

Steps to reproduce
Create a new console app, add the xunit package, write a few tests, try to dotnet restore and dotnet test.
Expected  behavior
An error message saying to add a test runner, ideally suggesting the right one based on what testing framework is referenced if we have a way to recognize that at this point.
Actual behavior
An error message suggesting to add "microsoft.testplatform.testhost". The right packages to add to make it work really are Microsoft.NET.Test.Sdk and xunit.runner.visualstudio.
Environment data
Reproduces on Preview 3 and Preview 4.

For one more scenario this message should be improved:
Scenario
- Create a unit test project targeting net46
- Try to run it with "dotnet vstest path-of-unit-test-net46-dll"

It should give message to check framework as well because when we run test from "dotnet vstest"
the default framework is netcoreapp1.0.


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/33490

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/34040

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/35283

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/35628

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/36128

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/53237

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/91879

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/92908

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/98239


Favicon is not set via manifest.json

Hello colleagues,
Could you please clarify a proper way to set the favicon.ico of the application? Let's assume that it's hosted as a standalone application on the SCP.
I see three ways to do it:

Set it via manifest.json file. This either doesn't work or I don't understand how to use this:

"sap.ui": {
	"technology": "UI5",
	"icons": {
		"icon": "",
		"favIcon": "img/favicon.ico",
		"phone": "",
		"phone@2": "",
		"tablet": "",
		"tablet@2": ""
	},

Set favicon via Shell object in index.html file - working fine:

new sap.m.Shell({
	homeIcon: {
		favicon: "img/favicon.ico"
	},
	app: new sap.ui.core.ComponentContainer({
		height: "100%",
		name: ""
	})
}).placeAt("content");

Set via image/x-icon link in index.html - working fine:

<link href="img/favicon.ico" rel="shortcut icon" type="image/x-icon" />
Thanks,
Ilya

As far as I understand it, the entry in the manifest.json is not meant to automatically show that icon, but it is an information for a surrounding launcher what icon could be used to represent that app.
The framework code that instantiates components (sap.ui.component) does not know how created components will be shown to the user or whether the currently created component qualifies as "the current application" or only as a helper (reuse component). Such a decision is left to the launcher.
Fiori Launch Pad (FLP) for example reads the entry from the manifest of the currently active application and - when not empty - sets it with a call
    jQuery.sap.setIcons({
         'phone': '',
         'phone@2': '',
         'tablet': '',
         'tablet@2': '',
         'favicon': oFavIcon,
         'precomposed': true
    });

I'm not aware of any code in the OpenUI5 repository that would do this automatically for a standalone app. Whether the Shell control would be in a position to take that decision, I can't judge. In the context of the FLP for example, the shell control is one layer deeper than the launcher, but that might not be a valid argument.

Same is discussed here - https://blogs.sap.com/2017/01/08/complete-the-branding-of-your-ui5-apps-with-a-favicon/
The settings in the manifest.json are for FLP - https://help.sap.com/saphelp_uiaddon10/helpdata/en/d4/09e4c8940743ccb173e9ee8bbaa3a8/frameset.htm
I am forwarding internally the issue for reference with #1870229007

Additionally to codewarriors comment ...
The favIcon can also be set using the Mobile utils module. You can find the details here:
Mobile.setIcons()

@stephania87 Could this issue be reopened?
I think the attribute sap.ui/icons should be either hidden completely from OpenUI5 documentation Descriptor for Applications, Components, and Libraries, or the attribute should be supported for stand-alone apps as well. Otherwise, it's confusing.


Implements support for PHP 5.3

Hi KuiKui,
your wish is my command. Can this be merged?
Have fun,
Hendrik

Can you add a new test in ServiceProvider.php :
    public function testRegisterWrapperSelection()
    {
        $app = new Application();
        $provider = new MemcacheServiceProvider\ServiceProvider();

        $provider->register($app);

        $this->object($app['memcache'])
            ->isInstanceOf('\KuiKui\MemcacheServiceProvider\AbstractWrapper')
        ;
    }

Or a better one :wink:

The pattern is fairly common in JavaScript, where closures are used a bit longer than in PHP. There you just don't have to make the copy of $this explicitly available in the closure as you have to in PHP.
I changed the name of the var, added a comment and your test.

Thanks !

Fichtre!
We have to fix tests :sweat:
I check this tomorrow.

Can you merge master in your feature branch (I've updated Travis config) ?

Master was merged into a new feature branch, see other pull request.


Put in higher rate limit jupiter swap url

This PR adjusts the API provided for the jupiter package. The default API was updated for swap/quote to leverage jupiterapi.com. The API offers higher rate limits (up to 10 rps) along with a faster updated market cache. The jupiterapi.com market cache surfaces newly launched tokens faster since it doesn't require a certain volume/liquidity threshold to be met, along with an improved cadence of it being updated. With that said, the API does take a small platform fee (0.2%) on swaps given the stability/rate limits/new markets it provides.
More details can be found on the site.

@ChewingGlass it should go without saying:
DO NOT APPROVE THIS PR.

This is clearly sus


[DUOS-1905][risk=no] Update Researcher Console to use Summary Endpoint

Addresses
https://broadworkbench.atlassian.net/browse/DUOS-1905
In dev, the summary endpoint does not return reference ids. We need them so that draft dar collections can be resumed. The backend change to add that is here: https://github.com/DataBiosphere/consent/pull/1670

Have you read Terra's Contributing Guide lately? If not, do that first.

Label PR with a Jira ticket number and include a link to the ticket
Label PR with a security risk modifier [no, low, medium, high]
PR describes scope of changes
Get a minimum of one thumbs worth of review, preferably two if enough team members are available
Get PO sign-off for all non-trivial UI or workflow changes
Verify all tests go green
Test this change deployed correctly and works on dev environment after deployment

Not sure where the bug originates, but if I set the page count to 100 on the researcher console, I get the following error:
ResearcherActions
src/components/dar_collection_table/ResearcherActions.js:48
  45 | const { collection, showConfirmationModal } = props;
  46 | const collectionId = collection.darCollectionId;
  47 |
> 48 | const uniqueId = (collectionId ? collectionId : collection.referenceIds[0]);
     | ^  49 |
  50 | const history = useHistory();
  51 |

Not sure where the bug originates, but if I set the page count to 100 on the researcher console, I get the following error:
TypeError: Cannot read properties of undefined (reading '0')
ResearcherActions
src/components/dar_collection_table/ResearcherActions.js:48
  45 | const { collection, showConfirmationModal } = props;
  46 | const collectionId = collection.darCollectionId;
  47 |
> 48 | const uniqueId = (collectionId ? collectionId : collection.referenceIds[0]);
     | ^  49 |
  50 | const history = useHistory();
  51 |

I'm not sure how to recreate this. I ran into the same bug when there was a bug with the consent backend; this happens in the case that there is a row which is both a draft DAR and the backend does not send a referenceId alongside it. When I create a new draft DAR locally, this doesn't happen to me. Would it be possible that there is a draft DAR in your database which does not have a referenceId? If so, that would cause this error, and I can add some guards to ensure legacy drafts don't break the UI.

Would it be possible that there is a draft DAR in your database which does not have a referenceId?

This scenario should be technically impossible since all dars, draft or not, are saved with a reference id. I'll spend some time digging further now that I have some more time.

Closing, changes already incorporated in https://github.com/DataBiosphere/duos-ui/pull/1699


test: add missing screenshots for safari

Codecov Report

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

@@           Coverage Diff           @@
##           master    #2423   +/-   ##
=======================================
  Coverage   94.00%   94.00%
=======================================
  Files          84       84
  Lines        5470     5470
  Branches      964      964
=======================================
  Hits         5142     5142
  Misses        297      297
  Partials       31       31

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 42cc5e5...54276da. Read the comment docs.

@ylemkimon I'll remember to check for Safari screenshots in the future.  Thanks adding these.


Adding slack link to Bug report template

Adding slack link to Bug report template

it should be at the end; else, users will put all the issues in slack, and we won't be able to meaningfully triage in time.


Add Cockpit

See Cockpit website.

Cool, thanks for the addition. Sorry for the delay in merging!


Error on portworx instance not found during install

Keep getting this while running the tf
Error: Error while removing volume attachment "0757-e8e9a0a9-3131-478e-9f06-54d452f510ef" for instance 0757_c74fb7bb-8aca-49ed-a071-a9f620eab1c6: "Instance not found"

@tcskill   Are you still facing issue with Portworx Instance.  We have recently upgraded our scripts to install CPD 4.5.0.  Can you please give it a try and let us know if the issue is resolved.  In case it is not,  Can you please provide more details on

Which environment you are trying to install Cloud Pak for Data on ?
Any logs from your installation that you could provide us for debugging purpose.
Thank you.

Closing this issue , hoping the issue is no longer faced


[MultiprocessFTPServer] TypeError: cannot pickle 'select.kqueue' object

Hello,
[ TypeError: cannot pickle 'select.kqueue' object ] occured when I tried to connect to the server.
I used MultiprocessFTPServer as Multiple process example shows in https://pyftpdlib.readthedocs.io/en/latest/tutorial.html
But If I change MultiprocessFTPServer() to FTPServer(), it works well. I can send files easily.
Please advise how to use multiprocess correctly.
My environment and terminal results from server&client are as follow.
Environment--------------
server:
python3.8.9
macOS 11.6.1 (M1)
pyftpdlib1.5.6
client:
python3.8.2
macOS 10.15.7
pyftpdlib1.5.6
Terminal results (server)--------------
notaryman@Notarymans-Mac-mini sign % python3 sign_server.py
[I 2021-11-18 10:30:12] concurrency model: multi-process
[I 2021-11-18 10:30:12] masquerade (NAT) address: None
[I 2021-11-18 10:30:12] passive ports: 60000->61999
[I 2021-11-18 10:30:26] ::ffff:<IP_ADDRESS>:55080-[] FTP session opened (connect)
[E 2021-11-18 10:30:26] Traceback (most recent call last):
      File "/Applications/Xcode.app/Contents/Developer/Library/Frameworks/Python3.framework/Versions/3.8/lib/python3.8/asyncore.py", line 83, in read
        obj.handle_read_event()
      File "/Applications/Xcode.app/Contents/Developer/Library/Frameworks/Python3.framework/Versions/3.8/lib/python3.8/asyncore.py", line 414, in handle_read_event
        self.handle_accept()
      File "../pyftpdlib/ioloop.py", line 1050, in handle_accept
        self.handle_accepted(sock, addr)
      File "../pyftpdlib/servers.py", line 458, in handle_accepted
        t.start()
      File "/Applications/Xcode.app/Contents/Developer/Library/Frameworks/Python3.framework/Versions/3.8/lib/python3.8/multiprocessing/process.py", line 121, in start
        self._popen = self._Popen(self)
      File "/Applications/Xcode.app/Contents/Developer/Library/Frameworks/Python3.framework/Versions/3.8/lib/python3.8/multiprocessing/context.py", line 224, in _Popen
        return _default_context.get_context().Process._Popen(process_obj)
      File "/Applications/Xcode.app/Contents/Developer/Library/Frameworks/Python3.framework/Versions/3.8/lib/python3.8/multiprocessing/context.py", line 284, in _Popen
        return Popen(process_obj)
      File "/Applications/Xcode.app/Contents/Developer/Library/Frameworks/Python3.framework/Versions/3.8/lib/python3.8/multiprocessing/popen_spawn_posix.py", line 32, in __init__
        super().__init__(process_obj)
      File "/Applications/Xcode.app/Contents/Developer/Library/Frameworks/Python3.framework/Versions/3.8/lib/python3.8/multiprocessing/popen_fork.py", line 19, in __init__
        self._launch(process_obj)
      File "/Applications/Xcode.app/Contents/Developer/Library/Frameworks/Python3.framework/Versions/3.8/lib/python3.8/multiprocessing/popen_spawn_posix.py", line 47, in _launch
        reduction.dump(process_obj, fp)
      File "/Applications/Xcode.app/Contents/Developer/Library/Frameworks/Python3.framework/Versions/3.8/lib/python3.8/multiprocessing/reduction.py", line 60, in dump
        ForkingPickler(file, protocol).dump(obj)
    TypeError: cannot pickle 'select.kqueue' object

Terminal results (client)--------------
Junyeops-iMac:Utils junyeopjeon$ python3 /Users/junyeopjeon/Downloads/ftp_test.py
Traceback (most recent call last):
  File "/Users/junyeopjeon/Downloads/ftp_test.py", line 465, in main
    obj_ftp.connectToServer()
  File "/Users/junyeopjeon/Downloads/ftp_test.py", line 449, in connectToServer
    self.ftp.login( self.SIGN_SERVER_LOGIN_ID, self.SIGN_SERVER_LOGIN_PW )    # user name, passwd
  File "/Applications/Xcode.app/Contents/Developer/Library/Frameworks/Python3.framework/Versions/3.8/lib/python3.8/ftplib.py", line 401, in login
    resp = self.sendcmd('USER ' + user)
  File "/Applications/Xcode.app/Contents/Developer/Library/Frameworks/Python3.framework/Versions/3.8/lib/python3.8/ftplib.py", line 275, in sendcmd
    return self.getresp()
  File "/Applications/Xcode.app/Contents/Developer/Library/Frameworks/Python3.framework/Versions/3.8/lib/python3.8/ftplib.py", line 238, in getresp
    resp = self.getmultiline()
  File "/Applications/Xcode.app/Contents/Developer/Library/Frameworks/Python3.framework/Versions/3.8/lib/python3.8/ftplib.py", line 224, in getmultiline
    line = self.getline()
  File "/Applications/Xcode.app/Contents/Developer/Library/Frameworks/Python3.framework/Versions/3.8/lib/python3.8/ftplib.py", line 206, in getline
    line = self.file.readline(self.maxline + 1)
  File "/Applications/Xcode.app/Contents/Developer/Library/Frameworks/Python3.framework/Versions/3.8/lib/python3.8/socket.py", line 669, in readinto
    return self._sock.recv_into(b)
ConnectionResetError: [Errno 54] Connection reset by peer

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/Users/junyeopjeon/Downloads/ftp_test.py", line 475, in <module>
    ret = main()
  File "/Users/junyeopjeon/Downloads/ftp_test.py", line 469, in main
    obj_ftp.disconnectToServer()
  File "/Users/junyeopjeon/Downloads/ftp_test.py", line 452, in disconnectToServer
    self.ftp.quit()
  File "/Applications/Xcode.app/Contents/Developer/Library/Frameworks/Python3.framework/Versions/3.8/lib/python3.8/ftplib.py", line 648, in quit
    resp = self.voidcmd('QUIT')
  File "/Applications/Xcode.app/Contents/Developer/Library/Frameworks/Python3.framework/Versions/3.8/lib/python3.8/ftplib.py", line 279, in voidcmd
    self.putcmd(cmd)
  File "/Applications/Xcode.app/Contents/Developer/Library/Frameworks/Python3.framework/Versions/3.8/lib/python3.8/ftplib.py", line 201, in putcmd
    self.putline(line)
  File "/Applications/Xcode.app/Contents/Developer/Library/Frameworks/Python3.framework/Versions/3.8/lib/python3.8/ftplib.py", line 196, in putline
    self.sock.sendall(line.encode(self.encoding))
BrokenPipeError: [Errno 32] Broken pipe

P.S. The same "TypeError: cannot pickle 'select.kqueue' object" message occurs during [make test ] with python3.10 as install section instructs in https://pyftpdlib.readthedocs.io/en/latest/install.html...

It was solved by changing the setting to use fork() instead of spawn() which is the default method on masOS. Thanks.


[FEATURE] clear build folder

This PR address the following issue: #45
Pull Request Checklist

[x] Reviewed the Contributing Guidelines

Especially the How to Contribute section

[x] No merge commits
[x] Correct commit message style

Coverage increased (+0.01%) to 84.672% when pulling acf55b2d479458a0ca1f61019e18d7334f66ea41 on mauriciolauffer:clear-build-folder into 6ec51dc668a355f3da50f8ded4a20012a5a3e8a0 on SAP:master.

I would rather not always clean the dist directory. There can be cases where you just want to overwrite specific files with a smaller and faster build (i.e. updating some files without re-doing the whole build).
An example for this might be the new JSDoc build and how it is used in the OpenUI5 testsuite:
https://github.com/SAP/openui5/blob/master/src/testsuite/package.json#L18-L19
You have to execute the full build one time. After that only a subset, of the build tasks is required to update the JSDoc. This speeds up the build time by about one minute in this example.
So maybe we could add an option to activate cleaning the dist directory before build? In that case I would also rather locate this in the UI5 CLI than in the UI5 Builder project, since I would not expect a function that is just called build to also remove things from my disk (remember that the UI5 Builder is not only used by the CLI but directly in other node scripts!).

Hi @RandomByte.
I've defined it as a parameter, false by default. I've added a test case for it. Also, I've moved rimraf from devDependencies to dependencies.

I'm not sure whether it should be a command or an option in ui5-cli. I'm more inclined to use it as an option... What do you think is better?

PS: I tried to create it as a task, but I couldn't find a way to pass the destPath as parameter to the task without changing too many files...

Replaced by #306


readme: make copy button appear for other things that need copying too

You might want to reformat the nuclear list urls too, but I deliberately left them with the old formatting so they're more difficult to accidentally copy by new users looking for the main lists.

Okay, will do!


bug: kflex create -p garbage claims success

Describe the bug
kflex create claims success when told to use a PostCreateHook that does not exist. The following typescript shows an example.
mspreitz@mjs13 kubestellar % kubectl --context kind-kubeflex get pch
NAME             SYNCED   READY   TYPE   AGE
openshift-crds                           5m49s

mspreitz@mjs13 kubestellar % kflex create its2 --type vcluster -p flagrant-garbage
✔ Checking for saved initial context...
✔ Switching to initial context...
✔ Creating new control plane its2 of type vcluster ...
✔ Waiting for API server to become ready...

mspreitz@mjs13 kubestellar % echo $?
0

Steps To Reproduce
Shown above.
Expected Behavior
Error message and non-zero exit status in this case.
Additional Context
https://github.com/kubestellar/kubestellar/issues/2069

Working as designed.


Use marshmallow for API models and/or query parameters

As a reminder... (Since models are not done yet)

Pydantic does not seem to be supported by Flasgger


Manual links to dotland-fresh.deno.dev instead of deno.land

When navigating to the main version of the manual, a banner appears at the top of the page (see screenshot below), including a link with the text "Click here to view the documentation for the latest release." which points to https://dotland-fresh.deno.dev/manual@DENO_VERSION/introduction. This link should direct the viewer to https://deno.land/manual@DENO_VERSION/introduction instead.

If this issue belongs in the denoland/dotland repo, please migrate it for me, thanks.

I think this may or may not be an issue with fresh (re @lucacasonato) or Deploy.
That banner is defined at UserContributionBanner() at manual.tsx:
https://github.com/denoland/dotland/blob/d633a47c74b8b6543d9061386793a55988e43bad/pages/manual.tsx#L393-L423
It's used in manual.tsx at lines 318 through 326.
https://github.com/denoland/dotland/blob/d633a47c74b8b6543d9061386793a55988e43bad/pages/manual.tsx#L318-L326
These are relative URLs, not set URLs, so the problem is not here. At this time, I'm not sure whether or not it's fresh or Deploy.


Update utils.py

Fixes hidden field rendering on Django 1.8
Error was:
Exception Value:
'wrapper_class'
Exception Location: C:\Python27\lib\site-packages\django-1.8.dev20140702115402-py2.7.egg\django\template\context.py in delitem, line 74

@maraujop Any chance for a django-crispy-forms release to make it Django 1.8 compatible? The deprecation warning on 1.7 was annoying, but in the latest builds of 1.8, django.utils.functional.memoize has been removed entirely, so django-crispy-forms doesn't work at all anymore.

See also: https://github.com/maraujop/django-crispy-forms/pull/364

Pinging @maraujop — Any chance of this (and other PRs) being pulled into a new release? Crispy Forms is great, and a lot of people rely on it, but it seems stalled at the moment. Do you need help maintaining to get it moving forward again?

Fix in dev branch now, see #417 thanks
Please read my changelog:
https://github.com/maraujop/django-crispy-forms/commit/2b808e955c74c87b31b8d2d66fb101f35cf15767#diff-4ac32a78649ca5bdd8e0ba38b7006a1eR9
Your patch would have broken backwards compatibility with previous Django versions, as you were using and importing things not available previously.
Not sure why you changed requiredField to required-field, we should add the class instead of replacing it if it's necessary, as it could break current users' CSS.
Cheers,
Miguel


Fail to send HTML5 Push notifications

Make sure you are running the latest version of Home Assistant before reporting an issue.
You should only file an issue if you found a bug. Feature and enhancement requests should go in the Feature Requests section of our community forum:
Home Assistant release (hass --version):
0.31.1 on Windows server 2012 r2
Python release (python3 --version):
3.5.2
Component/platform:
html5 push
Description of problem:
Fail to send notifications
Expected:
Problem-relevant configuration.yaml entries and steps to reproduce:
notify 4:
  name: WebPush
  platform: html5
  gcm_api_key: redacted
  gcm_sender_id: cloud-messaging-redacted

Traceback (if applicable):
←[32m16-10-24 12:09:48 INFO (CP Server Thread-18) [homeassistant.components.http
] Serving /api/services/notify/webpush_matthews_phone to <IP_ADDRESS> (auth: Tru
e)←[0m
←[32m16-10-24 12:09:48 INFO (MainThread) [homeassistant.core] Bus:Handling <Even
t call_service[L]: domain=notify, service_data=message=The sun is {% if is_state
('sun.sun', 'above_horizon') %}up{% else %}down{% endif %}!, service=webpush_mat
thews_phone, service_call_id=927770390992-1>←[0m
←[31m16-10-24 12:09:48 ERROR (ThreadPool Worker 9) [homeassistant.core] BusHandl
er:Exception doing job←[0m
Traceback (most recent call last):
  File "C:\Miniconda3\lib\site-packages\homeassistant\core.py", line 1221, in jo
b_handler
    func(*args)
  File "C:\Miniconda3\lib\site-packages\homeassistant\core.py", line 1075, in ex
ecute_service
    service_handler.func(service_call)
  File "C:\Miniconda3\lib\site-packages\homeassistant\components\notify\__init__
.py", line 111, in notify_message
    notify_service.send_message(**kwargs)
  File "C:\Miniconda3\lib\site-packages\homeassistant\components\notify\html5.py
", line 326, in send_message
    from pywebpush import WebPusher
  File "C:\Miniconda3\lib\site-packages\pywebpush\__init__.py", line 10, in <mod
ule>
    import http_ece
  File "C:\Miniconda3\lib\site-packages\http_ece\__init__.py", line 8, in <modul
e>
    import pyelliptic
  File "C:\Miniconda3\lib\site-packages\pyelliptic\__init__.py", line 43, in <mo
dule>
    from .openssl import OpenSSL
  File "C:\Miniconda3\lib\site-packages\pyelliptic\openssl.py", line 528, in <mo
dule>
    OpenSSL = _OpenSSL(libname)
  File "C:\Miniconda3\lib\site-packages\pyelliptic\openssl.py", line 67, in __in
it__
    self._lib = ctypes.CDLL(library)
  File "C:\Miniconda3\lib\ctypes\__init__.py", line 347, in __init__
    self._handle = _dlopen(self._name, mode)
OSError: [WinError 193] %1 is not a valid Win32 application←[0m```

I had success just now by copying the dlls libeay32.dll and ssleay32.dll to my C:\Windows folder. I used the same that were in my letsencrypt-win-simple\x64 folder. These are the same as the 1.0.2a version found here: http://indy.fulgan.com/SSL/


Add interface for breaking out position control interface

@brentyi  @gbalke  please review

I know but it is super long if I put it on the same line :(
On Wed, Aug 7, 2019 at 1:40 PM Brent Yi<EMAIL_ADDRESS>wrote:

@brentyi approved this pull request.
In blue_hardware_drivers/src/BLDCControllerClient.cpp
https://github.com/berkeleyopenarms/blue_core/pull/81#discussion_r311753007
:

 COMM_REG_RO_ROTOR_P, 9,                                          // Read

COMM_REG_VOL_QI_COMM, sizeof(value), reinterpret_cast<uint8_t*>(&value));  // Write

COMM_REG_VOL_SETPOINT_P, sizeof(position) + sizeof(feedforward), // Write

         reinterpret_cast<uint8_t*> (&values), num_registers

);

putting this on a new line is inconsistent with the rest of the file :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/berkeleyopenarms/blue_core/pull/81?email_source=notifications&email_token=ACUDTGEV3XVNXGAZCXGSJG3QDMXLJA5CNFSM4IJ32LG2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCA43DRI#pullrequestreview-272216517,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACUDTGCP3CPCKHFXEZFBUW3QDMXLJANCNFSM4IJ32LGQ
.

--
Greg Balke
University of California, Berkeley
*Electrical Engineering and Computer Science, B.S. *2019


fix: repeat breakpointLocations DAP request

Types

[x] 🐛 Bug Fixes

Background or solution
在接收 BreakpointEvent 这个自定义事件的时候，应该只改变状态，而不应该再触发断点的任何 DAP 请求
Changelog
修复 breakpointLocations DAP 重复请求的问题

Codecov Report

Merging #229 (83fdde1) into main (b5b75c6) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##             main     #229   +/-   ##
=======================================
  Coverage   59.28%   59.28%
=======================================
  Files        1181     1181
  Lines       72666    72666
  Branches    15061    15061
=======================================
  Hits        43080    43080
  Misses      26963    26963
  Partials     2623     2623

Impacted Files
Coverage Δ

packages/debug/src/browser/debug-session.ts
14.63% <0.00%> (ø)

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 b5b75c6...83fdde1. Read the comment docs.


Explain which files need to be updated for the schedule update

/cc @lionelvillard @dprotaso

/lgtm


feat: add support for all protocol 18 features

Update the generated_xdr to the latest version and add support to all the operations that were added since the last time the library was updated.
Because the xdr definitions have changed in a non-backwards compatible way, this required non-backwards compatible changes in the library's struct definitions themselves. Thus, it leads to a major version release.
I added all the changes that were necessary to the changelog.
Signed-off-by: Santiago Alessandri<EMAIL_ADDRESS>

Thanks. I will update the PR to resolve the merge conflicts.


CI .deb packages should be built on Ubuntu 18.04

Currently they target 20.04 and thus limit compatibility to that or newer, but we should be able to ~easily downgrade to 18.04 (the oldest release supported by GitHub actions).

The difficulty of this has been vastly underestimated. For one, bionic doesn't have dune so the Debian build script won't work.

Yeah, let's not do it.


add webhook for ansible galaxy

This way the build status is shown on ansible-galaxy.

Thanks @rndmh3ro for this addition!


(supp) pass arguments options to the package installer at firts install time

Please paste the output of the following command here: sudo edit-chroot -all

**there's no edit-chroot if i not have any installed** Pufff!!

Please describe your issue:
i need to pass  --no-install-recommends and a custom list of packages due the very limited space in the ARM based older chromiunbooks but it's hardcoded in the installer
also some other command option to apt, so then: there's some possiblitity to in future allow to pass options to the package installer of the release/distro option?
TAKE IN CONSIDERATION THAT: the SD or USBdrive are very slow and mounts "noexec,nodev"
so issues like #3587 never will work due all the chrrot are in a external mount
NOTE minimize/customize the chroot install after happened its very tedious and problematic due in ARM based there's limited support and we not know what to unninstall, a black method its better rather white list
If known, describe the steps to reproduce the issue:
normal install of stretch/jessy but does not fit/allow me space to my documents

USBdrives are very expense its no a solution in Venezuela, you are not provide a real solution..
a option to parse to the installer its better, i can see the hardcoded in winbuntu case and some packages the "no recommends" in the code, but you prefers do not made that improvement and close that issue!? i will not the only one for sure-.- and now that crouton does not work recently ...

@mckaygerhard,

USBdrives are very expense its no a solution in Venezuela, you are not provide a real solution..

Sorry but that's hardly a problem we can solve.

a option to parse to the installer its better, i can see the hardcoded in winbuntu case and some packages the "no recommends" in the code, but you prefers do not made that improvement and close that issue!?

'crouton' is an open system so you're welcome to submit solutions to your unique issues via 'pull requests'.
Hope this helps,
-DennisLfromGA

"your unique issues" seems some poor mind people thinks the resources in the planet are infinite, there more countries in economic problems rather thant countries that can buy a usbdrive all the dais..

@mckaygerhard,
Sorry, I didn't mean 'unique' in the way you interpreted it, my bad.
I was referring to the information & documentation you're requesting, nothing else.
-DennisLfromGA


tickets/PREOPS-4856

Overall I like how you've organized the notebook, but it seems like it might be hard for a user to understand how the cutout service works from this. It may need a little more explanation in Section 2.2.
The LBG example is cool! It's a little frustrating how long it takes for cells to execute, but I know that's out of our control...
The variable/transient example is good, but it may be useful to give a little more context. It is an abrupt change from the LBG to the variable, so I'd try to clarify as much as possible.

I guess the plan is for this demonstration to be preceded by a more technical notebook that demonstrates the more technical details of the cutout service. Leanne is working on this, so hopefully in the future the pair of notebooks will together do a better job.


support normal modules

Hey, so, if you didn't already know, this plugin breaks with webpack 4. I fixed the normal modules, but it breaks the other versions. Should be easy to fix that though. I'm going to re-add support for loaders next. Then, I'll make it compatible with all versions. Just making this PR now, so people can see it and comment if they want.
I can change formatting and stuff too if needed, or if you just want to take my branch and change it to you're liking, that's ok too. There're a lot of breaking changes in wp4 though

This took me all day, btw, because I've never made a wp plugin before. So, the code might not be totally polished. I tried to adhere to the current formatting though...I really hate writing tests, but I can do that too if needed.

Ok, actually, this supports loaders and babel too somehow

Just added support for all versions...tests are still broken though

#123 is better than mine, so I'm closing this, but please merge that one


Scrollbar instead of overview ruler

Clicking the overview ruler positions the cursor at the absolute position within the document.
I am used to page-up/ page-down when clicking in the scrollbar, rather than jumping to the clicked location.
So it would be nice to have a scrollbar instead of the overview ruler.
Or did I really miss this in the settings?
Thank you!

Adding the unlisted setting "editor.scrollbar.scrollByPage": true to your user settings and restarting VS Code makes the editor scrollbar and minimap scroll by page when clicking above or below the slider. See #104923

Great! Very helpful, thank you indeed!


SQLiteException: no such collation sequence

When initializing the database using
final Manager manager = new Manager(new AndroidContext(context), Manager.DEFAULT_OPTIONS);
database = manager.getDatabase(DatabaseManager.DATABASE_NAME);

I get an exception:

com.couchbase.lite.storage.SQLException: android.database.sqlite.SQLiteException: no such collation sequence: REVID (code 1): , while compiling: CREATE TABLE revs (         sequence INTEGER PRIMARY KEY AUTOINCREMENT,         doc_id INTEGER NOT NULL REFERENCES docs(doc_id) ON DELETE CASCADE,         revid TEXT NOT NULL COLLATE REVID,         parent INTEGER REFERENCES revs(sequence) ON DELETE SET NULL,         current BOOLEAN,         deleted BOOLEAN DEFAULT 0,         json BLOB,         UNIQUE (doc_id, revid))

When the manager will run this query:
CREATE TABLE revs (         sequence INTEGER PRIMARY KEY AUTOINCREMENT,         doc_id INTEGER NOT NULL REFERENCES docs(doc_id) ON DELETE CASCADE,         revid TEXT NOT NULL COLLATE REVID,         parent INTEGER REFERENCES revs(sequence) ON DELETE SET NULL,         current BOOLEAN,         deleted BOOLEAN DEFAULT 0,         json BLOB,         UNIQUE (doc_id, revid))

as part of the open method when initializing the schema on a db version of 0.
I'm using version 1.1.0 of the library and this crash seems to happen only on Android N. Upgrading to the latest version is not really an option since the project I'm involved with used private parts of the library which are no longer present in version 1.3.0

Hi @vanniktech,
CBL v1.2.x or earlier can not support Android N because Android N does not allow the application to use most of native libraries that are provided by Android.
Thanks,
Hideki

Thanks for your answer Hideki :)
So the only way is upgrading to 1.3? Also since I just got into that project and never used the couchbase library before, can the library handle an upgrade from one version to another (e.g. 1.1.0 to 1.3.0) without loosing the persisted data from the 1.1.0 time?

Hi @vanniktech,
CBL provides auto database upgrading functionality. With 1.2.x and 1.3.0, CBL Java has database upgrading issue. See #888 . This issue is addressed with 1.3.1 release which will be released soon. I recommend you to wait till 1.3.1 is released or make a build from source code.
Thanks!


Maintaining tqdm progress bar callback from keras-tqdm

Describe the feature and the current behavior/state.
Keras integration with TQDM progress bars.
The current repository is in https://github.com/bstriner/keras-tqdm and I would like to maintain/update it for tensorflow 2.0 and put it here in the contrib section so more people can easily use it.
Relevant information

Are you willing to contribute it (yes/no): Yes
Are you willing to maintain it going forward? (yes/no): Yes
Is there a relevant academic paper? (if so, where): No
Is there already an implementation in another framework? (if so, where): No
Was it part of tf.contrib? (if so, where): No

Which API type would this fall under (layer, metric, optimizer, etc.)
callbacks
Who will benefit with this feature?
Those who wants to use tqdm to have a prettier progress bar when training models.
Any other info.

Looks cool, any thoughts @tensorflow/sig-addons-maintainers

I think its a nice addition (and our first callback). Please be sure to update the callbacks README guidelines if there is anything you find missing from the documentation.

Hi,
Thank you so much for your comments! I have been busy with work lately but will work on this when I get time!


Merge the Rest7 branch back to the master

This PR allows us to merge back the rest7 branch in the master. We can use it to easily monitor merge conflict if we decide to still wait some days to have dspace-solr restored (it is disabled now in the pom.xml - we have commented out the dspace-solr profile)

with the commit https://github.com/DSpace/DSpace/pull/1679/commits/b66066ffb60ad0f8b38f144e2f792221dc198445
we have restored the dspace-solr profile. I'm able to succesful build and deploy solr locally so we should proceed with the merge now


Additional Pigment Colors / PrideLib Integration

Description
It would be awesome to see some PrideLib integration for those who want to add their own pride pigments. Additionally, could we also add the other Gay flag and maybe the Transspecies flag colors?
Notes
This is distinct from #212, as it involves PrideLib integration.

pridelib stuff might work better as an addon.


Consider linking the package to the SBT organisation

See http://www.scala-sbt.org/0.13/docs/Bintray-For-Plugins.html#Linking+your+package+to+the+sbt+organization
We've done this for a number of our plugins at the Guardian. It's pretty painless and means consumers don't have to specify a separate resolver.

Thanks @philwills! We will look into it the next time we touch this repository

+1  Note that with sbt 1.0 firming up, this might be a good time.

+1 on any route to a sbt 1.0 version


Required dispatch_sync to load constants for Orientation. This may lead to deadlocks

when I update RN from 0.30 to 0.34.1 I get this waring
Required dispatch_sync to load constants for Orientation. This may lead to deadlocks

it not appears in 0.30
is it serious？

Another developer has reported the same warning for my native module: https://github.com/idehub/react-native-google-analytics-bridge/issues/70
It seems to have showed up in 0.33 and be related to the Native Module exporting constants.
The framework loads the constants from native modules in RCTModuleData  gatherConstants. That method is used in RCTBatchedBridge, which has the following comment:

// Modules that need to be set up on the main thread cannot be initialized
// lazily when required without doing a dispatch_sync to the main thread,
// which can result in deadlock. To avoid this, we initialize all of these
// modules on the main thread in parallel with loading the JS code, so
// they will already be available before they are ever required.

I've also gotted this warning intermittently for other things like RCTUIManager and CodePush.
Tagging @javache since he introduced this warning.

This is a known issue with the current bridge initialisation. In practice, this won't lead to any issues with RN apps since the main thread is free, but in more complex integration scenarios this may result in issues, which is why I added this warning to chase it down.

Hm, and there isn't any way for me to implement constantsToExport in order to not get this error?
All I am really doing is:
NSString *staticTrackerId;

- (instancetype)init
{
    if ((self = [super init])) {
        ...
        staticTrackerId = [[NSBundle mainBundle] objectForInfoDictionaryKey:@"GAITrackingId"];
        ...
    }
    return self;
}

- (NSDictionary *)constantsToExport
{
    return @{ @"nativeTrackerId": staticTrackerId != nil ? staticTrackerId : [NSNull null] };
}

Or will I get it regardless if I export constants?

I am getting this error and chrome debugging is stuck!

@javache Is it safe to ignore all these warnings about possible deadlocks? kinda scary warning tho.


Creature matching fails if there's a space in a folder in the path

cachedTokens will contain %20 but tokenCheck will contain a space character  .
My suggestion on this would be to ignore folder names. Cache only file names and remove the CR check.
After that, match purely in Actor name VS File name.
The only situation I'd see this becoming an issue would be if there's two monsters with the same name in two different CRs. I know that for 5e this isn't possible as every monster has a unique name.
EDIT: The default Forgotten-Adventures token folder name for Patreon is Tokens (Patreon). Renaming to Tokens_Patreon resolved my issue.
EDIT2: Due to the indexOf check, this could be an issue if there's a CR 1 creature named "Goblin" and a CR 2 creature named "Goblin Archer". By adding a "Goblin" token, both creatures would be matched as they both contain the string "Goblin".
This is subjective, but my opinion is that CR shouldn't be the differenciating factor as you could have the same problem if both had the same CR. A more strict check than indexOf would allow clearer matching. Perhaps allow users to configure their name convention for tokens?
e.g. ${data.name}_${data.data.traits.size}_${data.data.details.type}
This might be too flexible and complex though.

I'll have a look at this. Alternately what you can currently do is remove the cr value from the NPC Difficulty Name and put everything in the root folder, that way the programs looks at all the files without worrying about folder structure.
I'll add some escapes around the whole string as well (currently just escaping the file names). That should at least fix any issue with spaces and other funky characters that get's replaced with HTML encoded values in the folder names too.

I've added some new basic functionality to define how the folder and image file names are defined. This way you can specify whether your folder and file names use spaces, underscores or dashes between the words.
So if you select the "Proper English" version for both folders and files, it'll be expecting "/cr 1/Monster Name.png".
If you select "Underscored" for both, it'll expect "cr_1/Monster_Name.png".
If you select "No Space" and "Underscored" for the folder and file names respectively (this is seen as default), it'll expect "/cr1/Monster_Name.png".
If you don't specify a NPC Difficulty name, it will ignore the folder and just compare things in the root folder.
This will be released later this week


trigger based query does not update results properly

notebook 2JEF7JF7C on tg6 shows that while running a continuous query the output is not updated even though firefox network tab shows that there is new data coming in via the websocket.
please investigate why.

only the first results are visible, rest are visible with the page refresh

Current status of the issue:
Trace log:

UI processes the paragraph as it should and sends a run request to server.
Server responds with two results for the paragraph. First result is a text field, second is a table with 1 column.
Server starts to update both result fields. The text field is updated via "Append_Paragraph_Output" and receives either "" or "\n" in updates, which makes the empty text field grow uncontrollably. The table constantly receives updates with identical information, which makes it stay the same.
No signs of any other data was found.

Conclusion: Server misbehaves, waiting for confirmation. Probably not a AJS_01 issue.

@BVVLD @ronja-ui please update status

@StrongestNumber9 reproduced this on tg7

@StrongestNumber9 reproduced this on tg7

tg7 notebook/2K1PAMQ6Q or test-combined

tested on AJS_01:
server indeed sends data updates, which UI ignores. However, once user switches to any graph and back to the table, the table starts to update. Here is the reason, why:
initially, to display the results, server sends the updated paragraph object. Data for the results is in the paragraph.results array, and the config for it is stored in paragraph.config.results array. The indexes of the arrays are (or at least should be) synchronized. Let's assume, that the data is paragraph.results[x].msg and config is paragraph.config.results[x].
Since all current results are now graphs, all config is stored in config.graph. Here it should be a "mode" parameter, that tells which representation to use, "setting" array, which stored specific settings for each mode, "commonSetting" object, and also here is stored setting fields, like "keys", "groups" and "values", which you can see if you open a setting panel.
The problem is: server does not initially send the config object. Therefore paragraph comes without it and on every update it tries to apply it with the config, but fails, since there is none. So updates are discarded.
Once user switches between graphs - it creates the configs and stores it in the notebook. After that, it starts to provide these configs every time the paragraph object is provided and everything starts to work as expected.
Current attempt to solve the problem: trying to make UI to render and process updates with empty config. If this will not work -> to add a defaulting mechanism, that will use at least some minimal configs to run.
What problem that might create? Sometimes server sends empty result updates or empty data for results, which might mess up result object. Previously, some of these empty updates were cut out with empty config checks.

ui should be responsible for providing the default rendering-config if it's not present, if it is use it. if there later comes update without one but ui has still previous config stored, use it.

The attempt to allow results render without config was a success. The "default rendering-config" will be implemented, since it is requested. It would be helpful to see at least some documentation on what should be there by default.
Otherwise, troubleshooting the issue also uncovered few other UX problems, mostly with DataTable controls, including flashing "default columns" button and bugs related to its functionality in raw-data tables. These issues are currently being investigated.

Previously mentioned issues have been resolved and basic config defaulting was implemented, issue is awaiting review


Python 2 gcc MatplotlibUsing.py triggers -Werror=unused-but-set-variable

Occurring on v<IP_ADDRESS> openSUSE 15.2 builds , on gcc only, and likely Python 2 only.  (clang without debug passed; clang with debug ongoing..)

The specific PyPI names and versions

python2-matplotlib-2.2.5-lp<IP_ADDRESS>

Provide in your issue the Nuitka options used

This had --debug enabled.

Note if this is a regression

Not sure as https://github.com/Nuitka/Nuitka/issues/1298 prevented testing in earlier versions.
[ 1695s] Consider output of standalone mode compiled program: MatplotlibUsing.py
[ 1695s] Comparing output of 'MatplotlibUsing.py' using '/usr/bin/python2' with flags silent, expect_success, --standalone, remove_output, cpython_cache, timing, ignore_warnings ...
[ 2092s] --- /usr/bin/python2 (stdout)
[ 2092s] +++ nuitka (stdout)
[ 2092s]
[ 2092s] @@ -1,2 +1 @@
[ 2092s]
[ 2092s] -OK.
[ 2092s]
[ 2092s] --- /usr/bin/python2 (stderr)
[ 2092s] +++ nuitka (stderr)
[ 2092s]
[ 2092s] @@ -1 +1,7 @@
[ 2092s]
[ 2092s] +module.pyparsing.c: In function 'impl_pyparsing$$$function__90_parseString':
[ 2092s] +module.pyparsing.c:53841:17: error: variable 'tmp_try_except_1__unhandled_indicator' set but not used [-Werror=unused-but-set-variable]
[ 2092s] +     nuitka_bool tmp_try_except_1__unhandled_indicator = NUITKA_BOOL_UNASSIGNED;
[ 2092s] +                 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
[ 2092s] +cc1: all warnings being treated as errors
[ 2092s] +scons: *** [module.pyparsing.o] Error 1
[ 2092s]
[ 2092s] Updating CPython cache by force due to non-matching comparison results.
[ 2092s] --- /usr/bin/python2 (stdout)
[ 2092s] +++ nuitka (stdout)
[ 2092s]
[ 2092s] @@ -1,2 +1 @@
[ 2092s]
[ 2092s] -OK.
[ 2092s]
[ 2092s] --- /usr/bin/python2 (stderr)
[ 2092s] +++ nuitka (stderr)
[ 2092s]
[ 2092s] @@ -1 +1,7 @@
[ 2092s]
[ 2092s] +module.pyparsing.c: In function 'impl_pyparsing$$$function__90_parseString':
[ 2092s] +module.pyparsing.c:53841:17: error: variable 'tmp_try_except_1__unhandled_indicator' set but not used [-Werror=unused-but-set-variable]
[ 2092s] +     nuitka_bool tmp_try_except_1__unhandled_indicator = NUITKA_BOOL_UNASSIGNED;
[ 2092s] +                 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
[ 2092s] +cc1: all warnings being treated as errors
[ 2092s] +scons: *** [module.pyparsing.o] Error 1
[ 2092s]
[ 2092s] Exit codes 0 (CPython) != 1 (Nuitka)

Oh, that's an interesting report. I will look into it. The module name is one that I believe is fequently compiled, it would be more interesting to know the pyparsing version, but since we are talking Python2, it's pretty likely just the latest one working for it, right?

btw, if you have like a set of things, e.g. a podman usable file to spin up somelike like a VM to reproduce openSUSE things, I would be super cool with that too.

ok, and I managed to reproduce it on stable version, but not on develop. I will try and bisect things, and see if the fix is something I should backport.

in the 0.6.19 release cycle, optimization for merges, loop variables, escaping and all got much better, so I think I know that this is something I had in fact triggered myself in other code, sometimes unused variables were not forward propagated because of being used in merges, and that is no longer relevant, which is why the new version won't have it. I thinking it won't be backportable at all. But there will be a new release this week anyway, probably within days even.

Somehow git biset is letting me down in new ways, I cannot git biset from master to develop branch, because they have no common base, and when I did it from 0.6.18 tag to develop, it gave up and listed a whole range of commits, because a merge base was bad. Now that's strange, never had that before. Maybe some rebase of mine has gone wrong, or git improved somehow to find out that I am doing bisects the wrong way. Anyway, I think this is something to be considered solved with next release. No way can I backport new optimization, as they uncover too many other wrong things as well, some of which got hotfixed already, but there will be regressions from that.

btw, if you have like a set of things, e.g. a podman usable file to spin up somelike like a VM to reproduce openSUSE things, I would be super cool with that too.

best strategy is to use a suse vm, install osc, then osc co devel:languages:python/python-Nuitka, then use osc build which builds in the VM. https://build.opensuse.org/package/show/devel:languages:python/python-Nuitka
Not sure about podman, but maybe there is tooling to support converting builds to podman, which is also supported by obs-build (https://github.com/openSUSE/obs-build#image-formats).

ok, and I managed to reproduce it on stable version, but not on develop ..

Somehow git biset is letting me down in new ways, I cannot git biset from master to develop branch ..

I'll do a build of develop to see if that fixes it.

@jayvdb I am actually sure of that it is fixed there, problem with these kinds of things is that any optimization can reveal others that are missing... these are going to crop up, and usually are valuable to recognize missed opportunities

I am also getting this on Python 3 when I enable --debug on gcc builds.  I havent been trying --debug on gcc for a while, so dont know if this is a recent problem.
[ 2291s] Consider output of standalone mode compiled program: FlaskUsing.py
[ 2291s] Comparing output of 'FlaskUsing.py' using '/usr/bin/python3.6' with flags silent, expect_success, --standalone, remove_output, cpython_cache, timing, ignore_warnings ...
[ 2930s] --- /usr/bin/python3.6 (stderr)
[ 2930s] +++ nuitka (stderr)
[ 2930s]
[ 2930s] @@ -1 +1,7 @@
[ 2930s]
[ 2930s] +module.pyparsing.core.c: In function â€˜impl_pyparsing$core$$$function__36_parse_stringâ€™:
[ 2930s] +module.pyparsing.core.c:25491:17: error: variable â€˜tmp_try_except_1__unhandled_indicatorâ€™ set but not used [-Werror=unused-but-set-variable]
[ 2930s] +25491 |     nuitka_bool tmp_try_except_1__unhandled_indicator = NUITKA_BOOL_UNASSIGNED;
[ 2930s] +      |                 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
[ 2930s] +cc1: all warnings being treated as errors
[ 2930s] +scons: *** [module.pyparsing.core.o] Error 1
[ 2930s]
[ 2930s] Updating CPython cache by force due to non-matching comparison results.
[ 2930s] --- /usr/bin/python3.6 (stderr)
[ 2930s] +++ nuitka (stderr)
[ 2930s]
[ 2930s] @@ -1 +1,7 @@
[ 2930s]
[ 2930s] +module.pyparsing.core.c: In function â€˜impl_pyparsing$core$$$function__36_parse_stringâ€™:
[ 2930s] +module.pyparsing.core.c:25491:17: error: variable â€˜tmp_try_except_1__unhandled_indicatorâ€™ set but not used [-Werror=unused-but-set-variable]
[ 2930s] +25491 |     nuitka_bool tmp_try_except_1__unhandled_indicator = NUITKA_BOOL_UNASSIGNED;
[ 2930s] +      |                 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
[ 2930s] +cc1: all warnings being treated as errors
[ 2930s] +scons: *** [module.pyparsing.core.o] Error 1

It very likely is recent, the next release doesn't have it as far as I can tell. The compiler is telling us, me, here that it can do an optimization that is indicative of one not being done by Nuitka, in principle for every optimization added to Nuitka, it can reveal the missing of another one... so these are going to come up. Not a huge problem per se.

Part of the 0.6.19 release I just made.


[The Grid] documentation has broken link

How to reproduce this bug:
The initial alert callout under https://foundation.zurb.com/sites/docs/grid.html#importing has a link (from XY Grid) to https://foundation.zurb.com/xy-grid.html
This link https://foundation.zurb.com/xy-grid.html gives a 404
What should happen:
I see sweet documentation
What happened instead:
I see a slightly less contextually sweet 404

But 404 is still sweet. Won't fix.

I'm joking. Thanks for the issue @nigelm

See https://github.com/zurb/foundation-sites/pull/10986

Closed by https://github.com/zurb/foundation-sites/pull/10986


add callback_query processing

This will be added with #102 when my PR to @tjhorner accepts mine and then @yagop accepts the combined PR.

tjhorner/node-telegram-bot-api#2 accepted :)

Tets retutns a Promise so no need for done call.
https://mochajs.org/#working-with-promises

@yagop Please create or accept callback_query processing.
Because other forks of this project don't generating /src and have problems with babel .

Looks like I have to change this library for smth else... (((

Thanks!
Done in #102


content refreshing

the daily refresh for your list seems to have stopped working can you please check it out for potential issues

Opps! Thanks, it should be back now.


Use the update state machine syntax in the JavaScript compiler

<EMAIL_ADDRESS>introduces a new syntax for transitions in states machines. While the syntax we use is still supported, we might want to update the code generator to use the new syntax.
Low priority.

Issue deprecated by #283


Add tdhttp.Path() function

to build paths with query params:
tdhttp.Path("/1.0/path", "key", "value", url.Values{"x": []string{"a", "ba"}}, "key2", true)

After some thought, it could be better to simply add a tdhttp.Q type:
type Q map[string]interface{}

Instances of this type can be directly used in net/http request builders, like tdhttp.NewRequest as in:
req := tdhttp.NewRequest("POST", "/pdf", body,
  "Content-type", "application/pdf",
  tdhttp.Q{
    "param1": 128,
    "param2": "value"
    "param3": [ 1, 2, 3],
    "param4": anyType, // passed to fmt.Sprint
  },
  "X-Test", "value",
  tdhttp.Q{
    "param3": "another", merged with previous "param3"
    "param5": 45.63,
  },
)


Fix updating metadata in CheckoutShippingMethodUpdate mutation

Fix for https://saleor.sentry.io/issues/4147808800/?project=6417854&referrer=slack
The CheckoutShippingMethodUpdate was failing for checkout without metadata_storage

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Ports of this PR is included in ports of PR https://github.com/saleor/saleor/pull/12646


Backport ID collision  fix to 1.0

Backport https://github.com/eclipse/microprofile-opentracing/issues/66 to 1.0 (as 1.0.1) to avoid failures when running TCKs for 1.0.
cc @pilhuhn

This has been done, closing


Move destination port NAT to conduit

Is your feature request related to a problem? Please describe.
The problem with the current implementation (using nftables with Flow) is that the source-port, source-subnet and the priority of the flow is not considered. So, if there is any overlap with any of the other flows, there will be "glitches" (traffic that should not be NATted will be NATted due to another flow).
Describe the solution you'd like
The NAT feature should be moved to the conduit level. A new property should be added to the conduit to support the Destination port NATing:
apiVersion: meridio.nordix.org/v1alpha1
kind: Conduit
metadata:
  name: conduit-a
  namespace: red
  labels:
    trench: trench-a
spec:
  type: stateless-lb
  destination-port-nats:
  - port: 80
    targetPort: 8080
    vips:
    - vip-a
    protocol: UDP
  - port: 90
    targetPort: 9999
    vips:
    - vip-a
    - vip-b
    protocol: TCP
  - port: 90
    targetPort: 5000
    vips:
    - vip-a
    - vip-b
    protocol: UDP

All traffic going through the conduit with the properties defined in the conduit will be NATted.

port: port the exposed by the service (in any of the conduit` flow).
targetPort: port used by the target pod.
vips: port will be NATted only if the traffic has one of the corresponding VIP address as destination address.
protocol: port will be NATted only if the traffic has to corresponding protocol.

port will be NATted to targetPort for the ingress traffic and targetPort will be NATted to port for the outgoing traffic.
Describe alternatives you've considered
To avoid having glitches while NATting the destination port of the traffic, the NAT could be implemented as part of NFQLB, so NFQLB could consider every flow properties (byte-match, source IP/Port, priority...).
With the NAT implemented in NFQLB, it will mean the traffic should go through NFQLB on the way back to be "Un-NATted".
Additional context
https://github.com/Nordix/Meridio/pull/187
https://github.com/Nordix/Meridio-Operator/pull/99

1
Only 1 protocol per entry (so -- can be used as nft chain name)
apiVersion: meridio.nordix.org/v1alpha1
kind: Conduit
metadata:
  name: conduit-a
  namespace: red
  labels:
    trench: trench-a
spec:
  type: stateless-lb
  destination-port-nats:
  - port: 80
    targetPort: 8080
    vips:
    - vip-a
    protocol: UDP
  - port: 80
    targetPort: 8080
    vips:
    - vip-a
    protocol: TCP
  - port: 90
    targetPort: 9999
    vips:
    - vip-a
    - vip-b
    protocol: TCP
  - port: 90
    targetPort: 5000
    vips:
    - vip-a
    - vip-b
    protocol: UDP

2
multi-protocols per entry (nft chain name TDB)
apiVersion: meridio.nordix.org/v1alpha1
kind: Conduit
metadata:
  name: conduit-a
  namespace: red
  labels:
    trench: trench-a
spec:
  type: stateless-lb
  destination-port-nats:
  - port: 80
    targetPort: 8080
    vips:
    - vip-a
    protocols:
    - TCP
    - UDP
  - port: 90
    targetPort: 9999
    vips:
    - vip-a
    - vip-b
    protocols:
    - TCP
  - port: 90
    targetPort: 5000
    vips:
    - vip-a
    - vip-b
    protocol:
    - UDP

3
Only 1 protocol and 1 VIP per entry (so --- can be used as nft chain name)
apiVersion: meridio.nordix.org/v1alpha1
kind: Conduit
metadata:
  name: conduit-a
  namespace: red
  labels:
    trench: trench-a
spec:
  type: stateless-lb
  destination-port-nats:
  - port: 80
    targetPort: 8080
    vip: vip-a
    protocol: UDP
  - port: 90
    targetPort: 9999
    vips: vip-a
    protocol: TCP
  - port: 90
    targetPort: 9999
    vip: vip-b
    protocol: TCP
  - port: 90
    targetPort: 5000
    vip: vip-a
    protocol: UDP
  - port: 90
    targetPort: 5000
    vip: vip-b
    protocol: UDP


document if decompilation of apk is possible

Hello,
I would like to be able to decompile apk in kotlin with kotlin-explorer.
Is it currently possible? Is it a compiler instead of a decompiler?
Best reagrds.

It disassembles the code it compiles. At no point does the readme nor the tool make any claims about APKs. So no, it won't decompile an APK into Kotlin code.


Update AsyncMqttClient.cpp

On bk7231t, having currentByte as a char causes a problem when receiving packet types >= 8 (in the packet the type is in the high nibble, ie 0x80), when byte shifting >> 4 they get sign extended so we end up with packet type 0xf9 instead of 0x9 for suback.

Thanks! I don't know why, but GitHub did not notice me in any way about this PR - not even an e-mail. So naturally I didn't notice it.
And I ended up taking MQTT crashing under a microscope, just to find the same issue as you did...


Weird output in content section if string given to content doesn't end in new line

Given the following declaration for a meta cfn init files object:
      agent_json_content = {
        "cloudwatch.emitMetrics" => false,
        "flows": external_parameters["kinesis"]["agent"]["flows"]
      }
      metadata["AWS::CloudFormation::Init"]["common_bootstrap"]["files"]["/etc/aws-kinesis/agent.json"] = {
        "content" => JSON.pretty_generate(agent_json_content)
      }

This is the output I end up with in the template:
            "/etc/aws-kinesis/agent.json":
              content: |-
                {
                  "cloudwatch.emitMetrics": false,
                  "flows": [
                    {
                      "filePattern": "/var/log/httpd/*log",
                      "deliveryStream": "eric-log-stream"
                    },
                    {
                      "filePattern": "/var/log/nginx/*log",
                      "deliveryStream": "eric-log-stream"
                    }
                  ]
                }

Notice the hyphen after content: |
If I add a new line character to the end of content:
      metadata["AWS::CloudFormation::Init"]["common_bootstrap"]["files"]["/etc/aws-kinesis/agent.json"] = {
        "content" => JSON.pretty_generate(agent_json_content) + "\n"
      }

The hyphen disappears:
              content: |
                {
                  "cloudwatch.emitMetrics": false,
                  "flows": [
                    {
                      "filePattern": "/var/log/httpd/*log",
                      "deliveryStream": "eric-log-stream"
                    },
                    {
                      "filePattern": "/var/log/nginx/*log",
                      "deliveryStream": "eric-log-stream"
                    }
                  ]
                }

I'm assuming you are generating yaml templates from the snippets you provided. The pipe | with ant without a hyphen - are different way that YAML treats scalars with newlines. See http://yaml.org/spec/1.2/spec.html section <IP_ADDRESS>. Block Chomping Indicator
Using IRB:
>> require 'yaml'
>> puts({ foo: "hey\nman" }.to_yaml)
---
:foo: |-
  hey
  man
=> nil
>> puts({ foo: "hey\nman\n" }.to_yaml)
---
:foo: |
  hey
  man
=> nil
>>

Yes.  Outputting yaml.
So this is a problem with Ruby's yaml parser.  If so, this ticket is obviously not for cfndsl to address.   :)

I wouldn't even consider it to be a problem, unless you are getting an error trying to process the YAML later? Your output looks like valid YAML to me.

I wasn't aware this was valid yaml.  Now I know.  :)


Add combine

https://htmlview.glitch.me/?https://gist.github.com/fonsp/056877529633cdbc4e91fcabdb056d3f#file-combine-jl-html

Very cool! What do you think about transforming into a tuple instead of a vector and also making a named tuple variant ?

Awesome idea @Pangoraw !

@disberd

Cool!


chore: release v1.0.0-2

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add introduction for empty string and empty quotes string

Learn Build status updates of commit 913d6a5:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/use-cli-effectively.md
:white_check_mark:Succeeded
View

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


Unknown error OverflowError

Hi, I have error, sometimes works fine:
Logger: custom_components.midea_ac_lan.midea.core.device
Source: custom_components/midea_ac_lan/midea/core/device.py:316
Integration: Midea AC LAN (documentation, issues)
First occurred: 22:58:52 (17 occurrences)
Last logged: 23:00:12
[142936511754902] Unknown error OverflowError('int too big to convert')
How to fix it?
From HA:
Air Conditioner 00000Q1A(8)
In real:
Versu Pure X 3,5 kW

Fixed in v0.3.17


CUDA_ERROR_UNKNOWN running examples

Running any of the examples using cuda gives me an unknown error.
System:
Lambda vector (2x4090)
Ubuntu 22
candle/candle-examples on  main [!] is 📦 v0.3.2 via 🦀 v1.73.0
❯ nvidia-smi
Mon Dec 25 14:22:05 2023
+---------------------------------------------------------------------------------------+
| NVIDIA-SMI 535.129.03             Driver Version: 535.129.03   CUDA Version: 12.2     |
|-----------------------------------------+----------------------+----------------------+
| GPU  Name                 Persistence-M | Bus-Id        Disp.A | Volatile Uncorr. ECC |
| Fan  Temp   Perf          Pwr:Usage/Cap |         Memory-Usage | GPU-Util  Compute M. |
|                                         |                      |               MIG M. |
|=========================================+======================+======================|
|   0  NVIDIA GeForce RTX 4090        On  | 00000000:41:00.0  On |                  Off |
|  0%   31C    P8              35W / 480W |    865MiB / 24564MiB |      5%      Default |
|                                         |                      |                  N/A |
+-----------------------------------------+----------------------+----------------------+
|   1  NVIDIA GeForce RTX 4090        On  | 00000000:61:00.0 Off |                  Off |
|  0%   27C    P8              15W / 480W |    687MiB / 24564MiB |      0%      Default |
|                                         |                      |                  N/A |
+-----------------------------------------+----------------------+----------------------+

Pytorch looks ok
❯ python3 -c "import torch; print(torch.cuda.device_count())"
2

Error:
candle/candle-examples on  main [!] is 📦 v0.3.2 via 🦀 v1.73.0
❯ cargo run --features cuda --example whisper --release
    Finished release [optimized] target(s) in 0.12s
     Running `/home/dres/src/gh/candle/target/release/examples/whisper`
thread 'main' panicked at /home/dres/.cargo/registry/src/index.crates.io-6f17d22bba15001f/cudarc-0.9.15/src/driver/safe/core.rs:51:24:
called `Result::unwrap()` on an `Err` value: DriverError(CUDA_ERROR_UNKNOWN, "unknown error")
stack backtrace:
   0: rust_begin_unwind
             at /rustc/cc66ad468955717ab92600c770da8c1601a4ff33/library/std/src/panicking.rs:595:5
   1: core::panicking::panic_fmt
             at /rustc/cc66ad468955717ab92600c770da8c1601a4ff33/library/core/src/panicking.rs:67:14
   2: core::result::unwrap_failed
             at /rustc/cc66ad468955717ab92600c770da8c1601a4ff33/library/core/src/result.rs:1652:5
   3: cudarc::driver::safe::core::CudaDevice::new
   4: <candle_core::cuda_backend::CudaDevice as candle_core::backend::BackendDevice>::new
   5: candle_core::device::Device::new_cuda
   6: candle_examples::device
   7: whisper::main
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

It's a bit to hard to diagnose what is going on as the error message is not very informative.
I would suggest trying to run the examples/tests from the cudarc crate (which candle uses behind the hood), if they don't work this would indicate more of a general cuda issue rather than a candle specific one.

erm disregard.  This was due to some cuda init issue.  A reboot fixed it.
❯ cargo run --features cuda --example whisper --release
   Compiling candle-transformers v0.3.2 (/home/dres/src/gh/candle/candle-transformers)
   Compiling candle-examples v0.3.2 (/home/dres/src/gh/candle/candle-examples)
    Finished release [optimized] target(s) in 9.45s
     Running `target/release/examples/whisper`
No audio file submitted: Downloading https://huggingface.co/datasets/Narsil/candle_demo/blob/main/samples_jfk.wav
loaded wav data: Header { audio_format: 1, channel_count: 1, sampling_rate: 16000, bytes_per_second: 32000, bytes_per_sample: 2, bits_per_sample: 16 }
pcm data loaded 176000
loaded mel: [1, 80, 3000]
0.0s -- 30.0s:  And so my fellow Americans ask not what your country can do for you ask what you can do for your country


coco::detail::promise::return_void(): Misplaced rethrow

Greetings.
I'm looking at task.hpp here.
You're using coro::detail::promise::return_void() to rethrow the exception after the end of the coroutine. That part I understand.
But according to the spec, promise::return_void() is also called automatically during co_return. From the coroutine itself. Where a rethrow makes no sense.
This is a strange double use of the function, and I think it's just a bug. Or I don't get it, which is always a possibility.
In the case of promise-of-not-void you also use return_value() function for two purposes, but that works since they are distinct by parameter overload.
Best Regards, Jørn

Been out and about, I'll take a look tomorrow if I have some time.

This does indeed look like a bug, the return_value(v) is overloaded here one with an arg and one without where return_value() -> T is used in the task itself later, not by the compiler.  I've incorrectly used this function twice on the void one in task.hpp around lines 208 and 230.
Probably renaming the libraries return_void() and return_value() -> T used within task to something else would remedy this?

@Ankkatalo I've posted a PR https://github.com/jbaldwin/libcoro/pull/110 if you have time please take a look over it.

looks good to me.

Merged and tagged as v0.3


Send multi-part MIME emails via Govdelivery

As a va.gov business owner
When communicating with veterans over email
I want to make sure that the email is rendered correctly
Even when user receives text only emails
Checklist:

[ ] create multi-part MIME message body to be sent to Govdelivery
[ ] test with email client that displays emails only as plain-text
[ ] test with email client that displays html emails

We could potentially handle it the same way as SES client does:
multipart_content_subtype = 'alternative' if html_body else 'mixed'
msg = MIMEMultipart(multipart_content_subtype)
part = MIMEText(body, 'plain')
msg.attach(part)
if html_body:
    part = MIMEText(html_body, 'html')
    msg.attach(part)


Bug: IWbemClassObject::Get params are not *mut

Which crate is this about?
windows
Crate version
0.46
Summary
Hi!
I'm maintaining a WMI crate, and I tried updating from 0.44 to 0.46.
One of the things I saw was that IWbemClassObject::Get is now defined as:
pub unsafe fn Get<P0>(
    ...
    ptype: Option<*const i32>,
    plflavor: Option<*const i32>
) -> Result<()>

However, according to the docs, ptype and plfavor should be *mut:
HRESULT Get(
  [in]            LPCWSTR wszName,
  [in]            long    lFlags,
  [out]           VARIANT *pVal,
  [out, optional] CIMTYPE *pType,
  [out, optional] long    *plFlavor
);

I also checked emitter.settings.rsp, and both are set to [Optional], which I guess means they should be Option<*mut _>?
The end result is that my code which uses the pType, "works":
let cim_type = -1;

unsafe {
  self.inner.Get(
      PCWSTR::from_raw(name_prop.as_ptr()),
      0,
      &mut vt_prop,
      Some(&cim_type),
      None,
  )?;
}

// cim_type is mutated by `Get`.
assert_ne!(cim_type, -1);

Thanks! 🦀 🪟
Toolchain version/configuration
Default host: aarch64-pc-windows-msvc
rustup home:  C:\Users\oravid.rustup
stable-aarch64-pc-windows-msvc (default)
rustc 1.67.1 (d5a82bbd2 2023-02-07)
Reproducible example
No response
Crate manifest
[target.'cfg(target_os = "windows")'.dependencies]
windows = { version = "0.46", features = [
    "implement",
    "Win32_Foundation",
    "Win32_Security",
    "Win32_System_Com",
    "Win32_System_Ole",
    "Win32_System_Rpc",
    "Win32_System_Wmi",
] }

Expected behavior
No response
Actual behavior
No response
Additional comments
No response

Indeed, the header declaration is as follows:
        virtual HRESULT STDMETHODCALLTYPE Get(
            /* [string][in] */ LPCWSTR wszName,
            /* [in] */ long lFlags,
            /* [unique][in][out] */ VARIANT *pVal,
            /* [unique][in][out] */ CIMTYPE *pType,
            /* [unique][in][out] */ long *plFlavor) = 0;

The out parameter attributes are being ignored. The IDL declaration which is also available in the Windows SDK is even clearer:
    HRESULT Get(
        [in, string] LPCWSTR wszName,
        [in] long lFlags,
        [out, OPTIONAL] VARIANT* pVal,
        [out, OPTIONAL] CIMTYPE* pType,
        [out, OPTIONAL] long* plFlavor
        );

I'll transfer to the Win32 metadata repo for resolution. I'm not sure what it ends up parsing, but the metadata doesn't quite line up with docs, the header, or the IDL.


Verify Image Signatures: invalid type: map, expected a sequence

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
Invalid yaml produced by UI
  settings:
    description: null
    rule: PublicKey
    modifyImagesWithDigest: true
    signatures:
      image: theimage
      keyless:
        issuer: ''
        subject: ''

Expected Behavior
Valid yaml example from policy readme
signatures:
- image: "ghcr.io/kubewarden/*"
  githubActions:
    owner: "kubewarden"

Steps To Reproduce
No response
Environment
No response
Anything else?
No response

Works on 1.0.3-rc1