Edit Analyzer Ranges Data

Definition of Ready

[x] Does this meet the INVEST criteria?
[x] Does this have a clearly defined user?
[x] Does this have an estimate?
[x] Does this have acceptance criteria?
[x] Does this have sufficient context (wireframes, business rules, etc.)?
[x] Is this traceable to a feature, epic, or PI?

As a user, I want the ability to edit the Analyzer Range data so that the data available is current
Acceptance Criteria:
Given the user is on the Component editor modal
When the user selects View/Edit for a Range
Then the system displays the Edit Analyzer Range modal
and the user can edit data
Range: drop down, required
Dual Range Indicator: Yes/No Radio Button
Start Date: date picker, required, format mm/dd/yyyy
Start Time: Text, required, format hh
End Date: date picker, format mm/dd/yyyy
End Time: Text, format hh
Refer to page 47 for values
Given the user has edited Analyzer Range data
When the user chooses to save and go back
Then the updated data is saved
And the user is navigated back to the Component editor modal
Given the user has edited Analyzer Range data
When the user chooses to Cancel the updates
Then the updates are not saved
And the user is navigated to the Component editor modal
Does this Story require 508 Testing?

[ ] Implementing a new authoring tool that will produce code / content for the application
[ ] Implementing a 3rd party widget that will be part of the application
[ ] Adding new / updating custom functionality
[ ] Adding new / updating a page template
[x] Adding new / replacing existing functionality (unless the replacement has already passed 508 testing)
[ ] Full Site Review prior to a scheduled EPA 508 review
[ ] Planning a video that will appear on the application
[ ] Producing a video that will appear on the application
[ ] Producing Training materials
[ ] Producing documents

Context
Design Wiki
Testing
Test Case:
Tester:
Definition of Done

[ ] Have all Acceptance Criteria been agreed to and validated by the scrum team?
[ ] Has the code been unit tested and peer reviewed?
[ ] Have the functional tests been executed?
[ ] Have all defects been reviewed and dispositioned, resolved, or deferred?
[ ] Has the user story been accepted by the Product Owner?
[ ] Is the user story ready to be deployed to the test/staging environments?

API work done in 5.2. UI still outstanding.


Add basic logging mechanism

Configure a logger to be used in the whole application.

Based on the table [here](https://docs.python.org/3/howto/logging.html#when-to-use-logging, I wonder if we should keep all the prints for stdout and stderr messages instead of replacing them.

It will probably depend on the action to run.  Actions like 'up' and 'destroy' should probably log almost everything. They will be used in CIs so the log should be available. I think that we can log more info (e.g. info from providers) to DEBUG level than we now print to console.
For human 'console' actions like 'list' and 'ssh' we probably will mostly log to INFO and print.

It will probably depend on the action to run. Actions like 'up' and 'destroy' should probably log almost everything. They will be used in CIs so the log should be available. I think that we can log more info (e.g. info from providers) to DEBUG level than we now print to console.
For human 'console' actions like 'list' and 'ssh' we probably will mostly log to INFO and print.

Agreed! Let me enhance this.

I see that there is used pattern:
        logger.info(f"Hosts to delete: {names}")
        print(f"Hosts to delete: {names}")

Do we want this code duplicity with strings?
In the previous comment I thought that we will:

log INFO level into console and DEBUG level into file implicitly.
log DEBUG into console when --debug is passed
have separate print() statements only for text which we don't want to log into file.

I'm also open to not have the console logger but in such case we might want to have some utility function like
def print_log(logger, msg, level="INFO"):
    file = sys.stdout
    if level in ("ERROR", "CRITICAL"):
       file = syst.stderr
    print(msg, file=file)
    logger.log(level, msg)

@pvoborni it was hard to feel comfortable with the idea of logging+printing a CLI application. Specially because the common practice is to log to stdout. But I think your points now made it clear to me.
About the duplication, I wonder if most of the strings would be the same or not. However, I guess all of this is fixed by having a custom print function. It looks strange, but I think is the best solution.

Earlier today @pvoborni and I talked about this and we agreed to the following points:

Remove all print and replace them with logger.info
One handler would output the messages to the console without timestamps
Another handler will display more info, such as the timestamps

@pvoborni  Let us merge this as well. Thanks @netoarmando


Jester requires nim >= 14.0 / PonyAPI didn't compile

Jester now requires nim >= 14.0 since June 11.
I've found a recent docker image running nim 14.0 (mdoboi/nim) but then, PonyAPI didn't compile.
Installing ponyapi-0.1.0
Building ponyapi/ponyapi using c backend...
Hint: used config file '/opt/Nim/config/nim.cfg' [Conf]
Hint: system [Processing]
Hint: ponyapi [Processing]
Hint: asyncdispatch [Processing]
Hint: os [Processing]
Hint: strutils [Processing]
Hint: parseutils [Processing]
Hint: math [Processing]
Hint: times [Processing]
Hint: posix [Processing]
Hint: oids [Processing]
Hint: endians [Processing]
Hint: tables [Processing]
Hint: hashes [Processing]
Hint: etcpriv [Processing]
Hint: macros [Processing]
Hint: heapqueue [Processing]
Hint: nativesockets [Processing]
Hint: net [Processing]
Hint: sets [Processing]
Hint: queues [Processing]
Hint: selectors [Processing]
Hint: epoll [Processing]
Hint: episode [Processing]
Hint: json [Processing]
Hint: lexbase [Processing]
Hint: streams [Processing]
Hint: unicode [Processing]
Hint: future [Processing]
Hint: jester [Processing]
Hint: asynchttpserver [Processing]
Hint: asyncnet [Processing]
Hint: uri [Processing]
Hint: httpcore [Processing]
Hint: strtabs [Processing]
Hint: re [Processing]
Hint: pcre [Processing]
Hint: rtarrays [Processing]
Hint: scgi [Processing]
Hint: sockets [Processing]
Hint: asyncio [Processing]
Hint: cookies [Processing]
Hint: mimetypes [Processing]
Hint: md5 [Processing]
Hint: logging [Processing]
Hint: patterns [Processing]
Hint: errorpages [Processing]
Hint: htmlgen [Processing]
Hint: utils [Processing]
Hint: cgi [Processing]
Hint: random [Processing]
Hint: stats [Processing]
lib/pure/asyncdispatch.nim(1521, 7) Hint: Processing match as an async proc. [User]
ponyapi.nim(76, 1) template/generic instantiation from here
lib/core/macros.nim(333, 70) template/generic instantiation from here
ponyapi.nim(82, 15) template/generic instantiation from here
ponyapi.nim(59, 21) Error: type mismatch: got (HttpHeaders, string)
but expected one of:
proc getOrDefault(t: StringTableRef; key: string): string
proc getOrDefault[A, B](t: Table[A, B]; key: A): B
proc getOrDefault[A, B](t: OrderedTable[A, B]; key: A): B
proc getOrDefault[A, B](t: TableRef[A, B]; key: A): B
proc getOrDefault[A](t: CountTableRef[A]; key: A): int
proc getOrDefault[A](t: CountTable[A]; key: A): int
proc getOrDefault[A, B](t: OrderedTableRef[A, B]; key: A): B
proc getOrDefault(node: JsonNode; key: string): JsonNode

Ah, that would be the change in how HTTP headers work now. Should be easy to fix, just need to dig into the type changes.

@mkody did the latest commit on HEAD fix this for you?

Didn't build.
root@BerryPunch:~/PonyAPI# docker build -t xena/ponyapi /root/PonyAPI
Sending build context to Docker daemon 449.5 kB
Step 1 : FROM coopernurse/docker-nim
 ---> 123c3ee53037
Step 2 : RUN apk update && apk add bash
 ---> Using cache
 ---> 6dd07b0216f7
Step 3 : EXPOSE 5000
 ---> Using cache
 ---> 9d01787ad03f
Step 4 : RUN adduser -D -g '' r
 ---> Using cache
 ---> 46fe4882d387
Step 5 : RUN chmod a+x /opt/Nim/bin/nim
 ---> Using cache
 ---> 65ef63fef748
Step 6 : ADD . /app
 ---> cbfe1fdec8fe
Removing intermediate container 837ac8a834ef
Step 7 : WORKDIR /app
 ---> Running in 7cea5d544073
 ---> 07c669a3428f
Removing intermediate container 7cea5d544073
Step 8 : RUN nimble update &&    yes | nimble install &&    nim c -d:release --deadCodeElim:on ponyapi
 ---> Running in 375c85b5d112
Downloading package list from https://github.com/nim-lang/packages/raw/master/packages.json
Done.
Traceback (most recent call last)
nimble.nim(990)          nimble
nimble.nim(959)          doAction
nimble.nim(685)          install
nimble.nim(539)          installFromDir
packageinfo.nim(259)     getPkgInfo
packageinfo.nim(158)     readPackageInfo
Error: unhandled exception: Invalid section:  [NimbleError]
The command '/bin/sh -c nimble update &&    yes | nimble install &&    nim c -d:release --deadCodeElim:on ponyapi' returned a non-zero code: 1

@mkody lol, i apparently forgot to commit my custom image change to the Dockerfile

(I don't have it as an autobuild image on the docker hub right now sorry)

Wow, am I missing something or I'm getting nothing?

wat. curl -v?

And, huh. Direct access from IP only gives application/json. (curl -v gives the good input)
http://<IP_ADDRESS>:5000/season/6/episode/11
(From Caddy: https://ponyapi.ponyfrance.net/season/6/episode/11 )

Oh boy
!()[https://i.imgur.com/MLM8C3a.png]

So it's not me!
I'm closing this, since there's a dedicated commit for it #14


Updates for release

Update setup.py details
Changes in dependencies:

Removes numpyro/jax dependencies which are incompatible with Windows
Adds explicit pandas and xarray dependencies which are used throughout the codebase

It would be nice to be able to use the jax samplers when available. I'll open an issue.


Trails without informal tag show informal tag in mouseover

When you mouse over a trail which does not have an informal tag set it shows informal=false in the tag list. This is not a tag which is present on the object on OSM and might confuse new mappers, since the intended value is informal=no
Example on OpenTrailMap

Way on OSM:
https://www.openstreetmap.org/way/268153916

Welcome, @elevenchars. This is an issue with planetiler treating this tag as a boolean when really we want all tags imported as strings. It will be fixed next time the tiles are updated (hopefully soon!) Thanks for reporting.

This was fixed by doing #4.


Auth should only allow to demote people in lower ranks, not the same

Eg. an admin should not be able to /deladmin another founder.
Depends on the "owner class in Auth" issue.

Crisbal says it shouldn't be the case, admins should be able to promote new admins.
Closing this for now, may discuss this later if this feature is requested.


[FIX] Fix vanilla framework bug

Fix vanilla framework installation bug

Hi @sixbag , would you please tell me what bug do you think you find out ? $1 aim to Vanilla install Root, $2 aim to OpenResty install Root.


Wrong definition of DoneCallback

The README specifically mentions we should pass err to done callback. However, this repository declares DoneCallback as parameterless function. This causes error when compiling my TypeScript project.
This is how done should be called:

This is the definition of DoneCallback (line 130):

Check this specific line: https://github.com/types/npm-pg/blob/c8826dc3f6afdbe02d7e9622a664d1be9095cf8e/index.d.ts#L9

On a separate note, another error (visible at line 123 above) is:
index.d.ts(123,10): error TS2305: Module '"tls"' has no exported member 'Co
nnectOptions

Which node definition are you using? It's based on https://github.com/types/env-node. I just updated the DoneCallback using https://github.com/types/npm-pg/commit/ba5ea76de73f2cd352e3b69f149cd97dfba1d4c9.


[BLCKCHN-187]  Add Base Url for local testing

Fix for getTransactionsByAddress, getTransactionsByHash (api prefix)
Add support for local server URL during development
Set production server as the default environment
Remove redundant environment checks

rebased

removed local url


Preserve line breaks on compile

Hello. Absolute noob here... new to gulp and JS...
Question, is there a way to preserve line breaks on compilation with an option or some sort of argument?

Node sass has 4 output format styles that's are non-configuable: nested, expanded, compact, compressed


apns: Does not support 'content-available'?

It seems that gaurun dose not support 'content-available' property on apns. Is there any reasons to do so? (or I misundestanding?)
I wrote a small patch to use 'content-available' on gaurun and I would like to send a PR.

It seems that gaurun dose not support 'content-available' property on apns. Is there any reasons to do so? (or I misundestanding?)

There is not any reasons in particular.

I wrote a small patch to use 'content-available' on gaurun and I would like to send a PR.

Nice work. Patches welcome.

Thanks for your reponse. I'm going to create a PR!


Projects API improvements & dnf mocking

This PR introduces support for mocking dnf-json, adds error reporting to dnf-json and implements one missing route for projects.
I've renamed rpmmd to dnfadapter. I don't like the name rpmmd - it isn't very descriptive and it was hard to work with it when I introduced the mocking part. I'm not sure if dnfadapter is the right name. Maybe dnfwrapper?

I've pushed a new version of the PR with all discussion points solved:

no more renaming rpmmd
mocking of rpmmd is now done on Go level

Don't merge this. I've figured out there's an error in dnf-json with depsolve MarkingError, will fix it after lunch and push a new version.

@teg @larskarlitski I fixed the error and repushed, can you approve this PR?


Style Utilities : BG

what is BG? Use proper heading.
"so in some cases you’ll want to use CoreClasses.COLOR.TEXT_"
suggestion - "so in some cases you’ll have to use CoreClasses.COLOR.TEXT_".
how can I know you used appropriate colors? Provide The document you use for colors.

Test Fail

Test fail

share me wrappid theme

make font color white

fix this

The issues related to the background color are resolved as of now. kindly check the changes in the next release!

As of now closing this issue


VSCode - Host disconnected network issue

<!--
For Visual Studio problems/feedback, please use the "Report a Problem..." feature built into the tool. See https://aka.ms/vsls-vsproblem.

**Steps to Reproduce / Scenario:**
1. Host disconnects from Live Share
2. My session as guest does not see that host has left for a while
3. I get a notification saying host has disconnected and rejoin
4. Clicking that leads to the error seen in the screenshot below

**Screenshots**
![image](https://user-images.githubusercontent.com/51928518/64385936-a0e36780-cfec-11e9-9a30-160df7ac74da.png)

[LiveShareLogs.zip](https://github.com/MicrosoftDocs/live-share/files/3581361/LiveShareLogs.zip)

LiveShareLogs.zip


Main Menu Panorama Stretched

The background panorama in the main menu is stretched horizontally. The texture file is not stretched, but the image becomes stretched when used in-game.

Fixed in 1.3


datapath: Fix race in the fake NodeHandler

The assumption that the NodeHandler callbacks are called sequentially
was unfounded. Add a mutex to prevent concurrent access.
Fixes: f0d9e65b35 ("test/control-plane: Add test for node handler")
Fixes: #20651

This fixes a failing unit test (https://app.travis-ci.com/github/cilium/cilium/jobs/577440300#L994), so relevant CI check that needs to pass is the "Travis CI" one.


Implement map.min/max keys in C++

Implements min/max pair validations in C++ and adds the map test cases to the test harness.
Progress on #130.

Awesome, thanks!


PHP version compatibility issue

What steps will reproduce the problem?
On php yii serve PHP Fatal error:  Cannot use 'Object' as class name as it is reserved, I already searched on similar issue but unable to solve it. I choose different php version but problem remain constants. Im new to Yii. can you guys guide how to resolve this issue
| ---------------- | ---
| Yii version      | 2.0.?
| PHP version      | < 7.0
| Operating system | Windows

use yii\base\BaseObject instaed of yii\base\Object

where to add this?

Likely you're using outdated Yii version.

How to update it im new in it.
composer.json have already "yiisoft/yii2": "*",

composer update
check yii version with command: composer info | grep yiisoft/yii2, or in developer panel left side
minimum version is 2.0.13, if this is not true then any package you are using requires an older version of yii.


This is actually available in Azure SQL Database.

Shows it isn't, though..

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

ID: 555df55e-bb95-0dc8-b7a6-c0c776903bef
Version Independent ID: 7dd977fc-bd60-b0e2-f7ff-c3a3aa8f340b
Content: sys.dm_xtp_gc_queue_stats (Transact-SQL) - SQL Server
Content Source: docs/relational-databases/system-dynamic-management-views/sys-dm-xtp-gc-queue-stats-transact-sql.md
Product: sql
Technology: system-objects
GitHub Login: @stevestein
Microsoft Alias: sstein

@LuisGonzalezLT Thank you for your feedback.  We have fixed the documentation, and it should be live tomorrow, so I am closing this Issue 2345 now.


Allow different tests to be run by concourse

Motivation and Context
30 million takes ages. 3.5 million takes 3 hours. 350k takes 40 minutes. PubSub takes a couple of minutes. We want a way to run the test we want, not have to run ALL of them.
What has changed
Made scenarios configurable for different poipoises.
How to test?
Fly. Fly. Fly.
Links
Trello: https://trello.com/c/zWLwrX72

 SPIKE: Multiple trigger jobs for performance pipeline for different types of test (2 days)

 Allow performance test choice before triggering


errors thrown by openerService.resolveExternalUri not surfaced to MainThreadWindow.$asExternalUri in VS Code Web

VS Code Version: 1.72.1 (main)
OS Version: N/A
Browser: Chrome

Steps to Reproduce:

git clone https://github.com/microsoft/vscode.git
run VS Code Web locally
install any extension that uses asExternalUri (i.e. this one)
run command "Hello World"
Open Browser Console

Expected
Should log error thrown 'Could not resolve external URI: ' + resource.toString()
Actual
Nothing
Notes
I believe this happens because there is no try/catch block here:
https://github.com/microsoft/vscode/blob/main/src/vs/workbench/api/browser/mainThreadWindow.ts#L63
When you add a try/catch block there like so:
		try {
			const result = await this.openerService.resolveExternalUri(URI.revive(uriComponents), options);
			return result.resolved;
		} catch (e) {
			console.error('something wrong', e);
		}
		const result = await this.openerService.resolveExternalUri(URI.revive(uriComponents), options);
		return result.resolved;

Then it logs the thrown error
Screenshot

Would you accept a PR to fix this?

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.71.2. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

Oops, I put the wrong version. This is off main so it's the latest. Updated issue description.

I don't think logging there is correct. Instead extensions should get an exception if asExternalUri throws
Looks like we sort of already do this but not for http uris: https://github.com/microsoft/vscode/blob/fa758a7cc52429139fb2a5fe5ee91c6182414d81/src/vs/workbench/api/common/extHost.api.impl.ts#L361
@joaomoreno The history oI this code is kind of messy but I believe these checks were first added in a6f7aa5e4c5832c3d788ab9a8146ac69a6e5a77f
Do you remember why exceptions are ignored for http uris?

Sorry for the long delay replying.
I don't really remember, other than this being very specific to how URL callbacks are used in vscode.dev/github.dev. Check this one out: https://github.com/microsoft/vscode/commit/f99e1c358f0fa9e9f5bd38022e7b8cac04553711


Support custom struct field names with new scalar function named_struct

Which issue does this PR close?
Closes #5861
What changes are included in this PR?

Add a new scalar function named_struct which accepts pairs of name and values, made possible with #8985
Transform sqlparser::Expr::Struct into a named_struct call, instead of the existing struct, and if any expression is a sqlparser::Expr::Named, use its name as the struct field name, otherwise fallback to the currently used cN convention.

Are these changes tested?
Yes, the new scalar function has a unit test similar to the existing struct function, and a sql logic test has been added too.
Are there any user-facing changes?
New scalar function.
Existing struct function is not changed.
Alternatives considered

Separate new Expr::Struct expression
Return the existing struct call wrapped in a Expr::Cast to a struct with custom field names, but this would only benefit SQL users.

It looks nice to me overall, and I also added some comment which may help you improve the PR :) @gstvg

Thank you guys for all the comments and the review.
Prior to applying the requested changes, I wanna say that I think I didn't elaborated enough what this PR does and most importantly, why it does that way, and now I'm afraid maybe this is not the best way.
How @alamb noted and @yyy1000 agreed, ideally we would simply extend the current struct function to support named arguments, but because of the scalar function signature fn invoke(&self, args: &[ColumnarValue]) -> Result<ColumnarValue>, I didn't figured out any way to associate a name or any kind of metadata with the args. Self in this case is the "global" function, not unique to any specific invocation. Changing it's signature it's a big change, and I personally like it's simplicity. We can add metadata to the ColumnarValue, but again, feels like a big change.
We could modify the existing struct func to accept pairs of name and value, but it would be breaking change, e.g: given an existing call struct("text", 1) should it create a struct with 2 fields named "c0" and "c1", or a single field named "text"?
And most importantly, It would not support the much nicer syntax struct("valueOrName?", "value" as another_name) because it would be ambiguos, is the first arg a value, or the name of the second arg? If it's a name, and it's corresponding value pair is a named arg, what name takes precedende?
So, we can create a new separate function, that accept pairs of name and value, but not named arguments, and it would give us a minimun level of support without a nice syntax, and also naming it named_struct it becomes compatible with Spark but that's not the priority now, of course.
Then, there's the improvement of supporting the struct(value as name) syntax.
Currently, this PR, while parsing sqlparser::Expr::Struct, inspect the args Expr and if it is Expr::Named, use its name, otherwise the cN convetion, then call named_struct, not struct.
But doing this, what exaclty  struct(...) becomes ? A function? Expression? "Meta-function"? How should the docs call it? Users inspecting projections and plans would only see named_struct calls, it is acceptable? It is indeed a bit confusing.
Also, if Expr-based code calls struct with datafusion::Expr::Alias, this PR can't identify it and return field names with the cN convetion, which is probably unexpected. Should we delete the struct function, so Expr code can't use it?
Prior opening this PR, I though this:

Return the existing struct call wrapped in a Expr::Cast to a struct with custom field names, but this would only benefit SQL users.

I was wrapping the call with a cast while parsing sqlparser::Expr.
But if instead we parse sqlparser::Expr::Named to datafusion::Expr::Alias, then add FunctionRewrite that wraps struct calls with cast, using the name of Expr::Alias, if any, or the cN convetion, we support both SQL and Expr users via the same code in a more expected way, i think.
The problem with it is that, per current FunctionRewrite docs
Trait for rewriting [`Expr`]s into function calls.

This trait is used with `FunctionRegistry::register_function_rewrite` to
to evaluating `Expr`s using functions that may not be built in to DataFusion

For example, concatenating arrays `a || b` is represented as
`Operator::ArrowAt`, but can be implemented by calling a function
`array_concat` from the `functions-array` crate.

we are using it in the opposite way it was designed for. It is okay to use it that way, updating the docs, or it's misuse?
It also create very long projection plan strings, which may be undesired.
So, given the options:

Add named_struct function
1.1. Forward struct calls to named_struct
Rewrite struct into a Expr::Cast(struct_fn_expr, struct_datatype_with_named_fields)

We can do:

Only 1
Only 2
1 & 1.1 (This PR)
1 & 2(for spark compatibility only, in a different PR)

What do you think?

Thanks for your explain, @gstvg !
Yeah, in my previous thought, struct udf will not be called after this PR, so if we just remove struct udf, would it also work? ( I think so but not sure)

@gstvg Is it possible to have a syntax like {"a": 1, "b": 2} for named struct?

Thanks for your explain, @gstvg ! Yeah, in my previous thought, struct udf will not be called after this PR, so if we just remove struct udf, would it also work? ( I think so but not sure)

Yes, you're right, named_struct doesn't require the struct udf, but some expr-based code somewhere may depend on it, it is enough to justify it's existance? I'm don't know much about the retrocompatibility guidelines of datafusion.

@gstvg Is it possible to have a syntax like {"a": 1, "b": 2} for a named struct?

It would require a PR to sqlparser-rs
Someone already created an issue

struct can remain for the cN convention when you don't care about the field name, so we don't have the ambiguous
signature issue about struct() or named_struct()
Do not allow partial named struct like struct("valueOrName?", "value" as another_name), I think it is confusing and also > not a helpful syntax.

Fair enough

After the named struct {"a": 1, "b": 2} is completed, we can easily rewrite struct to {"c0": 1, "c1": 2}, at the end we only need one struct function.

Yes, after minimal support is merged, and support for the syntax is added on sqlparser-rs, it would be trivial to support it, leaving us with a very nice support for structs!

Thank you @gstvg  -- I think your solution 1.1 sounds like the best plan to me (and basically what this PR does). I think this PR is very important and will unlock several other great usecases for DataFusion so thank you again for working on it

Yes, you're right, named_struct doesn't require the struct udf, but some expr-based code somewhere may depend on it, it is enough to justify it's existance? I'm don't know much about the retrocompatibility guidelines of datafusion.

I agree that it would be nice to remove the old struct udf -- let's do that as a follow on PR (where we can discuss the merits / requirements for backwards compatibility)
It looks like @jayzhan211  and you are already busy working on the sqlparser support for supporting the duckdb style literal syntax 🙏
Thus here is what I think we should do:

Polish up this PR (add the tests described in https://github.com/apache/arrow-datafusion/pull/9743#discussion_r1536799585)
Work on removing the existing struct udf impl as a follow on PR
Work on the named struct syntax support as a follow on PR (

I think you can get the CI passing by merging this PR up from main. We are getting very close

Thanks again @gstvg  🙏

And thanks for the reviews and suggestions @yyy1000 and @jayzhan211

I think this one may need some change in SqlParser, currently in Sqlparser we just parse syntax like "struct(xx, xx, xx, xx)" to be parsed to Expr::Struct, but when encounter some structure like "t2.struct(xx, xx, xx, xx)" the Sqlparser would parse them into two ident. I think right now it is not ready yet.


Fix collision detection

The issue was caused by:

The sim maintains the wrong keys of the agent collisions.
Pass a wrong vehicle ID to query the collision.

close #221

Maybe in another PR, is it possible to update the collision test to catch this issue?

Maybe in another PR, is it possible to update the collision test to catch this issue?

Yes, should have a goalkeeper to ensure that.

Nice test! I see that the updated test can catch the mistake in smarts.py, but doesn't catch the mistake in sensors.py. Perhaps it is more robust to check both events in observations and done?

@Gamenot We need to release a new version after this one goes in since the smarts service is using SMARTS as a library.


build(perception_online_evaluator): add lanelet_extension dependency

Description

This PR adds a dependency to lanelet2_extension to fix a missing include.
Tests performed

Not applicable.
Effects on system behavior

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

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

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

[ ] The PR follows the pull request guidelines.

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

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

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

@kosuke55 no problem, thanks for the quick review :slightly_smiling_face:


Immediate consistency or how to know if Register customer was successful

At the moment, CustomerController in Equinox.Services.Api will always return status 200 to any request to register a new customer since the consistency is not implemented and the error feedback loop from a Register method in CustomerAppService. Basically it will send a command to register a new customer to an in-memory bus, bus will write a generic (not linked to a particular transaction) message to a "notifications" store and that's it, the message will simply stay in that store. Yes, the Handle task will return false instead of a true, but since the repo is intended to teach users a clean architecture, may I ask - is this how it should be? What is the "by the books way" of validating if the "operation is successful" and feedback it to a calling API? Thanks.

This project uses imediate consistency and now is returning 204 from API


In the Apps page, the pods should be left aligned - today they are mid / right mixed

See screenshot.
I think the pods should be left aligned, then when you have 1, 2 or 3+ then its all showing nicely.

This used to be an issue but I thought it was fixed, have you cleared your cache recently?  Am not seeing it here...

I don't see it in hawtio-kubernetes; but do in fabric8-console. Wonder if its some CSS?

correctly - I get the same bug in Chrome on my mac with hawtio-kubernetes running locally too

correction - after flushing caches - I don't see this issue in either hawtio-kubernetes or fabric8-console locally - or in the docker container!
I did a docker pull fabric8/fabric8-console too and deleted the fabric8-console pod for good measure and its all working fine here

Yep just did the same as james, and there was a new version of the fabric8 console. And deleted all history on safari browser which I normally do not use. And now all of them is aligned nicely to the right.


Error: Ad is already loaded. (Warning)

I am trying to implement Admob Rewarded, and I have code similar to the one from Example folder. Everything works great but I receive warning in console that says Error: Ad is already loaded.  and another warning on the phone {"framesToTop":1",code":"E_AD_ALREADY_LOADED"}.
I receive it right after the page loads (before even clicking a button with an ad). Am I doing something wrong or should I just ignore the warning since everything is working great?
I am using 2nd version, and here the code I have:
    componentDidMount() {
    AdMobRewarded.setAdUnitID('XXX');

    AdMobRewarded.addEventListener('rewarded',
      (reward) => console.log('AdMobRewarded => rewarded', reward)
    );
    AdMobRewarded.addEventListener('adLoaded',
      () => console.log('AdMobRewarded => adLoaded')
    );
    AdMobRewarded.addEventListener('adFailedToLoad',
      (error) => console.warn(error)
    );
    AdMobRewarded.addEventListener('adOpened',
      () => console.log('AdMobRewarded => adOpened')
    );
    AdMobRewarded.addEventListener('videoStarted',
      () => console.log('AdMobRewarded => videoStarted')
    );
    AdMobRewarded.addEventListener('adClosed',
      () => {
        console.log('AdMobRewarded => adClosed');
        AdMobRewarded.requestAd().catch(error => console.warn(error));
      }
    );
    AdMobRewarded.addEventListener('adLeftApplication',
      () => console.log('AdMobRewarded => adLeftApplication')
    );

    AdMobRewarded.requestAd().catch(error => console.warn(error));
    }

    componentWillUnmount() {
    AdMobRewarded.removeAllListeners();
  }

  showRewarded() {
    AdMobRewarded.showAd().catch(error => console.warn(error));
  }

....

<Button
              onPress={() => this.showRewarded()
               }
            />

try this way:
AdMobRewarded.requestAd()
.then(() => AdMobRewarded.showAd())
.catch(error => console.warn(error))

same here. any solutions so far?

AdMobRewarded.requestAd().then(() => AdMobRewarded.showAd())
This will work! 👍🏻

Its only shows the Ad 1 time if its skipped/closed via "adClosed" event.
the error is
Error: Ad is already loaded.

Go to AdMobRewarded class in library folder in android folder and remove the following lines with red doodles.

await AdMobInterstitial.setAdUnitID(adUnitID.android.interstitial); // Test ID, Replace with your-admob-unit-id
await AdMobInterstitial.requestAdAsync({ servePersonalizedAds: true });
await AdMobInterstitial.showAdAsync();
// ad reloaded
setTimeout(() => {
AdMobInterstitial.dismissAdAsync();
}, 1000);

AdMobInterstitial.setAdUnitID(adInterstitialUnitID);
AdMobInterstitial.requestAdAsync({ servePersonalizedAds: true })
.then(async () => {
await AdMobInterstitial.showAdAsync();
setTimeout(() => {
//Here Write your code after user seen ad
}, 3000);
setLoading(false);
})
.catch((error) => console.log(error))
.finally(() => {
setTimeout(() => {
AdMobInterstitial.dismissAdAsync();
}, 1000);
});


Add support for bool in elementwise ops

Description
Adds support for bool tensors in element wise ops
Type of change
Please delete options that are not relevant and/or add your own.

New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the style guidelines of this project (You can use the linters)
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas and hacks
[x] I have made corresponding changes to the documentation
[ ] I have added tests to verify my fix or my feature
[x] New and existing unit tests pass locally with my changes

We support aten::to operator now which should handle this. Check out https://github.com/NVIDIA/TRTorch/blob/master/tests/core/conversion/converters/test_cast.cpp for usage.


Fix BelongsToMany relationship bugs when using custom keys

Hi there,
This PR fixes bugs in BlongsToMany relationship when custom keys defined.
I tested custom parentKey and relatedKey in #2667, but that was not enough. In this PR foreignPivotKey and relatedPivotKey keys have been tested and previous tests are improved.

Thank you @hans-thomas for all your work on relationships.

@GromNaN You're welcome buddy. Thank you for sharing your knowledge with me❤ I learned a lot.


docs: update documentation for deprecated components

Description
Updates documentation for deprecated components to give more accurate replacement options.
Deploy Preview
Include a direct link to your changes in this PR's deploy preview here (e.g., a specific component page).
Merge Checklist

[ ] If your update includes visual changes, have they been reviewed by a designer? Send them a link to the Netlify deploy preview, if applicable.
[ ] Does your change adhere to the documented style conventions?
[ ] Do any prop types have missing descriptions in the Props API tables in the documentation site (check deploy preview)?
[ ] Were your changes tested using all available themes (see theme switcher in the header of the deploy preview, under the "Settings" icon)?
[ ] Were your changes tested in the example app?
[ ] Is there adequate test coverage for your changes?
[ ] Consider whether this change needs to reviewed/QA'ed for accessibility (a11y). If so, please add wittjeff and adamstankiewicz as reviewers on this PR.

Post-merge Checklist

[ ] Verify your changes were released to NPM at the expected version.
[ ] If you'd like, share your contribution in #show-and-tell.
[ ] 🎉 🙌 Celebrate! Thanks for your contribution.

Hi @viktorrusakov! Please let me know if you have any questions regarding submitting your CLA form. If you are contributing as part of another organization, you'll need to be added to their entity CLA.

Hi @viktorrusakov! Please let me know if you have any questions regarding submitting your CLA form. If you are contributing as part of another organization, you'll need to be added to their entity CLA.

Hi @mphilbrick211, Viktor was previously under Raccoon Gang's CLA but he is no longer with Raccoon Gang. That said, he is still a Core Contributor and we believe he has signed an individual CLA form already. Perhaps he is falling into this camp now:

If you've signed an agreement in the past, you may need to re-sign.

@viktorrusakov When you get a chance, do you mind filling out the individual CLA form again?

Thanks, @adamstankiewicz, I appreciate the clarification, and apologies @viktorrusakov for not remembering you're a CC!
If you don't mind re-submitting the form, that would be great! If there's any issue, I can look into it for you.

Hi @adamstankiewicz, @mphilbrick211! Thanks for the clarification, I totally forgot that there was something wrong with my CLA check. I've resubmitted the form just now, let me now if anything else is required from my side.

Hi @adamstankiewicz, @mphilbrick211! Thanks for the clarification, I totally forgot that there was something wrong with my CLA check. I've resubmitted the form just now, let me now if anything else is required from my side.

Hi @viktorrusakov! Did you get confirmation of your CLA going through? If not, I can check on it for you.

Hi @mphilbrick211, no, I don't think so, I didn't receive any feedback after submitting the form and CLA check still fails after re-running it.
I've resubmitted the form just now just in case I made a typo somewhere last week.

Hi @mphilbrick211, no, I don't think so, I didn't receive any feedback after submitting the form and CLA check still fails after re-running it.
I've resubmitted the form just now just in case I made a typo somewhere last week.

Great, thanks, @viktorrusakov! I will check on Monday to make sure it's been processed.

Hi @viktorrusakov! I checked on your CLA status, and it has been received. Tomorrow, you should be able to rebase and your CLA check should turn green.

Thank you @mphilbrick211, it works now!

@viktorrusakov 🎉 Your pull request was merged!  Please take a moment  to answer a two question survey so we can improve your experience in the future.

:tada: This PR is included in version 21.1.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 22.0.0-alpha.6 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Log protocol runners stdout/stderr output

Very hacky implementation, not meant to be merged in its current state, but will be used for debugging.
What would be a better way to handle this? I saw that there is a tokio::process module that could be used to work with this in a more async way.

Ok, definitely a mix of what can be found in tokio::process and tokio::io is needed for a better solution that doesn't launch these extra threads, will look into it tomorrow.

@bkontur I pass the tokio runtime handle around to make things more explicit, but the alternative is to just obtain a handle through this method https://docs.rs/tokio/1.8.1/tokio/runtime/struct.Handle.html#method.current
Passing the handle around is more cumbersome but has the advantage of forcing you to make sure that things are initialized before trying to call stuff that depends on the tokio runtime being initialized (finding tests that needed to be updated for example was easy thanks to this).
Although I lean to the more cumbersome approach, I am ok with both, what do you think?


localizesFormat API unavailable on iOS

Hello! First of all, thank you for taking the time to upgrade all your packages (URKit, URUI) to support Swift 5.7, really appreciate it!
Currently, I'm running into an issue compiling URKit for iOS targets because it looks like Wolfbase uses an API that is only available on macOS.
We probably need to add #if os(macOS) before running that line. Let me know if this is something I can make a quick PR on. thanks! :)

looks like this was fixed in ad3bef161fe0f1f2b41389a4ff8a831409bd2ec6 -- thanks!


Switch video in player (play next video)

How can I switch the video to the next one or prev?
I want to show a list of videos and then when a user taps on any video, immediately play that video in the player

i need to answer too


MountImage: Get error when try to dismount after mounting iso

Details of the scenario you tried and the problem that is occurring
Get error when try to dismount after mounting iso.
Verbose logs showing the problem
Test-ConflictingResources : A conflict was detected between resources '[MountImage]MountMSSQL2016 (::8::10::MountImage)' and '[MountI
mage]DismountMSSQL2016 (::23::9::MountImage)' in node 'localhost'. Resources have identical key properties but there are differences
in the following non-key properties: 'Ensure'. Values 'NULL' don't match values 'Absent'. Please update these property values so that
 they are identical in both cases.
At line:289 char:9
+         Test-ConflictingResources $keywordName $canonicalizedValue $k ...
+         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (:) [Write-Error], InvalidOperationException
    + FullyQualifiedErrorId : ConflictingDuplicateResource,Test-ConflictingResources
Compilation errors occurred while processing configuration 'Basic_settings'. Please review the errors reported in error stream and mo
dify your configuration code appropriately.
At C:\Windows\system32\WindowsPowerShell\v1.0\Modules\PSDesiredStateConfiguration\PSDesiredStateConfiguration.psm1:3917 char:5
+     throw $ErrorRecord
+     ~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (Basic_settings:String) [], InvalidOperationException
    + FullyQualifiedErrorId : FailToProcessConfiguration

Suggested solution to the issue
N/A
The DSC configuration that is used to reproduce the issue (as detailed as possible)
Configuration Basic_settings
{
    Import-DSCResource -ModuleName StorageDsc #For ISO-mounting

    Node "localhost"
    {

         MountImage MountMSSQL2016
        {
            ImagePath   = 'C:\Install\Software\Microsoft SQL Server 2016 RTM with SP2 (MSDN)\en_sql_server_2016_enterprise_core_with_service_pack_2_x64_dvd_12124052.iso'
            DriveLetter = 'S'
            #Ensure = 'Present'
        }

        WaitForVolume WaitForISO
        {
            DriveLetter      = 'S'
            RetryIntervalSec = 5
            RetryCount       = 10
            DependsOn = '[MountImage]MountMSSQL2016'
        }

        MountImage DismountMSSQL2016
        {
            ImagePath   = 'C:\Install\Software\Microsoft SQL Server 2016 RTM with SP2 (MSDN)\en_sql_server_2016_enterprise_core_with_service_pack_2_x64_dvd_12124052.iso'
            DriveLetter = 'S'
            Ensure = 'Absent'
            DependsOn = '[WaitForVolume]WaitForISO'
        }
    }
}

$configData = @{
    AllNodes = @(
        @{
            NodeName = '*'
            PSDscAllowPlainTextPassword = $True
            PsDscAllowDomainUser = $true
        }
        @{
            NodeName = 'localhost'
            PSDscAllowPlainTextPassword = $True
            PsDscAllowDomainUser = $true
        }
    )
}

Basic_settings -OutputPath "C:\DSC\Configurations" -ConfigurationData $configData

The operating system the target node is running
OsName               : Microsoft Windows Server 2016 Datacenter
OsOperatingSystemSKU : DatacenterServerEdition
OsArchitecture       : 64-bit
WindowsBuildLabEx    : 14393.1794.amd64fre.rs1_release.171008-1615
OsLanguage           : en-US
OsMuiLanguages       : {en-US}

Version and build of PowerShell the target node is running
PSVersion                      5.1.14393.1884
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.14393.1884
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

Version of the DSC module that was used ('dev' if using current dev branch)
current

Hi @NemoDima , sorry for the delay in getting back to you.
This is by design and is how DSC should be used. This was discussed in https://github.com/dsccommunity/StorageDsc/issues/154.
I do understand the requirement though. In general, I'd recommend just leaving the ISO mounted permanently. @johlju - any thoughts you can share here? E.g. what do other users of SqlServerDsc tend to do? Do they just leave the ISO mounted?
The gist is that DSC is designed to define a desired state. So creating a configuration that has two different states for a resource defined is considered illegal - e.g. in this case, having an ISO both mounted and unmounted is a conflict.

@PlagueHO thank you for the answer and explanation.
Now I understand DSC logic quite better.
I'm not sure, but maybe it has the sense to add some remarks to readme about MountImage and row execution.
For my purpose, I'll try to look on Script(https://docs.microsoft.com/en-us/powershell/scripting/dsc/reference/resources/windows/scriptresource) or xScript from xPSDesiredStateConfiguration (https://github.com/dsccommunity/xPSDesiredStateConfiguration).
Thanks again

any thoughts you can share here? E.g. what do other users of SqlServerDsc tend to do? Do they just leave the ISO mounted?

@PlagueHO normally I would say you don’t mount an ISO. I would say that the normal production scenario is instead to extracted the ISO into an UNC path and the SourcePath parameter of SqlSetup resource is pointed to that UNC path. Mounting an ISO is probably best used in lab environments where you change SQL Edition very often and don’t want to change the configuration for whatever reason.
Although it seems for whatever reason the normal scenario is mounting an iso. Not sure if that is by old habit from click-install processes.
@NemoDima is there a reason you mount the image instead of using an UNC share. Interested if the normal scenario I describe above is not as normal as I expect. 😊

@johlju We have a scenario where we host clients on bare metal machines which only have Windows Server installed and we need to install everything else, most notably SQL Server.
Our ideal workflow would be: download ISO file, mount, run SQL Server setup, unmount image, delete file. I'm aware it's much more imperative than declarative :-)
@PlagueHO, the way we work around this is that we have a set of partial DSC configurations used for "initial configuration" - these involve bootstrapping our custom DSC resources module, setting up some very basic prerequisites (e.g. setting timezone, machine name, etc). That configuration set also contains a "one-off" partial configuration which sets up our SQL Server: downloads SQL Server ISO file, mounts it and runs SQL Server setup. For SQL Server 2017 and later, it also downloads and installs SSRS. That configuration set is run only once, against a brand-new machine.
After that, we have another set of partial configurations, which is a superset of the "initial configuration" partial configurations, with the one-off partial configuration removed.
@johlju, our approach does hit a snag at the moment, though. Our regular configuration unmounts the SQL Server ISO file and deletes it (no need to keep a few GB permanently mounted). Unfortunately, Get-TargetResource for MountImage throws an error if the target image file is not present on the system (Get-DiskImage throws), preventing our configuration from being re-runnable.
I think the simplest solution would be for MountImage to treat the mount as Absent if the target image is not present on the system instead of throwing an exception.

Another approach i've used in the past is to use a Script resource to install SQL based on a configuration file - see the example here: https://colinsalmcorner.com/install-and-configure-sql-server-using-powershell-dsc/
You do lose visibility within DSC with this approach, it used to be my go-to method when the install resource was less mature.
Now, the approach i typically take is to create a template that includes the SQL media already in situ, which could be kept up to date using something like packer (or a script).
Understand this means you've got ISOs copied everywhere, but in this age of de-duplicated storage and other wizardry it wasn't a worry for us.

@johlju, our approach does hit a snag at the moment, though. Our regular configuration unmounts the SQL Server ISO file and deletes it (no need to keep a few GB permanently mounted). Unfortunately, Get-TargetResource for MountImage throws an error if the target image file is not present on the system (Get-DiskImage throws), preventing our configuration from being re-runnable.

I think you will also hit a problem in SqlSetup if it is used since Get-TargetResource expects the setup.exe to be available to determine SQL major version. Major version is needed to determine the correct paths in a scenario where multiple instances is installed with different major versions.


Implenting very rudimentary http caching

POC
adds ETAG and Last-Modified headers
rudimentary handling of incoming if-none-match and if-modified-since
Currently only enabled if memcach cache is enabled

This is no longer relevant, closing.


Evaluation of CycleGAN-VC3 in your paper

Hi,
Thank you for sharing the paperwork!
I wonder if you trained the Resnet-9blocked generator in CycleGAN-VC3 for your paper evaluation.
Since I followed your command instruction for training CyleGAN-VC3, it uses the same "netG" as the CVC framework.

Hi @powei-C,
Thanks for your interest in our work.
Actually we used the Resnet-9blocked generator in CycleGAN-VC3 for fair comparison in our paper, since we'd like to demonstate that cycle-consistency loss can be replaced by the contrastive loss.

Thank you for your explanation!
I still have one question.
When listening to the demo audio,
I found that the audio's speed slows down a little bit compared to the source audio,
even though I retrain the model and test it, there is a similar situation.
Is it reasonable?

Thank you for your instant reply!
It makes sense!!!
Since I have run out of my GPU resources recently, I may test it a few days later.
Best Regards

It worked!
Thank you for your help!
Best Regards

How did you do the settings?

It worked! Thank you for your help!
Best Regards

@powei-C How did you do the settings?

@Quetzalquatl It's been a long time... I am not 100% sure right now. I remember that I just adjust the parameter's value in CVC/data/wav_folder.py to match the one in "/CVC/checkpoints/vocoder/config.yml" such as hop size, fft size and win_length ....
Hope this help~

@powei-C I changed the settings in wav_folder.py according to the parameters in vocoder/config.yml but the result is very bad. no matter what i did i couldn't fix it well. If yes, could you paste your settings here?


New broker to be added. Pepperstone

Hi, I am told to submit a ticket here for any new broker suggestions. Have you guys thought about adding Pepperstone to the brokers supported? If not, I am looking adding it myself.

Hi, Any updates here?

Guide to contributing brokerages - https://www.quantconnect.com/docs/v2/lean-engine/contributions/brokerages/setting-up-your-environment would be another new repo.


fix documentation warnings & 404 deadlink

/mnt/c/Users/yann/Desktop/code/nni/docs/en_US/community_sharings.rst:7: WARNING: toctree contains reference to nonexisting document 'CommunitySharings/NasComparison'
/mnt/c/Users/yann/Desktop/code/nni/docs/en_US/community_sharings.rst:7: WARNING: toctree contains reference to nonexisting document 'CommunitySharings/HpoComparison'
WARNING: autodoc: failed to import class 'smac_tuner.SMACTuner' from module 'nni.smac_tuner'; the following exception was raised:                   No module named 'ConfigSpaceNNI'
/mnt/c/Users/yann/Desktop/code/nni/src/sdk/pynni/nni/bohb_advisor/bohb_advisor.py:docstring of nni.bohb_advisor.bohb_advisor.BOHB:11: WARNING: Unexpected indentation.
/mnt/c/Users/yann/Desktop/code/nni/src/sdk/pynni/nni/bohb_advisor/bohb_advisor.py:docstring of nni.bohb_advisor.bohb_advisor.BOHB:13: WARNING: Block quote ends without a blank line; unexpected unindent.
/mnt/c/Users/yann/Desktop/code/nni/src/sdk/pynni/nni/bohb_advisor/bohb_advisor.py:docstring of nni.bohb_advisor.bohb_advisor.BOHB:25: WARNING: Unexpected indentation.
/mnt/c/Users/yann/Desktop/code/nni/src/sdk/pynni/nni/bohb_advisor/bohb_advisor.py:docstring of nni.bohb_advisor.bohb_advisor.BOHB:26: WARNING: Block quote ends without a blank line; unexpected unindent.

checking consistency... /mnt/c/Users/yann/Desktop/code/nni/docs/en_US/CommunitySharings/HpoComparision.md: WARNING: document isn't included in any toctree
/mnt/c/Users/yann/Desktop/code/nni/docs/en_US/CommunitySharings/NasComparision.md: WARNING: document isn't included in any toctree                  /mnt/c/Users/yann/Desktop/code/nni/docs/en_US/GPTuner.md: WARNING: document isn't included in any toctree                                           /mnt/c/Users/yann/Desktop/code/nni/docs/en_US/NniOnWindows.md: WARNING: document isn't included in any toctree                                      /mnt/c/Users/yann/Desktop/code/nni/docs/en_US/builtin_assessor.rst: WARNING: document isn't included in any toctree                                 /mnt/c/Users/yann/Desktop/code/nni/docs/en_US/builtin_tuner.rst: WARNING: document isn't included in any toctree

/mnt/c/Users/yann/Desktop/code/nni/docs/en_US/BohbAdvisor.md:44: WARNING: Pygments lexer name 'yml' is not known
/mnt/c/Users/yann/Desktop/code/nni/docs/en_US/BuiltinTuner.md:68: WARNING: Pygments lexer name 'yml' is not known                                   /mnt/c/Users/yann/Desktop/code/nni/docs/en_US/CommunitySharings/HpoComparision.md:37: WARNING: None:any reference target not found: ../gbdt_example /mnt/c/Users/yann/Desktop/code/nni/docs/en_US/GeneralNasInterfaces.md:114: WARNING: None:any reference target not found: ./multiPhase               /mnt/c/Users/yann/Desktop/code/nni/docs/en_US/GeneralNasInterfaces.md:49: WARNING: Could not lex literal_block as "json". Highlighting skipped.     /mnt/c/Users/yann/Desktop/code/nni/docs/en_US/GeneralNasInterfaces.md:49: WARNING: Could not lex literal_block as "json". Highlighting skipped.     /mnt/c/Users/yann/Desktop/code/nni/docs/en_US/HowToImplementTrainingService.md:10: WARNING: None:any reference target not found: FrameworkController/mnt/c/Users/yann/Desktop/code/nni/docs/en_US/Nnictl.md:741: WARNING: Pygments lexer name 'yml' is not known                                        /mnt/c/Users/yann/Desktop/code/nni/docs/en_US/Overview.md:53: WARNING: None:any reference target not found: BuiltinAssessors                        /mnt/c/Users/yann/Desktop/code/nni/docs/en_US/QuickStart.md:243: WARNING: None:any reference target not found: BuiltinAssessors                     /mnt/c/Users/yann/Desktop/code/nni/docs/en_US/QuickStart.md:24: WARNING: Pygments lexer name 'pseudo' is not known                                  /mnt/c/Users/yann/Desktop/code/nni/docs/en_US/Release.md:71: WARNING: None:any reference target not found: ./WindowsLocalMode                       /mnt/c/Users/yann/Desktop/code/nni/docs/en_US/Release.md:154: WARNING: more than one target found for 'any' cross-reference 'MetisTuner': could be :doc:`Metis tuner` or :py:class:`nni.metis_tuner.metis_tuner.MetisTuner`
/mnt/c/Users/yann/Desktop/code/nni/docs/en_US/Release.md:156: WARNING: more than one target found for 'any' cross-reference 'CurvefittingAssessor': could be :doc:`Curve fitting assessor` or :py:class:`nni.curvefitting_assessor.curvefitting_assessor.CurvefittingAssessor`
/mnt/c/Users/yann/Desktop/code/nni/docs/en_US/Trials.md:47: WARNING: None:any reference target not found: BuiltinAssessors

Solutions:

fix deadlinks
change lexer name: yml => yaml, json => 'javascript'

after fix:
/mnt/c/Users/yann/Desktop/code/nni/docs/en_US/community_sharings.rst:7: WARNING: toctree contains reference to nonexisting document 'CommunitySharings/NasComparison'
/mnt/c/Users/yann/Desktop/code/nni/docs/en_US/community_sharings.rst:7: WARNING: toctree contains reference to nonexisting document 'CommunitySharings/HpoComparison'
WARNING: autodoc: failed to import class 'smac_tuner.SMACTuner' from module 'nni.smac_tuner'; the following exception was raised:                   No module named 'ConfigSpace'
WARNING: autodoc: failed to import class 'bohb_advisor.BOHB' from module 'nni.bohb_advisor'; the following exception was raised:                    No module named 'ConfigSpace'
looking for now-outdated files... none found                                                                                pickling environment... done                                                                                checking consistency... /mnt/c/Users/yann/Desktop/code/nni/docs/en_US/CommunitySharings/HpoComparision.md: WARNING: document isn't included in any toctree
/mnt/c/Users/yann/Desktop/code/nni/docs/en_US/CommunitySharings/NasComparision.md: WARNING: document isn't included in any toctree                  /mnt/c/Users/yann/Desktop/code/nni/docs/en_US/GPTuner.md: WARNING: document isn't included in any toctree                                           /mnt/c/Users/yann/Desktop/code/nni/docs/en_US/NniOnWindows.md: WARNING: document isn't included in any toctree                                      /mnt/c/Users/yann/Desktop/code/nni/docs/en_US/builtin_assessor.rst: WARNING: document isn't included in any toctree                                 /mnt/c/Users/yann/Desktop/code/nni/docs/en_US/builtin_tuner.rst: WARNING: document isn't included in any toctree                                    done
preparing documents... done                                                                                writing output... [100%] tutorials                                                                                /mnt/c/Users/yann/Desktop/code/nni/docs/en_US/Release.md:154: WARNING: more than one target found for 'any' cross-reference 'MetisTuner': could be :doc:`Metis tuner` or :py:class:`nni.metis_tuner.metis_tuner.MetisTuner`
/mnt/c/Users/yann/Desktop/code/nni/docs/en_US/Release.md:156: WARNING: more than one target found for 'any' cross-reference 'CurvefittingAssessor': could be :doc:`Curve fitting assessor` or :py:class:`nni.curvefitting_assessor.curvefitting_assessor.CurvefittingAssessor`

@QuanluZhang how can I fix No module named 'ConfigSpace' error?

@leckie-chn should install the necessary packages through nnictl package install

great job, BTW.

@leckie-chn should install the necessary packages through nnictl package install

we can only specify dependencies inside requirements.txt, for we cannot customize docker image for  nni.readthedocs.io


[wip] Remove "view page source"

Removes the link to display raw page sources on each webpage

Issue addressed here: https://github.com/SubstraFoundation/substra-documentation/pull/86


Write a TypeScript program to convert minutes to seconds

Description
Write a TypeScript program to convert minutes to seconds
1 minute = 60 seconds
Input  : 5
Output : 300

How to contribute

Save the solution in ConvertMinutesToSeconds.ts file
Add ConvertMinutesToSeconds.ts file in convert-minutes-to-seconds folder

!assign

Hey @iamClutchX,
This issue has been assigned to you for more than 15 days.
We are unassigning you from this issue.
If you are still interested in contributing to this issue, please get auto-assigned on the issue by commenting !assign again.
Thanks for your contributions :)

this issue was merged as i see

!assign


Is it possible to have PC automatically put in sleep mode when TV is turn off (use power button on the remote control) just like the Ps5?

I was wondering if Is it possible to have PC automatically put in sleep mode when TV is turn off (use power button on the remote control) just like the Ps5?

I will consider this feature request, even though it's a bit incompatible with the scope of the application (to work like a normal PC-monitor)

yeah or even shutdown? honestly don't know how you'd do this though over ip though? unless you have an addon lg webos app? or maybe a phone home ping to the tv ip address every mins or so? anyway thanks for your work i couldn't believe in this day and age where TV's are used mainstream with pc gaming this isn't implemented at hardware level on 30x and 40x GPUs its not the 1990s TV's have vrr, gsync, game mode, all, freesync, etc and GPUs have hdmi, hdmi 2.1 etc so why not cec? i appreciate you coding this its exactly what everyone that wants to game in the living room with a 83inch tv on a sofa instead of in there basement on a 24 inch monitor lol, im sure ive heard your name "JPerssson77" did you do work on the lg root via web code? im actually running that right now lol to get add free youtube with sponsor skip etc


Revert "CLOUDP-242997: [FIXED] PoC translation layer for dbUser (#1688)"

This reverts commit 5c2146f93c6374e477e5fa755e609fa3fd7c0f5c which is causing issues with gov tests.
All Submissions:

[X] Have you signed our CLA?

CLOUDP-242997

@igor-karpukhin seems this revert does not help much, it also fails gov tests.
Let me know if you think we shall just close this one or if you see any value on it.


Merge pull request #1536 from MicrosoftDocs/master

Sync to live

Docs Build status updates of commit b0030e5:
:warning: Validation status: warnings
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.


Redesigned Preferences

I feel like the preferences panel should be redesigned, as there aren't that many things and settings to do in it. Not only that, there are 2 deprecated options which are unnecessary for future use.

#3812

Are you talking about the Cmd+, options? Those deprecated settings will be removed in the future. Otherwise I don’t see a need to “redesign” it as it’s just a bunch of check boxes…


node-modules is hidden

Hi,
On gitter few people confirmed the same issue.
This plugin is causing an issue with node-modules folder... no idea why

Can confirm

@mariaczi @crybat This is not a problem.I deliberately hide his show in the idea.

@henjue I think it's causing issues with some autocompletion and also, sometimes you want to take a look at some files inside node_modules to understand what's going on in background. Can you unhide it ?

我刚开始接触vue，今天也遇到了这个问题。刚开始以为是自己设置的有问题，花了2个小时的时间也没找到原因。最后无奈卸载webstorm重新安装的过程中才发现是vue for idea插件引起的。在学习过程中我会查看node_modules里面的源码，这个问题可以修复吗？

I have the same issue

@yin-fan 收到，下一班我加入设置项。是否显示node_modules

搞了半天问题出在这个插件身上，我还以为是 WebStorm 测试版的 bug。

i met the same problem

I can confirm. I have the same issue. node_modules is hidden when vue-for-idea is activated.
Version: PHPStorm 2016.2

Confirming for WebStorm 2016.1 / 2016.2

This is significant bug. Vue plugin shouldn't make such a big side effects as hiding important folders.

Definetely a but. This makes the plugin unusable!

Not only node_modules folder. I create a folder and a file in this folder, and then I delete the new file, the folder is hidden.

It affected all projects not only for vue projects!

还没有修复么？我搞了两个多小时才发现是插件问题

please remove this "feature"!

@henjue can we know why you do that? makes your plugin unusable for me.

Hiding a folder without explanation is frustrating. Forcing it to be excluded should be the decision of the plugin user, not the plugin author.
The Node.js plugin doesn't do this, so why should the Vue plugin? The PHP Composer support in PhpStorm doesn't do this with the vendor folder either (though it does put all the external libraries in the External Libraries part of the Project tree).

Yeah, this is obviously a bug that the developer refuses to look into.
Too bad.

After trying to find the code that hides the folder in this plugin, I can't see it consciously being being removed. It really looks like a bug. That said I don't know the inner workings of JetBrains/IDEA.
For the record, @henjue, I contacted JetBrains support about node_modules disappearing, and they were the ones who sent me a link to this GitHub issue, and within 12 hours too! They know this plugin is doing this weird thing.
💀
I've uninstalled this plugin, and have switched to John Kelly's Vue.js plugin. It has more stars.

@henjue we're receiving quite a lot of complaints about this issue via JetBrains support channels. We’d really appreciate if you fix this problem as well as incorrectly closed #28, otherwise we'll be considering removing this plugin from the plugin repository.

any news?

The plugin has been removed from the repository due to reasons stated in my previous comment and due to no cooperation from the author. Please use John Kelly's Vue.js plugin

Weird stuff, moving to John Kelly's Vue.js plugin

这个BUG还没修复吗？我也找了半天，还以为是webstorm设置问题

Just a note for anyone more at home in idea plugin development.
In the branches pw and develop there is a setting in the plugin.xml file:

This line references the file plugin/src/main/java/io/j88/idea/vue/structure/VueTreeStructureProvider.java. On lines 29, 33 and 34 is the offending code. If these lines are to be removed, I think it is best to remove the whole plugin.xml line and file, since the file won't do anything anymore.
Just my 2 ct.

@chancezeus if you've found a problematic code it may be a good idea to fork this repository. Consider this ;)

@denofevil as I see this plugin is published again without fixing reported bug: https://plugins.jetbrains.com/plugin/9188

@piotros, thank you for the update.
@henjue, please resolve issues mentioned above, or that upload will be removed too

Seems that latest version has also broken code completion and identation, too :/

@piotros, the plugin approval was removed, new versions (and re-uploads) won't get it until this issue is resolved

Fuck the scums who feedback this question to Jetbarins. "John Kelly's Vue.js plugin" is a lump of dog feces. Now we have no any workable Vue support plugin.

The reason this plugin is not viable is because there's a huge blocker bug in it at the moment which must be fixed.

@amcsi Sorry, it seems that I misunderstood

Still an issue as of 10.0.74.. :(

I had to remove the plugin.

我靠,终于找到了,原来是这个插件的原因啊?

Looks like this plugin is not going to be fixed any time soon unfortunately for vue developers that prefer using IDEA. @henjue, probably hasn't got time for it anymore or lost an interest.

@asolopovas: Check out this alternative, it's much better: https://github.com/postalservice14/vuejs-plugin

@callumacrae I am using this one. It has its own issues. It does not detect default export when components are imported. Because export default is within script tags. Sass does not work properly in components.

@asolopovas: If you put type="text/babel" on the script tags and rel="stylesheet/scss" on the style tags, it works. It sucks, but it's better than the problems with this plugin.

@callumacrae agree

This problem confused me for really serveral months, and I blamed to webstorm before...Really cann't agree with https://github.com/henjue/vue-for-idea/issues/46#issuecomment-227340288, so I have to drop this plugin...

楼上的各位，其实webstorm自身有个功能，在文件夹上右键 Mark Director as -> Excluded 。这个功能可以把文件夹从 webstorm 的文件加载中排除掉。目的就是为了避免加载大文件过慢的问题。
所以，插件的作者可以把这个默认隐藏去掉了，或者可配置。
@mqliutie

Having the same issue with 2106.3, installed on Nov. 22. I do not have the Vue plugin installed. Turns out that when Javascript Support is enabled, the node_modules directory is hidden. Hoo boy. I will submit this to a support channel.

@wvezey: Woah, seriously? I'm going to hold off updating, then!
@denofevil: Is this true?

@callumacrae cannot confirm yet, you can follow https://youtrack.jetbrains.com/issue/WEB-24387 for the updates

@callumacrae @denofevil l have not gone through all of the plugins to test their culpability in this issue, but I can say with certainty that the visibility of a node_modules directory toggles with the enabled state of the JavaScript Support plugin. I read the issue that @denofevil posted above; it looks like this behavior is intentional. Access to node_modules comes in pretty handy when debugging. Even on the day after Thanksgiving.

@wvezey can you please provide a screenshot including title bar like the one that I've posted above?

With JavaScript Support plugin enabled:

With JavaScript Support plugin disabled:

@wvezey you have Project files selected as a Project view scope, it shows you only project files without library ones (node_modules is considered to be library source). If you select Project via dropdown on the top you will see node_modules too

@denofevil Right you are! Thank you very much for your very quick and thorough help.

Wow, this cost me more than an hour fiddling around with my settings, filing a bug with WebStorm lead me to this issue.  😕
Plugins should never make these decisions for the user. If it really adds to the experience, offer it as an option in the plugin settings.

Guys if you would vote on this issue on the Jetbrains issue tracker, maybe we could get official VueJS support some time in the not so distant future: https://youtrack.jetbrains.com/issue/WEB-19082

Just unbelievable stupid!!! And I only say this because he (the owner) said he did it on purpose :O. Luckily I found out what caused this rather quick. I can imagine what some went through to find the issue and I bet there are still allot of people that did not found out...

两个月了，就是没一个说已经解决了的，白看了！！！

@wushuang5112 你把 vue-for-idea 这个插件给卸了就好了，应该时插件没写好


Cannot set enum to arguments in Function Mutation

Environment information
System:
  OS: macOS 14.5
  CPU: (8) arm64 Apple M2
  Memory: 173.20 MB / 24.00 GB
  Shell: /bin/zsh
Binaries:
  Node: 20.9.0 - ~/.nvm/versions/node/v20.9.0/bin/node
  Yarn: 1.22.19 - ~/.nvm/versions/node/v20.9.0/bin/yarn
  npm: 10.1.0 - ~/.nvm/versions/node/v20.9.0/bin/npm
  pnpm: 8.11.0 - ~/.nvm/versions/node/v20.9.0/bin/pnpm
NPM Packages:
  @aws-amplify/backend: 1.0.2
  @aws-amplify/backend-cli: 1.0.3
  aws-amplify: 6.3.4
  aws-cdk: 2.143.0
  aws-cdk-lib: 2.143.0
  typescript: 5.4.5
AWS environment variables:
  AWS_STS_REGIONAL_ENDPOINTS = regional
  AWS_NODEJS_CONNECTION_REUSE_ENABLED = 1
  AWS_SDK_LOAD_CONFIG = 1
No CDK environment variables

Description
When I try to set ENUM in the arguments of the Function Mutation, I get the following error and cannot do a sandbox deploy.
Failed to instantiate data construct
Caused By: Schema validation failed.\n\nUnknown type "TestFunctionResolverTestEnum".\n\nGraphQL request:25:34\n25 |   TestFunctionResolver(testEnum: TestFunctionResolverTestEnum): EchoResponse @fu\n   |                                  ^\n   | nction(name: "FnTestFunctionResolver") @auth(rules: [{allow: private}])

Resolution: See the underlying error message for more details.

This is the definition of my function:
  TestFunctionResolver: a
    .mutation()
    .arguments({ testEnum: a.enum(["A", "B", "C"])})
    .returns(a.ref("EchoResponse"))
    .authorization((allow) => [allow.authenticated()])
    // 3. set the function has the handler
    .handler(a.handler.function(sayHello)),

Hey👋, thanks for raising this! I'm going to transfer this over to our API repository for better assistance 🙂


[IOS] 0.1.230 EXC_BAD_ACCESS  TypedArray::update

Description
In Skia version 0.1.230 iOS devices started to crash with EXC_BAD_ACCESS: ZTUM > KERN_INVALID_ADDRESS at 0x40. when animated blurs were rendered conditionally. I tracked this down to #2062. However, I am not sure what specifically causes it.
Version
0.1.230
Steps to reproduce
After conditionally rendering the blur animation once or twice, the app immediately crashes.
Snack, code example, screenshot, or link to a repository
Here is a minimal code example to reproduce it in the Skia iOS example app in the repository.
import React, { useEffect, useState } from "react";
import { useSharedValue, withTiming } from "react-native-reanimated";

export const CrashingBlur = () => {
  const [crasher, setCrasher] = useState(false);

  const blur = useSharedValue(4);

  const switchCrasher = () => {
    setTimeout(() => {
      setCrasher((prev) => !prev);
      switchCrasher();
    }, 2000);
  };

  useEffect(() => {
    blur.value = withTiming(12, { duration: 10000 });

    switchCrasher();
  }, []);

  return (
    <Canvas
      style={{
        height: "100%",
        width: "100%",
      }}
    >
      {crasher && (
        <Path path="M 100 100 L 300 100 L 200 300 z">
          <BlurMask blur={blur} />
        </Path>
      )}
    </Canvas>
  );
};

@CreatlV this looks interesting. I wasn't able to reproduce the issue on iOS nor Android. Can you help me to reproduce the crash?

Curious to hear that you were not able to reproduce it. I think this only happens on iOS. I opened a draft PR that should help you reproduce it #2160 .  I tested on an iPhone 13 running 17.2.1 but have seen crash reports from a range of iOS devices.


Java 7u51 installer complains that Java 8 is already installed

I have manually installed Java 8. When I run Boxen, I get this error message:

Error: Execution of '/usr/sbin/installer -pkg /private/tmp/dmg.h7gWu3/Java 7 Update 51.pkg -target /' returned 1: installer: Error - You are trying to install Java 7 Update 51, however Java 8 is already installed.
Visit java.com/newerversionexists for more information.

Error: /Stage[main]/Java::V7u51/Java::Version[v7u51]/Java[7u51]/Package[jre-7u51.dmg]/ensure: change from absent to present failed: Execution of '/usr/sbin/installer -pkg /private/tmp/dmg.h7gWu3/Java 7 Update 51.pkg -target /' returned 1: installer:  Error - You are trying to install Java 7 Update 51, however Java 8 is already installed.
Visit java.com/newerversionexists for more information.

I would like to support having both versions installed at the same time.
I looked into command line parameters for the installer, but could not find anything relevant. Strangely, if I run the same command manually on the command line, Java installs without an error, but if I run it from Puppet, I get the above errors.

Fixed by #54.


Feature/oauth

Rework on authentication.
I've created example html/spec file (authentication.html/json) to show the different scenarios and gui. Tried OAuth2 flows with Identity Server setup (tried swashbuckle gui and then this and both worked).

More information in authentication list
add OAuth2 clientCredentials and authorizationCode support
Request information more info about required security
Request each authentication is seperated to own calls

note: if more are interrested to have more auth types maybe a look into swaggers own auth code should solve some things more generic. But I think this should work as a start.
Issues that talks about oauth

https://github.com/mrin9/RapiDoc/issues/142
https://github.com/mrin9/RapiDoc/issues/54

Thanks for the PR, Supporting OAuth has been a long pending feature, but there is no suitable workflow that we could come up with in the context of Custom-Element
For instance as a part of this PR, you created a file docs/oauth2-redirect.html which helps in token exchange, but this file will not be a part of the build output and therefore will break the flow
The build script generates a single RapiDoc.js file which is a pure java script Custom-Element no css, fonts, images or anything else is included with it
If this would have been a WebApp like MrinDoc we could have included docs/oauth2-redirect.html and can place it on the WebServer. but thats not the case with Custom Elements.
let me know your thoughts

Could it be that you mention the api docs and/or github readme.
That if authorizationCode is used the oauth2-redirect.html file is needed to finalize the authentication. It should be placed in the path root or use the redirectUrl parameter in specification file.
Then a direct link to the file in github where the developer can download the needed oauth html file.
If we get the html file in dist folder the file could be accessed thru https://unpkg.com/rapidoc/dist/oauth2-redirect.html so quick access for developer would just to add that as the redirectUrl.
Think it's inside the scope with OAuth to be a little more configuration. You would need a oauth server, setup client credentials, redirect urls and so on.
Can try to help write some kind of documentation help if you want?

The way I am envisioning it is to create another custom-element something like <oauth-receiver>  which will contain the functionality of oauth2-redirect.html
and some documentation which will state to create another .html file and include <oauth-receiver> in it.

I've started a new branch from the latest master and trying to adjust the authentication to match. Think what I'm doing now will work better and easier to implement all different kind of oauth types in a later stage.
So this branch we could throw out and I can create a new PR when I'm ready.

I've a much better pull request for this now. Will send a new pull request soon. Needs just a little more testing for many scenarios.


Memory leak with py_func

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes (see below)
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows 10
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): 1.7.0
Python version: 3.6.4 64bit
Bazel version (if compiling from source): N/A
GCC/Compiler version (if compiling from source): N/A
CUDA/cuDNN version: 9.0
GPU model and memory: GeForce 1080 Ti
Exact command to reproduce: (see below)

Issue
py_func which has a python function referencing the graph (might be an unobvious reference, see below) results in the graph never being garbage collected. If you know about this issue usually it can be worked around, but in general it's a pitfall which should be fixed in the code (see below for suggestion).
Source Code Example
The following source code produces the leak. If you comment out py_func.graph = graph you will see that "Deleted MyPyFunc" will be printed.
import gc
import tensorflow as tf

class MyPyFunc:
    def __init__(self):
        self.graph = None

    def __call__(self, inputs):
        return inputs

    def __del__(self):
        print("Deleted MyPyFunc")

def run_graph():
    graph = tf.Graph()
    with graph.as_default():
        inputs = tf.constant(0)
        py_func = MyPyFunc()
        op = tf.py_func(py_func, [inputs], [inputs.dtype])
        # Comment this out to fix the leak
        py_func.graph = graph

    with tf.Session(graph=graph) as sess:
        print("Result:", sess.run(op))

run_graph()
gc.collect()
run_graph()
gc.collect()

Analysis
The issue originates from script_ops.py:181. Here, all py_funcs get registered globally, i.e. these will only be deleted when CleanupFunc is garbage collected. It is instantiated at script_ops.py:222 and stored in the graph when py_func is called. The idea is, that the CleanupFunc.__del__ method is called when the graph is garbage collected, which in turn will delete the reference to the python function, so it can get garbage collected as well.
In the example, MyPyFunc contains a reference to the graph. By calling py_func the reference to MyPyFunc is stored globally in _py_funcs = FuncRegistry(). That means the graph is referenced globally, so it will never get garbage collected, which in turn means that the CleanupFunc.__del__ will never be called, so MyPyFunc will never be deleted from _py_funcs.
Importance
Consider the following:
class MyOp:
    def __init__(self, inputs):
        self.op = tf.py_func(self._py_func, [inputs], [inputs.dtype])

    def _py_func(self, inputs):
        return inputs

This looks like valid code, but having MyOp.op in your graph will make the graph leak. The problem lies in the fact that _py_func has is bound to the MyOp instance which contains a reference to a TensorFlow op which in turn has a reference to the graph (for the remaining inference see above). This is a big issue, because tf.estimator.train_and_evaluate creates a lot of graphs during training (i.e. when it switches between training and evaluation a new graph is created).
Suggestion
Instead of globally storing references to the functions and keys to them, they might be better stored in the graph directly. This also prevents them from being deleted while the graph exists. Thus they are only cyclic references in the graph and can be garbage collected with the graph. Instead of storing
graph._cleanup_py_funcs_used_in_graph.append(cleanup), the function should be stored directly like graph._py_funcs_used_in_graph.append(func).

Could weakref solve the problem?

@facaiy: I can easily fix my code, now that I know where the leak comes from. The problem here is rather the pitfall which creates the leak (i.e. hidden/unobvious references to the graph from your py_func, which can be really hard to find). So if the weakref was ment for my py_func, that's actually not the issue. If you suggest replacing the direct reference in the FuncRegistry, that is not helpful, because then I guess the functions might actually be deleted though they are being used internally by the graph.

/CC @zffchen78

Nagging Assignee @drpngx: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Can we close this?

Thanks!


Float History Viewer scrollbar

Right now it is hard to use the scrollbar if you want to look at changes that occur near the end of the notebook. We could float the scrollbar so users can look at a later part of the notebook but still have the scrollbar be visible so they can scroll back and forth to see the changes there

3aae53c

We want it so moving the slider does not cause the user to scroll all the way to the top of the notebook. I think we can do something like check their current scroll height, render the cells, and then set scroll height to the min of either the previous scroll height or current length.


[PE-D][Tester C] Unknown Command tasks

`tasks [index] is listed in the UG as a viable command but executing it in the app throws "Unknown Command"

Labels: type.DocumentationBug severity.High
original: SeekSaveServe/ped#6

Fixed.


Remove the deprecated --skip-preflight-checks for kubeadm

Use --ignore-preflight-errors=all instead.
removed in PR:
https://github.com/kubernetes/kubernetes/pull/62727
failing test tracking issue:
https://github.com/kubernetes/kubernetes/issues/66338
i don't have a good solution if someone tries an old kubeadm version with k-a....
this is difficult to maintain.
/assign @luxas
/cc @timothysc @BenTheElder

let's assign more approvers as we are not sure when/if Lucas can reply:
/assign @errordeveloper
/assign @mikedanese

/lgtm


bump payload and aws packages and reset yarn.lock to fix NoSuchKey ERROR

@pogseal @Dimbreath Check to make sure it doesn't break anything on payload please 🙏

Works on my end fine, though required a yarn clean to get rid of the NoSuchKey error


Resizing a DynamicFont by code can corrupt it

Operating system or device - Godot version: Windows 10, Godot 2.1 https://github.com/godotengine/godot/commit/c328693e837f5899b24504caef5a9ee5e4d4cb4e, self-compiled with Visual Studio 2015 Update 2
Issue description (what happened, and what was expected): When resizing a DynamicFont by code, it can get corrupt. This was tested with a DynamicFont as a theme's default font.
Steps to reproduce: Resize a DynamicFont from code.
Link to minimal example project (optional but very welcome): dpi_scaling.zip - Use the slider at the top left of the screen to change the GUI scale.

call update_changes() after doing changes to a font and it will update it


component failed to setup with KeyError: 'Body'

Caseta component setup failed after I upgraded to home assistant 0.108.0.dev20200330, with below exception.
2020-03-31 18:41:26 ERROR (MainThread) [homeassistant.setup] Error during setup of component lutron_caseta
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/setup.py", line 171, in _async_setup_component
    hass, processed_config
  File "/usr/src/homeassistant/homeassistant/components/lutron_caseta/__init__.py", line 53, in async_setup
    await bridge.connect()
  File "/usr/local/lib/python3.7/site-packages/pylutron_caseta/smartbridge.py", line 51, in connect
    await self._login()
  File "/usr/local/lib/python3.7/site-packages/pylutron_caseta/smartbridge.py", line 363, in _login
    await self._load_occupancy_groups()
  File "/usr/local/lib/python3.7/site-packages/pylutron_caseta/smartbridge.py", line 467, in _load_occupancy_groups
    for occgroup in occgroup_json["Body"]["OccupancyGroups"]:
KeyError: 'Body'

Oops... I've never seen this fail before on my system (but my system also has occupancy sensors attached to it).
@mdonoughe, this should probably be merged and another release cut before HA 0.108 goes out the door?

I have a non-pro version Smart Bridge and i do not have occupancy sensors, maybe that's why it failed...

I also have a non pro version. That’s why I needed to add support for occupancy sensors to this library.


🐛: Cursor changes to text on hovering over options

:red_circle: Title:
:red_circle: Bug: Cursor changes to text on hovering over options
:red_circle: Changes:  Will Turn it To pointer
Screenshots 📷

:white_check_mark: To be Mentioned while taking the issue :

Full name : Priya Raj
What is your participant role?  GSSOC

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎

No need. This just increase the load of the website


Set up directory structure

Set up directories to hold config-generating scripts. I think we should do this in a way that we can add more pipelines later on without too much trouble, but have some shared utils and Azure management stuff. Maybe something like:
pipelines/
    epinow2/
    <other models>/
    ....
utils/
    azure/
        storage.py
        auth.py
        ...

@natemcintosh @zsusswein what do you two think?

I really like it! That utils folder in particular will be useful.

Look good!


Extract out a kotlin-js-ext module from compose-html-ext

Quite a few features inside compose-html-ext are actually not directly tied to compose.
I realized this because I wanted to use one of the WindowExtensions extensions (i.e. invokeThenInterval) from a web worker, but the web worker does not use compose, so it wouldn't make sense for it to use compose-html-ext as a dependency.
Probably the way forward here is to create kotlin-js-ext as a project and make it an API dependency from compose-html-ext. Because if a project wants to use compose-html-ext they definitely want to use the kotlin JS utilities too.
The only tricky question about this is if we need to change the package name now. For example, not package com.varabyte.kobweb.compose.util but maybe package com.varabyte.kobweb.js.util?

We pulled out browser-ext which went live in 0.15.4


Record enter key presses in input fields

Still need to add integrations tests.
Pushed early as the integration tests dont all pass for me, but I think thats a local issue..
The ZestComment support isnt strictly needed, but I added it to help me debug something, and it could be useful in the future?

Added integration test, and fixed a problem with dup events that the test uncovered 😁

Checkmarx One – Scan Summary & Details – 051ef228-19a5-4345-bb3e-5235e72693eb
No New Or Fixed Issues Found

Thank you!


Explicitly require another entry chunk on-demand?

I have an app that I want to treat more-or-less like an SPA even though there are several distinct pages which which serve as entry points into it (e.g. /login, /signup, /password-reset). Each of these entry points corresponds to a named entry in webpack.config (e.g. {login: 'pages/login.jsx', signup:'pages/signup.jsx', ...} and so forth). I also emit a separate HTML page for each entry point using HtmlWebpackPlugin - it's a very small HTML wrapper that simply includes the one entry chunk used for that page. So far so good, it all works really well and we can optimize shared code chunks and so on as advertised.
What I'm having trouble with is allowing navigation between "pages" using pushState alone, minimizing needless downloads and code loading. Ideally I should be able to go from /login to /signup simply by loading the signup entry chunk at runtime (i.e. pages/signup.jsx). I'm not sure if this is actually possible... I might be missing something.
If I load something like require('bundle?lazy!pages/signup.jsx') (or use require.ensure without bundle-loader... though as far as I understand it amounts to the same thing), well.. that works actually, except a new chunk would be created, distinct from the entry chunk (and as such would download all over again when the /signup html page is accessed directly), even though the content should be identical. I thought maybe require('bundle?lazy&name=signup!pages/signup.jsx') might work (explicitly giving it the same name as the entry chunk) but this seems to confuse webpack, I get an uncaught TypeError: Cannot read property 'call' of undefined at runtime.
Is this outright not supported (if so, don't suppose it could be?) or am I just going about it wrong?

This works but is a bit more complex. You have two different chunk types: entry chunks and additional chunks. require.ensure loads additional chunks. entry chunks include the webpack runtime. You only want to have a single webpack runtime on the page. To archive this the orginal entry chunks shouldn't be entry chunks. This can be done with the CommonsChunkPlugin.
{
  entry: {
    a: ["./a"], // must be arrays
    b: ["./b"]
  },
  plugins: [
    new webpack.CommonsChunkPlugin({
      name: "commons",
      minChunks: Infinity // this is optional if you want only the runtime in this chunk
        // i. e. to inline it into the HTML
    });
  ]
}

// You must specify the name elsewise an additional chunk will be created
// Be careful: Loading this chunk also executes the entry point "./a" as sideeffect
require.ensure([], function() {

}, "a");
// or
require("bundle?lazy&name=a!./a");

Ok I think I understand the problem with the runtime and the entry chunks, but that aside, is require("bundle?lazy&name=a!./a"); the way to load another chunk by name? It still seems to get confused when using output.chunkFilename containing a hash... which makes sense since that call isn't really "loading another chunk by name" - it's creating a new chunk that maybe happens to have the same name as something else, which will (almost accidentally) be loaded instead. Is that about right?
Is there a way to actually, explicitly just load a chunk by name, in a way that would work with [hash] ?

Why doesn't it work with [hash]? The chunk names should be independent of the filenames.

Ah indeed, [hash] was not the problem - I was basically still just trying to manually load entry chunks.
I'm running into some further trouble trying to make sense of this while trying to load a chunk on-demand based on a run-time value. I do know, ahead of time, all the possible modules I might want loaded this way, however, it's not just all the files in a folder (in which case the default context behavior would more or less work out of the box) - it's just a list of arbitrary module paths (as in, what would be passed to require - both local-relative paths as well as module-relative paths). I as able to create a custom context for these using require.context and abusing DefinesPlugin a little, and it is correctly aware of all the possible modules it can load (and will correctly load them by calling the context module... but it will do so synchronously, not on-demand). Subsequent require.ensure calls don't seem to be aware of the previously defined context...
It's all gotten pretty messy. I'll probably just make do with using regular dynamic contexts and live without the ability to dynamically load non-entry chunks outside of the ./pages/ + viewName + '.jsx'  folder, unless there is actually a way more straight-forward way of doing this. I mean ideally I could just define the chunks I know I'll want up front (in webpack.config.js) and load them explicitly by chunk name in the run-time code.

Actually, @sokra, let me just try to clear up my general understanding of webpack concepts. Can you confirm as true or false the following statements:

In webpack, the only way  to load code on demand, at run time, is using require.ensure (or require(deps, callback)).
In webpack, the only way to load a module based on a run-time value is on-demand and using a context module.

Is that right so far? I have to use the require.ensure and contexts no matter what - the CommonsChunkPlugin only comes into deciding what code goes in what chunk?
If that is all the case, I have some less straight-forward follow-up questions:

Do you have to use dynamic contexts to load a module on demand based on a run-time value, or is it possible to combine this approach with explicitly created contexts (using require.context)
Are context modules by necessarily tied to a specific root folder and scanning the files within? Is there no way to just feed it a list of potential require paths (i.e. to concrete modules, not just a folder) instead?

I think you're mostly right. Webpack 2 introduces more convenient System.import syntax for chunk separation
btw you can create a loader that would generate your own context based on your own spec. That rules :)

In webpack, the only way to load a module based on a run-time value is on-demand and using a context module.

In webpack, the only way to load a module based on a run-time value is on-demand and using a context module.
A context doesn't load modules on-demand. You need to explicitly combine both concepts to archive this.

Are context modules by necessarily tied to a specific root folder and scanning the files within? Is there no way to just feed it a list of potential require paths (i.e. to concrete modules, not just a folder) instead?

Currently yes, they need to tied to a root folder, but for no technical reason. It's propably useful to add a ManualContextPlugin which overrides the list of requireable modules in a context. (This would require a small change in webpack...)

@webpack-bot I still care about this, but thank you for the warning.

@jakeNiemiec Fixed or caring no more?


[bug]: Wrong Bar Chart Tooltip For Horizontal Layout On Mobile

Describe the bug
When tapping on any bar in the chart, the tooltip for the first bar always appears first, as shown in the video below.
https://github.com/user-attachments/assets/86a5a274-7dd5-4878-a9cd-55eb8e610d98
Affected component/components
Bar Chart
How to reproduce

Open the bar chart page on the documentation site using a mobile device or Chrome's mobile simulator.
Tap on any bar other than the first one in a horizontal bar chart.

Codesandbox/StackBlitz link
No response
Logs
No response
System Info
Windows and Android, Google Chrome

Before submitting

[X] I've made research efforts and searched the documentation
[X] I've searched for existing issues

I think I found the solution but don't really understand what's going on.
Adding "onClick={(e) => e.preventDefault()}" seems to fix the issue.
Look for "ChartTooltipContent". On the return statement div, applying the following changes seems to fix the behavior. (not the underlying issue tho)
      <div
        ref={ref}
        className={cn(
          "grid min-w-[8rem] items-start gap-1.5 rounded-lg border border-border/50 bg-background px-2.5 py-1.5 text-xs shadow-xl",
          className
        )}
        onClick={(e) => e.preventDefault()} // <-- This line fixes the behavior
      >
...
    </div>

Is anyone working on this?

@heecheon92 Thanks for the reply, unfortunately preventDefault didn't solve the issue for me.
Please thumbs up the issue so they notice sooner.


Add support for implementing workflows and tasks with code from the SDK

Evaluate https://gist.github.com/vaskoz/10073335 usage as reference

Done: https://github.com/conductor-sdk/conductor-go/releases/tag/v1.2.3