Invalid VolumeID: error decoding volume ID (failed to decode CSI identifier, string underflow)

Hi,
I have a Ceph cluster, I want to clean up unnecessary files from time to time.
I have configured everything as described in the documentation.
https://github.com/csi-addons/kubernetes-csi-addons/blob/v0.8.0/docs/deploy-controller.md#installation-for-versioned-deployments
https://github.com/csi-addons/kubernetes-csi-addons/blob/v0.8.0/docs/reclaimspace.md#reclaimspacecronjob
reclaimspacecronjob:
apiVersion: csiaddons.openshift.io/v1alpha1
kind: ReclaimSpaceCronJob
metadata:
  name: reclaimspace-mongo
  namespace: mongo
spec:
  concurrencyPolicy: Forbid
  failedJobsHistoryLimit: 5
  jobTemplate:
    spec:
      backOffLimit: 6
      retryDeadlineSeconds: 600
      timeout: 3600
      target:
        persistentVolumeClaim: mongo
  schedule: '@hourly'
  successfulJobsHistoryLimit: 5

The controller works, finds all possible PVCs and their statuses.
CronJob triggers and is in mongo namespace.
My PVC and PV have the same names - mongo.
This is what my StorageClass looks like
apiVersion: storage.k8s.io/v1
kind: StorageClass
metadata:
  name: app-block-retain
provisioner: rook-ceph.rbd.csi.ceph.com
allowVolumeExpansion: true
reclaimPolicy: Retain
volumeBindingMode: Immediate
parameters:
  clusterID: rook-ceph
  csi.storage.k8s.io/controller-expand-secret-name: rook-csi-rbd-provisioner
  csi.storage.k8s.io/controller-expand-secret-namespace: rook-ceph
  csi.storage.k8s.io/fstype: ext4
  csi.storage.k8s.io/node-stage-secret-name: rook-csi-rbd-node
  csi.storage.k8s.io/node-stage-secret-namespace: rook-ceph
  csi.storage.k8s.io/provisioner-secret-name: rook-csi-rbd-provisioner
  csi.storage.k8s.io/provisioner-secret-namespace: rook-ceph
  imageFeatures: layering
  imageFormat: '2'
  pool: app
---

This is how the rbd was created: rbd create --size 1024000 app/mongo
CSI controller is not finding the mysql PV. But it exists, I can see it in the PVs list.
Here is the log from the controller.
2024-07-15T02:12:05.074Z	ERROR	Failed to make controller request	{"controller": "reclaimspacejob", "controllerGroup": "csiaddons.openshift.io", "controllerKind": "ReclaimSpaceJob", "ReclaimSpaceJob": {"name":"reclaimspace-mongo-1721009520","namespace":"mongo"}, "namespace": "mongo", "name": "reclaimspace-mongo-1721009520", "reconcileID": "ed1dc2d7-1e9f-415d-8a44-cd6be76308fe", "PVCName": "mongo", "PVCNamespace": "mongo", "PVName": "mongo", "NodeID": "app-worker1", "Timeout": "1h0m0s", "nodeClient": "rook-ceph/csi-rbdplugin-7zj59", "error": "rpc error: code = Aborted desc = failed to find volume with ID \"mongo\": invalid VolumeID: error decoding volume ID (failed to decode CSI identifier, string underflow) (mongo)"}

@JWebDev we dont support static pvc in cephcsi, it's only supported for dynamic PVC, the issue is not with csi-addons its with cephcsi. please feel free to open an issue in cephcsi if you need support for static pvc

@Madhu-1 Thank you for your answer.
I carefully reread your answerseveral times.
Is there something I don't understand? Or please tell me where it is written in the documentation that Ceph does not support Static PVs?
I have been using Static PVs with Ceph for at least a year now. Everything works fine.
Here is a link to the documentation. https://github.com/ceph/ceph-csi/blob/devel/docs/static-pvc.md
Why closed?

warning static PVC can be created, deleted, mounted and unmounted but currently ceph-csi doesn't support other operations like snapshot,clone, resize, etc for static PVC

@JWebDev  sorry if i was not clear, we have above warning in the cephcsi static PVC doc where it mentioned it supports only mounting and unmounting operations for static PVC nothing else. which means other operations like reclaimspace, clone,snapshot,encryption, Replication and may operations are not supported for static PVC in cephcsi.
The reason for closing the issue is that kubernetes-csi-addons repo is not specific to cephcsi, the current problem that you are experiencing is related to cephcsi not related to kubernetes-csi-addons.

@Madhu-1
This is a very important point. It needs to be emphasized more bolder! )))
I missed it.
It all falls into place now. I understand the problem.
Although very strange because I can resize Ceph image with rbd resize. I just tried it and it works. But after resizing in PVC. The size in PV did not change - this is true.
Thanks for the clarifying.


update url according to documentation

Hi,
I noticed a small typo in the addContactToJourney url according to the documentation : http://docs.autopilot.apiary.io/#reference/api-methods/contact-operations/get-list-of-journeys-with-api-triggers
I have tested it on my website and it is working with the new url.
Is this modification ok for you ?

Yes. You are correct.

@nicolasThal: We are no longer using this repo. Would you want to take over?

@ruskiyos : As we start to use it, why not :)

Do you want it under your username or do you have an organization to add it under?
I guess I'll probably just abandon the packagist repo and let you create a new one, unless there is a clean way to transfer that as well.

I would rather have it undner the dekalee organization, this way, everyone from my company can modify it.
I don't know if there is a clean way to transfer a repository with packagist...

I think I need to be added as a collaborator in dekalee in order to transfer ownership. I got this error when I tried.
You don’t have the permission to create repositories on dekalee

Done. Sorry. I was out of town.

@ruskiyos Thank you.
I have transfered the repository on my company's organisation.
I have checked on packagist and as long as you don't create a repository with the name finicprint/php-autopilothq it should be ok.
Thank you for all your work :)

👍

@ruskiyos could you also add me as maintainer on the packagist project ?
My username is also nicolasThal.
This way I could add some tags.
Thank you !

done


Components overhaul

Connected Components now use a work queue and no longer produce stack overflow.

Currently collapsing the hypergraph results in a segmentation fault, need to handle that later.


Sidebar to big

The sidebar wasn't suppose to be this big it should look more like this https://imgur.com/a/vMgw6C1

I will work on this

@JohJohan Great :)

Great work :)


Why here use "caffe" but not "tf"?

https://github.com/keras-team/keras/blob/d8fcb9d4d4dad45080ecfdd575483653028f8eda/keras/applications/vgg19.py#L237
I'm using tf.keras.applications.vgg19.VGG19 for transfer leraning, following the guides in tf docs and using tf.keras.applications.vgg19.preprocess_input to deal with inputs. But I'm very confused with the above source code, i.e., why here set mode as "caffee" but not "tf"?

@Zhaopudark,
Here mode caffe: will convert the images from RGB to BGR, then will zero-center each color channel with respect to the ImageNet dataset, without scaling
Where as  mode tf will scale pixels between -1 and 1, sample-wise.

@chunduriv Thank you!
I think this answer eliminate my confusion very well in detail.


Azure Data Studio (Not Responding) after changing Results Grid Font Size

Azure Data Studio Version:
Version: 1.19.0 (user setup)
Commit: 4095037f2578c23033867e611e82c13de114ca5a
Date: 2020-06-11T22:06:05.429Z
VS Code: 1.46.0
Electron: 7.2.4
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Windows_NT x64 10.0.19041

Steps to Reproduce:

Create a simple new notebook with one Code cell

SELECT TOP 10 * FROM sys.databases
SELECT TOP 10 * FROM sys.objects

Run and these should give 2 resultsets.

Open Settings, and search for grid:

Save the Settings.

Go back to the notebook and try to rerun the Code cell.

You should see a Not Responding error like this:

When Azure Data Studio eventually becomes responsive, continuing to navigate other tabs or execute cell etc, will start the Not Responding loop like this.
My current workaround is to kill and restart Azure Data Studio.

This issue seems to repro-able when Settings is opened. i.e. I've tested it with SQL File. It's not as severe, but having notebook file opened with Settings are the reliable combo to repro this.


Error when load Facebook login

Hi! I am using this library for the first time and I try to migrate some projects also.
I am trying make authentication with Facebook and email, email works fine but when I try Facebook the window only show "Loading..."
5-20 02:40:45.759 13896-13925/io.atoom E/DynamiteModule: Failed to load module descriptor class: Didn't find class "com.google.android.gms.dynamite.descriptors.com.google.firebase.auth.ModuleDescriptor" on path: DexPathList[[zip file "/data/app/io.atoom-1/base.apk"],nativeLibraryDirectories=[/vendor/lib, /system/lib]] 05-20 02:40:45.759 13896-13896/io.atoom D/FirebaseApp: Initialized class com.google.firebase.auth.FirebaseAuth. 05-20 02:40:45.759 13896-13925/io.atoom V/GoogleSignatureVerifier: com.google.android.gms signature not valid.  Found: XXXXXXXXXXXXXX 05-20 02:40:45.769 13896-13896/io.atoom D/FirebaseApp: Initialized class com.google.firebase.iid.FirebaseInstanceId. 05-20 02:40:45.769 13896-13925/io.atoom E/DynamiteModule: Failed to load module descriptor class: Didn't find class "com.google.android.gms.dynamite.descriptors.com.google.firebase.auth.ModuleDescriptor" on path: DexPathList[[zip file "/data/app/io.atoom-1/base.apk"],nativeLibraryDirectories=[/vendor/lib, /system/lib]]
And thanks for this!

I think this is a duplicate of #121 - the loading dialog is not being dismissed when authentication with an identity provider fails:
https://github.com/firebase/FirebaseUI-Android/blob/master/auth/src/main/java/com/firebase/ui/auth/ui/idp/AuthMethodPickerActivity.java#L226
You can log the failure details / set a breakpoint at the above line of code to see what exactly has happened. We've got a fix coming for failing to dismiss the loading dialog (being tracked in #124), and we are adding additional logging throughout the failure cases to help with future diagnosis. Sorry for the inconvenience!


Enable HTTP/3 in all tests

Title: Enable HTTP/3 in all tests
Description:
Right now HTTP/3 is disabled for quite a few tests. We're fixing that with PR #26197 but this issue exists to track future work in this domain.

/assign asingh-g

/assign @asingh-g


lax.custom_linear_solve appears to give incorrect gradients for complex dtypes

I'm not quite sure why, but a simple unit test with jtu.check_grads fails with even a generous tolerance.

Actually, custom_linear_solve was correct -- this was purely a bug in my test: https://github.com/google/jax/pull/2655


Sercomm FG1000B.11 // Glasfaser Modem 2 Config?

Hello iam new here so please have mercy.
i got a Glasfaser Modem 2 and whant to use it as my ONT. I managed to get on the ONT with telnet (not permanent only via browser console) shown on the wiki.
But every thing else is a show stopper to me. I dont understand the wiki :)
What i want to do is swap my  "HG8010Hv6" with the "Glasfaser Modem2".
I got my PW and SN from my HG8010Hv6. But how do i put them on the "NEW ONT".
i know i should work with telnet because there is no GUI. -> Only for Germany LWL Telekom Users.
Thank you

I could not find out the Problem every time is try the new Ont i get State: 03 Seriel Number.
is there no one who can help me? Thank you

You need to figure out the kind of authentication and checks done by your OLT. Feel free to join the telegram group and we can discuss this there

Did you ever find the problem?
I can change the serial number but only with the gponctl command.
Connection works. But after a reboot the original serial number is active. even after changing the gpon_sn file.

No did not find the Problem...
But my ISP changed my ONT to the "same Modell" so i know now that i only need the PW &SN but it still does not work.


Provide Community Meeting Invite and Notes

Is your feature request related to a problem? Please describe.
We do not yet have a doc to share the community meeting info,  meeting notes, and meeting recordings on this GitHub.
Describe the solution you'd like
Provide a community meeting doc to include the above info.
Describe alternatives you've considered
Or provide the above info in the wiki page or documentation website.

the meeting info is here

@husky-parul would you mind posting the meeting info on kepler-doc?

This could go under the new Project section on the docs website 😄


Dark mode

How do I enable dark mode?

The theme is determined based on your browser's default theme, so you'll
need to change your browser settings.
On Fri, 9 Sep 2022, 8:58 AM thezwap, @.***> wrote:

How do I enable dark mode?
—
Reply to this email directly, view it on GitHub
https://github.com/kingsthwaiteJ/jsonld-viewer-chrome-extension/issues/1,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANOOVODX3MCWDGF4WIG35VDV5JHOJANCNFSM6AAAAAAQIDWWRM
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Errors when calling the CLI on Colab

I have tried to run the CLI from within Colab (with GPU activated), as follows:
%pip install big-sleep

!dream "a pyramid made of ice"

First, CUDA is version 10.1 on Colab, so I encounter an error regarding PyTorch version, which can be fixed with:
%pip install torch==1.7.1+cu101 torchvision==0.8.2+cu101 torchaudio==0.7.2 \
 -f https://download.pytorch.org/whl/torch_stable.html

Second, I encounter an other error:
100%|██████████████████████| 353976522/353976522 [00:02<00:00, 135419788.41it/s]
Traceback (most recent call last):
[...]
  File "/usr/local/lib/python3.6/dist-packages/big_sleep/big_sleep.py", line 82, in __init__
    assert image_size in (128, 256, 512), 'image size must be one of 128, 256, or 512'
AssertionError: image size must be one of 128, 256, or 512

Weirdly, the error goes away after I run everything again.
100% 235773527/235773527 [00:03<00:00, 70344158.48B/s]
100% 800/800 [00:00<00:00, 1094761.24B/s]
Imagining "a pyramid made of ice" from the depths of my weights...
Warning: unknown mime-type for "./" -- using "application/octet-stream"
Error: no "view" mailcap rules found for type "application/octet-stream"
/usr/bin/xdg-open: 851: /usr/bin/xdg-open: www-browser: not found
/usr/bin/xdg-open: 851: /usr/bin/xdg-open: links2: not found
/usr/bin/xdg-open: 851: /usr/bin/xdg-open: elinks: not found
/usr/bin/xdg-open: 851: /usr/bin/xdg-open: links: not found
/usr/bin/xdg-open: 851: /usr/bin/xdg-open: lynx: not found
/usr/bin/xdg-open: 851: /usr/bin/xdg-open: w3m: not found
xdg-open: no method available for opening './'
epochs:   0% 0/20 [00:00<?, ?it/s]
iteration:   0% 0/1050 [00:00<?, ?it/s]
loss: -8.45:   0% 0/1050 [00:16<?, ?it/s]
loss: -8.45:   0% 1/1050 [00:16<4:48:39, 16.51s/it]
[...]

Weirdly, the error goes away after I run everything again.
100% 235773527/235773527 [00:03<00:00, 70344158.48B/s]
100% 800/800 [00:00<00:00, 1094761.24B/s]
Imagining "a pyramid made of ice" from the depths of my weights...
Warning: unknown mime-type for "./" -- using "application/octet-stream"
Error: no "view" mailcap rules found for type "application/octet-stream"
/usr/bin/xdg-open: 851: /usr/bin/xdg-open: www-browser: not found
/usr/bin/xdg-open: 851: /usr/bin/xdg-open: links2: not found
/usr/bin/xdg-open: 851: /usr/bin/xdg-open: elinks: not found
/usr/bin/xdg-open: 851: /usr/bin/xdg-open: links: not found
/usr/bin/xdg-open: 851: /usr/bin/xdg-open: lynx: not found
/usr/bin/xdg-open: 851: /usr/bin/xdg-open: w3m: not found
xdg-open: no method available for opening './'
epochs:   0% 0/20 [00:00<?, ?it/s]
iteration:   0% 0/1050 [00:00<?, ?it/s]
loss: -8.45:   0% 0/1050 [00:16<?, ?it/s]
loss: -8.45:   0% 1/1050 [00:16<4:48:39, 16.51s/it]
[...]


Be more strict with deserialization

Codecov Report

Merging #21 into master will increase coverage by 0.03%.
The diff coverage is 98.11%.

@@            Coverage Diff             @@
##           master      #21      +/-   ##
==========================================
+ Coverage   99.04%   99.07%   +0.03%
==========================================
  Files          42       46       +4
  Lines         525      543      +18
  Branches       50       54       +4
==========================================
+ Hits          520      538      +18
  Partials        5        5

Impacted Files
Coverage Δ

toofz.NecroDancer.Leaderboards/toofz/PlayerDTO.cs
100% <ø> (ø)
:arrow_up:

...croDancer.Leaderboards/toofz/HttpErrorException.cs
100% <ø> (ø)
:arrow_up:

...bApi/ISteamRemoteStorage/UgcFileDetailsEnvelope.cs
100% <ø> (ø)
:arrow_up:

...derboards/Steam/WebApi/ISteamUser/PlayerSummary.cs
100% <ø> (ø)
:arrow_up:

...oofz.NecroDancer.Leaderboards/LeaderboardHeader.cs
100% <ø> (ø)
:arrow_up:

...Steam/WebApi/ISteamRemoteStorage/UgcFileDetails.cs
100% <ø> (ø)
:arrow_up:

...Steam/WebApi/ISteamUser/PlayerSummariesEnvelope.cs
100% <ø> (ø)
:arrow_up:

toofz.NecroDancer.Leaderboards/toofz/ReplayDTO.cs
100% <ø> (ø)
:arrow_up:

...ofz.NecroDancer.Leaderboards/toofz/BulkStoreDTO.cs
100% <ø> (ø)
:arrow_up:

...z.NecroDancer.Leaderboards/toofz/ToofzApiClient.cs
100% <100%> (ø)
:arrow_up:

... and 19 more

Continue to review full report at Codecov.

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


cells: rename functional module

Workaround #24
Using:
sed -i -e 's/module \(.*\)(/module \1_func(/' cells/*/*.functional.pp.v

Regenerated the library file with open_pdks https://github.com/RTimothyEdwards/open_pdks/blob/master/common/foundry_install.py:
python3 common/foundry_install.py -source ~/src/github.com/google/globalfoundries-pdk-libs-gf180mcu_fd_sc_mcu7t5v0/ \
                                  -target /tmp/out \
                                  -library digital gf180mcu_fd_sc_mcu7t5v0 \
                                  -verilog 'cells/*/*.v' compile-only

for testing purpose: gf180mcu_fd_sc_mcu7t5v0.zip

Thanks @proppy!
Still getting some errors, even with the modified file:
/home/uri/p/gf180-game-of-life-cell/test/gf180mcu_fd_sc_mcu7t5v0.v:14585: error: Unknown module type: UDP_GF018hv5v_mcu_sc7_TT_5P0V_25C_verilog_pg_MGM_HN_IQ_FF_UDP
2 error(s) during elaboration.
*** These modules were missing:
        UDP_GF018hv5v_mcu_sc7_TT_5P0V_25C_verilog_pg_MGM_HN_IQ_FF_UDP referenced 1 times.

@urish - I assume it is related to https://github.com/google/globalfoundries-pdk-libs-gf180mcu_fd_sc_mcu7t5v0/pull/27 ?

TBH I've no idea - I'm just trying to get to a working gate level simulation of my gf180 submission, and thanks you @proppy slowly figuring it out

@urish - Maybe your test could be added to the repository as a way to verify that the Verilog is working?

To which repository, this one?

Maybe? We are making up this as we go.

The test is here: https://github.com/urish/gf180-game-of-life-cell/tree/gate-level/test
Right now, it uses @proppy's generated verilog model. If you have cocotb installed, you can run it by checking out the above repo (gate-level branch), and running:
make -C test GATES=yes

Feel free to adopt the test to this repo

@mithro :  The answer should be immediately obvious from the report:
*** These modules were missing:
        UDP_GF018hv5v_mcu_sc7_TT_5P0V_25C_verilog_pg_MGM_HN_IQ_FF_UDP referenced 1 times.

The primitive added in PR #27 is named gf180mcu_fd_sc_mcu7t5v0__udp_hn_iq_ff, not UDP_GF018hv5v_mcu_sc7_TT_5P0V_25C_verilog_pg_MGM_HN_IQ_FF_UDP.  So you added the UDP primitives, and changed their names according to your special syntax rules, but didn't bother to fix the places where those UDP primitives are called from other verilog modules.

Thanks @RTimothyEdwards. I renamed gf180mcu_fd_sc_mcu7t5v0__udp_hn_iq_ff to UDP_GF018hv5v_mcu_sc7_TT_5P0V_25C_verilog_pg_MGM_HN_IQ_FF_UDP, and finally the GL test passes! 🥳

The primitive added in PR #27 is named gf180mcu_fd_sc_mcu7t5v0__udp_hn_iq_ff, not UDP_GF018hv5v_mcu_sc7_TT_5P0V_25C_verilog_pg_MGM_HN_IQ_FF_UDP. So you added the UDP primitives, and changed their names according to your special syntax rules, but didn't bother to fix the places where those UDP primitives are called from other verilog modules.

It looks like all the FF latch and clock gating cells have this issue.


SW_MAXIMIZE and SW_SHOWMAXIMIZED have same value 3, which to use.

Link to article:
https://docs.microsoft.com/en-us/windows/desktop/api/winuser/nf-winuser-showwindow
Problem:
In the documentation on the ShowWindow function, there is a list of values for the int parameter nCmdShow.
However, there are 2 entries with the same 3 value, SW_MAXIMIZE and SW_SHOWMAXIMIZED.
Which one is it?
Thank you.
MSDN thread: https://social.msdn.microsoft.com/Forums/windowsdesktop/en-US/d2a8a7cf-1a11-4608-848e-fa447195578a/showwindow-ncmdshow-parameter-several-values?forum=windowsgeneraldevelopmentissues

@GrantMeStrength Can you please take a look at this?

This is correct - both values are valid.

I think the problem the customer is reporting here is that both SW_SHOWMAXIMIZED and SW_MAXIMIZE show the value 3. One should be different no?

The current contents are confusing because there are two names (SW_MAXIMIZE, SW_SHOWMAXIMIZED) mapping to the same value.
One might say: They're just alternate names for the same behavior.
However, if that's true, then the description of the behavior of the two entries should be the same. They're not. One says, "Maximizes the specified window.", and the other says, "Activates the window and displays it as a maximized window."
Thus, one claims to activate the window, while the other does not. This is bad documentation since they have the same value, thus the same behavior. (thus one of the descriptions is inaccurate, or at least incomplete)


Desktop client updates - Linux

> "Teams desktop client updates are released every two weeks..."
This does not appear to be the case for the Linux desktop client, which has not been updated for nearly two months now - putting it 6 releases and many features behind the Windows/Web clients.  Is there a release schedule for Linux, or is it just packaged ad-hoc?

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

ID: f6da28da-eb7b-d3ae-4c2f-8f87803b1830
Version Independent ID: 8c42d7a2-80c6-86dd-347a-ed6d8d909c41
Content: Teams updates - Microsoft Teams
Content Source: Teams/teams-client-update.md
Service: msteams
GitHub Login: @LolaJacobsen
Microsoft Alias: lolaj

@3vi1 Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

@3vi1 thank you for reaching out and helping improve MS Docs. Please see this link - https://support.microsoft.com/office/update-microsoft-teams-535a8e4b-45f0-4f6c-8b3d-91bca7a51db1

Please see this link - https://support.microsoft.com/office/update-microsoft-teams-535a8e4b-45f0-4f6c-8b3d-91bca7a51db1

@lindspea This link is not working for me ("Sorry, the page you’re looking for can’t be found.")

@3vi1 Thank you for submitting feedback. We understand that this issue has been resolved.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.


Testing remote access after overriding remote

#72  👈
main

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

Join @CatChen and the rest of your teammates on  Graphite


refactor(trie): move database implementations of trie cursors to reth-trie-db crate

About
This PR is about moving trait impls from reth-trie crate to their designated place.
Motivation
Expanding on #9282, for simplicity sake the reth-trie trait implementations using reth-db  were kept in an inappropriate place.
Goals

Move reth-db impls of reth-trie traits to reth-trie-db
Update all usages

Solution

The impls are moved to reth-trie-db
The tests dependent on those impls were moved to integration tests of reth-trie-db
All usages updated

Additional notes
After merging this and either #9970 or #9987 it should allow us for the removal of reth-db and reth-db-api dependencies from reth-trie crate.

oh hell yeah


(refactor) (olap) (refactor) optimize SegmentIterator::next_batch()

Proposed changes

refactor BitmapRangeIterator, avoid new/delete, check _eof in loop while _read_next_batch to simply outloop's check
modify evaluate_vec, avoid set and check new_flags out of block_column_predicate->evaluate_vec()
replace _block_rowids.reserve() with _block_rowids.resize(). access vector's data with subscript is illegal after reverving, when via at() or operator[] access vector's data may assert failed, the behavior is depend on STL's implements
optimize insert_default_value_res_column and insert_res_column by operating PODArray data directly
using empty() interface instead of 'size() > 0', because empty() is fittable, sometimes faster, such as for container list, refer to Effective C++
some other small changes

Issue Number: close #xxx
Problem Summary:
Describe the overview of changes.
Checklist(Required)

Does it affect the original behavior: (Yes/No/I Don't know)
Has unit tests been added: (Yes/No/No Need)
Has document been added or modified: (Yes/No/No Need)
Does it need to update dependencies: (Yes/No)
Are there any changes that cannot be rolled back: (Yes/No)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

split this PR to two separate PR


Request new Trove Classifier for pytest plugins

Currently pytest contains dozens of plugins on PyPI (100+), as a simple search for "pytest" shows.
We would like to make it easier for users to find these plugins, as some plugins are left out for not following this naming scheme (e.g. backports.unittest_mock, jaraco.mongodb), so we request to include a new trove classifier:
Framework :: Pytest :: Plugin

Pytest's maintainers (myself included) demonstrated interest of supporting this officially by advertising it in the documentation and the plugincompat service (which automatically finds plugins and check compatibility against different pytest versions).
(I created a duplicate of this issue at pypa/pypi as I'm not sure which repository handles the classifiers).

I have also requested new classifiers without any reply. I submitted two requests to pypa/pypi project on BitBucket over two years ago (see here), and then a bit later sent emails to catalog-sig and distutils-sig. No reply whatsoever.
I'm now sending an email to pypa-dev that is owner of this organization. I'll mention this issue in the email too.

Great, pypa-dev group shown on https://github.com/pypa is closed and my email bounced. This is getting annoying.

Yeah, I'm almost giving up on the whole idea... :worried:
Thanks for the note though

Here's the email I sent (needed to join the list first):
https://groups.google.com/forum/#!topic/pypa-dev/JRv7BXsNX_w
Hopefully we get a reply. Our request even happen to be very much related.

Yes, fingers crossed! :smile:

Btw, I appreciate you taking the time to ping me back on this issue. Thanks!

@nicoddemus I just brought this to the table again:
https://groups.google.com/d/msg/pypa-dev/JRv7BXsNX_w/AUfbrnvfBQAJ
Looks like @pekkaklarck's request has been already processed successfully, but yours has been overseen.

Thanks for letting me know! :smile:

@nicoddemus We have now a dedicated Trove Classifier:
Framework :: Pytest

This already shows up in the Trove Classifiers List.
I guess as soon as this is mentioned somewhere in the pytest documentation this issue can be closed.

@benjixx thanks for letting me know! I already sent an announcement email with this change, and will take care of updating the documentation myself later. :grin:


[Request] "axis" on responsive

I'm using this slider in the vertical axis for large screen, but I need to slide from left to right or right to left in mobile. So, please add this feature.

Sorry for the late response.
Changing axis will totally rewrite the slider. So I don't suggest to put this option into responsive which will make the slider even more complicated. You can manually destroy() and rebuild it again when reach the breakpoint.

Same isue here, will try suggestion of @ganlanyuan

Hi, destroy() not working on tiny-slider 2.8.6 on Chrome 69 - macos 10.14
throw error:
function getSlidePositions () {
	slidePositions = [0];

	var attr = horizontal ? 'left' : 'top',
		first = slideItems[0].getBoundingClientRect()[attr], // Uncaught TypeError: Cannot read property '0' of null
		position;

	for (var i = 1; i < slideCountNew; i++) {
		position = slideItems[i].getBoundingClientRect()[attr];
		slidePositions.push(position - first);
	}
}

My slider
var thumb_carousel = tns({
	loop: false,
	container: '#thumbnail-images',
	gutter: 10,
	items: 5,
	mouseDrag: true,
	nav: false,
	controls: false,
	axis: 'vertical'
});

if ( window.matchMedia( '( max-width: 767px )' ).matches ) {
	thumb_carousel.destroy();

	/*thumb_carousel = thumb_carousel.rebuild({
		axis: 'horizontal'
	});*/
}

// How i can rebuild with other option like comment above??

Issue still here...

Need solution for this issue...

@des-tes @notrealdev You can do something like this :
let tnsOptions = {
	loop: false,
	container: '#thumbnail-images',
	gutter: 10,
	items: 5,
	mouseDrag: true,
	nav: false,
	controls: false,
	axis: 'vertical'
};
if(document.getElementsById('thumbnail-images').length > 0 && tns !== undefined) {
    let thumb_carousel = undefined,
        thumbsOpts = {};
    let wait = null;

    function iniThumbs(){
        if(wait !== null) clearTimeout(wait);
        wait = setTimeout(function(){
            if(typeof thumb_carousel === 'object') thumb_carousel.destroy();
            if ( window.matchMedia( '( min-width: 768px )' ).matches ) {
		// Set options for screen wider than 768px width
                thumbsOpts = {}
            }
            else {
		// Set options for screen smaller than 768px width
                thumbsOpts = {};
            }
            thumb_carousel = tns(Object.assign({},tnsOptions,thumbsOpts));
        },250);
    }
    // Reset on orientation change for mobile
    window.addEventListener('orientationchange',iniThumbs);
    // Reset on resize
    window.addEventListener('resize',iniThumbs);
    // Initialize at load
    iniThumbs();
}

I've noticed that the error appears when you destroy then rebuild the slider too quickly (that's the reason for the time out).
Also, in the doc says

slider = slider.rebuild();
// this method returns a new slider Object with the same options with the original slider
Tht's why you have to reinit completely the slider after destroying it if you want to change the settings like the axis.

@ganlanyuan However, I'm of those who'd really like the axis option as a responsive feature :)

I can't make axis responsive, the solution still destroy the object and them rebuild it?
slider.destroy();
var slider = tns({
container : '#my-slider',
items : 7,
gutter : 5,
axis : "vertical",
mouseDrag : "mouseDrag",
useLocalStorage : true,
controlsContainer : '#controls',
prevButton : '.previous',
nextButton : '.next',
nav:false
});
			slider = slider.rebuild();


gitlab-release-cli: 0.19.0 -> 0.20.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/gitlab-release-cli/versions.
meta.description for gitlab-release-cli is: Toolset to create, retrieve and update releases on GitLab
meta.homepage for gitlab-release-cli is: https://gitlab.com/gitlab-org/release-cli
Updates performed

Golang update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.20.0 in filename of file in /nix/store/w5qwxdq6rlq5iqzrz4xi97p2syij4rxj-gitlab-release-cli-0.20.0

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

gitlab-release-cli

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/w5qwxdq6rlq5iqzrz4xi97p2syij4rxj-gitlab-release-cli-0.20.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

Or:
nix build github:r-ryantm/nixpkgs/e8151e1869500d65d3a65a6aaaa793e17a8b3fa9#gitlab-release-cli

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/w5qwxdq6rlq5iqzrz4xi97p2syij4rxj-gitlab-release-cli-0.20.0
ls -la /nix/store/w5qwxdq6rlq5iqzrz4xi97p2syij4rxj-gitlab-release-cli-0.20.0/bin

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

x86_64-linux

:white_check_mark: 1 package built:

gitlab-release-cli

Maintainer pings
cc @kilimnik for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

@NixOS/nixpkgs-merge-bot merge


Add support for a ready endpoint, based upon max_inflight and change timers to use a Context with a deadline.

Description

Add support for a ready endpoint, based upon max_inflight and change timers to use a Context with a deadline.
Motivation and Context

Ready endpoint introduced

Documentation updated for usage

Logic for timing and stopping requests that run for
too long has been switched over to use a context.

Currently, when a function is overloaded with too many
requests vs max_inflight i.e. 10 inflight with a limit of
2, then a 429 / retry message is returned to the caller.

By implementing a ready endpoint, the readiness check for an
individual function can be implemented, so that Kubernetes removes
it from the pool of endpoints when it exceeds its max_inflight.
It will return again when it's once again deemed to be ready.
Tested with a large amount of requests through the queue-
worker, pods were removed from the pool of endpoints as
expected.

The documentation was reorganise and updated for new options
and other were removed which were not relevant.

As per https://github.com/openfaas/of-watchdog/commit/47d5c31a5a182999ea3f904d187355023ad75955, there
were other modes which had an issue that could be resolved
by using a Context with a deadline instead of a timer.

How Has This Been Tested?

Tested with unit tests on the new change to the concurrency limiter to expose a Met() method used in the new Ready endpoint.
No harm to existing functions which won't use the /_/ready path.
Then tested with some load with the queue-worker. I observed Pods being removed from the readiness count and coming back online again as expected.
This testing revealed a bug in the scale from zero logic of the gateway which was fixed here: https://github.com/openfaas/faas/pull/1753
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I've read the CONTRIBUTION guide
[x] I have signed-off my commits with git commit -s
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

Thanks for the review @LucasRoesler

LGTM


Add arial-label to date in sample

Description
At sample site adding arial-label like yy/mm/dd to date will be more good for accessibility.
References

just add aria-label for basic example https://github.com/veccu/react-calendar/pull/88


Fix unclaimall not removing the player from claim queues

This is reverting a change that was made because if a mods did !unclaimall it would unclaim modules they didn't have a claim on. So every single module would be unclaimed.


class loading error when enchantment descriptions mod added

Describe the Bug
I added the file EnchantmentDescriptions-Forge-1.17.1-8.17.1.jar to my current modpack which had previously loaded with no errors and it had class loading errors on startup - poss a problem with compatibility with Flywheel mod - I've also logged the issue on their github
Reproduction Steps
1.add EnchantmentDescriptions-Forge-1.17.1-8.17.1.jar along with flywheel to mods folder
2.load up minecraft
3. get classloading error
...
crash-2021-12-22_20.51.29-fml.txt
Expected Result
I hoped to open Minecraft and play ;)
Screenshots and Videos
No response
Crash Report or Log
https://pastebin.com/zGEwCd1p
Operating System
Windows 10
What is your GPU?
AMD Radeon HD 7800 Series
Mod Version
0.3.0
Minecraft Version
1.17.1
Forge Version
37.1.0
Other Mods
see crashlog
Additional Context
No response

Crash log has nothing to do with Flywheel. Seems like Enchantment Descriptions was compiled against Java 17 instead of 16.
https://github.com/Darkhax-Minecraft/Enchantment-Descriptions/issues/92


Divide this library into two

Separate this library into:

ArchitectureKit
UIKit

FuntastyKit is currently only in maintenance mode. I am closing this since it is breaking change nad not worth the effort.


Missing atomic delivery of current state with subscription

There are some use cases where the observer needs to get the current state, as well as subscribe to changes.
As this are two independent calls to the provider, there's a race condition where, depending on the order of the calls, the observer may either miss notifications, or get notifications which are already outdated regarding the "current state" he fetched.
The observer pattern, as provided by the [System.IObservable<T>](https://docs.microsoft.com/en-us/dotnet/api/system.iobservable-1) class, does not provide a nice solution for this problem.
One solution is to provide an alternative Subscribe()-Method which returns a tuple containing both the current state and the IDisposable used to unsubscribe.
Maybe some hints to this or other solutions to that problem could be added to the article (or the "best practices" article next to it)?

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

ID: 5e8b6ec6-3773-879a-00e9-4bf113b9f362
Version Independent ID: 5c46289d-8473-623f-eb42-8bda01c30c63
Content: Observer Design Pattern
Content Source: docs/standard/events/observer-design-pattern.md
Product: dotnet-fundamentals
GitHub Login: @IEvangelist
Microsoft Alias: dapine

Hmm. As the IObserver<T> actually stays the same, and the role of IObservable<T> is a minor one in the pattern, I'm not sure whether it really is an API change, or actually just about documenting an alternative way of offering the pattern. I'll ask in the https://gitter.im/dotnet/coreclr gitter channel whether this is worth a runtime issue.

I'd expect any state to be broadcast to the IObserver<T>, not returned from the IObservable<T>.  Most publishers have no concept of "current state", they are "cold" and don't start emitting values until the subscriber connects.  In the case of a "hot" publisher, which it sounds like you are working with, I would expect the publisher to emit the last signal to the observer on subscription.  If the publisher itself doesn't support that there should be a reactive operator that will buffer the last received signal and broadcast that to observers as they subscribe.  IMO making a change to flow signals outside of that would break the reactive streaming pattern.

Ok, I'm somehow convinced now that the current state should be returned via event to the IObserver<T>.


Add path details and elevation widget

Here I added the heightgraph we use for GH maps currently. It's a modified version of the original heightgraph that is currently living here: https://github.com/easbar/Leaflet.Heightgraph and can be used with both Leaflet and Mapbox (in contrast to the original which was only usable with Leaflet).
Fixes #12
todo:

[x] svg icons from heightgraph are not loaded (need to adjust webpack config?!)
[x] hide widget before path is calculated
[ ] test (or hide?) on mobile

test (or hide?) on mobile

Mobile for this widget is tricky (e.g. how to do this "hover mouse" functionality?) and so I would just hide it for now.

Mobile for this widget is tricky (e.g. how to do this "hover mouse" functionality?) and so I would just hide it for now.

Moving the mouse over the diagram to show the little flag at the corresponding route position indeed seems tricky for touch devices, but at least showing the elevation and path details would be useful I think. But maybe we can do this later :)

@Janekdererste Can you take a look at this PR so we can merge it? And maybe you can help me to hide the widget for mobile? The css I pasted above does it, but I could not make it work with a @media query.

Another option that we currently do is to "close" the widget on mobile per default:

but maybe this takes too much space?

but maybe this takes too much space?

Mobile could also be a tablet device, so it should depend on screen size. Also I think seeing the elevation details is quite useful especially for outdoor profiles that might be used from mobile devices primarily. I'd say let's hide it for now and later think about options to show it on mobile devices as well.

test (or hide?) on mobile

Mobile for this widget is tricky (e.g. how to do this "hover mouse" functionality?) and so I would just hide it for now.

The Covid tracker on https://zeit.de has the exact same "hover" effect when the finger is moved on the diagram

Thanks for the review and code suggestions! Let's postpone mobile-related questions to #44.


2.x refactor tests

Ticket: #1793
Issue
We're moving to a new API that will restrict direct instantiation of core Timber objects (Post, User, Menu, etc.) in favor of corresponding Timber::get_* calls.
Solution
This refactors away most direct instantiations within the tests. Some refactors we'll need to do eventually are impossible right now, since the Timber class lacks some get_* methods we'll need to replace the calls with. I don't want to add any code just yet, so I'm just taking care of the low hanging fruit right now.
Impact
None.
Usage Changes
No changes to production code in this PR; only tests.
Considerations
???
Testing
You know I'm all about those tests tho.

@gchtr any more feedback before I merge this in?

@gchtr are you sure you meant to close this issue?

Nope, I didn’t. For the documentation in #2073, I first created a 2.x-api branch, that I then renamed to 2.x-docs-api. I saw yesterday that the 2.x-api branch still existed and assumed we didn’t use it, so I deleted it. Except this pull request is made against the 2.x-api branch, so we still need it 🙈. It’s back now 😅. Sorry!


Parsing style tag in HTML document that includes comments

How can I parse the following HTML and access the comment next to each CSS selector and know which selector the comment belongs too? Also, if there are comments on their own lines within the style tag, I'd like to be able to access those as well.
`

....

`

Comments are not part of the CSSOM. Once they could be included, but the solution was not elegant.
They may come back with AngleSharp.Css eventually, but the final decision on this has not been made.


BZ#145848583 - Introduce service tag filtering

Update, this pr is done, but there are two api bugs that I would like to be sorted prior to merging this in (detailed below)
Why are we wipping? present issues (and prs solving when they are up):

polling results in a call back sorta effect whenever filter or sorting is applied, need to rework this immediately
We stop polling whenever a new request for resources has been made, ie a filter or sorting event (or the component is destroyed)
After successfully grabbing these new resources, we begin polling on the successful call
This avoids the previous issue of stale polling
api anding of multiple filters does not quite work as expected, issue has been identified and is being looked at (not an sui issue)
https://bugzilla.redhat.com/show_bug.cgi?id=1470251
api throws error when changing sort direction of filtered tags
https://bugzilla.redhat.com/show_bug.cgi?id=1470260

No more polling artifact <3 <3 <3

What filtering by tag looks like, also what this polling artifact bug looks like :(

@chriskacerguis This pr is done, but there are two api bugs that I would like to be sorted prior to merging this in (detailed below)

@allenbw Can you please make BZs for those issues and put the links in the PR comments?

@chriskacerguis yeah! totally did, travis passed, but its not being reported, any idea whats up?

Looks like it passed: https://travis-ci.org/ManageIQ/manageiq-ui-service/builds/252562261
Do you want me to merge or wait for those bugs to pass?

@chriskacerguis thats a gooooooooooood question.... i know one is on dev presently, the other should be soonish... without them filtering by tags still works... its just breakable :P
As there is no more work to do here and bz are made... i think merging is ok to do

@ManageIQ/core-ux for your review.
/cc @Loicavenel

I anticipate one feedback might be "ew, those tags are long and silly" buttttttt this the pf-ang lib doesn't support an tag value alias, there might be another way to manage this, i have an idea, working it, but the need for the ability to display a tag alias (rather than the search value) has also been expressed here https://bugzilla.redhat.com/show_bug.cgi?id=1470868

Yeah totally possible, for filter thing to support alias! Thinking of doing that work in separate pr, aligned to the ref... Thoughts ?

@AllenBW since we are still waiting for UX review, I would say let's just add it to this one.

I will leave Serena reviewing this one. But, I see 2 issues:

this list can really long..
"Managed/owner/prod_ops"  is not really friendly. Each Category has a description, and each value has a description, do we retrieve it?  It will be a much better experience to have the description including in when we display Tags... If an API has to be updates, let's do it.
I think we should also offer a menu in 2 phases where user see first the categories and then when he is selecting a category, UI list all tags in the category...

@AllenBW definitely think that this is a useful capability.  Colleen and I worked on this design, and would like this to be implemented:

Upon selecting to filter by tag [1], then there will be two components added:
Filter by Category... [2] and Filter by Value... [3]
Both of those components should use the typeahead dialog select, as you did here:  https://github.com/ManageIQ/manageiq-ui-service/pull/821
When the value [3] is designated, a filter chip [4] should be added in the format:
Tag: :
Does this seem achievable?  Let me know your thoughts @AllenBW @Loicavenel !

@serenamarie125 looks great... I just want to look if we can get the Description and not the value.
Attached some screenshots showing what we see when we select Tags in VM

@serenamarie125 @Loicavenel not impossible (would totally love to as its far more usable), but angular-patternfly presently does not have provisions to support a multi-dropdown filter (what we have to work with presently: http://www.patternfly.org/angular-patternfly/#/api/patternfly.filters.component:pfFilter)
Thoughts on how to proceed?

Also, I lied above, alias isn't possible with ang-pf, a mild version is possible with some string hackery, but this involves deconstructing valid filter values from api, displaying them, then reconstructing them, undesired. Ideally, would prefer to have an field filterAlias or something here heck even an ignored param that could be passed around filter implementation would be handy

@jeff-phillips-18 just tagged you in the angular-patternfly repo.  You can see the desired design above ... I'm thinking you may have done something similar to this in the past.  If not, I think this would be great to have supported in PF as a generic option, would you agree?  @AllenBW of course contributing to PF is always a great option :)

@serenamarie125 yeah, I don't recall doing this in the past.

Thanks for the supa fast reply @jeff-phillips-18 ❤ 🙇♀ (I'm not confused, its the closet reaction github provides to a sad face)
@serenamarie125 given that this isn't present functionality and will require a nontrivial amounta effort to get in, you down with adding the present functionality as is with the pt to circle back around and "fix" when the desired look/feel exists?

@serenamarie125 would @AllenBW suggestion (https://github.com/ManageIQ/manageiq-ui-service/pull/840#issuecomment-316056592) be ok with you?
/cc @Loicavenel

@AllenBW I think there are usability issues with the current PR, and don't think that it should be added.  Although PF doesn't currently support it, I would think that you could either enhance the component and contribute it back, or just tailor in the SUI and contribute back at a later date?

+1

@serenamarie125 if we're to continue to use patternfly, this component would have to be added to patternfly, not something we can tailor in the sui, I'll keep this pr out here and mark the pt as blocked

@AllenBW i don't think this should block SUI implementing a usable solution.  PatternFly is an OpenSource community, please feel free to contribute what you need to Angular PatternFly.

@serenamarie125 it was my understanding that this pr couldn't go through until it met the mock above?
that can't happen until the additions are made to the repo, that we have to hold off on this pr until those changes are present means this pr is blocked 👍 🌮 😿

@AllenBW @chriskacerguis If this new "UI fonction" is not available today, why can you make it?
Why you guys cannot add it to PartternFly community? then everybody can benefits from it..

Woaaaaaah @Loicavenel I never said we couldn't add it to patternfly, apologies if I wasn't clear, the functionality is not in ang-pf at this very moment, which means this pr is blocked 😺

@AllenBW ok, so we can move on this PR with doing it in Partnerfly or doing it our own way close to Partnerfly and someone will move it to partnerfly later

@Loicavenel sounds good to me!

@chriskacerguis how did this get merged?  Both @Loicavenel and I requested a change

Whoops. My bad. You did. I'll back it out

Thanks @chriskacerguis


error when using telescope as picker

I really like these plugins but when I try to change the default picker to telesope I get an error like below. this error appears when i execute SqlsSwitchConnection
sqls.nvim configuration
on_attach = function(client)
    client.resolved_capabilities.execute_command = true
    require("sqls").setup({ picker = "telescope" })
end,

error message
E5108: Error executing lua ...cker/start/telescope.nvim/lua/telescope/actions/init.lua:29: Key does not exist for 'telescope.actions': get_selected_entry
stack traceback:
        [C]: in function 'error'
        ...cker/start/telescope.nvim/lua/telescope/actions/init.lua:29: in function '__index'
        ...nvim/site/pack/packer/opt/sqls.nvim/lua/sqls/pickers.lua:50: in function 'run_replace_or_original'
        ...packer/start/telescope.nvim/lua/telescope/actions/mt.lua:30: in function 'key_func'
        ...k/packer/start/telescope.nvim/lua/telescope/mappings.lua:235: in function 'execute_keymap'
        [string ":lua"]:1: in main chunk

Thanks for the report. It seems the telescope API changed recently, I updated the picker accordingly.
Note: I would recommend using the default configuration and overriding vim.ui.select() instead.  The pickers option will eventually be deprecated


babel support added

take this as a gift bro. ;)

Codecov Report

Merging #1 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master     #1   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           2      2
  Lines          68     68
=====================================
  Hits           68     68

Continue to review full report at Codecov.

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


Web GUI progress fields

Hello. I've just started using the Web GUI, and I really like it! I'm doing a sync between my local drive and google drive, and the GUI is great for a progress report, except it could be better. stats logs a little more info that would be great to have in the GUI.
Here's what I can see using stats:
Transferred: 1.195G / 62.546 GBytes, 2%, 464.619 kBytes/s, ETA 1d14h27m39s
Checks: 4299 / 4299, 100%
Transferred: 2 / 86, 2%
Elapsed time: 45m1.4s
The Web GUI doesn't show the overall GB to be transferred and the overall progress (2% above), not does it who the overall ETA. Finally, it doesn't show to the total number of files to be transferred (86 above). It would be great to have stats in the dashboard of the web GUI. All the stuff in bold above is not shown in the GUI, but would be great if it were.

Hi, @odinian  Thanks for the suggestions. It would be great to have those displayed in stats.
I am currently running low on bandwidth, would anyone like to take this up?
@ncw can you suggest someone?

hi. Unfortunately, I'm not a programmer, so I don't know how to make this enhancement. It does appear that one may just need to add "totalTransfers" and "totalBytes" to the stats array in GlobalStatus in the RunningJobs.js file. Possible this:
function GlobalStatus({stats}) {
    const {speed, bytes, checks, elapsedTime, deletes, errors, transfers, lastError, totalTransfers, totalBytes} = stats;
    return (
        <Card>
            <CardHeader>Global Stats</CardHeader>
            <CardBody>

                        Bytes Transferred:
                        {formatBytes(bytes)}

                        Total to Transfer:
                        {formatBytes(totalBytes)}

                        Average Speed:
                        {formatBytes(speed)}PS

                        Checks:
                        {checks}

                        Deletes:
                        {deletes}

                        Running since:
                        {secondsToStr(elapsedTime)}

 0 ? "table-danger" : ""}>
                        Errors:
                        {errors}

                        Transfers:
                        {transfers}

                        Total to Transer:
                        {totalTransfers}

                        Last Error:
                        {lastError}

            </CardBody>
            {/*<CardFooter></CardFooter>*/}

        </Card>);
}

For this rclone issue, the following parameters were exposed to the REST API, and can be displayed via an rclone rc core/stats command. It would be nice to display them in the web-UI, especially useful for sync operations (might be put in a dedicated sync tab?)
- totalChecks - total number of checks needed in sync
- totalTransfers - total number of transfers needed in sync
- totalBytes - total number of bytes to transfer in sync
- eta - eta of sync in seconds

rclone rc job/list --rc-addr=:5572 --rc-no-auth
rclone rc core/stats


SM v3 schema[Do not merge]

Replace or add openapi.json in hmr-playground.tsx specURL list to populate Redoc with new schema

not needed in anvu


use phantomjs-prebuilt instead of phantomjs

since phantomjs package is deprecated https://www.npmjs.com/package/phantomjs#deprecated

Coverage remained the same at 100.0% when pulling 5be068e974dc7076b783e7376082ea3234bf86cd on adam187:feature/update-phantom-js into 541a9df8aab2942e853cc21bd2568cf9187bf092 on ismarslomic:master.

Thanks, @adam187 !


TabPage sequence changes while saving desing of form.

When I save the form's design, the sequence of the tab pages gets changed inside InitializeComponent() method of form designer.

you can avoid the problem by saving the tab value .
go to program declare a varibale likje
pulic static int32 tab = 0;
on metrotab selectedtab change event write following
program.tab = metroTabControl1.SelectedIndex;
Now on form load
this.metroTabControl1.SelectedIndex =program.tab;
.//////////////////////////// this will surely solve your problem


will  DINO Code be implemented in mmdetection project?

@SlongLiu @FengLi-ust @IDEACVR  thanks for your greate work.

We will consider your suggestion to make the open-source code more user-friendly. Thank you.

It is recommended to release the original code, for example, yolov5, yolox, which can be more beneficial for developers to carry out in-depth development.

If it is integrated in mmdetection, it may become a simple parameter adjustment work, which is not conducive to development.

It's implemented on version 3 https://github.com/open-mmlab/mmdetection/tree/main/configs/dino


theme files with lowercase

consider to put theme files in lowercase as feature-main-type.hbs,
e.g: register-main-form.hbs instead of RegisterForm.hbs and keep a pattern in this.

deprecated


Don't ignore parse error in partial date time binder

Description
Make sure we fail request for export in date is incorrect.
Related issues
Addresses [#1967].
Testing
Unit tests
FHIR Team Checklist

[x] Update the title of the PR to be succinct and less than 50 characters
[x] Add a milestone to the PR for the sprint that it is merged (i.e. add S47)
[x] Tag the PR with the type of update: Bug, Dependencies, Enhancement, or New-Feature
[x] Tag the PR with Azure API for FHIR if this will release to the managed service
Review squash-merge requirements

Semver Change (docs)
Patch

/azp run

/azp run

/azp run


1.21 update [PR coming]

I was really enjoying this mod in 1.20.1, but I've mostly been playing 1.21. I spent some time yesterday and today updating Blazing Bamboo to 1.21. You can look at the current status here:
fabric-1.21-1.0.1
forge-1.21-1.0.1
neoforge-1.21-1.0.1
I've tested them all on 1.21 and 1.21.1. If nothing major changes, they may be able to load on future 1.21.x releases as well.
I didn't change the 1.0.1 version, because these changes mostly just changed registration and datapack stuff to keep it working on 1.21.
The reason they're not a PR already is that there are still a few remaining problems (and could be more that I haven't found):

[Forge] - The compat recipes are logging errors when the mods they depend on are not installed - even though they have the forge:mod_loaded condition and shouldn't be logging errors

I'm assuming that it's a Forge bug related to the 1.21 datapack changes. In any case, it's not causing any problems other than logging errors, lots of other 1.21 mods will probably run into the same thing, and it's Forge on 1.21 when Neoforge will probably be getting most of the attention/support.

[Neoforge/Forge] - When a blazing bamboo chest raft is spawned, the game crashes with this error: java.lang.IllegalStateException: Entity class net.paddedshaman.blazingbamboo.entity.BBChestRaftEntity has not defined synched data value 15

I'm a bit stuck here. I'm having trouble seeing what could even be setting a synched data value of 15. It seems to be happening down in the Entity class, and the same error happens if I remove defineSynchedData/setVariant from the BBChestRaftEntity class. I'm still thinking that it's most likely an error related to the DATA_ID_TYPE members in Boat and BBChestRaftEntity.
Here's the weirdest thing: I can place a vanilla boat with chest and get the same crash. If I disable the Blazing Bamboo mod and start the game again then I am able to place the vanilla boat with chest. Then if I enable Blazing Bamboo and start Minecraft again with a vanilla boat with chest already in the world I am able to place the blazing bamboo boat with chest.

Once that bug is fixed, I'm planning on submitting PRs for the 3 1.21 branches. Feel free to do whatever you want with this. I've already met my main goal of being able to have blazing bamboo in 1.21 Fabric :)
PS. I thought I liked being able to turn the blazing bamboo into blaze rods and gunpowder. I didn't even realize that there were lava boats, and discovering those made me like the mod a lot more. Thanks for making a fun mod!

The boat crash was fixed, and PRs have been submitted: #3, #4, #5

Wow... This is incredible! I didn't expect anyone to like my mod so much that they'd update it for me!!
Thank you for your hard work on all this. You even gave me support for all three major mod loaders... and extra mod compatibilities!
I'm gonna merge the PRs now, but I haven't finished learning how to mod 1.21 yet. I want to be sure I fully understand the changes before publishing these versions publicly, but when I do, I will give you all the credit!

Okay so I have no idea what I'm doing lmao, I'll have to look more closely at the PRs later because ideally I'd like them to go to a new branch named [modloader]-[mcversion]-[modversion] instead of committing them to the existing 1.20 branches. So like the fabric one should go to a new branch named fabric-1.21-1.0.1. Give me some time to work on this, but I'll get everything up and running!

I don't know if you've figured it out yet. I'm not sure how you could do it through the PR, but you could pull the branches from my fork and then push them in.
Something like this:
git remote add jship git@github.com:jshipley/BlazingBamboo.git
git fetch jship fabric-1.21-1.0.1
git push origin fabric-1.21-1.0.1

(it might have to be git push --set-upstream origin ...)
If you have any questions about what I changed or why, please let me know.
After you have a 1.21 Fabric branch, I can fix/redo the PR for the 1.21 Simple Copper Pipes compat so that it compares against the 1.21 branch instead of the 1.20.1 branch. Hopefully that's just changing a dropdown in the github web interface.

If you'd like, you could also create new 1.21 branches of the 1.20 code, and I can submit/update PRs against those new branches.

Awesome! Looking forward to 1.21.1 neoforge. This mod is great :) Any ETA for when it'll be on CF?


Adding a log entry when an argument has been omitted on a zuul query.

If I have the following query: cibyl --builds it will be equal to: cibyl --tenants --projects --pipelines --jobs --builds
I have added log entries indicating that those arguments have been added implicitly.

Closed in favor of #257.


Go modules

Introduce a go.mod and go.sum file, together with a vendor directory. This uses the standard go mod command and makes building micro a lot easier for Go 1.13 and later. This also makes the git submodules redundant, since go mod now handles dependencies properly.
There are also some changes to the Makefile, for building in a way that is compatible with the standard Go module system.

:+1:

Go modules are now supported.


Add CPHD, SICD, and SIDD to sar:product_type

I have actually two favors to ask:

One new Type called Phase History which is similar to, but not the same as, complex data
Three new sar:product_type:

Compensated Phase History Data (CPHD) | Phase History
Sensor Independent Complex Data (SICD) | Complex
Sensor Independent Detected Data (SIDD) | Amplitude

Each of these is a very highly specified form of standard ("sensor independent") metadata requirements and organization created by the National Geospatial Intelligence Agency. You'll start to notice them in the API and product documentation for every commercial customer with aspirations to sell to them, and it would be wonderful to help encourage these organizations to use STAC.

Please note that the table in https://github.com/stac-extensions/sar#sarproduct_type is not meant to be exhaustive / complete, it is just a list of suggestions. But I'm happy to discuss and accept PRs with the community. :-)

similar comment as per https://github.com/stac-extensions/sar/issues/9#issuecomment-1172046369.
I read about those sensor independent format some time ago and there are indeed interesting, especially in the ARD context.
IMO, It is worth to list them in the documentation but I'd like to have concrete samples to base STAC item examples on. @jmorrison1847, would you have some to recommend?

@emmanuelmathot apologies for the radio silence, I just forgot to check Github for a month. Showing my true colors...
A great example of this just happened in the time since posting it - Capella released sample data via AW that includes a collection of SICD files compliant with the STAC standard: https://capella-open-data.s3.us-west-2.amazonaws.com/index.html/BrVjRL9LGRjg8ipuaKLhdHr1kD4sbvyd8QTYGwbMTTtkR4fzPd4yeiApxyrQkiq/2U5bjtVSEPJUkNoHyocrdciedbcheZ6VAXprFmxH5bwLpegLEXWtWg5fUM2VVpdMQjFW9FaKBeHy8hgyTZqF5Eiacgwn4vn3adb?t=2
If that link is wonky, you can get to it by going to the main page on the AWS open data website and navigating to the STAC Catalog or STAC Browser on the right-hand side there: https://registry.opendata.aws/capella_opendata/

Well, this is probably going to open a can of worms, but does product type mean processing level or file format? Because a SICD is basically a SLC in NITF format. And a SIDD would be likely a GEC or GTC type product in NITF.  CPHD would be RAW (as S1 calls them).  I don't think these two things should be conflated.

Processing level!
File Format should be specified in the asset property type.

Yes the sar:product_type is the combination of the SAR operating mode and the processing level. It should easily give information about the data you'll find inside (amplitude, phase...) and it's refining (raw, geocoded, terrain corrected...). So it is better if we keep using "common" types, especially for cross-mission search in the catalogs.

Closing in favor of #14


Enabled to be detected multiple words.

In the conventional format of rules, it is necessary to increase the pattern for each word to be detected.
Therefore, when many words are detected, the number of lines in the rule increased, and it make to be difficult to read.
To solve this problem, we made it possible to detect many words in a rule.

Thank you for your Question.
I think this issue is resolved with Google Spread Sheet and CSV To JSON Converter
instead of an new feature of textlint-rule-ja-simple-user-dictionary.
GUI editor and collaborative editing features are also included in Spread Sheet.
Example Spread Sheet
https://docs.google.com/spreadsheets/d/1xYaCQ336RVru7nvxfbaBEO5TKuUBWjo6q9mA8Orw2I0/edit?usp=sharing
CSV To JSON Converter
https://www.convertcsv.com/csv-to-json.htm
Screencast from 2022年07月31日 16時16分02秒.webm


External temp. sensor readings not received by the thermostatic valve

Is your problem within Home Assistant (Core or Z-Wave JS Integration)?
NO, my problem is NOT within Home Assistant or the ZWave JS integration
Is your problem within Z-Wave JS UI (formerly ZwaveJS2MQTT)?
NO, my problem is NOT within Z-Wave JS UI
Checklist

[X] I have checked the troubleshooting section and my problem is not described there.

[X] I have read the changelog and my problem was not mentioned there.

Describe the bug
What causes the bug?
I don't know
What do you observe?
External temperature sensor readings (Aeotec Multisensor 7) are not received properly by the Eurotronics Spirit Z-wave+ thermostatic valve.
What did you expect to happen?
The TRV should use my multisensor temperature to drive the valve opening instead of its own measurements.
Steps to reproduce the behavior:

Add a group association on the Multisensor 7 => Number 11 for [Air temperature (Multivel)]

Set parameter 8 of the Eurotronics TRV to -128 to use external temperature sensor reports

Valve opening percentage will not vary anymore, if it is positive, it can reach 0 but it won't increase anymore. I did some monitoring for a month, as you can see it sticks to 0 (recent peaks corresponding to internal temperature measurement) :

If i set the paramteter 8 to a valid integer (offset), TRV opens up and down again.
I tried with two different Multisensor 7, and 3 TRV, same behaviour all the time.

My guess is that it should work out of the box, in the TRV documentation we can see what's expected to be reported to the valve for parameter 8 :

Device information
Sensor
Manufacturer: Aeotec
Model name: ZWA024 (Multisensor 7), fw 1.4.1
Node ID in your network:  40
TRV
Manufacturer: Eurotronics
Model name: Eurotronic Spirit Z-Wave Plus, fw 0.16
Node ID in your network:  26
How are you using node-zwave-js?

[X] zwave-js-ui (formerly zwavejs2mqtt) Docker image (latest)
[ ] zwave-js-ui (formerly zwavejs2mqtt) Docker image (dev)
[ ] zwave-js-ui (formerly zwavejs2mqtt) Docker manually built (please specify branches)
[ ] ioBroker.zwave2 adapter (please specify version)
[ ] HomeAssistant zwave_js integration  (please specify version)
[ ] pkg
[ ] node-red-contrib-zwave-js (please specify version, double click node to find out)
[ ] Manually built from GitHub (please specify branch)
[ ] Other (please describe)

Which branches or versions?
version: 8.4.1
node-zwave-js branch: 18.10.0
zwave-js-ui branch: latest
Did you change anything?
no
If yes, what did you change?
No response
Did this work before?
No, it never worked anywhere
If yes, where did it work?
No response
Attach Driver Logfile
Logs start when I'm doing the group association / setting parameter 8 to 128
z-ui_2022-11-08.log
zwavejs_2022-11-08.log

My bad, I added the zwave logs to the OP.

Are both devices included using the same S2 key/security class (Unauthenticated/Authenticated)? Everything else looks good IMO.
Theoretically they should work if they both share a common S2 key as long as the sending node has the same or higher highest security class. But in practice many devices can only inter-operate if both have the same highest class.

Hello, thx for your input.
Both devices have been included with S2_Unauthenticated security class.
I ended up using the ZwaveJS API through MQTT with my domotic software to send the sensor temperature to the valve.
So I can tell now that the valve correctly receives the payload from the controller. Then afaik there are two possibilities left :

The Multisensor 7 doesn't send anything to the valve through the group
The format is not readable by the valve software

Maybe I could put the controller in the association and see what happens in the logs ?

Worth a try. Then you can compare if the format is the same.


xml:base in Atom feeds does not appear to be respected

I'm using jekyll-postfiles to keep content (like images) local to the post, and the local references work with a feed reader thanks to an xml:base attribute on the content tag emitted by jekyll-feed.
Unfortunately those links are failing per netlify-plugin-checklinks:
9:37:55 PM:   ✖ FAIL load _site/f001.jpg
9:37:55 PM:   | operator: load
9:37:55 PM:   | expected: 200 _site/f001.jpg
9:37:55 PM:   |   actual: ENOENT: no such file or directory, open '/opt/build/repo/_site/f001.jpg'
9:37:55 PM:   |       at: _site/feed.xml:7:14 (inlined Html)

Is this more of a hyperlink problem, or is it HTML-only?

For anyone running into this in the future, add to your netlify.toml:
[[plugins]]

 package = "netlify-plugin-checklinks"
   [plugins.inputs]
   skipPatterns = [
     "_site/feed.xml",
   ]

main.css is also giving me grief for some yet-undetermined reason, so you might want to add that too if you're using checkExternal = true like I am.

@numist
Sounds like the missing inclusion of xml:base in the link resolution belongs in Assetgraph either in the RSS or the Atom asset type.
Could you create a reduced test case with one html-file that links to a feed, which links to an image in this manner, so we can add it to the Assetgraph test cases and base a path upon it?
The CSS issue you mention lacks a bit too much information for me to act upon. But you are welcome to open another issue with it so we can have a look if you found a bug, or if our error messages are maybe just too cryptic :P

I just added an exception for all css links for now, I'll open a new issue with a reduced example when I restyle my site.
The below will need some massaging to fit your test suite's architecture but here's some reduced code from my own website. Both files validate, the html with https://validator.w3.org/nu/ and the xml with https://validator.w3.org/feed/.
feed.xml:
The important things to modify for testing here are xml.feed.id (self-referential URI), xml.feed.entry.content[xml:base] (baseurl, obv), and xml.feed.entry.content.p.img[src] (relative path to test image from baseurl). Probably xml.feed.entry.id should point to something valid? I would have left it out, but it's required by Atom.
<?xml version="1.0" encoding="utf-8"?>
<feed xmlns="http://www.w3.org/2005/Atom" >
  <updated>2022-04-27T09:26:27+00:00</updated>
  <id>https://numi.st/feed.xml</id>
  <title type="html">my cool website title</title>
  <entry>
    <title type="html">my cool entry title</title>
    <updated>2022-04-21T00:00:00+00:00</updated>
    <id>https://numi.st/post/2022/travel-uke</id>
    <content type="html" xml:base="https://numi.st/post/2022/travel-uke/">
      <![CDATA[

]]>
    </content>
    <author><name>Not Blank</name></author>
  </entry>
</feed>

index.html:
Obv html.head.link[href] needs to point at the xml file above
<!DOCTYPE html>

    <title>my site title</title>
    <link type="application/atom+xml" rel="alternate" href="https://numi.st/feed.xml" />

@papandreou I'm pretty sure we have the correct modeling of an inline HTML-fragment in an atom <content> block. I didn't know bout the ability to set xml:base though. Do you think we could map the xml:base tag to what I guess would have to be a new baseUrl setter in Html or possibly all the way up to Asset ?
https://github.com/assetgraph/assetgraph/blob/master/lib/assets/Html.js#L40-L51

Html is already wired up to get the baseUrl from the superclass' baseUrl getter (and then possibly modify it if there's a <base href=...> in the HTML itself): https://github.com/assetgraph/assetgraph/blob/815ae4b44b30d004bd5fc247cd39c10523cd448e/lib/assets/Html.js#L41-L50
The superclass (Asset) will delegate to its first "non inline ancestor" when it's an inline asset: https://github.com/assetgraph/assetgraph/blob/815ae4b44b30d004bd5fc247cd39c10523cd448e/lib/assets/Asset.js#L556-L559
The challenge seems to be that you can have an xml:base attribute for each inline HTML snippet. I guess the easiest thing is to pick it up when resolving the relation here, add it as a baseUrl property to the to object, and then make sure that the baseUrl getter in Html supports that case also.


The demo is not accessible

GREAT WORK!!!
I found that the demo (https://huggingface.co/spaces/halleewong/ScribblePrompt) was not accessible.
Could you please update it?
BTW, I have no idea if the ScribblePrompt has interactive UI with users? just like annotators.   :D

Hi, the demo is accessible and currently working for me. Are you still having trouble accessing it?

Thanks for your reply :D
Unfortunately, I just tried to access the demo but failed.
Is there any limitation?

It seems that Hugging Face Spaces, where the demo is hosted, is blocked in China.
I've added the demo code to ./demos so you can run the app locally. Hope this helps!

wow! cool
I'll give it a try!   :D


Uncaught TypeError: this.textEditor.displayBuffer.getMaxScrollTop is not a function

[Enter steps to reproduce below:]

...
...

Atom Version: 1.1.0
System: Microsoft Windows 10 Home
Thrown From: autoclose-html package, v0.19.0
Stack Trace
Uncaught TypeError: this.textEditor.displayBuffer.getMaxScrollTop is not a function
At /C:/Users/anapr/.atom/packages/minimap/lib/minimap.coffee:169

TypeError: this.textEditor.displayBuffer.getMaxScrollTop is not a function
    at Minimap.module.exports.Minimap.getTextEditorMaxScrollTop (C:\Users\anapr\.atom\packages\minimap\lib\minimap.coffee:200:46)
    at Minimap.module.exports.Minimap.getTextEditorScrollRatio (C:\Users\anapr\.atom\packages\minimap\lib\minimap.coffee:216:36)
    at Minimap.module.exports.Minimap.getCapedTextEditorScrollRatio (C:\Users\anapr\.atom\packages\minimap\lib\minimap.coffee:223:50)
    at Minimap.module.exports.Minimap.getScrollTop (C:\Users\anapr\.atom\packages\minimap\lib\minimap.coffee:334:17)
    at Minimap.module.exports.Minimap.getFirstVisibleScreenRow (C:\Users\anapr\.atom\packages\minimap\lib\minimap.coffee:316:17)
    at atom-text-editor-minimap.MinimapElement.leftMousePressedOverCanvas (C:\Users\anapr\.atom\packages\minimap\lib\minimap-element.coffee:480:63)
    at atom-text-editor-minimap.MinimapElement.mousePressedOverCanvas (C:\Users\anapr\.atom\packages\minimap\lib\minimap-element.coffee:470:8)
    at HTMLCanvasElement.<anonymous> (C:\Users\anapr\.atom\packages\minimap\lib\minimap-element.coffee:174:31)

Commands
     -3:36.6.0 core:cut (atom-text-editor.editor.is-focused)
     -3:33.9.0 editor:newline (atom-text-editor.editor.is-focused)
     -3:32.9.0 core:paste (atom-text-editor.editor.is-focused)
     -3:30.2.0 core:save (atom-text-editor.editor.is-focused)
     -3:07.7.0 core:cut (atom-text-editor.editor.is-focused)
     -3:05.5.0 snippets:next-tab-stop (atom-text-editor.editor.is-focused)
     -3:05.5.0 snippets:expand (atom-text-editor.editor.is-focused)
     -3:05.5.0 editor:indent (atom-text-editor.editor.is-focused)
     -3:04.7.0 core:paste (atom-text-editor.editor.is-focused)
     -2:58.4.0 core:save (atom-text-editor.editor.is-focused)
  2x -2:52 core:backspace (atom-text-editor.editor.is-focused)
     -2:50.8.0 core:save (atom-text-editor.editor.is-focused)

Config
{
  "core": {}
}

Installed Packages
# User
autoclose-html, v0.19.0
minimap, v4.13.4
pretty-json, v0.4.1
symbols-tree-view, v0.10.0

# Dev
No dev packages

Sorry for not following up on this, the stack trace doesn't really have anything that indicates it has anything to do with autoclose-html, despite atom claiming it was thrown from here. Also I haven't heard anything else about this from other or been able to reproduce it myself, if it continues to be a problem I encourage you to try and debug it on your machine and send me your findings.


Suggest that concurrent-map would use custom shard_count for each instance

Hello, concurrent-map maintainers :)
I accidentally found this concurrent-map repository, which looks very cool!
Me and one of my team members suggested how to use custom SHARD_COUNT for each concurrent map instance instead of global SHARD_COUNT.
It looks like this one.
// A "thread" safe map of type string:Anything.
// To avoid lock bottlenecks this map is dived to several (SHARD_COUNT) map shards.
type ConcurrentMap[K comparable, V any] struct {
	shards     []*ConcurrentMapShared[K, V]
	sharding   func(key K) uint32
	shardCount uint32 // <- THIS IS ADDED
}

I'll submit my source code for it on PR, could you please review my changes?
Thanks!

Yeah, Honestly having this lever will enable users to tune the number according to their concurrency needs and memory overhead threshold.


print() is a function in Python 3

Also from six.moves import xrange

Thanks for this, however this code is not written or tested for Python 3 - there are probably other changes you'll need to make in order to get it fully Python 3 compatible.

Yes but this change is still worth making.


TkSetVisibilityTest.java:95-99: Make visibility functionality in MkAttributes...

Puzzle 255-bf924f1e in src/test/java/com/nerodesk/takes/doc/TkSetVisibilityTest.java:95-99 has to be resolved: Make visibility functionality in MkAttributes be a function of the underlying file instead of a boolean field. This is necessary in order for different instances of MkDoc which point to the same underlying file to see the same visibility status. Once this is done, un-ignore the test below.... The puzzle was created by Felipe Pina on 11-May-2015.
If you have any technical questions, don't ask me, submit new tickets instead

@yegor256 there is no milestone yet, so I set it to 0.7

@pinaf do this task

@karato please assign someone else

@karato please assign someone else

@pinaf deducted 30 from your rating :(

@karato please assign someone else

@pinaf all right, we'll find someone else for this task

@carlosmiranda do this task

@karato waiting for #285 here.

@karato waiting for #285 here.

@carlosmiranda yes, waiting for #285

@yegor256 I think we can continue, impediment #285 is closed

@yegor256 We are done here, thanks all

@carlosmiranda 30 mins sent to your balance (ID 58075257), many thanks! It took 92 hours and 42 mins.. +30 to your rating, your total score is +4201


To: field does not get changed in Tutorial?

Hi,
I am just trying out the tutorial to understand how to write plugins.  From the tutorial the To: was changed on the console -
[INFO] [rcpt_to.lookup] Email address now<EMAIL_ADDRESS>However when I checked in the /tmp/mail.eml file, the content says that the To: was not changed.
Fri, 11 Nov 2011 08:50:49 -0500
Date: Fri, 11 Nov 2011 08:50:49 -0500
To<EMAIL_ADDRESS>From<EMAIL_ADDRESS>Subject: test Fri, 11 Nov 2011 08:50:49 -0500
X-Mailer: swaks v20100211.0 jetmore.org/john/code/swaks/
This is a test mailing
I am just curious how does the mail get send in the next phase?  And where was the change saved to?  Thank you for your help.

There are two components to email - the envelope, which the end-user never sees, which is what is used for delivering the message, and the body of the email, which is what you see in email clients when you open the mail. The two don't have to match, and are only related to each other by courtesy. In Haraka you can modify the headers through the Transaction object API, but it's separate to modifying the envelope, which is all I do in the tutorial.

Thank you for your help.  I hope you don't mind I ask further questions to help clarify my understanding.  So in this case, the one used to deliver the mail was changed, which is in the envelop.  How does, let's say, qmail pick that up to know where to send the mail to?  Also, is there an example where the change is also pushed to the body of the email through Transaction object API?  I will be looking through the source code but a pointer would definitely be a great help.
Thank you.

No I don't mind more questions. But I'll keep the bug closed if that's OK.
The SMTP protocol carries the envelope data, so if you forward to Qmail it knows the changes you made behind the scenes.
To change the headers you'd need to first delete the original To line:
transaction.remove_header('To');

And then add the new one:
transaction.add_header('To', new_recipient);

Hope that helps.

FWIW the headers are always parsed, you don't need to enable parse_body for this. Also this should be done in data_post just for cleanliness sake.


Forget ssid

How can I forget a ssid from the raspberry pi?

Before starting wifi-connect I call these commands:
nmcli connection down id "ssid" || true
nmcli connection delete id "ssid" || true

To remove all existing wifi-connections I call:
nmcli con show | grep wifi |  awk '{print $2}' | while read line; do nmcli con down uuid $line && nmcli con delete uuid $line;    done

Similar approach:
nmcli --pretty --fields UUID,TYPE con show | grep wifi | awk "{print $1}" | while read line; do nmcli con delete uuid $line; done
Don't forget to install network-manager with your dockerfile for it to work but to avoid issues, be sure to mask the service:
RUN apt-get update && apt-get install -y network-manager && systemctl mask NetworkManager.service
See https://www.balena.io/docs/reference/OS/network/2.x/#changing-the-network-at-runtime for the warnings and issues network manager can cause.
Alternatively, build a Python script solution: https://github.com/balena-io/wifi-connect/issues/270

Hi everyone,
I also want to remove all wifi connection programmatically in the container. I was able to delete the specific connection with the following commands.

nmcli connection down id "ssid" || true
nmcli connection delete id "ssid" || true

But what I really want to is delete all wifi connections in my django container. Otherwise, is it possible to run those commands on hostOS in the container?
Here are all settings for django service in docker-compose.yml file

django:
build:
context: .
dockerfile: ./compose/django/Dockerfile-dev
network_mode: "host"
privileged: true
volumes:
- ./:/app/
- /run/dbus/system_bus_socket:/host/run/dbus/system_bus_socket
depends_on:
- db
restart: always
ports:
- "8000:8000"
devices:
- "/dev/ttyAMA0:/dev/ttyAMA0"
cap_add:
- NET_ADMIN
environment:
- DBUS_SYSTEM_BUS_ADDRESS="unix:path=/host/run/dbus/system_bus_socket"

To use nmcli command in the container, I installed Network-manager by the following command.

apt-get update && apt-get install -y network-manager && systemctl mask NetworkManager.service

And I run executed the following command in the container
nmcli connection down id "TP-LINK-REAL3D" || true
nmcli connection delete id "TP-LINK-REAL3D" || true
But I got the following error.

How can I solve this error?
Thanks in advance.
Anatoli

Hi everyone,
I also want to remove all wifi connection programmatically in the container. I was able to delete the specific connection with the following commands.

nmcli connection down id "ssid" || true
nmcli connection delete id "ssid" || true

But what I really want to is delete all wifi connections in my django container. Otherwise, is it possible to run those commands on hostOS in the container?
Here are all settings for django service in docker-compose.yml file

django:
build:
context: .
dockerfile: ./compose/django/Dockerfile-dev
network_mode: "host"
privileged: true
volumes:
- ./:/app/
- /run/dbus/system_bus_socket:/host/run/dbus/system_bus_socket
depends_on:
- db
restart: always
ports:
- "8000:8000"
devices:
- "/dev/ttyAMA0:/dev/ttyAMA0"
cap_add:
- NET_ADMIN
environment:
- DBUS_SYSTEM_BUS_ADDRESS="unix:path=/host/run/dbus/system_bus_socket"

To use nmcli command in the container, I installed Network-manager by the following command.

apt-get update && apt-get install -y network-manager && systemctl mask NetworkManager.service

And I run executed the following command in the container
nmcli connection down id "TP-LINK-REAL3D" || true
nmcli connection delete id "TP-LINK-REAL3D" || true
But I got the following error.

How can I solve this error?
Thanks in advance.
Anatoli

From https://github.com/balena-io/wifi-connect#balenaos-multicontainer-app it seems that you are missing
        labels:
            io.balena.features.dbus: '1'

in your docker-compose file.

From https://github.com/balena-io/wifi-connect#balenaos-multicontainer-app it seems that you are missing
        labels:
            io.balena.features.dbus: '1'

in your docker-compose file.

Thanks for your reply! @avandermeer
I know that labels option is only needed if I use Ballena OS. Right?
I installed raspbian OS on my raspberry pi4. And the base image of Dockerfile for django service is python3.7.3.

So I didn't add label option to docker-compose.yml file
You seem to have professional experience in this field. I really hope you give me a good solution to realize my ultimate goal.

How can I delete and deactivate the already configured wifi network connection in the container? Is it necessary to install network-manger in the container? I think there is a way to do that without installing network manager in the container
I also tried the following command to delete all the network connection, but it didn't work

nmcli con show | grep wifi |  awk '{print $2}' | while read line; do nmcli con down uuid $line && nmcli con delete uuid $line;    done

pi@raspberrypi:~/DTECTS_SW_Server_Side $ sudo nmcli con show | grep wifi |  awk '{print $2}' | while read line; do nmcli con down uuid $line && nmcli con delete uuid $line;    done
Connection 'Ergsense' deactivation failed: Not authorized to deactivate connections
Error: Connection deletion failed: Insufficient privilegesError: not all connections deleted.

Thanks for your reply! @avandermeer
I know that labels option is only needed if I use Ballena OS. Right?
I installed raspbian OS on my raspberry pi4. And the base image of Dockerfile for django service is python3.7.3.

So I didn't add label option to docker-compose.yml file
You seem to have professional experience in this field. I really hope you give me a good solution to realize my ultimate goal.

How can I delete and deactivate the already configured wifi network connection in the container? Is it necessary to install network-manger in the container? I think there is a way to do that without installing network manager in the container
I also tried the following command to delete all the network connection, but it didn't work

nmcli con show | grep wifi |  awk '{print $2}' | while read line; do nmcli con down uuid $line && nmcli con delete uuid $line;    done

pi@raspberrypi:~/DTECTS_SW_Server_Side $ sudo nmcli con show | grep wifi |  awk '{print $2}' | while read line; do nmcli con down uuid $line && nmcli con delete uuid $line;    done
Connection 'Ergsense' deactivation failed: Not authorized to deactivate connections
Error: Connection deletion failed: Insufficient privilegesError: not all connections deleted.

Similar approach:
nmcli --pretty --fields UUID,TYPE con show | grep wifi | awk "{print $1}" | while read line; do nmcli con delete uuid $line; done
Don't forget to install network-manager with your dockerfile for it to work but to avoid issues, be sure to mask the service:
RUN apt-get update && apt-get install -y network-manager && systemctl mask NetworkManager.service
See https://www.balena.io/docs/reference/OS/network/2.x/#changing-the-network-at-runtime for the warnings and issues network manager can cause.
Alternatively, build a Python script solution: #270

Hi,
Installing network manager inside the container is for running nmcli commands in the container?
I installed network manager manually in the container and run nmcli. Of course, I used your command to install it.
However, I still can't run nmcli in the container. Can you check my screenshot in above? My host OS is Raspbian and network manger is already installed on host OS. I really want to use nmcli commands in the container. Is there any good solution for it?
Thanks.
Anatoli

Similar approach:
nmcli --pretty --fields UUID,TYPE con show | grep wifi | awk "{print $1}" | while read line; do nmcli con delete uuid $line; done
Don't forget to install network-manager with your dockerfile for it to work but to avoid issues, be sure to mask the service:
RUN apt-get update && apt-get install -y network-manager && systemctl mask NetworkManager.service
See https://www.balena.io/docs/reference/OS/network/2.x/#changing-the-network-at-runtime for the warnings and issues network manager can cause.
Alternatively, build a Python script solution: #270

Hi,
Installing network manager inside the container is for running nmcli commands in the container?
I installed network manager manually in the container and run nmcli. Of course, I used your command to install it.
However, I still can't run nmcli in the container. Can you check my screenshot in above? My host OS is Raspbian and network manger is already installed on host OS. I really want to use nmcli commands in the container. Is there any good solution for it?
Thanks.
Anatoli

I know that labels option is only needed if I use Ballena OS. Right?

It will only work in Balena OS/with Balena Engine, but you will likely need to achieve the equivalent of this in other environments too.
Try adding -v /var/run/dbus:/var/run/dbus to your docker run command. This will mount the dbus controller in to your container, similar to what the label does in Balena.
Depending on what kind of permissions are required you may need to add --privileged to the container too, but try without as wouldn't recommend adding it unless necessary as it's generally not good practice security wise.
Try searching/reading around mounting DBus into Docker container for more info. As this is outside of the Balena infrastructure and not related to Balena software/wifi-connect you may be better off trying other forums for more info.

I know that labels option is only needed if I use Ballena OS. Right?

It will only work in Balena OS/with Balena Engine, but you will likely need to achieve the equivalent of this in other environments too.
Try adding -v /var/run/dbus:/var/run/dbus to your docker run command. This will mount the dbus controller in to your container, similar to what the label does in Balena.
Depending on what kind of permissions are required you may need to add --privileged to the container too, but try without as wouldn't recommend adding it unless necessary as it's generally not good practice security wise.
Try searching/reading around mounting DBus into Docker container for more info. As this is outside of the Balena infrastructure and not related to Balena software/wifi-connect you may be better off trying other forums for more info.

To remove all existing wifi-connections I call:
nmcli con show | grep wifi |  awk '{print $2}' | while read line; do nmcli con down uuid $line && nmcli con delete uuid $line;    done

Hi @avandermeer
I was able to run nmcli commands in my container. It works well. However, the above command you shared for removing all wifi network connections doesn't work. I get the following error when I try it.

What's the wrong? I can remove the specific network connection using nmcli command.
How can I remove all the network connections?
I hope you will get back to me with a good solution asap.
Thanks in advance.
Jense

To remove all existing wifi-connections I call:
nmcli con show | grep wifi |  awk '{print $2}' | while read line; do nmcli con down uuid $line && nmcli con delete uuid $line;    done

Hi @avandermeer
I was able to run nmcli commands in my container. It works well. However, the above command you shared for removing all wifi network connections doesn't work. I get the following error when I try it.

What's the wrong? I can remove the specific network connection using nmcli command.
How can I remove all the network connections?
I hope you will get back to me with a good solution asap.
Thanks in advance.
Jense

@jense-arntz sorry for the delay; you have to run every command with the right permissions, in your example you only put sudo before the first command (nmcli con show), but the second time you call nmcli (in the while loop), sudo is missing.

@jense-arntz sorry for the delay; you have to run every command with the right permissions, in your example you only put sudo before the first command (nmcli con show), but the second time you call nmcli (in the while loop), sudo is missing.


Fix problem with cameras that don't support time

Description:
Some onvif cameras don't support Date management. In that case None is returned and script crashes when trying to obtain date
Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.
[x] I have followed the development checklist

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[x] The manifest file has all fields filled out correctly. Update and include derived files by running python3 -m script.hassfest.
[x] New or updated dependencies have been added to requirements_all.txt by running python3 -m script.gen_requirements_all.
[x] Untested files have been added to .coveragerc.

If the code does not interact with devices:

[x] Tests have been added to verify that the new code works.

Make sure you are basing your PR against dev, not master.

I have a similar issue after upgrade to 0.93 and 0.93.1 too. My old onvif cameras crashes if i configure them on HA. With 0.92 everything worked fine.

Please open an issue if you suspect a bug. If you need help please use our help channels:
https://home-assistant.io/help/#communication-channels
Merged PRs should not be used for support or bug reports. Thanks!


Fix whitespace and add braces to one-line for loop

Prior to this change, viewing mixed tabs and spaces on GitHub caused the body of the for loop to have less visually preceding whitespace than the loop description itself.
I've included for loop braces to prevent any ambiguity for the reader.
Added a space between the reduce callback function and its opening brace to mirror the whitespace used on the first line.

LGTM
Thank you very much for your contribution! This will be included in the next beta release.

LGTM
Thank you very much for your contribution! This will be included in the next beta release.


Could the keybindings  shortkeys internationalized

VSCode Version: 1.10.2
OS Version: Windows 7

Steps to Reproduce:
There is a problem with keyboard layout (I'm using Azerty) when defiining a shortkey and even in the default keybinding file, for example shortcut labelled as xx+yy+\ but on an AZERTY keyboard it is xx+yy+:.

Is there a another  way of having the correct shortkeys,could it be possible to have several shortkeys default json one for each keyboard layout?

also russian

We don't ship with keyboard layout specific default keybindings, that is something you can upvote in #1240


add upload coverage unittest

fix #60

hi @ann26
it is quite independet between insight function with campaign
you need to make it linked
just put campaign as parameter on CoverageUpload initiation (you could see the code)
and coverage upload needs to be object, not class and not use static method. just use all of insight function methods

its is much much better
great!

hi @ann26
maybe you could make test to compare shapefile that loaded by insight function and by using shapefile directly

LGTM
please update the ticket


Make Docker tests work in forks/pull requests

Currently Docker assignment tests only run on pushes, not on pull requests.
Furthermore, the Dockerfile has the repo URL hardcoded. This causes it to break when it runs in a fork, as commits in forks are not present in the upstream repo. The URL should default to this one but it should be set in the CI via environment variables. GitHub provides such environment variables which we can use.

I no longer have permissions to assign issues, can I take this one? :)

Of course :)


Dev doc line highlighting misplaced

Describe the bug
In the current dev doc drawer nav item examples, the code block line highlighting is not properly aligned with the code snippet. Something similar happened on the Angular side too, but Evan wisely fixed it (https://github.com/brightlayer-ui/angular-component-library/pull/458#issuecomment-1198130174)
Screenshots or links to minimum reproduction example

Notice how the bottom edge of that background gradient is going through the middle of a line

This might need a custom style override to align highlight

BLUI-3501