Add Option to disable encrypting/signing by default

Currently there is no way to disable encryption other than doing that for each E-Mail that I send separately.

Technically a duplicate of #1743 but that one was closed for veering wildly off-topic.

Hi,
the switch from AGP to OpenKeychain just bit me too. K9 now won't encrypt emails: this is the error message: "No keys configured for signing, please check your settings".
But I don't want to GPG-sign my emails, I only want to send GPG encrypted emails. K9+OpenKeychain now (seem) to require the keypair to be on the Android phone. I'm not sure I want my secret key on a phone.
With AGP this was not a problem. You could simply encrypt it for the recipient, and send it off.
This is with OpenKeychain 4.2 and K9 4.3

@dingesista This is not an issue with OpenKeychain vs APG. It’s a design decision in K-9. @Valodim is supposed to write a blog post about it some time soon.

The use case people seem to have seems to be "I don't want to have my signing key on my phone". I wonder if there is a way to handle this situation with a mobile-specific signing subkey.
Sending messages without signing seems like a fine tradeoff on the sender side because it's so super convenient and doesn't have any apparent disadvantages, but it's a horrible cryptographic choice that has dire consequences for the pgp ecosystem.

I have my key on an NFC token but I don't want to use it for every mail.
I sign/encrypt only very important Mails.
And currently I have gpg completely disabled in k9 due to the described issue.

@Valodim Specific signing key is not suitable for all use case. For instance, I don’t trust my phone with it, and I would rather prevent any issue in that direction by not having subkeys at all on my phones than rely on the fact I can revoke this specific subkey. But I, for one, would definitively live without GPG support in K-9. The current state is almost fine for me excepted for the fact I have to move the slider on every mail I send (because I selected OpenKeychain but no key).
Also, just by curiosity, can hardware crypto modules issue subkeys for such use or is the better thing support for those modules in OpenKeychain anyway?

@Valodim et al, thanks for your responses.  I can only hope that there will be an option soon to encrypt without signing. I've had to stop sending emails from K9. I can elaborate on my use case: I am (was) used to share simple emails mostly with URLS to myself and a handful of contacts - using my and their public key to encrypt these emails for (only) the recipient. I don’t know how this has, as Vincent suggests has dire cryptographic consequences - I’m seriously interested to reading your explanation. My aim is to encrypt each and every email to contacts with PGP/GPG keys.

It feels like we're getting off topic again. In my opinion this issue should be about a configurable default value for the encryption slider.

Maybe its better when "Encrypt if possible" decides not to encrypt the mail when there is no signing key.?

I simply would like a system like that one used by APG: IF and only IF I want to sign the email will be signed and encrypted. Unfortunately now k9 wants to encrypt every mail I send to a certain person whose public key is in my storage. That's unacceptable.

At the moment, when I write a message to someone whos public key is in my keystore, k-9 always tries to encrypt. But I only sometimes want to encrypt mails, not as default.
There is no other way I see to disable this, as remove the key(s), disable the Cryptography by choosing "None" in the OpenPGP app or to freeze/uninstall OpenKeychain. Neither of this is comfortable. At the moment I disabled Cryptography ("none").
Did I miss something in the settings and this is not the design of it? Seems not optimal.
Thanks.

You perfectly described what happens to me. In fact, when I use mobile
terminals I use encryption but since I cannot install decrypting software
on the pc I use in  my workplace, emails I receive in this case, to be
visible, must be in the clear. The app, so far, forces decisions everytime
while in a different configuration it should suffice to select if and only
if someone wants encryption.
Hoping they will solve the issue.

Maybe somebody knows where exactly in the code the default setting is set, so maybe one could compile a version with default unencrypted?

I thought about it and I think if the last setting could just be saved this would be the best.
So if someone often needs it, he wouldnt need to change the switch. At the moment there is the default and no saved setting.

Ouff. I found it.
The hardcoded setting is in the RecipientPresenter.java in Line 81:
private CryptoMode currentCryptoMode = CryptoMode.OPPORTUNISTIC;
this is the path:
k9mail/k-9/k9mail/src/main/java/com/fsck/k9/activity/compose/RecipientPresenter.java
OPPORTUNISTIC means to encrypt if a key is available. I set this to:
private CryptoMode currentCryptoMode = CryptoMode.DISABLE;
and any new mail with a recipient where I have a key is set to not encrypt by default.
I can set it then manually to encrypt or encrypt and sign (called: PRIVATE / CryptoMode.PRIVATE)
But best would be, I think now, to use the last setting, because often I mail many times to one person and it is then normally each time encrypted or none. But maybe there is a better way.
I for myself now compile a version without the default encryption.

That's great!!
When will we see this is in a release?

I assume this change has made it into the release, because that's how my K-9 behaves recently. I have to think of activating encryption every time. I am not happy about it. I preferred the former default behaviour.
The title of this post was "add option to ..." (which is a good idea), not "change default", (which is what was implemented in https://github.com/k9mail/k-9/commit/5bc61258d1c864297a7e5bfe625cfa81f635bafc).

you can enable mutual mode, which will cause k-9 to encrypt to other people who have enabled this setting. please see here for a rationale

Aha, thanks for the answer. I don't really get Autocrypt yet and 5.5 is not yet on FDroid, but I see this change has not been a spontaneous action by a single user. Somebody seems to have thought a while about this.


Support quick normalization checks

This would be great.  I'd have a use for this in unicode-collation; if we could quickly determine that the text is already normalized and avoid the expensive normalization step (which accounts for about 1/3 of my run time according to profiling and benchmarks), that would help us achieve parity with text-icu.

We can do that, its just a matter of putting some time on it. In the meanwhile we can try to figure out how fast unicode-collation can go with the current impl itself. I guess there will be some avenues to optimize irrespective of this.

If you have any suggestions for optimizing, I'd be glad to hear them.


Code Parsedown unexcpected behaviour

When writing code in my Markdown, it gets escaped. Even if I try to disable SafeMode or StrictMode it won't change anything.
I tried looking a bit through the code but I don't see why it happens after about one hour, so I feel like it should be explained somewhere how to show code..
Can anyone help?
Dear regards

Hey I just wanted to explain the behaviour and found out this is a false alert, everything is working fine and I had a bug in my code.
Thank you a lot for trying to help though!


Move related files to optimizes module

Mars optimizes including three parts:

optimize on coarse-grained graph.
optimize on fine-grained graph.
runtime optimizations.

We can move all related code to optimizes module, for example, fuse.pyx is for fine-grained graph optimization which should be moved to chunk_graph(the name need further discussion) directory under the optimizes. Directory structure will be like this:
|-- optimizes
    |-- tileable_graph
        |-- ...
    |-- chunk_graph
        |-- fuse.pyx
        |-- ...
    |-- runtime
        |-- core.py
        |-- numexpr.py
        |-- jax.py
        |-- ...

It will be more clear for optimize module instead of separating them into other modules.

FYI @wdkwyf

Closed by #640.


Merge featureState from multiple sources

With the upcoming #65, we'll run into issues if multiple places are trying to setFeatureState on the same feature with different pieces of state. This could happen when JoinedData and a Layer with manageHoverState are both used.
The solution is probably to make a separate piece of code which can merge the hover state with other state. Then everything that currently calls $map.setFeatureState could call that code instead. While this could be made generic, it's probably simplest to just make it work with a hover value and one other optional state object.

Per @stuartlynn in #65, Maplibre already handles this internally.


[HUDI-6523] fix get valid checkpoint for current writer

Change Logs
filter commit action and deltacommit action when get valid checkpoint for current writer
Impact
NONE
Risk level (write none, low medium or high below)
NONE
Documentation Update
NONE
Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

@hudi-bot run azure

whats the bug here. can you please write a Unit test

@nsivabalan Thank you for your reply.
Every batch, we get checkpoint message from timeline firstly, if the latest commit action is compaction or clustering, HoodieCommitMetadata.fromBytes(timeline.getInstantDetails(instant).get(), HoodieCommitMetadata.class) will get an error:
Trace is here:
(https://issues.apache.org/jira/browse/HUDI-6523)
In my opine, we need filter the compaction and clustering.
Do you have any good suggestions?

@nsivabalan we parse every instants by HoodieCommitMetadata.fromBytes(InstantBytes[], HoodieCommitMetadata.class).
In normal circumstances，HoodieCommitMetadata.fromBytes() can parse the instants like commit and deltacommit correctly but in some corner case, if the latest instant's action which got from active timeline was compaction action,HoodieCommitMetadata.fromBytes() will get an error.
actually, we don't need compaction or clustering instants when get checkpoint message in extra metadata.
Does this correct?
so, I just get the commit and deltacommit to fix this issue.


Qwen 2 model broken

To reproduce:
cargo run --release -- --port 2000 --hf-token TOK --model-id Qwen/Qwen2-1.5B qwen2 --repeat-last-n 64

And send a curl request, for example:
curl -X POST "http://<IP_ADDRESS>:2000/v1/chat/completions"      -H "Content-Type: application/json"      -H "Authorization: Bearer YOUR_API_KEY"      -d '{
           "model": "qwen2",
           "messages": [
               {"role": "user", "content": "Explain how to best learn Rust."}
           ],
           "temperature": 0.7,
          "max_tokens": 128,
          "stop": {"Single":"</s>"}
       }'

The Phi 3 model works, however, and the Qwen 2 does not. I think this may be because the num_attention_heads != num_key_value_heads. In Phi 3 they are the same, but in Qwen 2 they are 12 and 2 respectively. Gentle ping to @guoqingbao, could you please take a look?
Error:
cannot broadcast [1, 2, 128, 13] to [1, 12, 128, 13]

To reproduce:
cargo run --release -- --port 2000 --hf-token TOK --model-id Qwen/Qwen2-1.5B qwen2 --repeat-last-n 64

And send a curl request, for example:
curl -X POST "http://<IP_ADDRESS>:2000/v1/chat/completions"      -H "Content-Type: application/json"      -H "Authorization: Bearer YOUR_API_KEY"      -d '{
           "model": "qwen2",
           "messages": [
               {"role": "user", "content": "Explain how to best learn Rust."}
           ],
           "temperature": 0.7,
          "max_tokens": 128,
          "stop": {"Single":"</s>"}
       }'

The Phi 3 model works, however, and the Qwen 2 does not. I think this may be because the num_attention_heads != num_key_value_heads. In Phi 3 they are the same, but in Qwen 2 they are 12 and 2 respectively. Gentle ping to @guoqingbao, could you please take a look?
Error: cannot broadcast [1, 2, 128, 13] to [1, 12, 128, 13]

My fault! I thought the candle tensor broadcast method can handle broadcast with [x,a,x,x...] to [x,b,x,x...] where a>1. Apparently, it requires stacking in this case. I will fix this later.

To reproduce:
cargo run --release -- --port 2000 --hf-token TOK --model-id Qwen/Qwen2-1.5B qwen2 --repeat-last-n 64

And send a curl request, for example:
curl -X POST "http://<IP_ADDRESS>:2000/v1/chat/completions"      -H "Content-Type: application/json"      -H "Authorization: Bearer YOUR_API_KEY"      -d '{
           "model": "qwen2",
           "messages": [
               {"role": "user", "content": "Explain how to best learn Rust."}
           ],
           "temperature": 0.7,
          "max_tokens": 128,
          "stop": {"Single":"</s>"}
       }'

The Phi 3 model works, however, and the Qwen 2 does not. I think this may be because the num_attention_heads != num_key_value_heads. In Phi 3 they are the same, but in Qwen 2 they are 12 and 2 respectively. Gentle ping to @guoqingbao, could you please take a look?
Error: cannot broadcast [1, 2, 128, 13] to [1, 12, 128, 13]

Fixed in #52

Close as the issue fixed in the latest update.


Feature base structure and constructs

Issue #, if available:
Description of changes:
Upgraded AWS SDK version from v2 to V3
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

changes tested and deployed


Not able too generate script to fill text boxes.

Tested this in Chrome as well as in Chromium. I'm not able to see any input script being generated for the keydown events recorded.

Hey @shobhitchittora you need to hit the TAB key to "commit" the key downs.
https://github.com/checkly/puppeteer-recorder#usage
There is no other way to register the begin / end of text input for now.


some problems on data augmentation

Hi:
Thanks for your great job. While I reading the code,I found that data augmentations used in the code were listed as follows:
1.Horizontally flip 50% of images
2.Vertically flip 50% of images
3.Translate 50% of images by a value between -5 and +5 percent of original size on x- and y-axis independently
4.Scale 50% of images by a factor between 95 and 105 percent of original size
But in the original FlowNet ,the kinds of augmentation were more than above and a little different.
1.Translate all of images by a value between -20 and +20 percent of original size on x- and y-axis independently
2.Scale all of images by a factor between 90 and 200 percent of original size
3.No horizontal flip and vertically flip is used
4.Add the Gaussian noise that has a sigma uniformly sampled from [0, 0.04]
5.Add the contrast sampled within [−0.8, 0.4]
6.Multiplicative color changes to the RGB channels per image from [0.5, 2]
7.gamma values from [0.7, 1.5] and additive brightness changes using Gaussian with a sigma of 0.2.
Will the network trained using the above methods be different?
Expecting your reply. Thanks in advance!

Hi @guuusss , I am also inspecting data augmentation of PWC-Net. Could you provide the paper reference?  I think that your point 3 might be mistaken. As far as I know in "Model matters..." https://arxiv.org/pdf/1809.05571.pdf they do horizontal flipping (no vertical) when fine tuining on sintel

PWC-Net uses caffe to do data augmentation as Flownet do. So, i refer to the Flownet paper https://lmb.informatik.uni-freiburg.de/Publications/2015/DFIB15/flownet.pdf.

Flownet doesn't use horizontal flip and vertically flip.
To inspect data augmentation I guess you might need this paper https://lmb.informatik.uni-freiburg.de/Publications/2018/MIFDB18/paper-1801.06397.pdf .
This paper shows that different data augmentations have different effects to the model
.

Thanks, you are right, they all use the same augmentation. But PWC-Net ROB, when fine tuning on sintel also performs horizontal frame flipping, It is written and I also have tested it empirically

Hey @ZhongkaiZhou and @jeffbaena ,
Thanks for your clarification about the augmentation details about PWCNet and FlowNet 2.0.
I wonder if you have successfully reproduced the results on mpi-sintel test set?
Recently, I am trying to reproduce the PWCNet myself. The training AEPE and validation AEPE looks great when fine-tuning sintel, however, it performed really bad on the test set.
The situation is really similar to the issue here https://github.com/lmb-freiburg/flownet2/issues/191 opened by @ZhongkaiZhou.
I am not sure if the bad result is because that, I didn't apply the whole augmentation as described in the paper. (i.e., I only use scale/flip/translate without incremental changes on second frames).

Sorry, i have not reproduced the results on MPI-Sintel test set successfully. I think MPI-Sintel train and MPI-Sintel test are very different, so low epe on train may not lead to the low epe on test. All we can do is avoid overfitting and the whole augmentation should heip it.

@ZhongkaiZhou ,
Thanks for your quick and kind reply. Your reference to that project is really helpful for me, as there are few projects implementing the flow augmentation based on python (most of them use C or Caffe).
I really appreciate your help. If I have progress someday, I'll let you know!


Xcode 13 Beta 5 missing

Today's Xcode 13 Beta 5 is missing (after running xcodes update).
Is this a bug? Or should I be more patient? (because you need to manually add releases?)

Oh: https://twitter.com/XcodeReleases/status/1425162386420363269

It is already available in the list:
13.0 Beta (13A5154h)
13.0 Beta 2 (13A5155e)
13.0 Beta 3 (13A5192i)
13.0 Beta 3 (13A5192j)
13.0 Beta 4 (13A5201i)
13.0 Beta 5 (13A5212g) (Installed, Selected)

Works fine now. Thanks!


#include Error

https://github.com/Genymobile/scrcpy/blob/c15df01171793dca7074a012f4703a185d99169d/app/src/events.h#L8
Cannot find SDL_events.h
Change SDL_events.h To SDL2/SDL_events.h

Thank you, rebased and merged on dev: 89c4f455b0f9a4ada6a6552ddc6684b0344115cd
I don't know why it works for me (i.e. why the cc commands executed include -I/usr/include/SDL2).
What is your environment? and meson and c compiler versions?


Illegal Memory access error

Hi, sometimes I get an error if someone else is also using the same GPU. Can you check if the following code runs fine for you?
import torch
import torch_blocksparse
Z, H, M, N, K = 100, 2, 1, 2048, 512
a = torch.rand((Z, H, M, K), dtype=torch.float32).cuda()
b = torch.rand((H, K, N), dtype=torch.float32).cuda()
block = 16
layout = torch.ones(H, K//block, N//block).long()
dot = torch_blocksparse.MatMul(layout, block, 'dds', trans_a=False, trans_b=False)
c = dot(a, b)
print(c.shape)

That's pretty interesting. At this moment I have no clue why this would be the case, but I'll look into that some more when I get some time :)

I fixed some undefined behavior in the way Triton handled work from CUDA contexts created by pytorch (https://github.com/ptillet/triton/commit/7710e048f457c46afb1d222afdb5eac35a51e035). I have no way to test whether or not it solved the problem, but I feel like it should have resolved the issue. Feel free to re-open it if it persists.


Rails can't find assets on Production env

Do i need to do something to make this work on production or staging?
Now i get this error:

nevermind, i solved it.. sorry for spaming :sweat_smile:

Please do tell us how you solved it. Maybe we need a README update?

We'll the error i was getting was because i was deploying to a staging env, but i didn't have a staging.rb on config/environments, so i gues it was defaulting to development.rb.
Other error i found is that you need to add rack-cors middleware and configure it to make swagger work with your api.


BUG: fix KDTree for input with many identical values

I found bug when running kdtree on my data. And I fixed it. I think so...
It was segfault when it's a lot of objects with few coordinate values

@mirzman can you provide a self-contained example that reproduces the segfault?

points = [(1.0,)] * 100000 + [(2.0,)] * 100000
tree = KDTree(points)

(it's from tests/test_kdtree.py)

Hmm, this somehow looks familiar. I seem to remember a fix for this, however I can reproduce the issue with latest master. It works fine for cKDTree, maybe it was only fixed there.

so... is it okay?

It's the same bug with cKDTree, but it doesn't fall
--- a/scipy/spatial/ckdtree/src/build.cxx
+++ b/scipy/spatial/ckdtree/src/build.cxx
@@ -3,6 +3,7 @@
#include <cmath>
#include <cstdlib>
+#include <cstdio>
#include <cstring>
#include
@@ -171,6 +172,8 @@ build(ckdtree *self, npy_intp start_idx, npy_intp end_idx,
q = end_idx - 2;
}
+        printf("%d %d %lf %lf %lf\n", p-start_idx, end_idx-p, minval, maxval, split);
+
if (NPY_LIKELY(_compact)) {
_less = build(self, start_idx, p, maxes, mins, _median, _compact);
_greater = build(self, p, end_idx, maxes, mins, _median, _compact);
I'm constructing cKDTree on my data and it causes:
...
9 14 54.204904 54.218963 54.213603
1 70 0.138629 0.179176 0.138629
1 69 0.138629 0.179176 0.138629
1 68 0.138629 0.179176 0.138629
1 67 0.138629 0.179176 0.138629
1 66 0.138629 0.179176 0.138629
1 65 0.138629 0.179176 0.138629
1 64 0.138629 0.179176 0.138629
1 63 0.138629 0.179176 0.138629
1 62 0.138629 0.179176 0.138629
1 61 0.138629 0.179176 0.138629
1 60 0.138629 0.179176 0.138629
1 59 0.138629 0.179176 0.138629
1 58 0.138629 0.179176 0.138629
1 57 0.138629 0.179176 0.138629
1 56 0.138629 0.179176 0.138629
1 55 0.138629 0.179176 0.138629
1 54 0.138629 0.179176 0.138629
1 53 0.138629 0.179176 0.138629
1 52 0.138629 0.179176 0.138629
1 51 0.138629 0.179176 0.138629
1 50 0.138629 0.179176 0.138629
1 49 0.138629 0.179176 0.138629
1 48 0.138629 0.179176 0.138629
1 47 0.138629 0.179176 0.138629
1 46 0.138629 0.179176 0.138629
1 45 0.138629 0.179176 0.138629
1 44 0.138629 0.179176 0.138629
...
it's not okay

@sturlamolden did you delete your own comment, or did I do that by accident? (it's still in my email)

@sturlamolden did you delete your own comment, or did I do that by accident? (it's still in my email)

I deleted it. I decided I wanted to do some experiments first.

I deleted it. I decided I wanted to do some experiments first.

ok thanks. then will delete all these comments as well:)

ok thanks. then will delete all these comments as well:)

I wonder why the input hasn't triggered
https://github.com/scipy/scipy/blob/master/scipy/spatial/ckdtree/src/build.cxx#L101
The tree should have been built with compact=True by default.

I will try to push a fix for cKDTree into PR #5647

I wonder why the input hasn't triggered
https://github.com/scipy/scipy/blob/master/scipy/spatial/ckdtree/src/build.cxx#L101
The tree should have been built with compact=True by default.

Yeah, that is why I deleted my comment. KDTree also has a similar check for ties, but obviously it is not invoked. This needs some investigation I think.

@mirzman I cannot reproduce your output on PR #5647 which does contain the same build.cxx as the lastest master. The commit that contains the test case is https://github.com/rainwoodman/scipy/commit/ca6ae588721ece795eb21e72c512617f162f67d2
I am guessing your version of cKDTree must have set compact_nodes=False by default.

That is an excellent test case :)

Why does it matter if we use compact_nodes=True?

I think the bug for compact_nodes=False might be on line 535 and 536 in ckdtree.pyx in PR #5647
Or am I missing something?

The example in the test passes for me now on main, so looks like this was fixed elsewhere at some point in the last 8 years... closing! Thanks for the initial report @mirzman


IPRoyal Coupon not valid

Is there an existing issue for this?

[X] I have searched the existing issues

I'm submitting a ...

[X] bug report
[ ] feature request
[ ] support request --> Contact me over mail for support https://github.com/MShawon

Description
We like the IPRoyal, but the Coupon from you is not more valid. Do you have a new one?
Environment
- OS :
- Python :
- Script version :

config.json
none

See my comment here https://github.com/MShawon/YouTube-Viewer/issues/340#issuecomment-1146694353


Improve parameter acceptance and validation in prepareDatahandleForIntegration

Summary
There are a few improvements in the interface of prepareDatahandleForIntegration which we would like to implement.

Rename the parameter that accepts a numerical integrator from solver to integrator.

[!CAUTION]
Don't forget to also adjust the documentation, in particular the readme and github-pages website! There may also be other pieces of documentation aside from the ones mentioned here, so try to be as thorough as possible to ensure consistency across the board.

Make the integrator parameter accept both function handles and char arrays as inputs.
Add user warnings for unrecognized or unused parameters.

Proposed solutions

Check if integrator parameter is a function handle using isa(x, 'function_handle' ). If so, then convert it to a char array using func2str(x).
Use optionlists for parameter validation. This functionality is bundled with IFDIFF and can be found in the Tools folder of the project.

We do not use MATLABs native argument validation because this feature was only introduced recently (2019b) and we want to stay backwards compatible (to a reasonable degree, so stick to around MATLAB2014 if possible).

Implemented in #20


Cartesian coe

This PR implements a hacked version of coe:

No diagonal cofibration yet!
Hacked impl of coe r r A a = a, see BetaExpander
No longer need eoc, coeFill!


[BEAM-13575] Sickbay test that is flaky to restore precommit test signal

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.
[ ] 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.

GitHub Actions Tests Status (on master branch)

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

R: @ibzib
Will merge early to restore precommit test signal.


Suggestion: Provide a policy for request ids.

Given that we expect clients to provide a request id, it probably makes sense to have a policy for it.

Fixed


[firebase_dynamic_link] Update README.md with up-to-date info

Description

Updated video link to non-deprecated info video.
Include instructions for using custom domains (as found in https://firebase.google.com/docs/dynamic-links/custom-domains)

Related Issues
Relates to #262
Although this issue was closed, the info was never added to the README
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] If the pull request affects only one plugin, the PR title starts with the name of the plugin in brackets (e.g. [cloud_firestore])
[x] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide). N/A
[x] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (flutter analyze) does not report any problems on my PR.
[x] I read and followed the Flutter Style Guide.
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy. N/A
[x] I updated CHANGELOG.md to add a description of the change. N/A
[x] I signed the CLA.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[x] No, this is not a breaking change.

iv wasted about 2 hours trying to figure this issue out , they really should update the read me. SAD!!


docs: add root files

Successes

Links on main page work, but no link to blog page.

Failures

No change.
Also, minima is not being used.
Times Roman instead of sans serif font.
No borders on tables.


msvc windows xp toolset(v141_xp) open include file: 'debugapi.h': No such file or directory

so lib failed to build for winxp, even when BOOST_URL_REPORT is not defined and ::DebugBreak not called from anywhere.

This needs a pull request would you like to submit it?

sure https://github.com/boostorg/url/pull/675


Whatsapp asking me to update my browser - and then downloading a deb package

Bug description:
When trying to use the Whatsapp Web service it says that it needs Chrome 49+, or I can use firefox, safari, edge, or opera.
I have double checked and both my Chrome and Firefox are the most up-to-date, and required version needed as the Whatsapp service suggests.
Steps to reproduce:
Open ferdi, start whatsapp service

Open Ferdi
Go to Whatsapp Service
Click Update Google Chrome (or "use Firefox")
Download a Deb package (can't use, as on Arch)error

Expected behavior
Recognise I have most up to date versions of both firefox and chrome
Screenshots

Environment:

Operating System: Manjaro Linux 56

Ferdi Version: Ferdi-git from AUR

Server: Ferdi

Debug information:

{"host":{"platform":"linux","release":"5.6.19-2-MANJARO","screens":[{"id":1658020820910526,"bounds":{"x":0,"y":0,"width":1920,"height":1080},"workArea":{"x":0,"y":0,"width":1920,"height":1080},"accelerometerSupport":"unknown","monochrome":false,"colorDepth":24,"colorSpace":"{primaries:INVALID, transfer:INVALID, matrix:INVALID, range:INVALID}","depthPerComponent":8,"size":{"width":1920,"height":1080},"workAreaSize":{"width":1920,"height":1080},"scaleFactor":1,"rotation":0,"internal":false,"touchSupport":"unknown"},{"id":4693199496013570,"bounds":{"x":1101,"y":1080,"width":1680,"height":1050},"workArea":{"x":1101,"y":1080,"width":1680,"height":1050},"accelerometerSupport":"unknown","monochrome":false,"colorDepth":24,"colorSpace":"{primaries:INVALID, transfer:INVALID, matrix:INVALID, range:INVALID}","depthPerComponent":8,"size":{"width":1680,"height":1050},"workAreaSize":{"width":1680,"height":1050},"scaleFactor":1,"rotation":0,"internal":false,"touchSupport":"unknown"},{"id":1658020820910339,"bounds":{"x":1920,"y":0,"width":1920,"height":1080},"workArea":{"x":1920,"y":0,"width":1920,"height":1080},"accelerometerSupport":"unknown","monochrome":false,"colorDepth":24,"colorSpace":"{primaries:INVALID, transfer:INVALID, matrix:INVALID, range:INVALID}","depthPerComponent":8,"size":{"width":1920,"height":1080},"workAreaSize":{"width":1920,"height":1080},"scaleFactor":1,"rotation":0,"internal":false,"touchSupport":"unknown"}]},"ferdi":{"version":"5.6.0-beta.1","electron":"8.4.1","installedRecipes":[{"id":"discord","version":"1.1.3"},{"id":"github","version":"2.0.0"},{"id":"googlecalendar","version":"1.0.3"},{"id":"messenger","version":"1.1.0"},{"id":"office365-owa","version":"1.1.0"},{"id":"telegram","version":"3.0.0"},{"id":"whatsapp","version":"2.0.3"},{}],"devRecipes":[],"services":[{"id":"edeb16a7-a094-4e05-b0eb-114ba91b249b","recipe":"whatsapp","isAttached":true,"isActive":true,"isEnabled":true,"isHibernating":false,"hasCrashed":false,"isDarkModeEnabled":true},{"id":"67e1c687-adbf-410d-92a9-8a7de084ef8b","recipe":"messenger","isAttached":true,"isActive":false,"isEnabled":true,"isHibernating":false,"hasCrashed":false,"isDarkModeEnabled":true},{"id":"6ae104fd-3bc9-4292-8f0c-a422dd6b7fc5","recipe":"telegram","isAttached":true,"isActive":false,"isEnabled":true,"isHibernating":false,"hasCrashed":false,"isDarkModeEnabled":true},{"id":"bd40487f-fd50-4357-bfcf-956d7d1657c5","recipe":"discord","isAttached":true,"isActive":false,"isEnabled":true,"isHibernating":false,"hasCrashed":false,"isDarkModeEnabled":true},{"id":"4c8cc726-b802-436f-8b70-8f8e19171cc6","recipe":"googlecalendar","isAttached":true,"isActive":false,"isEnabled":true,"isHibernating":false,"hasCrashed":false,"isDarkModeEnabled":true},{"id":"d0cc0000-8269-44f5-83dd-8653e0e02edb","recipe":"github","isAttached":true,"isActive":false,"isEnabled":true,"isHibernating":false,"hasCrashed":false,"isDarkModeEnabled":true}],"messages":[{"type":"error","request":{"result":null,"wasExecuted":true,"method":"getInfo"},"error":{},"response":{},"server":"https://api.getferdi.com"},{"type":"error","request":{"result":null,"wasExecuted":true,"method":"features"},"error":{},"response":{},"server":"https://api.getferdi.com"},{"type":"error","request":{"result":null,"wasExecuted":true,"method":"all"},"error":{},"response":{},"server":"https://api.getferdi.com"},{"type":"error","info":["error","Reaction doesn't converge to a stable state after 100 iterations. Probably there is a cycle in the reactive function: Reaction[bound _build]"]},{"type":"log","info":["log","Grace @ Broadgate Brides V2",{"icon":"blob:https://web.telegram.org/0ca0edfb-0096-4917-b330-86ef04c92e8b","body":"But studios dont have an oven","tag":"s1320625797_7335840666434244209","silent":true}]},{"type":"log","info":["log","Grace @ Broadgate Brides V2",{"icon":"blob:https://web.telegram.org/c54c30ab-1175-4df3-8aa5-30d6baddbd4a","body":"Just a fancy microwave","tag":"s1320625797_7335840666434244209","silent":true}]},{"type":"log","info":["log","Grace @ Broadgate Brides V2",{"icon":"blob:https://web.telegram.org/09249efb-07a6-4fa4-ab07-a85a3212d201","body":":sob::sob:","tag":"s1320625797_7335840666434244209","silent":true}]},{"type":"log","info":["log","Grace @ Broadgate Brides V2",{"icon":"blob:https://web.telegram.org/5a09af62-6957-40cf-99df-eac790e066d4","body":"Oh fair","tag":"s1320625797_7335840666434244209","silent":true}]},{"type":"log","info":["log","Ciara:two_hearts: @ Broadgate Brides V2",{"icon":"blob:https://web.telegram.org/b969ea3a-a756-4ad9-8dd9-3ccabf50e748","body":"🖼 Sorry to interrupt guys but I got my braces off and I’m so gassed","tag":"s1320625797_7335840666434244209","silent":true}]},{"type":"log","info":["log","Ciara:two_hearts: @ Broadgate Brides V2",{"icon":"blob:https://web.telegram.org/524b48d4-c8a4-45ae-a404-e44d35f755b8","body":":two_hearts::two_hearts:","tag":"s1320625797_7335840666434244209","silent":true}]},{"type":"log","info":["log","Shini @ Broadgate Brides V2",{"icon":"blob:https://web.telegram.org/4491da65-b6b1-4d25-9158-35b57162dd19","body":"Your teeth are straighter than tej :scream:","tag":"s1320625797_7335840666434244209","silent":true}]},{"type":"error","request":{"result":null,"wasExecuted":true,"method":"featured"},"error":{},"response":{},"server":"https://api.getferdi.com"},{"type":"error","request":{"result":null,"wasExecuted":true,"method":"featured"},"error":{},"response":{},"server":"https://api.getferdi.com"},{"type":"log","info":["log","[ServerApi::getRecipePackage] Using internal recipe file"]},{"type":"log","info":["log","/usr/lib/ferdi/app.asar.unpacked/recipes/googlecalendar.tar.gz"]},{"type":"error","request":{"result":null,"wasExecuted":true,"method":"featured"},"error":{},"response":{},"server":"https://api.getferdi.com"},{"type":"error","request":{"result":null,"wasExecuted":true,"method":"featured"},"error":{},"response":{},"server":"https://api.getferdi.com"},{"type":"error","request":{"result":null,"wasExecuted":true,"method":"featured"},"error":{},"response":{},"server":"https://api.getferdi.com"},{"type":"log","info":["log","[ServerApi::getRecipePackage] Using internal recipe file"]},{"type":"log","info":["log","/usr/lib/ferdi/app.asar.unpacked/recipes/github.tar.gz"]},{"type":"error","request":{"result":null,"wasExecuted":true,"method":"featured"},"error":{},"response":{},"server":"https://api.getferdi.com"},{"type":"error","request":{"result":null,"wasExecuted":true,"method":"featured"},"error":{},"response":{},"server":"https://api.getferdi.com"},{"type":"log","info":["log","Ciara:two_hearts: @ Broadgate Brides V2",{"icon":"blob:https://web.telegram.org/275320be-bd7a-4e7c-b46c-9f7d96214f48","body":"Aha he’s gonna be jealous","tag":"s1320625797_7335840666434244209","silent":true}]}],"workspaces":[],"windowSettings":{"width":800,"height":600,"x":1527,"y":1167,"isMaximized":true,"isFullScreen":false,"displayBounds":{"x":1101,"y":1080,"width":1680,"height":1050}},"settings":{"autoLaunchInBackground":false,"runInBackground":true,"reloadAfterResume":true,"enableSystemTray":true,"startMinimized":false,"minimizeToSystemTray":"","privateNotifications":false,"showDisabledServices":true,"showMessageBadgeWhenMuted":true,"showDragArea":false,"enableSpellchecking":"","spellcheckerLanguage":"\"\\\"\\\\\\\"[\\\\\\\\\\\\\\\"en-US\\\\\\\\\\\\\\\"]\\\\\\\"\\\"\"","darkMode":true,"locale":"en-US","fallbackLocale":"en-US","beta":false,"isAppMuted":false,"enableGPUAcceleration":"","serviceLimit":5,"server":"https://api.getferdi.com","predefinedTodoServer":"https://app.franztodos.com","autohideMenuBar":false,"lockingFeatureEnabled":false,"locked":false,"lockedPassword":"******","useTouchIdToUnlock":true,"scheduledDNDEnabled":false,"scheduledDNDStart":"17:00","scheduledDNDEnd":"09:00","hibernate":false,"hibernateOnStartup":true,"hibernationStrategy":300,"inactivityLock":0,"automaticUpdates":true,"showServiceNavigationBar":false,"universalDarkMode":true,"adaptableDarkMode":true,"accentColor":"#7367f0","serviceRibbonWidth":68,"iconSize":20,"sentry":false,"navigationBarBehaviour":"never","todoServer":"isUsingCustomTodoService","customTodoServer":""},"features":{"isSpellcheckerIncludedInCurrentPlan":true,"needToWaitToProceed":false,"needToWaitToProceedConfig":{"delayOffset":3600000,"wait":10000},"isServiceProxyEnabled":true,"isServiceProxyIncludedInCurrentPlan":true,"isAnnouncementsEnabled":true,"isWorkspaceIncludedInCurrentPlan":true,"isWorkspaceEnabled":true,"isCommunityRecipesIncludedInCurrentPlan":true,"isSpellcheckerPremiumFeature":false,"isServiceProxyPremiumFeature":true,"isWorkspacePremiumFeature":false,"isSettingsWSEnabled":false,"isServiceLimitEnabled":false,"serviceLimitCount":0,"isCommunityRecipesPremiumFeature":false,"isCustomUrlIncludedInCurrentPlan":true,"isMagicBarEnabled":true,"isTeamManagementIncludedInCurrentPlan":true,"isTodosEnabled":true,"isTodosIncludedInCurrentPlan":true,"defaultTrialPlan":"franz-pro-yearly","subscribeURL":"https://getferdi.com","planSelectionURL":"https://getferdi.com","hasInlineCheckout":true,"isPlanSelectionEnabled":false,"isTrialStatusBarEnabled":false,"canSkipTrial":true,"pricingConfig":{"currency":"$","currencyID":"USD","plans":{"personal":{"monthly":{"id":"ferdi-free","price":0,"billed":0},"yearly":{"id":"ferdi-completely-free","price":0,"billed":0}},"pro":{"monthly":{"id":"ferdi-still-free","price":0,"billed":0},"yearly":{"id":"ferdi-forever-free","price":0,"billed":0}}}}},"user":"82c1cf9d-ab58-4da2-b55e-aaa41d2142d8"}}

any fix for this ?

any fix for this? can't the user agent version be set to something like 2000 and be done with it ?

Haven't found one yet, but haven't looked into changing the user agent version, how would I go about doing this?

I made a fix. https://github.com/getferdi/recipes/pull/275

I think this is now fixed. Can someone close this issue?

can @FerdiBot do it ?

@tofran @beppe9000 thanks for the heads up!

I'm facing the same issue. Just had installed Ferdi from AUR package: fer.
https://aur.archlinux.org/packages/ferdi/

My Linux:
Linux stallman 5.10.42-1-MANJARO #1 SMP PREEMPT Thu Jun 3 14:37:11 UTC 2021 x86_64 GNU/Linux
My chrome is up to date:

@ovictoraurelio - this is referring to v5.5 - which is QUITE old. Due to issues in our CI pipeline, we are not yet able to release v5.6 - please use the ferdi-git package which is much newer. Or else, you can download from our nightliies page - the appropriate package for your OS.


More than one SoapClient in Project

When more than one WSDL generated SOAP-Client is used in the same project, the static SoapClient on AbstractSoapClientBase will be overriden for the first instantiated with the last instantiated service.
This is a problem for me at the moment.
There are two possibilities around: Drop the "staticness" completely or make the access to the property protected and use the static (instead of self) keyword in setter and getter. This way derived packages can decide to create their own instances by overriding $soapClient
What is your opinion on that?
Ah this issue does not really belong here - but in the PackageBase - Repository. Sorry.

Closing it here and opening it on PackageBase


[SPARK-40165][BUILD] Update test plugins to latest versions

What changes were proposed in this pull request?
This PR test updates plugins to latest versions.
PS: The PR following https://github.com/apache/spark/pull/37598
Why are the changes needed?
This brings improvment & bug fixes like the following:

1.scalacheck (from 1.16.0 to 1.17.0)
https://github.com/typelevel/scalacheck/releases
https://github.com/typelevel/scalacheck/compare/v1.16.0...v1.17.0

2.maven-surefire-plugin (from 3.0.0-M5 to 3.0.0-M7)
https://github.com/apache/maven-surefire/releases
https://github.com/apache/maven-surefire/compare/surefire-3.0.0-M5...surefire-3.0.0-M7

3.maven-dependency-plugin (from 3.1.1 to 3.3.0)
https://github.com/apache/maven-dependency-plugin/compare/maven-dependency-plugin-3.1.1...maven-dependency-plugin-3.3.0
https://github.com/apache/maven-dependency-plugin/commit/9646b0ed76a6d00e468c8eb1b6a27d260b09e944

Does this PR introduce any user-facing change?
No.
How was this patch tested?
Pass GA and testing with the existing code.

the root cause for the failure: https://github.com/apache/spark/pull/37598
when upgrade maven-dependency-plugin from 3.1.1 to 3.3.0, The build path is missing dependencies: org.apache.httpcomponents:httpcore

Otherwise yeah seems OK to update plugins

Thanks for invistigation, in previous PR the old cache included the httpcore, so the bug was not triggered.
I also think adding httpcore deps explicitly is right.

BTW, in order to avoid similar problems in future, I think we should remove all "restore-keys: |" (I mean in a separate PRs). Make sure that when the pom changes, we can invalidate the cache in time, right?

If it would only invalidate the cache when the POM changes (or ideally, the deps file) then yeah that's fine. Otherwise it may slow the build down a lot to never have the cache

Thanks for invistigation, if your analysis is right (you can easy to validate it by disable all cache step), in previous PR the old cache included the httpcore, so the bug was not triggered.
I also think adding httpcore deps explicitly is right.

Yes,
the first commit of pr, i remove following code:
https://github.com/apache/spark/blob/014165ff6ffad87c2d6e970a14c1a13e15ba178f/.github/workflows/build_and_test.yml#L698-L704
and add following code:

Scala 2.13 build with SBT run ok.

@srowen Yes, remove restore-keys only means remove loose hit, but still keep strict hit (key match)
restore-keys - An ordered list of keys to use for restoring stale cache if no cache hit occurred for key.

Can one of the admins verify this patch?

Merged to master


Edit Mapfiles / Map Source File

Hi,
great work! we would like to integrate your API in a local Recreation site and add a map only for the region. How to Edit the map .json file. Its easy to add only the needed regions and towns, but what tool /source did you use to edit the viewbox and dimensions. I would love to have a map only for our region in fullsize.
I already copyied all needed areas to a new file and got the map, but it is small and has a lot of space around.
Many thanks,
Martin

Hi @zeug-it
In the district.json you can set the width, height and viewbox. You have to play around with the numbers to find the best values for your selection.

Yes, I did this. Worked great!

Awesome!


General updates v5

[x] Notifications modal text is way too small
[x] Think about boot profile option in the power dropdown on the Linode details page
[x] All Linode details rows should not extend outside of typical padded area
[x] Drop all notifications checkmarks
[x] Add account link to sidebar

Just going to show screenshot for notifications changes:

Modifying row margins can be seen in #119.
I'm letting the boot profile go for now and spinning out another task to try mocking up a power dropdown with profile options or something similar.

Maybe a little smaller text for the word Notifications, it doesn't need to be that big of a focus item.  It is only there as a reference to people if and when they need it, which will be almost never, beyond the first time seeing it.

It's keeping in line with the current usage of header fonts and whatnot. So it's currently "h2" size. Though I guess it could be "h3" size.

For the notifications themselves, try out having the padding the same all around, or a little bit more on bottom, instead of on top.  Traditionally groups of text have more white space below.

The padding is currently the same all around. Personally, I'm not a fan of asymmetric padding.


Consider bumping PYTHON_VERSION_FOR_BENCHMARK_BUILDS from 3.8 to 3.10 or 3.11

https://github.com/voltrondata-labs/arrow-benchmarks-ci/blob/e4de700016e0d473d82199733b8e4dbe89112b88/config.py#L50
PYTHON_VERSION_FOR_BENCHMARK_BUILDS = "3.8"

Changing this changes the reference/baseline. However, every now and then (like once per year or once per two years) this needs to change, what do we think? I suppose the CPython performance improvements that came with the 3.11 release are an interesting motivator.

FWIW, this would have prevented the benchmark failures the last few days.


Add sv-tests to smoke tests.

The very small syntax tests in sv-tests make it
particularly suited to find specific issues.

(submitting now already to skip timezone shift; might already be useful now)


add version_info to onConfigUpdate and /routes output

agreed, i like that better @htuch. will put together a PR.


style: update theme colors and backgrounds

Updated the app's color scheme with a softer background (slate-50 & stone-900) for light mode and adjusted the tab bar background to match the theme colors

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

#129  👈
#128
main

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

Join @owengretzinger and the rest of your teammates on  Graphite

Merge activity

Nov 20, 1:42 AM EST: A user started a stack merge that includes this pull request via Graphite.


[SPARK-45534][CORE] Use java.lang.ref.Cleaner instead of finalize for RemoteBlockPushResolver

What changes were proposed in this pull request?

use java.lang.ref.Cleaner instead of finalize() for RemoteBlockPushResolver
Why are the changes needed?

The finalize() method has been marked as deprecated since Java 9 and will be removed in the future, java.lang.ref.Cleaner is the more recommended solution.
Does this PR introduce any user-facing change?

No
How was this patch tested?

Pass GitHub Actions
Was this patch authored or co-authored using generative AI tooling?

No

Please help me review it. @LuciferYang

cc @mridulm FYI

The change is fine to me. Could you take another look?  @mridulm @beliefer Thanks ~

A few comments, mostly looks good.

All has been modified. Thank you very much for your comments.

Merged to master.
Thanks for working on this @zhaomin1423 !
Thanks for the reviews @LuciferYang, @beliefer :-)

Thank you for your review and providing valuable suggestions. @mridulm @LuciferYang @beliefer


[#100214054] Return a list of file keys from an s3 bucket.

Returns a list of all file keys in an S3 bucket ordered by date, optionally filtered by files whose names contain a prefix and/or don't contain a delimiter.
For each key, we return a dictionary containing its:

path
filename
extension
last_modified date
size

>> Story on Pivotal

:+1:


feat(oci): deduplicate layers prior to push; archive if needed

The following refactor-slash-feature brings the following:
(refactor)

Remove extra 'layer_count' method for determining whether to employ archive layers when publishing.  Instead, assemble layers assuming we won't need to use archive layers.  After deduplicating the layers vec, if the count exceeds the max, then we re-assemble using archive layers.  Since use of archive layers is the rarer case, it seemed fitting to only apply the tax of re-traversing the locked app here, instead of always doing so.
Add ClientOpts and content_ref_inline_max_size as first opt so that this value can be configured.  Open to alternative ways to doing the same thing (admittedly, this need came about from unit testing).

(feature)

Deduplicate assembled layers. For apps with multiple components and multiple static assets (think: static website), oftentimes the same component content or file content may be reused/included across multiple components.  Previously, when publishing the app (via OCI), we would potentially be publishing the same content multiple times, which is both inefficient in bandwidth and final published artifact footprint.  With the added deduplication, we ensure content is only published once and the OCI artifact is at its minimum size.

Depends on https://github.com/fermyon/oci-distribution/pull/1 (or alternative to achieve similar means)
TODO:

[ ] Merge https://github.com/fermyon/oci-distribution/pull/1 and update reference in Cargo.toml (or explore alternative)

@vdice I added a "do-not-merge" label to flag the dependency on upstream OCI


PyPI accepts packages with dependencies on local versions (e.g., 0.1.0+local)

PyPI accepts packages with dependencies on local versions (e.g., 0.1.0+local). I'm not sure if this is intentional or not, since PyPI will reject packages whose version is a local version.
I tested this was the case using this test package:
import setuptools
import os
import re

setuptools.setup(
    name="test-pypi-version-specifier-main",
    version="0.0.2",
    author="Edward Z. Yang",
<EMAIL_ADDRESS>    description="Testing package",
    long_description="Yarr",
    long_description_content_type="text/markdown",
    url="https://localhost/",
    packages=setuptools.find_packages(),
    include_package_data=True,
    classifiers=[
        "Programming Language :: Python :: 3",
        "License :: OSI Approved :: MIT License",
        "Operating System :: OS Independent",
    ],
    install_requires=[
        "test-pypi-version-specifier-dep==0.0.1+cuda9",
    ],
)

Thanks for the report!

Need to take a different approach here than #6294, see #6336.


FontTools is a community effort...

Thanks for mentioning my name with FontTools, and while I wrote a lot of the initial code, it's a true community effort these days, with major roles for @behdad, @anthrotype and @brawer. I didn't write a single line of code for varLib for example :)

@justvanrossum Sorry about that. I just changed it to a community project with contributions by @justvanrossum, @behdad, @anthrotype, @brawer, and many more.
Let me know if you'd prefer it worded differently.

That's great, thanks :)


Change setup configuration for pip

This is a proposed internal setup makeover.
Proposed Changes

Add a setup.cfg file for pip to better handle the requirements installation
Move most of setup.py content into setup.cfg
Remove the packaged six and docopt, add them as requirements during installation.
Update imports to reflect the above-mentioned change
Add "extra" modules installation (e.g. h5py and scipy can be installed with pip install bioread[hdf5/mat/all]

Things that may be interesting to discuss about

At the moment, most of the requirements don't have minimal version indication. It might be interesting to add them.
ordereddict is still packaged, mainly due to the fact that it becomes deprecated with python > 3.6, but then some refactoring is necessary to drop the dependency. It's still a possibility!

Basically, this is what changes:

Hmmmm... it's probably worth dropping ordereddict, and with it the entirety of python2 support. Which means we can also drop six.
Also! It looks like [we might really want a pyproject.toml file instead of a setup.cfg file[(https://snarky.ca/what-the-heck-is-pyproject-toml/). I could even be talked into the idea that the version number belongs both there and in _metadata.py though it makes my soul cry, just a little bit. Though this offers some hope — looks like maybe introspecting with importlib is a reasonable thing to do?
Anyhow, I'm gonna merge this for now, as it's better than what I'm doing now, and on the right track to being The Right Thing, I think.

Well, on a clean vagrant box, after installing I now get:
$ acq_info
Traceback (most recent call last):
  File "/home/vagrant/.local/bin/acq_info", line 11, in <module>
    load_entry_point('bioread==1.0.5', 'console_scripts', 'acq_info')()
  File "/usr/lib/python3/dist-packages/pkg_resources/__init__.py", line 480, in load_entry_point
    return get_distribution(dist).load_entry_point(group, name)
  File "/usr/lib/python3/dist-packages/pkg_resources/__init__.py", line 2693, in load_entry_point
    return ep.load()
  File "/usr/lib/python3/dist-packages/pkg_resources/__init__.py", line 2324, in load
    return self.resolve()
  File "/usr/lib/python3/dist-packages/pkg_resources/__init__.py", line 2330, in resolve
    module = __import__(self.module_name, fromlist=['__name__'], level=0)
  File "/home/vagrant/.local/lib/python3.6/site-packages/bioread/__init__.py", line 13, in <module>
    from bioread import reader
  File "/home/vagrant/.local/lib/python3.6/site-packages/bioread/reader.py", line 31, in <module>
    from bioread import headers as bh
  File "/home/vagrant/.local/lib/python3.6/site-packages/bioread/headers.py", line 11, in <module>
    from bioread.struct_dict import StructDict
  File "/home/vagrant/.local/lib/python3.6/site-packages/bioread/struct_dict.py", line 13, in <module>
    from bioread.vendor.ordereddict import OrderedDict
ModuleNotFoundError: No module named 'bioread.vendor'

so something is Not Quite Right yet

Aw, sorry for that!
It might be because it's not initiating the vendor folder. Let me open a second PR.
@rmarkello set up versioneer for phys2bids. We could try to do the same here and drop _metadata.py.
Also, I think that the issue with vendor is due to the fact that I accidentally removed the __init__.py library from that folder. I can put it back. Would you like to setup a CI to test that these things won't happen?

I 100% would love to set up CI, though I haven't done so in yonks and have mostly forgotten how to do it :/
It looks like Github has its own solution for this now?
Also, hi, @rmarkello!

Yes it turns out that re-adding init.py to vendor/ makes this better, I've just done it.

Great, sorry for the bug!
Github does have CI solutions now! We still use Travis and azure (still thanks to our great @rmarkello), but why not stay in house? (Probably there's a reason, but I don't know it)

Well, let me know if you cut a release for pypi any time soon, I can't wait to update our installation to make it flawless!

It looks like versioneer expects to be making a _version.py file which won't be compatible with updating a pypyroject.toml file... we'll figure something out though

Okay! I've upload 1.0.5 to pypi, let me know if it makes life better. There's more stuff to do here, but this might make your life better now.

As a little aside: I've set up a CI build with Github Actions and it works a charm. I'm gonna release a 2.0 (only supporting py3.6+) probably today... the last major change to make there is going to be to use pyproject.toml and make setup.py a stub.

Okay again! Just FYI, I've yanked 1.0.5 from pypi and replaced it with a full-on 2.0. I figured that a major release was good for the whole "dropping support for a major language version" thing.
Now I am 100% done commenting on this closed PR.


When saving or publish I get "Error: 422 error making request POST … "Unprocessable Entity"  vendor.e2e21d9c993fa4b677a1.js:96:11726"

So, I making new Article(post) and want to save or publish, I get this error:
Error: 422 error making request POST https://api.github.com/repos/miroque/myblog/git/trees: "Unprocessable Entity"  vendor.e2e21d9c993fa4b677a1.js:96:11726
What I do wrong? Or It's a bug?

I am getting the same

#496
refresh the posts fixes the issue

I would say this is duplicate to #496

Yep it's duplicate to #496

refresh the posts fixes the issue

In my case: refresh or synchronization not  fixes issue at all
It works only once, when I create new Post and save. Next I edit this Post, and I get this error "Error: 422..."

Hi here, the 0.8.1 is out, could you upgrade your instance and tell me if now everything is ok please?

@anthonny You are my hero!
It's gone! The problem has gone!
At last my thoughts can fly freely! Yippee

I apologise, i took so much to fix that. Thank you so much for having been so patient 👍

You know, I've found situation when I get again this error 422.
in this situation:
when I using special symbols like this one « »
== «My very special Topic»
and again I see this 422. (it saves but not publish)
When I deleted this symbols, all started publishing.
I don't know reopen this issue or create new, or just not using this symbols :)

Symbols are not the cause, delete is...
The problem is the cache of the browser, so i have to store the sha1 of the last commit i made to be sure that the next commit will be make on the HEAD and not on a old sha1
And i do not store the sha1 in the delete workflow
I will fix that asap (maybe at lunch)

Also, just now I tested more.
Also it not work when I use Non Latin characters in Main Topic - Page Title. It works and not works.
Now I test with :hp-alt-title:
= Шустрая IDE
:hp-tags: черновик
:hp-alt-title: Speedy IDE  <--- this may be help?

Thanks for your work!
still testing...

Yes Non latin character are not supported, that's why there is the hp-alt-title you are right.
It is because of the library that i use to slugify the title. I have to work on it but it is not really a problem since there is a workaround with the hp-alt-title attribute

Thanks for clarifying!
Now I trying clear my this post, and redone in new with this info

Damn, it not work not with help-tags and not with new post with Latin in Topic ))))
maybe I using FireFox, and it cashes in deeps this js
I try in another browser. But later, not today.

Same problem for hp-tags and actually there is not alternative sorry.
For the title, this must work
= Шустрая IDE
:hp-tags: черновик
:hp-alt-title: Speedy IDE


[PDI-18508] changed toInt, toLong, and toDouble to match the new logi…

…c for checking if string is null

@MarcDiamantHitachi Please add to the existing unit test cases for these methods to include at least tests for null values

:white_check_mark: Build finished in 11m 27s
Build command:
mvn clean verify -B -e -Daudit -amd -pl core

:exclamation: No tests found!
:information_source:  This is an automatic message

:white_check_mark: Build finished in 15m 26s
Build command:
mvn clean verify -B -e -Daudit -amd -pl core

:exclamation: No tests found!
:information_source:  This is an automatic message

:white_check_mark: Build finished in 10m 25s
Build command:
mvn clean verify -B -e -Daudit -amd -pl core

:exclamation: No tests found!
:information_source:  This is an automatic message

:white_check_mark: Build finished in 11m 36s
Build command:
mvn clean verify -B -e -Daudit -amd -pl \
core,plugins/streaming/impls/jms

:exclamation: No tests found!
:information_source:  This is an automatic message


setting exposure not working for Logitech cameras

### Expected behaviour
since version <IP_ADDRESS>, setting exposure was not possible anymore, I had to use other cameras and they worked! but not for Logitech C930 or C270 along with opencv-python, using different platforms macOS, Linux and windows
Write here how did you expect the library to function.
to set the exposure using the function, I've used before
cap = cv2.VideoCapture(0)
cap.set(cv2.CAP_PROP_EXPOSURE, -11)
### Actual behaviour
now setting the exposure has no effect
even cap.get(cv2.CAP_PROP_EXPOSURE)
returns 0.0
Write here what went wrong.
### Steps to reproduce
use any of the mentioned cameras
for the code use a simple video acquisition
- example code
import cv2
cap = cv2.VideoCapture(0)
cap.set(cv2.CAP_PROP_EXPOSURE, -11)
print(cap.get(cv2.CAP_PROP_EXPOSURE))
while True:
#cap.set(cv2.CAP_PROP_FPS, 30)
_, frame = cap.read()
#cv2.imshow('frame', frame)
cv2.imwrite('image.png', frame)
key = cv2.waitKey(1)
if key & 0xFF == ord('s'):
    cv2.imwrite('image.png', frame)
if key & 0xFF == ord('q'):
    break

cap.release()
cv2.destroyAllWindows()

- operating system
macOS Catalina, linux ubuntu 18.04, windows 10

- architecture (e.g. x86)
64-bit

- opencv-python version
opencv-python -> 4.1.1(26)
macOS Catalina, python 3.7.5
linux ubuntu 18.04, python 3.6.8
windows 10, python 3.7.5

Please report the issue to the OpenCV repository: https://github.com/opencv/opencv. I cannot fix OpenCV bugs here.

@skvark it came out that it uses this backend
CODE
import cv2
print(cv2.getBuildInformation())
cap = cv2.VideoCapture(0, cv2.CAP_PROP_BACKEND)
#if not cap.isOpened():
#raise SystemError("could not open")
retval = cv2.videoio_registry.getBackendName(cv2.CAP_PROP_EXPOSUREPROGRAM)
print(retval, cap) #getBackendName(cap))
cap.set(cv2.CAP_PROP_EXPOSURE, -11)
print(cap.get(cv2.CAP_PROP_BACKEND))
cap.release()
cv2.destroyAllWindows()
returned value
General configuration for OpenCV 4.1.1 =====================================
Version control: 4.1.1
Platform:
Timestamp: 2019-09-02T14:54:43Z
Host: Darwin 16.7.0 x86_64
CMake: 3.9.4
CMake generator: Unix Makefiles
CMake build tool: /usr/bin/make
Configuration: Release
CPU/HW features:
Baseline: SSE SSE2 SSE3 SSSE3
requested: DETECT
Dispatched code generation: SSE4_1 SSE4_2 FP16 AVX AVX2
requested: SSE4_1 SSE4_2 AVX FP16 AVX2 AVX512_SKX
SSE4_1 (13 files): + SSE4_1
SSE4_2 (1 files): + SSE4_1 POPCNT SSE4_2
FP16 (0 files): + SSE4_1 POPCNT SSE4_2 FP16 AVX
AVX (4 files): + SSE4_1 POPCNT SSE4_2 AVX
AVX2 (27 files): + SSE4_1 POPCNT SSE4_2 FP16 FMA3 AVX AVX2
C/C++:
Built as dynamic libs?: NO
C++ Compiler: /usr/local/opt/ccache/libexec/clang++ (ver <IP_ADDRESS>20042)
C++ flags (Release): -stdlib=libc++ -fsigned-char -W -Wall -Werror=return-type -Werror=non-virtual-dtor -Werror=address -Werror=sequence-point -Wformat -Werror=format-security -Wmissing-declarations -Wmissing-prototypes -Wstrict-prototypes -Wundef -Winit-self -Wpointer-arith -Wshadow -Wsign-promo -Wuninitialized -Winit-self -Wno-delete-non-virtual-dtor -Wno-unnamed-type-template-args -Wno-comment -fdiagnostics-show-option -Wno-long-long -Qunused-arguments -Wno-semicolon-before-method-body -ffunction-sections -fdata-sections -fvisibility=hidden -fvisibility-inlines-hidden -O3 -DNDEBUG -DNDEBUG
C++ flags (Debug): -stdlib=libc++ -fsigned-char -W -Wall -Werror=return-type -Werror=non-virtual-dtor -Werror=address -Werror=sequence-point -Wformat -Werror=format-security -Wmissing-declarations -Wmissing-prototypes -Wstrict-prototypes -Wundef -Winit-self -Wpointer-arith -Wshadow -Wsign-promo -Wuninitialized -Winit-self -Wno-delete-non-virtual-dtor -Wno-unnamed-type-template-args -Wno-comment -fdiagnostics-show-option -Wno-long-long -Qunused-arguments -Wno-semicolon-before-method-body -ffunction-sections -fdata-sections -fvisibility=hidden -fvisibility-inlines-hidden -g -O0 -DDEBUG -D_DEBUG
C Compiler: /usr/local/opt/ccache/libexec/clang
C flags (Release): -fsigned-char -W -Wall -Werror=return-type -Werror=non-virtual-dtor -Werror=address -Werror=sequence-point -Wformat -Werror=format-security -Wmissing-declarations -Wmissing-prototypes -Wstrict-prototypes -Wundef -Winit-self -Wpointer-arith -Wshadow -Wsign-promo -Wuninitialized -Winit-self -Wno-delete-non-virtual-dtor -Wno-unnamed-type-template-args -Wno-comment -fdiagnostics-show-option -Wno-long-long -Qunused-arguments -Wno-semicolon-before-method-body -ffunction-sections -fdata-sections -fvisibility=hidden -fvisibility-inlines-hidden -O3 -DNDEBUG -DNDEBUG
C flags (Debug): -fsigned-char -W -Wall -Werror=return-type -Werror=non-virtual-dtor -Werror=address -Werror=sequence-point -Wformat -Werror=format-security -Wmissing-declarations -Wmissing-prototypes -Wstrict-prototypes -Wundef -Winit-self -Wpointer-arith -Wshadow -Wsign-promo -Wuninitialized -Winit-self -Wno-delete-non-virtual-dtor -Wno-unnamed-type-template-args -Wno-comment -fdiagnostics-show-option -Wno-long-long -Qunused-arguments -Wno-semicolon-before-method-body -ffunction-sections -fdata-sections -fvisibility=hidden -fvisibility-inlines-hidden -g -O0 -DDEBUG -D_DEBUG
Linker flags (Release): -Wl,-dead_strip
Linker flags (Debug): -Wl,-dead_strip
ccache: YES
Precompiled headers: NO
Extra dependencies: -framework OpenCL ade /usr/local/lib/QtGui.framework /usr/local/lib/QtTest.framework /usr/local/lib/QtCore.framework
3rdparty dependencies: ittnotify libprotobuf zlib libjpeg-turbo libwebp libpng libtiff libjasper IlmImf quirc ippiw ippicv
OpenCV modules:
To be built: calib3d core dnn features2d flann gapi highgui imgcodecs imgproc ml objdetect photo python3 stitching video videoio
Disabled: world
Disabled by dependency: -
Unavailable: java js python2 ts
Applications: -
Documentation: NO
Non-free algorithms: NO
GUI:
QT: YES (ver 4.8.7 EDITION = OpenSource)
QT OpenGL support: NO
Cocoa: YES
VTK support: NO
Media I/O:
ZLib: build (ver 1.2.11)
JPEG: build-libjpeg-turbo (ver 2.0.2-62)
WEBP: build (ver encoder: 0x020e)
PNG: build (ver 1.6.37)
TIFF: build (ver 42 - 4.0.10)
JPEG 2000: build (ver 1.900.1)
OpenEXR: build (ver 2.3.0)
HDR: YES
SUNRASTER: YES
PXM: YES
PFM: YES
Video I/O:
DC1394: NO
FFMPEG: YES
avcodec: YES (58.35.100)
avformat: YES (58.20.100)
avutil: YES (56.22.100)
swscale: YES (5.3.100)
avresample: YES (4.0.0)
GStreamer: NO
AVFoundation: YES
Parallel framework: GCD
Trace: YES (with Intel ITT)
Other third-party libraries:
Intel IPP: 2019.0.0 Gold [2019.0.0]
at: /Users/travis/build/skvark/opencv-python/_skbuild/macosx-10.8-x86_64-3.7/cmake-build/3rdparty/ippicv/ippicv_mac/icv
Intel IPP IW: sources (2019.0.0)
at: /Users/travis/build/skvark/opencv-python/_skbuild/macosx-10.8-x86_64-3.7/cmake-build/3rdparty/ippicv/ippicv_mac/iw
Eigen: NO
Custom HAL: NO
Protobuf: build (3.5.1)
OpenCL: YES (no extra features)
Include path: NO
Link libraries: -framework OpenCL
Python 3:
Interpreter: /Users/travis/build/**skvark/opencv-python/**venv/bin/python (ver 3.7)
Libraries: /Library/Frameworks/Python.framework/Versions/3.7/lib/libpython3.7m.dylib (ver 3.7.0)
numpy: /Users/travis/build/skvark/opencv-python/venv/lib/python3.7/site-packages/numpy/core/include (ver 1.14.5)
install path: python
Python (for build): /Users/travis/build/skvark/opencv-python/venv/bin/python
Java:
ant: NO
JNI: /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.12.sdk/System/Library/Frameworks/JavaVM.framework/Headers /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.12.sdk/System/Library/Frameworks/JavaVM.framework/Headers /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.12.sdk/System/Library/Frameworks/JavaVM.framework/Headers
Java wrappers: NO
Java tests: NO
Install to:/Users/travis/build/skvark/opencv-python/_skbuild/macosx-10.8-x86_64-3.7/cmake-install-----------------------------------------------------------------
UnknownVideoAPI(17009)

I have no idea what's the issue. This indicates that you are out of luck, some cameras just won't work:
https://answers.opencv.org/question/175793/why-am-i-not-able-to-change-all-settings-of-logitech-c920-camera-using-opencv-python/

Also, this doesn't look right:
cap = cv2.VideoCapture(0, cv2.CAP_PROP_BACKEND)
try for example:
cap = cv2.VideoCapture(-1)

Also from official docs:
cv.CAP_PROP_BACKEND : Current backend (enum VideoCaptureAPIs). Read-only property.
It is not a valid backend but a readonly property. The valid backends can be found here: https://docs.opencv.org/3.4/dc/dfc/group__videoio__flags__others.html

tried the same code and camera on windows 10, I was able to adjust exposure!
so I guess I need to build a valid release from opencv.org, would that guarantee, valid or official backend, whatever the OS is? please correct me if I'm wrong... I'm new to this -backend- expression

tried the same code and camera on windows 10, I was able to adjust exposure!

It depends whats type of API bindings does on windows 10 OpenCV is using, it may default to correct backend automatically. So you must always use correct code to access valid backend.

I need to build a valid release from opencv.org

To test official version, You first need to completely remove the opencv-python pypi version and then install official OpenCV from scratch. You can follow this tutorial for installing official OpenCV.

would that guarantee, valid or official backend, whatever the OS is?

Nope, you first have to use correct code and a valid backend to ensure everything is ok. But still you cannot guarantee anything as it depends upon your camera hardware.


error when launching single dispatcher using --hunger-policy

When (locally) I follow the docs to run an event and run a single dispatcher using a given command:
~/ramp_deployment $ ramp launch dispatcher --event-config events/iris_test/config.yml --hunger-policy sleep -vv
I am receiving an error:

Error: no such option: --hunger-policy

It runs alright if the --hunger-policy option is removed

Description

Steps/Code to Reproduce

Expected Results

Actual Results

Versions

True. We need to add the parameter to the signature. The code already use a default: https://github.com/paris-saclay-cds/ramp-board/blob/master/ramp-engine/ramp_engine/cli.py#L81


Add citation information to makhzan dataset

This PR adds the citation information to makhzan datasets, once they have replied to our request for that information:

https://github.com/zeerakahmed/makhzan/issues/43

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Fix custom fonts do not load correctly on Android < 10

This hotfix should resolve this issue:
https://github.com/mit-cml/appinventor-sources/issues/2858

Can one of the admins verify this patch?

@patryk84a since this affects behavior of the companion the base branch should be ucr rather than master.

Yes it's a good idea. If it had been done this way when adding custom fonts, we would have avoided the changes now.

Please answer the questions so I can continue.

@AppInventorWorkerBee ok to test

I fixed the compilation error and reverted the visibility change in MediaUtil. This has been tested with Android 5.1.1 (macOS emulator) and Android 13 (Google Pixel 6a).


[SPARK-5801] [core] Avoid creating nested directories.

Cache the value of the local root dirs to use for storing local data,
so that the same directories are reused.
Also, to avoid an extra level of nesting, use a different env variable
to propagate the local dirs from the Worker to the executors. And make
the executor directory use a different name.

Test build #27903 has started for   PR 4747 at commit 18ee0a7.

This patch merges cleanly.

Hold off a little bit on this; let me see how to handle the SparkConf issue (I think Josh added that for the unit tests).

Test build #27903 has finished for   PR 4747 at commit 18ee0a7.

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

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

retest this please. We really need to fix the JavaAPISuite at some point.

Test build #27908 has started for   PR 4747 at commit 18ee0a7.

This patch merges cleanly.

Test build #27909 has started for   PR 4747 at commit e0114e1.

This patch merges cleanly.

Test build #27908 has finished for   PR 4747 at commit 18ee0a7.

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

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

Test build #27909 has finished for   PR 4747 at commit e0114e1.

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

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


column: add support for Arel::Nodes::SqlLiteral

This PR resolves #252, allowing for more more advanced columns, such as extracting elements out of json arrays, or fields out of json objects. This is intended to be entirely backward compatible.

Is this going to be released? I don't see this code in v.2.3.6

Can we get some proper documentation about how to use this?

Can we get some proper documentation about how to use this?

me too

➕ 1⃣ on the docs. Or just a stupid-simple example for moots like myself.

Found something useful in tests. An example, that works for me:
class Car < ApplicationRecord
  store_accessor :options, :number
end

class Driver < ApplicationRecord
  has_many :cars

  pg_search_scope(
    :pg_search,
    against: [:first_name, :last_name, :middle_name],
    associated_against: {
      cars: [Arel.sql("cars.options->>'number'")]
    },
    using: { tsearch: { prefix: true } }
  )
end

But looks like tsearch does not work for "cars.options->>'number'") column. Trying to fix it right now. It works well for first_name, etc. of Driver still.

did you ever get this working?


⚠ Mobitopup has degraded performance

In e58f669, Mobitopup (https://napcard.voz.tech) experienced degraded performance:

HTTP code: 200
Response time: 8131 ms

Resolved: Mobitopup performance has improved in a484f3e.


Feature: sensu message processing rates in api /info call

It would be awesome to see some extended details in the /info about message.  If we run into a message processing issue (sometimes an artifact of a network outage/degradation), we often go directly to RabbitMQ to inspect the transport message rates.
It'd be great to add these metrics into the /info api call, which could also then be used by Sensu internals to make decisions about the state of the Sensu transport (ex: should I add more messages to an underwater keepalive queue?)
Specific metrics that might be nice:

Messages published in last 60s/300s/3600s
Messages delivered in last 60s/300s/3600s
Messages acked in last 60s/300s/3600s
Messages unacked in last 60s/300s/3600s

@xyntrix these statistics are provided by a RabbitMQ management plugin and are not available via the AMQP protocol/library. AFAIK these metrics are only available via the RabbitMQ management HTTP API.
Sensu supports other Transports (via sensu-transport), which do not support these metrics or a method of accessing them. We are not going to implement  this, the RabbitMQ management remains the best way of accessing this information.
@xyntrix you could use a RabbitMQ management plugin to collect these metrics, send them to your TSDB of choice, and then embedded the graphs in Uchiwa for a more streamlined experience. You could also monitor these metrics with set thresholds to trigger notifications etc :+1:


Tutorials missing workflows

This is a list of tutorials that are missing a workflow, if anybody would like to run this tutorial and extract the workflow from their history afterwards and upload it here, this would be a great help for us!

[ ] Topic: computational-chemistry Tutorial: setting-up-molecular-systems  Link
[ ] Topic: ecology Tutorial: species-distribution-modeling  Link
[x] Topic: epigenetics Tutorial: tal1-binding-site-identification  Link
[ ] Topic: genome-annotation Tutorial: genome-annotation  Link
[x] Topic: genome-annotation Tutorial: tnseq  Link
[x] Topic: introduction Tutorial: galaxy-intro-101  Link
[ ] Topic: introduction Tutorial: galaxy-intro-ngs-data-managment  Link
[x] Topic: introduction Tutorial: galaxy-intro-peaks2genes  Link
[x] Topic: introduction Tutorial: igv-introduction  Link
[ ] Topic: introduction Tutorial: r-basics  Link
[x] Topic: proteomics Tutorial: labelfree-vs-labelled  Link
[x] Topic: transcriptomics Tutorial: rb-rnaseq  Link
[ ] Topic: transcriptomics Tutorial: rna-seq-counts-to-viz-in-r  Link
[x] Topic: transcriptomics Tutorial: scrna-plates-batches-barcodes  Link
[x] Topic: transcriptomics Tutorial: scrna-umis  Link
[x] Topic: variant-analysis Tutorial: non-dip  Link
[x] Topic: variant-analysis Tutorial: somatic-variants  Link

And if you run into any other issues or questions when doing the tutorial, please add a fix or file an issue for it :)

Trying out the Statistics and machine learning  tutorial.

Updated with fresh data, ca 75% of tutorials have workflows so that's great! :)

@Delphine-L would you have a history where you could extract the tn-seq workflow? I have a user who wants to run that tutorial and want to make sure I've collected all the needed tools for it.

Hi @hexylena @shiltemann !
I have created a workflow for the tutorial "tal1-binding-site-identification", and I have identified some updates to the existing tutorial that I am happy to put in a PR. I noticed that at the top of the tutorial it says "Under Development! This tutorial is not in its final state. The content may change a lot in the next months. Because of this status, it is also not listed in the topic pages.". So, I have two questions:

Does the TAL1 tutorial still need a workflow?
Would it be useful for me to make a PR with updates to the tutorial itself, or is someone else working on it?

Thanks!
#Hacktoberfest

Hi @malloryfreeberg I'm not sure anyone is currently working on it. It looks like it hasn't seen any updates in some time. If you have some updates to the tutorial itself to bring it up to quality, then that's fantastic! We can maybe set enable: true finally :) It looks like it still needs a WF so that would be great!

Thanks @hexylena ! I have made the PR updating the TAL1 tutorial (https://github.com/galaxyproject/training-material/pull/2047) including adding a workflow and updating enabled: true.
I am happy to keep going through the tutorials listed in this ticket and generate workflows. The interface has changed quite a bit since I last used Galaxy - it looks great!

I was going to generate a workflow for scrna-umis but realised there was no way to include a pasted text as input in the workflow-test.yml. I opened a ticket on the main Galaxy repo (https://github.com/galaxyproject/galaxy/issues/10415) and am happy to finish the workflow for this tutorial as a test case when this issue is addressed.

Is it actually possible to make workflows for the R-based tutorials?

Tutorial r-basics
Tutorial: rna-seq-counts-to-viz-in-r

Nope! Not currently. Maybe someday thanks to new ITs which are workflow compatible, we'll get that, but, not yet.

@bebatut Hi! I think I can help with this
also, this list needs updating, some of the unchecked tutorials already have workflows


Add core http compat to AI Metrics Advisor

Packages impacted by this PR
@azure/ai-metrics-advisor
Issues associated with this PR

https://github.com/Azure/azure-sdk-for-js/issues/20917
https://github.com/Azure/azure-sdk-for-js/issues/15594

Describe the problem that is addressed by this PR
AI Metrics Advisor has already migrated to the new Core client and Core rest pipeline libraries, however, this introduced too many breaking changes. In order to minimize the braking changes, core-http-compat was created. This PR regenerates the client with the latest version of autorest, with the core-http-compat-mode flag set to true, and no longer depending on core-http.
Checklists

[X] Added impacted package name to the issue description
[X] Does this PR needs any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here)
[X] Added a changelog (if necessary)

API changes have been detected in @azure/ai-metrics-advisor. You can review API changes here
API changes
- export interface MetricsAdvisorAdministrationClientOptions extends CommonClientOptions { }
+ export interface MetricsAdvisorAdministrationClientOptions extends ExtendedCommonClientOptions { }
- export interface MetricsAdvisorClientOptions extends CommonClientOptions { }
+ export interface MetricsAdvisorClientOptions extends ExtendedCommonClientOptions { }


Sandbox

I want to use this API to generate BCH wallet , send BCH from one to another address and get transaction details.
Is there any sandbox or testnet server associate with it ????
If there something else please suggest me..
Thanks in advance.

There is no sandbox functionality at this time.

Still not possible to test?

No, there is no sandbox at this time, and no plans to support one in the near term.


BAEL-3938: Add new section on Java 10 Collectors.toUnmodifiableList()

I had to change the java version to 11 to support Collectors.toUnmodifiableList() and to enable the --enable-preview flag for the maven compiler plugin that I saw was already used on other modules.
I also added an example of using Java 9's List.copyOf() which is not mentioned in the article. My intention with this was to follow the existing pattern, where there are more cases presented in the tests than in the article itself.
Besides these is the test that is included in the article, following the pattern of the existing tests.

@ashleyfrieze I'm force-pushing to fix the compile error that fails the CI check. It only affects the first commit where the pom.xml is modified, so I hope if you comment on the tests in the second commit that won't be affected.


One touch zoom in & out function

Hi,
it would be cool to have a one finger zoom in and out functionality like Google Maps.
Meaning to touch and hold with one finger & moving finger up to zoom in & moving finger down to zoom out again.

This happens in Google Maps? The app or the website? Desktop or mobile? I've never seen this in Google Maps. Are you talking about pressing in the map itself or on some sort of control?
At any rate, you could certainly write it with OSD by overriding the normal mouse controls.

yes, Google Maps provides this quite handy functionality on mobile devices, when touching the map.  :)
Would be cool, to have it ootb in OSD.

The Google Maps app? Or the website? Is this on iOS or Android? I've tried it on iOS in both the app and the website and I can't make it happen.

For me it works on the website (Safari & Google on iPad & iPhone) & in the app.
You need to double tap & keep holding the finger on the second tap. When moving the finger up you are zooming in, when moving the finger down you are zooming out.

Oh! Double tap first! Yup, I see it now. Cool… I had no idea that existed!
I wonder how common knowledge of this gesture is… I don't know that we want to add it to OpenSeadragon directly, but I imagine you should be able to build it on top, using a MouseTracker.

Hi,
this sound like a cool feature to improve accessibility and usability!
I would be happy if you would add to your roadmap.
Thanks!

@SMA-cgn Nice! That's a great idea! When I follow the link, however, it says "You don't currently have access to this page". Do you need to publish it or something?

Thanks for the note! Not sure how to share a public link? Probably on will need an account. Development will start soon now.
It would be great if you could advice how to integrate it best.
The developer will join the discussion here soon.

How many hours effort would you estimate to get that done?
The alignment on Upwork is already to heavy for such a small feature 😵💫

Thanks @SMA-cgn, I just had a look at the code.
Based on my initial understanding, there is a onCanvasClick() function defined in Viewer.js
Would it be a good decision to implement this feature in that function as clickToZoom is also implemented there
I think that another property such as doubleTouchDrag should also be added in settings which will be then used to override clickToZoom if it is turned on. Any help is highly appreciated :)

The double tap zoom in functionality should only be enabled for mobile web view and mobile app but not for desktop view of OSD. Going with this approach. If anyone thinks different, let me know

Hi Hamza,
right, this should only be available on mobile touch devices. On desktop PCs the mouse wheel already does the job :)
Thanks!

Gesture Settings exist for each pointer type.
        gestureSettingsMouse {...}
        gestureSettingsTouch {...}
        gestureSettingsPen {...}
        gestureSettingsUnknown {...}

These settings are as follows
            dragToPan: true,
            scrollToZoom: true,
            clickToZoom: true,
            dblClickToZoom: false,
            pinchToZoom: false,
            zoomToRefPoint: true,
            flickEnabled: false,
            flickMinSpeed: 120,
            flickMomentum: 0.25,
            pinchRotate: false

I have to add mine for this feature. Any idea what should I name it or if there is a proper name for it. Maybe DoubleTapDrag ?

Another Thing Possible is that it should follow the setting of dblClickToZoom. I think it is a better approach because this feature is actually an extended feature of dblClickToZoom and follows the same boolean value for each screen type

When do gesture settings change, How do I test the feature on mobile.

Sry, not sure if I got the question right.
I would expect to execute this feature when you double tap on the screen & keep your finger hold on the screen on the second tap. In this hold position move up to zoom in and move down to zoom out.
It should still be possible to just double tap (without hold) in order to zoom in.

Hi, this would be fine for me!

@HamzaTatheer Welcome! Thank you for tackling this!
Implementing it in onCanvasClick seems like a good start, though that might not be the exact right place. I think I would keep track of the most recent click event there, but then in onCanvasPress note whether it was soon enough to be considered a double click, and then in onCanvasDrag do the right thing based on that information.

I would prefer to have a new feature called dblTapHoldToZoom which is disabled by default.

Sounds good, or how about dblTapDragToZoom? Also, it should probably be click, rather than tap, just to be consistent with dblClickToZoom (which also supports both mouse and touch). So dblClickDragToZoom?

When do gesture settings change, How do I test the feature on mobile.

I believe it's based on the type of event. To get touch events on your computer, you need to switch into mobile mode in the development tools.

@iangilman Okay, great. That clear things out. I am stuck in a weird problem. I am testing my code using http://localhost:8000/test/demo/basic.html
However, whenever I Zoom It zooms in  or zooms out too much. This is the code I wrote for zooming in while dragging
if (gestureSettings.dblClickToZoom && THIS[ this.hash ].draggingToZoom){
    var factor = Math.pow( this.zoomPerDblTapDrag, event.delta.y );
    this.viewport.zoomBy(factor);
    this.viewport.applyConstraints();
}

I know that the factor comes out to be a huge number but even if i hard code factor by 0.1, it gives the result.
Can you please have a look at my repo and give me some insight on how i can make it zoom less.

I finally found one equation that works much better now. Do let me know some improvements. Hardcoding 100 means i get a less zoom factor.
if (gestureSettings.dblClickToZoom && THIS[ this.hash ].draggingToZoom){
    var factor = Math.pow( this.zoomPerDblTapDrag, event.delta.y / 100 );
    this.viewport.zoomBy(factor);
    this.viewport.applyConstraints();
}

@iangilman  from my perspective the new zoom functionality works fine! It would be great to have it in the core of openseadragon!
Hamza still needs your feedback on the code for the zoom speed. It would be great if you could review it please.
@HamzaTatheer thanks for the great work! Would you mind to share the link to your test environment here in order to demo the new zoom functionality?
Thanks a lot!

Sure,
Demo: https://master--vocal-wisp-c1e55d.netlify.app/test/demo/collections/index.html
Code: https://github.com/HamzaTatheer/openseadragon

@HamzaTatheer, I'm blanking on the proper equation at the moment, but basically what you want is something where if you drag in one direction x pixels, the factor is 2, and if you drag in the other direction the same amount, the factor is 0.5. You can then multiply that by zoomPerDblTapDrag so the user can help specify how fast it goes. I don't think what you have here is quite that.
I went to take a look at your demo just now, and it seems it's not showing the new feature at the moment? I couldn't get it to do it.
I also took a quick look over your code, and it looks like a good start. Maybe it's time to start a pull request, so we can be discussing various aspects directly in the code?

Hmm... Looks like the scroll handler does exactly what you're doing, so yeah, I guess you've got it! It's just been a while since I thought about that part of the code.

@HamzaTatheer i just tested it again in the posted test environment. It seems that there is again this behaviour that it zooms only when releasing the finger after the double tap and then drag up/down. Could you please check?
Could you also please set viewer maxZoomLevel = 1000 to enable a deeper zoom in the test environment?

@SMA-cgn Regarding zoom in after releasing the double tap and then dragging, can you tell me that does it also work for the situation where you dont release the finger on double tap and keep dragging.
Moreover, yes i wil change the zoom level in the morning

@iangilman Sure. I will create the PR. Moreover, i am not sure why the feature is not working for you because when i test it on my own mobile and laptop, it is working.
are you double tapping and then not releasing finger on second tap but instead dragging up and down ?

@HamzaTatheer I've just retested and I'm still getting the same results. I double click and on the second click, I don't release the mouse button, but instead drag up and down. The image pans just like a normal drag. When I look at the code:
https://master--vocal-wisp-c1e55d.netlify.app/test/demo/collections/main.js
... I don't see any special double tap drag options in the config object at the top. Shouldn't there be?
Tested on Mac Firefox and Chrome.

Look at the 5th option. I added this in openseadragon.js, should it be some place else ?
        gestureSettingsMouse:   {
            dragToPan: true,
            scrollToZoom: true,
            clickToZoom: true,
            dblClickToZoom: false,
            dblTapDragToZoom: false,
            pinchToZoom: false,
            zoomToRefPoint: true,
            flickEnabled: false,
            flickMinSpeed: 120,
            flickMomentum: 0.25,
            pinchRotate: false
        },

The Reason it was not working on your laptop was because it is disabled through gesture settings on laptop. Besides this regarding the formula, I originally tried to avoid throttling in the original formula used by onCanvasScroll but upon analyzing it further, It seems that I should follow the same approach. I will make this change and one other change regarding a bug mentioned by @SMA-cgn  and then create a PR so we can proceed ahead by code

I added the same formula as used in onCanvasScroll, it seems to be not giving smooth results. I will try some other approach. I have also created PR now

@HamzaTatheer I'm not seeing any reference to gestureSettingsMouse in your demo file:
https://master--vocal-wisp-c1e55d.netlify.app/test/demo/collections/main.js
Perhaps that's why it's not working there? Anyway, now that you have the PR in, I can test locally :)

@SMA-cgn @liveman1 The conversation is continuing in #2225 if you want to check in there :)


Form dialog recognizer type (.lu) is overwritten when you make any changes in Composer to *.lu.qna

Describe the bug
A form dialog is generated with the recognizer of type .lu, but editing the dialog in Composer seems to change the recognizer to .lu.qna causing the bot to give an "Object reference not set to an instance of an object" error.
Version
main branch, pulled November 18
Browser

[ ] Electron distribution
[x] Chrome
[ ] Safari
[ ] Firefox
[ ] Edge

OS

[ ] macOS
[x] Windows
[ ] Ubuntu

To Reproduce
Steps to reproduce the behavior:

Turn on the form dialog UI by going to Settings > App settings > Preview features and checking "Form dialogs."

Create a new bot (I used the the EchoBot template).

Create a form by clicking on the table-looking icon in the side nav, pressing the + button in the Schema pane, and adding a name. Add properties (or you can import a sample form schema form here: https://github.com/microsoft/BotBuilder-Samples/tree/main/experimental/generation/docs/example-schemas)

Generate the dialog by pressing the "Generate dialog" button.

Connect the form dialog to the main dialog (I added a new trigger that used a Begin Dialog node to start the form dialog).

Start the bot. The bot should work fine.

Edit the form dialog (I added a trace event to the Dialog started trigger).

Restart the bot. Now there is an error:

I removed the trace event from the Dialog started trigger and restarted the bot, and the error still appeared. Then I compared the .dialog file with a copy I made before making any changes to it, and only the recognizer line was different:

Removing the .qna and restarting the bot removed the error.

Expected behavior
Either the recognizer for the form dialog should start off as .lu.qna or Composer should not change it.
Screenshots

Additional context

@boydc2014

@boydc2014 Chris suggested you might have more context for this.

Yes. The conversation to .lu.qna is expected, because the convention used in Composer is CrossTrainedRecoginzerSet(.lu.qna.dialog).
So i guess this issue is not because .lu => .lu.qna, is more because the correct structure .lu.qna isn't correctly generated.
I will ask @zhixzhan to look at this soon.

@chrimc62 adding you for visibility.


Fix preview window size in visual shader editor

Fix https://github.com/godotengine/godot/issues/72949

Superseded by #72991.


refactor: abstract checkbox presentation to standalone component

Moves the presentation logic from the md-checkbox to a md-pseudo-checkbox that can be re-used in other components, like md-select and md-selection-list. Renames and refactors a bunch of CSS classes, unit tests, properties etc.
Relates to #2412.

Sorry for the huge PR @jelbourn, but there's no new functionality, just moving stuff around. The main thing is the MdSelectionModule. It's intended to contain the functionality from #2562.

Closing in favor of https://github.com/angular/material2/pull/2619


Refactor and improve tests

Resolves #36

Fix Remove directory exists check: https://github.com/webdriverio-community/wdio-rerun-service/blob/main/src/index.js#L71

Based on #42, see https://github.com/SCG82/wdio-rerun-service/pull/1 to view the diff.
Merge #42 before merging this PR.

Same here, it seems like similar PR as to #42 and #44, any chance you can chain them?

Doesn't look like that's possible unless I can push these branches to this repository.

@SCG82 I merged #42, can you fix these conflicts? I will merge the remaining PRs then do the release.

I also added you (@SCG82) to the org, so feel free to go ahead and just merge directly into main. Thanks for everything!

closing in lieu of #46


Need to have internet access when doing enable_mathjax=true

Hi,
Last wednesday, I wanted to enable mathjax for the first time. So I set enable_mathjax=true. But it was not working when previewing the file in my browser.
I finally found that it was because I didn't have an internet access at the time and that I needed to have internet access for mathjax to download in the background. I found this information on Timon Wong's blog.
I think it could be a good thing to add it in the README, after the small example in Mathjax support.
What do you think?
I can do the PR if you want.
Albert

I think that sounds reasonable.  I think a pull request would help others.

@albertcthomas please close this


Need to create a daemonjob operator

We have the default daemonset objects as part of native Kubernetes API.
However there are situations when we need to run a code on a kubernetes node one time only (like running a script to install Antivirus software on the physical host operating system).
For this we need a daemonjob controller which will run one time and complete the task.
Ref. code from Metacontroller : https://github.com/GoogleCloudPlatform/metacontroller/tree/master/examples/daemonjob
Acceptance Criteria :

the script runs successfully
The daemonjob object get into completed state and does not keep running.

I think we can do most of this -- the difference in the daemonjob is that they have an arbitrary Python script running. Our health check will return as successful as soon as the pods pass their readiness check, which doesn't quite handle this.
However, we do have the semantics for deleting these resources in a step, so I think we're close. We could have a different type of task for now (until we figure out something more extensible) that watches for completion of the end pods it cares about rather than the fact that they're running. This could be used in situations other than a DaemonJob.
Also, @sb1975, I'm transferring this over to to the KUDO repo proper.


pylintrc: re-enable unused-private-member check

the only offender of this check was the plugin summary functions that are named in "__[d]+_summary" format. Added the new summary_function decorator that stores the summary function metadata (name, index) into the decorated function (i.e. summary function). Decorated functions in such manner get automatically picked up by the PluginPartBase class.

@dosaboy I've made the changes, ready for review.


Feature/dbc22 1050

Implemented event and camera list along the route.

Closed as this wasn't meant for this sprint.

Branch deleted as code is obsolete. To be recreated with up-to-date code when we start this ticket again.


Cold start takes 2-3 minutes. Any way to improve?

Is it a big project? Try to start with just ts-node the same result?

Around 20k lines, express, redux, sockets.io. Ts-node is also a bit slow, but not that much.

node_modulesare excluded by default,
So you have 1000 of TS files, each in average 20 lines, required by single running node process?

Yes, kind of.

Interesting what kind of process/service is launched this way?

Just a node server, nothing too fancy. Is just a backend server for a game.

If I run this command it takes around 20 seconds. With ts-node-dev takes around 5 minutes now.

We also have a projet, not as large as @hackhat but still, it takes about 5 seconds to load on MacOS but about 5 minutes to load on windows

@MatxBerg with --transpileOnly?

Will close for now