Download the face mesh v2 tflite model

MediaPipe Solution (you are using)
face blendshape
Programming language
c++
Are you willing to contribute it
Yes
Describe the feature and the current behaviour/state
Now I am developing an app that drives my own designed avatar using 52 blendshape. The blendshape coefficients and model inference are performed in c++. mediapipe solution shows very excellent solution for this. And face_blendshape.tflite and face mesh v2(478 landmark) are both needed. For now, I cannot find any download information about face mesh v2(mentioned in blendshape model card). The face mesh outputing 468 model (with or without attention) is the only model I can find. So it is possible to download face mesh v2 model?

Will this change the current API? How?
No

Who will benefit with this feature?
No response
Please specify the use cases for this feature
Using blendshape to drive my own avatar

Any Other info
No response

@jeremmoore,
Thank you for sharing your request. We are forwarding issue internally based on the discussion team will prioritise the request.

The only models we offer are linked here: https://github.com/google/mediapipe/blob/master/docs/solutions/models.md#face-mesh

@jeremmoore
You can access the tflite by by downloading the face landmarker task file here and then unzip it from within the terminal.
unzip face_landmarker.task
Archive:  face_landmarker.task
 extracting: face_detector.tflite
 extracting: face_landmarks_detector.tflite
 extracting: geometry_pipeline_metadata_landmarks.binarypb
 extracting: face_blendshapes.tflite


我冒昧的提一点建议，你们的流式项目好像不支持在云服务器上运行。。。

我个人因为设备资源有限，在网上租了云算力，在单次调用的录音输入输出中，是可以进行正常问答的，但是流式的就不行，遇到了
“OSError： [errno -9996] 无效的输入设备（无默认输出设备）
追踪：
File "/root/miniconda3/envs/skl/lib/python3.10/site-packages/streamlit/runtime/scriptrunner/exec_code.py", line 85, in exec_func_with_error_handling
result = func()
File "/root/miniconda3/envs/skl/lib/python3.10/site-packages/streamlit/runtime/scriptrunner/script_runner.py", line 576, in code_to_exec
exec(code, module.dict)
File "/root/mini-omni2/webui/omni_streamlit_2.py", line 331, in
main()
File "/root/mini-omni2/webui/omni_streamlit_2.py", line 304, in main
recording(status)
File "/root/mini-omni2/webui/omni_streamlit_2.py", line 183, in recording
stream = audio.open(
File "/root/miniconda3/envs/skl/lib/python3.10/site-packages/pyaudio/init.py", line 639, in open
stream = PyAudio.Stream(self, *args, **kwargs)
File "/root/miniconda3/envs/skl/lib/python3.10/site-packages/pyaudio/init.py", line 441, in init
self._stream = pa.open(**arguments)”
错误。
希望您能提几点建议（ps：我的设备可以正常录音和外放，然后正常的权限我也给浏览器了，相关代码我不懂如何修改，请各位大佬指点一二）

can you tell me exactly what error are you facing maybe I can help. in english :)

hi, @daifuliu for now, the streamlit demo does not support running in cloud server, you may refer to: https://github.com/gpt-omni/mini-omni2/pull/27 and https://huggingface.co/spaces/freddyaboulton/mini-omni2-webrtc, as the gradio demo can run in cloud server.

那是代码运行会使用你本地的麦克风。你云服务器当然没有麦克风。

那是代码运行会使用你本地的麦克风。你云服务器当然没有麦克风。

THAT'S RIGHT

I am also curious about how it would go on cloud server, so I translate the issue here in english: sunkeli-ux was running this model on a cloud based GPU server, it worked well in the manner of recorded audio input, while failed with audio streaming, the error on server showed: OSError： [errno -9996] invalid input device（No default output device） Trace： File "/root/miniconda3/envs/skl/lib/python3.10/site-packages/streamlit/runtime/scriptrunner/exec_code.py", line 85, in exec_func_with_error_handling result = func() File "/root/miniconda3/envs/skl/lib/python3.10/site-packages/streamlit/runtime/scriptrunner/script_runner.py", line 576, in code_to_exec exec(code, module.dict) File "/root/mini-omni2/webui/omni_streamlit_2.py", line 331, in main() File "/root/mini-omni2/webui/omni_streamlit_2.py", line 304, in main recording(status) File "/root/mini-omni2/webui/omni_streamlit_2.py", line 183, in recording stream = audio.open( File "/root/miniconda3/envs/skl/lib/python3.10/site-packages/pyaudio/init.py", line 639, in open stream = PyAudio.Stream(self, *args, **kwargs) File "/root/miniconda3/envs/skl/lib/python3.10/site-packages/pyaudio/init.py", line 441, in init self._stream = pa.open(**arguments)”
@Gittomon have you got it running on a cloud server and function well?

INTERFACE MODFICATION LIKE A THIRD OPTION FOR TEXT-TO-TEXT AS WELL ( AUIDO WILL PE PLAYED THERE TOO FOR THIS VOICE ) AND I HAVE SEEN A BIT MORE ISSUES THERE TO FIX THEM AS WELL.


Displaying errors assigned to :base?

Am I doing something wrong, or does this library not handle errors added to :base?  The only errors that show up for me are those attached to individual attributes.

It doesn't currently handle them, and I don't intend to (since any way I chose might not be acceptable for some). You should be able to easily insert your own error markup for errors on :base, though.

Thanks.

No worries. Let me know if you have any more questions.


Add custom_init logic

This reverts commit 4c4fe8a6aa8ece15fdf93326c23961a9f881b38b - built by reverting the commit that removed custom_init in #565

Quoting from #565
The principle is working, the only potential problem I see is that the data kwargs aren't the original, instead they're of the form
{'validated_data': ValidatedData(model_dict={'a': 1, 'b': 3}, fields_set={'b'})}

where ValidatedData is a custom type that's internal to pydantic-core and can't be constructed outside.

Codecov Report

Merging #566 (f2368a4) into main (4f1ce33) will increase coverage by 0.01%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #566      +/-   ##
==========================================
+ Coverage   94.13%   94.14%   +0.01%
==========================================
  Files          94       94
  Lines       12762    12794      +32
  Branches       24       24
==========================================
+ Hits        12013    12045      +32
  Misses        743      743
  Partials        6        6

Impacted Files
Coverage Δ

src/input/input_abstract.rs
87.87% <ø> (ø)

src/input/input_python.rs
98.74% <ø> (ø)

pydantic_core/core_schema.py
96.86% <100.00%> (+<0.01%)
:arrow_up:

src/argument_markers.rs
91.48% <100.00%> (+3.61%)
:arrow_up:

src/validators/model.rs
98.33% <100.00%> (+0.12%)
:arrow_up:

Continue to review full report in Codecov by Sentry.

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

AFAIK this is ready to merge.

LGTM, no issues with the pydantic PR after making the changes I suggested there

Hmm I take it back, I raised a couple points on https://github.com/pydantic/pydantic/pull/5648 that may mean this offers less backward compatibility than I thought

We decided to close this, and instead add a init_vars option to config.extra.


Add support for sbt 0.13.8

Hello,
Can you release a new version of sbt-dependency-graph-sugar as sbt-dependency-graph have just bump to 0.7.5 to support sbt 0.13.8 (https://github.com/jrudolph/sbt-dependency-graph/commit/2e3a8fa168b57004ef867481a5974c1f734484f1).
Thanks in advance

+1

+1

0.7.5 is now available @ https://oss.sonatype.org/content/repositories/releases/com/gilt/sbt-dependency-graph-sugar_2.10_0.13/0.7.5/


[DOC] Group documents into clusters using Cluster layer

What do you want to do?

[ ] Request a change to existing documentation
[x] Add new documentation
[ ] Report a technical problem with the documentation
[ ] Other

Tell us about your request. Provide a summary of the request and all versions that are affected.
Add cluster layer to Maps documentation.
What other resources are available? Provide links to related issues, POCs, steps for testing, etc.
https://github.com/opensearch-project/dashboards-maps/issues/250

@vagimeli , @kolchfa-aws - Fanit is assigned to the doc issue, but Melissa is assigned to the github issue. Who is covering this one? Thanks!

@hdhalter I'm ok with any option. I'll discuss with @vagimeli.

@kolchfa-aws +1

@hdhalter Assign the issue to me. cc: @kolchfa-aws

Moved to 2.11, as per project meeting 8/10.

@VijayanB Will you send me the demo and endpoint I can use to finish drafting the documentation for this feature? Once the draft content is ready, I'll send to you for technical review. Thanks!

Per the roadmap, this feature is for 3.0.0

@VijayanB Is this issue still on hold?

@VijayanB Can we close this issue and open a mew issue or PR when the feature needs documentation?
cc: @hdhalter

[Triage -- attendees 1, 2, 3, 4, 5, 6, 7]


open('foo', 'w').write('bar') not handled correctly

I had a program that did
open('foo', 'w').write('bar')

(which is not ideomatic but in CPython works fine), and after that the file is just not there even though I was writing over an existing file created with fs.create_file.

Thanks for the report!
Hm, the following works fine for me, at least under Windows:
    def test_open_write(self):
        file_path = 'foo'
        self.fs.create_file(file_path)
        open(file_path, 'w').write('bar')
        self.assertTrue(os.path.exists(file_path))

This is using fake_filesystem_unittest.TestCase. Though generally it is not a good idea to do this as you don't have a file object to close the file now. Leaving the file open will probably not write the file contents - the caching/flushing behavior is different under different systems and different Python versions, so while we try to emulate this in pyfakefs, the behavior may not match in all cases.
What are your OS and Python version? Can you write a minimal test that succeeds in the real OS and fails under pyfakefs?

macOS. I'm sorry, but I found like 5 more issues that were all fatal to my use of this lib, so have given up on the entire thing :(

that's unfortunate... ah well, it's obviously not suited for your use then.

I don't have a Mac, so I can test locally only under Linux and Windows - both did not show the problem.

Ok, the test also works under MacOS (tested in Travis.CI) - so I don't know what the problem is.
Closing this - feel free to reopen it if you want to have another go or have more information.


XXcleanUp.sh - add resetBlockchain option

In some cases people want to trigger a reset on the blockchain data. Add that option to the XXcleanup.sh script and start the "get blockchain" script again and see that it offers reboot when done.

How can I add the XXcleanup.sh script ?

+1 , Thanks @rootzoll  that is what I was asking for in issue  #596

@btc21cluib correction: the script is calles XXcleanHDD.sh and is already part of RaspiBlitz. If you call it, it cleans everything from the HDD (also the LND data), but is keeping the blockchain data ... so you can use the HDD for a fresh RaspiBlitz setup, without the need to download the blockchain data.
What we taking about here is to extend the functionality to also use the script to just delete the blockchain data. That will be part of a future release.

Added a -blockchain parameter to XXcleanHDD.sh - that will delete all the blockchain data and trigger the new XXrepairBlockchain.sh. To be final tested.

Tests look good. This is part of v1.3 release.


PLT-8212: Keep min length check while saving team name

Summary
Minimum Length check on team name has been added when editing the name from Team settings
Ticket Link
Github
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x ] Ran make check-style to check for style errors (required for all pull requests)
[x ] Ran make test to ensure unit and component tests passed
[x ] Has UI changes

Should I be using Utils.localizeMessage or FormattedMessage as used in the create team page is fine??

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: http://i-0d834190c8130de61.spinmint.com
Test Account 1: Email<EMAIL_ADDRESS>| Password: passwd
Test Account 2: Email<EMAIL_ADDRESS>| Password: passwd
Instance ID: i-0d834190c8130de61

Spinmint test server destroyed

@esethna push the changes by evening

@esethna done

Hey @pruthvip, let us know if we can help with the above changes?

@jasonblais sorry for the delay. I have made the changes. Please have a look, if its fine.

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: http://i-028e46690a42bb0db.spinmint.com
Test Account 1: Email<EMAIL_ADDRESS>| Password: passwd
Test Account 2: Email<EMAIL_ADDRESS>| Password: passwd
Instance ID: i-028e46690a42bb0db

Spinmint test server destroyed

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: http://i-0698e426aca78695f.spinmint.com
Test Account 1: Email<EMAIL_ADDRESS>| Password: passwd
Test Account 2: Email<EMAIL_ADDRESS>| Password: passwd
Instance ID: i-0698e426aca78695f

Spinmint test server destroyed

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: http://i-05de2abb28d2fabcc.spinmint.com
Test Account 1: Email<EMAIL_ADDRESS>| Password: passwd
Test Account 2: Email<EMAIL_ADDRESS>| Password: passwd
Instance ID: i-05de2abb28d2fabcc

Thanks @pruthvip!
Not sure why, but the error text seems to be the old one, instead of the one proposed by @esethna: Team Name must be 2 or more characters up to a maximum of 15. You can add a longer team description.

@jasonblais hey thanks for pointing that out. I was making a mistake. I have fixed it now. But I still have an issue.  I am using FormattedMessage message to show the error, since min and max are dynamic values , I cant use Utils.localizeMessage directly.
Hence in the render, SettingItemMax is expecting the error as a string, but  FormattedMessage returns an object. Which throws a prop mismatch error.  Can you guide me how to go about it.

Spinmint test server destroyed

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: http://i-0318930db5bdca88c.spinmint.com
Test Account 1: Email<EMAIL_ADDRESS>| Password: passwd
Test Account 2: Email<EMAIL_ADDRESS>| Password: passwd
Instance ID: i-0318930db5bdca88c

@MusikPolice Can you help with this question?

But I still have an issue.  I am using FormattedMessage message to show the error, since min and max are dynamic values , I cant use Utils.localizeMessage directly.

Hence in the render, SettingItemMax is expecting the error as a string, but  FormattedMessage returns an object. Which throws a prop mismatch error.  Can you guide me how to go about it.

Functionally this looks good, thanks for the quick fixes!

Spinmint test server destroyed

@jasonblais merged.

@MusikPolice The client_error which i pass to SettingItemMax is an object, because FormattedMessage gives back an object. But SettingItemMax is expecting a string as you can see in the propTypes.. Hence in development mode , react will throw an error of prop mismatch! Is this fine?
SettingItemMax.propTypes = {     inputs: PropTypes.array,     client_error: PropTypes.string,     server_error: PropTypes.string,     extraInfo: PropTypes.element,     infoPosition: PropTypes.string,     updateSection: PropTypes.func,     submit: PropTypes.func,     saving: PropTypes.bool,     title: PropTypes.node,     width: PropTypes.string,     submitExtra: PropTypes.node,     cancelButtonText: PropTypes.node };

@pruthvip it's certainly not ideal. I didn't realize that we still had this outstanding issue before I merged. Can you fix it in a separate PR? I suspect that you can pass some property of client error in to SettingItemMax - can you inspect it and see if it has a string field that contains the error message?

ya sure @MusikPolice . Vl look into that and vl raise a PR.

@MusikPolice , i have fixed the issue https://github.com/mattermost/mattermost-webapp/pull/478 . Have a look.


NBuilder doesn't work with VB.NET

NBuilder does not work as expected in VB.NET
Test code:
Imports FizzWare.NBuilder
Module Module1
Sub Main()
Dim x = Builder(Of Something).CreateListOfSize(5).All().With(Function(s) s.Int = 1).Build()
Console.WriteLine(x.All(Function(s) s.Int = 1)) 'returns false
End Sub
End Module
Public Class Something
Public Property Int() As Integer
Public Property Bool As Boolean
Public Property Str() As String
End Class

x is a list. try looping over x and console.writeline each item

Don't think the linq query is going to output something else then Console.Writeline() but here is the output of the property Int:
1,2,3,4,5
Just try it. C# works as expected

Your example doesn't really make sense, can you explain what you're trying to do and I'll tell you how to do it?

I want a list of 5 elements of type Something with all elements having property Int 1
Above example is just a quick setup to see if nbuilder works as expected. In C# I get expected result (a list of 5 Something elemens with all instances having Int = 1) in VB.NET I get a list with 5 Something elements having Int 1 for element 1, 2 for element 2 etc...
So behaving not as expected. Try above code and you'll see.

In VB.Net, use .Do instead of .With and use an anonymous Sub.


Title: Shifting Search box at the top -right  side  of the navbar with the icons on it

Which pre-existing JavaScript project will you be working on?
Search-box is not placed properly . There is no search- icons available on it.

Describe your enhancement/improvement
Search-box should be placed at the top-right side of the  title  [30 days of Javascript] and the  search-icons should be provided within it.
Anything you need to say to the Reviewers
I am a GSSoc'24 contributor .
kindly assign this issue to me
please add gssoc label to it.
Add any other context or relevant screenshots about the enhancement here.
No response

@swapnilsparsh please label this under gssoc and assign this to me

@codewithishu Thank you for providing your insight but it is not required as of now.


ERROR - mqtt:// is disabled on this system

⚠ Please verify that this bug has NOT been raised before.

[x] I checked and didn't find similar issue

🛡 Security Policy

[X] I agree to have read this project Security Policy

📝 Describe your problem
Trying to use the apprise mqtt notification agent and I'm getting this:
ERROR - mqtt:// is disabled on this system.
The URL is
mqtt://user:password@<IP_ADDRESS>:1883/UPTIME
I can't find much on this, except that Apprise state they support mqtt, but the notifications pinned tab in this repo doesn't show mqtt as available?
Thanks.
🐻 Uptime-Kuma Version
1.11.3
💻 Operating System and Arch
Ubuntu 20.04.2 LTS (GNU/Linux 5.4.0-92-generic x86_64)
🌐 Browser
Firefox 96.0.2 (64-bit)
🐋 Docker Version
Docker version 20.10.8, build 3967b7d
🟩 NodeJS Version
v10.19.0

I think the docker image needs to be rebuilt with a package.  See @louislam 's comment and beyond on Issue 125: https://github.com/louislam/uptime-kuma/issues/125#issuecomment-927139135

Thanks - I have just installed both via Exec in the container and the issue still persists, even after rebooting.

Same for me, installing all python libs
pip3 install --upgrade --no-cache-dir wheel setuptools
pip3 install --no-cache-dir paho-mqtt

using Apprise to send mqtt:

Error: 2022-02-02 23:29:40,933 - ERROR - mqtt:// is disabled on this system.
2022-02-02 23:29:40,933 - ERROR - You must specify at least one server URL or populated configuration file.
Usage:  [OPTIONS] SERVER_URL [SERVER_URL2 [SERVER_URL3]]

  Send a notification to all of the specified servers identified by their
  URLs the content provided within the title, body and notification-type.

  For a list of all of the supported services and information on how to use
  them, check out at https://github.com/caronc/apprise

Options:
  -b, --body TEXT                Specify the message body. If no body is
                                 specified then content is read from <stdin>.
  -t, --title TEXT               Specify the message title. This field is
                                 complete optional.
  -c, --config CONFIG_URL        Specify one or more configuration locations.
  -a, --attach ATTACHMENT_URL    Specify one or more attachment.
  -n, --notification-type TYPE   Specify the message type (default=info).
                                 Possible values are "info", "success",
                                 "warning", and "failure".
  -i, --input-format FORMAT      Specify the message input format
                                 (default=text). Possible values are "text",

For now, I'm using this image to solve the problem, it works ok.
FROM louislam/uptime-kuma:latest
RUN apt update
RUN apt --yes install python3
RUN apt-get install python3-setuptools
RUN pip3 install --no-cache --upgrade pip paho-mqtt

@agustinvinao - thank you for the build file. I have it pushed to my repo here:
https://hub.docker.com/repository/docker/jasonshearer/uptime-kuma-mqtt

Any thoughts on adding paho-mqtt to the build file?

Trying to help others avoid this rabbit hole. Anyone having MQTT issues and thinking paho-mqtt not being installed could be the cause of the issue, uptime-kuma docker image version 1.19.2 does not have paho-mqtt and does not need it. On version 1.19.2, MQTT works fine without paho-mqtt.

Trying to help others avoid this rabbit hole. Anyone having MQTT issues and thinking paho-mqtt not being installed could be the cause of the issue, uptime-kuma docker image version 1.19.2 does not have pip package paho-mqtt and does not need it. On version 1.19.2, MQTT works fine without needing to install anything.

That is good to know! What MQTT client is included in 1.19.2?

@dadcoachengineer Apologize, I was mistaken and didn't realize this issue was related to notifications via MQTT. I was having an issue with monitoring an MQTT server.

Agreed. Would love to see Palo-mqtt incorporated into the base image.
In the meantime.....I have this build but I do not update it on a regular basis.
https://hub.docker.com/r/jasonshearer/uptime-kuma-mqtt

It's been nearly a year and it seems this problem still exists.
Ubuntu 23.10
Using latest Uptime Kuma docker image: louislam/uptime-kuma:latest
Trying to set up Notification:
Notification Type:  Apprise
Apprise URL: mqtt://blah:blahpass@<IP_ADDRESS>:1883/homeassistant/uptime_kuma/plex
Status:  Apprise is installed
Pressing Test produces 'Process Exited with Code 1'
Docker log files show:
2024-06-16T00:48:11-07:00 [AUTH] INFO: Login by username + password. IP=<IP_ADDRESS>
2024-06-16T00:48:11-07:00 [RATE-LIMIT] INFO: remaining requests: 19
2024-06-16T00:48:11-07:00 [AUTH] INFO: Successfully logged in user moa. IP=<IP_ADDRESS>
Error: Process exited with code 1
    at ChildProcess.done (/app/node_modules/promisify-child-process/index.cjs:70:19)
    at ChildProcess.emit (node:events:517:28)
    at maybeClose (node:internal/child_process:1098:16)
    at ChildProcess._handle.onexit (node:internal/child_process:303:5) {
  code: 1,
  signal: null,
  stdout: '2024-06-16 00:48:21,526 - ERROR - mqtt:// is disabled on this system.\n' +
    '2024-06-16 00:48:21,527 - ERROR - There are no service(s) to notify\n',
  stderr: ''

}

Temporary Fix:
I was able to get MQTT notifications working again. The root cause was some breaking changes in the paho-mqtt library between versions 1.6.1 and version 2.0.0 (link). It appears that the Apprise libraries included in the uptime-kuma:latest are not yet compatible with the v2 initialization calls.
By pinning the paho-mqtt library to v1.6.1 in my Dockerfile, I was able to get MQTT event notifications via Apprise to work again. Including my Dockerfile below:
FROM louislam/uptime-kuma:latest
RUN apt update
RUN apt --yes install python3
RUN apt-get install python3-setuptools
RUN pip3 install --upgrade pip
RUN pip3 install --no-cache-dir paho-mqtt==1.6.1


Bug when formatting php inside 'script' tags

Hello,
if I have a php code inside script tags like this:
<script>
<? if (true) { ?>
alert('test');
<? } ?>
</script>

a space is being added between '<' and '?' so it becomes like this after formatting :
<script>
< ?
if (true) {
    ? >
    alert('test'); <
    ?
} ? >
</script>

this problem occures only between script tags

@tariqalkhassa what about, seems to work fine
<script>
<?php if (true) { ?>
  let objStr = JSON.stringify(obj);
<?php } ?>
</script>

Unfortunaly there are too many use cases to support <? short open tags, especially when they are discouraged from use anyways.
https://secure.php.net/manual/en/language.basic-syntax.phptags.php

PHP also allows for short open tag <? (which is discouraged since it is only available if enabled using the short_open_tag php.ini configuration file directive, or if PHP was configured with the --enable-short-tags option).

This extension just uses the HTML portion JS Beautify
You can test it out here if you want we are using the same code.
https://beautifier.io/


consul_installation: Fix binary base url for Linux x86

At the moment consul_installation_binary#binary_basename returns an invalid base url if node['kernel']['machine'] is not literally 'i386'.
Reproduced on CentOS 5.11 x86: attribute node['kernel']['machine'] is "i686" there, so the url became the following:
https://releases.hashicorp.com/consul/0.6.4/consul_0.6.4_linux_i686.zip
# 403 "Forbidden"

However it should be:
https://releases.hashicorp.com/consul/0.6.4/consul_0.6.4_linux_386.zip

This PR fixes that issue.

Current coverage is 69.02%

Merging #310 into master will increase coverage by +19.81% as of 850d71e

@@            master    #310   diff @@
======================================
  Files            7       9     +2
  Stmts          380     381     +1
  Branches         0       0
  Methods          0       0
======================================
+ Hit            187     263    +76
  Partial          0       0
+ Missed         193     118    -75

Review entire Coverage Diff as of 850d71e

Powered by Codecov. Updated on successful CI builds.


*** JIRA REQUIRED: This is an initial test for upgrading surefire to …

…get around issues with CI failing with the dummy:dummy:1.0 artifact not being able to be downloaded.
I keep seeing intermittent  failures with:
Error:    org.apache.maven.surefire:surefire-junit47:jar:2.22.0
Error:
Error:  from the specified remote repositories:
Error:    jboss-public-repository (https://repository.jboss.org/nexus/content/groups/public/, releases=true, snapshots=false),
Error:    central (https://repo.maven.apache.org/maven2, releases=true, snapshots=false)
Error:  Path to dependency:
Error:      1) dummy:dummy:jar:1.0

This comes from the maven-surfire-plugin. This was removed in 3.0.0 so this is to test that 3.0.0 works for RESTEasy and hopefully makes this issue go away.

This seems less stable so we won't be trying this.


GitOps Service: deploy to staging only, update components/gitops/staging/base/kustomization.yaml

Cherry pick https://github.com/redhat-appstudio/infra-deployments/pull/2080/commits to staging, only

Merging as the HAC-E2E tests are optional, and failing equally on other PRs as well.


DSB resection PMID:21931565

decreased DNA double-strand break resection (cf. fypo:5059, but generic)
decreased DNA resection during double-strand break repair via homologous recombination (is_a fypo:new1)

also: standardize resection vs. resectioning

decreased DNA double-strand break processing FYPO:0006686
decreased DNA double-strand break processing during double-strand break repair via homologous recombination FYPO:0006687
edit file: 186769f12d6c0b85492b700abe9d5bae4843ae5c
release: 8c59a4d07cde37fd310986b6f87ce1097b1507c9


Vulnerability roundup 84: fontconfig-2.10.2: 1 advisory

search, files

[ ] CVE-2016-5384 CVSSv3=7.8 (nixos-20.03)

Scanned versions: nixos-20.03: 82b5f87fcc7. May contain false positives.

See also: #73630

PR for unstable: https://github.com/NixOS/nixpkgs/pull/92919
PR for 20.03: https://github.com/NixOS/nixpkgs/pull/92921

This is still an issue.

oops, sorry

We only use config files. We could probably just delete everything but /etc from the derivation, or remove it from top-level.

Yeah, but we still ship the fontconfig binary in nixpkgs. Is there any reason we still need to provide config files parse-able by 2.10.x fontconfig implementations?

I doubt there is much of a reason these days, nobody will be running programs compiled against fontconfig 2.10. I just do not want to remove it now so that possible breakage caused by #73795 is not mixed with potential downfall from removing the unversioned path.

https://github.com/NixOS/nixpkgs/pull/93562 is a new attempt to address this.


sorting the content by search result

I'm doing the pg_search by title and text_body. i first want to order the record with the highest number of occurence of the search text in the title and then i want to order the record with the highest number of occurence of the search text in  text_body
person_1 = Person.create!(title: "tom Grant grant", text_body: "ji")
person_2 = Person.create!(title: "Grant", text_body: "call")
person_3 = Person.create!(title: "Grant Grant Grant",, text_body: "help")
person_4= Person.create!(title: "wick", text_body: "grant calls" )
person_5 = Person.create!(title: "john", text_body: "grant grant")

i need the out put like this
Person.search_full_text("Grant") # => [person_3, person_1, person_2, person_5, person_4]
my code

pg_search_scope :search_full_text,
                  :against => [
                      [:title, 'A'],
                      [:text_body, 'B'],
                  ],
                  :using => {
                      :tsearch => {
                          tsvector_column: "tsv_body",
                          any_word: true,
                          prefix: true,
                          dictionary: "english"
                      },
                      :dmetaphone => {:any_word => true, :sort_only => true},
                      :trigram => {
                          only: :title,
                          :threshold => 0.3
                      }
                  }

please help me to fix this

Experiment with the normalization option for :tsearch
https://github.com/Casecommons/pg_search#normalization

@nertzy  how to rank association between :title and :text_body


Blank page instead of management interface

Describe the problem
I followed the self-hosting guide and after following all the steps, hitting https://netbird.mydomain.com produces a blank page
To Reproduce
Steps to reproduce the behavior:

Set up OIDC provider (Keycloak) at keycloak.mydomain.com (and verify that https://keycloak.mydomain.com/realms/corp/.well-known/openid-configuration works)
Follow steps 1 to 5 in https://netbird.io/docs/getting-started/self-hosting
Run docker-compose up -d
Access https://netbird.mydomain.com in a web browser

Expected behavior
I expect to be able to see the management interface
Screenshots

Additional context
This is the output of curl https://netbird.mydomain.com:
<!doctype html><meta charset="utf-8"/><link rel="icon" href="[/favicon.ico](view-source:https://netbird.mydomain.com/favicon.ico)"/><meta name="viewport" content="width=device-width,initial-scale=1"/><meta name="theme-color" content="#000000"/><meta name="robots" content="noindex"><meta name="description" content="NetBird Management Dashboard"/><link rel="manifest" href="[/manifest.json](view-source:https://netbird.mydomain.com/manifest.json)"/><title>NetBird</title><link href="[/static/css/main.90cc2dd8.css](view-source:https://netbird.mydomain.com/static/css/main.90cc2dd8.css)" rel="stylesheet">NetBird Management Dashboard.

Output of docker-compose logs management
oot@netbird-basic:~/netbird/infrastructure_files# docker-compose logs management
Attaching to infrastructure_files_management_1
management_1  | time="2023-03-22T21:06:58Z" level=info msg="loading OIDC configuration from the provided IDP configuration endpoint https://keycloak.mydomain.com/realms/corp/.well-known/openid-configuration"
management_1  | time="2023-03-22T21:06:58Z" level=info msg="loaded OIDC configuration from the provided IDP configuration endpoint: https://keycloak.mydomain.com/realms/corp/.well-known/openid-configuration"
management_1  | time="2023-03-22T21:06:58Z" level=info msg="overriding HttpConfig.AuthIssuer with a new value https://keycloak.mydomain.com/realms/corp, previously configured value: "
management_1  | time="2023-03-22T21:06:58Z" level=info msg="overriding HttpConfig.AuthKeysLocation (JWT certs) with a new value https://keycloak.mydomain.com/realms/corp/protocol/openid-connect/certs, previously configured value: "
management_1  | time="2023-03-22T21:06:58Z" level=info msg="overriding DeviceAuthorizationFlow.TokenEndpoint with a new value: https://keycloak.mydomain.com/realms/corp/protocol/openid-connect/token, previously configured value: "
management_1  | time="2023-03-22T21:06:58Z" level=info msg="overriding DeviceAuthorizationFlow.DeviceAuthEndpoint with a new value: https://keycloak.mydomain.com/realms/corp/protocol/openid-connect/auth/device, previously configured value: "
management_1  | time="2023-03-22T21:06:58Z" level=info msg="overriding DeviceAuthorizationFlow.ProviderConfig.Domain with a new value: keycloak.mydomain.com, previously configured value: "
management_1  | 2023-03-22T21:06:58Z INFO management/server/telemetry/app_metrics.go:144: enabled application metrics and exposing on http://<IP_ADDRESS>:8081
management_1  | 2023-03-22T21:06:58Z INFO management/server/account.go:655: single account mode enabled, accounts number 0
management_1  | 2023-03-22T21:06:58Z INFO management/cmd/management.go:221: running gRPC backward compatibility server: [::]:33073
management_1  | 2023-03-22T21:06:58Z INFO management/cmd/management.go:253: running HTTP server and gRPC server on the same port: [::]:443

After some debugging, it looked like the old version of docker-compose (and/or its interaction with podman) I was using wasn't passing through the $AUTH_AUTHORITY env variable.
Replacing podman with docker-engine and using the docker-compose-plugin and it all worked.

Any chance you could elaborate please? Am having the same issue but am not familiar with Docker...


How many months back will scaffold search for reuser?

Do we know how many months back for previous scans for a project that scaffold will search for doing the reuser for an incremental scan?
I think it might be the previous 12 months, but can't confirm easily without doing a scan, and might need this extended to a longer time frame, perhaps 24 months.

I have confirmed it's only going back 12 months:
cncf/grpc: no prior upload found in preceding 12 months, skipping reuser
We should make this 24 months if it's not too much work.

if this is a quick fix can we get this in the next week?

if this is a quick fix can we get this in the next week?

I'll start working on it on Wednesday - have a talk in San Diego on Tuesday.

Sounds good, please coordinate with me before pushing any new code to production so we can be sure that Scaffold is not running.


refactor: Moving elementwise core to impl (FX Converter Refactor [3/N]) <Target: converter_reorg_proto>

Description
Moves Elementwise and Unary core functions to impl in preparation for moving other ops
Type of change
Please delete options that are not relevant and/or add your own.

Refactor

Checklist:

[x] My code follows the style guidelines of this project (You can use the linters)
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas and hacks
[x] I have made corresponding changes to the documentation
[x] I have added tests to verify my fix or my feature
[x] New and existing unit tests pass locally with my changes
[x] I have added the relevant labels to my PR in so that relevant reviewers are notified

@wushirong Can you review this PR. This is gating release of 1.4

Closing in favor of #2070


Authenticate hook using strategies parameter incorrectly?

The README for the authenticate hook says that "You can chain multiple strategies" (by passing an array)  but in the code it only uses the first value of the array (and even ignores that if hook.data.strategy is set).
It seems that the array passed into the hook is meant to limit the strategies that can be used, but that doesn't seem to be happening, unless I'm missing something.

I agree with @hubgit.
I change my authentication config to set only the 'jwt' strategy:
app.service('authentication').hooks({
    before: {
      create: [
        authentication.hooks.authenticate('jwt'),
      ],
    },
  });

and I am still able to get an accessToken thanks to this payload:
{
	"strategy": "local",
	"email"<EMAIL_ADDRESS>	"password": "passw0rd"
}

You do but it's not going to contain any useful information (as in it will be anonymous). I agree that it should error in that case as well. The related issue is #392

There seems to be an issue with the fact that you can pass an array of strategies but only the first one is used. Is there a reason why you allow for an array but only use the first item?
      if (Array.isArray(strategies)) {
        strategy = strategies[0];
      } else {
        strategy = strategies;
      }

When "local" strategy is first, this allows for credentials to be used to create a token, an not allowing anonymous tokens, but using the "jwt" option first allows for anonymous but does not allow credentials to be used.

The short answer is, "it's complicated".
You're only looking at one part of the authentication. Yes, the code you are pointing to it does not matter. What we are doing there is if you didn't provide a strategy when you try to authenticate with the server, we just grab the first one that has been registered server side and assume that is the one you are trying to auth with. Not ideal but it's not likely to happen because feathers-authentication-client forces you to provide one.
Although looking at this again with fresh eyes I think we should just force the strategy option from the client and I can update this to support arrays a bit better.

I think we talked about doing this before. Totally makes sense to force the strategy in order to avoid any ambiguity.

Note that if the strategy is forced (which makes sense) there's still a bug here, as it's not actually validating that the strategy from the client is in the server's array of allowed strategies.

Hi guys, I receive the message...
error: NotAuthenticated: Invalid login
at new NotAuthenticated (C:\Projetos\SkeewsMe\node_modules@feathersjs\errors\lib\index.js:93:17)
at app.authenticate.then (C:\Projetos\SkeewsMe\node_modules@feathersjs\authentication\lib\hooks\authenticate.js:76:31)
at
I using
"@feathersjs/authentication": "^2.1.7",
"@feathersjs/authentication-jwt": "^2.0.1",
"@feathersjs/authentication-local": "^1.2.1",
"@feathersjs/authentication-oauth2": "^1.1.0",
i my user hooks
module.exports = {
before: {
all: [],
find: [],
get: [],
create: [local.hooks.hashPassword({ passwordField: 'password' })],
thanks for help me

Since https://github.com/feathersjs/authentication/pull/663, the strategy is now being verified against the list registered in the hook.


Using ActiveMQ and bitronix-transaction-manager redelivery issue

Hello, I'm trying to test bitronix-transaction-manager with activemq. I'm running into this problem in my test cases. It would be great if anyone can help.
When the JMS listener fails to process the message with an exception, the transaction gets rolled back, but  It's not following the redelivery policy I defined to redeliver three times on exception with delay of 5 seconds for each retry .
This is the configuration.
resource.mq1.className=org.apache.activemq.ActiveMQXAConnectionFactory
resource.mq1.uniqueName=jms/connectionFactory
resource.mq1.minPoolSize=0
resource.mq1.maxPoolSize=10
resource.mq1.driverProperties.brokerURL=tcp://localhost:61616
resource.mq1.driverProperties.redeliveryPolicy.initialRedeliveryDelay=0
resource.mq1.driverProperties.redeliveryPolicy.redeliveryDelay=5000
resource.mq1.driverProperties.redeliveryPolicy.maximumRedeliveries=3
resource.mq1.shareTransactionConnections=false
resource.mq1.automaticEnlistingEnabled=true
resource.mq1.allowLocalTransactions=true
resource.mq1.applyTransactionTimeout=true

Well, looking at your description of the problem, that doesn't look like a transaction manager's issue, but rather a MQ one.
I'd suggest you to investigate deeper to get a better idea of what's going on, and eventually be able to point your finger exactly where the problem is.


fix the case of per-function custom Role

I had a trouble with per-function custom Role, and needed to fix it.
I llooked around issues and pull requests, and found #87 and #88.
My custom role is in the same stack, so @bencodner 's solution does not apply.
So, I wrote another patch. It will cover cross-stack cases.

Any progress on this PR?  I seem to be having the same issue that this PR is supposed to fix (custom roles, same stack, for each lambda function)

could u add support for serverless-iam-roles-per-function?

Sorry, it works perfect with serverless-iam-roles-per-function. Why it's not merged yet?

tests added here https://github.com/HyperBrain/serverless-aws-alias/pull/157

Any update on this?  Did the merge fail just because the code coverage percentage was slightly decreased?

@dwkerwin serverless-iam-roles-per-function works perfect in latest release 2.0.1


[SPARK-12950][SQL] Cache newly accessed hashtable entry in BytesToBytesMap

JIRA: https://issues.apache.org/jira/browse/SPARK-12950
As described in JIRA, it is observed that when aggregate with grouping keys, profiling show that lookup in BytesToBytesMap took about 90% of the CPU time.
This patch doesn't change how the lookup works. But we find that every time it is going to call safeLookup method, we need to look up for a key from the beginning. So if we want to find a same key, we will perform the same lookup process (comparing hashcode, comparing keys,hitting collision...). This patch tries to cache the newly accessed key and returns the entry early if it is in cache.

Test build #49917 has started for PR 10877 at commit 0add15c.

Test build #49917 has finished for PR 10877 at commit 0add15c.

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

Merged build finished. Test PASSed.

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

Did you do any performance test?

I don't think this could save much, even with cache hit. It will still calculate the hash, compare the key as bytes, update the address.

Ok.

The point is if you need to find your key after several comparings, next time you face the same key, you still need to go through this process of wrong comparings. With cache, you can get the address early without above.

I don't think you can get final address without any hashcodeing or key comparing.

With the fill ratio under 70% (will grow after that), it usually does one probe most of the time. Another question is that, the cache hit rate will be very low in practice (considering thousands of unique keys).

If so, I am curious what makes lookup cost much cpu time, hashcoding? key comparing?

Profiling using jvisualvm will help a lots, here are the things that take most of the time:

decode address from Long to baseObject and offset
calculate hash code
compare the bytes (equality check)


Merge branch 'master' into live

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Add a jpeg decoding function that is based on libjxl internals.

This is now equivalent to the lossless-jpeg roundtrip version, but is faster because we do not create the actual jxl bitstream.

Nice!
Should we add something like this to the libjxl decoder so you can take a jpeg bitstream as input and decode it as if it's a recompressed jxl file? Doing that in a streaming/progressive way will be tricky and quite a bit of work, but if it works already in the simple way (full input bitstream available), that's already quite useful for many use cases: basically it can mean that applications can drop a dependency on libjpeg-turbo or whatever they're currently using to decode jpeg, and use libjxl for both jpeg and jxl decoding...

Yes, that would be the long-term plan. I did not want to add it to djxl yet, as it is still a work in progress and there are no tests yet, I am only testing it through the benchmark with a few input files.


Add release note on publishing ng

for https://github.com/threefoldtech/js-ng/issues/434

js-ng.md is just a copy of README.md and it's not referenced anywhere.
@xmonader I think we need to move this section to another file or README.md and remove this duplicated one?

Codecov Report

Merging #563 (c8fabd5) into development (51fa5bc) will decrease coverage by 0.80%.
The diff coverage is n/a.

@@               Coverage Diff               @@
##           development     #563      +/-   ##
===============================================
- Coverage        70.68%   69.87%   -0.81%
===============================================
  Files               84       84
  Lines             3892     3818      -74
===============================================
- Hits              2751     2668      -83
- Misses            1141     1150       +9

Impacted Files
Coverage Δ

jumpscale/servers/gedis/server.py
58.20% <0.00%> (-7.59%)
:arrow_down:

jumpscale/core/executors/remote.py
73.68% <0.00%> (-3.59%)
:arrow_down:

jumpscale/clients/redis/redis.py
85.29% <0.00%> (-3.28%)
:arrow_down:

jumpscale/servers/openresty/server.py
69.66% <0.00%> (-2.51%)
:arrow_down:

jumpscale/core/base/store/__init__.py
64.60% <0.00%> (-2.07%)
:arrow_down:

jumpscale/core/executors/command_builder.py
90.00% <0.00%> (-0.91%)
:arrow_down:

jumpscale/core/executors/local.py
86.66% <0.00%> (-0.84%)
:arrow_down:

jumpscale/core/logging/logging.py
93.56% <0.00%> (-0.79%)
:arrow_down:

jumpscale/core/events/__init__.py
88.37% <0.00%> (-0.76%)
:arrow_down:

jumpscale/servers/openresty/location.py
91.17% <0.00%> (-0.72%)
:arrow_down:

... and 16 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 51fa5bc...c8fabd5. Read the comment docs.


Use Alluxio Java API pom dependency, commit job service does not work!

Page
Include the link to the page you are reporting a defect
https://docs.alluxio.io/os/user/stable/en/api/Java-API.html
Summary
What is the defect and suggested improvement

In my project ,for warm up data, i use Alluxio Java API to submit a load job service, there happens an error !

alluxio version is 2.9.3
When i change the pom dependency to alluxio-core-client-fs , it does work.
<dependency>   <groupId>org.alluxio</groupId>   <artifactId>alluxio-core-client-fs</artifactId>   <version>2.9.3</version> </dependency>
So i suggest this version client may have some changes ,but the document not updated.

Thanks for reporting the issue!


Non-uniform memory sampling

An easy way to improve the learning efficiency might be, focusing on memories the agent is uncertain about. This can be achieved by couppeling the sample probability to the uncertainty.

This also means that the uncertainty over the states has to be continuously be updated

Run experiment 169

Evaluate impact of hyperparameter


Document Photo API terms of use

Other applications may want to display/download pictures from OpenAEDMap (e.g. MapComplete).
It may be useful to document usage terms for photo API (i.e. maximum number of requests allowed).
(though usually they will do so from URL on single OSM object and cache the request.)
Also, does image stay hosted if no longer used in OSM? If so, for how long?
(given there are no plans to host images elsewhere.)

We don't have any hard limits on number of requests (it's fair use). The userbase is not significant and I don't think it's worth setting up atm. We usually have 2-stage cache system (Cloudflare+Varnish) so the requests are pretty lightweight. Cloudflare may eventually block abusive clients.

Also, does image stay hosted if no longer used in OSM? If so, for how long?

Currently, there is no cleanup system for unused images (except for manually reported ones). I would say it's very low priority. Perhaps, that's something we could think about if we grow further and it starts becoming a real issue. Today we are talking just megabytes of wasted space.

Closing as not planned. Feel free to re-open when needed.

No problem and thank you! It's important that we consider such issues from time to time. I like how your questions are well thought out.


New sentry options

@fgimenez I think this PR doesn't cover the current requirements for the secure validator to work.

sentry nodes: --sentry <list of validator addresses separated by spaces>
validator nodes: --sentry-nodes <list of sentry addresses separated by spaces>

thanks for the heads up @ilhanu for now we are only changing public nodes, the changes to validator confirm are more involved in a VPN setup.


Open Quickly

Will fix #42
Video demo
https://user-images.githubusercontent.com/17158860/159169615-cbe2775a-7efe-4cc3-b3ca-88aaedfcf18c.mov
Current state

I'm experiencing difficulties optimizing "Open Quickly".
The problem is that, because we are using NavigationLinks in the sidebar if the NavigationLink has not appeared in the sidebar (because folders containing them aren't expanded), then navigation to file won't work because there won't be NavigationLink in the UI hierarchy which is responsible for the specified ID.
And we cannot add a view to NavigationView to handle this behavior because it will lead to views rebuilding twice instead of once and, I guess, memory leaks. (Right now this branch contains this way and it's horrible in terms of memory usage when there are many files opened)
And expanding all the parent directories to the file is not really the way it should be done, because there can be a lot of subdirectories and files which will mess up the whole sidebar content. Especially if we are speaking about searching inside caches and modules directories.
Obviously, I cannot use NavigationLink in the OpenQuicklyPanel, because it's a different window not related to the main one. I'm selecting file by changing WorkspaceDocument.selectedId property.
So, what should we do about that? Because it may not allow us to make lookups and other features as well.

Right now I'm working on a solution for this problem without changing how we implement SideBar. So, I'll write if it won't work.

To be honest I'm not quite happy yet with how the sidebar is implemented yet. I would rather have a main window which hosts the editor/placeholder and the sidebar should be embedded as a package. But as of now (using NavigationLink) it needs to be as it is now. One option would be to ditch the List {} and implement a completely custom one imitating the native look.

Looks good! Does it fix the issues described earlier?

No, I've tried, but I think it would be better for someone to fix them in #123.

Ok so for now some files will not open if not present in the sidebar. Correct?

Ok so for now some files will not open if not present in the sidebar. Correct?

They will open because I've made a small workaround with WorkspaceCodeFileView inside NavigationView at the initial state. So, right now if the user opens anything through the sidebar or "Open Quickly" everything works fine.

Perfect!

Fixes #42

Implements overlay for "Open Quickly"
Creates a new module with OverlayPanel
OverlayPanel automatically closes when it's out of focus (as "Open Quickly" and other overlays do in Xcode)
Some changes in how navigation is handled (just workaround at the moment)

Video demo
demo_openquickly.mov
Screenshots


Loosen Python dependency requirements and upgrade dependencies

Motivation and context

Now that we're using pip-compile-multi, we don't have to strictly pin our dependency versions in the .in files. We can define loose version requirements and let PCM find appropriate upgrades when we want.
Make use of that and loosen up most of the version constraints in base.in and production.in (testing.in and development.in aren't as important, so I'll deal with them some other time). Then upgrade dependencies within the new constraints.
This fixes some of the issues reported by Snyk (vulnerabilities in sqlparse and NumPy).
Some packages listed in base.in were used only transitively, or seemingly just not at all. Remove those entirely.
Don't upgrade some of the packages, because I feel that it could cause more breakages that I currently have time to deal with. I'll come back to them another time.
How has this been tested?

Not at all! Relying on CI for this.
Checklist

[x] I submit my changes into the develop branch
[ ] I have added a description of my changes into the CHANGELOG file
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[ ] I have linked related issues (see GitHub docs)
[ ] I have increased versions of npm packages if it is necessary
(cvat-canvas,
cvat-core,
cvat-data and
cvat-ui)

License

[x] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.

Looks like I bit off more than I could chew here. I'll redo this via more gradual changes.


pytest_plugin.py: replace deprecated pkg_resources with importlib.metadata

This PR replaces the deprecated use of pkg_resources  with importlib.metadata.
As importlib.metadata.version() is only availeable for python>=3.8 I add importlib-metadata as dependency for python<3.8 (not sure if I have done this right).

Hi @lorddavidiii thanks for the contribution.
We don't actually test on python <3.8 so as long as your happy with this on 3.7 my only requirements is the tests pass.
(If we get complaints though about breaking some older version I likely will revert)


Border not working for custom module

Default border control border not applying and when i try put border on specific class then it is also not working.
$advanced_fields['border'] = array(
'label'    => esc_html__( 'Text', 'bpd-booster-pack-divi' ),
'css'      => array(
'main' => array(
'border_styles' => "{$this->main_css_element} .bpd-flip-box-wrapper",
// also tried     'border_styles' => "%%order_class%% .bpd-flip-box-wrapper",
),
'defaults' => array(
'border_radii' => 'on|3px|3px|3px|3px',
),
),
'toggle_slug' => 'action_button_style',
);
I am using this code and default border radii also not apply.

QA Checklist
Reporter

[ ] Helpful title used

Helpful title format: Scope(s) :: One Sentence To Describe The Issue
Example: VB :: Floating Toolbar :: Jumps To Top Of Page

[ ] Detailed description

Description should usually be 150+ characters.

[ ] Steps to reproduce included
[ ] Support Ticket URL included
[ ] Image and/or gif

Developer

[ ] Steps to reproduce proven solved
[ ] Test included and coverage acceptable and all Tests passing?
[ ] PR description has layman's description of problem and solution and "Areas Affected" listed/explained?
[ ] PR branch includes latest commits from master branch (at the time of checking)?
[ ] Estimated points on issue
[ ] Proper code documentation and conventions?
[ ] Proper RTL appearance
[ ] i18n used if applicable
[ ] Ajax/Form used? Nonce used? Perms checked? All user input sanitized and validated? Intentional unsanitized/unescaped functions used?
[ ] Database saved/updated? Data sanitized and validated?
[ ] Variable output properly escaped and validated?

Peer Review

[ ] PR branch includes latest commits from master branch (at the time of checking)?
[ ] Steps to reproduce proven solved
[ ] Affected areas are tested and ok
[ ] Proper code documentation and conventions
[ ] Proper RTL appearance
[ ] i18n used if applicable
[ ] Test(s) included and coverage acceptable
[ ] Ajax/Form used? Nonce used? Perms checked? All user input sanitized and validated? Intentional unsanitized/unescaped functions used?
[ ] Database saved/updated? Data sanitized and validated?
[ ] Variable output properly escaped and validated?

Quality Assurance

[ ] All tests passed on CircleCI.
[ ] Steps to reproduce passed, the fix works.
[ ] No errors in console or output
[ ] Affected areas are tested and ok
[ ] The fix does not change a customer's website in any surprising or unintended way

[ ] If Divi's design is adjusted, settings are migrated gracefully and invisibly

Design Review

[ ] UI is to spec
[ ] UX is to spec
[ ] Design is to spec
[ ] No surprising visual changes to a customer's website will be experienced after updating

[ ] Any/all visual changes are migrated gracefully

Dev Review

[ ] Test(s) included and coverage acceptable
[ ] Ajax/Form used? Nonce used? Perms checked? All user input sanitized and validated? Intentional unsanitized/unescaped functions used?
[ ] Database saved/updated? Data sanitized and validated?
[ ] Variable output properly escaped and validated?
[ ] i18n used if applicable
[ ] Implementation acceptable, DRY, Scaleable, future safe
[ ] Best practices used, cleverness and grokage, ES6 use, etc.

The format of your border configuration is incorrect. It should be nested under the default key as shown in the documentation here


Library view text is misaligned when displaying list (overflow, line wrap)

When the text has enough horizontal space to expand:

When the window width is reduced, the text reflows and line-wraps (the the rightmost column), causing the other columns' content to misalign:

Suggestion: row height should be locked to a fixed dimension, text should have ellipsis / crop (this will prevent really long titles or author names to mess-up the layout)

fixed in 1c42a18

PR: https://github.com/readium/readium-desktop/pull/442


bug: packaged version is wrong

the version packaged into ./internal/version/detail.go is lagging behind the tags
e.g. when you go install the v0.4.1 tag and run go version you get alpha 0.3.1 version details:
Version:              "0.3.0-alpha+e5f64d9.32-improve-install-story.dirty"

https://github.com/davidalpert/go-git-mob/blob/v0.4.1/internal/version/detail.go#L29

this is not a problem in the go-releaser-built binaries.
when I downloaded go-git-mob_0.4.1_Darwin_arm64.tar.gz from the v0.4.1 release page that binary has the correct version:
❯ ~/Downloads/go-git-mob_0.4.1_Darwin_arm64/go-git-mob version                                                                                git-mob 0.4.1+14743e9
❯ ~/Downloads/go-git-mob_0.4.1_Darwin_arm64/go-git-mob version -oj                                                                                {
  "app_name": "git-mob",
  "build_date": "2022-07-17 18:03:11.427311225 +0000 UTC m=+0.021318129",
  "core_version": "0.4.1",
  "branch": "main",
  "commit": "14743e9",
  "commit_summary": "release notes for v0.4.1",
  "dirty": false,
  "dirty_modified": false,
  "dirty_staged": false,
  "dirty_untracked": false,
  "working_state": "",
  "summary": "",
  "user_agent": "git-mob/0.4.1+14743e9 (user: dalpert) darwin (arm64)",
  "version": "0.4.1+14743e9"
}
❯

so this problem seems limited to the sequence of steps inside make release which result in checking in the version/detail.go file


create-react-app v2.0.4 // styled(MyComponent) yields error 1

System:

OS: Windows 10
CPU: x64 Intel(R) Core(TM) i7-4700HQ CPU @ 2.40GHz
Memory: 7.03 GB / 15.95 GB

Binaries:

Yarn: 1.7.0 - C:\Program Files (x86)\Yarn\bin\yarn.CMD
npm: 6.0.1 - C:\Program Files\nodejs\npm.CMD

I am trying to migrate from<EMAIL_ADDRESS>to<EMAIL_ADDRESS>styled-components@latest works like a charm <3 in our CRA v1 setup, but breaks on CRA v2.
It only breaks on yarn run build && yarn run serve but yarn run start is perfectly fine.
The error imessage links to a component that was working perfectly fine prior to this upgrade.
const ThumbnailImage = styled(MyImageComponent)`
  max-width: 100%;
  max-height: 100%;
  height: auto;
  width: 100%;
`

MyImageComponent is a function which execute some logic and return a React component.
Anybody running<EMAIL_ADDRESS>and styled-components@latest having issues with the styled() function? Any idea what may be the issue here?
It seems that MyImageComponent is not a valid tag. But why?

Great question for the React repo. As you can see it’s a React core
dependency we consume, so there’s probably nothing to be done on our end.
On Fri, Oct 5, 2018 at 1:57 PM Sébastien Fichot<EMAIL_ADDRESS>wrote:

System:

OS: Windows 10
CPU: x64 Intel(R) Core(TM) i7-4700HQ CPU @ 2.40GHz
Memory: 7.03 GB / 15.95 GB

Binaries:

Yarn: 1.7.0 - C:\Program Files (x86)\Yarn\bin\yarn.CMD
npm: 6.0.1 - C:\Program Files\nodejs\npm.CMD

[image: image]
https://user-images.githubusercontent.com/5808108/46551644-900c0c80-c8d8-11e8-9b82-b0edd5f3b73d.png
I am trying to migrate from<EMAIL_ADDRESS>to
<EMAIL_ADDRESS>styled-components@latest works like a charm in our CRA v1 setup, but
breaks on CRA v2.
It only breaks on yarn run build && yarn run serve but yarn run start is
perfectly fine.
The error imessage links to a component that was working perfectly fine
prior to this upgrade.
const ThumbnailImage = styled(MyImageComponent)  max-width: 100%;  max-height: 100%;  height: auto;  width: 100%;
MyImageComponent is a function which execute some logic and return a
React component.
Anybody running<EMAIL_ADDRESS>and styled-components@latest
having issues with the styled() function? Any idea what may be the issue
here?
It seems that MyImageComponent is not a valid tag. But why?
[image: image]
https://user-images.githubusercontent.com/5808108/46551721-c8134f80-c8d8-11e8-9fbb-1c7d3813e3da.png
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/styled-components/styled-components/issues/2068, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAiy1rRF_Z46ww6o8l-WPeYbvZj0OOAUks5uh512gaJpZM4XKqzB
.

Thanks @probablyup


Not really an issue. iOS 14

Hi, since public iOS 14 version release I detect some strange behaivour in accessories (some of them). Some of then become "No answer" after working for years, and I can't say why.
This is started from my home automation center (appletv) being updated to iOS 14. Same time, enabling the home automation center on other device iOS12 or iOS13 (i have both) making the accessories to work again.
Offcourse I am talking about self-made (not certified) devices.
Is there any major changes in HAP or the way its implemented in iOS 14 we need to know to understand what happened?
Thanks in advance.

i don't really know... but to be clear: shelly devices are ok, right?

It's not about hardware, it's about software. @rojer which Shelly firmware you're referring to?

https://github.com/mongoose-os-apps/shelly-homekit
i replied thinking it was filed there, didn't realize this was notification for an issue in the adk repo.
please ignore my comment.

@rojer no, its good u r here :) I have shelly too, will flash it with your fw to try in my home. this will definately shed some light on the problem nature

https://github.com/mongoose-os-apps/shelly-homekit
@rojer your fw works just great. with or without home hub active 🥇

The problem was fixed. The accessory discovery sf flag was not triggered when paired (it was done intentionally to connect single accessory to multiple homes)
After sorted to HAP requirements - all come alive. Closed.

@d4rkmen Which firmware was it?

@maximkulkin it was mongoose http-server based library. Now I can say for sure: old iOS does not pay attention to the mDNS txt record sf flag. And new iOS14 update fixed this. It filters paired accessories by this flag now, and reports unavailable status.

@d4rkmen so, you're saying that mongoose library initially didn't work with your hacks, and then you removed the hacks and it worked?

From the start the txt record update was not implemented, then I have found it convenient and left as is :) So was not really a hack. When implemented record update - all accessories came back online


Feature: Abillity to Copy properties of markers to a new marker

First off, incredible plugin. It took a few minutes to really understand how to use it, but its so powerful I cant imagine using anything else.
Praise aside, the biggest problem I have with it now is the inability to edit markers. I know its on the possible roadmap if this amazing plugin gets continued support, but making a small mistake while making a big area marker and having to go back and fix it later is a pain.
As an interim solution, what about the ability to copy a marker's properties to a new marker? It would allow someone to 'edit' a marker by creating a copy, updating the things they want to change (fill opacity got me twice now on a big area, and the height got me once), while leaving all of the other info like Description, Name, and Edges alone so you don't need to trace out the area for the 4th time.

Obviously, if this is as 'complex' as adding the ability to Edit a marker, that would be the preferable solution, but Ill leave the decision to you.

Thank you for your kind words :)
Your idea how to implement it sounds interesti

Thank you for your kind words :)
I think editing a marker should be the same effort like cloning a marker, but it sounds very interesting to add anyways. I'll try to find some time in the next days (exam time...) to implement both.
Sadly i'm struggling with multiversion support, because my command system (i use minecrafts inbuilded) do not work in other versions than the one implemented (1.19.2 for this project). Currently that is on top of my todo list

Added :)
https://modrinth.com/mod/bmarker


Request sent in XML instead of JSON when com.fasterxml.jackson.dataformat.xml.XmlMapper exists in the classpath

We are running into issue with request sent in XML format instead of JSON, when com.fasterxml.jackson.dataformat.xml.XmlMapper exists in the classpath. This can be fixed by specifying the content-type to application/json, but it's very cumbersome to do this each time we want to make Vault request.
Also in certain places such as ClientCertificateAuthentication the content-type is set to null, resulting in the MappingJackson2XmlHttpMessageConverter being used and XML being sent

That's fixed now with 1.0.0.BUILD-SNAPSHOT available from https://repo.spring.io/snapshot/.


Inbound sample code is not working

Executing inbound sample code, freeswitch is returning -ERR command not found
{'Content-Type': 'command/reply', 'Reply-Text': '-ERR command not found'}

Could you give context?

Hello,
i got the same issue runnning https://github.com/Otoru/Genesis/wiki/Quickstart#inbound-socket-mode.
python test_sample-inbound-socket-mode
{'Content-Type': 'command/reply', 'Reply-Text': '-ERR command not found'}

I'm closing this issue due to inactivity.
In case someone has the same problem in the future, please open a new issue providing more information about the environment where the problem happened, as well as freeswitch logs and a network capture (wireshark or ngrep) filtered with the communication between the client in python and the freeswitch server.


Add link to Atom plugin python-black

Pull request #456 linked a Black for Atom named atom-black in the Readme.  A plugin named python-black already existed. I suggest to link both in the Readme.

Pull Request Test Coverage Report for Build 665

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage remained the same at 96.074%

Files with Coverage Reduction
New Missed Lines
%

tests/test_black.py
1
97.72%

Totals

Change from base Build 659:
0.0%

Covered Lines:
2692

Relevant Lines:
2802

💛  - Coveralls

That makes sense.
Both packages provide similar functionality:

Formatting with a shortcut
Toggle for formatting on save
Format shortcut in menu -> packages.

Minor advantages for python-black:

Supports non-default line length
Python-black is more commonly used. 422 downloads vs 47 downloads.

Python-black would be my preference, since:

I was using it already (and so were others),
It supports non-default line lengths.

In the end, it's your choice.

Hello, author of atom-black here. There's no sense in having two plugins; I clearly could have googled better. I would just archive mine, but it has some minor advantages (better handling of unsaved files, support for all of black's options by changing the black command used). That being said, this repo isn't the place to discuss this. How about I contact the author of atom-python-black and attempt to merge the relevant changes in, and update this PR once things have been resolved?

If you can resolve this between the authors of the two plugins and then let me know, that's definitely the way to go. Keeping my fingers crossed!


Questions regarding performance

In your presentation you talk about achieving a 12x performance improvement for validating a list of dicts with length of 100 elements, is this test consistently accomplishing the same numbers for bigger lists?
The other question that I have is regarding the decision to choosing Rust as the language for the core of pedantic, which was the criteria to choose Rust over other languages like C or C++ or even dotnet?

Thanks so much for attending.

In your presentation you talk about achieving a 12x performance improvement for validating a list of dicts with length of 100 elements, is this test consistently accomplishing the same numbers for bigger lists?

Yes, but I would encourage you to checkout the code, build an optimised binary and run your own benchmarks. Once you have rust nightly installed (which is very simple with rustu), building and installing pydantic-core is as simple as pip install ..

choose Rust over other languages like C or C++ or even dotnet?

I'm not clever enough to write good c, AFAIK there's no good python bindings for C#.
The only other viable alternative was C++ and I've heard a lot of good things about rust vs C++.
In the end it was rust vs. python, not rust vs. something else.

thank you for your prompt reply. keep up the awesomely good work.
I did try to write a binding for python using the C driver of MongoDB and I have some troubles with dependencies even using CMake, so I will check Rust as you suggest.
I optimistically hope to sponsor you this year.


feat(core): add support for prefers-reduced-motion

📥 Proposed changes
Turn off all transitions and animations when the user prefers reduced motion
☑ Submission checklist

[x] I have read the CONTRIBUTING doc
[x] yarn build works locally with my changes
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have added necessary documentation (if appropriate)

💬 Further comments
affects: @fremtind/jkl-core

Later on

We also need to define a strategy for javascript animations. I don't think we can express our visual identity properly without them, and they will probably be most relevant for prefers-reduced-motion.


Supporting the  tag

First I would like to thank you for approving Open Surge on Flathub :)
Parts of the project description are being displayed as (null). In the appdata XML, I'm using the  tag for emphasis, as described in https://www.freedesktop.org/software/appstream/docs/chap-Metadata.html#tag-description . It turns out that whatever appears inside ... is being rendered as (null).

      How do I play?

      You can play using a keyboard or a joystick.

Can it be fixed? Is the  tag supported? The appdata is being validated correctly.
Thank you!

Lists don't work as well, at least not the unordered ones.
In contrast to the  tag they don't show up at all.

I tracked it down to appstream-glib issue and reported it at https://github.com/hughsie/appstream-glib/issues/350.


Update README.md

Fix Travis build status URL, and link to serde as a use case of syntex instead of regex_macros (which doesn't currently have a syntex edition).

Thanks!


ELK stack Open Source (or SSPL): ElasticSearch, Logstash, Kibana

Project description
Upstream has stopped distributing builds of their SSPL-licensed software, which means that we currently only have recent packages for the non-SSPL variants of these packages.
This is a loss, even for non-SSPL users for whom the SSPL-based VM test on ofborg and hydra serves as a canary.
This adds to the other benefits of adding a source-based package.
Metadata

homepage URL: https://www.elastic.co/
source URL: https://github.com/elastic
license: sspl
platforms: unix, linux, darwin, ...

@dpausp here

seems like they are opening up again https://www.elastic.co/blog/elasticsearch-is-open-source-again

Yeah. Originally even building the SSPL stuff was troublesome, but having AGPLv3 as an option would make that even more interesting to take a look at again. This requires version 8.16+.


Add the documentation for http_server pkg

Originally opened as dart-lang/sdk#12740
This issue was originally filed by<EMAIL_ADDRESS>
I find this package very useful, but lack documentation. While we can just read the source, I think it is tedious to switch tab between the source file.
http://pub.dartlang.org/packages/http_server

 Comment by skabet

Closign for now. Let me know if you think it could use more information.

Added Fixed label.

 Comment by skabet

The readme is now up. PTAL: http://pub.dartlang.org/packages/http_server

Set owner to @skabet.
Added NeedsInfo label.

 Comment by iposva-google

Added Area-Pkg, Triaged labels.

 Comment by anders-sandholm

Removed Library-HttpServer label.
Added Pkg-HttpServer label.

 Comment by sethladd

cc @skabet.
Removed Type-Defect label.
Added Type-Enhancement, Library-HttpServer labels.


Description of inputs for AzureRmWebAppDeployment

Hi, apologies if this is a dumb question but I can't find any description of what inputs are available for

task: AzureRmWebAppDeployment@4
Are there any references you could point me at?
Thanks for your help

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

ID: 7a40a853-83d6-844a-4752-8b00909c9ffb
Version Independent ID: e3121964-de38-2a57-2414-c1e29a880477
Content: Deploy an Azure Web App - Azure Pipelines
Content Source: docs/pipelines/targets/webapp.md
Product: devops
Technology: devops-cicd
GitHub Login: @juliakm
Microsoft Alias: jukullam

Hi @timothies , you can find it here.

Thanks @iamsrivatsa.  That page looks like its missing some of the inputs for AzureRmWebAppDeployment@4 (e.g. packageForLinux) but I'll raise the issue on that page.
Really appreciate your help!

Hi @timothies, the inputs are not missed in the document. The  packageForLinux is the alias used for Package same is already documented in the Arguments table.

@timothies Thank you for the feedback.
Please refer to the source documentation and if anything is missing from the doc referenced by @iamsrivatsa please comment below.

Thanks guys,  perhaps I'm wrong and all the inputs are listed but the alias packageForLinux is definitely missing though.

The changes are pushed. You can now see the updated document.

Thanks guys, appreciate your help


pre-commit: test PR76203

PR Link: https://github.com/llvm/llvm-project/pull/76203

Can it be tested again?

Can it be tested again?

Done.


Redux package: access to login & logout functions

We currently have not access to login & logout function into Redux package.
Does it make sens to expose react-oicd actions to do that? (it's already possible into Context package)

authenticating props for OidcProvider is too exposed by context package but not by redux package.
Ta wrap it, is it possible to obtain:

authenticating props into redux Oidc provider
login & logout functions into Oidc entry of redux store
?

Thxs

Thank you for the issue, yes it should be exposed. I do not have time to work on the project for the moment @oyouf ?

Any update on this? 🙏

In v4 version you can switch to oidc context which in fact does not use any react context and is compatible with redux.
Login and logout are available.

I close the issue. It is fixed in v4, feel free to reopen it if you need.


flat_hash_map::try_emplace() is not exception safe

The current try_emplace() implementation in raw_hash_map.h reads:
  template <class K = key_type, class... Args>
  std::pair<iterator, bool> try_emplace_impl(K&& k, Args&&... args) {
    auto res = this->find_or_prepare_insert(k);
    if (res.second)
      this->emplace_at(res.first, std::piecewise_construct,
                       std::forward_as_tuple(std::forward<K>(k)),
                       std::forward_as_tuple(std::forward<Args>(args)...));
    return {this->iterator_at(res.first), res.second};
}

In case the key of the value being emplaced is not already in the table, find_or_prepare_insert() will prepare the table metadata for insertion before constructing in-place the value from the supplied arugments via emplace_at().
The problem is that if the construction of the value in emplace_at() throws, then the table will be left in a state which is inconsistent:: although according to the metadata a value is present, in reality no value was actually constructed because of the exception that was raised. This will lead to UB and erratic runtime behaviour (e.g., double free on table destruction).
I took a look at some abseil internals and it seems to me that there might be an easy fix in terms of the erase_meta_only() member function of raw_hash_set.h:
  template <class K = key_type, class... Args>
  std::pair<iterator, bool> try_emplace_impl(K &&k, Args &&... args) {
    auto res = this->find_or_prepare_insert(k);
    if (res.second)
      try {
        this->emplace_at(res.first, std::piecewise_construct,
                         std::forward_as_tuple(std::forward<K>(k)),
                         std::forward_as_tuple(std::forward<Args>(args)...));
      } catch (...) {
        this->erase_meta_only(this->iterator_at(res.first));
        throw;
      }
    return {this->iterator_at(res.first), res.second};
  }

The only problem is that erase_meta_only() needs to become a protected function, rather than private, in order for it to be usable in that context.
With this modification, my test case does not crash any more in face of exceptions thrown in try_emplace(). I suspect a similar problem (and perhaps a similar solution) exists in the insert() function.
Do you think this fix is correct? If so, would you like me to open a PR?
I would really like to get this behaviour fixed in my code, so, if you say the fix is ok, I could in the meantime patch my own copy of abseil while waiting for an official resolution.
Thanks!

While this will work for flat_hash_map, it will leak memory for node_hash_map.

The current try_emplace() implementation in raw_hash_map.h reads:
  template <class K = key_type, class... Args>
  std::pair<iterator, bool> try_emplace_impl(K&& k, Args&&... args) {
    auto res = this->find_or_prepare_insert(k);
    if (res.second)
      this->emplace_at(res.first, std::piecewise_construct,
                       std::forward_as_tuple(std::forward<K>(k)),
                       std::forward_as_tuple(std::forward<Args>(args)...));
    return {this->iterator_at(res.first), res.second};
}

In case the key of the value being emplaced is not already in the table, find_or_prepare_insert() will prepare the table metadata for insertion before constructing in-place the value from the supplied arugments via emplace_at().
The problem is that if the construction of the value in emplace_at() throws, then the table will be left in a state which is inconsistent:: although according to the metadata a value is present, in reality no value was actually constructed because of the exception that was raised. This will lead to UB and erratic runtime behaviour (e.g., double free on table destruction).
I took a look at some abseil internals and it seems to me that there might be an easy fix in terms of the erase_meta_only() member function of raw_hash_set.h:
  template <class K = key_type, class... Args>
  std::pair<iterator, bool> try_emplace_impl(K &&k, Args &&... args) {
    auto res = this->find_or_prepare_insert(k);
    if (res.second)
      try {
        this->emplace_at(res.first, std::piecewise_construct,
                         std::forward_as_tuple(std::forward<K>(k)),
                         std::forward_as_tuple(std::forward<Args>(args)...));
      } catch (...) {
        this->erase_meta_only(this->iterator_at(res.first));
        throw;
      }
    return {this->iterator_at(res.first), res.second};
  }

The only problem is that erase_meta_only() needs to become a protected function, rather than private, in order for it to be usable in that context.
With this modification, my test case does not crash any more in face of exceptions thrown in try_emplace(). I suspect a similar problem (and perhaps a similar solution) exists in the insert() function.
Do you think this fix is correct? If so, would you like me to open a PR?
I would really like to get this behaviour fixed in my code, so, if you say the fix is ok, I could in the meantime patch my own copy of abseil while waiting for an official resolution.
Thanks!

Close

While this will work for flat_hash_map, it will leak memory for node_hash_map.

Close

While this will work for flat_hash_map, it will leak memory for node_hash_map.

Close

The current try_emplace() implementation in raw_hash_map.h reads:
  template <class K = key_type, class... Args>
  std::pair<iterator, bool> try_emplace_impl(K&& k, Args&&... args) {
    auto res = this->find_or_prepare_insert(k);
    if (res.second)
      this->emplace_at(res.first, std::piecewise_construct,
                       std::forward_as_tuple(std::forward<K>(k)),
                       std::forward_as_tuple(std::forward<Args>(args)...));
    return {this->iterator_at(res.first), res.second};
}

In case the key of the value being emplaced is not already in the table, find_or_prepare_insert() will prepare the table metadata for insertion before constructing in-place the value from the supplied arugments via emplace_at().
The problem is that if the construction of the value in emplace_at() throws, then the table will be left in a state which is inconsistent:: although according to the metadata a value is present, in reality no value was actually constructed because of the exception that was raised. This will lead to UB and erratic runtime behaviour (e.g., double free on table destruction).
I took a look at some abseil internals and it seems to me that there might be an easy fix in terms of the erase_meta_only() member function of raw_hash_set.h:
  template <class K = key_type, class... Args>
  std::pair<iterator, bool> try_emplace_impl(K &&k, Args &&... args) {
    auto res = this->find_or_prepare_insert(k);
    if (res.second)
      try {
        this->emplace_at(res.first, std::piecewise_construct,
                         std::forward_as_tuple(std::forward<K>(k)),
                         std::forward_as_tuple(std::forward<Args>(args)...));
      } catch (...) {
        this->erase_meta_only(this->iterator_at(res.first));
        throw;
      }
    return {this->iterator_at(res.first), res.second};
  }

The only problem is that erase_meta_only() needs to become a protected function, rather than private, in order for it to be usable in that context.
With this modification, my test case does not crash any more in face of exceptions thrown in try_emplace(). I suspect a similar problem (and perhaps a similar solution) exists in the insert() function.
Do you think this fix is correct? If so, would you like me to open a PR?
I would really like to get this behaviour fixed in my code, so, if you say the fix is ok, I could in the meantime patch my own copy of abseil while waiting for an official resolution.
Thanks!

Close all

The current try_emplace() implementation in raw_hash_map.h reads:
  template <class K = key_type, class... Args>
  std::pair<iterator, bool> try_emplace_impl(K&& k, Args&&... args) {
    auto res = this->find_or_prepare_insert(k);
    if (res.second)
      this->emplace_at(res.first, std::piecewise_construct,
                       std::forward_as_tuple(std::forward<K>(k)),
                       std::forward_as_tuple(std::forward<Args>(args)...));
    return {this->iterator_at(res.first), res.second};
}

In case the key of the value being emplaced is not already in the table, find_or_prepare_insert() will prepare the table metadata for insertion before constructing in-place the value from the supplied arugments via emplace_at().
The problem is that if the construction of the value in emplace_at() throws, then the table will be left in a state which is inconsistent:: although according to the metadata a value is present, in reality no value was actually constructed because of the exception that was raised. This will lead to UB and erratic runtime behaviour (e.g., double free on table destruction).
I took a look at some abseil internals and it seems to me that there might be an easy fix in terms of the erase_meta_only() member function of raw_hash_set.h:
  template <class K = key_type, class... Args>
  std::pair<iterator, bool> try_emplace_impl(K &&k, Args &&... args) {
    auto res = this->find_or_prepare_insert(k);
    if (res.second)
      try {
        this->emplace_at(res.first, std::piecewise_construct,
                         std::forward_as_tuple(std::forward<K>(k)),
                         std::forward_as_tuple(std::forward<Args>(args)...));
      } catch (...) {
        this->erase_meta_only(this->iterator_at(res.first));
        throw;
      }
    return {this->iterator_at(res.first), res.second};
  }

The only problem is that erase_meta_only() needs to become a protected function, rather than private, in order for it to be usable in that context.
With this modification, my test case does not crash any more in face of exceptions thrown in try_emplace(). I suspect a similar problem (and perhaps a similar solution) exists in the insert() function.
Do you think this fix is correct? If so, would you like me to open a PR?
I would really like to get this behaviour fixed in my code, so, if you say the fix is ok, I could in the meantime patch my own copy of abseil while waiting for an official resolution.
Thanks!

Close

Hello!
Any updates on this issue?

@higher-performance I also stumbled upon this error in our codebase. If you could point me to the right direction I would also contribute a fix.
The problem seems to be a bit in the architecture:

There's one function for preparing the slot (prepare_insert) which sets the slot to a state where it assumes that there is a valid element.

The (possible) allocation for a slot + the construction in the element are done in one go (via PolicityTraits::construct. There's the issue, that those two can fail independently (allocation(for the node based containers) or construction.

In case the construction fails, we have to undo two steps: the (possible) allocation (to avoid the memory leaks mentioned above) and the setting of the metadata.

The setting of the metadata can be undone from the raw_hash_set interface, one must just remember to call it on all cases.

The memory leak for the node hash maps that would occur by this fix are a separate bug imho, namely the involved policies not being leak-free. So the fix would be to ensure this to also make sure those policies immediately deallocate in the case of failure before rethrowing.

Please let me know what you think about this issue (it seems doable, as the implementation of all the involved containers boils down to raw_hash_set.h), but I would first like to discuss the structure of a possible fix, as it somewhat would change the architecture of this central class.

The performance of this class is critical Google. Google also somewhat famously does not use exceptions in its C++ code. The SwissTable maintainers think it is not in Google's interest to support exception safety in this class because it will make future optimizations (some of which are being worked on right now) more difficult to implement. Exception-safe implementations will likely either have a runtime cost in the non-exception-safe case, or two code paths would have to be maintained. Exception-safe implementations will likely be under-tested and prone to bugs.
I think for now we may want to document the fact this this class is not exception safe.

Expanded on here: https://github.com/abseil/abseil-cpp/issues/1690#issuecomment-2181371793
https://github.com/abseil/abseil-cpp/commit/e181410198b58b8dffe6e62e24ad56e5d78e2790 documents that these containers are not exception safe.


feat(color): color consistency enhancements

SUMMARY

In the current world, we have this problem:
• SharedLabelColors are calculated when the dashboard loads, but only for the visible tab(s)
• SharedLabelColors are only "reserved" if the label is used twice or more in the VISIBLE charts at load time.
• This means if a label is used once at load time, and then used by another chart when a hidden tab is loaded, the color will be inconsistent.
Current solution by @rusackas and @geido :
• When loading the initial dashboard, make ALL labels use reserved colors... not just those that are used more than once. In other words, assume that ALL colors will be shared/reused.
• Then, when you change tabs, any labels that are used the second time (or more) will already be in the SharedLabelColors
• When loading/exposing a hidden tab, you might see a new series/label for the first time. These should also be added to SharedLabelColors, in case a third or subsequent tab uses those.
Module Diagrams:

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

@stephenLYZ thanks for this!
I tested it and I found that when setting up custom label colors, they only work for the first tab now, but not the other one. Have a look at the screenshots. I forced Michael to be yellow in the metadata

@stephenLYZ thanks for this!
I tested it and I found that when setting up custom label colors, they only work for the first tab now, but not the other one. Have a look at the screenshots. I forced Michael to be yellow in the metadata

Good catch! This is a case that I have missed, I only dealt with the case of changing the color scheme.

/testenv up

@stephenLYZ as I work on the E2E I report here any issue that I encounter.
I just found an issue with already rendered charts under a hidden tab. If you first visit the tab and let the chart load, then move away to another tab, add a custom label color and go back to the tab with the chart, you will see that the custom label color isn't applied.
https://user-images.githubusercontent.com/60598000/192529879-db758eff-e290-4973-a39e-e4edc9365da2.mp4
Same thing happens with changing the color scheme
https://user-images.githubusercontent.com/60598000/192530373-a55553ce-6289-4887-8947-26de4be503ca.mp4

@stephenLYZ the above happens also for main tabs. If you go to tab B, then go to tab A, change scheme, go back to tab B and you will see the color scheme isn't applied.
I also found an even weirder bug where if you change the color scheme from tab B, it picks different colors than if you did change it from tab A.

@stephenLYZ not sure if this is all about the same issue, but when changing color scheme, it works for a nested tab in the initial dashboard tab, but it is not applied correctly to the second tab. As you can see the label colors for "Anthony" are different
https://user-images.githubusercontent.com/60598000/192785479-c6bd1910-22cc-485e-b536-8ce06b5caa84.mp4

@geido It looks like I can't reproduce in my local. I actually fixed this issue in the 'Chart.jsx' file:

https://user-images.githubusercontent.com/11830681/193212693-da333241-40a3-412a-85b9-db9f686532e1.mov

@geido It looks like I can't reproduce in my local. I actually fixed this issue in the 'Chart.jsx' file:

2022-09-<IP_ADDRESS>.mov

@stephenLYZ you are looking at nested tabs as I explained in my latest comment. You should be looking at the main tabs as well. Please follow the steps of my video. As I recommended, you can use my E2E tests to verify whether your changes are working as they are expected to https://github.com/apache/superset/pull/21622

@geido It looks like I can't reproduce in my local. I actually fixed this issue in the 'Chart.jsx' file:

2022-09-<IP_ADDRESS>.mov

@stephenLYZ I can reproduce this pretty consistently with the "Tabbed Dashboard" that is available as a sample dashboard in the E2E testing env. I suggest that you try your changes using my E2E tests PR https://github.com/apache/superset/pull/21622 where the above issues should be apparent immediately.

Adding @jinghua-qa for an upmost confirmation from QA. Let's hold for her feedback before merging

/testenv up

I found an issue that if a series in different charts in the same dashboard, then it is not consistent for the color on that series
here how i reproduce this issue:
step 1: Run this sql query and create dataset:
SELECT * FROM
(SELECT 'a' as color, 1 as val
UNION
SELECT 'b', 1
UNION
SELECT 'c', 1
UNION
SELECT 'd', 1
UNION
SELECT 'e', 1
UNION
SELECT 'f', 1
UNION
SELECT 'g', 1
UNION
SELECT 'h', 1
UNION
SELECT 'i', 1
UNION
SELECT 'j', 1
UNION
SELECT 'k', 1
UNION
SELECT 'l', 1
UNION
SELECT 'm', 1
UNION
SELECT 'n', 1
UNION
SELECT 'o', 1
UNION
SELECT 'p', 1
UNION
SELECT 'q', 1
UNION
SELECT 'r', 1
UNION
SELECT 's', 1
UNION
SELECT 't', 1
UNION
SELECT 'u', 1
UNION
SELECT 'v', 1
UNION
SELECT 'w', 1
UNION
SELECT 'x', 1
UNION
SELECT 'y', 1
UNION
SELECT 'z', 1
) as _view
ORDER BY COLOR
step2: create 3 charts using the same dataset:
chart 1: metrics: count(), dimension: color
chart 2:metrics: count(), dimension: color, filter: color in a, b, c
chart 3:metrics: count(*), dimension: color, filter: color in d, e, f
add 3 charts to same dashboard
step 3: create native filter for color name, then apply filter with color name a, e
Expect:
chart 1: a, e show different color
chart 2, a show same color with chart 1, series a
chart 3, e show same color with chart 1 series ec
Actual:
chart 2 series a show same color as series e

@jinghua-qa Thanks for reporting this issue. This is a bug once we enter the dashboard from explore. I have fixed it and maybe we can add some e2e tests for the native filter scene in the future. cc @geido

/testenv up

The problem may have existed before. I will fixed this in this PR.

/testenv up


Cannot uninstall or update

Microsoft PowerToys version
0.69.1
Installation method
GitHub
Running as admin
Yes
Area(s) with issue?
Installer
Steps to reproduce
I have receiving several running errors from PowerApps when closing or restarting my system.
The program doesn't update itself to the new version, and an automatic crash occurs when trying to run the installer from the "General" tab.
Attempting to uninstall the application results in the "Setup Failed" error (I'm including the log file).
[28482D88][2023-08-14T182537]i001 B.txt
✔ Expected Behavior
Uninstall the app or update it,.
❌ Actual Behavior
Setup failure error.
Other Software
No response

Could you attach /bugreport?
Are you using more than one user accounts?
Also, could you try downloading manually this installer and running it?

The problem no longer remains as, yesterday, I did a complete formatting for my PC. Sorry to not be able to get more info.


Title: Add good vector image in about us

Is your feature request related to a problem? Please describe.
i want to add a good vector image according to ui of webiste
Describe the solution you'd like.
nope
Describe alternatives you've considered.
nope
Add any other context or screenshots about the feature request here.
No response

Sure @Abhinavcode13

Kindly assign this project


Enforce "Days to/from" is integer

Days to Birth
Days to Death
Days to Follow Up
Days to Adverse Event
Days to Comorbidity
Days to Progression
Days to Progression Free
Days to Recurrence
Days to Imaging
Days to Treatment End
Days to Treatment Start
Days to Last Follow up
Days to Last Known Disease Status
Days to Diagnosis
Library Preparation Days from Index
Single Cell Dissociation Days from Index
Sequencing Library Construction Days from Index
Nucleic Acid Capture Days from Index
Collection Days from Index
Sectioning Days from Index
Processing Days from Index
Start Days from Index
Stop Days from Index

Comments added to Google Sheet HTAN schema v22.02 for fields that need controlling and assigned to @elv-sb for modification

Also tracked in [HTAN-57](https://sagebionetworks.jira.com/jira/servicedesk/projects/HTAN/queues/custom/1/HTAN-57


Potential gam4 plot and external csv file architecture

[x] Although not pushing out "gam4" plots near term, we should consider whether/when to build the architecture to handle flow adjusted results in this revision even though we do not populate it with plots at this time.

[x] Consider moving csv files that we have pre-loaded in the package to be external as we have done with the pop-up plots.  (If we do this, we'll need to consider how to make the radio buttons on tab 1a dynamic.)

Part 2 as Issue #54

Rename plots folder "nlt_fp" to "plots_NLT_FA_F_FP" to match data file naming.

Update Shiny app to use new plots folder.
Issue #54

this seems done


Only Plays One Song, and I Can't Change The Song or Add New Songs to Playlists

Expected Behavior
Clicking on a song selects it. You can add songs to playlists.
Actual Behavior
It will play one song only, and selecting other songs does nothing. Adding new songs to playlists also does nothing. The Google Drive Music Player is virtually useless in this state.
Steps to Reproduce the Problem

Open any playlist in Google Drive Music Player.
Select a song.
Add a song to the playlist.

Specifications

Browser: Google Chrome
OS: Android

Seems like others are having similar issues. Too bad the creators of this app don't respond to our posts.

Having same issue from couple of days!

Same problem using Chromebook.

I right clicked the entire music folder and chose the add to music player option. It took about 6 seconds to load, and its still a little buggy but I got my music to play from a google drive folder. Acer Chromebook 15

Only thing I can say is maybe your dick is to short to reach the correct
button .