Allow returning types from functions

Search Terms
Not easily googleable because all the words are the same for extracting the function's return type.
export type from function, return type from function
Suggestion
Make it possible to return a type from a function.
The type doesn't need to depend on the runtime values passed to the function, only on their types. This feature is just syntactic sugar.
Desired:
const {decorated_fn, DecoratedFnParams} = decorate(fn);

declare const decorated_fn_params: DecoratedFnParams;
decorated_fn(decorated_fn_params);

Workaround:
const decorated_fn = decorate(fn);
type DecoratedFnParams = DecoratedParams<typeof fn>;

declare const decorated_fn_params: DecoratedFnParams;
decorated_fn(decorated_fn_params);

Alternative syntax, since this only seems useful for functions that run once:
import {decorated_fn, DecoratedFnParams} = decorate(fn);

(Shouldn't be a compatibility breaking change for the same reasons import ... = require() isn't).
Use Cases
My case is decorating a class with multiple generic parameters. Currently I have to do this:
interface Bar extends BaseBar {}
interface Baz extends BaseBaz {}
class Foo<Bar, Baz> extends BaseFoo {}

const DecoratedFoo = decorator(Foo);
type DecoratedFooBar = ExtractBar<DecoratedFoo>;
type DecoratedFooBaz = ExtractBaz<DecoratedFoo>;

But it could be just const {DecoratedFoo, DecoratedFooBar, DecoratedFooBaz} = decorator(Foo);
Checklist
My suggestion meets these guidelines:

[x] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[x] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, etc.)
[x] This feature would agree with the rest of TypeScript's Design Goals.

const {decorated_fn, DecoratedFnParams} = decorate(fn);

This is just the regular destructuring syntax, for when decorate returns an object with a shape like
{ decorated_fn: unknown; DecoratedFnParams: unknown; }

How would you disambiguate the two?

We could prefix types with type:
const {decorated_fn, type DecoratedFnParams} = decorate(fn);

What is the definition of decorate in these examples? What is fn ? BaseBaz ? BaseBar ? What is ExtractBaz ?

Honestly I was hoping that this issue is yet another duplicate!
I'll come up with an actual concrete example.

It probably is, I just can't point you at it until I understand the suggestion 😅

Found a simple and relatively common use case:
const guests = ['Alice', 'Bob', 'Charlie'];
type Guest = typeof guests[number];

This defines a list of values that can be used both at runtime and at type level.
Currently it takes two lines for each declaration. It would be nice to be able to write a function that also returned a type like this:
function List<T extends unknown[]>(x: T): T {
  export type Item = typeof x[number];
  return x;
}

import guests, {Item: Guest} from List(['Alice', 'Bob', 'Charlie']);

Yet another option:
function List<T extends unknown[]>(x: T): T {
  return x;
  return type typeof x[number];
}

// Guest is both a type `"Alice" | "Bob" | "Charlie"`
// and an array with all the values
const Guest = List(['Alice', 'Bob', 'Charlie']);

// example usage
function printGuests(me: Guest) {
  for (const guest of Guest) {
    console.log(guest === me ? 'me' : guest);
  }
}


Dependency installation errors occurring on win32

This is occurring very frequently in the wild:
Exit code:<PHONE_NUMBER>
Signal: null
Stdout: ➤ YN0000: ┌ Resolution step

Stdout: ➤ YN0002: │ swagger-parser@npm:10.0.3 doesn't provide openapi-types (p54c62), requested by @apidevtools/swagger-parser
➤ YN0000: │ Some peer dependencies are incorrectly met; run yarn explain peer-requirements <hash> for details, where <hash> is the six-letter p-prefixed code

Stdout: ➤ YN0000: └ Completed in 2s 840ms
➤ YN0000: ┌ Fetch step

Stdout: ➤ YN0013: │ @appland/appmap@npm:3.93.1 can't be found in the cache and will be fetched from the remote registry

Stdout: ➤ YN0019: │ @appland-appmap-npm-3.93.0-25f9ac707c-99e7cb8946.zip appears to be unused - removing

Stdout: ➤ YN0000: └ Completed in 11s 519ms

Stdout: ➤ YN0000: ┌ Link step

Stdout: ➤ YN0000: └ Completed in 5s 929ms

Stdout: ➤ YN0000: Done with warnings in 20s 663ms

at ChildProcess.<anonymous> (\src\services\nodeProcessService.ts:238:20)
at Object.onceWrapper (node:events:628:26)
at ChildProcess.emit (node:events:525:35)
at ChildProcess.emit (node:domain:489:12)
at Process.ChildProcess._handle.onexit (node:internal/child_process:293:12)```

This looks likely to be the culprit
Stdout: ➤ YN0001: │ Libzip Error: Failed to open the cache entry for d3-color@npm:3.1.0: Not a zip archive
at Ai.makeLibzipError (c:\Users\Dustin\AppData\Roaming\Code\User\globalStorage\appland.appmap\yarn.js:346:12685)
at new Ai (c:\Users\Dustin\AppData\Roaming\Code\User\globalStorage\appland.appmap\yarn.js:346:12126)
at j (c:\Users\Dustin\AppData\Roaming\Code\User\globalStorage\appland.appmap\yarn.js:461:2968)
at c:\Users\Dustin\AppData\Roaming\Code\User\globalStorage\appland.appmap\yarn.js:461:3032
at YS (c:\Users\Dustin\AppData\Roaming\Code\User\globalStorage\appland.appmap\yarn.js:422:11171)
at Vh.factory (c:\Users\Dustin\AppData\Roaming\Code\User\globalStorage\appland.appmap\yarn.js:461:3023)
at get baseFs [as baseFs] (c:\Users\Dustin\AppData\Roaming\Code\User\globalStorage\appland.appmap\yarn.js:346:39931)
at Vh.readFilePromise (c:\Users\Dustin\AppData\Roaming\Code\User\globalStorage\appland.appmap\yarn.js:346:37479)
at Da.readFilePromise (c:\Users\Dustin\AppData\Roaming\Code\User\globalStorage\appland.appmap\yarn.js:346:37486)
at ol.loadFile (c:\Users\Dustin\AppData\Roaming\Code\User\globalStorage\appland.appmap\yarn.js:422:32946)


Tenant cordoning state

Closes #307.

The desired additional printer columns are there, @bsctl!
NAME   STATE      NAMESPACE QUOTA   NAMESPACE COUNT   OWNER NAME   OWNER KIND   NODE SELECTOR                  AGE
oil    cordoned   3                 0                 alice        User         {"kubernetes.io/os":"linux"}   23s


[Tree] [Nested Set] Cannot delete root node

I have a nested set in a forest configuration (multiple roots), and I do not want to allow cascade deletion - A node can be deleted, only if it has no children:
    /**
     * @Gedmo\TreeRoot
     * @ORM\ManyToOne(targetEntity="Group", fetch="EAGER")
     * @ORM\JoinColumn(name="tree_id", referencedColumnName="id")
     */
    private $root;

In the controller, I do
                $em->remove($group);
                $em->flush();

And I get:
SQLSTATE[23000]: Integrity constraint violation: 1451 Cannot delete or update a parent row: a foreign key constraint fails (`db`.`groups`, CONSTRAINT `FK_F06D397078B64A2` FOREIGN KEY (`tree_id`) REFERENCES `groups` (`id`))

which is not too surprising really.
The obvious workaround was to run this first, for nodes without children:
$group->setRoot(null);
$em->flush();

but then I get Root cannot be changed manually, change parent instead
So how else can I delete a root node then?
I would probably expect to see this handled in the Nested listener inside processScheduledDelete where a node would be checked if having children and if not, set it's root to null before deleting it?

Never mind, there's actually a simple way around this. I just had to allow cascading on delete:
     * @ORM\JoinColumn(name="tree_id", referencedColumnName="id", onDelete="CASCADE")
Since there is another restricting FK on the parentId field, when there are children, the root cascade woudn't cascade anything.


Add implementation to InlinedS3BucketSpec endpoint() function

Description

Review Checklist

[ ] Code contains useful comments
[ ] (Integration-)Test cases added (or not applicable)
[ ] Documentation added (or not applicable)
[ ] Changelog updated (or not applicable)
[ ] Cargo.toml only contains references to git tags (not specific commits or branches)

Once the review is done, comment bors r+ (or bors merge) to merge. Further information

bors r+


Add :angry: for mean ∘ skipmissing

This comes from a recent gripe on slack. Somebody wants a shorter name for mean ∘ skipmissing and I figure this is a great opportunity to expand the user-base of this package.
While I'll admit :angry; is not ideal (:mean: would be better, were it a thing), I think this face looks a little mean: 😠, and I think the slight deviation is also appropriate as it reflects the semantic variation from traditional mean. If we used :mean:, folks might mistakenly think we were aliasing to Julia's mean.
Also, the motivation for this addition is a user's anger at not having a brief way of expressing mean ∘ skipmissing.

we need @😠 for making everything skipmissing, just like @.

Sounds great could you just add it to the func_to_emojis Dict in WatchJuliaBurn.jl such that it also gets rendered in the README?
Just add a new line as:
:(mean ∘ skipmissing) = (:(😠),)

Lovely! I didn't know that that dict could handle non-symbols on the LHS (e.g. sorting for the README & for equality in emojify).

Oh you're right... It might need some more hacking around. LGTM and I'll add some shady fix


cap&u - remove extra host lookup

Currently, assigned_to_target already checks the presence of
a tag for all parent nodes, including the ems_cluster.
No reason to run this twice.
This only performs ~1 fewer query per host. But removing the code tightens it up.
@gtanzillo and I saw this and thought it wouldn't be necessary. I added the test to ensure
and went from there
before

ms
bytes
objects
queries
query (ms)
rows
comments

2,190.8
82,528,460*
1,973,298
1,052
759.5
11,168
before

after

ms
bytes
objects
queries
query (ms)
rows
comments

2,249.4
82,486,408*
1,969,429
1,050
705.8
11,168
after

👍 LGTM


Record autoderef adjustments

cc #9475

This doesn't record adjustments for method receiver coercions yet as I'm unsure how to incorporate the special array unsizing for it.

bors r=flodiebold


limited use of [index:string]

Currently some function definitions like propEq use the index signature to type object arguments. This seems of limited use, because the supplied object needs to have this index signature explicitly defined. For example.
One of the type signatures of propEq is
propEq<T>(name: string, val: T, obj: {[index:string]: T}): boolean;

Using this function with var obj = {a: 1, b: 2} like
propEq('a', 1, obj); // Type error

is correct, however TypeScript complains with:
argument of type 'string' is not assignable to parameter of type 'number' pointing to the a parameter.
This is because obj  does not have a index signature. If we add this signature to obj explicitly the type error disappears:
var obj: {[index: string]: number} = {a: 1, b: 2}
propEq('a', 1, obj); // OK

This reason for this behavior is explained in http://stackoverflow.com/questions/22077023/why-cant-i-indirectly-return-an-object-literal-to-satisfy-an-index-signature-re#22077024
I think the requirement that only objects with an index signature can be used is too strict and that we therefore should limit the use of index signature to those cases where it is obviously (but I can not think of one at the moment).
Any thoughts on this issue?

I have just trapped on this, and it confused me a lot. I expected "props" to have the same signature as "prop"  except for input args. Why isn't it so? I believe loosing a bit of type safety is a affordable trade off here. At least there could be all possible overloads to choose from.
So as for me "props" should look like this.
props<T>(ps: string[]): (obj: any) => T[];

Hey, I'm sorry for chiming in, but I think there are a lot of issues here:

The issue originally report by @donnut seems to no longer apply as of typescript 1.8, as indicated by the referenced stackoverflow question. Hence, we can (and should?) have the [index:string]

1.a) I really don't understand the third to six lines of the type signature in propEq:
    propEq<T>(name: string, val: T, obj: any): boolean;
    propEq<T>(name: number, val: T, obj: any): boolean;
    propEq<T>(name: string, val: T): (...args: any[]) => boolean;
    propEq<T>(name: number, val: T): (...args: any[]) => boolean;
    propEq<T>(name: string): (val: T, ...args: any[]) => boolean;
    propEq<T>(name: number): (val: T, ...args: any[]) => boolean;

Shouldn't it be:
    propEq<T>(name: string, val: T, obj: any): boolean;
    propEq<T>(name: number, val: T, obj: any): boolean;
    propEq<T>(name: string, val: T): (obj:any]) => boolean;
    propEq<T>(name: number, val: T): (obj:any) => boolean;
    propEq<T>(name: string): (val: T, obj:any) => boolean;
    propEq<T>(name: number): (val: T, obj:any) => boolean;

Or better yet, obj:{[index:string]: any } ?

I totally agree with @yazla that props and prop should have related signatures, which is not the case, and it is confusing. But given the new capabilities of the typescript compiler I mentioned above, I actually think it is  prop which needs to be changed.

What do you think?

Hum, I stand corrected, this seems to only be fixed on typescript 1.9, which is now bleeding edge / alpha. So, I guess if we are still some months away of having this PR fixed in a more 'production ready' environment :(

That explains why my quick try wasn't able to reproduce it.

At this point, TS 2.0 is stable, so the original issue should no longer be a concern. propEq has been adjusted in line with @jcristovao's request by now.


Remove Inher from ulox

as part of #111 test removal of inher showed that it uses a chunk of code and 3 opcodes, but is not really found use in demos to this date

Coverage increased (+1.2%) to 94.863% when pulling d15f329ed202ab81ca731ce39651f768aa2ba6e7 on imagine-no-inher into f48751b09c00f479600b9e9a966daa3507968499 on main.


Automated cherry pick of #4683: hostman: distro name OpenWrt instead of OpenWRT

Cherry pick of #4683 on release/2.13.
#4683: hostman: distro name OpenWrt instead of OpenWRT

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.13@5944c9c). Click here to learn what that means.
The diff coverage is 0%.

@@              Coverage Diff               @@
##             release/2.13   #4687   +/-   ##
==============================================
  Coverage                ?   6.86%
==============================================
  Files                   ?     523
  Lines                   ?   89028
  Branches                ?       0
==============================================
  Hits                    ?    6114
  Misses                  ?   82406
  Partials                ?     508

Flag
Coverage Δ

#aFlag
6.86% <0%> (?)

Impacted Files
Coverage Δ

pkg/hostman/guestfs/fsdriver/linux.go
0% <0%> (ø)

pkg/hostman/guestfs/fsdriver/base.go
13.08% <0%> (ø)

Continue to review full report at Codecov.

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

/lgtm

/approve


If requested temporal range is larger than data range - then give back original files - requires alignment to be truncated into range

This is relevant to this code:
https://github.com/roocs/rook/blob/master/rook/director/alignment.py#L14-L30
The code needs to:

Get requested time range
Get full time range of dataset (i.e. all files)
If requested time range exceed full time range:

Return all original files as the aligned request.

Should be fixed in: https://github.com/roocs/rook/commit/63b9f25c354a341fb02d1b2013b39f9972d641ed
@ellesmith88 please merge into your "resolve_urls" branch.

Should be fixed in: https://github.com/roocs/rook/commit/63b9f25c354a341fb02d1b2013b39f9972d641ed
@ellesmith88 please merge into your "resolve_urls" branch.

Hi @agstephens, @ellesmith88 and @cehbrecht,
Does rooki know how to handle the 'original files' response? Earlier this week I was able to pass a temporal subset that was larger than the data, but I now get a "Sorry, process failed" error from rooki (version 0.3.0). I ran the following experiments:

Temporal subset much larger than the data range (failed)
Temporal subset that exactly matches the data range (failed)
Temporal subset very slightly smaller than the data range (succeeded)

Let me know if you'd like more information.
Many thanks!
James

Hi @agstephens, @ellesmith88 and @cehbrecht,
Does rooki know how to handle the 'original files' response? Earlier this week I was able to pass a temporal subset that was larger than the data, but I now get a "Sorry, process failed" error from rooki (version 0.3.0). I ran the following experiments:

Temporal subset much larger than the data range (failed)
Temporal subset that exactly matches the data range (failed)
Temporal subset very slightly smaller than the data range (succeeded)

Let me know if you'd like more information.
Many thanks!
James

@JamesVarndell I think rooki should simply represent the original files as those available from URLs on the data node. In my tests, I get files back from each response (so the server is working).
This works for me:
http://rook-wps1.ceda.ac.uk/wps?service=WPS&version=1.0.0&request=Execute&identifier=subset&status=true&storeExecuteResponse=true&DataInputs=collection=c3s-cmip6.ScenarioMIP.INM.INM-CM5-0.ssp245.r1i1p1f1.Amon.rlds.gr1.v20190619;time=1900-01-01/2500-12-30
Do you get a valid response from it?

Implemented here:
https://github.com/roocs/rook/blob/master/rook/director/alignment.py


we should considering always pushing an OCI manifest config blob with "{}" rather than empty

Definitely do:

allow a non-empty config and an unrecognized config media type when we download a module (currently we throw)
allow multiple module layers when we download (currently we throw)

Consider (would break previous Bicep versions from being able to download newly-published modules, including those in the public registry):

Write "{}" as the config contents
Change config media type to application/vnd.oci.empty.v1+json?
We could consider a phase out with warning approach...

Context:
I discovered that ACR requires the sources manifest config to be valid JSON or it won't recognize it as an attachment.  Upon asking the ACR guys if this was expected, I got this answer:

Hey Stephen, empty manifest support is somewhat inconsistent industry wide. Configs have only recently become that free as of the most recent RC3 image spec and we have not yet implemented support for that at the config level. I'll make sure to add it as a workitem nonetheless, thanks for bringing it to our attention. Regardless we would not suggest for the use of actually empty  configs or layers for that matter though since it would not be portable as different registries have different levels of support for empty layers/config. The general guidance is to use the empty descriptor https://github.com/opencontainers/image-spec/blob/main/manifest.md#guidance-for-an-empty-descriptor as you have done in the example above if no config is wanted. RC3 Also allows for the complete removal of the config field but we haven't completed support for that (Its an ongoing change one of my colleagues is working on though so that is also an option in the future). P.S did you observe this behaviour only with the empty config or also with a binary one?

We will need to emit a warning about the breaking change as the first step.

related branch:  sw/allow-multiple-layers

The ACR guys suggested the following changes:
{
"schemaVersion": 2,
"mediaType": "application/vnd.oci.image.manifest.v1+json",   <<<<<<<<<<< ADDED
"artifactType": "application/vnd.ms.bicep.module.artifact",
"config": {
"mediaType": "application/vnd.ms.bicep.module.config.v1+json",
"digest": <digest of {}>,  <<<<<<<<<<<< CHANGED
"size": 0,   <<<<<<<<<<<<<<< CHANGED FROM 0 TO 2 (pointing to "{}" blob)
"annotations": {}
},
"layers": [
{
"mediaType": "application/vnd.ms.bicep.module.layer.v1+json",
"digest": "sha256:fa703356d827467cede2e38c00501fc9fc8f66eba3e6e13a0a3f37adae50528a",
"size": 117570,
"annotations": {}
}

SUMMARY OF CHANGES
Now

[ ] allow a non-empty config and an unrecognized config media type when we download a module (currently we throw)
[ ] allow multiple module layers when we download (currently we throw)
[ ] obsolete warning?

Later (breaking changes)

[ ] Write "{}" as the config contents instead of null/empty
[ ] Change manifest media type to application/vnd.oci.image.manifest.v1+json instead of null

See https://github.com/Azure/bicep/pull/12057 for breaking change details


[zattoo] Add support for zattoo.com and its partner sites

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2017.10.29. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2017.10.29

Before submitting an issue make sure you have:

[x] At least skimmed through the README, most notably the FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[x] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):
The videos are usually only available for like 7 days! So the links probably won't work, but the scheme should be clear.

Single video: https://zattoo.com/watch/3plus/130760870-bauer-ledig-sucht
Live stream: https://zatto.com/watch/srf1

Note that youtube-dl does not support sites dedicated to copyright infringement. In order for site support request to be accepted all provided example URLs should not violate any copyrights.

Description of your issue, suggested solution and other information
zattoo.com is a big TV streaming service provider. With a free account you can only watch the live program, with a paid account you get access to the replay functionality. All the video should be available for at least 7 days. You can make a 30 days premium account for free on zattoo.com. It's possible that some content is geo restricted.
Zattoo has several partners, which use exactly the same interface:

https://mobiltv.quickline.com
https://www.netplus.tv
https://tvplus.m-net.de

Would it be possible to get support for zattoo.com and its partners into youtube-dl?
I have written a information extractor for that. As soon as it's finished, I will open a pull request.

This seems broken atm, worked for me when I checked out the zattoo branch a couple of weeks ago, now I only get an error.
youtube-dl -v "https://zattoo.com/watch/srf_zwei/131484236-feuer-und-flamme/99350971/1511253900000/1511256300000/-300000" -o feuer_20.11.ts
[debug] System config: []
[debug] User config: [u'--username', u'PRIVATE', u'--password', u'PRIVATE']
[debug] Custom config: []
[debug] Command-line args: [u'-v', u'https://zattoo.com/watch/srf_zwei/131484236-feuer-und-flamme/99350971/1511253900000/1511256300000/-300000', u'-o', u'feuer_20.11.ts']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2017.10.29
[debug] Python version 2.7.9 - Linux-3.16.0-4-amd64-x86_64-with-debian-8.10
[debug] exe versions: ffmpeg 3.2.5-1, ffprobe 3.2.5-1
[debug] Proxy map: {}
[Zattoo] Downloading https://zattoo.com
[Zattoo] Say hello
[Zattoo] Logging in
[Zattoo] 131484236: Downloading video information
[Zattoo] 131484236: Downloading dash formats
ERROR: Unable to download JSON metadata: HTTP Error 403: Forbidden (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl-2017.10.29-py2.7.egg/youtube_dl/extractor/common.py", line 503, in _request_webpage
    return self._downloader.urlopen(url_or_request)
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl-2017.10.29-py2.7.egg/youtube_dl/YoutubeDL.py", line 2195, in urlopen
    return self._opener.open(req, timeout=self._socket_timeout)
  File "/usr/lib/python2.7/urllib2.py", line 437, in open
    response = meth(req, response)
  File "/usr/lib/python2.7/urllib2.py", line 550, in http_response
    'http', request, response, code, msg, hdrs)
  File "/usr/lib/python2.7/urllib2.py", line 475, in error
    return self._call_chain(*args)
  File "/usr/lib/python2.7/urllib2.py", line 409, in _call_chain
    result = func(*args)
  File "/usr/lib/python2.7/urllib2.py", line 558, in http_error_default
    raise HTTPError(req.get_full_url(), code, msg, hdrs, fp)

Or could that be something on my end? I see it's timeouting somewhere, but I can watch zattoo and recordings via web.

@troych Unfortunately I don't have a zattoo premium account, so I can not test this issue on my site. I have an account for a zattoo partner though, but there I have only 7 days replay available, and your video seems to older than that. Have you encountered the same issue for a recent video?

Ye the video in question is a recording, not via replay. Just tested a file within the 7 day replay period and there it works indeed. So I guess the recording bit is broken. I guess I could give you access to my zattoo account if that would help? Not sure how that works with geo restrictions and the likes though, since I'm situated in Switzerland.

@troych I was not aware that there is a recording feature for zattoo, because I use the zattoo partner Quickline and they don't offer this. I have just seen that I can record up to 20 shows with my free zattoo account. I am recording something now and I will test this recording with this information extractor afterwards.

@troych I have added support for zattoo, you just need to pull my latest commit https://github.com/rg3/youtube-dl/pull/14676/commits/ce8053b6464a3c02d1fd1c7a9d32869bb99ecb0a. Please test it with your premium account and let me know if everything works as expected.

Just had a quick go, recording downloading as we speak. Thank you very much for the help and fix. :)

I have a premium account. Downloading my Zattoo recordings works using goggle's commit ce8053b. Thanks!

@troych @dstftw Do we have any news about this?

Haven't received any credentials.

@dstftw I sent you an email on the 4th, never got an answer though?

@goggle Can you get in contact with @dstftw, I'll be happy to lend my account for testing but I can't seem to get a response via email.

@troych Sure, I can try. Can you PM me your email address, so I can give it to dstftw, that he knows who you are?

I guess I made it quite clear: if you want to provide account credentials send account credentials not requests to contact somebody at somewhere.

Well, I sent you an email asking you about the need for vpn since zattoo is geo restricted. I was just looking for a quick sign of life so I can change my password to something I can send out. I did not want to make this overly complicated, and wasn't intending to make your access any harder then it needs to be.
I hope you understand that I didn't just want to send account credentials without knowing if a) they're actually useful to you because of geo restrictions and b) the email I wrote to is actually in use.
Sorry for any misunderstandings.

@dstftw I've sent credentials to your gmail account, if I can provide anything else let me know.


Extract POSIX implementation of Process and Signal to a portable API

Prepare Process API for Windows implementation. After discussion in #8518 i extract the system implementation which provide
the following private interface (all methods are protected):
# for current running program
self.exit_system(status : Int32) : NoReturn
self.pid_system : Int64
self.ppid_system : Int64
self.fork_system(&) : Process
self.fork_system : Process?
self.chroot_system(path : String) : Nil
self.times_system : Tms

# for Process instance
terminate_system : Nil
kill_system : Nil
close_system: Nil
exist_system? : Bool
signal_system(signal : Signal) : Ni
wait_system : Nil
pgid_system : Int64
create_and_exec(command, *args, ...) : Int64

# for given pid
self.exists_system?(pid : Int64)
self.pgid_system(pid : Int64) : Int64

# util
self.prepare_shell_system(command : String, args : Enumerable(String)) : {String, Enumerable(String)}

Public API design for Process
There need to be some changes to remove POSIXisms and make it portable.

kill method is renamed to signal move to #8642
new methods terminate (graceful), interrupt (immediately)

Precursor PR: #8518

@ysbaddaden it's done, please look at it.

Oh, and thank you for spending so much time on making this super clean!

@RX14 Let's avoid holding things up and keep them hanging.
IMO: the abstraction here is good. It doesn't remove any functionality and will be enough to implement Process on Windows. We can make it perfect afterwards, deciding in their own dedicated issues, whether we keep fork, chroot or include (e)(uid|gid) or not.

What does exists_system mean?  The system part of the method name is a bit confusing to myself... :)

@RX14 Process.kill wasn't removed but renamed to Process.signal which is a better name. No need to open repetitive PR.

@ysbaddaden I agree - but it still needs to be in seperate PRs.

Ok  @RX14, @straight-shoota i move kill method renaming to new PR #8642. But because of this a our intention with @ysbaddaden to have portable API to stopping processes i rename proposed kill method that immediately terminate process to interrupt. Otherwise it would be super unclear until we do remove original kill methods. The more so that someone already suggested to make it only deprecated in #8642 (which I expected).

@jan-zajic I'd rather just have no kill or interrupt or terminate, just signal(:kill) and signal(:interrupt). On windows, signal would only be able to send sigkill, an attempt to send sigint would raise. This is similar to how go approaches it.
Please no user-visible API changes in this PR.

@RX14 i rearranged it as you wish. But i insist on to have some methods to stop Process (no matter how they are named, it can be discussed). We try to create portable API (at least @ysbaddaden and i), that can be implemented on every platform. If you can create Process, you must be able to stop it. Do not confuse it with that signal stuff. It's only implementation detail on unix. Stopping or killing most of the processes on Windows, for example, have nothing to do with signal API. When you mention go for example, it also has Kill() method on Process that causes the Process to exit immediately in platform specific way. But Go lang is not a good example of pretty apis at all, please let's not be influenced by previous compromises of other teams.

@bcardiff builds now starts to fail on:

HTTP::Cookie::Parser expired? not expired
Failure/Error: parse_set_cookie("bla=1; expires=Thu, 01 Jan 2020 00:00:00 -0000").expired?.should eq false
Expected: false
got: true

P.S. happy New Year to everyone involved in Crystal Language!

Stopping or killing most of the processes on Windows, for example, have nothing to do with signal API

That is true, but that does not mean that the API for killing a process on windows cannot be exposed as signal(:kill), which is then translated to TerminateProcess. This is how go handles it, and is a subjective API design issue. I'm fine with either way.

@jan-zajic You need to rebase on master in order to get green CI builds.


 This feature will allow users to load data only when they need it, which can improve performance and reduce bandwidth usage. The lazy loading feature is implemented using the splashscreen component. This pattern allows the data to be loaded in the background while the user is still interacting with the page. This can provide a better user experience, as the user does not have to wait for the data to load before they can start using the page.

hey @M-A-N-I-K please close this PR & create a new fresh PR we refactored our code & added few features so please fetch upstream


fix(ffe-form-react): updated type for input

Input now extends React.DetailedHTMLProps to make it compatible with external libraries.
We have started to use react-hook-form and somehow the old definition was not enough for this component, the ref-definition in the React.InputHTMLAttributes did not accecpt ref={register()} (register from react-hook-form).

The library is changed now, so I close this PR.


⚠ ClamAV has degraded performance

In fc01c08, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 580 ms

Resolved: ClamAV performance has improved in ed904b2.


1.0.8 aar in maven repository have problem

I wrote a project using your lib, when package release variant, proguard fail, and print the following errors:

Warning: com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager$ScrollListener: can't find referenced method 'int access$102(com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager,int)' in program class com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager
Warning: com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager$ScrollListener: can't find referenced method 'float access$202(com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager,float)' in program class com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager
Warning: com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager$ScrollListener: can't find referenced method 'float access$300(com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager)' in program class com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager
Warning: com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager$ScrollListener: can't find referenced method 'int access$400(com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager,int,int)' in program class com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager
Warning: com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager$ScrollListener: can't find referenced method 'int access$500(com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager)' in program class com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager
Warning: com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager$ScrollListener: can't find referenced method 'int access$100(com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager)' in program class com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager
Warning: com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager$ScrollListener: can't find referenced method 'java.util.List access$600(com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager)' in program class com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager
Warning: com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager$ScrollListener: can't find referenced method 'android.support.v7.widget.RecyclerView$OnScrollListener access$700(com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager)' in program class com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager

I thought maybe you forgot to clean project before assembling the aar file.
Hope this bug could be fixed!

Your RecyclerView version？？

22.2.1

I have the same problem too
22.2.1 RecyclerView version

I change the version back to 1.0.6 is OK

I encountered the same problem as well. Tried using v22.2.0 for the RecyclerView support library, but the problem still occurs. Had to resort to downloading and adding the module manually in Android Studio instead of using the Maven repository. Thankfully that works :)

Does 1.0.10-SNAPSHOT  resolved this problem??

@filbertteo @wonTonx @lyrixliu

@lsjwzh just tested. still has the problem. build log is as follows:
Warning: com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager$ScrollListener: can't find referenced method 'int access$102(com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager,int)' in program class com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager
Warning: com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager$ScrollListener: can't find referenced method 'float access$202(com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager,float)' in program class com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager
Warning: com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager$ScrollListener: can't find referenced method 'float access$300(com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager)' in program class com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager
Warning: com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager$ScrollListener: can't find referenced method 'int access$400(com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager,int,int)' in program class com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager
Warning: com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager$ScrollListener: can't find referenced method 'int access$500(com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager)' in program class com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager
Warning: com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager$ScrollListener: can't find referenced method 'int access$100(com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager)' in program class com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager
Warning: com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager$ScrollListener: can't find referenced method 'java.util.List access$600(com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager)' in program class com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager
Warning: com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager$ScrollListener: can't find referenced method 'android.support.v7.widget.RecyclerView$OnScrollListener access$700(com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager)' in program class com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager
Warning: com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager$ScrollListener: can't find referenced method 'java.util.List access$800(com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager)' in program class com.lsjwzh.widget.recyclerviewpager.RecyclerViewPager

@filbertteo Could you post your build.gradle?

repositories {
    mavenCentral()
    maven {
        url "https://oss.sonatype.org/content/repositories/snapshots"
    }
}

dependencies {
    // RecyclerViewPager library
    compile('com.lsjwzh:recyclerviewpager:1.0.10-SNAPSHOT')
}

@filbertteo  your build options? ex.... lintOption and so on....

@filbertteo @wonTonx @lyrixliu
Adding the line below into proguard  file will resolve it.
-dontwarn com.lsjwzh.widget.recyclerviewpager.**
It is just a warn info.
http://stackoverflow.com/questions/23883028/how-to-fix-proguard-warning-cant-find-referenced-method-for-existing-methods

i am getting - failed to resolve failed to resolve com.github.lsjwzh.RecyclerViewPager:lib:v1.1.0


mvcc: skip accumulated locks using seek in forward scanner

What is changed and how it works?

Issue Number: Ref #13694
What's Changed:

This commit adds support of skipping versions with a seek to the LatestKvPolicy
of the forward scanner.

Now we don't add this to other policies because they are usually used for large
range of scanning. So, the influence of accumulated locks of certain keys should
be amortized.

Related changes

PR to update pingcap/docs/pingcap/docs-cn:
Need to cherry-pick to the release branch

Check List
Tests

Unit test

Release note
None

/merge

/test


[Grafana 9.1.2, plugin 4.2.10] context deadline exceeded (Client.Timeout exceeded while awaiting headers)

Hello. We have a problem with Grafana and Zabbix 5.4.x (in DirectDB connection to PostgreSQL configuration)
I have a bunch of similar hardware and dashboard for them. About 150 total. Every device is working well, but on two of them I get error "context deadline exceeded (Client.Timeout exceeded while awaiting headers)" in dashboard.
They are exactly the same with same template etc. The only difference is that they have about 16000 metrics in Zabbix instead of about 5000-10000 metrics for every other devices.
They are multi-slot GPON-equipment with different number of active clients so the problem devices are the most huge devices.
I guess there is some bottlneck between grafana/plugin and zabbix/postgresql, and if device has too much metrics - grafana cannot take them. And I couldn't found any limitations in configs or logs. What could it be?
The same problem was with Grafana 8.x and older versions of plugin, I just tried to update Grafana/Plugin to avoid this issue.

Well, since no one answered. The problem was in Zabbix settings: Administration - General - Other parameters - Communication with Zabbix server timeouts. I raised Network timeout for script execution parameter and the problem solved


⚠ TADA Wallet Web has degraded performance

In 5f9f6d9, TADA Wallet Web ($WALLET_WEB_URL) experienced degraded performance:

HTTP code: 200
Response time: 8791 ms

Resolved: TADA Wallet Web performance has improved in 4c1badf.


MIP starts - Starting from a solution

How can I supply hints to help a solver find an initial solution similar to addMIPStart in CPLEX Concert Technology?

CPLEX hinting is currently not implemented, it is supported for SCIP, Gurobi, and BOP (our own binary optimizer).
We do welcome contributed code :-)


Small improvements

:tada: This PR is included in version 1.1.3 :tada:
Your semantic-release bot :package::rocket:


⚠ GitLab Server has degraded performance

In 8a01a52, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 11709 ms

Resolved: GitLab Server performance has improved in de3dee5.


feat(time-picker): support titleColor

Checklist
List of tasks you have already done and plan to do.

[x] Fix linting errors
[x] Tests have been added / updated (or snapshots)

Change information
Describe your modifications here.
Issues
The issues you want to close, formatted as close #1.
Related Links
Links related to this pr.

@haoziqaq  I had updated the default value of color and titleColor in docs.


Updated github issue numbers on gCNV TODO comments

No review is necessary here. I'll wait for the tests and merge.

@mbabadi Didn't you get the email this morning that merges into master in either gatk-protected or gatk are frozen until further notice?


UI Chart - Add x/y axes labels options

Description

Add x-axis label property
Add y-axis label proeprty

Related Issue(s)
Closes #382
Examples:

Thanks Joe and CSTNS!  Your responsiveness is incredible!


Fix bug of equals method in address

Fixed the bug where an address with the same value of empty address will be equal to an empty address

LGTM

LGTM


Don't delete existing boards.

If there's an existing board don't delete it, just overwrite the contents.
This prevents open tabs (including pins) from closing when the board is updated.
Fixes #21

Thanks so much for this fix! I wondered if an update function like that existed when I made this :)


Missing file

Despite this file is not mandatory to build the ext, it is part of the project sources, and may be required for downstream (if patch need to be applied on this file)
Minimum pear version is the one with PHP 7 support.

Despite this file is not mandatory to build the ext, it is part of the project sources, and may be required for downstream (if patch need to be applied on this file)

Oh - I wasn't sure of the stance on packaging those since it's new functionality, but it's convenient to have. For example, the ext_functions and class registry in _arginfo.h is only generated when using gen_stubs.php from 8.1 and newer.


luci-theme-material: upgrade jquery

This upgrade jquery to latest version and solve 404 error for the useless .map file. Also it's 85kb instead of 97kb
Signed-off-by: Ansuel Smith<EMAIL_ADDRESS>

merged, thanks


[AUTOPATCHER] Patches CVE-2021-37600 for util-linux

        Patches CVE-2021-37600 for util-linux.
        This patch and PR was automatically generated and needs human verification before merging.

Issue already manually resolved.  Closing autopatch issue.


Add github sponsor button, and link with patreon

Displaying a sponsor button in your repository
You can add a sponsor button in your repository to increase the visibility of funding options for your open source project.

Patreon
patreon: USERNAME

See: https://help.github.com/en/github/building-a-strong-community/displaying-a-sponsor-button-in-your-repository

Thanks for the heads up, I've been meaning to do this.
Done!

Thank you so much :+1:


Add fail2ban and unattended-upgrade for Ubuntu

Hello, wouldn't it be nice to do basic hardening to the ubuntu nodes (and others also) ? Like installing fail2ban and making sure unattended-upgrades are activate for security patches? @geerlingguy I know you have tons of playbooks for this, what do you think?

Hi, my optinion: it's out of scope, these playbooks are of build and maintain k3s cluster. I don't say hardening is not necessary but it can and have to be done with other playbooks.

Agreed; I was going to mention that this playbook benefits from the simple focus of setting up K3s and nothing else. It is a good playbook that can be integrated into another one that does any hardening and also deploys your applications into the cluster.

Ok I understand, thanks!


Implement "Client" functionality

Should be able to fetch TUF metadata from various locations. Needs to support:

local FS
HTTP/S
In memory (for testing)

for example
trait Client {
  /* functions */
}

struct HttpClient {}
impl Client for HttpClient {}

struct FileSystemClient {}
impl Client for FileSystemClient {}

struct InMemoryClient {}
impl Client for InMemoryClient {}

These would be crate private and be instantiated based off URL of the Tuf struct when updates are required.

Because of how the Tuf struct works, doing this via a client makes it a many step process to:

parse bytes
verify bytes
pull out inner structure (meta)

And so on.
What we really want is: "ability to load files from the local FS, remote via HTTP, and from an in memory store (for testing)."

Update: with the addition of the tuf-test-vectors submodule, there only needs to be FS and HTTP/S implementations (for now).

HTTP client is in progress on branch hyper.


yo command to accept cwd uri and support explorer context menu

This mainly allows generating sub projects with yeoman in mono repos, maven child projects, etc.
Also minor cleanups were done, added category to commands and capitalize titles.

@bfitzpat could you please review/comment on this PR?

@bfitzpat could you please review/comment on this PR?

Yes, adding to my list today! Thanks for the PR!

My only other question is how this will affect the current workflow.
Right now the user simply uses the command palette to trigger a Yeoman flow. When triggered, it takes the currently open folder in the editor (i.e. workspace.rootPath) and calls the Yeoman cli from there.
With this new method, we're forcing users to right-click on the folder they want to use as the root, effectively cutting off the way this tool used to work.
Is there a way for us to handle both cases?

Both ways still work. There is a condition and if the argument uri is not truthy the old flow is run.
let cwd = currentFolderUri ? currentFolderUri.fsPath : workspace.rootPath;
I'll mark the argument as optional as well.

I update the readme.


Beehive and Wither Armor: Special Abilities missing

HI I encounter an issue with certain armors: Beehive and Wither Armor.
Problem: The Special Abilities do not apply/overwrite when being replaced from the previous armor. (The previous armor abilities are still active)
I have attached a few images.
I'm not sure what's causing the issue since I can overwrite other unique armors; like the Halloween items with no issues.
I am doing something wrong or is this a bug?
Thanks

My bad. I didn't notice I have to change the armor properties as well. That is cool!   Thank you so much for the tool!
I will close the issue as I answered my own question.

My bad. I didn't notice I have to change the armor properties as well. That is cool!   Thank you so much for the tool!
I will close the issue as I answered my own question.

I apologize that the UI wasn't clear enough. Alas, I am not a designer. Thanks for the feedback!

I apologize that the UI wasn't clear enough. Alas, I am not a designer. Thanks for the feedback!


Fix NumPy(Minimum)EigensolverResult typehints

Summary
Small fix of the TypeHints in the NumPyEigensolverResult and NumPyMinimumEigensolverResult from np.ndarray to list[Statevector] and Statevector, respectively.
Details and comments

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

5 of 5 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.007%) to 84.543%

Totals

Change from base Build<PHONE_NUMBER>:
0.007%

Covered Lines:
62457

Relevant Lines:
73876

💛  - Coveralls


Which IDE for Lua?

Hi all,
Recently, I turn to Torch, I am a newer. I want to know which tool you use in your project? I want to quickly to know it so that I can use Torch. Thank you for your advices.

The only thing like an IDE that I use for Lua/Torch is iTorch, mostly as a superpowered interactive shell for development. For text editing I just use vim.

Ok. Thanks a lot @jcjohnson


Make dimension search work with ClickHouse tables

Not able to see search option with clickHouse backed dashboards.

@nishantmonu51 Is this related to https://github.com/rilldata/rill/issues/5082 ?

Tested with adbids data. Couldnt reproduce this. If we face this again please send the project as well.


Add automatic numeric conversion for HGET and HGETALL

Where possible, the values returned by HGET and HGETALL should be converted to numeric format

@alex6058 HGET is done. HGETALL returns any data types in the fields and can't be converted easily. If you do grafana Transformation, you can get numeric first/last fields.


Add .tex to text_extensions

I added .tex to the list of text_extensions. Useful when creating a LaTeX layout (I stumbled upon this when creating a latexmk filter for nanoc).

Thanks!


CUDA compatibility issue

Getting this issue, thoughts?
/Tools/kohya_ss-21.8.2/venv/lib/python3.10/site-packages/bitsandbytes/cuda_setup/paths.py:93: UserWarning: /Tools/anaconda3-Fonchai/envs/Fonchai did not contain libcudart.so as expected! Searching further paths...
warn(
/Tools/kohya_ss-21.8.2/venv/lib/python3.10/site-packages/bitsandbytes/cuda_setup/paths.py:27: UserWarning: WARNING: The following directories listed in your path were found to be non-existent: {PosixPath('/Tools/kohya_ss-21.8.2/venv/lib/python3.10/site-packages/cv2/../../lib64')}
warn(
/Tools/kohya_ss-21.8.2/venv/lib/python3.10/site-packages/bitsandbytes/cuda_setup/paths.py:105: UserWarning: /Tools/kohya_ss-21.8.2/venv/lib/python3.10/site-packages/cv2/../../lib64: did not contain libcudart.so as expected! Searching further paths...
warn(
/Tools/kohya_ss-21.8.2/venv/lib/python3.10/site-packages/bitsandbytes/cuda_setup/paths.py:27: UserWarning: WARNING: The following directories listed in your path were found to be non-existent: {PosixPath('https'), PosixPath('//nodejs.org/dist')}
warn(
/Tools/kohya_ss-21.8.2/venv/lib/python3.10/site-packages/bitsandbytes/cuda_setup/paths.py:27: UserWarning: WARNING: The following directories listed in your path were found to be non-existent: {PosixPath('local/Fonchai'), PosixPath('@/tmp/.ICE-unix/2507,unix/Fonchai')}
warn(
/Tools/kohya_ss-21.8.2/venv/lib/python3.10/site-packages/bitsandbytes/cuda_setup/paths.py:27: UserWarning: WARNING: The following directories listed in your path were found to be non-existent: {PosixPath('/org/gnome/Terminal/screen/0426ca50_bb8f_45d2_b907_b6a91d5218e9')}
warn(
/Tools/kohya_ss-21.8.2/venv/lib/python3.10/site-packages/bitsandbytes/cuda_setup/paths.py:27: UserWarning: WARNING: The following directories listed in your path were found to be non-existent: {PosixPath('1'), PosixPath('0')}
warn(
/Tools/kohya_ss-21.8.2/venv/lib/python3.10/site-packages/bitsandbytes/cuda_setup/paths.py:27: UserWarning: WARNING: The following directories listed in your path were found to be non-existent: {PosixPath('/run/user/1000/fnm_multishells/24930_1690415446698')}
warn(
/Tools/kohya_ss-21.8.2/venv/lib/python3.10/site-packages/bitsandbytes/cuda_setup/paths.py:27: UserWarning: WARNING: The following directories listed in your path were found to be non-existent: {PosixPath('/etc/xdg/xdg-ubuntu')}
warn(
/Tools/kohya_ss-21.8.2/venv/lib/python3.10/site-packages/bitsandbytes/cuda_setup/paths.py:27: UserWarning: WARNING: The following directories listed in your path were found to be non-existent: {PosixPath('/tmp/torchelastic_0dn8tzln/none_9dgt79nw/attempt_0/0/error.json')}
warn(
CUDA_SETUP: WARNING! libcudart.so not found in any environmental path. Searching /usr/local/cuda/lib64...
CUDA SETUP: CUDA runtime path found: /usr/local/cuda/lib64/libcudart.so
CUDA SETUP: Highest compute capability among GPUs detected: 8.6
CUDA SETUP: Detected CUDA version 122
CUDA SETUP: TODO: compile library for specific version: libbitsandbytes_cuda122.so

Dont know if you have windows system ...
on Windows i had to search 24hours for solution regarding conflict between CUDA and bitsandbytes.
For me the following windows version of bitsandbytes helped:
pip install bitsandbytes-windows
search on github ... there is a repository for bitsandbytes-windows

Thanks!  I tried updating the bitsandbytes package, and also creating symbolic links to libbitsandbytes_cuda122.so in multiple directories.... right now trying to downgrade to cuda 11.8

OR you are  Linux distribution (Ubuntu, MacOS, etc.)system ,AND CUDA Version: 11.X.
Bitsandbytes can support ubuntu.（yuhuang）
1  open folder J:\StableDiffusion\sdwebui，Click the address bar of the folder and enter CMD
or WIN+R, CMD  。enter，cd /d J:\StableDiffusion\sdwebui
2  J:\StableDiffusion\sdwebui\py310\python.exe -m pip uninstall bitsandbytes
3 J:\StableDiffusion\sdwebui\py310\python.exe -m pip uninstall bitsandbytes-windows
4 J:\StableDiffusion\sdwebui\py310\python.exe -m pip install https://github.com/TimDettmers/bitsandbytes/releases/download/0.41.0/bitsandbytes-0.41.0-py3-none-any.whl
Replace your SD venv directory file（python.exe Folder） here（J:\StableDiffusion\sdwebui\py310）


Minor edit: Capitalisation

I always try to stay consistent how it's in the screenshots or on the original page,
If you change it like it's on the original wiki then i will approve and merge it.

Both earlier ones were wrong. Corrected now


[FLINK-33029][python] Drop python 3.7 support

What is the purpose of the change

Python 3.7.17 is the latest security patch of the 3.7 line. This version is end-of-life and is no longer supported: https://www.python.org/downloads/release/python-3717/
Miniconda 4.7.10 is released on 2019-07-29 which is 4 years old already and not supporting too many architectures (x86_64 and ppc64le)
The latest miniconda which has real multi-arch feature set supports the following python versions: 3.8, 3.9, 3.10, 3.11 and no 3.7 support

In this PR  python 3.7 support dropped and upgrade miniconda to the latest version which opens the door to other platforms + python 3.11 support
Brief change log

Python 3.7 support dropped
Upgrade miniconda to the latest version
Add arm support
Upgrade mypy==1.5.1 (latest)
flink-python/apache-flink-libraries/setup.py python versions fix (still 3.6 was there which is dropped long time ago)

Verifying this change

Existing python tests.
Manually

dev/build-wheels.sh

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

@flinkbot run azure

cc @gyfora @mbalassi

Not sure how to test tools/releasing/create_binary_release.sh which is called by the nightly build. I'm not able to test it locally but the mentioned job only executed when a PR is merged. Honestly no idea 🤔

Asked it on the mailing list: https://lists.apache.org/thread/7rk9g921nl0mc4y8b2xsnpqd6b1nh17z

@gaborgsomogyi @gyfora is this mergable?

Asked in the dev discussion thread whether somebody has addition, nothing arrived so I would say yes. I'm intended to merge it soon...

Rebase to the latest master to trigger azure.


DeepL Translator

Ref: Conversation #6012 / Walter

That would be awesome since it is fast and really good. I have played around a bit using the html interface but there's also an undocumented json.
I have however not been able to figure out yet if there is a way to display the html output in the popclip response overlay. It does not seem to support that? It would be a nice capability since with translators the value does not come from just returning the the top hit word but rather the whole context such as: https://dict.deepl.com/english-german/search?ajax=1&query=house&source=english&onlyDictEntries=1&translator=dnsof7h3k2lgh3gda
Having that opened in a browser tab is too intrusive as it breaks the workflow on the same window in a popclip overlay would be just right.

I have hacked together a version using python3 and the jsonrpc api. Good enough for my personal use. I already like it much more than the Microsoft powered one. The python3 dependency does probably not make it suitable for the masses though. I can make it available if there's interest.

Ref: Conversation #6021 / Tomasz

My solution is available here:
https://github.com/binaryanomaly/DeepL.popclipext
if anybody is in need of a pragmatic solution before an official extension is made available.


[1.28] sap.m.Dialog: way to disable scrolling completely

In sap.m.Dialog there is a special handling for Dialogs containing single scrollable controlls. The function for checking this has the following comment:
 * If a scrollable control (sap.m.NavContainer, sap.m.ScrollContainer, sap.m.Page) is added to dialog's content aggregation as a single child or through one or more sap.ui.mvc.View instances,
 * the scrolling inside dialog will be disabled in order to avoid wrapped scrolling areas.
 *
 * If more than one scrollable control is added to dialog, the scrolling needs to be disabled manually.

The last line states that there is a way to disable scrolling manually, but it seems like there is no way to achieve the same behavior without setting the "sapMDialogWithScrollCont" class by hand which seems to be a pretty unstable things since the name of internal css classes might change.
Heres a Plunker with my exact problem: http://plnkr.co/edit/4aHmI0iIVeJWAqPsUaeI
As you can see, the Splitter does not behave correctly and is even shrinking. That might make users laugh, but does not provide a good ui to be productive ;)
I'd like if there's a way to force this "don't care about scrolling"-mode on dialogues.
Setting horizontal- & verticalScrolling = false does not help. What about a new Flag "enableScrolling" which defaults to true? That would match already existing behavior in views.

Hi themash,
thanks for reporting this. I notified the developer to have a look. Internal references is<PHONE_NUMBER>
Best! Thilo

Hi themash,
the developer responsible added some samples to clarify this issue and how to disable scrolling in the described case.
Please have a look into this commit:
https://github.com/SAP/openui5/commit/c61faeb2ec77e6d84311b644c52eec2d5df9dc7b
This also fixes the issue in your plunker.
Best! Thilo

Nice, quick and easy fix. Like it :+1:!


feat(Popup): add eventsEnabled prop

Fixes #3700.

Codecov Report

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

@@           Coverage Diff           @@
##           master    #3715   +/-   ##
=======================================
  Coverage   99.83%   99.83%
=======================================
  Files         175      175
  Lines        3120     3120
=======================================
  Hits         3115     3115
  Misses          5        5

Impacted Files
Coverage Δ

src/modules/Popup/Popup.js
98.68% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


docs(readme): fix typos and improve formatting

docs(readme): remove space between slash in Colemak/Dvorak
docs(readme): wrap text at 80 characters
style(readme): apply HTML LSP formatting
docs(readme): remove trailing list seperator
style(readme): properly format paragraphs
docs(readme): use are instead of is for plural references

Thanks; I think only 6cdf7cc9243470ac593309cf694eec9c029af710 and d4e744161c20f82c6a045ac84d0e9bd028194e59 are non-stylistic in nature, and I don't think the others are necessarily "improvements" over the current state. Could you remove all commits but those two?

Thanks


SetValue for TwoOptions doesn't work for checkboxes

When using the "SetValue(TwoOption option)" method, it doesn't properly work for checkboxes. It clicks next to the box, so the box doesn't get (un)checked.
It currently clicks the fieldelement and not the imput element that is a child of the fieldelement. the fieldelement is too big, so it clicks next to the box. If you click on the input element, it seems to work.

Hello,
We have the same issue. However the deprecated method SetValue(string, bool) is working fine.

Thanks Nicolas for a workaround.
Which version and which client are you working with?

Hello @dtu11 ,
We are working with Chrome and D365 Version 1710 (<IP_ADDRESS>40) online.

@dtu11
For me exactly the same. Good plan to use the deprecated method for now.

When formatted as a list it doesn't work either. Going from false to true opens the drop down but never sets. Going from true to false doesn't open the drop down at all.


Packaging of codegen that supports CADL and Swagger

We plan three different deliverables to NPM (names to be confirmed):

@cadl-tools/pythoncodegen: The CADL emitter for Python
python/codegen: The core codegen agnostic of CADL/Swagger (based on shared YAML format)
autorest.python: The codegen plugin of "autorest" that reads Swagger

Notes:

1/3 have a dependency on 2.
1/3 owned their venv. The idea of the installation is:

1/3 creates a fresh venv
1/3 installs 2 as Python code in this venv (2 is not published in PyPI)
1/3 then installs themselves as Python code, and the Python dependency on 2 is solved because pre-installed
1 may not have Python code at all if it's just sub-processing a Python process, but it still needs to host the venv somewhere.

More details to come, this is the overall architecture

See https://github.com/Azure/cadl-azure/issues/1759 for Python name @azure-tools/cadl-python


Automator: update common-files@master in istio/istio@master

Generated by Automator - 2022-10-21T18:22:06+00:00

@istio-testing: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-helm_istio
1c7fcb8ad514bea5cded5b02c8640bac64b8cf26
link
true
/test integ-helm

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test integ-helm


RDK-35648: Override HDMI Color Depth

Reason for change:
Override HDMI Color Depth
Test Procedure: None
Risks: Low
Change-Id: I9e18de182d516d65b1f794163c238621a08212ed
Signed-off-by:Anooj Cheriyan<EMAIL_ADDRESS>

Can one of the admins verify this patch?

Review comments fixed: 2523


Mattermost 7 app jitsi problem

Hello,
OS= windows 11, Macbook intel, Macbook m1
When I want to  a call with jitsi, the application does not open. redirects to the browser. Can you help me?
Mattermost app version: 5.1.1

@iwyz0 This might be something that has to be fixed in the Jitsi plugin, since the plugins don't have direct access to screen share resources in the Desktop App.
Can you confirm that the screen sharing is the only issue going forward? If so, I can send this over to the Jitsi plugin repo.

Hello @devinbinnie
i see 2 problems
Screen recording doesn't work
Screen share doesn't work
Can you send it if you don't mind? to the jitsi repo
Thank you.


Add Staticfile and use buildpack path in manifest.

@AbelHu
I'd like to revert this PR so that cf push won't download staticfile_buildpack from internet.
By default, staticfile_buildpack is installed in the CF deployment.
Downloading staticfile_buildpack from Github may be blocked in some network environment (e.g. in China).
What do you think?

LGTM


warning C4201: nonstandard extension used: nameless struct/union

I guess this is the same issue as was the case with GCC inside typedef struct _cl_image_desc for eg. which was circumvented with the code:
#ifdef __GNUC__
    __extension__   /* Prevents warnings about anonymous union in -pedantic builds */
#endif
    union {
      cl_mem                  buffer;
      cl_mem                  mem_object;
    };

MSVC has exactly the same problem when compiled with /Za, strict ANSI conformance.

By the way, would giving this enum a name break the ABI? Or are there other implications? Isn't it more elegant to actually conform to ANSI C than to disable extensions?

It wouldn't break the ABI, but it would potentially break existing code, which is undesirable. We'll certainly consider avoiding anonymous unions like this in the future though.
I've merged PR #35, so closing this issue now.


chore: updates arquillian-cube-openshift dependency.

Wow... you even fixed the conflict! that's awesome! Thanks!


Unable to get custom template to work

Thank you for this amazing package! Saved me a lot of time.
I am able to get the basic example working in my website. However when I tried to implement the custom template example, it doesn't work. The suggestions just don't show up. I have done everything shown in the example. I am using meteor <IP_ADDRESS>
I was also wondering how the text search is done in case of custom template. the find().fetch() returns objects right..so how does it know which property should be used to match the text entered with?
Thanks!
Best,
Vivek

@vivekkatta please look at demo application in this repository with example of custom template usage. re text search - by default value property is used for search, but you should try data-value-key attribute to specify the search property.

thats what I missed out!! Thanks, its working now.


Error handling in Mapping

Introduces a Mapping::Error that tracks multiple errors for a given
record and handles messaging.
In the Mapping, we catch errors for each declaration on a given
record, storing them with the relevant property name, and re-raise. The
Mapper then simply catches the error and logs its message.
This simplifies handling outside the DSL code, and creates richer error
messages.

Just to check - since you're defining Error as in inner class of Mapping, does that mean that you don't anticipate using this type of error handling for any other classes?

My question above is mostly academic.  This is really neat!  :+1: from me.

does that mean that you don't anticipate using this type of error handling for any other classes?

Yeah. This is a pattern we could deploy elsewhere, but the specific behavior is all particular to mapping.
We might benefit from some digging to find other people using error collector patterns in Ruby---and think about a refactor to use a base class if we find ourselves reaching for this again.
Since implementing this, I've already found references to this kind of code in the Fowler DSL book. Everything old is new again.

I can generally see what's going on by reading the tests, but maybe there could be some @examples in the doc comments.

I'll work on adding some examples. :+1: for pushing on docs. :)

Does that look better?

Yes, thanks!

Happy to merge; assuming you want to squash it first.

:squash:'d


Roll Flutter Engine from 36dccf7bb25c to 38bef17b7bdc (21 revisions)

https://github.com/flutter/engine/compare/36dccf7bb25c...38bef17b7bdc
2024-07-12<EMAIL_ADDRESS>Roll Dart SDK from 797d3df745d1 to e986ed9d0bc1 (1 revision) (flutter/engine#53840)
2024-07-12<EMAIL_ADDRESS>Roll Skia from 7a91f0a4b7a0 to 9ea5603242c1 (1 revision) (flutter/engine#53839)
2024-07-12<EMAIL_ADDRESS>Roll Skia from 9529b8ad9e45 to 7a91f0a4b7a0 (2 revisions) (flutter/engine#53838)
2024-07-12<EMAIL_ADDRESS>Roll Skia from 38f355af4f36 to 9529b8ad9e45 (1 revision) (flutter/engine#53837)
2024-07-12<EMAIL_ADDRESS>Roll Skia from 14c8d318615d to 38f355af4f36 (1 revision) (flutter/engine#53836)
2024-07-12<EMAIL_ADDRESS>Roll Skia from ddf045505cb9 to 14c8d318615d (1 revision) (flutter/engine#53835)
2024-07-12<EMAIL_ADDRESS>Roll Fuchsia Linux SDK from 0e47sje8wkJ08sGJ6... to VlZIUknh6dnA23owe... (flutter/engine#53834)
2024-07-12<EMAIL_ADDRESS>Manual roll Dart SDK from fb546f313557 to 797d3df745d1 (8 revisions) (flutter/engine#53832)
2024-07-11<EMAIL_ADDRESS>[Impeller] Ensure full transform is applied to text contents (flutter/engine#53819)
2024-07-11<EMAIL_ADDRESS>Roll ICU from 43953f57b037 to 9408c6fd4a39 (6 revisions) (flutter/engine#53827)
2024-07-11<EMAIL_ADDRESS>Update Life-of-a-Flutter-Frame.md (flutter/engine#53829)
2024-07-11<EMAIL_ADDRESS>Update Setting-up-the-Engine-development-environment.md (flutter/engine#53828)
2024-07-11<EMAIL_ADDRESS>[web] retrieve hostElement for an implicit view (flutter/engine#53296)
2024-07-11<EMAIL_ADDRESS>[dart:ui] remove expensive index assertion in Vertices. (flutter/engine#53558)
2024-07-11<EMAIL_ADDRESS>[Impeller] Enable fixed-rate compression support in Vulkan. (flutter/engine#53292)
2024-07-11<EMAIL_ADDRESS>Roll Skia from 037d5f8a727f to ddf045505cb9 (1 revision) (flutter/engine#53824)
2024-07-11<EMAIL_ADDRESS>Add instructions for source debugging with Xcode when using RBE. (flutter/engine#53822)
2024-07-11<EMAIL_ADDRESS>Roll Skia from 004c81523e44 to 037d5f8a727f (1 revision) (flutter/engine#53818)
2024-07-11<EMAIL_ADDRESS>[Impeller] move more aiks tests to DL.  (flutter/engine#53792)
2024-07-11<EMAIL_ADDRESS>Roll Skia from ec4a1e03f7b0 to 004c81523e44 (23 revisions) (flutter/engine#53813)
2024-07-11<EMAIL_ADDRESS>Roll Skia from 2783ba54bf8e to ec4a1e03f7b0 (9 revisions) (flutter/engine#53797)
Also rolling transitive DEPS:
fuchsia/sdk/core/linux-amd64 from 0e47sje8wkJ0 to VlZIUknh6dnA
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC bdero@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


How to Flash mac OS image to my Local HDD drives?

Hello,
I want flash mac OS image （.dmg） to my HDD drives,Can the next version add this feature?
thanks very much!

Hey there, you can already do that, but need to enable "Unsafe mode" in the settings. Be aware that doing so will make previous data on that drive unrecoverable, so please be very sure you know what you're doing.


The database on the right side of the editing page cannot be painted out terms of settlement:

Search before asking

[X] I searched the issues and found no similar issues.

Linkis Component
linkis-commons
What happened + What you expected to happen
The database on the right side of the editing page cannot be painted out
terms of settlement:
Relevent platform
.
Reproduction script
.
Anything else
This question comes from the QA documentation of the Linkis community
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

A. Linkes metadata strictly abides by hive authorization. Only hive library tables authorized by grant select can be displayed by the front end.
This function relies on hive to enable authorized access:
hive.security.authorization.enabled=true
可以参考：https://blog.csdn.net/yancychas/article/details/84202
B. If you don't want to open hive permission, you can modify it: COM / webank / wedatasphere / links / metadata / hive / Dao / impl / hivemetadao Remove the permission control part of SQL in XML.


Entity with children cannot be expanded

Some entities with children cannot be expanded in the Architectural Map.
Steps to reproduce the issue:
It does not happen consistently so it might be hard to reproduce, but you can try loading this image and following these steps:

Go to the Architectural Map window which is open.
Under the "Audio" tag, you should find the folders "alsa" and "alsamidi". While both have children, only "alsa" can be expanded by right-clicking in and then selecting "expand".

Expected behaviour: both "alsa" and "alsamidi" should be expandable. By by inspecting them it is possible to see that the "children" attribute exists for these entities, therefore children should be visible in the visualisation as well.

Moved to https://github.com/moosetechnology/MooseIDE/issues/882


SecretAgentMan: Watch brunocampana.engineer

SecretAgentMan requests the watch of the watch_keyword brunocampana\.engineer. See the MS search here and the Stack Exchange search here.

brunocampana\.engineer has been seen in 0 true positives, 3 false positives, and 0 NAAs.

Approved by Machavity in Charcoal HQ


Production WSGI server

Use a production WSGI server to host the WSGI Connaisseur application. Load and stress testing led us to choose cheroot by a slim margin.
PR also contains some loadtests for the pipeline integration tests

Codecov Report

Merging #299 (c7c6499) into develop (485e4f7) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop     #299      +/-   ##
===========================================
- Coverage    96.66%   96.65%   -0.01%
===========================================
  Files           22       22
  Lines         1080     1077       -3
===========================================
- Hits          1044     1041       -3
  Misses          36       36

Impacted Files
Coverage Δ

connaisseur/validators/static/static_validator.py
93.75% <0.00%> (-0.37%)
:arrow_down:

...naisseur/validators/notaryv1/notaryv1_validator.py
95.83% <0.00%> (-0.07%)
:arrow_down:

connaisseur/validators/interface.py
100.00% <0.00%> (ø)

connaisseur/validators/validator.py
100.00% <0.00%> (ø)

connaisseur/validators/notaryv1/notary.py
98.30% <0.00%> (ø)

connaisseur/validators/cosign/cosign_validator.py
98.61% <0.00%> (ø)

Continue to review full report at Codecov.

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

Looks very good! Can we delete flask from the requirements?

No, since we're still using the flask WSGI application. Just no longer the server. But that's a great point, gotta rename the flask_server file


A reload function would be nice

Thanks again for your control.
For usage in cells the option of reloading w/o recreating would be a great addition.
I have added one via an extension (see below), but it is of course not a clean and has to be added to your source file (private access).
Kind Regard
Graham
extension CalendarHeatmap {
    public func reload() {
        DispatchQueue.global(qos: .userInteractive).async {
            DispatchQueue.main.async { [weak self] in
                // then reload
                self?.collectionView.reloadData()
                self?.scrollToEnd()
            }
        }
    }
}

Sounds great, I will add the reload function, thank you for the advice.

Thank you, this solves my issue.


Remove bower. Fetch plyr from npm instead

Hey,
I discovered this app recently and I love it. Thought I could help. This should close #227

Hope this gets merged :) :+1:

Thanks for the PR. Couple of issues:

You are targetting multiple issues with one PR. Please consider separate them into multiple PR's.
With each fix/feature, please squash into one commit only.
What's the motivation behind changing the plyer class names in the scss file?

Sure, I can do one PR/Commit per issue. Should we close this one and I can create 3 PRs later ?
For the plyer class names, I believe koel was using version 1.3.6, but plyr has only been on npm since v1.5.9 ... One of the changes they made was to change their class names from .player* to .plyr*, so I made the necessary adjustements in Koel's css, and copied the scss from their package.

Yeah, please create individual commits, thanks.
On Wed, Feb 17, 2016 at 10:57 PM, Mickael Burguet<EMAIL_ADDRESS>wrote:

Sure, I can do one PR/Commit per issue. Should we close this one and I can
create 3 PRs later ?
For the plyer class names, I believe koel was using version 1.3.6, but
plyr has only been on npm since v1.5.9 ... One of the changes they made was
to change their class names from .player* to .plyr*, so I made the
necessary adjustements in Koel's css, and copied the scss from their
package.
—
Reply to this email directly or view it on GitHub
https://github.com/phanan/koel/pull/235#issuecomment-185240235.


Implementing the random operator

If we think the VTL spec is stable we may start implementing the random operator.
The current state of the spec is:

Syntax
random ( { seed (op) })
Input parameters
op ::  	    		number
Examples of valid syntax
random()
random(seed (10))
Semantics for scalar operations
The operator returns a random decimal number >= 0 and <1. By default, random() produces different results each time it is calculated. Set the seed parameter to a constant to return the same pseudo-random numbers.

Hi ! Anything new about this topic ? Thanks !


Old, simple method for flat return sections

Everything looks like it should:

https://scanpy.readthedocs.io/en/return-formatting/api/scanpy.pp.calculate_qc_metrics.html
Here the rst is weird. Just get rid of the *!

https://scanpy.readthedocs.io/en/return-formatting/api/scanpy.pp.normalize_total.html
Short prose, looks fine.

https://scanpy.readthedocs.io/en/return-formatting/api/scanpy.tl.leiden.html
This should be converted into a definition list as well.

https://scanpy.readthedocs.io/en/return-formatting/api/scanpy.tl.dpt.html
Mixed. Looks great.

No rtype interfering anywhere

Wait, aren’t the docs supposed to be like this?
def myfunc():
    """\
    First line
    ...
    """

No, they aren't. Most of them start with """my title. But we can happily migrate... see Slack...

Well, it wasn’t that one anyway. I’ll figure it out

Ah, the problem was that the string actually contained that return type!
Theoretically that should have parsed as a 1-element definition list, but I don’t think it’s a problem that it doesn’t.

Ah, the problem was that the string actually contained that return type!

This is the standard way of writing a numpydoc returns section. Also see https://scanpy.readthedocs.io/en/return-formatting/api/scanpy.pp.filter_cells.html, for instance.
This solution is dropping support for them.
Also, do you by chance have another simple solution for having the styling of the return sections similar to the parameters section (what numpydoc did 🙂)?

Unfortunately, changing
.rst-content dl:not(.docutils) dl dt {
    font-weight: normal;

to bold changes a whole lot of other stuff. https://scanpy.readthedocs.io/en/return-formatting/api/scanpy.tl.dpt.html
The space could be inserted manually before the colon, that would be fine as it's consistent with how the parameters sections are written.

@ivirshup @flying-sheep I would remove the , optional statement from the docstrings, as, what we mean with this is "a parameter has a default value". Hence, it's redundant.

In the auto-generated type annotations, the default values miss completely, and I don't think we'll ever restore the , optional descriptor, there.

This is the standard way of writing a numpydoc returns section. […] This solution is dropping support for them.

It certainly shouldn’t, since the definition lists are rendered in other cases. IDK why not here, this should render as a definition list with one item.
However, I don’t like indenting the whole section except for the first line, so in case it always works once there are multiple definition list items, I don’t worry too much here.

Also, do you by chance have another simple solution for having the styling of the return sections similar to the parameters section (what numpydoc did :slightly_smiling_face:)? Bold font and spacings around colons?

I’ll figure it out.

I would remove the , optional statement from the docstrings, as, what we mean with this is "a parameter has a default value". Hence, it's redundant. However, it's consistently used in all of numpy, scipy, sklearn, pandas, etc.

We should definitely put the defaults inline, and I also think the “optional” is redundant. What would it even mean to have “a parameter that isn’t optional but has a default value”?
I’m pretty sure people will understand it.

IDK why not here, this should render as a definition list with one item.

Hm, maybe because a new line is missing before the start of the list. I don't know now....

However, I don’t like indenting the whole section except for the first line, so in case it always works once there are multiple definition list items, I don’t worry too much here.

Absolutely agreed! So, let's just forget about the one-item definition list.

I figured the one-item-thing out: The emitted code is:
:param copy: If an :class:`~anndata.AnnData` is passed, determines whether a copy
             is returned.
:type copy: `bool`, optional (default: `False`)

:returns: AnnData, None
              Depending on `copy` returns or updates `adata` with the corrected data matrix.

And since :returns: is part of a field list, and field lists are defined by the indentation of the block starting in the second line, the additional indentation of the second line is ignored.
So yes, only numpydoc-style sections with one item are affected.

OK I have something good. I’ve got an appointment now but will fix the remaining docstrings soon.

This should work for now. The problem is that this really needs to live in scanpydoc, where we have access to the fancy parsing.
My plan is to merge this now, which has a solution for the simple case of
parameter : some.type
    Description

Later I’ll introduce the same behavior as what scanpydoc does with the parameters:
If the return type is ...) -> Tuple[foo.bar, baz.zab]:, then I’ll check if there’s a section like
one_identifier
    Desc
second_identifier
    Desc

and replace them with the same code as parameters.

This leaves 3 styles:

Prose for a single return value
The above for returning a tuple
“this method adds some anndata fields”

For 3., we have like 3 styles floating around and need to fix one:

**dpt_pseudotime** : :class:`pandas.Series` (`adata.obs`, dtype `float`)
``adata.obs['louvain']`` (:class:`pandas.Series`, dtype ``category``)
`adata.uns['leiden']['params']` : `dict`

As mentioned in the commit, fantastic! :smile:
Which style mentioned in https://github.com/theislab/scanpy/pull/610#issuecomment-484124854 did you decide for?
Most manual corrections I saw in your PR pointed to solution (1) [**dpt_pseudotime** : :class:pandas.Series (adata.obs, dtype float)]. That's fine, but we should mention it in CONTRIBUTING.md. 🙂What do you think, @flying-sheep?

Why did you decide against the sklearn-style solution, which would have been simple to get without manual fixes (adding **name**)? https://github.com/theislab/scanpy/pull/610#issuecomment-484027492

I think you misunderstood a bit.

There’s three types of return sections – prose, tuple, added anndata.obs/var fields.
There’s one style for tuple return sections (like parameters). I implemented an automatic way to handle simple name : type cases of those and manually formatted the rest. I’ll automate the other cases in scanpydoc, then we can remove the manually formatted ones.
There’s 3 styles for anndata.obs/var field return sections. I left them as they are for now, since we have to decide one.

I only reformatted tuple return sections, I neither formatted nor decided on anything about anndata.obs/var field return sections.
I hope I was more clear this time :smile:

This is unrelated to your post just now. :smile:
Also, Parameters etc. don't render as a heading (rubric) like the other sections (Notes, Examples, See also etc.) anymore. They simply render as

Parameters

whereas

Notes

This is the new styling decision around numpydoc, also visible here: https://scikit-learn.org/stable/modules/generated/sklearn.decomposition.PCA.html
In order to have it look consistent (as in sklearn), I think we do need
.rst-content dl:not(.docutils) dl dt {
    font-weight: bold;
    font-size: normal

https://github.com/theislab/scanpy/pull/610#issuecomment-484427693: Great! Let's leave the remaining ones as a task for moving towards more consistency in the future. :smile:

Adapted the css to enforce the consistent styling of headings on each page: https://github.com/theislab/scanpy/commit/1f579f6f745d01c599a39f38abadb8a32f95c12b

Tried to meaningfully convey what you described in https://github.com/theislab/scanpy/commit/0dcf6fcd831eadfc64b1c9485d1b7021347db0b9.

Good! I fixed some typos and improved the examples a bit. Specifically when there’s something to link, they should do it! :class:`pandas.Series` instead of ``pd.Series``

Nice! 🙂


Update AMI ID in CloudFormation config to latest ECS release

Update AMI ID in cloudformation.json to the latest ECS AMI released on 06/08. See https://aws.amazon.com/marketplace/ordering?productId=4ce33fd9-63ff-4f35-8d3a-939b641f1931&ref_=dtl_psb_continue&region=us-east-1.

Awesome. Thanks for this!


fix: Requesting ERC-20 or ERC-677 Transfer events in log filter.

This PR fixes some issues in the treasury_exporter for ERC-677 transfer events which have a different signature than usual Transfer events:
function transferAndCall(address receiver, uint amount, bytes data) returns (bool success)
https://github.com/ethereum/EIPs/issues/677
Prior to this fix, these events fail the exporter because the topic filter for the usual ERC-20 transfer event doesn't match with the first topic of the received event.

I'm looking at the specific example with LINK and it appears we just have the wrong ABI cached.
>>> Contract('0x514910771AF9Ca656af840dff83E8264EcF986CA')
<ERC20 Contract '0x514910771AF9Ca656af840dff83E8264EcF986CA'>  # wrong

>>> Contract('0x514910771AF9Ca656af840dff83E8264EcF986CA')
<LinkToken Contract '0x514910771AF9Ca656af840dff83E8264EcF986CA'>  # right

@banteg
updated according to recommendations from review


Fix build in Webpack 2 project with default config

Previous versions built fine with the default Webpack 2 config (resolve.extensions = ['.js', '.json']) https://webpack.js.org/configuration/resolve/#resolve-extensions
The new version uses a .web.js extension so breaks:

By using plain .js for the web version this is fixed. react-native will continue to prefer the .native.js version.
Test plan (required)
Setup a Webpack 2 app

Does this work when you change your resolve.extensions = ['.js', '.json']?


Set custom path for chunk files

Laravel Mix Version: 1.6.0 (npm list --depth=0)
Node Version (node -v):
NPM Version (npm -v):
OS:

Description:
Hello, My laravel structure is :
/myApp/ (/app, /config, /resources....) <= laravel
/wpublic (/js, /css) <= here i put my index.php file, this is the content of my public folder
How can i set a custom path for chunk files?
For example for js files i want using this path /wpublic/js and for chunks /wpublic/js/chunks
Steps To Reproduce:

Take a look here - https://github.com/JeffreyWay/laravel-mix/pull/1248

Search before creating new issue
//webpack.mix.js
mix.webpackConfig({
    output: {
        chunkFilename: 'js/[name].js',
    },
});

@ankurk91 This is not what i want. please try to undestand question before answering

@leonfather

How can i set a custom path for chunk files?

I gave answer to above question.

@ankurk91 Thank you

Hi @ankurk91 ,, I have a problem. When I access a url with parameters like "http://localhost/laravel/5.4/edit-user/1" with the vue router everything is fine. But when I refresh the browser, I found the error and redirect to url "http://localhost/laravel/5.4/".

I suspect you have enabled html5mode in your Vue router.
Usually js routes url includes a hash bang # .
You are getting 404 because that routes does not exits.
You should watch Vue js routing video on laracast.

Read more about html5mode here :
https://router.vuejs.org/en/essentials/history-mode.html

Solve the issue when I edit the webpack.mix. js from "publicPath:/protected/public/" to "publicPath : http://localhost/laravel/5.4/protected/public/"

But how to be a dynamic baseurl? without adding "http://localhost/laravel/5.4/" on publicPath

Public Path should be a physical harddisk location.
There is a difference between path and url.
I would suggest to create a virtual host for your  laravel installation.

empp,, so what if I upload to hosting later?

Don't run too fast. Solve your issues one by one. Learn the basics, clear the concepts.
I can't teach you the whole development flow.
Jeffrey way runs a free website called Laracasts for people like you.
Invest some time in watching those webseries and you will no longer require asking such questions.

@ankurk91 Thank you so much bro


optimize events creation and improve indexes

Proposed Changes

optimize events creation and improve indexes
closes https://github.com/ohcnetwork/care/issues/2369

Merge Checklist

[ ] Tests added/fixed
[ ] Update docs in /docs
[ ] Linting Complete
[ ] Any other necessary step

Only PR's with test cases included and passing lint and test pipelines will be reviewed
@ohcnetwork/care-backend-maintainers @ohcnetwork/care-backend-admins

https://github.com/ohcnetwork/care/pull/2593


Yajra datatable date range report

Hi for everyone on the cross of the globe.
I use Yajra datatable to filter my report as bellow.
Dear @yajra and anyone knows my problem, please help me to solve this issue.
This is my issue.
This is my view.

It's my script.
    <script>
        $("#dbutton").click(function()
        {
            $('#servicestable').DataTable(
                {
                    ajax: {
                        url:'{{route("report.show")}}',
                              data: function (d)
                                {
                                    d.from       = $('input   [name=from]').val();
                                    d.to         = $('input   [name=to]').val();
                                    type:      'service'
                                }
                        },
                        columns:
                        [
                            { data: 'number',        name: 'number' },
                            { data: 'name',          name: 'name' },
                            { data: 'description',   name: 'description' },
                            { data: 'income',        name: 'income' },
                            { data: 'rate',          name: 'rate' },
                            { data: 'measure',       name: 'measure' },
                            { data: 'cost_account',  name: 'cost_account' },
                            { data: 'amount',        name: 'amount' },
                        ]
                });
        });
    </script>

It's my report controller with its functions.
  public function index(Request $equest ,$type)
 {

    if($type == "services")
    {
 	    return view('services.report');
    }
 }
 public function show(Request $request)
{

    if($request->ajax()) // This is check ajax request
    {
        if($request->has('type') && $request->get('type') == 'service')
        {
            if($request->has('from') && $request->has('to'))
            {
                $from   = $request->get('from');
                $to     = $request->get('to');
                  $service=$this->item->getServices($from,$to);
                  return Datatables::of($service)->make(true);
            }

        }
    }
}

Its my model codes
 public  function getServices($from , $to)
    {
        return $this->leftjoin('accounts as incomes', 'items.account_id', '=', 'incomes.id')
            ->leftjoin('costs', 'costs.item_id', '=', 'items.id')
            ->leftjoin('measure_units', 'items.um_id', '=', 'measure_units.id')
            ->leftjoin('accounts as cost_accounts', 'costs.account_id', '=', 'cost_accounts.id')
            ->where('items.type','service')
            ->whereBetween('items.created_at',[$from],[$to])
            ->orderBy('item_id', 'desc')
            ->get(
                [
                'items.id',
                'items.um_id',
                'items.number',
                'items.name',
                'items.description',
                'measure_units.name as measure',
                'incomes.name as income',
                'cost_accounts.name as cost_account',
                'items.rate',
                'costs.amount',
                ]);
    }

And finally its my routes.
 Route::get('report/{type}',['as'=>'report.index','uses'=>'ReportController\ReportController@index']);
 Route::post('report/',['as'=>'report.show','uses'=>'ReportController\ReportController@show']);

I think the issue here is on your show route. From and To variable should be optional and should be loaded only if value was set?
Also, I think there is an issue on your query.
->whereBetween('items.created_at',[$from],[$to])
// should be
->whereBetween('items.created_at', [$from, $to])

Lastly, better inspect the ajax request and dump the any errors that appear. That will give you more hint on what causes the issue.
PS: See https://datatables.yajrabox.com/eloquent/advance-filter for some ref.


Typesafe introspection

Sorry to reopen the introspection discussion again, but I've been wondering if there should be support for introspecting schemas without listing them all in a union yourself.
In the meantime I've created valibot-introspection, which essentially is just a large union of all schemas .
Here's an example of how it could be used to generate constraint validation attributes from a valibot schema: TS Playground
Let me know if this is something you are interested in supporting, or if it should stay as a separate library.

Personally, I recommend specifying GenericSchema and/or any specific schema type you support. This gives you more control over your code and allows you to specify what kind of schema your function supports. Note that we have added an isOfKind and isOfType type guard utility for better DX.


fix: v2 reporter app wrong commit message

Checklist:

[ ] Either (a) I've created an enhancement proposal and discussed it with the community, (b) this is a bug fix, or (c) this does not need to be in the release notes.
[ ] The title of the PR states what changed and the related issues number (used for the release note).
[ ] The title of the PR conforms to the Toolchain Guide
[ ] I've included "Closes [ISSUE #]" or "Fixes [ISSUE #]" in the description to automatically close the associated issue.
[ ] I've updated both the CLI and UI to expose my feature, or I plan to submit a second PR with them.
[ ] Does this PR require documentation updates?
[ ] I've updated documentation as required by this PR.
[ ] I have signed off all my commits as required by DCO
[ ] I have written unit and/or e2e tests for my change. PRs without these are unlikely to be merged.
[ ] My build is green (troubleshooting builds).
[ ] My new feature complies with the feature status guidelines.
[ ] I have added a brief description of why this PR is necessary and/or what this PR solves.
[ ] Optional. My organization is added to USERS.md.
[ ] Optional. For bug fixes, I've indicated what older releases this fix should be cherry-picked into (this may or may not happen depending on risk/complexity).

@CodiumAI-Agent /review

PR Reviewer Guide 🔍

 ⏱ Estimated effort to review [1-5]
3

 🧪 Relevant tests
No

 🔒 Security concerns
No

 ⚡ Key issues to review

Error Handling: The refactored method getResourceActualState in application_event_reporter.go simplifies error handling by moving the logic into a separate function. However, it's crucial to ensure that all error scenarios are adequately tested, especially the context deadline and not found errors.

GZIP Compression: In client.go, the addition of gzip compression for sending events is a significant change. It's important to verify that the receiving server supports gzip decompression and that this change doesn't introduce any issues with data integrity or transmission.