[SPARK-27633][SQL] Remove redundant aliases in NestedColumnAliasing

What changes were proposed in this pull request?
In NestedColumnAliasing rule, we create aliases for nested field access in project list. We considered that top level parent field and nested fields under it were both accessed. In the case, we don't create the aliases because they are redundant.
There is another case, where a nested parent field and nested fields under it were both accessed, which we don't consider now. We don't need to create aliases in this case too.
How was this patch tested?
Added test.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/10417/
Test PASSed.

Test build #105128 has started for PR 24525 at commit f24b1d5.

Test build #105128 has finished for PR 24525 at commit f24b1d5.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/105128/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/10418/
Test PASSed.

Test build #105129 has started for PR 24525 at commit 578663c.

Test build #105129 has finished for PR 24525 at commit 578663c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/105129/
Test PASSed.

cc @dongjoon-hyun @cloud-fan

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/10425/
Test PASSed.

Test build #105136 has started for PR 24525 at commit a009d3e.

Test build #105136 has finished for PR 24525 at commit a009d3e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/105136/
Test PASSed.

Retest this please.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/10546/
Test PASSed.

Test build #105267 has started for PR 24525 at commit a009d3e.

Test build #105267 has finished for PR 24525 at commit a009d3e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/105267/
Test PASSed.

ping @dongjoon-hyun

@viirya . Actually, I've been investigating this PR multiple times from the first day. But I couldn't make me sure about this approach.
One big thing is nestedFields.forall(f => f == n || child.find(_.semanticEquals(f)).isEmpty). semanticEquals is not a robust choice in Struct related operations. It erases many information and frequently it bites us at some corner cases.
I'm interested in this PR and testing this. Could you test this more seriously, too?

Thanks for review @dongjoon-hyun
I understood your concerns. Thanks for test, also. Can you provide few cases that you think might be problematic, if you find any?
In case here, the intent is to see the child of a nested field accessor is already presented. Do you think it is more robust, if comparing them exactly equally, not semantically?

Test build #105622 has started for PR 24525 at commit 412401b.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/10873/
Test PASSed.

Test build #105622 has finished for PR 24525 at commit 412401b.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/105622/
Test PASSed.

Test build #105649 has started for PR 24525 at commit 3fd4cc5.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/10904/
Test PASSed.

Test build #105649 has finished for PR 24525 at commit 3fd4cc5.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/105649/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/13875/
Test PASSed.

@cloud-fan @dongjoon-hyun Please let me know if you have more comments or thoughts on this? thanks.

Test build #108795 has started for PR 24525 at commit 4ff4112.

@viirya . I has been not sure about this PR due to the old reason, but I don't have a counter example you asked. Sorry about that. If the other committers think okay, I will not be against this PR. Thanks for working on nested column improvement, @viirya .
cc @cloud-fan , @dbtsai , @gatorsmile

Test build #108795 has finished for PR 24525 at commit 4ff4112.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/108795/
Test FAILed.

retest this please.

@dongjoon-hyun I see. Thanks for the comment. Do you more agree on this if changing semanticEquals to exact comparing? Can it resolve your concern?

Test build #108845 has started for PR 24525 at commit 4ff4112.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/13919/
Test PASSed.

It's been just my personal concern, @viirya . You may not need to revise your PR~

Test build #108845 has finished for PR 24525 at commit 4ff4112.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/108845/
Test PASSed.

cc @cloud-fan Do you have more thoughts on this change?

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/14435/
Test PASSed.

Test build #109372 has started for PR 24525 at commit 7a18a0a.

Test build #109372 has finished for PR 24525 at commit 7a18a0a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/109372/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/15699/
Test PASSed.

retest this please.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/19126/
Test PASSed.

Test build #114283 has finished for PR 24525 at commit 7a18a0a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/114283/
Test PASSed.

retest this please

Test build #119144 has started for PR 24525 at commit 7a18a0a.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/23886/
Test PASSed.

Test build #119144 has finished for PR 24525 at commit 7a18a0a.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/119144/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/23939/
Test PASSed.

retest this please

Test build #119199 has started for PR 24525 at commit 7a18a0a.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/23940/
Test PASSed.

@HyukjinKwon Thanks for reopening this. I think I need to sync it up with latest change.

Test build #119199 has finished for PR 24525 at commit 7a18a0a.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/119199/
Test FAILed.

Test build #119206 has started for PR 24525 at commit a027946.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/23947/
Test PASSed.

Test build #119206 has finished for PR 24525 at commit a027946.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/119206/
Test FAILed.

retest this please

retest this please

Test build #119213 has started for PR 24525 at commit a027946.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/23954/
Test PASSed.

Test build #119213 has finished for PR 24525 at commit a027946.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/119213/
Test PASSed.

ping @cloud-fan

@viirya, can you rebase this please? I merged a couple of your PRs and seems it caused the conflicts.

@HyukjinKwon rebased, thanks.

Test build #123950 has started for PR 24525 at commit 40521aa.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/28572/
Test PASSed.

Test build #123950 has finished for PR 24525 at commit 40521aa.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/123950/
Test FAILed.

retest this please

Test build #123974 has started for PR 24525 at commit 40521aa.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/28597/
Test PASSed.

Test build #123974 has finished for PR 24525 at commit 40521aa.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/123974/
Test PASSed.

Test build #124009 has started for PR 24525 at commit fb1d3f3.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/28630/
Test PASSed.

Test build #124009 has finished for PR 24525 at commit fb1d3f3.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/124009/
Test PASSed.

Merged to master.


Add language line for file validation rule

See: https://github.com/laravel/laravel/commit/cd032040441787c827aa07c428ae753281b685df

Thanks :)


Avoid filling the ThreadNet mempools with random transactions

Fixes #2581. At least as stop-gap until PR #2472.
I don't like estimating the transaction sizes and/or throttling the size of a batch of randoms. It all seems much too pessimistic.
For example, for the repro I was working with, the average tx per block fell from 10-20 to 2-3. These are random transactions, so they're not particularly important, but more seems better than less regarding code coverage.

The only attractive alternative we've considered is the discussion on #2581. That path would add to the Mempool API so the node can check if a sequence of hypothetical transactions would be valid or invalid regardless of the current size of the mempool. If I understand correctly, this is already loosely planned functionality. That functionality would let us mitigate #2581 because we're fairly certain that the mempool only fills up far after all the crucial transactions are relevant (in RealTPraos at least; not sure about RealPBFT, but we haven't seen errors related to full mempools there). So even if the mempool were full, we'd still be able to see immediately that the crucial tx would be rejected (eg because it's already on the chain).
I proved that out on this branch, for comparison. https://github.com/input-output-hk/ouroboros-network/tree/nfrisby/issue-2581-precheck-crucials

Closed, favoring PR #2643 instead.


Investigate MDAL

The Mesh Data Abstraction Library (MDAL) is able to read the HEC-RAS mesh format, as well as number of other hydrodynamic mesh formats.
Investigate MDAL. Consider:

Should we introduce MDAL as a dependency to rashdf?

Is it able to correctly handle the sorts of edge cases that we're aware of?

If so, how would we integrate it? What would it take to convert a mesh extracted from MDAL to a GeoDataFrame?

MDAL Python API

For MDAL exploration, a sample data file in the repo was used as the data input "test.p01.hdf". The library is pretty handy reading the file and different objects generated can be convert to GeoDataframe. The objects often have a method returning their values as numpy.ndarray. However, I tried to use MDAL to open some of the HEC-RAS p01.hdf files on my laptop but failed. MDAL can load the file but was unable to display any information regarding the mesh. Not sure what the cause is. This requires further investigation into input files.
I basically ran through the code example on PYPI and repo, plus digging into the source code for more methods. I documented some methods/objects in this library that we might find useful and codes used to transform MDAL object into GeoDataframe.
MDAL Exploration Notes.docx
Please feel free to check and let me know what you think!

@qijuechenwsp can you elaborate on the HDF files on your laptop that failed? What errors did you get? Was there anything strange about those models? (They were 2D models, right?)

They are .p01.hdf and .g01.hdf from 1D HECRAS model - the output hdf should be 2D. And the test.p01.hdf in the repo is under folder 1D area so I think 1D output hdf should be a valid input here. I don't have any 2D model output files to test.
The error shows below - I found the error defined in the code but couldn't locate which line reports it


nip19.decode() does not return in some case

I found that following code does not return and eating memory. (using Node.js v18.14.0, nostr-tools 1.10.0)
require('nostr-tools').nip19.decode('nevent1qq5v4rkz')

I think here (parseTLV) is the thing happening.
Since with above example l (rest[1]) is undefined, and 2 + l = undefined, it does rest.slice(undefined) which is noop, and v.length < l is false and pushing v to result.  (eating memory.) And rest is still not changed, loop continues forever.
I expect it throws an Error if such broken (incomplete) input is given.
I've tested this with older nostr-tools but I think it still left on current master. (bb5acfc19764d1c4e46bba6b1065ef4a25121951)

Lol this is catastrophic. Simply requesting a URL on my server with this nostr ID nevent1qq5v4rkz makes the CPU go up to 100% and never come down until the server is restarted.

Before the request:

After making the request and even closing the tab:

No way. This must be a joke.

I confirmed it throws "Error: malformed TLV 0" for nevent1qq5v4rkz with nostr-tools 1.11.2
Thanks for quick fix!


initial implementation of extractor usage report

Adds a new /api/reports/metrics/extractors endpoint with since and until flags, e.g.
http://localhost:9000/api/reports/metrics/extractions?since=2020-08-12T18:05:45.582Z
Due to database design (we don't store user data in the extractions table) we have to get those data from the datasets to make that association and the query is likely to be slow and taxing - we should discuss.

To test this, unless you have a very new database from last month or so, you need to run update script to add user_id to your extractions:
mongo clowder --quiet scripts/updates/UpdateUserId.js
Then you can hit the api:
http://localhost:9000/api/reports/metrics/extractors?since=2020-11-01T18:05:45.582Z


Updated Booking form

Enhanced Booking Form UI with Hover Effects and Improved Styling
In this update, I have enhanced the booking form's UI to improve the user experience and visual appeal. The following changes were made:

Hover Effect: Added a scaling hover effect to the entire form, making it grow slightly when hovered. This helps draw attention and provides a more interactive feel.

Border and Shadow: Introduced a border and shadow to the form for a clean, modern look. The shadow intensifies on hover, adding depth.

Button Styling: Updated the "Book now" button with a new color scheme and hover effect that changes its background color and adds a subtle transformation.

Input Focus: Inputs and select fields now have a focus state with a border color change for better accessibility.

Responsiveness: Improved the responsiveness of the form to ensure it adapts well on smaller screens.

Fixes: #173
Before:

After:

Kindly merge my PR and  I've added Responsiveness too,... so could you please make it level2...
Thank you the oppurtunity :)

@neeru24 Please see the conflicts highlighted in the image above. Resolve the conflicts in the mentioned files. Ensure you pull all the latest changes before pushing your feature/bug fixes to the main branch. After resolving the conflicts, please open a new PR for review.
Thank you!

Hii,
I didn't get like why this appears highlighted?
Can you please help me with it like is it because of alignment?


v10 cannot find a way to sync production to local storage emulator - destination is always considered as a directory on disk

Hi when trying to do:
azcopy 'https://xxxxxxxx.blob.core.windows.net/media?<SAS Query String Goes Here>'  'http://<IP_ADDRESS>:10000/devstoreaccount1/media' --recursive
It fails as it tries to append the local running directory before the destination url 'c:\xxx\xxxx\xxx\<IP_ADDRESS>:10000/devstoreaccount1/media'
I've tried using the --from-to flag but there doesn't seem to be a blob to storage emulator or blobblob option..

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

ID: 15911160-5ba2-fb0a-0b24-16b3857e20ce
Version Independent ID: d925bc9f-f825-2f82-168c-6c30d716b9ce
Content: Copy or move data to Azure Storage by using AzCopy v10
Content Source: articles/storage/common/storage-use-azcopy-v10.md
Service: storage
Sub-service: common
GitHub Login: @normesta
Microsoft Alias: normesta

@owlyowl Thanks for your question. We are checking on this and will respond to you soon.

Thanks very much :)
On Mon, Dec 9, 2019 at 3:56 PM SumanthMarigowda-MSFT <
<EMAIL_ADDRESS>wrote:

@owlyowl https://github.com/owlyowl Thanks for your question. We are
checking on this and will respond to you soon.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/44278?email_source=notifications&email_token=AAOT4CSSR5ZUHH4TW36VX7TQXXFW7A5CNFSM4JYCZV4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEGH2QPA#issuecomment-563062844,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAOT4CXG3IHNSBTF5AXDOVDQXXFW7ANCNFSM4JYCZV4A
.

@owlyowl Apologies for delay in responding here!
At the moment, using the storage emulator is broken. This is a known issue that has an open PR. https://github.com/Azure/azure-storage-azcopy/pull/743 should fix this.
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Timestream endpoint url not present

We are using wrangler in a lambda within a vpc to communicate with timestream. When trying to connect to timestream using vpc  endpoints we are unable to connect due to not being able to set the endpoint url for timestream.
We would like to request enabling timestream-write and timestream-query url endpoints so wrangler can be used to connect to timestream using vpc endpoints as mentioned in this aws blog:
https://aws.amazon.com/blogs/database/use-vpc-endpoints-with-amazon-timestream/

Hi @ben-w1 thanks for bringing this to our attention. Personally, I'm in agreement that this is a necessary enhancement and we will get this on the roadmap.


Investigate testnet calldata carto logging

All transfers on testnet have logged no calldata (0x). What is wrong here?

On mainnet we are capturing xcall calldata in the tables

I see plenty of non-0x calldata transfers:

I think you were looking at testnet staging which might not have any calldata transfers.


Questions about Resampling primitive

For resize_blinear algorithm,  tensorflow defines:

align_corners: An optional bool. Defaults to False. If true, the centers of the 4 corner pixels of the input and output tensors are aligned, preserving the values at the corner pixels.

I wonder whether the resampling support align_corner when using Bilinear algorithm.

Resampling primitive behaves in the same way as Tensorflow 2.0's resize implementation. This implementation does preserve the image corners similar to TF 1.x with align_corners=True, but interpolation result will be different. Looking at the docs TF 2.0 behavior matches TF 1.x with align_corners=True and half_pixel_centers=True.

@vpirogov I got it! Thanks for your explanation.


[AutoPR automation/resource-manager] Remove automation parameters from client

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/2741

This PR has been merged into https://github.com/Azure/azure-sdk-for-go/pull/1411


Allow params and query in get_task_logs()

Make get_task_logs() use 'api/case/task/log/_search/' API endpoint. This allows specifying params and also query. Current implementation of this method only returns 10 case task logs at max. I believe this addresses issue #160

@nadouani I see 1.7.2 has been tagged last week. When do you estimate 1.8.0 to happen? I'm asking because I need this functionality, and if that's a 'reasonable wait', I'll just wait. But if that's multiple weeks, I'll have to write some work around code for now. Thank you!

Hello @jnahorny I don't know if you are using TheHive4py, could you update your PR to be up to date so I can merge it? Thanks

Hello @jnahorny I don't know if you are using TheHive4py, could you update your PR to be up to date so I can merge it? Thanks

Hi @nadouani
Yes, I am using TheHive4py all the time. I've rebased my PR on the top of the current state of 1.x branch. I'm not sure why here in the PR it shows 7 files modified :/
When I visit https://github.com/TheHive-Project/TheHive4py/compare/1.x...jnahorny:allow-params-and-query-in-get_task_logs
it correctly shows just one files as modified, with only my changes.
Git is so simple, but sometimes I can't figure out what is it trying to tell me :)

If I did something incorrectly, I'll try to fix it :/

@nadouani sorry, but I had to delete this PR. I've messed something up, and wasn't able to fix it.
I've created a new PR, number 200 (https://github.com/TheHive-Project/TheHive4py/pull/200). This one looks properly, it only changes one file.
Again, I'm very sorry for that!


Gradius galaxies hard locks after start message printed

Dumping out the core after 1500 frames I see it in a tight loop in IRQ mode running code in 0x0300_xxxx region doing two lines:
03000904: E1C310B8 - strh r1, [r3, #8]
03000908: E169F000 - msr spsr_fc, r0
0300090C: E12FFF1E - bx lr=0x0300_0904

So we're stuck in a daft loop with IRQs disabled storing r1 (1) into 04000208 (IME) to enable interrupts.
At this point PPU interrupts are enabled and all are active for some reason, presumably because the code to clear them hasn't run.
Presumably LR is wrong here, we'd expect that this code is part of the ISR and that it returns to BIOS with BX LR at the end of executing. It does the BX LR but LR is just an earlier instruction! So what splatted LR??

IRQ line going high IE=VBL,HBL,VCount IF=VBL
IRQ line going high IE=VBL,HBL,VCount IF=HBL
IRQ line going high IE=VBL,HBL,VCount IF=HBL
IRQ line going high IE=VBL,HBL,VCount IF=HBL
IRQ line going high IE=VBL,HBL,VCount IF=HBL
IRQ line going high IE=VBL,HBL,VCount IF=HBL
IRQ line going high IE=VBL,HBL,VCount IF=HBL
IRQ line going high IE=VBL,HBL,VCount IF=HBL
IRQ line going high IE=VBL,HBL,VCount IF=HBL
IRQ line going high IE=VBL,HBL,VCount IF=HBL
IRQ line going high IE=VBL,HBL,VCount IF=HBL
IRQ line going high IE=VBL,HBL,VCount IF=HBL
IRQ line going high IE=VBL,HBL,VCount IF=HBL
IRQ line going high IE=VBL,HBL,VCount IF=HBL
IRQ line going high IE=VBL,HBL,VCount IF=HBL

That's interrupt logs from the section where the failure happens.
VBL,HBL and Vcount IRQs are all enabled and the specific interrupt which is "last" is just a HBL one same as all the others so is presumably going through much the same code.
We can work out the exact scanline this is happening on by just counting things. There were 14 HBL interrupts after the VBL one. VBLank triggers on line 160 and hblank happens later on the same line. There are 68 vbank lines so this is just "somewhere in VBlanking" not line 0 or anything interesting like that

Notably disabling hblank IRQ firing (by commenting out the relevant ppu line) makes this issue go away although the ship goes unbearably slowly.
Next question is - "what is hlank DMA actually doing". Might need to do some disassembling to find out!

Asking the question "when is R14 set to the RAM value that causes the loop" gives me:

 r0:6000003F   r1:00000001   r2:00012007   r3:04000200
 r4:03004410   r5:03000070   r6:00000009   r7:00000000
 r8:03000028   r9:00000000  r10:00000000  r11:00000000
r12:0805DCFA  r13:03007F88  r14:03000904  r15:00000018
cpsr: 00000092 -----I- Arm Irq
spsr: 00000092 -----I- Arm Irq
Cycle: 129600697
Pipeline not refilled

Which is really weird beacuse I think it means we have reentrant IRQs except that the IRQ disable flag should be set for the duration of IRQ handling in Gradius (as far as I can tell)

Ooh, this might be a genuinely interesting bug. I've got a breakpoint on my "service interrupt" routine which fires when the return address is 0x0300_0904 (that dodgy R14). When that fires it's clear it comes from the IRQ servicing routine and that on the cycle it fires the CPSR IRQ Disable bit is set.
I was trying to nail down the implementation of the IRQ synchronizer in #55 and as part of that there's a 3 cycle IRQ sync delay, in my implementation the IRQ is only cancelled if the IRQ disable bit is set during the first 2 of those cycles.
Testing what happens if we allow the IRQ disable bit to affect IRQ all the way through to the last cycle and TADA Gradius gets in game.

Timer Countup and irq tests both still pass the same number (#55) after that change, not surprising. However Fleroviux's irq delay test has now stopped showing identical values as hardware.
I still don't think I have a really good handle on what happens during the sync process and what latching occurs for various IRQ registers so I'm not exactly sure what to make of this.

this is pretty much the only image I can find describing an IRQ synchronizer (and it's for the arm cortex)


Localazy 1.0.2

What to do if the build fails?
At the moment contributors do not have access to the CI workflow triggered by StepLib PRs. In case of a failed build, we ask for your patience. Maintainers of Bitrise Steplib will sort it out for you or inform you if any further action is needed.
New Pull Request Checklist
Please mark the points which you did / accept.

[x] I will not move an already shared step version's tag to another commit
[x] I read the Step Development Guideline
[x] I have a test for my Step, which can be run with bitrise run test (in the step's repository)
[x] I did run bitrise run audit-this-step (in the step's repository - note, if you don't have this workflow in your bitrise.yml, you can copy it from the step template.)
[x] I read and accept the Abandoned Step policy

Hi @vaclavhodek,
If i get it correctly, this PR is the combination of #2774 and #2775. Is it ok for you if we only merge this and not the other two?

Hi @vaclavhodek,
If i get it correctly, this PR is the combination of #2774 and #2775. Is it ok for you if we only merge this and not the other two?

Yes, this one is enough. It contains the icon and fixed description as well as removed list of supported platforms.


Auth: Generator should build a config object for feathers-authentication

Selecting an auth provider should add the config for that provider to the environment configs.

Something like
auth: {
  secret: 'master-token-generation-secret',
  local: {
    loginEndpoint: '/login'
  },
  facebook: {
    appId: '149b0wv1n4v4',
    secretKey: '1g444g209n'
  },
  twitter: {
    appId: '240v9ndsvn23',
    secretKey: '1g444g209n'
  }
}

Related to and depends on feathersjs/feathers-authentication#6.

This is fixed in the app-architecture branch.

Still in the works but this stuff is now in master.

Damn this is actually pretty hard with yeoman. @daffl any chance that we can change our config files to be js files rather than JSON? I need to import the strategy somehow. The typical config for OAuth is like this:
"facebook": {
  "strategy": FacebookStrategy,
  "clientID": "your-facebook-client-id",
  "clientSecret": "your-facebook-client-secret",
  "permissions": {
    "scope": ["public_profile", "email"]
  }
},


Segmentation fault

compiled on commit made on feb 19,2016 still not running
/opt/home/admin # unbound -v
[1455965500] unbound[9751:0] notice: Start of unbound 1.5.8.

Segmentation fault
/opt/home/admin #

commit Feb 24, 2016
/opt/home/admin # unbound -v
[1456336813] unbound[28821:0] notice: Start of unbound 1.5.8.
Segmentation fault
/opt/home/admin #

Unfortunately, "Segmentation fault" is not enough to have a clue about what is wrong.

I have no knowledge on stack trace.

If only there is tag version, we can eliminate the issue. Currently using unbound version 1.5.8, file created on 12/2/15.
Is there a way to roll back, so I can see what commit failed?

Sure, you can use git checkout <commit id> to switch the current view to any commit ID.

Ok, after exhaustive compilation, error start on commit dated Jan 26, 2016.
Currently running Jan 25, 2016 compiled version.

commit made on march 3, 2016
fixed segmentation issue. Tnx.


Git: failed to show text document

With todays insider build I am often seeing this error when jumping from a diff to the source file or vice versa:

I can't reproduce.
Can you narrow down? Are you doing the jump through the editor title action? How many editor views do you have open?

Reproduced with 2 editor panes.

Bisect returns 4339a247abcdd7072336cd1731298cbca4c472b1 as the cause for this.

Nice catch! Listeners were not registered for the already instantiated code editors, so the models and editors tracker was not updating when the workbench would call .setModel() on those code editors, thus failing in correlating things.


Could not find 'php-cs-fixer'. The program may not be installed.

I got this issue when i try to format the html page that contain PHP. Anyone got solution for this ?

Please follow the issue template provided. More specifically, adding a link to the required debug.md Gist which includes debugging information that answers our most commonly asked questions: https://github.com/Glavin001/atom-beautify/blob/master/ISSUE_TEMPLATE.md#how-to-create-debugmd-gist
Thank you.

I really want to focus on improving the installation experience for users. I have created a new Issue, #1687, to target this problem. Please provide your feedback! Thanks in advance.


[fixed] element with display 'contents' is visible and is tabbable

Fixes #905.
This PR is a follow-up of https://github.com/reactjs/react-modal/pull/906 addressing the PR feedback.
Changes proposed:

element with display 'contents' is visible and is tabbable.

Upgrade Path (for changed or removed APIs):

none

Acceptance Checklist:

[ ] The commit message follows the guidelines in CONTRIBUTING.md.
[ ] Documentation (README.md) and examples have been updated as needed.
[ ] If this is a code change, a spec testing the functionality has been added.
[ ] If the commit message has [changed] or [removed], there is an upgrade path above.

@aloker @diasbruno I have opened this PR just so we have the latest master on the branch. I've also addressed the PR comments on https://github.com/reactjs/react-modal/pull/906.
I'd appreciate if you guys could take a look and review it, please 🙏

Awesome, @galexandrade.
Everything looks great. I'll merge and make a release tomorrow.

Awesome, @galexandrade. Everything looks great. I'll merge and make a release tomorrow.

Thank you. I appreciate


Fix race condition with search e2e test

In the search e2e tests, the returns empty results test has a race condition between clearing the input and typing in the next search term.
The test is suppose to search for "empty" which is setup to return an empty set of results, but at times it only types "ty" or "mpty".

I haven't seen this crop up since the change you made, I did make one small tweak for the term "empty" though since it's now returning results.

I think this is now fixed in newest PR.


add gvisor based service library

This adds a gvisor based service library which allows for using a user space TCP stack instead of a native tun device.
This is an initial implementation and only handles TCP and ipv4 connections. This implementation is a bit rough so feedback is welcome :slightly_smiling_face:
This refactors the Main function to take in a DeviceFactory -- so any users that require tunFd need to use NewFdDeviceFromConfig(tunFd) as the factory instead
See examples/go_service/main.go for how this might be used
Test plan:
$ go test ./service

Manual
$ go run ./examples/go_service
$ socat <IP_ADDRESS> -
hello world
asdf
echo: "asdf"
duck
echo: "duck"

I'm OK with this landing. It adds some nice, useful functionality, modifies the Main() interface a little, and doesn't appear to me to add any interfaces that we can't live with going forward.
Will see if @wadey or @nbrownus have any thoughts on it.

Error: service/service.go:111:10: undefined: context.AfterFunc

My bad...apparently context.AfterFunc in go 1.21.0, but Nebula isn't on that golang version yet; it reduces Windows supported O/S's. Sorry! Will need to implement that functionality without that context function...

I believe this PR would close #538.

@maggie44 I'm currently using this for my own (limited) use and it's been working just fine. You're right that ipv6 isn't supported by Nebula currently so that's just behavior as usual so just UDP isn't supported.
I've been using this for SSH and HTTP connections without any issues


make it easy to switch between types of work

types could be: "work", "meeting" - things that one want to distinguish

fixed and merged


[WIP] Drop Result dependency

This PR drops Result dependency

Bump ReactiveSwift dependency to 6.0 that also drops Result

I wasn't able to update Package.resolved since of Alamofire, so it may be correct to rebase this PR on #1810.

Tried and faced no problems with a big project 👍

Will this be merged?

@metisdev I really want to have it merged. But not sure about Alamofire dependency. We may need to wait new Alamofore version to be merged. In other case SPM cannot be used.
I also want some feedback from core team

@larryonoff yeah we need to finish Alamofire transition first. Then we would focus on this one. I’m really excited about droping Result in the next version, though.

@sunshinejr awesome! I really looking forward to the update. I also would like to bump ReactiveSwift and RxSwift dependencies.


In-app update check (Restart and install) does not work

Describe the bug
update check does not work
To Reproduce
Steps to reproduce the behavior:

Go to settings
Click on check for updates

Expected behavior
update gets downloaded
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):
Ferdi: Version 5.4.1-beta.4 (5.4.1-beta.4.952)
OS: macOS Catalina
{"host":{"platform":"darwin","release":"19.3.0","screens":[{"accelerometerSupport":"unknown","bounds":{"height":1120,"width":1792,"x":0,"y":0},"colorDepth":24,"colorSpace":"{primaries_d50_referred: [[0.6818, 0.3196],  [0.2846, 0.6747],  [0.1562, 0.0554]], transfer:0.0777*x + 0.0000 if x < 0.0450 else (0.9495*x + 0.0495)**2.3955 + 0.0003, matrix:RGB, range:FULL}","depthPerComponent":8,"id":69734662,"internal":false,"monochrome":false,"rotation":0,"scaleFactor":2,"size":{"height":1120,"width":1792},"touchSupport":"unknown","workArea":{"height":1097,"width":1792,"x":0,"y":23},"workAreaSize":{"height":1097,"width":1792}}]},"ferdi":{"version":"5.4.1-beta.4","electron":"7.1.6","installedRecipes":[{"id":"android-messages","version":"1.2.0"},{"id":"googlekeep","version":"1.0.0"},{"id":"messenger","version":"1.0.8"},{"id":"skype","version":"1.1.1"},{"id":"slack","version":"1.0.9"},{"id":"telegram","version":"1.0.5"},{"id":"whatsapp","version":"1.0.17"},{}],"devRecipes":[],"services":[{"id":"382d78ee-b143-4043-a461-acc0e2351d88","recipe":"whatsapp","isAttached":true,"isActive":true,"isEnabled":true,"isHibernating":false,"hasCrashed":false,"isDarkModeEnabled":true},{"id":"2537052a-2047-424e-9ee5-1e014f29622c","recipe":"slack","isAttached":true,"isActive":false,"isEnabled":true,"isHibernating":false,"hasCrashed":false,"isDarkModeEnabled":true},{"id":"c4db7b48-691f-4192-b08b-5a07dea10d30","recipe":"slack","isAttached":false,"isActive":false,"isEnabled":false,"isHibernating":false,"hasCrashed":false,"isDarkModeEnabled":""},{"id":"487b879d-5471-4760-9af1-59358d6e7a7b","recipe":"messenger","isAttached":false,"isActive":false,"isEnabled":false,"isHibernating":false,"hasCrashed":false,"isDarkModeEnabled":false},{"id":"55b43572-1733-4bb7-99a3-7edfe9ae16c5","recipe":"telegram","isAttached":true,"isActive":false,"isEnabled":true,"isHibernating":false,"hasCrashed":false,"isDarkModeEnabled":true},{"id":"47a7366d-2bc0-467a-b610-eeee1594b027","recipe":"skype","isAttached":false,"isActive":false,"isEnabled":false,"isHibernating":false,"hasCrashed":false,"isDarkModeEnabled":false},{"id":"39fc1e9d-cd05-42cf-90f3-ebe7d13da49e","recipe":"android-messages","isAttached":false,"isActive":false,"isEnabled":false,"isHibernating":false,"hasCrashed":false,"isDarkModeEnabled":""},{"id":"4816b9e8-d38e-4e1b-b902-96593813efed","recipe":"googlekeep","isAttached":true,"isActive":false,"isEnabled":true,"isHibernating":false,"hasCrashed":false,"isDarkModeEnabled":""},{"id":"bfb78fe2-9a55-4145-b3b7-0845b87892e3","recipe":"slack","isAttached":true,"isActive":false,"isEnabled":true,"isHibernating":false,"hasCrashed":false,"isDarkModeEnabled":false},{"id":"612e9f70-9ad6-45d0-b51d-d9536feec9e8","recipe":"slack","isAttached":true,"isActive":false,"isEnabled":true,"isHibernating":false,"hasCrashed":false,"isDarkModeEnabled":true}],"messages":[{"type":"error","request":{"result":null,"wasExecuted":true,"method":"getInfo"},"error":{},"response":{},"server":"https://api.getferdi.com"},{"type":"error","request":{"result":null,"wasExecuted":true,"method":"features"},"error":{},"response":{},"server":"https://api.getferdi.com"},{"type":"error","request":{"result":null,"wasExecuted":true,"method":"all"},"error":{},"response":{},"server":"https://api.getferdi.com"},{"type":"log","info":["log","[ServerApi::getRecipePackage] Downloading recipe from server"]},{"type":"log","info":["log","/Users/stan/Library/Application Support/Ferdi/recipes/temp/telegram-react/recipe.tar.gz"]},{"type":"error","info":["error",{"errno":-2,"syscall":"open","code":"ENOENT","path":"/Users/stan/Library/Application Support/Ferdi/recipes/temp/telegram-react/package.json"}]},{"type":"error","info":["error","Warning: You cannot PUSH the same path using hash history"]}],"workspaces":[],"windowSettings":{"width":1421,"height":866,"x":192,"y":147,"isMaximized":false,"isFullScreen":false,"displayBounds":{"x":0,"y":0,"width":1792,"height":1120}},"settings":{"autoLaunchInBackground":false,"runInBackground":false,"enableSystemTray":false,"minimizeToSystemTray":false,"privateNotifications":false,"showDisabledServices":false,"showMessageBadgeWhenMuted":true,"enableSpellchecking":false,"spellcheckerLanguage":"en-us","darkMode":true,"locale":"en-US","fallbackLocale":"en-US","beta":true,"isAppMuted":false,"enableGPUAcceleration":true,"serviceLimit":5,"server":"https://api.getferdi.com","todoServer":"https://app.franztodos.com","autohideMenuBar":false,"lockingFeatureEnabled":false,"locked":false,"lockedPassword":"******","scheduledDNDEnabled":false,"scheduledDNDStart":"17:00","scheduledDNDEnd":"09:00","hibernate":false,"hibernationStrategy":300,"noUpdates":false,"showServiceNavigationBar":false,"universalDarkMode":false,"adaptableDarkMode":true,"accentColor":"#7367f0","serviceRibbonWidth":68},"features":{"isSpellcheckerIncludedInCurrentPlan":true,"needToWaitToProceed":false,"needToWaitToProceedConfig":{"delayOffset":3600000,"wait":10000},"isServiceProxyEnabled":true,"isServiceProxyIncludedInCurrentPlan":true,"isAnnouncementsEnabled":true,"isWorkspaceIncludedInCurrentPlan":true,"isWorkspaceEnabled":true,"isCommunityRecipesIncludedInCurrentPlan":true,"isSpellcheckerPremiumFeature":false,"isServiceProxyPremiumFeature":true,"isWorkspacePremiumFeature":false,"isSettingsWSEnabled":false,"isServiceLimitEnabled":false,"serviceLimitCount":0,"isCommunityRecipesPremiumFeature":false,"isCustomUrlIncludedInCurrentPlan":true,"isMagicBarEnabled":true,"isTeamManagementIncludedInCurrentPlan":true,"isTodosEnabled":true,"isTodosIncludedInCurrentPlan":true,"defaultTrialPlan":"franz-pro-yearly","subscribeURL":"https://getferdi.com","planSelectionURL":"https://getferdi.com","hasInlineCheckout":true,"isPlanSelectionEnabled":false,"isTrialStatusBarEnabled":false,"canSkipTrial":true,"pricingConfig":{"currency":"$","currencyID":"USD","plans":{"personal":{"monthly":{"id":"ferdi-free","price":0,"billed":0},"yearly":{"id":"ferdi-completely-free","price":0,"billed":0}},"pro":{"monthly":{"id":"ferdi-still-free","price":0,"billed":0},"yearly":{"id":"ferdi-forever-free","price":0,"billed":0}}}}},"user":"82c1cf9d-ab58-4da2-b55e-aaa41d2142d8"}}
Additional context
Add any other context about the problem here.

Seems related to https://github.com/electron-userland/electron-builder/issues/4583.

Or could it be because of the mishaps that took place between these tags?

Maybe some clients first picked up the deleted releases/versions and got stuck once they were removed? Adding this as a note to double check in the source code later if that could have happen.

Can confirm it happens on macOS 10.14.6.
In 5.4.3, the update button showed Update available, downloading... for only a couple frames (had to screen record to read it). Then, it goes back to normal.
In 5.4.4-beta.2, the behavior was different for me. The update button changed to Update available, downloading... and after a minute-ish, it went back to normal without performing any update.

Reviewing priorities 🙏 See https://github.com/getferdi/ferdi/issues/536#issuecomment-619562782

I had a successful update download from the nightly.11 to nightly.12. No ideas how that happened, will have to double check but took a screenshot for reference.

Until this bug fixed, do we just download the latest version manually? Will installing it automatically overwrite the old version?

Update: here's the console output when clicking "Check for updates":

What's the chmod 'C:\' about?

@lukefisklennon You will have to update manually for now, yes 🙏 I think it will override the old version, but I am not 100% sure.

Regarding your second question, are you using Ferdi without an account?
Can you please update to 5.5.0 manually and see if you still see this chmod?
Thank you!

This issue is still occurring.
Currently on v5.5.0.
In the setting with "Check for betas" checkbox selected, the 5.6.0-beta.3 does not show as visible.

Still does not work. It now says "downloading" but then nothing happens

@Stanzilla - what version of Ferdi are you trying to upgrade from? At some point in between 5.6.betas, the upgrade checker was broken, and to "jump past that build", you need to manually upgrade. Also, if you are on mac, the signed vs unsigned issue for Apple's security will not allow an app (that is not downloaded from the Apple App Store) to autorun through the installation process.

Windows and upgrade was from 5.6.0-beta.5 to beta.6

mac does not work either, I just tried updating from .7 to .8 and when you hit update & restart it just closes and does not update nor restart

Yes - this is a known issue #1476 and I am unable to find the fix since we are using the in-app updater from electron-updater module.

Might be related (need to test): https://github.com/electron-userland/electron-builder/releases/tag/v22.14.6

@kytwb - as you can see there are numerous tickets being raised for this. Could you please prioritize fixing of this ticket soon?

I switched from Ubuntu (.deb) to Linux Manjaro (AppImage) recently and this does work for me now (or always worked?)
I get the "Update & Restart" prompt, then once it quits, I get a prompt asking me to overwrite the current Ferdi install with the latest nightly. After confirming, the latest version starts.
So AppImage seems to work as expected, but macOS .dmg does not.

5.6.4 deb just installed.
If I click on the button to search for upgrades, nothing happens.
Good job! ;D

Still not working for macos

Not working for Linux either.
By the way, the download link for Ubuntu on https://getferdi.com/download/ yields a HTTP 404 error.

The following commits are related to the in-app updater and landed in 5.6.5: https://github.com/getferdi/ferdi/commit/547d462243b26b3c1ee1dcf040f7d63081a1a4af, https://github.com/getferdi/ferdi/commit/8ba714e856420da09620a7d55d6412f5e8c5c027, https://github.com/getferdi/ferdi/commit/18494f66a7e0f4020cbc77374d4679421fcf469a, https://github.com/getferdi/ferdi/commit/01062664af1a9564d50759d1c23ab849376a6615.

Dear sender,
due to a ransomware attack and necessary security measures, CGM is temporarily not reachable in some regions via the usual eMail addresses and landline numbers.
In these cases, please feel free to use SMS and mobile telephony of your individual contact persons, as far as they are known to you or inform yourself about further contact possibilities via the respective company or product page.
Thank you for your understanding and best regards,
Your CGM Team

Sehr geehrte Absenderin,
sehr geehrter Absender,
aufgrund einer Ransomware-Attacke und notwendiger Sicherheitsmaßnahmen, ist CGM in einigen Regionen vorübergehend nicht über die gewohnten eMail-Adressen und Festnetznummern erreichbar.
 Produktseite über weitere Kontaktmöglichkeiten.
Vielen Dank für Ihr Verständnis und freundliche Grüße,
Ihr CGM-Team

Running 5.6.5 on latest Mac OS 12.1, and just received the update notification for 5.6.6. The "restart and update" button would not work, but then I noticed that the app would not actually terminate. So I disabled the "Keep Ferdi in background when closing the window" option, and then tried again. That time the update worked w/o any issues.
I hope that this helps.

Dear sender,
CGM is temporarily not reachable in some regions via the usual eMail addresses and landline numbers.  In these cases, please feel free to use SMS and mobile telephony of your individual contact persons, as far as they are known to you or inform yourself about further contact possibilities via the respective company or product page.
Thank you for your understanding and best regards,
Your CGM Team

Sehr geehrte Absenderin,
sehr geehrter Absender,
CGM ist in einigen Regionen vorübergehend nicht über die gewohnten eMail-Adressen und Festnetznummern erreichbar.   Produktseite über weitere Kontaktmöglichkeiten.
Vielen Dank für Ihr Verständnis und freundliche Grüße,
Ihr CGM-Team

@all-contributors please add @klonos for userTesting

Updating from 5.6.6 to 5.6.7, the update was downloaded but clicking on Restart & install update failed with the following error in the console: Updater error: Error: The network connection was lost.
Using a VPN fixed it for me.

5.6.7 Ubuntu 64.
Still hangs on "searching for updates".
I really don't come to terms with developers.

Still relevant: https://github.com/getferdi/ferdi/issues/512

Updating from 5.6.6 to 5.6.7 on MacOS has the same problem. Disabling the "Keep Ferdi in background when closing the window" option is the workaround.
@kytwb I see that you have closed this issue. Does that mean that it has been fixed in 5.6.7? I don't see anything relevant in the changelog.

@klonos yes, fixed via https://github.com/getferdi/ferdi/commit/610923e48784352fcb45c6a56a185682690c16a3

Cool 👍thanks.

I can't confirm this is fixed with "Keep Ferdi in background when closing the window" turned on. I removed Ferdi, installed 5.6.9 and tried to run the update which failed until I unchecked the option as @klonos described.

@kytwb I can't confirm this is fixed with "Keep Ferdi in background when closing the window" turned on on macOS 12.1 (arm). I removed Ferdi, installed 5.6.9 and tried to run the update which failed until I unchecked the option as @klonos described.

I have the same problem.

@kytwb check this: https://github.com/electron-userland/electron-builder/issues/1604
It probably  makes sense to run
if (focusedWindow != null) {
    focusedWindow.close()
}

as suggested there (and as per unchecking the "background when closing the window" fixes the issue)

@Privatecoder the docs says:

Under the hood calling autoUpdater.quitAndInstall() will close all application windows first, and automatically call app.quit() after all windows have been closed.

It's not necessary to manually call params.mainWindow.close() here, but removing the close listener from mainWindow should help.

@kytwb you're right about the docs however I cannot confirm that updating Ferdi works after commit 610923e, i.e. build 5.6.9+ to any newer version. I will build it locally, check if I can find something and report back

@Privatecoder I have pushed a fix via https://github.com/getferdi/ferdi/commit/820a92de96652967e809e7f3ba163a4654e40ffc.

I just checked the commit and must admit that I did not read properly what you wrote in your previous answer.
Yes, removing the handler before calling quitAndInstall() should indeed fix it (also with that handler in place my proposed solution would not have worked either as it would have been intercepted by named handler..)
Thanks @kytwb


error

paper 2105.06070v1 1804.04829v2 2201.07422v1
Traceback (most recent call last):
  File "/home/wzp/anaconda3/envs/python3.8/bin/paper", line 45, in <module>
    check_update()
  File "/home/wzp/anaconda3/envs/python3.8/lib/python3.8/site-packages/arxiv_dl/updater.py", line 28, in check_update
    latest_version = check_latest_version()
  File "/home/wzp/anaconda3/envs/python3.8/lib/python3.8/site-packages/arxiv_dl/updater.py", line 10, in check_latest_version
    response = requests.get(pypi_url)
  File "/home/wzp/anaconda3/envs/python3.8/lib/python3.8/site-packages/requests/api.py", line 73, in get
    return request("get", url, params=params, **kwargs)
  File "/home/wzp/anaconda3/envs/python3.8/lib/python3.8/site-packages/requests/api.py", line 59, in request
    return session.request(method=method, url=url, **kwargs)
  File "/home/wzp/anaconda3/envs/python3.8/lib/python3.8/site-packages/requests/sessions.py", line 589, in request
    resp = self.send(prep, **send_kwargs)
  File "/home/wzp/anaconda3/envs/python3.8/lib/python3.8/site-packages/requests/sessions.py", line 703, in send
    r = adapter.send(request, **kwargs)
  File "/home/wzp/anaconda3/envs/python3.8/lib/python3.8/site-packages/requests/adapters.py", line 519, in send
    raise ConnectionError(e, request=request)
requests.exceptions.ConnectionError: HTTPSConnectionPool(host='pypi.org', port=443): Max retries exceeded with url: /pypi/arxiv-dl/json (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x7f7f54990640>: Failed to establish a new connection: [Errno -2] Name or service not known'))

Hi @simplew2011 , I am unable to replicate this error. Based on the error msg provided, it is likely due to the pypi.org being unavailable. It might be a temporary thing, or something wrong with your network condition. Please try connecting to another network and try using the command again.

Do you consider adding IP proxy functionality or interfaces？

Do you consider adding IP proxy functionality or interfaces？

Everyone's network condition can be quite different, proxy should not be done at this application level I believe.

I just pushed a minor version (30be9a44bb74c039bfea04e7d796485efe275a2f) to handle this error further in case pypi.org is not available during the function call check_update(). If your network has an issue with pypi.org only, the new version should not throw an error anymore.


self._connection.destroy();

C:\csgoo\csgo\csgo-bot-master\node_modules\steam-client\lib\cm_client.js
:177
                                self._connection.destroy();
                                                 ^

TypeError: Cannot read property 'destroy' of undefined
    at Socket.<anonymous> (C:\csgoo\csgo\csgo-bot-master\node_modules\st
eam-client\lib\cm_client.js:177:22)
    at Object.onceWrapper (events.js:285:13)
    at Socket.emit (events.js:202:15)
    at Socket._onTimeout (net.js:447:8)
    at listOnTimeout (timers.js:327:15)
    at processTimers (timers.js:271:5)

changed 2 things
I was erroring on
handlers[EMsg.ChannelEncryptRequest] = function(data) {
	// assume server isn't dead
	this._connection.setTimeout(0);

I made it
handlers[EMsg.ChannelEncryptRequest] = function(data) {
	// assume server isn't dead
	this._connection.setTimeout(5000);

Was
			self._connection.setTimeout(1000, function() {
self.emit('debug', 'socket timed out');
self._connection.destroy();
});
made it
			self._connection.setTimeout(10000, function() {
				self.emit('debug', 'socket timed out');
				self._connection.destroy();
			});

This isn't really supported anymore. Please use steam-user v4.


IOS 11 FIX

In your open func show(_ contentView: UIView, fromView: UIView) {
please re-make this code. This will only get the keyWindow in the main app
guard let window = UIApplication.shared.keyWindow else {
        return
    }
self.show(contentView, fromView: fromView, inView: window)

Dont use [window].

Thanks issue.
What kind of change do you want?
Please try out the PR.
The current code is below.
https://github.com/corin8823/Popover/blob/master/Classes/Popover.swift#L204

Ok thanks.


add support of parse json field

I sync a table from mysql 5.7 with json field,  if I insert a value like blow:
"info" : "{"name": "wx"}"
and will get :
"info" : "eyJuYW1lIjoid3gifQ=="
in the elasticsearch.
So I add the parse of json field.

Thanks @WangXiangUSTC
Could you add some tests for JSON type?

@siddontang  travis ci's mysql version need to upgrade to 5.7

Hi @WangXiangUSTC
Does Travis CI support 5.7 by default now? Maybe we can update the travis now.

@siddontang Travis CI doesn't support mysql 5.7 default, but we can update to 5.7 use script.

Cool, @WangXiangUSTC

@WangXiangUSTC  does it support json array?


4 windows split

hello i have 2560x 1440 resolution and im searching to split a window not only for 2 but for 4
is it possible?

not possible currently. enhancement.

Would love to see this too. I'm using 2160p and the extra screen real estate could really do with being put to use.


setting memory parameters - docs correct?

Docker image used: 6.2.2 from elastic.inc
Operating System: RHEL 7.4

The docs https://www.elastic.co/guide/en/elasticsearch/reference/current/docker.html
say to set - "ES_JAVA_OPTS=-Xms512m -Xmx512m"
which I do:
elasticsearch:
    image: fsdepot.evry.com:8085/davidkarlsen/elasticsearchhq:6.2.2
    restart: always
    volumes:
      - /srv/elasticsearch/data:/usr/share/elasticsearch/data
      - /srv/elasticsearch/config/elasticsearch.yml:/usr/share/elasticsearch/config/elasticsearch.yml
      - /etc/localtime:/etc/localtime:ro
    environment:
      - bootstrap.memory_lock=true
      - "ES_JAVA_OPTS=-Xms12g -Xmx12g"
      - SERVICE_TAGS=prom_monitored,metrics_path=/_prometheus/metrics
      - SERVICE_NAME=applog-elasticsearch
      - SERVICE_CHECK_HTTP=/
    ulimits:
      memlock:
        soft: -1
        hard: -1
      nofile:
        soft: 65536
        hard: 65536
    mem_limit: 24g
    ports:
     - "<IP_ADDRESS>:9200:9200"
    cap_add:
      - IPC_LOCK

but as you see:
!ps
ps xa|grep elast|grep Xmx
26753 ?        SLsl  23:17 /usr/lib/jvm/jre-1.8.0-openjdk/bin/java -Xms1g -Xmx1g -XX:+UseConcMarkSweepGC -XX:CMSInitiatingOccupancyFraction=75 -XX:+UseCMSInitiatingOccupancyOnly -XX:+AlwaysPreTouch -Xss1m -Djava.awt.headless=true -Dfile.encoding=UTF-8 -Djna.nosys=true -XX:-OmitStackTraceInFastThrow -Dio.netty.noUnsafe=true -Dio.netty.noKeySetOptimization=true -Dio.netty.recycler.maxCapacityPerThread=0 -Dlog4j.shutdownHookEnabled=false -Dlog4j2.disable.jmx=true -Djava.io.tmpdir=/tmp/elasticsearch.yVP7xQoF -XX:+HeapDumpOnOutOfMemoryError -XX:+PrintGCDetails -XX:+PrintGCDateStamps -XX:+PrintTenuringDistribution -XX:+PrintGCApplicationStoppedTime -Xloggc:logs/gc.log -XX:+UseGCLogFileRotation -XX:NumberOfGCLogFiles=32 -XX:GCLogFileSize=64m -Des.cgroups.hierarchy.override=/ -Xms12g -Xmx12g -Des.path.home=/usr/share/elasticsearch -Des.path.conf=/usr/share/elasticsearch/config -cp /usr/share/elasticsearch/lib/* org.elasticsearch.bootstrap.Elasticsearch -Ebootstrap.memory_lock=true

it is confusing as it says -Xms1g -Xmx1g  as well as -Xms12g -Xmx12g

I spotted the same issue. ES_JAVA_OPTS doesn't work anymore.
Changing values via jvm.options file works correctly.

The option is working as expected. The actual heap size should be checked from the ES API endpoint ; all the gory details can be found in https://github.com/elastic/elasticsearch-docker/issues/22#issuecomment-360139880. Since this a frequently asked question it could warrant a clarification in an faq section in the docs.

Also related to https://github.com/elastic/elasticsearch-docker/issues/43#issuecomment-343997733

Yes, the heapsizing works just fine - what I am reporting a bug on is the sizing of the young generation which leads to the excessive gc.

Oh, sorry - that was https://github.com/elastic/elasticsearch-docker/issues/155 - and yes, this issue #155 can be closed - it looks strange, but works ok.


Remove Swagger UI from Basic CFT

Remove Swagger UI from Basic CFT for marketplace

This was soft deleted from basic before being migrated to marketplace.


build/make error

Hello there,
When I resync my forked repo, and "rebuild the project" in CLion, I got the following cmake error, I also got the following make error via the command line.
Any ideas?

Try https://github.com/facebookincubator/velox/pull/3295

Thanks, @mbasmanova ! The BaseRuntimeStatWriter error is gone, here is a new make error, which seems to be related to class SelectiveColumnReaderInternal.h

@darrenfu Try adding FOLLY_NULLABLE to raw pointers the compiler complains about.

Thanks @mbasmanova.
If you look at the failure error stacks, they mostly occurred at lines when include a header, such as <optional>, <algorithm>, <memory>, <folly/Range.h>. This is different from fixes like #2026. Since it reports at nullptr in its dep libraries, I have less confidence adding FOLLY_NULLABLE to velox code will help, any other thoughts?
e.g.
In file included from /Users/dofu/git/velox/velox/common/base/RandomUtil.cpp:19:
In file included from /opt/homebrew/opt/llvm/bin/../include/c++/v1/optional:163:
In file included from /opt/homebrew/opt/llvm/bin/../include/c++/v1/__concepts/invocable.h:13:
In file included from /opt/homebrew/opt/llvm/bin/../include/c++/v1/__functional/invoke.h:15:
In file included from /opt/homebrew/opt/llvm/bin/../include/c++/v1/__type_traits/apply_cv.h:16:
In file included from /opt/homebrew/opt/llvm/bin/../include/c++/v1/__type_traits/remove_reference.h:13:
/opt/homebrew/opt/llvm/bin/../include/c++/v1/cstddef:50:9: error: no member named 'nullptr_t' in the global namespace
using ::nullptr_t;

[Update] I tried to resync the forked repo, reinstall most dependency libraries (and upgrade llvm to 15), clean the build folder and rerun "make debug", it doesn't help. Looks like it requires some code fixes for M1 build?
Btw, I'm using Mac M1, macOS v12.6.1, CLion 2022.2.4.

@kgpai and I paired and located the issue, it relates to the incompatible versions of my Mac's CommandLineTools and XCode. Once I manually removed the -isystem /Library/Developer/CommandLineTools/SDKs/MacOSX12.3.sdk/usr/include in the cmake command, the cmake command will pass. I will look into it further for an ultimate fix.

@darrenfu Can we close this issue if your problem is resolved ?


Feature Request:Smart search

Sorry, could find where to post feature requests.
Creating smart searches in packages.
Som useful examples (Not uninstallable, Not unattended, Does not contain a description, Minimum OS is 10.9+, Exists in catalog X and Y, Has no icon, Destination Path is "/Applications/Utilities/", Force install after 12/12/16, Modified after 09/09/12 or created after 10/10/12)

Thanks for reporting! And yes, this is the right place for feature requests. The manifests view got this kind of search functionality recently and I'm planning to implement it in the packages view too at some point. Already on my TODO list.

I would like add +1 to this feature request. List all items with Force install after date set would be very useful feature for our setup


Allow custom JSON Schema validator in validator middleware (fixes #467)

This changeset provides an option to allow a custom JSON schema validator to be passed to the validation middleware.  See further discussion in #467.
In general this actually a small change. validateAgainstSchema() in validators.js already supported passing an external validator to it. So this change is mostly just adding another parameter to the options
that swagger-validator accepts, and then passing any validator down until it gets to validateAgainstSchema().
There are other minor changes around error handling to be slightly more flexible in the error handling for validation errors, so that the custom validator doesn't have to be exactly identical in response to z-schema (the default validator).  If the response is not identical you will still get errors; you just won't get the slightly improved error messages that swagger-tools generates from the z-schema error messages in some corner cases.
This change also includes updated unit tests to check that:

with no custom validator provided (the default), the existing tests continue to pass.
the tests also pass when provided with suitably configured z-schema and ajv custom validators.
the z-schema and ajv custom validators can also validate custom formats and vendor extensions (also known as custom keywords) that are not able to be verified using the default validator.

NOTE: this ONLY allows custom validation of schema objects in the  body of a request or response.
NOTE: path and query parameters, and the schema itself will still be validated by the inbuilt validators, irrespective of any validator passed in.
Finally, the Middleware.md docs have been updated with the new capability.
Test Plan:

Run all unit tests and ensure they pass

I'll give this a peek.

Thanks, I'll look forward to your comments.

Any chance this could get into the next release?

This is unlikely.  Long story short, implementing this yourself in your own middlewares would be very, very simple and any enhancements to swagger-tools right now is not likely due to deprecation (#335).

Ahh, bummer. Ok, thanks for the explanation.


[RFE] Flag to pass kube context to kube burner explicitly

Is your feature request related to a problem? Please describe.
Setting current k8s context so that kube-burner uses the correct cluster is unergonomic and potentially dangerous.
Describe the solution you'd like
I would like a flag that allows setting the k8s context that should be used by kube burner explicitly. The flag can be optional, with the current behaviour if the flag isn't set.

I don't think this feature is necessary, context should be set upfront by the user and is out of scope kube-burner, as far as I know most of CLI tools of the k8s ecosystem don't have anything like this.

@rsevilla87 I am fine with that decision, but it is a common feature for CLI tools in the k8s ecosystem. For example, kubectl has --context, helm has --kube-context, k9s has --context etc.
Kubernetes client-go library has explicit support for overwriting the current context when loading a kubeconfig:
overrides := &clientcmd.ConfigOverrides{
  CurrentContext: kubeContext,
}
configLoadingRules := &clientcmd.ClientConfigLoadingRules{ExplicitPath: kubeconfigPath}
restConfig, err := clientcmd.NewNonInteractiveDeferredLoadingClientConfig(configLoadingRules, overrides).ClientConfig()

Kubernetes client-go library has explicit support for overwriting the current context when loading a kubeconfig:

ok, I don't have much cycles now, but feel free to add this feature yourself, shouldn't be very complex, we'll review it asap

@Nick-Triller Hey, just checking in if you;d be interested to contribute and needed any help?

Hi @smalleni, I am not planning to work on this issue because of lack of time

Hi @smalleni and @rsevilla87,
Is it okay if I try implementing this feature? I'm interested in it as well.
I think directly mimicking the kubectl flags might lead to confusion when they are listed alongside flags that kube-burner already has. So, I intend to start by considering the specifications for this aspect.

are listed alongside

Hey @hhiroshell, I'd personally prefer to mimic the kubectl flags, --kubeconfig and '--context'  if possible, I can't see any conflict or confusion with the flags that kube-burner init, kube-burner destroy or kube-burner measure already provide

Hi @rsevilla87
Let me elaborate on my concerns in more detail.
Besides --context, kubectl also includes --username and --namespace flags that modify a portion of the context settings.
Therefore, if kube-burner introduces a --context flag, it might leads users to expect kube-burner's --username and --namespace flags to function the same as kubectl's.

@rsevilla87
I have created a PR for this issue. Could you please review it?
https://github.com/kube-burner/kube-burner/pull/604


chore(deps): bumps aws-cdk from 1.79.0 to 1.80.0

Codecov Report

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

@@            Coverage Diff             @@
##             master       #82   +/-   ##
==========================================
  Coverage          ?   100.00%
==========================================
  Files             ?         1
  Lines             ?         8
  Branches          ?         1
==========================================
  Hits              ?         8
  Misses            ?         0
  Partials          ?         0

Continue to review full report at Codecov.

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


What is this package actually for?

Sorry for the dumb question but why is it useful to open multiple browser instances... is it any different to opening a single browser (with puppeteer.launch()) and using that to open multiple pages (with browser.newPage())?  Is there a performance benefit of one way or the other?  Wouldn't it be better to have a package for pooling pages instead?  Or, since there isn't much overhead with opening and closing pages, unlike opening and closing a browser instance... don't even both with any page pool... just open/close pages as and when required?

There are things where the separate processes are necessary for parallelization. The screenshot method is one.

So screenshoting multiple pages of the same browser definitely doesn't happen in parallel?  Doesn't the screenshoting of each page start in its own process... why should it have to happen in series?

The internal implementation of the screenshot feature requires the browser to bring the tab being captured to the front / make it active. It then has to draw the page and then capture the image. If you're wanting to take multiple screenshots very quickly, you have to split it out into multiple processes so they can each be rendering a single page and not locking on the active page.


Docker images with missing prophet fix

IMPORTANT: Please do not create a Pull Request without creating an issue first.
Before submitting (must do checklist)

[ ] Did you read the contribution guide?
[ ] Did you update the docs? We use Numpy format for all the methods and classes.
[ ] Did you write any new necessary tests?
[x] Did you update the CHANGELOG?

Type of Change

[ ] Examples / docs / tutorials / contributors update
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Improvement (non-breaking change which improves an existing feature)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Proposed Changes

Related Issue

Closing issues

Codecov Report

Merging #767 (06adc2f) into master (c35055e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #767   +/-   ##
=======================================
  Coverage   83.54%   83.54%
=======================================
  Files         122      122
  Lines        6764     6764
=======================================
  Hits         5651     5651
  Misses       1113     1113

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

Still strange why pip does not install prophet from requirement.txt file. But if that allows to fix the bug then LGTM


flutter naming project bug with git worktrees

Steps to Reproduce

create a simple app called bug with flutter create bug
call git init to init a git repo, git add ., git commit -m 'init' and then connect this repo to any remote
now delete the repo from your machine and clone it as a worktree with the command git clone --bare my_remote_url
now you will have a bare repo of your project on your machine, let's pull our main branch (it could also be master depending on your git config)  by using git worktree add main
so now your code of your main branch exists at bug.git/main folder. now everything works just fine and your bundle id is something like com.example.bug
now we're assuming some new stuff came to flutter and we want to reinit android folder for example. we can just remove the android folder rm -r android and then call flutter create .

Expected results:
what flutter should do is to get the name from pubspec.yaml because the name of the folder in this case presents the branch name and not the project name .
Actual results:
although my project name in pubspec.yaml is still bug, flutter will take it from the folder name which currently is main as we're in the main branch of the worktree (bug.git/main).

Logs

[✓] Flutter (Channel stable, 3.0.0, on macOS 12.3.1 21E258 darwin-arm, locale en-TR)
    • Flutter version 3.0.0 at /Users/adnanfahed/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision ee4e09cce0 (3 days ago), 2022-05-09 16:45:18 -0700
    • Engine revision d1b9a6938a
    • Dart version 2.17.0
    • DevTools version 2.12.2

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0-rc2)
    • Android SDK at /Users/adnanfahed/Library/Android/sdk
    • Platform android-31, build-tools 33.0.0-rc2
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7772763)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.3.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.11.3

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

[✓] Android Studio (version 2021.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7772763)

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

[✓] Connected device (3 available)
    • sdk gphone arm64 (mobile) • emulator-5554 • android-arm64  • Android 11 (API 30) (emulator)
    • macOS (desktop)           • macos         • darwin-arm64   • macOS 12.3.1 21E258 darwin-arm
    • Chrome (web)              • chrome        • web-javascript • Google Chrome 101.0.4951.64
    ! Error: Adnan’s iPhone X is not connected. Xcode will continue when Adnan’s iPhone X is connected. (code -13)

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

• No issues found!

here is an example of the changes after flutter create .:

you can use the repo I've created

Hi @huycozy, thanks for the feedback, I have updated my issue accordingly

Actual results: although my project name in pubspec.yaml is still bug, flutter will take it from the folder name which currently is main as we're in the main branch of the worktree (bug.git/main).

What do you mean by "flutter will take it from the folder name"? What are the actual results you are seeing? Do you mean the project name in your IDE?

what i mean by folder name is the root folder which contains the project (the parent of pubspec.yaml, lib, android etc.). on a normal git clone you would download the project normally with the folder name as the same as the project name, but on a worktree the folder name is the branch name. the solution for this would be for flutter to depend on the name tag inside of the pubspec.yaml instead of the folder name.

the solution for this would be for flutter to depend on the name tag inside of the pubspec.yaml instead of the folder name.

I'm unclear on how flutter depends on the folder name. What is the result of Flutter depending on the folder name?

the result is that when we execute flutter create . it will get the project name from the folder name and fill all the new bundle names from the folder name.

What do you mean by "flutter will take it from the folder name"? What are the actual results you are seeing? Do you mean the project name in your IDE?

Hi guys, in order to make matters clearer, I provide some more screenshots for this:

The project name in IDE vs pubspec's project name
The IDE's project name will take the name by git worktree add main. In this case, it's main.

The re-created android project took wrong package name
After executed step 6, the re-created android project will have different id with current other projects:

android: com.example.main
ios, macos, windows, linux: com.example.bug

As predicted by @adnanjpg, it looks like the command (flutter create .) has combined the new id (com.example.main) from IDE's project name or somewhere else.

Additional suggestion
There is a configuration of project name inside {project dir}/.dart_tool/package_config.json that I think it should be used to fix this issue. Let's consider about this!

@adnanjpg to unblock yourselves, you can explicitly provide the project name to flutter create, like flutter create --project-name=bug.
I will leave this feature request for flutter create . to default to reading from pubspec.yaml open.


Update requests library

What does this PR do?

Bump requests library.
Motivation

Additional Notes

Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached
[ ] If the PR doesn't need to be tested during QA, please add a qa/skip-qa label.

Could you have a look at the agent build?


RateLimitPolicy does not work after being modified

Overview
It seems that after RateLimitPolicy (RLP) CR is modified - e.g. limit  is changed to some other number - it stops working. By that I mean that the 429 Too Many Request is never returned, one receives as many 200 OKs as they want.
The only suspicious entry from log is found in Limitador pod log. The following entry appears in that log after each hit of API endpoint that is rate-limited by modified RLP:
thread 'main' panicked at limitador/src/storage/in_memory.rs:106:58:
called `Option::unwrap()` on a `None` value

Once limit it reverted back to original value everything starts working as expected.
This has been discovered using Kuadrant nightly build from November 20
Steps to reproduce
Create 1 GW, 1 HTTPRoute and one RLP attached to GW (does not matter, could be also attached to HTTPRoute instead). Make sure everything works as expected and the 429 is returned after expected no. of 200s.
Edit the RLP so that different no. of requests is allowed.
Try again, this time 429 is not returned as expected, one continues getting 200s. And see Limitador pod logs for the entry mentioned above.

Is this happening with 1.6 as well?


shouldn't use "using namespace std"

An explanation why you shouldn't use "using namespace ..."
https://stackoverflow.com/questions/1452721/why-is-using-namespace-std-considered-bad-practice

In the context of a simple hello world program, the use of the namespace isn't too much of a problem.


Error with Generic Enum

Bug Report

🔎 Search Terms

Generics Enum Error
🕗 Version & Regression Information

Tested on bunch of versions (4.4.2, 4.3.5, 4.2.3, 4.1.5, 4.0.5, 3.9.7, 3.8.3, 3.7.5, 3.6.3, 3.5.1, 3.3.3, Nightly - 4.5.0-dev.20210910) on playground all failed.

⏯ Playground Link

Playground link with relevant code
💻 Code

export enum OrderStatus {
  Created = "created",
  Cancelled = "cancelled",
  AwaitingPayment = "awaiting:payment",
  Complete = "complete",
}

type Unpacked<T> = T extends (infer U)[]
  ? U
  : T extends ReadonlyArray<infer U>
  ? U
  : T;

type AnyArray<T> = T[] | ReadonlyArray<T>;

type QuerySelector<T> = {
  $in?: T extends AnyArray<any> ? Unpacked<T>[] : T[];
};

type QuerySelectorIn = OrderStatus[] extends AnyArray<any>
  ? Unpacked<OrderStatus>[]
  : OrderStatus[];

const querySelector: QuerySelector<OrderStatus> = {};
querySelector.$in = [OrderStatus.AwaitingPayment, OrderStatus.Cancelled];

🙁 Actual behavior

Type of $in must be OrderStatus[] but I got its wrong. I don't know why but QuerySelectorIn has right type but $in have not.
🙂 Expected behavior

Type of $in must be OrderStatus[] like QuerySelectorIn

[Distributive Conditional Types](Distributive Conditional Types)
This seems more like a question than a bug report.

@MartinJohns

First $in has WeirdType[] type means T extends AnyArray<any> is falsy. But then

Somehow, after checking T extends AnyArray<any>, T  is not OrderStatus anymore.  If I remove the checking part, it works

If this is not a bug please tell me how to code $in?: T extends AnyArray<any> ? Unpacked<T>[] : T[] part correctly.
Actually this code is a part of mongoose.

You need wrap T into [] to avoid union distribution.
type QuerySelector<T> = {
-  $in?: T extends AnyArray<any> ? Unpacked<T>[] : T[];
+  $in?: [T] extends [AnyArray<any>] ? Unpacked<T>[] : T[];
};

The Unpacked<T>[] is redundant too.
type QuerySelector<T> = {
  $in?: [T] extends [AnyArray<any>] ? T : T[];
};

@ducnguyen96 You didn't read the link I provided, did you? Filipe provided you the solution, which is mentioned in the provided documentation as well.

If this is not a bug please tell me how to code $in?: T extends AnyArray<any> ? Unpacked<T>[] : T[] part correctly.

Just because you don't know how it's done correctly doesn't mean it's a bug. :-) You're just dealing with behaviour your don't understand, but don't try to line out what exactly is the bug, instead want to know how to achieve the desired result.

Actually this code is a part of mongoose.

Terrible library. I always feel pain having to use it.

@MartinJohns
Sorry my bad, clearly I don't understand enum  behavior here.


Plugins working in debug mode but not in profile / release mode.

I converted my existing flutter (Flutter Version 3.10) project into flutter-elinux (Flutter Version 3.13) based one as mentioned in the project wiki. Everything including plugins work fine when run in the debug mode, but the plugins don't working properly when run in profile / release mode. I am running Ubuntu 20.04 and have followed the steps as mentioned in the wiki of this repo to setup flutter-elinux.
For example when running the camera plugin from the flutter-elinux-plugins repo in debug mode ( flutter-elinux run -d elinux-x11 ):
( flutter-elinux run -d elinux-x11 )
══╡ EXCEPTION CAUGHT BY WIDGETS LIBRARY ╞═══════════════════════════════════════════════════════════
The following LateError was thrown building RtCamFeedView(dirty, state: _RtCamFeedViewState#d023d):
LateInitializationError: Field '_controller@277206264' has not been initialized.

The relevant error-causing widget was:
  RtCamFeedView
  RtCamFeedView:file:///home/arna/wasteboard_edge_application/wasteboard_ui_flutter/lib/ui/welcome/we
  lcome_view.dart:68:27

Pixel buffer size: width = 1280, height = 720

But when I run the same application in release mode ( flutter-elinux run -d elinux-x11 --release ) :
flutter: LateInitializationError: Field '_controller@552206264' has not been initialized.

flutter: #0      MethodChannel._invokeMethod (package:flutter/src/services/platform_channel.dart:308)
flutter: <asynchronous suspension>
flutter: #1      MethodChannel.invokeListMethod (package:flutter/src/services/platform_channel.dart:496)
flutter: <asynchronous suspension>
flutter: #2      AndroidCamera.availableCameras (package:camera_android/src/android_camera.dart:74)
flutter: <asynchronous suspension>
flutter: #3      _RtCamFeedViewState.startCamera (package:wasteboard_ui_flutter/ui/rtcamfeed/rtcamfeed_view.dart:30)
flutter: <asynchronous suspension>

Is there something I am missing that needs to be done extra to run the app in release mode?

I suppose this issue is the same as https://github.com/sony/flutter-elinux-plugins/issues/78. Are you using the latest video_player plugin (flutter-elinux-plugins)?

This is a flutter-elinux-plugin issue, but I might be wrong. camera plugin for elinux might need to be fixed to fix this issue.

I suppose this issue is the same as sony/flutter-elinux-plugins#78. Are you using the latest video_player plugin (flutter-elinux-plugins)?

Didn't try it yet. Although, I tried some other plugins which are not specifically elinux plugins but plugins which support the Linux platform installed from pub.dev which worked fine in debug mode as well but not in profile/release. Will test and make some detailed observations and report back today.

This is a flutter-elinux-plugin issue, but I might be wrong. camera plugin for elinux might need to be fixed to fix this issue.

So this is specific to the flutter-elinux for flutter version 3.13 right and this does not exist for flutter 3.10 right?

So this is specific to the flutter-elinux for flutter version 3.13 right and this does not exist for flutter 3.10 right?

Yes, correct.

This issue should be fixed in https://github.com/sony/flutter-elinux-plugins/pull/81. Thanks.

Okay, let me check.

@Arna-Maity Is it possible to share your test code (flutter project?) to reproduce a rotation issue?

Have a look at this. The StatefulWidget managing the camera is the same as the one we are using in our project.
Try running it in all the three debug, profile and release modes. In debug mode the orientation is correct but in profile and release modes it is rotated.
https://github.com/Arna-Maity/camera_test

The camera rotation issue seems to be fixed after the latest platform fix (x64) (#213)   you merged into the main branch.  All the linux platform plugins which failed to run previously on 3.13.1 are now running fine. Thank you!

I see. Thanks.


Feature: pinch vertical gesture

Fixes #130
(see demo in https://github.com/hoangnm/react-native-week-view/issues/130#issuecomment-1546529513)

Will open a new one when I fix the tests


Can't show video feed

Describe the bug
I've recently added my first device - a 2k battery doorbell with homebase. I've set it up following your guide, and I get the entities and all. The camera lovelace card also shows the snapshot from the recording last time a person was detected.
Since it's a battery doorbell (doesn't support RTSP I think), I've also installed the RTSP add-on.
When I call eufy_security.start_livestream, the default camera view goes completely grey, while the WebRTC card shows the following error:
ERROR: NotSupportedError: Failed to execute 'addSourceBuffer' on 'MediaSource': The type provided ('video/mp4; codecs="hvc1.016000"') is unsupported.
To Reproduce
Steps to reproduce the behavior:

Set up 2k battery doorbell
Follow setup instructions from readme.md
Call eufy_security.start_livestream

Expected behavior
Camera feed to stream
**Hardware Information about Home Assistant
**Hardware Information about Cameras

Eufy 2k battery doorbell
Live Streaming Protocol: P2p with RTSP addon in HA.

Can you check doorbell streaming quality with low?

Didn't change anything. Any other recommended settings? Didn't actually look through much of it, so it's all on default.

Wait now I'm seeing errors in log about ffmpeg not running.
Traceback (most recent call last):
File "/usr/local/lib/python3.9/site-packages/aiohttp/http_websocket.py", line 673, in ping
await self._send_frame(message, WSMsgType.PING)
File "/usr/local/lib/python3.9/site-packages/aiohttp/http_websocket.py", line 650, in _send_frame
self._write(header + message)
File "/usr/local/lib/python3.9/site-packages/aiohttp/http_websocket.py", line 660, in _write
raise ConnectionResetError("Cannot write to closing transport")
ConnectionResetError: Cannot write to closing transport
2021-11-18 22:09:26 ERROR (Thread-66) [custom_components.eufy_security] eufy_security Doorbell video_thread exception: [Errno 32] Broken pipe- traceback: Traceback (most recent call last):
File "/config/custom_components/eufy_security/camera.py", line 162, in write_bytes_to_ffmeg
self.ffmpeg.process.stdin.write(frame_bytes)
BrokenPipeError: [Errno 32] Broken pipe
2021-11-18 22:09:26 ERROR (Thread-66) [custom_components.eufy_security] eufy_security Doorbell - video ffmpeg error - ffmpeg is not running

this is about h265 and it is not supported by webrtc and currently, native streaming inside Home Assistant is not working for the integration.
Please follow here: https://github.com/fuatakgun/eufy_security/issues/102

Ok, so just to be clear. Right now, it's not possible to get the camera feed to HA? Which is also fine, since it looke like you're working on it. Just curious if I missed a workaround.

Using webrtc, you can have low quality (h264 codec) video stream. If your camera supports h265 media codec, it is not supported by WebRTC. Otherwise, you have to wait for native HA streaming


How to Display Additional Content Correctly with Swiper Slide Active

Clear and concise description of the problem

Suggested solution
How to make the active slide display additional content
Alternative
No response
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

Would you like to open a PR for this feature?

[ ] I'm willing to open a PR

Can using Swiper achieve dynamic display of content when a card is activated

Do you want to ask a question? Are you looking for support? The Framework7 forum and Stack Overflow are the best places for getting support

Please, don't use GitHub issues for questions


Modify qwen2 model ID in tests

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


ARROW-12278: [Rust][DataFusion] Use Timestamp(Nanosecond, None) for SQL TIMESTAMP Type

Rationale
Running the query CREATE EXTERNAL TABLE .. (c TIMESTAMP) today in DataFusion will result in a data type pf "Date64" which means that anything more specific than the date will be ignored.
This leads to strange behavior such as
echo "Jorge,2018-12-13T12:12:10.011" >> /tmp/foo.csv
echo "Andrew,2018-11-13T17:11:10.011" > /tmp/foo.csv

cargo run -p datafusion --bin datafusion-cli
    Finished dev [unoptimized + debuginfo] target(s) in 0.23s
     Running `target/debug/datafusion-cli`
> CREATE EXTERNAL TABLE t(a varchar, b TIMESTAMP)
STORED AS CSV
LOCATION '/tmp/foo.csv';

0 rows in set. Query took 0 seconds.
> select * from t;
+--------+------------+
| a      | b          |
+--------+------------+
| Andrew | 2018-11-13 |
| Jorge  | 2018-12-13 |
+--------+------------+

(note that the Time part is chopped off)
Changes
This PR changes the default mapping from SQL type TIMESTAMP

FYI @Dandandan  / @seddonm1  / @ovr

Random though: I think that we would benefit from not writing a test on Context to test the SQL planner. It could be easier to identify which tests are testing what by writing unit tests more specialized to the component that they affect, as they usually declare the API behavior and are easier to find.

@jorgecarleitao  I think the core of the problem is that the tests in context.rs are largely end to end tests of DataFusion rather than tests for the ExecutionContext itself. I think they ended up in context.rs because that is typically the main entry point to run queries.
What would you think about refactoring the non ExecutionContext specific tests to some other file (perhaps the tests/sql.rs end to end test?)


Yashdev9274 patch 1

🛠 Fixes Issue

👨💻 Changes proposed

✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[ ] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers

📷 Screenshots

Updated Solidity tutorial content


Simplify SSL Certificates Handling for Tailscale's serve Command by Replacing Dot with Dash in Machine.GetFQDN

Why
The Tailscale serve command starts a server that only respond to its specific FQDN (e.g. not its IP address), and that's set up in headscale's codebase as Machine.GivenName + dot + Machine.User.Name. Because of that dot between Machine.GivenName and Machine.User.Name, you'd need an extra wildcard SSL certificate for each User.Name (to cover all of her nodes) if you want to make it accessible to everyone online.
But if we used a dash instead of the dot, it would simplify things. It would just count as one subdomain, so a single wildcard SSL certificate could cover all users in the tailnet.
To test if I am right I came up with a simple patch for this. It makes the server return Machine.GivenName + - + Machine.User.Name. If you get a wildcard SSL certificate from Caddy and set up a reverse proxy to the node running tailscale serve http:1234 / text:"hello world!", it works perfectly. Anyone online would see "hello world!" at the same URL used in the tailnet, and the wildcard SSL certificate would have it covered.
Description
Here's the simple patch which does it:
modified   hscontrol/types/machine.go
@@ -269,7 +269,7 @@ func (machine *Machine) GetFQDN(dnsConfig *tailcfg.DNSConfig, baseDomain string)
 	var hostname string
 	if dnsConfig != nil && dnsConfig.Proxied { // MagicDNS
 		hostname = fmt.Sprintf(
-			"%s.%s.%s",
+			"%s-%s.%s",
 			machine.GivenName,
 			machine.User.Name,
 			baseDomain,

With this setup, a single generic Caddy snippet can handle all users. Without it, you'd have to add every new user to Caddy individually. That would be way more work to keep up with.
I wonder if there's any interest in having something like this as part of the configuration directive. I am happy to discuss it more and eventually make a PR.

Not necessary based on the same motivation but would also result in the same functionality for you (for tagged devices at least): #1405
From my understanding the given name should be unique by appending a suffix if two machines do have the same given name.


$stateProvider and  $urlRouterProvider need better visibility in the doc

The new doc is missing $stateProvider and  $urlRouterProvider. They are present in the old doc. Am I missing something?
The link to $stateProvider and  $urlRouterProvider in the new doc are not too visible. They should be normal text buttons instead of just a gear icon

The link to $stateProvider and $urlRouterProvider in the new doc are not too visible. They should be normal text buttons instead of just a gear icon

This is the way ngDocs works. I'm open to alternative (specific) suggestions of where else we can link them.

Now that the gear icon has text next to it, it's pretty obvious. Thanks!


Variable Height Sortable - unable to pull some items all the way down

I implemented a variable height sortable, but something about the way it's observing intersections is preventing me from dragging larger items below smaller ones in my list. Is there anything in the library that can solve something like this?
https://github.com/clauderic/dnd-kit/assets/159949/29f618ee-08b1-489d-84fa-19b8f42f7d15
Current code:
<DndContext
  modifiers={[restrictToVerticalAxis, restrictToParentElement]}
  onDragEnd={({ over, active }) => {
    if (active.id && over?.id) {
      const fromId = active.id;
      const toId = over.id;
      if (typeof fromId === "string" && typeof toId === "string")
        moveFeature(fromId, toId);
    }
  }}
>
  <SortableContext
    items={tasks}
    strategy={verticalListSortingStrategy}
  >

      {tasks.map((task) => (
        <TaskRow key={task.id} {...task} />
      ))}

  </SortableContext>
</DndContext>

export function TaskRow(task: Feature) {
  const {
    attributes,
    listeners,
    setNodeRef,
    transform,
    transition,
    isDragging,
    setActivatorNodeRef,
  } = useSortable({ id: task.id });

  const style = {
    transform: CSS.Translate.toString(transform),
    transition,
    zIndex: isDragging ? 1 : undefined,
    position: isDragging ? ("relative" as const) : undefined,
    height,
  };

  return (

        <button
          className={classNames(
            ROW_HEIGHT,
            "group cursor-move w-6 flex items-center justify-end outline-none touch-none"
          )}
          ref={setActivatorNodeRef}
          {...attributes}
          {...listeners}
        >
          <GripVertical className="w-4 h-4 opacity-20" />
        </button>
        {/** ... */}

  );
}

try to use a prop collisionDetection, but i'm not sure it will help you, maybe it will..

use the DnDContext prop named collisionDetection, try to set closestCenter, im almost sure it will help you.
doc: https://docs.dndkit.com/api-documentation/context-provider/collision-detection-algorithms