Error on installing git-lfs

When I install git-lfs, I get the following error:
dpkg: error processing package git-lfs (--configure):
 installed git-lfs package post-installation script subprocess returned error exit status 2
Errors were encountered while processing:
 git-lfs
E: Sub-process /usr/bin/dpkg returned an error code (1)

I think the gitconfig binding in the root environment is related to this problem. I suggest binding with the file from the user folder.

I'm not sure but I think something had changed on docker API. I think we have a new behavior for how it sets the user id. Maybe we don't even need to use gosu anymore.
Perhaps this change is causing the git-lfs error somehow...

I think it would be good to remove the gosu stuff with the intent of simplifying it if possible.


Update README for setup on non-Unreal projects and add error check

This PR updates the README file with instructions for building/including in a non-Unreal C++ project (it's currently a bit of a pain and took me a while to figure out). It also updates the example and provides some troubleshooting steps.
I also included an error check in the STL file read ifstream mode to ensure the file was opened properly. It earlier failed on the parsing step and produced an endianness error; it now properly reports the error to the user. I included this because it also tripped me up for a bit.
The test file passes when I run it with -C Release, but not -C Debug (standing issue, not caused by my changes).
Let me know if I should change or add anything!

Thanks for updating the readme . The additional error check is a good idea :+1
But could you add the name of the file to the error message, this might be usefull if the user generates its filenames.

But could you add the name of the file to the error message, this might be usefull if the user generates its filenames.

Pushed!


provide crypto safe rnglib

We had better provide crypto safe rnglib

windows system uses: CryptGenRandom()
*nix system uses: /dev/urandom
Hardware: RDRAND instruction.

11/8/21 - agree to provide example.

I will check this.


_.pickRandom()

Hi!
Here is a proposed addition to underscore, a function that picks a number of random elements from an array. Functionally, _.pickRandom(array, [n]) is equivalent to _.first(_.shuffle(array), [n])), but much more effective. Takes time proportional to n rather than the length of the array.
The implementation might seem odd. It actually mutates the input array while it is working, but restores everything back to normal before returning. Hope you see how it works. I couldn't think of a better way to ensure that elements are picked from unique indices. One alternative would be to keep an array of indices picked so far and keep trying until a unique index was found. This would work well for picking 5 elements out of 1000, but would perform badly when the number of elements to pick is close to the length of the array.
Documentation and unit tests included. Here is some extra testing code: speed test (compared with shuffle/first) and "fair coin toss" test.
I'm new to git and github, hope I am doing this the right way.
Regards,
Simon Kågedal Reimer

Thanks -- this is great stuff. But I'm afraid that we're trying to keep Underscore small and focused, and the since we already have a good array shuffling function, I think that's what we should be using here.
_.first(_.shuffle(array), n)

... is a fine way to do it.

Too bad JS isn't lazy. These would have the same performance.


Add cloudtrail endpoint for ap-northeast-3

Describe the bug
Cloudtrail exists in ap-northeast-3, but there is no end-point for it listed in the endpoints file: https://github.com/boto/botocore/blob/742df860f38bc3d70a0a74a1cf7efdb3bed776fb/botocore/data/endpoints.json#L1736
As a result, tooling that calls get_available_regions is not able to know Cloudtrail exists there.

Hi @0xdabbad00, thanks for bringing this to our attention. Endpoints are generated upstream by service teams so I created a ticket for the Cloudtrail team to add this endpoint. I'll post an update here when I hear back from them.

P54774474

The ap-northeast-3 endpoint has been added for Cloudtrail here. Thanks for reporting this, closing the issue now.


Topic fixes and new MD

I've fixed some minor errors in Topics.st - syntax errors and missing sections.
Also I've generated the MD file directly from the Topics.st so we can probably automate this step.

Great work Peter. How to you automate the generation ?
I start to work on doing it by hand, but better if you do it automatically !

I think that we still should aim towards an html page on gsco.pharo.org this will look much more professional and we also discussed with Peter possibilities of automatically deploying the page

ok, you are better than me ;-) Go ahead !

I've created an extension for the HTML generator that generates the markdown.
I wrote to Sven as I would like push it back to the project, also I asked him about the automation, so we'll see.

Ok I merge what you have done until now. Thank you Peter for your help.
I will ask people on the mailing to update the Topics.st file and that we are working on generating the html file from it

We will also add a Travis job so we will be (almost) instantly informed if there is any issue in projects format ;)

On 17 Feb 2016, at 12:06, Serge Stinckwich<EMAIL_ADDRESS>wrote:
Ok I merge what you have done until now. Thank you Peter for your help.
I will ask people on the mailing to update the Topics.st file and that we are working on generating the html file from it
—
Reply to this email directly or view it on GitHub https://github.com/pharo-project/pharo-project-proposals/pull/19#issuecomment-185156462.

Great ! I know why I ask you to join as gsoc 2016 co-admins ;-)

We will also add a Travis job

As I am not member of pharo-project github team, I will need someone who is to sign up on travis-ci.org and enable in Accounts settings building for this project.

I ask people on pharo-users if someone can add you.


Investigate Double Build in Heroku

The bug relates to:

[ ] api - An issue with the API

Describe the bug
When deploying the api builds twice in Heroku.
Expected behavior
It should only run build once.

I think I may have accidentally solved this today :). I'll close for now


Surveyor View: Hookup Assignment “cluster” dropdown up to actual API

Whenever a surveyor selects homes on the dashboard and then clicks Generate Map, the selected homes are added to surveyor.selectedHome in the redux store. On the subsequent page /surveyor/map the homes are not displayed properly (everything's undefined).
In general we should probably make this workflow a bit smoother but that's out of the scope of this issue, see #335

This is as opposed to being hooked up to the jsonserver? It is currently pulling from jsonserver using the apiSlice hooks, so I think just changing the baseUrl in apiSlice to the url of the actual API would take care of this.

Merging #318 should resolve the URL situation! If this is already hooked up to RTK query then this is done, but we should confirm.

Okay I dug into this a bit - I think what we need to "hook up" here is the /surveyor/map page where it shows the list of homes you've selected on the actual surveyor dashboard page. Right now it's actually putting the selected homes in redux, so there's no actual API involved here! I'll update the issue name + description.

Actually my bad, I got this confused with #295 so I don't think anything else needs to be done here. Closing.


Split content with 2.42 OLED Display

Hi,
I really like this very cool project and want to build my own WebRadio.
I would like to use a 2.42 OLED display that I have around. This uses an SSD1309 Chip, which (according to most Docu) is almost identical to the SSD1306. But when I configured this project to use the OLED display, the image is split in half and the left side is right and the right side is left (see image).

If I use an example sketch with u8g2 the display works just fine.
Is there any idea what could cause this issue? Or is this type of display simply not supported? I think there's not much missing for it to work correctly...

You need to modify the constructor and the init call on line 23 and 36 of "oled.h". It is now:
U8G2_SSD1306_128X64_NONAME_1_SW_I2C
And it should be the one for 1309.  See https://github.com/olikraus/u8g2/wiki/u8g2setupcpp.
For example:
U8G2_SSD1309_128X64_NONAME2_1_SW_I2C

Hi @Edzelf, first thank you very much for your answer. I just noticed I'm in the wrong project with my question (I'm using an ESP32) - but I think you already noticed - reading your answer :)
To be honest, I don't quite understand it yet. If "OLED" is defined, the file SSD1306.h is included. Here, oled.h is never used. If I instead include oled.h (where I have changed the two lines of your comment), it doesn't compile since oled.h does not have all the required functions (like for example displayvolume() ).
I think I did not understand something correct here :(

You are right.
If you uncomment the #define OLEd in line 178 o main.cpp and comment the line with BLUETFT.
Then SSD1306.h is included.
The file oled.h is not in use anymore.
The driver for the SSD1306 is not quite compatible with the 1309.
I will try to fix that later.

Thank you very much!

Try the new version.  I did not test it, because I don't have the right module.
3 models are available:
//#define OLED1306                     // 64x128 I2C OLED SSD1306
//#define OLED1309                     // 64x128 I2C OLED SSD1309
//#define OLED1106                     // 64x128 I2C OLED SH1106

Wow - I'm really impressed with how fast you did the refactoring of the oled drivers. I will try it as soon as possible :) Thank you so much!

Hi @Edzelf, I've just tested it. It's a lot better but not yet quite right. The lines seem to start at different columns Maybe an off-by-one error?

Did you try all 3 models?  Please give me the exact type of the display.  Maybe I can find a hint in the datasheet.Vriendelijke groeten,Ed.
-------- Oorspronkelijk bericht --------
Van: martix90<EMAIL_ADDRESS>Datum: 20-02-21  10:56  (GMT+01:00)
Aan: Edzelf/Esp-radio<EMAIL_ADDRESS>Cc: Ed Smallenburg<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Onderwerp: Re: [Edzelf/Esp-radio] Split content with 2.42 OLED Display (#197)
Hi @Edzelf, I've just tested it. It's a lot better but not yet quite right. The lines seem to start at different columns Maybe an off-by-one error?
—You are receiving this because you were mentioned.Reply to this email directly, view it on GitHub, or unsubscribe.
[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/Edzelf/Esp-radio/issues/197#issuecomment-782598065",
"url": "https://github.com/Edzelf/Esp-radio/issues/197#issuecomment-782598065",
"name": "View Issue"
},
"description": "View this Issue on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]

Hi Ed, I now tried all three models (OLED1306, OLED1309 and OLED1106).
All of them behave exactly the same.
The model I use is called "DIY MORE" and says on the back "2,42' OLED, SSD1309", here's a links:
https://www.diymore.cc/products/2-42-inch-12864-oled-display-module-iic-i2c-spi-serial-for-arduino-c51-stm32-green-white-blue-yellow
Thank you and greetings from Germany
Martin

I think I found the issue:
If I comment out  this line, it works :)
//i2c_master_write ( i2Cchan, fillbuf, 4, true ) ;      // Yes, fill extra RAM with zeroes

Great!


[RHBPMS-4267] add jbpm-jpdl-migration into jbpm-bom

Related PRs:
https://github.com/droolsjbpm/droolsjbpm-build-bootstrap/pull/292
https://github.com/droolsjbpm/jbpm/pull/632
https://github.com/droolsjbpm/jbpm-designer/pull/469

Closing as it has been decided to not pursue this way (after discussion with @krisv).


Handling NULL values inside array of uuid[] columns

I'm running EFCore 2.2.6 and Npgsql.EntityFrameworkCore.PostgreSQL 2.2.4.
I have a database column type of uuid[]. Postgres seems to support NULL values inside of the arrays (e.g. { NULL, "8AF...", NULL } ).
I've marked the EF Core column type as Guid?[] , but get an exception trying to handle it with Npgsql:
System.InvalidCastException: Can't cast database type uuid[] to Nullable´1[]
How do I support possible null values inside uuid[] columns?

The basic ADO level of support for nullable arrays is tracked by https://github.com/npgsql/npgsql/issues/443. Once that's done, some changes will likely be needed in the EF Core provider as well - I'll keep this issue to also track that.


fix(45501): split insertSpaceAfterOpeningAndBeforeClosingEmptyBraces to insertSpaceInEmptyObjects/insertSpaceInEmptyBlocks

Fixes #45501
Fixes https://github.com/microsoft/TypeScript/pull/45682

\cc @andrewbranch

@andrewbranch Sure. I've created a PR to clean up my TODO :).

@andrewbranch I lost track of this. Did we ever get PRs out to VS and VS Code to match this?

The current status quo is more like

object type literals get no space (not configurable)
object literals get no space (not configurable)
every other pair of empty braces is configurable with the mega-setting, defaulting to having a space

???

I think the underlying aesthetic rationale, insofar as there is one, is/was for things that look like they got pulled straight out of C# to be consistent in having a space.
// Neither SyntaxKind.Block nor very JS-objecty
enum E { }
class C { }
interface I { }
module M { } // SyntaxKind.ModuleBlock but not sure why
switch (true) { } // SyntaxKind.CaseBlock but again ??

// Actually SyntaxKind.Block
{ }
if (true) { }
function f() { }
const x = () => { }

// JS object and direct type equivalent?
const o: {} = {}

// Binding patterns also feel like they should
// behave the same as object literals, this default
// may be a mistake, though the syntax is truly useless
const { } = o

// Unclear where these fit, but to me they feel similar
// to shorthand object literals and binding patterns. The
// defaults include a space though.
import { } from "typescript"
export { }

I can understand the desire to have the top 4 examples have the same default and be controlled by the same setting, though I think 4 and 5 are a closer fit with the second group.
So now I’m not sure how to group all these into a reasonable number of options or what to name those options.

Giving my +1 for the two categories of block versus object, where enums/class/etc are block-like.
As for binding patterns, I'd say that it's object-y. For import/export, they also seem object-y too (as they are more binding patterns, sort of).

I'd also make the assertion that if people are getting particular about the formatting of their code, it sounds like they should be using prettier, dprint, etc, rather than whatever the formatting is that a particular editor defaults to.

Why do you consider enums and classes block-like? They declare actual object-like values and their bodies are lists of members, not statements. How about interfaces? Do you think it’s weird for interfaces to be grouped apart from object type literals?

For both, I can reduce it down to the notion that their contents look more like declarations or assignments (or potentially worded as "things that feel legal inside of a class body").
enum Foo {
    X,
    Y = 1234,
    Z
}

interface Bar {
    x: number;
    someMethod(): void;
}

Whereas, the things in my object-y category use colons to re-bind from one name to another, or no colons at all.

The outlier of course is object types, which I stuck into the object category, wheras they're more like interface without the keyword in contexts where we don't need to have the keyword interface, except that they are no longer their own declaration and wouldn't be placed directly in any scope with other declarations.

@andrewbranch @jakebailey do you want to pick this back up? Is it worth hashing through the design or should we close it?

My feeling about the formatter in generall hasn't really changed, in that I think that if people care about these details they probably should be using prettier, dprint, etc, which aim to be opinionated enough to enforce one style. It's not very clear to me that the demand is there for our formatter to do anything more than leave what the user wrote in this particular case; this is just more knobs to maintain.
It seems to me like there are still unanswered questions about where we should stick spaces, what we consider to be a block vs not (especially considering interfaces, enums, object literals which all look different), so I don't think this PR would be going in as-is, at least, but I'm sure it could be mended.
But, @andrewbranch may have a different opinion here.


remove SpaceMapRBTree

What problem does this PR solve?
Issue Number: close #6289
Problem Summary:
What is changed and how it works?
Remove SpaceMapRBTree
Check List
Tests

[x] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/merge

/hold

/unhold


Music Notification Artwork Not Showing

I set the artworkUrl, but it doesn't show anything. Did I set it wrong?
My MediaItemBuilder:

               addMediaItem(
                            MediaItem.Builder()
                                .setMediaId(track.id.toString())
                                .setMediaMetadata(
                                    MediaMetadata.Builder()
                                        .setTitle(track.name)
                                        .setArtist(track.ar.joinToString(", ") { ar -> ar.name })
                                        .setMediaUri(Uri.parse("$RainMusicProtocol://music?id=${track.id}"))
                                        .setArtworkUri(Uri.parse(track.al.picUrl))
                                        .build()
                                )
                                .build()
               )

None of the UI components load from the artworkUri field currently, so you have to use setArtworkData. That's obviously bad for playlists though, since there will be a large amount of associated memory usage.
This is something we should address for 1.0.0 stable, so I'll mark it as a bug. We'll need to make sure our approach is compatible with image caching libraries (e.g., Glide), without mandating use of one. For internal devs, see [Internal ref: cr/383239306] for some previous discussion about this.

Is there any progress?

We added a Bitmap loader with this commit. It is already part of the released alpha version.
Did you try after updating and it did not work. Please let us know if this is the case.

Do I need extra code to make it work? I'm using alpha03 and it still doesn't work

Closing as this will be in the next release that is coming soon.

I'm having a similar issue as the original bug, but local artwork seems to not show.
Here is my MediaMetadata
val mediaMetaData = MediaMetadata.Builder().apply {
                        setAlbumTitle(audiobook.audiobookTitle)
                        setAlbumArtist(audiobook.audiobookAuthor)
                        setGenre(audiobook.audiobookGenre)
                        setArtworkUri(audiobook.imgThumbnail) //Doesn't work -> content://com.android.externalstorage.documents/tree/primary%3AAudiobooks/document/primary%3AAudiobooks%2FCreativity%2C%20Inc%2FCreativity%2C%20Inc-Cover.jpg
                        //setArtworkUri(Uri.parse("https://m.media-amazon.com/images/I/51vXKxIu6xL._SL500_.jpg")) //Works
                        setTitle(track.chapters.getOrNull(index)?.chapterName?.trim() ?: track.track.trackTitle ?: audiobook.audiobookTitle)
                    }.build()

When I try to set the artwork via a content:// Uri, the notification artwork is missing. (I use the same Uri in other areas of the app)
setArtworkUri(audiobook.imgThumbnail) //Doesn't work -> content://com.android.externalstorage.documents/tree/primary%3AAudiobooks/document/primary%3AAudiobooks%2FCreativity%2C%20Inc%2FCreativity%2C%20Inc-Cover.jpg

When I use a https:// reference, it shows the artwork in the notification. I'm not storing a url to the artwork though, so it's not really a viable solution. The above example was just for testing purposes.
Am I doing something wrong? (Or does this necessitate a separate bug?)


[BUG] Draw borders on top does not work

Module Version: v0.4.7
As per #6 , this feature should be added but it does not appear in the settings at all and by default it draws underneath all tokens:

I believe the two tickets are separate ticket #6 refers to the fact that the border of the token was not "above" the frame, now it is as show in the image.

In the image you passed it looks like you want the "secondary border" of the frame (not the border of the token), to be above the image of the token ?

Correct. That's what I was requesting in the previous request. I'd like to essentially cover up the "border" built into the token image with the border from this module, since that used to be possible with the module.


Disabled input styles for forms

Darkens the background of disabled inputs that have non-default background colors. Also includes changes to jquery.customforms.js for form.custom support.

Thanks!


Popups and Dropdowns does not work on stock Android 6.0 browser (Webview 4.0)

The following error is thrown when using the stock Android 6.0 browser and no popup appears:
Transition: Element is no longer attached to DOM. Unable to animate.  Use silent setting to surpress this warning in production. scale in <div class="ignored ui popup very wide top left">…
Other settings for transition have the same effect. e.g. "fade"
The issue can be reproduced on most of the popups on the definitions page for the Semantic UI popup. (probably all those using JavaScript). https://semantic-ui.com/modules/popup.html#/definition
Any workaround for this would be appreciated.

Is there any plan to fix this? It is kind of a show-stopper as most older phones have this browser installed by default, rather than Chrome.

Debug output from console. In transition stage:
Converted arguments into settings object
Preparing animation
Determining whether animation exists
No animation defined in css
Storing instance of module
Works fine on other browser. The problem seems to be the line "No animation defined in css". Why would this only be happening in Webview browsers?

@merravid There are not so many contributors to help with every issue, if you think you can solve a problem, open a PR


Change method of choosing next available config..

Made it much more randomised..

@ingwarsw Is there a reason for re-opening?

Tried to force rebuilding..
But its not working :|


Duplicate JSDoc in union of derived interfaces

interface I {
    /** The colour */
    readonly colour: string
}
interface A extends I {
    readonly colour: "red" | "green";
}
interface B extends I {
    readonly colour: "yellow" | "green";
}
type F = A | B
const f: F = { colour: "green" }
f.colour/**/

Expected behavior:
Quickinfo at /**/
(property) I.colour: string

The colour

Actual behavior:
(property) I.colour: string

The colour
The colour

From my notes on discord explaining to the person who noticed this:
The function in the TS services is called getDocumentationComment, and it does three things:

calls another function to get  the comment text to display
loops over each declaration after uniquifying
per-declaration, gets the comment text from the base, if there is any

For the example above, colour has two declarations, since it's a union: Apple.colour and Banana.colour. So:
Step 1. No comment text from either declaration.
Step 2. There are two unique declarations to iterate over.
Step 3. Each declaration has the same base, which contributes the same comment.
#35170 has separate jsdocs for each, as you point out.
#32708 only has one extended declaration, just instantiated with two generic types

I'd like to take a look at this.

Submitted a fix in https://github.com/microsoft/TypeScript/pull/43240


[DOC] Add ingest processors documentation

Description
Add ingest processors documentation
Issues Resolved
#4193
Checklist

[X] By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@heemin32 Please review, at your availability, the draft processors files ingest-processors.md and append.md. I'm working on several pieces of processors documentation, and I'll incorporate your feedback on these initial documents into the other processors. We need to make sure we have consistently style and level of detail for all the processors. Thank you, Melissa

Wasn't able to review the entire PR. There are quite a few errors in the doc. Please test if each example is working correctly.

For visibility: @vagimeli tested each processor; working as expected

Great start. Some high-level comments:

We should add a section that describes error handling
We should add a section that describes conditionally executing a processor
We should add a section that describes mustache template snippets.
Then we can link to these sections from the parameter tables.
I'm happy to sync offline if you have any questions. Thanks!

Added these sections or topics within the index.md or create-ingest.md files.

Wasn't able to review the entire PR. There are quite a few errors in the doc. Please test if each example is working correctly.

All processors have been tested. For visibility, an example response is shown in Step 2 of each processor documentation.


Fixed broken kubernetes operator link

Reading up on the k8s operator I noticed that this link is broken.
Instead of linking to https://kubernetes.io/docs/concepts/extend-kubernetes/operator/ it links to https://www.rabbitmq.com/kubernetes/operator/(https://kubernetes.io/docs/concepts/extend-kubernetes/operator/) which is a broken link.

Thank you!


Use 86 sentences to test this method but the result so bad.

Hi,
I use 86 sentences to train model , but the result is so bad.
If I need more data to train my model?
And when i am training my model I have large the occupancy of virtual memory, how do me tune parameter to have a less resource consumption.
Thank you very much!

I don't know how many minutes of speech you have, but I think transferring from pretrained SEGAN can help if you have low amount of your own data. Here you have the observations https://arxiv.org/abs/1712.06340

In terms of tuning the amount of consumption, it is the tensorflow Session configuration I guess. It is a TF inner thing, but you can specify a dynamic behavior rather than the full occupancy

Thanks for your reply sir. I used for reverb dataset about 86 sentences and maybe have 30+-min.The result is worse to result which I had not processing before not use segan,


Unexpected dataType of class HKActivitySummaryType for startDate predicate

Describe the bug
I have this error:
 "Unexpected dataType of class HKActivitySummaryType for startDate predicate"
2020-12-23 14:19:03.814970+0300 Runner[5703:578480] *** Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: 'Unexpected dataType of class HKActivitySummaryType for startDate predicate' *** First throw call stack: (0x1bce75344 0x1bcb8acc0 0x1bcd6551c 0x1d0bcbfa8 0x1d0b2005c 0x1d0b1fec8 0x1d0b1fb94 0x1d0aeb2d0 0x1d0ae89c4 0x1d0b08824 0x1049fb9c0 0x1049fb930 0x1049fb45c 0x104b974d8 0x104b97970 0x104b9212c 0x104b97a2c 0x104b97b2c 0x1032a1c84 0x102a603b4 0x102d3ac18 0x102a69948 0x102a6c074 0x1bcdf13a8 0x1bcdf10c0 0x1bcdf0740 0x1bcdeb538 0x1bcdeabc8 0x1c71cc5cc 0x1c0f9d744 0x1026c1b48 0x1bcc67384) libc++abi.dylib: terminating with uncaught exception of type NSException *** Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: 'Unexpected dataType of class HKActivitySummaryType for startDate predicate' terminating with uncaught exception of type NSException (lldb)
Smartphone:

Device:  iPhone11
OS: iOS13.6.1

Please help.

Thank you,
Have provided a fix, please review version 1.0.6

Thank you,
Have provided a fix, please review version 1.0.6

Ok, now everything works fine, thank you very much!

Ok, now everything works fine, thank you very much!


[hotstar] added support for downloading playlist

Please follow the guide below

You will be asked some questions, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your pull request (like that [x])
Use Preview tab to see how your pull request will actually look like

Before submitting a pull request make sure you have:

[x] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[x] Searched the bugtracker for similar pull requests

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[ ] I am the original author of this code and I am willing to release it under Unlicense
[x] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[ ] Bug fix
[ ] Improvement
[ ] New extractor
[x] New feature

Description of your pull request and other information
Support for download all episodes(playlist) from HotStar.
Steps to get playlist url:

Open any show
Click on 'Episodes'
Copy the url
E.g http://www.hotstar.com/tv/pow-bandi-yuddh-ke/10999/episodes/10856/9993

Hotstar works with --prefer-ffmpeg option only. This is an existing issue with single video download also.

Fix issues.
Check code with flake8.
Squash commits.
Provide reliable evidence that the original code is in public domain or released under Unlicense.

Plz let me know how to squash... never did it

Plz dnt merge it, i wl create new pull request as some commits were made from different email


Check if Sylius 1.9 migration is already executed

Fixes #132
This behaviour is also used within Sylius (for example in the refund plugin) to make sure these exact same table alterations are not being executed twice on Sylius 1.9.

I have now tested this while upgrading an 1.8 application to 1.9 and it works 💯.


SSL decryption via the $SSLKEYLOGFILE setup?

It's simple. While this is may Youtube-dl:
$ youtube-dl --version
2016.12.22-gentoo_no_offensive_sites
$
I prepared a page telling what my desired feature would be, by comparison with
what the Wget does. And I don't know the details how, I'm just a user.
Pls. see:
http://www.croatiafidelis.hr/foss/cap/cap-170105_wget-ssl/
and notice that Wget now decrypts SSL traffic, since you can decrypt the:
http://www.croatiafidelis.hr/foss/cap/cap-170105_wget-ssl/dump_170105_1733_g0n.pcap
by using the two effemeral keys in the
http://www.croatiafidelis.hr/foss/cap/cap-170105_wget-ssl/dump_170105_1733_g0n_SSLKEYLOGFILE.txt
such as by downloading those and issuing:
$ wireshark -o "ssl.keylog_file: dump_170105_1733_g0n_SSLKEYLOGFILE.txt" \
dump_170105_1733_g0n.pcap
My wish is that Youtube-dl would do that too! Thanks for your kind
consideration!

( I didn't know you need to insert two backslashes to get one out ;-) )
$ wireshark -o "ssl.keylog_file: dump_170105_1733_g0n_SSLKEYLOGFILE.txt" \
dump_170105_1733_g0n.pcap

( I didn't know you need to insert two backslashes to get one out ;-) )
$ wireshark -o "ssl.keylog_file: dump_170105_1733_g0n_SSLKEYLOGFILE.txt" \
dump_170105_1733_g0n.pcap

So, are you requesting youtube-dl to be able to write TLS session keys in a file pointed by SSLKEYLOGFILE in order to able to decrypt caps later in wireshark?

Yes, I believe, that is what Wget has recently starting to do, IIUC.
(IIUC: previously it was not the case! Now it is, as anybody can see, also with, say tshark-streams.sh from my:
https://github.com/miroR/tshark-streams repo, also, say:
$ tshark-streams.sh -r dump_170105_1733_g0n.pcap -k dump_170105_1733_g0n_SSLKEYLOGFILE.txt -Y 'tcp.stream==5'
and then:
$ cat dump_170105_1733_g0n_s005-ssl.txt | tail -450 | head -449 > dump_170105_1733_g0n_s005-ssl.html
and anybody would get:
$ <your-browser> dump_170105_1733_g0n_s005-ssl.html
that http-over-tls RFC from the video on that www.CroatiaFidelis.hr page that I linked above.
Would be terribly interested to know how Wget does it, but programming real is still overkill for me...
Wget surely are fine and capable people, but there are capable people here, I'm sure Youtube-dl devs can do it...

What's the format of keylog files? In youtube-dl multiple hosts are involved in each invocation, and, as SSL session resumption (http://bugs.python.org/issue19500) is not implemented in youtube-dl, there are multiple session keys even for the same host. How to generate a keylog file for such scenarios?

By the way, Python does not support exposing session keys yet. A patched Python is necessary.
References:
https://www.openssl.org/docs/manmaster/man3/SSL_SESSION_get_master_key.html
https://hg.python.org/cpython/file/tip/Modules/_ssl.c#l4427

Depends on https://github.com/openssl/openssl/pull/1646, too

There is any updates regarding this issue? Assuming that I'm using the patched Python and the right openssl, this is a solvable problem?

Looks like with Python 3.8 and OpenSSL 1.1.1, keys will be logged if you specify the environment variable $SSLKEYLOGFILE [1][2]. Closing as there is nothing to do in youtube-dl.
[1] https://github.com/python/cpython/commit/c7f7069e77c58e83b847c0bfe4d5aadf6add2e68
[2] https://github.com/openssl/openssl/pull/2287#issuecomment-288369442


Fix Node Selector Resource Plugin

With latest version of Liqo, we don't need to subtract pod resources from node resources. This PR will fix this.

@cheina97 @aleoli Can I get review please? Thanks

Hi @Sharathmk99! Can you explain your changes and why they are needed?

Hi @aleoli,
In version 0.8.x Resource plugin was providing
aggregated node resources - pod usage resources but this logic doesn't work with  latest version of liqo.
With liqo version 0.10.x resource plugin should just provide aggregated node resources without subtracting pod usage resources.
This PR will remove pod resources calculation and provide only aggregated node resources only.

Hi @Sharathmk99, can you please squash your commits before merging? Thanks


User Story: Contact Page with Route

The Contact Page has been created but I am trying to working on the link between the Admin Page and the Contact Page so it is a few bug there.

Closing this pull request


Feature: document the thing

This adds three usage guides:

Overview (basic usage, including standard styles, modifiers, and browser states)
Media queries
Computed styles

Resolves #36
/cc @colinmegill @eastridge @rgerstenberger @aykayen @paulathevalley

Updated based on @eastridge's feedback and API changes since this was opened. Going to merge in now for the sake of keeping things moving. If anyone has any additional feedback or suggestions, please open an issue or submit a PR!


Miscellaneous resources

Added references to the official list of HTTP headers (permanent and provisional) and a link to a list of HTTP/REST APIs standards and specifications

oh. I must have added too many files. It's just regarding miscellaneous.md will correct!


what would happen if the log data has some minutes delay?

If I set the period time as 30 minutes, frequency time as 5 minutes, but the log data would has 6 minutes delay, then what would happen?

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

ID: ab89353c-6acc-9944-40e0-d8ab067e7d41
Version Independent ID: b8c18210-d14f-4b7c-aee9-545cafd20582
Content: Log alerts in Azure Monitor
Content Source: articles/azure-monitor/platform/alerts-unified-log.md
Service: azure-monitor
GitHub Login: @msvijayn
Microsoft Alias: vinagara

@imeya Thanks for the Comment. We are actively investigating and will get back to you soon.


How to deeplink back to wallet app after connection.

Hey there, after connecting the modal, if I try to do a personalSign action, it doesn't redirect me back to the metamask app, I have to manually go back, is this expected behaviour? The signature shows on metamask and deeplinks back to app but no initial deeplink into metamask.

Hi @ZeroTimeDrift - you can use connector.openWalletApp() when you want to navigate back to the Wallet app. Have a look at the example app, it's being used when the testing amount is sent.

Gotcha, thanks for that, it works great but run into issues when doing a sign transaction after awaiting the walletconnect connection.
For context, here is my code:
await initializeProvider(           context,           "https://bridge.walletconnect.org",           'MoonGate',           'An example app showcasing Sign in With Ethereum',           'localhost:3000',           'https://www.joinef.com/wp-content/uploads/2023/03/MoonGate-5-Praneet-Sinha-700x700.png',         );         await signInWithEthereum(             'localhost:3000', 'I love pizza', 'localhost:3000', '1', '1');
        Where signInWithEthereum() calls `openWalletApp()`, are there any callbacks that I can use to ensure that signInWithEthereum is only called once the user is fully back in the app?

You can detect if the app is back in the foreground using WidgetsBindingObserver with didChangeAppLifecycleState method. Would that work for you?

Worked like a charm, cheers @3ph!


Performance degradation in 0.10.0 due to DNS

Hello,
Here, you can see some perf tests done in 0.9.9. Our setup is quite simple: we have a client querying a backend either directly ("Without Kong") or through Kong ("With Kong"). With upstream keep-alive properly set, the latency induced by kong is approxymately 1ms (networking time being negligible in our setup).

We were really looking forward to deploying Kong 0.10 because of the dynamic upstream keep-alive configuration. That being said, we ran our very same performance tests and got those results:

As you can see, latency distribution has clearly shifted right, with the mode of the distribution now being 15ms.  After investigating, it turns out this time is spent in Kong, and more exactly in this call stack:

kong.access()
-> kong.core.before()
--> kong.balancer.execute()
---> toip()
----> resolve()

toip() does a DNS call way too frequently, it seems. Is it a misconfiguration on my side? Is it expected? Is there any issue when caching IPs?
I'm still investigating and will let you know of my results, and I'll be more efficient with your insights :)

@Vermeille what is the ttl of your dns records?

The TTL is 3600

The DNS call is always performed because somehow, upstream_id is always nil here . Continuing my investigations.

Okaaaaaaaaaaaaay. singletons.dao.upstreams:find_all() checks a new /upstreams endpoint that was not mentioned in the UPGRADE.md document and that somehow require special attention. Migration did not apparently do The Expected Thing, which is, if my understanding is correct, migrate upstream_url for APIs to that table.
Should we write a migration procedure?
Is my understanding totally wrong?
Or should the code handle the "base/simple" case (nothing specified in /upstreams)?
I'll gladly help, but need your guidance to fully understand what /upstreams is all about

The DNS call is always performed because somehow, singletons.dao.upstreams:find_all() returns an empty table. That makes upstream_id always nil here . So, get_balancer() returns false . balancer is false and falls down to toip.

This is completely logical, it should as it tests whether the hostname refers to an upstream entity. If it doesn't then there is no ring-balancer, and hence it must invoke the dns resolver. Which results in the toip call.

Okaaaaaaaaaaaaay. singletons.dao.upstreams:find_all() checks a new /upstreams endpoint that was not mentioned in the UPGRADE.md document and that somehow require special attention. Migration did not apparently do The Expected Thing, which is, if my understanding is correct, migrate upstream_url for APIs to that table.

No upstreams is a completely different mechanism to do loadbalancing

Should we write a migration procedure?

No

Is my understanding totally wrong?

Yes 😄

Or should the code handle the "base/simple" case (nothing specified in /upstreams)?

yes. The upstream name is used as a virtual host name, see my previous comment. So it first checks whether a ring-balancer for that upstream is available, if not, it will fall through to regular dns resolution.

I'll gladly help, but need your guidance to fully understand what /upstreams is all about

see https://getkong.org/docs/0.10.x/loadbalancing/

This is completely logical, it should as it tests whether the hostname refers to an upstream entity. If it doesn't then there is no ring-balancer, and hence it must invoke the dns resolver. Which results in the toip call.

Okay, I think I got the general idea. However it seems the IP isn't cached, considering how long name resolution takes. Why so? Why Is it a good default behavior?

(Somewhat off topic) @Vermeille what are you using for load testing and generating those pretty charts? Is your testing script something you could share? Thanks!

@coopr
We are using jmeter. Kong and the upstream server run in docker containers.
Jmeter 3 has super sexy charts. If you can't or don't use jmeter 3, we analyze the csv produced with a python/numpy script. I'll see on monday if I'm allowed to open source that script (I see no reason they refuse, but procedures...)

@Vermeille
kong.access()
-> kong.core.before()
--> kong.balancer.execute()
---> toip()
----> resolve()

toip() will

get a record through resolve()
round robin on that dns record

resolve() will

identify which record type to query (last successful one)
try each record type through lookup() until successful

lookup()

query the cache for the given name and type
if not found query the server

What do your dns records look like? for SRV, A and CNAME queries, eg. dig <name> SRV etc

an interesting experiment would be to add
    order = { "last", "A", "SRV", "AAAA", "CNAME" },

to the opts table here

My DNS is a CNAME that points to an A.
My guess now is that there is a mistake in the cache.
Say I have
a.com CNAME b.com
b.com A <IP_ADDRESS>

When calling resolve("a.com") and the cache is empty, it will perform a DNS resolution. the DNS answer is:
a.com CNAME b.com
b.com A <IP_ADDRESS>

And that will be cached. Meaning the cache now is
"5:a.com" => "CNAME b.com"
"1:b.com" => "A <IP_ADDRESS>"

plus the "last" entry
"a.com" => 1

Which is the mistake.
The next query will then look for "a.com" in the cache, get the last successful type in the cache, which in this case is 1, and check for "1:a.com" in the cache, which doesn't exist. I guess you meant to save a.com => 5 as a last successful query, so that you check the cache for "5:a.com" and recursively to "1:b.com" for the CNAME.

The mischievous line is this one. I guess the correct second argument is not qtype but records[1].type or something like this.

Okay got a quick fix I guess. Certainly not what's best though.
Adding this after this line puts in cache the first A response returned by the DNS, after following CNAMEs.
if answers and (#answers > 0) then
  answers[1].name = qname
  cacheinsert(answers, qname, qtype)
end

By doing so, the cache entry for "1:a.com" is populated.

@Vermeille can you give this branch a try: https://github.com/Mashape/lua-resty-dns-client/tree/fix/cname-caching
check it out using git, and then do a luarocks make from the repo directory.

That works super well for me, thanks!

@Vermeille dunno how much work is involved, but could you share the update graphs from the original post as a comparison?

That fix, my lord, is amazing.

thx! I love the way those graphs look :heart:

You're the real MVP!
Many thanks!


Implement display profile UI

Add UI to display profile
Cleaned up old UI (navigation bar, results page)
Results page now displays 4 personality outputs with collapsible dropdowns
closes #25

TODO:

Replace filler text with actual descriptions of personality outputs

https://user-images.githubusercontent.com/32944104/108580379-1d046480-72e0-11eb-8020-6c51babf1a50.mov

@jenny-l-li Since Jessica undid (is that really the past tense form of undo?) most of her changes from #34, I think that it should be safe for you to merge these changes into master now. The other changes can be added back into the PR for #34 after your changes are merged in.


[thread-cert] print elapsed time of each test when running cert suite

This is helpful to know the exact time that each test take when running bunch of tests in cert suite.

https://github.com/openthread/openthread/actions/runs/10299741646/job/28507637930?pr=10593#step:7:43 shows the how the new log looks like


possible setFilter bugs

I found, I think, 2 'bugs'.

When, using a mouse, you select first item from the list (see previously attached picture) and than programmatically unselect all using selectNothing filter api, first item is still selected. This behavior is not happening if you repeat process with any other item from the filter list.
If you programmatically unselect all items from filter list using unselectValue('unique_filter_name') it works correctly (all items are unselected) but 'Select All' check box is still selected.

http://www.angulargrid.com/forum/showthread.php?tid=2615

this is now working.


[Offworlders] MaxHP limited to 3

The maxhpvalue has a max of 3 and min of -1, it should just be min 0 like hpvalue below?
`

`

Also, ticking 'ship' on the character sheet doesn't seem to persist, always defaults back to normal character.

I submitted a fix for the ship sheet thing, but didn't touch the maxhp thing yet. Am I reading it right that the min should be set to zero, and the max should be removed entirely, as it's by default defined as a number way higher than 3.


Anteupplaya on  yt

its now giving me a new error

okay so the access deny means there's an app or av that's holding it back from saving the file

Okay I ran it as administrator and it gave me the original erro

r

paste the playlist here

https://open.spotify.com/playlist/5Yr3eK6N4bfCWg9j4QBRAw?si=ce30cfed09fc4ff7

it works fine on my end.
Most Likely you'll need to try a vpn

can you recommend a free vpn?

tbh i've never used a vpn haha :( sorry man

damn...is there maybe a setting i can change on the device that may fix the issue?

the old version use to work or you're now using this ?

im now using this. i have a huge list and saw your yt short

Try this temporarily and if it not work. delete
https://protonvpn.com/free-vpn/pc
their mobile app is really good so i gues their pc version should be good too

lol. i just installed that by myself also good news. it is working
thanks for your help, i cant believe how nice you were! id suggest on the github saying you may need a vpn. if you dont have one use protonvpn.

btw heres proof its working

okayyy greaet !!! so it's working i now will close this :)
thanks for suggestion will add it in a few

goodbye!


Safe ADC for pedal position read

The ESP32's ADC peripheral does not guarantee proper voltage detection throughout the voltage range. There have been some discussions  about it and ideas how to fix it. I am thinking about another Pedal implementation that uses and ADS1115, hooked up via SPI. Ideas, experiences, or contributions are welcome.

With the right input circuitry and calibration (see here), I think the on-board ADC is adequate for a vehicle application. Adding an ADS1115 seems like unecessarily high precision for automotive applications.  Also it seems that the ESP32-S3, especially with the mini variant which has a cleaner PCB layout, are better performers than the original ESP32.
On my ESP32-S3 dev kit, just by putting my fingers on the pins, with no front end components at all, the Arduino Plotter shows a reasonable result IMO.


use TRUNCATE TABLE in auditlogflush command

use TRUNCATE TABLE in auditlogflush command

maybe a check on DB type before running truncate?

That would be an option. but please consider the ORM will also follow foreign key references and delete objects in other tables.
So, I would suggest implementing this as a flag in auditlogflush management command. the default behavior would be still delete and by providing the flag, the management command will use the truncate instead.
Please also consider adding:

Tests
Document for the new flag in https://github.com/jazzband/django-auditlog/blob/master/docs/source/usage.rst#management-commands
Changelog in https://github.com/jazzband/django-auditlog/blob/master/CHANGELOG.md

Feel free to open the PR when you are ready.


Automatically release container image for ARM

This PR adds (using buildx and cross compilation) support for ARM arch (https://www.docker.com/blog/faster-multi-platform-builds-dockerfile-cross-compilation-guide/)
Also removes the support to dockerhub
Checklist

[x] Commits are signed with Developer Certificate of Origin (DCO - learn more)
[x] Tests have been added
[x] A PR is opened to update our Helm chart (repo) (if applicable, ie. when deployment manifests are modified)
[x] A PR is opened to update the documentation on (repo) (if applicable)
[x] Changelog has been updated

Fixes https://github.com/kedacore/keda/issues/2263

@JorTurFer are we going to merge this one and give it a try?

Sure, during this week I'll update the PR


Support Discord reply

@lekoOwO we already supported discord replies. What issues do you have with the current implementation?

@lekoOwO we already supported discord replies. What issues do you have with the current implementation?

Um do we have one?
I've set up a matterbridge bot which connects telegram and discord and I dont see my discord reply on telegram until I made this approach.

So no message appears on telegram when using reply on discord?

Message appears on telegram with no reply info.

@42wim discord has changed the way it handles replies, please merge it into master.

+1

just ran into this issue on a new install, looking forward to this pr merged
:+1:

Works perfectly for me.

Yesterday tested master branch without this merge requests. Replying from discord works fine in telegram. Don't understand why we need this change then.

@sleroq can you show us a screenshot? You might be misunderstanding what this PR does


 on CSG substraction  - performance?

error: TypeError: Cannot read properties of undefined (reading 'count')
when try to put <Text3D> from @react-three/drei
What I did was to create a  of three (obsolete). Also working with text in CSG takes a lot of real-time processing.
codesandbox here : https://codesandbox.io/s/fervent-diffie-qr140c?file=/src/App.js

Hi, does anyone know how to improve this performance? I'm subtracting a textGeometry and takes a lot of time of process..

Hello! Any feedback on this issue?

Not from my side. I have limit the amount of text allowed to only 1.. but still running slow


Add support for test artifacts

We want to add the following to an artifact directory for each failed test:

Video of the test from start to finish (using fbsimctl record)
Device log (can be extracted from ~/Library/Logs/CoreSimulator/{udid}/system.log
Print hierarchy with testIDs (we already print that to log on a failed expectation)
Screenshots (I actually believe a video will have a much higher value in understanding what went wrong)

Not just for failure. We should support artifacts in general.

@rotemmiz sometimes screenshots enough. In most cases, video don't need

Artifacts could include the app’s log, screenshots, video of the test run, profiling data, etc. After the test suit finishes, we need to find a way to take this data from the device and transfer it for export. On simulator, this is easy, but it may be challenging on real devices. Xcode is able to take an app’s container data and export it in an archive. Perhaps we could use this somehow.

What about just for now use xcrun simctl io booted screenshot testId.png in the mochajs tests?

Hi Is there an update on this/ETA or a workaround - My tests fails for proxy errors on the server and I would like to take screenshots while its running on the CI to rule out proxy issues.

Please support this ticket guys, we expect detox will be the first pioneer all-in-one toolkit for QA/automation things dedicated in RN world.
Tons of thanks

Thoughts on our current state, and how I see this feature:
I wasn’t able to merge the current implementation since I felt like it’s not scalable enough. Meaning, my initial approach, the one I asked  @donatas to implement (mentioned at the top of this issue), has a serious flaw. It is very hard to add a new artifacts to the system.
Android logs, for instance, were not implemented in this branch, and it seems like it’s going to be very hard to add this capability, let alone adding more artifacts in the future, will cause a code overflow in deviceDrivers.
ArtifactsCopier+ArtifactsPathProvider themselves needs a bit refactoring, and should be merged into one class (?) and maybe name change as well (?), so that it will be able to fully control the artifact creation lifecycle, including file naming and triggering start, stop, move, copy on each artifact handler.

Each ArtifactHandler will have a single responsibility, implementingstart, stop, move, copy methods, for each artifact type.
Let’s take android video artifact handler for example, it should extend the functionality of AndroidArtifact to not only handle copying and moving, but control the whole thing  :
class AndroidVideoArtifactHandlern {
  ...

  async start() {
    //spawn video process creation here
  }

  async stop() {
    //stop the spawned process
  }

  async move() {
    //as currently implemented in AndroidArtifact
  }

  async copy() {
    //as currently implemented in AndroidArtifact
  }
}

something like that, not sure its a good enough approach, we need to test it. I’d like your inputs on that @donatas.
I think I need your help with explaining the caveats, since you’re the only one who encountered all of them, and maybe help designing it.

More considerations in the future:

When we find time to integrate Detox Instruments with Detox, we will need a mechanism to transfer files from and to devices.
When we decide to add support for real devices, we will need a mechanism to transfer user notifications and user activity files to the devices.

Android emu or device will likewise need that functionality for notifications

Before implementing artifacts, I think we need to first implement the file transfer infra for each platform.

Agree with @rotemmiz, current implementation in https://github.com/wix/detox/tree/screenshots-and-videos should be improved. Drivers got polluted with data that could be easily contained in artifact handlers, e.g. AndroidVideoArtifact could hold current process object and handle it's creation and termination. Also, currently Device object is orchestrating recording of screenshots and videos, but these could be delegated to ArtifactsManager which should subsume and replace ArtifactsCopier. New options (--record-videos and --take-screenshots) should be passed as parameters to this ArtifactsManager, thus freeing Device of this knowledge and logic.
What Device should do in the end is to tell ArtifactsManager what to record and what not to (--record-videos and --take-screenshots), pass list of DeviceDriver's artifacts to ArtifactManager, and then tell it when to start, stop, and copy or move those artifacts.

Hello to all interested,
I indeed started my work on the feature, and the current challenge is to refactor it properly.
As it was stated previously, we want to avoid tight coupling of Detox internals with the test artifacts feature, since most of it is about spawning and controlling a handful of CLI processes at the right times.
Also, we might benefit from making methods takeScreenshot('screenshot name') and recordVideo('video name') public, because it opens a possibility for visual comparison tests and other creative uses.
Last but not least, in the interim I've noticed that in the artifacts branch Jest support was overlooked.
As you may know, the artifacts feature, as of right now, relies on the knowledge of current test name and status. Incidentally, it took me a couple of days to figure out a satisfactory way to convey current test name and status into beforeEach and afterEach  functions, and it significantly differs from Mocha way, to my great displeasure.
This omission concerns me in a sense that Detox, being agnostic to test frameworks and runners, at the moment has no explicit contract for an arbitrary test runner. It is true that we did not need that in the past.
In fact, as a user, you were required only to call detox.init(config) in the very beginning and detox.cleanup() in the end.
Unfortunately, the new artifacts feature needs more points of interaction, for instance:

before test runs, get current test name and make the runner wait till the video recording process is successfully spawned;
after the test ends, stop video recording (which is also an async op), ascertain the status of the test (whether it passed or failed), and according to the preferences, decide whether we keep the video or delete it;
since "keeping video" in a case with Android assumes copying files from device (or emulator) filesystem to the local artifacts folder, we'd want to perform that in the background (to avoid blocking the next test execution), but ultimately we'll still need to wait for the all copy operations to complete before we allow the runner itself to terminate.

Altogether, it brought me to the following stance:

We should split the artifacts feature into two parts:

provide imperative API for taking screenshots and recording videos
integrate this API with CLI options and test runner lifecycle in a scope of a separate feature.
I assume both parts can be worked on separately.

While 1.i is a relatively standalone feature, we need to make several preparations before doing 1.ii:

Define Detox <-> Test Runner contract:

detox init -r runnername implementation for scaffolding initial Detox suite with a proper runner config.

detox test -r runnername partial implementation. It regards the part which prepares command-line arguments, environment variables and whatever the runner needs to work, and triggers the process synchronously. Something like (detoxConfig) => cp.execSync("runnername --runnerArgsBuiltFromDetoxConfig...").

within test runner execution context, piece detox options together into a single detoxConfig.
While it is not self-evident, but, for instance, with Jest (which rejects unknown flags), we are forced to pass detox options via environment variables, while with Mocha, we are freely handing command-line arguments as they are. That means that Jest integration should mostly look into process.env, while Mocha integration should be parsing process.argv. To my taste, I would prefer two different and clear implementations, so to say, instead of generic know-it-all detox/src/utils/argparse.js.

provide either a method like setup(detoxTestRunnerHooks), or a particular module which ensures that:

Like in a turn-based game, Detox gets its time to make its moves:

on start, before any test runs;
before a test;
after a test;
on exit, when all tests have already run.

Detox gets information about the current test:

before a test: test name, test full name (optional).
after a test: test name, test status (passed or failed), test full name (optional).

I'm still hesitant about a way to achieve the functionality above. For example, if we go with a setup() method, its encapsulated nature is going to contradict the current Detox approach of straightforward and open to editing setup files like e2e/init.js that Detox scaffolds upon init.
On the other hand, if we go with extending init files, there are problems also. In fact, I could not meet the requirements above (a.a-a.d,b.a-b.b) with less than 50 lines of code for Jest, and that, by itself, ruins the simplicity of startup files and complicates introducing changes in future to these files as they might grow and differ from project to project. This is a place where I'm stuck currently and thinking on a solution.

Extract detox integrations with mocha and jest into two separate modules, e.g.: detox-runner-mocha and detox-runner-jest.

Provide minimum viable API for Detox extensions (plugins):

CLI and package.json options registration and their values' retrieval in runtime.
asynchronous onStart, beforeTest, afterTest, onExit hooks with test information that can be used.
provide runtime info (like current platform, various executables paths) and convenience helper methods to the plugins.

Completing the artifacts feature:

Optionally, extract the whole artifacts feature as a plugin using API in 2.iii.
Alternatively, hard-code CLI hooks without doing 2.iii and wire the methods like (.recordVideo() and .takeScreenshot()) straight to the place where detox gets control from test runner in (before|after)(All|Each) calls. I don't favor this way, but it can serve us as a transit point to the cleaner integration in the future.

It's a bit hard to track my progress since it is quite chaotic movement here-and-there, which is uniformly and messily distributed between three branches: feature/detox-artifacts-plugin, refactor/test-runners, and the original one, plus some local files and quite a recent Jest PoC.
I'll do my best to make it scope of work more focused in the nearest days and will be gladly updating you and listening to your input.
Cheers!

Quick update, guys. We've found a sweet spot between what is best and what is just good enough.
The work is going on in this (last and final) artifacts branch: https://github.com/wix/detox/compare/feature/test-artifacts . To great relief, there's no need to scatter work across branches - I have a focus now (at last :).
Within next couple of days, I'll create a new WIP Pull Request instead of https://github.com/wix/detox/pull/541 , so that my progress will be more visible and transparent to us all. I'll make sure to create a checklist in the PR description and keep it in sync with my progress.
I have a cautious hope to finish work on artifacts quite soon. Still, besides the parts I clearly understand how to accomplish, there are less predictable ones: infrastructure work to improve iOS log recording code, merging low-level parts of iOS screenshotting and screen recording from the former pull request. Maybe we'll postpone implementing Android log recording. Anyway, a few big challenges on structuring have been solved, as well as integration with both Mocha and Jest (as a bonus, detox init -r jest was also implemented).
Many thanks for your patience!
Stay tuned.

@rotemmiz , can we close this?


How to get reference documents of the global-search-result

Is there an existing issue for this?

[X] I have searched the existing issues
[ ] I have checked #657 to validate if my issue is covered by community support

Describe the issue
I want to know how to get reference documents of the global-search-result?
Steps to reproduce
No response
GraphRAG Config Used
# Paste your config here

Logs and screenshots
No response
Additional Information

GraphRAG Version:
Operating System:
Python Version:
Related Issues:

Moving to discussions with a reply: https://github.com/microsoft/graphrag/discussions/931


start fetcher while initializing Fetcher

Description
This PR is ready to review and merge.
Main change with is PR is to enable start the fetcher while launching fetcher service, where aggregators will be filtered by active status.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

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

Checklist before requesting a review

[x] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.

Deployment

[ ] Should publish npm package
[x] Should publish Docker image

@martinkersner Thanks for your comments. I resolved the changes which you highlighted.
I am going to merging the PR.


CORS policy

url: https://assets3.lottiefiles.com/packages/lf20_2SsXhx.json

navigate to the url, download the file to your angular assets folder and then fetch the json file from your development server.


Show curated lists for mainstream browse pages

The links on a mainstream browse page can be curated into lists (see  https://github.com/alphagov/collections-publisher/pull/71).
This PR makes the change to make those lists visible on subsection pages:

Note that when the links are not curated into lists, the links will be displayed as usual, in alphabetical order.
Trello: https://trello.com/c/dkaGFxTT/123

Thanks @edds! https://github.com/alphagov/collections/commit/fbdcf72afda199a5cf4a5e9a0663567905fb4084 addresses the feedback. I've inverted a_to_z? to make it easier to read.

That looks great, for some reason github hasn't updated this pull requests diff.
Is it worth also adding a test for curated_links? and exists??

Good point, those were tested in lists? and indirectly via a controller test. I've pulled them apart and added to sub_section_test.rb.
Rebased & force pushed.

Nice work - and thanks for the reviews, @edds

:+1:

Thanks @edds !


Add sys::socket send and recv

Adds the libc recv and send functions to sys::socket. I wanted this for doing low-level tcp with nix.
Let me know if you need me to improve the patch in some way, or if there's a better way to achieve the same thing. Thanks!

Thanks! merged: e71077d824f8d5e7003c00c0c1e939ba5955be8f


JsonException Syntax error when loading not existing storage key

In a webapp I'm developing, I found this error:
JsonException Syntax error on /vendor/defstudio/telegraph/src/Storage/FileStorageDriver.php in json_decode at line 39

/** @phpstan-ignore-next-line */
return rescue(fn () => json_decode($json, true, flags: JSON_THROW_ON_ERROR), []);
} catch (JsonException|FileNotFoundException) {

Why is the catch block not working?
After throwing this error (into the webhook controller file) the bot didn't reply to the message, the execution stopped.
After enabling debug logs, I saw this warning:
json_decode(): Passing null to parameter #1 ($json) of type string is deprecated in REDACTED/vendor/defstudio/telegraph/src/Storage/FileStorageDriver.php on line 39

This was triggered by calling
$this->message->from()->storage()->get('user', null);

It should be a Sentry bug... It's configured to log that exception even if it's expected.
After ignoring it from config, everything was working fine again.


HibernateDomainExporter and inheritance with composite-id.

Ported from pull request #1460 and issue #1459.

The code looks good now, but the commits are a bit noisy. Would you mind squashing them? Sorry for being so pedantic. :)

I did the best I could. Everything looked good locally, I then had to merge & push, now it looks that there are even more commits. Sorry, I'm no Git guru...

Is this what you intended? I'm no Git expert, :)

That's pretty much what I did and it looks right.
The CI is failing with a "Killed" message. I don't think that it is related to this squash as the log looks right. Can you confirm that is indeed the case?

Yes, sorry about that. Occasionally Travis thinks the Scala compilation uses too much resources, so it kills it.
I'll restart it.

@vveloso Sorry about that. I hadn't refreshed my browser and github was showing the original commits as well as the squashed commit.

@johnktims No problem. It happened to me as well. :)
@Shredder121 It seems that CIs have minds of their own.
Thank you both.

I'll just merge this since it's the same as #1460.
@vveloso Thanks again for the contribution and for taking the time to port it to 4.x.


fix --leader flag for dcos-log [WIP]

currently the command
dcos node log --leader will not get logs from mesos leader node. It will make
a request against dcos_url variable. We should redirect the request to a
leading mesos node unless --component dcos-marathon is used. In this case
we have to redirect the request to a leading marathon master (which can be
a totally different node)

@tamarrow backend doesn't support the endpoints used in this PR yet. We'll have to hold on for a little bit. However this is ready for review

blocked by https://github.com/dcos/adminrouter/pull/37

blocked by https://github.com/dcos/dcos/pull/1182

@darkonie is this unblocked now?

@tamarrow yes this is unblocked! can we merge this?

yep, just waiting for tests to pass


Fix usage with react-native-service-location

Fix usage of navigator.geolocation.getCurrentPosition is null
When use react-native-service-location like
navigator.geolocation = require('react-native-geolocation-service');

When use this library it comes in default property.
navigator.geolocation.getCurrentLocation is undefined but navigator.geolocation.default.getCurrentLocation is currently defined.

Thanks for the PR! I will test and merge next week hopefully.

Published as v1.9.0


[Feature Request] Ability to change the default file extensions when creating a file using a VSCode command

As an example, when using the command to create an entity or animation controller file, it creates .entity.json & .controller.json files respectively.
I prefer using the file extensions of .e.json & .ac.json, so when I use the command I have to manually rename the file each time.
It would be great if there was a section in the VSCode extension settings that allows me to configure the automatic file extensions when creating a file using the commands.
I could just create the files manually or not change the file extensions but the latter isn't really a reasonable solution for me at the moment.
I imagine this could be done in the user settings.json (see below), or maybe in the actual Settings Editor.
    "BC-MC.defaultFileExtensions": {
        "animationController": ".ac.json",
        "animation": ".anim.json",
        "entity": ".e.json",
        "geometry": ".geo.json"
    }

Solved with PR: #387


Deprecate MempoolTransaction Type

Deprecate MempoolTransaction Type
Fix broken test

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#1320  👈
#1319
#1317
#1316
#1315
#1312
#1311
feature/proof-of-stake

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

Join @tholonious and the rest of your teammates on  Graphite

Merge activity

May 14, 6:40 PM EDT: @tholonious started a stack merge that includes this pull request via Graphite.


Segmentation fault before replying

Fresh build today, armv7, running on baremetal.
Backtrace:
(gdb) bt full
#0  0xb6b6b976 in gst_element_set_state (element=0x1, state=GST_STATE_PLAYING) at ../subprojects/gstreamer/gst/gstelement.c:2896
        __inst = 0x1
        __t =<PHONE_NUMBER>
        __r = <optimized out>
        oclass = <optimized out>
#1  0x7f5e7544 in genie::AudioPlayer::resume (this=0x80b60a00) at ../src/audioplayer.cpp:407
No locals.
#2  0x7f5e784e in genie::STT::on_message (conn=0x80b732a8, type=1, message=0x813f6c20, data=0x80b8cc50) at ../src/stt.cpp:94
        obj = 0x80b8cc50
        status = 400
        sz = 34
        ptr = 0x811e5fb8 "{\"status\":400,\"code\":\"E_NO_MATCH\"}"
        parser = 0x81267170
        reader = 0x80cbb890
        __func__ = "on_message"

Logs:
Conversation id: main
Start speak text: Hello! How can I help you?
[02:22:19.37448] on_message (../src/wsclient.cpp:153): Start speak text: Hello! How can I help you?
[02:22:19.61684] dispatch_queue (../src/audioplayer.cpp:357): gst pipeline started
[02:22:19.178266] bus_call_queue (../src/audioplayer.cpp:89): Stream status changed
[02:22:19.178685] bus_call_queue (../src/audioplayer.cpp:89): Stream status changed
[02:22:20.793676] bus_call_queue (../src/audioplayer.cpp:89): Stream status changed
[02:22:24.197840] bus_call_queue (../src/audioplayer.cpp:93): End of stream
[02:22:24.197929] bus_call_queue (../src/audioplayer.cpp:94): Pipeline elapsed: 5136153us
[02:22:24.956941] loop (../src/audioinput.cpp:234): detected keyword
detected keyword
Gst element could not be created
[02:22:27.936971] dispatch_queue (../src/stt.cpp:209): STT xfer elapsed: 1025007us
[02:22:27.937101] loop (../src/audioinput.cpp:256): STT last frame
STT status 400
Gst element could not be created
Resume current pipeline
Segmentation fault (core dumped)

corefile-genie-1035-1629138148.gz

This should've been addressed with the latest master


can install waldur

when i install with
helm install waldur waldur
and got result
Error: INSTALLATION FAILED: unable to build kubernetes objects from release manifest: unable to recognize "": no matches for kind "ClusterIssuer" in version "cert-manager.io/v1alpha2"
and check cert-manager.io

Hi!
Could you please provide a version of your k8s cluster and apiVersion of the ClusterIssuer?

Hi!
Could you please provide a version of your k8s cluster and apiVersion of the ClusterIssuer?

Fixed here: https://github.com/waldur/waldur-helm/commit/287498233d9a8e79b6b5fc1f3d593ea9b45ae465


Hide dismissal button when restrictDismissal is set to true

App Review is starting to reject my updates for two reasons:

In the permission prompt screen, I allowed users to dismiss it. Apparently this is a no-no. It was solved by setting restrictDismissal to true.
Now that user's cannot dismiss the screen, this is the rejection notice:
"Specifically, the cancel (“x”) button on the permission screen was still present but unresponsive."

So essentially, Apple wants us to hide the dismissal button completely. This is blocking me from submitting an app update

@khuffie did you get your app submitted with this library?

@xpnewmedia - I created a branch with the source code and did a fix to hide the button, which allowed it to go through the approval process.

Fixed by PR 5395d233b21d190950ed254700961bcd9298f5f0


Tweak POI and landuse zoom filters

Change the zoom / area filter for a bunch of different features based on @nvkelso's feedback. There's probably still a whole bunch of tweaking to do, but this is a step in the right direction.
Connects to #195. See the parent issue for screenshots.

@rmarianski Can you review, please?

:+1:
We're adding the min zoom to the label feature. I've got some pull requests open that will auto-generate the label features though (issue https://github.com/mapzen/tilequeue/issues/21). @zerebubuth Should the min zoom calculation become a part of that post process step?

Should the min zoom calculation become a part of that post process step?

I'm not sure. We'd have to be careful about making sure the min_zoom calculation in the post-processing step matched the queries. Otherwise we'd end up missing stuff, or having stuff with min_zoom=N in tiles at zoom N-1 or something.

@zerebubuth Can we move it to the landuse layer? If we wanted, we could trim that property out in the post process transform except for label features.

Sure. If we add it to the landuse, and the calculation is the same for the label and the polygon (or at least the min of the two) then we can always strip stuff out later. I'm just concerned that we'll end up with "missing" features and end up bouncing between the two implementations; one in the database and the other in Python.

If we add it to the landuse, and the calculation is the same for the label and the polygon (or at least the min of the two) then we can always strip stuff out later.

Yeah. Sometimes there will be less labels than parent features, but never the other way around.

@rmarianski could you review, please?

:+1:


How to make it compile with "--cudart=shared" when running luarocks install

Thank you very much.

Try:
TORCH_NVCC_FLAGS="-cudart=shared" luarocks ...

Sorry, wrong description, actually it does compile with --cudart=shared but I don't know how to make it use /usr/local/rCUDA/lib instead of /usr/local/cuda/lib64 .
Thanks a lot

You will probably have to muck with how CMakeLists.txt finds the CUDA library. You could do something like:
SET(CUDA_TOOLKIT_ROOT_DIR /usr/local/rCUDA/) before calling FIND_PACKAGE(CUDA ...) in the top-level CMakeLists.txt file.

Alternatively, try using LD_PRELOAD=/usr/local/rCUDA/lib/libcudart.so.7.5 th some_script.lua

Thank you all for your tips.
TORCH_NVCC_FLAGS="--cudart=shared" do the job but as there's no such env var for cunn/cudnn cudart is not shared for these luarocks.
LD_PRELOAD is fine (although deepLearning frameworks like Torch aren't supported by rCUDA yet).


defect #432027 - [LA] Jenkins doesnt send logs of foldered jobs

Changed the code to support folder jobs, and also made some additional refactoring so that we will handle the full job path in only one place.
JENKINS-46711

java doc will be set as seperate pr.


Fix applyMatrix in lightning.gpu

Before submitting
Please complete the following checklist when submitting a PR:

[ ] All new features must include a unit test.
If you've fixed a bug or added code that should be tested, add a test to the
tests directory!

[ ] All new functions and code must be clearly commented and documented.
If you do make documentation changes, make sure that the docs build and
render correctly by running make docs.

[ ] Ensure that the test suite passes, by running make test.

[x] Add a new entry to the .github/CHANGELOG.md file, summarizing the
change, and including a link back to the PR.

[x] Ensure that code is properly formatted by running make format.

When all the above are checked, delete everything above the dashed
line and fill in the pull request template.

Context:
[SC-74899]
lightning.gpu stores gate data to a unordered_map object. The gate data can be accessed with a key (opsName, Param) . This design can avoid the excessive data copy between the device and host. However, the way a matrix gate is stored causes the bug since lightning.gpu will only add first matrix data to the cache since the keys of all matrix will be the same ({},{}) pair. Following applyMatrix calls will just result in getting data from the cache and the matrix used for operation will be always the first matrix.
Description of the Change:
Benefits:
Possible Drawbacks:
Related GitHub Issues:

Looks good to me! Thanks @multiphaseCFD!


gorm 分组查询文档描述用法的问题

Your Question
//这是官网对分组查询的描述
https://gorm.io/docs/advanced_query.html#Group-Conditions
db.Where(  db.Where("pizza = ?", "pepperoni").Where(db.Where("size = ?", "small").Or("size = ?", "medium")),).Or(  db.Where("pizza = ?", "hawaiian").Where("size = ?", "xlarge"),).Find(&Pizza{}).Statement

// SELECT * FROM `pizzas` WHERE (pizza = "pepperoni" AND (size = "small" OR size = "medium")) OR (pizza = "hawaiian" AND size = "xlarge")

但实际上不是这样的 我测试后的结果是
//实际结果
SELECT
	*
FROM
	`test_models`
WHERE
	pizza = 'pepperoni'
	AND size = 'small'
	OR size = 'medium'
	AND ( pizza = 'pepperoni' AND size = 'small' OR size = 'medium' )
	AND (
		pizza = 'pepperoni'
		AND size = 'small'
		OR size = 'medium'
	AND ( pizza = 'pepperoni' AND size = 'small' OR size = 'medium' ))
	AND pizza = 'hawaiian'
	AND size = 'xlarge'
	OR (
		pizza = 'pepperoni'
		AND size = 'small'
		OR size = 'medium'
		AND ( pizza = 'pepperoni' AND size = 'small' OR size = 'medium' )
		AND (
			pizza = 'pepperoni'
			AND size = 'small'
			OR size = 'medium'
		AND ( pizza = 'pepperoni' AND size = 'small' OR size = 'medium' ))
	AND pizza = 'hawaiian'
	AND size = 'xlarge')

//正确的写法
db.Where(
            db.Session(&gorm.Session{NewDB: true}).Unscoped().Where("pizza = ?", "pepperoni").Where(db.Where("size = ?", "small").Or("size = ?", "medium")),
        ).Or(
            db.Session(&gorm.Session{NewDB: true}).Unscoped().Where("pizza = ?", "hawaiian").Where("size = ?", "xlarge"),
        ).Find(&pizza)

The document you expected this should be explained

Expected answer

Most of the documents do not describe what db is, but the default db refers to a shareable db instance, which can be created through gorm.Open or created through .Session

After a Chain method, Finisher Method, GORM returns an initialized *gorm.DB instance, which is NOT safe to reuse anymore, or new generated SQL might be polluted by the previous conditions, for example:

https://gorm.io/docs/method_chaining.html#content-inner


backgroundColor returns warning for having a capital letter in strict mode

Warning: React does not recognize the backgroundColor prop on a DOM element. If you intentionally want it to appear in the DOM as a custom attribute, spell it as lowercase backgroundcolor instead. If you accidentally passed it from a parent component, remove it from the DOM element.
in img (created by Badge)
in Badge (at Landing.js:52)
in div (at Landing.js:51)
in div (at Landing.js:47)
in div (at Landing.js:45)
in label (at Landing.js:44)
in section (at Landing.js:42)
in LandingPage (at App.js:49)
in Router (created by BrowserRouter)
in BrowserRouter (at App.js:49)
in Switch (at App.js:46)
in div (at App.js:44)
in App (created by Context.Consumer)
in withRouter(App) (at src/index.js:11)
in Router (created by BrowserRouter)
in BrowserRouter (at src/index.js:11)
in StrictMode (at src/index.js:10)

Yup this occurs because its just passing the props straight through to the img element. It needs to remove the custom props like backgroundColor, logoColor, name and label. Ill get it fixed asap.


Refactor Scripter examples using syncContinue

The method syncContinue is obsolete, but there are still many usages of it in the system.
Sends of syncContinue should be removed, and steps should have play added to them.
Essentially code that looks like this:
request_focus
	<gtExample>
	| aScripter |
	aScripter := self scripterWithCompletion.
	aScripter
		substep: 'Request focus'
		do: [ :aStep |
			aStep requestFocus; onSelf ].
	^ aScripter syncContinue

should be rewritten to look like this:
request_focus
	<gtExample>
	| aScripter |
	aScripter := self scripterWithCompletion.
	aScripter
		substep: 'Request focus'
		do: [ :aStep |
			aStep requestFocus; onSelf ];
		play.
	^ aScripter

Refactored the examples in GtWorldByScripterExamples.


Roll engine e52287a1b557..62b9594d2167 (3 commits)

https://github.com/flutter/engine/compare/e52287a1b557...62b9594d2167
git log e52287a1b5572c73b19d085ccbe404f0aeddb5f6..62b9594d21673b8988587ff3ab02222029da38e6 --no-merges --oneline
62b9594d2 Roll src/third_party/skia 4683942a45f6..76616a10a786 (8 commits) (flutter/engine#7365)
23d087420 Roll Dart to version ec86471ccc47a62df8b4009e1fb37c66ff9dc91b (flutter/engine#7363)
78f6eb5c8 Roll src/third_party/skia 3b7e65107f67..4683942a45f6 (10 commits) (flutter/engine#7362)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (amirha@chromium.org), and stop
the roller if necessary.

Trybots failed. These were the failed builds: https://cirrus-ci.com/task/5232222925750272

Commit queue failed; closing this roll.


ScrollViewer hijacks touch pointer captures

Describe the bug
When controls that rely on dragging elements are placed within a scroll viewer, touch events cease to be captured outside said controls. In other words - when dragging a slider within a scroll viewer via touch the drag ceases if touch leaves the confines of the slider track. This does not happen with mouse pointers. When control is not inside a scroll viewer - behavior is as expected.
To Reproduce
Steps to reproduce the behavior:

Create a slider within a scroll viewer
Observer behavior on a mobile or touch device

Expected behavior
When dragging a slider, I should have a "capture" so that I'm allowed to move my finger outside the confines of the small slider track while still controlling the slider until I release my touch.
Desktop (please complete the following information):

OS: Windows 10 & Android
Version 11-preview4

Slider doesn't seem to mark the event as handled, so ScrollViewer has no idea that it shouldn't handle the event. I suppose the problem here is in the Slider code.

@kekekeks It's a good thing you mentioned this. I noticed the same thing, but I found this issue originally with a custom slider control I made from scratch that does handle the events, and I still have the same issue.

PointerEventArgs now have PreventGestureRecognition() which is called from Thumb


Fix Control Panel: Feat/service metadata

What this PR does:
https://github.com/apache/dubbo-go-pixiu/pull/540
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

Codecov Report

Merging #570 (70dedeb) into develop (c0bf34e) will increase coverage by 0.05%.
The diff coverage is 50.00%.

:exclamation: Current head 70dedeb differs from pull request most recent head 9decde0. Consider uploading reports for the commit 9decde0 to get more accurate results

@@             Coverage Diff             @@
##           develop     #570      +/-   ##
===========================================
+ Coverage    54.63%   54.69%   +0.05%
===========================================
  Files          671      671
  Lines        78589    78589
===========================================
+ Hits         42940    42983      +43
+ Misses       31981    31947      -34
+ Partials      3668     3659       -9

Impacted Files
Coverage Δ

...networking/dubbo/v1alpha1/servicemetadataserver.go
0.00% <0.00%> (ø)

pilot/pkg/bootstrap/server.go
54.46% <100.00%> (+0.74%)
:arrow_up:

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


Implement parallel writes

This is a big feature that would allow Etcher to be able to stream the same image to multiple drives at the same time. This feature should be implemented by extending the already existing etcher-image-write NPM module. The module currently accepts the path to a drive device as an argument to the main function. If the user passes an array of drives, the multi-writer logic should come in.
Some of the challenges for this task are:

Cross platform support.
Validating the drives at the moment the writing starts.
Handling validation correctly for all of them (there are some quirks to make validation work in certain operating systems, like in OS X, where we have to use OS X specific frameworks to prevent the drives from being mounted right when the flash ends, so the operating system doesn’t write files like .DS_Store that messes up the checksum).
Handling error conditions correctly (e.g: if a certain drive fails, the rest should continue).
Making sure the whole stream is streamed to all devices (e.g: no data is lost because it has been piped to another drive).
Imposing a reasonable limit of drives that can be written at the same time, depending on the capabilities of the operating system to prevent potential kernel panics, etc. If a user tries to write to more images than he should, we should make of use of a queue.

The module returns an EventEmitter instance which emits state information. When multiple writings are taking place, the emitted events could have a property containing the drive they correspond to.
It's important that this feature contains extensive end to end tests.
This project should not contain native addons by any means, since it's used by other projects that are installed by end users without necessarily having a toolchain in place.

And I guess this will also need to cope with some of the drives being written to having much slower write speeds than others (e.g. a Class4 SD card in a USB2 cardreader vs. a Class10 SD card in a USB3 cardreader). So perhaps there needs to be separate decompression queues for each drive, rather than trying to stream the same decompressed data to each of the drives? Unless of course you're happy for the whole process to just run at the speed of the slowest drive.

Hey there,
That's spot on. We've experimented with opening different file
descriptors from the input image, and handling decompression separately,
which helps avoiding faster drives being bottlenecked by slower drives,
however that also means that when uncompressing many multiple streams
(think >20), the CPU usage rises a lot, and we're basically duplicating
computation.
Another thing we found out is that performing much I/O on a single
NodeJS process (even with different file descriptor) slows things down a
lot, given the single-threaded nature of NodeJS. Seems like the way to
go is to spawn multiple NodeJS processes.
On Thu, Sep 29, 2016 at 05:30:10AM -0700, Andrew Scheller wrote:

And I guess this will also need to cope with some of the drives being written to having much slower write speeds than others (e.g. a Class4 SD card in a USB2 cardreader vs. a Class10 SD card in a USB3 cardreader). So perhaps there needs to be separate decompression queues for each drive, rather than trying to stream the same decompressed data to each of the drives? Unless of course you're happy for the whole process to just run at the speed of the slowest drive.
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/resin-io/etcher/issues/267#issuecomment-250452357

--
Juan Cruz Viotti
Software Engineer

Yeah, and I guess by running all-drives at the speed of the slowest drive, you only need a single progress bar, rather than separate progress bars for each drive ;-)

@jviotti I think we can push this off the v1 release.

Done!

Lets try using web workers with node integration for this: https://github.com/electron/electron/pull/8852

Resolved by #2124

I've flashed about 20 cards now and I have noticed that the time taken more or less scales linearly with the transfer speed of one card. e.g. if I have 3 cards that are independently capable of flashing @21MB/s, flashing 3 of them simultaneously has them proceed at almost exactly 7MB/s, making it so that I have to wait 3 times as long for the batch. Now, this is a process improvement, however I am confused as to why there is no speedup as you would see if you just ran concurrent dds on multiple drives.
This is using a MacBook with multiple usb-c to sd card adapters.

Same issue, same setup with @unphased on Etcher Version 1.5.100. Any way to improve speed to full in parrallel, @jviotti ? That would be great.

I've flashed about 20 cards now and I have noticed that the time taken more or less scales linearly with the transfer speed of one card. e.g. if I have 3 cards that are independently capable of flashing @21MB/s, flashing 3 of them simultaneously has them proceed at almost exactly 7MB/s, making it so that I have to wait 3 times as long for the batch. Now, this is a process improvement and already a big win, however I am confused as to why there is no speedup as you would see if you just ran concurrent dds on multiple drives. The reason I use etcher is so I can avoid building a dd runner and figuring out how to properly verify a disk, but this seems... not ideal.
This is using a MacBook with multiple usb-c to sd card adapters.


Request for inverse of forced only

I feel it would be very handy to have a way to inverse the -D command.  Something like a --non-forced option that would export everything but the forced subtitles.  That way if forced subtitles are burned into a movie file and subtitles are turned on you don't get two copies of the subtitles over the top of each other for any parts with forced subs.

I as well would love to see this option as I prefer to burn in my forced subtitles as kcchris mentioned.  In order to make everything look right that means I have to manually tell removed all the forced subs from the export (which works just slow and very manual).


Purge unused CSS via postCSS

Is this a BUG REPORT or FEATURE REQUEST?:
feature
What happened:
Currently the full CSS file is delivered even unused parts.
What you expected to happen:
Purge unused CSS parts to reduce delivery size.
Anything else we need to know?:
Might be possible via https://gohugo.io/hugo-pipes/postprocess/#css-purging-with-postcss

This works quite easily but it requires installation of npm packages on user websites. We can add it to our exampleSite and use it in our demos, it'll reduce the css size by around 80% (~58KB -> ~11KB). I'll create a PR adding the feature and we can add an explanation on how to do so in a new section in Documentation when we address #761. What do you think?

So it would be a configuration feature, if we wanna call it that. Basically the feature is already available besides the resources.Postcss call within Hugo.
So Syna change would be adding the resources.PostCSS hook and then add docs on how to configure it on Hugo's site.
As this needs a js prerequisite I feel like that makes sense. On that note. Should we add in more hooks that could be used by our users or is PostCSS the main one?
Should we document more or add a documentation section on hooks? Aka there are various hooks one can use such as PostCSS hook to purge css or do RTL etc.?
Basically having the feature/config documented, but also a general hooks section linking back to the various features it enables on Syna side? Does that sound reasonable? Also probably agree to push this for after #717 761.

Also damn that change is nice. This paired with fontawesome size decreases with selective fonts would do great for performance. Desktop subsetter is still not available for Linux so. :roll_eyes:
Once we have this I'm happy to push this to the various netlify configs so we optimize the speed.


Routing: setArgument() is not taking arrays any more

Hello,
upgrading from V3 to V4 we suddenly receive this error message:
Argument 2 passed to Slim\\Routing\\Route::setArgument() must be of the type string, array given, called in ...

Example (only for showing the system; namespaces and values renamed for public presentation):
$app->group("/demo", function(RouteCollectorProxy $grp) {
    $grp->post("/test/{token}", "App\TestController:test")
        ->setArgument("param1", false)
        ->setArgument("param2", array("value1" => "N", "value2 => 15, ...))
        ->setName("test1");
});

That's "a bit" annoying for us, because we pass arrays as parameters to the route very successfully in our running slim app.
A quick change is not the best option: Our App contains several thousand routes and more files. We currently give service to also several 1000 customers with our app and are not is the position to "quickly" rebuild the whole routing.
Any idea or solution ?

Use setArguments()


I2C read error is not handled correctly in TT21100 driver

I2C read error is not handled correctly as ret_val == ESP_OK and data_len > 0 is not checked here:
https://github.com/espressif/esp-box/blob/c46079095e4115282babbddb78179af547b6d548/components/i2c_devices/touch_panel/tt21100.c#L122
This will cause that tt21100_read will be called with data_len == 0
https://github.com/espressif/esp-box/blob/c46079095e4115282babbddb78179af547b6d548/components/i2c_devices/touch_panel/tt21100.c#L123
https://github.com/espressif/esp-box/blob/c46079095e4115282babbddb78179af547b6d548/components/i2c_devices/touch_panel/tt21100.c#L33
esp_err_t i2c_master_read(i2c_cmd_handle_t cmd_handle, uint8_t *data, size_t data_len, i2c_ack_type_t ack)
{
...
    ESP_RETURN_ON_FALSE(data_len > 0, ESP_ERR_INVALID_ARG, I2C_TAG, I2C_DATA_LEN_ERR_STR);

You are right. It's strange that we haven't encountered data_len=0 yet. Anyway, it'll be fixed soon.

Hi, if you don't have any questions, please close this issue. Thank you!


Automated validation of PR against a set of pre-defined guidelines

We have @orcabot for that - currently working on fully extending it and will update the issue once it's deployed.

@orcabot is now in action at #148

@orcabot can now merge PRs with the help of approved community members (see #150)


Navigation Pane HTML

The Navigation pane is incorrectly showing HTML values for page hyperlinks for several of the links:

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

ID: 553d3ade-8cba-49b6-b83a-36088034ca2f
Version Independent ID: 553d3ade-8cba-49b6-b83a-36088034ca2f
Content: Analyze data in list pages using data analysis mode - Business Central
Content Source: business-central/analysis-mode.md
Service: dynamics365-business-central
GitHub Login: @jswymer
Microsoft Alias: jswymer

Thanks for reporting this. Seems like the problem has been fixed.


Change color for Tooltip on hover

The Dashboard and Server link in the menu does not have the right color on the Tooltip while hovered. At least if it should be consistent with how the rest of the items behave while hovered (IE Iam, Settings and Logout)
Also I replaced the location of the two Tooltip lines in the code where I find it more logical to have them. Hope I didn't miss any obvious reason why they might have been placed there in the first place :)

Again thanks for this magical app!

Thanks for the PR! 💜
I have a small question (see above).


Poke Location Scraper

Create an auto scraper for pokemon data (removes need to create datasets by hand)

Created Pokémon location scraper using python. Works at least for 1st gen games, unsure whether or not it would work for future ones. Takes a long time per run due to scraping constraints listed on bulbapedia’s robot.txt file


Restore RSSIntegrationTest

Refer to this link for build results (access rights to CI server needed):
https://fabric8-ci.fusesource.com/job/wildfly-camel-pullreq/211/
Test PASSed.


Adding cartridge subcommand to CLI and bat tests

This is dependent on
https://github.com/Accenture/adop-platform-management/pull/18
Once that is supported, I want people to be able to boostrap cartridges from a template inside their repo using this job:
$ adop cartridge init -h
usage: adop cartridge init []
Options:
-s           : Name of subfolder create the cartridge in (Optional) (Default : adop)
-t
HINT: Run 'adop cartridge init' to create a cartridge in the current directory
HINT: If you are looking to load a cartridge, instead try: adop platform cartridge
I think this is pretty exciting as it hopefully lowers the barrier to people creating cartridges - in particular when the templates get better.

Hi @kramos
I've tested this and it works as expected :+1:
Couple of points to note:

There is a typo in this line https://github.com/Accenture/adop-docker-compose/pull/134/files#diff-95d916467777772a62b21d85774164c7R60 where it should say for -t "Do not combine with -c" but it says "Do not combine with -t"
The -p flag is given in the help, when it's not actually supported and comes up as an illegal option https://github.com/Accenture/adop-docker-compose/pull/134/files#diff-95d916467777772a62b21d85774164c7R61

Everything else if fine, tested in both regular bash and Git bash

LGTM.