How can I know which promises is failed in .catch?

Hello! I have 2 promises, which have to execute one by one.
First I need to upload images on server, then I need to update user info (not in a row).
So I wrote this code:
        firstly {
            ProfileManager.uploadImages(photos)
        }
        .then { (photos)  in
            ProfileManager.updateProfile(name, bio: bio ?? "", birthdayDate: birthdayDate ?? "")
        }
        .done { (result) in
                self.view?.showAlert(title: nil, message: "Данные успешно изменены")
        }
        .catch { (error) in
                print() // how can I know uploadImages or  updateProfile was failed??
        }
        .finally {
                        self.view?.stopLoading()
                }

how can I know in the .catch section uploadImages or  updateProfile was failed??
And if I want to do something with "photos", which uploadImages returned , I need to write .done section before .then, right?
As I understand I need to write (photos)  in in .then section, but I can't do anything with this variable.

I can do nothing with photos in then, there is error then:

Swift can’t infer the return type of closures with 2+ lines. You’ll have to specify the type of promise you’re returning from the closure.

Ok, thanks!
But I thought, that there is something like when(fullfiled) when I can get each promises .
Thought that in .done I can get all of previous completed results
And that .catch will return promise which failed.
Or that I can write something like that:
 firstly {
            ProfileManager.uploadImages(photos)
        }
        .catch { (error) in
                print() // error for photos
        }
        .then { (photos)  in
            ProfileManager.updateProfile(name, bio: bio ?? "", birthdayDate: birthdayDate ?? "")
        }
        .done { (result) in
                self.view?.showAlert(title: nil, message: "Данные успешно изменены")
        }
        .catch { (error) in
                print() // error for profile
        }
        .finally {
                        self.view?.stopLoading()
                }

So thanks for your help, I think all is fine now..

We do have when which behaves as you describe. Though I assumed that you wouldn’t want to update he profile until after the photos upload. If you want to do them at the same time, then when is what you want, and the following closure will contain the results of each of the promises passed in.
You can’t chain anything after a catch except for a finally. A catch will get called for any promise the errors prior in the chain. It’s up to you to determine which promise failed via the error object. This is a feature of promises. It reduces your error handling to a single place. Though there are alternatives when you really them.
done does not in fact contain the results of all prior promises in the chain as you describe. We don’t have a method for that. Though it sounds like when will fit the bill for you in this case.

I meant, that it’s cool that in .when i can get each promises.
Nevermind) anyway than you so much)


does the api support events/push updates?

Describe Your Problem:
is it possible to get accessory value push updates with the homebridge-config-ui or should a client connect to the evented http server directly?
all existing issues i have found are only concerned with controlling homebridge accessories from the outside. but not with receiving updates.
one more api question: is there an endpoint that returns the port for the evented http server?

i have just implemented proof of concept socket.io kommunikation in perl an can connect to the homebridge gui server from fhem. subscribing to messages works and i'm getting the accessory data and events. this looks very good so far.
that said: would you consider having event push as part of the official api? there is already accessory query and control. only the event part is missing. an optional upgrade to a websocket on the main accessories endpoint for example.
i think this would be a mich better option. also in comparison to the api from northernman as you already resolve the uuids to human readable names. this avoids having to look them up on the client side.


Setup2 dektop vertion

Some CSS properties were updated to better adapt the desktop version
Problems with the contac-form were also corrected
A new working branch was created because the previous branch was used in error and was getting bugs

@Fernando426, thank you for your work on this PR. Could you please share the link to the previous PR where you received some feedback for changes? This will help me review your code more effectively.

@topeogunleye ok here is the link although you can review it since I am trying to delete that branch because it has errors and it usually breaks my code
https://github.com/Fernando426/Portfolio-mobile-website/pull/16

@topeogunleye ok here is the link although you can review it since I am trying to delete that branch because it has errors and it usually breaks my code #16

@Fernando426, please kindly request for review again after you are done.

Hello @Fernando426, please take a look at the changes requested by the previous reviewer in the previous PR and try to address them before asking for a new review by doing that you could improve the quality and professionalism of your portfolio. Here's a rundown:

[ ] It would be great if you could follow the suggested change from the previous reviewer. On desktop devices, there should be three cards per row, but your current design only has two cards. Here's the required change link. 💻

[ ] In your "About Me" section, there are still some blank spaces that could be reduced. As requested by the previous reviewer in here. 📄

[ ] [OPTIONAL] Kindly use the correct fonts for your page.

[ ] [OPTIONAL] You might consider changing the font-family to enhance the overall look. 😊


Create a contact info block

Add links to social networks (LinkedIn, Facebook, Instagram), phone number, email, Telegram username, etc.

Links to social networks added


Commands working even with PATH unset

Our commands shouldn't work without PATH in the environment but they are working :/

that's because unset probably is not working...


Create venv workflow is broken for a new Mac M1

Mac M1
Install brew & a few other tools
I didn't manually install Python, however Python 3.9 and 3.11  gets installed either via brew or some other tool
Install VS Code, Jupyter and Python
Ensure mru option is selected, and restart VS Code
Create a blank notebook and attempt to run a cell

Problem 1:

Not prompted to select a kernel, and get an error about ipykernel not being available**
Some how Python 3.9 got selected as a kernel for a new machine

Create a venv and ipykernle gets installed

Now run a cell and I get prompted to install notebook.

Problem2
When installing packages, we should prompt user to select an interpreter to run Jupyter and then install missing packages into that.**
Problem 3
Reload VS Code and run a cell
We now get two progress indicators! Why not just one instead of two?

Should be fixed in latest pre-release


Project doesn't build

On checking out master, doing a dotnet build gives the following errors:
error MSB4184: The expression "registry:HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\Microsoft SDKs\Windows\v10.0@InstallationFolder" cannot be evaluated.
warning NU1503: Skipping restore for project 'C:\Users\Isaac\Source\Repos\EFCore.FSharp\EFCore.FSharp\EFCore.FSharp.fsproj'. The project file may be invalid or missing targets required for restore. [C:\Users\Isaac\Source\Repos\EFCore.FSharp\EFCore.FSharp.sln]
error MSB4184: The expression "registry:HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\Microsoft SDKs\Windows\v10.0@InstallationFolder" cannot be evaluated.

Restore completed in 46.54 ms for C:\Users\Isaac\Source\Repos\EFCore.FSharp\EFCore.FSharp.Test\EFCore.FSharp.Test.fsproj.

error MSB4184: The expression "registry:HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\Microsoft SDKs\Windows\v10.0@InstallationFolder" cannot be evaluated.
error MSB4184: The expression "registry:HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\Microsoft SDKs\Windows\v10.0@InstallationFolder" cannot be evaluated.

How can I fix this?

This appears to be caused by these lines in EFCore.FSharp.fsproj
<WindowsSdkDir Condition=" '$(WindowsSdkDir)' == '' ">$(registry:HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\Microsoft SDKs\Windows\v10.0@InstallationFolder)</WindowsSdkDir>
<WindowsSDKVersion Condition=" '$(WindowsSDKVersion)' == '' ">$(registry:HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\Microsoft SDKs\Windows\v10.0@ProductVersion)\</WindowsSDKVersion>
<SignToolCommand>"$(WindowsSdkDir)bin\$(WindowsSDKVersion)x86\signtool.exe" sign /t $(TimestampUrl) /q</SignToolCommand>

If you comment them out then it should build, although it will not be signed.
This issue also prevents dotnet build from running on any non-Windows platform
@bricelam We'll need to find a way to sign the package in a more cross-platform friendly manner

Yes, I intend to address this. It think it will be fixed in the next version of the SDK. To workaround it, just remove those properties.
By default, it won't sign anyway. Those are just there for me so I can sign it before publishing release to NuGet.org.

FYI, to get this to build in VS 2017 I also had to remove the line:
<AssemblyOriginatorKeyFile>key.snk</AssemblyOriginatorKeyFile>

This wasn't needed to get it to build from the cmdline, though.

This is preventing Appveyor and Travis builds


MouseHoverEventTestCase: Dispatching event on_cursor_leave to cleanup some tests

Maintainer merge checklist

[x] Title is descriptive/clear for inclusion in release notes.
[x] Applied a Component: xxx label.
[ ] Applied the api-deprecation or api-break label.
[ ] Applied the release-highlight label to be highlighted in release notes.
[x] Added to the milestone version it was merged into.
[ ] Unittests are included in PR.
[ ] Properly documented, including versionadded, versionchanged as needed.

@matham Can you review this pull request? We will have to disable MouseHoverEventTestCase again as tests in MultitouchSimulatorTestCase can still fail although they passed in https://github.com/kivy/kivy/pull/7483.

I'm not sure what to make of this because as you noted the tests fail again. I would rather not merge something that causes the tests to fail. Is the changes needed to properly cleanup and the test failing is just a consequence, or are the changes to try to fix the failing tests?

Changes in this request ensure that "end" event is dispatched to cleanup internals of EventLoop and they are not fix or cause of failing tests in MultitouchSimulatorTestCase. I'm still trying some ideas to fix that failing class.

Maybe we can just disable all these tests on Windows CI for now, if you want to merge this before figuring out the issues.
BTW, have you tries running mint in a VM to see if perhaps you can directly replicate the issue like @Zen-CODE mentioned? That could help!?


See if we can improve the pymatgen.get_all_distances function

This function: https://github.com/GEMDAT-repos/GEMDAT/blob/b6d6f98e9a25d2f93c7073b2950838581d294bd5/src/sites.py#L195 already takes 15s, and we expect it to become a bottleneck for larger datasets, so we should have a look at it.
Periodic boundaries should be taken into account
Possible paths to explore:

LazyTensor: http://www.kernel-operations.io/keops/python/api/common/GenericLazyTensor.html
Some other matrix(multiplication)Operator which is fast and does not store the full matrix
Custom code from Julia/Rust to define such an operator
using another method to find the closest neighbours https://docs.scipy.org/doc/scipy/reference/generated/scipy.spatial.KDTree.html#scipy.spatial.KDTree

Note that we are not interested in the distances between all positions, but in the index and distance to the nearest site only.
pdist = lattice.get_all_distances(atom_coords, site_coords)

# index of nearest site
nearest = pdist.argmin(axis=1, keepdims=True)

# True if atom is close enough to a site
is_at_site = np.take_along_axis(pdist, nearest, axis=1) < dist_close

# Site index when close, NOSITE when in transition
atom_site = np.where(is_at_site, nearest, NOSITE)


Changelog

It would be cool to have a shortlist of features/bugfixes for each release. This can be done either in a (plain)text file or directly on GitHub.

@superbobry, I am assuming you are referring to the new tag(v2.0.9) I created yesterday.
I didn't know that we can add release notes in github. That looks like a good idea to me. I will update and include the release notes for the tags I create. Thanks for letting us know.
I would also urge you to create tags or let us know the deployed version of Dr. Elephant you are running in prod. In that way, we won't limit the official releases to be only from Linkedin.

Yep, I was referencing 2.0.9. We are currently running the latest master, because we needed one of the latest features.
Have you considered detailing the release policy for Dr. Elephant? E.g. master is guaranteed to be prod. ready or only tagged releases should be installed.

No, we do not have a release policy yet.
We have been trying to ensure that the master is always prod ready by carefully merging every change to the master. Whenever we work on some major changes, we try to use a separate branch for it and merge it after completely testing it.
But in fact, it is quite hard to guarantee, master or any specific commit to be prod ready without having a comprehensive testing suite. The least we can do is to tag the commit which we are successfully running in production which in some ways tells the developer that the specific tag is stable enough to be deployed to production.
Until we build the test suite, we have decided to tag our internal releases. We also recommend others to follow the same. Do let us know if you have a better approach?


Add gutter icon for the And keyword and proper test path entry to all…

… test nested in And
Greetings.
First of all thank you for the great testing framework. And I hope you will find my first commit useful.
I've noticed that for and keyword gutter icons are not generated. And even more, on clicking on gutter icon for any test nested in and test path misses and-part which leads to malformed test path and effectively to fail to run the test.
After some time I've managed to build plugin from sources and come up with the following solution.
I've tested it on my local IDEA 2020.3.2 installation, and it does work.
Alas, I couldn't test it on fresh master branch as it's broken at the moment. So I took latest stable revision (ae8e77ea49367f00845ed1c56d63aaa71e354fc1 Fixed is spec from 22/12/2020). But I hope that pretty trivial code would work on fresh master and IDEAs as well.
Please review my solution. )
P.S. I've seen similar issues on context keyword and believe it could be fixed pretty much the same way. But first I want to check if my approach meets contribution guidelines if any.

This is awesome, thank you.


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-k8s-25-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 2458

Questions ?
Please refer to the Backport tool documentation


Unable to read k3s.yaml

Imaging new host via refactored ISO from rc2.  Getting error running kubectl logged in as the rancher user.  Run the same command with sudo and it will succeed.  This is new behaviour in rc2, not sure if it's intentional or not.
s10618svr [~]$ kubectl get node
WARN[2019-09-13T13:59:26.157674721Z] Unable to read /etc/rancher/k3s/k3s.yaml, please start server with --write-kubeconfig-mode to modify kube config permissions
error: Error loading config file "/etc/rancher/k3s/k3s.yaml": open /etc/rancher/k3s/k3s.yaml: permission denied
s10618svr [~]$ ls -l /etc/rancher/k3s/k3s.yaml
-rw------- 1 root root 1052 Sep 13 13:56 /etc/rancher/k3s/k3s.yaml
s10618svr [~]$ sudo kubectl get node
NAME         STATUS     ROLES    AGE     VERSION
k3os-18447   NotReady   master   6m24s   v1.14.6-k3s.1
s10618svr    Ready      master   5m17s   v1.14.6-k3s.1

@dweomer In k3s we changed the k3s.yaml to not be world readable by default.  In k3os I think should allow the rancher user to read the k3s.yaml by default.

fixed by #181


Tree view message: Does not wrap

Issue Type: Bug
Testing #64205

VS Code version: Code - Insiders 1.30.0-insider (fa87e67ebfd6b8014ba697ddbc6c2d6c0e2cb970, 2018-12-04T09:26:33.257Z)
OS version: Darwin x64 17.7.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-4770HQ CPU @ 2.20GHz (8 x 2200)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: enabledrasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
4, 4, 3

Memory (System)
16.00GB (0.71GB free)

Process Argv
--extensions-dir /Users/chrmarti/Development/repos/tmp/ext --user-data-dir /Users/chrmarti/Development/repos/tmp/user

Screen Reader
no

VM
0%

Extensions: none

It stops at the longest non-wrapping part:
tree.message = new MarkdownString(`[Refresh](command:githubIssuesPrs.refresh) &

- [PRs](https://www.microsoft.com)
- Other

Markdown | Less | Pretty
--- | --- | ---
*Still* | \`renders\` | **nicely**
1 | 2 | 3

Here's our logo (hover to see the title text):

Inline-style:
![alt text](https://github.com/adam-p/markdown-here/raw/master/src/common/images/icon48.png "Logo Title Text 1")

Reference-style:
![alt text][logo]

[logo]: https://github.com/adam-p/markdown-here/raw/master/src/common/images/icon48.png "Logo Title Text 2"

\`\`\`javascript
var s = "JavaScript syntax highlighting";
alert(s);
\`\`\`

\`\`\`python
s = "Python syntax highlighting"
print s
\`\`\`

\`\`\`
No language indicated, so no syntax highlighting.
But let's throw in a <b>tag</b>.
\`\`\`

1. asdf
2. adfasdf
`);

We do not want to support all features of markdown in tree view message which could cause all such sort of issues. This will get fixed by reducing the list or not supporting the markdown at all. Tracking this here - https://github.com/Microsoft/vscode/issues/64406


Feature: output additional AudioSensors info

When I add Audio sensor my output is always "High Definition Audio Device" but when I try to get my default audio device via powershell cmdlet https://github.com/frgnca/AudioDeviceCmdlets I get additional info f.e.
Name    : Headphones (High Definition Audio Device)

Interesting! I'll have a look, thank you :)

CoreAudio version 1.17.0 property FriendlyName return exactly what @dimmuboy wrote and the old FriendlyName property has been changed to DeviceInterfaceFriendlyName.

Awesome, nice find @tachibanayui ❤


Danphoenixjess/upgrade gisaid workflow

Summary:
These changes aim to:

Make the daily GISAID job (in gisaid.wdl) run 2 additional steps to mask and filter all GISAID data, and use the output as starting point for downstream tree build (in nextstrain-dbg.wdl,  run_nextstrain_ondemand.sh and run_nextstrain_scheduled.sh). This cut our longest tree build (overview tree) from 7.5hr to 5hr.

Remove ncov in the GISAID docker (Dockerfile.gisaid) and only have it cloned in gisaid.wdl to make version management more straightforward. I searched around to make sure it's not used elsewhere but I couldn't test the impact of this change.

builds.yaml is updated to include what is required by the newer ncov than the pinned ncov in GISAID docker.

Unpin ncov in tree runs (in nextstrain-dbg.wdl,  run_nextstrain_ondemand.sh and run_nextstrain_scheduled.sh). They were pinned when ncov had a bug upstream. All tests were done with the current newest ncov versions and they seem to work fine.

Tests:
Phoenix tested the gisaid.wdl and I tested the nextstrain-dbg.wdl on danrlu/improve_gisaid branch, which are the first 2 files here when compared against the current branch.
Demos:

Notes:
This is the set up Nextstrain uses to preprocess their GISAID data, for reference in case we want to cut back on our memory.
Checklist:

[x] I merged latest <base branch>
[x] I manually verified the change
[x] I added labels to my PR
[ ] I tested in multiple browsers
[ ] I added relevant unit tests
[ ] I have notified others of changes they need to make locally (migrations, jobs, package updates, etc)

@jgadling is it good with you if we merge it now into staging?


How to download the next batch while processing the current batch

My producer produce messages in a specific order. To preserve this specific order while consuming, we have single partition and a single consumer.
For performance reasons, I'm consuming batches and maxBytesPerPartition = 1024 * 1024 * 50
I'm seeing a delay of couple of seconds between batch ends to process until the next one is starting to be processed.
My Request
KafkaJs should download the next batch while my app is processing the current batch.
Why
The processing time of a batch takes 0.5s, while the download of the batch takes much longer (couple of seconds).

We cannot download a new batch while you are processing the current batch, because we won't know what offset to fetch from until you are done processing the current batch.
The delay you are seeing is possibly due to the configuration of minBytes, maxBytes, maxBytesPerPartition and minWaitTimeInMs, which control whether you want to prioritize throughput or latency.
If you want to improve performance, the way to do that is to have multiple partitions and consumers, and key your messages by your aggregation key. Unless you need global ordering between all messages in the system, which is usually not the case. If you do need global ordering, you don't have a way to scale.


iframe exporst are not supported for saved snippets

Describe the bug
New iframe export urls are broken
I'm trying to export this snippet to an iframe, https://carbon.now.sh/R1akLq42QmwZxET5KwoV, however the src generated below doesn't display my snippet.
Generated code
<iframe
  src="https://carbon.now.sh/embed/R1akLq42QmwZxET5KwoV"
  style="transform:scale(0.7); width:1024px; height:473px; border:0; overflow:hidden;"
  sandbox="allow-scripts allow-same-origin">
</iframe>

To Reproduce
Steps to reproduce the behavior:

Go to https://carbon.now.sh/R1akLq42QmwZxET5KwoV
Click on copy embed iframe
then go to: https://carbon.now.sh/embed/R1akLq42QmwZxET5KwoV and see that the snippets do not match, the snippet displayed is the carbon demo snippet

Expected behavior
See my snippet in the embedded iframe instead of the carbon demo snippet
Screenshots
My snippet code:

What I get in the embedded iframe:

Info (please complete the following information):

OS [e.g. macOS, Linux, Windows, iOS]: macOS
Browser [e.g. chrome, safari]: Chrome

We currently only support iFrame exports for GitHub gists, not Carbon snippets. I will mark this feature as an enhancement to be considered.


Card slider becomes confused if used twice in same section

The ellipsis of the cards in a card slider seem to become confused if they're in the same section. The ellipsis of the first line one will be too agressive.
Example:

See https://main--caesars--hlxsites.hlx.page/caesars-palace/things-to-do/garden-of-the-gods-pool-oasis

Confirmed issue happens only when multiple sliders are in the same section.


Module update and testing 1.19

Hi team!
This PR contains a new update of the logging packages including a major upgrade of elasticsearch.
Includes:

Add checks in the CI and fix policeman issues (super-linter).
Add checks in the CI and fix license headers.
Add 1.19 e2e testing pipeline.
Add an automatic release pipeline.
Update cerebro. From version 0.8.5 to 0.9.2.
Update fluentd. From version 1.10.2 to 1.11.2.
Update fluent-bit. From version 1.4.4 to 1.5.6.
Update elasticsearch. From version 6.8.8 to 7.9.1
Update kibana. From version 6.8.8 to 7.9.1

The pipeline is passing right now: http://ci.sighup.io/sighupio/fury-kubernetes-logging/278/7/3
Thanks!

I have some general questions:

we are using a very basic readiness probe (which gave me a lot of problems with cluster nigthly shutdown) I think it would be valuable to use a more refined one (I can share some code).

we are using a liveness probe, is it really needed with elasticsearch? If it's indeed the case I believe that the initialDelaySeconds: 60 is not enough; I would at least double that, probably triple that to stay on the safe side.

we are not using podManagementPolicy: Parallel but instead we are using the default ordered behaviour, why?

In fluentd-index-sighup-template.json I would change the default replicas to 1, and add "codec" : "best_compression" in the settings. It decrease the size of logs by 15-25% and the cpu impact while it exists it's invisible to the human eye^^

I have some general questions:
* we are using a very basic readiness probe (which gave me a lot of problems with cluster nigthly shutdown) I think it would be valuable to use a more refined one (I can share some code).

Absolutely, I tried the upstream readiness probe script: https://github.com/elastic/helm-charts/blob/master/elasticsearch/templates/statefulset.yaml#L225
But, deploying a single elasticsearch cluster didn't work as it remains in the yellow state forever.
Maybe we can patch the triple.yml to modify the readinessProbe adding the upstream script.

* we are using a liveness probe, is it really needed with elasticsearch? If it's indeed the case I believe that the `initialDelaySeconds: 60` is not enough; I would at least double that, probably triple that to stay on the safe side.

Modified to 2x both liveness and readiness. ✅

* we are not using `podManagementPolicy: Parallel` but instead we are using the default ordered behaviour, why?

Modified! ✅

* In `fluentd-index-sighup-template.json` I would change the default replicas to 1, and add `"codec" : "best_compression"` in the settings. It decrease the size of logs by 15-25% and the cpu impact while it exists it's invisible to the human eye^^

✅


Too many zeros

I cant change the screenshot, so hopefully that can be updated by you.
Type of changes

[ ] Bug fix
[ ] New feature
[x] Documentation / docstrings
[ ] Tests
[ ] Other

Checklist

[ ] I've run the latest black with default args on new code.
[ ] I've updated CHANGELOG.md and CONTRIBUTORS.md where appropriate.
[ ] I've added tests for new code.
[x] I accept that @willmcgugan may be pedantic in the code review.

Description
Please describe your changes here. If this fixes a bug, please link to the issue, if possible.

Thanks


Import presentation order for never seen levels

Hello,
It seems that when importing courses, that the things are inserted in Anki in not necessarily the same order you would learn them on the memrise site.
I did a bit of digging around in order to better understand the data that memrise2anki-extension uses in order to hopefully point out a track toward a solution.
It seems that the appropriate presentation order should use the boxes property from the ajax/session endpoint to schedule learning when the thingusers property is empty. Not doing so currently results in a somewhat random scheduling order. Scheduling is well separated between levels, but the ordering within the levels makes it harder to learn things when you're shown the composite word before its base word.
The boxes property is an array with items
{
  "template": "presentation",
  "thing_id": THING_ID,
  "column_a": 1,
  "column_b": 2
}

which appears to follow the order used when displaying the cards in preview mode.
Thus, it seems that given an empty thingusers, the boxes list order should be used in order to create the cards/notes in the proper order to ease learning.
As a side request (if this change is implemented), it would be nice if one could only update the schedule/presentation order of unseen cards/notes through the import mechanism. Otherwise I guess one can delete the whole deck and reimport.

Could you please manually update the addon from the repo and try the latest changes? The positions will always be imported or updated, but of course already learned cards/notes are left untouched. Does everything work for you as intended?

@wilddom After a quick test, it seems to work as intended!
Thanks for the prompt update!

Okay, thanks for testing, I think I know what's going on, but I'll have to take a closer look.

@tomzx Have another go

Perfect! Thanks for the followup. :+1:


Correct the code snippet in the "Sync" section

The current code snippet does not include the NSError parameter in the completion block. I corrected it by adding the NSError parameter to the completion block.

@SirSJR thanks :heart: :bouquet:


Make Http2ClientSession inactive on EOS event

Fix #6671 & add a release assert to make sure any active session is not freed.
I could reproduce the issue in my local box with h2spec. It looks like ATS jump in Http2ConnectionState::release_stream() with fini_received = true & ua_session->is_active() = true in error cases.
As @maskit pointed out on #6729, the issue is introduced by #4936 which was released by 8.0.3.
It looks like what we should have was moving only the problematic vc->add_to_keep_alive_queue(); at that time.
How to reproduce
$ h2spec -h localhost -p 4443 -t -k http2/5.1/7

Backtrace
[Jun  9 11:25:06.343] [ET_NET 6] DEBUG: <Http2ConnectionState.cc:864 (rcv_continuation_frame)> (http2_con) [18] [1] Received CONTINUATION frame
[Jun  9 11:25:06.343] [ET_NET 6] DEBUG: <Http2ConnectionState.cc:1874 (send_goaway_frame)> (http2_con) [18] Send GOAWAY frame, last_stream_id: 1
[Jun  9 11:25:06.343] [ET_NET 6] DEBUG: <Http2ClientSession.cc:324 (set_half_close_local_flag)> (http2_cs) [18] session half-close local
[Jun  9 11:25:06.343] [ET_NET 6] DEBUG: <Http2ConnectionState.cc:1336 (delete_stream)> (http2_con) [18] [1] Delete stream
[Jun  9 11:25:06.343] [ET_NET 6] DEBUG: <Http2ConnectionState.cc:1803 (send_rst_stream_frame)> (http2_con) [18] [1] Send RST_STREAM frame
[Jun  9 11:25:06.343] [ET_NET 6] DEBUG: <Http2Stream.cc:301 (change_state)> (http2_stream) [18] [1] Http2StreamState::HTTP2_STREAM_STATE_CLOSED
[Jun  9 11:25:06.343] [ET_NET 6] DEBUG: <Http2Stream.cc:425 (initiating_close)> (http2_stream) [18] [1] initiating_close
[Jun  9 11:25:06.343] [ET_NET 6] DEBUG: <Http2Stream.cc:466 (initiating_close)> (http2_stream) [18] [1] send EOS to read cont
[Jun  9 11:25:06.343] [ET_NET 6] DEBUG: <Http2Stream.cc:765 (destroy)> (http2_stream) [18] [1] Destroy stream, sent 0 bytes
Process 29730 stopped
* thread #8, name = '[ET_NET 6]', stop reason = breakpoint 1.1
    frame #0: 0x000000010012bf43 traffic_server`Http2ConnectionState::release_stream(this=0x00000000090edd48, stream=0x00000000091cf3e0) at Http2ConnectionState.cc:1396:9
   1393         // We were shutting down, go ahead and terminate the session
   1394         // this is a member of Http2ConnectionState and will be freed
   1395         // when ua_session is destroyed
-> 1396         ua_session->destroy();
   1397
   1398         // Can't do this because we just destroyed right here ^,
   1399         // or we can use a local variable to do it.
(lldb) bt
* thread #8, name = '[ET_NET 6]', stop reason = breakpoint 1.1
  * frame #0: 0x000000010012bf43 traffic_server`Http2ConnectionState::release_stream(this=0x00000000090edd48, stream=0x00000000091cf3e0) at Http2ConnectionState.cc:1396:9
    frame #1: 0x000000010013f9e6 traffic_server`Http2Stream::destroy(this=0x00000000091cf3e0) at Http2Stream.cc:784:36
    frame #2: 0x000000010013cb94 traffic_server`Http2Stream::terminate_if_possible(this=0x00000000091cf3e0) at Http2Stream.cc:414:5
    frame #3: 0x000000010013b4cd traffic_server`Http2Stream::main_event_handler(this=0x00000000091cf3e0, event=104, edata=0x0000000009815f00) at Http2Stream.cc:149:3
    frame #4: 0x0000000100008286 traffic_server`Continuation::handleEvent(this=0x00000000091cf3e0, event=104, data=0x0000000009815f00) at I_Continuation.h:160:12
    frame #5: 0x0000000100319b0f traffic_server`EThread::process_event(this=0x0000000002c34000, e=0x0000000009815f00, calling_code=104) at UnixEThread.cc:131:22
    frame #6: 0x0000000100319e7d traffic_server`EThread::process_queue(this=0x0000000002c34000, NegativeQueue=0x0000000004a26f10, ev_count=0x0000000004a26ee0, nq_count=0x0000000004a26ee4) at UnixEThread.cc:170:7
    frame #7: 0x000000010031a386 traffic_server`EThread::execute_regular(this=0x0000000002c34000) at UnixEThread.cc:230:5
    frame #8: 0x000000010031aa65 traffic_server`EThread::execute(this=0x0000000002c34000) at UnixEThread.cc:331:11
    frame #9: 0x0000000100318ce8 traffic_server`spawn_thread_internal(a=0x00000000008220a0) at Thread.cc:85:12
    frame #10: 0x00007fff7198f109 libsystem_pthread.dylib`_pthread_start + 148
    frame #11: 0x00007fff7198ab8b libsystem_pthread.dylib`thread_start + 15
(lldb) p ua_session->is_active()
(bool) $0 = true

[approve ci autest]

ATS crashed on openclose_h2 autest.
https://ci.trafficserver.apache.org/autest/ausb-6876.13460/openclose_h2/_output/openclose_h2-ts/stream.all.txt

[approve ci autest]

Cherry-picked to 8.1.x
Cherry-picked to v9.0.x branch.


camelCase is wrongly represented as CamelCame

In the guides,
You mistakenly suggest throughout the pages that things should be named using CamelCase notation.
But it is wrongly represented, the correct names should be:
camelCase: lower case for first word, Capital for initial of following words.
PascalCase: Initial for all words should be uppercase.
snake_case: compound word separated by underscore
Despite that, the docs are really good, thanks to put it together.

Thanks for reporting the issue and feedback. Will check and update in this weekend.


Virtual Joystick issue in HITL Simulation

Issue 1) I am using Pixracer in HITL. I followed instructions on https://dev.px4.io/en/simulation/hitl.html?q  to perform HITL simulation. I wanted to use virtual joystick, which I enabled from Application Settings > General and set COM_RC_IN_MODE parameter to "virtual RC by joystick"
However when I fly in any mode requiring RC input, the vehicle keeps yawing even though my joysticks are centered as shown below:

Issue 2) When I move the left joystick up/down, the vehicle yaws, whereas moving left joystick right/left , vehicle pitches. For right joystick, up/down is throttle and left/right is roll. This is not the same as normal mode 2 joysticks/ RCs. How can we change this?
I tried both Gazebo and Jmavsim and I tried both the latest master and stable release v1.7.3 of Firmware.
@dagar

Do you mean physical (usb) joystick? QGC on screen thumbsticks are also sent over mavlink.
Related - https://github.com/PX4/Firmware/issues/9318

@dagar I mean QGC on screen thumbsticks

I think this might be a QGC issue? @dogmaphobic


Optimize PGBK table to only update cache when there is a large enough size change. #21250

This prevents an expensive scenario where a user is outputting lots of small values (e.g. ints) to be precombined and hence takes little to no space to store so updating the cache provides little value.
Note the 5-10x change for all types except for unique keys. Some early profiles show that there is an issue with the G1 garbage collector when storing so many small values that the GC management overhead dominates 75% of the execution which requires further investigation.

Before:
Benchmark                                                 (distribution)  (globallyWindowed)   Mode  Cnt   Score   Error  Units
PrecombineGroupingTableBenchmark.sumIntegerBinaryCombine         uniform                true  thrpt    5   8.306 ± 1.255  ops/s
PrecombineGroupingTableBenchmark.sumIntegerBinaryCombine         uniform               false  thrpt    5   7.849 ± 0.476  ops/s
PrecombineGroupingTableBenchmark.sumIntegerBinaryCombine          normal                true  thrpt    5  10.575 ± 1.295  ops/s
PrecombineGroupingTableBenchmark.sumIntegerBinaryCombine          normal               false  thrpt    5  10.772 ± 0.141  ops/s
PrecombineGroupingTableBenchmark.sumIntegerBinaryCombine          hotKey                true  thrpt    5   9.131 ± 2.761  ops/s
PrecombineGroupingTableBenchmark.sumIntegerBinaryCombine          hotKey               false  thrpt    5   8.302 ± 1.078  ops/s
PrecombineGroupingTableBenchmark.sumIntegerBinaryCombine      uniqueKeys                true  thrpt    5   3.899 ± 1.737  ops/s
PrecombineGroupingTableBenchmark.sumIntegerBinaryCombine      uniqueKeys               false  thrpt    5   4.203 ± 2.170  ops/s

After:
Benchmark                                                 (distribution)  (globallyWindowed)   Mode  Cnt   Score   Error  Units
PrecombineGroupingTableBenchmark.sumIntegerBinaryCombine         uniform                true  thrpt    5  88.740 ± 8.925  ops/s
PrecombineGroupingTableBenchmark.sumIntegerBinaryCombine         uniform               false  thrpt    5  76.005 ± 5.150  ops/s
PrecombineGroupingTableBenchmark.sumIntegerBinaryCombine          normal                true  thrpt    5  43.388 ± 1.966  ops/s
PrecombineGroupingTableBenchmark.sumIntegerBinaryCombine          normal               false  thrpt    5  37.804 ± 7.177  ops/s
PrecombineGroupingTableBenchmark.sumIntegerBinaryCombine          hotKey                true  thrpt    5  84.881 ± 5.040  ops/s
PrecombineGroupingTableBenchmark.sumIntegerBinaryCombine          hotKey               false  thrpt    5  74.183 ± 2.063  ops/s
PrecombineGroupingTableBenchmark.sumIntegerBinaryCombine      uniqueKeys                true  thrpt    5   5.567 ± 4.068  ops/s
PrecombineGroupingTableBenchmark.sumIntegerBinaryCombine      uniqueKeys               false  thrpt    5   6.957 ± 1.508  ops/s

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @bhisevishal

Run Java PreCommit

Thank you @lukecwik this is really awesome news

@bhisevishal this is ready for review.


recording the user's historical resource usage

We need to periodically record the resource usage of the user's all containers, especially the CPU usage.
Note that for a container named ct, if we stop it and restart it, the lxc-info -n ct command will only get the current usage info from the restarting time. The previous history usage info will get lost. So we need a deliberate design to record all the historical resource usage.
@FirmlyReality
@leebaok @zhongyehong @monumentality

git clone https://github.com/unias/dockmeter.git
cd dockmeter
sudo ./main.py
curl -L -X POST -F uuid=[lxc-name] http://<IP_ADDRESS>:1729/v1/billing/increment


Log inputs of function too

I'm trying to decide the best UX to support inputs too, This is what I came up with for now:

keeping the macro the same but from now the logs will look like this: fibonacci(3) => 8.
The problem with that is what if the input is too big?

Do the second option but with a bool like with_inputs, if so should this be opt-in or opt-out?

Make another macro with the same functionality and options as this one but for inputs only,
pros: more functionality. cons: Harder to use and means 2 macro over the functions

keeping the macro the same but from now the logs will look like this: fibonacci(3) => 8. The problem with that is what if the input is too big?

This also raises the question, what do you do if the inputs don't impl fmt::Debug?

Do the second option but with a bool like with_inputs, if so should this be opt-in or opt-out?

If you do this, I'd suggest making it opt-in, and possibly allowing the format string to use named arguments instead of positional ones.

Make another macro with the same functionality and options as this one but for inputs only,
pros: more functionality. cons: Harder to use and means 2 macro over the functions

If you did this, how would you get the inputs and outputs into a single log-line?

@TedDriggs Thanks for the feedback.
That's exactly my problems, the second one means basically no normal nice configurations.
About the third I'm not sure how the TokenStream looks with 2 macros on the same function, Do I get the original TokenStream or the one that came back from the last macro?(and which one runs first)

That's exactly my problems, the second one means basically no normal nice configurations.

As I think about this, input logging comes at a cost, both in terms of constraints on function input (it requires all arguments impl fmt::Debug or fmt::Display, which may make a generic function less useful), runtime performance (extra formatting calls, and keeping the formatted args in memory until the log line can be written), and log size. I think a lightweight opt-in is very reasonable.
darling helps here: You can derive FromMeta on enums to get very expressive options with minimal effort. If you think about what the options should look like, I can show you how to define those.

About the third I'm not sure how the TokenStream looks with 2 macros on the same function, Do I get the original TokenStream or the one that came back from the last macro?(and which one runs first)

I just tested this; it looks like the macros are invoked in order, and each one gets the token stream from the one before it.

I'm more thinking about the UX side.
I don't care if it will even be very hard for me to implement.
i.e. if It's opt-in in the same macro how would it look like to pass multiple fmts and different log levels etc.

Done :)
and darling is a cool library, and I think i'll use it's Error's as my main errors.


LastInsertId and postgres

Should this work? I know with PostgreSQL you need to add the RETURNING clause...
CREATE SEQUENCE foo_id_seq;
CREATE TABLE foo (
  id bigint PRIMARY KEY default nextval('foo_id_seq'::regclass),
  created timestamp with time zone NOT NULL,
  name varchar(255) NOT NULL
);

sql := `INSERT into foo (created, name) VALUES ($1, $2) RETURNING id`
result, err := db.Exec(sql, time.Now(), "somename")
if err {
  panic(err)
}
lastid, lastiderr := result.LastInsertId()
fmt.Println("LastId?", lastid, lastiderr) // prints: LastId? 0 no LastInsertId available

Seems like it works with lib/pq by using QueryRow

Hello!
Quoting from https://godoc.org/github.com/lib/pq:
"pq does not support the LastInsertId() method of the Result type in database/sql. To return the identifier of an INSERT (or UPDATE or DELETE), use the Postgres RETURNING clause with a standard Query or QueryRow call".
Perhaps you misread the statement.

Hi, thanks -- I did miss that line about using Query or QueryRow -- thanks.  I have it working now.

I might be missing something but there's no Named version that returns a single row? That would be handy

There's no one-liner for it.  NamedExec and NamedQuery came before PrepareNamed, and I decided to keep them mostly for convenience.  Rather than cluttering up the DB, Tx, etc structs with Named variants of everything (since sqlx adds a couple new verbs as well like Select, Get, etc.), I decided to leave most of that on the NamedStmt.  So you can do:
stmt, err := db.PrepareNamed("INSERT INTO person (name, email) VALUES (:name, :email) RETURNING id")
// check error
var id int
err = stmt.Get(&id, arg)

It isn't much more difficult to use an interface and define an all-in-one type function:
type namedPreparer interface {
    PrepareNamed(string) (*sqlx.NamedStmt, error)
}

func NamedGet(db namedPreparer, dest interface{}, query string, arg interface{}) error {
    stmt, err := db.PrepareNamed(query)
    if err != nil {
        return err
    }
    return stmt.Get(dest, arg)
}

FWIW in most drivers this is basically what happens when you use eg Query or QueryRow with bindvars + args, anyway.

I'd like to leave a note about the suggestion above.

There's no one-liner for it. NamedExec and NamedQuery came before PrepareNamed, and I decided to keep them mostly for convenience. Rather than cluttering up the DB, Tx, etc structs with Named variants of everything (since sqlx adds a couple new verbs as well like Select, Get, etc.), I decided to leave most of that on the NamedStmt. So you can do:
stmt, err := db.PrepareNamed("INSERT INTO person (name, email) VALUES (:name, :email) RETURNING id")
// check error
var id int
err = stmt.Get(&id, arg)

Actually there are situations when you don't want to create prepared statement as suggested.
For example, when your work with PgBouncer in Transaction pooling mode, when prepared statements are disabled.
So the most accurate suggestion is to use sql.Named like this:
query, args, err := sqlx.Named("INSERT INTO person (name, email) VALUES (:name, :email) RETURNING id", arg)
if err != nil {
	<handle error>
}
query = db.Rebind(query)
var id int
err = db.Get(&id, query, args...)
...


Bump carbon identity framework version

$subject

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/6307844239

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/6307844239
Status: success


env: node\r: No such file or directory

Hi everyone, I am trying to transform a file called baseSteps.js to webdriverIO file using command
npx jscodeshift ../node_modules/@wdio/codemod/protractor/ test/steps/baseSteps.js
Before this i have installed :
npm install --save-dev @wdio/cli
npx wdio config  (ran the configuration wizard)  I have configured cucumber framework.
After which I ran
npx jscodeshift ../node_modules/@wdio/codemod/protractor/ test/steps/baseSteps.js
In package.json I have all the dependencies

So when I run npx jscodeshift ../node_modules/@wdio/codemod/protractor/ test/steps/baseSteps.js
it throws :
env: node\r: No such file or directory
Not sure what I am missing. though baseSteps.js file exists in test/steps folder. Can someone suggest?
Note: I am using mac OS. I was going through help in some other articles and I think this issue is caused by line endings with CRLF, where as its must be LF. For ex:  https://github.com/mrmlnc/svg2sprite-cli/issues/4
and many more when we search for env: node\r: No such file or directory hopefully this will give an idea

You were missing a parameter, see our Usage section, I believe it is suppose to be this way:
npx jscodeshift -t ./node_modules/@wdio/codemod/protractor ./test/steps/baseSteps.js

Please let me know if you have further questions.

Unfortunately, using -t parameter dint help.  Its still throws env: node\r: No such file or directory.

For reference, I think this is an upstream issue with jscodeshift: https://github.com/facebook/jscodeshift/issues/424

It would be helpful to update the docs to indicate that the workaround described here https://github.com/facebook/jscodeshift/issues/424#issuecomment-888243971 works.

@kliph-gladly mind raising a PR?

I've encountered larger issues with the @wdio/codemod/async adding await keyword to function calls for functions that do not return promises and failing to add async to anonymous inline functions.  I anticipate that I won't be using this in the future.

You will only need the codemod package as utility to rewrite your tests.

Thanks.  Yes I'm aware.  The larger issue that I mentioned is that the utility produces code with syntax errors.

Can you provide a code example that causes a syntax error when rewriting the code with this codemod?

Sure:
const foo = function() {
  mySyncFunctionDoesNotReturnAPromiseOrEvenExist();
};

produces
const foo = function() {
  await mySyncFunctionDoesNotReturnAPromiseOrEvenExist();
};

This is syntactically incorrect for two reasons, the outer function context is not async so it doesn't make sense to use await within it.  And the function called in the inner scope does not return a promise, so it doesn't make sense to use await.

Yep, this is a bug, mind raising a new issue for it?

Will do.  Thank you.


chore: add Anomaly Tests

Comments for the various test cases will help

There have been a lot of bugs for anomaly over the time, are you sure you have covered all those cases?

I've tried covering all the cases that I know of. Also for every function I've added test cases for both daily and hourly.

For comments, each test case is assigned an id (inside the @pytest.mark.parametrize) which describes what the test case is about.

LGTM


Need to free graphics memory when Flutter screen dismissed in add-to-app

Reproducible with https://github.com/flutter/samples/tree/master/add_to_app/fullscreen/android_fullscreen. The Flutter engine is long lived. When we navigate to the Flutter activity, we end up consuming more native memory and graphics memory. This is as expected. But when we dismiss the Flutter activity and keep the Flutter engine, we should expect to free all surfaces and graphics buffers associated with that view and free as much of the graphics memory as possible.
It didn't seem to happen currently. On a Pixel 3, the graphics memory went from 24.7 to 49.1 and stayed at 49.6 after leaving the Flutter activity.

@gaaclarke related to our customer conversation today. cc @blasten too.

@GaryQian will take a look.
It seems like we are missing a benchmark for this scenario. The benchmark should be pretty straightforward to add.

@GaryQian have you had a chance to look into this issue?

Not yet, I'll try to get to this next!

Ran a AAR build of Android Fullscreen sample app, and was able to get a version that does not repro this. Was able to repro with the direct dependency version of Android Fullscreen though.

That's really interesting. I can't think of a reason why they'd be different. I think I tried the release version above.

It did not have anything to do with the use of AAR. It looks like this issue is fixed in the latest version of flutter. I can no longer repro it on master debug profile or release, while I was able to repro it before.

I'll close this issue for now.

How hard would it be to add a benchmark that measures the graphics memory after dismissing the Flutter screen?

I don't think we have any tooling/infra setup to get the detailed profiling data from an AVD or device as far as I know, @godofredoc can you confirm? I'll file an issue to track it.

I thought we could get the data with adb dumpsys. We do something similar with the gallery app.

If this bug happened and went away without any of us noticing in code reviews, then a benchmark would definitely be useful :)


Duplicate class found

After adding the library in my app-level build.gradle file, I am now getting tons of error saying Duplicate class found. I checked my libs folder as well but it's empty. Almost tried all the solutions on StackOverflow but no luck. Any guesses?

could you please share the details of classes, and problem details as well

There are tons of classes which shows duplicated.
On Sun, 11 Dec 2022 at 12:01 PM, Chirag Gupta @.***>
wrote:

could you please share the details of classes, and problem details as well
—
Reply to this email directly, view it on GitHub
https://github.com/chirag-ji/DocuConverter/issues/15#issuecomment-1345475037,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANKR7ERD5TP2EWQABEWTWQTWMV36HANCNFSM6AAAAAASXISTRU
.
You are receiving this because you authored the thread.Message ID:
@.***>

I have also attached the screenshot of the errors. Please see attached.
On Sun, Dec 11, 2022 at 1:18 PM Muhammad Salman Khan <
@.***> wrote:

There are tons of classes which shows duplicated.
On Sun, 11 Dec 2022 at 12:01 PM, Chirag Gupta @.***>
wrote:

could you please share the details of classes, and problem details as well
—
Reply to this email directly, view it on GitHub
https://github.com/chirag-ji/DocuConverter/issues/15#issuecomment-1345475037,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANKR7ERD5TP2EWQABEWTWQTWMV36HANCNFSM6AAAAAASXISTRU
.
You are receiving this because you authored the thread.Message ID:
@.***>

for those who got error, just add this in android (build.gradle)
configurations {
        all {
            exclude group: 'org.apache.poi', module: 'ooxml-schemas'
            exclude group: 'org.apache.poi', module: 'poi-ooxml-lite'
        }
    }

please take a look at https://github.com/chirag-ji/DocuConverter/issues/14#issuecomment-1235036211.
Thanks

okay so I added this in app level build.gradle and now I am getting the
following errors. Please see attached. Also please note that I want to
target android 5.0 and above.
On Wed, Dec 14, 2022 at 7:59 AM hmduc1603 @.***> wrote:

for those who got error, just add this in android (build.gradle)
configurations {
all {
exclude group: 'org.apache.poi', module: 'ooxml-schemas'
exclude group: 'org.apache.poi', module: 'poi-ooxml-lite'
}
}
—
Reply to this email directly, view it on GitHub
https://github.com/chirag-ji/DocuConverter/issues/15#issuecomment-1350308313,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANKR7EW7PIS75KY5W7G3X7DWNEZY3ANCNFSM6AAAAAASXISTRU
.
You are receiving this because you authored the thread.Message ID:
@.***>


install platform video

hello
now my server ready to install platform but i don't know how can i open the platform to counties the install steps
if there are video will be more than easier

Hello @saad-elalfy ,
We don't have video for now. We will try to add some video guide asap. In the meantime you can look at the install doc : https://github.com/Cocolabs-SAS/cocorico/tree/master/doc


python311Packages.pontos: 23.12.1 -> 24.1.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python311Packages.pontos is: Collection of Python utilities, tools, classes and functions
meta.homepage for python311Packages.pontos is: https://github.com/greenbone/pontos
meta.changelog for python311Packages.pontos is: https://github.com/greenbone/pontos/releases/tag/v24.1.1
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 24.1.1 with grep in /nix/store/apn869n30n2mca1jg1i4r5i3l8x6w4g9-python3.11-pontos-24.1.1
found 24.1.1 in filename of file in /nix/store/apn869n30n2mca1jg1i4r5i3l8x6w4g9-python3.11-pontos-24.1.1

Rebuild report (if merged into master) (click to expand)

5 total rebuild path(s)

5 package rebuild(s)

First fifty rebuilds by attrpath
faraday-agent-dispatcher
gvm-tools
python311Packages.gvm-tools
python311Packages.pontos
python311Packages.python-gvm

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/apn869n30n2mca1jg1i4r5i3l8x6w4g9-python3.11-pontos-24.1.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python311Packages.pontos https://github.com/r-ryantm/nixpkgs/archive/a83fe2fde4ad2f62d66bf1a393939740221d8d10.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/a83fe2fde4ad2f62d66bf1a393939740221d8d10#python311Packages.pontos

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/apn869n30n2mca1jg1i4r5i3l8x6w4g9-python3.11-pontos-24.1.1
ls -la /nix/store/apn869n30n2mca1jg1i4r5i3l8x6w4g9-python3.11-pontos-24.1.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

3 packages failed to build:

faraday-agent-dispatcher
gvm-tools (python311Packages.gvm-tools)
python311Packages.python-gvm

1 package built:

python311Packages.pontos

Maintainer pings
cc @fabaff for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python311Packages.pontos is: Collection of Python utilities, tools, classes and functions
meta.homepage for python311Packages.pontos is: https://github.com/greenbone/pontos
meta.changelog for python311Packages.pontos is: https://github.com/greenbone/pontos/releases/tag/v24.1.2
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 24.1.2 with grep in /nix/store/43ssrbhk5fvlwyixwl0yhk69x0n9cpbs-python3.11-pontos-24.1.2
found 24.1.2 in filename of file in /nix/store/43ssrbhk5fvlwyixwl0yhk69x0n9cpbs-python3.11-pontos-24.1.2

Rebuild report (if merged into master) (click to expand)

5 total rebuild path(s)

5 package rebuild(s)

First fifty rebuilds by attrpath
faraday-agent-dispatcher
gvm-tools
python311Packages.gvm-tools
python311Packages.pontos
python311Packages.python-gvm

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/43ssrbhk5fvlwyixwl0yhk69x0n9cpbs-python3.11-pontos-24.1.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python311Packages.pontos https://github.com/r-ryantm/nixpkgs/archive/3322ec64f613709b6a2bab0876e33d214846f3ae.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/3322ec64f613709b6a2bab0876e33d214846f3ae#python311Packages.pontos

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/43ssrbhk5fvlwyixwl0yhk69x0n9cpbs-python3.11-pontos-24.1.2
ls -la /nix/store/43ssrbhk5fvlwyixwl0yhk69x0n9cpbs-python3.11-pontos-24.1.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

4 packages built:

faraday-agent-dispatcher
gvm-tools (python311Packages.gvm-tools)
python311Packages.pontos
python311Packages.python-gvm

Maintainer pings
cc @fabaff for testing.

@ofborg eval


enables EBC for test inside semeru builds

It add a function called allocateEBCnodesForTest which return label for new generated EBC nodes and let test use it.

a sample run for this new EBC+AQA tests in Semeru-JDK22 build:
https://hyc-runtimes-jenkins.swg-devops.com/job/build-scripts/job/jobs/job/jdk22/job/jdk22-linux-x64-openj9/50/consoleFull

@AdamBrousseau the PR is ready for review. Could you please advise?


Embedded-SDK getGuestTokenRefreshTiming missing buffer.from

A clear and concise description of what the bug is.
Followed along with example deployment. Successfully passing in the first token.
Afterwards it attempts to do getGuestTokenRefreshTiming and fails as Buffer.from is not found
How to reproduce the bug

Follow along with example
Navigate to the webpage

Expected results
Function runs without issue
Actual results
guestTokenRefresh.js:41 Uncaught (in promise) ReferenceError: Buffer is not defined.       at getGuestTokenRefreshTiming (guestTokenRefresh.js:41:1)        at _callee3$ (index.js:137:1)        at tryCatch (runtime.js:63:1)        at Generator.invoke [as _invoke] (runtime.js:293:1)        at Generator.next (runtime.js:118:1)        at asyncGeneratorStep (asyncToGenerator.js:3:1)         at _next (asyncToGenerator.js:25:1)
Environment

browser type and version: chrome 105.0.5195.102
superset version: 2.0.0
node.js version: v16.17.0

Checklist
Make sure to follow these steps before submitting your issue - thank you!

[x] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[x] I have reproduced the issue with at least the latest released version of superset.
[x] I have checked the issue tracker for the same issue and I haven't found one similar.

Additional context
Adding a buffer replacement to guestTokenRefresh.js like npmjs.com/package/buffer removes the error

refer to: https://github.com/apache/superset/pull/21641

Awaiting on this fix. Hope it merges soon


Option to store images in an external folder #FeatureRequest

Hi @scubajeff,
I know, that you probably thought about this a lot, since you mentioned your reasoning (privacy) in the readme file, but I wanted to give you 2 arguments for giving the user the option to store media in an external directory.

Preventing data loss. Uninstalling les pas would no longer remove all users media (see https://github.com/scubajeff/lespas/issues/56).
Decoupling from nextcloud backend. Les pas could be used as a local gallery and album manager and the external media directory could be synced for example through syncthing (p2p) or foldersync (FTP, SSH, Samba, Webdav, a lot of cloud providers).

I'm curious about your opinion.

All your points are valid. But there is a technical reason why the change is not a easy jump. Les Pas saves all local image files in a single folder and name them with a unique internal id, the actual folder/file structure map is stored in a database. I would say it's nearly impossible to do the refractoring.

Okay, now I understand your decision (of course my first point would still be valid in this case).
I think les pas needs a way to export all media from les pas private directory in that case (something like a new hamburger-menu entry point "Export all local media"). I would definetly feel more safe to "move" data

I would rather pay more attention on fixing the 'waggy' sync

Since les pas saves media in its private storage and currently there is no way to extract them (but going in each album individually and selecting each image individually and then share them to a file app), I think, there should be a better way to extract images without having access to nextcloud. Without this it is dangerous to move files to les pas.


[guides] update site/en/guides/build/logs.md

Update site/en/guides/build/logs.md. For the NCP section, the instructions are pretty outdated and I failed to let wpantund forward logs to syslog. Anybody can point me a doc/codelab to setup NCP & wpantund? Thanks!

@superwhd  Any update on this PR?

@superwhd Any update on this PR?

Hi Jeff, this PR is not ready since I don't have much idea on how to properly print NCP logs via spinel. The argument OPENTHREAD_CONFIG_LOG_OUTPUT_APP seemed not to work.
@bukepo @librasungirl What's your suggestion?

Updated after https://github.com/openthread/wpantund/issues/494 was solved. @simonlingoogle Please take a look.


DoS protection

There are a number of things we can do to help protect ourselves from DoS attacks.   This isn't a live or NRT system.  We don't have a need for multiple/successive POST/GET requests from a single client.
In addition to the things in these articles, we should modify the ufw firewall.  We do have the AWS Security Groups, but this would be another layer.  (Obviously doesn't help "allowed" ports)
https://www.unixmen.com/protecting-apache-server-denial-service-dos-attack/
https://bobcares.com/blog/apache-prevent-ddos/

https://github.com/FreeLoaderz/rwd-frontend/pull/79


[SD-488] [SD-779][SD-717] Segment/GA with turbolinks

After selecting place order button on checkout/confirm step I get this error: Uncaught SyntaxError: Unexpected token '{' in console . Please verify it before merging.

@Punkbooster could you please update the description on what this PR is about?


Adding a mark as important button in SmartNotezz-WebApp

Adding a mark as important is a feature commonly found in productivity applications. Its purpose is to allow users to prioritize and highlight specific items, messages, tasks, or notes as important or high priority. In this SmartNotezz-WebApp, it can be used to highlight the important notes added.

Please assign me this under SSOC


dependency problem with fsnotify?

simplehttpserver version:
latest as of 12/09/2023
Current Behavior:
On windows/amd64, trying to install as per README does not work. However, compiling from trunk does.
Expected Behavior:
Package installs successfully and simplehttpserver.exe becomes available.
Steps To Reproduce:
> go version
go version go1.21.1 windows/amd64

> go install -v github.com/projectdiscovery/simplehttpserver/cmd/simplehttpserver@latest
github.com/projectdiscovery/simplehttpserver/internal/runner
# github.com/projectdiscovery/simplehttpserver/internal/runner
..\..\..\go\pkg\mod\github.com\projectdiscovery\simplehttpserver@v0.0.6\internal\runner\watchdog.go:19:32: watcher.Events undefined (type *fsnotify.Watcher has no field or method Events)
..\..\..\go\pkg\mod\github.com\projectdiscovery\simplehttpserver@v0.0.6\internal\runner\watchdog.go:28:19: watcher.Errors undefined (type *fsnotify.Watcher has no field or method Errors)

>

After this, where simplehttpserver.exe cannot find the file.
However, cloning the repo and go build does work, and produces the exe - which runs fine.
Anything else:

Thanks for the issue! We'll be taking a look soon.


adding a green flash to new rows in the history table on refresh

I had this refactor left over from the observe ajax work and didn't want to throw it away

I renamed array to nodes in pluckData since the argument should be a NodeList instance and added a docstring to try to clarify that for future developers. I also added a mention of the functionality change to the changes documentation. When CI passes, I'll merge it.


Is there a way to use difftastic for rendering diffs on github pullrequests?

Would be cool.

There isn't I'm afraid. Long term I want to add an HTML output option (see #45) which you might be able to build something on top of.


Error in build/npm/waypoint.js?

Hello,
I believe that your build process has created an error in the built file.
At the top we find:
'use strict';

var _createClass = (function () { function defineProperties(target, props) { for (var i = 0; i < props.length; i++) { var descriptor = props[i]; descriptor.enumerable = descriptor.enumerable || false; descriptor.configurable = true; if ("value" in descriptor) descriptor.writable = true; Object.defineProperty(target, descriptor.key, descriptor); } } return function (Constructor, protoProps, staticProps) { if (protoProps) defineProperties(Constructor.prototype, protoProps); if (staticProps) defineProperties(Constructor, staticProps); return Constructor; }; })();

Object.defineProperty(exports, "__esModule", {
  value: true
});

var _react = require('react');

var _react2 = _interopRequireDefault(_react);

var _reactDom = require('react-dom');

var _reactDom2 = _interopRequireDefault(_reactDom);

function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }

function _classCallCheck(instance, Constructor) { if (!(instance instanceof Constructor)) { throw new TypeError("Cannot call a class as a function"); } }

function _possibleConstructorReturn(self, call) { if (!self) { throw new ReferenceError("this hasn't been initialised - super() hasn't been called"); } return call && (typeof call === "object" || typeof call === "function") ? call : self; }

function _inherits(subClass, superClass) { if (typeof superClass !== "function" && superClass !== null) { throw new TypeError("Super expression must either be null or a function, not " + typeof superClass); } subClass.prototype = Object.create(superClass && superClass.prototype, { constructor: { value: subClass, enumerable: false, writable: true, configurable: true } }); if (superClass) Object.setPrototypeOf ? Object.setPrototypeOf(subClass, superClass) : subClass.__proto__ = superClass; }

var POSITIONS = {
  above: 'above',
  inside: 'inside',
  below: 'below',
  invisible: 'invisible'
};

var propTypes = {
  // threshold is percentage of the height of the visible part of the
  // scrollable ancestor (e.g. 0.1)
  threshold: _react.PropTypes.number,
  onEnter: _react.PropTypes.func,
  onLeave: _react.PropTypes.func,
  fireOnRapidScroll: _react.PropTypes.bool,
  scrollableAncestor: _react.PropTypes.any
};

In the case that the result of var _react = require('react'); is an es5 module (so that the actual React object is present at _react.default), the var _react2 = _interopRequireDefault(_react); will do nothing, as _react.default already exists. I'm no expert, but I guess the idea is to have all react accesses replaced with _react2.default. However, the PropTypes are still accessed through the original _react object, which throws an error since it does not contain any of the React suff. All other calls still works fine though, as they have properly been replaced with _react2.default. This is no problem when _react is exposed as a non-es5 module, since _react then contains all React stuff.
Any thoughts?

Sorry about this. This is my fault. I converted this to use ES6 export but didn't realize that Babel 6 changed it to make this inconvenient for people not using require instead of import.
The quick fix for you would be to either use import Waypoint from 'react-waypoint'; or .default as you've pointed out. Long term, we should either convert this back to use module.exports instead of export, or add this Babel plugin to smooth over this for folks. https://www.npmjs.com/package/babel-plugin-add-module-exports

I'm a little confused @lencioni, I added the add-module-exports plugin in b7452e687.
@wnr, could you share a bit more detail on your setup? Are you using webpack, babel or some other bundling tool? And what version of react-waypoint & react are you using?

Ah, I didn't notice that. That should definitely take care of this problem.

Hello, and sorry for my late response.
The issue is not to require react-waypoint, but rather how react-waypoint requires react.
@trotzig Sure. I use browserify with Babel and all that stuff. I want to split up my code into 1) global singletons such as React and 2) my code. To do this i use browserify -x react to exclude the react source from my code bundle. Then I have a react-shim.js file that contains export default window.React; that gets built with browserify and told to export the react name with browserify -r react-shim.js:react. So my index.html file looks something like this:
<script src="cdn/react.js"></script>
<script src="react-shim.js></script>
<script src="app.js"></script>

Everything works fine, except for react-waypoint which throws the following error: Uncaught TypeError: Cannot read property 'number' of undefined at line 35 which I talked about in my first post.
If I change my react-shim.js file to instead export with the CommonJS syntax module.exports = window.React;, everything works fine. So it seems to me that the built react-waypoint file is dependent on react being exported with the CommonJS syntax, which seems like an error (probably in your build tool). Or am I missing something?

I have to admit, this is a little outside my comfort zone. I wonder if @lencioni has anything thoughtful to share?
If you are looking for a workaround in the meantime, would adding this line to the bottom of react-shim.js help?
module.exports = exports['default'];

That's what babel-plugin-add-module-exports does for us in our build process.

I have been using the module.exports =  as a workaround so it's not really any problem for me. Just wanted to let you know :)

I'm glad that works for you. Unless others are also bitten by this, I'm don't think this is worth fixing. Others: if you end up having the same problems as @wnr, please reopen this issue!


ValueError: Input 0 of layer gru_cell_2 is incompatible with the layer: its rank is undefined, but the layer requires a defined rank.

hi,
I met the problem ValueError: Input 0 of layer gru_cell_2 is incompatible with the layer: its rank is undefined, but the layer requires a defined rank.
Do you know how do solve it, I have no idea.

solved it, maybe the version of tensorflow is not suited.

i met the problem too. what's the suitable version of tf. thx.


Need one more click on Macos system tray

When user right click tray icon many times.
Reproduce step:

right click icon on tray icon
right click again. (it wired from here, the tray icon still at select status)
we need right click again to leave this status.

I found biyi have same issue. other application did not have this issue.

Fixed, will be released in next version


Step prerequisite blanks out the page

when typing into a prerequisite will blank out the page and everything disappears

Fixed with 79aa810318f158312f8829aeeb2816795f8cd393


[kafka-ws] Adding NOTICE file for license exceptions

Please check if the PR fulfills these requirements

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

What is the current behavior? (You can also link to an open issue here)

What is the new behavior (if this is a feature change)?

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

Is there any issue related to this PR? (Link to an issue, e.g. #99999)
This PR is related to dojot/dojot#2082.

Other information:

Plase, create to release/v0.6.0 too.


Distinguishes zoom event and zoom option in the docs (#6660).

Just a quick PR to fix the duplicate IDs for 'zoom' event & option.
:wave:

This might be indicative of a deeper problem with LeafDoc. Is this specific of the 1.5.0 docs, or does it happen when building the docs from source?


Refactor enumerations

Hi there,
second attempt for a new package structure at the enumerations level. All enumerations related to autodiscover have been moved there. Additionally other enumerations are moved to core -> enumeration -> XXXX. Splitted enumerations to several subpackages:

attribute
availability
dns
misc
notification
permission
property
search
service
sync

Any feedback or suggestions are welcome.

Hi @serious6, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
We will now validate the agreement and then real humans will evaluate your PR.
TTYL, AZPRBOT;

related to #39

:+1:  - did we based this re-structuring on ews-managed-api or did you come up with the split?

@vboctor this is some kind of split with ews-managed-api since all enumerations are stored in the enumerations package there. Nevertheless I think this PR provides a more fitting aproach due to the already refactored Interfaces and Exceptions that also make a split from ews-managed-api.

@vboctor are you fine with this changes?

yes


Got unexpected error without any clue how to work with it in CRA

Current behavior
npm run build
><EMAIL_ADDRESS>build
> react-scripts build

Creating an optimized production build...
Failed to compile.

fork-ts-checker-webpack-plugin error in undefined(undefined,undefined):
Value undefined out of range for undefined options property undefined  TSINTERNAL

npm start
Files successfully emitted, waiting for typecheck results...

Expected behavior
It should build without error and with npm start show compiled files
Steps to reproduce the issue
I've just migrated code when update react-hook-form from v6 to v7. It's really hard to reproduce, this error message gives no clue where the issue occurs
Issue reproduction repository
It's very hard to reproduce, looks like I need to make some mistake in code, but don't know which
Environment

react-scripts: [4.0.3`]
typescript: [4.3.5]
os: [e.g. OS X Big Sur 11.5.2]

I've found the reason of this bug. That was a nested type and it takes couple days to find that with removing files one by one from a codebase.
Who's using react-hook-form@v7 avoid approach like below on your integration components:
export const MultipleAutocompleteField = (props: MultipleAutocompleteFieldProps<T>) => {
  return (
    <Controller<ReactElement<MultipleAutocompleteFieldProps<T>>>
      ...
    />

Sry for opening the issue here.


Need to stop using grpc.load

grpc.load is deprecated so we need to switch to proto-loader. This refactor was started in #229 but some more work needs to be done.
Also, I'm planning to rethink the idea of COMMON_PROTOS: since proto-loader supports searching in several directories (includeDirs option), this logic can be offloaded to it.
TODO:

Do not use grpc.load.
Simplify COMMON_PROTOS logic.
Make tests pass.
???
PROFIT!

The merged change fails on LRO, still working on a fix.


Feature Request: Add ability to exclude youtbe playlists when downloading from whole channel

Checklist

[x] I'm reporting a feature request
[x] I've verified that I'm running youtube-dl version 2019.05.20
[x] I've searched the bugtracker for similar feature requests including closed ones

Description
If possible, there could be an ability exclude certain or more playlists when downloading from the whole youtube channel, some channels create smaller videos and don't put them all into a playlist, but they do have a playlist for longer "full-version" videos such as podcasts, and I have to separate the download script so I download the full ones which much lower video quality, no need for video quality if there's 2 heads talking in an office for 3 hours.

Duplicate of #8413.


[help] Does it require CUDA 10.x or can it run on CUDA 11.x

Describe the bug
When running main.py will the instruction in readme,
log complaint that I don't have "libcusolver.so.10". So I wasn wondering if it looks specifically for CUDA 10.x
'save_config' require google cloud crediential. Do I have to use google cloud, or can I use GPU locally?

I can confirm that I have run the 2.7B parameter model on my RTX 3090.  It takes some bug fixing of the code, but it works with CUDA 11.  I will go over it more later.


[DOCS-76] algolia test

Changelog
Added

Page
Deploy Preview
Comment

[Name]
[Link]
[Comment]

Updated

Page
Deploy Preview
Comment

[Name]
[Link]
[Comment]

Preview This Change
To see how this change looks in production, scroll down to Deploy Preview. Select the link that looks like https://deploy-preview-<num>--cobalt-docs.netlify.app/
Variables
Help us support a “Write once, publish everywhere” single source of truth. If you see a line that looks like:

You’ve found a shortcode that we include in multiple documents.
You’ll find the content of the shortcode in the following directory:
https://github.com/cobalthq/cobalt-product-public-docs/tree/main/layouts/shortcodes
That shortcode has the same base name as what you see in the PR, such as asset-categories.html.

Closing. We've implemented GCS, at least for now.


Vulnerable dependencies found by OWASP Dependency Check

One or more dependencies were identified with known vulnerabilities in shift-rest:
jackson-databind-<IP_ADDRESS>.jar<EMAIL_ADDRESS>cpe:2.3:a:fasterxml:jackson:<IP_ADDRESS>:::::::, cpe:2.3:a:fasterxml:jackson-databind:<IP_ADDRESS>:::::::) : CVE-2019-14540, CVE-2019-16335, CVE-2019-16942, CVE-2019-16943, CVE-2019-17267, CVE-2019-17531

@InfoSec812 ; seems like this can be closed.


Add ability to configure arbitrary additional CLI args for each dependency

I'm imagining a feature like this:
config.py:
YOUTUBEDL_ARGS = ['-f', 'bestvideo[filesize<500M][height<=?480]+bestaudio/best']
...
WGET_ARGS = ['--no-warc-compression']
...

extractors/youtubedl.py:
...
CMD = [
    YOUTUBEDL_BIN,
    ...
    *YOUTUBEDL_ARGS,
]
...

This will allow people to self-configure the extractor behavior with much more customizability, without requiring archivebox-level config flags for each separate feature.

How does this work with the refactor to django's settings? Can it be done at the same time? should we get this one out first?

Don't bother with the Django settings refactor for now, I have an idea of how to do it and I'd like to do it myself later on with a module re-organization. Get this one out first as it's quite simple.

@pirate we have the following cmd in the media extractor:
cmd = [
        YOUTUBEDL_BINARY,
        '--write-description',
        '--write-info-json',
        '--write-annotations',
        '--write-thumbnail',
        '--no-call-home',
        '--no-check-certificate',
        '--user-agent',
        '--all-subs',
        '--extract-audio',
        '--keep-video',
        '--ignore-errors',
        '--geo-bypass',
        '--audio-format', 'mp3',
        '--audio-quality', '320K',
        '--embed-thumbnail',
        '--add-metadata',
        *(['--yes-playlist'] if SAVE_PLAYLISTS else []),
        *([] if CHECK_SSL_VALIDITY else ['--no-check-certificate']),
        link.url,
    ]

The idea is to remove SAVE_PLAYLISTS and CHECK_SSL_VALIDITY and pass the new YOUTUBEDL_ARGS config, right? The default should contain all of these defaults?

@pirate I just pushed a commit (https://github.com/pirate/ArchiveBox/pull/506). Can you please check if that is what you had in mind? I will wait until you check it to move forward with other extractors.

@pirate Closing this. Please reopen if there is something else we are missing for this.


Loop?

Seems like the loop attribute isn't supported. Is this true?

I added it in a test version, can you try this?
https://raw.githubusercontent.com/bfred-it/iphone-inline-video/90e437733e67b7209c2f07972c3002f9aec54111/dist/iphone-inline-video.browser.js

Works perfectly! Thanks!


⚠ Bofrost.be has degraded performance

In 0469ebb, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1807 ms

Resolved: Bofrost.be performance has improved in e666eb4.


[Runtime] Short-circuit search for context descriptor for protocol extensions

When looking for a context descriptor for a protocol extension, we
search based on the mangled name--"x", a generic type parameter with
depth and index 0--and are guaranteed to fail because there is no such
concrete type. However, _findContextDescriptor will fail very slowly,
spinning through all of the types and conformances in all of the
loaded images. Moreover, negative results aren't cached, so this can
happen repeatedly.
Short-circuit _findContextDescriptor when it receives a dependent
generic type parameter type, avoiding the expensive search when it
will find nothing.
Potential fix for rdar://problem/53560010.

@swift-ci please smoke test


How to use NextUI with twind?

Nextui can be easy to set up with tailwindcss. But I dont know how to use it with twind. I have tried many way and spent several days in that.
Or can I use nextui with pure tailwindcss in this project? How to do that?
Thank you for your reply!

The underlying problem is probably a bunch of unfamiliarity, as I'm not an advocate for tailwindcss.

@lalawila Did you manage to setup tailwind with NextUI? I managed to setup tailwind but it looks like the tailwind styles are not being applied to those NextUI components.

Had similar issue. The problem was:
As documentation suggests, you should  extend tailwind.config.js by adding "./node_modules/@nextui-org/theme/dist/**/*.{js,ts,jsx,tsx}" into content array
Given that I extended the config file inside the @extension/popup module, "./node_modules/@nextui-org/theme/dist/**/*.{js,ts,jsx,tsx}" was linking to non-existing directory (actually, @nextui-org was installed to the root project node_modules)
Hence I linked it this way - "../../node_modules/@nextui-org/theme/dist/**/*.{js,ts,jsx,tsx}"
Works!


Relative Links are not clickable in Terminal

Issue Type: Bug
Building using gcc and none of the relative links are clickable in the Terminal. Absolute links work.
Tried setting "fileLocation": [ "relative", "${workspaceFolder}/out/build"], and still no luck.
Tried possible solutions in #58256 but no luck.
VS Code version: Code 1.41.1 (26076a4de974ead31f97692a0d32f90d735645c0, 2019-12-18T15:04:31.999Z)
OS version: Linux x64 5.0.0-37-generic

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-6700HQ CPU @ 2.60GHz (8 x 3088)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: disabled_softwareskia_renderer: disabled_offsurface_control: disabled_offsurface_synchronization: enabled_onvideo_decode: unavailable_offviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
3, 3, 3

Memory (System)
31.31GB (1.89GB free)

Process Argv
--no-sandbox --unity-launch

Screen Reader
no

VM
0%

Extensions (15)

Extension
Author (truncated)
Version

alignment
ann
0.3.0

codealignment-vscode
cpm
2.0.19

gitlens
eam
10.2.0

cortex-debug
mar
0.3.4

vscode-docker
ms-
0.9.0

python
ms-
2019.11.50794

cmake-tools
ms-
1.2.3

cpptools
ms-
0.26.2

vs-keybindings
ms-
0.2.0

vsliveshare
ms-
1.0.1404

clang-tidy
not
0.2.2

code-spell-checker
str
1.7.20

cmake
twx
0.0.17

vscodeintellicode
Vis
1.2.2

better-align
wwm
1.1.6

/needsMoreInfo
Can you provide examples of paths that are not clickable?
Most likely duplicate of #21125

I'm pretty sure this is a regression. The link below (src/md-sections.test.ts:22:36) isn't clickable.

Still work fine for me in the terminal, it's probably related to the cwd of the terminals.

Same screen capture, full paths work, relatives don't.

Not sure about the cwd comment, all relative paths are from build folder.

@sharadgupta-north tasks doesn't do anything special for clickable links in the terminal, it just relies on the terminal link detection. If you run the same command that your task runs manually in the terminal, are the links clickable?

No they are not.

The fileLocation you mentioned in the original post is for the links in Problems that are created from running the task, so it also has no affect on the terminal links.
I suppose tasks could create a new terminal link matcher based on the problem matcher, but right now that's out of scope.

That was me just trying to fix my problem not the expected solution.
As 'jacksteamdev' mentioned it seems to be a regression and not sure how a regression could be a design decision?
So what you are saying is relative links should not work as clickable links? you are basically excluding a major chunk of your user base.

Relative links that are relative to the terminal's cwd should be clickable.
Example where the following exist:
c:\one\two\filetwo.txt
c:\one\fileone.txt
Terminal cwd is c:\one.
This relative path should be clickable: fileone.txt
This relative path should be clickable: \two\filetwo.txt
This path is not relative to the cwd and shouldn't be clickable: filetwo.txt.
@Tyriar please correct me if I'm wrong!

This relative path should be clickable: fileone.txt
This relative path should be clickable: \two\filetwo.txt
This path is not relative to the cwd and shouldn't be clickable: filetwo.txt.

Paths must start with .\ or ..\ for Windows. Simply fileone.txt is not specific enough currently (improvements planned in https://github.com/microsoft/vscode/issues/21125, https://github.com/xtermjs/xterm.js/issues/583)

@Tyriar thanks for the clarification!
Corrected example:
c:\one\two\filetwo.txt
c:\one\fileone.txt
Terminal starting cwd is c:\one.
This relative path should be clickable: .\fileone.txt
This relative path should be clickable: .\two\filetwo.txt
This path is not relative to the_starting_ cwd and shouldn't be clickable: .\filetwo.txt.


#447 uui-box: add a property to control header variant

Description
enhancement #447
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Chore (minor updates related to the tooling or maintenance of the repository, does not impact compiled assets)

Motivation and context
How to test?
Story at /docs/layout-box--docs been updated to include new property headerVariant
Change dropdown to a different H tag and inspect element to see component now renders with the H tag selected rather than then always h5
Screenshots (if appropriate)
Checklist

[x] If my change requires a change to the documentation, I have updated the documentation in this pull request.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.

Hi @Digipex thanks for your contribution, well done.
And I'm happy to see that this does not enable the developer to visually pick different headline sizes, but just enabling to correct the tag for Accessibility concerns.
I've added a comment to emphasize this, so no one in the future thinks that's a mistake.
I also did a few minor corrections to the code, to make it work properly with ESLint.
I'm unsure when those commits will be picked up by GitHub. But when they are in place I will approve and merge :-)

Hi @nielslyngsoe
Thanks for the feedback.
Yeah was struggling to figure out the ESLint issue so be good to see what you did.


Add item label to builder

This PR adds an item label to the Builder. Its basically just a copy of the functionality already existing on the repeater and is used in the same way:
Forms\Components\Builder::make("xxx")
    ->itemLabel(fn (array $state): ?string => $state['title'] ?? null)

This is very usefull, when sorting long lists, by collapsing:

I think ->itemLabel() should be on the Block level instead of the Builder level, because then you can have a different label pattern for each block which is good for different data structures.

Are you able to implement the requested changes, @bernhardh? 🙂

Great, @bernhardh, thanks!

@zepfietje
I think I have changed it as suggested. You can now set the itemLabel on Block  level instead of globally for the whole Builder.
One minor thing I changed compared to the behavior of Repeater: You can also use/retour HtmlString's:
->itemLabel(fn (array $state): HtmlString => new HtmlString(
    $state['title'] ?? "TITLE MISSING!!!"
))

Closed in favor of #6321


Require a git pull before a terraform apply

Is there a way to add a hook into Terraform that requires the current branch be up-to-date before running terraform apply? If so, should we add that? It doesn't stop emergency applys - just git commit -m "temp" and undo it after, which doesn't make the branch out-of-date locally - but it does prevent situations liking accidentally taking down March Madness for a few minutes...

Related issue: https://github.com/hashicorp/terraform/issues/32930
Since we have the Nix shell at our disposal, it would be really easy for us to completely hide terraform from the user and enforce deploying via our shell scripts instead.

Related issue

That would definitely be a great help...

Since we have the Nix shell at our disposal, it would be really easy for us to completely hide terraform from the user and enforce deploying via our shell scripts instead.

Which scripts?

Which scripts?

We'll just make them:
writeShellScriptBin "deploy" ''
  # source the scripts/lib thing
  if [[ $(git branch --show-current) != main ]]; then
    fatal "You're not in the main branch. Deployments are allowed from that branch only."
  fi

  originHEAD=$(git rev-parse origin/main)
  behind=$(git rev-list --count HEAD..${originHEAD})
  if (( behind > 0 )); then
    log "Your local repository is behind upstream by $behind commits."
    log 'Ensure you ran `git pull` before proceeding.'
    exit 1
  fi

  # Run a Nix-imported Terraform here.
  # This way, we don't need Terraform in our $PATH.
  ${lib.getBin pkgs.terraform} apply "$@"
''


Address Get-PSSession Remote Session not available

If the script is run on a EMS session that has expired, you will see a similar error
Get-PSSession : Remote session is not available for c1b4025a-2d0b-41b3-9f84-2fcc3fcb026d.
At
C:\Users\user.domain\AppData\Roaming\Microsoft\Exchange\RemotePowerShell\:117
char:14
+             (& $script:GetPSSession `
+              ~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidArgument: (c1b4025a-2d0b-41b3-9f84-2fcc3fcb026d:Guid) [Get-PSSession], ArgumentException
    + FullyQualifiedErrorId : RemoteRunspaceNotAvailableForSpecifiedRunspaceId,Microsoft.PowerShell.Commands.GetPSSessionCommand

Should address this by making the script include this error in the handled errors.

I was able to repro this behavior:

Run HealthChecker.ps1
Run Get-PSSession | Remove-PSSession
Run HealthChecker.ps1 again and you will get error

done in commit bd0f12471a3f64588147f6930e8ad4f1c307286a


[BEAM-7109] Do not reconnect logging at termination

Indefinite attempt to reconnect logging client causes SDKHarness to get stuck during termination and eventually cause thread leak as each attempt to reconnect create some threads on GRPC side.
The pr deletes the logging stub before reconnecting so that grpc can clean its state.
The PR also introduces _alive flag which is used to determine if logging client should reconnect on disconnect.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

R: @robertwb @aaltay

cc: @yifanzou

Run Portable_Python PreCommit

Thanks for fixing this, Ankur!


Widget screenshots in documentation

Closes #1027
This PR ensures each widget has an associated screenshot, displayed in a gallery (in modules/gui/index.rst) and in the class docstring.
plot and plot3d widgets have additional galleries at the module documentation level, so for the moment I did not add the screenshots to each class docstring.
To be discussed:

should we also add a gallery of screenshots of higher level applications using silx widgets?
is there a way to avoid code duplication for the plot and plot3d galleries? I could not find a way to use .. include::  because the relative paths of the images are not the same in modules/gui and modules/gui/plot

I think it would be better to avoid to break the overview of the API page.
Then i think it would be better to

Remove

+Sub-packages and modules
+------------------------

Put the gallery as a main entry of the doc
Remove the include of the gallery from modules/gui/index, optionally we can create a link to the gallery, there is no problem.

What do you think?

LGTM


MoveAxesAsync Generating Incorrect G-Code for Z-Axis Moves

Looking at the code, it looks like the MoveAxesAsync method is incorrectly generating the gcode for Z-axis moves. Instead of using Z in the gcode, Y has been inserted. You can view the problem here:
https://github.com/AndreasReitberger/KlipperRestApiSharp/blob/b3e5604758cc137fbb790aed3a34a0d78c479af9/source/MoonrakerSharpWebApi/MoonrakerSharpWebApi/MoonrakerClient.cs#L5660

Hi @jerretDavis
Thank you for the hint and the PR to fix it 😊


./CharacterAI_Discord_Bot doesn't work

When starting in Linux with ./ it says there is no file with the name CharacterAI_Discord_Bot and when using bash , It says this root@localhost:~# bash CharacterAI_Discord_Bot CharacterAI_Discord_Bot: CharacterAI_Discord_Bot: cannot execute binary file ....
Someone else posted this issue but you didn't answer correctly.. #41 I really needed this bot.. pls fix it..
Thanks in advance....
@drizzle-mizzle

Still not responding....... Pls fix it screen shot down here @drizzle-mizzle

The CharacterAI_Discord_Bot file does exist but when I run it , it says this

You are in the wrong directory
https://www.google.com/search?q=linux+cd&oq=linux+cd
Why in the world you use Linux if can't even manage to change directory? Sorry, but I'm not a Google and I'm not going to teach you how to use your OS.

And I'm 100% not going to deal with Termux and problems you may encounter on it.


Review shutdown overrides

run all CS phases.
enable JVM shutdown hooks.
Related to https://github.com/lagom/lagom/issues/1379

mid-air collision. https://github.com/lagom/lagom/pull/1453/commits/69e920834b93772e14c0f0ee7bb58905a0e6eff6
This PR is no longer necessary.