Document Links

When providing a link in a step (like Go to Azure Portal), that link should open in a new tab or window so the instructions don't go away!

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

ID: 479bff62-927a-9f71-c85f-4a1188c72bb2
Version Independent ID: df934d01-7a9d-f7f9-d9ea-320454e32d77
Content: Quickstart: Add sign-in with Microsoft to an ASP.NET Core web app - Microsoft identity platform
Content Source: articles/active-directory/develop/quickstart-v2-aspnet-core-webapp.md
Service: active-directory
Sub-service: develop
GitHub Login: @jmprieur
Microsoft Alias: jmprieur

@bjstigall, Thanks for the feedback! We are taking a look into this and will get back to you soon.

@mmacy do you know how we do that in docs.ms?

@jmprieur Yes, though we need to apply it very judiciously. A link to the Azure portal in the first step of a series of steps is one of the very few situations that qualify for this, so @bjstigall's suggestion here is something we can consider adopting here and in docs with similar procedures.
@msewaweru is currently doing a consistency pass on all articles that contain app registration steps, so we'll either roll this into that work or record it as a new work item. For now, I'll #reassign:mmacy for tracking and well cover this in our next sync.
Thanks for the heads-up, @jmprieur, and for the request to look into it, @bjstigall!

@MarileeTurscak-MSFT @SaurabhSharma-MSFT Can we please get this recategorized as a doc-experience/doc suggestion/doc enhancement? It's not a doc bug. Thanks!
CC @mmacy

@MarileeTurscak-MSFT @AjayKumar-MSFT Pinging on this - as Celeste mentions, this is not a doc bug, but an enhancement. Can one of you relabel it as doc-experience/doc suggestion/doc enhancement, please? Thanks.
@msewaweru This would be a good one for you to tackle since you've made the most recent pass on standardizing all the app registration steps and are familiar with each instance of "Sign in to the Azure portal." I'll submit a work item. #reassign:msewaweru
Cc: @CelesteDG

@MarileeTurscak-MSFT @AjayKumar-MSFT Pinging on this - as Celeste mentions, this is not a doc bug, but an enhancement. Can one of you relabel it as doc-experience/doc suggestion/doc enhancement, please? Thanks.
@msewaweru This would be a good one for you to tackle since you've made the most recent pass on standardizing all the app registration steps and are familiar with each instance of "Sign in to the Azure portal." I'll submit a work item. #reassign:msewaweru
Cc: @CelesteDG

Well noted @mmacy. Taking up this task.

Well noted @mmacy. Taking up this task.

The following PRs updated all manual registration steps "Sign in the to the Azure portal" with force-new-open-tab behavior:

https://github.com/MicrosoftDocs/azure-docs-pr/pull/142483
https://github.com/MicrosoftDocs/azure-docs-pr/pull/142487
https://github.com/MicrosoftDocs/azure-docs-pr/pull/142488

@msewaweru As a follow-up, can you add a work item to also update the links in the Option 1 sections with this change? Example here:
https://docs.microsoft.com/azure/active-directory/develop/quickstart-v2-aspnet-core-webapp#option-1-register-and-auto-configure-your-app-and-then-download-your-code-sample
Thanks again for the suggestion, @bjstigall! Closing this one out for now; we'll follow up on the last few stragglers as time allows. #please-close

The following PRs updated all manual registration steps "Sign in the to the Azure portal" with force-new-open-tab behavior:

https://github.com/MicrosoftDocs/azure-docs-pr/pull/142483
https://github.com/MicrosoftDocs/azure-docs-pr/pull/142487
https://github.com/MicrosoftDocs/azure-docs-pr/pull/142488

@msewaweru As a follow-up, can you add a work item to also update the links in the Option 1 sections with this change? Example here:
https://docs.microsoft.com/azure/active-directory/develop/quickstart-v2-aspnet-core-webapp#option-1-register-and-auto-configure-your-app-and-then-download-your-code-sample
Thanks again for the suggestion, @bjstigall! Closing this one out for now; we'll follow up on the last few stragglers as time allows. #please-close


Update Readme with Setup instructions

Why do we need this ?

Acceptance Criteria

[ ] Make Readme More attractive.
[ ] Add Setup instructions

Further Comments / References

I would like to work on this.

I have made changes @samad-yar-khan. Patiently waiting for your feedback on changes.


feat: use THROTTLE to output log

Description
When a lane has stopline but a path does not intersect with a stopline, the following log continues to output.
This PR suppreses its frequency.
[component_container_mt-45] [WARN<PHONE_NUMBER>.549768965] [planning.scenario_planning.lane_driving.behavior_planning.behavior_velocity_planner.stop_line_module] modifyPathVelocity(): is no collision

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@yukkysaito
this may happen when avoidance or lane change is enabled for stop line lane but it's obvious error for proper behavior what do you think about this

@taikitanaka3 Could you tell me more details of your question?

It is difficult it should be a warning or an error.
I think it is ok as the warning because it doesn't mean the system is down.

@h-ohta

Let's think about no collision with stop line case, I think it's worth sending warning log every time.

@taikitanaka3 Thank you for your information. I understand.
In such a situation, no collision is not failure...
How about changing to RCLCPP_DEBUG?
Is RCLCPP_WARN better?


Make translatable string.xml

Hi, I would like send a pull request for translate new Fontster app in French, but it seems that's not possible right now.

I need to go through the app and move all my hardcoded strings into the external file at some point. Feel free to do this if you'd like.

String have been extracted. Please feel free to translate, thanks!


user-jwts does not support different Name and NameIdentifier claim

First I opened this bug in the asp.net core and Safia Abdalla debugged the issue that I mentioned and she suggested that the bug is related to this Repo
here is a copy paste of the bug from ASP.NET Core project
Describe the bug
basically what I want is to set name and nameIdentifier to different values, this is what I am trying to accomplish.
I am using user-jwts and here is my command
dotnet user-jwts create --claim<EMAIL_ADDRESS>--claim "http://schemas.xmlsoap.org/ws/2005/05/identity/claims/name=hakan"
I am not providing the --name, I know, but I tried that, and did not achieve what I wanted either.
what I am getting basically from the above command is duplicated claims, I am getting the name claim twice, and the name identifier twice,
The first pair of name and nameidentifier claims are set to the default windows user (this is expected by the documentation as I am not specifying anything to the --name option)
the second pair of name and nameidentifier claims are set to the second claim I specify in the above command hakan.
What I am trying to do is just set those 2 claims to different values (without duplicating them).
Is this a bug? or is this by design?
I tried to work around this by using IClaimsTransformation interface, but it only allow to add new claims not to modify or deleted the duplicated claims.
if this is confirmed a bug, maybe I can help with it
Thank you
Expected Behavior
name and nameidentifier claims should be allowed to set to different values
.NET Version
7.0.401

here is the replay of Safia Abdalla

I did a bit of debugging into this and I believe the issue might be in the Microsoft.IdentityModel.Tokens APIs. The JWT that is produced has the correct properties when I analyze it with a different parser but duplicate claims are produced when the we decode the JWT bearer token.
Try filing an issue over at AzureAD/azure-activedirectory-identitymodel-extensions-for-dotnet to see if they might be able to provide a clue there.

Sorry but I am not sure if I understood this correctly, please correct me if you feel I am going in the wrong direction.
this library set the NameIdentifier claim to whatever has been set to the NameId claims correct?
and How this is related to my duplicated claims? can you elaborate a little bit please
Thank you

@HakamFostok it seems to me you are on the right track, it appears that the code is additive.
Not sure if there is an api to control this, but I don't believe the issue is in this repo.
I was calling out the 'long names' for the claims which are being remapped.
"http://schemas.xmlsoap.org/ws/2005/05/identity/claims/nameidentifier" => "nameid" "http://schemas.xmlsoap.org/ws/2005/05/identity/claims/name" => "unique_name"

Not sure if there is an api to control this, but I don't believe the issue is in this repo.

Resurrecting a bit of an old thread here but I'm finally looking through old issues. Is there a reason we suspect the issue is not in this repo? Do you have an intuition on where the issue might be?
What dictates that the mapping is additive instead of using override behavior?


Added Visual Studio Code Pony language extension to editor support list

Please let me know if this is the right branch (didn't find a develop?) and whether you prefer the marketplace site or GitHub repository link!

I'm not really familiar with Visual Studio - what is the difference between the "Visual Studio" and "Visual Studio Code" items?

Visual Studio is the full blown Windows IDE with solutions, compiler and debugger integration, profiler and targets large-scale enterprise solutions (similar to Eclipse).
Visual Studio Code is a small, fast, free editor for all platforms, with very tidy interface and the same syntax highlighter as Visual Studio (similar to Atom, Sublime).
http://visualstudio.com/

@SeanTAllen asked me on Twitter to issue a pull request :)

@npruehs - thanks for the explanation!  Just checking to make sure it wasn't redundant.

Sure, that's always a good thing! :)


Changes make the code work in 2023

Hello everyone,
First of all, this is not exactly an issue, but some changes i made to the code so the project work (i tested on kaggle). I haven't test on my computer yet (it's old and can't run those code) (so yet can't make pull requests). I used this in code used in researching so I can't share publicly, sorry.
Code:
import glob
filenames = glob.glob('/kaggle/working/ssd_keras/keras_layers/*.py')
for filename in filenames:
    with open(filename, 'r') as file :
        text = file.read()
    text = text.replace('keras.engine.topology', 'tensorflow.keras.layers')
    text = text.replace('K.image_dim_ordering()', 'K.image_data_format()')
    text = text.replace('._keras_shape', '.shape')
    text = text.replace('.trainable_weights', '._trainable_weights')
    text = text.replace('tf', 'channels_last')
    with open(filename, 'w') as file:
        file.write(text)
with open('/kaggle/working/ssd_keras/keras_loss_function/keras_ssd_loss.py', 'r') as file :
    text = file.read()
text = text.replace('from __future__ import division', '')
with open('/kaggle/working/ssd_keras/keras_loss_function/keras_ssd_loss.py', 'w') as file:
    file.write(text)
with open('/kaggle/working/ssd_keras/keras_loss_function/keras_ssd_loss.py', 'r+') as f:
        content = f.read()
        f.seek(0, 0)
        f.write('from __future__ import division\nimport tensorflow as tf\ntf.to_float = lambda x: tf.cast(x, tf.float32)'.rstrip('\r\n') + '\n' + content)
with open('/kaggle/working/ssd_keras/data_generator/object_detection_2d_data_generator.py', 'r') as file :
    text = file.read()
text = text.replace('yield ret', 'yield tuple(ret)')
with open('/kaggle/working/ssd_keras/data_generator/object_detection_2d_data_generator.py', 'w') as file:
    file.write(text)

with open('/kaggle/working/ssd_keras/eval_utils/average_precision_evaluator.py', 'r') as file :
    text = file.read()
text = text.replace('if len(predictions) == 0:',
                    'if len(predictions) == 0:\n                cumulative_true_positives.append(1)\n                cumulative_false_positives.append(1)')
with open('/kaggle/working/ssd_keras/eval_utils/average_precision_evaluator.py', 'w') as file:
    file.write(text)

with open('/kaggle/working/ssd_keras/keras_loss_function/keras_ssd_loss.py', 'r') as file :
    text = file.read()
text = text.replace('tf.log', 'tf.math.log')
text = text.replace('self.neg_pos_ratio = tf.constant(self.neg_pos_ratio)', '')
text = text.replace('self.n_neg_min = tf.constant(self.n_neg_min)', '')
text = text.replace('self.alpha = tf.constant(self.alpha)', '')
text = text.replace('tf.count_nonzero', 'tf.math.count_nonzero')
text = text.replace('tf.to_int32(n_positive)', 'tf.cast(n_positive, tf.int32)')
with open('/kaggle/working/ssd_keras/keras_loss_function/keras_ssd_loss.py', 'w') as file:
    file.write(text)

import os
os.environ['PYTHONPATH'] += ':/kaggle/working/ssd_keras/:/kaggle/working/ssd_keras/keras_layers/'

This is great. I am trying to get the code to run too.
I will try your changes.

For information sharing, i am writing this.

ssd7 case, csv file coordinate must be not corner form but minmax form. Almost all of the problems were due to this issue. In def parse_csv, "The expected strings are 'image_name', 'xmin', 'xmax', 'ymin', 'ymax', 'class_id'."

ssd300 case, In evaluation mode, mode must be 'training' not 'inference'

Then, it will work in tensorflow 2.12.

For information sharing, i am writing this.
1. ssd7 case, csv file coordinate must be not corner form but minmax form. Almost all of the problems were due to this issue. In def parse_csv, "The expected strings are 'image_name', 'xmin', 'xmax', 'ymin', 'ymax', 'class_id'."

2. ssd300 case, In evaluation mode, mode must be 'training' not 'inference'

Then, it will work in tensorflow 2.12.

I am doing another push with this code and I think I am hiting the problem you mention because when I start the training I get the error "SSDInputEncoder detected degenerate ground truth bounding boxes". I think the code is not using the correct format for the coordinates of the boxes.
I know I can supply the argument coords='minmax' to build_model and also to SSDInputEncoder; but I am still getting the same error about degenerate boxes. Can you tell me if I need to change the argument 'coords' anywhere else?


fix bug in transforming empty strings in scan key

This PR mainly fixes the bug in issue #143.
As mentioned in that post, Postgres uses generic plan after several rounds of execution. In a generic plan, a parameter will be filled with a Param node rather than a Const node, and will be replaced by runtime_key in executor. However, Peloton does not handle some data types correctly (like strings and lists) when transforming the Param node.
This PR first fixes the bug in handling text, varchar, bpchar. What we do here is to simply transform it to an empty Value, since it will be replaced by runtime key anyway.
Things are more complicated for arrays. An ArrayExpr is passed as runtime_key, and Peloton does not support transforming ArrayExpr now. Since this is sort of independent issue, I will send a separate PR when I finish it. I added a TransformArrayExpr to transform ArrayExpr to VectorExpression in our VoltDB-like expression system.

UPDATE: bugs in transforming arrays have been fixed.


Max users per room. Limit increase.

The official documentation (https://docs.livekit.io/home/get-started/intro-to-livekit/) says that the self-hosted version has a limit of 3000 users in a room. What is the reason for this limitation? Is there any way to bypass it?

there are no artificial limits and the 3k number may be out of date - there have been a lot of improvements to performance since this benchmark was released. the only way to accurately estimate the single node capacity for your use case is to benchmark your workload on your hardware.
if you need more capacity than a single instance can provide you could load balance your room across multiple servers or check out livekit cloud - https://cloud.livekit.io/ - which does this for you


Use the default pose instead of frame 0 for base pose

Hi,
Is it possible to use the default pose rather than frame 0?
It causes problems when the artist use frame 0 as an animation frame.

No this feature is not available because in a great many cases Maya fails to restore the binding pose because of constraints, expressions, scripts etc...
But you can pick any frame for the default pose, e.g. -100 or so. You pass this with the -ivt parameter

Although the documentation states that this should be possible by disabling all modifiers, this didn't always work for us..
https://knowledge.autodesk.com/support/maya/learn-explore/caas/CloudHelp/cloudhelp/2015/ENU/Maya/files/GUID-D47C31C7-428F-4A54-B104-49C51F3856E5-htm.html
The code for going to the bind pose and using that as the default one could be added again, it might work in most cases.

@Ziriax
Hi, Ziriax, below are just my two cents for what it's worth, please correct me if any information is wrong:
In situations like exporting I don't think we need or want to restore the bind pose in the scene, as that might mess up the animation. What can be done though is to fetch the bind pose transform data and use those as the default pose for the skeleton in the exported data. That way we can get accurate skeleton pose data from the output.
There's just too many situations where having to manually create bind pose in the animation timeline is inconvenient especially if the animator for one reason or another forgot to do it before hand and Maya fails to restore bind pose.

Very true.
Do you know a reliable way of getting to the bind pose transform data?
Please note that Maya2glTF is a world-space-sampler. After 20 years of Maya and 3D hacking, we found this to be the only reliable way to export scenes, because complex rigs always have a lot of expressions, constraints, nodes, etc... So if the bind pose transform data can be extracted, we need it in world space, not local space (most likely it is)

@Ziriax I'm not familiar with the c++ package for Maya, in python we can fetch the joint(bone) bind pose with cmds.getAttr(bone + ".bindPose") to get the bind pose matrix. I believe the matrix is in world space, but I could be wrong. Those same data should be fetch able in c++ as well.
The follow attributes are available in python/mel as well for bind pose:
bindRotation
bindRotationX
bindRotationY
bindRotationZ
bindJointOrient
bindJointOrientX
bindJointOrientY
bindJointOrientZ
bindRotateAxis
bindRotateAxisX
bindRotateAxisY
bindRotateAxisZ
bindScale
bindScaleX
bindScaleY
bindScaleZ
bindInverseScale
bindInverseScaleX
bindInverseScaleY
bindInverseScaleZ
bindSegmentScaleCompensate

I will investigate this attribute, thanks for the info!
The other problem is what default weights to pick for the blend shapes. I guess all zero should work, if this is not desired, then one should apply an animation that contains the desired weights
The same question must be answered for the transforms of all other non-joint nodes, and cameras what are the defaults?
To solve all these problems we just ask the artist to create a single frame that provides the "pose"

Hm, that's a very good question lol. I was thinking only in terms of character animation, but I'll take a crack at this.
Let's see, the objects themselves should have a starting point for the animation, since the animation data are technically just offsets applied onto their starting transform. So if we can somehow strip away animation data or access the default transforms pre-animation, we should be able to get accurate starting transform (and attributes) for those nodes. I'll poke around in Maya and see if I can find anything for this.

On the other hand, the two concepts (binding pose matrices and initial non-animated scene values) are orthogonal.
So it should be possible to use your bindPose attribute to extract the inverse bind poses, and have another discussion about what values to pick for the "default" scene :-)


[BUG]: Adjust Background Image to Fill Entire Screen on Landing Page

Currently, the landing page of the foodie website displays a background image that does not cover the entire screen, resulting in a visual inconsistency. This issue aims to improve the aesthetics and user experience by adjusting the background image to fill the complete screen. By ensuring the background image covers the entire viewport, we aim to create a more visually appealing and cohesive landing page.

Kindly assign this task to me, and I'll proceed with implementing the necessary changes. Thank you for the opportunity to contribute!

Kindly assign this task to me, and I'll proceed with implementing the necessary changes. Thank you for the opportunity to contribute!

i can add a background image and we can do one thing also , to add image inside a  text that will give an attractive look.
kindly assign this task to me.

@anurag-jaiswal-aj I have assigned you the issue. Go ahead . All the best.

I wanted to provide an update regarding the issue reported in this thread. The necessary changes have been implemented in pull request #223 to address the issue.
Could you please review and merge at your earliest convenience?


Fix HTTP3 reverse proxy mode for chrome clients

Description
Handles fragmented QUIC client hello
refs #7025
Checklist

[ ] I have updated tests where applicable.
[ ] I have added an entry to the CHANGELOG.

Chrome doesn't seem to trust the certificate that we provide during the QUIC handshake:
Client QUIC handshake failed. 199:TLS handshake failure (ENCRYPTION_HANDSHAKE) 46: certificate unknown
Workaround:
.\chrome.exe --ignore-certificate-errors-spki-list="<cert fingerprint>" --origin-to-force-quic-on=localhost:443
We can generate cert fingerprint using:
openssl x509 -pubkey -noout -in <cert path> | openssl rsa -pubin -outform der | openssl dgst -sha256 -binary | base64

@errorxyz: this is for reverse proxy mode and the cert is installed in the system?

@errorxyz: this is for reverse proxy mode and the cert is installed in the system?

Yes. Searching for the error a bit online, it seems it is an issue with quiche?

Certificate issue fixed in https://github.com/mitmproxy/mitmproxy/pull/7073

@mhils chrome still does not trust the self-signed certificate i.e accessing our http3 reverse proxy on localhost using chrome doesn't work
error: Client QUIC handshake failed. 199:TLS handshake failure (ENCRYPTION_HANDSHAKE) 46: certificate unknown
However, curl works with the mitmproxy cert specified using --cacert
check this out: https://github.com/GoogleChrome/samples/blob/8223e1b7f619bde037cf010dd9a2d899556a2d89/quictransport/quic_transport_server.py#L44-L75
Following these instructions fixes the issue
Using a trusted cert for eg. from letsencrypt on our test server works as intended

We still get an error related to not handling StopSendingReceived event and a stream related error, but I'm confident that these changes are unrelated to those bugs.

Changelog entry and this is good to go.
(Focus on user-facing changes 😉 )


Fix ng-messages for formField.

Coverage remained the same at 52.892% when pulling 143568f29c9b4e9dc3b334f02798b54c106c1ed7 on ynurmahomed:fix-messages into<PHONE_NUMBER>f3ccc4a97848b74e507906da3e6572 on esaude:master.


Process for translating a new article

Copying from https://github.com/github/opensource.guide/pull/985#issuecomment-524490596

[we] might need to add something to tests to allow for not-yet-translated articles. I don't think any articles have been added since the original set so we haven't had the issue yet. Or maybe the right thing is to add stubs for to each translation for a new article asking for translations? Either would seem reasonable to me.

Think I prefer the stubs route: no test change needed, visible to people looking through _articles/{lang} who might be interested in translating.

I think the higher-level goal is "no new articles". I don't think we're at a place to really be able to accept them and give them a decent level of review. I also think the original set of articles stand alone nicely. Perhaps that should be documented explicitly somewhere?

I think the new article being worked on in #985 would be a worthy addition, and we have someone working on it. Happy to be wrong about even that one though. In any case, 👍 to documenting at least an extremely high bar for any new articles.


GetBalancesByAddresses

We need a way to get balances across a set of accounts in a single request.
One possible API but open to all of your suggestions here :)
{ result: balances: [ {  asset: "XCP", total: 10, { [address_1]: 5, [address_2]: 4 } }]
I'm not sure if we need pagination here, practically.  Although technically, it's probably the right thing to do.

done here https://github.com/CounterpartyXCP/counterparty-core/pull/2041


Fix spelling error in README

Changed 'detatched' on line 169 to 'detached'

Thank you Anthony for taking time to put in this pull request, we really appreciate.


SSL - SecurityContext - Unhandled Exception: FileSystemException: Cannot open file - Error: No such file or directory, errno = 2

When I try to set certificate, I get an error:
E/flutter ( 7195): [ERROR:flutter/lib/ui/ui_dart_state.cc(148)] Unhandled Exception: FileSystemException: Cannot open file, path = 'assets/ssl/STAR_octopusspace_com.crt' (OS Error: No such file or directory, errno = 2)
E/flutter ( 7195): #0      _File.throwIfError (dart:io/file_impl.dart:643:7)
E/flutter ( 7195): #1      _File.openSync (dart:io/file_impl.dart:487:5)
E/flutter ( 7195): #2      _File.readAsBytesSync (dart:io/file_impl.dart:547:18)
E/flutter ( 7195): #3      _SecurityContext.useCertificateChain (dart:io/runtime/binsecure_socket_patch.dart:175:40)

... and this is my yaml file:

Jelenas-MacBook-Pro:flutter-comment-wall jelenalecic$ flutter doctor -v
[✓] Flutter (Channel master, v1.2.1-pre.66, on Mac OS X 10.13.6 17G4015, locale en-US)
    • Flutter version 1.2.1-pre.66 at /Users/jelenalecic/projects/flutter
    • Framework revision 496ddc580c (2 days ago), 2019-02-02 16:23:39 -0800
    • Engine revision 15f2b92cce
    • Dart version 2.1.1 (build 2.1.1-dev.3.2 71bee8f05e)

[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at /Users/jelenalecic/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1136-b06)
    • All Android licenses accepted.

[✓] iOS toolchain - develop for iOS devices (Xcode 10.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 10.1, Build version 10B61
    • ios-deploy 1.9.4
    • CocoaPods version 1.5.3

[✓] Android Studio (version 3.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 31.3.1
    • Dart plugin version 181.5656
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1136-b06)

[✓] Connected device (2 available)
    • Android SDK built for x86 • emulator-5554                        • android-x86 • Android 9 (API 28) (emulator)
    • iPhone XR                 • 5B28E6C4-2F19-4FA4-B3E9-AA64830E1B75 • ios         • iOS 12.1 (simulator)

• No issues found!

You probably need to load the file using AssetBundle.

@wouterhardeman hm.. not sure.. I need to provide only path(String) and not a file.

Instead of useCertificateChain and usePrivateKey you can use useCertificateChainBytes and usePrivateKeyBytes with the bytes loaded from AssetBundle. Here's a possible working example:
final List<int> certificateChainBytes = (await AssetBundle.load('your asset url')).buffer.asInt8List();
context.useCertificateChainBytes(certificateChainBytes);

Not sure if this will work, I just typed this in here.

@wouterhardeman
yup, I get to read them like this:
final List<int> certificateChainBytes =
        (await rootBundle.load('assets/ssl/STAR_octopusspace_com.crt')).buffer.asInt8List();
    context.useCertificateChainBytes(certificateChainBytes);
    final List<int> keyBytes =
    (await rootBundle.load('assets/ssl/client_octopusspace.key')).buffer.asInt8List();
    context.usePrivateKeyBytes(keyBytes);

    httpClient = HttpClient(context: context);

but now I get this exception:
 E/flutter (11733): [ERROR:flutter/lib/ui/ui_dart_state.cc(148)] Unhandled Exception: TlsException: Failure in usePrivateKeyBytes (OS Error:  E/flutter (11733): 	KEY_VALUES_MISMATCH(ssl_cert.cc:494), errno = 0) E/flutter (11733): #0      _SecurityContext.usePrivateKeyBytes (dart:io/runtime/binsecure_socket_patch.dart:164:50)
so it must be somethin key-related.

Maybe you forgot to add the password of the key? Or maybe it doesn't match the public key in the certificate? It does seem related to your keys.

@wouterhardeman
this is what worked for me:
`
final SecurityContext context = SecurityContext.defaultContext;
final ByteData crtData = await rootBundle.load('assets/ssl/STAR_octopusspace_com.crt');
context.setTrustedCertificatesBytes(crtData.buffer.asUint8List());

final ByteData authoritiesBytes = await rootBundle.load('assets/ssl/STAR_octopusspace_com.ca-bundle');
context.setClientAuthoritiesBytes(authoritiesBytes.buffer.asUint8List());

final ByteData keyBytes = await rootBundle.load('assets/ssl/client_octopusspace.key');
context.usePrivateKeyBytes(keyBytes.buffer.asUint8List());

httpClient = HttpClient(context: context);

`
thanks a lot for your help :)

@jelenalecic No problem! Could you please close the issue?

@wouterhardeman Hi, I am seeing the same issue, it wasted me a few hours until I saw this page. I think this is a walk-around, not a fix.
I use File(), it doesn't throw error "no such file or directory"
File file = File(path)
then a few lines below, I am using following it throw error "no such file or directory"
context.setClientAuthorities(path)
Why one throw the error but the other doesn't??
At least, we should change the error message to let user knows they need to load the bytes instead.


AT-37, AT-27, AT-26 Playlist page

Please check if the PR fulfills these requirements

[ ] The commit message follows our guidelines. For the guidelines check the README.
[ ] Tests for the changes have been added (for bug fixes / features). As we discussed this is optional but encouraged.

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
This PR introduces creating a playlist with selected artists.

What is the current behavior? (You can also link to an open issue here)
Playlist page shows dummy page.

What is the new behavior (if this is a feature change)?
New playlist page shows an enter artist input form. User can select / deselect searched artists, enter name and description of the playlist. After creating a playlist we show an iframe of the created playlist.

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

Other information:

Ill review this one after we get the navbar one done. That way we wont have to deal with the same changes in this one.

Currently doesn't conflict with master.

I have done what you asked for but it created conflict and i don't know why.

Found another bug. If the user enters the same artist twice, it shows up twice when selecting the artist. Here is an example.


enh: support list styles in node/edge constructors

No test-case :-(

Hi @ankostis, thanks for this PR and sorry for my late reply. Hope you are doing ok.
I like your idea, but I will need some more time to think about it. It won't go in pydot 1.4.2 anyway, because that will just be a bugfix release.
With regard to pydot 2.0.0, I am hesitant if we should implement this already for only this single Graphviz-known DOT attribute (style), while there are so many other attributes for which there are also obvious conversions from certain Python types. If we do style, we should also be prepared to do other obvious ones. Of course, we don't have to do them all in one time, but still it is a goal that we would set then. Also, later changes to DOT attributes should also accept the same input (e.g. set_style(...), set('style', ...)). On the other hand, we should also consider that not every DOT language user may be targeting Graphviz (e.g. use the style attribute for a custom purpose), so we might need an option to prevent these kind of conversions. I don't know if you would be interested in working more on all this? Because I feel I already have more than enough on my plate for pydot 2.0.0. Perhaps we could create a separate feature branch for this and merge it once it has a bit more substance, whether that be pydot 2.0.0 or a later version. Please let me know what you think.
Anyway, I will try to respond in more detail during development of pydot 2.0.0.
In case you got prompted to write this PR by the lack of quoting of comma-separated strings: I certainly intend to get that solved with pydot 2.0.0. I see you already made a comment about it in issue https://github.com/pydot/pydot/issues/187#issuecomment-623127490. Issue https://github.com/pydot/pydot/issues/85 is more specific. The problem is with the regex that checks if the string should get quotes or not.
About the second commit you made (commit c8ff1330): I think that can be applied independent of this PR, right? Perhaps you can open a separate PR for it and that can go in pydot 2.0.0 already. Just two remarks:

The commit only changes Edge.__init__(), but it seems the same change could be made in Node.__init__() and Graph.__init__() as well.
Maybe improve the description a bit, because the current one ("fix: copying **kw now really needed") is a bit cryptic. Perhaps something like "Drop excess copying of fresh keyword dictionaries".

Thanks, take care,
Peter

Hi @ankostis, thanks for this PR and sorry for my late reply. Hope you are doing ok.
I like your idea, but I will need some more time to think about it. It won't go in pydot 1.4.2 anyway, because that will just be a bugfix release.
With regard to pydot 2.0.0, I am hesitant if we should implement this already for only this single Graphviz-known DOT attribute (style), while there are so many other attributes for which there are also obvious conversions from certain Python types. If we do style, we should also be prepared to do other obvious ones. Of course, we don't have to do them all in one time, but still it is a goal that we would set then. Also, later changes to DOT attributes should also accept the same input (e.g. set_style(...), set('style', ...)). On the other hand, we should also consider that not every DOT language user may be targeting Graphviz (e.g. use the style attribute for a custom purpose), so we might need an option to prevent these kind of conversions. I don't know if you would be interested in working more on all this? Because I feel I already have more than enough on my plate for pydot 2.0.0. Perhaps we could create a separate feature branch for this and merge it once it has a bit more substance, whether that be pydot 2.0.0 or a later version. Please let me know what you think.
Anyway, I will try to respond in more detail during development of pydot 2.0.0.
In case you got prompted to write this PR by the lack of quoting of comma-separated strings: I certainly intend to get that solved with pydot 2.0.0. I see you already made a comment about it in issue https://github.com/pydot/pydot/issues/187#issuecomment-623127490. Issue https://github.com/pydot/pydot/issues/85 is more specific. The problem is with the regex that checks if the string should get quotes or not.
About the second commit you made (commit c8ff1330): I think that can be applied independent of this PR, right? Perhaps you can open a separate PR for it and that can go in pydot 2.0.0 already. Just two remarks:

The commit only changes Edge.__init__(), but it seems the same change could be made in Node.__init__() and Graph.__init__() as well.
Maybe improve the description a bit, because the current one ("fix: copying **kw now really needed") is a bit cryptic. Perhaps something like "Drop excess copying of fresh keyword dictionaries".

Thanks, take care,
Peter

Oh, I think this is now fixed — if I'm reading this right, it's the same issue that my #320 addressed.
Ironically enough, it was the numeric value handling that was tripping it up — the regular expressions used by quote_if_necessary, seemingly written for European-style number formatting, were quoting values containing . but not quoting values containing ,. Which bled into its handling of style attribute quoting, and obviously caused problems because attributes are comma-separated in graphviz.
But, like I said, I believe now fixed by #320.


ipsec: fixes for key rotation

Fixups to ensure keys are correctly in place before deleting the old key.

Instantiate all xfrm rules when new key shows up instead of waiting for nodeUpdate events through validator to happen. Its not always the case on larger clusters that the nodeUpdate logic will trigger before the key retention logic triggers to delete old keys. In this case we may be missing xfrm rules for new keys for some nodes.

Ensure rules are in place before key is updated. Current logic updates datapath key before updating the BPF maps creating smallish window where pkts can be marked for the new key but the BPF maps and datapath wont have the mapping to do this yet resulting in dropped pkts.

Both of these intend to address an increased policy block counter in recent versions.

/test

/test

/test

Some tests failed on force push but really all I did was change a comment so seems tests are buggy.

Marking as backport/author to avoid this getting in the backporter bucket, as it's expected to have conflicts. @jschwinger233 will take care of the backports.

I've added a release note.


Vickel: Watch findingschool(?!.com)

Vickel requests the watch of the watch_keyword findingschool(?!\.com). See the MS search here and the Stack Exchange search in text, in URLs, and in code.

findingschool(?!\.com) has been seen in 1 true positive, 0 false positives, and 0 NAAs.


Won't compile with g++

As I don't have clang installed, I tried changing the Makefile to use g++, which worked for the last version I tried. Now I get this error:
commands.c:18:46: Fehler: ungültige Umwandlung von »void*« in »unsigned char*« [-fpermissive]
recv = calloc(wLength, sizeof(unsigned char));
(This translates to "Error: invalid conversion from »void*« to »unsigned char*« [-fpermissive]
recv = calloc(wLength, sizeof(unsigned char));")

Hi Gotos,
the latest code base is written in C, not C++ anymore. Please use gcc instead of g++. Please report back.
Cheers,
Tolga

That did the trick, thanks!


Retrieving the draggable position when bounds is set to a parent.

logging the response from onDrag or stopDrag gives me 4 or 5 different x,y positions and none of them match the position that's in local state. I'm looking everywhere to try and figure out how I can get the exact location. Can anyone enlighten me?

As far as I can tell, the stopDrag seems to return the mouseEvent position and not the position of the draggable object internal state. Is there no way to expose the internal position? It's a card that's redraggable by clicking anywhere on it so recalculating based on the clicked position is fairly impossible.

ok.. it seems I was looking at the mouseEvent data instead of the draggableData, my apologies @mzabriskie
Maybe we could update the readme a little to clarify what exactly is in the callback?


Fix broken paths to images in jupyter notebooks

Some of the markdown in the jupyter notebook tutorials had broken paths to images.

No longer applies


pep517 support

Some pip installs are failing for uavro because of a lack of pep517 support.  Simple solution to support pep517.
Prior case failed on command:
pip wheel --use-pep517 "uavro==0.1.2"
Tested manually via:
python setup.py sdist
pip wheel --use-pep517 "dist/uavro-0.1.2.tar.gz"
Reference: https://github.com/conda-forge/uavro-feedstock/pull/10
@martindurant should I bump version to 0.1.3 in this PR?

should I bump version to 0.1.3 in this PR?

Sure, why not. I can push the tag and wheel when this is in. Is there anything left to do?

Version bumped.

Is there anything left to do?

Nothing else that I know of.

I guess we should copy the CI from fastparquet to here to build all the wheels, I'd rather not make them all by hand. I had forgotten this needs compiling... If I push the sdist, we'll get conda packages automatically.


Gatineau, QC, CA

https://github.com/openaddresses/openaddresses/blob/master/sources/ca/qc/gatineau.json
Way back I notified you that the click through didnt work for gatineau, because you needed to accept the license terms. I've been processing Gatineau address data in the CSV format to add to osm and upload that month's file to dropbox. Would it be useful for me to give you a dropbox link to the latest file that I could update when I update the file in osm:
https://www.dropbox.com/s/oddopnkftwsytju/ADRESSE_LATEST.7z?dl=1

Nice! I see what you mean about the license click-thru (and it's just CC-BY, sigh).
We generally do S3 uploads so I've done that step and it's here:

https://s3.amazonaws.com/data.openaddresses.io/cache/uploads/nvkelso/6e49cb/Gatineau_QC_CA_ADRESSE.zip

Looks like this data source has not been added to the map yet (though it was indexed):

Yeah, the license page has to receive a "accept license" cookie before download proceeds, which is annoying. I update this data monthly(when they release a new file) older files can be found here: https://www.dropbox.com/sh/n8tpe1p45fvzmrk/AAB0YFAIjCpZfF8aZxUXf8uYa?dl=0

I just checked and the csv seems to work directly now with this link(didn't before): http://gatineau.ca/upload/donneesouvertes/ADRESSE.csv

Shapefile even works now: http://gatineau.ca/upload/donneesouvertes/ADRESSE.zip


Fix path (nav->navbar) typo in docs

Short description of what this resolves:
Changes proposed in this pull request:

Ionic Version: 1.x / 2.x
Fixes: #

thanks!


Generate Panels at runtime / Reduce Filesize of winutil

This PR moves the generation of the Xaml Code to display the Panels from the Compile script to winutil itself to be built at runtime.
This has a few advantages over the previous solution:

The compiled winutil.ps1 file is shrunk by about 150 KB (Before 884KB, After 728KB)
The line count is reduced by about 1200 Lines in the compiled script. (Before 15323, After 14090)
The amount of duplicate information in the compiled script is reduced which adds clarity.
Smaller File = Faster Download = Faster Start

Downsides:
I measured the time it takes to build the panels at runtime which came out to add about 250ms. In my case, this wasn't noticeable though, because Powershell + WPFApps do take a moment to start eighter way, and taking into account the time saved downloading the script, this could very well still be faster on a slow internet connection.
IMHO:
The added time is a tradeoff worth taking, considering all the pros mentioned above.

Yeah this is a hard decision to make. It can be beneficial, but on a potato machine with fast internet it would be slower. Since most of the internet falls in to this category I'm going to close this as it is too close to call.


 There was a removal of the
exception handler call, however, it didn't work when the removing is
calling inside the handler. see the comments in code change for detail

Hi @leirocks, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Lei Shi). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

Replaces https://github.com/Microsoft/ChakraCore/pull/34. See previous discussion there.


Link to Document inspection syntax

@flbulgarelli, are the titles ok?, we don't have an English version of this post, do we?

@gcrespi  Yes, titles are ok :smile:
Please check my comment about t and rebase this branch against master.

Yes, it works, I had already checked!. If you end the translation key with "_html", you don't need that "raw"

Oh, awesome!

Fixes #153


strain energy function for StVenant

@lijas I got this invariant formulation of the strain energy function, but I'm getting for the tangent a result which doubles all entries. Could it be that your AD of the stress needs a factor of 2? Otherwise, I need to double check the strain energy formula again
and maybe we should declare one ElasticState which is shared by all hyperelastic materials, if you agree, I can include it in this PR
We can also think about using the analytic forms in material_response and use the AD ones only for testing

Codecov Report

Merging #34 (ee5859a) into main (2a4ef29) will increase coverage by 0.10%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #34      +/-   ##
==========================================
+ Coverage   94.87%   94.97%   +0.10%
==========================================
  Files          10       10
  Lines         351      358       +7
==========================================
+ Hits          333      340       +7
  Misses         18       18

Impacted Files
Coverage Δ

src/FiniteStrain/stvenant.jl
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

Good to add proper testing :P
I dont know where the factor 2 is coming from... C = 4 d^2PhidC^2 as you have written, and S = 2dPhidC.
I will check the _SPK function

I kind of like that all materials have their own state, even though it adds some extra code
Sounds good to AD only for testing.

do you have some pdf with the analytic derivatives of Yeoh and/or NeoHook?

No PDF unfortunately.
In Nonlinear continuum mechanics for finite elment analysis by Javier Bonet, I found derivations of the tangent and stress for the NeoHook material:
function check_tangents_AD(material::NeoHook,loading::Vector)
    state = initial_material_state(material)
    for C in loading

        invC = inv(C)
        J = sqrt(det(C))

        S = material.μ*(one(SymmetricTensor{2,3}) - inv(C)) + material.λ*log(J)*inv(C)
        T = material.λ*(invC⊗invC) + 2*(material.μ-material.λ*log(J))*otimesu(invC,invC)
        T = symmetric(T)

        ∂²Ψ∂C², ∂Ψ∂C, _ =  hessian(C -> MaterialModels.ψ(material, C), C, :all)
        S_AD = 2*∂Ψ∂C
        T_AD = 4*∂²Ψ∂C²

        @show S_AD ≈ S
        @show T_AD ≈ T
    end
end

I had to take symmetrize the analytical tangent for some reason. I remember my proffessor mentioning this at one time, but i never understood why.
I also found analytical derivations for Yeoh material in an old matlab code (voigt notiation):
S2 = 2*( mu/2*I + ...
    2*c2*(Ic-3)*I + ...
    3*c3*(Ic-3)^2*I + ...
    -mu*dlnJdc + ...
    lambda * log(J) * dlnJdc );

dS2_dE = 4 * (...
         2*c2*(I*I') + ...
         6*c3*(Ic-3)*(I*I') + ...
         -mu*d2lnJdcdc + ...
         lambda*(log(J)*d2lnJdcdc + (dlnJdc*dlnJdc')) );

No derivation for StVenant material though :P

I (or someone else) can add these for analytical derivations for the meterial_response instead of the using AD as we are doing now

hm it's true that the symmetric statement is needed, otherwise the test fails, also for Yeoh model

Other than the symmetric issues this should be good to go. Im not sure where or how to investigate into this, if you ever find it out please let me know :)


Windows 10 VSCode icon has black background in many places

Issue Type: Bug
Not always, but most of the time VSCode icon shows with black background, instead of transparent background. It doesn't look too good.
The Shell extension icon is not affected, but others are as seen in the screenshot.
VS Code version: Code 1.21.1 (79b44aa704ce542d8ca4a3cc44cfca566e7720f1, 2018-03-14T14:46:47.128Z)
OS version: Windows_NT x64 10.0.16299

![black icon](https://user-images.githubusercontent.com/1581639/37830506-ccef6322-2e78-11e8-8c42-70259946fc63.png)
System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-7820HQ CPU @ 2.90GHz (8 x 2904)

Memory (System)
31.85GB (24.23GB free)

Process Argv
C:\Program Files\Microsoft VS Code\Code.exe C:\Users\Public

Screen Reader
no

VM
0%

Extensions (8)

Extension
Author (truncated)
Version

python
ms-
2018.2.1

cpptools
ms-
0.15.0

vscode-icons
rob
7.22.0

hyperworks
sle
0.1.0

optistruct
sle
0.1.0

radioss
sle
0.1.0

tcl
sle
0.2.0

templex
sle
0.1.1

Reproduces without extensions

Please reopen if you still see this with our upcoming stable release. See https://github.com/Microsoft/vscode-docs/blob/vnext/release-notes/v1_35.md#updated-visual-studio-code-icon


Unexpected breakage of existing functionality

Seems like this buildpack was moved towards more generic rake execution, which is totally cool.  I've updated my app to use the new buildpack-ruby-rake-deploy-tasks.
However....I discovered this because I did a deploy of my production app, and the migrations didn't run, breaking the app!  Eek.  Might I suggest that the behavior in this buildpack be restored, and maybe print out a deprecation warning so others have a smoother transition?  I'm guessing there are a number of folks who are having a bad surprise right now.
Shame on me for not forking the buildpack (I've done that with others, but referenced this directly).  I'm guessing I'm not the only one....

I'm really sorry for breaking your deploy process. I updated our own applications before pushing this change and completely forgotten that there might be other people using this tool as well. Thank you for pointing out this issue.
Like you suggested I restored the original functionality by defaulting DEPLOY_TASKS to db:migrate and added a deprecation warning.

Thanks psyho.  No worries on the build, we learned quickly and fixed it.  Thanks for your work on these buildpacks in general, they are simple but helpful to not have to reinvent.  Hopefully this revert will make someone else's upgrade path smoother!


ENYO-2128 : Fix reading text twice on marquee item

Issue
If the string of the SelectableItem(or RadioItem) has a long text truncation, screen reader reads string twice.
Cause
Screen reader reads spotlight focused text when audio guidance is on and component is focused with 5-way mode. However, if component internally has marquee mixin for reading long text like moonstone SelectableItem, this issue is reproduced.
Screen reader tries to read dom content but marquee mixin creates marqueeWrapper component dynamically as soon as component is focused, so screen reader first reads content and text of marquee that is created after focusing.
Fix
add aria-hidden:true when marquee is started and remove it when component is blured.
Enyo-DCO-1.1-Signed-off-by: Bongsub Kim<EMAIL_ADDRESS>

This branch has a lot of changed commit, so I will re-create this branch with last commit.


"Upgrade" form nits (add help tooltips/labels)

If there are only two steps, consider removing the step progress line (or enumerate the dots); as is, it looks awkward and confusing
I know Stripe says CVC, but IME I've seen CVV (or CV2) more often
Add tooltip/help text for CV2 (consider using this awesome vanilla JS tooltip lib)
Consider removing Card Holder (is this actually required? I know Stripe doesn't require this, so I don't think it's required)
Rename Card Holder to Name on Card possibly?
Change Month field from input[type=text] to input[type=range]
Add to Month field maxlength="2" min="1" max="12" (curious how this works with i10n; I assume Arabic numerals for CCs are universal, but I don't know)
Change Year field from input[type=text] to input[type=range] (consider adding a pattern that requires the card to be equal to or greater than the current year)

Any fields that seem like they shouldn't be required are there because it makes it easier to deal with fraud, and most password managers are good.
Month/Year need to stay [type=text] to be friendly with some weird password managers. I've had a world of trouble.
Arabic numerals aren't supported by Stripe anyway, IIRC, so that's moot.
I considered adding a tooltip for CVC but honestly if you don't know what that is by now, how have you used the internet. On mobile, Chrome will auto-fill it. And frankly the tooltip is unhelpful if you don't know what your CVC is, since it's in different places for many card types.
"CVV" alone isn't actually used by any credit cards to describe the number. "CVC" matches the docs that I link folks to when they have trouble, so I'm going to stick with that.

Arabic numerals aren't supported by Stripe anyway, IIRC, so that's moot.

Arabic numerals, also called Hindu-Arabic or Hindu numerals, are the ten digits: 0, 1, 2, 3, 4, 5, 6, 7, 8, 9.

what about at least renamnig Card Holder to Card Holder Name to be consistent with the labels for the bank info. fields?

The card holder name field is already changed, but I need to bump the versions of the packages on NPM and I'd rather not do that until I have other changes that need to go out. It's a real production.

It's a real production.

So don't give it the production-quality treatment 👑 👍


howto reference action of custom integration?

I love how the YAML can reference actions from integrations, this promotes re-use of actions, instead of hardcoding methods into the integration.
paths:
    "/db/:id":
        get:
            action: mongodb/findOne

But, how can i reference an action of my own custom integration?
paths:
    "/db/:id":
        get:
            action: mongodb/findOne
        post:
            action: ./myfile.js/mypostAction

This does not work :/ Probably because it applies require() on it.
How about an optional spaceseparated action? something like this:
paths:
    "/db/:id":
        get:
            action: mongodb/findOne
        post:
            action: ./myfile.js mypostAction

or this:
paths:
    "/db/:id":
        get:
            action: mongodb/findOne
        post:
            file: ./myfile.js
            action: mypostAction

If you create your custom integration in the ./integrations directory, you can just reference it by it's ID.  For example, if your integration is in ./integrations/petstore/index.js, you can reference its actions as petstore/getPetById.
That said, it'd be better not to force users to use a particular directory structure to organize their integrations.  Maybe the DataFire.yml config should have an integrations section which pulls in custom integrations, e.g.
integrations:
  petstore: ./petstore-integration.js

paths:
  /rover:
    get:
      action: petstore/getPetById
      input:
        petId: 23

This feature is now available in 2.6.0.


The SAS is not a valid account SAS. An account SAS with service ('ss') and resource type ('srt') parameters is required.

Preflight Checklist

[X] I have installed the latest version of Storage Explorer.
[X] I have checked existing resources, including the troubleshooting guide and the release notes.
[X] I have searched for similar issues.

Storage Explorer Version
1.34.0
Regression From
No response
Architecture
i86
Storage Explorer Build Number
No response
Platform
All
OS Version
No response
Bug Description
The SAS is not a valid account SAS. An account SAS with service ('ss') and resource type ('srt') parameters is required.
Steps to Reproduce

Launch storage explorer.
What kind of Azure resource do you want to connect to? Select Storage Account or Service.
Select SAS and copy shared access signature
and then we get error The SAS is not a valid account SAS. An account SAS with service ('ss') and resource type ('srt') parameters is required.

Actual Experience
Not able to access container using Azure storage explorer using sas
Expected Experience
No response
Additional Context
No response

I have the same issue. @patelamit06 is it fixed in any version?


[Bug]

[REQUIRED] Please fill in the following fields:

Unity editor version: 2020.3.37f1
Firebase Unity SDK version: 10.2.0
Source you installed the SDK: Unity Package Manager
Problematic Firebase Component: Database
Other Firebase Components in use:
Additional SDKs you are using: Facebook
Platform you are using the Unity editor on: Windows
Platform you are targeting: Android
Scripting Runtime: Mono and IL2CPP
Pre-built SDK from the website or open-source from this repo: _____

[REQUIRED] Please describe the issue here:
The application only updates with the data from the database when i run the game twice (on the second time it updates) and testing it also i noticed that if i change a value (e.g. [day = 1]) directly in the database and running the game will not update, if i change the same value again in the database again (e.g. [day=3]) and running the game, it will update the modified value with the previous change (e.g. the value that will be get from the database is [day = 1]) and not with the value i just put in the database (e.g. [day = 3]).
Is there any way I can force the application to get the data directly from the database and not from local data or there is a resolution from my issue?

Previously I was using 'GetValueAsync() and the same issue was happening, so currently I'm using 'GetValueAsync() to get a boolean value in user data to check if user exists and using event listener to get all user data.
The code below was slightly changed to keep it safe, but the structure is the same as in the project.
//Firebase authenticator Initialize on Start(), checking and fixing dependencies Asynchronously, then starts SignIn()

public class Example
{
    DatabaseReference userDataRef;

    void SignIn()
    {
        StartCoroutine(LoadUserData(StartListeningToUserBranch));
    }

    IEnumerator LoadUserData(Action loadSuccessfullCallback)
    {
        var userId = Firebase.CurrentUser.UserId;
        var userProperty = //Get a especifiqui bool item on user branch

        var DBTask = DBReference.Child("users").Child(userId).Child(userProperty).GetValueAsync();

        yield return new WaitUntil(predicate: () => DBTask.IsCompleted);

        if (DBTask.Exception != null)
        {
            //Debug's the DBTask Exception
        }
        else if (DBTask.Result.Value == null)
        {
            //Does what neds if user doesn't exist
        }
        else
        {
            loadSuccessfullCallback();
        }
    }

    private void StartListeningToUserBranch()
    {
        if (FirebaseAuthenticator.Instance.IsAuthenticated)
        {
            string userId = Firebase.CurrentUser.UserId;

            if (!userId.IsNullOrWhitespace())
            {
                userDataRef = DBReference.Child("users").Child(userId);

                userDataRef.ValueChanged += UserBranchChagedEventHandler;
            }
        }
    }

    private void UserBranchChagedEventHandler(object sender, ValueChangedEventArgs args)
    {
        if (args.DatabaseError != null)
        {
            //Debug's the DatabaseError message on args
            return;
        }

        DataSnapshot userData = args.Snapshot;

        //Does what neds to do with the user data

        StopListeningToUserBranch();
    }

    private void StopListeningToUserBranch()
    {
        if (FirebaseAuthenticator.Instance.IsAuthenticated)
        {
            if (userDataRef != null)
            {
                userDataRef.ValueChanged -= UserBranchChagedEventHandler;
            }
        }
    }
}

Thanks for the update, @MimoLopes. I haven't successfully run the code you provided so far. Could you provide a minimal, reproducible example of your implementation so we can have a better understanding on what's causing this to happen? You may upload this to a repository and add me as a collaborator. Make sure that all private information is scrubbed.

Thanks for this, @MimoLopes. I've been unsuccessful in reproducing the issue using the files you provided. It would be helpful if you upload a runnable project in your repository in order to avoid any issues that prevents the scene from running.

Sorry for the delay in answering, I had another priority on the project, I will make an executable project available in my repository as soon as possible.


service option changes

Adding support to send service options for worker only nodes
Bringing in changes from @thxCode's PR https://github.com/rancher/rancher/pull/22443
Supporting vendor updates for RKE and Kontainer-Engine

- Save Windows RKE service options into CRD
- Output the appropriate version images

**Related PR:**
https://github.com/rancher/kontainer-engine/pull/185

**Issue:**
https://github.com/rancher/rancher/issues/22470

LGTM


Bunny drill example

Description of the problem

We had been thinking about making a bunny drill example for a while. Here it is. It is a bit more light hearted than the other examples.

Description of the solution

The bunny drill is here. It's a chill example. It works quite well actually.

How Has This Been Tested?

Ran the drill

Documentation

Documented the drill

Future changes

Short examples like this one are also a nice way to show features. I like it.

@lpsaavedra I need your OK to be able to merge :)


temp script to kill workflow in the agent

test this please

test this please

Seems to work fine, I've just used it to kill an aborted workflow in cmssrv217.
You may want to add .py extension to it though

Oh, I guess you want to execute it in the following way
$manage execute-agent kill-workflow-in-agent workflow_name
if so, then just make it executable and it's Ok to be merged from my side.

@ticoann you can merge this one as well


Doesn't work for standalone installation

Hey! Did you try it without using Filament Panel? With just Form/Table component?
It doesn't work:
Call to a member function getPlugin() on null

We have not try this.

1.0.0


Context dependant comment string

Hi, is it possible to comment with different comment strings based on cursor position in multi-context files like those of Vue and React? There is Shougo/context_filetype.vim which could help with figuring out the current context, but not sure how to make it work with nvim-comment.

Realised by https://github.com/JoosepAlviste/nvim-ts-context-commentstring.

Apologies for opening old issue, but was thinking whether it would be possible to use above plugin's 'on demand' functionality. That is with enable_autocmd set to false, meaning that plugin itself does not run on CursorHold but it is expected from comment plugin to invoke it. Kommentary solved it  by providing hook_function which can call nvim-ts-context-commentstring's update_commentstring() function. This can be set up for specific filetypes only.
Is that something you would be willing to seeing your plugin?
Thanks!

Hey, ATM I wanted to maintain v < 0.5 compatability, but once 0.5 is released I'm up for giving it ago... or if I find the time will do on a branch.
Let's leave this open for now so i don't forget

Hey! Just wanted to let you know that integrating nvim-ts-context-commentstring can be done without explicitly adding any Neovim 0.5 specific code. An easy way to do it would be to make the user configure it in a "hook" which is triggered before any commenting is done.
The user might configure it like this:
require('nvim_comment').setup({
  hook = function()
    -- They can do anything here, e.g.:
    require('ts_context_commentstring.internal').update_commentstring()
  end
})

A good place to trigger this hook might be the start of your comment_toggle function or at any time before you read commentstring.
Here's how b3nj5m1n/kommentary added a similar configuration option: https://github.com/b3nj5m1n/kommentary/issues/43

Hey does @gegoune PR cover your needs? I believe it does, but wanted to check in first.
I've just asked to add some tests and made some comments on the PR


Add Alicia to the Authors file

Add Alicia to the Authors file
No ticket

Before getting review, please check the following:

[x] Does frontend functionality render and work correctly in IE?
[x] Have we load-tested any new pages or internal API endpoints that will receive significant traffic?
[x] Are interactive elements accessible to screen readers?
[x] Have you checked for tech debt you can address in the area you're working in?
[x] If this change involves routes, does it work correctly with pertinent "unusual" routes such as the combined Green Line, Silver Line, Foxboro commuter rail, and single-direction bus routes like the 170?
[x] Are the changes organized into self-contained commits with descriptive and well-formatted commit messages?

@alicia-mbta I used your MBTA email address because that’s what I know. Let me know if you’d prefer to use a personal email (or if you’d prefer not to be listed at all).


Fix some leftover headless issues

This PR fixes some issues where bevy_rapier enabled unnecessary features for headless applications. It also fixes a compilation error that happened if the async-collider feature was disabled, but headless was not enabled.
With these changes #296 should be fully resolved. Not only will rapier no longer panic at runtime due to missing assets/resources, but you can also run a bevy app with the absolute minimum features like this
bevy          = {version = "0.10", default-features=false}
bevy_rapier3d = {version = "0.21", default-features=false, features = ["dim3", "headless"]}

Thanks!

@NiseVoid This PR introduces a new feature headless which seems a bit unnecessary. All it does is that it disables the colliders_with_aabb_intersecting_aabb method which uses bevy::render::primitives::Aabb. Can't we just use the rapier::prelude::Aabb type here so that the method can continue to be exposed?

@Neo-Zhixing is you have a rapier::prelude::Aabb you can call directly ctx.query_pipeline.colliders_with_aabb_intersecting_aabb(&your_aabb, callback);.
All the colliders_with_aabb_intersecting_aabb adds is a handy method when bevy_render is enabled, especially to avoid converting to rapier specific.
We could make it a bit better by taking impl Into<rapier::prelude::Aabb> but I'm not sure it would be a net improvement.


Rename seedu.address to seedu.officeharbor

This package name was a remnant of the previous project AddressBook v3. So this commit just refactors everything to a package reflecting the new project name "OfficeHarbor".

I just realised that there's more things to change. Reverting back to draft


Update all deps

This is just the result of running bundle upgrade so I think syntax wise it should definitely be safe! As long as things install and the site still generates this should be safe to merge IMO.


REQUIRES FACILITATOR REVIEW: Facilitator modifications to Arbitration scope.

Requires new CVCs to provide a draft Governance Strategy Position Document.
Makes CVC Decisions less painful, by allowing optimistic resolution with a single signed message from any CVC member.
Clarifies rules + requirements around CVC Member participation rewards.
Fixes incorrect dot-headings in section 4.
Clarifies rules + requirements around Constitutional Delegate meeting participation requirements.
Constitutional Delegates only count MKR that is delegated through contracts that point to CVC strategies.
Clarifies the calculation used to determine the outcome of CVC decisions to MKR-weight majority.
Clarifies that verified MKR for CVC decisions includes MKR delegated from verified address.
Modified CD Submission process to reflect realities of current delegation contract system.

Closing as submitted in separate PR


Incorrect license

The tag in setup.py says BSD, but the LICENSE.txt is the license text from Matplotlib, which is PSF. It also contains text about pre-/post-1.3.0 which doesn't seem relevant to this repo.

Oops, this is a duplicate of #29


Hotfix: fixes servlet endpoint name error. /translate to /translator

What was the error here? Why is this needed?

The frontend wasn't pulling from the correct servlet location.


Support for the Modrinth API

The modrinth platform has support for mods and plugins, and can be used to fetch download links for jars and mod/plugin version information from provided modids , being the only thing that can change other than a rare api update (recently from v1 to v2)
It could be its own subcommand eg:
pap modrinth install mod fabric-api
modrinth may be omitted or shortened if you deem it appropriate, and it would be similar for plugins of course:
pap modrinth install plugin invsee
Somewhat of the same code can be reused between them as they share many fields and attributes. Everything gets hosted on modrinth cdn (with uniform naming).
This could also replace your static files all together, for config manipulation, you either could use a section for mods and their ids and a section for plugins and their ids.
Modrinth can show information on loaders supported by mods (quilt support might help, but quilt is mostly like fabric) this could also be used to tell pap operators of incompatibilities, if also has "breaks" and dependencies fields. All of this with the filtering of client sided mods with the fields related to "side" support it would be a clean and great addition.
Also to note, you may also want to regex /config folder (using modid/pluginid and named to search) instead of keeping these static files holding their config locations, the only issue may be keeping track of if a mod/plugin used a folders or a lose file, but maybe it is smart to just try both and regex files ending in anything?
It is also optional to show the pap operator possible locations and making them confirm the ones to delete.
If this is unfeasible, it can be held with a file somewhat like before but just kept in two lists, one being plugin and one being mod. Each holding a dict of ids that have a dict each holding folder or file and the relative location, these could also be normally lists, allowing for the option to pass multiple files or folders, pap will load these and if no file is specified then keep track that the mod/plugin does not use it and vice versa for folder.

Hope you somewhat agree with these changes and think they are a good idea, I can't PR right now as I forgot to bring adequate development "gear" with me on an excursion (laptop).

This is actually already implemented in pap, and I should really make it known to the user. You use it by doing pap plugin install modrinth:pluginslug. I'll keep this issue open until I figure out a better way to communicate this feature.

by doing pap plugin install modrinth:pluginslug

I think it would be useful, if it would be possible to make a shourtcut for pap pl gen b <name> and pap pl i <name>.json, similar to the modrinth one you mentioned just for spigot, bukkti, etc., but I can understand if you prefer not to do that.

Not really sure what you are talking about, are you talking about the command itself or about shortening modrinth to m for example?

Also, here was the original issue that caused the feature to be already implemented

oh wow that is great!


Expose start delay in child workflow options

See https://github.com/temporalio/features/issues/515 for more information

This is a feature request. Thanks for suggesting this feature! To work around this feature you can set the search attribute on the Workflow when you start the Workflow Execution. You can also add the search attribute from within your Workflow using UpsertSearchAttributes. You can use temporal workflow list to query Workflow Executions by search attribute. See Temporal CLI Workflow command reference.


Clarify docs on which files are ignored when publishing

The docs for pub publish say that all hidden files are ignored and not published. That's only true if the package isn't using git. If it is, it will include any "." files unless they are explicitly gitignored. We should either clarify the docs or change the behavior.

This issue has been moved to dart-lang/pub#376.


Proprietary fonts used in playground

There is a proprietary font used in the playground which can cause troubles. How about Shabnam?

Hi @par6n and thanks for reaching out. I've bought the license for the font, but I'm quite ambivalent whether I'm using the .current license plan or not. I gotta ask Moslem Ebrahimi for it.

I asked Mr.Ebrahimi, and the project is using the appropriate license plan now 🎉


Internal server error: java.lang.IllegalArgumentException: Script pool for database  is not configured

OrientDB Version, operating system, or hardware.

2.2.11

Operating System

[ ] Windows

Expected behavior and actual behavior
Executed without errors
Steps to reproduce the problem
I create OrientDB function on JavaScript,
`var boxTypeRid = doc.field('@rid');
boxTypeRid = String(boxTypeRid);
var functionName = boxTypeRid.replace(/#/g, '');
functionName = functionName.replace(/:/g, '');
functionName = 'DashboardBoxTypeFunction' + functionName;
db.executeCommand('DELETE FROM OFunction WHERE name = "'+ functionName +'"');
code = doc.field('code');
if (code != '') {
code = code.replace(/'/g, "'");
code = code.replace(/"/g, '\"');
db.executeCommand('CREATE FUNCTION '+ functionName +' "'+ code +'" IDEMPOTENT true LANGUAGE JavaScript');
db.executeCommand('UPDATE DashboardBoxType SET onAfterUpdate = "'+ functionName +'" WHERE @rid = '+ boxTypeRid);
print('Update');
}`
Function fully completed and after this I get error in console:
Internal server error: java.lang.IllegalArgumentException: Script pool for database MyDatabase  is not configured
For example, If function name is DeleteMePlese and this function exist I get error. If this function are not exsit - function complete successfully without errors.

Hi @stas0
I think the error message is not reporting the exact problem.
Could you please try to do the following:

add a db.commit() after the db.executeCommand('DELETE FROM...
surround the function body with a try/catch block and see if it's throwing another exception

Thanks
Luigi

@luigidellaquila  I add try/cath for delete and commit, in console I see the same error


Use the appropriate quote in Windows (#393)

This is probably not the ideal solution, but it makes stuff work until someone come up with a better one

Will do. I’ll probably get to that in Sunday

I pushed another fix. Seems to be okay now

Sorry for bugging, but is there anything else that I should do before we can merge?


linux: 5.8.15 -> 5.8.16

Motivation for this change
Potentially security critical PR in the context of BleedingTooth. 5.8.16 releases a lot of bluetooth updates.
Things done
Ran nix-build nixos/tests/kernel-latest.nix with kernelPackages set to linuxPackages_5_8.

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

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

[X] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests):

$ cd nixos/tests
$ sed -e s/linuxPackages_latest/linuxPackages_5_8/ kernel-latest.nix > foo.nix
$ nix-build foo.nix

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

Anyone, please also approve the cherry-pick https://github.com/NixOS/nixpkgs/pull/100817

Mentioning CVE-2020-12351 to ease searching for this issue.

Seems to have been already updated, closed.


feat: add services spec draft

Closing this for now, if you want to continue the work you can re-open it in the new rfcs repository.


support configure Checks name

for now the Checks name is 'jest-github-action', but i'd like to give it a custom name.
thanks

Having no check-names option means that the checks generated by this action get overwritten. If you have two steps that run the action (no matter if in the same job/workflow) the latter will overwrite the first.
Adding a check-name prevents this.

LGTM, thx


Dropdown opens after clear after reselecting menu

It seems react-virtualized-select opens the menu on clear after selecting an option then reselecting the menu, and then off click before clearing. In comparison react-select does not behave this way. See GIFs below. Do you think this is a react-virtualized-select or react-select bug?
We are experiencing a similar bug where the react-virtualized-select dropdown opens on clear without doing the below but I haven't figured the exact cause. I will post another issue once I get to the bottom of exactly what is causing it. Perhaps your insight on this issue will help me solve that one.

Hm. I'm not seeing the behavior you show for react-virtualized-select:

I'd be happy to review a PR from you if you look into this though 😄  Unfortunately I don't have time to look because I'm swamped with work stuff.

@bvaughn You didn't re-select the menu after selecting an option before clearing if that makes sense. It may be an edge case but I will take a deeper look.

@bvaughn To clarify the issue.

You select an option.
You click outside the dropdown so it is not focused.
Then you click clear
The option menu renders and pops open

If an option is selected and the dropdown is not focused and then you click clear, the menu will render.
If an option is selected and the dropdown is focused and then you click clear, the menu does not render.
In react-select, regardless of whether the dropdown is focused or unfocused, when you click clear it does not render the menu.
I will continue looking at the code. I can see that the optionRenderer is being called on dropdown unfocused clear but is not being called on focused clear. Still trying to figure out why.
Further gifs below:
react-virtualized-select unfocused clear bug:

react-virtualized-select focused clear:

react-select unfocused clear desired behavior:

Makes sense 😄 Let me know what you find

Hi @bvaughn, it seems like it is an issue in the custom menuRenderer react-select code. Looks like @tosho-ait had the same issue here. If you would like we can close this issue and I will open an issue on react-select.
Some how this._openAfterFocus gets set to true when using the react-select custom menuRenderer and you click clear when the dropdown is not focused.
If we delete the line: 291, it fixes the issue but I'm not sure if it breaks anything else. All react-select tests pass and all examples seem the same to me with the line deleted.

Thanks for the update. Let's close this issue then and open one for
react-select. Cheers!
On Jul 30, 2017 10:16 PM, "Niles Turner"<EMAIL_ADDRESS>wrote:

Hi @bvaughn https://github.com/bvaughn, it seems like it is an issue in
the custom menuRenderer react-select code. Looks like @tosho-ait
https://github.com/tosho-ait had the same issue here
https://github.com/JedWatson/react-select/issues/1348. If you would
like we can close this issue and I will open an issue on react-select.
Some how this._openAfterFocus
https://github.com/JedWatson/react-select/blob/master/src/Select.js#L313
gets set to true when using the react-select custom menuRenderer and you
click clear when the dropdown is not focused.
If we delete the line: 291
https://github.com/JedWatson/react-select/blob/master/src/Select.js#L291,
it fixes the issue but I'm not sure if it breaks anything else. All
react-select tests pass and all examples seem the same to me with the line
deleted.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bvaughn/react-virtualized-select/issues/71#issuecomment-318970032,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABznS8_qrkUCstReMAvJVE_vPN8QTpRks5sTWNLgaJpZM4OmCrU
.

@bvaughn, thanks, I've posted the issue on the react-select repo here: #1901

@agirton's new PR #1902 fixes this issue.


Error in Layout/MultilineAssignmentLayout

Just upgraded to 0.58.0 and I got an error with the Layout/MultilineAssignmentLayout cop.
Expected behavior
No error.
Actual behavior
Getting this error:
An error occurred while Layout/MultilineAssignmentLayout cop was inspecting [...]/app/models/concerns/app_scoped.rb:9:4.
undefined method `line' for nil:NilClass
/Users/maxime/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/rubocop-0.58.0/lib/rubocop/cop/layout/multiline_assignment_layout.rb:59:in `check_new_line_offense'
/Users/maxime/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/rubocop-0.58.0/lib/rubocop/cop/layout/multiline_assignment_layout.rb:52:in `check_assignment'
/Users/maxime/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/rubocop-0.58.0/lib/rubocop/cop/mixin/check_assignment.rb:18:in `on_send'
/Users/maxime/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/rubocop-0.58.0/lib/rubocop/cop/commissioner.rb:57:in `block (2 levels) in trigger_responding_cops'

Steps to reproduce the problem
My file is:
# frozen_string_literal: true

module AppScoped

  def self.included(base)
    base.belongs_to(:app, inverse_of: nil, required: true)

    # Creates a default_scope on the class
    base.send(     # this is line 9
      :default_scope,
      lambda do
        current_app = AppScope.current

        if current_app
          base.where(app_id: current_app.id)
        else
          raise(
            ::AppScope::ViolationError,
            "Trying to query an AppScoped model without a configured app"
          )
        end
      end
    )
  end

end

If I write base.send(:default_scope,, I still get the error.
Simplified:
# error:
base.send(
  :default_scope,
  -> {
    where(foo: "bar")
  }
)

# error:
base.send(:default_scope, -> {
  where(foo: "bar")
})

# no error:
base.send(:default_scope, -> { where(foo: "bar") })

## RuboCop version

$ rubocop -V
0.58.0 (using Parser <IP_ADDRESS>, running on ruby 2.5.1 x86_64-darwin17)

This is another example where the cop currently errors:
foo << items.map do |item|
  "#{item}!"
end

Thanks for the feedback. I confirmed that and I opened a PR #6091.


Chinese translation of extract

extract should be translated to 提取, not 解压缩.

VSCode Version: 1.18
OS Version: win10

Steps to Reproduce:

debug any extension
view some .ts file

Reproduces without extensions: Yes/No

Duplicate of https://github.com/Microsoft/TypeScript/issues/19863


App sell

https://github.com/github/docs/blob/main/content/developers/github-marketplace/index.md

@quinio01


Add profile builder tool

Create a tool that users can use to

[ ] Users should be able to edit their existing data using GUI
[ ] Users should be able to use drag and drop functionality to rearrange their data

Adding these functionalities will help the users to easily create and edit their terminal-portfolio.

Join our Discord server to discuss ideas and ask questions.

Hi, I would like to work on this topic :)

@oliCHECK24 go ahead


Generate pkg-config with cmake

This would make getting started with nana easier (e.g. no guessing which libraries to link against).

Do you mean CMake config?

see #438


NO-ISSUE: Remove microshift-gateway-api from rhel94-crel-with-optionals.toml until the package is published on mirror

The problem was introduced in this commit.
Otherwise, the following error is encountered during the rpm-ostree builds.
ERROR: BlueprintsError: rhel-9.4-microshift-crel-optionals: DNF error occurred: MarkingErrors: Error occurred when marking packages for installation: Problems in request:
missing packages: microshift-gateway-api-4.18.0~ec.2-202410101849.p0.ge6b2865.assembly.ec.2.el9
blueprint: rhel-9.4-microshift-crel-optionals v0.0.1

/test ?

/test e2e-aws-tests-periodic
/test e2e-aws-tests-periodic-arm


Please also add .net as TargetFramework (makes it possible to unit test my maui project)

In the following file:
https://github.com/dotMorten/MauiEx/blob/main/MauiEx/MauiEx.csproj
Please change:
<TargetFrameworks>net7.0-android;net7.0-ios;net7.0-maccatalyst</TargetFrameworks>
To:
<TargetFrameworks>net7.0;net7.0-android;net7.0-ios;net7.0-maccatalyst</TargetFrameworks>
That way a unittest project can also run against a maui project that uses this package.

There is no implementation for net7.0 though - are you using ui in your unit tests?

An  implementation for net7.0 is not needed imho.
I need to unittest certain (businesslogic) in my maui app that aren't ui related.
Cause the "standard net7.0" is missing in this project, my unittest fail to build.
All other packages I used (f.e. https://github.com/Alex-Dobrynin/Controls.UserDialogs.Maui/blob/master/Controls.UserDialogs.Maui/Controls.UserDialogs.Maui.csproj) also have "the standard .net framework" as a TargetFramework.
So to resume: you won't need to change anything beside adjusting the csproj file. No concrete implementation is needed.

No concrete implementation is needed.

That's not quite true. There needs to be code to handle and provide developers with feedback that platforms aren't supported like if they try and run it on tizen or macos.
Why aren't you just making your unit tests target the platforms that are actually supported? Maui needs a platform specific target to actually function

Why aren't you just making your unit tests target the platforms that are actually supported?

UnitTest won't work in a android project (they don't run).
See also: https://www.youtube.com/watch?v=C9vIDLQwc7M and https://github.com/jfversluis/MauixUnitTestSample

There needs to be code to handle and provide developers with feedback that platforms aren't supported like if they try and run it on tizen or macos.

I don't think that's needed, since you don't target them by adding net7.0

They will work in a Windows project or use a test runner to run on actual devices

They will work in a Windows project or use a test runner to run on actual devices .
I use xunit to run my unit-tests, why would I want to run those on a device? The only thing that's blocking me, is that this package also needs to target the "standard framework". When you look at for example: https://github.com/Alex-Dobrynin/Controls.UserDialogs.Maui/tree/master this doesn't need any specific implementation when using the default framework (= net8.0 for this example). So I'm pretty sure, that besides adding it in the csproj file, no changes are required.

The net7 lib is used if there's no platform match so it is needed
I don't understand what you're saying here, to be honest. Can you elaborate please?

I don't understand what you're saying here

If you use my library in a Tizen app it will use the net7.0 library since there isn't a platform match and it won't work.
I'm not saying it isn't possible, but it isn't true that there are no code changes needed. At a minimum there needs to be platform support attributes, so the tizen developer at least gets build warnings.

why would I want to run those on a device

Why wouldn't you? Your users will be running on a device. You'd be surprised about the number of platform differences. I daily run all my unit tests on windows, ios, catalyst and android devices.
Is the real reason here that your view models are not separate from your view code, so you can't build those without bringing in all the view controls?

Is the real reason here that your view models are not separate from your view code, so you can't build those without bringing in all the view controls?

That's correct indeed. Indeed splitting them would also solve this. But any guide/faq/example for Maui I have seen so far, is build that way. Don't get me wrong, I truly have respect for the work you are doing and the package itself, but this is really blocking me.
Every other package that I use for gui stuff, also have the standard network as build target.

If you use my library in a Tizen app it will use the net7.0 library since there isn't a platform match and it won't work.

Maybe you can do something like:
#if TIZEN #error NOT SUPPORTED #endif

Again I'm not saying I won't address this - just that it's a little bit more work than you think if you want to do it right and I'm pretty backed up with work for the next month or so. It is open source so you can use a custom local build until I get to it.
#if TIZEN and throw won't work. That would require I also target tizen and there are still more platforms like macos which from the sound of it is what you're running on. It would also prevent anyone from adding a custom tizen handler

I'm very happy with your lib @dotMorten, but I'm facing the same issue as @3sRykaert. I have non-platform specific unit-test on services and view-models. For this reason, I target my own app for .net-8.0.
I might be able to make my client support you, @dotMorten what would I take to make you prioritize adding .net support?

Can you help me understand how you’re using this control in your services and view models? That sounds counter to the mvvm patterns


localization and change of appearance of the phrase about the remaining time

About the PR

Localization and change of appearance of the phrase about the remaining time. This phrase had not been localized for many years, so I decided to localize it myself. The line break has been removed so that all the information is on one side, it looks more compact.
Why / Balance

Technical details

Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

:cl: to4no_fix

locale support PRs shouldnt have a changelog

locale support PRs shouldnt have a changelog

Okay


Adds last updated date

Completes #266
Contribution requirements

[x] I've checked that this isn't a new swag opportunity proposal.
[x] I've checked that this isn't a duplicate pull request.

Changes made

[x] Adds last updated date at bottom of each swag card.
[x] Adds to body.pug and src/js/index.js

Oops, it looks like you opened the PR in the wrong repo. Please open it again in here: https://github.com/swapagarwal/swag-for-dev/compare


Adds ETHLend Token

ETHLend token addition in contract-map.json and lend.svg added to images folder
Source on etherscan:
https://etherscan.io/token/0x80fB784B7eD66730e8b1DBd9820aFD29931aab03
Project website:
https://ethlend.io
Dapp website:
https://app.ethlend.io/
Project whitepaper:
https://github.com/ETHLend/Documentation/blob/master/ETHLendWhitePaper.md
Public logo:
https://github.com/ETHLend/logos

All problems solved, metamask reputation neutral, all necessary data published, if you need anything else more just let me know.

Hello @estebanmino, Thanks for your review, I have deleted the old logo and added a new one in svg format with a size less than 2KB.

@sirsoth  perfect, thanks.

@estebanmino Thanks a lot, when would be our token updated in metamask? In the next update?

@sirsoth yes, in the next release.


Initial Setup

[x] Vite
[x] React
[x] TypeScript
[x] Tailwind
[x] Vitest (Unit Test)
[ ] Playwright (Component Test)
[x] ESLint
[x] Prettier

https://playwright.dev/docs/testing-library
https://playwright.dev/docs/accessibility-testing


Mod Name
JFA
Author(s)
D4RKN3R
License
MIT
Homepage link
https://forum.kerbalspaceprogram.com/topic/223852-jfa-jeb-far-away-1125-alpha-12/
Download link
https://github.com/D4RKN3R/JFA

"Hey, this is already on CKAN!" I just need it updated and have assumed this is the correct way to do that. I am sorry for the trouble if it isn't, but if so, please direct me to the proper way to do this.

I’m not on the ckan team but just gathering information, do you plan to post this release to spacedock too or will you only be using github releases from now on? As that’s what i think reading the release notes

I’m not on the ckan team but just gathering information, do you plan to post this release to spacedock too or will you only be using github releases from now on? As that’s what i think reading the release notes

It's a spacedock issue. There should be details in the comments of the forum page of the mod. I plan to return to spacedock after it's resolved.

Hi @D4RKN3R, sorry for the inconvenicence; this mod has a mysterious inflation error that I haven't been able to figure out yet:

http://status.ksp-ckan.space/

"central directory" most likely means there's something wrong with the ZIP file, but when I downloaded it and inspected it locally, the usual tools for validating ZIP files don't report any issues (otherwise I would have tried to contact you with a recommendation of how to fix it).
So I have to try to think of other places to start investigating: Which tools are you using to makethe ZIPs for this mod? Maybe there's some quirk of incompatibility between them and the ZIP library that CKAN uses. If we can figure that out, then maybe we can also figure out a workaround.

I’m not on the ckan team but just gathering information, do you plan to post this release to spacedock too or will you only be using github releases from now on? As that’s what i think reading the release notes

It's a spacedock issue. There should be details in the comments of the forum page of the mod. I plan to return to spacedock after it's resolved.

Just read your forum message, I believe that to be a firefox issue not a spacedock issue. Try using a different browser to access spacedock

"central directory" most likely means there's something wrong with the ZIP file, but when I downloaded it and inspected it locally, the usual tools for validating ZIP files don't report any issues (otherwise I would have tried to contact you with a recommendation of how to fix it).

I use the default compression thing on my PC (Pop OS). The file extracts fine, and works fine after being extracted. Not sure what the issue with it would be.

Well, the good news is that whatever that issue is, it might be specific to the upload that's currently on SpaceDock.
Or it might start happening again in the next release after this one. 🤷
So switching the hosting over to GitHub should address both problems. I can do that in a couple of minutes...

Just read your forum message, I believe that to be a firefox issue not a spacedock issue. Try using a different browser to access spacedock

Alternately, a Firefox dev (!) apparently just posted a very jargon-heavy explanation of the issue that includes a workaround:

https://forum.kerbalspaceprogram.com/topic/170865-spacedockinfo-mod-hosting-site/?do=findComment&comment=4428045


Raspberrypi4-64, usb no power

Description
Keyboard is not powered, kernel reports errors
Steps to reproduce the issue:

build meta-raspberry @ 6531d1d (HEAD -> ag/rpi4-uboot-update, origin/ag/rpi4-uboot-update) u-boot: Update patches for RPi4

Describe the results you received:
Errors encountered in kernel :
[    2.530074] usb 1-1: new high-speed USB device number 4 using xhci_hcd
[    2.550489] usb 1-1: device descriptor read/8, error -61
[    2.678539] usb 1-1: device descriptor read/8, error -61
[    2.914059] usb 1-1: new high-speed USB device number 5 using xhci_hcd
[    2.934469] usb 1-1: device descriptor read/8, error -61
[    3.062515] usb 1-1: device descriptor read/8, error -61
[    3.170269] usb usb1-port1: unable to enumerate USB device
Describe the results you expected:
Usb keyboard to work.
Additional information you deem important (e.g. issue happens only occasionally):
Always happens.
It did not happen with meta-raspberry@ 1e329d36c99718194d20e6216f1c13199e124bb7
Additional details (revisions used, host distro, etc.):
layers used:
-poky@97c5921666
-meta-openembedded@27d1d3dfa

meta-raspberry @ 6531d1d
local.conf has:
MACHINE ?= "raspberrypi4-64"
DISTRO_FEATURES_append = " opengl"
DISTRO_FEATURES_append = " polkit"
u-boot is not enabled or used.
-Image : core-image-x11
Other packages in image: xfce4

dmesg.txt

What power supply are you using? Are you testing it with an official charger?

Official power source for rpi4.
usb is powered and no issue present if i build a couple of days old commit, as specified.  I suspect something wrong on firmware side.

That is worrying. Would you be able to give it a try with an older firmware and test that theory?

dmesg.txt
This works ok

Using https://github.com/agherzan/meta-raspberrypi/commit/8d5fc54f41dab42d765644ea953bf451ced78f74 i can attach my Razer Blackwidw Lite keyboard and type.
Using 69da331154852f5d62d6cf7a69c4ddb34f5564d0 it does not go more than colored screen at boot.
Using 0d641c8fa918b1c5ecce2cb9918c47bb3c4e1510, and RPI_USE_U_BOOT = "1" it does not appear again.
Using a USBtester that shows Voltage and amps, it never shows amps above 0.
Is anyone else able to reproduce issue ?

The relevant kernel issue is: https://github.com/raspberrypi/linux/issues/3093

@iohe if you are willing to compromise on 1Gb of ram, here is a PR ^

ok for now, will revisit later for full 4g.
Thank you very much!


Code with Harry DSA whole notes

Make sure some important points:
1)It's a single pdf
2)The official document available on Code with Harry website is incomplete
3)Just don't upload incomplete pdf available on any 3rd party website
4)You need merge all the pdf available after the videos
5)Also merge required data from Code with Harry site written by him which is only available online.

Look at this image make sure that your pdf contain this . This is available after each video in the playlist .you need to upload this too either in form of img(clear images) or something else it's up to you

I can work on this issue can u assign it to me i am new to open source

Has this issue already been assigned?
I can work on it

Is this issue not assigned to anyone?
I want to work on it

Just make a valid pr for your request

Just make a valid pr for your request


Closing bracket is missing in token_counts.py for DistilBERT

🐛 Bug

Model I am using (Bert, XLNet....): DistilBERT
The problem arise when using:

[x] the official example scripts: (give details)

I'm trying DistilBERT, and get an "invalid syntax error" when I run examples/distillation/scripts/token_counts.py.
A closing bracket seems missing at line 27.
    parser.add_argument("--data_file", type=str, default="data/dump.bert-base-uncased.pickle",
                        help="The binarized dataset."

Environment

OS: Linux
Python version: 3.6.9
PyTorch version: 1.2.0
PyTorch Transformers version (or branch): 1.1.0
Using GPU ? yes
Distributed of parallel setup ? no
Any other relevant information:

Hi! Indeed, the closing bracket was missing, fixed it with caf1d11! Thanks for the bug report :).