For Loop issue with cutting commands

For loop seems to be causing SIM3D to interpret the current command as NON-CUTTING, because the current context shows the command as a FORLOOPGCODE, instead of the expected G01 on example.

Fixed by using a newly added property IsInCuttingMode in MachineBase.SimControl, which checks if the current context represents a for loop, and then checks if the loop's current context is of type G01.


added filter functionality and integrated get patients api

added filter functionality

integrated get patients api

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


chore(dependencies): upgrade autoprefixer

Related Ticket(s)
Refs #1644.
Refs #3614.
Description
This change adds autoprefixer to devDependenvies, with its latest version, as our WebPack configurations explicitly imports it.
Such change seems to ensure that autoprefixer covers iOS Safari for text-size-adjust.
Changelog
Changed

Adds autoprefixer to devDependenvies, with its latest version, as our WebPack configurations explicitly imports it.

@asudoh thanks for this fix! It looks like the offline mirror is missing the file for autoprefixer.

Good catch @jeffchew - Fixed.

Thanks @jeffchew - I think the other PR needs to be rectified (removing prefixed style rules), but once it's done it'll be good to go.


Show log info method added in datasource

Show log info method added in datasource

Log info method to print the required information


SchedulerSuite.TestJobStatus: timed out waiting for job events

FAIL: test_scheduler.go:247: SchedulerSuite.TestJobStatus

++ 18:56:20.805 created app antelope-gratify-saugeen (3aa2e6b8b9a84f5388d10c333d6b2b9d)
++ 18:56:20.955 waiting for job events: map[crasher:map[up:1] :map[up:1] printer:map[up:1]]
++ 18:56:20.977 got job event:  ddde7ef5-6e3487c6ca0d4aefaf104a5a166504a3 starting
++ 18:56:21.017 got job event: printer 4e8f5e1a-a8ceeb0dae9f4169ac956eb84d816567 starting
++ 18:56:21.927 got job event: printer 4e8f5e1a-a8ceeb0dae9f4169ac956eb84d816567 up
++ 18:56:22.040 got job event: crasher ddde7ef5-81e2d19013324ff5bb03dc975c4b5022 starting
++ 18:56:22.067 got job event:  ddde7ef5-6e3487c6ca0d4aefaf104a5a166504a3 up
++ 18:56:22.767 got job event: crasher ddde7ef5-81e2d19013324ff5bb03dc975c4b5022 up
++ 18:56:22.774 crasherjob started with ID ddde7ef5-81e2d19013324ff5bb03dc975c4b5022
++ 18:56:22.774 printerjob started with ID 4e8f5e1a-a8ceeb0dae9f4169ac956eb84d816567
++ 18:56:22.774 job started with ID ddde7ef5-6e3487c6ca0d4aefaf104a5a166504a3
++ 18:56:22.774 stopping job 4e8f5e1a-a8ceeb0dae9f4169ac956eb84d816567
++ 18:56:22.845 waiting for job events: map[printer:map[up:1 down:1]]
++ 18:56:22.845 got job event: printer 4e8f5e1a-a8ceeb0dae9f4169ac956eb84d816567 down
++ 18:56:22.845 got job event: printer c87d9865-111bceef3e87479da646baac378b658c starting
++ 18:56:24.007 got job event: printer c87d9865-111bceef3e87479da646baac378b658c up
++ 18:56:24.015 stopping job ddde7ef5-6e3487c6ca0d4aefaf104a5a166504a3
++ 18:56:24.100 waiting for job events: map[:map[down:1]]
++ 18:56:24.100 got job event:  ddde7ef5-6e3487c6ca0d4aefaf104a5a166504a3 down
++ 18:56:24.108 stopping job ddde7ef5-81e2d19013324ff5bb03dc975c4b5022
++ 18:56:24.820 waiting for job events: map[crasher:map[down:1 up:1]]
++ 18:56:24.820 got job event: crasher ddde7ef5-81e2d19013324ff5bb03dc975c4b5022 crashed
test_scheduler.go:303:
    waitForJobEvents(t, stream, events, jobEvents{"crasher": {"down": 1, "up": 1}})
test_scheduler.go:74:
    t.Fatal("timed out waiting for job events: ", expected)
... Error: timed out waiting for job events: map[crasher:map[down:1 up:1]]

++ 18:57:30.583 Booting 1 VM
++ 18:57:30.630 Starting flynn-host on <IP_ADDRESS> [id: f00531ab]
++ 18:57:39.060 Booting 1 VM
++ 18:57:39.105 Starting flynn-host on <IP_ADDRESS> [id: b652fea8]
++ 18:57:48.052 removing host f00531ab
++ 18:57:58.263 removing host b652fea8

https://ci.flynn.io/builds/20150312184659-c9263595

Fixed in c54a751.


[Set Audio Device] ...Select favorite does not work (Ventura OS).

Extension – Set Audio Device
Author: @benvp

Description
Steps To Reproduce

The current behavior
The expected behavior
Raycast version
Version: 1.42.2

Can confirm. Even the Set Output Device action itself does not list any devices when run.

Alas the favourite component still doesn’t work

@tsugur Yes, when "Airplay Devices" are disabled, the action "Set Output Device" displays available devices but the Set "Output Device to Favourite" still does not work

Not fixed yet


Portable test suite

👍

You should be able to make a release now

Let's give it a shot! 💃

SUCCESS!! https://github.com/caiogondim/logdown.js/commit/b14f0a3aa10561a14e3cd7545e2da09c2ae56a79


Task - Edit Nomenclature - Author by source

If I add Author by Source, the created citation should be automatically checked as original citation.

Agreed.
On Tue, Sep 25, 2018, 5:23 PM Dmitry Dmitriev<EMAIL_ADDRESS>wrote:

If I add Author by Source, the created citation should be automatically
checked as original citation.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/SpeciesFileGroup/taxonworks/issues/523, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AABYSDiK9_XiADOOalzKZ9-MGH0qX_dnks5ueqzSgaJpZM4W5lJ6
.

Already done it


Filter out deprecated feeds from search

Describe the problem
It's confusing to see deprecated feeds in the search result.
Proposed solution
Filter out deprecated feeds from the search page. Users can still navigate to the deprecated feed if it's linked or they search it on their own.
Alternatives you've considered
No response
Additional context
No response

Add a parameter with the list of status we want.

@emmambd currently I don't plan to add a UI element that would let a user ask for deprecated feeds. So there will be no way to list them in the UI, for now. Is this acceptable?

@jcpitre Yes! We want to filter them out for now, no option to select them.

Since @jcpitre, would it make sense for someone else to take on finishing this? cc @davidgamez

Since @jcpitre, would it make sense for someone else to take on finishing this? cc @davidgamez

The PR will be ready for review soon ;-)


update to go-github v53

Update to go-github v53

nevermind that commit ba894f52c51e3ce42a2b9fe47ffb1291d879460c is in pr #1335 will be merged in rebase to main after it's merged

/lgtm


Blob (block blobs and append blobs only) - footnote 3 confirmation

"The Archive tier is available at level of an individual blob only, not at the storage account level. Only block blobs and append blobs can be archived. For more information, see Azure Blob storage: Hot, Cool, and Archive storage tiers."
I am using server-level auditing, the account kind is BlobStorage, on the Standard access tier.  In the portal, I select Change tier for an append blob audit log under sqldbauditlogs and receive a message popup: Changing the access tier is only allowed on block blobs.
As the observed behavior appears contradict what in the doc, are you able to illuminate on this?

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

ID: 2665929f-ca2f-897a-7de3-6a9d9fecd3d2
Version Independent ID: e7604774-9953-2942-19eb-7956f437cdc8
Content: Storage account overview - Azure Storage
Content Source: articles/storage/common/storage-account-overview.md
Service: storage
Sub-service: common
GitHub Login: @tamram
Microsoft Alias: tamram

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

@langstonj  Your feedback has been shared with the content owner for further review.

#reassign: @mhopkins-msft
@mhopkins-msft , could you address this one?

@langstonj
Sorry that seems to be an error in the storage account overview documentation.
Only block blobs support blob level tiering and archive, which was correctly called out in the Access Tiers doc page.
I've submitted a fix for the documentation.
#please-close

@langstonj Thanks for your feedback. You're correct, append blobs are not supported in archive storage at this time. I've updated the documentation, it will be published today.
Thanks,
~ Mark
#please-close


Rework reconcile loop for workspace controller

What does this PR do?
Reworks the controller reconcile loop pretty thoroughly. For reviewing this PR, the diff will be nearly useless -- apart from a few copied sections (e.g. webhooks, some specs), the state of this PR represents a ground-up rewrite of most of the controller code.
A more detailed commit history is contained in repository https://github.com/amisevsk/che-workspace-operator-rework; see the design doc for a high-level overview.
The vast majority of the additions are because we store PodAdditions (a list of elements to be added to the workspace deployment, e.g. pods) in the status of subcomponents. This results in their full spec being included in the relevant CRDs. To make even looking at this PR slightly less daunting:
$ git diff master --stat pkg/
82 files changed, 4405 insertions(+), 4194 deletions(-)

What issues does this PR fix or reference?
https://github.com/eclipse/che/issues/16494
https://github.com/eclipse/che/issues/15786
supercedes PR https://github.com/che-incubator/che-workspace-operator/pull/22
Is it tested? How?
Tested java-mysql and cloud shell with and without webhooks, oauth, on crc.

Tried this out and ran into a few things:

When running oc get workspace the url given doesn't have http infront anymore, not a big issue I just like being able to click on it in the terminal

NAME          WORKSPACE ID                PHASE      URL
cloud-shell   workspacee0cc88538f2c4e58   Starting   workspacee0cc88538f2c4e58-cloud-shell-proxy-4400.apps-crc.testing

When testing with webhooks enabled and routing as openshift-oauth I'm running into The only workspace creator has exec access. For some reason org.eclipse.che.workspace/creator is set to ''. I feel like I've run into that issue on master before though

Tested with webhooks disabled and everything was working 👍
In terms of the code, it LGTM but I don't have that much experience in go/kubernetes/the operator world yet. I'm going to take another look through it all again tomorrow though

When testing with webhooks enabled and routing as openshift-oauth I'm running into The only workspace creator has exec access. For some reason org.eclipse.che.workspace/creator is set to ''. I feel like I've run into that issue on master before though

I've seen this before, but I think usually it's due to starting with webhooks disabled and then enabling them (then the webhook would add the annotation but not have a user for it)? I'm not sure exactly what causes it, but could not reproduce with a fresh deployment
I've added https(s) back into the workspace URL :) -- I didn't add it initially because clicking the url never works in my terminal for some reason.
While looking into including http(s) in the URL status, I came across a few issues:

We weren't listing endpoints in the runtime annotation if:

The endpoint was for a plugin
The plugin had an alias
The plugin alias didn't match the container name for that plugin

This means the problem basically only manifested for cloud-shell, where it doesn't matter; in the java-mysql workspace theia is aliased to theia-ide.
The solution for now is to disable aliases for Che plugins and editors; from checking the full Che server, I don't see how aliases are used currently. We might still run into issues with component name <-> container name matching, but I don't have a good solution in mind. Suggestions here are more than welcome.

We currently still rely on ingress.global.domain for correctly setting hostnames, even when running on OpenShift where it's not necessary. I have no idea how I didn't have this issue during testing, but for now it's required to set ingress.global.domain manually (e.g. app-crc.testing for crc). I'll look into fixing this more generally, but the issue is that even with openshift-oauth routing we create ingresses.

The makefile I merged had some minor issues, I've opened PR to address those https://github.com/che-incubator/che-workspace-operator/pull/37 (they're largely from rebasing over #35). The changes are duplicated to this PR but will be removed once #37 is merged.

@sleshchenko Commit 22ab54d  should fix your issue with command -v kubectl in the makefile, please double check.

Tested again briefly on crc, and I suspect there may be a regression somewhere. With the java-mysql workspace I'm getting frequent issues opening a terminal.

Tested again, and the issues seem to be related to crc being somewhat unresponsive. Cloud shell works with both basic routing and OAuth + webhooks.
I'm merging this PR to unblock things; we can address edge cases in separate issues.

@sleshchenko I missed your last comments before merging; they are addressed in PR https://github.com/che-incubator/che-workspace-operator/pull/46


minor fixes

well, for the db server - it was localhost because I am using SQL Server Developer edition. But it's OK to have Express in configs by default


Increment minor version and update localized resource strings

Increments the minor version from 0.3.0 to 0.4.0.
Updates localized resource strings

Microsoft Reviewers: Open in CodeFlow

You probably know this, but just FYI the locale strings won't be updated for your latest changes yet


Tolerations parameter type is invalid

With added schema validation deployment fails when having tolerations configured.
Used config:
...
tolerations:
  # allow scheduling on any node
  - operator: Exists
...

Error message:
Helm upgrade failed: values don't meet the specifications of the schema(s) in the following chart(s):
starboard-exporter:
- tolerations: Invalid type. Expected: object, given: array

Whoops, sorry about that. Merged https://github.com/giantswarm/starboard-exporter/pull/158, will get a release out shortly


is CPU option available ?

Dear friends,
Is there any way to test in CPU ? We have not GPU: but plenty of cpu cores.
We would like to test and use it at CPU
Is it possible? and how?
thanks

any help ?

Allowing for a CPU only option would be convent for the tests that I am currently doing: we are trying to see if our model can be performant on a system without GPUs.

@tweiand Hello, have you been realized without GPUs？ Thank you!


Fix bug "error: [Errno 24] Too many open files".

Fix bug "error: [Errno 24] Too many open files".
Open only one memmap for all segmengts.
issue #515

Hello @samuelgarcia! Thanks for submitting the PR.

In the file neo/rawio/winwcprawio.py, following are the PEP8 issues :

Line 37:1: W293 blank line contains whitespace
Line 78:1: W293 blank line contains whitespace
Line 140:9: E741 ambiguous variable name 'l'

Coverage decreased (-0.01%) to 48.878% when pulling 0e31596401750e49bf18665513a0eb3bd6ab6237 on samuelgarcia:fix_winwcp_515 into e70c6f5e62cdfc2f79f4f337c449032ffa1f77c7 on NeuralEnsemble:master.

Coverage decreased (-0.01%) to 48.878% when pulling 0e31596401750e49bf18665513a0eb3bd6ab6237 on samuelgarcia:fix_winwcp_515 into e70c6f5e62cdfc2f79f4f337c449032ffa1f77c7 on NeuralEnsemble:master.

Coverage decreased (-0.01%) to 48.878% when pulling 0e31596401750e49bf18665513a0eb3bd6ab6237 on samuelgarcia:fix_winwcp_515 into e70c6f5e62cdfc2f79f4f337c449032ffa1f77c7 on NeuralEnsemble:master.


GPL license?

It looks like utf8.js is licensed under MIT, but I see in your component.json file, the license is listed as "MIT/GPL". Can you clarify the intended licensing, please? Thanks!

I'm very curious as well, @mathiasbynens  Is there a particular reason for that?
Matthias Is there any chance we can change the license to just MIT?


Game crashs with serene seasons

According to your testing, does the issue occur with ONLY Immersive Portals mod?
Yes
Forge Version
40.1.74
ImmersivePortals Version
immersive-portals-1.4.9-rc3-mc1.18.2-forge.jar
Latest Log
https://gist.github.com/agentwolf88/1ce26c86ce5236f71fbf7db449fcadfc
Crash Report (if applicable)
https://gist.github.com/agentwolf88/57a6ff875e34c50bad32bf2c975e6923
Steps to Reproduce
Use serene seasons + immersive portals and start the game.
What You Expected

What Happened Instead
The game crashs with serene seasons.
Additional Details
No response
Please Read and Confirm The Following

[X] I have confirmed this bug can be replicated without the use of Optifine.
[X] I have confirmed the details provided in this report are concise as possible and does not contained vague information (ie. Versions are properly recorded, answers to questions are clear).
[X] I have confirmed this does not happen on the fabric version (If it does please report to here instead).
[X] I have confirmed this issue is unique and has not been reported already.

Fix cf57371: Game crashs with serene seasons
This fixes the crash. Note, the mods are still incompatible. There are several rendering issues. Enabling compatibilityRenderMode in imm_ptl-common.toml fixes most of them.
I'm not going to fix this further, as I'm not interested in debugging closed-source ARR mods. If you want further compat, tell them to provide compat on there side or to provide their source.

What do you mean by closed-source ARR? Serene Seasons have github page with Issues and Source

Yeah, but there GitHub does not contain any source.

What do you mean by closed-source ARR? Serene Seasons have github page with Issues and Source

Glitchfiend only has open source for TerraBlender iirc. The other mods have closed source, so you must ask for permission to get access

Only their API code, which on its own is completely useless.

Ah oof

They said in Discord that you can get access for it


docs: add documentation for deploying probot on cloudlfare workers

Feature Request
Is your feature request related to a problem? Please describe.
I don't know how to deploy Probot on Cloudflare workers and I'm for some guide, docs, etc...
Describe the solution you'd like
Please add good documentation that tells us how to deploy Probot on Cloudflare workers.
Teachability, Documentation, Adoption, Migration Strategy
I would request someone from the Probot team to add the documentation as soon as possible please.

I don't think Probot will be compatible with Cloudflare workers, but I did get it to work using the underlying Octokit a while ago, maybe give that a try first?
https://github.com/gr2m/cloudflare-worker-github-app-example

Thanks, it will do the work for me.


Update instruction to run single Cucumber feature

What does this PR change?
It updates the README_TESTING.md instruction as the current one will not work.

@Bischoff, @srbarrios : my understanding is that the splitting of features into the core/ and secondary/ subdirectories unexpectedly broke this Cucumber convention. From cucumber --help:
Usage: cucumber [options] [ [FILE|DIR|URL][:LINE[:LINE]*] ]+
[...]
    -r, --require LIBRARY|DIR        Require files before executing the features. If this
                                     option is not specified, all *.rb files that are
                                     siblings or below the features will be loaded auto-
                                     matically
[...]

Since *.feature files now live either under spacewalk/testsuite/features/core or spacewalk/testsuite/features/secondary instead of just spacewalk/testsuite/features this means that support and step_definition are not sibling directories any longer - thus an explicit -r flag is needed.
So the semi-philosophical question now is: do we accept to break the Cucumber convention here and require users to specify -r (thus this PR might be just accepted I guess) or are we considering other solutions, if any exist?

I vote to let the users know that need to use -r parameter. IMHO, it make sense to keep this organization.
I don't understand why cucumber framework don't recomend it as the default structure.

I vote to let the users know that need to use -r parameter. IMHO, it make sense to keep this organization.
I don't understand why cucumber framework don't recomend it as the default structure.

Without pipelines, it used to work without -r.
Don't ask me why pipelines makes it necessary.

So the semi-philosophical question now is: do we accept to break the Cucumber convention here and require users to specify -r (thus this PR might be just accepted I guess) or are we considering other solutions, if any exist?

I don't really think it is really a "convention". I think it was just not thought enough :troll: .
The solution is to document -r as you did here.

Merging not to feed the :trollface: !


Phase 2 of XLA int8 convolution on CUDA.

This is a breakdown of previous PR (https://github.com/tensorflow/tensorflow/pull/29158) per @timshen91 's suggestion.
This PR doesn't depend on (https://github.com/tensorflow/tensorflow/pull/30761).

Refactor RunCudnnConvImpl in cudnn_conv_runner.cc to dispatch CuDNN function calls based on input/output types in addition to convolution kind.
Modify dnn, cuda_dnn, and rocm_dnn to separate convolution output type from input/element type.
Add int8 ThenConvolveWithAlgorithm overloads to stream.h.

@timshen91 I have updated the PR to address your comments.  Please review and approve it again.

@yongfeng-nv Could you please resolve the conflicts? Thanks!

@yongfeng-nv  gentle ping to resolve the conflicts. Thanks!

I'm going to go ahead and close this PR, because it seems to have stalled. If you're still interested in pursing this (and responding to my comments), please feel free to reopen!

Since Phase 1 patch is not needed anymore, I'm re-opening this for committing into the repo.


Show more detail for UnprocessableEntity

Danger crashed and stopped CI Job because of Octokit::UnprocessableEntity when use github inline-comments.
This is maybe Danger's bug but the causes could not found because its error has no information.
So, it makes to show more detail for UnprocessableEntity, and not fail CI job.
See detail: https://github.com/danger/danger/issues/774

          1 Warning

:warning:
Tests were not updated

          1 Message

:book:
@litmon is not a contributor yet, would you like to join the Danger org?

  Generated by :no_entry_sign: Danger

@orta bundle exec rubocop lib has failed due to below. Should I fix this?
Offenses:

lib/danger/commands/init.rb:185:21: C: Prefer using YAML.safe_load over YAML.load.
      config = YAML.load(File.read(".travis.yml"))
                    ^^^^
lib/danger/commands/init.rb:198:21: C: Prefer using YAML.safe_load over YAML.load.
      config = YAML.load(File.read("circle.yml"))
                    ^^^^

Can you disable those checks please? (YAML.load is on Ruby 2.0, which we need to support)

Awesome - thanks! Will see if I can get #853 in and ship these two

Thanks - shipped!


Crosschain transfer did not arrive

I transferred BNB from my own Binance chain network wallet to my Binance smart chain network account and it never arrived
I reached out to Binance.com support they told me it is not their business but Binance.org's. I reached them whom told me that the transaction that i have sent is invalid.
I'm really starting to worry, can some of you guys help me by chance?
TxHash:
673DE15FAF50D397A755AE1D4AB8607E547B8F037DBE5EA50BFAF6CF5FE73DC4
TxHash On BSC:
0x5df9a0f1d9e1e6e947447a6e4ae636939a152efff0eea2064656a3272b2953ac
TxReceipt Status:
Success
Block Height:
215597861
TimeStamp:
1 day 5 hrs ago [Jan-14-2022 18:55:20]
Transaction Type:
Crosschain transfer out
Fee:
0.000075 BNB
Asset:
BNB
Memo:
From:
bnb16p4v7ptsledv6y2ngzma9sk55dceg4wzm0dcxt
Value:
1.89527815 BNB
Kind regards.

I transferred BNB from my own Binance chain network wallet to my Binance smart chain network account and it never arrived I reached out to Binance.com support they told me it is not their business but Binance.org's. I reached them whom told me that the transaction that i have sent is invalid.
I'm really starting to worry, can some of you guys help me by chance?
TxHash: 673DE15FAF50D397A755AE1D4AB8607E547B8F037DBE5EA50BFAF6CF5FE73DC4 TxHash On BSC: 0x5df9a0f1d9e1e6e947447a6e4ae636939a152efff0eea2064656a3272b2953ac TxReceipt Status: Success Block Height:
215597861
TimeStamp:
1 day 5 hrs ago [Jan-14-2022 18:55:20]
Transaction Type:
Crosschain transfer out
Fee:
0.000075 BNB
Asset:
BNB
Memo:
From: bnb16p4v7ptsledv6y2ngzma9sk55dceg4wzm0dcxt Value: 1.89527815 BNB
Kind regards.

we are so sorry for the late reply kindly contact there live chat support on whatsapp -  +19546679281


Implement Pseudoterminal.onDidChangeName

This PR fixes #114898

Sorry about the delay, have to push this again

Didn't make April unfortunately, I missed the auto merge CI failure.


Unanble to complie or open file in Ardunio IDE

Description
Unable to open file in ardunio ide. Considering you are using node MCU or esp, it would be great if you could have the ino files in the directory as well. How can i compile this in Ardunio IDE ?
Suggested Actions
Additional

This project is made for PlatformIO, not the Arduino IDE. Please use those:
https://github.com/timmo001/ESP8266-MQTT-JSON-Lights#software-setup


When i type quote like " or '. Memos type a second time quote.

Describe the bug
When i type quote like " or '. Memos type a second time quote.
In French it's a real problem for some word I type : "j'ai" "Aujourd'hui" for exemple
bug appear in 0.19.0
Steps to reproduce

Create a new memos
type ' or "
memos return : "" or ''

i would like memos return just " or '
Screenshots or additional context

I see this is closed and I found the toggle for the power editor as well, but there's no explanation for what all the power editor does. Does it only add the double " feature or does it do more? Could you please add this information to the usememos website? Thanks!


Ledger cannot verify

On Pera wallet, using Ledger, I get the error "In-App sdkError(error: Optional(Error Domain=go Code=1 "incorrect signature length expected 64, got 86" UserInfo=(NSlocalizedDescription=incorrect signature length expected 64, got 86})), when Pera attempts to verify the transaction with ledger.

We don't support using Pera. We only support Defly. Try using them instead.


(PUP-10379) Load bolt project into environment module list

If the keyword bolt_project is defined, the Bolt project should be
loaded at the head of the Puppet::Node::Environment#modules list of
modules. Bolt sets a temporary environment when running which it uses to
load modules on it's modulepath. We want to maintain this behavior and
also load the Bolt project directory configure by the user as a
standalone module. This ensures that any modules with the same name as
the Bolt project aren't loaded, so that the Bolt project can safely
shadow existing modules only when specified in Bolt. Additionally this
ensures that only the Bolt project is loaded as a module, not any
sibling directories. The keyword should only ever be set in Bolt
contexts and should not impact Puppet environments.

Well, third time's a charm :four_leaf_clover:

CLA signed by all contributors.

Ok sorry all the churn @joshcooper but this is our final iteration and is ready for review.

Awesome - is there anyone else we need review from? I think we're good on the Bolt side.

Sorry had a thought last night. What do you think about creating a tmp directory, linking the bolt project into the directory, and adding the directory to the front of the modulepath? Would that eliminate the need to make any puppet changes?

That seems a bit hard to debug if something goes wrong, and seems kind of like a hack...plus raises a lot of other questions like where do we make the tmpdir (/tmp? what if users don't have access. Current dir? What if it doesn't get cleaned up properly?). It seems like an indirect way to solve the problem. This is still technically experimental in Bolt, but I don't expect us to back it out and I think eventually we're going to want this.

That's fine, just wanted to raise/discuss the issue, and see what you thought. 👍

That seems a bit hard to debug if something goes wrong

Yeah that's actually a long standing issue in the existing code in puppet as there is no debug output when there are duplicate modules, or modules are skipped due to incorrect file permissions (puppetserver is running as puppet but module is not readable by that user), etc.

Yeah totally worth discussing! Thanks @joshcooper


Merge type and id fields in ClosureColor

These two fields were redundant with each other. Merging them lines things up a bit more nicely internally (closure data starts at offset 16 instead of 20).
Also added some convenience methods to reduce the need for verbose casts in client code.

LGTM

LGTM2


cannot detect communication between clusters when CollectiveAllReduceStrategy

tensorflow 1.12
CUDA 10
Ubuntu 16.04
Here is the part of code in the main script:
strategy=tf.contrib.distribute.MirroredStrategy(num_gpus=FLAGS.n_gpus)#devices=["/gpu:0", "/gpu:1"]
if FLAGS.strategy=="ps":
os.environ['TF_CONFIG'] = json.dumps({
'cluster': {
"worker": ["<IP_ADDRESS>:8081", "<IP_ADDRESS>:8082"],
"ps": ["<IP_ADDRESS>:8083"]
},
'task': {'type': FLAGS.type, 'index': FLAGS.index}
})
strategy = tf.contrib.distribute.ParameterServerStrategy(num_gpus_per_worker=FLAGS.n_gpus)
elif FLAGS.strategy=="collective":
os.environ['TF_CONFIG'] = json.dumps({
'cluster': {
"worker": ["<IP_ADDRESS>:8888","<IP_ADDRESS>:8888", "<IP_ADDRESS>:8888"],
},
'task': {'type': FLAGS.type, 'index': FLAGS.index}
})
strategy = tf.contrib.distribute.CollectiveAllReduceStrategy(num_gpus_per_worker=FLAGS.n_gpus)
run_config = tf.estimator.RunConfig(
model_dir=FLAGS.output_dir,
save_checkpoints_steps=FLAGS.save_checkpoints_steps,
train_distribute=strategy
)
estimator = tf.estimator.Estimator(
model_fn=model_fn,
config=run_config,
params={"batch_size": FLAGS.train_batch_size}
)
if FLAGS.do_train:
tf.logging.info("***** Running training *****")
tf.logging.info("  Batch size = %d", FLAGS.train_batch_size)
train_input_fn = input_fn_builder(
input_files=input_files,
max_seq_length=FLAGS.max_seq_length,
max_predictions_per_seq=FLAGS.max_predictions_per_seq,
is_training=True)
estimator.train(input_fn=train_input_fn, max_steps=FLAGS.num_train_steps)
I have three machines each has 3 gpus.
When I run CollectiveAllReduceStrategy, set type as 'worker' and index, the num of gpus for each worker as 3.I run the script independently on three clusters,  and use iftop -n to monitor the network, I did notice any communication between the clusters.
No errors, and from the log I did't know if it now runs the correctly way. Or just single machine 3 gpus, running independently.
Any suggestions or reference links?

This question is better asked on StackOverflow since it is not a bug or feature request. There is also a larger community that reads questions there.


Use LLMs to infer arg names of generated configurations

Tailcall uses the power of AI to auto-generate configurations. This feature is primarily used to identify the name of the type currently and works reliably well.
Before
type Query {
  f1(p1: Int): [T1] @http(path: "/users/{{args.p1}}")
}

After infer_type_names: true
type Query {
  f1(p1: Int): [User] @http(path: "/users{{args.p1}}")
}

After infer_arg_names: true
type Query {
  f1(id: Int): [User] @http(path: "/users{{args.id}}")
}

Technical Requirements

NOTE: Ensure the mustache templates are also updated.
Try to produce minimal diff and abstract out common parts out of infer_type_name.rs and the new infer_arg_name.rs.
Add snapshot tests for the generated configurations

Additional Links

Refer to the implementation here — https://github.com/tailcallhq/tailcall/blob/main/src/cli/llm/infer_type_name.rs

/bounty 65$

/attempt 2652

/attempt #2652

/attempt #2652


Bootstrap need CSS Variable Support

In my project i need to use CSS Variables. But if i use css variables with bootstrap it does not working as its using lighten, darken etc functions.
So please make a solution to give support css variables which is very essential for color configuration ( Ex. Dark mode, light mode for template )
Ref. https://github.com/twbs/bootstrap/issues/26596
Can't use primary color in css variable : https://codepen.io/themeix/pen/LYGvjdP

Duplicate https://github.com/twbs/bootstrap/issues/26596


Update core integration secret managment - Google Vertex

[ ] Code (c.f https://github.com/deepset-ai/haystack-core-integrations/pull/347)
[ ] Documentation
[ ] Tutorials
[ ] Cookbooks
[ ] Articles
[ ] Integration Tiles

No changes required as authentication is handled by Google Cloud Application Default Credentials .


Update automerge.yml

This Pull Request is auto-merged by approval of @trung1051996

This Pull Request is auto-merged by approval of @trung1051996


(Staging) Server side rendered feeds are reloaded on page load

Expected behavior
Server side rendered feeds should be visible on page load and shouldn't need to refresh the first time.
Actual behavior
When the page is loaded the feed is rendered however after a short amount of time it's reloaded unnecessarily.
How to reproduce

Visit a page that is rendered server side like https://staging.busy.org/trending
The feed content should be visible
Wait a moment and the feed should refresh

Recording
https://streamable.com/7rtjh

Seems like it only happens if you're logged out

Fixed in #2042.


New menu

This PR features many edits to the website and its examples in support of the new menu construct which uses flatter selectors.

CLA is valid!


Shadow-DOM support

First of all thank you for the wonderful library. We've been having trouble using Ribbon within Shadow-DOM. Tried loading Metro's JavaScript?CSS from within it and externally from light-dom, to no avail. Maybe it's the same case as with Bootstrap who simply for now does not support and there's no easy way around it?
The effect is that the Metro's elements remain raw and 'not-rendered'

Please, create an example with your problem on codepen.io

Olton, please, here's a sample:
https://codepen.io/rafalsk/pen/KKzREaX

https://codepen.io/olton/pen/bGpMyJE?editors=1010

But this is not a good way, because you will download CSS for each shadow.


Syncing latest changes from bundle-deps for odf-operator

PR containing the latest commits from bundle-deps branch

/retest

/override ci/prow/ci-bundle-odf-operator-bundle
/override ci/prow/odf-operator-e2e-aws
/override ci/prow/images
The same PR was passed on the main, We don't need to retest.


Add GitHub organization option

close #1

Codecov Report

Merging #4 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master     #4   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           2      2
  Lines          37     39    +2
=====================================
+ Hits           37     39    +2

Impacted Files
Coverage Δ

app/index.js
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


AMI ID not found

Hi!
I want to build a pipeline around this technology. However I can't find this AMI on AWS:
AMI ID: ami-b9be23ae
AMI NAME: waifu2x-server 20160807

Sorry, it seems to have deleted by my mistake.
I am planning to update this project, so it may take several months to release a new AMI.


Fix for magiclink bug

Issue #HAV-319
Signed-off-by: Christopher Mundus<EMAIL_ADDRESS>Description
Magiclink was failing due to a dead end url

Thank you!


Use a plain bool instead of an Option<bool>

All consumable values are wrapping in Some. This unnecessary wrapping is avoided with using just a bool.

This actually looks like a good approach for fixing some bugs.

The fix seems to be treating a symptom. I see that there could be other issues with the way the Object is modelled.
pub struct Object {
    pub label: Vec<String>,
    pub description: String,
    pub location: Option<usize>,
    pub destination: Option<usize>,
    pub item: bool,
    pub enemy: bool,
    pub health: Option<u64>,
    pub attack: Option<u64>,
    pub consumable: bool,
}

I don't suppose all the possible states represented by the struct are valid. Can it be the case that item and enemy both are true?


Builder.py throwing error

Traceback (most recent call last):
File "G:\R\RR\Loki-master\builder\builder.py", line 14, in
from lib.aes import CryptoAES
File "G:\R\RR\Loki-master\builder\lib\aes.py", line 5, in
from Crypto.Cipher import AES
ModuleNotFoundError: No module named 'Crypto'
I am getting this error, I've tried to manually do pip install pycrypto to see if that helped and it did not.

$> pip install pycryptodome

This solved my problem, thanks for the support!


Add psql instructions

Closes open-contracting/kingfisher-process#218

@duncandewhurst I added a tiny commit; to clarify which user to use (I know it says above but that may be read as being for something else and being explict doesn't hurt) and to add a ; to your example so your example matches the instruction :-)


stake-pool-cli: Default to simulated compute units

Problem
#6550 fixed the token-cli for offline signing, but still defaults to using simulated compute units, while #6499 in the stake-pool-cli forces people to specify that they want to use SIMULATED compute units, and otherwise adds nothing. This is inconsistent.
Solution
Make the stake pool CLI consistent with the token CLI by defaulting to simulated compute units. On the other hand, this PR allows people to specify that they want DEFAULT compute units.
Let me know what you think!

i like the latest ux the most! unfortunately i ran into two issues:

now, proving no compute flags at all introduces a simulated compute budget; i believe this should be default ie no instruction
there is no way to use a simulated budget with a price because the budget flag is required and SIMULATE is no longer a valid option

this is what i believe the grammar should look like:

no price flag, no budget flag -> no instructions
no price flag, budget flag with default -> no instructions
no price flag, budget flag with number -> fixed budged instruction
price flag, no budget flag -> price instruction, simulated budget instruction
price flag, budget flag with default -> price instruction, no budget instruction
price flag, budget flag with number -> price instruction, fixed budget instruction

budget options remain number or DEFAULT. remove the requirement for a budget flag if price flag is given
ive pushed a commit that implements this logic, lmk what you think!

That makes sense to me, thanks for doing the work! I'll do the same thing with the monrepo CLI


[ISSUE #7189] Optimize time complexity of loop in ConsumerGroupInfo#updateSubscription

Which Issue(s) This PR Fixes

Fixes #7189
Brief Description
The current implementation of the updateSubscription method in the code has a potential issue with the time complexity of the loop. The loop iterates over the subscription table to check if a topic exists in the subscription list. This results in a time complexity of O(n) for each iteration.
To optimize the time complexity, we can introduce an additional HashSet to store the new topics. By doing so, we can perform a constant time (O(1)) lookup instead of iterating over the subscription table in the loop. This will significantly reduce the time complexity of the loop.
How Did You Test This Change?

No need.

Codecov Report

Merging #7190 (484b67f) into develop (f534501) will decrease coverage by 0.03%.
Report is 5 commits behind head on develop.
The diff coverage is 100.00%.

:exclamation: Current head 484b67f differs from pull request most recent head 96cf0ec. Consider uploading reports for the commit 96cf0ec to get more accurate results

@@              Coverage Diff              @@
##             develop    #7190      +/-   ##
=============================================
- Coverage      42.52%   42.50%   -0.03%
+ Complexity      9357     9355       -2
=============================================
  Files           1149     1149
  Lines          82043    82042       -1
  Branches       10701    10699       -2
=============================================
- Hits           34889    34870      -19
- Misses         42792    42815      +23
+ Partials        4362     4357       -5

Files Changed
Coverage Δ

...ache/rocketmq/broker/client/ConsumerGroupInfo.java
79.09% <100.00%> (-0.74%)
:arrow_down:

.../org/apache/rocketmq/proxy/config/ProxyConfig.java
54.20% <100.00%> (ø)

... and 19 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


[SDS-NNN] pylintrc incompatibilities solved

Warnings about pylintrc incompatible configuration

Coverage: 100.0%. Remained the same when pulling 950725ccebbc309958f5da3f6faec13286f37816 on fix/SDS-NNN/pylintrc-changes into 5d1dd1344170446c91490c5c4b7e45e7173a5aa4 on dev.


Tracking last patch date

Is your feature request related to a problem? Please describe.
An option to track the last time a zarf package was updated. We are forced to comply with strict policies around patching and its always a pain to prove the last patch date to show we are in compliance. If zarf provided an additional column to zarf package list that shows its initial deployment date and last updated date then life would be made easy for those who work in the environments Zarf is targeting.
Describe the solution you'd like

Given a state - A zarf package is deployed/updated
When an action is taken - zarf package list is executed maybe with a flag or an info subcommand
Then something happens - Zarf shows all the packages deployed to the cluster along with their last patch date

Describe alternatives you've considered
There are other ways to do this for sure but having the information built into zarf natively would be amazing
Additional context
Add any other context or screenshots about the feature request here.

Since moving to v0.42.2 we now have time stamps on the package secrets created for each package deploy. The Zarf secret for each package now creates a managedFields key for each of the package secrets. These managedFields show include the timestamp of when the package was created. See the timestamp with k get secrets -n zarf zarf-package-example --show-managed-fields -o yaml. There is also a .creationTimestamp field under .metadata. This at least makes it possible to get the data.
Example:
  managedFields:
  - apiVersion: v1
    fieldsType: FieldsV1
    fieldsV1:
      f:data:
        f:data: {}
      f:metadata:
        f:labels:
          f:app.kubernetes.io/managed-by: {}
          f:package-deploy-info: {}
      f:type: {}
    manager: zarf
    operation: Apply
    time: "2024-11-08T19:51:04Z"

LMK if you'd still like to see this information in zarf package list


struct syntax

in C struct {} ID;
defines ID as a variable of type struct. In our grammar, this defines ID as the name of the struct.
Change grammar to require:
typdef struct {} ID;

Fixed in v0.1


Functions specified in Chat Options is not returned in the getToolCalls() method

Bug description
I have two functions that I have specified in Chat Options. I can see from the logs that these functions are getting invoked by the LLM, but when I try to retrieve the function details using the getToolCalls() method they are not being returned (even when the functions have been invoked by the LLM).
Code snippet -
 Prompt prompt = new Prompt(List.of(systemMessage, userMessage),
                                OpenAiChatOptions.builder().withTemperature(0.7f)
                                                .withModel("gpt-4o")
                                                .withFunction("findPapers")
                                                .withFunction("summarizePaper")
                                                .withParallelToolCalls(false)
                                                .build());
                Flux<ChatResponse> chatResponseStream = chatModel.stream(prompt);

                chatResponseStream.map(response -> response.getResult().getOutput().getToolCalls())
                                .doOnNext(toolCalls -> {
                                        logger.info("Tool calls: {}", toolCalls); // returns an empty list when the function has actually been invoked
                                        }
                                })
                                .onErrorContinue((e, o) -> logger.error("Error occurred while processing chat response",
                                                e))
                                .subscribe();

Environment
Spring AI version - 1.0.0-M2
Steps to reproduce
Please see sample code above for the steps.
Expected behavior
Expecting the getTools() method to return the list of functions invoked by the LLM.

How did you find logs to show you that?

+1

The misconception is that the call response.getResult().getOutput().getToolCalls() would return the conversation that happened back and forth with the model for the tool requests, there can be multiple tool calls.
The only way now to gain access to that conversation is to take over control of the function calling youself via the so called 'proxy' feature in Spring AI.   An example of getting close to the conversation is here.
That said, we would like to collect this information for you so you don't have to get down so low into the code.  To achieve that we need to improve the class ChatGenerationMetadata to contain a hashmap.  I've created an issue for this #1722

Similar to what @markpollack said, I experience the same when using Ollama sync case, you need to add this line to get control.
        val prompt = Prompt(
            "what's news in kotlin 2.1",
            OllamaOptions.builder()
                .withFunctionCallbacks(listOf(getExternalKnowledge))
                .withToolContext(mapOf("userId" to "user123"))
+                .withProxyToolCalls(true)
                .build()
        )

When withProxyToolCalls(false)

When withProxyToolCalls(true), you will get back the control but you need to trigger the function on your own.

@markpollack There is also a bug on calculating the token usage, when withProxyToolCalls(false). It doesn't calculate the tools call usage. I'm not sure if other platforms would have the same behaviour.
Suggestion
Capture all the generations into results list.


Use JSON for serializing the graph

Binary files are compact but difficult to read without special tools. A structured text document should be a decent solution.

Best solution would be to support both and let the user decide.

Actually, the solution is simple: Qt's JSON code supports text-based and binary JSON files. You can check it out in the Qt example.
I want to get rid of my hand-made binary serialization and go for JSON in whatever form the user wants.

Maybe you could abstract the serialization away so that the user can specify a custom serialization, with QJson as default serializer. This way everybody could plugin the serialization format they want.

Writing the abstracted serialization code is way not trivial. The serialization itself is a complex problem, thus, it is hard to foresee what a client's serialization implementation could look like. For not just the Json serialization is implemented (in its simplest form, without versioning).

Isn't this done?

Well, I made some sort of serializaiton to JSON, there is a way to construct JSON on-the-fly from your data structure and pass to deserialization.


MINOR: Refactor SslFactory

SslFactory: split the part of SslFactory that creates SSLEngine instances into SslEngineBuilder.  When (re)configuring, we simply create a new SslEngineBuilder.  This allows us to make all the builder fields immutable.  It also simplifies the logic for reconfiguring.  Because we   sometimes need to test old SslEngine instances against new ones, being able to use both the old and the new builder at once is useful.
Create an enum named SslClientAuth which encodes the possible values for ssl.client.auth.  This will simplify the handling of this configuration.
SslTransportLayer#maybeProcessHandshakeFailure should treat an SSLHandshakeException with a "Received fatal alert" message as a handshake error (and therefore an authentication error.)
SslFactoryTest: add some line breaks for very long lines.
ConfigCommand#main: when terminating the command due to an uncaught exception, log the exception using debug level in slf4j, in addition to printing it to stderr.  This makes it easier to debug failing junit tests, where stderr may not be kept, or may be reordered with respect to  other slf4j messages.  The use of debug level is consistent with how we handle other types of exceptions in ConfigCommand#main.
StateChangeLogMerger#main: spell out the full name of scala.io.Source rather than abbreviating it as io.Source.  This makes it clearer that it is part of the Scala standard library.  It also avoids compiler errors when other libraries whose groupId starts with "io" are used in the broker.

Can we make sure that the changes here are consistent with the proposal in https://cwiki.apache.org/confluence/display/KAFKA/KIP-383%3A++Pluggable+interface+for+SSL+Factory?

I think they are consistent.  I think SslEngineBuilder is pretty similar to what KIP-383 calls SslEngineFactory.  The KIP makes the point that "the idea is to make the configuration immutable in the pluggable factory"-- exactly what this PR does with SslEngineBuilder.
{{SslEngineFactory}} is also supposed to have only one method: SSLEngine createSslEngine(String peerHost, int peerPort);.  This is very similar to SslEngineBuilder, which mainly has SSLEngine createSslEngine(Mode mode, String peerHost, int peerPort) Since the KIP hasn't been accepted yet, there is still time to modify it or think it over.  But overall it seems compatible.

We have definitions of client auth in KafkaConfig, we probably want to use the new enum there as well.

Good point.  I added the new enum to {{KafkaConfig.scala}}.

There seem to be a lot of changes to tests for which it is hard to tell why the changes were made. Do you have a list of changes or comments that could be useful in reviewing these?

SslTransportLayerTest.java: we no longer test that SslFactory#reconfigure validates that the reconfiguration can be done.  The reason is because validation only happens in SslFactory#validateReconfiguration, rather than being duplicated in both functions.

SslFactoryTest.java: mainly, the changes here are to test SslFactory#sslEngineBuilder rather than SslFactory#sslContext.  For example, we check that the engine builder is re-created on re-configuration, rather than checking that the ssl context is re-created.  The rationale here is that SSL context is somewhat of an implementation detail and it would be better to not poke at it unless we have a reason.  This will help if we someday have another way of creating SSL engines besides JSSE, etc.  There is also a change where we compare sets rather than comparing arrays.  The rationale here is that in the collection we're looking at, ordering doesn't matter.

@cmccabe Thanks for the updates, PR builds are failing, can you take a look? Also left a comment on validation.

OK.  I revised the patch so that reconfigure also does validation.

Can we revert the changes to SslTransportLayerTest.java since we validate configs during reconfiguration

Yeah, reverted.
Thanks for the reviews.  I'll push a new version and get a build from Jenkins :)


0.4.9 breaks standard Rails setup

The update from test-prof v0.4.8 to v0.4.9 broke my test setup.
The guides describe to require the RSpec recipe "before_all" in the spec_helper.rb. Commit https://github.com/palkan/test-prof/commit/4ffac875229fa0bd8c317abef20726117a6048b7#diff-dee29c4f171f219aa13eb1dc872f1775R3 breaks the setup because ActiveRecord is not loaded in the spec_helper.rb but in the rails_helper.rb.
I'd suggest to either adjust the guides or to move the AR check to somewhere where the class is available.

Hi @schmijos,
Well, the guide is confusing( it should state "in spec_helper.rb (or in rails_helper.rb for Rails applications)".
Not every application (even Rails application) has rails_helper.rb, that's why we have spec_helper.rb everywhere.
Would you like to propose a PR to update the guide (and, maybe, other guides too)?

Hi @palkan,
The "before_all" recipe only works together with Rails (ActiveRecord to be specific) and RSpec, is that correct? If that's the case, I'd rather not confuse users of the standard setup (rails generate rspec:install)
So I would create a PR mentioning the dependencies of the "before_all" recipe and the rails_helper.rb, of that's ok for you.

So I would create a PR mentioning the dependencies of the "before_all" recipe and the rails_helper.rb, of that's ok for you.

Sure it's ok for me!


[FEAT]: 5Q proposal the fifth question does not work

Description
The 5th question does not work
Additional Information
No response

need to add a few-shot example prompt

fixed in #682


Showing number of Search Results

Feature
Hii everyone,
I saw in the website that it's not showing the number of search results.
So, I would like to add a new feature that shows no. of search results

The search feature is for searching for a specific icon.  I don't think there is any value in showing totals here.

Closing. Feel free to reopen it if you can provide some good use cases showing totals.


Add env var for applying flags to cargo commands

Problem
I prefer to use --locked in some environments where the use of the lock file to ensure an exact reproducible install of a binary is important. However, it is too easy to forget to add the --locked.
Proposed Solution
Add an environment variable for each cargo command that can specify flags that will be applied to the use of that command.
For example:
A new env var could be named CARGO_INSTALL_FLAGS. If the env var is defined it is assumed as part of the arguments to the cargo command as if those arguments had been supplied at that specific invocation.
Notes
This proposal is similar to the RUSTFLAGS env var that exists for rustc.

Closing as a duplicate of #8207.  If it is added to the config, then it will also get an environment variable.


Repl

Specifically, this does not look to implement everything in #48, though I do hope to get completion working in a somewhat basic state before I move on to making things line up for the modern language. It's worth thinking about how this influences the design though.
The primary goal of this PR is to reintegrate the REPL module and start solidifying it. Unfortunately that means we are back to dealing with #20 which this reverts the partial fix for in a PR it should never have landed in to begin with: #51.
fixes #42
reopens #20

First pass review done.
Overall, I think the only big open question is how to decouple Runtime from the repl and if we want to block this PR on that work.
Also, as usual, error handling could be better.


optimise undo history

each undo sequence stores the full state of each cursor.
quite expensive, especially considering each cursor can be hundreds of bytes.
undo sequence should be stored as flat array of cursor x, y, offset positions

undo history now only 3 numbers per cursor. i think that's 24 bytes on luajit


[dove] Migration to latest feathers dove version

Summary
This PR address the migration of this package to work in feathers dove.
This is related to #195 too, as the actual package in 4.0.2 is not 100% compatible with dove.
Other Information
This is still a work in progress.
Compilation is ok, and the dist/ is working on my test project.
But all tests are not OK. I'm working on it.
I may refer to this PR to ask for help.

I also opened a new branch: https://github.com/feathersjs-ecosystem/feathers-authentication-management/tree/dove
Could you please make the PR against this branch instead of master? Thanks!

feathers v5 is not compatible with node12 anymore. So fam does not need to support it as well. So please remove it here:
https://github.com/feathersjs-ecosystem/feathers-authentication-management/blob/master/.github/workflows/node.js.yml#L13
And here: https://github.com/feathersjs-ecosystem/feathers-authentication-management/blob/master/package.json#L29
The minimal supported node version is >= 14 as can be seen here: https://github.com/feathersjs/feathers/blob/dove/package.json#L23
That's why the CI test for 12.x fails.

Issues fam+fhc are now understood. They were on my side.
Waiting for error feedback.
@fratzinger do you see anything else ?
And do you agree with my thoughts on error package ?

Without any further looking. What about: const err = new BadRequest(undefined, data) ?

This "could" work as BadRequest constructor signature accept it :
export class BadRequest extends FeathersError {
  constructor(message?: ErrorMessage, data?: any) {
    super(message, 'BadRequest', 400, 'bad-request', data)
  }
}

But, as FeathersError doesn't accept it (constructor(err: ErrorMessage, name: string, code: number, className: string, _data: any)), I think either BadRequest shouldn't accept a message?: ErrorMessage or FeathersError should accept a err?: ErrorMessage. This seems more consistent ?
Actually, my IDE is not happy with error file for this reason.

Do you prefer I use your hack ?

Thanks for your work!
Released as<EMAIL_ADDRESS>🎉

Thanks !


provide ability to request a fresh connection

Is your feature request related to a problem? Please describe.
Certain retry behaviors specify a fresh connection (and sometimes a full DNS re-lookup) to direct traffic away from what might be a problematic host. I need the ability to somehow mark a request to Hyper as not reusing a connection that may be in the pool
Describe the solution you'd like
I can imagine a few ways the API might be structured, but in general I'd like some way as marking a request as "needs fresh connection", potentially giving a reference to the old connection / response which was problematic.
Describe alternatives you've considered
I could rebuild the entire client & pool when a request fails in a way that necessitates a re-lookup. However, it seems like that may cause unnecessary churn on the client side.
Additional context
https://docs.aws.amazon.com/AmazonS3/latest/userguide/optimizing-performance-design-patterns.html#optimizing-performance-timeouts-retries

I like the idea of having hyper::client::connect::Fresh. Usage could look something like:
use hyper::client::connect::Fresh;

let req = Request::builder()
    .method("GET")
    .uri("https://example.com")
    .extension(Fresh)
    .body(())?;
let resp = client.request(req).await?;

This would work for our use-case where we need a new connection on retry since we can just add the Fresh extension as part of our Tower retry policy.
I'm not really sure how this would carry over to hyper 1.0 yet though. From reading the code, it looks like the connection pool was moved to hyper-util, but I can't find any client code that actually references it.

This feature would be really helpful to my team too - one thing we've seen is that we can have stale connections that just don't work anymore or the host they're connected to on the backend had some problem. It'd be nice if the Fresh extension didn't just bypass the lookup in the conn-pool but was a way to force the expiration and replacement of the existing pool in the connection.

@tjandra-amzn - That seems like a good idea to me.
I talked to @rcoh about this, and he had an interesting idea about having the PoolClient place an extension in responses that identifies the Key in the connection pool used to produce that response. Then a method could be added to client to evict the connections for a given response.
Thinking about that more, in the case where DNS starts responding with a new IP, I wonder if it would be better to evict only the connection that failed rather than all the connections with the same key, as there could be some newer connections in the pool that are still good.
To accomplish that, each PoolClient's connection info could add an extension value unique to that connection to each response so that the individual connection from the pool could be identified and evicted. Or, perhaps this extension value could be the resolved IP address so that all the connections for that given IP could be evicted?
@seanmonstar, what do you think about this approach?

@jdisanti That sounds like a good idea to me - we probably want to be invalidating as few connections as possible to prevent concerted storms of new connection attempts if the backing endpoint is partially available which the proposal would help with.

I think the general approach sounds reasonable. It kind of sounds like 2 separate (though related) features. So, it's probably easier to review and merge them separately. But I'm excited at the notion, making the pool more configurable/controllable is part of why I wanted to move it into hyper-util at first. There's lots of things to explore, and it shouldn't be hindered by "but should that API be frozen in place?"


Tweak Variables incl. manpage, docstrings

Added link anchors in variables-related funcs/methods, and link to them - these methods are not part of the generated link setup.
Clarified that vars set to defaults are not saved.
Updated docstrings in the Variables source (for API docs).
Added return-type annotations to Variables.
Fix for converter function possibly failing if it tries to access an environment.  Fixes #2064.
Fixed up the behavior of aliases to variables, and added docu.  Fixes #3869.
Fix PathIsDirCreate validator to catch permission problems.  Fixes #2828

Signed-off-by: Mats Wichmann<EMAIL_ADDRESS>Contributor Checklist:

[X] I have created a new test or updated the unit tests to cover the new/changed functionality.
[X] I have updated CHANGES.txt (and read the README.rst)
[X] I have updated the appropriate documentation

sigh, the update won't push.  will have to try to resolve later.


Changing your GitHub email affects your UIverse.io account

Currently I changed some settings on my GitHub such as name and email
Old email<EMAIL_ADDRESS>Old username: eduardo-amaro-maciel
I have some projects registered with this username on UIverse, but when I logged in this week the platform created a new account for me with the username edu-amr (which is the new username of the account as I log in with GitHub), would there be any way to pass this on to my user?

Changing your GitHub email may affect your UIverse.io account if it's linked to GitHub. Update and verify your new email on GitHub, then check if UIverse.io reflects the change. Contact their support if issues arise.


SSL Configuration for admin node for Dynamic and Configured Cluster Offer

Problem: the hostname for the app gateway (and admin console) are auto generated using a guid to ensure uniqueness.  However, SSL certs must have a hard-coded hostname.  We have two choices at deployment time:

Allow the user to provide the hostname as a parameter.  We would have to assume the provided hostname is unique.
~a. Allow the user to provide the value that is used as the guid.  In the ARM template, if this value is set, use it instead of calling the guid function.~
~Allow the user to derive what the hostname will be.~

Jacob and I discussed this today and we agreed to a simplifying assumption: we assume the user will provide a value that, when combined with the rest of the parts of the name, will be a unique hostname.

User Experience

Update the App Gateway blade to have a field that accepts the fully qualified DNS name to bind to the public IP for the App Gateway.  We assume this DNS name already exists.  This is based on Integrate Azure DNS.  If not specified, the existing guid-based approach is used.
Update the ARM template to account for this provided DNS name.

I was able to configure an Azure DNS Alias for the WebLogic Admin Server and the Application Gateway in the WebLogic Cluster Offer.
Following are the steps followed to complete the setup:

Registered the Domain - wlseng-azuretest.com

Created a DNS Zone in the same Resource Group where the WebLogic Admin Server & Application Gateway are already created.

Note down the Name Servers associated with the DNS Zone and update them as the name servers in the Registered Domain (This step is called Azure DNS Delegation)

Run the command nslookup -type=SOA wlseng-azuretest.com to ensure that the configured Azure Name Servers are listed as the name servers for the domain.

Create an Azure DNS alias (subdomain) for the Public IP Address of the WebLogic Admin Server & Application Gateway

admin.wlseng-azuretest.com
appgw.wlseng-azuretest.com

Access the URLs using the wlseng-azuretest.com domain.
WebLogic Admin Server
http://admin.wlseng-azuretest.com:7001/console/login/LoginForm.jsp
https://admin.wlseng-azuretest.com:7002/console/login/LoginForm.jsp
Application Gateway
http://appgw.wlseng-azuretest.com/replicationwebapp/FirstServlet
https://appgw.wlseng-azuretest.com/replicationwebapp/FirstServlet

Thanks @gnsuryan .  Let me spell out the steps I see to automate this process with template deployment.

Modify the createUiDefinition for the cluster offer.  Modify the App Gateway blade.   Make it so the UI has a text field that allows the user to input a fully qualified domain name, assumed to be pre-created and already in public DNS.
Modify the app gateway nested template.
If the DNS field was filled in

Create DNS zone using knowledge from docs
Do the necessary delegation actions from docs.
Make it so the public hostname of the App Gateway is the one provided in the text field. This will involve lots of conditionality, particularly in the parts that formerly used the guid.
Make it so the template spits out the ip addresses of the nameservers so the user can add them to the pre-created DNS entry.

I may have missed something, but this should be a good starting point for the work.

@gnsuryan please determine if the above actions are better implemented as an entirely separate nested template that just deals with the DNS actions, or if, as I was writing, they are better implemented as modifications to the app gateway nested template.  I'm not sure right now.

I tried including the ns1-01.azure-dns.com. in the list of default name servers (that starts with @ symbol ).
I am getting the following error message during deployment.
Deployment failed. Correlation ID: 6dcdadc0-a465-46ad-bf0d-a9fb783b1375. {
"status": "Failed",
"error": {
"code": "ResourceDeploymentFailure",
"message": "The resource operation completed with terminal provisioning state 'Failed'.",
"details": [
{
"code": "DeploymentFailed",
"message": "At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/DeployOperations for usage details.",
"details": [
{
"code": "BadRequest",
"message": "{\r\n  "code": "BadRequest",\r\n  "message": "The 'nsdname' of a record set with type 'NS' and name '@' cannot be changed."\r\n}"
}
]
}
]
As per the Azure DNS operations documentation, I found that we will not be able to add/delete/modify the name servers which get created by default.
You cannot add, remove, or modify the records in the automatically created NS record set at the zone apex (--Name "@", including quote marks). For this record set, the only changes permitted are to modify the record set TTL and metadata.
https://docs.microsoft.com/en-us/azure/dns/dns-operations-recordsets-cli

The only option left now is to use the default Name Servers created by Azure and output them after deployment is complete, so that users can then use it for Azure DNS Delegation at their Domain Registry.

I have completed the creation of Azure DNS Zone and DNS Alias (A) record entry for WebLogic Administration console (in Admin Only, Cluster & Dynamic Cluster offers) and DNS Alias Record (A) entry for Azure Application Gateway for Cluster offer.
Currently, I am working on making changes to the createUIDefinition.json file to accept user inputs for Custom SSL and DNS Configuration.

I have completed the changes to createUIDefinition.json file to accept users inputs for Custom SSL and DNS configurations for all the three offers (admin Only, Cluster & Dynamic Cluster).
I am currently working on merging the changes in to the wls-eng branch, as there are many changes that have happened in the last few weeks. I will merge the changes, test and then create a Pull Request once everything is found to be working fine.

Notes for 2020-12-23 Weekly Oracle sync

What is the status of this work and #185 ?
If the work is stalled, please share the current state, because are working on this regarding #145.
If the work is still ongoing, we need coordinate integrating it with our work on #145 .

Hello @gnsuryan This one is very important for the Q1 release.  We are counting on you to implement this feature for the Dynamic and Configured Clusters, in similar to what you did on the admin offer.  After you have done it, @zhengchang907  will improve the experience similar to what he did on the admin offer.

@edburns @gnsuryan I've created related task for adding key vault automation to the SSL configuration blade for the Configured Clusters offer #AB1275906


when expanding check in data quality report, all checks are expanded

In the development version of Ares when expanding check in the data quality report, all checks are expanded.

The task is done, merged with the Developer branch.

The task is done, merged with the Developer branch.


Enable SerializeFeature.MapSortField for deterministic order

test_f() in JSONPParseTest2.java parses a text to a JSONPObject and compare the toJSONString representation to see if it is same as a hard-coded string. The underlying inner map of jsonpObject is using a HashMap.
As you may tell from the test itself, the original input is
"parent.callback ({'id':1, 'name':'idonans'} );   /**/ ";
but the test assertion is asserting
"parent.callback({\"name\":\"idonans\",\"id\":1})"
However, HashMap does not guarantee any specific order of entries. Therefore, the assertions in those test cases will fail if the order is different.
This PR proposes to enable  SerializerFeature.MapSortField to make the order deterministic and modify the corresponding test assertions. We believe this will make the test more stable.

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

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

Codecov Report

Merging #2799 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #2799      +/-   ##
============================================
- Coverage     80.13%   80.12%   -0.02%
+ Complexity     7144     7143       -1
============================================
  Files           145      145
  Lines         26961    26961
  Branches       6448     6448
============================================
- Hits          21606    21603       -3
- Misses         3495     3497       +2
- Partials       1860     1861       +1

Impacted Files
Coverage Δ
Complexity Δ

...com/alibaba/fastjson/serializer/SerialContext.java
72.91% <0%> (-4.17%)
19% <0%> (ø)

...main/java/com/alibaba/fastjson/util/FieldInfo.java
86.2% <0%> (-0.35%)
91% <0%> (ø)

Continue to review full report at Codecov.

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

@Omega-Ariston any updates on this PR?


libretuya-esphome support for wb2l_m1 board?

Hi,
i have some bulbs with wb2l_m1 like this one https://www.elektroda.com/rtvforum/topic3880540.html .
I have succesfully flashed it with openbeken using tuya-cutter.
But I would like to use it with libretuya-esphome.
I tried to place
libretuya:
  board: wb2l_m1
  framework:
    version: latest

to config, but it does not compile.
Is there a way to compile it?

The bulb in the linked post doesn't have WB2L_M1, but from what I know WB2L_M1 is BK7231N, despite the "WB" prefix.
It's not in the board list, but CB2L is, so you can use that. You can also use generic-bk7231n-qfn32-tuya.

Sorry i messed up the links then, I was flashing it several months ago.
In smartlife app it showed firmware version 1.3.21 , so i used it in tuya cutter and then uploaded openbk to it.
So is it safe to use the board you suggested?

Yes, 1.3.21 is BK7231N. You can use the boards I suggested.


io.binary: make le> and be> faster (20% and 75%, respectively).

Removed primitive byte-array>bignum and digit_stream_to_bignum from vm/.

I'm glad the simpler pure Factor implementation is faster. In theory, for large bignums, calling shift in a loop is O(n^2) in the length of the bignum. If so, we could add a conditional to use two different implementations here. But that is probably a rare case, I imagine most times be> and le> are called with 2, 4 and 8.
As for removing byte-array>bignum completely, there are case where it could be useful, for constructing very large bignums. Didn't Doug at some point want '1024 random-bits' and stuff to be fast?

You can merge this

It seems that the new implementation is faster for large byte arrays and only a little slower for 1024 random-bits, for example:
old:
( scratchpad ) 1024 iota >byte-array [ 10,000 [ dup le> drop ] times ] time drop
Running time: 5.871235712 seconds

( scratchpad ) 1024 iota >byte-array [ 10,000 [ dup be> drop ] times ] time drop
Running time: 6.220240729 seconds

( scratchpad ) [ 1,000 [ 1024 random-bits drop ] times ] time
Running time: 0.345316604 seconds

new:
( scratchpad ) 1024 iota >byte-array [ 10,000 [ dup le> drop ] times ] time drop
Running time: 3.110973963 seconds

( scratchpad ) 1024 iota >byte-array [ 10,000 [ dup be> drop ] times ] time drop
Running time: 3.631767304 seconds

( scratchpad ) [ 1,000 [ 1024 random-bits drop ] times ] time
Running time: 0.363106837 seconds

Also, benchmark.sfmt and benchmark.mt seem 4-6% faster.

I clicked the button "merge pull request" without thinking, and then separately pushed to factorcode... I hope this doesn't break anything...

You can always git push --force the right head if things get messed up.


Support templates hosted on SourceHut and GitLab (e.g. wofi)

As described in the commit:
The template source from base16-templates-source contains a link to a
SourceHut repository (e.g. wofi) that doesn't exist on GitHub.
Only GitHub is supported at the moment by base16-universal-manager.
This is a commit with sane defaults that should work for now. But later
we could add support for user settings to set the raw base url as well
as the main repository branch (which isn't 'master' by default in most
new repos).
Notes

First time I write some code in Go... Happy to hear any suggestions on doing things differently.
Can you also update the wiki? I've pushed some changes adding wofi in my fork of your wiki

git clone https://github.com/pinpox/base16-universal-manager.wiki
git pull https://github.com/paul-ri/base16-universal-manager.wiki.git
git push

Testing done

Ran tests with IDE :heavy_check_mark:
The following config was failing, but now isn't :heavy_check_mark:

  wofi:
    enabled: true
    files:
      default:
        path: ~/.config/wofi/style.css
        mode: rewrite

PR will fix #43


DocuSign login failure due to replyURL mismatch

The reply URL is incorrect in this document

The correct value is
https://.docusign.com/organizations//saml2
Error reference:
Error: AADSTS50011: The reply URL specified in the request does not match the reply URLs configured for the application: 'https://xxx/organizations/xxx/saml2'
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 40756c7c-63eb-10f8-c8c4-17b57877bb7f
Version Independent ID: 9de82050-9230-0236-c246-b2fd273888ce
Content: Tutorial: Azure Active Directory single sign-on (SSO) integration with DocuSign
Content Source: articles/active-directory/saas-apps/docusign-tutorial.md
Service: active-directory
Sub-service: saas-app-tutorial
GitHub Login: @jeevansd
Microsoft Alias: jeedes

@TonySh127-ms Thank you for your feedback. We will investigate and update the thread .

I do see the value of the AssertionConsumerService URL as per the document only. Can you please provide screenshot (covering the account id) of your Docusign configuration

Hi Jeevan,
Thank you for your response.
Problem description:
The user account Oganesyan, Grachya (object id: ce0a37f0-4153-4139-b2f9-906c056af862) cannot login at application TCW Docusign SSO QA (object id: b6c77c12-9cc7-4f31-b6ec-8c323f5090ff)
Error: AADSTS50011: The reply URL specified in the request does not match the reply URLs configured for the application: 'https://account-d.docusign.com/organizations/9c8e2a4f-907d-4fe4-a3b8-74bcc95b9d91/saml2'
Tenant id: b730b432-2098-413f-bd4a-014acdf7c72e
Checked AAD sign-ins log
Resent issue occur time:
2021-03-18 02:47:21 UTC time
Detail error:
Saml2:Processrequest
errorCode: 50011,
Correlation Id: 6a9bf6b2-151b-44d3-862e-187df20d1d8a
Request Id: 4e7875a9-c178-418f-bd7d-11aa0ba3f600
failureReason: The reply URL specified in the request does not match the reply URLs configured for the application: '{identifier}'. {detail}",
additionalDetails:
Developer error - the app is attempting to sign in without the necessary or correct authentication parameters
Request Has Failed With The Following:
AADSTS50011: The Reply URL Specified In The Request Does Not Match The Reply URLs Configured For The Application: 'Https://Account-D.Docusign.Com/Organizations/9c8e2a4f-907d-4fe4-A3b8-74bcc95b9d91/Saml2'.
Checked Application reply URL:
Object ID: b6c77c12-9cc7-4f31-b6ec-8c323f5090ff
App ID: 7a9fe208-8473-4023-971b-8b1c46772e69
replyUrls: https://account-d.docusign.com/organizations/9c8e2a4f-907d-4fe4-a3b8-74bcc95b9d91/saml2/login/6124102https://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Faccount-d.docusign.com%2Forganizations%2F9c8e2a4f-907d-4fe4-a3b8-74bcc95b9d91%2Fsaml2%2Flogin%2F6124102&data=04|01|Grachya.Oganesyan%40tcw.com|7046f7c2a61b4feca3c908d8edb92092|b730b4322098413fbd4a014acdf7c72e|0|0|637520726660216184|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=qXGmngSXOu%2B1MenzUaji1tBdR1XE4XU4Ru9cMqdgM2g%3D&reserved=0
DocuSign configuration:
@.***D72080.0321D2D0]
@.***D72080.0321D2D0]
@.***D72080.0321D2D0]
Please let me know if you need more information. Many thanks!
Best regards,
Tony Shen
From: Jeevan Desarda @.>
Sent: Wednesday, March 24, 2021 2:11 AM
To: MicrosoftDocs/azure-docs @.>
Cc: Tony Shen @.>; Mention @.>
Subject: Re: [MicrosoftDocs/azure-docs] DocuSign login failure due to replyURL mismatch (#72612)
I do see the value of the AssertionConsumerService URL as per the document only. Can you please provide screenshot (covering the account id) of your Docusign configuration

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F72612%23issuecomment-805119654&data=04|01|Feipeng.Shen%40microsoft.com|ac36d74bc4464c88ebb108d8ee26fc8e|72f988bf86f141af91ab2d7cd011db47|1|0|637521198524368233|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=8ZqXK24wOIrE3n%2F68a%2FBBS3PDCSCq%2BJZ2VfXEUFGb1Y%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAQHECIUXYHBL5NPQCNQPEULTFDKSRANCNFSM4ZUWOVFA&data=04|01|Feipeng.Shen%40microsoft.com|ac36d74bc4464c88ebb108d8ee26fc8e|72f988bf86f141af91ab2d7cd011db47|1|0|637521198524368233|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=TiE6EsMgMP8mZ8GqTNi2KBqi2rjiKbFhooIW9g9cxAY%3D&reserved=0.

@TonySh127-ms This article is talking about using Docusign Production instance. What you are trying with is using Docusign QA/Demo environment which is very different. We will add these patterns here and that way you can use these URLs here.

Hi Jeevan,
Thank you for your time to investigate the issue.
Understand, the issue URL is for DocuSign QA/Demo environment. Hope we will add this note in our doc soon. Many thanks your quick assistance and effort! 😊
Best regards,
Tony Shen
From: Jeevan Desarda @.>
Sent: Wednesday, March 24, 2021 10:54 PM
To: MicrosoftDocs/azure-docs @.>
Cc: Tony Shen @.>; Mention @.>
Subject: Re: [MicrosoftDocs/azure-docs] DocuSign login failure due to replyURL mismatch (#72612)
@TonySh127-mshttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FTonySh127-ms&data=04|01|Feipeng.Shen%40microsoft.com|4ded84e3c3e048e5336f08d8eed4a63c|72f988bf86f141af91ab2d7cd011db47|1|0|637521944412240058|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=rDwPZohuw8aFp8h8GOXq1xQOowAXiNKAQz2g3nfHW3I%3D&reserved=0 This article is talking about using Docusign Production instance. What you are trying with is using Docusign QA/Demo environment which is very different. We will add these patterns here and that way you can use these URLs here.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F72612%23issuecomment-805887286&data=04|01|Feipeng.Shen%40microsoft.com|4ded84e3c3e048e5336f08d8eed4a63c|72f988bf86f141af91ab2d7cd011db47|1|0|637521944412250052|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=CfE9TvBuXF9pRtXKmIYxPXkPZ6gizqG7YwjRLuzvm4Q%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAQHECIRE2V6VHR5MROGOB7TTFH4IHANCNFSM4ZUWOVFA&data=04|01|Feipeng.Shen%40microsoft.com|4ded84e3c3e048e5336f08d8eed4a63c|72f988bf86f141af91ab2d7cd011db47|1|0|637521944412250052|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=sOMYZ6nTWNqbJGVMocWO2BbAIt%2F2rhcWnZHx0%2F5wCTA%3D&reserved=0.

Issue resolved as per the above conversation.
#please-close


break out hasPVs into two separate functions

@alaypatel07 @dymurray @sseago what's left on this PR?

too late to merge this now

too late to merge this now


(#367) Navigate to nested items on hash change

Description Of Changes
The implementation for navigating to a hash that located inside of a
collapsed or tabbed element already existed, however it was commented
out in choco-theme. This has been updated and is now enabled for use.
On the install page, a user will now correctly be navigated when
clicking on a link in the right hand nav under "More Install Options".
Motivation and Context
Links should always navigate to where they are pointing to.
Testing

Linked choco-theme to both docs and community
On the docs site, went to /en-us/choco/setup
In the right hand nav, clicked on "Completely Offline Install" (2nd option under "More Install Options")
Clicking opened up the collapsed element and navigated me to the correct spot.
On community, I ensured all of the functionality still worked as expected on the packages page when trying to navigate to a link inside a collapsed element.

Change Types Made

[x] Bug fix (non-breaking change)
[ ] Feature / Enhancement (non-breaking change)
[ ] Breaking change (fix or feature that could cause existing functionality to change)
[ ] PowerShell code changes.

Related Issue

https://github.com/chocolatey/choco-theme/issues/142
https://github.com/chocolatey/docs/issues/367

Fixes #367
Change Checklist

[ ] Requires a change to the documentation
[ ] Documentation has been updated
[ ] Tests to cover my changes, have been added
[x] All new and existing tests passed.
[ ] PowerShell v2 compatibility checked.

@st3phhays thanks for getting this updated!


[Feature][UI Next]Fix the routing problem of the user management module

Purpose of the pull request
Please refer to the main issue #8070 .

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

Codecov Report

Merging #8251 (fc7e1dd) into dev (7f4d26e) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev    #8251      +/-   ##
============================================
- Coverage     44.95%   44.94%   -0.01%
+ Complexity     3951     3950       -1
============================================
  Files           672      672
  Lines         26182    26182
  Branches       2825     2825
============================================
- Hits          11769    11768       -1
  Misses        13304    13304
- Partials       1109     1110       +1

Impacted Files
Coverage Δ

...dolphinscheduler/remote/future/ResponseFuture.java
81.96% <0.00%> (-1.64%)
:arrow_down:

Continue to review full report at Codecov.

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

good job

good job


Can not see all deployments

I'm not able to see all deployments in the dashboard. Yesterday i was able to see different builds deployed to what I can see today - there doesnt seem to be any configurability I can see around time periods for deployments or other things that would explain this?

Not currently.
It may be that we need to query further back if the missing deployment was quite old.

@pvillalvagu thanks for the report. Sorry it is not working as intended.
If you can let us know some details about your pipelines, it will help us fix the bug.
Could you include what pipelines you expected to see, which deployments were the latest, and what you actually see on the dashboard instead?
You might like to capture the network responses for pipeline information in developer tools as well. If you are comfortable sharing that it might help identify the bug.
Thanks for the report.

As an example, here are some pipelines executed on Thursday 26/9

And here is what i see on the dashboard:

I can only see the run-publisher deployment but not the other two pipelines in that folder. List view also produces the same output.

Can I confirm that these pipelines have deployment jobs (docs)>?

Yes, we are using deployment jobs:

Hi @pvillalvagu
We are releasing a new feature in the dashboard settings that let's you download some diagnostic data.
If you like, you can send that to me directly or include it in the issue here (be aware it contains information about your devops project, pipelines and deployments).
Just go to Settings, click the ellipsis and export diagnostics to download a json file.


No errors about function parameter option properties when they are unused

Flow Try Link
No errors from this code:
type StorageMonitorOptions = {|
    keepRunning: boolean,
    onError: (Error) => void,
    personIdUsedByRemote: string,
    addAccessibleHashToCache: (hash: string) => boolean,
    idLinkHashes: Set<string>,
    addIdLinkHashes: (Array<string>) => void,
    idObjectsLatestOnly: boolean
|};

function initStorageMonitor (
    {
        sss,
        ddd,
        fff,
        ___,
        äää,
        ß,
        __
    }: StorageMonitorOptions
): void {
  // ...
}

Errors will start to be reported as soon as a value is actually used in the function body.

Flow now errors on them.


Alignment replaces indentation with spaces

Whenever I trigger an alignment, my tab indentations are replaced with spaces. This makes the plugin too destructive for me to use. Why is it even touching my indentation in the first place?

Fixed https://github.com/chemoish/atom-valign/commit/ffd80e184c36108181c1713d13768eaf5b1de93a

Sorry for the extremely late update.
The code does not use the Grammar API for one thing… but each line is made up of multiple parts. Indentation, keywords, operators, etc. and…
It is touching the indentation because entire lines are replaced during the formatting process. This was the easiest way of doing it (at the time of writing), rather than inserting spaces/formatting at specific editor coordinates.


Managed storage clarifications

It would be great if this article spelled out the following:

Per the article: container images are "encrypted at rest". Is BYOK supported? If not, are keys managed per customer?
In managed storage, how are different customers' images separated from each other? Does each customer get a distinct managed storage account, or ?
Thanks!

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

ID: 194963f9-afb6-3db2-7995-bf3a4aa3ca0e
Version Independent ID: c899bf22-66de-39b6-952f-faef21f3a8ce
Content
Content Source
Service: container-registry

@plzm  Thanks for the feedback, I assigned the case to the author.

Hi @plzm, thanks for your interest!

No, BYOK is not supported. The storage account that backs your registry uses Storage Service Encryption (SSE). There is a FAQ at the bottom of that article with a few details on keys:

Q: Who manages the encryption keys?
A: The keys are managed by Microsoft.
Q: Can I use my own encryption keys?
A: We are working on providing capabilities for customers to bring their own encryption keys.
Q: Can I revoke access to the encryption keys?
A: Not at this time; the keys are fully managed by Microsoft.

Yes, each registry is backed by a unique storage account.

#in-process

The docker images and layers can only be accessed though the registry.
Regarding the question

In managed storage, how are different customers' images separated from each other? Does each customer get a distinct managed storage account, or ?

There is no concept of managed storage. Customers don't have access to the underlying layer implementations. They did with the classic model, but not with a managed registry
/cc @mmacy

Hi @plzm, I'm marking this as in-process as we have a new article on storage that I anticipate will published in the next week or so. Once that's published, I'll paste the link here, and then we'll close this issue if you're all set.
#in-process

Hello Patrick @plzm, the new article on ACR storage has been published:
Container image storage in Azure Container Registry
I believe we've taken care of the items you raised, so I'll request to have this issue closed. Please let us know if you have any further questions, however, and we'll address them here. Thanks for using Azure!
#please-close


Go to symbol in Workspace without type exactly its name.

Go to symbol in Editor works find but Go to symbol in Workspace can't do this.
Go to symbol in Workspace in vscode:

Go to symbol in Workspace in sublime text:

@nguyencobap what language is this?

@nguyencobap what language is this?

@bpasero I use default for language settings so English I think.

@nguyencobap I meant what file language in that example.

@nguyencobap I meant what file language in that example.

@bpasero It's english.


[dv/kmac] Fix unmasked regression err

This PR fixes kmac unmasked regression error from PR #14178. The RTL changes the key masking for register input only, but the DV changes impacts the sideload key as well.
So this PR adds a logic to gate key masking if it is sideload key.
Signed-off-by: Cindy Chen<EMAIL_ADDRESS>

Thanks @cindychip , this looks good. And sorry for introducing this problem in the first place!

no worries at all. It helped me better understand masking for surre hehe


More questions than problems

I decided to use the plugin because of the youtube video and now that I try it I have some doubts that I am still not able to answer are 3:
1.- How can I disable the permissions of the files and folders that the file tree shows me?

2.- How can I make that the statusline is not shared between the panels?

3.- How can I open the left side and also the terminal?
To look like this:

Instead of this:

Please create a series of Q/A for this in discussion.


Update README.md

Thanks for your valuable contribution 😊. Don't forget to ✨ our Repository.
If you want to contribute to our new Ideas Project -https://github.com/KapilKumar7/CreativeAppIdeas.


Support on memory API to make Menoh model data

support menoh_make_model_data_from_onnx_data_on_memory API

Codecov Report

Merging #29 into master will increase coverage by 0.58%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #29      +/-   ##
==========================================
+ Coverage   87.77%   88.35%   +0.58%
==========================================
  Files           4        4
  Lines         180      189       +9
==========================================
+ Hits          158      167       +9
  Misses         14       14
  Partials        8        8

Impacted Files
Coverage Δ

runner.go
82.4% <100%> (+1.36%)
:arrow_up:

Continue to review full report at Codecov.

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


Unable to convert Iterable to List - It Throws a Null Exception

ArgumentError (Invalid argument(s) (input): Must not be null) on converting the returned Iterable to a List.

Any solution for this? Stuck here for two days now.

It turns out when you uncheck All Exceptions (in visual studio code)  there is no exception thrown and everything works correctly.

Can you provide some code?

This is the line where exception is experienced: List<Contact> _contacts = (await ContactsService.getContacts())          .toList(growable: false);

Maybe the exception is being handled internally by the plugin? Please, provide where the exception is happening exactly

See thread in #195