OpenAI

your import seems outdate.
Corrected:from langchain_openai.chat_models import ChatOpenAI

Thanks~ I'm planning to update langchain version as latest sooner or later in v2 repo.


Make DateTime friendlier

Add time for "Today" and "Yesterday" for completeness.
Fixes https://github.com/WalletWasabi/WalletWasabi/issues/13206

Sorry for the late action on this. I hope it's fine now.


Type stubs for versions 0.20.18 and up

Thanks for working on this! Would it be possible to cut a new release with the stubs for the recent versions of Polars (0.20.18 -> 0.20.22)? Hoping its not too much trouble based on https://github.com/StefanBRas/polugins/issues/57.
Thanks!

Yeah sure, I'll cut a release later today.

Done!


Add SWAG Vision Transformer Weight

subtask of https://github.com/pytorch/vision/issues/5708
Adding weight of ViT_B_16 and ViT_L_16 model from https://github.com/facebookresearch/SWAG

@datumbox here is the command I used for testing the vit_b_16_swag model:
python -u ~/script/run_with_submitit.py --timeout 3000 --ngpus 1 --nodes 1 --partition train --model vit_l_16 --data-path="/datasets01_ontap/imagenet_full_size/061417" --test-only --batch-size=1 --weights="ViT_L_16_Weights.IMAGENET1K_SWAG_V1"

The failures are unrelated. cc @pmeier
@YosuaMichael you are good to merge.


Can't scroll vertical on touch device when bound to x-axis

I'd like to use draggabilly for a swipe-to-dismiss feature, similar to gmail on Android. I've built a demo and I think that draggabilly performs and feels quite nice. It performs better and required much less code than using hammer.js. However...
When the elements fill the view, you can only scroll by dragging the small space between the elements, even though draggabilly is restricted to the x-axis.
Mobile test case: http://codepen.io/okfuture/pen/KVbYLr?editors=0010
Is there a way to allow default scroll behavior for the y-axis when restricted to x-axis?

Yes, this can be done. See demo http://codepen.io/desandro/pen/LGvYNy
// duck punch Draggabilly
var proto = Draggabilly.prototype;

var pointerMove = proto.pointerMove;
proto.pointerMove = function( event, pointer ) {
  var moveVector = this._dragPointerMove( event, pointer );
  this.touchVerticalScrollMove( event, pointer, moveVector );
  pointerMove.apply( this, arguments );
}

proto.hasDragStarted = function( moveVector ) {
  return !this.isTouchScrolling && Math.abs( moveVector.x ) > 3;
};

proto.canPreventDefaultOnPointerDown = function() {
  return false;
};

var touchScrollEvents = {
  touchmove: true,
  MSPointerMove: true
};

proto.touchVerticalScrollMove = function( event, pointer, moveVector ) {
  var isTouchScrollEvent = touchScrollEvents[ event.type ];
  // start scrolling if touch event and y moved 10px
  if ( this.options.touchVerticalScroll && isTouchScrollEvent &&
      !this.isTouchScrolling && Math.abs( moveVector.y ) > 10 ) {
    this.isTouchScrolling = true;
  }
};

var pointerUp = proto.pointerUp;
proto.pointerUp = function() {
  pointerUp.apply( this, arguments );
  delete this.isTouchScrolling;
};

But now that we have enabled scrolling, users can scroll to the right by dragging an item to the right.
At this point, the logic is different enough that I would consider writing your own Draggabilly-like class on top of Unidragger, rather than re-writing Draggabilly to do it.

Thanks for writing up the demo @desandro! Very much appreciated.


Python 3 port

Here is the full python 3.4.2 port changes for cocotb. Travis successfully builds both python2.7 and python3.4.2 now.
As mentioned in https://github.com/potentialventures/cocotb/issues/186

Great, thanks very much @jeremyherbert for your contributions :)


Stale reference to Docker ACI development repo

"Provide feedback about the Docker-Azure integration by creating an issue in the Docker ACI Integration GitHub repository."
That repo is now read-only.

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

ID: 920c8ddf-83d1-ba47-5a6e-270b390b33b5
Version Independent ID: df5d92c4-217b-aa4c-ba21-141850548e4a
Content: Tutorial - Use Docker Compose to deploy multi-container group - Azure Container Instances
Content Source: articles/container-instances/tutorial-docker-compose.md
Service: container-instances
GitHub Login: @tomvcassidy
Microsoft Alias: tomcassidy

Thanks for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response. Though the scope of our feedback channel here on GitHub covers specific documentation fixes, we can help redirect you to the right support channel to get an answer to your question: [reference to alternative product/service or community endpoint]. Because this issue does not relate to documentation, we are closing this issue. #please-close

My comment is about docs and is still relevant, I checked that the sentence and link are still present.  So this issue persists.
Not urgent for me - I gave up on docker-compose with ACI as it had too many limitations - but FYI to others.


Update rootcacertificates-csp.md

This PR Solved the Issue #3290 I clarify each principal Node, please review and merge

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@Tdiwestyguy: May we have your comments on this PR content and if this is the sort of change you were asking for in your issue ticket #3290 ?

@Tdiwestyguy: Are you available for comments on this PR content?
Does this PR contain the changes you asked for?

@manikadhiman The PR that addressess issue #3290 is available for final review. Thanks.

Hi @e0i, From #3290, the doc should be updated to depict nodes with their complete paths and their respective descriptions.
For example:
RootCATrustedCertificates/Root
Defines the certificate store that contains root, or self-signed certificates, in this case, the computer store.
RootCATrustedCertificates/Root/CertHash
Defines the SHA1 hash for the certificate. The 20-byte value of the SHA1 certificate hash is specified as a hexadecimal string value.
The supported operations are Get and Delete.
RootCATrustedCertificates/Root/CertHash/EncodedCertificate
Specifies the X.509 certificate as a Base64-encoded string. The Base-64 string value cannot include extra formatting characters such as embedded linefeeds, etc.
The supported operations are Add, Get, and Replace.
RootCATrustedCertificates/Root/CertHash/IssuedBy
Returns the name of the certificate issuer. This is equivalent to the Issuer member in the CERT_INFO data structure.
The only supported operation is Get.
RootCATrustedCertificates/Root/CertHash/IssuedTo
Returns the name of the certificate subject. This is equivalent to the Subject member in the CERT_INFO data structure.
The only supported operation is Get.
RootCATrustedCertificates/Root/CertHash/ValidFrom
Returns the starting date of the certificate's validity. This is equivalent to the NotBefore member in the CERT_INFO data structure.
The only supported operation is Get.
RootCATrustedCertificates/Root/CertHash/ValidTo
Returns the expiration date of the certificate. This is equivalent to the NotAfter member in the CERT_INFO data structure.
The only supported operation is Get.
RootCATrustedCertificates/Root/CertHash/TemplateName
Returns the certificate template name.
The only supported operation is Get.
You can follow the same model for CA, TrustedPublisher, TrustedPeople, and UntrustedCertificates nodes.
I understand this will involve replicating the same details under each parent node, but this will bring consistency and completeness to the doc.
@Tdiwestyguy: Please confirm if the above mentioned changes are the desired updates per #3290.
Thanks!

Thanks!
Looks good.

From: Manika Dhiman<EMAIL_ADDRESS>Sent: Monday, September 23, 2019 11:35 AM
To: MicrosoftDocs/windows-itpro-docs<EMAIL_ADDRESS>Cc: Alex<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/windows-itpro-docs] Update rootcacertificates-csp.md (#4934)
Hi @e0ihttps://github.com/e0i, From #3290https://github.com/MicrosoftDocs/windows-itpro-docs/issues/3290, the doc should be updated to depict nodes with their complete paths and their respective descriptions.
For example:
RootCATrustedCertificates/Root
Defines the certificate store that contains root, or self-signed certificates, in this case, the computer store.
RootCATrustedCertificates/Root/CertHash
Defines the SHA1 hash for the certificate. The 20-byte value of the SHA1 certificate hash is specified as a hexadecimal string value.
The supported operations are Get and Delete.
RootCATrustedCertificates/Root/CertHash/EncodedCertificate
Specifies the X.509 certificate as a Base64-encoded string. The Base-64 string value cannot include extra formatting characters such as embedded linefeeds, etc.
The supported operations are Add, Get, and Replace.
RootCATrustedCertificates/Root/CertHash/IssuedBy
Returns the name of the certificate issuer. This is equivalent to the Issuer member in the CERT_INFO data structure.
The only supported operation is Get.
RootCATrustedCertificates/Root/CertHash/IssuedTo
Returns the name of the certificate subject. This is equivalent to the Subject member in the CERT_INFO data structure.
The only supported operation is Get.
RootCATrustedCertificates/Root/CertHash/ValidFrom
Returns the starting date of the certificate's validity. This is equivalent to the NotBefore member in the CERT_INFO data structure.
The only supported operation is Get.
RootCATrustedCertificates/Root/CertHash/ValidTo
Returns the expiration date of the certificate. This is equivalent to the NotAfter member in the CERT_INFO data structure.
The only supported operation is Get.
RootCATrustedCertificates/Root/CertHash/TemplateName
Returns the certificate template name.
The only supported operation is Get.
You can follow the same model for CA, TrustedPublisher, TrustedPeople, and UntrustedCertificates nodes.
I understand this will involve replicating the same details under each parent node, but this will bring consistency and completeness to the doc.
@Tdiwestyguyhttps://github.com/Tdiwestyguy: Please confirm if the above mentioned changes are the desired updates per #3290https://github.com/MicrosoftDocs/windows-itpro-docs/issues/3290.
Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/windows-itpro-docs/pull/4934?email_source=notifications&email_token=AJI7GR7UWG4GWYNO5DYHHR3QLED7BA5CNFSM4IWQYV2KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7L2YCY#issuecomment-534227979, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AJI7GR5PVTW4FYY5HEM47OLQLED7BANCNFSM4IWQYV2A.


[APIM 2.6] 405 Method Not Allowed on Service provider > Forgot Password

Description:
On a stock and clean installation of the WSO2 API Manager 2.6, when trying the Recover username/Forgot password/Self signup features of a new service provider, a 405 Method Not Allowed is shown. The final endpoint: https://<hostname>:9443/accountrecoveryendpoint/recoverpassword.do?etc..

Steps to reproduce:

Install WSO2 API Manager 2.6
Create a new service provider and configure OAuth2
Go to: https://<hostname>:9443/oauth2/authorize?client_id=<YOUR_CLIENT_ID&redirect_uri=<YOUR_REDIRECT_URI>&response_type=code&scope=openid
Click on Forgot password

Affected Product Version:
API Manager 2.6

On WSO2 Identity Server 5.7 (compatible with APIM 2.6) this feature works, so I will proceed by using that instead as identity provider.


Redirects in pretty much all of ggplot2 pkg fail

In the Details section of the ggplot2::geom_bar() page, there are links to several position_*() functions such as position_dodge().

Clicking them redirects to the ggplot2 package page, not the appropriate function page.

Update: it's pretty much every function to function link in ggplot2 that fails.
Here's an example of a working link in the dplyr package:
https://www.rdocumentation.org/link/everything?package=dplyr&version=0.7.6&to=tidyselect
Here's an example of a broken link in ggplot2:
https://www.rdocumentation.org/link/aes()?package=ggplot2&version=3.0.0&to=%3Daes
Notice the extra () after the function name.


reader: rename value to get ?

Maybe rename value* methods to get* to match setters in the writer. And exists to has. While it is not used a lot

TODO: update docs !

done


Update languages for tasks 927 - 1088 and address some human feedback.

Some of the human feedbacks* didn't make sense. My guess is that it had something to do with the unfamiliarity of the languages of the tasks, and/or the characters not rendering correctly.

938 - 952

Thanks!
Yeah, there were a few mistakes on my side that caused some issues in rendering the examples, unfortunately.

Some of the human feedbacks* didn't make sense. My guess is that it had something to do with the unfamiliarity of the languages of the tasks, and/or the characters not rendering correctly.

938 - 952

Hey @Palipoor
for the tasks 939, 941, 943 the output language should be english I have addressed the missing tags issue in #586


ENH: linalg: add the function 'solve_circulant' for solving a circulant system

I updated the pull request.  The proposed signature of solve_circulant is now
def solve_circulant(c, b, singular='raise', tol=None, caxis=-1, baxis=0, outaxis=0):

When c is a 1-d array, @argriffing's suggested behavior is followed: solve_circulant(c, b) is a specialized implementation of solve(circulant(c), b), where solve is scipy.linalg.solve.
Note that scipy.linalg.solve and numpy.linalg.solve do not handle high dimensional b arrays the same way.  If, for example, a has shape (3, 3) and b has shape (3, 2, 5), scipy.linalg.solve(a, b) returns an array with shape (3, 2, 5);  numpy.linalg.solve(a, b) raises a ValueError. solve_circulant follows the convention of scipy.linalg.solve.

I updated the pull request.  The proposed signature of solve_circulant is now
def solve_circulant(c, b, singular='raise', tol=None, caxis=-1, baxis=0, outaxis=0):

When c is a 1-d array, @argriffing's suggested behavior is followed: solve_circulant(c, b) is a specialized implementation of solve(circulant(c), b), where solve is scipy.linalg.solve.
Note that scipy.linalg.solve and numpy.linalg.solve do not handle high dimensional b arrays the same way.  If, for example, a has shape (3, 3) and b has shape (3, 2, 5), scipy.linalg.solve(a, b) returns an array with shape (3, 2, 5);  numpy.linalg.solve(a, b) raises a ValueError. solve_circulant follows the convention of scipy.linalg.solve.

This looks ready. Besides a rebase, is there anything left to do here?

Rebased.  I don't have any more changes in mind.

@pv @argriffing maybe time for a final review and then green button it?

Yes, it has tests asserting that solve_circular(c, b) is almost equal to solve(circular(c), b), including tests for the 2d rhs mentioned by @pv, so I think it is safe to merge.

Thanks @WarrenWeckesser sorry it took a year to merge :/


fix: Ensure that we set to midnight the same day

…request from FE

Currently when we set our enddate to midnight we set it to midnight the day before, this ensures that we get the midnight of the requested date

Codecov Report

Merging #614 (fc690ea) into main (21978a1) will decrease coverage by 34.40%.
Report is 273 commits behind head on main.
The diff coverage is 35.30%.

@@             Coverage Diff             @@
##             main     #614       +/-   ##
===========================================
- Coverage   83.07%   48.67%   -34.40%
===========================================
  Files         456      364       -92
  Lines       10418     7116     -3302
===========================================
- Hits         8655     3464     -5191
- Misses       1438     3550     +2112
+ Partials      325      102      -223

Files
Coverage Δ

...ivedMessages/ArchivedMessageGetDocumentListener.cs
0.00% <ø> (ø)

source/Api/Common/HttpHeadersExtensions.cs
0.00% <ø> (ø)

...iddleware/Authentication/ActorForAuthentication.cs
0.00% <ø> (ø)

...ntication/Bearer/BearerAuthenticationMiddleware.cs
0.00% <ø> (ø)

...MarketActors/MarketActorAuthenticatorMiddleware.cs
0.00% <ø> (ø)

...ation/Middleware/Correlation/CorrelationContext.cs
66.66% <ø> (ø)

.../Middleware/Correlation/CorrelationIdMiddleware.cs
0.00% <ø> (ø)

...iguration/Middleware/Correlation/UserProperties.cs
0.00% <ø> (ø)

...figuration/Middleware/FunctionContextExtensions.cs
0.00% <ø> (ø)

...rce/Api/Configuration/ServiceBusMessageMetadata.cs
0.00% <ø> (ø)

... and 95 more

... and 505 files with indirect coverage changes
:mega: Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!


Multiple libraries were found for SD.h

Hello, when i try to compile the example of Arducam_ESP8266_UNO_V1_Capture2SD i have an error of sd library
Arduino: 1.8.5 (Windows 7), Board: "ArduCAM ESP8266 UNO, 80 MHz, 921600, 4M (3M SPIFFS)"
WARNING: library Wire claims to run on (esp8266) architecture(s) and may be incompatible with your current board which runs on (ArduCAM_ESP8266_UNO) architecture(s).
WARNING: library SPI claims to run on (esp8266) architecture(s) and may be incompatible with your current board which runs on (ArduCAM_ESP8266_UNO) architecture(s).
In file included from C:\Users\Visual\Documents\Arduino\libraries\SD\src/utility/Sd2Card.h:26:0,
             from C:\Users\Visual\Documents\Arduino\libraries\SD\src/utility/SdFat.h:29,

             from C:\Users\Visual\Documents\Arduino\libraries\SD\src/SD.h:20,

             from C:\Users\Visual\AppData\Local\Arduino15\packages\ArduCAM_ESP8266_UNO\hardware\ArduCAM_ESP8266_UNO\2.2.4\libraries\ArduCAM\examples\ESP8266\ArduCAM_ESP8266_UNO_V1_Capture2SD\ArduCAM_ESP8266_UNO_V1_Capture2SD.ino:17:

C:\Users\Visual\Documents\Arduino\libraries\SD\src/utility/Sd2PinMap.h:524:2: error: #error Architecture or board not supported.
#error Architecture or board not supported.
^
Multiple libraries were found for "SD.h"
Used: C:\Users\Visual\Documents\Arduino\libraries\SD
Not used: C:\Users\Visual\AppData\Local\Arduino15\packages\ArduCAM_ESP8266_UNO\hardware\ArduCAM_ESP8266_UNO\2.2.4\libraries\SD
Not used: C:\Program Files\Arduino\libraries\SD
exit status 1
Error compiling for board ArduCAM ESP8266 UNO.
How i can fix this error?
Regards

Hi, I had the same issue on a different program.
Deleting the folder C:\Users...\Documents\Arduino\libraries\SD solved it for me.

hi,
same issue
but after deleting the folder when i recompile it again the files created in this folder and same error occured please sugess

Hi,
Same for me too.
Its driving me mad, i have no idea why the issue came about..
ive uninstalled arduino, removed all temp files etc and as soon as i install the library i need i get the multiple library error

@bunglex
Have you try to delete all of the C:\Users...\Documents\Arduino folder?

yes we deleted all C:\Users...\Documents\Arduino folder but as we compile the directory arduino automatic created but its blank no files are available in it .
all libraries are in c program file

@bipinpcl22
If so, I advise you delete the Arduino IDE and re-install it.

Same problem here. I tried all options above, but nothing seems to work

The best method is to completely remove the Arduino IDE related folders and libraries and then reinstall the Arduino IDE and the Arducam esp8266 uno board.

Same for me too but ı updated esp8266 configuration. ıt works.  ı don't take same error anymore

Same for me too but ı updated esp8266 configuration. ıt works. ı don't take same error anymore

Can you please explain in more detail what you did?

Having the same problem with the latest 1.8.14 windows client and the esp32 package version 1.0.6. Arduino now selects the default SD library instead of the one in the esp32 package. Deleting the SD subfolder in the installation directory works as a temporary workaround, but it's not ideal.

How do you install the lib try installing from repo and board manager.

C:\Users\Visual\Documents\Arduino\libraries\SD
delete the Arduino folder, it worked for me

I did a   full IDE install of 2.2.1 windows version after getting the same problem.  This included removing all registry lines that contained "Arduino" and any and all Arduino files and directories on my C: drive. I still ended up with the same error except this time it said:
Used: C:\Users\XXXXXX\AppData\Local\Arduino15\packages\esp8266\hardware\esp8266\3.1.2\libraries\SD
Not used: C:\Users\XXXXXX\AppData\Local\Arduino15\libraries\SD
In my case I think the SD.h in the ESP8266 lib. and the standard lib. are different and the IDE had to figure that out.  At least this time it only loaded the right SD.h for my board.


Get rid of the redundant Injector wrapper

This is the proof of concept:

Hm, looking at that code I actually wonder if that class component needs to be there. Converting a whole thing to a single functional component would yield tree-like.

https://github.com/mobxjs/mobx-react/pull/715#issuecomment-504358745
Tests are green, but I did not check it on a real production codebase.

I deleted isStateless and now we apply React.forwardRef unconditionally.

Can you also add the test case you drafted here in that PR?

I will do it tomorrow because I have some problems with mobx-react tests.

I will do it tomorrow because I have some problems with mobx-react tests.

Done.

Well, nobody else seems to care, so let's ship it and fix later if necessary :)


Using v0x and v1x Components in the same component

[x ] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior
Both the v0x and v1x components should work in the same component
Current Behavior
While trying to do the above, I am getting errors
Steps to Reproduce (for bugs)
I am having an existing app where i have used the material 0x component and now I am trying to migrate to the V1 component. I have successfully installed the version V1. I am trying to use the SelectField component from 0x and new Select component from 1x like this.
import SelectField from "material-ui/SelectField"
import { Select as V1Select} from 'material-ui-next/Select'
But its giving me lot off errors.
is it possible to use both version components together?

@pavanshinde47 Yes, it should be working with no issue. Please provide a reproduction example.

@oliviertassinari  https://codesandbox.io/s/v6v01rqpy7 Please check this sandbox. The same error i am getting in my application.

@pavanshinde47 I see no error.

@oliviertassinari It was a wrong import, fixed that later. Thanks


Install and configure eslint-plugin-lodash

Importing individual lodash functions help keeping the bundle size small. This PR adds the eslint-plugin-lodash with some config to throw a warning when we attempt to use the second syntax - I updated the code in a few places to match the new rule.
// this only imports the groupBy function 👍
import groupBy from "lodash/groupBy";

// This imports all the lodash methods 👎
import { groupBy } from "lodash";

It wasn't working for me either, I had to install the ESLint VSCode Extension and enable it, then the squiggly line appears

It does show the error during the pre-commit checks too


Setup initial Hearth/Laravel codebase

Describe the solution you'd like
The LCIL will likely be embedded in the Right to Decide platform. To get an understanding of some of the technical requirements we should setup a basic Hearth/Laravel instance and develop the LCIL inside of it.
Describe alternative solutions you've considered
An alternative would be to just create a standalone instance of LCIL. However, it may not surface all of the technical requirements, possibilities, and etc. that might occur when used within the platform. That being said, if the LCIL will be a completely standalone app in the future, starting embedded in a Laravel app may introduce unnecessary dependencies and structure.

We have explored an initial spike of the LCIL using Airtable.


[main] Update actions

Cron -knative-prow-robot
/cc we don't use Prow
/assign we don't use Prow
Produced by: knative-sandbox/knobots/actions/update-actions
Details:

Codecov Report

Merging #451 (1e8a571) into main (94c349e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #451   +/-   ##
=======================================
  Coverage   81.83%   81.83%
=======================================
  Files          27       27
  Lines        1382     1382
=======================================
  Hits         1131     1131
  Misses        223      223
  Partials       28       28

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 94c349e...1e8a571. Read the comment docs.


fix(@vtmn/svelte): add correct types for Svelte popover component

Changes description
Svelte popover component had not correct type for the position attribute. I changed it for the correct one.
Context
We use this component in OneShop project
Checklist

[x] Make sure you are requesting to pull a topic/feature/bugfix branch. Please, don't request directly from your main!
[x] Check commits & PR names matches our requested structure. It must follow the https://www.conventionalcommits.org pattern.
[x] Check your code additions will fail neither code linting checks.
[x] I have reviewed the submitted code.
[x] I have tested on related showcases.
[x] If it includes design changes, please ask for a review with a core team designer.

Does this introduce a breaking change?
No

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

Thanks @Klescouar
Can you please sign the CLA? https://cla-assistant.io/Decathlon/vitamin-web?pullRequest=1199
(Feel free to wait, this software is sometimes long...)
Thanks a lot.


add ignored files will ignore all files

Short summary
when enable gomnd in golangci-lint
add ignore files will ignore all files

when excludes is blank, ignore files is ,ignores, then will match all files
Steps to reproduce the behavior
config:
gomnd:
settings:
mnd:
# the list of enabled checks, see https://github.com/tommy-muehle/go-mnd/#checks for description.
checks: case,condition,operation,return
ignored-numbers: 0,1
ignored-functions: fanout.*,GOMAXPROCS
ignored-files: "lexer.go"
go-mnd version or commit ref
v2.3.2
Go version (output of 'go version')
go1.16
Operating system / Environment
darwin
Expected behavior
only ignore some file
Actual behavior
ignore all files

https://github.com/tommy-muehle/go-mnd/pull/24

@XyuWang
Thanks for the report! Good catch!


installing pep8 fails

Environment data
VS Code version:
Python Extension version:
Python Version:
OS and version:
ubuntu 17.10
anaconda python 3.6.3
Actual behavior
/home/yolen/Desktop/data-analysis/azureenv/bin/python -m pip install pep8
yolen@yolen-ThinkPad-X260:~/Desktop/data-analysis$ /home/yolen/Desktop/data-analysis/azureenv/bin/python -m pip install pep8
Traceback (most recent call last):
  File "/home/yolen/scicomp/anaconda3/lib/python3.6/runpy.py", line 183, in _run_module_as_main
    mod_name, mod_spec, code = _get_module_details(mod_name, _Error)
  File "/home/yolen/scicomp/anaconda3/lib/python3.6/runpy.py", line 142, in _get_module_details
    return _get_module_details(pkg_main_name, error)
  File "/home/yolen/scicomp/anaconda3/lib/python3.6/runpy.py", line 109, in _get_module_details
    __import__(pkg_name)
  File "/home/yolen/Desktop/data-analysis/azureenv/lib/python3.6/site-packages/pip/__init__.py", line 28, in <module>
    from pip.vcs import git, mercurial, subversion, bazaar  # noqa
  File "/home/yolen/Desktop/data-analysis/azureenv/lib/python3.6/site-packages/pip/vcs/mercurial.py", line 9, in <module>
    from pip.download import path_to_url
  File "/home/yolen/Desktop/data-analysis/azureenv/lib/python3.6/site-packages/pip/download.py", line 36, in <module>
    from pip.utils.glibc import libc_ver
  File "/home/yolen/Desktop/data-analysis/azureenv/lib/python3.6/site-packages/pip/utils/glibc.py", line 4, in <module>
    import ctypes
  File "/home/yolen/scicomp/anaconda3/lib/python3.6/ctypes/__init__.py", line 7, in <module>
    from _ctypes import Union, Structure, Array
ImportError: /home/yolen/Desktop/data-analysis/azureenv/lib/python3.6/lib-dynload/_ctypes.cpython-36m-x86_64-linux-gnu.so: undefined symbol: _PyUnicode_AsWideCharString
yolen@yolen-ThinkPad-X260:~/Desktop/data-analysis$ ^C
yolen@yolen-ThinkPad-X260:~/Desktop/data-analysis$

Expected  behavior
Steps to reproduce:

Logs
Output from Python output panel

Output from Console window (Help->Developer Tools menu)

Please could you try installing pep8 manually from the terminal.
Try the following command:
/home/yolen/Desktop/data-analysis/azureenv/bin/python -m pip install pep8

that virtual env died so I cannot try. I think that my problem is due to my conda installation. When I click install pep8 it uses /usr/bin/python and not my conda installation elsewhere. So in the vs code terminal i get
yolen@yolen-ThinkPad-X260:~$ which python
/home/yolen/scicomp/anaconda3/bin/python

but clicking the "install autopep8" uses /usr/bin/python. Therefore I have tried to change the pythonpath in settings. Then I get

yolen@yolen-ThinkPad-X260:~$ /home/yolen/scicomp/anaconda3/bin/python -m pip install autopep8
Requirement already satisfied: autopep8 in ./scicomp/anaconda3/lib/python3.6/site-packages
Requirement already satisfied: pycodestyle>=2.3 in ./scicomp/anaconda3/lib/python3.6/site-packages (from autopep8)
yolen@yolen-ThinkPad-X260:~$

restart and it works it seems

Awesome

not working again :-(

Please could you be a bit more specific.

What's not working?
What's the error?
Is there any output in the Python output window and the console window (Help->Toggle Developer tools)

Looks like machine configuration issue, specifically C++ side of things.
lib/python3.6/lib-dynload/_ctypes.cpython-36m-x86_64-linux-gnu.so: undefined symbol: _PyUnicode_AsWideCharString

This mean Python binary is unable to locate PyUnicode_AsWideCharString entry point which usually means some installation placed shared library that is not compatible with this specific Python.
See https://stackoverflow.com/questions/45998875/tensorflow-python-importerror-undefined-symbol-pyunicode-aswidecharstring


change deprecated method

remove force unwrapped
swift 5 available

Thanks for the update (sorry it took so long to merge it in)


Signup Screen and Signout Button fixed

added working sign up screen and made sign out button lead back to login screen

LGTM!!


Blur resizes picture unexpected

I have a picture with 3000x3000 pixel. This operation ...
goeffects -effect=gaussian out.png out-gaussian-89-1.png 89 1
... resizes the new picture to 2912x2912 pixel.
How to avoid the resizing?

Hi - the code takes half the kernel size as a border around the image so that you are not trying to use pixels that fall outside the bounds of the image. Hence in your case the image is losing 2xborderWidth, where borderWidth is roughly kernelSize/2.
There is no way currently to avoid the resizing, you could update the code to maybe take an option that simply uses the edge pixel values to use outside the bounds of the image.

Thank for looking into the issue.  But I'm sorry ... with this very uncommon behavior goeffects is not really useful for me.


3.0.1

Add optional status description to outcomes v1 responses (e.g. ReadResultResponse).
Add properly formatted body to outcomes v1 responses when status code != 200.
Update version to 3.0.1.

Added two more commits to pull request. Based on recommendations (most recently https://medium.com/bynder-tech/c-why-you-should-use-configureawait-false-in-your-library-code-d7837dce3d7f), I added ConfigureAwait(false) to the await calls within public async methods in both NetCore and AspNetCore libraries.

RE: Github Releases, I try to remember to release when I publish to NuGet, but do forget sometimes. I haven't checked whether AppVeyor or Travis would automate that process.
RE: More detail, I agree. "Language usage." is not very helpful. Is "Add ConfigureAwait(false) to all await calls within public async methods in the AspNetCore library" better?

As for the releases, it could be pretty simple: publish every tag build to GitHub releases and Nuget feed.
If approve only OR with good messages/descriptions then it's possible to generate release notes with the tool like that.

I tried to automate publish on every tag build, but it seemed to get into a loop of building and publishing. Backed that out, but did experiment with manual publishing to both GitHub and NuGet. That worked, so I'll clear my head and try again later.
I'll also investigate GitReleaseManager. In the mean time, I've duplicated the NuGet release notes in the GitHub release notes, which--while hand written--are better than nothing.


namespaceSelector wrong in Gateway examples?

What happened:
Tried to apply the current examples on a k3d.
error: error validating "service_api.yaml": error validating data: [ValidationError(Gateway.spec.listeners[0].routes): unknown field "namespaceSelector" in io.x-k8s.networking.v1alpha1.Gateway.spec.listeners.routes, ValidationError(Gateway.spec.listeners[1].routes): unknown field "namespaceSelector" in io.x-k8s.networking.v1alpha1.Gateway.spec.listeners.routes]; if you choose to ignore these errors, turn validation off with --validate=false

What you expected to happen:
That it works ;-)
How to reproduce it (as minimally and precisely as possible):
Try to apply the examples.
Anything else we need to know?:
I figured that a definition like
  - protocol: HTTP
    routes:
      routeNamespaces:
        namespaceSelector: {}
      routeSelector:
        matchLabels:
          "controller": "traefik"
  - protocol: HTTPS

works. Now my question is, are the examples out-of-date or is there something wrong with the spec? To me, it looks like "old" examples but I wanna confirm here first. :-)

Thanks for raising this issue, we've not been good at keeping the examples up to date as we update the API. @jpeach has raised a PR that should help us catch this earlier in the future: https://github.com/kubernetes-sigs/service-apis/pull/201

Thanks. The PR looks good to me. Shall I raise a PR to fix the examples @robscott? Would that help?

Hey @SantoDE, I actually already had something in the works to deal with this so I just finished it up today, thanks for the offer though! There ended up being more examples that were out of date as well. Thanks for catching this!


Rounded taskbar support.

Hey Chris Andriessen,
I love TaskbarX and it made my pc look so much cleaner.
But I would really like a rounded taskbar setting, just like - RoundedTB
I think it would look really nice.

To be able to focus on major/current bugs I had to close all issues. The issues section has gotten very messy to a point that I couldn't follow it anymore. Most issues are duplicates, very old or are already fixed/solved. Feel free to reopen this issue if you still have this issue on the latest version of TaskbarX to get my attention. All feature requests have been put in a list here: 💡| Feature requests #870
I hope you understand ❤


Update Get Started

The get started guide says to include react-big-calendar/lib/css/react-big-calendar.css, but that file doesn't seem to be present in the repo. It's a bit confusing and it's just at the beginning which can easily mislead or discourage newcomers.
Thank you for the awesome work!

It's in the npm package, not the repo, since it's a generated file. The direction though is correct still :)


Fix "ERROR There is no current event loop in thread"

Closed in favor of https://github.com/openshift-eng/art-bot/pull/96


source uses wrong environment during debugging

Describe the bug
Parameter b is not passed to the sourced b.R when the source command is in a function.
Remove rm(b) to see that b.R sees b == 1 incorrectly.
To Reproduce
Debug the code below.
Your R code
file_conn <- file("b.R")
writeLines("message(b)", file_conn)
close(file_conn)

# this works
b <- 1
source("b.R")
rm(b)

# this fails
call_b <- function() {
    b <- 2
    source("b.R")
}
call_b()

Your Launch config
If applicable, the launch config that causes the bug. E.g.:
{
    "name": "R: Current File",
    "presentation": {
        "group": "00 User",
        "order": 20,
    },
    "type": "R-Debugger",
    "request": "launch",
    "file": "${fileDirname}/${fileBasenameNoExtension}.R",
    "debugMode": "file",
    "allowGlobalDebugging": true,
    "workingDirectory": "${fileWorkspaceFolder}",
}

Expected behavior
1
2

Actual behavior
1
Error in message(b) : object 'b' not found

Desktop (please complete the following information):

OS: Windows 10 21H1 remoting into Linux
R Version: 4.0.4
vscDebugger Version: 0.4.6
vscode-r-debugger Version: rdebugger.r-debugger v0.4.6

(I was quite sure I tested that this works in Rscript, but it seems it does not, either. Investigating if this can be closed immediately.)

Glad you could figure this out. Quick side note on source(): This function is by default overwritten by .vsc.debugSource(), which does (mostly) the same thing, but respects breakpoints. I tried writing this function so that it can be used just like source(), but some arguments that work with base::source are simply ignored by .vsc.debugSource. The local argument should work as expected, some other arguments might not work properly, though.


Popup does not work after page change when on AppBar

Current Behavior
Popup behavior is odd when placed onto AppShell/AppBar. It stops working when either there is a hot-reload on dev mode, or when you change the page.
You can see the button on each page that is outside the AppBar always works.

Steps To Reproduce

Load Stackblitz link
Click "Authenticated" in top nav bar
Click button on top right of AppBar - it should "flash".

Anything else?
Note that each page has its own layout with its own AppShell. This issue may be related to #1281
StackBlitz link: https://stackblitz.com/edit/skeletonlabs-repl-8scgxj?file=src/routes/(public)/+layout.svelte

@saturnonearth, going forward feel free to merge similar issues in one ticket. We can divide after the fact if we determine that's a good reason to. It just helps keep things organized. It also makes it easier to keep the discussion localized in one place.
For now I'll opt to discuss both issues in this ticket.
So the weird per here is your repro obvious replicates the issues described, but we have a prime example of this functionally working - which is the Explore and Theme popups used in the Skeleton docs site at the top of each page:
https://www.skeleton.dev/
These have been working for a long while without issue. Even in our local dev branch currently running the latest SvelteKit/Tailwind dependencies. The best way to diagnose the issues is to figure out what's different between your replications and our site.
I'll try to circle back on this soon, but off hand I'm not sure what's up here.

On second thought, I'm going to go ahead and link the other issue. Both these tasks can be handled together. We'll use this ticket as the centralize location for these tasks:
https://github.com/skeletonlabs/skeleton/issues/1281

I would like to work on this issue can you please assign it to me?

Thanks @sumitshinde-84, done. Please make sure you you review #1281 as well, which may be related. If the issue resides on Floating UI's end, let's try to report the issue over to them here:
https://github.com/floating-ui/floating-ui

@sumitshinde-84 checking on the progress for this. I haven't seen a draft PR come through, which is typically our recommendation for contribution:
https://www.skeleton.dev/docs/contributing/requirements
If something has come up and you won't be able to address this just let me know. No harm done. I just don't want this to sit idle too long.

I fixed the issue by changing one of both popup settings target.
The target is used to query the element and when duplicated will always return the first one.
This is a working Stackblitz link.
Maybe this should be mentioned in the documentation to avoid confusion.

I fixed the issue by changing one of both popup settings target. The target is used to query the element and when duplicated will always return the first one.

Good find!!
This makes sense now. It was a similar issue I was having when having the same targets from #each. I think in my mind, since they were different components or pages, and in Svelte, components are usually "self-contained", it would not matter - like other things such as CSS are abstracted without "global" tags, or even just normal vars.
But since it's just a normal HTML data attribute as a target, that makes sense!
I believe the PR for https://github.com/skeletonlabs/skeleton/issues/1336 would actually fix this issue in a way, as it would not rely on static data attribute values, but instead, actual references to the node.
But yea a note in the docs is a good idea as well.


Convert Input boxes to TextAreas

Is your feature request related to a problem? Please describe.
Input boxes are one liners so it becomes very uncovinient while working with sentences/paragraph.
Describe the solution you'd like
Converting all inputs to text areas should improve the user's experience
Describe alternatives you've considered
An alternative would be that instead of converting all inputs to text areas, there can be an option of one line or multiline in the TextField.
Additional context
None

I find this issue searching for hacktoberfest tag and i want to contribute but i see that @christeen24 already proposed a solution.
I will write here my opinion about the closed PR: the use of TextArea is right but maybe is better with this syntax:
<TextArea rows={1} autoSize={true} />

With these parameters, the TextArea will be in a single line (as a simple input) and when space is not enough or the 'enter' button is clicked the TextArea will automatically expand

Hey @lomby92,
I really like your proposed solution, this allows the current look to stay the same but also allows it to change when needed. Since I haven't heard from @christeen24 I can assign you is this issue. Let me know.

I will be happy to provide my PR. Tomorrow morning I will do it 😉
Now it's 17:57 in Italy


Update .hidden seminar for November

How much for kebab in your country man ? 🥙 @Costinteo

for free if you're coming with me to Efendi ;)

Nice to hear you still following my recommendation


[PORT] fix strict mode issue in expander

Port this change from botbuilder-dotnet/master branch:
https://github.com/microsoft/botbuilder-dotnet/pull/4057

fix  #3972: strict mode issue in expander
Changed projects

Microsoft.Bot.Builder.LanguageGeneration
Microsoft.Bot.Builder.LanguageGeneration.Tests

N/A


Object type PDWS - Workflow Template

The code here is just a shell. I'm sure you can submit the full class when done. Can we close?

The code here is just a shell. I'm sure you can submit the full class when done. Can we close?

The code here is just a shell. I'm sure you can submit the full class when done. Can we close?

Using a draft PR is intentional for purposes of visibility, at least that was the idea when discussed with @larshp.
I know there's been little progress, had an unusually busy December and now we're in full lockdown so haven't had time to work on it further. But do plan to pick it up again in February. If there's a preference we can close the PR and reopen a new one when I resume.

The code here is just a shell. I'm sure you can submit the full class when done. Can we close?

Using a draft PR is intentional for purposes of visibility, at least that was the idea when discussed with @larshp.
I know there's been little progress, had an unusually busy December and now we're in full lockdown so haven't had time to work on it further. But do plan to pick it up again in February. If there's a preference we can close the PR and reopen a new one when I resume.

well, I dont think we should have PRs open that are very old, but right now we have a PR that is older than this one 😁

well, I dont think we should have PRs open that are very old, but right now we have a PR that is older than this one 😁

Closing until I (or someone) picks this up again, we still have the branch for visibility.

Closing until I (or someone) picks this up again, we still have the branch for visibility.


Missing types in TS declarations.

For example, the queue class extends Array, but it most likely would extend Array, since it's an array of tracks.
I am unsure if this was intentional, but if not, it would greatly assist me in using TypeScript.
The Poru class also seems to be missing a few types, such as the return type of the functions decodeTrack(s), getLavalinkInfo, and getLavalinkStatus.
It's a really great package, but I'm having a bit of trouble with TypeScript regarding missing type declarations.

TypeScript issue will fix in next commit

Thank you very much, have a lovely day!


No easy way of predictable termination when working with streams (sequences)

Problem
Let's take a simplest problem - "print each number in a stream". Turns out current implementation doesn't allow one to do so without introducing some extra complexity.
Demonstration
Here's naive attempt to program this logic:
$lst -> iter -> unwrap
unwrap:some -> println -> del
unwrap:none -> :stop

The problem is with this line unwrap:none -> :stop.
There's no guarantee that unwrap:none is sending after all elements from unwrap:some are printed. In fact, there's higher chance that we terminate before we print all the elements because printing takes time.
The root of this problem is parallel processing without synchronization. If only we could somehow await for all elements to be printed.
Bad solution
One way to fix this is to get rid of parallelism and introduce a pipeline. If data moves through the single flow then there's no asynchronism and everything happens in sequential manner.
Let's print messages before unwrapping them!
$lst -> iter -> println -> unwrap
unwrap:none -> :stop

Because every message that comes to -> unwrap is already printed there's a guarantee that when unwrap:some -> is sending, all values are processed (printed in this case).
However, there's a problem with this solution. Let's say our $lst is [1,2,3]. The output will be not 1\n2\n3\n, it will be 1\n2\n3\n<nil>\n! It could be even worse if we change how maybe is represented as a string in the future. In other words - this is not a solution because program operates in a different way.

Important to understand that the problem is not about maybe, it's about parallelism and lack of synchronization. However, changing how stream works OR maybe how Iter works could fix this. As well as changing something else.

Use structs with indexes instead of maybe
Imagine each stream's element is wrapped into something like this
pub type StreamItem<T> struct {
  data T
  idx int
}

Component that emits stream uses something like this StreamItem | StreamDelimiter
StreamDelimiter could be {idx int} or even just int
Lock
When unwrap:none (or it's alternative) is sending we lock until some "processor" is done with the message from unwrap:some (or it's alternative). The trick is to unlock only if message from :some and :none has the same index (or if they are equal themselves, depending on the API)

^ question is how we handle maps this way?

When this will be implemented/fixed it might be possible to #568

This is how stream item could look instead of maybe. If handlers will receive it instead of just data we can get rid of parallelism (but the API for user won't be as handy)
pub type stream<T> struct {
    idx int
    last bool
    data T
}

Closed in favor of #644


[Docs] Correct example showing the record passed in as param

When following this example I discovered I could not call $this->record from that context. After a quick check I discovered the $record must be passed in as a parameter into the closure.
NOTE: There are other examples on this docs page of $this->record that I suspect may also need to be updated but I have not tested that so I am not updating them, unless someone can confirm that is also the case there. I am making this a draft until I can get confirmation on this.

Hmm I think you're right and we should move all examples to $record. But $this->record does still work in some places.

Yeah I wasn't sure. But I can go ahead and just switch all the examples to $record. Personally that's probably how I'd actually do it instead of having to figure out/remember where $this does and does not work.


why can my Docker host can fill up with a lot of unused images?

Even though the buildkite agent cleans up the images after each run, inevitably over time your Docker host can fill up with a lot of unused images.

if it cleans up images after each run, how would my host fill up?

The agent doesn't seem to clean up images after every run, at least not in the buildkite docker build plugin.
Also what is https://github.com/buildkite/agent/blob/master/packaging/linux/root/usr/share/buildkite-agent/docker-cleanup.sh for ?
And the docs should only recommend a single tool for cleaning up images. Or explain when docker system prune would be preferable to docker-gc.

The agent doesn't seem to clean up images after every run, at least not in the buildkite docker build plugin.

No, it doesn't, we didn't include any on purpose. The Docker Compose plugin does clean up the project-based images (but not standard tags, such as alpine:latest. Are you finding it a problem?
docker-cleanup.sh is an old file I believe — I've just opened https://github.com/buildkite/agent/pull/914 to remove it.
I believe  docker system prune should be the preferred default, but the idea is that it's good to know that there are alternatives such as docker-gc.

No, it doesn't, we didn't include any on purpose. The Docker Compose plugin does clean up the project-based images (but not standard tags, such as alpine:latest. Are you finding it a problem?

Nah, just trying to understand the precise behaviour here. The docs should be modified to reflect that the only docker compose plugin does cleanup.

I believe docker system prune should be the preferred default, but the idea is that it's good to know that there are alternatives such as docker-gc.

It adds more choice for a user which is confusing when you can't really tell the difference. See https://github.com/spotify/docker-gc/issues/199

There was a time where docker system prune was very new, and so we had to give alternatives for newer and older Dockers… and prune didn’t have all the options it does now. But having looked at those (and comparing them more closely) we can probably just remove the docker-gc link now! 👌🏼

The docs should be modified to reflect that the only docker compose plugin does cleanup.

Which doc are you referring to?

@toolmantim https://buildkite.com/docs/tutorials/docker-containerized-builds#adding-a-cleanup-task

Thanks @nhooyr! I've just opened a PR to clarify things.


No example for row header template

The documentation says "If you would like to alter a row header visual, you should first provide one by altering the DataGrid's control template." but gives no example how to accomplish this.
The documentation has examples of changing all other templated sections of the DataGrid, but not the row header. After hours of research, it actually appears to be impossible in the current version of the DataGrid. Am I missing something?

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

ID: 30c8bdc3-08a5-6c3d-a652-429098db76ba
Version Independent ID: 877092f0-922e-b2bc-2b65-62781bdf6efc
Content: How to - Customize the DataGrid control through UI formatting options - Windows Community Toolkit
Content Source: docs/controls/datagrid_guidance/styling_formatting_options.md
Product: community-toolkit
GitHub Login: @harinikmsft
Microsoft Alias: mhawker

@michael-hawker , @anawishnoff, @harinikmsft , @RBrid: It's very disappointing that a year later this is still undocumented. If row headers are indeed supported, please add them to the DataGrid in the Windows Community Toolkit Sample App to demonstrate that they work.

@EddieLotter Depending on what you want to do with the Header, you can alter it with simple styling.
For example, this

Gives

Or, I can modify the template:
<Style TargetType="wctprimitives:DataGridColumnHeader">
    <Setter Property="Foreground" Value="Purple" />
    <Setter Property="FontFamily" Value="Courier New" />
    <Setter Property="Template">
        <Setter.Value>

            <ControlTemplate TargetType="wctprimitives:DataGridColumnHeader">
                <Grid x:Name="ColumnHeaderRoot">
                    <!-- Omitted VisualStates -->
                    <Grid.ColumnDefinitions>
                        <ColumnDefinition Width="*"/>
                        <ColumnDefinition Width="Auto"/>
                    </Grid.ColumnDefinitions>

                    <Rectangle x:Name="BackgroundRectangle" Stretch="Fill" Fill="{ThemeResource DataGridColumnHeaderBackgroundBrush}" Grid.ColumnSpan="2"/>

                    <Grid HorizontalAlignment="{TemplateBinding HorizontalContentAlignment}" VerticalAlignment="{TemplateBinding VerticalContentAlignment}" Margin="{TemplateBinding Padding}">
                        <Grid.ColumnDefinitions>
                            <ColumnDefinition Width="*"/>
                            <!-- I've added this column -->
                            <ColumnDefinition Width="Auto"/>
                            <ColumnDefinition MinWidth="32" Width="Auto"/>
                        </Grid.ColumnDefinitions>

                        <ContentPresenter Content="{TemplateBinding Content}" HorizontalAlignment="Left" VerticalAlignment="Center"/>

                        <!-- I've added this symbol in every header -->
                        <SymbolIcon Symbol="Emoji" Grid.Column="1" />
                        <FontIcon Grid.Column="2" x:Name="SortIcon" FontFamily="{ThemeResource SymbolThemeFontFamily}" Glyph="{ThemeResource SortIconAscending}" FontSize="12"
                        Foreground="{ThemeResource DataGridColumnHeaderForegroundBrush}" HorizontalAlignment="Center" VerticalAlignment="Center" Opacity="0"/>
                    </Grid>
                    <Rectangle x:Name="VerticalSeparator" Grid.Column="1" Width="1" VerticalAlignment="Stretch" Fill="{TemplateBinding SeparatorBrush}" Visibility="{TemplateBinding SeparatorVisibility}"/>

                    <Grid x:Name="FocusVisual" IsHitTestVisible="False" Opacity="0">
                        <Rectangle x:Name="FocusVisualPrimary" Stroke="{ThemeResource DataGridCellFocusVisualPrimaryBrush}" StrokeThickness="2" Fill="Transparent"
                        HorizontalAlignment="Stretch" VerticalAlignment="Stretch" IsHitTestVisible="False"/>
                        <Rectangle x:Name="FocusVisualSecondary" Stroke="{ThemeResource DataGridCellFocusVisualSecondaryBrush}" StrokeThickness="1" Fill="Transparent"
                        HorizontalAlignment="Stretch" VerticalAlignment="Stretch" IsHitTestVisible="False" Margin="2"/>
                    </Grid>

                </Grid>
            </ControlTemplate>
        </Setter.Value>
    </Setter>
</Style>

[Most of the above came from the source.]
And I get an emoji symbol added to every header. You, of course, could modify however you wish.

It's not the simplest thing to work out how to do. What do you think the modified documentation should show how to do?

@mrlacey, thanks for the time and effort you put into your response. Unfortunately you misread my comment.
I'm talking about the row header, not the column header.
As you point out, the column headers are well documented. I'm pointing out that the row headers are not.
Have you successfully used row headers?

@EddieLotter Sorry, my mistake, but I used the same approach to identify how to modify row headers as I did for column headers.
ROW Headers seem like an unloved feature. By default you get nothing in them.
For anything to show up you have to provide a style that includes the content you want.
As a simple example, I took the sample app and added this to DataGridPage.xaml
      xmlns:wctprimitives="using:Microsoft.Toolkit.Uwp.UI.Controls.Primitives"
      mc:Ignorable="d">
    <Page.Resources>
        <Style TargetType="wctprimitives:DataGridRowHeader">
            <Setter Property="Foreground" Value="Orange" />
            <Setter Property="Background" Value="Red" />
            <Setter Property="Template">
                <Setter.Value>
                    <ControlTemplate TargetType="wctprimitives:DataGridRowHeader">
                        <Grid Background="{TemplateBinding Background}">
                            <TextBlock Text="{Binding Mountain}" />
                        </Grid>
                    </ControlTemplate>
                </Setter.Value>
            </Setter>
        </Style>
    </Page.Resources>

[Colors chosen to make it clear they're being used and not for aesthetic purposes]
This produces the following when HeadersVisibility is set to All (or Row).

I couldn't get this to work by setting the RowHeaderStyle directly though.
The use of RowHeaders is definitely something that looks like it needs lots of attention if taken forward in WinUI (/cc @anawishnoff ) - the documentation is lacking and the code contains lots of commented out functionality.

@mrlacey, thanks once again for your time, effort and detailed responses.
Your example is clear and concise and has given me what I need to proceed with my project.
I do hope the team opts to take this forward in WinUI. There are legitimate use cases for row headers and using a frozen, read-only column as a work-around is less than ideal.
Do you recommend I close this issue or leave it open until the documentation is updated?

Do you recommend I close this issue or leave it open until the documentation is updated?

I'd leave it open to signal that there's still an issue with the documentation.

Thanks @mrlacey for the solution, and thanks @EddieLotter for bringing this back up. It does look like the only way to change the row header styling is to edit the control template for the DataGridRowHeader, rather than using the RowHeaderStyle property.
I'm not sure if this is by design (and if so, why). I'll dig a bit deeper into this and update the docs accordingly.

Thanks @mrlacey for the solution, and thanks @EddieLotter for bringing this back up. It does look like the only way to change the row header styling is to edit the control template for the DataGridRowHeader, rather than using the RowHeaderStyle property.
I'm not sure if this is by design (and if so, why). I'll dig a bit deeper into this and update the docs accordingly.

Hi again folks, it looks like most of @mrlacey's comments are correct except that you actually should be able to set the RowHeaderStyle property directly. I've tweaked the code sample a bit to follow along with the doc tutorial (changed variable names in binding and set colors directly). I also added a name to the style resource.
      xmlns:wctprimitives="using:Microsoft.Toolkit.Uwp.UI.Controls.Primitives"
      mc:Ignorable="d">
      <Page.Resources>
            <Style TargetType="wctprimitives:DataGridRowHeader" x:Name="myRowHeader">
                <Setter Property="Foreground" Value="Orange" />
                <Setter Property="Template">
                    <Setter.Value>
                        <ControlTemplate TargetType="wctprimitives:DataGridRowHeader">
                            <Grid Background="Red">
                                <TextBlock Text="{Binding LastName}" />
                            </Grid>
                        </ControlTemplate>
                    </Setter.Value>
                </Setter>
            </Style>
        </Page.Resources>

Assigning to this DataGrid:

<controls:DataGrid x:Name="dataGrid1"
    Height="600" Margin="12"
    AutoGenerateColumns="True"
    ItemsSource="{x:Bind Customers}"
    HeadersVisibility="Row" <!-- Can also be "All" -->
    RowHeaderStyle="{StaticResource myRowHeader}"/>

@mrlacey / @EddieLotter was there a specific issue you were hitting when trying to assign to the RowHeaderStyle property directly?
I think it's important to mention the following things in our DataGrid docs, based off of this thread:

In order to set up your DataGrid with row headers, you must provide a style targeted at the DataGridRowHeader type (which is a part of the Microsoft.Toolkit.Uwp.UI.Controls.Primitives library) that includes a control template.
This style can be defined as a global resource and be used for all DataGrid objects in your app, or it can be assigned to a specific DataGrid using the DataGrid.RowHeaderStyle property.
Code sample similar to what @mrlacey showed above.

@mrlacey, let me know if you'd be interested in updating the docs. If not, I'll do my best to add these updates in ASAP. Thanks again for bringing this up!

Hi again folks, it looks like most of @mrlacey's comments are correct except that you actually should be able to set the RowHeaderStyle property directly. I've tweaked the code sample a bit to follow along with the doc tutorial (changed variable names in binding and set colors directly). I also added a name to the style resource.
      xmlns:wctprimitives="using:Microsoft.Toolkit.Uwp.UI.Controls.Primitives"
      mc:Ignorable="d">
      <Page.Resources>
            <Style TargetType="wctprimitives:DataGridRowHeader" x:Name="myRowHeader">
                <Setter Property="Foreground" Value="Orange" />
                <Setter Property="Template">
                    <Setter.Value>
                        <ControlTemplate TargetType="wctprimitives:DataGridRowHeader">
                            <Grid Background="Red">
                                <TextBlock Text="{Binding LastName}" />
                            </Grid>
                        </ControlTemplate>
                    </Setter.Value>
                </Setter>
            </Style>
        </Page.Resources>

Assigning to this DataGrid:

<controls:DataGrid x:Name="dataGrid1"
    Height="600" Margin="12"
    AutoGenerateColumns="True"
    ItemsSource="{x:Bind Customers}"
    HeadersVisibility="Row" <!-- Can also be "All" -->
    RowHeaderStyle="{StaticResource myRowHeader}"/>

@mrlacey / @EddieLotter was there a specific issue you were hitting when trying to assign to the RowHeaderStyle property directly?
I think it's important to mention the following things in our DataGrid docs, based off of this thread:

In order to set up your DataGrid with row headers, you must provide a style targeted at the DataGridRowHeader type (which is a part of the Microsoft.Toolkit.Uwp.UI.Controls.Primitives library) that includes a control template.
This style can be defined as a global resource and be used for all DataGrid objects in your app, or it can be assigned to a specific DataGrid using the DataGrid.RowHeaderStyle property.
Code sample similar to what @mrlacey showed above.

@mrlacey, let me know if you'd be interested in updating the docs. If not, I'll do my best to add these updates in ASAP. Thanks again for bringing this up!

@anawishnoff docs change PR created

@anawishnoff docs change PR created

@EddieLotter was there a specific issue you were hitting when trying to assign to the RowHeaderStyle property directly?

@anawishnoff, no I didn't get as far as trying to style the row header because I couldn't get any content into the row header.
Since there is a DataGridTemplateColumn template in the Microsoft.Toolkit.Uwp.UI.Controls.DataGrid name space, I was trying unsuccessfully to find something similar for the row header, something like DataGridTemplateRowHeader. I would never have thought to use the Microsoft.Toolkit.Uwp.UI.Controls.Primitives name space the way @mrlacey did. 🤪

@EddieLotter was there a specific issue you were hitting when trying to assign to the RowHeaderStyle property directly?

@anawishnoff, no I didn't get as far as trying to style the row header because I couldn't get any content into the row header.
Since there is a DataGridTemplateColumn template in the Microsoft.Toolkit.Uwp.UI.Controls.DataGrid name space, I was trying unsuccessfully to find something similar for the row header, something like DataGridTemplateRowHeader. I would never have thought to use the Microsoft.Toolkit.Uwp.UI.Controls.Primitives name space the way @mrlacey did. 🤪

Ah, I see. Thanks @mrlacey for the PR! Reviewing it now.

Ah, I see. Thanks @mrlacey for the PR! Reviewing it now.


[Snyk] Fixing go.mod snyk CVEs

Description of your changes:

Fixing go.mod dependencies/packages related snyk CVEs.
Imported my fork in synk and made sure the relevant CVEs were fixed with this update.
Added a replace block for each CVE with the fix version suggested by Snyk.
Ran go mod tidy. Wherever go version compatibility issues came up, ran go mod tidy -go=1.16 && go mod tidy -go=1.17
Pushed changes to my fork and ensured the CVEs were resolved.
Also executed a go build . locally and ensure it doesn't return any errors.
These updates resolve 24/27 Snyk High Level errors for the go.mod.
Testing instructions
Pull the changes in this PR, push them to your fork's main (make sure the main branch of the fork is in sync with odh-io/argo-workflows main). Import your fork in snyk and verify.

Closing as this has been duplicated by https://github.com/opendatahub-io/argo-workflows/pull/5


Fixed syntax error in markdown for Standard LRS Storage link

@garvinmsft,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@garvinmsft : Thanks for your contribution to the Azure documentation! The author, @jboeshart, has been notified to review your proposed change.

Thanks for the correction @garvinmsft!
#sign-off


thephpleague/oauth2-client on openhub

Is there thephpleague/oauth2-client page on openhub?
I should provide for our customers estimated cost info for development of oauth2 client with all implemented clients.
Some of them cannot understand if I try develop Facebook Client myself I got problems with Google Client and other  auth services.
I can find only info on deprecated HA libraray

I'm open the idea of having a page on OpenHub. Won't be adding it myself though.


ERR: pngdefry : seen 1 file(s), wrote 0 file(s) fs.js:640

@bumaociyuan hi, I get an error below:
pngdefry : seen 1 file(s), wrote 0 file(s)
fs.js:640
  return binding.open(pathModule._makeLong(path), stringToFlags(flags), mode);
...

and my install steps:
git clone https://github.com/bumaociyuan/ios-ipa-server.git
cd ios-ipa-server
npm install
npm link
ios-ipa-server

and I also modify the template.plist with my app identifier(same as .ipa's name)
at last, it crashed after visited https://my_local_ip:1234/download
please do some help when your are free, thanks!

Thank you for report this
I'll check this on Monday

git clone https://github.com/bumaociyuan/ios-ipa-server.git
cd ios-ipa-server
npm install
npm link
cd ipa-folder/
ios-ipa-server

On my mac everything works fine.
Would you post your system version and
node --version
npm --version

@bumaociyuan node: v6.6.0, npm: 3.10.7, my verbose:
$ ios-ipa-server
https://<IP_ADDRESS>:1234/download
./icon.png : not an -iphone crushed PNG file
pngdefry : seen 1 file(s), wrote 0 file(s)
fs.js:640
  return binding.open(pathModule._makeLong(path), stringToFlags(flags), mode);
                 ^

Error: ENOENT: no such file or directory, open './icon_tmp.png'
    at Error (native)
    at Object.fs.openSync (fs.js:640:18)
    at Object.fs.readFileSync (fs.js:508:33)
    at base64_encode (/Users/mac/Desktop/ipa-automic/ios-ipa-server/bin/ios-ipa-server.js:207:19)
    at /Users/mac/Desktop/ipa-automic/ios-ipa-server/bin/ios-ipa-server.js:189:49
    at Array.forEach (native)
    at itemInfoWithName (/Users/mac/Desktop/ipa-automic/ios-ipa-server/bin/ios-ipa-server.js:183:14)
    at /Users/mac/Desktop/ipa-automic/ios-ipa-server/bin/ios-ipa-server.js:124:20
    at /Users/mac/Desktop/ipa-automic/ios-ipa-server/node_modules/graceful-fs/graceful-fs.js:78:16
    at FSReqWrap.readFileAfterClose [as oncomplete] (fs.js:445:3)

I just quick fixed this bug
cd ios-ipa-server-repo/
git pull
cd ipa-location/
ios-ipa-server # try again

@bumaociyuan yeah! It works now after I git pull and git checkout templates/template.plist and delete the same bundleIdentifier app,  now it still shows some error but app can be installed:
$ ios-ipa-server
https://<IP_ADDRESS>:1234/download
./icon.png : not an -iphone crushed PNG file
pngdefry : seen 1 file(s), wrote 0 file(s)
{ Error: ENOENT: no such file or directory, open './icon_tmp.png'
    at Error (native)
    at Object.fs.openSync (fs.js:640:18)
    at Object.fs.readFileSync (fs.js:508:33)
    at base64_encode (/Users/mac/Desktop/ipa-automic/ios-ipa-server-lib/bin/ios-ipa-server.js:214:19)
    at /Users/mac/Desktop/ipa-automic/ios-ipa-server-lib/bin/ios-ipa-server.js:193:51
    at Array.forEach (native)
    at itemInfoWithName (/Users/mac/Desktop/ipa-automic/ios-ipa-server-lib/bin/ios-ipa-server.js:187:16)
    at /Users/mac/Desktop/ipa-automic/ios-ipa-server-lib/bin/ios-ipa-server.js:125:20
    at /Users/mac/Desktop/ipa-automic/ios-ipa-server-lib/node_modules/graceful-fs/graceful-fs.js:78:16
    at FSReqWrap.readFileAfterClose [as oncomplete] (fs.js:445:3)
  errno: -2,
  code: 'ENOENT',
  syscall: 'open',
  path: './icon_tmp.png' }
./icon.png : writing to file ./icon_tmp.png
pngdefry : seen 1 file(s), wrote 1 file(s)

@litt1e-p It's just a quick fix. I print the error log. Just ignore that.

@bumaociyuan all right, thank you

@litt1e-p It looks like that there are some problem with your app's icon, make sure it is a iphone crushed PNG file

@bumaociyuan I archived two ipa file which one was archive by Xcode and other one archived by shenzhen tools(this ipa caused the error), so it is not ios-ipa-server's fault, you are right, I will fix it myself. thanks a lot!


Write tests for *bob rendering

This works in general but the test coverage has dropped significantly so some unit testing is required.
This is also important because it will help us to recognise when we have changed things that will break this.

Completed in #108


[Feature] Add a "template" command to the vela cli

Is your feature request related to a problem? Please describe.
Its important to template out the k8s resources in the CI Pipeline to run tools like Datree for policy management. Currently there is a vela dry-run but it requires access to a k8s cluster (kube api server to be more precise). This isn't possible in most ci layers.
Describe the solution you'd like
A vela template command to spit out the k8s resources without requiring a kubeapi. We can provide it a directory containing all the trait and component definitions so it doesn't have to query kubernetes.
Describe alternatives you've considered
Dry run the resources and commit it in git alongside the application file. This isn't the best option.
Additional context

I think you can refactor the vela dry-run command , if the definition and other related things are provided, it can avoid access to K8s cluster.

Let me try making the necessary modifications. Any help in directing me to the right file / function would be great

The basic code of vela dry-run is here: https://github.com/kubevela/kubevela/blob/master/references/cli/dryrun.go
The mechanism is read from K8s for Definition and other information, finally it will revoke the function same with the controller and do the rendering.
The work here is to avoid connect to K8s cluster, provide the basic information from local.


Update Linux-CIP to kernel version 6.1

This pull request is an initial port to version 6.1 of linux-cip. Basic functionality seems to be working, including ethernet with the upstream version of the prueth driver. Some of the patches from 5.10 still need to be addressed, like EFI and some more advanced ethernet functionality.

Ethernet is not working yet, driver does not bind to the device.

My Ethernet issue has been understood and resolved - firmware bug<EMAIL_ADDRESS>

@diogo-ivo Any updates on this?

Just tested this in secure boot setup, and our reboot errors are back:
...
[  OK  ] Stopped systemd-remount-fs…ount Root and Kernel File Systems.
         Stopping tee-supplicant.service - TEE Supplicant...
[  OK  ] Stopped tee-supplicant.service - TEE Supplicant.
[  OK  ] Reached target shutdown.target - System Shutdown.
[  OK  ] Reached target final.target - Late Shutdown Services.
[  OK  ] Finished systemd-reboot.service - System Reboot.
[  OK  ] Reached target reboot.target - System Reboot.
[  693.436109] watchdog: watchdog0: nowayout prevents watchdog being stopped!
[  693.443675] watchdog: watchdog0: watchdog did not stop!
[  693.484356] systemd-shutdown[1]: Using hardware watchdog 'K3 RTI Watchdog', version 0, device /dev/watchdog0
[  693.494562] systemd-shutdown[1]: Modifying watchdog timeout is not supported, reusing the programmed timeout.
[  693.504586] systemd-shutdown[1]: Watchdog running with a timeout of 1min.
[  693.551009] systemd-shutdown[1]: Syncing filesystems and block devices.
[  693.579837] systemd-shutdown[1]: Sending SIGTERM to remaining processes...
[  693.600918] systemd-journald[1450]: Received SIGTERM from PID 1 (systemd-shutdow).
[  693.620817] systemd-shutdown[1]: Sending SIGKILL to remaining processes...
[  693.641910] systemd-shutdown[1]: Unmounting file systems.
[  693.650484] (sd-remount)[1829]: Remounting '/etc' read-only with options 'lowerdir=/root/etc,upperdir=/root/var/local/etc,workdir=/root/var/local/.etc-atomic'.
[  693.676796] (sd-umount)[1830]: Unmounting '/etc'.
[  693.705311] EXT4-fs (dm-2): unmounting filesystem.
[  693.742062] systemd-shutdown[1]: All filesystems unmounted.
[  693.747690] systemd-shutdown[1]: Deactivating swaps.
[  693.752764] systemd-shutdown[1]: All swaps deactivated.
[  693.758000] systemd-shutdown[1]: Detaching loop devices.
[  693.766053] systemd-shutdown[1]: All loop devices detached.
[  693.771654] systemd-shutdown[1]: Stopping MD devices.
[  693.777039] systemd-shutdown[1]: All MD devices stopped.
[  693.782362] systemd-shutdown[1]: Detaching DM devices.
[  693.789535] systemd-shutdown[1]: Detaching DM /dev/dm-2 (252:2).
[  693.834609] systemd-shutdown[1]: Detaching DM /dev/dm-1 (252:1).
[  693.886590] systemd-shutdown[1]: Not all DM devices detached, 1 left.
[  693.893379] systemd-shutdown[1]: Detaching DM devices.
[  693.899806] systemd-shutdown[1]: Not all DM devices detached, 1 left.
[  693.906502] systemd-shutdown[1]: Detaching DM devices.
[  693.912544] systemd-shutdown[1]: Not all DM devices detached, 1 left.
[  693.919001] systemd-shutdown[1]: Cannot finalize remaining DM devices, continuing.
[  693.926574] watchdog: watchdog0: nowayout prevents watchdog being stopped!
[  693.933451] watchdog: watchdog0: watchdog did not stop!
[  693.943170] systemd-shutdown[1]: Failed to finalize DM devices, ignoring.
[  693.951167] systemd-shutdown[1]: Syncing filesystems and block devices.
[  693.958145] systemd-shutdown[1]: Rebooting.
E/TC:? 0 get_rpc_alloc_res:645 RPC allocation failed. Non-secure world result: ret=0xffff000c ret_origin=0x2
E/TC:? 0 get_rpc_alloc_res:645 RPC allocation failed. Non-secure world result: ret=0xffff000c ret_origin=0x2
E/TC:? 0
E/TC:? 0 TA panicked with code 0xffff000c
E/LD:  Status of TA bc50d971-d4c9-42c4-82cb-343fb7f37896
E/LD:   arch: aarch64
E/LD:  region  0: va 0x40004000 pa 0x9ee1a000 size 0x002000 flags rw-s (ldelf)
E/LD:  region  1: va 0x40006000 pa 0x9ee1c000 size 0x008000 flags r-xs (ldelf)
E/LD:  region  2: va 0x4000e000 pa 0x9ee24000 size 0x001000 flags rw-s (ldelf)
E/LD:  region  3: va 0x4000f000 pa 0x9ee25000 size 0x004000 flags rw-s (ldelf)
E/LD:  region  4: va 0x40013000 pa 0x9ee29000 size 0x001000 flags r--s
E/LD:  region  5: va 0x40014000 pa 0x9eeb1000 size 0x011000 flags rw-s (stack)
E/LD:  region  6: va 0x40025000 pa 0xf2b9c000 size 0x002000 flags rw-- (param)
E/LD:  region  7: va 0x40044000 pa 0x00001000 size 0x068000 flags r-xs [0]
E/LD:  region  8: va 0x400ac000 pa 0x00069000 size 0x01f000 flags rw-s [0]
E/LD:   [0] bc50d971-d4c9-42c4-82cb-343fb7f37896 @ 0x40044000
E/LD:  Call stack:
E/LD:   0x40082c18
E/LD:   0x40044b58
E/LD:   0x400451d8
E/LD:   0x40064a40
E/LD:   0x4007dc0c
E/LD:   0x40045a84
E/LD:   0x40088ec8
E/LD:   0x4007ddbc
[  694.079466] tpm tpm0: ftpm_tee_tpm_op_send: SUBMIT_COMMAND invoke error: 0xffff3024
[  694.087146] tpm tpm0: tpm_try_transmit: send(): error -53212
[  694.093553] sd 0:0:0:0: [sda] Synchronizing SCSI cache
[  694.118914] reboot: Restarting system

I don't get it yet, tee-supplicant is shut down and should have disconnected the fTPM driver, but there is still some command issued that fails in the absence of the supplicant daemon.

Shutdown issue resolved via optee-client patch in isar, see https://groups.google.com/g/isar-users/c/T5FuJo9A2hc.

@diogo-ivo DCO check failed, see the details:
Commit sha: 53eb5db, Author: Diogo Ivo, Committer: Diogo Ivo; Expected "Diogo Ivo<EMAIL_ADDRESS>but got "Diogo Ivo<EMAIL_ADDRESS>

@diogo-ivo DCO check failed, see the details:
Commit sha: 53eb5db, Author: Diogo Ivo, Committer: Diogo Ivo; Expected "Diogo Ivo<EMAIL_ADDRESS>but got "Diogo Ivo<EMAIL_ADDRESS>
I guess you might have to update your email in your account settings?


(next) checkbox label-input order causes toggle not to work

If the order of the input and label inside a checkbox is input first, label second (just like in the docs) everything works as expected. However, if the order is switched, the checkbox appearance doesn't change when toggling the control. In 1.x it was working both ways, so if this behavior is intended, then people (like me) might find their websites breaking.
See [fiddle] (http://jsfiddle.net/trigoesrodrigo/fbwr7kj4/1/)

This was a straight up bug. create labels was failing to pick up elements initialized this way.


Start of Beta5.1 development.

Description

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] I have read the contribution guidelines.
[x] The pull request does not introduce breaking changes.

General Guidelines

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

SDK Generation Guidelines

[ ] If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
[ ] The generate.cmd file for the SDK has been updated with the version of AutoRest, as well as a link to the swagger spec, used to generate the code.
[ ] The project.json and AssemblyInfo.cs files have been updated with the new version of the SDK.

Sign Job: http://azuresdkci.cloudapp.net/view/2-.NET SDK/job/sdk-for-net-sign/1401/


Flaky TestMultipleClientsWithSameLabelWithClusterLabelVerification

The test failed the first run, but succeeded the second. Revision is https://github.com/grafana/dskit/commit/126e8a816b1701654dd6ef3977f513fc67ac5c80
--- FAIL: TestMultipleClientsWithSameLabelWithClusterLabelVerification (24.37s)
    memberlist_client_test.go:672: Waiting before start
    memberlist_client_test.go:680: Observing ring ...
    memberlist_client_test.go:694: Update 68.522298ms : Ring has 2 members, and 256 tokens, oldest timestamp: 825.359625ms avg timestamp: 825.359625ms youngest timestamp: 825.359625ms
    memberlist_client_test.go:694: Update 153.817701ms : Ring has 3 members, and 384 tokens, oldest timestamp: 910.655027ms avg timestamp: 910.655027ms youngest timestamp: 910.655027ms
    memberlist_client_test.go:694: Update 459.555858ms : Ring has 4 members, and 512 tokens, oldest timestamp: 1.216393192s avg timestamp: 1.216393192s youngest timestamp: 1.216393192s
    memberlist_client_test.go:694: Update 460.579224ms : Ring has 5 members, and 640 tokens, oldest timestamp: 1.217416552s avg timestamp: 1.217416552s youngest timestamp: 1.217416552s
    memberlist_client_test.go:694: Update 461.713676ms : Ring has 6 members, and 768 tokens, oldest timestamp: 1.218551007s avg timestamp: 1.218551007s youngest timestamp: 1.218551007s
    memberlist_client_test.go:694: Update 463.242127ms : Ring has 7 members, and 896 tokens, oldest timestamp: 1.220079455s avg timestamp: 1.220079455s youngest timestamp: 1.220079455s
    memberlist_client_test.go:694: Update 503.895559ms : Ring has 10 members, and 1280 tokens, oldest timestamp: 2.260732883s avg timestamp: 2.260732883s youngest timestamp: 1.260732883s
    memberlist_client_test.go:694: Update 558.913038ms : Ring has 10 members, and 1280 tokens, oldest timestamp: 2.315750343s avg timestamp: 2.315750343s youngest timestamp: 1.315750343s
    memberlist_client_test.go:694: Update 1.002496549s : Ring has 10 members, and 1280 tokens, oldest timestamp: 2.759333874s avg timestamp: 1.759333874s youngest timestamp: 759.333874ms
    memberlist_client_test.go:694: Update 1.084007026s : Ring has 10 members, and 1280 tokens, oldest timestamp: 2.840844352s avg timestamp: 1.840844352s youngest timestamp: 840.844352ms
    memberlist_client_test.go:694: Update 1.13045055s : Ring has 10 members, and 1280 tokens, oldest timestamp: 2.887287876s avg timestamp: 1.887287876s youngest timestamp: 887.287876ms
    memberlist_client_test.go:694: Update 1.136422462s : Ring has 10 members, and 1280 tokens, oldest timestamp: 2.89325979s avg timestamp: 1.89325979s youngest timestamp: 893.25979ms
    memberlist_client_test.go:694: Update 1.258210064s : Ring has 10 members, and 1280 tokens, oldest timestamp: 3.015047392s avg timestamp: 2.015047392s youngest timestamp: 1.015047392s
    memberlist_client_test.go:694: Update 1.278908586s : Ring has 10 members, and 1280 tokens, oldest timestamp: 3.035745902s avg timestamp: 2.035745902s youngest timestamp: 1.035745902s
    memberlist_client_test.go:702: Ring updates observed: 14
    memberlist_client_test.go:723: KV 0: number of known members: 10
    memberlist_client_test.go:620:
        	Error Trace:	memberlist_client_test.go:620
        	Error:      	Received unexpected error:
        	            	Member 0: invalid state of member Member-9 in the ring: 2
        	Test:       	TestMultipleClientsWithSameLabelWithClusterLabelVerification

Not flaky anymore. Haven't managed to trigger failures for those in hundreds of runs. See: https://github.com/grafana/dskit/pull/600. We can reopen if they show up again


Custom effects

Just look into what it would take to add such functionality

I want to use logux with redux-saga,
but it needed an opportunity to use custom dispatch function:
Instead of Redux, in Logux Redux you have 4 ways to dispatch action:

store.dispatch(action) is legacy API. Try to avoid it since you can’t
specify how clean this actions.
store.dispatch.local(action, meta) — action will be visible only to current
browser tab.
store.dispatch.crossTab(action, meta) — action will be visible
to all browser tab.
store.dispatch.sync(action, meta) — action will be visible to server
and all browser tabs.

I think it could be solved by implementing custom effect like PUT_LOCAL. Are there any updates about this issue?

@borovik96 I suspect you can implement it right now  with emitter option https://github.com/redux-saga/redux-saga/blob/master/packages/core/test/middleware.js#L105-L113

@borovik96 your use case seems simple enough that using lower-level runSaga API should get you everything you need. You'd have to provide a specialized dispatch to runSaga that would interpret input action based on some prop and would use appropriate logux dispatch flavour accordingly, so you could just use smth like
yield put({ type: 'SOME_TYPE', meta: { crossTab: true } })

where ofc you could hide this behind a helper function
const putCrossTab = ac => put({ ...ac, meta: { ...(ac.meta || {}), crossTab: true } })

I need an effect to handle call and logging for each API call for me, for now I have a universalCall function which call fetch in nodejs and browser for me. It is async function and I have await in it. I want to call yield put(logActions.writeLog(logInfo));  before and after API call.
Do you think it is good to have callAndLog custom effect and do logging inside it?

@sayjeyhi Possibly I will do logging with saga monitor like this,
class MyMonitor implements SagaMonitor {
  private _trackIds = {};
  private _filter: string[] = [];

  setCallApiFunctionNames(list: string[]) {
    this._filter = [...list];
  }

  effectResolved(effectId: number, result: any): void {
    if (this._trackIds[effectId]) {
      this._trackIds[effectId] = Object.assign({}, this._trackIds[effectId], {
        end: new Date().getTime()
      });

      // console.log(this._trackIds);
    }
  }

  effectTriggered(options: {
    effectId: number;
    parentEffectId: number;
    label?: string;
    effect: any;
  }): void {
      if (options.effect.type === "CALL"
          && options.effect.payload.fn.name
          && this._filter.some(fn => fn === options.effect.payload.fn.name)) {
        this._trackIds[options.effectId] = {
          start: new Date().getTime(),
          functionName: options.effect.payload.fn.name
        };
      }
  }
}

const monitor = new MyMonitor();
monitor.setCallApiFunctionNames(["getCosts"]);

const sagaMiddleware = createSagaMiddleware({
  sagaMonitor: monitor
});

I had a chat with andarist and we decided to not implement this feature.  Instead, if we want to add more effects, we could create a superset library that provides more effects.


Solved responsiveness issues in the docs section

Signed-off-by: AnshumanDhiman<EMAIL_ADDRESS>

@AnshumanDhiman Can you kindly check with the failed Test Sass workflow, I think there are a few indentation and spacing problems. You can run npm run test:sass in order to run this test locally.


[MRG] Add voice recognizer, voice generator doc

The details of the pull request:
Add doc for voice recognizer and voice generator

Coverage decreased (-0.1%) to 93.62% when pulling 7821e88e1a38789c6e1df17e9e790e397eef390f on voice_doc into 76b07e64cf693cff7531f0bd7b5bca4fc9faf18c on master.


Is it possible to setting automation of email verification when amplify auth add ?

Simply, usage or feature request.
I want to automate email verification.
It is easy to realize using lambda pre-signup event trigger.
In aws-amplify/cli, is it possible ?
var AWS = require('aws-sdk');
exports.handler = (event, context, callback) => {
    var confirm = {
        "autoConfirmUser": true,
        "autoVerifyEmail": true
    };
    event.response = confirm;
    context.done(null, event);
};

@yukitaka13-1110 We're actively working on automating this for you using lambda triggers. You would see an update regarding this soon. I'll update this thread once we have the release out.

@kaustavghosh06 Thank you for quick response !
That's great !! I'm looking forward to it.

i guess its in the new rls 1.8.0 now.

@davidbiller Oh, really? I tried to find the change about cloudformation but couldn't find it.
Now I found out the target commits c6fc838
Thank you for quick announcement !

We've added support for Lambda Trigger support for the dynamo tables managed/added by the CLI using the amplify add storage command. Let me know if you find that useful. Please take a look at https://aws.amazon.com/blogs/mobile/amplify-framework-adds-supports-for-aws-lambda-triggers-in-auth-and-storage-categories/ for more details.


updates dependencies, allows emojis

I set the option for allowSurrogateChars: true in the config objects to allow emojis and other multi byte characters to be written to sheets.
Thanks so much for a cool module!

Reference to the xmlbuilder-js repo: https://github.com/oozcitak/xmlbuilder-js/issues/98


Healthcheck controller

If possible, we should also check the health of the resources that are deployed through the managed resource. This can be done by implementing a healthcheck controller like for example: https://github.com/gardener/gardener-extension-shoot-lakom-service/blob/main/pkg/controller/healthcheck/add.go

Then we should also check if the accounting-exporter has issues sending reports.


Fix bug in bfloat16 optimizer related to checkpointing

Overview
Fixed conditional branching where errors occurred.
-        elif self.bfloat16_enabled() and not self.zero_optimization():
+        elif self.bfloat16_enabled() and hasattr(self.optimizer, "bf16_groups"):

Error Detail
I encountered the following error while training with zero 1 optimization and bf16 using Megatron-DeepSpeed.
.env/lib/python3.11/site-packages/deepspeed/runtime/engine.py", line 3288, in _get_zero_param_shapes
    ) == 2 else self.optimizer.fp16_groups
                ^^^^^^^^^^^^^^^^^^^^^^^^^^
                ^^^^^^^^^^^^^^^^^^^^^^^^^^
    ) == 2 else self.optimizer.fp16_groups
    ) == 2 else self.optimizer.fp16_groups
AttributeError: 'BF16_Optimizer' object has no attribute 'fp16_groups'
    ) == 2 else self.optimizer.fp16_groups
AttributeError: 'BF16_Optimizer' object has no attribute 'fp16_groups'. Did you mean: 'bf16_groups'?
                ^^^^^^^^^^^^^^^^^^^^^^^^^^
                ^^^^^^^^^^^^^^^^^^^^^^^^^^
                ^^^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: 'BF16_Optimizer' object has no attr

and also when loading the checkpoint, I got another error.
.env/lib/python3.11/site-packages/deepspeed/runtime/engine.py", line 2873, in _load_zero_checkpoint
    self.optimizer.load_state_dict(state_dict_list=zero_sd_list,
TypeError: BF16_Optimizer.load_state_dict() got an unexpected keyword argument 'load_serial'

Related Pull Request
https://github.com/microsoft/DeepSpeed/pull/3759
Related Issue
https://github.com/microsoft/DeepSpeed/issues/4272

I have confirmed that checkpoints can be saved & loaded with bf16 in Megatron-DeepSpeed!
(A100 40GB x 16(2node))

@okoge-kaz, thanks for sharing your results.

Fix #3759

@tjruwase All test passed!
May I ask for a merge?


something error with README

in 3. Inference topic proportions for documents, i.e., P(z|d)
$ ./src/btm inf   <docs_pt> <model_dir>
The <docs_pt> is confused, it is the <dwid_pt> right?

Yes:)


Bump Java and Gradle version in Android sample

Description

Bump Java and Gradle version in Android sample

General Checklist

[ ] Added new tests to cover change, if needed
[ ] Security oriented best practices and standards are followed (e.g. using input sanitization, principle of least privilege, etc)
[ ] Documentation update for the change if required
[ ] PR title conforms to conventional commit style
[ ] If breaking change, documentation/changelog update with migration instructions

By submitting this pull request, I confirm that my contribution is made under the terms of the MIT-0 license.

For build-android actions, Please fix the build error:

error: invalid source release: 21

For the build-ios actions, Please update the runs-on parameter in
https://github.com/aws-samples/swift-chat/actions/runs/12339018149/workflow#L11,
https://github.com/aws-samples/clickstream-sdk-samples/blob/main/.github/workflows/build-rn.yml#L37
to  macos-latest and try again.


coqPackages.reglang: init at 1.1.2

Motivation for this change
Add https://github.com/coq-community/reglang
Things done

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

[x] NixOS
[x] macOS
[ ] other Linux distributions

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

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

[x] Fits CONTRIBUTING.md.

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

1 package built:

coqPackages.reglang


Business name shows on expanding incontext window

On expanding incontext window, business name for the seller shows up:

Default behaviour is that, if logo is not setup during checkout then business name will show up. checkout.js does not give an option to pass a logo which means seller can not setup a logo and every time consumer expands the incontext window, we will have business name on checkout page.
This is reproducible on https://developer.paypal.com/demo/checkout/#/pattern/client
Thanks,
Shoaib

checkout.js does not give an option to pass a logo

It's still totally possible to pass a merchant logo in the api call -- checkoutjs doesn't prohibit this. The logo is just not shown for the default window size.


Plugin conflicting with another push enabled plugin

Build is failing with following error:
UNEXPECTED TOP-LEVEL EXCEPTION:
com.android.dex.DexException: Multiple dex files define Lcom/google/android/gms/actions/ItemListIntents;

        at com.android.dx.merge.DexMerger.readSortableTypes(DexMerger.java:579)
        at com.android.dx.merge.DexMerger.getSortedTypes(DexMerger.java:535)
        at com.android.dx.merge.DexMerger.mergeClassDefs(DexMerger.java:517)
        at com.android.dx.merge.DexMerger.mergeDexes(DexMerger.java:164)
        at com.android.dx.merge.DexMerger.merge(DexMerger.java:188)
        at com.android.dx.command.dexer.Main.mergeLibraryDexBuffers(Main.java:504)
        at com.android.dx.command.dexer.Main.runMonoDex(Main.java:334)
        at com.android.dx.command.dexer.Main.run(Main.java:277)
        at com.android.dx.command.dexer.Main.main(Main.java:245)
        at com.android.dx.command.Main.main(Main.java:106)

 FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':transformClassesWithDexForDebug'.

>
com.android.build.api.transform.TransformException: com.android.ide.common.process.ProcessException: org.gradle.process.internal.ExecException: Process 'command '/home/karan/jdk1.8.0_91/bin/java'' finished with non-zero exit value 2

Please suggest me some work around for this situation.
Thanks

Any clue?

Not really sure how to address this.
We will be updating for FCM shortly, hopefully it will solve the issue by itself.

Please try the latest version of the plugin and open this again if the problem persists.


docs: include YouTube walkthrough in the README.md

Hey Michael,
You made such a good YouTube video setting up the starter kit. It makes sense to mention it in the README. What do you think?
Btw, I left a comment on the YouTube video.

thanks for adding this


fix typo

@FGRibreau Thank you for your PR, and nice to meet you!

Nice work! I think I'm soon going to benchmark this lib for writing/reading data into Redis :)


Set the 'lowest-speed-limit' parameter will stop the low download speed task and the webui-aria2 can not restart it all.

I need Aria2 to automatically stop the low download speed task, and then restart it to get the highest download speed (which can usually be achieved).
But now I need to handle these tasks manually.
It would be nice if webui could achieve this function.

I use the following script solved
function loop (){
  const a = document.querySelectorAll('[ng-click="restart(download)"]')
  a.forEach(($e,i)=>$e.click())
  console.log('Restart',a.length,'tasks')
}
setInterval(loop,5000)