Added get USD value snapshots endpoint

Summary
Added get USD value snapshots endpoint
Additional Information
N/A
Checks

[ ] Changes tested locally (if applicable)
[ ] Updated related documentation (if applicable)
[ ] Updated/added related tests (if applicable)

Happy to contribute 🫂


chore: adaptations for nightly-2024-03-24

There are some regressions in rw? and rw_search, so some test cases are commented out. Joe is looking at these today, and they are not used in committed code in Mathlib, so I think it is okay to proceed as is.

bors r+


v2.3.0 should be deployed to official S3 repository

Issue:
Currently the v2.3.0 states on its release notes, that
Customers should now download the Customizations for AWS Control Tower CloudFormation Template from GitHub instead of S3
As CfCT lab leverages the official S3 template, others will fail to use the right one.
Request:
Push v2.3.0 template file to https://s3.amazonaws.com/solutions-reference/customizations-for-aws-control-tower/latest/custom-control-tower-initiation.template

Hey @mdaehnert, per the release notes, GitHub is now the official source of distribution for CfCT as of v2.3.0.
We will work with the owners of CfCT lab to update documentation to reflect this.

Sorry to highjack this issue, but what about the deploy zip files in the s3 solutions bucket?
For example the following S3 file does not exist for v2.3.0
s3://solutions-reference/customizations-for-aws-control-tower/v2.3.0/custom-control-tower-state-machine.zip

but v2.2.0 does exist
s3://solutions-reference/customizations-for-aws-control-tower/v2.2.0/custom-control-tower-state-machine.zip

Will v2.3.0 be added to the solutions-reference bucket?

@jrosco the bucket location was updated as part of the v2.3.0 release, which was automatically included in the new source for the CfCT template
For non-self deployed/hosted CfCT deployments, the files are all sourced from this new bucket, as an example for the state machine zip:
s3://control-tower-cfct-assets-prod/customizations-for-aws-control-tower/v2.3.0/custom-control-tower-state-machine.zip


refactor(wrong name): Wrong name for locale

Hmmm no? si stands for Sinhala.
Please check https://en.wikipedia.org/wiki/List_of_ISO_639-1_codes

sl stands for Sri Lanka
si stands for Slovenia (slovenian domains are .si)

What you are talking about is domain name, which is not the same thing as language code.
https://en.wikipedia.org/wiki/Country_code_top-level_domain


Modified grammar.

I'd also like to propose that we add running the models from the command line, rather than via Visual Studio - most engineers / data scientists won't be using VS as their preferred development environment, especially if they're leveraging a Linux DLVM.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/ai/tensorflow-vm.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Serializing methods that reference "Smalltalk"

If we have store a method that references "Smalltalk", we try to store all the binding in "Smalltalk globals", which fails due to Process being a global var
--> for now, store "Smalltalk" similar to "Smalltalk globals"

The reason is not the globals, but the Class Variable LowSpaceProcess, which references a process.


Migration nuxt2 to nuxt3

Once we will finish #1060 we can look at how we can ape into Nuxt3
Or maybe just install the Nuxt bridge 👀
Chart from https://v3.nuxtjs.org/getting-started/introduction

Continue to

#2692
#2081
#2690


Update $timeout JsDoc.

Angular v1.4 allows additional callback arguments to be passed to $timeout.
https://github.com/angular/angular.js/commit/3a4b6b83efdb8051e5c4803c0892c19ceb2cba50

LGTM.


Some Questions About Autopilot

Hey!
Recently I'm trying a pipeline-way to control a car in the simulator. So I have read part of the source codes of autopilot. But I still have some questions.

Which method is used to control a car to turn at an intersection? Does "CalcStreeringValue(Direction)" solve this problem?
What does TargetLocation mean? I understand it as a planned route consist of a series points vector. But why it executes "CalcStreeringValue(Direction)" when "TargetLocation is empty". if I can understand the TargetLocation as a straight-line route without any turning?
How can I make sure that the car has reached a point on the planned route and to compute the "steering" with the next point? It should have a method to judge whether the car has reached a point, but I haven't found it.
How does autopilot create a planned route automatically and update the variable “Locations”?(And I also think "Locations" is the same as "TargetLocations", because TargetLocations.emplace(Location), am I right?)

Thank you !!

Also, a question does not belong to autopilot, where can I find the relationship between start point index and its coordinate in the map? Should it be in a file?

Hey @yuanzhiyu
From my understanding, when Autopilot Mode is switched on,

The car proceeds to move in a forward direction at the speed which is set by the speed limit for the given region (default - 30 km/hr)
The car stops when it detects the presence of other cars or pedestrians in its surroundings and if it encounters a red traffic light achieved using ray tracing and trigger overlap information (see WheeledVehicleAIController)
The car uses the roadmap data to steer itself to ensure it stays on the right side of the road and also ensures that car doesn’t enter the sidewalk/footpath. (see CalcStreeringValue(Direction) in WheeledVehicleAIController)
When the car enters an intersection, an overlap trigger event calls the corresponding Route Planner (visible in Town maps) which provides the car a route as a set of spline points that determine the direction (left/right/straight) the car will head towards. These spline points are sent to the VehicleController through the SetFixedRoute() as the 'TargetLocations'. (See RoutePlanner)
The routes are picked randomly using the RandomEngine based on the probabilities that have been set for the routes. (These probabilities can be changed in the editor or in-game.)

I hope this answers some or most of your questions.

@shriram17
Thanks a lot for answering my question! But still, the algorithm on autopilot is not easy to understand, I choose pure-pursuit algorithm to solve my problem finally.


There is no Metrics under Features in cluster settings page

Describe the bug
There is no Metrics under Features in cluster settings page
To Reproduce
Steps to reproduce the behavior:

Fresh install lens 4.1.4
Add a cluster just created by k3sup

Expected behavior
There should be metrics button in Features setting block.
Screenshots

Environment (please complete the following information):

Lens Version: 4.1.4
OS: OSX
brew install lens

Lens on Windows connected to the same cluster works fine

Can you provide logs?

I did reproduce this, and after reloading (Cmd-R) the Metrics Stack appeared. But can't seem to reproduce it again...


Building a Password Strength Classifier Model Using Machine Learning

Article checklist
Attention
Our Peer Reviewer and Content Moderator teams do NOT provide any revisions services. All revisions and edits should be completed on your own forked repo (as to not take up room in the queue) in order for our team to review them all in a timely manner.
ANY ARTICLE SUBMITTED WITH GLARING ERRORS WILL BE IMMEDIATELY CLOSED.

As a rule of thumb - please be sure to only submit articles (pull requests) that are fully polished and ready to be published. Be sure to go through our resources documents for extra and 3rd party (vetted) resources to help improve overall technical writing.

Pre-submission checks

Please ensure that you have only one open topic suggestion (issue) + in-review article (linked pull request) at a time. This will ensure that we complete the review process in a timely manner from inception to publishing

If you have not already, please go over our Resources Page for more tips and tricks on how to improve your overall technical writing so reviews are swifter, increase the chance of a payout and provide more value to our readers.

For first-time contributors and for more details on our submission guidelines, see our Contributing Guidelines.

Formatting and structure checks

[x] Have you used the correct folder and file structure? - see Contribution Guidelines

[x] Is your article properly formatted in Markdown? - see Github Markdown guide

[x] Have you used our preferred heading structure? - H3 (###) and up

[x] Does your article follow any of the suggested structure formats? - see example formats

[x] Is your article over 750 words? - Tool: Wordcounter.net

[x] Is the article you are submitting an in-depth and unique article? Does it go beyond what is in the official docs and what is covered in other blog sites. See these articles as examples.

[x] Does your article provide enough value and detail about your topic? - Articles should be clear, accurate, and fully explained.

[x] Can your article be understood by beginners? - Assume the audience is smart but has no prior exposure to the common terminology in your article.

[x] Have you included a hero image that is 600x400 pixels, under 300KB in size, and have the copyright to use it? - Tools: Pixlr Image Editor for resizing and Unsplash for Creative Commons images

Grammar & plagiarism checks

[x] Have you spellchecked and grammar checked your article? - Tools: Write&Improve - ProWritingAid - Grammarly
[x] Please place your article through a 3rd party plagiarism checker. We suggest using Quetext, this tool is free and has a daily limit. We typically accept articles with 10% or less.
[x] Have you checked your article for readability? - Tool: Hemmingway
[x] Have you added sources for quotes and images that aren't yours?

Technical checks

[x] Are your code snippets properly formatted for syntax highlighting - see Syntax guide
[x] Have you checked your code runs correctly and you've highlighted all necessary dependencies for installation?
[x] Are the software programs and packages you're highlighting in your article up to date, using current versions and not deprecated?

Topic suggestion this closes

Remove the backticks and add the issue number below to link and close your Topic Suggestion (issue) when your article has been published (PR has been merged). See this video for more details.

This closes #5722

Finally, delete the article checklist notes in blockquotes and submit your PR. We look forward to reviewing your article.

@hectorkambow, I'd like to review this article.

Plagiarism check done ✅

🚀 awesome article @kelvinkimani501, please ensure you go through the changes I made in the commits to help improve flow and format for your future articles. Thanks.

Awesome article @kelvinkimani501.
Thank you for the review @M0nicah.


Update Spotless + ktlint to latest versions

IMPORTANT: All PRs must be linked to an issue (except for extremely trivial and straightforward changes).
Description
Updates Spotless and ktlint, and re-runs ktlint.
ktlint had a bug that results in a cryptic error when attempting to use trailing commas (a valid Kotlin 1.4 feature). Updated ktlint to the latest version. Updated Spotless as well, and ran spotlessApply to apply the updated ktlint rules.
Type
Choose one: Code health
Checklist

[x] I have read and acknowledged the Code of conduct
[x] I have read How to Contribute
[x] I have read the Developer's guide
[x] I have signed the Google Individual CLA, or I am covered by my company's Corporate CLA
[ ] I have discussed my proposed solution with code owners in the linked issue(s) and we have agreed upon the general approach
[x] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the style guide of this project
[x] I have run ./gradlew check and ./gradlew connectedCheck to test my changes locally
[x] I have built and run the reference app(s) to verify my change fixes the issue and/or does not break the reference app(s)

thanks so much for this!


String which contains the substring "EOF"?

Some poorly-written applications may treat the string "EOF" as an end of file without regard for context (as opposed to bash)

Can you please explain more on this issue? Maybe something like https://bugs.java.com/bugdatabase/view_bug.do?bug_id=4209933

Hahah!! Yeah sounds cool! Something I'd miss! 🤣


Merging thesis_gemein_2022 into TrixiAtmo.jl

This is a draft PR to better understand https://github.com/trixi-framework/TrixiAtmo.jl/issues/1.
TODO

[ ] Add tests for elixirs

[ ] Resolve repetition from Trixi in the process of adding tests. That is, this file https://github.com/trixi-framework/TrixiAtmo.jl/blob/ab_lucas/test/test_trixiatmo.jl is copied from Trixi.jl.
[ ] Are the 1920 allocations expected?
[ ] Add tests for the remaining 5/7 elixirs

This is a good start for TrixiAtmo.jl, and should be quite useful. I will have some time to look into the specifics more in the next week, but my main comment is that we should perhaps set up testing/CI (similarly to TrixiShallowWater.jl) before merging this to main.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 567 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-99.3%) to 0.701%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/equations/compressible_moist_euler_2d.jl
0
567
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-99.3%

Covered Lines:
4

Relevant Lines:
571

💛  - Coveralls

Resolve repetition from Trixi in the process of adding tests. That is, this file https://github.com/trixi-framework/TrixiAtmo.jl/blob/ab_lucas/test/test_trixiatmo.jl is copied from Trixi.jl.

At first I though that we could maybe move some parts to TrixiBase.jl, but then I realized that these macro e.g. explicitly include the Trixi. Maybe we should there add our own macro as need be.

Are the 1920 allocations expected?

Which test case was it?

It looks good to me, although I am wondering if either of you (@Arpit-Babbar or @benegee) know the reason for the warnings copied below? My tests have always been very simple so I've never run into this.
WARNING: replacing module TrixiAtmoTestModule. WARNING: could not import TestExamples2DMoistEuler.TRIXI_EXAMPLES_DIR into TrixiAtmoTestModule

You define a module with the same name every time in
https://github.com/trixi-framework/TrixiAtmo.jl/blob/1c7efd9be9a80019df8d74bb765ffdccd6a90ba1/test/test_trixiatmo.jl#L28
If you want to get rid of that warning, you could gensym an appropriate name - but I don't think that's necessary right now.


[no-use-before-declare] False positive for keyof/typeof parameter types

Bug Report

TSLint version: 5.9.1
TypeScript version: 2.7.0-dev*
Running TSLint via: (pick one) CLI / Node.js API / VSCode / grunt-tslint / Atom / Visual Studio / etc

TypeScript code being linted
interface IGroupTypes {
    Scenery: { type: "scenery "},
    Solid: { type: "solid" },
}

function addToGroup(thing: IGroupTypes[typeof groupName], groupName: keyof IGroupTypes): void {
    // ...
}

with tslint.json configuration:
{
    "rules": {
        "no-use-before-declare": true
    }
}

Actual behavior
ERROR: 6:47   no-use-before-declare  variable 'groupName' used before declaration

Expected behavior
There shouldn't be complaints about type parameters, as their types aren't runtime constructs.

Marking this as very low priority because no-use-before-declare is no longer a recommended rule. Sorry, @JoshuaKGoldberg from January!

This might be re-opened per discussion in #4789.


Flutter Web: Preflight check doesn't send cookie, creates new session (NodeJS backend)

I am using Dio. I'll be posting the same issue in both places, because I'm not sure where the problem lies.
I detail below how I can use an alternative method in the server, so preflight isn't triggering the full request, but then the main-request gives me a different session ID each time. The method I'm demonstrating here shows that the sessions are persisting, and that that Flutter-Web is getting, and sometimes sending the cookie.
Steps to Reproduce

I've stripped the demo server (Node JS) down to what I think is the bare minimum

import express from "express";

const app: express.Application = module.exports = express(),
  http = require("http"),
  session = require('express-session');

app.use(session(
  {
    secret: 'demo',
    resave: true,
    saveUninitialized: true,
    cookie: {
      path: '/',
      crossDomain: true,
      httpOnly: true,
      secure: false,
      maxAge:<PHONE_NUMBER>,
    },
    rolling: true,
  }));

app.use('/', (req, res, next) => {
  res.header({
    'Access-Control-Allow-Methods': 'GET, OPTIONS',
    'Access-Control-Allow-Origin': req.headers.origin,
    'Access-Control-Allow-Headers': 'Content-Type, Authorization',
    'Access-Control-Allow-Credentials': 'true'
  });
  console.log(req.method, req.session.id);
  res.end();
});

const server = http.createServer({
  rejectUnauthorized: false,
  host: '<IP_ADDRESS>'
}, app).listen(3000, function () {
  console.log(`Demo Server started.`);
});

I make a request in Dio like this.

Dio dio = Dio();
dio.options.baseUrl = baseURL;
dio.options.connectTimeout = 5000;
dio.options.extra['withCredentials'] = true;
Map<String, String> sendHeaders = {};
sendHeaders['Content-Type'] = 'application/json';
    await dio.post(url, options: Options(headers: sendHeaders));

Expected results:
Both the preflight and the request will trigger the app.use('/'), so I expect to see...
 I'm deliberately taking advantage of it here.
If I do that, and the browser only sends one request, then that main request does not send the cookie, and each session is different, making maintaining login state impossible.
This is frustrating, because this is easy in Android/IOS/etc. You catch the cookie on the first request and send it with subsequent requests.

Logs

The logs aren't outputting anything with the request, so that might be irrelevant here. I'll post the logs in a comment if you like but they're very long and probably not helpful here.

   info - The import of 'package:flutter/foundation.dart' is unnecessary because all of the used elements are also provided by the import of 'package:flutter/material.dart' - lib\UI\card_admin_script_new.dart:3:8 -
          unnecessary_import
   info - The value of the field 'indexedDB' isn't used - lib\fab\core\platform.dart:3:40 - unused_field
   info - The value of the field 'url' isn't used - lib\fab\core\platform.dart:3:51 - unused_field

[√] Flutter (Channel stable, 2.10.4, on Microsoft Windows [Version 10.0.22000.613], locale en-US)
    • Flutter version 2.10.4 at D:\src\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision c860cba910 (10 weeks ago), 2022-03-25 00:23:12 -0500
    • Engine revision 57d3bac3dd
    • Dart version 2.16.2
    • DevTools version 2.9.2

[!] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
    • Android SDK at D:\Users\Abby\AppData\Local\Android\Sdk
    X cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    X Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/windows#android-setup for more details.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop for Windows (Visual Studio Community 2019 16.10.4)
    • Visual Studio at D:\Program Files (x86)\Microsoft Visual Studio\2019
    • Visual Studio Community 2019 version 16.10.31515.178
    • Windows 10 SDK version 10.0.19041.0

[√] Android Studio (version 2020.3)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)

[√] Android Studio (version 2022.1)
    • Android Studio at D:\Program Files\Android\Android Studio1
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)

[√] VS Code (version 1.67.0)
    • VS Code at C:\Users\Abby\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.40.0

[√] Connected device (4 available)
    • Android SDK built for x86 (mobile) • emulator-5554 • android-x86    • Android 10 (API 29) (emulator)
    • Windows (desktop)                  • windows       • windows-x64    • Microsoft Windows [Version 10.0.22000.613]
    • Chrome (web)                       • chrome        • web-javascript • Google Chrome 102.0.5005.63
    • Edge (web)                         • edge          • web-javascript • Microsoft Edge 101.0.1210.53

[√] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

@cfuj Thanks for the report. Since you are using dio, I suggest you to follow-up in the issue you reported in that repo since it is third party for Flutter.
You may also try to use http package and see if it gives same results or not. If not, then, please do the same by reporting in dedicated repo.
Closing out from here as this doesn't seem to be specific to flutter web. If you disagree, write in comments and I'll reopen it.


Reconstruct the fully qualified pathname

Without this path reconstruction, files will only match if the working directory matches the basepath variable derived from doublestar.SplitPattern().

LGTM.
I'll just fix the issue of directories


sui_getEventsByModule returns no events for my module

Steps to Reproduce Issue
curl --location --request POST https://fullnode.devnet.sui.io \
--header 'Content-Type: application/json' \
--data-raw '{  "jsonrpc": "2.0",  "id": 1,  "method": "sui_getEventsByModule",  "params": [    "0xa28c569cb1e272ad0d0c18b5ad2f48fa0c4e425e",    "playerstate_module",    5,    0,<PHONE_NUMBER>00000  ]}' | json_pp

Expected Result
return the events, e.g. this transaction has the move events I want to get back: tx exlorer
Actual Result
{
"id" : 1,
"jsonrpc" : "2.0",
"result" : []
}
Having this issue since this morning. I also republished my module multiple times. The strange thing is that the devnet nft module returns some events:
curl --location --request POST https://fullnode.devnet.sui.io \
--header 'Content-Type: application/json' \
--data-raw '{  "jsonrpc": "2.0",  "id": 1,  "method": "sui_getEventsByModule",  "params": [    "0x0000000000000000000000000000000000000002",    "devnet_nft",    5,    0,<PHONE_NUMBER>00000  ]}' | json_pp

{
   "id" : 1,
   "jsonrpc" : "2.0",
   "result" : [
      {
         "event" : {
            "moveEvent" : {
               "bcs" : "FE7RsUK52ldXLFJ6qyqGo0wNR+CE95YqNrDaVKZvKKCEBsNjE+ygqQtFeGFtcGxlIE5GVA==",
               "packageId" : "0x0000000000000000000000000000000000000002",
               "sender" : "0x84f7962a36b0da54a66f28a08406c36313eca0a9",
               "transactionModule" : "devnet_nft",
               "type" : "0x2::devnet_nft::MintNFTEvent"
            }
         },
         "timestamp" :<PHONE_NUMBER>086,
         "txDigest" : "IlTDmgsiXDc+VaDiHJ3nGyHgVvsTP9wY65JDq228t+c="
      },
      {
....

System Information

OS: Windows
Compiler: sui.exe-move 0.1.0

Also tried different languages / platforms, not just curl.

There is an ongoing problem with the fullnode indexer on DevNet, which might cause transactions/events to be missing from the fullnode

Hi there, any update on this? From my tests it seems it is still an ongoing issue, not getting back my events. Devnet explorer seem to have some issues

Based on my tests, it seems to be still an ongoing issue, can you confirm please that there are problems on your side?

APIs changed, closing this


python311Packages.bimmer-connected: 0.14.2 -> 0.14.3

Diff: https://github.com/bimmerconnected/bimmer_connected/compare/refs/tags/0.14.2...0.14.3
Changelog: https://github.com/bimmerconnected/bimmer_connected/releases/tag/0.14.3
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

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

[ ] Fits CONTRIBUTING.md.

Priorities
Add a :+1: reaction to pull requests you find important.

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

5 packages built:

home-assistant-component-tests.bmw_connected_drive
python310Packages.bimmer-connected
python310Packages.bimmer-connected.dist
python311Packages.bimmer-connected
python311Packages.bimmer-connected.dist


update readme

related to #423
addAdditionalSuccessPurchaseListenerIOS needs to be called prior to buyProduct if one expects it to catch successful purchases.
This PR reflects this on the example usage of buyProduct

I haven't tested this but it looks like for a normal flow (not the StokeKit flow), this.handlePurchaseSuccess(purchase); will be called twice

@Edgpaez I partially agree with @vafada. You should only subscribe to the listener when there was a failure in the purchase unless you should put any other condition inside the listener that would avoid recording double purchases.
In addition, the react-native-iap won't affect on duplicate purchase just by adding listener. The side effect on doing this is receiving the same result twice. This may cause duplication when recording successful result in your database (if you are doing so).

@vafada, yes. This is not related to storekit flow. TBH, I don't understand how to address your comment.
@hyochan I don't think you can subscribe AFTER you get the error. Not if you want to be sure you'll catch the error.
Can we be sure the second purchaseProcess is called after the user caught the error and called addAdditionalSuccessPurchaseListenerIOS ?
I think we need to be clear to users that this.handlePurchaseSuccess(purchase); can be called more than once.
What do you think?

What I'm trying to do is fix the documentation since I believe the example code has a bug.
The bug (I believe) is that addAdditionalSuccessPurchaseListenerIOS won't catch all success purchases in the storekit flow.
Current Implementation
Internally addAdditionalSuccessPurchaseListenerIOS relies on NativeEventEmitter to listen for the iap-purchase-event event.
Inside RNIapIos (the class handling the purchases natively) when the iap-purchase-event event is emitted there is no guarantee that addAdditionalSuccessPurchaseListenerIOS has already registered to listen that event.
So, what's the problem?
I think that means that there is no way to be sure that when the iap-purchase-event is emitted natively, there is a listener registered.
That means there is a possibility that users successfully pay but JS doesn't acknowledge  the payment.
The fix?
Registering the listener for the iap-purchase-event event before making a purchase.
I hope that's clearer @hyochan

@Edgpaez Thanks for your through explanation and I could understand you much better! I've not yet reproduced any missing event in my other app. Also, the listener is registered when there was failure in the purchase as in line. I am not quite sure tough whether there will be any hole doing the implementation this way since this was just the work around described in #348. If there is any clear reproducible senario, I would like to try to consider this again.

@hyochan
@Edgpaez has a point. There is NO GUARANTEE that the listener would be added before the second success transaction comes to the queue.
So this flow is possible:

Failed transaction enters the Payment Queue
Success transaction enters the Payment Queue
code goes to catch block
listener gets added

At this point, the success transaction is now lost.

@vafada The code goes to catch block right after failed transaction and starts the listener. Then the success transaction enters the payment queue and listener will receive it.
If we use the updated code, the listener will receive the success queue at the same time with the result of buyProduct. We are getting two results for ios for every success results which we need more codes to prevent receiving the same results and doing things twice.

@hyochan
We don't know how fast or slow StoreKit will put the success transaction in the queue.
It's all async so it is not safe to assume that the success transaction will always come in after adding the listener.
I agree also with the listener getting two event for the same successful transaction.
This is not an easy problem :smile:
I'm now using this fork https://github.com/superandrew213/react-native-in-app-utils/tree/listen-for-purchase-event#helpers
And here's my pseudo code to deal with StoreKit flow:
https://github.com/chirag04/react-native-in-app-utils/issues/202#issuecomment-492618182

@hyochan IMHO it's better if users handle purchases more than once instead of having no guarantee that all successful purchases are handled. Since transactions have IDs users have data available to avoid duplicates on their end.
I know it's not ideal but IMHO it's the best suggestion we can give users until we can come up with a better API that hides all these details.
@vafada I'll take a look at your fork 👍

@Edgpaez If you'd like to support a better approach to the readme please do not erase the previous one and add to another paragraph for your approach and add the description on why it'd be better.
Since current approaches are not either perfect it'd be better to have more references.
Thanks for all your supports in advance.

@vafada Since version 3 is out today. This doc is no longer needed. I may go to deprecated readme.


aks-preview: Add podMaxPids argument for node config

I decided to re-apply changes as there was a significant amount of rework before the last PR was attempted to be merged.

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

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your PR is merged into master branch, a new PR will be created to update src/index.json automatically.
The precondition is to put your code inside this repo and upgrade the version in the PR but do not modify src/index.json.

aks-preview

Could you add some tests for this new parameter?

@justindavies May I ask what's the update here?

@justindavies Could you please resolve those conflicts and add some related tests?

Please solve these comments and conflicts in time today, otherwise this PR will miss the release of this sprint and can only be released in the next sprint (09-07)

@justindavies Any update?

Conflicts fixed

@justindavies  Could you add some tests for this new parameter in test_aks_commands.py ?

@justindavies Could you fix the CI issue?

@zhoxing-ms done

@justindavies There is still a conflict file, please address it~


ROX-12692: Remove pods/attach resource from admission controller resources, and fail open on unsupported request kinds/api verbs

Description
The admission controller configuration specifies that we want to listen on pods/attach resource access. However, we don't actually handle that.
This PR does two things: first, it removes the pods/attach resource from the admission controller configuration. This will not cause anything to break, as we're not handling pod attach events anyway, but it will ensure customers kubectl attach calls will not break when they have configured enforcement for K8s events. Second, it makes sure we simply log an error and fail open if we receive a request that we should not receive (per ValidatingWebhookConfiguration), instead of blocking the request.
Checklist

[ ] Investigated and inspected CI test results
[x] ~Unit test and regression tests added~
[x] ~Evaluated and added CHANGELOG entry if required~
[x] Determined and documented upgrade steps: https://docs.engineering.redhat.com/display/StackRox/Upgrade+to+73.0
[x] ~Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)~

If any of these don't apply, please comment below.
Testing Performed
TBD

@md2119 do you have any historic insight as to why pods/attach was added here?

Test failures are known flakes


Multiple Cookie authentication

I edited the sample page to add another cookie which is supposed to redirect to a second login page.
It keeps going back to the VS generated Index page.
services.AddAuthentication()
               .AddCookie(CookieAuthenticationDefaults.AuthenticationScheme).AddCookie("Admin", o =>
               {
                   o.Cookie.HttpOnly = true;
                   o.ExpireTimeSpan = TimeSpan.FromMinutes(15);
                   o.LoginPath = "/Account/TestLogin";
                   o.SlidingExpiration = true;
                   o.Cookie.IsEssential = true;
               });
 services.AddAuthorization(options =>
           {
               var defaultAuthorizationPolicyBuilder = new AuthorizationPolicyBuilder(CookieAuthenticationDefaults.AuthenticationScheme,"Admin");
               defaultAuthorizationPolicyBuilder =
                   defaultAuthorizationPolicyBuilder.RequireAuthenticatedUser();
               options.DefaultPolicy = defaultAuthorizationPolicyBuilder.Build();
           });

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

ID: ee71441f-471a-89cd-0ef4-9b32b5dd85cf
Version Independent ID: 0096c365-2f4e-5d21-89a0-c4f10fcb0ed9
Content: Authorize with a specific scheme in ASP.NET Core
Content Source: aspnetcore/security/authorization/limitingidentitybyscheme.md
Product: aspnet-core
Technology: aspnetcore-security
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Can you phrase this as what's wrong with the doc and what needs to be fixed?

moved to Master issue for: Authorize with a specific scheme #16190


Lambda@Edge and OAuth doc tweak?

The docs about private static websites show a bucket configuration that implies usage of the website endpoint (WebsiteConfiguration):
Resources:
  WebBucket:
    Properties:
      BucketName: !Ref WebDomainName
      WebsiteConfiguration:
        ErrorDocument: 404.html
        IndexDocument: index.html
    Type: AWS::S3::Bucket

But the CloudFront distribution created there is not pointing at the website endpoint:
Origins:
          - DomainName: !Sub ${WebBucket}.s3.amazonaws.com
            Id: !Ref WebBucket
            S3OriginConfig:
              OriginAccessIdentity: !Sub origin-access-identity/cloudfront/${WebOriginAccessIdentity}

Which, if I understand it correctly, is because accessing S3 objects through the website endpoint is incompatible with the origin access identity feature that is used here.
First, this is an awesome project, thank you! I found it to be very helpful.
Second, a question: is my reading accurate that WebsiteConfiguration is not really doing anything here? If so I think it would be helpful to remove it from the template to clarify that the WebsiteConfiguration and S3 website endpoint are incompatible with locking down access to the bucket in this way.
I'm happy to send a PR if it makes sense.

Whether it makes sense or not I'd like to see a PR here.


Tabs component doesn't propagate theme to sub components

When you create a Tabs component and put a theme property on it, then the theme is not passed on to the Tab and TabContent component. This prevents theme styling of these elements.
I fixed it by adding a theme property in the renderHeaders and renderContents methods, but I'm not sure that that is the correct way to solve this issue. If you agree that it is, I will create a PR.

Yeah, we took the approach of giving a single theme for those components that are composed with subcomponents. In this case, since we can inject the theme in the component export, it's possible to forget passing down the theme which definitively should be done.
Your changes sound good, PR it!

When do you expect to push a new release to npm?

Mmm maybe a change affected in the good way 😎
A new release should come during the week, I just wanted to review some pendings PRs to release it all together :)


VS Tools Release - 16.5

VS Tools Release
VS Version: 16.5
Tasks
There is overlap between the tasks here and those for a Patch Tuesday release. In the case where a VS release is on the same day as Patch Tuesday, the tasks should be combined into one workflow. Tasks specific to only a VS release have been bolded.

[x] Merge any pending PRs or commits from dev branch:

[ ] <placeholder link>

[x] Look up the latest NuGet CLI versions and update the nuget-info.json file with the latest patch of each major/minor version listed in the file

[x] Update the vsVersion in vs-info.json to be the new VS version being released

[x] Wait for VS to be released

[x] Install or update the new version of VS on your local machine

[x] Create a zip file named MSBuild.Microsoft.VisualStudio.Web.targets.<YYYY>.<MM>.zip that contains the MSBuild\Microsoft\VisualStudio\v16.0\Web and MSBuild\Microsoft\VisualStudio\v16.0\WebApplications folders from the VS installation location

[x] Upload the zip file to https://dotnetbinaries.blob.core.windows.net/dockerassets

[x] Update the webTargetsUrl in vs-info.json to reference the new zip file name

[x] Gather the Build Tools and Test Agent download URLs by navigating to https://visualstudio.microsoft.com/downloads. Download links can be found under the Tools for Visual Studio section. Prior to clicking the download button, open a network profiler like your browser's dev tools or Fiddler so that you can capture the URL that gets downloaded. For Build Tools for Visual Studio look for a file named vs_BuildTools.exe being downloaded. For Agents for Visual Studio, ensure the radio button is defaulted to Agent and look for a file named vs_TestAgent.exe. Copy those two URLs and paste them into vs-info.json.

[x] Run the update-dependencies tool to update all the necessary files:

[ ] dotnet run --project .\eng\update-dependencies\update-dependencies.csproj --datestamp-sdk <YYYYMMDD> Note that this uses a different option than a Patch Tuesday release because only the SDK images should be updated here. If VS and Patch Tuesday releases align, then --datestamp-all should be used.

[x] Inspect generated changes for correctness

[x] Commit generated changes

[x] Create PR

[x] Get PR signoff

[x] Merge PR

[x] Wait for changes to be mirrored to internal dotnet-framework-docker repo (internal MSFT link)

[x] Queue build of dotnet-framework-docker pipeline (internal MSFT link)

[x] Confirm images have been ingested by MCR

[x] Confirm READMEs have been updated in Docker Hub for microsoft-dotnet-framework

Release is complete.


northd: remove Controller_Event from sb_input_relations and get_sb_ops

Since Northd does not modify Cotroller_Event table remove it from
sb_input_relations array and get_sb_ops routine
Signed-off-by: Lorenzo Bianconi<EMAIL_ADDRESS>

Manually applied this patch.


ffprobe fails silently when running inside a separate spawned thread

Hey, is there a reason why this does not work in Rust threading that I haven't grasped, or is there something up with ffprobe?
fn main() {
    std::thread::spawn(
        move || match ffprobe::ffprobe("D:\\video.mp4") {
            Ok(info) => {
                dbg!(info);
            }
            Err(err) => {
                eprintln!("Could not analyze file with ffprobe: {:?}", err);
            }
        },
    );
}

You are not waiting for the thread to finish, so main() will exit immediately before the thread has a chance to run.
You need to do spawn().join().


Activate Proven Headers using deployments

Currently we activate PH using a param in the network class (this will be ok for testnet) we sould consider using BIP9 deployment mechanisem to activate it on mainnet

+1 for using the BIP9 mechanism. We may have to introduce a network-specific enum so that we don't have overlapping bit definitions, or bits that are applicable to BTC and not STRAT, or vice versa.

https://github.com/stratisproject/StratisBitcoinFullNode/issues/2464

Tracking this https://github.com/stratisproject/StratisBitcoinFullNode/pull/2680

Fixed by having it activated by default instead of having activation height


feature request - retrict_to option for leiden clustering

Hi,
I was wondering, if you can synchronize the functionality of the louvain and leiden clustering algorithm implementations.
sc.tl.louvain has the restrict_to parameter, which allows subclustering of a specific cluster (set), while sc.tl.leiden does not (Note: I have scanpy==1.4+18.gaabe446).
I'd be happy to have that.
Best,
M

sure, sounds sensible! related issue: #570

Hi! I just wrote a quick solution in https://github.com/theislab/scanpy/pull/586
I tested it manually and it seems to work (I used louvain code as template).
I assumed it could be interesting to work on a separate leiden function, due to possible argument clashes with louvain, instead of merging the two functions together.

It should be fixed on master by https://github.com/theislab/scanpy/pull/586

Great, thank you again!


PHP Notice: Undefined variable: output in .../VariantBuilder.php on line 243

Fixes #919. Replaced $output with $prefix, use printf() for cleaner error message.

Coverage remained the same at 52.293% when pulling b416d07dbc838754359321dca49e0d97d3641a42 on morozov:issue-919 into 6fff9cd60d6e41c1ca8ee007731157a1b306a4a4 on phpbrew:master.


Source maps in flutter run -d chrome --release  doesn't work for path dependencies (and maybe other situations)

If we take the flutter counter app (created via flutter create app) and modify it slightly via
dependencies:
+  helper:
+    path: ../helper

and
+ import 'package:helper/helper.dart';
...
+ int i = 0;
class _MyHomePageState extends State<MyHomePage> {
  int _counter = 0;

  void _incrementCounter() {
+    if (++i % 5 == 0) helper(() => throw 'baz');
    setState(() {

and have the helper method in package:helper call some things which eventually call the given closure to it
// package:helper/helper.dart (aka ../helper/lib/helper.dart)
helper(void Function() fun) {
  print('helper');
  helper1(fun);
}

helper1(void Function() fun) {
  print('helper2');
  helper2(fun);
}

helper2(void Function() fun) {
  print('helper2');
  fun();
}

, then we can break on the exception in chrome.
What we'll observe is that

The core library methods have no source maps (e.g. js_helper.dart:throwExpression)
The lib/main.dart has valid source maps
The package:helper/helper.dart has invalid source maps (for me it shows a html file in chrome)
The package:flutter/* code has valid source maps

So either

we have to instruct dart2js to emit different source maps for packages (e.g. package:helper in this case) or
we have to make the flutter run tool's http server serve the files for the filepaths currently emitted by dart2js

(On dart2wasm this issue doesn't exist because we currently bake in absolute file url which work very well on developer machines, see https://github.com/dart-lang/sdk/issues/56232 for follow up tasks)
/cc @yjbanov @sigmundch @a-siva  for finding someone to own this

Reproducible using the steps outlined above.

flutter doctor -v
[✓] Flutter (Channel stable, 3.22.2, on macOS 14.4.1 23E224 darwin-arm64, locale en-GB)
    • Flutter version 3.22.2 on channel stable at /Users/nexus/dev/sdks/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 761747bfc5 (5 weeks ago), 2024-06-05 22:15:13 +0200
    • Engine revision edd8546116
    • Dart version 3.4.3
    • DevTools version 2.34.3

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/nexus/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • Java binary at: /Users/nexus/Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.10+0-17.0.10b1087.21-11609105)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.3)
    • Xcode at /Applications/Xcode-15.3.0.app/Contents/Developer
    • Build 15E204a
    • CocoaPods version 1.15.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2024.1)
    • Android Studio at /Users/nexus/Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.10+0-17.0.10b1087.21-11609105)

[✓] IntelliJ IDEA Ultimate Edition (version 2023.2.5)
    • IntelliJ at /Users/nexus/Applications/IntelliJ IDEA Ultimate.app
    • Flutter plugin version 77.2.2
    • Dart plugin version 232.10286

[✓] VS Code (version 1.91.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.92.0

[✓] Connected device (4 available)
    • Nexus (mobile)                  • 00008020-001875E83A38002E • ios            • iOS 17.5.1 21F90
    • macOS (desktop)                 • macos                     • darwin-arm64   • macOS 14.4.1 23E224 darwin-arm64
    • Mac Designed for iPad (desktop) • mac-designed-for-ipad     • darwin         • macOS 14.4.1 23E224 darwin-arm64
    • Chrome (web)                    • chrome                    • web-javascript • Google Chrome 126.0.6478.127
    ! Error: Browsing on the local area network for Dean’s iPad. Ensure the device is unlocked and attached with a cable or associated with the same local area network as this Mac.
      The device must be opted into Developer Mode to connect wirelessly. (code -27)

[✓] Network resources
    • All expected network resources are available.

• No issues found!

[!] Flutter (Channel master, 3.24.0-1.0.pre.93, on macOS 14.4.1 23E224 darwin-arm64, locale en-GB)
    • Flutter version 3.24.0-1.0.pre.93 on channel master at /Users/nexus/dev/sdks/flutters
    ! Warning: `flutter` on your path resolves to /Users/nexus/dev/sdks/flutter/bin/flutter, which is not inside your current Flutter SDK checkout at /Users/nexus/dev/sdks/flutters. Consider adding /Users/nexus/dev/sdks/flutters/bin to the front of your path.
    ! Warning: `dart` on your path resolves to /Users/nexus/dev/sdks/flutter/bin/dart, which is not inside your current Flutter SDK checkout at /Users/nexus/dev/sdks/flutters. Consider adding /Users/nexus/dev/sdks/flutters/bin to the front of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision dddea4d155 (10 hours ago), 2024-07-11 17:16:10 -0700
    • Engine revision 36dccf7bb2
    • Dart version 3.6.0 (build 3.6.0-26.0.dev)
    • DevTools version 2.37.1
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/nexus/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • Java binary at: /Users/nexus/Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.10+0-17.0.10b1087.21-11609105)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.3)
    • Xcode at /Applications/Xcode-15.3.0.app/Contents/Developer
    • Build 15E204a
    • CocoaPods version 1.15.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2024.1)
    • Android Studio at /Users/nexus/Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.10+0-17.0.10b1087.21-11609105)

[✓] IntelliJ IDEA Ultimate Edition (version 2023.2.5)
    • IntelliJ at /Users/nexus/Applications/IntelliJ IDEA Ultimate.app
    • Flutter plugin version 77.2.2
    • Dart plugin version 232.10286

[✓] VS Code (version 1.91.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.92.0

[✓] Connected device (4 available)
    • Nexus (mobile)                  • 00008020-001875E83A38002E • ios            • iOS 17.5.1 21F90
    • macOS (desktop)                 • macos                     • darwin-arm64   • macOS 14.4.1 23E224 darwin-arm64
    • Mac Designed for iPad (desktop) • mac-designed-for-ipad     • darwin         • macOS 14.4.1 23E224 darwin-arm64
    • Chrome (web)                    • chrome                    • web-javascript • Google Chrome 126.0.6478.127
    ! Error: Browsing on the local area network for Dean’s iPad. Ensure the device is unlocked and attached with a cable or associated with the same local area network as this Mac.
      The device must be opted into Developer Mode to connect wirelessly. (code -27)

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

I believe the proper fix for this issue lies needs to be done within the flutter tool.

Context
We have a very deliberate agnostic approach in dart2js. When in the past we tried to match one use case and ended up breaking others. Today we support local development (see sources in the debugger), crash reports analysis (using a detached deobsfuscation tool), and QA debugging (sources in the debugger of a deployed asset), by following a common recipe.
Our recipe is a combination of a careful use of input URIs while building, and post-processing of source-maps before serving.
Input URIs: Internally, dart2js will use whatever URIs the CFE uses for resolved file URI in kernel. However, these are not always file:// URIs (as you noted for the SDK). We often invoke the CFE with a file system abstraction (e.g. the multi-root file system) and provideda custom URI scheme (such as org-dartlang-sdk://). This scheme is an abstract representation of the root of a project, and allows us to encode path in a root-agnostic way. Even .dart_tools/package_spec.json is served via this abstraction and can be resolved with the custom URI scheme. While in some cases we can work without them, custom URIs are necessary whenever distributed builds are used (like g3) or when artifacts are prebuild (like our SDK). Before using custom URIs, we tried absolute file URI, but they included paths belonging to non-local machines, and we tried relative file URIs, but they were not rooted properly and didn't compose well when using modular compilation across different roots.
Post processing: whenever custom-schemes are used, the post-processing step is where we change the .map file to point to the proper location where they are meant to be found. This location also varies (e.g. a local server may have a special root path that it uses to serve contents, like localhost:8080/sources/packages/...), but a deobfsucation tool may lookup files differently. In the QA scenario, it's possible to use a server that supports versioned sources (like cs).

Circling back to the issue here, I believe we need to adapt the flutter tool to follow this recipe as well. In particular:

The missing location for core libraries is already because the SDK has a custom URI, but the .map file is not post-processed before being served.
My guess with the path dependency is that the source-map encodes the relative path and it goes up above the path used by the flutter tool server to host source files. Instead, we need to properly encapsulate the project root to fix the issue.


Question: can I apply a partial segmentation control image?

Summary
I would like to achieve some masked inpainting, but want to limit the segmentation to only a part of the image, and left regions out of the segmentation mask without constraints.
Motivation
The motivation comes from the pains of regular inpainting - if I only supply a mask and a prompt without further constraints, the model will usually add unwanted features to the subject. See below example, the inpainted image grows extra legs on the dog.
Prompt: "dog on the beach, best quality"

Original image from segment-anything example
Mask
Inpainted image with controlnet-inpaint

To prevent this, I am thinking of using a segment-mask control net to constrain the extent of the primary subject. I use the following script to assign the masked region as segment 1, and other region as segment 0, but it results in the model recognizing everything else uniformly. See produced image.
        def mask_to_control(mask):
            # mask is a binary mask after HWC3(...)
            res = np.zeros((mask.shape[0], mask.shape[1], 3))
            res[:, :, 0] = np.where(mask[:, :, 0] >= 128, 0, 1)
            return res

I would like the image not be bounded by the segmentation mask outside of the masked region, is that possible?
I am still new with controlnet. Please also don't hesitate to point out any mistakes. Glad to hear from all of ya!

@ikuinen thank you for your quick reply! I tried lower mask alignment strength, and it is not very optimal - it sometimes will grow extra legs, while not diversifying the background much.

Another way is to give pixel-wise control scales instead of a single scalar.

Can you elaborate on this approach? Which parameter should I supply? 🙏

Thanks for your interest. The mask alignment strength is a control scalar, while I think you could use a heatmap-like control scale. The value in region of dog could be one. The area around the dog could be lower and the rest could be zero. Just a simple idea and haven't tried yet. This may be a further feature that can be added into EditAnything.

@ikuinen Thank you for your insight - I would like to experiment your approach, but I didn't locate a way to provide a heatmap as you described.
In StableDiffusionControlNetInpaintPipeline, __call__: I think control_net_conditioning_scale is a float or List[float], and the alignment_ratio looks like a Optional[float].
Could you give a code pointer of how this is achieved? Much thanks!

That is amazing. You are awesome. I'll try it at my earliest convenience.


service component could not start popup when opening Flat File Import Wizard

Azure Data Studio Version:
Version: 1.40.0-insider
Commit: 9d921e4efc2a00c5c9c8f440c32fa2f6c9277737
Date: 2022-11-02T05:43:35.004Z
VS Code: 1.67.0
Electron: 17.4.11
Chromium: 98.0.4758.141
Node.js: 16.13.0
V8: <IP_ADDRESS>-electron.0
OS: Linux x64 5.10.150-gentoo
OS Version: Linux x64 5.10.150-gentoo

Steps to Reproduce:

open "Flat File Import Wizard"
error gets displayed

the installation seems to work fine, additional deps get pulled in on first start but after that the generic error message is displayed and thats it.
Nothing suspicious in the output window, selecting "Flat File Import Service" in the dropdown comes up empty.
There is an error in the JS console though:

The same in Ubuntu 22.10 and Azure Data Studio 1.40.1 with SQL Server Import 1.5.3

dup of #21313


ENH: Switch to codecov

Switch from buggy coveralls to codecov

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@4eeaaa8). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #91   +/-   ##
=========================================
  Coverage          ?   94.25%
=========================================
  Files             ?       15
  Lines             ?     2490
  Branches          ?        0
=========================================
  Hits              ?     2347
  Misses            ?      143
  Partials          ?        0

Continue to review full report at Codecov.

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


Close #3: Add testing workflows

I added the most recent workflows that you sent me in e54a705425d8ca2a731103fad7e23c30d923dfee (the're in vipe/oozie/test/data directory). Apart from these workflows, the commit contains YAML files generated from these workflows using vipe/oozie/test/data/run_oozie2yaml_on_workflows.sh script.
Now for the workflows: I need workflow definitions that potentially would be executable if appropriate Java jars, Pig scripts etc. were present. Such testing workflows would be close enough to real-life workflows for my current needs. Can you modify the workflows so they are like that?
For example, in "bypass" workflow the "consumer" ingests metadata that is not produced anywhere earlier in this workflow. This is bad since it will upset the code that analyzes such things that will be implemented in #5.

Now for the workflows: I need workflow definitions that potentially would be executable if appropriate Java jars, Pig scripts etc. were present. Such testing workflows would be close enough to real-life workflows for my current needs. Can you modify the workflows so they are like that?

This is what I asked about in https://github.com/openaire/vipe/issues/3#issuecomment-152222681. So they should be kind of executable then. What is the priority of this task?

This is what I asked about in #3 (comment). So they should be kind of executable then.

Yeah.

What is the priority of this task?

If I I could get them today this would be great.

I've just updated workflows.

In conditional, fs, distcp, and pig examples you use a Finalizer class in the final node. It doesn't ingest any data so the data flow analysis won't show much. Is it possible to add some real data processing to these examples (so e.g., fs action produces some data that is then ingested by a consumer)? This is certainly possible for the pig example, but I'm not sure if for the rest of them it would be a "realistic" case.
Can you add a new fork example (fork2?) with a single consumer that ingests document and person data stores?

OK, I am getting back to this issue.

In conditional, fs, distcp, and pig examples you use a Finalizer class in the final node. It doesn't ingest any data so the data flow analysis won't show much. Is it possible to add some real data processing to these examples (so e.g., fs action produces some data that is then ingested by a consumer)? This is certainly possible for the pig example, but I'm not sure if for the rest of them it would be a "realistic" case.

I've slightly improved fs and pig workflows. Yaml files should be regenerated then.
Anyways, shouldn't we keep those workflows compact (now I am thinking of conditional, distcp)? When I start to supplement each one of them by complex processing we will loose focus on nodes under test and workfows would become similar to each other.

Can you add a new fork example (fork2?) with a single consumer that ingests document and person data stores?

Done, yaml file is not generated yet.

I've slightly improved fs and pig workflows. Yaml files should be regenerated then.

Thanks.

fs worfklow looks strange from POV of producing and consuming data, namely init-workingDir action creates an empty directory and check-empty-dir consumes it by verifying that it's empty. But maybe that's the most realistic producer-consumer case related to fs that we can get?
Since you haven't introduced any changes to conditional and distcp, I guess the present form of the worflows is the most realistic producer-consumer scenario that we can get?

Anyways, shouldn't we keep those workflows compact (now I am thinking of conditional, distcp)? When I start to supplement each one of them by complex processing we will loose focus on nodes under test and workfows would become similar to each other.

I don't want to have "complex" processing but real-life-like data processing producer-consumer scenarios. This way the data data flow analysis method has non-trivial scenarios to work on.

Can you add a new fork example (fork2?) with a single consumer that ingests document and person data stores?

Done, yaml file is not generated yet.

Ok.

BTW, I re-generated YAML files for the tests. The tests pass correctly now.

fs worfklow looks strange from POV of producing and consuming data, namely init-workingDir action creates an empty directory and check-empty-dir consumes it by verifying that it's empty. But maybe that's the most realistic producer-consumer case related to fs that we can get?

Yeah, I realize it is far from realistic use case but shouldn't it be enough in terms of the action flow? We  could rename EmptyDirChecker to be more sophisticated than the current name suggests...

Since you haven't introduced any changes to conditional and distcp, I guess the present form of the worflows is the most realistic producer-consumer scenario that we can get?

I was trying to find both easy (time-wise) and complex enough scenarios. If complexity is not enough we could always think of something better but let's not forget about time factor.
I am not sure we need to follow producer-consumer scenario here since those examples could follow different pattern than the one used in integration tests. E.g. why would we need producer in distcp example? conditional example also covers core functionality and adding producer wouldn't bring much to the table unless in the next step we wan't to execute those workflows.
Btw, how do reply to comments with attaching quoted text? I just realized this can be that easy:
https://storify.com/TheAllBox/reply-with-quoted-text-on-github

fs worfklow looks strange from POV of producing and consuming data, namely init-workingDir action creates an empty directory and check-empty-dir consumes it by verifying that it's empty. But maybe that's the most realistic producer-consumer case related to fs that we can get?

Yeah, I realize it is far from realistic use case but shouldn't it be enough in terms of the action flow? We could rename EmptyDirChecker to be more sophisticated than the current name suggests...

I'm interested more in data flow, not in what is the order of execution of different actions. But this should be fine for now.

Since you haven't introduced any changes to conditional and distcp, I guess the present form of the worflows is the most realistic producer-consumer scenario that we can get?

I was trying to find both easy (time-wise) and complex enough scenarios. If complexity is not enough we could always think of something better but let's not forget about time factor.
I am not sure we need to follow producer-consumer scenario here since those examples could follow different pattern than the one used in integration tests. E.g. why would we need producer in distcp example? conditional example also covers core functionality and adding producer wouldn't bring much to the table unless in the next step we wan't to execute those workflows.

Ok.

Btw, how do reply to comments with attaching quoted text? I just realized this can be that easy:
https://storify.com/TheAllBox/reply-with-quoted-text-on-github

Cool. It's a pity that it doesn't handle text that is already quoted properly though.

OK, I guess you can merge it, I don't have any new comments to add.


[CL][Swapping]: investigate implementation of swap and calc pool interface methods within a bucket

Background
This issue is focused on investigating, prototyping, and documenting how to implement our calc and swap methods to swap within a single bucket with hard-coded values. The goal is to gain an understanding of specific steps needed to implement:

SwapOutAmtGivenIn / CalcOutAmtGivenIn
SwapInAmtGivenOut / CalcOutAmtGivenOut

Acceptance Criteria

Summary of the subsequent steps
Summary of constraints that we need to tackle (e.g. logarithm implementation)
New issues created

Closed by: https://github.com/osmosis-labs/osmosis/pull/3047


add a getter for the shared secret of Box

Useful if you need key agreement for something else than encrypting messages.

Alternatively this could be a toBytes() method. That would mean Box would inherit from Key. Would that be better?


Fix minor memory leaks

While debugging https://github.com/runtimeverification/llvm-backend/issues/770, I found a few small memory leaks in the C bindings and associated tests. This PR cleans up places where we were not properly freeing temporary objects.
The fixes are unrelated to the Python bindings, but allow me to check that nothing obvious is being leaked by the underlying associative desugaring code.

pyk integration tests are passing.


fix: bigint corruption in lmdb

An attempt at fixing the bigint decoding corruption that we've seen sporradically in CI

#6002  👈
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @alexghr and the rest of your teammates on  Graphite


Multiple sources path in config.

Is there a way to provide multiple folders for compilation? My goal is to have contracts for testing purposes only and I don't want to put them inside the "contracts" folder.
Originally I wanted to put them inside the test folder, but I couldn't find a way to provide that in the config.
Something like this:

Or is there any other way I can achieve something similar?

Hey @kraikov, I don't think this is possible with Hardhat. Looking at their documentation, the sources field seems to be a single directory where all contracts are stored. What I do in my projects is to create a subfolder named "test" within the "contracts" folder.
At any rate, I would re-ask this question in the Hardhat Discord. They may be able to implement this feature at some point.


ci: use minimal docker image for build test

Any idea why it failed?

Ah, might be related to https://github.com/golioth/golioth-firmware-sdk/actions/runs/6499180729/job/17652019275#step:5:7. I've seen request issues in the past.

I figured out the issue with the build (zephyrproject-rtos/zephyr/issues/63948,) and now can successfully build again.
Here's what I changed in the base image:

Moved the SDK to align what's in this workflow
Added a global override for Python / Ubuntu warnings

We'll need to decide if this is okay or move to either venv, a 3P PPA or build from source (WHY PYTHON, WHYYYY)

I verified this with my test RD template repo. There's a few things I'm doing there. I think the only thing that needs to be added now is the Setup West workspace step or similar. This was probably already done in the massive CI image.


Adding default rule support for subscription creation

Description
(TODO - Yet to add recording for integration tests. Expect one more commit adding recording files. As such the code is complete)

Adding default rule support to subscriptions. Today there is a gap from when a subscription is created , to when correct rules are configured when incorrect messages can get through. This addresses that gap.
This also fixes the issue of missing tests for sync admin client which caused a hotfix release. (Added Sync AdminClientIntegrationTests).

All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

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

Testing Guidelines

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

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-messaging-servicebus


Apache Age install required

Maybe this should have been obvious, but I ran into errors when running bin/rails apache_age:install because I hadn't installed https://github.com/apache/age on my machine yet. I'll try to create a PR with updated documentation, just wanted to leave this here as a reminder
error looked like this:
PG::FeatureNotSupported: ERROR:  extension "age" is not available

Thanks, I will have added this to the documentation.


[BUG] 8*A100（80G）lora训练为啥会OOM，每次50步的时候oom

是否已有关于该错误的issue或讨论？ | Is there an existing issue / discussion for this?

[X] 我已经搜索过已有的issues和讨论 | I have searched the existing issues / discussions

该问题是否在FAQ中有解答？ | Is there an existing answer for this in FAQ?

[X] 我已经搜索过FAQ | I have searched FAQ

当前行为 | Current Behavior
CUDA_VISIBLE_DEVICES 0,1,2,3,4,5,6,7
swift sft
--model_type minicpm-v-v2_5-chat
--model_id_or_path "/mnt/workspace/pre_train_model/openbmb/MiniCPM-Llama3-V-2_5"
--dataset "/mnt/workspace/app_source/data/last_1000_records.json"
--output_dir "/mnt/workspace/app_source/output"
--max_length 1024
--batch_size 1
--gradient_accumulation_steps 8
--resume_only_model True
--sft_type 'lora'
期望行为 | Expected Behavior
No response
复现方法 | Steps To Reproduce
No response
运行环境 | Environment
- OS:
- Python:
- Transformers:
- PyTorch:
- CUDA (`python -c 'import torch; print(torch.version.cuda)'`):

备注 | Anything else?
torch.cuda.OutOfMemoryError: CUDA out of memory. Tried to allocate 320.00 MiB. GPU 6 has a total capacty of 79.15 GiB of which 15.31 MiB is free. Process 122606 has 79.12 GiB memory in use. Of the allocated memory 77.49 GiB is allocated by PyTorch, and 683.81 MiB is reserved by PyTorch but unallocated. If reserved but unallocated memory is large try setting max_split_size_mb to avoid fragmentation.  See documentation for Memory Management and PYTORCH_CUDA_ALLOC_CONF

您可以打印一下输入的shape是不是第50步的时候 输入比较大

swift的eval有问题，在训练中作验证时导致非常大的显存增加，此时应该避免eval，将eval_steps调整到一个非常大的值


The debug console still has  character encoding issue in flutter project

I have to look up each error number or guess the error. really give me hard time.

No such issue If it is a pure c/c++ project.

Please file this on the Flutter extension


fix(solid-query): Simplify SSR Hydration Strategy

Noticed while creating another example with Astro integration that a query with select option did not correctly hydrate the underlying query. This simplifies the hydration payload by adding a hydrationData to the SSR payload

Codecov Report

Merging #7291 (350515f) into main (93674fe) will decrease coverage by 0.48%.
Report is 81 commits behind head on main.
The diff coverage is 28.57%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files

@@            Coverage Diff             @@
##             main    #7291      +/-   ##
==========================================
- Coverage   41.42%   40.95%   -0.48%
==========================================
  Files         184      182       -2
  Lines        7331     7381      +50
  Branches     1531     1535       +4
==========================================
- Hits         3037     3023      -14
- Misses       3889     3946      +57
- Partials      405      412       +7

Components
Coverage Δ

@tanstack/angular-query-devtools-experimental
∅ <ø> (∅)

@tanstack/angular-query-experimental
86.46% <0.00%> (ø)

@tanstack/eslint-plugin-query
85.29% <100.00%> (ø)

@tanstack/query-async-storage-persister
43.85% <100.00%> (-5.35%)
:arrow_down:

@tanstack/query-broadcast-client-experimental
∅ <ø> (∅)

@tanstack/query-codemods
0.00% <0.00%> (ø)

@tanstack/query-core
92.86% <79.59%> (-0.40%)
:arrow_down:

@tanstack/query-devtools
3.92% <0.00%> (-0.02%)
:arrow_down:

@tanstack/query-persist-client-core
57.73% <ø> (ø)

@tanstack/query-sync-storage-persister
82.50% <ø> (ø)

@tanstack/react-query
92.77% <100.00%> (ø)

@tanstack/react-query-devtools
10.71% <ø> (ø)

@tanstack/react-query-next-experimental
∅ <ø> (∅)

@tanstack/react-query-persist-client
100.00% <ø> (ø)

@tanstack/solid-query
77.27% <63.88%> (-5.00%)
:arrow_down:

@tanstack/solid-query-devtools
∅ <ø> (∅)

@tanstack/solid-query-persist-client
100.00% <ø> (ø)

@tanstack/svelte-query
62.68% <100.00%> (ø)

@tanstack/svelte-query-devtools
∅ <ø> (∅)

@tanstack/svelte-query-persist-client
100.00% <100.00%> (ø)

@tanstack/vue-query
71.04% <90.47%> (+0.23%)
:arrow_up:

@tanstack/vue-query-devtools
∅ <ø> (∅)


Rotate with offset/anchor/origin

Hi,
is it possible to add a function for rotating an image and specify the offset ourself? I've checked the code and it seems the function for this is already there ... Just private ...
This would be a great help.

I'm closing the issue for now. Feel free to comment/reopen if you have any further questions.


pull request for e2e test at 11/09/2016 08:45:06

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

Details

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


ScrollController should notify when its offset changes

This change make it easier to track the position of the scroll view without
having to worry about the position object changing out from under the
controller.

@Hixie @apwilson

LGTM.


Steps 3a and 3b

Step 2 refers to steps 3a and 3b, which do not exist in the current document. It seems to have been this way since the doc was moved to its current location.
Can you either restore the differentiation between 3a and 3b, or remove the references to them and clarify what you are supposed to do in each case?
As far as I can tell, all the info in the current Step 3 is a single step,

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

ID: 7c1d3d23-bf8a-1b97-0d2c-81f37f30bca1
Version Independent ID: ebc70b53-8796-7a16-a781-a6986d3e823a
Content: Monitor your SQL pool workload using DMVs - Azure Synapse Analytics
Content Source: articles/synapse-analytics/sql-data-warehouse/sql-data-warehouse-manage-monitor.md
Service: synapse-analytics
Sub-service: sql-dw
GitHub Login: @ronortloff
Microsoft Alias: rortloff

Perhaps Step 3a is supposed to be 3, and 3b is step 4?

@dcbrown16 Thanks for your contribution, I've addressed this issue in PR: https://github.com/MicrosoftDocs/azure-docs-pr/pull/127585
The changes should be live EOD or tomorrow. In future, if you see any minor bug fixes in the doc, feel free to click on the edit button and submit a PR to update. we love to take community contributions.

I wasn't sure what the right answer was, otherwise I would have made the change myself. Thanks for checking!


[bug][Knowhere2.0] RangeSearch is slower than version 1.3, and precision is not allowed

ivfflat index latency:

In version 1.3, nprobe is [1, 2, 4, 8, 16, 32, 64, 128, 256, 512] and the delay is
[0.5701, 1.0174, 2.0392, 3.8497, 7.4166, 12.7004, 23.1321, 45.3427, 91.5857, 186.9473]

/assign @cydrain

fixed by #618

After verification, the performance problem is solved, but the problem of precision of 0 still exists

When I use the new sift dataset the case passes and the precision is fine. But when I use the glove dataset, the case still does not pass
pytest test_ivf.py::TestIvf::test_ivf_recall --data_type glove

After verification and replacement of the sift-range dataset, the case passed.


feat: Added regexp_match operator support for bigquery

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[x] Make sure to open an issue as a bug/issue before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[x] Ensure the tests and linter pass
[x] Code coverage does not decrease (if any source code was changed)
[x] Appropriate docs were updated (if necessary)

Adds support for regexp_match sqlalchemy operator https://docs.sqlalchemy.org/en/14/core/sqlelement.html#sqlalchemy.sql.expression.ColumnElement.regexp_match

Hey folks 👋 just wanted to loop back in here. Let me know if you need anything else from my end to move forward with the PR 🙂

Hey folks 👋 just wanted to loop back in here. Let me know if you need anything else from my end to move forward with the PR 🙂

@parthea can you please review this PR. We are waiting for this to be merged. If there is any feedback please let us know. Thanks

@alvarowolfx @chalmerlowe @parthea we really need help with merging this PR. We are essentially maintaining a fork just to get this PR into our releases. Please let us know what we need to do get this merged in. Thanks

@harshach I will do what I can to clear my schedule to prioritize looking at this PR this week.
Your patience is appreciated. I will provide feedback as soon as I am able to examine this item.

Thanks @chalmerlowe I'll address the comment. 🙂

@chalmerlowe just pushed the changes based on your comments. Quick question regarding import test_utils.prefixer and import test_utils.retry, are these internal packages? I was trying to install these to execute the tests on my side but could not figure out what package this is.

@chalmerlowe just pushed the changes based on your comments. Quick question regarding import test_utils.prefixer and import test_utils.retry, are these internal packages? I was trying to install these to execute the tests on my side but could not figure out what package this is.

I will take a look at the changes!
test_utils should be based on this public repo managed by Google:

It should be possible to install it locally using the following command:
pip install google-cloud-testutils

Thanks @chalmerlowe. I was able to run the tests on my end. Tests are passing in my environment ✅

@TeddyCr The coverage tests are failing in kokoro. The coverage doesn't believe the new lines of code are adequately covered by testing.
Let me figure out how I wanna handle that.
I will provide some feedback as soon as I can.

@TeddyCr The coverage tests are failing in kokoro. The coverage doesn't believe the new lines of code are adequately covered by testing.
Let me figure out how I wanna handle that.
I will provide some feedback as soon as I can.

@chalmerlowe let me know how you want to proceed

@TeddyCr
I have been looking at some options and conferring with some colleagues. I am hoping to have an answer by Monday.

@TeddyCr
I have been looking at some options and conferring with some colleagues. I am hoping to have an answer by Monday.

Thank you 🙏

@TeddyCr Had a headache yesterday. Should have something for you today.

@TeddyCr
I suggested a few items for your consideration. Those suggestions might be showing up on your original commits on your repo as opposed to being present in this thread. Sorry about that. Computers are hard. : )

@TeddyCr
I have some time today. I am gonna try to migrate my suggested changes into this repo along with your changes.
If the tests pass, I plan on trying to close out this PR.

Sounds good, thanks @chalmerlowe let me know if you need me to do anything.

@TeddyCr
Nothing I do seems to help. I have been trying to add edits, etc to this PR, but I am failing. All my suggestions go straight to your original repo as suggestions from me.
Can you please go to your repo and accept those changes to the PR and see if we can get them pulled over to this repo that way? Here is a link:
TeddyCr#1

Just merged the changes to my PR 🙂

@TeddyCr Thank you for your time and effort. And thanks for your patience.
This should be included in the next release.


[BUG]: CUDA extension build skipped when installing from source

🐛 Describe the bug
Hi,I use Install From Source option to install colossalAI,but i encouter problem like:
/path/to/myconda/anaconda3/envs/py37-pt111-cu111-colai/lib/python3.7/site-packages/torch/autocast_mode.py:162: UserWarning: User provided device_type of 'cuda', but CUDA is not available. Disabling   warnings.warn('User provided device_type of \'cuda\', but CUDA is not available. Disabling') Colossalai should be built with cuda extension to use the FP16 optimizer If you want to activate cuda mode for MoE, please install with cuda_ext!
I have installed torch1.11+cu11.3 and using cuda 11.1
any suggestion?
Environment
Pytorch 1.11+cu11.3
CUDA 11.1

Hi, you should use torch built with cuda 11.1 as your cuda version is 11.1.

Generally, this is allowed in Colossal-AI to build with torch 1.11+cu11.3 and cuda 11.1. However, I think the problem is that you don't have a GPU on your system as the log says CUDA is not available.

You can check this by
python -c "import torch;print(torch.cuda.is_available());"

You can check this by
python -c "import torch;print(torch.cuda.is_available());"

thanks for your reply.Now I assured my cuda is available,but I got error like:
/mnt/cache/share/cuda-11.1/bin/nvcc -I/tmp/pip-req-build-2nvgz3bf/colossalai/kernel/cuda_native/csrc/kernels/include -I/path/to/conda/anaconda3/envs/py
37-pt111-cu111-colai/lib/python3.7/site-packages/torch/include -I/path/to/conda/anaconda3/envs/py37-pt111-cu111-colai/lib/python3.7/site-packages/torch/in$
lude/torch/csrc/api/include -I/path/to/conda/anaconda3/envs/py37-pt111-cu111-colai/lib/python3.7/site-packages/torch/include/TH -I/mnt/cache/yangwendi.ven$
or/anaconda3/envs/py37-pt111-cu111-colai/lib/python3.7/site-packages/torch/include/THC -I/mnt/cache/share/cuda-11.1/include -I/path/to/conda/anaconda3/env$
/py37-pt111-cu111-colai/include/python3.7m -c colossalai/kernel/cuda_native/csrc/scaled_upper_triang_masked_softmax_cuda.cu -o build/temp.linux-x86_64-3.7/colossalai/k$
rnel/cuda_native/csrc/scaled_upper_triang_masked_softmax_cuda.o -D__CUDA_NO_HALF_OPERATORS__ -D__CUDA_NO_HALF_CONVERSIONS__ -D__CUDA_NO_BFLOAT16_CONVERSIONS__ -D__CUDA$
NO_HALF2_OPERATORS__ --expt-relaxed-constexpr --compiler-options '-fPIC' -O3 --use_fast_math -DVERSION_GE_1_1 -DVERSION_GE_1_3 -DVERSION_GE_1_5 -U__CUDA_NO_HALF_OPERAT$
RS__ -U__CUDA_NO_HALF_CONVERSIONS__ --expt-relaxed-constexpr --expt-extended-lambda -gencode arch=compute_70,code=sm_70 -gencode arch=compute_80,code=sm_80 -DTORCH_API$
INCLUDE_EXTENSION_H -DPYBIND11_COMPILER_TYPE="_gcc" -DPYBIND11_STDLIB="_libstdcpp" -DPYBIND11_BUILD_ABI="_cxxabi1011" -DTORCH_EXTENSION_NAME=colossal_scaled_upper_tria$
g_masked_softmax -D_GLIBCXX_USE_CXX11_ABI=0 -std=c++14
    /path/to/conda/anaconda3/envs/py37-pt111-cu111-colai/lib/python3.7/site-packages/torch/include/torch/csrc/api/include/torch/nn/cloneable.h: In instant$
ation of ‘std::shared_ptr<torch::nn::Module> torch::nn::Cloneable<Derived>::clone(const c10::optional<c10::Device>&) const [with Derived = torch::nn::CrossMapLRN2dImpl$
’:
    /tmp/tmpxft_000187da_00000000-6_scaled_upper_triang_masked_softmax_cuda.compute_80.cudafe1.stub.c:209:27:   required from here
    /path/to/conda/anaconda3/envs/py37-pt111-cu111-colai/lib/python3.7/site-packages/torch/include/torch/csrc/api/include/torch/nn/cloneable.h:57:59: erro$
: invalid static_cast from type ‘const torch::OrderedDict<std::basic_string<char>, at::Tensor>’ to type ‘torch::OrderedDict<std::basic_string<char>, at::Tensor>&’
    /path/to/conda/anaconda3/envs/py37-pt111-cu111-colai/lib/python3.7/site-packages/torch/include/torch/csrc/api/include/torch/nn/cloneable.h:69:61: erro$
: invalid static_cast from type ‘const torch::OrderedDict<std::basic_string<char>, std::shared_ptr<torch::nn::Module> >’ to type ‘torch::OrderedDict<std::basic_string<$
har>, std::shared_ptr<torch::nn::Module> >&’
    ...
    ...
    /path/to/conda/anaconda3/envs/py37-pt111-cu111-colai/lib/python3.7/site-packages/torch/include/torch/csrc/api/include/torch/optim/sgd.h:49:48:   requir
ed from here
    /path/to/conda/anaconda3/envs/py37-pt111-cu111-colai/lib/python3.7/site-packages/torch/include/torch/csrc/api/include/torch/nn/cloneable.h:57:59: error
: invalid static_cast from type ‘const torch::OrderedDict<std::basic_string<char>, at::Tensor>’ to type ‘torch::OrderedDict<std::basic_string<char>, at::Tensor>&’
    /path/to/conda/anaconda3/envs/py37-pt111-cu111-colai/lib/python3.7/site-packages/torch/include/torch/csrc/api/include/torch/nn/cloneable.h:69:61: error
: invalid static_cast from type ‘const torch::OrderedDict<std::basic_string<char>, std::shared_ptr<torch::nn::Module> >’ to type ‘torch::OrderedDict<std::basic_string<c
har>, std::shared_ptr<torch::nn::Module> >&’
    error: command '/mnt/cache/share/cuda-11.1/bin/nvcc' failed with exit status 1
    ----------------------------------------
ERROR: Command errored out with exit status 1: /path/to/conda/anaconda3/envs/py37-pt111-cu111-colai/bin/python -u -c 'import io, os, sys, setuptools, token
ize; sys.argv[0] = '"'"'/tmp/pip-req-build-2nvgz3bf/setup.py'"'"'; __file__='"'"'/tmp/pip-req-build-2nvgz3bf/setup.py'"'"';f = getattr(tokenize, '"'"'open'"'"', open)(_
_file__) if os.path.exists(__file__) else io.StringIO('"'"'from setuptools import setup; setup()'"'"');code = f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();e
xec(compile(code, __file__, '"'"'exec'"'"'))' install --record /tmp/pip-record-b988udni/install-record.txt --single-version-externally-managed --compile --install-heade
rs /path/to/conda/anaconda3/envs/py37-pt111-cu111-colai/include/python3.7m/colossalai Check the logs for full command output.

Any possible solution? @FrankLeeeee

Any possible solution? @FrankLeeeee

Hi, one possible reason is that you PyTorch is built with cuda 11.3 and your CUDA is 11.1. Can you tried this version?
pip install torch==1.10.1+cu111 torchvision==0.11.2+cu111 torchaudio==0.10.1 -f https://download.pytorch.org/whl/torch_stable.html

Any possible solution? @FrankLeeeee

Hi, one possible reason is that you PyTorch is built with cuda 11.3 and your CUDA is 11.1. Can you tried this version?
pip install torch==1.10.1+cu111 torchvision==0.11.2+cu111 torchaudio==0.10.1 -f https://download.pytorch.org/whl/torch_stable.html

Thanks for the prompt reply, I will try it.

well,I tried the abve command line and test it like:
>>> import torch
>>> torch.__version__
'1.10.0+cu111'
>>> import torchvision
>>> torchvision.__version__
'0.11.2+cu111'

and still can't compile it.Maybe you have a prebuilt package for release?

@imabackstabber Sorry makes you struggle to install our software. We will solve the cuda dependency problems ASAP. Could you please use the docker container to install and run?

@imabackstabber We are building the self-hosted pip source, we will let you known ASAP.

ok,looking forward to your reply!

Colossal-AI pre-built with CUDA extension is provided at https://www.colossalai.org/download now.


Customized ,next and ,debate command

This pull request will allow Jacob to extend review or debate by adding a time, in minutes, as an optional argument to the ,next and ,debate commands.
So, for instance, typing ,debate like he does now will do the standard 5 minute debate period. Typing ,debate 6 will grant a 6 minute debate period.

I'm pretty sure this doesn't actually work, because teh argument to handle_debate is a string, not an int.


insure the correct format parameters

insure the correct format parameters

Not sure what this was fixing, but the tests pass so if needed please reopen as an issue with data to test against


Overriding max TTL with 300s by ngx_pagespeed

Hi,
I've set ForceCaching flag, and initially all resources have cache-control:max-age=31536000 header. But after few reloads of page, more and more resources gets cache-control:max-age=300,private according to PageSpeed Insights. What could be a reason for that? I have even global config in Nginx like this:
location ~* \.(3gp|gif|jpg|jpeg|png|ico|wmv|webp|avi|asf|asx|mpg|mpeg|mp4|pls|mp3|mid|wav|swf|flv|exe|zip|tar|rar|gz|tgz|bz2|uha|7z|doc|docx|xls|xlsx|pdf|iso|js|css|html|htm|txt|eot|svg|ttf|woff|woff2)$ {
    gzip_static off;
    add_header Access-Control-Allow-Origin *;
    add_header Cache-Control "public, must-revalidate, proxy-revalidate";
    access_log off;
    expires 365d;
    break;
}

@oschaaf told me in #1251 that ForceCaching flag is for test purpose only and I'm ok with that, but isn't that strange that even with this flag more and more resources get short TTL?
Also @oschaaf siad:

usually the max-age:300 in Cache-Control is emitted by pagespeed when it still finishing up optimizations

But I see reverse situation. All non optimized resources have long TTL, and those optimized have short one: https://developers.google.com/speed/pagespeed/insights/?url=naprawimyto.pl
One more thing - Is it really necessary to apply short TTL to not-yet optimized resources? They are already downloaded, there is no reason to replace them with minified/compressed one ASAP IMO.
Thanks,
Tom.

RE: "more and more resources gets cache-control:max-age=300,private according to PageSpeed Insights"
What strikes me as odd is that Insights keeps getting a (cache-extended) jpg version served:
https://naprawimyto.pl/wp-content/uploads/2016/08/Galeria_00011-870x430.jpg.pagespeed.ce.ImXV83iJsp.jpg

While I get the webp version served with chrome (with a long TTL):
https://naprawimyto.pl/wp-content/uploads/2016/08/xGaleria_00011-870x430.jpg.pagespeed.ic.CO58ci4Obh.webp

I'm not sure why that is yet, I would have thought that PSI would receive the WebP versions as well.
RE: "One more thing - is it really necessary to apply short TTL to not-yet optimized resources? They are already downloaded, there is no reason to replace them with minified/compressed one ASAP IMO."
Someone else may know better but my 2p:

For responses that are optimized in-place, I believe only a short s-maxage header will be set to avoid shared caches from holding on to unoptimized responses too long as of https://github.com/pagespeed/mod_pagespeed/commit/689e3f1a0dd70c8585a2304c044b4408a8da0bc3
For .pagespeed. urls,  shared caches like proxies also have to be considered. But in addition fully optimized resource urls that get written into html/css will be different from partially optimized ones, and as such setting  a long TTL wouldn't make much sense either (it would be cache unfriendly).

I was experimenting with different ngx_pagespeed configs for few hours, but without success. Still I get short TTL on PSI test :(
What else could I check?

Ok, found answer in https://github.com/pagespeed/mod_pagespeed/issues/989 I've set:
pagespeed ImplicitCacheTtlMs<PHONE_NUMBER>0;
pagespeed LoadFromFileCacheTtlMs<PHONE_NUMBER>0;

and now everything work great. Thanks for all help :)

Note: the s-maxage change isn't in any released version of PageSpeed yet; it will be in the next trunk release


Contextual learning

Sorry if this question is more generic but I'm fairly new to the TTS field and I'm not sure if what I want to ask is a project-specific thing or a TTS-specific thing.
I'd like to ask how much is the context of sentences in WAV files important for training.
I'm working on my own set of WAV files using the LJSpeech structure. While testing this project's demo output by using the model generously provided by you (thank you!), I realized that the output is sometimes sounding "wrong" at the end of a sentence.
In some sentences, the voice is going up, as if there was a comma or a questionmark at the end instead of a full-stop. In other sentences this does not occur.
When I played back some of the LJSpeech sentences used for training, I found that this is exactly the same.
What I'm not sure about is whether the model learns to make the same mistake by the context of the sentence itself or it's just repeating the same thing because there are the same / similar words towards the end of the sentence?
I'm trying to understand how to best create my WAV files, so the model is trained well with regards to the emotional context of the sentence.
Example: "Oh my god! How did that happen?!", exclaimed Anna with a tone of irritation in her voice.
If I use this sentence as a whole, would the TTS learn to use an irritated surprise emotion where the context of irritation is present? Or does this not matter and the model would only learn the irritated tone from the sound in that quote, regardless of the context following it?
Thanks for reading and sorry again for a super-long question!

Yes, this is one major problem of the current TTS model which treats each sentence independently. In reality, the audio clips are always dependent on its context (previous or next sentence), so you may also want to use that information in training. You may want to add the style of the previous sentence along with the style of the current sentence to synthesize the current sentence. This is one way to achieve context-aware TTS.
As for your example, if you feed the sentence as a whole, of course it can learn the emotion this way. One problem of LJSpeech dataset is that it cuts the audio arbitrarily by silence, which is why you see some seemingly unnatural intonation and also why our model is even better than ground truth recording.

Hello, I have been thinking of your suggestion for achieving context-aware TTS. To employ this method, would it be done during the training? or during inference? I am pretty new to this, but this has piqued my interest. Thank you.


Computed view not updating

I'm trying to marry s-js with lit-html, but I can't get the view to update. I've reduced the issue down to the smallest example I could.
This should output 123, but outputs 12.
import { html, render } from 'lit-html';
import S from 's-js';

const nums = S.data([1, 2]);
const view = S(() => html`<div>${nums()}</div>`);

const r = S.root(() => {
  render(view(), document.getElementById('app'));
});

nums(nums().push(3));

The render function is only getting called once. As far as I understand it, that function should also be called as a result of nums(nums().push(3)).
Can you enlighten me as to what I'm missing?
Edit: I also tried this with the same output result:
const nums = S.data([1, 2]);

const view = S.root(() => html`<div>${nums()}</div>`);

render(view, document.getElementById('app'));

nums(nums().push(3));

The call to render isn't 'live' because it's not wrapped in a computation.  So you want something like:
S.root(() => S(() =>
  render(view(), document.getElementById('app'))
));

S.root() controls computations' lifecycles but isn't a computation itself: it only runs once.
I'd be curious to hear how these explorations with lit-html go.  I've done a couple thought experiments about how the two could be combined.

Great, that works now. Thanks.
I'm just at the start of the project, but will update you as it gets more mature.


Process Topology Task context as an object instead of string

Based on Topology changes from https://github.com/ManageIQ/topological_inventory-ansible_tower/pull/20
Task object from Topology use to return the context property as a string.
Before:
{
  context: "{"service_instance":{"source_id":"352","source_ref":197,"id":"2058","url":"http://<IP_ADDRESS>:8080/api/topological-inventory/v1.0/service_instances/2058"},"remote_status":"successful"}",
  created_at: "2019-08-06T21:20:45Z",
  id: "119",
  state: "completed",
  status: "ok",
  updated_at: "2019-08-06T21:22:50Z"
}

After:
{
  context: {
    remote_status: "successful",
    service_instance: {
      id: "2061",
      url: "http://<IP_ADDRESS>:8080/api/topological-inventory/v1.0/service_instances/2061",
      source_id: "352",
      source_ref: 199
    }
  },
  created_at: "2019-08-09T16:34:07Z",
  id: "120",
  state: "completed",
  status: "ok",
  updated_at: "2019-08-09T16:34:43Z"
}

Any thoughts @eclarizio?

@gmcculloug DO we have a Jira ticket to track this bug?