Imported SVG semi-transparent areas become fully opaque

Godot version
4.2beta1
System information
macOS 13.6.0 - Vulkan (Mobile) - integrated Apple M2 Pro - Apple M2 Pro (10 Threads)
Issue description
Importing SVGs (via editor or in code using the Image.load_svg_from_string() etc) that have semi-transparent areas result in these areas being fully opaque instead of having the expected semi-transparent alpha values.
Left imported svg in Godot, right same svg in Firefox

This seems this is an issue with ThorVG as the same issue is present in their viewer. So I've opened an issue there as well ( https://github.com/thorvg/thorvg/issues/1716 ). But since this affects Godot I'm opening this here so it's known and we can keep track of the issue.
If you don't want these kinds of double issues feel free to just close this.
EDIT: Here's a better example image showing off the issue, made by @capnm, where you can see that the alpha is also incorrect in the overlapping area.

Steps to reproduce
Import the following svg image into a godot project

Minimal reproduction project
see image file above

Looks like the issue has been fixed on the ThorVG side now and a new 0.11.2 release is available that we can update to in Godot to hopefully fix this issue.

Godot + v0.11.2

PNG

SVG


RNaD Entropy Schedule

s = EntropySchedule(sizes=(10,))
print([s(i) for i in range(20)])
[
(0.0, False), (0.2, False), (0.4, False), (0.6, False), (0.8, False),
(1.0, False), (1.0, False), (1.0, False), (1.0, False), (1.0, False),
(0.0, True), (0.2, False), (0.4, False), (0.6, False), (0.8, False),
(1.0, False), (1.0, False), (1.0, False), (1.0, False), (1.0, False)
]

Shouldn't the alpha value be 1.0 when the regularization nets are updated?
Since it is 0 this effectively disrupts the linear interpolation between regularization policies

@perolat, @bartdevylder: any ideas?

Hi @spktrm ,
Thanks for the question.
It should becomes 0 after we update the two regularisation policies. Unless we missed an edge case there should only be continuous interpolations between regularisation networks.
It should go this way:

alpha goes from 0 to 1 linearly over half of the interval (interpolate from pi_{reg, 0} to pi_{reg, 1}),
then alpha stays at one for the rest of the interval (here the regularisation policy is pi_{reg, 1}),
we update the networks (to interpolate between pi_{reg, 1} and pi_{reg, 2})
start a new interval from alpha=0 (so at the first step of this interval we start with the regularisation policy pi_{reg, 1})

Let me know if you see something that doesn't match this intended behaviour.
Julien

Update: We have identified this as a bug and Eugene Tarassov has a fix. Will be fixed on next sync to github.


Permissions need to be changed on CentOS 6.8

Per slack

Max Chadwick [4:43 PM]
service kapacitord start
kapacitord: unrecognized service

[4:43]
How does one start the kapacitord service?

Nathaniel Cook [4:43 PM]
`service kapacitor start`

Max Chadwick [4:43 PM]
that doesn't seem to start kapacitord for me

[4:44]
i'm on CentOS

[4:44]
downloaded the Kapacitor like this

[4:44]
wget https://dl.influxdata.com/kapacitor/releases/kapacitor-0.13.1.x86_64.rpm
sudo yum localinstall kapacitor-0.13.1.x86_64.rpm

[4:45]
or maybe it's just not working at all?

[4:45]
[root@sd-ops-diagnostics kapacitor]# service kapacitor start
Starting the process kapacitor [ OK ]
kapacitor process was started [ OK ]
[root@sd-ops-diagnostics kapacitor]# ps aux | grep 'kap'
root      7913  0.0  0.0 103312   876 pts/2    R+   15:44   0:00 grep kap
[root@sd-ops-diagnostics kapacitor]#

Jack Zampolin [5:05 PM]
What centos version? did you check `service kapacitor status`?

Max Chadwick [5:07 PM]
CentOS 6.8

[5:07]
Potentially permissions issue?

[5:07]
==> /var/log/kapacitor/kapacitord.err <==
2016/06/29 16:07:30 Using configuration at: /etc/kapacitor/kapacitor.conf
[run] 2016/06/29 16:07:30 E! init logging: open /var/log/kapacitor/kapacitor.log: permission denied
run: init logging: open /var/log/kapacitor/kapacitor.log: permission denied

[5:08]
i'm starting the service as root

[5:08]
root@sd-ops-diagnostics kapacitor]# sudo service kapacitor start

[5:09]
OK yea

[5:09]
==> /var/log/kapacitor/kapacitord.err <==
run: open server: open service *storage.Service: open boltdb @ "/var/lib/kapacitor/kapacitor.db": open /var/lib/kapacitor/kapacitor.db: permission denied

[5:10]
yeah, it was permissions

[5:10]
I did _not_ have to change any permissions for InfluxDB FWIW

new messages
Jack Zampolin [5:18 PM]
Did you try running with `sudo`

[5:18]
yes

[5:19]
sorry for not reading that. You shouldn’t have had to change any permissions for kapacitor either

[5:19]
@mpchadwick please open an issue on the `influxdata/kapacitor` repo with the tag `packaging` (edited)

@mpchadwick What where the permissions after installing? I just performed an install on a new CentOS image.
[root@c74ad51706a9 /]# curl https://dl.influxdata.com/kapacitor/releases/kapacitor-0.13.1.x86_64.rpm > kapacitor-0.13.1.x86_64.rpm
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 10.8M  100 10.8M    0     0  8417k      0  0:00:01  0:00:01 --:--:-- 8415k
[root@c74ad51706a9 /]# yum localinstall kapacitor-0.13.1.x86_64.rpm
Loaded plugins: fastestmirror, ovl
Examining kapacitor-0.13.1.x86_64.rpm: kapacitor-0.13.1-1.x86_64
Marking kapacitor-0.13.1.x86_64.rpm to be installed
Resolving Dependencies
--> Running transaction check
---> Package kapacitor.x86_64 0:0.13.1-1 will be installed
--> Finished Dependency Resolution

Dependencies Resolved

=====================================================================================================================================
 Package                     Arch                     Version                       Repository                                  Size
=====================================================================================================================================
Installing:
 kapacitor                   x86_64                   0.13.1-1                      /kapacitor-0.13.1.x86_64                    41 M

Transaction Summary
=====================================================================================================================================
Install  1 Package

Total size: 41 M
Installed size: 41 M
Is this ok [y/d/N]: y
Downloading packages:
Running transaction check
Running transaction test
Transaction test succeeded
Running transaction
  Installing : kapacitor-0.13.1-1.x86_64                                                                                1/1
  Verifying  : kapacitor-0.13.1-1.x86_64                                                                                1/1

Installed:
  kapacitor.x86_64 0:0.13.1-1

Complete!
[root@c74ad51706a9 /]# ll /var/lib/
total 0
drwxr-xr-x 1 root      root       44 Mar 31 17:01 alternatives
drwxr-xr-x 1 root      root        0 Nov 20  2015 dbus
drwxr-xr-x 1 root      root        0 Aug 12  2015 games
drwxr-xr-x 1 root      root        0 Dec  3  2015 initramfs
drwxr-xr-x 1 kapacitor kapacitor   0 May 13 22:44 kapacitor
drwx------ 1 root      root        0 Mar 31 17:02 machines
drwxr-xr-x 1 root      root        0 Aug 12  2015 misc
drwxr-xr-x 1 root      root      322 Mar 31 17:03 rpm
drwxr-xr-x 1 root      root        0 Aug 12  2015 rpm-state
drwxr-xr-x 1 root      root       52 Mar 31 17:02 systemd
drwxr-xr-x 1 root      root       68 Jun 30 15:09 yum
[root@c74ad51706a9 /]# ll /var/lib/kapacitor/
total 0
[root@c74ad51706a9 /]# ll /var/log/
total 4
drwxr-xr-x 1 root      root      186 Mar 31 17:03 anaconda
drwxr-xr-x 1 kapacitor kapacitor   0 May 13 22:44 kapacitor
-rw------- 1 root      root       53 Jun 30 15:09 yum.log
[root@c74ad51706a9 /]# ll /var/log/kapacitor/
total 0
[root@c74ad51706a9 /]#

All permissions look correct. Can you provide more details as to which permissions you fixed to get it working?

I chmod-ed /var/log/kapacitor and /var/lib/kapacitor to 777. I'm not sure what they were before making the perissions changes. I'm going to close this issue if it's not reproducible. It might have just been me. If someone else reports it it would be worth investigating, I guess

I ran into this bug trying to upgrade from 0.12.1 to 1.2

Error : It does not have permission to open previously installed version's(0.12.1) kapacitor.db
Removed that file and gave 666 on entire kapacitor folder, trying to reinstall
ran into create server: open /var/lib/kapacitor/cluster.id: permission denied
gave 666 on that file too but still now working

I got the similar issue while upgrading the kapacitor image
"open service *storage.Service: open boltdb @ "/var/lib/kapacitor/kapacitor.db": open /var/lib/kapacitor/kapacitor.db: permission denied"
-- I am using aws ebs volume for persistent storage.
-- Access mode setup in dynamic allocation PVC - ReadWriteOnce
When upgrading the kapacitor-- pod is getting into crashloopbackoff with above error

One way to deal with above issue--
-- Get the PV and PVC with ReadWriteMany
-- aws doesn't supports RWX with ebs volume, switch to EFS volume which supports RWX access mode


bug: bang dquote is special in zsh

Describe the bug
Our documentation tells people to copy-and-paste some commands that include !".  But that is special to zsh.
mspreitz@mjs13 kubestellar % echo $0
-zsh

mspreitz@mjs13 kubestellar % echo "Hey! "
Hey!

mspreitz@mjs13 kubestellar % echo "Hey!"
dquote>

There are four instances of this problem in docs/content/Coding Milestones/PoC2023q1/example1-subs/example1-stage-4.md
There are also !" appearing in some scripts, but those are all scripts that go specifically to bash.
Steps To Reproduce
See above
Expected Behavior
A user of a common shell can copy-and-paste what our docs offer.
Additional Context
No response

squotes are different.
mspreitz@mjs13 kubestellar % echo 'Hey!'
Hey!

fixed in main and 0.9.0 version of doc


God Fist Lee Sin Q and Auto Attack Animation

Can this be fixed? Or is it problem #1 ? thank you for your understanding ^^

#1


Parser output empty file

Hi all,
Thank you a bunch for the parser.
I have a problem in the parser application.
I have created the three folders as you have mentioned and added some CM xml files in the input data folder. However, after running the code in the cmd, the output is a CSV file named "fileFooter" which doesn't have any data in it.
The command that I am running is:
java -jar C:\HUNTER\Boda\Huawei_4G_XML\Scripts\boda-huaweicfgsynparser.jar C:\HUNTER\Boda\Huawei_4G_XML\Data\MyFolderWithMyRawData C:\HUNTER\Boda\Huawei_4G_XML\Output\MyOutputFolder

@Salari7, what format is your dump?


Added .usf and .ush

Add support for Unreal Engine shader file

I would be great, but Unreal Shader would require a little bit more work to make it functional :)
At least a .tmlanguage would be needed to have the proper syntax definition.
In the meanwhile, you can associate .usf and .ush files in your VSCode user settings
Files: Associations
Configure file associations to languages (e.g. "*.usf": "hlsl"). These have precedence over the default associations of the languages installed.
Edit in settings.json


Works with eclipseLink

Is there any possibility of working for eclipseLink

Hello @frafaelptu
This library is fully based on Hibernate so it is not compatible with EclipseLink.
However, I think you could try to apply the same principle to EclipseLink by looking for an interceptor.
I close the issue because it is not in the library scope,
Good luck!


Minor updates

App-specific configurations - raised the prominence of the paragraph about only one being active at any time, addition of info about the alternation operator.
Package-specification - make homepage: key obligatory, to facilitate successful Hub merges

Every little counts 🤘


Hint at what orientation means in the error message

Small PR that changes the error text when you pass in e.g. boundary nodes with the wrong orientation.  Feel free to close/edit if this doesn't make sense.
Should DelaunayTriangulation just reverse the order of points and go on with life?

The advice depends on whether the curve is contiguous or has multiple sections.
julia> v = [[1, 2, 3], [3, 4, 5], [5, 6, 1]];

julia> rev = [[1, 6, 5], [5, 4, 3], [3, 2, 1]];

julia> reverse(v) # the sections don't line up
3-element Vector{Vector{Int64}}:
 [5, 6, 1]
 [3, 4, 5]
 [1, 2, 3]

julia> reverse(reverse.(v)) == rev # need to reverse sections and the entire curve
true

julia> v_contiguous = [1, 2, 3, 4, 5, 1]; rev_contiguous = [1, 5, 4, 3, 2, 1];

julia> reverse(v_contiguous) == rev_contiguous # good
true

julia> reverse(reverse.(v_contiguous))
ERROR: MethodError: no method matching reverse(::Int64)
The function `reverse` exists, but no method is defined for this combination of argument types.

Closest candidates are:
  reverse(::Base.AnnotatedString)
   @ Base strings\annotated.jl:318
  reverse(::SubString{<:Base.AnnotatedString})
   @ Base strings\annotated.jl:328
  reverse(::Base.Order.Ordering)
   @ Base ordering.jl:56
  ...

Stacktrace:
 [1] _broadcast_getindex_evalf
   @ .\broadcast.jl:673 [inlined]
 [2] _broadcast_getindex
   @ .\broadcast.jl:646 [inlined]
 [3] getindex
   @ .\broadcast.jl:605 [inlined]
 [4] copy
   @ .\broadcast.jl:906 [inlined]
 [5] materialize(bc::Base.Broadcast.Broadcasted{Base.Broadcast.DefaultArrayStyle{…}, Nothing, typeof(reverse), Tuple{…}})
   @ Base.Broadcast .\broadcast.jl:867
 [6] top-level scope
   @ REPL[31]:1
Some type information was truncated. Use `show(err)` to see complete types.

So the advice needs to be

If the curve is contiguous, reverse(curve).
If the curve is made of multiple sections, reverse(reverse.(curve)).

We could pass whether the curve has multiple sections or not into the struct defining the error (see has_multiple_sections), or just breaking the advice into those two parts is probably fine..

Should DelaunayTriangulation just reverse the order of points and go on with life?

This is an option as well but I usually prefer not to modify the user's input at all and error instead. We could continue, but then there's a mismatch between their provided boundary nodes and those stored inside the Triangulation. I would also need to then add support for reversing the boundary nodes for general boundaries since they don't have to be vectors. It's easier to just error I think

I didn't think of the impact on node order, it does make sense to simply error out then.  Will implement a has_multiple_sections parameter in the error then!

Will implement a has_multiple_sections parameter in the error then!

This also turned out to be annoying since the error isn't actually look at the boundary nodes it's looking at the PolygonTree. We could change it to also take the boundary nodes but that's a bit of work

Ahh actually this advice also doesn't really work for curve-bounded domains where reverse wouldn't really be defined... I will make some edits.

See what you think now @asinghvi17. Also added some tests.

Looks good to me - thanks!

Thanks!


ECR repository is not created on change of docker registry

Checklist:

[x] I've searched in the docs and FAQ for my answer
[x] I've included steps to reproduce the bug.

Description
When we build and push any image from Devtron to ECR, it creates the repository at ECR side if it doesn't already exist but only if we select ECR as the docker registry during first save of docker build config but if we use any other docker registry at first save and then update it with ECR for further builds, it doesn't create the repository if it doesn't already exist.
To Reproduce

Create an app and use any registry other than ECR
Save the build config and trigger pipeline
Now change docker registry to ECR in build config and update
Trigger pipeline again

Expected behavior
If the repo isn't created at ECR side, it should create the repo with the given name and push successfully  as it happens when we select ECR during app creation
Current behavior
The build fails because push didn't succeed to ECR as repository doesn't exist at ECR side

While reproducing the bug, found another extension of the same bug that even if ECR is selected in the initial docke build config, it will create a repository for the first time. But if the repo is changed to an uncreated repo, the same bug occurs, i.e. it does not create a repository at ECR.


No Module found

Traceback (most recent call last):
File "C:\Program Files (x86)\Steam\steamapps\common\Blender\3.0\scripts\modules\addon_utils.py", line 351, in enable
mod = _import_(module_name)
File "C:\Users\User\AppData\Roaming\Blender Foundation\Blender\3.0\scripts\addons\io_import_pskx-develop\_init__.py", line 39, in <module>
from io_import_pskx import op
ModuleNotFoundError:
No module named 'io_import_pskx'

I followed the instructions as specified but I was greeted with this error, not sure what I did wrong.

Yeap, same here.

Reinstall the addon, but remove -develop from the zip filename. Alternatively go to the addons folder and remove -develop from the folder name and activate it again in blender. That should fix it.
I don't know why people are using this addon though, nothing exports to these modified formats.

i removed -develop from the folder name and still got the same error,
Traceback (most recent call last):
File "C:\Program Files\Blender Foundation\Blender 3.1\3.1\scripts\modules\addon_utils.py", line 351, in enable
mod = import(module_name)
File "C:\Users\ksorc\AppData\Roaming\Blender Foundation\Blender\3.1\scripts\addons\io_import_pskx-develop_init_.py", line 39, in
from io_import_pskx import op
ModuleNotFoundError: No module named 'io_import_pskx'

are you sure?

 "C:\Users\ksorc\AppData\Roaming\Blender Foundation\Blender\3.1\scripts\addons\io_import_pskx**-develop**_init_.py"

I had the same mistake, thanks to yretenai for solving it


Develop a screen for selecting the workout

[x] Create a file
[x] Pass Muscle Name as parameter
[x] Update Img
[x] Find a layout for design of component
[x] Pass image and Workout clicked as parameter for the next component from workout list
[x] Load data as workout clicked

Code for this issue is completely done


scipy.rosen

I would like to pass in a simple float as well, right now this throws an error and I need to pass in an array-like type.

Do you mean just one float as input? Isn't the Rosenbrock function only defined for multidimensional inputs (see Wikipedia) or is there any other implementation of it for one-dimensional inputs?
Using rosenbrock( [arbitrary float] ) results in 0 at the moment (short test for 100 integers and 100 random normally distributed values). Is zero the desired output for rosenbrock( arbitrary float ) as well?

Yes, you are right, please just add an informative error message when passing in a single variable.

Philipp Eisenhauer
Economist
Mail<EMAIL_ADDRESS>Web https://peisenha.github.io
Repository https://github.com/peisenha
Sent with ProtonMail Secure Email.
‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐
On Monday, November 9, 2020 8:37 PM, manuhuth<EMAIL_ADDRESS>wrote:

Do you mean just one float as input? Isn't the Rosenbrock function only defined for multidimensional inputs (see Wikipedia) or is there any other implementation of it for one-dimensional inputs?
Using rosenbrock( [arbitrary float] ) results in 0 at the moment (short test for 100 integers and 100 random normally distributed values). Is zero the desired output for rosenbrock( arbitrary float ) as well?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Done.


Generate Coral's RelNode for views from base table schema

What changes are proposed in this pull request, and why are they necessary?

This PR includes the following fixes:
(1) Includes changes from https://github.com/linkedin/coral/pull/394 which aimed to remove CAST in HiveViewTable.toRel to solve Nested View Stale Schema Issue.
(2) Enables avro schema generation for a dataset with map field with NULL key
(3) When a field schema is a UnionType of a single data type, such as uniontype<array<string>>, surface the underlying field & its schema in Coral's SqlNode RelNode representation, such as array<string>. SingleUnionFieldReferenceTransformer has been introduced for backward compatibility.
For a table t1: (f1: uniontype ) and a view v1 defined on top of it SELECT custom_udf_to_remove_single_type_union(f1) AS f2 FROM t1
v1 has a schema: (f2: int)
The RelNode representation, prior to dropping the CAST operator in the RelNode, would force the derived data type to be int for f2, as expected. Hence, the generated avro schema could be analyzed by the Spark engine. However, after dropping the CAST, as documented here, the derived type is Struct<tag_0:int>. The Avro schema is incorrectly generated and cannot be analysed by the Spark engine.
The resolution is to derive the data type of single uniontype fields like f1 as the underlying data type (int in this case).
How was this patch tested?

./gradlew spotlessApply
./gradlew build
Updated UTs
Thoroughly tested production views across all supported engines.

@aastha25: Thanks for this PR. Could we create a PR for union type handling and keep #394 ? Or keep the author information of #394 if this PR can't be splitted?

@ljfgem Unfortunately the PR cant be split up. All these changes need to go in together. I have updated the author's information for the commits related to #394.

@ljfgem I have added an example clarifying the usecase in the PR description.

Should not this PR change the behavior of CoalesceStructUtility so that if the input is a field with single type, it returns a struct?

This PR sets the return type of extract_union(string) as string, making the operator a no-op. I can modify the comment on the class to document this change if required.


Add json format support

The javascript version of mjml handles json MJML both as input (if you give a dict to the mjml2html function) and as an output (in a json field).
Before sending you a PR:

is it OK to add this feature?
for input, the API is not exactly the same as in js (python accepts a file descriptor, whereas js gets a string), is it OK to accept a dict for json-like format?

Thank you for sending the PR.

is it OK to add this feature?
for input, the API is not exactly the same as in js (python accepts a file descriptor, whereas js gets a string), is it OK to accept a dict for json-like format?

I was not aware of json MJML but if you need this we can add it of course.
Also I don't think we need bug-for-bug compatibility. The Python implementation should be reasonably similar but we can change the details to make the API easier to use in Python.
I'll look at your bug report later today - it's a very busy time here.

closed by #13 (but I'd appreciate a response regarding hiding children)


Reduce travis errors and move to trusty

This is a replacement to #283 which:

brings travis to trusty (with sudo: false it results in travis throwing away source: libicu-dev, but everything seems to install and run ok, so maybe that source is no longer needed on trusty?)
improves reliability of bundle install on travis by deleting the .lock files for specific gemfiles and overriding/removing the --deployment option to bundler so that bundler will never complain about the gemfile.lock being updated, and will always use latest dependencies (similar to a current clean install on any system)

Given that e27f50a7676480511e91f897ca56cc34a5fbcb9b passed, why is dist: trusty necessary?

Given that e27f50a passed, why is dist: trusty necessary?

@gjtorikian because travis is moving its default from precise to trusty anytime now, I thought it would be better to be specific (rather than just wait for it possibly to fail). True, it has now succeeded so could just wait for the automatic default to change, but I think the more predictability in CI the better

but I think the more predictability in CI the better

My concern is that we will forget to remove or change this value when Travis inevitable moves off of trusty and onto The Next Great Thing, leaving us stuck in the past.
Given that this PR works with the current Travis setup and the future Travis setup I'm going to go ahead and merge it but take that line out.

Thank you!


Add comma(,) in index.md

adding the appropriate comma(,) in order to make the sentence more understandable to the readers.

Thank you. For future patches, could you bundle these changes a bit more? Now the commit are usually really small, while it's more useful for us if these kinds of changes are more combined.


Docs broken link

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[ ] Bug report
[ ] Performance issue
[ ] Feature request
[x] Documentation issue or request
[ ] Support request
[ ] Other... Please describe:

Current behavior
https://netbasal.gitbook.io/transloco/comparison-to-other-libraries
The links for Pluralization and Locale are broken

@WLun001 You are viewing the old documentation site, see our new docs

@shaharkazaz So just to clarify is https://ngneat.github.io/transloco/docs/translation-api the official up to date documentation? As I've seen a lot of links to https://netbasal.gitbook.io/transloco/

@ChazUK ngneat.github.io/transloco is the correct documentation 🙂
Where have you seen links to our old docs?

It's the second link on Google, and the top two articles also link to the old docs, one from April 2020.

@ChazUK thanks for bringing it to my attention, I'll take care of it.

@shaharkazaz the new docs do not mention anything about comparison anymore. So the link you previously shared does not work anymore: https://ngneat.github.io/transloco/docs/comparison

@Boosten We decided to remove this section 🙂


Hand pose weird results

I'm running a real-time application to detect hands and drive a standard smplx model (default flat hand pose) to act, but got some weird stretch for both hands. Like, all fingers move to the wrong side. Any idea?
I set flat_hand_mean=True here,
https://github.com/mks0601/Hand4Whole_RELEASE/blob/21ce3c03a4018d6b97324e0df061f026e4d276c0/common/utils/human_models.py#L80
and tried mano.layer['left'] here,
https://github.com/mks0601/Hand4Whole_RELEASE/blob/21ce3c03a4018d6b97324e0df061f026e4d276c0/main/model.py#L29

You should set flat_hand_mean=False

Actually nothing changes when changing this parameter. The rotation keeps the same

That option adds some constant rotation to the input hand joint rotations in SMPLX layer. Although the estimated rotation is the same, SMPLX applies different rotations by adding a constant rotation.

I mean, after using the default flat_hand_mean=False, the displayed pose keeps almost the same as above pictures.


Fix bullet alignment errors in localization.md

When syncing with this file
content/en/docs/contribute/localization.md

find the list and table are incorrectly aligned.

Corrected preview: https://deploy-preview-37510--kubernetes-io-main-staging.netlify.app/docs/contribute/localization/

Screenshot of the preview:

/lgtm

Thank you for the update, the change is an improvement
/approve


Consider adding a language services embed for HTML as part of this extension?

First of all: thank you for providing and officially supporting such a fine VSCode extension for the Phoenix framework!
After working with this extension and HEEx templates for a while I noticed that HTML Intellisense behaviour, such as automatic indentation between tags, autocomplete of attributes, etc., was not available when working  with HEEx files or inside of ~H sigils.
Out of the box, VSCode only provides the enhanced Intellisense experience for .html files. While we can configure other extensions like emmet or bradlc.vscode-tailwindcss to be enabled in HEEx via user settings, expanding the built-in HTML Intellisense to HEEx appears to be bit more involved.
Microsoft provides a detailed write-up on how this functionality can be accomplished via language extensions at [1] with source code examples at [2], [3]. For instance, the vscode-html-erb extension uses such a language services embed to offer HTML-Intellisense in Ruby (.erb) templates.
Given the similarity of HEEx and ERB syntax, aki77's code with some minor modifications implemented in a quick and dirty prototype showcases that a similar approach could perhaps be viable for HEEx.
Interestingly a language services embed seems to add TMScope to HTML tags in HEEx files, possibly providing an answer to #18.

https://github.com/microsoft/vscode/pull/146731 added a feature for extensions to enable the builtin html-language extension on their languageId
"extensionDependencies": [
	"vscode.html-language-features"
],
"contributes": {
	"htmlLanguageParticipants": [
		{
			"languageId": "HEEx",
                        "autoInsert": true
		}
	]
}

related: https://github.com/microsoft/vscode/pull/198583


Typo in directory name

Typo in directory name: "D:\PL-300-Microsoft-Power-BI-Data-Analyst-Main\Allfiles\Labs\01\Starter\09erform-data-analysis-in-power-bi-desktop".
Should be "09-perform-data-analysis-in-power-bi-desktop".

Thank you. We will update this on Friday.

Update has been made.


Questions about the paper, need your answers.

Thank you for your great work. I currently have a question that I would like you to answer. Does sunlight have an impact on sky image predictions? Will this lead to models skewing PV predictions towards sun position or light intensity rather than cloud movement?

Thanks for your question. I am not quite sure what you mean by "does sunlight have impact on sky image prediction" or you actually mean would sunlight have impact on PV output prediction. Our model can capture the movement of both the sun and the clouds. Yes, the sunlight (i.e., intensity and position) will have impact on the PV output prediction and these behaviors can be well modeled by either clear sky models based on physics or learned by deep learning models based on sky images. The difficulties for current solar forecasting research (especially short-term forecasting) really are the rapidly changing cloud dynamics. You can take a look at our repo here. PV output on sunny days can be predicted almost perfectly by the model, while cloudy days obviously have much larger errors.

Thank you for your answer, your answer solved my confusion.


Linking when using kotlin not possible? NativeModule.RCTAsyncStorage is null.

You want to:
Since I was getting warnings that the react-native asyncStorags is becomming deprecated and is moved (here) to individual package, I decided to install... This seems to be a pain in the a** every time, was also troubleful when I installed RCs' extracted WebView. (grumpy sorry)....
Anyway, this time I am stuck with failing jest tests that I have not been able to resolve after 24 hours of trying. Have had multiple colleges passing by as well as asked for help on SO.
Please check this link for further issue despription: https://stackoverflow.com/questions/55418910/jest-test-fails-after-installing-react-native-async-storage
I did manual linking since react-native link will not work for this module as MainApplication.java does not exist when using kotlin. Instead I have MainApplication.kt
After linking manually the async functions are working just fine. Except the jest test are unable to find the module.
 @RNCommunity/AsyncStorage: NativeModule.RCTAsyncStorage is null.
Any suggestions?

Hi, I answered your post on stackoverflow.
Also see our repo if you like.

@rpuls 	Hey, thanks for reaching out.
I'm on the way of writing mocks to be used with Jest. I'm aiming to finish in few days, alongside with docs explaining how to test it.
For now, to get by this error, you'd have to mock NativeModule from react-native, to return either of those modules.

@Krizzu

I'm on the way of writing mocks to be used with Jest. I'm aiming to finish in few days, alongside with docs explaining how to test it.
For now, to get by this error, you'd have to mock NativeModule from react-native, to return either of those modules.

Hey thanks for replying! I found a solution that worked. Tried many different ways of mocking the AsyncStorage. 4th attempt worked like a charm. I updated my stackoverflow with the solution https://stackoverflow.com/questions/55418910/jest-test-fails-after-installing-react-native-async-storage/55421364?noredirect=1#comment97619261_55421364

Hey @rpuls,
I've created https://github.com/react-native-community/react-native-async-storage/pull/53, so future cases with Jest testing will be easier.
thanks.

v1.3.0 released, please check it out 🙏


Shell expansion syntax not recognized for imports

Minor issue but could be worth fixing.
The following code is marked as a syntax error by IntelliJ (PyCharm 2024.1.4)
import x'envs/${JUST_COMMON:-common}.just'

With the following error:  JustToken.IMPORT_PATH or JustToken.QUESTION_MARK expected, got 'x' but the expansion works as expected as documented at the bottom of the page here:

However, this allows shell expanded strings to be used in places like settings and import paths, which cannot depend on just variables and .env files.

Note that the shell expansion is correctly recognized for settings, the code:
set dotenv-filename := x'envs/${JUST_ENV:-dummy}.env'

does not trigger a syntax error.

Fixed with version 0.5.2


Specific team/channel URL as server URL does not display a notification about an invalid setting; in consequence boards can not be displayed.

I confirm (by marking "x" in the [ ] below: [x]):

[x] This is not a troubleshooting question. Troubleshooting questions go here: https://docs.mattermost.com/install/troubleshooting.html.
[x] This doesn't reproduce on web browsers (such as in Chrome). If it does, issue reports go to the Mattermost Server repository.
[x] I have read contributing guidelines.

Summary
Desktop App should validate the server URL when adding a server configuration and report issues if the URL seems to be a team or channel URL or something similar; if not, boards can not be displayed.
Environment

Operating System: Windows 10
Mattermost Desktop App version: v5.0.4
Mattermost Server version: v6.3.3

Steps to reproduce
If users enter a specific channel URL like "https://mattermost.example.com/myteam/channels/mychannel" instead of just "https://mattermost.example.com/" the configuration is saved without issues. The app behaves as usual most of the time. Unfortunately, if the users switches to the boards, the boards are not displayed, but a permanent "loading ..." message appears.
Expected behavior
In the moment when users add server configurations an invalid server URL should be reported to the user.
Observed behavior
No validation message is displayed if a team or channel URL is entered as server URL, in consequence the boards are not displayed if user tries to switch to them.
Remarks
I post this issue here and not in the Mattermost/Focalboard issue-tracker because I think that this should be handled in the Mattermost Desktop App in the moment when users add the server configurations.
Do you think it is possible to detect and validate these sort of ill configured server URLs? Otherwise it is hard to identify the problem if people just experience the permanent "loading ..." message when switching to the boards.
Please let me know if you want me to create an issue in the Mattermost/Focalboard issue-tracker as well.
Thank you very much in advance for your feedback.

@christianhueserhzdr Thanks for the report.
At this time it is currently not possible to validate whether a subpath that happens to match one that Mattermost server uses is an invalid subpath. Because it is technically possible for a server to be setup at the path https://mattermost.example.com/myteam/channels/mychannel, so we shouldn't invalidate it.
What we are looking into in a future release is to make sure that we can communicate with a Mattermost server when the URL is entered and perform some validation that the URL entered is a valid Mattermost server.
Thanks for the input, but I'm going to close this issue as I don't believe validating based off of URL patterns will be the way forward for the app.


fix(scroll): 修复移动端快速滚动时控制台报错 close #2266

👀 PR includes

🐛 Bugfix

[x] Solve the issue and close #2266

📝 Description

移动端滚动时通过 preventDefault() 阻止父容器滚动, 在快速滚动时, 会有报错
参考解决方案: https://developer.mozilla.org/zh-CN/docs/Web/API/Event/cancelable#示例

🖼 Screenshot

Before
After

❌
✅

🔗 Related issue link

🔍 Self-Check before the merge

[ ] Add or update relevant docs.
[ ] Add or update relevant demos.
[ ] Add or update test case.
[ ] Add or update relevant TypeScript definitions.

:tada: This PR is included in version @antv/s2-v1.50.0-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version @antv/s2-react-v1.42.2-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version @antv/s2-v1.50.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version @antv/s2-react-v1.43.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


C/CPP: RECURSIVE BFS

Description 📜
MADE A DOCUMENT ON RECURSIVE BFS
Fixes #6225

Type of change 📝

[x] Documentation (Content Creation in the form of codes or tutorials)

Domain of Contribution 📊

[x] C/CPP

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

Screenshots / Gif (Optional) 📸

@kashishahuja2002 made required Changes..... Plz check now


endpoint health/status doesn't show all etcd members' values

In ETCD v2, the cluster health will show all the etcd members' health status. And in ETCD v3, the etcdctl can't show all the members' status.
After reading the source code of the endpoint command, it only connect to the endpoints configured in the command argument. It's very inconvenient to catch all members' status.
func epStatusCommandFunc(cmd *cobra.Command, args []string) {
	c := mustClientFromCmd(cmd)

	statusList := []epStatus{}
	var err error
	for _, ep := range c.Endpoints() {
		ctx, cancel := commandCtx(cmd)
		resp, serr := c.Status(ctx, ep)
		cancel()
		if serr != nil {
			err = serr
			fmt.Fprintf(os.Stderr, "Failed to get the status of endpoint %s (%v)\n", ep, serr)
			continue
		}
		statusList = append(statusList, epStatus{Ep: ep, Resp: resp})
	}

	display.EndpointStatus(statusList)

	if err != nil {
		os.Exit(ExitError)
	}
}

this is by design. it is under endpoint, not cluster anymore, which means it only cares about the endpoint you provide.
a cluster level ping is usually broken by design for various reasons. we do not want to go that route again.

@xiang90 Here is what I do to make it:
https://github.com/armstrongli/etcd/commit/0c7034cd907169723ee7cb60833e5128cdcca9f3

If the endpoint which is being checked is not in the soft balancer, the ping will be failure. I haven't check the root cause.

I'll commit the PR first.
https://github.com/coreos/etcd/pull/8116
If it's design level, I hope add one cluster sub-command to support that.

Cluster level command is too easy to be confusing. please open issue for discussion before implement it.

@xiang90 I created one issue here: https://github.com/coreos/etcd/issues/8117


Remove mirror stream on update to NATS 2.10

In #79 we added some things around a mirror stream for combining streams. In NATS 2.10, we will be able to have multiple filter subjects and won't need it any more. So once we update, we should remove the need for a mirror stream

@thomastaylor312 was this okay to close as stale?

Nope. Let me label

Done in https://github.com/wasmCloud/wadm/pull/310


Add waiting screen after QR Code is scanned while wait for presentation

Description of feature / user story
As a verifier I want feedback whether or not the QR code was scanned affectively so that I can tell if there was an error or not.
Acceptance Criteria
Given the holder and verifier are using the mobile verifier
When the QR code is scanned by the holder
Then the QR code screen on the verifier device changes to the proof request sent loading screen
And there's a "go back" button that takes them back to the request for proof screen (qr code screen)
Wireframes or relevant image assets / links
See wireframes and user flow from ticket #1238

Looks good.

@wadeking98 There's a bug when I follow these steps:

Scan QR code from verifier
Verifier taps "go back" on the "Waiting for response" screen
Holder shares information
Verifier goes to their home screen and sees a new proof request notification that errors out when you try to open it (on Android) or has a perpetual load (on iOS). You can't delete/dismiss or cancel this proof request notification, you have to close the app and open it again and try to delete/dismiss/cancel the proof request notification again.

Note: error behaviour is different for Android and iOS.

Can we make a separate ticket for this? I think I know what's causing it and it'll need a fix in credo

@wadeking98 There's a bug when I follow these steps:

Scan QR code from verifier
Verifier taps "go back" on the "Waiting for response" screen
Holder shares information
Verifier goes to their home screen and sees a new proof request notification that errors out when you try to open it (on Android) or has a perpetual load (on iOS). You can't delete/dismiss or cancel this proof request notification, you have to close the app and open it again and try to delete/dismiss/cancel the proof request notification again.

Note: error behaviour is different for Android and iOS.

This issue seems to be fine now in build 1877 for both iOS and Android @wadeking98


Issue Integrating Dagster with OpenMetadata on Local WSL Environment

Affected module
Does it impact the UI, backend or Ingestion Framework?
backend I believe
Describe the bug
A clear and concise description of what the bug is :
I'm trying to integrate Dagster as a service into OpenMetadata to fetch its metadata. Both OpenMetadata and Dagster are running locally on my WSL Ubuntu environment (on Windows 11). However, I’ve encountered an issue where testing the connection in OpenMetadata consistently fails, and I’m unable to add Dagster successfully.
To Reproduce
Screenshots or steps to reproduce :
-installing opnemetadata successfully on an ubuntu wsl
-installing dagster successfully on an ubuntu wsl
-The Problem:
When I test the Dagster connection in OpenMetadata using either http://localhost:3000/ or http://<IP_ADDRESS>:3000/ in the url field in openmetadata I always get the following error showed in the image below :

However, when I run a curl request directly to the Dagster GraphQL API (http://localhost:3000/graphql) like this :
curl -X POST http://localhost:3000/graphql
-H "Content-Type: application/json"
-d '{
"query": "query PipelineRuns { repositoriesOrError { __typename ... on RepositoryConnection { nodes { id name location { id name } pipelines { id name description } } } } }"
}'
I can successfully retrieve the pipeline metadata:
Response from Dagster:
{
"data": {
"repositoriesOrError": {
"__typename": "RepositoryConnection",
"nodes": [
{
"id": "423451fde460f7f3e2271e37b7006a84d40efa4a::2efeefa6393af3b161eec7662556feb5758538a0",
"name": "deploy_docker_repository",
"location": {
"id": "example_user_code",
"name": "example_user_code"
},
"pipelines": [
{ "id": "3ba0abeb52911b8035672174f3f13a35c2b2e69e", "name": "__ASSET_JOB", "description": null },
{ "id": "b4013548ab09ca3536a591ea9cafca9a7776011b", "name": "my_job", "description": null },
{ "id": "a29265b92dfdce506d8a4a9e146a97ff0834ee34", "name": "my_step_isolated_job", "description": null }
]
}
]
}
}
}
Expected behavior
openmetadata should test successfully the connection
Version:

OS: Windows 11
Python version: 3.10
OpenMetadata version: 1.5.10
dagster version : 1.9.2
OpenMetadata Ingestion package version: should it be installed in this step ? i'm asking because this a service to integrate and not an ingestion tool to be used instead of airflow, because I already successfully configured bigquery (as a service) and successfully tested the omd-bq connection. (if I should install it where should i do it, within the docker of omd or the docker of dagster)?

Additional context
Add any other context about the problem here.

try host.docker.internal

@3angour Can you confirm if the above comment from @harshach helped in solving the Issue, you can reach out in slack community for more support

@harshach when I tested the connection with host.docker.internal I got the following error :
test-connection-Dagster-63BX4dOd Failed to trigger workflow due to airflow API returned Bad Request and response {"error": "Request Validation Error parsing payload: 1 validation error for DagsterConnection\nhost\n Input should be a valid URL, relative URL without a base [type=url_parsing, input_value='host.docker.internal', input_type=str]\n For further information visit https://errors.pydantic.dev/2.7/v/url_parsing"}

@Prajwal214 can you reopen the issue please ? it is clearly not solved yet


Set does not support writing flags

Flags is being retrieved and is the third parameter of the get callback. Flags is an arbitrary 32-bit unsigned integer (written out in decimal) that the server stores along with the data and sends back when the item is retrieved. Clients may use this as a bit field to store data-specific information; this field is opaque to the server.
Memjs should support setting it as well.

Duplicate of #53


Ubuntu 12.10 build vernemq from sources

I have server with ubuntu 12.10 installed.
First I built erlang latest (18) version, but I can't build vernemq with it.
Then I removed erlang and installed 17.5 version. Now I can build vernemq, but after build I can't start it. I get this error message:
ubuntu@ip-10-80-162-94:~/vernemq/_build/default/rel/vernemq$ bin/vernemq start
vm.args needs to have a -name parameter.
  -sname is not supported.

Also I can't find any vernemq.conf file.
I tried latest version of vernemq and also 0.10.1 version.
What I'm doing wrong?

Hi there.
thanks for reporting.
yes, you need 17.5, unfortunately 18 isn't yet working.
can you start from a fresh clone of 0.11.1 using 17.5.
the vernemq.conf should be in _build/default/rel/vernemq/etc/vernemq.conf


code folding state lost when cutting and pasting folded code

Description
code folding state does not maintain when pasting code that has been folded.
Steps to Reproduce

fold the elements to be copied
copy then to the clipboard
paste them in desired location

Expected behavior: Expected the code to remain folded
Actual behavior:  The code expands upon being pasted
Versions
Windows 10 Pro on intel core i5 M20
Release 1.8 build 1.8.0-17108 (alf_localization_release_1.8 3af64fae4)
build timestamp: Wed Nov 02 2016 12:07:29 GMT+0000

Yup, that's still happening with me as well. It's happening on most languages that I have tested. I'm going to look for the issue and try to fix it.


nef - integration with Carbon

Description
We've added an integration with Carbon in our tool nef - you can read more about it here
What is nef?
In a general way, nef is a CLI that lets you have compile-time verification of your documentation written as Xcode Playground. Now, from version 0.3.0 we have integrated Carbon, so you can write in official Apple playgrounds, and export the whole snippets with just one command.
Request
We love Carbon, and we usually use it; this is the reason for this integration and the request for adding in the Carbon's tool list.

@mfix22 LGTM!
Following your suggestions, I have updated the link to nef in README - now it is pointing to Carbon section

Note, it contains an expandable section with more details


How to change ssh client to e.g. tabby under windows 11 to get rid of  the "too open keyfiles"

Hi,
I would like to give x-pipe a try but stuck at the very beginning. I have difficulties to get ssh running under windows 11 with the built in ssh client ( I never use elsewhere). It (the annoying built in ssh client of windows) is complaining about "too open key files". I would like to change the by x-pipe used ssh client to e.g. tabby or at least putty or something else to get rid of such bullshit bingo with "too open key files". I am not willing to fight my way down to a "not too open key file" by changing the file permissions. My key files are good as they are :-). How is it done in x-pipe to force it to use another ssh client of my choice.
Thanks alot for your help
Michael

Yeah the key file permissions and Windows with OpenSSH are weird. Essentially it checks whether a file is either located in a subdirectory of your user home, e.g. C:\Users\...\** or the file permissions are set to only allow the current user. Sadly, OpenSSH does not allow you to overwrite that behavior, so I can't disable it.
Right now, it is not possible to switch the type of ssh client to use because others like tabby or putty have a different interface that is not supported right now. It is possible to change the terminal to open the connection in, but not the ssh client itself.
I hope to somehow implement a workaround for this problem soon somehow. But right now there are sadly not a lot of options other than maybe copying the key file into a user directory like C:\Users\<User>\.ssh and using it from there. This problem annoyed me as well with OpenSSH, I honestly don't understand why they do not give uses the option to disable that behavior.

@crschnick thanks for the reply. I feared you answer will be that way.  I will try your suggestion, thanks.

solved

If you want, you can try whether the latest PTB can fix your original issue: https://github.com/xpipe-io/xpipe-ptb. I added an option to automatically adapt file permissions when OpenSSH complains. However I'm not entirely sure whether that will work for everyone as Windows permissions are weird.

Thanks alot, I will give it a try. But I fear the same as you as I don´t believe that Windows will be cooperative concerning its permission bullshit. As my "problem" is solved in general with the solution you proposed before the latest PTB, I think I will stay with it at the very end - as it works as designed and will not have fight against the rediculous windos permission system.

I just implemented an option to run essentially this: https://superuser.com/a/1329702 when OpenSSH throws in error if the user chooses too.

I guess this can be closed now as xpipe now shows the user possible options on how to fix the problem, even when the manual permissions change does not work.


Long delay on restart of units after each cert renewal following a CA renewal

After a successful CA renewal, 2 issues occur:

After certificate renewal - for +4 minutes the following error occurs:

unit-main-0: 15:14:59 ERROR unit.main/0.juju-log opensearch-peers:1: Cannot connect to the OpenSearch server...
unit-main-0: 15:15:00 ERROR unit.main/0.juju-log opensearch-peers:1: [Errno 111] Connection refused
unit-main-0: 15:15:03 ERROR unit.main/0.juju-log opensearch-peers:1: [Errno 111] Connection refused
unit-main-0: 15:15:06 ERROR unit.main/0.juju-log opensearch-peers:1: [Errno 111] Connection refused
unit-main-0: 15:15:09 ERROR unit.main/0.juju-log opensearch-peers:1: [Errno 111] Connection refused
unit-main-0: 15:15:12 ERROR unit.main/0.juju-log opensearch-peers:1: [Errno 111] Connection refused
unit-main-0: 15:15:15 DEBUG unit.main/0.juju-log opensearch-peers:1: Getting secret app:admin-password
unit-main-0: 15:15:15 DEBUG unit.main/0.juju-log opensearch-peers:1: Starting new HTTPS connection (1): <IP_ADDRESS>:9200
unit-main-0: 15:15:15 DEBUG unit.main/0.juju-log opensearch-peers:1: Error when checking if host <IP_ADDRESS> is up: HTTP error self.response_code=None
self.response_text="HTTPSConnectionPool(host='<IP_ADDRESS>', port=9200): Max retries exceeded with url: / (Caused by SSLError(SSLCertVerificationError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1007)')))"
unit-main-0: 15:15:18 DEBUG unit.main/0.juju-log opensearch-peers:1: Getting secret app:admin-password
unit-main-0: 15:15:18 DEBUG unit.main/0.juju-log opensearch-peers:1: Starting new HTTPS connection (1): <IP_ADDRESS>:9200
unit-main-0: 15:15:18 DEBUG unit.main/0.juju-log opensearch-peers:1: Error when checking if host <IP_ADDRESS> is up: HTTP error self.response_code=None
self.response_text="HTTPSConnectionPool(host='<IP_ADDRESS>', port=9200): Max retries exceeded with url: / (Caused by SSLError(SSLCertVerificationError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1007)')))"
unit-main-0: 15:15:21 DEBUG unit.main/0.juju-log opensearch-peers:1: Getting secret app:admin-password
unit-main-0: 15:15:21 DEBUG unit.main/0.juju-log opensearch-peers:1: Starting new HTTPS connection (1): <IP_ADDRESS>:9200
unit-main-0: 15:15:21 DEBUG unit.main/0.juju-log opensearch-peers:1: Error when checking if host <IP_ADDRESS> is up: HTTP error self.response_code=None
self.response_text="HTTPSConnectionPool(host='<IP_ADDRESS>', port=9200): Max r

....

unit-main-0: 15:18:13 DEBUG unit.main/0.juju-log Executing command: openssl pkcs12 -export -in /tmp/tmp1uvvhlyo.cert -inkey /tmp/tmp2hqex7t8.pem -out /var/snap/opensearch/current/etc/opensearch/certificates/app-admin.p12 -name app-admin -passout pass:xxx
unit-main-0: 15:18:13 ERROR unit.main/0.juju-log err: No cert in -in file '/tmp/tmp1uvvhlyo.cert' matches private key
4007AF64F67E0000:error:05800074:x509 certificate routines:X509_check_private_key:key values mismatch:../crypto/x509/x509_cmp.c:405:
/ out:
unit-main-0: 15:18:13 ERROR unit.main/0.juju-log Error storing the TLS certificates for app-admin:
unit-main-0: 15:18:13 INFO unit.main/0.juju-log TLS certificate for app-admin stored.

An endless deferral events queue happen:

The issue comes from two separate root causes:

When requesting new admin certificates (after the CA certificate has been updated), a new private key is generated as well. This leads to situations where the app-admin secret already contains the new private key, but not yet the new certificate (which can only be updated by the leader unit). This is addressed in https://github.com/canonical/opensearch-operator/pull/436.

When processing the newly requested certificates (from 1.), the operator defers the CertificateAvailableEvent even after updating the certificate on disk and on the secret, in cases when the old-ca has not been removed from the truststore yet (see here). This is not necessary, but causes almost endless deferral loops.

Issues are addressed in https://github.com/canonical/opensearch-operator/pull/436.

Was resolved with https://github.com/canonical/opensearch-operator/pull/436.


Redefine the url path of the pool api according to swagger api.

Coverage remained the same at 41.656% when pulling dcb80237579cf56033295a6991aa69ee510dacc6 on wisererik:development into bd82444289846d4b3b1d2b46be4731c3141e0b57 on opensds:development.


Code Analysis fail build with "Error parsing IL"

Hi. I've just turned on code analysis in my solution and all of a sudden usage of InfoOf.Method breaks my build.
The error I'm getting says

CA0001 : An error was encountered while parsing IL for method: 'Class1..cctor', instruction at offset '0x5' with opcode 'Ldtoken'.

Here's my usage:
private static readonly MethodInfo Method = Info.OfMethod(
            "mscorlib",
            "System.Tuple",
            "Create",
            "T1");

Here's the generated IL:
.method private hidebysig specialname rtspecialname static void .cctor () cil managed
{
	IL_0000: ldtoken method class [mscorlib]System.Tuple`1<!!0> [mscorlib]System.Tuple::Create(!!0)
	IL_0005: call class [mscorlib]System.Reflection.MethodBase [mscorlib]System.Reflection.MethodBase::GetMethodFromHandle(valuetype [mscorlib]System.RuntimeMethodHandle)
	IL_000a: castclass [mscorlib]System.Reflection.MethodInfo
	IL_000f: stsfld class [mscorlib]System.Reflection.MethodInfo InfoOfProblem.Class1::Method
	IL_0014: ret
}

Any ideas? I'm not even it's a problem with InfoOf of VS 2015 Code Analysis. Who would I even ask?

what version of .net r u targeting?

I'm targeting 4.5. The same is happening with 4 and 4.6

so i just added some more tests with your scenario. and it passes PEVerify so essentially the IL is good, and it runs ok as well. so ATM it looks like a false negative in VS 2015 Code Analysis
I pinged some twitter accounts https://twitter.com/SimonCropp/status/736859076513042432
let see what comes of that

was told to raise it on MS connect. not really a path that has ever worked for me in the past. feel free to raise it on MS Connect yourself

FYI, I created a Connect issue: https://connect.microsoft.com/VisualStudio/feedback/details/3103266


Implement column resizing

https://github.com/cardstack/boxel/assets/12637010/3a578a71-addf-4fa0-b588-f9edde899ee5

[x] Resizing by dragging the resizer button.
[x] Open/close the left/right panel by double-clicking the resizer button.
[x] Persist this via local storage (applies to all of the code mode).

*) The resizer button icon needs to be updated.

I tested this and it works nice, great job!
There is one thing that is bothering me-I think the whole logic for this layout feature should be moved out of the code-mode component otherwise we risk this file getting huge and having too many responsibilities. I believe ideally code mode component should stick to managing data flows while layout logic is dealt with in another place.
Perhaps you can consider extracting this resizing logic somehow. I'm wondering if a combination of a modifier and state service would work. Maybe you could implement a modifier which would enhance the columns with the resizing feature when attached to the code mode component. One other option could be a new ResizableCodeModeColumns component with named blocks for columns

I've decided to create a new component in Boxel UI called 'resizable-panel'.
https://github.com/cardstack/boxel/assets/12637010/dd05f78e-332b-4e7b-8173-a4d840336802

The resizing handle works well, but there seems to be something off with the column widths. I get something like this:

Fixed!
https://github.com/cardstack/boxel/assets/12637010/2a7d5a4e-bcaf-4bcc-950e-62925afb1012

After updating the resize handler button:
https://github.com/cardstack/boxel/assets/12637010/8d36d616-e737-48fd-bec2-30ec06e00e62


fetch list of collectors from source of truth

Source of truth:

RIPE RIS: https://stat.ripe.net/data/rrc-info/data.json
RouteViews: https://api.routeviews.org/collector/?format=json

Created a new cratebgpkit-commons to gather such information.
https://crates.io/crates/bgpkit-commons
See the documentation and the following example on how to use it:
use bgpkit_commons::collectors::get_routeviews_collectors;
fn main() {
    println!("get route views collectors");
    let mut routeviews_collectors = get_routeviews_collectors().unwrap();
    routeviews_collectors.sort();
    let earliest = routeviews_collectors.first().unwrap();
    let latest = routeviews_collectors.last().unwrap();
    println!("\t total of {} collectors", routeviews_collectors.len());
    println!(
        "\t earliest collector: {} (activated on {})",
        earliest.name, earliest.activated_on
    );
    println!(
        "\t latest collector: {} (activated on {})",
        latest.name, latest.activated_on
    );
}


[Feature Request] Support the Spotify URI scheme

I think it'll be useful to implement the Spotify URI scheme, it's used on the web player and in discord embeds, and it can be inconvenient to manually look for the song on Spot.
The only crate I found that can register a URI handler is system_uri which is archived and not maintained anymore, I forked it and updated it's dependencies, I'll try publishing it to crates.io, alternatively we can just do it manually in Spot because we don't have to worry about other platforms, we just need to make a desktop file in ~/.local/share/applications/ with the proper information.
According to wikipedia, this is how the Spotify URI scheme is used

spotify:<artist|album|track>:<id> or
spotify:search:<text> or
spotify:user:<username>:playlist:<id>
example:
spotify:track:2jCnn1QPQ3E8ExtLe6INsx

Hi! This is indeed something that would be very handy :) I guess we only need to handle the http(s) scheme (users are unlikely to encounter a Spotify URI in the wild)

I've only seen the URI used (in the web player if you right click and select open in desktop app, it redirects you to a spotify:... link), afaik the offical client doesn't handle http(s).
I'll try working on this, I'm still learning GTK and I'm still not super comfortable in Rust so I'll see how far I can get.


Update user-defined-functions.md

The current docs refer to the function as a table.  This doesn't seem intentional.

@hallipr : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@shsagir
Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off
approved by ADX content team

Invalid command: '#sign-off'. Only the assigned author of one or more file in this PR can sign off. @shsagir

Docs Build status updates of commit d023aaa:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

data-explorer/kusto/query/functions/user-defined-functions.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


btn-group-fluid class missing in documentation

Hi Maya,
just to report that the .btn-group-fluid class is missing from the documentation.
I gonna take a MOOC on Github to propose Pull requests instead of bothering you.
Thanks for all.

.btn-group-fluid is already covered in http://daemonite.github.io/material/docs/4.0/material/buttons/#toggle-buttons (scroll past the first two examples in this section).

Sorry for this mistake. I couldn't find it again.
Should get some sleep.
Thx for all Maya


[event-hubs] Load balancing can starve out new consumers

When fixing a bug in load balancing I noticed a possible deadlock. Let's assume the following:

We have two event processors ('a' and 'b')
We have 4 partitions (although the number isn't important but it makes the example simpler)

Let's say that 'a' owns all partitions. So our partition map looks like this:
'a': [ 0, 1, 2, 3 ]
Now 'b' comes up at some later time (perhaps we're just scaling out). At this point we have two processors.
What we'd like to end up with is something like this:
'a': [ 0, 1 ]
'b': [ 2, 3 ]
Where the partitions are evenly divided, thus splitting the load. And under ideal conditions this will happen since 'a' will detect 'b' and be okay with owning fewer partitions (ie, 'b' and 'a' won't keep fighting over time).
However, under un-ideal conditions we run into an issue in our current implementation - whether 'a' can even detect 'b' exists. Currently, our only way of knowing which event processors exist is by checking the ownership map. If 'b' fails to ever win a fight to own a partition it'll never be seen and never accounted for in our calculation of number of partitions to own.
This leads to a potentially never-ending fight between 'a' and 'b' - 'b' is fighting to exist, and 'a' is continually asserting that it should continue to own partitions that it already has.
Now..assuming we solve that our second problem is whether 'a' will ever stop trying to own excess partitions. As is, we never "abandon" a partition unless 'a' is actively stopped, which won't work for this case.
Some things we could do:

Create a concept of an active processor blob/entity that isn't tied to ownership. This will solve the first problem (detecting additional processors). That blob/entity needs to have a lease or some sort of active timeout associated with it, much like an ownership blob does today.
Active processors to stop re-asserting ownership if they detect they have too many owned partitions.

@chradek, @srnagar Can this be related to the customization that we want to do for load balancing strategies?

Hmmm, possibly. With a little creativity we could fix this issue with a load balancing strategy. I think even if we just changed how we claim partitions such that a processor doesn't claim every partition it owns every time it lists claims, and instead only reclaims partitions it thinks it owns after some multiple of the processor loop delay, that would solve this problem too.
Basically the issue now is that one processor that has a claim on all partitions might reclaim each of the partitions at the same time as another processor attempts to claim one. Since the etag for the claims are updated by the original processor, the new processor fails to get the claim due to having an old etag, and the cycle continues because each tick of the processor loop the original processor will try to claim everything it currently owns unless it knows about another processor.

This should be partially mitigated by #9706 for the reasons listed here:
https://github.com/Azure/azure-sdk-for-js/issues/7017#issuecomment-652556082
Basically, #9706 reduces the amount of time between when the list of partition ownership is retrieved, and when a client attempts to claim a partition. This should help reduce the frequency with which the etag of an ownership is changed before the 'new' client has a chance to update an ownership.
Technically this issue could still happen unless we either:

let the user configure an interval for reclaiming partitions
devise a way for clients to communicate their existence separate from the partitionOwnership types

However this should now be very rare, and only temporary (as new clients will randomly choose which partition to steal from existing clients, whereas reclaiming partitions is in a non-random order)

Closing this as it has been mitigated in event hubs 5.3.0-preview.1 to no longer be something that can happen permanently.


AppRTC between android and IOS device

When I try to do video chat between Android-Android,IOS-IOS devices ,it is working,But If i try from Android-IOS,IOS-Android,It is not working.
Can anyone help me to resolve this problem?

Did you got solution for this ??

Did you got solution for this

Sorry, it still stands unresolved. Here is an article about my problem,you can refer it.
https://stackoverflow.com/questions/47462247/webrtc-is-not-working-connecting-safari-with-chrome-for-android


Rename repo

We're following naming conventions here with node-red-contrib-<name>

@johanstokking, @htdvisser suggested we'd rename the repos for clarity as we'll be adding more device and application libraries, which I think makes a lot of sense.

<lang|platform>-<app|device>-lib

The package name of the node is still node-red-contrib-ttn.

Following the naming conventions are fine for when we publish to npm, but in our repository we should stick with our own naming.


Adding RedwoodJS framework to the list supported frameworks

Which project is this feature idea for?
Turborepo
Describe the feature you'd like to request
I would like to be able to run the create-turbo command and use a different initial framework (different than nextjs that is).
Realizing that you already have support for several frameworks - like svelte, these can only be added to the initial content of the monorepo (nextjs framework). I would like to get the initial monorepo host RedwoodJS and then continue adding various packages from the IoT context.
Is the approach documented as Add Turborepo to your existing project equally valuable as my proposal "Adding RedwoodJS framework to the list examples?
In summary:

Is adding RedwoodJS compatible with Vercel's future vision?
Would Vercel be interested in a contribution that would do the necessary work for item  1. above?

Describe the solution you'd like
Too soon for that as my understanding of how to do this depends on the comments I hope to get from you.
Describe alternatives you've considered
Too soon for that as my understanding of how to do this depends on the comments I hope to get from you.

@adriatic feel free to contribute to the examples/ directory, a PR would be great! create-turbo doesn't currently support templates, it has only a single template with some variations for using npm/yarn/pnpm, so it probably can't support what you're asking for.

As mentioned by Mehul, feel free to open a PR so users could use npx create-turbo@latest -e with-redwood and get started.
Short of that, though, I'll get this closed for inactivity!


Prevent black flash after starting a compositor?

A short moment after starting Sway and RiverVM, the screen goes black for 2 seconds to apply new refresh rate and VRR mode. From user's perspective this looks like the monitor turns off and on after entering the compositor.
For comparison, if you configure output directly in Sway without Way-Displays, monitor doesn't flash black after starting since the monitor mode is applied synchronously with the start of the compositor instead of asynchronously.
/tmp/way-displays.1.game.log
I [12:48:59] way-displays version 1.8.2
I [12:48:59]
I [12:48:59] Found configuration file: /home/game/.config/way-displays/cfg.yaml
I [12:48:59]
I [12:48:59] Active configuration:
I [12:48:59]   Arrange in a ROW aligned at the TOP
I [12:48:59]   Order:
I [12:48:59]     Dell Inc. Dell AW3821DW #HLAYMxgwABTR
I [12:48:59]   Scaling: OFF
I [12:48:59]   Auto scale: OFF
I [12:48:59]   Mode:
I [12:48:59]     Dell Inc. Dell AW3821DW #HLAYMxgwABTR: 3840x1600@120Hz
I [12:49:00]
I [12:49:00] DP-3 Arrived:
I [12:49:00]   info:
I [12:49:00]     name:     'DP-3'
I [12:49:00]     make:     'Dell Inc.'

Is there any way to prevent the screen from going black after starting the compositor (meaning is there a way to update monitor settings earlier/synchronously with the start)?

wlr and compositors will always set the preferred mode on startup before way-displays is connected, hence the change in mode.
It may be possible to prevent compositors from doing this initial mode set:

Who does this? wlroots or the compositor?
Look at adding an option to wlroots / sway / river / hyprland


training error

I have used facescrub dataset and dlib to align them before running facenet_train.py
After 1st epoch of training and it has the following error at validation below. Do you know what is wrong?
Traceback (most recent call last):
File "./facenet_train.py", line 274, in
tf.app.run()
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/platform/app.py                                                                   ", line 30, in run
sys.exit(main(sys.argv))
File "./facenet_train.py", line 144, in main
global_step, embeddings, loss, 'validation', summary_writer)
File "./facenet_train.py", line 212, in validate
image_paths, num_per_class = facenet.sample_people(dataset, nrof_people, FLA                                                                   GS.images_per_person)
File "/mnt/2TB/src/facenet/facenet/src/facenet.py", line 606, in sample_people
class_index = class_indices[i]
IndexError: index 52 is out of bounds for axis 0 with size 52

Hi,
It looks like there are too few images in the validation set.
The function "sample_people" tries to sample 45 persons with 40 images per person. But if some of the classes in the dataset contains less than 40 images more classes will be used to get 45*40 images.
One way to solve it would be to use a smaller "train_set_fraction", e.g. 0.8 or something. Then the validation set will be larger. Another (ugly) way could be to change the seed. Then a different set of classes will be included in the validation set, and perhaps these classes together will have enough images for validation.

Can I reduce 40 images/person? What is the impact of doing this?
Thanks
On Thu, May 19, 2016 at 10:56 PM, David Sandberg<EMAIL_ADDRESS>wrote:

Hi,
It looks like there are too few images in the validation set.
The function "sample_people" tries to sample 45 persons with 40 images per
person. But if some of the classes in the dataset contains less than 40
images more classes will be used to get 45*40 images.
One way to solve it would be to use a smaller "train_set_fraction", e.g.
0.8 or something. Then the validation set will be larger. Another (ugly)
way could be to change the seed. Then a different set of classes will be
included in the validation set, and perhaps these classes together will
have enough images for validation.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/davidsandberg/facenet/issues/18#issuecomment-220523749

Yes, reducing either the number of images per person or the number of people would help to solve the problem. But i'm not sure how it will affect the training. Also, you need to make sure that you can get a whole number of batches, i.e. that people_per_batch*images_per_person = batch_size * n, where n is an integer.

Also making sure the batch_size and people_per_batch are multiples of 3 helps.

@davidsandberg I put --lfw_dir place empty as i don't want to see the validation accuracy. Then also I am getting the error as below,
Traceback (most recent call last):
File "src/train_tripletloss.py", line 486, in
main(parse_arguments(sys.argv[1:]))
File "src/train_tripletloss.py", line 186, in main
args.embedding_size, anchor, positive, negative, triplet_loss)
File "src/train_tripletloss.py", line 212, in train
image_paths, num_per_class = sample_people(dataset, args.people_per_batch, args.images_per_person)
File "src/train_tripletloss.py", line 327, in sample_people
class_index = class_indices[i]
IndexError: index 4 is out of bounds for axis 0 with size 4

@PoonamZ any findings for the above error, I am getting the same error, Thanks in advance

@KumailHussain I didn't get any solution for this.

Hi, I am using "train_tripletloss.py" as training algorithm, but every time I try to run it, I get the error below and
I can't find any solution for that, please help me.
Restoring pretrained model: ./models/20170512-110547/model-20170512-110547.ckpt-250000
Traceback (most recent call last):
File "train_tripletloss.py", line 493, in
main(parse_arguments(sys.argv[1:]))
File "train_tripletloss.py", line 192, in main
args.embedding_size, anchor, positive, negative, triplet_loss)
File "train_tripletloss.py", line 218, in train
image_paths, num_per_class = sample_people(dataset, args.people_per_batch, args.images_per_person)
File "train_tripletloss.py", line 333, in sample_people
class_index = class_indices[i]
IndexError: index 2 is out of bounds for axis 0 with size 2

@sassiaicha  , was facing the same error during training. Same Index error. Looks like you have chosen only 2 classes which is causing the problem.
I myself was using only 10 custom classes with 3 images for each. But when I increased my dataset (used LFW dataset for training), the code ran for training. Maybe try increasing your dataset.


Really obscure rust crash when compiling seemingly innocent library

On Darwin, when I was trying to package a rust library, I noticed that an innocent program would crash the compiler with the following message:
error: failed to build archive: bad archive: truncated or malformed archive (offset to next archive member past the end of the archive after member std-23c7ab017b56c6b0.0.bytecode.deflate)

I'm filing the issue under nixpkgs because it doesn't seem to happen in the upstream version of rustc, so something about how we do things is either triggering an obscure bug in rustc (that doesn't normally get triggered) or we've caused it to break in some other way.
This is the fruit of my minimization:
with import <nixpkgs> {};

let
  src = writeText "repro.rs" ''
    pub extern fn test() {}
  '';
# If you s/dylib/cdylib/, it works fine
in runCommand "repro" {} ''
  mkdir -p $out
  ${rustc}/bin/rustc --crate-type dylib ${src} --out-dir $out
''

I'm aware that dylib isn't really intended as a user-facing feature, but apparently cdylib isn't quite there yet and is breaking things for at least one downstream package that still uses dylib. Also, if that library were really corrupted, wouldn't cdylib fail as well?

nix-build ./test.nix                                                                                ~
these derivations will be built:
  /nix/store/vig547mnaccijrz47c1cv2s275a6wixw-repro.drv
building path(s) ‘/nix/store/y7skd7js8xibvlc2lz1rsbfz8bl8jqvl-repro’
builder for ‘/nix/store/vig547mnaccijrz47c1cv2s275a6wixw-repro.drv’ failed to produce output path ‘/nix/store/y7skd7js8xibvlc2lz1rsbfz8bl8jqvl-repro’
error: build of ‘/nix/store/vig547mnaccijrz47c1cv2s275a6wixw-repro.drv’ failed

cc @wizeman @Mic92 @the-kenny who seem to be fairly active Nix rust contributors

Can't say anything about Darwin, don't have a Mac handy. Works fine on NixOS as of 1e1472e

@copumpkin what llvm version is used by upstream rustc?

They keep their own custom version, which we support in our expression but don't use by default. I tried switching back to it but it didn't help. I'll probably iteratively get closer and closer to their build until I figure out what's causing this and file a bug :(

On Jul 18, 2017, at 06:09, Jörg Thalheim<EMAIL_ADDRESS>wrote:
@copumpkin what llvm version is used by upstream rustc?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

I tried switching on forceBundledLLVM (and disabling --enable-llvm-link-shared which I think is required if you force the bundled LLVM) and it doesn't help; in fact it gives a less useful error message because I guess errors in this space improved in LLVM 4 😄

So, I'm getting this same issue when trying to compile Firefox on MacOS. Don't have much to add, just that this still seems to be an issue.

Oh, firefox now includes rust code?

@copumpkin Yes, iirc some quantum/stylo 'code' is in Firefox 57, though it needs to be enabled via about:config I think.

For some reason it checks for it? Here's the branch if anyone wants to look at it:
https://github.com/matthewbauer/nixpkgs/tree/firefox-on-macos

And version 56 already has an option to enable Rust implementation of URL parser.

Anyone in the Nix+Rust community know how we might go about debugging this? I filed https://github.com/rust-lang/rust/issues/42857 but since it's almost impossible for anyone outside of Nix+Darwin to reproduce it, I'm not sure how to proceed.

Anyone in the Nix+Rust community [...]

cc @garbas

@copumpkin does the same happen with rust overlay? https://github.com/mozilla/nixpkgs-mozilla

Have you tried to set a different linker with: rustc -C linker=?

Yeah, I've tried with a different linker (pretty sure) but haven't tried the rust overlay. Is that meaningfully different in the build of rustc itself? I assumed it was a different package set but the compiler was largely the same.

No, the overlay uses a pre-compiled version from mozilla.

Actually rust does not deal with ar-archives at all. This is all delegated to the dedicated linker.

Here's the full log for building firefox on macosx:
checking for libffi > 3.0.9... yes
checking MOZ_FFI_CFLAGS... -I/nix/store/n0sxvxbwzbvjgnxb06cyigacmp5ks839-libffi-3.2.1-dev/include
checking MOZ_FFI_LIBS...
DEBUG: Its error output was:
DEBUG: | error: failed to build archive: bad archive: truncated or malformed archive (offset to next archive member past the end of the archive after member std-6227198845c1feb5.0.bytecode.deflate)
DEBUG: |
ERROR: Cannot compile for x86_64-apple-darwin17.3.0 with /nix/store/qhv9ddgwpn3zsjy33awk7zxg2jq2p5cb-rustc-1.20.0/bin/rustc
The target may be unsupported, or you may not have
a rust std library for that target installed. Try:

  rustup target add x86_64-apple-darwin

builder for '/nix/store/w0sbrgpljm7cl27jmbkpg025nrggy9xs-firefox-unwrapped-56.0.2.drv' failed with exit code 1
cannot build derivation '/nix/store/if4y6jaf9mjlgla2z92s1cdxxn346xx0-firefox-56.0.2.drv': 1 dependencies couldn't be built
error: build of '/nix/store/if4y6jaf9mjlgla2z92s1cdxxn346xx0-firefox-56.0.2.drv' failed

Is firefox using dylib? Does cdylib work? It might be a less obscure place to start the conversation on why they're not equivalent for these uses, since I think they're trying to discourage folks from using dylib except for compiler plugins.

Oh, interesting, it's happening with --crate-type staticlib (sorry I missed that in your log above). I haven't seen that before.

It might be worth trying to upgrade rust itself, we are two minor versions behind? The only issue is that last time I tried firefox wasn't compiling. Maybe with the firefox 57 release that issue has been fixed.

It looks like we're only one version behind? From @matthewbauer's log:
checking rustc version... 1.20.0
checking cargo version... 0.22.0

Back when I first noticed this, we were on 1.17.

Err just re-read my last comments in 8 days and I will be right :) https://forge.rust-lang.org/

There's pr for rust ready, now that ff 57 is released we should be able to update.

Rust 1.22 is in #31982 if you want to give it a try

@copumpkin I think this issue has been fixed. At least your .nix file succeeds under nixpkgs-unstable.

Yeah, #34227 fixed this.

Awesome, thanks!


Don't use #send etc. to include or prepend modules

Back in the day (before Ruby 2.1), Module.include and Module.prepend were private methods. So when people authored gems / patched monkeys, they wrote e.g. Foo.send(:include, MyPatch). In our day and age, I think we should encourage everybody to just write Foo.include(MyPatch).
Describe the solution you'd like
Recommend calling Foo.include(Bar) over Foo.send(:include, Bar), Foo.public_send(:include, M), etc.
Recommend calling Foo.prepend(Bar) over Foo.send(:prepend, Bar), Foo.public_send(:prepend, M), etc.
Describe alternatives you've considered
None
Additional context
None

I think it makes sense because it can be unified with a new mix-in style using public include and prepend (and extend) methods. I opened a PR #7279.


Supply a license

Could you add a license to this repository? I'm hesitant to use this library without one.
https://help.github.com/articles/open-source-licensing/

Added Apache2. If that causes any issues for you let me know.
Thanks!
James

thanks!


[Enhancement] Allow a widget (eg a table) to be marked read-only, to use it as record selector only

Following this interesting discussion, I put a part of @dsagal proposal here, to avoid drowning it in the pile of messages.
Allow a widget (e.g. a table) to be marked read-only. So even if you have permissions to edit, it behaves as read-only.
The read-only table could be used to select a record, or to select linked records from a linked table. This prevents accidental edits of the master record, while the user is creating linked records.

Some users are creating specific widgets to be used as record selectors. This can result in widget code duplication, with additional maintenance burden.
IMHO it would probably be easier to implement a read only setting for widgets as proposed here.

Thanks @emanuelegissi ! I took a brief look, and I don't think it's a good first issue, but it may not be too hard.
Grist already implements read-only views for viewers (users with no edit access), and some aspects of summary tables. We could reuse that logic.
Specifically, Grist has code to pay attention to gristDoc.isReadonly (defined in GristDoc.ts), to field.disableEditData (defined in ViewFieldRec.ts), and to viewSection.disableAddRemoveRows (defined in ViewSectionRec.ts).
If we added a readOnly option to ViewSectionRec's "options", with UI to toggle it in the side panel, the following could be a good path to reuse existing logic maximally:

Add viewSection.isReadOnly() to be gristDoc.isReadonly || viewSection.optionsObj.prop("readOnly")
Update field.disableEditData and viewSection.disableAddRemoveRows to respect viewSection.isReadOnly() when it's set to true.
Replace certain references to gristDoc.isReadonly with viewSection.isReadOnly.

@dsagal thank you for pointing out the main hooks for such development. I am going to take a look in the next few weeks, and see if I can contribute a pull request. I'm going to really try hard.
( At first humble sight, it seems beyond my capabilities, but who knows...)

It'll be hard to navigate, in particular because this code uses observables, and different bits of code mix older knockout.js observables and newer grainjs ones. If you dive into it and need to figure out what all that is about, check out https://github.com/gristlabs/grainjs/blob/master/docs/basics.md#observables and https://github.com/gristlabs/grainjs/blob/master/docs/misc.md#integrating-observables.


A0 2712: Feature net minor revamp

Description

Feature nets should be started from scratch, chainspec generated from node PR version, binary PR test version (short session)
Small refactor of create and destroy feature net actions
minor bugifx for e2e completion check in PR pipeline so that workflow does not error in case it is cancelled either by dev or by concurrency (ie pushing another commit to PR)
remove trigger:create-hotfix-featurenet as it is not needed anymore
added manual workflow dispatch possibility to create feature net
changed runner for updatenet and featurenet spawn jobs to ubuntu-20.04
Added expiration to feature nets

Type of change
Please delete options that are not relevant.

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)

Checklist:

I have made neccessary updates to the Infrastructure
I have made corresponding changes to the existing documentation

This was made by mistake from wrong branch, closing.


fix installation instructions for pulumi esc

Just using curl get.pulumi.com will install the pulumi CLI.  While that embeds esc as pulumi env it's not what we want to direct users to when they look at esc, especially as the guide linked above refers to plain esc.

looks like this is a dupe of https://github.com/pulumi/esc/pull/374 :)

Oh whoops, I didn't realize that.  Let's merge that one since it's the exact same anyway.


Allow for enhancement of existing DB with older seasons

When I first gave it a try I used the existing bash scripts. It created a new DB filled with data.
When I went to explore I noticed it was from the latest season only.
I went back to see if I could add another season and got some errors in the process.
I would like to have the opportunity to add older seasons after I have a DB in place.
Current State:
Create a DB and populate with data on first creation only.
Subsequently a refresh of data is supported for the latest season
Enhancement
Together with the above also add the option to load one or more specific seasons after a DB is initialized
Requirements

It needs to see if a season was already loaded to avoid the extra requests

Could be overwritten with a force flag

Need to add an UPSERT capability in the DB

Ah yeah the bash scripts are just for testing. I believe if you use the commandline options then you can achieve what you want using the --season flag. Lemme know if that works.


[ja] OTel L10N announcement blog post doesn't show up

What needs to be changed?
#5044 added the Japanese translation of the announcement of OTel docs multilingalization, but it doesn't show up on the website.
https://opentelemetry.io/ja/blog/2024/
What is the name + path of the page that needs changed?
ja/blog/2024/docs-localized/
Additional context:
#5035 has the same directory structure to add, so it may cause the same issue.

I seem to see it now, @ymotongpoo . Could you confirm?
https://opentelemetry.io/ja/blog/2024/docs-localized/

Oh, it came out now! There should be some lag t

Oh, it came out now! There should be some lag t

Nice! Thanks for checking!


Rust checker: allow to ignore secondary spans

Have a look at the JSON object in :ALEInfo and see what information is there. If there's no other information, we can always look at the message string. I would hope there's something like an error type.

A bunch of comments:

There can be multiple primary spans in any given error (see #47481 for example).
The secondary spans do not have extra semantic meaning embedded in the output. They can point to the definition, add necessary context ("can't assign closure bound borrow to external binding"/"external binding defined here"/"this is the closure the borrow is valid for"). Lifetime errors are probably the ones that will be the hardest to deal with and present nicely when inline in an editor, specially if you want to distill it to a single line.
From a cursory look, you only use the gutter and the lower status line for output. This should be enough for the great majority of the errors, but for things like lifetime errors it'd be nice if you could open a secondary pane with the full text (which is sometimes needed to be able to understand the error).
It is conceivable that there are errors that point to the place with a problem using a secondary span, while maybe pointing at something less specific with the primary span (as it was before the above referenced PR, for example). This should get better over time, but it is a caveat to be aware of. If you see some like this, file a ticket.
We want to cater for single line output errors, going as far as having a flag for it, but I don't feel the support is perfect yet.

@estebank Cool, thank you for all of the information. That's very helpful.

Thanks @estebank!
@w0rp I think it's your call on what to pick between the options: sort, ignore-always, ignore-optionally (the PR at its present state), open-special-window-for-secondaries, or do-nothing. Or if you more ideas that's also great.

Having read everything, filter out secondary spans completely, then put text from them in the detail key, so you can see extra information with :ALEDetail. The secondary spans aren't errors worth flagging in files, and only seem to be useful for getting some additional information, and that's what :ALEDetail is for.

I don't understand how multiple primary spans are supposed to work. It makes sense to return a list of errors, and then some additional information about each error. How can an error exist in two locations at the same time?

@w0rp a case off the top of my mind:
error: multiple unused imports
LL | use foo::{bar, qux};
   |           ^^^  ^^^

Before it was
error: unused import
LL | use foo::{bar, qux};
   |           ^^^

error: unused import
LL | use foo::{bar, qux};
   |                ^^^

I would count those as two errors, not one error with two "spans". If spans are mixed together in one error object, how do you know which secondary spans are associated with which primary spans?

We purposely avoid having cases where different secondary spans refer to different primary spans.

Could you provide an example, using the JSON format, of how you expect someone to use the JSON information? What should you do if you just want to get an error, and stuff additional information into a string someone can view if they want to?

Closing due to inactivity.

:disappointed:

@da-x We can do this for now, as the secondary spans are too hard to figure out. Can you add some tests for this?

@w0rp should tests include actual Cargo project execution and letting ALE check it, or something more whitebox-y to drive the JSON parsing code? Some reference to similar tests will be helpful.

The tests won't execute Cargo itself. Look at test/handler/test_rust_handler.vader. You can provide some output for the handler there. You can Save and restore the setting, and you can pass in the same test data with secondary spans.

@da-x do you mind if I push some commits containing test?

@da-x do you mind if I push some commits containing test?

Go for it. Whatever it takes to push this forward. :)

@RyanSquared Go ahead, every help is appreciated!

I found a potential issue. JSON dump below:
{
  "message": "multiple unused formatting arguments",
  "code": null,
  "level": "error",
  "spans": [
    {
      "file_name": "<anon>",
      "byte_start": 21,
      "byte_end": 27,
      "line_start": 1,
      "line_end": 1,
      "column_start": 22,
      "column_end": 28,
      "is_primary": false,
      "text": [
        {
          "text": "fn main() { println!(\"Test\", 1, 2) }",
          "highlight_start": 22,
          "highlight_end": 28
        }
      ],
      "label": "multiple missing formatting specifiers",
      "suggested_replacement": null,
      "suggestion_applicability": null,
      "expansion": {
        "span": {
          "file_name": "<println macros>",
          "byte_start": 120,
          "byte_end": 154,
          "line_start": 4,
          "line_end": 4,
          "column_start": 29,
          "column_end": 63,
          "is_primary": false,
          "text": [
            {
              "text": "( $ crate :: io :: _print ( format_args_nl ! ( $ ( $ arg ) * ) ) ) ; } # [",
              "highlight_start": 29,
              "highlight_end": 63
            }
          ],
          "label": null,
          "suggested_replacement": null,
          "suggestion_applicability": null,
          "expansion": {
            "span": {
              "file_name": "<anon>",
              "byte_start": 12,
              "byte_end": 34,
              "line_start": 1,
              "line_end": 1,
              "column_start": 13,
              "column_end": 35,
              "is_primary": false,
              "text": [
                {
                  "text": "fn main() { println!(\"Test\", 1, 2) }",
                  "highlight_start": 13,
                  "highlight_end": 35
                }
              ],
              "label": null,
              "suggested_replacement": null,
              "suggestion_applicability": null,
              "expansion": null
            },
            "macro_decl_name": "println!",
            "def_site_span": {
              "file_name": "<println macros>",
              "byte_start": 0,
              "byte_end": 244,
              "line_start": 1,
              "line_end": 5,
              "column_start": 1,
              "column_end": 78,
              "is_primary": false,
              "text": [
                {
                  "text": "(  ) => ( print ! ( \"\\n\" ) ) ; ( $ ( $ arg : tt ) * ) => (",
                  "highlight_start": 1,
                  "highlight_end": 59
                },
                {
                  "text": "{",
                  "highlight_start": 1,
                  "highlight_end": 2
                },
                {
                  "text": "# [ cfg ( not ( stage0 ) ) ] {",
                  "highlight_start": 1,
                  "highlight_end": 31
                },
                {
                  "text": "( $ crate :: io :: _print ( format_args_nl ! ( $ ( $ arg ) * ) ) ) ; } # [",
                  "highlight_start": 1,
                  "highlight_end": 75
                },
                {
                  "text": "cfg ( stage0 ) ] { print ! ( \"{}\\n\" , format_args ! ( $ ( $ arg ) * ) ) } } )",
                  "highlight_start": 1,
                  "highlight_end": 78
                }
              ],
              "label": null,
              "suggested_replacement": null,
              "suggestion_applicability": null,
              "expansion": null
            }
          }
        },
        "macro_decl_name": "format_args_nl!",
        "def_site_span": null
      }
    },
    {
      "file_name": "<anon>",
      "byte_start": 29,
      "byte_end": 30,
      "line_start": 1,
      "line_end": 1,
      "column_start": 30,
      "column_end": 31,
      "is_primary": true,
      "text": [
        {
          "text": "fn main() { println!(\"Test\", 1, 2) }",
          "highlight_start": 30,
          "highlight_end": 31
        }
      ],
      "label": null,
      "suggested_replacement": null,
      "suggestion_applicability": null,
      "expansion": null
    },
    {
      "file_name": "<anon>",
      "byte_start": 32,
      "byte_end": 33,
      "line_start": 1,
      "line_end": 1,
      "column_start": 33,
      "column_end": 34,
      "is_primary": true,
      "text": [
        {
          "text": "fn main() { println!(\"Test\", 1, 2) }",
          "highlight_start": 33,
          "highlight_end": 34
        }
      ],
      "label": null,
      "suggested_replacement": null,
      "suggestion_applicability": null,
      "expansion": null
    }
  ],
  "children": [],
  "rendered": "error: multiple unused formatting arguments\n --> <anon>:1:30\n  |\n1 | fn main() { println!(\"Test\", 1, 2) }\n  |                      ------  ^  ^\n  |                      |\n  |                      multiple missing formatting specifiers\n\n"
}
{
  "message": "aborting due to previous error",
  "code": null,
  "level": "error",
  "spans": [],
  "children": [],
  "rendered": "error: aborting due to previous error\n\n"
}

As is shown in the above JSON, there is a case where an is_primary: true value does not have a label, but a secondary one does. In the current PR, it is possible for the message to be completely unnoticed due to the lack of a label.
This could possibly be an issue with Rust. Either way, for now, I am putting my development on the tests on hold due to this issue (and this is why we do tests!!!).
I think, if this is intended behavior, then we should use a "candidate" system, which could unfortunately result in only one span being highlighted per section. If no label exist, the message is updated regardless of primary-ness. If a label does exist, and the new span is primary, the message is updated. I'm not sure how I would deal with multiple primary spans.

@RyanSquared that can be considered a bug in the Rust output. The primary spans weren't given a label as that would have added too much clutter:
error: multiple unused formatting arguments
 --> src/main.rs:1:30
  |
1 | fn main() { println!("Test", 1, 2) }
  |                      ------  ^  ^
  |                      |
  |                      multiple missing formatting specifiers

but in this case we could switch the spans around and make the string literal the primary span (which makes sense, it's what needs to be fixed half of the time). This would reverse the emphasis of the error but not the spirit of it (you have a mismatch between formatting specifiers and arguments).
Created https://github.com/rust-lang/rust/issues/55350 as a follow up.

Tests that broke are not relevant to the current changes, and I think this should merge cleanly into master. I tested it out locally and it works fine. @w0rp looks good?

Cheers! :beers: