Seems Azure Blob Storage documentation changed. Need suggestion on right documentation for v12.4.0

Package Name: @azure/storage-blob
Package Version: 12.4.0
Operating system: Windows 10
[X] nodejs

version: v10.16.3

[ ] browser

name/version:

[ ] typescript

version: v1.5.3

Is the bug related to documentation in

[ ] README.md
[ ] source code documentation
[X] SDK API docs on https://docs.microsoft.com

Describe the bug
A clear and concise description of what the bug is.
Question on @azure/storage-blob documentation:
We are seeing two different versions of the documents in the documentation.
New one seems:- https://docs.microsoft.com/en-us/javascript/api/@azure/storage-blob/?view=azure-node-latest
Old one following till today:- https://docs.microsoft.com/en-us/javascript/api/@azure/storage-blob/?view=azure-node-preview
What is the difference between these two documentations azure-node-latest vs azure-node-preview?
As per the SDK it is inline with  azure-node-preview documentation. Please let us know the correct documentation for Azure Storage Blob client library for JavaScript - Version 12.4.0.

Hey @prashanthmadduri, Thanks for reporting
The one with view=azure-node-latest as the query parameter in the url corresponds to the latest stable version of the package
The one with view=azure-node-preview as the query parameter in the url corresponds to the latest preview version of the package

As per the SDK it is inline with azure-node-preview documentation.

Can you elaborate on this? Are you seeing a link to the preview documentation from somewhere? If so, then where?

Oh wait, I realize there is no preview version which is higher than the latest stable version. In that case, the link with the query parameter for preview should have re-directed to the stable one because there is no latest preview version.
@sima-zhu, @danieljurek, Can we have some automation around this?
@ljian3377 The 12.4.0-beta.1 version on npm has the next tag when it should not because the version 12.4.0 with the latest tag is newer than the beta version

Oh wait, I realize there is no preview version which is higher than the latest stable version. In that case, the link with the query parameter for preview should have re-directed to the stable one because there is no latest preview version.
@sima-zhu, @danieljurek, Can we have some automation around this?
@ljian3377 The 12.4.0-beta.1 version on npm has the next tag when it should not because the version 12.4.0 with the latest tag is newer than the beta version

You are right. @EmmaZhu Let's remove the "next" tag on 12.4.0-beta.1 using the js - npm-admin-task pipeline.

Oh wait, I realize there is no preview version which is higher than the latest stable version. In that case, the link with the query parameter for preview should have re-directed to the stable one because there is no latest preview version.
@sima-zhu, @danieljurek, Can we have some automation around this?
@ljian3377 The 12.4.0-beta.1 version on npm has the next tag when it should not because the version 12.4.0 with the latest tag is newer than the beta version

@ramya-rao-a Just to summarize the documentation with view=azure-node-preview as the query parameter in the URL corresponds to the latest stable version of the package and view=azure-node-latest as the query parameter in the URL corresponds to the 12.4.0-beta.1  version of the package.
Correct us here in case any understanding gap.

@prashanthmadduri It is the other way around
Ideally, you should only be looking at the url with query parameter view=azure-node-latest, as this corresponds to the latest stable version. As of today this would be for 12.4.0
You should care about the url with the query parameter view=azure-node-preview only if you wanted to look at the preview i.e. beta versions of the package.

@ramya-rao-a if that is the case then the view=azure-node-latest documentation is not matching with the samples available in repository here https://github.com/Azure/azure-sdk-for-js/tree/%40azure/storage-blob_12.4.0/sdk/storage/storage-blob/samples/javascript.
The samples here https://github.com/Azure/azure-sdk-for-js/tree/%40azure/storage-blob_12.4.0/sdk/storage/storage-blob/samples/javascript is matching with view=azure-node-preview.
If we take ContainerClient, BlobClient we can not see them in  view=azure-node-latest documentation.

@ramya-rao-a  could you please help with the inputs here. Thanks.

@prashanthmadduri That must be a point in time error which should be fixed now.
The latest package as of today is version 12.4.1 and the latest preview is of version 12.5.0-beta.1
There is a gap between package release date and the docs getting updated. We are working on reducing this gap.
Please let us know if you are still seeing issues


Revisit immutable tag propogation for workflow(-dev/-canary) chart

Our current CI model for deis component changes to the workflow-dev chart is as follows:

submit component PR
CI creates/pushes immutable git-<SHA> tag and mutable canary tag of said component
CI runs e2e tests against updated chart with immutable tag from step 2, sending back success or failure
Once PR is green, commit is merged to master
Steps 2 and 3 are re-run against this master commit
If tests are green, CI pushes commit to charts repo substituting immutable tag produced from step 2 for appropriate component Docker image

This process is great in theory.  Unfortunately, in practice, the e2e tests aren't 100% reliable and so some perfectly good component changes (immutable tags) fail to get into the master version of the workflow-dev chart in a timely manner.
However, if we went the alternate route of having every component image in the workflow-dev chart use the canary tag, we run the risk of hitting a situation where the master version of the chart is unstable, since the mutable canary tag is produced during component build/deploy stage of CI and before e2e has a chance to notify with a failure.
These are a few options I see moving forward:

Keep model same for now, meaning workflow-dev uses immutable tags but chart may not be as fresh
Switch to canary model so chart is always fresh but offer no guarantees that chart will be stable at any given point
Switch to canary model and revisit CI build/deploy logic thusly:

on component PR build/deploy, only push immutable tag
if e2e green (and other prerequisites met, i.e. LGTMs), merge to master
on component master merge build/deploy, push immutable tag and canary tag
run e2e,

if green we are all good and workflow-dev components already using canary tags
if red, chart may be unstable/broken and could remain until a fix is provided

The benefit of option 3 is the chance of a broken chart gets moved to a later stage (after greenlit PR gets merged, rather than before) but there still exists a chance of a unstable/broken chart, though hoping the cases will be rarer assuming PR vetting is dependable.
Note: if we go any of the canary routes, I vote we change the chart name to workflow-canary
Looking for feedback on those 3 options and/or other ideas in this area.  Thanks!

I vote option 3 but keeping the workflow-dev name - Stabalizing the development workflow at this point is important
For controller (as an example) locally I only use the immutable tag to update my own registry when I'm developing so that I know what branch I was working against anyway.

My 2 cents:
I like the current model, because it is a persistent reminder that our E2E tests are currently non-deterministic. Recent work on our E2E implementation has improved runtimes, so I think it's reasonable to expect the merger to investigate failed post-merge E2E test failures (retry, take notes, etc), and to use that investigation towards a more deterministic E2E paradigm.
I don't think 100% E2E tests are realistic, but I dislike the idea of building rather complicated processes to account for mysterious E2E failure because however clever we are in working around such mysteries, we  will end up in the situation where mystery is institutionalized: "E2E tests are flaky and we don't trust them and that is just that." Now is the time to at least aim for better than that.

Why are we using canary builds at all?  Not only are mutable tags a widely recognized Docker anti-pattern, they're a disaster from a bug reporting standpoint.

Unfortunately, in practice, the e2e tests aren't 100% reliable and so some perfectly good component changes (immutable tags) fail to get into the master version of the workflow-dev chart in a timely manner.

Why don't we put the effort into fixing this?

my $0.2 - we should put effort into speeding and determinizing (yes, I made that word up) our e2e tests, and remove canary. Right now, it just seems like these tags are a band-aid on this larger problem of e2e testing...
Real example: we don't go build our CLI binaries and upload them to deis-canary on Bintray before we run go test.

I'm happy with option 1 and fixing up CI to deploy git SHAs in a timely manner through enhancements to the test suite. If we can make the tests 95% reliable then I'd be completely happy with a few hiccups here and there. That's just the nature of end-to-end test suites, unfortunately.

Thank you for the feedback.  It sounds like the consensus is to keep our current process of promoting immutable tags in tact, maintaining focus on e2e determinism.
Many have questioned the process of pushing mutable/canary tags of components along with immutable, therefore https://github.com/deis/workflow-e2e/issues/152 was created to track the proposal to remove this process and let that particular conversation hash out there.


How to get the coefficients of RNS-CKKS ciphertext polynomial？

How to get the coefficients of RNS-CKKS ciphertext polynomial？

seal::Ciphertext::data() returns the address of the core data of a ciphertext. The coefficients are stored as $uint64_t$ and the data at a low address is a coefficient of a low degree term.

It is not surprised that some libraries do not offer interfaces to read the data of ciphertexts. After all, we hope the data of ciphertext will be not leaked. So we expect that the public interfaces of a well-designed class provide enough functions in application.


[Yunikorn 2573] Flaky test TestUpdateNodeCapacityWithMultipleNodes

What is this PR for?
TestUpdateNodeCapacityWithMultipleNodes occasionally failed in pipeline because of node and partition resources inconsistency.
What type of PR is it?

[x] - Bug Fix
[ ] - Improvement
[ ] - Feature
[ ] - Documentation
[ ] - Hot Fix
[ ] - Refactoring

Todos
N/A
What is the Jira issue?
[YUNIKORN-2573]
How should this be tested?
make test
Screenshots (if appropriate)
N/A
Questions:
N/A

@arhtur007 As we are discussing off-line, could you please change this PR to draft?

Hi @chia7712 @chenyulin0719 ,
I've updated the PR.
please take a review when you're free, thank you

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 77.01%. Comparing base (a1a10f8) to head (21e3c93).
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #862      +/-   ##
==========================================
+ Coverage   76.99%   77.01%   +0.01%
==========================================
  Files          97       97
  Lines       11998    11998
==========================================
+ Hits         9238     9240       +2
+ Misses       2422     2421       -1
+ Partials      338      337       -1

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


Logged exceptions should contain more detail

#182 was implemented to fix #177, but this still does not provide as much information about an exception as it should provide.
When using doOnError(), just using exception as a parameter for a {} placeholder results in an empty string in the output. Using exception.getMessage() results in the exception message in the output, but a more complete cause tree or stack trace would be better.

I'm not sure we need to be adding a stack trace to the log here. doOnError() is a side effect, but the error is still propagated up the chain.
https://projectreactor.io/docs/core/release/reference/#_log_or_react_on_the_side

@Albertoimpl I'm considering closing this. Thoughts or concerns?

Agreed, we did some improvements recently, @royclarkson.

👍


Consider wrapping the plugin with jQuery closure

I got the errors:
Uncaught TypeError: Cannot read property 'fn' of undefined
TypeError: $ is undefined in jquery

Till I wrapped the plugin with the regular closure:
(function($) {

})(jQuery);

I wonder if you could consider wrapping it that way in the next release?
Thanks.

Definitely should've done that. Added in v0.1.17.


fixed camfib-axes for summary page, to match amp plots aesthetic

addresses issue #65
fixes camfiber axes so only Z cams have xaxis tick marks, matches the spacing on the summary page

Seems to work well on my end!


PowerShell - Windows Defender - Lack of documentation

[Enter feedback here]
Normally, Microsoft documentation for PowerShell commands for common products usually includes actual examples of using the commands, not just a link to the cmdlet.
This page seems like a waste of space if it's just a reference to the cmdlet.
There's even this enticing line at the top, "You can use PowerShell to perform various functions in Windows Defender." ... Yes, but what functions, and how, and where are the examples? And why am I even on this page if I just need to go read the cmdlet docs? Just redirect me there instead of wasting two paragraphs of my time to link me there. It goes on to say nothing about how exactly you can do that except generically referencing PowerShell.
I can figure it out, it's just crappy documentation, IMHO.
It would be helpful to maybe expand upon this like other modules.

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

ID: 65f5f37f-9e03-eac1-715b-4a01acfff33e
Version Independent ID: b7e76481-3859-c8d1-c22d-ee238fadd1e9
Content: Use PowerShell cmdlets to configure and run Windows Defender AV
Content Source: windows/security/threat-protection/windows-defender-antivirus/use-powershell-cmdlets-windows-defender-antivirus.md
Product: w10
Technology: windows
GitHub Login: @Dansimp
Microsoft Alias: dansimp

@officedocsbot assign @e0i

I'm kind of tired of seeing this on my profile and it's taking long enough that I just don't really care at this point so I'm closing it.


Allow to copy data at the level of a 1D spectrum

Like for range add a contextual right click menu on a 1D spectrum that allows to copy the data as a {x:[], y:[]} json. This would help to debug.

ok i will
في السبت، 25 أبريل 2020 في 7:17 م تمت كتابة ما يلي بواسطة Luc Patiny <
notifications@github.com>:

Assigned #394 https://github.com/cheminfo/nmr-displayer/issues/394 to
@hamed-musallam https://github.com/hamed-musallam.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/cheminfo/nmr-displayer/issues/394#event-3272114229,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIQ2Q3DPFDZTSTG364Y66Y3ROMLJRANCNFSM4MQ2SNKQ
.

--
Eng. Hamed S. Musallam
*Software Engineer *
Mobile:00972599870960


Feature/callback messages

Golang impl of https://github.com/flutter/engine/commit/6aad13910c9b2c6a86cf9e9b5ff80389bc97945e
I chose to break the plugin api:
@@ plugin/method-channel.go
-  _, err := MethodChannel.InvokeMethod("popRoute",nil)
+  err := MethodChannel.InvokeMethod("popRoute",nil)

@@ plugin/basic-message-channel.go
-  _, err := BasicMessageChannel.Send("xxx")
+  err = BasicMessageChannel.Send("xxx")

Why?
Peoples are used to not return result in their flutter setMethodCallHandler/setMessageHandler.
example:
  platform_channel
      .setMethodCallHandler((MethodCall methodCall) async {
          // do sutff

          // rarely calls:
          // return "42";
  });

With a InvokeMethod that expect a reply, the embedder will wait forever, (not blocking the UI but the behavior isn't normal).
The uses of InvokeMethodWithReply and SendWithReply is enough to let the user know he has to return a value in the flutter handler.
*WithReply is also used in the flutter/engine desktop shell, users will not get lost.

I have checked on github, the changes in the plugin api shouldn't affect anyone.


Account for Sidekiq vs. Flipper Notifications load order when configuring sidekiq_options

In #4 , I failed to consider Rails load order between Sidekiq and this gem.  The Flipper::Notifications::WebhookNotificationJob was being loaded before Sidekiq had added the sidekiq_options method to ActiveJob::Base.  Therefore the WebhookNotificationJob did not yet respond to sidekiq_options when it was trying to disable Sidekiq retries 😩 .
This PR hooks into the Flipper Notifications Railtie to disable sidekiq retries after the Rails application has finished initializing.

Sorry @erichmachado , I merged #4 without testing it as much as I should have 😖 .  I loaded this gem into our Rails app locally and confirmed that this configuration does override the default ActiveJob::Base sidekiq options.
./bin/rails c

> Flipper::Notifications::WebhookNotificationJob.sidekiq_options
{
    "retry" => 0,
    "queue" => "default"
}


host binary releases

It would be great to have the binary releases hosted on GitHub too to make it easier to download the latest one automatically.
https://github.com/VirusTotal/yara/releases
Also prebuilt releases for other OS could be useful imo.
Because the current solution with Dropbox and others is not ideal.
Also relevant:
https://github.com/VirusTotal/yara/issues/841
https://github.com/VirusTotal/yara/issues/993

@DanielRuf Isn't the releases on GitHub just right now? so what's wrong?

Also prebuilt releases for other OS could be useful imo.

Currently at least the Windows builds are now part of the releases page here on GitHub.
Closing as we can build the others with GHA.

Reopening since I see that there is a new build setup with GitHub Actions and the question: could we not upload the artifacts of these builds to GitHub for every release / tag?
https://github.com/VirusTotal/yara/blob/master/.github/workflows/build.yml

Still waiting for Linux binaries.


Hide bullboard routes in fastify adapter.

Reattempting https://github.com/felixmosh/bull-board/pull/762 with the typescript error ignored for this special case.

Ot still fails

It's eslint yelling about ts-ignore. I guess I'll ignore that too?

Just use as any on the type of schema obj

I've fixed it, thank you 🙏

Thank you! Sorry I ran out of time yesterday.


WA- Missing LOINC in OBX-1 and OBR-1

On 1/13, WA reported missing LOINC from OBX-1 and OBR-1 fields in their messages. Sample message IDs noted below.
0205ef2c-8505-4894-850b-5f15862bcacb
30a223bf-e0e7-4aef-9d98-7c29b846a942
2ba6b95c-2cdd-4233-b679-5741bcebfa49
c8adefc5-276f-4f13-aca7-606bbf65e56b
7fc5d08a-ca2b-4bcc-a3fd-985430c47765
fc4ea3c7-1554-4238-9eb5-242e733e1cc9
a0555cc6-b8d5-4799-b414-52bb9d144364
8ce54afd-cf36-4042-bfe5-b82180659045
c50c66a9-ca0d-4ac3-9515-aa590ea0ff22
3f9bb3ac-d1aa-4046-bfbc-7318e8d3995a
8c489c7b-1723-48c1-9b0d-4a945bdc7175
89f68423-3b83-4aa5-a77c-041bfd82d61a
f01caca7-ce0b-484c-a388-560a34e08ff4

@zdeveloper these are SimpleReport messages. Could you please resubmit these?

Done! resent 13 events.

2/11: WA was notified of resubmissions. Awaiting confirmation from state.


Refactor

This PR contains a refactor of the main code. It fixes many subtle issues that could occur in specific cases. In particular, it fixes #1 and closes #8. I also added type checking to the rule function, as suggested in https://github.com/pauladam94/curryst/issues/9#issuecomment-1961291650.
This PR also improves the README file, with a more detailed documentation of the different functions and better examples.
I also added tests, and updated the typst.toml file to add a link to this repository, as well as categories and disciplines. I updated the keywords and the package description to better match what a user would search.
Finally, this PR raises the minimum required compiler version to Typst 0.11.0, which is not released yet at the time of writing this, but should be in a few days.

Even if it's not in the classical coding habits, I prefer to keep the philosophy of having only one file for the entire project.
The goal is to be able to use the package by simply copy pasting this one file... and maybe modifying some things and learn about typst.
One of the things that made me appreciate Typst is being able to copy and paste someone else's random code and get the precise behavior I wanted 👍
So, keeping the code as simple as possible, especially in its structure, seems super important to me 👍

Otherwise, thank you very much ! It's great work ! The example are very nice, there are a lot of them and they explain well how to use the package.

Done!


  The extension tries to select the active Windows Codepage on win32 by default, and it may not be getting the correct result.

Does setting "cmake.outputLogEncoding": "utf8" fix it?  The extension tries to select the active Windows Codepage on win32 by default, and it may not be getting the correct result.

Does setting "cmake.outputLogEncoding": "utf8" fix it? The extension tries to select the active Windows Codepage on win32 by default, and it may not be getting the correct result.

It's not work as you expect,
  "cmake.outputLogEncoding": "gbk",

"cmake.outputLogEncoding": "utf8" cause encoding problem
The ninja is output with windows codepage. does cmake treat ninja output always to be utf8?
I think that works for non-win platform but not for windows

Does setting "cmake.outputLogEncoding": "utf8" fix it? The extension tries to select the active Windows Codepage on win32 by default, and it may not be getting the correct result.

It's not work as you expect,
  "cmake.outputLogEncoding": "gbk",

"cmake.outputLogEncoding": "utf8" cause encoding problem
The ninja is output with windows codepage. does cmake treat ninja output always to be utf8?
I think that works for non-win platform but not for windows

does cmake treat ninja output always to be utf8?

Unfortunately, I don't know the answer to that.  Does the output look correct when you run the cmake command in the VS Code Terminal, or a PowerShell/CMD prompt?

does cmake treat ninja output always to be utf8?

Unfortunately, I don't know the answer to that.  Does the output look correct when you run the cmake command in the VS Code Terminal, or a PowerShell/CMD prompt?

does cmake treat ninja output always to be utf8?

Unfortunately, I don't know the answer to that. Does the output look correct when you run the cmake command in the VS Code Terminal, or a PowerShell/CMD prompt?

Valid in PowerShell/CMD prompt, VS Code Terminal not working because the MSVC bat env not setup properly

does cmake treat ninja output always to be utf8?

Unfortunately, I don't know the answer to that. Does the output look correct when you run the cmake command in the VS Code Terminal, or a PowerShell/CMD prompt?

Valid in PowerShell/CMD prompt, VS Code Terminal not working because the MSVC bat env not setup properly

@taoyouh, @lygstate, if you would like, you can try this vsix (https://github.com/microsoft/vscode-cmake-tools/actions/runs/811030625) to get unblocked with a fix for this issue before we release 1.7.2.

CMake Tools 1.7.2 was released today and it contains a fix for this problem. Upgrade the extension and let us know if you encounter any other issues.

thanks, works now


[Module Proposal]: avm-res-servicebus-namespace (bicep) (res)

Check for previous/existing GitHub issues/module proposals

[X] I have checked for previous/existing GitHub issues/module proposals.

Check this module doesn't already exist in the module indexes

[X] I have checked for that this module doesn't already exist in the module indexes; or I'm proposing the module to be migrated from CARML/TFVM.

Bicep or Terraform?
Bicep
Module Classification?
Resource Module
Module Name
avm-res-servicebus-namespace
Module Details
Migrate from CARML
Do you want to be the owner of this module?
Yes
Module Owner's GitHub Username (handle)
ChrisSidebotham
(Optional) Secondary Module Owner's GitHub Username (handle)
No response

Hi @ChrisSidebotham ,
Thanks for requesting/proposing to be an AVM module owner!
We just want to confirm you agree to the below pages that define what module ownership means:

Team Definitions & RACI
Shared Specification (Bicep & Terraform)
Module Support

Any questions or clarifications needed, let us know!
If you agree, please just reply to this issue with the exact sentence below (as this helps with our automation 👍):
"I CONFIRM I WISH TO OWN THIS AVM MODULE AND UNDERSTAND THE REQUIREMENTS AND DEFINITION OF A MODULE OWNER"
Thanks,
The AVM Core Team
#RR

NOTE:

This Module Proposal issue MUST remain open until the module is fully developed, tested and published to the relevant registry. Do NOT close the issue before the successful publication is confirmed!
Once the module is fully developed, tested and published to the relevant registry, and the Module Proposal issue was closed, it MUST remain closed.

I CONFIRM I WISH TO OWN THIS AVM MODULE AND UNDERSTAND THE REQUIREMENTS AND DEFINITION OF A MODULE OWNER

Hi @ChrisSidebotham ,
Thanks for confirming that you wish to own this AVM module and understand the related requirements and responsibilities!
We just want to ask you to double check a few important things and take the next steps before you start the development.
Please use the following values explicitly as provided in the module index page:

For your module:

ModuleName - to name your module
TelemetryIdPrefix - to be used in your module's telemetry

For your module's repository:

Repository name and folder path are defined in RepoURL
Create the GitHub teams defined in the ModuleOwnersGHTeam and ModuleContributorsGHTeam columns and grant them permissions as described here.

In case of a Bicep module, please let the AVM core team know when you created these teams, and we'll help you with assigning the necessary permissions as described here.
In case of a Terraform module, follow the instructions here and assign the necessary permissions.

Grant the right level of permissions for  the AVM core team and PG teams on your GitHub repo as described here.
You can now start the development of this module! ✅ Happy coding! 🎉
Please respond to this comment, once your module is ready to be published! 🙏
Any further questions or clarifications needed, let us know!
Thanks,
The AVM Core Team

NOTE:

This Module Proposal issue MUST remain open until the module is fully developed, tested and published to the relevant registry. Do NOT close the issue before the successful publication is confirmed!
Once the module is fully developed, tested and published to the relevant registry, and the Module Proposal issue was closed, it MUST remain closed.

@ChrisSidebotham,
Due to a recent change in the naming convention for AVM Bicep modules, this module is getting an updated name.
The title and description of this GH issue has been updated to reflect the name change. Please make sure you adhere to the new naming convention when publishing your module!
Please see the new naming convention in:

RMNFR1 for resource modules
PMNFR1 for pattern modules

Please note that the Telemetry ID and name of GitHub Teams haven't changed.
If you feel uncertain about any details, please check the Bicep modules indexes.

PR Opened https://github.com/Azure/bicep-registry-modules/pull/670

https://github.com/Azure/bicep-registry-modules/pull/670 > Merged and Published @matebarabas


<cuda-runtime.h> not found while building netlrts-linux-x86_64-cuda builds

Autobuild log: http://charm.cs.illinois.edu/autobuild/old.2019_07_25__01_04/netlrts-linux-x86_64-cuda.txt
In file included from ampi.h:11:0,
                 from compat_ampius.C:2:
/scratch/autobuild/netlrts-linux-x86_64-cuda/charm/netlrts-linux-x86_64-cuda/bin/../include/hapi.h:3:26: fatal error: cuda_runtime.h: No such file or directory
 #include <cuda_runtime.h>
                          ^
compilation terminated.
/bin/cp -f sim.h ../../../../include/pose/sim.h

Any update on this? It's release-critical

This appears to be caused by the fix to bug #2062 (#2121). Not sure what the best way to fix this is. Not only is the include path missing, but also the library path and the -lcudart library.

One problem is the ordering of libraries being linked in with AMPI: -lcudahybridapi must appear before -lcudart because HybridAPI depends on the CUDA runtime library. Matt is working on a quick fix, but a better long-term solution would be to make HybridAPI a header-only library, which I believe should be possible.

Patch #2460 caused new linking issues.

After reverting the PR above (#2466), this issue has been resolved. The current cuda autobuild failure is tracked in #2467.


Possibly make way to batch active record without batching the query

I ran into this one just now, whereby I ran an order summary screen which shows all orders for the day along with their products.
I joined, naturally, with orders and products so that I got all orders and their products back in a single query:
Order::find()->joinWith('products')

After which I use each() which gives me the problem of actually getting duplicate orders back since an order might break batch halfway through its products.
One solution is, yes, to get products separately when I display the order, like so:
    $products = $order->getProducts()->joinWith(['product', 'product.title'])->all();

But this hits into a wall. The whole reason I am doing batching is because of memory. I do not wish my script to exceed 134MB RAM for just 1067 orders (for example) but the only way is via batching the active record models (since that is what takes all the memory). Doing these separately in the foreach seems to cause a slow but sure memory leakage (I am currently looking into how to properly destruct the products var).
Even doing it this way (which does work) causes much higher memory usage than if I could just do:
Order::find()->joinWith('products')

What would be good is a way to get the query all in one and store it in RAM (most servers can allow like 50K results to be stored in RAM) but batch the active model.
What do you think?

I just found this as looking back, somehow I overlooked it sorry.
Since this page no longer exists in this form on my site I cannot actually test it now, but that does sound like the reason, so I will say that, as you noted in the tags, the doc probably just need updating on the subject a little.

Closing since resolved with distinct and documentation of framework query builder should rather not try to show how to solve SQL problem, but what tools are available to solve it.

Closing since resolved with distinct and documentation of framework query builder should rather not try to show how to solve SQL problem, but what tools are available to solve it.


move stepper package location to outside cloud directory

Summary
I plan to use it for improving the "install package" UX in https://github.com/jetpack-io/devbox/pull/491
How was it tested?
compiles

Current dependencies on/for this PR:

main

PR #495   👈

This comment was auto-generated by Graphite.


get-data --datastore operational with a subtree filter does not appear to be returning the correct data

Using the model for issue1, I made the configuration nodes be "in use" and also setup an operational callback for the "serial-num".
> get-data --datastore operational --filter-xpath /issue1:*
DATA
<data xmlns="urn:ietf:params:xml:ns:yang:ietf-netconf-nmda">
  <hardware xmlns="i1">
    <component>
      <name>ComponentName</name>
      <class>O-RAN-RADIO</class>
      <serial-num>1234</serial-num>
      <feature>
        <wireless>true</wireless>
      </feature>
    </component>
  </hardware>
</data>

Now I try using the same subtree filter as in the test_get_operational_data.   The filter looks like
brett@brett-Precision-7740:~/git/netconf/sysrepo/build$ cat /home/brett/git/netconf/filter.xml
<hardware xmlns="i1">
    <component>
      <name>ComponentName</name>
      <class>O-RAN-RADIO</class>
      <serial-num>1234</serial-num>
      <feature>
        <wireless>true</wireless>
      </feature>
    </component>
  </hardware>
brett@brett-Precision-7740:~/git/netconf/sysrepo/build$

I now try using that filter

> get-data --datastore operational --filter-subtree=/home/brett/git/netconf/filter.xml
DATA
<data xmlns="urn:ietf:params:xml:ns:yang:ietf-netconf-nmda"/>

>

This does not return anything.
Using get with the same filter does return correctly
> get  --filter-subtree=/home/brett/git/netconf/filter.xml
DATA
<data xmlns="urn:ietf:params:xml:ns:netconf:base:1.0">
  <hardware xmlns="i1">
    <component>
      <name>ComponentName</name>
      <class>O-RAN-RADIO</class>
      <serial-num>1234</serial-num>
      <feature>
        <wireless>true</wireless>
      </feature>
    </component>
  </hardware>
</data>

However using get is not performant when there are callbacks to be invoked and the advise is to use get-data as the replacement.

This filter works
<hardware xmlns="i1">
    <component>
      <name>ComponentName</name>
      <class></class>
      <serial-num></serial-num>
      <feature>
              <wireless></wireless>
      </feature>
    </component>
  </hardware>

and return
> get-data --datastore operational  --filter-subtree=/home/brett/git/netconf/filter.xml
DATA
<data xmlns="urn:ietf:params:xml:ns:yang:ietf-netconf-nmda">
  <hardware xmlns="i1">
    <component>
      <name>ComponentName</name>
      <class>O-RAN-RADIO</class>
      <serial-num>1234</serial-num>
      <feature>
        <wireless>true</wireless>
      </feature>
    </component>
  </hardware>
</data>

>

As soon as you add a value to be filtered on like
<hardware xmlns="i1">
    <component>
      <name>ComponentName</name>
      <class></class>
      <serial-num></serial-num>
      <feature>
              <wireless>true</wireless>
      </feature>
    </component>
  </hardware>

Then the class and serial-num are no longer returned in the output
DATA
<data xmlns="urn:ietf:params:xml:ns:yang:ietf-netconf-nmda">
  <hardware xmlns="i1">
    <component>
      <name>ComponentName</name>
      <feature>
        <wireless>true</wireless>
      </feature>
    </component>
  </hardware>
</data>

>

Strangely this filter works
<hardware xmlns="i1">
    <component>
      <name>ComponentName</name>
      <class>O-RAN-RADIO</class>
      <serial-num></serial-num>
      <feature>
              <wireless></wireless>
      </feature>
    </component>
  </hardware>

get-data --datastore operational  --filter-subtree=/home/brett/git/netconf/filter.xml
DATA

ComponentName
O-RAN-RADIO
1234

true

Can you please attach the YANG module(s)? No idea what issue1 is. Also, what versions are you using, of netopeer2 at least?

libyang v2.0.112
sysrepo v2.0.53
Netopeer2 v2.0.35
The issue1.yang is part of the Netopeer2 tests
https://github.com/CESNET/netopeer2/blob/master/tests/modules/issue1.yang
The test is basically this one
https://github.com/CESNET/netopeer2/blob/e6cdbb912cb82b81d7698a3f3d6c15a17f805cc4/tests/test_filter.c#L825
but trying to use get-data instead of get.   There seem to be no tests for get-data.
I will see if I can write a test case using get-data and this module that is equivalent to test_get_operational_data test.

I added this pull request with a new test.  The test fails because I think the filtering is not working correctly at this point.
https://github.com/CESNET/netopeer2/pull/1105

Okay, thanks for the test (they are obviously needed for get-data) and it should work now.


Need to add form validation

Potentially use https://vuetifyjs.com/en/components/forms#vee-validate to add validation to forms app-wide

@jtomeck, is this more of a Jared task or more of a Roni task?


Accordion enhancements

Change icon to chevron ("down" on default, "up" when opened) EDIT: Breaking change so icon is not changed now
Text is dark red in expanded state (dark red is also the hover color)
Child closer to parent
More space below the child.

Moved to trello


Fix typo

@el-mapache hey, the package is broken by this commit. @koddsson @keithamus @manuelpuyol

Thanks for this, both of you. I've merged https://github.com/github/textarea-autosize/pull/34 and releasing as v0.3.1. Sorry that we shipped a broken release :disappointed:


Black background

Current bug behaviour

Black background showed before my game.
Expected behaviour

I want to remove black background.
Steps to reproduce

Open game
Black background showed
Flutter doctor output

Output of: flutter doctor -v

More environment information

Log information

Enter log information in this code block

More information

This isn't a bug, the black background is just the default color, you can change that by overriding the backgroundColor method on your Game class and return the desired color instead.


Prepare "production" build of React app

We should try publishing it in "release" mode (as opposed to "dev") and see how big it is and how it looks (CSS/JS/WASM). Then we can see if it's ok or that we should work on trying to make it smaller.
This also includes minimizing the compiled WASM size (i.e. using -c flag to ask compiler to optimize for generated code size).

It looks like the resulting JS weights about 230k:

115k - react-dom
70k - lodash
11k - my own code
~35k - the rest

This data can be visualized with tree-maps with source-map-explorer as explained here: https://create-react-app.dev/docs/analyzing-the-bundle-size
WASM file is quite large: 315k
Perhaps we should invest some effort in making it smaller.


Getting translation alternatives via API

deepl.com shows me multiple alternatives for a translation:

Is this feature available via the API? I see that deepl.com sends a jsonrpc to retrieve this, but I couldn't find any docs on how to use this endpoint.

Hi @nielsrolf Niels, thanks for creating this issue.
No, for business reasons the DeepL API does not provide the alternative translations.

This is unfortunate. It would be great it this was referenced in the documentation as I spent a bit of time trying to figure out how to return alternate translations.
(For reference: I'm building a flash card app, and wanted to provide suggested translations. Providing a single response is insufficient for words with multiple meanings.)


Update RELEASE.md with Release Manager checklist

Motivation
Based on our June 28, 2022 standup call, it was identified that we should have a checklist for release managers to ensure that no steps are missed during the process of release and deployment.
Description
This list reiterates the steps already outlined, but also include steps related to our internal operations such as testing and stable deployments.

can you merge the latest unstable on top? there seem to be some stray files updated here


Handle url query strings with net/url package

Description
This PR replaces our old method of putting together query strings with a more robust way that is less error prone. Now we are using the net/url package to resolve any inconsistencies for example with '/' at the end of the string or no '/'. This PR also includes:

Unit testing for URI formatting
GitHub plugin refactor to pass query params separate from relative path string
GitLab plugin refactor for the same as GitHub

Does this close any open issues?
https://github.com/merico-dev/lake/issues/804

LGTM


docs(readme): app structure update for better accuracy

Current coverage is 70.58% (diff: 100%)

Merging #952 into master will not change coverage

@@             master       #952   diff @@
==========================================
  Files            15         15
  Lines            85         85
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits             60         60
  Misses           25         25
  Partials          0          0

Powered by Codecov. Last update 3fabc5f...fdbf987

Looks good, thanks!


when I run: yarn add postcss-pxtorem, I get a warning " ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>

This is my project`s package.json.

The waning

This is this postcss-pxtorem`s package.json

This is this postcss`s package.json

@cuth How to solve this problem? Do you need to modify the source code?

postcss is built in vite. I don't want to reinstall postcss.

Try adding Postcss to the package.json of your project.

@cuth It works。Thanks

Help.
I got the same problem when running with npm: Syntax Error: Error: PostCSS plugin postcss-pxtorem requires PostCSS 8.
And then I add postcss v8.3.0 to the package.json, but it does't work.

PostCSS plugin postcss-pxtorem requires PostCSS 8.

I downloaded postcss-pxtorem@lastest，postcss@lastest，postcss-loader@lastest，But still report an error.


the vnc video of linux screen is unclear by examples/client

https://user-images.githubusercontent.com/25843983/184138612-92ed3744-6d17-4dc7-ba15-005537965b0a.mp4
What should be configured to fix this issue?

encoding problem


ESP32-S2: Add SPI Slave support

Thank you for your contribution!
We appreciate the time and effort you've put into this pull request.
To help us review it efficiently, please ensure you've gone through the following checklist:
Submission Checklist 📝

[x] I have updated existing examples or added new ones (if applicable).
[x] I have used cargo xtask fmt-packages command to ensure that all changed code is formatted correctly.
[x] My changes were added to the CHANGELOG.md in the proper section.

Extra:

[x] I have read the CONTRIBUTING.md guide and followed its instructions.

Pull Request Details 📖
Description
Adds SPI Slave for ESP32S2 and removes ESP32 parts from it.
Testing
Tested with spi_slave_dma example.
Builds on top of #1514 but removed all ESP32-related changes from the driver. After discussion with @bjoernQ, we agreed it makes sense to add just S2 and remove ESP32 parts, we've burnt so much time on that and the thing doesn't work.

Given that the example is almost a test, could we add some test cases? I think we will need some hardware changes as we'll need more pins wired up.

Given that the example is almost a test, could we add some test cases? I think we will need some hardware changes as we'll need more pins wired up.

This will be fun. See https://github.com/esp-rs/esp-hal/issues/1332#issuecomment-2059093479 to have some reference when choosing the new pins so we can avoid having too many cfgs arround

This will be fun. See https://github.com/esp-rs/esp-hal/issues/1332#issuecomment-2059093479 to have some reference when choosing the new pins so we can avoid having too many cfgs arround

Maybe we can do this in another PR if it will be a pain, but it seems like something we can test in the future at least which would be nice!

This will be fun. See #1332 (comment) to have some reference when choosing the new pins so we can avoid having too many cfgs arround

Maybe we can do this in another PR if it will be a pain, but it seems like something we can test in the future at least which would be nice!

Let's add this in another PR, Tomas is on leave this week and we need to buy cables as well, in a meantime I can try to figure out the pins.


_valueAsNumber can't be found

In the documentation, demo, and tests I could not find any reference to how to extract the numeric value of the input through the use of the apparent _valueAsNumber
<NumericInput ref="myNumericInput"/>

this.refs.myNumericInput._valueAsNumber
>>undefined
this.refs.myNumericInput.valueAsNumber
>>undefined
this.refs.myNumericInput.refs.input.valueAsNumber
>>NaN
this.refs.myNumericInput.refs.input._valueAsNumber
>>undefined
this.refs.myNumericInput.refs.input.value
>>"5.00"

Can you please indicate how to use the _valueAsNumber  property ?

That is correct. The documentation for that is outdated and I will fix it ASAP. That property is now replaced with getValueAsNumber() function. Something like this should work:
this.refs.myNumericInput.refs.input.getValueAsNumber()


Add ERC: Minimal Modular Smart Accounts

This proposal outlines the minimally required interfaces and behavior for modular smart accounts and modules to ensure interoperability across implementations.

Hi @SamWilsn just a gentle poke to review and merge this if it looks good.


Torch with an AMD GPU: 'repeats can not be negative'

Hi, I just tried running the demo on an AMD 6900 XT with torch 2.0.1+rocm5.3 installed. It runs fine if I set everything to use the cpu, but with cuda() I get the following error:
In [38]: matches01 = matcher({'image0': feats0, 'image1': feats1})
---------------------------------------------------------------------------
RuntimeError                              Traceback (most recent call last)
Cell In[38], line 1
----> 1 matches01 = matcher({'image0': feats0, 'image1': feats1})

File ~/.pyenv/versions/3.10.13/lib/python3.10/site-packages/torch/nn/modules/module.py:1501, in Module._call_impl(self, *args, **kwargs)
   1496 # If we don't have any hooks, we want to skip the rest of the logic in
   1497 # this function, and just call forward.
   1498 if not (self._backward_hooks or self._backward_pre_hooks or self._forward_hooks or self._forward_pre_hooks
   1499         or _global_backward_pre_hooks or _global_backward_hooks
   1500         or _global_forward_hooks or _global_forward_pre_hooks):
-> 1501     return forward_call(*args, **kwargs)
   1502 # Do not call functions when jit is used
   1503 full_backward_hooks, non_full_backward_hooks = [], []

File ~/.pyenv/versions/3.10.13/lib/python3.10/site-packages/lightglue/lightglue.py:440, in LightGlue.forward(self, data)
    419 """
    420 Match keypoints and descriptors between two images
    421
   (...)
    437     matches: List[[Si x 2]], scores: List[[Si]]
    438 """
    439 with torch.autocast(enabled=self.conf.mp, device_type="cuda"):
--> 440     return self._forward(data)

File ~/.pyenv/versions/3.10.13/lib/python3.10/site-packages/lightglue/lightglue.py:476, in LightGlue._forward(self, data)
    474 desc1 = self.input_proj(desc1)
    475 # cache positional embeddings
--> 476 encoding0 = self.posenc(kpts0)
    477 encoding1 = self.posenc(kpts1)
    479 # GNN + final_proj + assignment

File ~/.pyenv/versions/3.10.13/lib/python3.10/site-packages/torch/nn/modules/module.py:1501, in Module._call_impl(self, *args, **kwargs)
   1496 # If we don't have any hooks, we want to skip the rest of the logic in
   1497 # this function, and just call forward.
   1498 if not (self._backward_hooks or self._backward_pre_hooks or self._forward_hooks or self._forward_pre_hooks
   1499         or _global_backward_pre_hooks or _global_backward_hooks
   1500         or _global_forward_hooks or _global_forward_pre_hooks):
-> 1501     return forward_call(*args, **kwargs)
   1502 # Do not call functions when jit is used
   1503 full_backward_hooks, non_full_backward_hooks = [], []

File ~/.pyenv/versions/3.10.13/lib/python3.10/site-packages/lightglue/lightglue.py:73, in LearnableFourierPositionalEncoding.forward(self, x)
     71 cosines, sines = torch.cos(projected), torch.sin(projected)
     72 emb = torch.stack([cosines, sines], 0).unsqueeze(-3)
---> 73 return emb.repeat_interleave(2, dim=-1)

RuntimeError: repeats can not be negative

Edit:
This isn't a lightglue issue. See https://github.com/pytorch/pytorch/issues/103973, https://github.com/pytorch/pytorch/issues/110532, https://github.com/RadeonOpenCompute/ROCm/issues/2055.
1.13.1+rocm5.2 works okay for the time being.


I2C Workflow

[x] I2C Component (bus) Initialization
[x] I2C Scan
[x] I2C Device Initialization
[x] I2C Device Deletion
[x] I2C Device Update
[x] I2C Device Publishing

I2C workflow completed with https://github.com/adafruit/Adafruit_Wippersnapper_Arduino/pull/189


Fix a bug in topk_op.cc

307df7080a8fe0538e05754170695b9925e2cea2 is causing failures in
http://ci.tensorflow.org/job/tf-master-win-bzl/1110/console :
03:15:56 //py_test_dir/tensorflow/python/kernel_tests:metrics_test                FAILED in 3 out of 3 in 54.3s
03:15:56 //py_test_dir/tensorflow/python/kernel_tests:topk_op_test                FAILED in 3.6s

The test output doesn't show useful information:
03:13:23 FAIL: //py_test_dir/tensorflow/python/kernel_tests:metrics_test (shard 3 of 3) (see C:/tmp/_bazel_system/424zmya1/execroot/tf-master-win-bzl/bazel-out/msvc_x64-py3-opt/testlogs/py_test_dir/tensorflow/python/kernel_tests/metrics_test/shard_3_of_3/test.log)
03:13:23 INFO: From Testing //py_test_dir/tensorflow/python/kernel_tests:metrics_test (shard 3 of 3):
03:13:23 ==================== Test output for //py_test_dir/tensorflow/python/kernel_tests:metrics_test (shard 3 of 3):
03:13:23 .....C:\Program Files\Anaconda3\lib\site-packages\tensorflow\python\util\tf_inspect.py:45: DeprecationWarning: inspect.getargspec() is deprecated, use inspect.signature() instead
03:13:23   if d.decorator_argspec is not None), _inspect.getargspec(target))
03:13:23 ....................\\?\c:\tmp\Bazel.runfiles_tbl0oj21\runfiles\org_tensorflow\py_test_dir\tensorflow\python\kernel_tests\metrics_test.py:163: DeprecationWarning: Please use assertEqual instead.
03:13:23   set(expected), set(v.name for v in variables.local_variables()))
03:13:23 ..................================================================================================

But when running the test locally I saw:

This change fixed them.
@gunan

http://ci.tensorflow.org/job/tensorflow-pr-win-bazel/26/console

LGTM
On Jun 16, 2017 5:28 AM, "Yun Peng"<EMAIL_ADDRESS>wrote:

http://ci.tensorflow.org/job/tensorflow-pr-win-bazel/26/console
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/pull/10762#issuecomment-309013656,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABtim0ecewZtPZ9Yj5aLM6z5KZboMWFpks5sEnTdgaJpZM4N8ZIO
.


InvalidRequestError: The model: gpt-4 does not exist

NOTE: I have a gpt-4 (paid) account
final cell returned:
generate_optimal_prompt(description, test_cases, NUMBER_OF_PROMPTS, use_wandb)

InvalidRequestError                       Traceback (most recent call last)
 in <cell line: 1>()
----> 1 generate_optimal_prompt(description, test_cases, NUMBER_OF_PROMPTS, use_wandb)
6 frames
/usr/local/lib/python3.10/dist-packages/openai/api_requestor.py in _interpret_response_line(self, rbody, rcode, rheaders, stream)
761         stream_error = stream and "error" in resp.data
762         if stream_error or not 200 <= rcode < 300:
--> 763             raise self.handle_error_response(
764                 rbody, rcode, resp.data, rheaders, stream_error=stream_error
765             )
InvalidRequestError: The model: gpt-4 does not exist

To check what models you have access to:
curl https://api.openai.com/v1/models -H "Authorization: Bearer sk-your-openai-key" | grep -i "id"


PR for merging Frontend and Backend

Issue #110
Commit References: https://github.com/roshansadath/CareerPaths-SOEN6011Summer2023/commit/117f989a33c4c45c5e34267a65a681f3a56aaa26 https://github.com/roshansadath/CareerPaths-SOEN6011Summer2023/commit/0f9b07a6e1ecd07ee4d91e5bfce51bb0f91bceff https://github.com/roshansadath/CareerPaths-SOEN6011Summer2023/commit/bbd98cc0e138d93a4d06144992d1e652e9a86988 https://github.com/roshansadath/CareerPaths-SOEN6011Summer2023/commit/5de027e903adb82be86ec5dd390a1dd6a752e433 https://github.com/roshansadath/CareerPaths-SOEN6011Summer2023/commit/9e174c23da01863c63ef5b0bceb4f95ba8463477 https://github.com/roshansadath/CareerPaths-SOEN6011Summer2023/commit/6e5f0b60cc2727db7c6e78d6bac06d76ff6fc24b

looks good

Looks good!


Wrong AssemblyVersion

The AssemblyVersion attribute is not set correctly in the NuGet package for version 2.7.1; it is set to <IP_ADDRESS>, which is a bit odd when setting binding redirects from an older version with a correct version.

I think this has already been addressed in issue #200 that I raised and was fixed with #201.
Although I think this build has a different key from the one published previously on nuget.org as V2.5.0 so you can't use binding redirects for that version.

Yes sorry for the version - it was working at some point but we apparently didn't check when we actually released it since the NuGet version was correct. I'm closing this issue as it has been fixed.


Fix playwright tests

Description
Fix the Playwright tests
Related Issue
The playwright tests were failing because of a change in the layout.
How Has This Been Tested?
These are tests
Types of changes

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

Checklist:

[X] I have signed the Adobe Open Source CLA.
[X] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[X] I have read the CONTRIBUTING document.
[X] I have added tests to cover my changes.
[X] All new and existing tests passed.

Playwright tests are fixed now.
The unit tests are still failing due to some date formatting issue. I'll file a separate PR for that 😄


Update README prior to release

Review it in general, but specifically for:

latest features
roadmap - maybe a generic link to milestones and issues?

Closed by #640


[TC-425] TPv2 - add the ability to manage federations

In order to achieve feature parity with the existing TO UI, TPv2 must support the management of Federations.
Author: Jeremy Mitchell
JIRA Link: https://issues.apache.org/jira/browse/TC-425

duplicate


Fixes to celldive_deepcell DAG found in testing, plus some pylint

This PR contains the code to add the metadata data_path to the data directory, plus some pylint cleanup.

I also turned off --singularity, at least for now.

Hah - this was formatted through black and isort, so apparently pylint and black disagree about indentation of arguments (after with DAG(), and pylint and isort disagree about import order 🙂
Looks good to me though.


Add test for refund_order function of facade module

Closes #7

Looks good! Please create PR directly into django-oscar-paypal.

Closed because of https://github.com/django-oscar/django-oscar-paypal/pull/253


Add slider_perf_android task to ci.yaml

This adds devicelab task slider_perf_android to .ci.yaml, which was added in https://github.com/flutter/flutter/pull/125296
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[ ] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Looks like this needs test owner ship defined in https://github.com/flutter/flutter/blob/master/TESTOWNERS
# Test ownership check failed.
Builders missing owner: [Linux_android slider_perf_android]
Please define ownership in https://github.com/flutter/flutter/blob/master/TESTOWNERS

Thank you! I'm adding this task for the first time.


Improve typography for links

Improves typography for links for browsers that support this feature.
See the results for all links including collapsible, accordion etc...
Closes #1306

 Improve typography rendering of links for modern browsers

 Improve typography rendering of links for modern browsers


Colab installation issues

Hello,
I'm trying to run this project on Colab, but when I run "python setup.py develop" I get this problem:
No CUDA runtime is found, using CUDA_HOME='/usr/local/cuda' running develop running egg_info writing pcdet.egg-info/PKG-INFO writing dependency_links to pcdet.egg-info/dependency_links.txt writing requirements to pcdet.egg-info/requires.txt writing top-level names to pcdet.egg-info/top_level.txt adding license file 'LICENSE' writing manifest file 'pcdet.egg-info/SOURCES.txt' running build_ext /usr/local/lib/python3.8/dist-packages/torch/utils/cpp_extension.py:387: UserWarning: The detected CUDA version (11.2) has a minor version mismatch with the version that was used to compile PyTorch (11.6). Most likely this shouldn't be a problem.   warnings.warn(CUDA_MISMATCH_WARN.format(cuda_str_version, torch.version.cuda)) building 'pcdet.ops.votr_ops.votr_ops_cuda' extension /usr/local/lib/python3.8/dist-packages/torch/cuda/__init__.py:497: UserWarning: Can't initialize NVML   warnings.warn("Can't initialize NVML") Traceback (most recent call last):   File "setup.py", line 34, in <module>     setup(   File "/usr/local/lib/python3.8/dist-packages/setuptools/__init__.py", line 153, in setup     return distutils.core.setup(**attrs)   File "/usr/lib/python3.8/distutils/core.py", line 148, in setup     dist.run_commands()   File "/usr/lib/python3.8/distutils/dist.py", line 966, in run_commands     self.run_command(cmd)   File "/usr/lib/python3.8/distutils/dist.py", line 985, in run_command     cmd_obj.run()   File "/usr/local/lib/python3.8/dist-packages/setuptools/command/develop.py", line 34, in run     self.install_for_development()   File "/usr/local/lib/python3.8/dist-packages/setuptools/command/develop.py", line 136, in install_for_development     self.run_command('build_ext')   File "/usr/lib/python3.8/distutils/cmd.py", line 313, in run_command     self.distribution.run_command(command)   File "/usr/lib/python3.8/distutils/dist.py", line 985, in run_command     cmd_obj.run()   File "/usr/local/lib/python3.8/dist-packages/setuptools/command/build_ext.py", line 79, in run     _build_ext.run(self)   File "/usr/local/lib/python3.8/dist-packages/Cython/Distutils/old_build_ext.py", line 186, in run     _build_ext.build_ext.run(self)   File "/usr/lib/python3.8/distutils/command/build_ext.py", line 340, in run     self.build_extensions()   File "/usr/local/lib/python3.8/dist-packages/torch/utils/cpp_extension.py", line 843, in build_extensions     build_ext.build_extensions(self)   File "/usr/local/lib/python3.8/dist-packages/Cython/Distutils/old_build_ext.py", line 195, in build_extensions     _build_ext.build_ext.build_extensions(self)   File "/usr/lib/python3.8/distutils/command/build_ext.py", line 449, in build_extensions     self._build_extensions_serial()   File "/usr/lib/python3.8/distutils/command/build_ext.py", line 474, in _build_extensions_serial     self.build_extension(ext)   File "/usr/local/lib/python3.8/dist-packages/setuptools/command/build_ext.py", line 202, in build_extension     _build_ext.build_extension(self, ext)   File "/usr/lib/python3.8/distutils/command/build_ext.py", line 528, in build_extension     objects = self.compiler.compile(sources,   File "/usr/local/lib/python3.8/dist-packages/torch/utils/cpp_extension.py", line 649, in unix_wrap_ninja_compile     cuda_post_cflags = unix_cuda_flags(cuda_post_cflags)   File "/usr/local/lib/python3.8/dist-packages/torch/utils/cpp_extension.py", line 548, in unix_cuda_flags     cflags + _get_cuda_arch_flags(cflags))   File "/usr/local/lib/python3.8/dist-packages/torch/utils/cpp_extension.py", line 1780, in _get_cuda_arch_flags     arch_list[-1] += '+PTX' IndexError: list index out of range
Do you know how to solve it?
Thanks in advance

This seems to be an issue caused by your cuda environment and pytorch version, so please try again using the same setup environment as in readme:
Ubuntu 18.04
Python 3.9.13
PyTorch 1.8.1
Spconv 2.1.22 # pip install spconv-cu111
NVIDIA CUDA 11.1

The problem was that I forgot to initialize the runtime with a GPU.
There was also an issue with the pytorch version, but I solved installing an older version:
! pip install torch==1.8.1+cu111 torchvision==0.9.1+cu111 torchaudio==0.8.1 -f https://download.pytorch.org/whl/torch_stable.html
Now setup.py seems to work on Colab


Move this repo to tc39-transfer

Since Intl.DurationFormat is at Stage 1, this repo should be moved to the tc39-transfer org such that the chairs can move it to tc39.

@younies can you please do this or give me access to this repo so I could do it? Thanks.

Thanks @ryzokuken .... I have just transferred it :)


Think about migrating to the sbt-organization

As your plugin is the defacto standard for sbt-junit5, please think about if you want to migrate it to the sbt organization.
github.com/sbt/junit5-interface would be a fitting name (IMHO).
please see this issue for reference: https://github.com/sbt/junit-interface/issues/75
This is written on the offical sbt page (https://www.scala-sbt.org/1.x/docs/Community-Plugins.html)

The sbt organization is available for use by any sbt plugin. Developers who contribute their plugins into the community organization will still retain control over their repository and its access. The goal of the sbt organization is to organize sbt software into one central location.

Hi and thank you for bringing this to my attention!
If it helps, I'll gladly do this. I'm not sure what's exactly involved, though. Is it just about requesting the project to be moved to github.com/sbt/ ? The documentation seems a bit dated with its references to bintray !?

If you could transfer the repo to me, I can transfer it to sbt orgazation and then create a team with you in it who would own the repo.

Great! Transfer has been initiated.

Thanks. The transfer is complete, and I've invited you to https://github.com/orgs/sbt/teams/junit5-interface-team, which has full rights on this repo.


Apply website doc changes after 3.15.0 release

Website/doc changes after Camel 3.15.0 release

@djencks Thanks for your feedback. I have made the requested updates, could you please take a quick look?

The updates here look good, but we need similar changes in camel-spring-boot and camel-karaf.  I've made what's needed for the docs in https://github.com/apache/camel-spring-boot/pull/434 and https://github.com/apache/camel-karaf/pull/113 and set up a preview build with https://github.com/apache/camel-website/pull/775.  The website PR has 2 commits: after we like the preview the 3 content branches need to be merged, the 2nd commit removed from the preview PR and the first commit merged.  I hope we can make this process simpler soon!!!

The updates here look good, but we need similar changes in camel-spring-boot and camel-karaf. I've made what's needed for the docs in apache/camel-spring-boot#434 and apache/camel-karaf#113 and set up a preview build with apache/camel-website#775. The website PR has 2 commits: after we like the preview the 3 content branches need to be merged, the 2nd commit removed from the preview PR and the first commit merged. I hope we can make this process simpler soon!!!

Thanks for reviewing and making the necessary changes in the Spring Boot and Karaf subprojects. Can I go ahead and merge this PR?


Add promisc option

https://github.com/rancher/rancher/issues/9090

Ran validation tests and there are no major failures caused due to this change. Only known or irrelevant failures are seen.
Test Result (10 failures )
cattlevalidationtest.core.test_container.test_ip_inject
cattlevalidationtest.core.test_rancher_cli.test_cli_catalog_list
cattlevalidationtest.core.test_rancher_cli.test_cli_env_create_rm_cattle
cattlevalidationtest.core.test_rancher_cli.test_cli_env_create_rm_kubernetes
cattlevalidationtest.core.test_rancher_cli.test_cli_env_create_rm_swarm
cattlevalidationtest.core.test_rancher_cli.test_cli_env_create_rm_mesos
cattlevalidationtest.core.test_rancher_cli.test_cli_env_deactivate_activate
cattlevalidationtest.core.test_rancher_compose_inservice_upgrade.test_rancher_compose_inservice_upgrade_retainip_during_upgrade_rollback
cattlevalidationtest.core.test_rancher_compose_inservice_upgrade.test_rancher_compose_inservice_upgrade_remove_sk_rollback
cattlevalidationtest.core.test_service_selector.test_selectorLink_lbservice

Verified the following:
Steps: (On a AWS cluster running rancher/server:1.5.10)

Launch a test stack with a test service (leodotcloud/swiss-army-knife:dev) enabling a published port 20001:80/tcp.
From inside the container, verified the URL is accessbile: http://{IP_ADDR_OF_CONTAINER}
From inside the container, verified the URL is accessible: http://{PRIVATE_IP_OF_HOST}:20001
From inside the container, verified the URL is accessible: http://{PUBLIC_IP_OF_HOST}:20001
From the host CLI, verified the URL is accessible: http://{PRIVATE_IP_OF_HOST}:20001
From the host CLI, verified the URL is accessible: http://{PUBLIC_IP_OF_HOST}:20001
From a different container on the same host, verified the URL is accessible: http://{PRIVATE_IP_OF_HOST}:20001
From a different container on the same host, verified the URL is accessible: http://{PUBLIC_IP_OF_HOST}:20001


fix: Switching Carbon tabs can cause content to scroll off-viewport

carbon-components-react 7.23.0 (versus 7.22.0) had a few DOM-altering changes:

a -> button for Tab child
[aria-hidden=true] => [hidden]

This required a few changes to CSS rules and test selectors.
Fixes #6014

Description of what you did:

My PR is a:

[ ] 💥 Breaking change
[x] 🐛 Bug fix
[ ] 💅 Enhancement
[ ] 🚀 New feature

Please confirm that your PR fulfills these requirements

[x] Multiple commits are squashed into one commit.
[x] The commit message follows Conventional Commits, which allows us to autogenerate release notes; e.g. fix(plugins/plugin-k8s): fixed annoying bugs
[x] All npm dependencies are pinned.

sigh, carbon 7.23 deprecated Tab props, but itself persisted in using those props.
https://github.com/carbon-design-system/carbon/issues/7237
who knows when that fix will be released...


Update AVS intro articles

Before selecting the "Create pull request" button:

Enter a meaningful title above^, using a prefix if necessary and keywords "New" or "Update" indicating the nature of changes.

Describe the summary, scope, and intent of this PR:
Freshness update for main AVS CAF articles, index.md and eslz intro.

@Zimmergren
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


[occm] make floating subnet optional

What this PR does / why we need it:
In PR  floating subnet was made mandatory, for backward compatibility we should make it optional again. I have also added docs to clearly specify that floating-subnet is an optional argument.
Release note:

NONE

/assign @lingxiankong

/ok-to-test

/override openlab/cloud-provider-openstack-acceptance-test-csi-cinder
/override openlab/cloud-provider-openstack-acceptance-test-csi-manila

/lgtm
/approve

/retest

/retest

/retest

/retest

/retest

/retest


docs(demo): separate root traffic split service from backend

These changes update the manual demo to use the more standard method of
traffic splitting, where the root service is not also a backend service.
To do this, the current bookstore service has been renamed to
bookstore-v1 and a new bookstore service will serve as the root for
the TrafficSplit.
The only change in the overall steps of the demo is a new step to update
the BOOKSTORE_SVC environment variable in the bookbuyer and
bookthief from bookstore-v1 to bookstore right before creating the
first TrafficSplit.

This is a subset of the changes from #195. When following along with these changes, make sure to use the demo manifests from this specific commit instead of what appears in the doc: https://github.com/openservicemesh/osm/pull/4180/commits/8eb99b2fcf3591904263d28a3f796655242481f6. These changes will be merged to the osm repo after this PR.

After talking with @shashankram and @michelleN offline, keeping the demo as-is for now seems like the better option, so I'll close this PR.
@shashankram, I'll make sure to take the comments you've made here re: the table and links and apply them to #195.

After talking with @shashankram and @michelleN offline, keeping the demo as-is for now seems like the better option, so I'll close this PR.
@shashankram, I'll make sure to take the comments you've made here re: the table and links and apply them to #195.

@nojnhuh This change looks good to me. I think we could use this one and also add additional documentation around the root apex service. Regardless, we need to clearly document the caveats that exist with using SMI traffic split (such as needing to change the service FQDN).
What was the reason to close this?

What was the reason to close this?

Overall it doesn't seem like it provides any benefit if users are more likely to set up a traffic split where the apex service is also a backend service the way the demo describes now.


Level transition animation

Some kind of transition animation between levels as well as from the start menu to level one and level four to game complete.

Will try this in another project once i've figured at a better way to keep track of what is going on in a level (like what state it is in, etc.)


Pump treatments are 12 hours in the future

I'm new but after days of digging and asking online and reading the docs I'm stumped.
Most Open APS treatments are shown as in the future in NS admin tools. But all timestamps match across all my devices and the UI in NS is current to what is happening i.e. tmp basals are current in the UI. How is NS deciding there is an issue? https://www.facebook.com/groups/TheLoopedGroup/permalink/2272196186330368/
My local time is Pacific/Auckland and is set on NS and OAPS, and manually on my pump (though that gets synced)
Inspecting the logs I see...
Checking deliverAt: 2019-05-27T11:44:29.995Z is within 1m of current time: Mon May 27 23:44:30 NZST 2019
When you drop the first timestamp in a converter it's 12 hours out to what my local time is (11:44pm)
https://www.silisoftware.com/tools/date.php?inputdate=2019-05-27T11%3A44%3A29.995Z&inputformat=text
I have triple checked all my device clocks are correct and have rerun setup and instructions here...
https://www.silisoftware.com/tools/date.php?inputdate=2019-05-27T11%3A44%3A29.995Z&inputformat=text
To make matters worse, these future treatments appear in xDrip when 12hours rolls round.

@old-square-eyes Can you post some sample data from your Nightscout in JSON format or given me your NS URL so I can check things? By having set the time zone, you mean this is set in the Nightscout profile and for the OpenAPS rig Linux environment?

In OpenAPS -
Time zone Pacific/Auckland
~# date
Tue May 28 07:28:11 NZST 2019
Which is correct.
'Checking deliverAt: 2019-05-27T19:34:23.008Z is within 1m of current time: Tue May 28 07:34:23 NZST 2019'
From log tail

I have it set in both NS profile and OAPS Linux and NS profile is Pacific /Auckland.
Will pm NS on gitter.

https://files.gitter.im/nightscout/intend-to-bolus/bjRs/image.png
This doesn't seem right myOpenAPS/monitor/clock-zoned.json
Or is it?

Umm... is that what time it was, or not? What makes you think it’s not right?

Clutching at straws. Time zones confuse me. To verify.
At 13:41 right now in Auckland the contents of this file is "2019-05-28T13:39:30+12:00"

That means it was 13:39 in the time zone 12h ahead of UTC, which is correct.

Hey to confirm - what’s the time zone setting on the computer you’re using to access Nightscout?

As Scott says, NZST is 12 hours ahead of UTC, so the time comparison sample above is correct (even if the log message is confusing given the time logged are in different zones).

OK great. At least we're ruling things out. Much appreciated. TZ of NS computer is set to auto (Mac Mojave).
Currently...

Looks like this is a regression in the Nightscout admin tool (or, some layer below). The admin tool implementation for the feature is not very robust, but also looks like the query it's making is interpreted wrong by some layer of the database. Checking.

This should fix this issue: https://github.com/nightscout/cgm-remote-monitor/pull/4658

Thanks for looking at this. I'm only really just starting to increase my NS usage since closing the loop (previously just a convenient monitor). How long would a change like this take to make it to master branch?

This making it to master depends on how much testing help I get :D You switch branches by using the upgrade tool and pulling from dev instead of master. The current dev should be stable, so you’re getting a master early if you halo test this now.

Fair point. I'll give it a try. Will it permanently alter data going in to Nightscout? Are there any risks I should be aware of?

It’ll change how treatment timestamps are saved. Existing data is not changed. If you see any adverse effects during the first 12 hours after the update (your time difference to UTC), I’d love to hear about that.

OK will do :)

Curious. Do the OAPS devs know? Presumably it's better in the long term for the most appropriate format to be used.

It sounds like this was resolved in NS; please open a new issue if needed if this is still an issue with the latest NS release and 0.7.0-dev on the OpenAPS side.

May need to reopen this one per @sulkaharo
https://omniaps.slack.com/archives/CNW9XR741/p1571635660170700
Interestingly I got some strange TZ issues after upgrading from Dev to 0.7.0 Master. Loops are failing saying BG is old. Reported delta is ~60 minutes and we are in DST currently.


fix issue#455

When the seed node enabled, the mining relative functions should disable.

Assigning @AlexandraRoatis, as Yulong may be too busy to review


Documentation page for Kusto (and only this specific page) renders in high-contrast mode when dark mode is selected

[Enter feedback here]

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

ID: 62643597-337f-69f2-2ed9-bc9e6ddc1bf7
Version Independent ID: 3f140887-e703-105c-1b27-e29e3b001324
Content: as operator - Azure Data Explorer
Content Source: data-explorer/kusto/query/asoperator.md
Service: data-explorer
GitHub Login: @orspod
Microsoft Alias: orspodek

@elmarsto Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

Hi @AshokPeddakotla-MSFT - this looks like more of a docs ops question than a docs question. I am unable to reproduce the difference between this doc and any others in dark mode.

yeah almost immediately after I left this the problem occurred on other pages. It was uncanny.  suspect some rollout somewhere

@YaelSchuster Thanks for looking into this issue.
@elmarsto Thanks for the confirmation. We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Fix corrupted field in data file bug

Description
Fix previous bug where an invalid name in data file will cause whole app to crash. Currently any invalid field represented in the data file will result in the app launching with an empty addressbook.
Type of change

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
[ ] Code quality improvements

How to test
Change field in data file to an invalid field. App will launch with an empty addressbook.
Checklist

[X] I have tested this code
[ ] I have updated the documentation

Some invalid outputs but address-book still opens normally without any error or can't open at all.

Attribute
Invalid Value
Result

ClientId
Alex and Bernice has same clientId (i.e. "1")
Address-book parses json and both Alex and Bernice share the same id

ClientId
Alex has clientId "00"
json parses successfully and Alex's id is 00

NextMeeting
Alex has an outdated NM (i.e. 25-02-2021 (08:00~09:00), Starbucks @ UTown)
address-book fails to start (occurs for other values that causes invalid NM, refer to #198)

Might be worth noting that the user can also change the clientCounter to an invalid value (-1, 0 when there's already a client) or rename it to clientCounter**s**. This causes the clientCounter to no longer be accurate and the duplicate clientId issue to occur again

Thanks for the feedback, will take this into account and rectify

Will post another pull request with the changes


Use json.NET for all json related things instead of the DataContractJsonSerializer and/or the JsonReaderWriterFactory.

The usage of json.NET instead of the DataContractJsonSerializer/JsonReaderWriterFactory should make the life easier with wonky, incomplete data coming from the Tumblr api/SVC (Tumblr website service) as it can be setup more fault tolerant. Additionally, it's easier to use.
The main reason TumblThree currently doesn't use it, is because I wanted to learn the .NET framework and C# by developing TumblThree.
As such, I wanted to use as few third party libraries as possible with the additional goal of implement as much as possible by myself using the raw .NET framework.
In retrospect and having "finished" my initial goal, I think json.NET would be a great library to use.

Many people also seem to have incomplete data (json) serialized during their use of TumblThree.


Map-Tab

[ ] It's not possible to link a geometry (a geo object) with a context from the database. This should be possible, very much like the way photos are linked with contexts.
[x] Currently child-context are inheriting the geodata of their parent-contexts. This should not be the case. Like that only the topmost marks on the map can be selected, which are the child-elements usually.
[x] During the first loading of the map, the center point of the Popups are way off. Only after panning or zooming around, the Popups are centered on top the their respective markers. This should be fixed.
[x] When selecting a marker, the Popup is centered on the point one has clicked, not the center of the marker (which should be the case).
[x] The legend should be as wide, as the longest label. Currently 'Architektureinheiten' are cut off and placed in a new line.
[x] There should be a Layer-Selection that gives the user the ability to switch between different background maps and enable layers provided through the GeoServer (WMS)

Currently child-context are inheriting the geodata of their parent-contexts. This should not be the case. Like that only the topmost marks on the map can be selected, which are the child-elements usually.

This is not correct. This was a bug in an old version and is already fixed. It only happens for elements created before the bug was fixed.

It's not possible to link a geometry (a geo object) with a context from the database. This should be possible, very much like the way photos are linked with contexts.

Any idea how we should implement this in the UI?

I would suggest that we put this into the pop-up window, which by the way should be collapsed by default. I'll suggest a new button that 'gives the magic' :smile:


[do-not-merge] Adapt helm files for release version 1.4.0

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 64.51%. Comparing base (bb134cb) to head (dfa73ee).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #4182   +/-   ##
=======================================
  Coverage   64.51%   64.51%
=======================================
  Files         399      399
  Lines       26790    26790
=======================================
  Hits        17284    17284
  Misses       8145     8145
  Partials     1361     1361

Flag
Coverage Δ

unittests
64.51% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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


Cleanups and additional tests to improve test coverage

No actual new functionality here, this is just some code cleanup and additional testing to improve test coverage on the C++ side.
The new coverage with these changes is 94.4% on lines of code. The original coverage (after merging the fixes for #2814 and #2815, which I discovered while doing this exercise and which already helped things) was 90.4%. Not a huge improvement, but certainly a step forward.

@bp-kelley Thanks for catching those incorrectly added files. They're now gone.

@bp-kelley : ok to merge this?


[Needs Review]b07902005_劉冠鴻

A Pull Request (PR) should explain why we need this PR and how you implement it, so others could understand it quickly.
Since this is just a homework, you do not need to explain why and how.
Do NOT change the status to NeedsReview if you do not want to be graded yet.
Please fill in the following questions as feedback. It is totally optional and would not be graded.
What is the most challenging part in this homework?
N
Is this homework helpful for you?
Y
Is there any suggestion?
N

PR readiness check list

[v] Did it pass the Flake8 check?
[v] Did you test this PR?
[v] Did you change the title of this PR to the correct format ([status] student_ID)?

Sorry, I don't understand what means my commit message is not linked to my profile.
My last commit is at Files changed.

If they are linked, the two would have the same picture:

Ok, I see.
So whether can my commit successfully be merged or not?
And how can I solve the problem after I changed my author name email?
Thanks.

@sam2068 I don't quite understand your questions. See https://help.github.com/articles/why-are-my-commits-linked-to-the-wrong-user/ and https://www.git-tower.com/learn/git/faq/change-author-name-email . What have you done to fix it?

Done.
Please merge, thanks.


generate requirements lockfile in ci upon requirements change in

Describe your changes
Related Issues / Projects
Checklist

[x] I've manually tested my code
[x] The changes pass pre-commit checks (make lint)
[x] The changes follow coding style

@NiceAesth nvm ez

wouldn't it make a bit more sense to do it the other way around? (e.g. have dependabot solely update pipfile and generate reqs from that)
changes look fine other than that

wouldn't it make a bit more sense to do it the other way around? (e.g. have dependabot solely update pipfile and generate reqs from that)

Yeah either way seems fine - didn't know what was possible/just went for a quick solution


[Question] Any way to send a HTTP 1.0 request?

As the subject says, is there any way I can instruct Insomnia to issue a HTTP 1.0 request (e.g. the equivalent of a curl --http1.0 https://site.tld command)?

Hi @pgp,
Are you still experiencing this issue, even with our newest release: https://github.com/Kong/insomnia/releases/tag/core%402022.5.0

Hi @pgp
You can change the HTTP request Preferences as seen below.
https://kongstrong.slack.com/files/U02JCR2AB1V/F03T8NS15FY/image.png?origin_team=T0DS5NB27&origin_channel=D02KGD00TNY
Thank you and have a nice day.


Fix stub for Builder::havingBetween()

havingBetween accepts iterable, not arrays only:
https://github.com/laravel/framework/blob/136ed754f2d80e5c7cd407a4262eb2eac478e7e8/src/Illuminate/Database/Query/Builder.php#L2221-L2243

Hi,
This looks like it got changed in 10.x. In 9.x it is still array Since we still support both we'd need to add different stub files for different Laravel versions.
But in this specific case the docblocks looks exactly the same between our stubs and Laravel. I honestly forgot why I added all those method to stubs 😅  I'll try to refactor the stubs instead.


keymanager-api-common: Implement Zeroize for secret types

See #1683
TODO

[x] Check if there are other places which need this.

lg, if #[zeroize(drop)] does what it sounds like

It implements Drop that simply calls self.zeroize() :-)


D<IP_ADDRESS> Profile overrides in the target instance summary

The target instance summary page contains the option to override the profile at the instance level. This functionality does not yet support imported profiles. We will add an XML editor as in the case of D<IP_ADDRESS>.

Working on this issue in branch: [feature/D<IP_ADDRESS>_profile_overrides_in_target_instance_summary] (https://github.com/DIA-NZ/webcurator/tree/feature/D<IP_ADDRESS>_profile_overrides_in_target_instance_summary).

@hannakoppelaar I've made the first set of changes -- that the editor appears when the Base Profile is imported. Can you verify that it looks and works the way you want?

I'm a little unclear about the behaviour. If the profile is imported then we show the XML editor (whether or not there's an override on it already). And if we apply the profile, does that changed profile then replace the target's profile string? Or does it only replace the profile override rawProfile?

There's a few problems:
We create a copy of the Target and in the 'Profile' tab we Override Imported Profile: [x].

If the HarvesterH3 tries to validate the override XML by creating an 'validateJob' job and for some reason that job isn't cleaned up (the directory 'validateJob' in the H3 harvester live instance), then that directory will persist, and from then on the state will be 'Ready' and not 'Unbuilt', and no validations will succeed. See HarvesterH3.java method build where if (jobStatus.statusDescription.equals("Unbuilt")).
 So is there an override limit? Or do we just use a CLOB, the same as P_PROFILE_STRING in the PROFILE table?

You're right, 1) could actually occur, although I haven't seen it happen yet. Do you think it's worth the trouble to automatically detect and fix that problem when it occurs? (Not a rhetorical question ;-))

In wct-core/db/latest/upgrade/wct-upgrade-1_6_3-1_7-oracle.sql on the D<IP_ADDRESS> branch the PO_H3_OR_RAW_PROFILE field is of type clob. (And text in mysql and postgresql.) Perhaps you've added that field manually, using a different dataype?

I'm a little unclear about the behaviour. If the profile is imported then we show the XML editor (whether or not there's an override on it already). And if we apply the profile, does that changed profile then replace the target's profile string? Or does it only replace the profile override rawProfile?

I feel the ti summary page is a bit ambiguous anyway, but I guess updating just the PO_H3_RAW_PROFILE makes more sense to me, since it's supposed to be an override of the base profile (the title of that section of the screen is "Profile Overrides" after all).

For 1) I'll have it automatically detect and fix the problem.
For 2) that's the field type in the H2 database, so I'll look into why it didn't use the CLOB type, since that's the type that shows up for P_Profile_String.
I'll also have the 'apply profile' button make the change the PO_H3_Raw_Profile.
I see that you've pushed another commit to D<IP_ADDRESS>, so I'll pull that into my branch as well.

Pull request pull/55. Note that D<IP_ADDRESS> should be merged into master first, as those features are included. See the pull request itself for identified issues.

Closed by #56


Level bounds fix

Fix an issue where the level bounds can become an invalid value, Causing Mega Man Maker to crash upon loading a level with screwed up bounds. Now the level bounds are rounded to 16, and each one is either minned or maxxed to their max value. This should solve all issues, and Definitely solved the issue I was having, which had to do with not having any objects in a level messing it up.


Datepicker visual bug when using ng-repeat

Bug description:
Having multiple datepicker-components in one page causes the active value to jump between the different instances. This does not happen immediately on click but with a delay of one choice in another datepicker when using more than two datepicker.
Only one element in one datepicker stays active although the correct values of all datepickers are saved within the scope.
Plunker:
http://plnkr.co/edit/wbqsauMzYBwcVMQP5LNQ?p=preview
Version of Angular, UIBS, and Bootstrap
Angular: 1.5.8
UIBS: 2.4.0
Bootstrap: 3.3.6

no response at all?
i think this is a big issue :(

Anything on this?


Reconnect on quit 431

This should help in situations where an IRC proxy (e.g. znc) may send a QUIT event, it will ensure matterbridge reconnects, even if the disconnect was not in error.
Also ran the code through gofmt/goreturns to fix some of the inconsistencies.

Had to force push to fix https://github.com/42wim/matterbridge/pull/445/commits/3b72594a9dafa85a6292403813ee557fccf8fb67#diff-228410cdab7b67ae6ed9a2aa28c657d0 not using proper import names.
Just FYI, when importing, if the package name you use doesn't match the folder name used in the import (e.g. https://github.com/42wim/matterbridge/pull/445/commits/3b72594a9dafa85a6292403813ee557fccf8fb67#diff-8cf1b62ff356263dd5a7c773e800dcc7, actual package being samechannelgateway but directory name being samechannel), this causes most editors to not properly provide intellisense/code-completion, and isn't standard. It should be concise and clear to the person reading the code, where the package came from.

Also, I would use something like https://github.com/sqs/goreturns (or at least goimports) in your editor. VS Code has amazing support for these types of things, and it's my favorite editor for Go (even if it is a Microsoft product..).
FWIW, vim-go should also support goreturns/goimports/etc, among other editors, if that's what you use.

Thanks for your PR!
I actually use goimports and it doesn't do the stuff you add.
Also it seems like vim-go can do the code-completion without the package matching the foldername.

Don't use goreturns though


[BACKEND] prefer less ptr arithmetic

Not sure if this makes a difference. But this uses fewer optimized TTIR instructions.

Unfortunately, I think this would overflow since AddIOp may be done in int32?

Hmmm, I think it is safe since the AddIOp will happen in INT64, but I am not sure how this would help? I can try to run some workloads with this PR and see the empirical effect

The layout is not handled properly for the nvidiagpu case. So I restrict to triton case for now (sm < 90).

I can try to run some workloads with this PR and see the empirical effect

Yes. While I would not believe that the pointer arithmetic itself would have an effect, I'm curious if the way that the passes interact with the specific choice of IR operations (splat v.s. broadcast etc) would have an effect.

I am not seeing any benefit to this change after running some benchmarks, and even https://github.com/openai/triton/issues/2301#issuecomment-1722342744 seems to come from int64 arithmetic? I understand the idea of specializing kernel based on grid size, but I'd rather avoid -- it could make recompilation very unpredictable.

I understand the idea of specializing kernel based on grid size, but I'd rather avoid -- it could make recompilation very unpredictable.

Yes, it is more of a corner case when there is severe register pressure. I'd keep the issue open in case it becomes useful e.g. if there is a measured difference in concrete use case (e.g. in matmul/flash attention). But it's definitely low priority + solution is high complexity.
We can close this PR if you think it's not helpful.


Encoding should be enforced on key while using a get

Change to angular-cookie.js
ipCookie will not be able to fetch values stored in cookie if the key is not encoded while fetching. Correct me if am wrong

Reason for my pull request is if my key has a value like this<EMAIL_ADDRESS>ipCookie will encode the key and while getting it. It will not fetch the value because encoding of key is not enforced while fetching

Encoding should be enforced on key while fetching or removing a cookie

:+1:


Can't get tape dependency through the following xml section.

<dependency>
  <groupId>com.squareup.tape2</groupId>
  <artifactId>tape</artifactId>
  <version>2.0.0-SNAPSHOT</version>
</dependency>

The above one described in README.md can't be resolved through Maven.
However the following works.
<dependency>
    <groupId>com.squareup</groupId>
    <artifactId>tape</artifactId>
    <version>1.2.3</version>
</dependency>

You also need the snapshot repo, something like this https://github.com/segmentio/analytics-java/blob/master/README.md#snapshots

@f2prateek yeah, it works. thank you very much.


Fix grammatical mistake in linear-classify.md

note that to classifying => note that classifying

Thanks :)


Deleted created_time in "status"

This is for master, but is also included in "Data loader fixes #7672" (for 0.15.0).
Issue:

Druid console shows blank screen when clicking on Tasks > group by Datasource (> or group by Type)
When group by status, only 1 status shows up:

Expected:

Current UI after this fix:
it can be grouped by type:

Or grouped by status:

This is not the correct approach to this issue it effectively undoes the tie-breaker reverting back to the state where the tasks are ordered incorrectly:

(see how the time within successful tasks is ascending, it should be descending)
The correct approach is to define a toString method on the object returned by the accessor.
Have a look at what I did in this backport PR: https://github.com/apache/incubator-druid/pull/7672/files#diff-4aaac42e3d5f769f344608b14e686f88R604
This gives that object a canonical string value and tricks ReactTable to work as intended.

Looks good to me now. Could we change the PR title to something like Web console: fixed issue when grouping tasks by different attributes

Awesome thank you! 👍


WIP [DUOS-491][risk=no] Transition Dac Member Votes

WIP
Needs some more tests.
Addresses
https://broadinstitute.atlassian.net/browse/DUOS-491
Changes

When adding a Dac member, create new votes for that member in the current open elections for the Dac.
When removing a Dac member, delete votes for that member in the current open elections for the Dac.
Prevent the removal of all Dac chairs which would leave the Dac in an invalid state.

Coverage decreased (-0.2%) to 60.053% when pulling bade689b4ac366457694f99710503ad13784e6f9 on DUOS-491 into ab318a523e60ae61c41740fe778ac3cd894604e6 on develop.


filecache not ram cache (or optional cache)

when you get a few million variations on your megagrid, let it run through the night and the next day, because this script caches the images to allow the webpage to load faster, it gets to be massive. just checked my ram this evening after work, of my 96 gb of ram, I was using 350 gb (pagefile of course), this meant I couldnt do anything.
also, at that amount of variations, it takes nearly 2 hours to start the script. make the webpage caching optional, as I just access from the file directly, and have the cache builder write a json array of all the prompts to a file so that as long as the yaml isnt modified, the json can be fed directly into the api after it is made and no extra startup time is needed (hash the yaml)

No part of this script does any form of image caching.
If it did, even The MegaGrid, which has 445k images in it and took around a full week together to render, only takes 5.2 GiB on disk, and so if it were cached would only occupy 5.2 GiB of RAM, not 350 GiB.
The situation you described is not only not accurate to how anything works (there is no cache in the script), the numbers extra so make no sense (you cannot possibly generate 350 gig of memory from SD in a human duration of time, it would take months if not years on high end hardware).

@mcmonkey4eva

this is roughly 2 minutes after hitting the start button. do you know if stable diffusion reserves memory for each image it will generate? because idle stable diffusion uses around 50 mb, and it is only after starting the generate that this happens. it continues to climb as it goes, but I had to restart it due to the aforementioned massive memory leak. since I took that screenshot, it has climbed another 10 gb as I sit here being a slow typer.

another few minutes later. still climbing. I will note that my yaml does have a few too many variations

but something is being done incorrectly. either here with generation of the queue or with a1111 with inserting the new image to the queue.

actual number being generated is around 4 million. I just have those prompts there because they are ideas I want to test but doubt I will ever get to unless I rent a server farm. I know that even 4 million will not be done any time soon, I just generate every once in a while. it still takes upwards of an hour to get to that point though where I can even see the total number to be generated.
overall: the generation params should be cached in pycache as a json or something so that startup time and background ram usage can be reduced.

There's a button in the UI for if you care about load time for large runs

Note that 4 million is still a huge amount. At one image per second that will take over a month to complete.
You can expect up to maybe around a kilobyte or two per target generation of RAM usage, so for 4 million images queued up expect around 4-8 GiB of RAM to be taken. This of course can be reduced by not running a month worth's of image generations all in one go.


Double check ArgumentExceptions across the Form Recognizer SDK

We were incorrectly passing the wrong argument to an ArgumentException:
https://github.com/Azure/azure-sdk-for-net/blob/5cb4b83b190cc6295d01ecc8796620fa07d73e80/sdk/formrecognizer/Azure.AI.FormRecognizer/src/RecognizeCustomFormsOperation.cs#L159
We should have wrapped operationId in nameof() so the error message contains the actual parameter name, and not its value:
throw new ArgumentException($"Invalid {nameof(operationId)}. It should be formatted as: '{{modelId}}/analyzeresults/{{resultId}}'.", nameof(operationId));

We must double check occurrences of ArgumentException across the Form Recognizer SDK and make sure we are passing the correct arguments.

Double-checked. No issues found.