rewrite k8s.py to use requests

In fixing up k8s.py, I realize this code will be overly repetitive without using the python requests library to store headers and SSL cert in in a Session.

+1

Fixed in #20.


RTL support

Hello,
I would like to know any plan for RTL support?

+1

What kind of things you need to do differently to support RTL languages? Would a plugin adding these classes be enough? https://developer.mozilla.org/en-US/docs/Web/CSS/CSS_Writing_Modes

+1

It can be done using a package like this one here but unfortunately we should rewrite the whole framework.

@ybaruchel I ended up overriding directional styles for example: padding-left to padding-right and ...ETC

@Dahdoul I Did the same thing, I wish they had RTL support out of the box.
Note that sometimes you will find yourself using a lot of unset and !important :/

@Dahdoul @ybaruchel So is there any plans to share what you did to help others and of-course this will help with contribution

@AhmedFawzy I just patched the selectors used by my specific app.. It can take forever to change the whole framework :/

Someone mentioned creating a custom variant for RTL stuff on Twitter, maybe worth looking at:
https://twitter.com/patrikengborg/status/1047802931343499264
You'd have to enable the variant plugins experiment in your config file first, then add that function to your plugins.
By the sounds of it, it would let you do stuff like this:

...

I actually created a Tailwind plugin to do this for LTR and RTL a few days ago using the variant plugin API. Feel free to use it!
https://github.com/RonMelkhior/tailwindcss-dir

seems promising Thanks @adamwathan @RonMelkhior

This issue should be prioritized.

I recommend using the plugin @RonMelkhior linked to above:
https://github.com/RonMelkhior/tailwindcss-dir
No concrete plans to add this to core right now as I have never built an RTL site personally, so I am not the best person to implement or maintain such a feature. Thankfully we have a plugin system so it's very straightforward to augment Tailwind with functionality like this yourself if you need it.

https://github.com/RonMelkhior/tailwindcss-dir

Thank you Bro, It's work like a charm.


contact box on website no appearing

Hello
i*ve already tried the "Please refresh the page if the contact box does not appear." but its not appearing.
just wanted to ask if there is a way to add more than one folder to an hub.if not i hope for 2.0.0
Greets

Thank you for posting 🙇
I'm aware of the issue: the provider I was using for feedback eliminated their free tier.
When I'm back from my vacation in late June I'll put in some replacement 👍

ps - sorry Video Hub App 2.0.0 will not allow to select more than one folder per hub.
Two solutions I can think of:

Create several hubs, and you can switch between them with 1 click (just toggle in settings to show recent hubs) in the left sidebar.
Create a symlink to all the folders you'd like within one folder, and create a hub from that one folder. I'm 99% sure it will work, though have not yet tried.

Please do share any other ideas if you think of anything.
While it's possible to rearchitect VHA to allow more than one folder, it would add a lot of extra complexity (and an annoying user interface problem to solve, followed by lots of logic of what to do when users want to add or remove previously-added folders, etc). At least for this year VHA will not have this functionality built in.
Conceptually -- I thought people would have their movies either all in one folder, or compartmentalized in several folders (e.g. "Home Movies", "Cartoons", etc). But I never conducted a survey ... was just hoping that it would be enough for most use cases 😓

Thank you again @Poddi -- I've finally fixed the contact form on the website:
https://videohubapp.com/contact.html
Huge thank you to https://formspree.io/ for making it so easy 🙇 🙇 🙇
Glad I don't have to spend time setting up my own form with PHP 👌


fix: remove last child node when forceFallback is active

When forceFallback is active, childNode gets created, and when you drag and drop to the same position, this check fails because nodes are no longer the same length and it causes first two elements to shift positions in DOM.
Note: This example is in vue2
Screencast from 2024-11-04 09-06-08.webm

@Alfred-Skyblue Any update here :slightly_smiling_face:

The compatibility of toSpliced is not very good, so we should try to use other methods to achieve this.


Add GDExtension compatibility methods

Requires #76446. Split into many commit to make it possible to cherry pick only the relevant bits (assuming that #76446 gets backported). I will add a commit with compat bindings for https://github.com/godotengine/godot/issues/75779, if that becomes possible.

Compared to 4.0.2 this leaved one reported issue:
Validate extension JSON: API was removed: classes/AnimationTrackEditPlugin

Which was completely useless and intentionally removed.
Compared to 4.0.1 a whole bunch of additional issues appear, that are all the hash changes that can't have compat methods atm:
Validate extension JSON: Error: Hash mismatch for 'classes/AnimatedSprite2D/methods/play'. This means that the function has changed and no compatibility function was provided.
Validate extension JSON: Error: Hash mismatch for 'classes/AnimatedSprite3D/methods/play'. This means that the function has changed and no compatibility function was provided.
Validate extension JSON: Error: Hash mismatch for 'classes/Animation/methods/compress'. This means that the function has changed and no compatibility function was provided.
Validate extension JSON: Error: Hash mismatch for 'classes/AnimationPlayer/methods/play'. This means that the function has changed and no compatibility function was provided.
Validate extension JSON: Error: Hash mismatch for 'classes/AudioStreamPlayer/methods/play'. This means that the function has changed and no compatibility function was provided.
Validate extension JSON: Error: Hash mismatch for 'classes/AudioStreamPlayer2D/methods/play'. This means that the function has changed and no compatibility function was provided.
Validate extension JSON: Error: Hash mismatch for 'classes/AudioStreamPlayer3D/methods/play'. This means that the function has changed and no compatibility function was provided.
Validate extension JSON: Error: Hash mismatch for 'classes/CanvasItem/methods/draw_set_transform'. This means that the function has changed and no compatibility function was provided.
Validate extension JSON: Error: Hash mismatch for 'classes/Curve2D/methods/sample_baked'. This means that the function has changed and no compatibility function was provided.
Validate extension JSON: Error: Hash mismatch for 'classes/Curve2D/methods/sample_baked_with_rotation'. This means that the function has changed and no compatibility function was provided.
Validate extension JSON: Error: Hash mismatch for 'classes/Curve2D/methods/tessellate_even_length'. This means that the function has changed and no compatibility function was provided.
Validate extension JSON: Error: Hash mismatch for 'classes/Curve3D/methods/sample_baked'. This means that the function has changed and no compatibility function was provided.
Validate extension JSON: Error: Hash mismatch for 'classes/Curve3D/methods/sample_baked_with_rotation'. This means that the function has changed and no compatibility function was provided.
Validate extension JSON: Error: Hash mismatch for 'classes/Curve3D/methods/tessellate_even_length'. This means that the function has changed and no compatibility function was provided.
Validate extension JSON: Error: Hash mismatch for 'classes/DisplayServer/methods/tts_speak'. This means that the function has changed and no compatibility function was provided.
Validate extension JSON: Error: Hash mismatch for 'classes/Font/methods/find_variation'. This means that the function has changed and no compatibility function was provided.
Validate extension JSON: Error: Hash mismatch for 'classes/GridMap/methods/make_baked_meshes'. This means that the function has changed and no compatibility function was provided.
Validate extension JSON: Error: Hash mismatch for 'classes/Image/methods/save_jpg_to_buffer'. This means that the function has changed and no compatibility function was provided.
Validate extension JSON: Error: Hash mismatch for 'classes/Image/methods/save_webp_to_buffer'. This means that the function has changed and no compatibility function was provided.
Validate extension JSON: Error: Hash mismatch for 'classes/Image/methods/bump_map_to_normal_map'. This means that the function has changed and no compatibility function was provided.
Validate extension JSON: Error: Hash mismatch for 'classes/PhysicsBody2D/methods/move_and_collide'. This means that the function has changed and no compatibility function was provided.
Validate extension JSON: Error: Hash mismatch for 'classes/PhysicsBody3D/methods/move_and_collide'. This means that the function has changed and no compatibility function was provided.
Validate extension JSON: Error: Hash mismatch for 'classes/PhysicsBody3D/methods/test_move'. This means that the function has changed and no compatibility function was provided.
Validate extension JSON: Error: Hash mismatch for 'classes/RandomNumberGenerator/methods/randfn'. This means that the function has changed and no compatibility function was provided.
Validate extension JSON: Error: Hash mismatch for 'classes/RenderingServer/methods/environment_set_ambient_light'. This means that the function has changed and no compatibility function was provided.
Validate extension JSON: Error: Hash mismatch for 'classes/RenderingServer/methods/canvas_item_set_canvas_group_mode'. This means that the function has changed and no compatibility function was provided.
Validate extension JSON: Error: Hash mismatch for 'classes/RenderingServer/methods/force_draw'. This means that the function has changed and no compatibility function was provided.
Validate extension JSON: Error: Hash mismatch for 'classes/Window/methods/popup_centered_ratio'. This means that the function has changed and no compatibility function was provided.
Validate extension JSON: Error: Hash mismatch for 'classes/Window/methods/popup_centered_clamped'. This means that the function has changed and no compatibility function was provided.

And finally 4.0.0 has a few more changes where no compat code can be created:
Validate extension JSON: Error: Field 'classes/MenuBar/properties/start_index': type changed value in new API, from "bool" to "int".
Validate extension JSON: Error: Field 'native_structures/PhysicsServer3DExtensionMotionCollision': format changed value in new API, from "Vector3 position;Vector3 normal;Vector3 collider_velocity;real_t depth;int local_shape;ObjectID collider_id;RID collider;int collider_shape" to "Vector3 position;Vector3 normal;Vector3 collider_velocity;Vector3 collider_angular_velocity;real_t depth;int local_shape;ObjectID collider_id;RID collider;int collider_shape".
Validate extension JSON: Error: Field 'native_structures/PhysicsServer3DExtensionMotionResult': format changed value in new API, from "Vector3 travel;Vector3 remainder;real_t collision_safe_fraction;real_t collision_unsafe_fraction;PhysicsServer3DExtensionMotionCollision collisions[32];int collision_count" to "Vector3 travel;Vector3 remainder;real_t collision_depth;real_t collision_safe_fraction;real_t collision_unsafe_fraction;PhysicsServer3DExtensionMotionCollision collisions[32];int collision_count".

Additionally a bunch of virtual methods have changes between 4.0 and 4.1 but they are currently not validated.

Thanks, these look great!
I manually tested the compatibility function for RichTextLabel::push_list() and it worked perfectly. I didn't test any of the others, but skimming the code, it all looks good to me :-)

Discussed at the GDExtension meeting, and it looks good!
Otherwise this just needs to be rebased after PR #76446 is merged, and then the remaining commits squashed into one commit, and then we'll approve it!

Added more of the same, 3 more PR with similar breaking changes have been merged since.
And https://github.com/godotengine/godot/pull/69988 has removed a whole bunch of APIs but I didn't do anything about that (also IIRC these apis were marked experimental before anyways)

@TokageItLab Protected methods were made public, can we check the AnimationNode?

Needs a rebase, by the way.

So what do you think we should do with this PR then? Reject it, but return to it post 4.1? Or merge anyway as a reference point?

That's a good question.
Maybe leave it be for a little while as we watch how things turn out after #77410? But assuming that all turns out good, I'd personally probably lean towards "Reject it, but return to it post 4.1?" It doesn't really make sense to merge code that won't ever get used, and I'm sure we'll have no shortage of compatibility methods to add after 4.1 stable is out.
What do others think?

I guess we can leave it be for now, so when we need to make a 4.1+ version, we have a starting point. I'll turn it into a draft for now.

Closing as there are probably enough compat methods in the engine so that this is no longer needed as reference.

Indeed, but still thanks for work and for providing everyone with a framework!


Colab Assertion Failed Issue

I've created my own copy of the colab project and have managed to successfully implement it up until the final stage. I get this error when running the program using the sample image of Lincoln:
`/usr/local/lib/python3.7/dist-packages/torch/nn/functional.py:3635: UserWarning: Default upsampling behavior when mode=bilinear is changed to align_corners=False since 0.4.0. Please specify align_corners=True if the old behavior is desired. See the documentation of nn.Upsample for details.
"See the documentation of nn.Upsample for details.".format(mode)
python test.py /content/Time-Travel-Rephotography/results/Abraham Lincoln_01/input_sibling /content/Time-Travel-Rephotography/results/Abraham Lincoln_01/skin_mask/face_parsing
0it [00:00, ?it/s]

error                                     Traceback (most recent call last)
 in ()
19         results_dir="results/"
20 ))
---> 21 main(args)
2 frames
/content/Time-Travel-Rephotography/tools/match_histogram.py in main(args)
144
145     mask_A = cv2.resize(cv2.imread(args.ref_mask, 0), A.shape[:2][::-1],
--> 146                         interpolation=cv2.INTER_NEAREST) > 0 if args.ref_mask else None
147     mask_B = cv2.resize(cv2.imread(args.src_mask, 0), B.shape[:2][::-1],
148                         interpolation=cv2.INTER_NEAREST) > 0 if args.src_mask else None
error: OpenCV(4.1.2) /io/opencv/modules/imgproc/src/resize.cpp:3720: error: (-215:Assertion failed) !ssize.empty() in function 'resize''

Looks like tools/parse_face wasn’t executed properly and mask was not generated.
Were the checkpoints downloaded properly? Check third_party/face_parsing/res/cp

The solution to #3 fixed it! You're right that the checkpoints weren't properly downloaded. Code is working now!


IBM: Extract cluster settings

Describe the issue
Automate how to extract all the cluster settings for reuse later
Which Sprint Priority is this issue related to?
The 'Milestone' should clearly list Sprint Priorities - which one is this issue related to?
Additional context
Add any other context, attachments or screenshots
Definition of done

CLI script to do this

Terraform takes care of this for us


Allow deleting draft groups

Sometimes people will make errors while creating draft groups.
It's impossible to modify the drafts themselves, so the only way to fix the error is to create another draft group.
This might cause significant storage overhead. To overcome this issue new methods are created, allowing to mark draft group as discarded and then delete drafts from it.

@evgenykuzyakov this MR should be merged after the #9


some fixes to get CVV module compiling with Visual Studio 2013

This should address #166 . I tested it on Windows 10 with Visual Studio 2013 with both Microsoft and Intel compilers. I have not tested the patch on other OS's, but hopefully the build bot will succeed.

Actually buildbot doesn't test this module, because there is no QT5 on Windows slaves.

:+1:


include BuildResult in public Mustache API

This small PR includes the Mustache build results in the public API to allow developers to better annotate types and pass the build result around if needed.
The discussion for this originated on Discord, so I'll include the important bits below for future readers:

me: Is there a design decision behind why the MustacheBuildResult type isn't marked public?

renerocksai: At the time, I probably thought it wouldn't be necessary to make it public, as it is returned by a pub fn anyway, which makes it kind-of-public. This choice makes it impossible (I think) to declare variables of this type in user code and encourages just assigning the result of the build function.
Do you have a need for var x : zap.Mustache.MustacheBuildResult = .{}  and assigning later or similar? Ah, maybe passing it around or storing it in your own struct: I would rather use its .str() result to hold on to.
Feel free to submit a PR to make it pub. I might do so myself; but then I'd strip the Mustache off the type names (type shaving 🤣) of the load args and the build result.

me: I've switched to passing around strings at this point (maybe that's better anyway...?), but I found it strange that I couldn't add annotations for a return type of a public function. Not saying I need it changed; I just found it slightly jarring from an API perspective.

Thank you for bringing this up, fixing it, and documenting it so excellently! 👍

I know it's a bit off-topic. But could you also rename the MustacheLoadArgs into LoadArgs? I am on vacation with limited computing capabilities.

@renerocksai I did some more type shaving in ee94128c7ab61faa232df7da95401478b62c0e6a.

Perfect! Thank you very much!!!!


Error when client sends "OPTS UTF8 ON"

Hi! I'm currently having this issue, trying to connect a pyftpsync client to a pyftpdlib server.
The client response is:
"Could not send 'OPTS UTF8 ON': '501 Invalid argument.'"
The pyftpdlib server logs no output.
Python 3.7
pyftpdlib 1.5.6
pyftpsync 3.1.0
Shouldn't the pyftpdlib server deal with this?! Any workarounds?
I appreciate any help!

Same thing here https://github.com/giampaolo/pyftpdlib/issues/266
TLDR from that issue:

From g.rodola on July 16, 2013 05:55:29

...

As per RFC-2640 client and server should just use UTF-8 by default.
Expressively enabling it via OPTS is uncompliant and I find astonishing that a
company like Microsoft, with thousands of developers, can produce software of
this quality.

...

I'm getting this error. The ftp host is HostGator.com.


chore: bdd-spring-1577989795 to 0.0.1

chore: Promote bdd-spring-1577989795 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


feat [SWA-276] update user email error

The type of this PR is: feat
This PR resolves SWA-276 : updates error message for an invalid email address which was not completely human-readable

Description

PR Checklist (tick all before merging)

[x] I have included screenshots or videos to illustrate my changes, or I have not changed anything that impacts the UI.
[x] I have tested my changes on iOS and Android.
[x] I have added tests/stories for my changes, or my changes don't require testing/stories, or I have included a link to a separate Jira ticket covering the tests.
[x] I have added a feature flag, or my changes don't require a feature flag. (How do I add one?)
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added an app state migration, or my changes do not require one. (What are migrations?)
[x] I have added a changelog entry below or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

ⓘ 'User-facing' changes will be published as release notes.

Cross-platform user-facing changes

Rename modal title - patrinoua

iOS user-facing changes

Android user-facing changes

Dev changes

This PR contains the following changes:

Cross-platform user-facing changes (Rename modal title - patrinoua)

  Generated by :no_entry_sign: dangerJS against cccbe5955c9e025cf87f8afca9601bc860034f85


Separate intermediate item directories for .NET 5 and 6 builds

Fixes intermittent build failures of IronPython.Wpf and IronPython.SQLite when parallel builds are enabled with multiple target frameworks.

Thanks!


Mobile - UI bugs

Describe the bug

[x] Update text: "recovery phrase" -> "seed phrase"

[x] UI bug on NFT detail screen

[x] When scroll
https://github.com/Koniverse/SubWallet-Mobile/assets/117999489/31171b37-9269-4ab3-8ff1-62dc0c919073

[x]  Properties

[x] UI bug on list NFT Conlection detail

[x] UI bug when import multi account

[x] UI bug when turn on network

Expect: Show only 1 toast

[x] UI bug Mission Pools detail screen on IOS

[x] Show blank screen when cancel Wallet Connect app to app
Steps:

On the SubWallet app, open Scan QR screen on WalletConnect
Perform connect WalletConnect app to app
Cancel request and observe the displayed screen

Actual: Show blank screen

https://github.com/Koniverse/SubWallet-Mobile/assets/117999489/c19ff68f-b292-4a19-948d-1f8b7e438dc0
Expect: Display Settings screen when there is no connection, display Connection List when there is at least 1 connection

Version test : https://file.cdmteck.com/s/DB9JLAnjJczopcC/download?path=%2F&files=SubWalletMobile-build-317-1487-merge-2024-02-20T05-46-31.ipa
Device test : IP 7+

[ ] UI bug when turn on network
Expect : If popup Connected appeared then popup Connecting close

https://github.com/Koniverse/SubWallet-Mobile/assets/155931563/b1e37cda-ddfd-4dbb-9227-c0a7dd2b6fc7

@Quangdm-cdm Please update icon when search on Earning positions, Earning options

Version test : https://file.cdmteck.com/s/DB9JLAnjJczopcC/download?path=%2F&files=SubWalletMobile-build-323-1487-merge-2024-02-26T04-09-28.ipa
Device test : IP 7+

[ ] Change text color

Version test : https://file.cdmteck.com/s/DB9JLAnjJczopcC/download?path=%2F&files=SubWalletMobile-build-325-1487-merge-2024-03-01T11-54-50.ipa
Device test : IP 7+

[ ] Update font-size seed phrase
Expect : Font-size = 12 according to design

Version test : https://file.cdmteck.com/s/DB9JLAnjJczopcC/download?path=%2F&files=SubWalletMobile-build-326-1487-merge-2024-03-04T09-02-28.ipa
Device test : IP7 +

[ ] Error when input Verify seed phrase
https://github.com/Koniverse/SubWallet-Mobile/assets/155931563/98dbee3b-011b-4e28-9da9-b4113f11c9b8

Version test :

IOS : https://file.cdmteck.com/s/DB9JLAnjJczopcC/download?path=%2F&files=SubWalletMobile-build-332-1487-merge-2024-03-11T03-32-36.ipa

Android : https://www.diawi.com/qrcode/link/VowMxd
Device test : IPHONE 7+, SAMSUNG

[ ] Update text color on IOS

[ ] UI bug of button Stake more

Version test : https://www.diawi.com/qrcode/link/FwjnFY
Device test : IP 7+

[ ] Check if the seed phrase has duplicate characters

[ ] UI bug

Expect :

[ ] Ui bug on Mission pools detail screen

[ ] Re-check Align text and icon on button Stake more
Expect : Align = center

Version test: https://www.diawi.com/qrcode/link/eJiCYm
Device test: IP 11, MI 8 Lite
Validation passed in these case.


New taxon name that includes author name prevents rank determination.

If the full name of a novel taxon (i.e. unknown to the taxon spine) is entered including any author names, this would mess up the code trying to guess the taxon rank. Leaving out the author name will make it harder to precisely identify the taxon in question. So there should be some way to include author name without causing problems for rank guessing.

I cannot think any clever solution that would not require special action from the side of the digitizer... This is because the author field is quite unpredictable as far as number of tokens and punctuation.
There are the following options:

Add an extra input field for the author name alone
Instruct the digitizer to separate author from the rest of the full taxon name with an underscore.

@bhsi-snm Please evaluate these two suggestions.

Alternatively, we could simply add a dropdownlist from which the digitizer can choose the taxon rank. This is probably the most fail-safe.

I have discussed it with @chelseagraham. She thinks that underscore + taxon rank dropdown is the best solution, same as you but I think for now we can live with the underscore because we can easily incorporate that (only the _before authorname). It is easiest to implement and would be a minor change in the digitiser's workflow.

@FedorSteeman can you please give a couple of examples to explicitly show how this is supposed to be used ?

@bhsi-snm Exactly as @chelseagraham surmised.

Just as an update: I have communicated to the digitizers the protocol for when encountering new taxon. They informed me that often the folders do not include the year, only author.


feat: Created cluster workflow templates based on the provided file paths, made changes to #11925. Signed-off-by: Simardeep Singh https://github.com/SimardeepSingh-zsh

Fixes #TODO
Motivation

Modifications

Verification

Closing as spam based on https://github.com/argoproj/argo-workflows/pull/11925#pullrequestreview-1677278400


Allow build systems to specify the files that SourceKit-LSP should watch

Build systems managed by BuildSystemManager get file change notifications forwarded from the client. But SourceKitLSPServer specifies the patterns to watch for. Build systems should be able to specify their file watch patterns and we should re-register the file watching as a new build system is started, which adds new file watch patterns.

Synced to Apple’s issue tracker as rdar://136014553


Allow the prod ningx.conf to proxy backend if the URL is present in env

Add a conditional rule in nginx.conf to proxy backend requests to /api/* if the BACKEND_URL environment variable is present

we are moving away from proxy configuration, so this is not required


{aksarc} and {akshybrid} - autogenerated content contains broken links

Describe the bug
This is coming to my attention from the autogenerated Available Azure CLI extensions doc.

Line 36: [Suggestion] Link 'https://dev.azure.com/msazure/msk8s/_git/hybridaks-utils/azcli/aksarc' points to a page that doesn't exist. Check the path or URL and update the link.
Line 36: [Suggestion] Link 'https://dev.azure.com/msazure/msk8s/_git/hybridaks-utils/azcli/aksarc' points to a page that doesn't exist. Check the path or URL and update the link.
Line 37: [Suggestion] Link 'https://dev.azure.com/msazure/msk8s/_git/hybridaks-utils/azcli/akshybrid' points to a page that doesn't exist. Check the path or URL and update the link.
Line 37: [Suggestion] Link 'https://dev.azure.com/msazure/msk8s/_git/hybridaks-utils/azcli/akshybrid' points to a page that doesn't exist. Check the path or URL and update the link.

Please route to service team and/or update lines 9817 and 9871 (%/azcli/aksarc), and lines 9983 and 9928 (%/azcli/akshybrid) in azure-cli-extensions/src/index.json that contain these "Home" links.
Related command
az aksarc and az akshybrid
Errors
See broken links reported in azure-docs-cli build report and copied above and hilighted below.

Issue script & Debug output
n/a
Expected behavior
The build report for autogenerated content may always have warnings of some type, but it should not have broken links that cause a customer to receive a 404.
Environment Summary
n/a
Additional context
n/a

Thank you for opening this issue, we will look into it.

@HariPrasadV, please help take a look.

Describe the bug
This is coming to my attention from the autogenerated Available Azure CLI extensions doc.

Line 36: [Suggestion] Link 'https://dev.azure.com/msazure/msk8s/_git/hybridaks-utils/azcli/aksarc' points to a page that doesn't exist. Check the path or URL and update the link.
Line 36: [Suggestion] Link 'https://dev.azure.com/msazure/msk8s/_git/hybridaks-utils/azcli/aksarc' points to a page that doesn't exist. Check the path or URL and update the link.
Line 37: [Suggestion] Link 'https://dev.azure.com/msazure/msk8s/_git/hybridaks-utils/azcli/akshybrid' points to a page that doesn't exist. Check the path or URL and update the link.
Line 37: [Suggestion] Link 'https://dev.azure.com/msazure/msk8s/_git/hybridaks-utils/azcli/akshybrid' points to a page that doesn't exist. Check the path or URL and update the link.

Please route to service team and/or update lines 9817 and 9871 (%/azcli/aksarc), and lines 9983 and 9928 (%/azcli/akshybrid) in azure-cli-extensions/src/index.json that contain these "Home" links.
Related command
az aksarc and az akshybrid
Errors
See broken links reported in azure-docs-cli build report and copied above and hilighted below.

Issue script & Debug output
n/a
Expected behavior
The build report for autogenerated content may always have warnings of some type, but it should not have broken links that cause a customer to receive a 404.
Environment Summary
n/a
Additional context
n/a

It's an issue due to authentication as their code is in private repository.

Sorry for the delayed response, I'll work with my team to update this link.

We've updated the link with the latest version of akarc 1.2.20

@HariPrasadV , @jsntcy, The broken link oversight committee is notifying me again of the following in azure-cli-extension-listmd.
Link 'https://dev.azure.com/msazure/msk8s/_git/hybridaks-utils/azcli/akshybrid' points to a page that doesn't exist. Check the path or URL and update the link.
You can see the link in line number 37 -->

Can you kindly give this another pass as this is our one remaining broken link in the extension's doc.
Thank you.


Bot activates when toggle mute/unmute is used

This happens when the following is done:

be in a voice channel
toggle mute

toggle unmute

the line "[USER] moved to General Hangout." is sent in the #malabot channel

Repeat from 2

Thank you for the lovely bug report.


Example Pod to connect Ray client to remote a Ray cluster with TLS enabled

Why are these changes needed?
Provide an example of connect Ray client to a remote Ray cluster with TLS enabled on the ray client port.
Related issue number
Closes #992
Checks
Setup the environment
# Create k8s cluster
kind create cluster
kubectl cluster-info --context kind-kind

# Deploy kuberay
export KUBERAY_VERSION=v0.5.0
kubectl create -k "github.com/ray-project/kuberay/ray-operator/config/default?ref=${KUBERAY_VERSION}&timeout=90s"

# Get this example
git clone https://github.com/tedhtchang/kuberay
cd kuberay
git checkout connect-remote-raycluster-tls

# Start a cluster with TLS enabled
kubectl apply -f ray-operator/config/samples/ray-cluster.tls.yaml

# Create client pod and connect to cluster
kubectl apply -f ray-operator/config/samples/ray-pod.tls.yaml
kubectl logs ray-client-tls

Verify the log contains IP of the head node.
Defaulted container "client" out of: client, gen-cert (init)
OpenBLAS WARNING - could not determine the L2 cache size on this system, assuming 256k
{'node:<IP_ADDRESS>': 1.0, 'object_store_memory': 521180774.0, 'CPU': 1.0, 'memory':<PHONE_NUMBER>.0}

Thank you for this contribution! I will review this PR after I finish the release process.

Thanks for the detailed feedback. I will add a step to the md. It will be something like:
Step 5: Connect the cluster with Ray client using TLS for interactive development...

@kevin85421  Added a step to the doc. Please take a  look.

@kevin85421 Could you provide some context about the Ray client limitation and reasons for demote/promote Ray client/Ray job submission?

@kevin85421 Could you provide some context about the Ray client limitation and reasons for demote/promote Ray client/Ray job submission?

The Ray client is very sensitive to both the Ray version and the Python version being used in your laptop and the RayCluster.

If you use the Ray client on your laptop to connect to a RayCluster, you will need to install all related dependencies on your laptop, such as TensorFlow and PyTorch, even if the libraries only run on the RayCluster.

The network connection between your laptop and the RayCluster must be very stable.

On the other hand, ray job submission will copy your Python script to the head Pod and execute the script in the head Pod, so (1) (2) (3) are solved. However, ray job submission may not fulfill the requirement of "interactive development".

@kevin85421 Thanks. I will share this with my team.


[Concurrency] Task "nurseries" or "Task scopes"

~Introduces very very basic "nursery" types -- they will be extensively documented in an evolution pitch.~
Proposal is now online: https://github.com/DougGregor/swift-evolution/blob/structured-concurrency/proposals/nnnn-structured-concurrency.md
This PR only reflects the proposal, does not add or polish anything yet.
It is just the API shape as we can't yet really get to implementing it.
I'll continue here shortly.
Another piece of rdar://70141994

@swift-ci please smoke test

@swift-ci please smoke test

Thanks to Kavon's https://github.com/apple/swift/pull/34551 a lot of pending TODO's about missing awaits resolved here :)
Renames of those types may follow; for now let's get them in and keep iterating.

@swift-ci please smoke test


patch: drop DoctrineAnnotations

BC: AnnotationReader constructor changed and it's methods stops triggering exceptions
TestControllerWithInvalidParameterAnnotation deleted as for is ignored when HideParameter used as parameter attribute
Also attribute exceptions and classes polished
Fixes: #675

Codecov Report
Attention: Patch coverage is 97.93814% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 95.16%. Comparing base (53f9d49) to head (96ee8d8).
Report is 79 commits behind head on master.

Files
Patch %
Lines

...notations/Exceptions/InvalidParameterException.php
50.00%
1 Missing :warning:

src/Annotations/Input.php
66.66%
1 Missing :warning:

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #677      +/-   ##
============================================
- Coverage     95.72%   95.16%   -0.57%
- Complexity     1773     1815      +42
============================================
  Files           154      171      +17
  Lines          4586     4795     +209
============================================
+ Hits           4390     4563     +173
- Misses          196      232      +36

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

@oojacoboo I marked this PR as draft because I though it would be nice to fix documentation with it (and remove php7 examples as we do not support it). Or I can have make that in separate PR. What is better for you?

@oojacoboo I marked this PR as draft because I though it would be nice to fix documentation with it (and remove php7 examples as we do not support it). Or I can have make that in separate PR. What is better for you?

Agreed.  Let's update the docs in this PR.

@fogrye thanks for the docs updates.  I think we should go ahead and update the Annotations Reference for attributes and drop the "annotations" name.  So the list of attributes are no longer @whatever, but #[whatever] or whatever.


Add vim-node plugin

Added the vim-node plugin to help with node development.

More plugins are needed for development, but this is a start.


Support alternative xUnit (JUnit) formats

Please add support for creating JUnit xml output report, we need this badly. Since most other tool support that.
Like right now we have pybot -x for generating xUnit xml format report files.
Similarly plz add support to generate JUnit xml format report file.
Or maybe point me to some script which will convert robot output.xml to JUnit format (http://llg.cubic.org/docs/junit/)

How would that jUnit output be different to the current xUnit output?

Related to my previous comment, have you tested using the xUnit output?
See also https://en.wikipedia.org/wiki/XUnit.

Yes I have tried the xUnit output using the pybot -x option. One difference is that the xunit report file is missing the top level "testsuites" node. But you might have seen the Junit report file begins with "testsuites" node.
We are using couple of other tools which gives output in stantard Junit format, so it would be nice if we could get a similar junit format output from pybot.
I am new to this, so not sure how to fix this

The problem is that there is no such thing as the standard jUnit format. This is highly annoying.
The current format used by RF is based on some jUnit output variation, but there could be others. If you can find statistics that other formats are used a lot more widely, we can consider changing the current format. Alternatively you can create a script that converts the current format to a format understood by tools you use. I don't like the idea of us adding multiple xUnit formats when nobody has bothered to create an official standard.

ok thanks, will workaround my creating our own custom output. This bug can be closed now.

After thinking this a little bit more, xUnit format configurable is not a totally impossible idea. I definitely don't want a totally new output, but having an option like --xunitformat to control the format would be OK. That would basically have two preconditions:

Someone studies what xUnit XML flavors there are. Based on that we can design this functionality in more detail.
Someone is interested enough about this to provide a pull request based on the agreed design.

Do you @mverma-va have interest to look at this?

Anyone interested to add support for xUnit file configuration? If not, I'll close this issue.

We are using Bamboo and Robot FW. There is only JUnit parser in Bamboo so we'd appreciate to have JUnit output from Robot FW.

@davidk1: Robot already provides JUnit output, we just use more general xUnit name for it. The problem is that there is no official standard for that format and Robot only implements one variant. I'm fine making xUnit format configurable. See my comment above for what it would require.

I consider this a valid enhancement and set labels accordingly. Actually getting this implemented requires someone providing a pull request or paying me or someone else for development.

See #442 for discussion about adding the original xUnit support.

Just adding for reference that soapui junit xml report is really well formatted and includes most of the required info.

It would be great if people using xUnit/jUnit outputs would provide a little more information about the current situation about the output "standard":

List in comments tools that work with Robot's current --xunit outputs.
Even more importantly, list tools that don't work with the current --xunit outputs.
Add references to specs about the output format if you know any.
Add references to outputs files produced by other tools. Adding an attachment is fine too.

I'm wanting the same thing as @mverma-va.  I've been using junit-viewer to view my test outputs, but it doesn't appear to be working with robot generated xunit xml files.
I'd love to have this functionality

List in comments tools that work with Robot's current --xunit outputs.

I've been using Bamboo's JUnit Parser to get my results listed and that works great.

@drew-royster: Could test why junit-viewer doesn't work with Robot's xUnit outputs? It would be great if you could try manually changing outputs to make them compatible so that we'd know what kind of changes would be needed.

It would be great if someone could study what's the current status with different xUnit/jUnit XML outputs. If there are different standards in wide use, we could make it somehow possible to use select which one to use. If there's a dominant standard that's different to what we currently use, we could even consider changing out xUnit output to use that. That could only happen in a major version, though.
Anyway, this needs external help from people familiar with this topic or interested to study that. I don't expect the actual implementation to be too complicated.

The xunit report generated cannot be used in Jenkins Zephyr plugin to update test cases on Jira. Because it uses Junit report or might Xunit report format is not compatible with that plugin. :(

xUnit and jUnit reports should in theory be the same. The problem is that at least earlier (see #442) there has been no official spec for xUnit/jUnit compatible reports. If there is now, we can either change xUnit output or make it configurable somehow. Someone interested in this topic should study the current situation and report the findings here.

This is the issue and Jobprogress token might be missing in Xunit format.

I seriously doubt jUnit outputs have a standard attribute jobProgressToken. Related to that, what we'd need to get forward is somekind of jUnit output specification. Last time I checked such standard didn't exist but that was long time ago.

OP's linked description of junit format looks about right - atleast in the form that it can be digested by most of the tooling (eg, jenkins). I have not really used "xUnit" format ever but my understanding is that there are difference (as there are also multiple versions?).   As a side comment, even if there is no such thing as "standard" for junit, it would be acceptable if the format is something that CI tools with their report plugins can digest and if im not mistaken, atleast jenkins does have xsd schema defined so it could be used as a basis ?
Anyway, I would be interested on picking this up when i have downtime.

Thanks for your investigation @rasjani and hope you have time to look at this more closely at some point. It's interesting, but not really that surprising, to see that other tools are having the same problem. It's super annoying nobody standardizing jUnit/xUnit back in the early days. Do you know the origins of the format Jenkins uses? Jenkins is so ubiquitous that they standardizing to certain format is likely to make their format at least a de-facto standard.

junit-10.xsd: doesnt validate "hello world" like xunit report produced by robot due to testsuite attribute "skip" not being allowed as it has been removed - only "skipped" is left.
junit-9.xsd and junit-8.xsd against the same helloworld xunit report passes validation just fine.
@KishoreNatesh i think your your issue most likely has nothing to do with the xunit format itself.  Looking at the code (https://github.com/jenkinsci/zephyr-for-jira-test-management-plugin/blob/master/src/main/java/com/thed/zephyr/jenkins/utils/rest/TestCaseUtil.java) and the stacktrace you provided, the failure happens with the request response for from zephyr when it tries to assing a test to a cycle.

@rasjani Is that junit-10.xsd used by Jenkins nowadays? Can you test would it be enough to change skip to skipped in Robot's xunit outputs? The place where to do that is here:
https://github.com/robotframework/robotframework/blob/master/src/robot/reporting/xunitwriter.py#L54

@pekkaklarck It's used by Jenkins xUnit plugin. When I edit xunitwriter.py L54 and changed skip to skipped, Jenkins doesn't throw that error anymore.
Plugin's Schema Definitions
junit-10.xsd only allows skipped for testsuite: https://github.com/jenkinsci/xunit-plugin/blob/master/src/main/resources/org/jenkinsci/plugins/xunit/types/model/xsd/junit-10.xsd#L164
junit-9.xsd allowed skip, skips, or skipped: https://github.com/jenkinsci/xunit-plugin/blob/master/src/main/resources/org/jenkinsci/plugins/xunit/types/model/xsd/junit-9.xsd#L111

Feed output.xml or xunit.xml to junitth(http://junitth.sourceforge.net) failed, junitth is great tool for reporting, It will be great if robot will support it.
@pekkaklarck

@pekkaklarck @kanchi240  regarding missing time attribute: https://issues.jenkins-ci.org/browse/JENKINS-51797  - it was reported as bug in jenkins side and was quickly fixed.

Did I get these points about the time attribute right?

[xj]Unit format supports optional time attribute in testsuite elements.
Robot doesn't add such attribute (but it adds to testcase elements).
Jenkins had a bug that time attribute wasn't optional but that's now fixed.

If I got all that right, then we don't need to do anything, but I don't see problems us adding the time attribute to testsuite elements either. If @kanchi240 or someone else considers that to be a good idea, please submit an issue about it. Although that probably is a a safe addition, it's still best done in a major release and now when RF 3.1 is under development would be a good time to add it.

Have I got these points about Jenkins compatibility right?

The problem is that we use skip, not skipped, attribute with testsuite elements.
Changing skip to skipped doesn't break any other tool using [xj]Unit outputs.

If the above points are correct, then we don't need separate jUnit output support nor do we need to make the xUnit output we produce configurable. All we need is simply changing skip to skipped. If that's the case, then we need to update the title of this issue or possibly submit a new one about skip -> skipped change and close this one. Regardless of that, we'd also need a PR with code and test changes. Shouldn't be a big task.

I did some experimentation with Jenkins addons and here's the findings:

JUnit Plugin 1.26.1

Works with both "skip" && "skipped".

xUnit plugin 2.3.0 with "Junit" format:

Works Only with "skipped" due to recent changes.

I've provided a patch that fixes this issue..
Also, worth to empasize that when using "xunit" addon, you have to pick "JUnit" format, nothing else will work and doesn't even make sense to support..

Thanks @rasjani for investigation! I'll add this RF 3.1 scope and since you apparently have a PR ready I target this to beta 1 that's planned for next Wednesday.
PS: The list in your comment may not be correctly formatted.

Or should we actually submit a new issue about skip -> skipped? This issue is about configuring xunit outputs and it seems that's not needed.

Å

 DATADOG_API_KEY=api_key DD_ENV=ci datadog-ci junit upload \                          ✔  282  09:17:31
--service robot-framework
execution_reports/
Starting upload with concurrency 20.

I am getting this error as the parser is looking for testsuites or testsuite in the output xml. How can I fix this?


Installation problem

When trying to install it on Ubuntu with microk8s I try to use this command, but it doesn't work:
 ./installer-[darwin|linux|windows.exe] -namespace=compose -etcd-servers=http://compose-etcd-client:2379
This doesn't work too:
./installer-linux -namespace=compose -etcd-servers=http://compose-etcd-client:2379
Do I miss sth?

@stefaner1 I am having the same issue. Looking at the compose/compose-api pods, they are logging an error: W0318 20:36:44.333026       1 clientconn.go:1120] grpc: addrConn.createTransport failed to connect to {http://compose-etcd-client:2379 0  <nil>}. Err :connection error: desc = "transport: Error while dialing dial tcp: lookup compose-etcd-client on <IP_ADDRESS>:53: no such host". Reconnecting...
I've tried replacing the compose-etcd-client with localhost or <IP_ADDRESS> and doesnt work either.
did you figure it out?

They fixed It here: #151
Run installer-[darwin|linux|windows.exe] --namespace=compose --etcd-servers=http://compose-etcd-client:2379
But it looks like it all buggy. I couldn't make it to the end of installation.

@stefaner1 I got it figure it out.
here, in the instructions to install etcd, the Option 1 says is only for evaluation; apparently that is the service the http://compose-etcd-client:2379 is trying to hit. After I did that, it started working.

This is my setup:

Ubuntu Server 18.04
microk8s 1.14.10
helm 2.16.3
sudo apt-get install iptables-persistent to make iptables forward policies persistent, then sudo iptables -P FORWARD ACCEPT
run microk8s.inspect to make sure everything is fine.
then follow the instructions from the compose on kubernetes page

hope this helps.
Now I will try to deploy a compose using docker 😅

Hi! Thanks for your feedback. Unfortunately we're no longer maintaining this repository so I will close this issue.


Launching Binder fails

Launching Binder to run this code fails with an error on the Binder site of "error connecting to event stream"

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

ID: e4c8bd02-16d6-d043-4f2c-a337a70954f5
Version Independent ID: 8fa84a6d-5c1e-3562-43aa-019f968edd0f
Content: Quickstart: Extract printed and handwritten text - REST, Python - Azure Cognitive Services
Content Source: articles/cognitive-services/Computer-vision/QuickStarts/python-hand-text.md
Service: cognitive-services
Sub-service: computer-vision
GitHub Login: @PatrickFarley
Microsoft Alias: pafarley

@jimbobbennett Thanks for your question. We are checking on this and will respond to you soon.

@jimbobbennett Could you please share the snapshot of the error.

Hi @jimbobbennett , I can't reproduce this error. Is it still occurring?

@PatrickFarley - still happens to me. I click on the Launch Binder button in Safari on macOS and get the error. It works in Edge on macOS, just not on Safari.

I believe this is an issue with MyBinder itself, though it doesn't seem to be well understood. Compare: https://github.com/jupyterhub/mybinder.org-deploy/issues/1281
#pleaseclose


[EDID_VIDEO_IN] error with kali

name: kali
encrypted: no
Entering /mnt/stateful_partition/crouton/chroots/kali...
crouton: version 1-20170619181610~master:980f090f
release: kali-rolling
architecture: amd64
targets: kde
host: version 9592.71.0 (Official Build) stable-channel banjo
kernel: Linux localhost 4.4.64-09098-g4504f6dcbb08 #1 SMP PREEMPT Wed Jul 26 23:33:44 PDT 2017 x86_64 GNU/Linux
freon: yes
Unmounting /mnt/stateful_partition/crouton/chroots/kali...
name: kali-rolling
encrypted: no
Entering /mnt/stateful_partition/crouton/chroots/kali-rolling...
crouton: version 1-20170619181610~master:980f090f
release: kali-rolling
architecture: amd64
targets: xfce
host: version 9592.71.0 (Official Build) stable-channel banjo
kernel: Linux localhost 4.4.64-09098-g4504f6dcbb08 #1 SMP PREEMPT Wed Jul 26 23:33:44 PDT 2017 x86_64 GNU/Linux
freon: yes
Unmounting /mnt/stateful_partition/crouton/chroots/kali-rolling...

Please describe your issue:
So I am trying to install the latest version of Kali on an Acer Chromebook 15, but after a minute or so of installing, I get these errors:
++++++++++++++++++++++++++++++++++++++++++++
Compiling CRAS (i386)...
common/edid_utils.c: In function 'show_edid':
common/edid_utils.c:687:29: error: '*' in boolean context, suggest '&&' instead [-Werror=int-in-bool-context]
(edid_data[EDID_VIDEO_IN] * 0x10) ? "Blank to black" : "",
common/edid_utils.c:688:29: error: '*' in boolean context, suggest '&&' instead [-Werror=int-in-bool-context]
(edid_data[EDID_VIDEO_IN] * 0x08) ? "Separate" : "",
~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~
common/edid_utils.c:689:29: error: '*' in boolean context, suggest '&&' instead [-Werror=int-in-bool-context]
(edid_data[EDID_VIDEO_IN] * 0x04) ? "Composite" : "",
~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~
common/edid_utils.c:690:29: error: '*' in boolean context, suggest '&&' instead [-Werror=int-in-bool-context]
(edid_data[EDID_VIDEO_IN] * 0x02) ? "On-green" : "",
~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~
common/edid_utils.c:691:29: error: '*' in boolean context, suggest '&&' instead [-Werror=int-in-bool-context]
(edid_data[EDID_VIDEO_IN] * 0x01) ? "Serration V" : "",
~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~
cc1: all warnings being treated as errors
Failed to complete chroot setup.
Unmounting /mnt/stateful_partition/crouton/chroots/kali-rolling...

++++++++++++++++++++++++++++++++++++++++++++

#### If known, describe the steps to reproduce the issue:
After powerwashing, enabling dev mode, and entering the crosh shell, enter
`sudo sh -e ~/Downloads/crouton -r kali-rolling -t xfce`
The above errors will halt the download. Trying again will return:

++++++++++++++++++++++++++++++++++++++++++++
/usr/local/chroots/kali-rolling already has stuff in it!
Either delete it, specify a different name (-n), or specify -u to update it.
++++++++++++++++++++++++++++++++++++++++++++

Trying to update with
`sudo sh -e ~/Downloads/crouton -r kali-rolling -t xfce -u`
will produce the same cc1 message as above.

This is my first time trying this, and all the guides I have found are outdated, recommending sana and stuff like that. Any help would be appreciated.

I got the same errors on my HP chromebook 11(ARM). Is there any method to solve it ?

@Void1ight,
DUP - Take a look at this issue for the root cause & a workaround: #3352
Hope this helps,
-DennisL


Use markdown for mailtos

See [Paul D. Hunt](mailto:opensourcefonts@adobe.com?subject=[GitHub] Source Code Pro) for instance.

Should also suggest a subject line and use the Jekyll configuration variable.


Known Issue: Plugin does not block controls when player dies

Background info: The plugin blocks input so it won't break the death screen.
Did not occur before. Unknown cause. (Maybe another plugin as I was developing a plugin.)

Fixed.


TypeError: exceptions must derive from BaseException

When I tried to run the command
!python train.py --dataset csv --csv_train ../training.csv  --csv_classes ../wheat_head_class_list.csv  --csv_val ../validation.csv
got the error:
**CUDA available: True
Traceback (most recent call last):
File "/content/pytorch-retinanet/retinanet/dataloader.py", line 166, in _parse
return function(value)
ValueError: invalid literal for int() with base 10: '245.0'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "train.py", line 180, in
main()
File "train.py", line 57, in main
transform=transforms.Compose([Normalizer(), Augmenter(), Resizer()]))
File "/content/pytorch-retinanet/retinanet/dataloader.py", line 153, in init
self.image_data = self._read_annotations(csv.reader(file, delimiter=','), self.classes)
File "/content/pytorch-retinanet/retinanet/dataloader.py", line 270, in _read_annotations
x1 = self._parse(x1, int, 'line {}: malformed x1: {{}}'.format(line))
File "/content/pytorch-retinanet/retinanet/dataloader.py", line 168, in _parse
raise(ValueError(fmt.format(e)), None)
TypeError: exceptions must derive from BaseException**
Here's the link to my notebook

The important part is here:
ValueError: invalid literal for int() with base 10: '245.0'
It looks like your class file is messed up somehow? Could you post it here?

here I am posting out the files that I have created

You need to have only 1 bounding box per line, i.e.:

/data/imgs/img_001.jpg,837,346,981,456,wheat-head
/data/imgs/img_002.jpg,215,312,279,391,wheat-head
/data/imgs/img_002.jpg,22,5,89,84,wheat-head
Values should be int, not float, so 942.0 -> 942
On Sat, Jul 18, 2020 at 2:40 PM soumochatterjee<EMAIL_ADDRESS>wrote:

here I am posting out the files that I have created
[image: image]
https://user-images.githubusercontent.com/42636586/87859619-0fef5380-c954-11ea-9039-538e1fb7d836.png
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/yhenon/pytorch-retinanet/issues/175#issuecomment-660523710,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACL3RSUUBUODNIVKFLMIZOLR4HUALANCNFSM4O7Y5KEA
.


[DEV-11473] Feature - Update data model to support EmbedNode layout

I'll merge this into my branch to add providers support to EmbedNode.

@MohammadxAli but we don't need providers property in the EmbedNode interface. Do we?

Yes, right. The type was moved:
https://github.com/prezly/slate/pull/469/commits/c6f4765226d3ecb1a89f14633d6bf96624ac958f#diff-06466310fd5774266b941c53ded026bfca72e0ded9245f86ced9e9030fb07076R20-R30


Source code of showcase in README.md?

https://github.com/AvaloniaUI/Avalonia#showcase
It will be great to learn these code samples.

These can be found here and here.

We should definitely add links to the readme for these.

Adding now and closing issue.


Automated cherry pick of #6951

Cherry pick of #6951 on release-5.29.
/cc  @mkraft

Will try to auto merge this PR once all tests and checks are passing. This might take up to an hour.

Trying to auto merge this PR.

Pull Request successfully merged
SHA: bc4d0cec7334efd1d011f6fdb147f287d75f1612


assert messages for std.string

starting to work on dlang/phobos/project/2

CircleCi failure was due to the PR targetting an old master of Phobos. Rebasing against latest upstream/master helps (just did that for you).

BTW we have a .dscanner.ini file with a check for this (this gives you an overview of all places where to look and also prevents regressions):
diff --git a/.dscanner.ini b/.dscanner.ini
index 4d33cde60..89f63d64c 100644
--- a/.dscanner.ini
+++ b/.dscanner.ini
@@ -196,7 +196,6 @@ assert_without_msg="-etc.c.SQL_,\
 -std.signals,\
 -std.socket,\
 -std.stdio,\
--std.string,\
 -std.traits,\
 -std.typecons,\
 -std.uni,\

run with e.g.
make -f posix.mak dscanner

@wilzbach thank you


External doc referred in Helidon's rest client doc needs update

Environment Details

Helidon Version: 2.3.2
Helidon MP

Problem Description
Customer is asking how to disable hostname verification for a rest client in Helidon MP. The repo doc: https://github.com/oracle/helidon/blob/master/docs/mp/restclient/09_rest-client.adoc suggests reading the microprofile rest client configuration doc in https://download.eclipse.org/microprofile/microprofile-rest-client-1.2.1/microprofile-rest-client-1.2.1.html, but doesn't provide tips on how to disable hostname verification. However, the new version of the microprofile rest client doc does provide this tip in: https://download.eclipse.org/microprofile/microprofile-rest-client-2.0/microprofile-rest-client-spec-2.0.html#_hostname_verification. This issue was created to find out if doc can be changed with this new link.

Hi Lisa @ljamen, Thanks for assigning this to yourself so I'll let you work on this and remove myself as an assignee. Basically the goal of this change is just to replace
the link  https://download.eclipse.org/microprofile/microprofile-rest-client-1.2.1/microprofile-rest-client-1.2.1.html with https://download.eclipse.org/microprofile/microprofile-rest-client-2.0/microprofile-rest-client-spec-2.0.html
in https://github.com/oracle/helidon/blob/master/docs/mp/restclient/09_rest-client.adoc#rest-client-configuration. Let me know if you have further questions.

https://github.com/oracle/helidon/pull/3909


How to create a cluster where the masters reside in the publice subnets and nodes in the NATed private subnets

I just tried kops-cn to create a cluster. It looks both masters and nodes must be in the public subnets. Is there any way to separate them into the different types of subnet?

it should support. please refer to the kops upstream docs and issues like
https://github.com/kubernetes/kops/issues/1097
https://github.com/kubernetes/kops/issues/4548#issuecomment-422072394


Warning: _interopDefaultLegacy is not defined Use --force to continue.

Describe the bug
after installing packages and grunt globally , when we run grunt , it throws this warning and aborts the testing.

Thanks for opening this request.
This is a great question for Stack Overflow.

Perhaps is related to https://github.com/rollup/rollup/issues/3748


"Failed to Deploy"

Your tool looks great and I was able to create a test path and deploy the generated artifact, but all subsequent attempts to deploy a path result in the error message "Failed to deploy". Currently running the software on Windows 11. I dug around, but there appears to be no logging of the output of the generation process that allows me to see what might be failing. Looking through the source, generation seems to be calling gradlew; are there any specifics that need to be in place for gradlew to run correctly?

Please disregard ... This tool is too clever; deploy does an actual Roborio deploy. My miss.


Don't install ghc before restoring dependencies

GitHub runners currently use stack 2.13.1 (e.g., Ubuntu).
Currently, the action runs stack query compiler and downloads GHC before restoring the deps cache.
https://github.com/freckle/stack-action/blob/ac0dd8dbe5850e6597a5a4ce5fe4e06613f490ce/action.yml#L177-L179
https://github.com/freckle/stack-action/blob/ac0dd8dbe5850e6597a5a4ce5fe4e06613f490ce/action.yml#L196
In stack 2.14.0, it became possible to run stack path  without installing GHC (https://github.com/commercialhaskell/stack/issues/6340).
To restore deps, the action runs stack path.
https://github.com/freckle/stack-action/blob/ac0dd8dbe5850e6597a5a4ce5fe4e06613f490ce/action.yml#L189-L191
When stack 2.14.0 (or newer) is available on runners, stack path won't install GHC.
Possibly, stack query compiler will still install GHC. If it's the case, this command should be run after the dependencies are restored, so that this command doesn't install a GHC.

Thanks for the report.
This seems pretty annoying to work around (though PRs welcome), so I'm optimistic it'll Just Work with 2.14 and would prefer to check that before doing anything. I think if we tossed a stack update in our own CI, that should get 2.14 and we could see how it works?

Sorry for the delayed response.
I think there will be 2.15 (https://github.com/commercialhaskell/stack/issues/6397).
It's not currently available, but I think it's possible to build stack from source if you'd like to experiment.

Since this has been an issue since the get-go, I'm comfortable leaving it open for a while longer till it's easier to experiment (e.g. until a stack update will get 2.15).

I've dug into this a little more now that 2.15 is available.
It seems that stack path --stack-root specifically no longer requires a GHC install to run. But a bare stack path or stack path --most-other-things will still install GHC.
In order to restore a cache, we need to know the paths and key. Right now that means:

stack path --stack-root
stack path --programs
stack query compiler to include the compiler in the cache key

To avoid doing any of this before we've restored a cache, we can:

Rely on stack 2.15 for stack path --stack-root to not install GHC
Use "stack.yaml"[resolver] for that part of the cache key, instead of calling stack query compiler early
Move calling stack path and/or stack query compiler for the purposes of setting outputs back to the end of the action

That just leaves stack path --programs.
On my system (Linux) this returns a directory that is under ~/.stack. That means there is no reason to include it as its own cache path, it will be included by caching ~/.stack already. However, it was added when we were working on Windows caching stuff, so I worry it returns a path outside of ~/.stack on Windows.
@deemp can you check that? If it does... is it worth caching stack path --programs (and so installing GHC, etc) before we restore a cache? Is there any other way we can find this path?

Yes, newer stack path --programs downloads GHC.
I think the easiest solution is to create an issue and ask to not download GHC for stack path --programs.
Here's why (not sure whether it's correct).
As I understand, there's config.yaml with global settings (link).

If the file is deleted, and Stack needs to consult it, Stack will create a file with default contents.

link

Stack obtains global configuration from a file named config.yaml.

It's possible to configure stack path --programs via local-programs-path (link). The paths are different on Linux and Windows.
The local-programs-path belongs to non-project-specific configuration (link).

Non-project configuration options can be included in a project-level configuration file (stack.yaml) or in global configuration files (config.yaml).

So, to determine the output of stack path --programs, we'll need to consult several config files in the same way stack does it.

As I understand, there's config.yaml with global settings

Well, we can ignore (~/.stack/)config.yaml because that is local to a users machine, and is not part of the cloned project on a GitHub Runner -- I seriously doubt GitHub is using the file to adjust anything on their runners, right?

The local-programs-path belongs to non-project-specific configuration

We could (easily) just look at stack.yaml ourselves for local-programs-path. In the TypeScript branch I'm working on I already have the code to decode the Yaml and could add whatever keys we're interested in.
The only real downside is we will be duplicating the knowledge of what the default is when the key is not specified:

Linux: {caching stack-root includes it}
Windows: %LOCALAPPDATA%\Programs\stack, if the %LOCALAPPDATA% environment variable exists. Otherwise, {caching stack-root includes it}

The windows logic is a bit annoying, but I think that's kind of worth it to do now, even if we report a Stack Issue, rather than waiting for them.
WDYT?

Good news! It seems stack path --programs doesn't, in fact, install GHC:

So we should be good. I also see that v2.15 is already the version of stack on the GitHub runners now. So once the TypeScript branch is merged, this Issue will be addressed.

I think that's kind of worth it to do now

I agree. We seem to have no other options until new versions of stack are available on runners.
I regret a bit that I thought their fix covered more options than only --stack-root. I could have created an issue about stack path before the release of stack 2.15. Anyway, now there's an issue in the stack repo.

Good news! It seems stack path --programs doesn't, in fact, install GHC:

After I had removed ~/.stack, it decided to download GHC.

Yeah, I think I just misread the output. Alright, we'll I'll re-implement the logic for now and we can go back to stack path if/when it's addressed, again.

Well, we can ignore (~/.stack/)config.yaml because that is local to a users machine

Well, a user may write a custom file, but probably it's a rare case.

Yeah, I think I just misread the output.

Perhaps you know that https://github.com/mxschmitt/action-tmate allows to connect to a runner. So, you can execute some commands manually to inspect what's going on.

That's neat!

The new stack should be available in "a few weeks" (link).

This is now addressed. We are coding our own logic for stack path --programs, and will want to switch that when a fixed Stack is released, but we are not installing GHC ahead of caching anymore.


Switch from hardcoded "https" protocol

Bug fix: addresses issue #4523
Description

Related issue and scope

[ ] I opened an issue to propose and discuss this change (#????)

My changes affect the following components

[ ] API
[ ] Controller
[ ] Message Bus (e.g., Kafka)
[ ] Loadbalancer
[ ] Invoker
[ ] Intrinsic actions (e.g., sequences, conductors)
[ ] Data stores (e.g., CouchDB)
[ ] Tests
[ ] Deployment
[ ] CLI
[ ] General tooling
[ ] Documentation

Types of changes

[x] Bug fix (generally a non-breaking change which closes an issue).
[ ] Enhancement or new feature (adds new functionality).
[ ] Breaking change (a bug fix or enhancement which changes existing behavior).

Checklist:

[x] I signed an Apache CLA.
[x] I reviewed the style guides and followed the recommendations (Travis CI will check :).
[ ] I added tests to cover my changes.
[ ] My changes require further changes to the documentation.
[ ] I updated the documentation where necessary.

Tagging folks for review :D @rabbah @dgrove-oss @mrutkows @bdoyle0182

@rraavi @rabbah This indeed appears to fix the issue as described since the regex for parseActionUrl() would fail. Just concerned that the action url may have a hardcoded prefix set elsewhere.  Can an integrated test be added to test this indeed carries through?

@mrutkows I checked the source, didn't find any other references.


Modified how the home directory in posix compliant environments is de…

…termined using posix methods for PHP since Apache sets the HOME environment variable incorrectly or not at all in posix environments.

This leaves in tact the current code for windows why replacing non windows code with posix compliant calls because they will always find the user home directory whereas getenv('HOME') will not.

@logistiker this is failing in every travis build, so it does not appear to be a safe or equivalent change.


When entering a Chinese punctuation, it acts like keeping hitting the key

Environment
Windows build number: 10.0.19030.1
Windows Terminal version: 0.7.3291.0

Steps to reproduce
Hit the key "，" (or other punctuation) in windows terminal with Chinese input method
Expected behavior
Output a "，" (I delete the extra "，" in the following picture manually)

Actual behavior
keeps outputting "，" just like I keep hitting the "，"

Some other information

I have reproduced it in Windows Terminal using wsl, cmd and powershell7-preview on my two PCs.
Another phenomenon is that Windows Terminal add space behind Chinese character automatically after my inputting one character.

For me, I have to type twice to input one Chinese puncutation, like period(。), I'm also using terminal version 0.7.3291.0
It reminds me this issue #2191 , one Chinese character rendered as two small charactors.
It seems wide character is not handled properly.

I am in the same situation as @etern 's instead of OP's

I have the same problem as @etern which has been discussed in #3745
Though looking opposite, I wonder whether these issues are related in some aspect.

I have encountered the same bug @wrvsrx

wt version: 0.7.3382.0

Recently, I test it on another PC whose windows build number is 10.0.18363.535, and it behave like what #3745 describes. Comparing this to windows terminal's behavior mentioned above on my PC  with windows build number 10.0.19030.1, I think it might be related to the windows version.

@wrvsrx And my windows version is: 10.0.19037.1
Seems that it did relate to the win10 version......


Pattern: Feature Card - The link (Arrow) on the card is not aligned and with long text, link goes out of card layout.

Detailed description
Device:- Windows 10, Mac and Mobile
Browser:- Firefox, Chrome, Safari
Issue :-

The link (right arrow) on the card is not aligned as per design
When we enter long text on the card the link arrow alignment is going out of the card layout.

Steps to reproduce the issue

Open the browser ( Firefox, chrome or Safari)
Open the Storybook URL on the browser
Select Feature Card section
On knobs, enter long text
Verify the link on the card.

Additional information

Item 1 is a dup to issue (#1071).  Item 2 will be enforced by the content management system or the application using this pattern.  Closing.


Update (M)KDAKernel to do naive looping over studies when number of studies is large

Summary
In #386, I modified the (M)KDA map generation process to do multiple studies in parallel. This produced a significant speed gain (> 30%), but I didn't consider the memory impact: when the number of studies is very large, memory explodes, as we internally construct a 4d image (number of studies x spatial dimensions).
We don't want to give up the speed gains, because in most cases memory won't be an issue, but in the specific case of, e.g., setting up the entire Neurosynth dataset, the kernel transformation step becomes impractical and the user is forced to go to lowmem, which will probably be very slow. Proposed solution is to add a check for the number of studies, and if it's above some conservative value we can experiment with (say, 500 or 1000), to naively loop over studies being passed to compute_kda_ma from MKDAKernel._fit(), instead of passing them all in one shot. Alternatively, we could add a flag in the estimator, but I suspect people will miss this, so probably better to make an implicit decision for them.
Additional details
We get the best of both worlds: increased speed under normal conditions, and acceptable memory consumption at the cost of slower processing when the dataset is particularly large. (We could even try to be clever and do something in between, by passing batches of studies, but this feels unnecessary.)
Next steps
This block probably just needs to be wrapped in a loop that only kicks in if the number of studies is above some threshold.

I finally stumbled across this issue today. Here's what the traceback looks like:
Traceback (most recent call last):
  File "/home/tsalo006/nimare/joblib/conda_env/lib/python3.8/site-packages/nimare/base.py", line 314, in fit
    maps = self._fit(dataset)
  File "/home/tsalo006/nimare/joblib/conda_env/lib/python3.8/site-packages/nimare/meta/cbma/base.py", line 78, in _fit
    ma_values = self._collect_ma_maps(
  File "/home/tsalo006/nimare/joblib/conda_env/lib/python3.8/site-packages/nimare/meta/cbma/base.py", line 176, in _collect_ma_maps
    ma_maps = self.kernel_transformer.transform(
  File "/home/tsalo006/nimare/joblib/conda_env/lib/python3.8/site-packages/nimare/meta/kernel.py", line 200, in transform
    transformed_maps = self._transform(mask, coordinates)
  File "/home/tsalo006/nimare/joblib/conda_env/lib/python3.8/site-packages/nimare/meta/kernel.py", line 403, in _transform
    transformed = compute_kda_ma(
  File "/home/tsalo006/nimare/joblib/conda_env/lib/python3.8/site-packages/nimare/meta/utils.py", line 347, in compute_kda_ma
    kernel_data = np.zeros(kernel_shape, dtype=type(value))
numpy.core._exceptions.MemoryError: Unable to allocate 13.8 GiB for an array with shape (2050, 91, 109, 91) and data type int64

memory_limit should address the problem, so I think the question is whether the kernel should have a limit that is independent of the memory_limit parameter, or if memory_limit should be set to some non-None value by default.


Update all subclasses of NSObject to pass InstanceManager and BinaryMessenger to parent class and override copy()

This was finished with https://github.com/flutter/plugins/pull/5961


Fixed an issue where the forgot_password textview would be pushed off the screen by the signup textview when in landscape.

Resolves #654
Description
Adjusted the forgot_password text to appropriately align below the sign up text. Before it would be squashed when in landscape on some devices.

Fixes #(issue)
Type of change
Just put an x in the [] which are valid.

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings

Is it going to create any issues with portrait mode? Paste a screenshot of that too


make a new dimension with dataset.merge

Is your feature request related to a problem?
I was trying to combine datasets with a field data (i.e. radar signal data on a lat and lon grid) each dataset being from a different point of time. As far as I could understand, there currently is no straight forward way to do this in xarray (or at least I did not find it).
Describe the solution you'd like
Maybe this would be a nice thing to add internally (e.g. xr.merge([ds1, ds2], newaxis='time')). Or in case I just did not find it, lets add it to the Combining data Documentation site for easy access.
Describe alternatives you've considered
I added a new coordinate and dimension (i.e. 'time') to every dataset individually before using merge for the datasets.
Additional context
No response

Hi @TomNicholas and @dcherian ,
thanks for your replies! I am super sorry, I must have overseen the part were it says "or new dimension" in the concatenate documentation. This indeed is exactly what I thought about.
Sorry for the bummer!

No problem @PLSeuJ - if you think the docs could be clearer in any way then we are open to suggestions!

I put some thoughts to that in the last hour, but in fact the documentation is very good, if you don't happen to give up on the first sentence.
I looked at the documentation style of all the methods and will propose a minor change to the concatenate docs first sentence. highlighting the words dataset and data array, as in the other methods descriptions.

Thanks @PLSeuJ let us know if you need any help with making that change

thanks @dcherian this first time took me a bit, but I think it worked out. It's a very minor change though. If i find the time, I will definitely come back to this.


draw model on center of line at any angle

How to draw model on center of line at any angle.
It draws above the line.

Code I am trying is as below :
`
const totalDistance = makerjs.measure.pointDistance(line.origin, line.end);
const halfDistance = totalDistance / 2 - moduleWidth / 2;
const angle = makerjs.angle.ofLineInDegrees(line);
makerjs.model.rotate(module, angle);
const _angle = makerjs.angle.ofPointInRadians(line.origin, line.end);
const x = line.origin[0] + halfDistance * Math.cos(_angle);
const y = line.origin[1] + halfDistance * Math.sin(_angle);

module.origin = [x, y];

`
Expectation is I am trying to make something below. Any idea what should be the correct approch?

This can be wrong or not the case, but I feel it's worth sharing.
When you try to draw two shapes that occupy the same space, one of them gets pushed. I suggest doing this:
var mainModel = { models: {}, paths: {} }; mainModel[0] = line; mainModel[1] = module; // calculate the position x and y module.origin = [x, y]

@MontassarAbdelhedi  It may works when you have no angle specified or simple case, Question is on when you have line with any angle.

First you'll want to check the origin of the model you're rotating. Some models, like Polygon and Star are built around an origin at the center of their paths:

Others like Rectangle and Square are built around an origin in the lower left:

If your model is symmetrical, you can convert between these modes with zeroing and centering. If your model is not symmetric or you want an origin elswhere, you can try what @MontassarAbdelhedi suggested - "pushing" or "wrapping" your model inside another model. For example, with a Square with a side of 100:
//this will have origin at lower left
var square = new makerjs.models.Square(100);
square.paths['center hole'] = new makerjs.paths.Circle([50,50], 2);

//offset the square to its center
square.origin = [-50, -50];

//rotate
makerjs.model.rotate(square, 45);

//can't set square's origin! - it needs to stay at [-50, -50] because that's where its children are referencing from.
//square.origin = [20, 0];	//bug

//so we "push" or "wrap" or "encapsulate" the rotated square into a model that we can move freely
var newModel = {
  models: [ square ],

  //now we can move to the exact location we want
  origin: [20, 0]
};

Latest version childrenOnPath  fixed under #549


add tests for fix for RT #126740

Added tests for RT #126742 and RT #130477 (which were fixed with additions to rakudo PR #1291.


next is not a function in src/utils/catchAsync

@hagopj13  TypeError: next is not a function
at /home/.../src/utils/catchAsync.js:2:54
at processTicksAndRejections (internal/process/task_queues.js:97:5)

@AJONCODE I cannot reproduce your error, can you please provide more details? Which node version are you using?
I ran the app with different node versions and it worked for me in all environments (production, dev, testing).

It happens when I have an error occurring in controller module
On Thu, 20 Aug 2020 at 11:06 PM, Hagop Jamkojian<EMAIL_ADDRESS>wrote:

@AJONCODE https://github.com/AJONCODE I cannot reproduce your error,
can you please provide more details? Which node version are you using?
I ran the app with different node versions and it worked for me in all
environments (production, dev, testing).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/hagopj13/node-express-mongoose-boilerplate/issues/23#issuecomment-677802131,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGNURLYOBG2KCMBMGFEBOGTSBVNIXANCNFSM4QGMM2OA
.

It happens when I have an error occurring in controller module

@AJONCODE I still can't reproduce your error. I introduced different types of errors in one of the controllers and what I get back is always a properly formed error response, and not what you are getting.
Make sure you have installed all the dependencies.
If you are getting this error because of some changes you did in the code, kindly create a fork and do the same changes there, and I can have a look.
If you are getting this error without having done any changes, then give me a step by step guide on how to reproduce it (what type of request, to which endpoint, what are you giving as body, etc.)

I have just created a controller
`const registerAdmin = catchAsync(async () => {
const user = await userService.createAdmin({
...config.administrator,
referral_code: new mongoose.mongo.ObjectId(),
account_verification: {
is_verified: true,
},
});
if (user.message === 'New administrator registered successfully') {
// add a referrer collection for the user
await referrerService.addReferrer({ referrer: user._id }); // no _id present
}
if (user.success) {
return true;
}
});`
Focus on comment "no _id present", the error was because of this. But catchAsync throws TypeError: next is not a function error.

@hagopj13 Is linking active controller to a route must. I guess the error was because of this. I was using controller independent of route. I guess I should have used service instead of controller for this. Is that right?

@AJONCODE yes linking the controller to a route is a must, otherwise your controller will not get the necessary parameters (req, res, next) and hence catchAsync will not have access to next. There you have it, this is why you were getting an error.
You should use the controllers only for routes, and use services for all other things.


Contributing

Purpose
Add guidelines for contributing to the repo
Changes
Adds CONTRIBUTING.md, which should be displayed whenever someone makes an issue or pull request.
Side-effects
More organized pull requests!

Thank you!


Implement GitHub actions for GAMBLR installation and build

This PR implements the GitHub actions to install and build GAMBLR package on every PR :partying_face: directly addressing the issue #168. In the process of preparation for this PR I have also addressed issues #169, #170, #171, and #174

All checks passed! With the latest nloptr version!
This is ready to go!

Thanks Adam!


[FEATURE REQ] Remove System.Memory.Data

Library name
Azure.Core
Please describe the feature.
Follow up for https://github.com/Azure/azure-sdk-for-net/pull/39478#discussion_r1400969855

This is addressed by #39998.   I'm going to close this out, as I don't think we need a separate tracking issue.


ci build script randomly exits when testing appsody build

Describe the bug
When running the build script, sometimes during the test phase, specifically appsody build the build will freeze and become unresponsive. This causes Travis builds to "error out" forcing a restart. (Testing locally you can Ctrl-C to exit and re run)
To Reproduce
Steps to reproduce the behavior:

Run the build script locally
See:  > appsody build freeze
On Travis: See the build error out.
Locally: See the build freeze forcing you to Ctrl-C and restart.

Expected behavior
Build script should finish successfully
Actual behaviour
Build script freezes locally and errors out on Travis due to no response

This should be resolved now as we have re-implemented a different way to build and test stacks


Support RTL mode

In some languages, websites are arranged from right to left, eg. Arabic, Uyghur...
So, supporting RTL mode feature for i18n is really useful. My thought as followed:

work like dark mode by class

we can add rtl keyword to the class list of html tag to activate RTL mode

replace the content of some utilities in RTL mode

for example, we can replace flex-row as flex-row-reverse

provide rtl: variant for manual control

I think we do support RTL mode, but it’s not written in the documentation.

I think we do support RTL mode, but it’s not written in the documentation.

I have not found this in the document, and open this issue☹

https://twitter.com/windi_css/status/1373999256814587908
Feel free to open a PR to complete the docs. Thanks!


Improve opt-out link

Make the link for the opt-out also work for non-Chromium-base browsers

OK, tested and working :+1:


getfastq download options

@Hego-CCTB Could you remind me why those options are set to no by default? I thought they should be yes by default and users should deactivate it if they have a particular reason to avoid some of them.
  --ncbi yes|no         default=no: Download SRA files using wget from NCBI cloud, if available.
  --aws yes|no          default=no: Download SRA files from Amazon Cloud (AWS), if available.
  --gcp yes|no          default=no: Download SRA files from Google Cloud (GCP), if available.

No particular good reason. When I implemented those options I treated them as an alternative to the back then only option. I preserved the original behaviour by making these optional. But yeah, I exclusively use these options myself anyways, so might as well turn them on by default. Will come with the next update.


Unexpected mime type for zip files

Code
(new \Mimey\MimeTypes())->getMimeType('zip');

Expected result
'application/zip'

Actual result
'application/x-gzip'

This wrong mime type means that e.g. Firefox and Chrome handle the file as gzip instead of a zip file and actively renaming the file.
The problem is probably the order in the default configuration:
https://github.com/ralouphie/mimey/blob/develop/mime.types.php#L34
Is there a reason for this order or is this a bug?

That's a good catch I think and it should be fixed :).
The zip and gzip are different MIME types.

@ralouphie, do you have any idea about this :)?

This repository is meant to be a wrapper for httpd's mime types mapping. If a different mapping is needed, please leverage the custom mappings capability described in the documentation.

@ralouphie, thanks for your reply.
According to the httpd mime type doc, it seems that this zip is different mime types.
They're as follows:

application/zip					zip
# application/gzip

Do you mean the # application/gzip is customized mapping?
If yes, and I think you're right. And this issue will be closed :).

Hi @ralouphie
According to the doc @peter279k mentioned, mime type for zip extension should be application/zip
zip files are widespread and are important part of web services, so I see it as important to resolve.
Would you let us know what's needed to help progress resolution of this issue, a PR? :)

It seems that the repository owner doesn't concern this PR for about whiles.
And I think it should add some collaborator on this repository.


Unclear on how you would get a JwtSecurityToken, Securely

Suppose I have a public key and I received a jwt string randomly from the internet
(for the sake of argument you want to write the function JwtSecurityToken GetVerifiedJwt(string jwtString, RsaSecurityKey securityKey) and to throw an exception on an invalid jwt  or "alg": "none")
I know you can write new JwtSecurityToken(jwtString) but that wouldn't detect whether the jwt was falsified.

@DSUK I owe you a sample.

@DSUK I am moving all questions about documentation into #601.
Could you chime in on what your priorities would be?


Improve service denied error message

If you click to connect to Drive and then you deny the permission, a JSON error message is displayed in the browser: {"errors":{"imports":"validate_callback: access_denied @ gdrive"}}
cc @CartoDB/backend

data services?

Probably is Editor because the message itself is ok, we'll probably need to handle via JS or serverside that error and show it more clearly, no?
(I'm unsure of ownership of this things)

But in this case, JS don't know what is happening with the dialog, because the error is not arriving there. Could we do something with that?

There has been no activity on this issue for more several months. We are closing it. If you think this still needs to be addressed please open a new issue.


Bump oneui version number

Looks good to me. I can't actually review this because I don't have permissions, but I can give you this +1?
+1


AccountLocks: Remove per-transaction allocation

Problem

get_account_locks performs allocations to store the account locks for each transaction

Summary of Changes
Follow-up to #2291

iterate over keys in account keys using is_writable to distinguish

Fixes #

Comparing against master 21fbec7eae0971312a8a209d8936924a7d3dd519
For low numbers of accounts this is a significant improvement.
For higher numbers of accounts this is less of an improvement - likely because the bottleneck for those is the poor performance in has_duplicates; that will be addressed by a separate PR.

Benchmark Results
bench_lock_accounts/batch_size_1_locks_count_2
                        time:   [104.48 µs 104.62 µs 104.79 µs]
                        thrpt:  [9.7723 Melem/s 9.7876 Melem/s 9.8011 Melem/s]
                 change:
                        time:   [-63.482% -63.302% -63.101%] (p = 0.00 < 0.05)
                        thrpt:  [+171.01% +172.49% +173.84%]
                        Performance has improved.
Found 8 outliers among 100 measurements (8.00%)
  1 (1.00%) low severe
  3 (3.00%) high mild
  4 (4.00%) high severe
bench_lock_accounts/batch_size_1_locks_count_2_read_conflicts
                        time:   [105.33 µs 105.84 µs 106.52 µs]
                        thrpt:  [9.6136 Melem/s 9.6746 Melem/s 9.7214 Melem/s]
                 change:
                        time:   [-62.130% -61.608% -61.002%] (p = 0.00 < 0.05)
                        thrpt:  [+156.42% +160.47% +164.06%]
                        Performance has improved.
Found 9 outliers among 100 measurements (9.00%)
  6 (6.00%) high mild
  3 (3.00%) high severe
bench_lock_accounts/batch_size_1_locks_count_64
                        time:   [2.9694 ms 2.9782 ms 2.9874 ms]
                        thrpt:  [342.77 Kelem/s 343.84 Kelem/s 344.85 Kelem/s]
                 change:
                        time:   [-2.9409% -2.6065% -2.2464%] (p = 0.00 < 0.05)
                        thrpt:  [+2.2980% +2.6762% +3.0301%]
                        Performance has improved.
Found 1 outliers among 100 measurements (1.00%)
  1 (1.00%) high mild
bench_lock_accounts/batch_size_1_locks_count_64_read_conflicts
                        time:   [2.6679 ms 2.6707 ms 2.6740 ms]
                        thrpt:  [382.95 Kelem/s 383.42 Kelem/s 383.83 Kelem/s]
                 change:
                        time:   [-4.7773% -4.6048% -4.4298%] (p = 0.00 < 0.05)
                        thrpt:  [+4.6351% +4.8270% +5.0170%]
                        Performance has improved.
Found 5 outliers among 100 measurements (5.00%)
  4 (4.00%) high mild
  1 (1.00%) high severe
bench_lock_accounts/batch_size_32_locks_count_2
                        time:   [124.35 µs 124.54 µs 124.73 µs]
                        thrpt:  [8.2094 Melem/s 8.2226 Melem/s 8.2348 Melem/s]
                 change:
                        time:   [-52.905% -52.796% -52.688%] (p = 0.00 < 0.05)
                        thrpt:  [+111.36% +111.84% +112.34%]
                        Performance has improved.
Found 4 outliers among 100 measurements (4.00%)
  1 (1.00%) low mild
  3 (3.00%) high mild
bench_lock_accounts/batch_size_32_locks_count_2_read_conflicts
                        time:   [89.107 µs 89.297 µs 89.498 µs]
                        thrpt:  [11.442 Melem/s 11.467 Melem/s 11.492 Melem/s]
                 change:
                        time:   [-62.153% -62.016% -61.873%] (p = 0.00 < 0.05)
                        thrpt:  [+162.28% +163.27% +164.22%]
                        Performance has improved.
Found 7 outliers among 100 measurements (7.00%)
  5 (5.00%) high mild
  2 (2.00%) high severe
bench_lock_accounts/batch_size_32_locks_count_64
                        time:   [4.6396 ms 4.6621 ms 4.6842 ms]
                        thrpt:  [218.61 Kelem/s 219.64 Kelem/s 220.71 Kelem/s]
                 change:
                        time:   [-12.069% -11.485% -10.922%] (p = 0.00 < 0.05)
                        thrpt:  [+12.261% +12.975% +13.725%]
                        Performance has improved.
bench_lock_accounts/batch_size_32_locks_count_64_read_conflicts
                        time:   [3.2557 ms 3.2623 ms 3.2688 ms]
                        thrpt:  [313.27 Kelem/s 313.89 Kelem/s 314.52 Kelem/s]
                 change:
                        time:   [-9.7076% -9.3523% -9.0042%] (p = 0.00 < 0.05)
                        thrpt:  [+9.8952% +10.317% +10.751%]
                        Performance has improved.
bench_lock_accounts/batch_size_64_locks_count_2
                        time:   [120.47 µs 120.61 µs 120.74 µs]
                        thrpt:  [8.4809 Melem/s 8.4902 Melem/s 8.4999 Melem/s]
                 change:
                        time:   [-53.679% -53.572% -53.466%] (p = 0.00 < 0.05)
                        thrpt:  [+114.89% +115.39% +115.88%]
                        Performance has improved.
Found 16 outliers among 100 measurements (16.00%)
  8 (8.00%) low mild
  5 (5.00%) high mild
  3 (3.00%) high severe
bench_lock_accounts/batch_size_64_locks_count_2_read_conflicts
                        time:   [100.16 µs 100.29 µs 100.44 µs]
                        thrpt:  [10.195 Melem/s 10.210 Melem/s 10.223 Melem/s]
                 change:
                        time:   [-56.819% -56.641% -56.457%] (p = 0.00 < 0.05)
                        thrpt:  [+129.66% +130.63% +131.58%]
                        Performance has improved.
Found 16 outliers among 100 measurements (16.00%)
  3 (3.00%) high mild
  13 (13.00%) high severe
bench_lock_accounts/batch_size_64_locks_count_64
                        time:   [4.8995 ms 4.9146 ms 4.9341 ms]
                        thrpt:  [207.54 Kelem/s 208.36 Kelem/s 209.00 Kelem/s]
                 change:
                        time:   [-5.6832% -5.3286% -4.9527%] (p = 0.00 < 0.05)
                        thrpt:  [+5.2108% +5.6285% +6.0256%]
                        Performance has improved.
Found 9 outliers among 100 measurements (9.00%)
  5 (5.00%) high mild
  4 (4.00%) high severe
bench_lock_accounts/batch_size_64_locks_count_64_read_conflicts
                        time:   [3.5609 ms 3.5665 ms 3.5725 ms]
                        thrpt:  [286.64 Kelem/s 287.12 Kelem/s 287.57 Kelem/s]
                 change:
                        time:   [+3.1218% +3.3252% +3.5397%] (p = 0.00 < 0.05)
                        thrpt:  [-3.4187% -3.2182% -3.0273%]
                        Performance has regressed.
Found 7 outliers among 100 measurements (7.00%)
  7 (7.00%) high mild


test: Block HubbleObserveFollow until ready

One of our flaky tests has been failing because we can't find the expected flow event in the hubble observe output. Looking at the logs revealed that the expected flow event was emitted before our hubble observe command running in the background was ready.
This happened because we start hubble observe in the background and directly make the test request without waiting [1]. This pull request changes that to ensure HubbleObserveFollow (which starts hubble observe in the background) waits for Hubble to be ready before returning.
To that end, debug mode is enabled. In that mode, hubble observe will the following debug message once connected to the server:
time="2023-04-21T16:29:45Z" level=debug msg="Sending GetFlows request" request="follow:true whitelist:{event_type:{type:129}}"

We can wait until we see that message to exit from HubbleObserveFollow. After that we'll simply call WaitUntilMatchFilterLineTimeout as usual, but we'll be sure Hubble was ready when we made the test query.
This works because the debug message is sent over stderr, whereas WaitUntilMatchFilterLineTimeout reads from stdout. WaitUntilMatchTimeout, the helper function we use to block in HubbleObserveFollow, searches for a match in both stdout and stderr.
1 - In that particular flaky test, we were actually waiting 2s, but that wasn't always enough.
Fixes: https://github.com/cilium/cilium/pull/11232.
Fixes: https://github.com/cilium/cilium/issues/22056.

I fixed both nits without rebasing in the last force push. The end-to-end ginkgo tests were green before that and I don't think we need to rerun them given I only touched an error string and a comment. Merging once linters are passing.


Autocompletion not working

2020-04-15T01:20:16.748 ERROR (pid:51762) [completion-complete] - Complete error: tailwindCSS TypeError: Cannot read property 'newText' of undefined
    at Languages.convertVimCompleteItem (/Users/myuser/.config/nvim/plugged/coc.nvim/build/index.js:51844:58)
    at /Users/myuser/.config/nvim/plugged/coc.nvim/build/index.js:51629:37
    at Array.map (<anonymous>)
    at Object.doComplete (/Users/myuser/.config/nvim/plugged/coc.nvim/build/index.js:51628:43)

I just noticed that the autocompletion is not working, I updated tailwindcss version from 1.1.3 to 1.2.0 a few weeks ago but not sure if that's related. I'm new to coc so I'm not sure what else to do to give you more information, please let me know how I can help.
Here's my tailwindcss config if it helps: https://github.com/apipath/ppoker-graphql-react/blob/master/tailwind.config.js
Thanks for your awesome work!!

coc's issue, update your coc