MeshInstance3D disappears at runtime if its skeleton is not a direct parent

Tested versions
Reproducible in 4.3.stable and 4.4.dev2
NOT reproducible in 4.2.stable
System information
Godot v4.3.stable - Windows 10.0.22631 - GLES3 (Compatibility) - NVIDIA GeForce GTX 1660 SUPER (NVIDIA; <IP_ADDRESS>09) - Intel(R) Core(TM) i7-10700F CPU @ 2.90GHz (16 Threads)
Issue description
When a MeshInstance3D's skeleton is not a direct parent, it works fine in editor, but at runtime, the mesh disappears. This worked previously in 4.2. In the attached video, the red fish has the skeleton under the mesh, the green fish has the mesh under the skeleton, and the blue fish has skeleton and mesh as siblings. When you run the game, the red fish and the blue fish disappear.
https://github.com/user-attachments/assets/d515c724-bc86-4996-bcf8-04696d870107
Steps to reproduce

Open test.tscn in the attached project
Run the game
Observe that only the green fish remains

Minimal reproduction project (MRP)
bug-report.zip

CC @TokageItLab

Wait a second, is the root node a Node, will that cause trouble compared to a Node3D? I remember something didn't act well without a path that is Node3D.

Wait a second, is the root node a Node, will that cause trouble compared to a Node3D? I remember something didn't act well without a path that is Node3D.

The bug appears regardless.
But in any case, this was working fine in 4.2, and I have been using it all over the place in my project, so I'm not sure what changed.

I have no idea. The standard procedure for debugging a regression is to run git-bisect until godot starts working again.

If the Skeleton change is having an effect, it may be because the SkeletonModifier implementation has reduced unnecessary duplicated Skin updates in the Skeleton and only does it once in the deferred process. However, I think this should not be reverted back to the old Skin updating way from a performance standpoint.
One thing I noticed is that when the Mesh and Skeleton are siblings, like the blue fish, the rendering works when the Skeleton is first. However, changing ProcessPriority has no effect. (I assume this is because the Deferred process does not affect ProcessPriority)
So I think the right direction would be to make some call to Skin in RenderingServer::skeleton_bone_set_transform(), or do it in Skeleton before rs->skeleton_bone_set_transform() line.
However, in my opinion, MeshInstance not being a child of a Skeleton is a pretty niche case, and I think it's not so worth to deal with that case by reducing the performance. Probably the ability to assign any non-ancestor Skeleton to a MeshInstance should be discontinued, although it breaks compatibility. How do you think about it? @reduz

Or the problem may be that Skin is discarded by the rendering server before the deferred process. If so, making it retain it might solve everything, but I am not familiar with rendering servers so I am not sure. Do you have any ideas? @clayjohn

Or the problem may be that Skin is discarded by the rendering server before the deferred process. If so, making it retain it might solve everything, but I am not familiar with rendering servers so I am not sure. Do you have any ideas? @clayjohn

I don't think anything has changed in the rendering server that would impact this. Note, the RenderingServer has no concept of Skin, it just takes the list of bones from the skeleton and uses it to transform the mesh, nothing in the node hierarchy should impact the RenderingServer either as it doesn't reference the node hierarchy anywhere

@clayjohn My thinking is that the MeshInstance is rendered without the skeleton, so when the MeshInstance is iterated by SceneTree, the mesh is registered with the rendering server and drawn.
Then, I expected that if the skeleton is first, the result of applying the skeleton's bone transformation will be rendered, but if the skeleton is later and the process is deferred, the MeshInstance has already been rendered, and if the skeleton's bone transformation is newly passed to it and the past drawing of the MeshInstance would disappear, is that not the case?

No, the renderer doesn't work that way. The entire scene system executes before rendering even starts. The calls to the RenderingServer just register resources with the RenderingServer, then at the end of the frame, the RenderingServer takes all the resources and draws them.
Even if you defer an update of the skeleton, the update will happen before anything is drawn. The order in the scene tree has no impact on drawing

If the Deferred process and its order in the scene tree is not directly related to the render server, then this issue is quite mysterious.
We know that it is Skeleton that has _process, and found that the order of Mesh and Skeleton in the scene tree has an effect, but Process Priority does not, so it is doubtful that it is even a process order issue.
If we can confirm that somewhere in the process of passing Skin and Bone information to the render server in Skeleton changes depending on the order in the scene tree, it may help to solve the problem, but we will have to investigate in detail.

If the Deferred process and its order in the scene tree is not directly related to the render server, then this issue is quite mysterious.
We know that it is Skeleton that has _process, and found that the order of Mesh and Skeleton in the scene tree has an effect, but Process Priority does not, so it is doubtful that it is even a process order issue.
If we can confirm that somewhere in the process of passing Skin and Bone information to the render server in Skeleton changes depending on the order in the scene tree, it may help to solve the problem, but we will have to investigate in detail.
For now, the workaround I can say at this point is, if the Skeleton and MeshInstance are siblings, always put the Skeleton above the MeshInstance in the SceneTree. If the Skeleton is a child of the MeshInstance, the structure must be changed.

If it helps at all, I have some other MI3D in my game with skeleton children that ARE working properly, though I have no idea what the differentiating factor is. Every time I try to repro a working case from scratch, it just ends up not working.

If there is a difference in the project, it could be the Skin importing way and where it is stored, since only the runtime has problems, it is feasible that the init process is being affected by them. For example, depending on the order of Skeleton and Mesh declarations, the Skin may be shrinking oddly in the init process.

If there is a difference in the project, it could be the Skin importing way and where it is stored, since only the runtime has problems, it is feasible that the init process is being affected by them. For example, depending on the order of Skeleton and Mesh declarations, the Skin possibly be shrinking oddly in the init process.

Interesting. Is there a rule of thumb for where I should be storing the skin to avoid this? I honestly don't know much about how all these pieces come together, the documentation is pretty sparse.

In my experience, there have been cases where some CSG software has bound a Skin with an oddly small scale to a seemingly correct bone, but this is probably not the case with MRP.
One case I can think of where a mesh would be invisible is when a Skeleton is specified in the MeshInstance and a new Skin is assigned to it. Possibly the init process does not handle the Skin binding information correctly and this is causing such a situation.


LookupError: No installed app with label '<module 'django.contrib.admin.models' from '[...]'>'.

Hello,
When I run one of these commands, I get the correct output:
python manage.py dumpscript auth
python manage.py dumpscript admin

But if I run the same without app name, I get an error
python manage.py dumpscript
Traceback (most recent call last):
  File "D:\Dev\path\to\project\venv\lib\site-packages\django\apps\registry.py", line 148, in get_app_config
    return self.app_configs[app_label]
KeyError: <module 'django.contrib.admin.models' from 'F:\\Flux\\CaffeineServer\\venv\\lib\\site-packages\\django\\contrib\\admin\\models.py'>

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "manage.py", line 10, in <module>
    execute_from_command_line(sys.argv)
  File "D:\Dev\path\to\project\venv\lib\site-packages\django\core\management\__init__.py", line 353, in execute_from_command_line
    utility.execute()
  File "D:\Dev\path\to\project\venv\lib\site-packages\django\core\management\__init__.py", line 345, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "D:\Dev\path\to\project\venv\lib\site-packages\django\core\management\base.py", line 348, in run_from_argv
    self.execute(*args, **cmd_options)
  File "D:\Dev\path\to\project\venv\lib\site-packages\django\core\management\base.py", line 399, in execute
    output = self.handle(*args, **options)
  File "D:\Dev\path\to\project\venv\lib\site-packages\django_extensions\management\utils.py", line 58, in inner
    ret = func(self, *args, **kwargs)
  File "D:\Dev\path\to\project\venv\lib\site-packages\django_extensions\management\commands\dumpscript.py", line 102, in handle
    models = get_models(app_labels)
  File "D:\Dev\path\to\project\venv\lib\site-packages\django_extensions\management\commands\dumpscript.py", line 136, in get_models
    models += [m for m in get_models_compat(app)
  File "D:\Dev\path\to\project\venv\lib\site-packages\django_extensions\compat.py", line 153, in get_models_compat
    return apps.get_app_config(app_label).get_models()
  File "D:\Dev\path\to\project\venv\lib\site-packages\django\apps\registry.py", line 155, in get_app_config
    raise LookupError(message)
LookupError: No installed app with label '<module 'django.contrib.admin.models' from 'D:\\Dev\\path\\to\\project\\venv\\lib\\site-packages\\django\\contrib\\admin\\models.py'>'.
(venv)

Setting some breakpoints try to find a way to fix, I saw the error comes from that block in dumpscript.py:
    # If no app labels are given, return all
    if not app_labels:
        for app in get_apps():
            models += [m for m in get_models_compat(app)
                       if m not in EXCLUDED_MODELS]
        return models

get_apps() returns a list of applications when get_models_compat(app) seems to require an app_label instead of an app as argument. I don't know how to fix this issue, so I'm sorry, I will not be able to create a PR.
Regards
Note: Django 1.9.5, django-extensions1.6.3

3a0ca9e40a11729d30cc3f2d916c446321805f97 should hopefully fix this.

Yeah that sounds good. I can't test right now, but the patch make sense. I will be able to test as soon as a next release will be published on Pypi.
Thanks !

1.6.4 is up on PyPi :)
(Please reopen or create a new ticket if this isn't solved)

Actually 1.6.5 is the latest version since PyPi borked while uploading 1.6.4...
PyPi seems broken at the moment for uploading wheel packages :(


chore: rename constant

Motivation
Rename ENTITYID to snake_case ENTITY_ID for readability.
Solution
Rename the constant.

@adamegyed @fangting-alchemy @jaypaik could you please take a look? Thanks

lgtm! happy to cut a new tag including this change once this is in

That'd be great, thank you!

@huaweigu This is now available on https://github.com/erc6900/reference-implementation/releases/tag/v0.8.0-rc.6

@huaweigu This is now available on https://github.com/erc6900/reference-implementation/releases/tag/v0.8.0-rc.6

Thank you!


Custom wlr_buffer from wl_resource

Custom wlr_buffer from wl_resource take two
Supersedes https://github.com/swaywm/wlroots/pull/2934

BTW, any reason why this is still a draft?

No reasons, I'll put it as a standard patch.

I've tried using this work to turn wl_shm into a buffer resource impl: https://github.com/swaywm/wlroots/pull/3061
Seems to work fine!

I'm not a huge fan of introducing global state for this, OTOH avoiding global state would require more work. Maybe something like registering the resource impls in wlr_compositor would do the trick?

what about wlr_renderer ?

I'm suggesting wlr_compositor, since this is the parent object for surfaces, where the wl_buffer resources are turned into wlr_buffer. However I now realize turning a wl_buffer resource into a wlr_buffer happens elsewhere too, e.g. for screencopy.

I'd rather not add this to the renderer: the renderer shouldn't really care about Wayland protocol related things.

I've checked the code a bit and I think adding a parent object would add a lot of clutter to the current interface.
Mostly because the only place we create a custon wlr_buffer is wlr_buffer_from_resource, which we already cleaned to have just a wl_resource argument. Plus I can't find where to take the reference to the wlr_compositor.

Yeah, it's probably fine for now. Something to think about before stabilization I guess.


Operations Manager Eventlog Spam

Version: <IP_ADDRESS>
The pure management pack is spamming the Operations Manager log with performance metrics. Every single metric for every array is being logged as an informational message. In our case this generating around 11,000 events per hour on the management servers and making the log very difficult to use.
Example Logs.
03/12/2020 08:34:57           2104 Information      HostPerformanceRule.ps1 [<IP_ADDRESS>] : Saving metric : @{writes_per_sec=0; name=host1; usec_per_write_op=0; output_per_sec=5631; san_usec_per_read_op=1; reads_per_sec=7; input_per_sec=0; time=12/03/2020 08:34:57; san_usec_per_write_op=0; usec_per_read_op=237}
03/12/2020 08:34:57           2104 Information      HostPerformanceRule.ps1 [<IP_ADDRESS>] : Saving metric : @{writes_per_sec=0; name=host2; usec_per_write_op=0; output_per_sec=8191; san_usec_per_read_op=1; reads_per_sec=2; input_per_sec=0; time=12/03/2020 08:34:57; san_usec_per_write_op=0; usec_per_read_op=457}
03/12/2020 08:34:57           2104 Information      HostPerformanceRule.ps1 [<IP_ADDRESS>] : Saving metric : @{writes_per_sec=0; name=host3; usec_per_write_op=0; output_per_sec=4095; san_usec_per_read_op=2; reads_per_sec=1; input_per_sec=0; time=12/03/2020 08:34:57; san_usec_per_write_op=0; usec_per_read_op=130}
03/12/2020 08:34:57           2104 Information      HostPerformanceRule.ps1 [<IP_ADDRESS>] : Saving metric : @{writes_per_sec=0; name=host4; usec_per_write_op=0; output_per_sec=4095; san_usec_per_read_op=1; reads_per_sec=1; input_per_sec=0; time=12/03/2020 08:34:57; san_usec_per_write_op=0; usec_per_read_op=162}

Have you altered the thresholds from the defaults to suit your environment? Also, the latest <IP_ADDRESS> release addresses some of the alert issues.

Yes I have increased the collection frequency. While this will increase the amount of messages generated, I do not believe this kind of information should be logged to the Operations Manager log.
Maybe it would be useful to have an override to enable debug or some such so as not to fill the log with performance metrics. Or a single log entry with a summary of everything collected in that run.

RFE created.


Create primarygoal.txt

Our website does not describe in detail about the course so I have provided some information below.
Primary Goal is a registered Learning Provider in UK [UKPRN: 10054804]. Our courses are for ages 16-25.
Each course lasts for 1 year. The course is a Level 4 qualification, classed as higher education ( REF: https://www.gov.uk/what-different-qualification-levels-mean/list-of-qualification-levels )
Below is the course content:
•	Networking and Architecture
•	Mobile and Operating Systems
•	Cloud Services
•	Coding and Logic
•	Business Processes
•	CompTIA A+ – 220-901 & 220-902
•	MTA Cloud Fundamentals – 98-369
•	MTA Server Admin Fundamentals – 98-365
•	MTA Network Fundamentals – 98-366
•	ITIL Foundation
•	Interconnecting Cisco Networking Devices
•	Cyber Security
•	Office 365
•	Computing Curriculum
•	Using technology across core subjects
•	Microsoft Office Essentials
•	iPads in the Classroom
Please reply if there is more information needed.

@pgbentrigg Do I understand correctly that your organization provides teachers and courses to other schools, and it effectively means that Primary Goal does not have students enrolled to Primary Goal only, is it correct? Is Primary Goal a college or a university in terms of common understanding of these terms - does it have classrooms in the organization's building, does it enroll students who study in Primary Goal only in the period of their studies, does it provide the students with the course content in the classrooms?

@philipto

"Is Primary Goal a college or a university in terms of common understanding of these terms - does it have classrooms in the organization's building, does it enroll students who study in Primary Goal only in the period of their studies, does it provide the students with the course content in the classrooms?"

The establishment is a specialist college. We teach people to become IT teachers. The courses are funded by government.
We have classrooms and we teach our students here. The students are enrolled in full-time Education with us. It is compulsory to be in full-time education till age 18.

"Do I understand correctly that your organization provides teachers and courses to other schools, and it effectively means that Primary Goal does not have students enrolled to Primary Goal only, is it correct?"

The teachers that our website explains are actually our students who are learning to become teachers.
This is how it works for a student:

Student finds our course through local school looking for teacher or government organisation "National Careers Service".
We teach the students our specialty course, in our real-world classrooms. Content of the course stated in previous comment. Students attend the college 5 days a week.
During a vocational part of the course, the students will work in a local school (our website is more aimed towards getting more schools involved with the course). Students are mentored by Primary Goal and attend the college 1 day a week during vocational.

I can understand your confusion as this is very uncommon.
Please reply if you need anything else.

@pgbentrigg Thank you for all the details. Your explanation clearly shows that your institution is a college of a kind which is absolutely eligible for being listed in this repository. Your pull request is now merged. Thank you!


Ruby 2.4 - rspec test failure - Validator Spec - returns stray quote instead of unclosed quote

Checked out the repository. Rspec tests pass on Ruby 2.1.X, 2.2.X and 2.3.X but a test failure happens on 2.4.0 - not sure why yet, will investigate further. Possibly ordering?
Failures:

  1) Csvlint::Validator validation with multiple lines:  .each() -> `parse_contents` parses malformed CSV and catches unclosed quote
     Failure/Error: expect(validator.errors.first.type).to eql(:unclosed_quote)

       expected: :unclosed_quote
            got: :stray_quote

       (compared using eql?)

       Diff:
       @@ -1,2 +1,2 @@
       -:unclosed_quote
       +:stray_quote
     # ./spec/validator_spec.rb:70:in `block (3 levels) in <top (required)>'

Ah, after a bit if digging, I think I'm right in that it is actually the Ruby CSV class which throws the error when parse_line is called. I think I was confused as LineCSV also has a parse_line method ;) - so I can only assume that this is a change to how the CSV handles unclosed and stray quotes. In this case, 2.4 considers it a stray quote rather than an unclosed quote.

Hey @jamesjefferies - did you encounter this while attempting to bring CSVlint into Octopub? I ask given the name of your local branch To create a Ruby 2.4 / Rails 5 compatible version of CSVLint, as it raises a good question of whether this gem needs to be considerate of rails apps its included within

@quadrophobiac yes, that rings a bell - it's a dependency of Octopub and as we were updating Octopub to 2.4/5 I created that branch of CSVlint which was compatible and therefore could be used..

I've "resolved" this in the tests now. I can't make Ruby throw a stray quote error on 2.4, so for now it's in there but pending. The unclosed quote test is improved to make more sense, and works fine.


added Hindi translation

Thanks!
I'm curious about one thing: you translated Catima itself to कैटीमा, is it normal in Hindi to translate app names?
I also noticed you translated the full description and then deleted the translation again: https://hosted.weblate.org/translate/catima/fastlane/hi/?checksum=502bfece59e1216b&sort_by=-priority,position#history. Was this deletion by accident?

Thanks!
I'm curious about one thing: you translated Catima itself to कैटीमा, is it normal in Hindi to translate app names?
I also noticed you translated the full description and then deleted the translation again: https://hosted.weblate.org/translate/catima/fastlane/hi/?checksum=502bfece59e1216b&sort_by=-priority,position#history. Was this deletion by accident?

if viewer reading in Hindi then this read कैटीमा same as Catima, word which have no meaning exist in Hindi are write in this way.
I deleted because i think because their are some conflict at that time.

if viewer reading in Hindi then this read कैटीमा same as Catima, word which have no meaning exist in Hindi are write in this way.

Ah, I see what you mean, if I look up "Coca Cola" on Wikipedia I see their name is also written in Hindi: https://hi.wikipedia.org/wiki/कोका_कोला.

I deleted because i think because their are some conflict at that time.

I'm not seeing any conflicts there so I'm a bit confused (there is a conflict in this MR because someone else added their name to the end too, you can use GitHub's "Resolve conflicts" button to fix that)

I resolve Conflict you Can Check!

But what is up with https://hosted.weblate.org/translate/catima/fastlane/hi/?checksum=502bfece59e1216b ? You wrote it and then deleted it? Can you put that back now if it the translation is correct?

I just added you can check now.


on android if you change setting for fonts to 'Huge' the letters in text input are cut by a third

Though the fonts are increasing proportionally the hight of the component does not.  As a temporary fix I just pass the height for the text input 45 but it is not the best solution.

bump


Fix build scripts to comsume msbuild v12.0

Currently, we are using the msbuild with v4.0 to compile .NET code. this change is to consume v12.0 one.

What is the reason for this change?

[Current coverage][cc-pull] is 88.71%

Merging [#475][cc-pull] into [master][cc-base-branch] will increase coverage by 0.42%

@@             master       #475   diff @@
==========================================
  Files            47         47
  Lines          4952       4952
  Methods           0          0
  Messages          0          0
  Branches        649        649
==========================================
+ Hits           4372       4393    +21
+ Misses          562        541    -21
  Partials         18         18

Powered by Codecov. Last updated by [6cecf1c...a87e105][cc-compare]
[cc-base-branch]: https://codecov.io/gh/Microsoft/Mobius/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/Microsoft/Mobius/compare/6cecf1c7b247abe93a9683cc64a9996bd9a4fc10...a87e1052137704702514858ea0caad32a7fa2646
[cc-pull]: https://codecov.io/gh/Microsoft/Mobius/pull/475?src=pr

The version of msbuild consumed in existing script is 4.6.1038.0 (under C:\Windows\Microsoft.NET\Framework), which is very old. From VS2013, the default path of MSBUILD is changed to "C:\Program Files (x86)\MSBuild\12.0\bin". You can get the path from your machine's regedit: HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\MSBuild\ToolsVersions\12.0.
Currently, we are officially support Visual Studio 2013 and later, so we should change the msbuild version to match it.
Another reason to change is that the msbuild 4.0 does not support C++ 11. my native code cannot be compiled.

LGTM other than the minor comment on adding comments to batch scripts

LGTM

LGTM


prepare submommand applies patches, but not excluded files

I'm using charts-build-scripts on the rke2-charts repo. Let's assume that we have a rke2-xxx package/chart with the following structure:
packages/rke2-xxx/
├── generated-changes
│   ├── exclude
│   └──  patch
└── package.yaml

I have a "clean" state of my repo and some files both in exclude/ and path/ directories.
Then I'm doing:
make prepare

The chart structure changes to:
packages/rke2-xxx/
├── charts
├── generated-changes
│   ├── exclude
│   └──  patch
└── package.yaml

My expectation would be that charts content:

Has the patches applied (which is true, that happens)
Doesn't have exluded files (which is not true, that doesn't happen and the files are still there)

I would expect the condition 2 to be also true. To make my patches correct again, I need to manually remove files I want to exclude.

This is currently functional in rancher/charts. Example:
https://github.com/rancher/charts/tree/dev-v2.6/packages/rancher-monitoring/rancher-monitoring/generated-changes/exclude
https://github.com/rancher/charts/tree/dev-v2.6/charts/rancher-monitoring/100.1.0%2Bup19.0.3
I'd be interested in understanding what the edge case that does not result in the excluded files being removed since the logic for that part has not changed in a while: https://github.com/rancher/charts-build-scripts/blob/b21b7fdd58cdb8e4403404a8ecc3c2d74c972604/pkg/change/apply.go#L46-L54
@vadorovsky Can you provide more details on the specific chart where you encountered this issue? (feel free to ping me on Slack otherwise)

Closing as stale since Arvind asked for more info from the reporter and hasn't gotten it. This can be reopened with more info.


libsoup_build failed

Trying to compile OpenWebRTC on my Mac OSX 10.10.1, got this error running ./build-all.sh -Cbr osx ios-simulator
9 warnings generated.
CCLD     libsoup-2.4.la
CC       soup-cookie-jar-sqlite.lo
CC       soup-gnome-features.lo
CC       soup-proxy-resolver-gnome.lo
CC       soup-password-manager-gnome.lo
CCLD     libsoup-gnome-2.4.la
libtool: link: warning: /opt/workspaces2/ios/fwk-openwebrtc/openwebrtc/scripts/dependencies/out/i386-apple-darwin10/libsoup/../gettext-<IP_ADDRESS>/lib/libintl.la' seems to be moved libtool: link: warning: /opt/workspaces2/ios/fwk-openwebrtc/openwebrtc/scripts/dependencies/out/i386-apple-darwin10/libsoup/../libxml2-2.7.8/lib/libxml2.la' seems to be moved
libtool: link: warning: `/opt/workspaces2/ios/fwk-openwebrtc/openwebrtc/scripts/dependencies/out/i386-apple-darwin10/libsoup/../sqlite-autoconf-3080500/lib/libsqlite3.la' seems to be moved
GISCAN Soup-2.4.gir
Can't open directory
Command '['/opt/workspaces2/ios/fwk-openwebrtc/openwebrtc/scripts/dependencies/build/i386-apple-darwin10/libsoup/libsoup/tmp-introspect3VAbRC/Soup-2.4', '--introspect-dump=/opt/workspaces2/ios/fwk-openwebrtc/openwebrtc/scripts/dependencies/build/i386-apple-darwin10/libsoup/libsoup/tmp-introspect3VAbRC/functions.txt,/opt/workspaces2/ios/fwk-openwebrtc/openwebrtc/scripts/dependencies/build/i386-apple-darwin10/libsoup/libsoup/tmp-introspect3VAbRC/dump.xml']' returned non-zero exit status 1
make[3]: *** [Soup-2.4.gir] Error 1
make[2]: *** [all] Error 2
make[1]: *** [all-recursive] Error 1
make: *** [all] Error 2
Failed building...libsoup
mv libsoup libsoup_build_failed_on_150219-19:26:35_using_script_with_md5sum_5838cdedea082deecc9acb79c05ccc91c4f53bf4

Have you had any luck with this issue? It looks like a GObject Introspection issue and we don't use any soup API through introspection so if you like you can edit the build-libsoup.sh script to add --disable-introspection to the autogen.sh parameters, say after --disable-shared

Closing as we have now moved to a new build system called cerbero. Please try to build using that.
https://github.com/EricssonResearch/openwebrtc/wiki/Building-OpenWebRTC


fix(front): 稳定依赖包版本

Codecov Report

Merging #70 (4705f4f) into V1.0.3rc (3e04fbc) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           V1.0.3rc      #70   +/-   ##
=========================================
  Coverage     53.28%   53.28%
=========================================
  Files           177      177
  Lines          8758     8758
=========================================
  Hits           4667     4667
  Misses         4091     4091

Continue to review full report at Codecov.

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


Breaking changes (Code Cleanup)

Check roll direction for VTOL

(https://github.com/PX4/Firmware/issues/4797)

Air frame init script files attempt to set rates that will fail on all FMU PWM

(https://github.com/PX4/Firmware/issues/7261)

Make airframe outputs consistent

https://github.com/PX4/Firmware/issues/7474

Make FW throttle placement consistent

http://discuss.px4.io/t/pixhawk-mini-pwm-on-pin-3-is-a-constant-1500-us-resolved/3656

Merge duplicates and delete old, many configs don't even set airframe specific parameters, but have a unique mixer that is identical to others

Many identical airframe configurations
Need to be consolidated/removed
Redesign channel assignments
Ability to configure rates, no rate per channel setting
https://github.com/PX4/Firmware/issues/7261

QGC: only show selection of airframes applicable to that HW
make sure we have a set of generic airframes that cover the vast majority of use cases
should the convention be for VTOL to instead use the MAIN mixer for FW (px4io backup) and not duplicate the more complicated FW mixers? Then we'd only need to simpler MC AUX mixers.

https://github.com/PX4/Firmware/issues/5608

Hey, this issue has been closed because the label status/STALE is set and there were no updates for 30 days. Feel free to reopen this issue if you deem it appropriate.
(This is an automated comment from GitMate.io.)

This message was created automatically by mail delivery software.
A message that you sent could not be delivered to one or more of its
recipients. This is a temporary error. The following address(es) deferred:
<EMAIL_ADDRESS>Domain evercl.com has exceeded the max emails per hour (200/200 (100%)) allowed.  Message will be reattempted later
------- This is a copy of the message, including all the headers. ------
Received: from o7.sgmail.github.com ([<IP_ADDRESS>]:51196)
by apollo.graphium.net with esmtps (TLSv1.2:ECDHE-RSA-AES128-GCM-SHA256:128)
(Exim 4.89_1)
(envelope-from<EMAIL_ADDRESS>id 1eoU6e-0007i4-Hc
for<EMAIL_ADDRESS>Wed, 21 Feb 2018 08:06:44 -0500
DKIM-Signature: v=1; a=rsa-sha1; c=relaxed/relaxed; d=github.com;
h=from:reply-to:to:cc:in-reply-to:references:subject:mime-version:content-type:content-transfer-encoding:list-id:list-archive:list-post:list-unsubscribe;
s=s20150108; bh=maTTg2qcTluIfn5tbobTZsjtSFc=; b=BYxxqo6VffL9V7cl
e12OOf/oPmLnwbUgi93bOuz8XwIPnJrcSc6+isG8Vp4e30az7/huRtWu8LgwESG9
P4EMqk3BgIfXPzHSBfBqI89kLGre1K5/JrlbP/LV9E2KACaguGittH6mqIdFObyz
bt9Hozv29SbO3Fu6LBG91zBr8M4=
Received: by filter0001p1iad2.sendgrid.net with SMTP id filter0001p1iad2-8884-5A8D6E42-4
2018-02-21 13:04:02.164726088 +0000 UTC
Received: from github-smtp2b-ext-cp1-prd.iad.github.net (github-smtp2b-ext-cp1-prd.iad.github.net [<IP_ADDRESS>])
by ismtpd0002p1iad1.sendgrid.net (SG) with ESMTP id fe2OSdD7RuGW9urPljSkMw
for<EMAIL_ADDRESS>Wed, 21 Feb 2018 13:04:01.992 +0000 (UTC)
Date: Wed, 21 Feb 2018 13:04:02 +0000 (UTC)
From: PX4 Build Bot<EMAIL_ADDRESS>Reply-To: PX4/Firmware<EMAIL_ADDRESS>To: PX4/Firmware<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Message-ID<EMAIL_ADDRESS>In-Reply-To<EMAIL_ADDRESS>References<EMAIL_ADDRESS>Subject: Re: [PX4/Firmware] Breaking changes (Code Cleanup) (#7650)
Mime-Version: 1.0
Content-Type: multipart/alternative;
boundary="--==_mimepart_5a8d6e3ff1e89_73dd3fac5e1aaf281747f3";
charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: PX4BuildBot
X-GitHub-Recipient: evercltech
X-GitHub-Reason: subscribed
List-ID: PX4/Firmware <Firmware.PX4.github.com>
List-Archive: https://github.com/PX4/Firmware
List-Post<EMAIL_ADDRESS>List-Unsubscribe<EMAIL_ADDRESS>https://github.com/notifications/unsubscribe/ACSImJD-M08O3f7gyjFI6htGfIUgodeyks5tXBQ_gaJpZM4OdNwX
X-Auto-Response-Suppress: All
X-GitHub-Recipient-Address<EMAIL_ADDRESS>X-SG-EID: f9JBnm3v7f66Apd0aaWYVdsqW29S4Z0GExn3vPRUuGkuSCud51EdJkwBSMdkQT1j0uh3EFeePPHt+G
f0GD8jVGyPXNLw6FXeoarZS6tJDmufU5dTOM/taxfj+8ZBwg/ky4IqsQDV8weWr9azTxtjdvjDl4A5
U9nmvMsEB6FRbdCaY+25TtCMEgXUUziKFOk8+rRrswYArk5MyNJm7ZC28A==
----==_mimepart_5a8d6e3ff1e89_73dd3fac5e1aaf281747f3
Content-Type: text/plain;
charset=UTF-8
Content-Transfer-Encoding: 7bit
Closed #7650.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/PX4/Firmware/issues/7650#event-1484605155
----==_mimepart_5a8d6e3ff1e89_73dd3fac5e1aaf281747f3
Content-Type: text/html;
charset=UTF-8
Content-Transfer-Encoding: 7bit
Closed #7650.
—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or mute the thread.

----==_mimepart_5a8d6e3ff1e89_73dd3fac5e1aaf281747f3--


ValueError: Expected value argument (Tensor of shape (100,)) to be within the support (GreaterThan(lower_bound=0.0)) of the distribution Weibull

When I run this command: (It is a test command)
python tools/train_net.py --num-gpus 4 --eval-only --config-file ./configs/OWOD/t1/t1_test.yaml SOLVER.IMS_PER_BATCH 4 SOLVER.BASE_LR 0.005 OUTPUT_DIR "./output/t1_final" MODEL.WEIGHTS "/home/multiai3/Jiuqing/OWOD-master/output/t1/model_final.pth"
I got an Error as follows:
Traceback (most recent call last):
File "tools/train_net.py", line 169, in
args=(args,),
File "/home/multiai3/Jiuqing/OWOD-master/detectron2/engine/launch.py", line 59, in launch
daemon=False,
File "/home/multiai3/anaconda3/envs/OWOD/lib/python3.7/site-packages/torch/multiprocessing/spawn.py", line 230, in spawn
return start_processes(fn, args, nprocs, join, daemon, start_method='spawn')
File "/home/multiai3/anaconda3/envs/OWOD/lib/python3.7/site-packages/torch/multiprocessing/spawn.py", line 188, in start_processes
while not context.join():
File "/home/multiai3/anaconda3/envs/OWOD/lib/python3.7/site-packages/torch/multiprocessing/spawn.py", line 150, in join
raise ProcessRaisedException(msg, error_index, failed_process.pid)
torch.multiprocessing.spawn.ProcessRaisedException:
-- Process 2 terminated with the following error:
Traceback (most recent call last):
File "/home/multiai3/anaconda3/envs/OWOD/lib/python3.7/site-packages/torch/multiprocessing/spawn.py", line 59, in _wrap
fn(i, *args)
File "/home/multiai3/Jiuqing/OWOD-master/detectron2/engine/launch.py", line 94, in _distributed_worker
main_func(*args)
File "/home/multiai3/Jiuqing/OWOD-master/tools/train_net.py", line 139, in main
res = Trainer.test(cfg, model)
File "/home/multiai3/Jiuqing/OWOD-master/detectron2/engine/defaults.py", line 525, in test
results_i = inference_on_dataset(model, data_loader, evaluator)
File "/home/multiai3/Jiuqing/OWOD-master/detectron2/evaluation/evaluator.py", line 145, in inference_on_dataset
evaluator.process(inputs, outputs)
File "/home/multiai3/Jiuqing/OWOD-master/detectron2/evaluation/pascal_voc_evaluation.py", line 122, in process
classes = self.update_label_based_on_energy(logits, classes)
File "/home/multiai3/Jiuqing/OWOD-master/detectron2/evaluation/pascal_voc_evaluation.py", line 102, in update_label_based_on_energy
p_known = self.compute_prob(energy, self.known_dist)
File "/home/multiai3/Jiuqing/OWOD-master/detectron2/evaluation/pascal_voc_evaluation.py", line 87, in compute_prob
pdf = distribution.log_prob(dx).exp()
File "/home/multiai3/anaconda3/envs/OWOD/lib/python3.7/site-packages/torch/distributions/transformed_distribution.py", line 149, in log_prob
log_prob = log_prob + _sum_rightmost(self.base_dist.log_prob(y),
File "/home/multiai3/anaconda3/envs/OWOD/lib/python3.7/site-packages/torch/distributions/transformed_distribution.py", line 138, in log_prob
self._validate_sample(value)
File "/home/multiai3/anaconda3/envs/OWOD/lib/python3.7/site-packages/torch/distributions/distribution.py", line 289, in _validate_sample
"Expected value argument "
ValueError: Expected value argument (Tensor of shape (100,)) to be within the support (GreaterThan(lower_bound=0.0)) of the distribution Weibull(scale: 3.947634220123291, concentration: 2.3650074005126953), but found invalid values:
tensor([-0.0010,  0.0091,  0.0192,  0.0293,  0.0394,  0.0495,  0.0596,  0.0697,
0.0798,  0.0899,  0.1000,  0.1101,  0.1202,  0.1303,  0.1404,  0.1505,
0.1606,  0.1707,  0.1808,  0.1909,  0.2010,  0.2111,  0.2212,  0.2313,
0.2414,  0.2515,  0.2616,  0.2717,  0.2818,  0.2919,  0.3020,  0.3121,
0.3222,  0.3323,  0.3424,  0.3525,  0.3626,  0.3727,  0.3828,  0.3929,
0.4030,  0.4131,  0.4232,  0.4333,  0.4434,  0.4535,  0.4636,  0.4737,
0.4838,  0.4939,  0.5040,  0.5141,  0.5242,  0.5343,  0.5444,  0.5546,
0.5647,  0.5748,  0.5849,  0.5950,  0.6051,  0.6152,  0.6253,  0.6354,
0.6455,  0.6556,  0.6657,  0.6758,  0.6859,  0.6960,  0.7061,  0.7162,
0.7263,  0.7364,  0.7465,  0.7566,  0.7667,  0.7768,  0.7869,  0.7970,
0.8071,  0.8172,  0.8273,  0.8374,  0.8475,  0.8576,  0.8677,  0.8778,
0.8879,  0.8980,  0.9081,  0.9182,  0.9283,  0.9384,  0.9485,  0.9586,
0.9687,  0.9788,  0.9889,  0.9990])
could you tell me why this error happened?

I find the result in the issue. At frist I didn't find because of the title. I think this title should be correct.
https://github.com/JosephKJ/OWOD/issues/66#issuecomment-1066089566
Thank you for your job!


Amp 56383 go sdk docs

Dev Docs PR
Description
Adds docs for Go SDK (Alpha) for Analytics.
Update Go SDK (Alpha) docs

Updated SDK sources list

Deadline
When do these changes need to be live on the site?
Change type

[ ] Doc bug fix. Fixes #[insert issue number]. Amplitude contributors include Jira issue number.
[ ] Doc update.
[X] New documentation.
[ ] Non-documentation related fix or update.

PR checklist:

[x] My documentation follows the style guidelines of this project.
[x] I previewed my documentation on a local server using mkdocs serve.
[x] Running mkdocs serve didn't generate any failures.
[x] I have performed a self-review of my own documentation.

@bgiori when you get the chance, can you take a look at the code sample also? curious if the usage is the ideal DX

The above changes are meant to resolve conflicts with main branch. It should be the correct version and thanks @liuyang1520 for helping.
Try to explain what was going on:

somehow I forked this repo and git origin was set to the forked repo
I forgot origin was set to forked repo and rebased branch to main


Extension causes high cpu load

Issue Type: Performance
Extension Name: vsliveshare
Extension Version: 0.3.1071
OS Version: Darwin x64 18.2.0
VSCode version: 1.30.0

:warning: Make sure to attach this file from your home-directory: /Users/bxj6191/ms-vsliveshare.vsliveshare-unresponsive.cpuprofile.txt [ms-vsliveshare.vsliveshare-unresponsive.cpuprofile.txt](https://github.com/MicrosoftDocs/live-share/files/2696509/ms-vsliveshare.vsliveshare-unresponsive.cpuprofile.txt)  :warning:
Find more details here: https://github.com/Microsoft/vscode/wiki/Explain:-extension-causes-high-cpu-load

ms-vsliveshare.vsliveshare-unresponsive.cpuprofile.txt


test_CanHaveNestedTryBlocksInASingleFunction_ThrowOutside() failed!

Test test_CanHaveNestedTryBlocksInASingleFunction_ThrowOutside(), ' TEST_ASSERT_EQUAL(2, i) ' failed, i's value should be still zero.

I'm assuming by this very brief bug that you are seeing a failure in the self-tests for your toolchain and target?
To help you debug this, It would be good to get some additional information. What compiler are you using? Are you running your tests natively or on a simulator or a target? etc.
Given which test you're seeing fail, it's possible your compiler has decided it can optimize i by not setting it inside the loop there... or that it hasn't written it back to the actual location of i in memory but still has it in a scratch register. If you change i to volatile, I suspect this test will get back to passing.
Otherwise maybe your concern is that the test itself is wrong? If this is the case, I'd encourage you to read the documentation. C doesn't have full stack unwinding the way C++ does, and CException's concept isn't to create a system with full stack unwinding.  It doesn't guarantee it will reverse the effects of any operations that happened on local variables within the Try blocks.
If your question was something else, I'd appreciate a bit more detail. Thanks so much! :)


Demo phase

After training the model and all, I would like to ask when we enter a new image so that our model would recognize it.
Is it able to recognize any composition of image? I mean even a text the model has never seen (been trained on) before?
For example:
We trained the model only on something like this:
AAA
BBB
CCC
And in the demo phase, we give it an image that contains ABC in it, will it be able to recognize it, or it can only recognize something it saw before?

Hi, yeah, it can recognize ABC!

We can consider that if we need to give all the condition, then when len(alphabet) = 3000 and the txt_len = 10, we need 3000^10 sample. Of course it's impossible.

You can think it as splited, whitch menas AAA can be splited as A, A, A, so as B, C... as for the reason why we need more data, it's we need the net to learn the right feature of the alphabet and diff different alphabets.
Look at the encode function, you can see the labels are splited treated
def encode(self, text):
    length = []
    result = []
    for item in text:
        item = item.decode('utf-8','strict')
        length.append(len(item))
        for char in item:
            index = self.dict[char]
            result.append(index)

    text = result
    return (torch.IntTensor(text), torch.IntTensor(length))

I see, so that's the work of CNN and RNN combined together, right?

Yes, you are right!


Update readme to provide more context on compilation from source

Provide some prerequisite info on how to set up the toolchain for building the SDK
Fix quotes in "Hello World Test!" which was causing an issue in zsh in MacOS related to the ! being treated as a special character.
Switch the script that's passed to the build example to use the bundle.js instead of script.js, which isn't buildable since it's not in cjs format.

Issue caused by the ! (unterminated quote denoted by the dquote> prompt)
➜  js-pdk git:(main) ✗ extism call out.wasm count_vowels --wasi --input="Hello World Test!"
dquote>
➜  js-pdk git:(main) ✗ extism call out.wasm count_vowels --wasi --input='Hello World Test!'
{"count":4}% ```

Thanks! I'm using fish so I need to watch out for these bash syntax issues


Modify TF code for test

Terraform Plan against dev environment
Refreshing Terraform state in-memory prior to plan...
The refreshed state will be used to calculate this plan, but will not be
persisted to local or remote state storage.

module.demo-cicd.aws_s3_bucket.demo: Refreshing state... [id=concourse-demo-dev-s3-bucket]

------------------------------------------------------------------------

An execution plan has been generated and is shown below.
Resource actions are indicated with the following symbols:
-/+ destroy and then create replacement

Terraform will perform the following actions:

  # module.demo-cicd.aws_s3_bucket.demo must be replaced
-/+ resource "aws_s3_bucket" "demo" {
      + acceleration_status         = (known after apply)
        acl                         = "private"
      ~ arn                         = "arn:aws:s3:::concourse-demo-dev-s3-bucket" -> (known after apply)
      ~ bucket                      = "concourse-demo-dev-s3-bucket" -> "concourse-demo-dev-bucket" # forces replacement
      ~ bucket_domain_name          = "concourse-demo-dev-s3-bucket.s3.amazonaws.com" -> (known after apply)
      ~ bucket_regional_domain_name = "concourse-demo-dev-s3-bucket.s3.eu-west-1.amazonaws.com" -> (known after apply)
        force_destroy               = false
      ~ hosted_zone_id              = "Z1BKCTXD74EZPE" -> (known after apply)
      ~ id                          = "concourse-demo-dev-s3-bucket" -> (known after apply)
      ~ region                      = "eu-west-1" -> (known after apply)
      ~ request_payer               = "BucketOwner" -> (known after apply)
      - tags                        = {} -> null
      + website_domain              = (known after apply)
      + website_endpoint            = (known after apply)

      ~ server_side_encryption_configuration {
          ~ rule {
              ~ apply_server_side_encryption_by_default {
                    sse_algorithm = "AES256"
                }
            }
        }

      ~ versioning {
          ~ enabled    = false -> (known after apply)
          ~ mfa_delete = false -> (known after apply)
        }
    }

Plan: 1 to add, 0 to change, 1 to destroy.

------------------------------------------------------------------------

Note: You didn't specify an "-out" parameter to save this plan, so Terraform
can't guarantee that exactly these actions will be performed if
"terraform apply" is subsequently run.

Releasing state lock. This may take a few moments...

Terraform Plan against stage environment
Refreshing Terraform state in-memory prior to plan...
The refreshed state will be used to calculate this plan, but will not be
persisted to local or remote state storage.

module.demo-cicd.aws_s3_bucket.demo: Refreshing state... [id=concourse-demo-stage-s3-bucket]

------------------------------------------------------------------------

An execution plan has been generated and is shown below.
Resource actions are indicated with the following symbols:
-/+ destroy and then create replacement

Terraform will perform the following actions:

  # module.demo-cicd.aws_s3_bucket.demo must be replaced
-/+ resource "aws_s3_bucket" "demo" {
      + acceleration_status         = (known after apply)
        acl                         = "private"
      ~ arn                         = "arn:aws:s3:::concourse-demo-stage-s3-bucket" -> (known after apply)
      ~ bucket                      = "concourse-demo-stage-s3-bucket" -> "concourse-demo-stage-bucket" # forces replacement
      ~ bucket_domain_name          = "concourse-demo-stage-s3-bucket.s3.amazonaws.com" -> (known after apply)
      ~ bucket_regional_domain_name = "concourse-demo-stage-s3-bucket.s3.eu-west-1.amazonaws.com" -> (known after apply)
        force_destroy               = false
      ~ hosted_zone_id              = "Z1BKCTXD74EZPE" -> (known after apply)
      ~ id                          = "concourse-demo-stage-s3-bucket" -> (known after apply)
      ~ region                      = "eu-west-1" -> (known after apply)
      ~ request_payer               = "BucketOwner" -> (known after apply)
      - tags                        = {} -> null
      + website_domain              = (known after apply)
      + website_endpoint            = (known after apply)

      ~ server_side_encryption_configuration {
          ~ rule {
              ~ apply_server_side_encryption_by_default {
                    sse_algorithm = "AES256"
                }
            }
        }

      ~ versioning {
          ~ enabled    = false -> (known after apply)
          ~ mfa_delete = false -> (known after apply)
        }
    }

Plan: 1 to add, 0 to change, 1 to destroy.

------------------------------------------------------------------------

Note: You didn't specify an "-out" parameter to save this plan, so Terraform
can't guarantee that exactly these actions will be performed if
"terraform apply" is subsequently run.

Releasing state lock. This may take a few moments...

Terraform Plan against stage environment
Refreshing Terraform state in-memory prior to plan...
The refreshed state will be used to calculate this plan, but will not be
persisted to local or remote state storage.

module.demo-cicd.aws_s3_bucket.demo: Refreshing state... [id=concourse-demo-stage-s3-bucket]

------------------------------------------------------------------------

An execution plan has been generated and is shown below.
Resource actions are indicated with the following symbols:
-/+ destroy and then create replacement

Terraform will perform the following actions:

  # module.demo-cicd.aws_s3_bucket.demo must be replaced
-/+ resource "aws_s3_bucket" "demo" {
      + acceleration_status         = (known after apply)
        acl                         = "private"
      ~ arn                         = "arn:aws:s3:::concourse-demo-stage-s3-bucket" -> (known after apply)
      ~ bucket                      = "concourse-demo-stage-s3-bucket" -> "concourse-demo-stage-bucket" # forces replacement
      ~ bucket_domain_name          = "concourse-demo-stage-s3-bucket.s3.amazonaws.com" -> (known after apply)
      ~ bucket_regional_domain_name = "concourse-demo-stage-s3-bucket.s3.eu-west-1.amazonaws.com" -> (known after apply)
        force_destroy               = false
      ~ hosted_zone_id              = "Z1BKCTXD74EZPE" -> (known after apply)
      ~ id                          = "concourse-demo-stage-s3-bucket" -> (known after apply)
      ~ region                      = "eu-west-1" -> (known after apply)
      ~ request_payer               = "BucketOwner" -> (known after apply)
      - tags                        = {} -> null
      + website_domain              = (known after apply)
      + website_endpoint            = (known after apply)

      ~ server_side_encryption_configuration {
          ~ rule {
              ~ apply_server_side_encryption_by_default {
                    sse_algorithm = "AES256"
                }
            }
        }

      ~ versioning {
          ~ enabled    = false -> (known after apply)
          ~ mfa_delete = false -> (known after apply)
        }
    }

Plan: 1 to add, 0 to change, 1 to destroy.

------------------------------------------------------------------------

Note: You didn't specify an "-out" parameter to save this plan, so Terraform
can't guarantee that exactly these actions will be performed if
"terraform apply" is subsequently run.

Releasing state lock. This may take a few moments...

Terraform Plan against dev environment
Refreshing Terraform state in-memory prior to plan...
The refreshed state will be used to calculate this plan, but will not be
persisted to local or remote state storage.

module.demo-cicd.aws_s3_bucket.demo: Refreshing state... [id=concourse-demo-dev-s3-bucket]

------------------------------------------------------------------------

An execution plan has been generated and is shown below.
Resource actions are indicated with the following symbols:
-/+ destroy and then create replacement

Terraform will perform the following actions:

  # module.demo-cicd.aws_s3_bucket.demo must be replaced
-/+ resource "aws_s3_bucket" "demo" {
      + acceleration_status         = (known after apply)
        acl                         = "private"
      ~ arn                         = "arn:aws:s3:::concourse-demo-dev-s3-bucket" -> (known after apply)
      ~ bucket                      = "concourse-demo-dev-s3-bucket" -> "concourse-demo-dev-bucket" # forces replacement
      ~ bucket_domain_name          = "concourse-demo-dev-s3-bucket.s3.amazonaws.com" -> (known after apply)
      ~ bucket_regional_domain_name = "concourse-demo-dev-s3-bucket.s3.eu-west-1.amazonaws.com" -> (known after apply)
        force_destroy               = false
      ~ hosted_zone_id              = "Z1BKCTXD74EZPE" -> (known after apply)
      ~ id                          = "concourse-demo-dev-s3-bucket" -> (known after apply)
      ~ region                      = "eu-west-1" -> (known after apply)
      ~ request_payer               = "BucketOwner" -> (known after apply)
      - tags                        = {} -> null
      + website_domain              = (known after apply)
      + website_endpoint            = (known after apply)

      ~ server_side_encryption_configuration {
          ~ rule {
              ~ apply_server_side_encryption_by_default {
                    sse_algorithm = "AES256"
                }
            }
        }

      ~ versioning {
          ~ enabled    = false -> (known after apply)
          ~ mfa_delete = false -> (known after apply)
        }
    }

Plan: 1 to add, 0 to change, 1 to destroy.

------------------------------------------------------------------------

Note: You didn't specify an "-out" parameter to save this plan, so Terraform
can't guarantee that exactly these actions will be performed if
"terraform apply" is subsequently run.


Frontend service fails with 503 errors after running for about a week

Bug Report
Which version of the demo you are using?
Commit fc01d8f
Symptom
After being up for about a week, frontend container returns errors to after running for about a week.
Specifically, http get requests to frontend return a 503 error.  The logs of the container show exceptions with a stack trace.
I can delete the frontend pod, causing it to be respawned.  The new container runs for about 8 hours before showing the same problem.  I have been running for about 2 weeks now, and it stays up for just about a workday.  Every morning I have to restart the frontend pod.
I have tried restarting other pods instead, including adservice and featureflag, but neither of those will fix it -- I have to restart frontend.
Stack traces are not always the same - it fails in different places on different days.  However, on any single day the stack traces are always the same.  Attached are 3 files for different stacktraces.
What is the expected behavior?
Frontend continues to work after a week.
What is the actual behavior?
Frontend works for about a week, then fails to respond to requests and logs show .js exceptions with stack trace.  Restarting the pod fixes it for about 8 hours, then it fails again.
Reproduce
Run the demo for a week straight on EKS (environment details below).  After about a week, frontend starts returning 503 errors to requests.
Additional Context
Environment:
Amazon EKS Running Kubernetes 1.28.  Node AMI is 1.28.1-20230919.
4 Nodes in Cluster:
2 of m5.large (2vCPU, 8GB memory, x86)
2 of t2.xlarge (4 vCPU, 16GB memory, x86)
I built the images myself, with no code changes.  Images are hosted on ghcr.io.
Stack trace Files
resource_exhausted_error_trace.txt
error_trace_recommendations_failure.txt
error_trace_ff_failure.txt
Helm deployment values file:
otel-demo-astronomyshop.yaml.txt
Add any other context about the problem here.

I've got the same issue, 503 error on frontend service, after that the demo has been running for 8 hours

I got the same issue. But I think I might have found the root-cause. It looks like the Productcatalogservice has a memory leak. After the second crash (in this example 20:40) the frontend starts to hang and send 503 responses.
See this screenshots:

This is a known issue with 1.6. It is resolved in the main branch and should be fixed with the next release.

This issue should be resolved with the 1.7.x release. If you are still having problems please open a new issue


how to run?

run in your heart.

That is totally easy!
Just open:
Then type this command:
Hit enter and it's running
Was a pleasure to help you


[HDBDD] Engine Support for processing annotation data

Each Symbol.class holds a collection of annotations but currently we do nothing with the provided data.
The current processors and xsk data model should be enhanced to include the annotation data and make the expected changes in the database.
For example the @Catalog annotation is parsed and the table should be created base on the @Catalog.tableType - #COLUMN or #ROW
The @Catalog.index[] should be processed and the already existing XSKDataStructureHDBTableConstraintsModel.class should be used to create new indices. The transformation between the CatalogObjectAnnotation and the XSKDataStructureHDBTableConstraintsModel could be done in HdbddTransformer.class
Cds annotation documentation: https://help.sap.com/viewer/09b6623836854766b682356393c6c416/2.0.03/en-US/8217aac86d9748d8b034797ecc8065b6.html#

Should be able to reuse idea from https://github.com/SAP/xsk/pull/819

related to

#1198

https://help.sap.com/viewer/9bf0d965efc74c51b0fbf177d3f96e5b/1.0.12/en-US/f7d6e1733c63488d9d568af2303251aa.html
Add support for parsing the following annotations:

EnterpriseSearch
EnterpriseSearchHana
Hierarchy

related to

#1280

to be tracked in separate issues


Added my book Bootstrapping Microservices to the list.

I'll update it to the full name, so it's clearer.


Synapse COPY INTO data type mapping

Regarding the COPY INTO option documented on https://docs.microsoft.com/en-us/sql/t-sql/statements/copy-into-transact-sql?view=azure-sqldw-latest
The statement expects a table with a certain schema so that the schema matches the files in the storage account. You need to create your table with that specific schema or you get unclear errors (or no errors at all, which is also not good because your table remains empty for an unknown reason...)
Please document for the supported file formats which data types map to which data types in Synapse SQL.

@sdebruyn The documentation for SQL server uses this github repo.
You can provide feedback by raising an issue on this repo.

Reported as https://github.com/MicrosoftDocs/sql-docs/issues/7377


In openssl-alpha17 release, symlinks of man/man* are introduced which are giving packaging issues

Issue not observed with openssl-3.0.0-alpha16.
While building openssl-3.0.0-alpha17, packaging issues are observed for docs, man/man1, man/man3, man/man7 etc as there are symbolic links introduced.
errors:
.
.
.
Symlink points to BuildRoot: /usr/share/man/man1/asn1parse.1ssl.gz -> /usr/src/photon/BUILDROOT/openssl-3.0.0-alpha17.1.ph4.x86_64/usr/share/man/man1/openssl-cmds.1ssl.gz
Symlink points to BuildRoot: /usr/share/man/man1/c_rehash.1ssl.gz -> /usr/src/photon/BUILDROOT/openssl-3.0.0-alpha17.1.ph4.x86_64/usr/share/man/man1/openssl-rehash.1ssl.gz
Symlink points to BuildRoot: /usr/share/man/man1/ca.1ssl.gz -> /usr/src/photon/BUILDROOT/openssl-3.0.0-alpha17.1.ph4.x86_64/usr/share/man/man1/openssl-cmds.1ssl.gz
Symlink points to BuildRoot: /usr/share/man/man1/ciphers.1ssl.gz -> /usr/src/photon/BUILDROOT/openssl-3.0.0-alpha17.1.ph4.x86_64/usr/share/man/man1/openssl-cmds.1ssl.gz
Symlink points to BuildRoot: /usr/share/man/man1/cms.1ssl.gz -> /usr/src/photon/BUILDROOT/openssl-3.0.0-alpha17.1.ph4.x86_64/usr/share/man/man1/openssl-cmds.1ssl.gz
Symlink points to BuildRoot: /usr/share/man/man1/crl.1ssl.gz -> /usr/src/photon/BUILDROOT/openssl-3.0.0-alpha17.1.ph4.x86_64/usr/share/man/man1/openssl-cmds.1ssl.gz
Symlink points to BuildRoot: /usr/share/man/man1/crl2pkcs7.1ssl.gz -> /usr/src/photon/BUILDROOT/openssl-3.0.0-alpha17.1.ph4.x86_64/usr/share/man/man1/openssl-cmds.1ssl.gz
Symlink points to BuildRoot: /usr/share/man/man1/dgst.1ssl.gz -> /usr/src/photon/BUILDROOT/openssl-3.0.0-alpha17.1.ph4.x86_64/usr/share/man/man1/openssl-cmds.1ssl.gz
Symlink points to BuildRoot: /usr/share/man/man1/dhparam.1ssl.gz -> /usr/src/photon/BUILDROOT/openssl-3.0.0-alpha17.1.ph4.x86_64/usr/share/man/man1/openssl-cmds.1ssl.gz
Symlink points to BuildRoot: /usr/share/man/man1/dsa.1ssl.gz -> /usr/src/photon/BUILDROOT/openssl-3.0.0-alpha17.1.ph4.x86_64/usr/share/man/man1/openssl-cmds.1ssl.gz

.
..
..
Request to fix this by removing the symlinks.

Does #15437 fix the problem?
Removing the symbolic links isn't going to happen.  They do need to be relative.

Something to be noted is that OpenSSL 1.1.1 symlinks manpages...  but they are relative symlinks, so unless 1.1.1 has been problematic because of this (I would be surprised, since that was never reported), #15437 should indeed make the problem go away.

pulled the patch from #15437 and it seems to have worked. Thanks @paulidale & @levitte


Problem with v-for + v-if in firefox

Vue.js version
1.0.20
Reproduction Link
https://jsfiddle.net/2so6kua0/2/
Steps to reproduce
Start example
What is Expected?
We should get
val == 1
val == 2
val == 3
val == 4
val == 5
val == 6

What is actually happening?
In firefox we get nothing, but if we will change

val == {{val}}

to
<template v-for="val in arr">

val == {{val}}

</template>

all work correctly.

Actually the second snippet doesn't work for me either (Firefox 44.0.2, Yosemite).

Interesting. To make it work you need to put v-if before v-for in Firefox, but to work in Chrome it's the other way around. Might have something to do with priorities? https://github.com/vuejs/vue/issues/2573

Yes, in 1.0.20 v-if and v-for is dependent on appearance order due to conflicting priority. This should have been fixed by https://github.com/vuejs/vue/commit/89b4ee9cd0ec2305199e9ce3dd199e5bf7653ac4

@yyx990803 I'm not 100% sure this is problem here. Do we maybe have mater/dev branch build anywhere so I can test if it works same in Chrome and Firefox?

@zigomir in Firefox element.attributes enumeration order is in reverse, therefore in the fiddle v-if is compiled first in root scope.

Ah, great, thanks for explanation.
On Wed, Mar 30, 2016 at 9:36 AM Evan You<EMAIL_ADDRESS>wrote:

@zigomir https://github.com/zigomir in Firefox element.attributes
enumeration order is in reverse, therefore in the fiddle v-if is compiled
first in root scope.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/vuejs/vue/issues/2592#issuecomment-203516136

Is this fix in a distribution? How can I get it into my project?

@jeffmarshall you will have to do the dev build: http://vuejs.org/guide/installation.html#Dev-Build

@yyx990803 ok, thanks


Update survey image link

Codecov Report

Merging #880 into dev will increase coverage by 0.03%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev     #880      +/-   ##
============================================
+ Coverage     48.98%   49.02%   +0.03%
- Complexity     2821     2822       +1
============================================
  Files           116      116
  Lines         27877    27877
  Branches       4650     4650
============================================
+ Hits          13655    13666      +11
+ Misses        12104    12087      -17
- Partials       2118     2124       +6

Flag
Coverage Δ
Complexity Δ

#JDBC42
48.58% <ø> (+0.39%)
2784 <ø> (+34)
:arrow_up:

#JDBC43
48.63% <ø> (-0.29%)
2795 <ø> (-21)

Impacted Files
Coverage Δ
Complexity Δ

...c/main/java/com/microsoft/sqlserver/jdbc/Util.java
61.63% <0%> (-0.44%)
92% <0%> (-1%)

...in/java/com/microsoft/sqlserver/jdbc/IOBuffer.java
58.21% <0%> (-0.04%)
0% <0%> (ø)

...om/microsoft/sqlserver/jdbc/SQLServerBulkCopy.java
52.85% <0%> (+0.12%)
264% <0%> (+1%)
:arrow_up:

...ncurrentlinkedhashmap/ConcurrentLinkedHashMap.java
39% <0%> (+0.21%)
43% <0%> (ø)
:arrow_down:

...rc/main/java/com/microsoft/sqlserver/jdbc/DDC.java
45.95% <0%> (+0.21%)
107% <0%> (-1%)
:arrow_down:

...m/microsoft/sqlserver/jdbc/SQLServerResultSet.java
32.77% <0%> (+0.3%)
256% <0%> (+3%)
:arrow_up:

...om/microsoft/sqlserver/jdbc/ReaderInputStream.java
47.25% <0%> (+2.19%)
15% <0%> (ø)
:arrow_down:

Continue to review full report at Codecov.

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


Defer loading plots and the block data

The code should all still be there for this, just need to parcel up the block loading into multiple requests and perhaps extend the API routes to make this easy (with e.g., response field URL query parameters).

Just going through old "high priority" issues, I guess this one was done a long time ago...


Niche Spell book display issue

When picking the urban druid archetype the spellbook UI logs a lvl 0 Cleric spellbook that does nothing, causes no issues, and disappears on reload.
I am putting this here to remind myself to look at it when more awake as with it causing no issues I will forget about it otherwise.

Fixed


Fix/introduce functional tests

It looks like the functional tests were never brought across or reintroduced after the split from molecule core
This pull request drags the functional tests across from molecule core
Specifically copying test_command.py and adapting it for ec2 use, significant changes:
* Replace with_scenario helper
* Replace all test parameters to use our scenarios
* Consolidated the parameters to a custom decorator
* Dropped dependency tests as they aren't driver related
test_command_init_scenario() is xfail due to https://github.com/ansible-community/molecule/issues/2797
test_ec2.py was dropped as the test it had is now covered by test_command_init_role()
Updates to setup.cfg, tox.ini and pytest.ini support the new tests and were mostly lifted piecemeal from the core molecule project.
Tests have been run through tox against python 3.8 and all pass.
The sequential test suite takes roughly two hours to execute.
Zuul changes have also been made to mimic the core molecule however they are untested.
I expect the maintainers will need to adjust zuul or the zuul configuration files to run.
There may also be issues with parallel execution and clashing AWS resources, I have another PR coming which will help address that.
There are also changes to the cookiecutter files to support the test passing:
* Allow the subnet to be omitted, this allows a generic template to be executed and tests to not be locked to an AWS account
* Add custom molecule.yml file with example ubuntu image
* Fixed ansible lint [503] errors, when: changed --> handlers
Futher details are provided in the relevant commit messages

Hi @lod
would you find some time to fix the issues within this PR?

Closing as not updated in a long time.

Hi, trying to resurrect this work.
Sorry, I didn't follow up at the time.
It obviously needs to be updated to reflect the considerable work in create.yml
Are there any other specific issues with this that need to be addressed @tbugfinder?


Avoid plucking single-use ids

I think you're good to merge this if you rebase it to master again! There have been more changes to the base since, so I'd recommend merging it as soon as tests pass after the rebase.


automate releases

Now that otel-cli has a good functional testing story, it's time to finally automate the last bits of the release process to get my computer out of the loop. I believe the remaining steps are something like this:

create a new token with permissions to write to packages and releases
set up a new GH Action that fires on new tags

build the release and run a go test -v which will run functional tests to validate the tag

current tests already do this

run goreleaser and it will do the rest

this must depend on a successful functional test

Once this is complete, a release can be made by tagging a sha and pushing tags, so package & release write access can be restricted to actions only.

Also the release build should set the linker flags with the tag version and shas. See: https://github.com/equinix-labs/otel-cli/blob/92a6f0acdf4affe4f6a7f1127f00cb44c57bc2f7/.github/workflows/ci.yml#L21

goreleaser should be setting the three linker flags automatically for release builds but for clarity there are three: main.version, main.commit, and main.date. These are printed by the otel-cli version subcommand and are invaluable for debugging.
The variables are defined in main.go: https://github.com/equinix-labs/otel-cli/blob/main/main.go

Instructions are in the Releases section of the README.md. A GitHub action could be used to trigger the tag and release.
In other Equinix Labs projects, Conventional Commits and Semantic Release have been introduced to simplify this process and ensure that the correct semver revision is applied based on the commit messages.

Releases follow semantic versioning since v0.0.1 and I've been writing a CHANGELOG.md for the last few releases. There have been no other maintainers to coordinate with so I feel the conventions are not as valuable as they are in a larger project context. For this project, releasing on tag push makes the most sense to me.
I now work on this in my personal time for free, and would prefer to not have the overhead. My feeling is that choices about how to maintain as a team should be made by that team when/if it materializes. For now, keeping the barrier to submitting & reviewing PRs low is paramount.

An on-push GitHub Actions workflow should have write access to packages and releases, no need to mint a personal token. You may want to explicitly specify the permissions though. https://docs.github.com/en/actions/writing-workflows/choosing-what-your-workflow-does/controlling-permissions-for-github_token


Make calls to getenv(3) safer when setuid.

Change all calls to getenv() inside libcrypto to use a new wrapper function that use secure_getenv() if available and an OPENSSL_issetugid() then getenv() if not.

I left the getenv(3) calls alone in the apps and tests.  Neither should be running setuid or similar.

I'd say bad idea in the suggested form. For two reasons. 1. Does it actually have to be non-discriminating?  Well, it's not like I have ready answer to the question, but at least I think that it should be posed, and I don't see that it was. I mean it pretty much looks like "replace everything just because". 2. What about other systems? Why does it have to be not only Linux-specific, but even glibc-version-specific? I mean if we consider this a security issue, then there should be fall-back code that provide corresponding functionality on other systems. In other words even from this angle it looks pretty much like "just because"...

It isn't Linux specific although it is Unix specific.  With glibc the safe secure_getenv(3) call is used, otherwise there is a check for privileged execution before allowing getenv(3) which wasn't present previously.
Unprivileged execution is unchanged -- this change only impacts programs that are setuid or have otherwise raised capabilities.  Applications that are run either as a user or as root have unchanged access to their environment.
I'm having difficulty thinking of a situation where a setuid program would rely on unfettered access the environment unless I'm trying to hack the system.

It isn't Linux specific although it is Unix specific.

I apparently missed extra if in non-glibc case. I suppose I was tipped off by subject, which is actually misleading. As this is not about using secure_getenv when available, but changing getenv's behaviour completely. So what's with that? I mean totally misleading subject. [BTW, is this a feature? This is reference to assertion made elsewhere that current definition of "feature" is too imprecise to be useful.] But back on topic...

I'm having difficulty thinking of a situation where a setuid program would rely on unfettered access the environment unless I'm trying to hack the system.

Yet it's not about reading variable per se but what effect do they have, is it? Consider locale for example, would you deny setuid program to read LANG? One can ask if all variables interpreted by OpenSSL lib are equally sensitive, but you just can't make such general assertion about all environment variables, and base merge request [with misleading subject] on it. I probably should clarify that I'm not saying that I know the answer to the implied question, only that it's not given that such generalization is warranted.

Yeah, bad subject choice.  For that I apologise.  I'm way too good at picking bad names and titles.
I too am not sure if all of the getenv(3) calls need to be changed or if some are intrinsically safe.  Using your example, I can't think how an attacker could change LANG to get an exploit.  However, I've repeatedly seen similarly benign things turn out to be exploitable.  I erred heavily on the side of caution here.  I'll go back and revisit each of the calls and try to make a more discerning determination.

What at least needs to use this is everything related to filenames and paths.

I'd gone back to getenv(3) for the override processor flags.
I'm unsure about the LEGACY_GOST_PKCS12 environment variable in p12_mutl.c  Safe or not?
I'm more sure about the _CONF_get_string which reads the environment in two places -- we don't know what these would be used for.

Overall I'd argue against using safe_getenv. Less Linux dependencies is better. Because our goal is wider than Linux. Trouble is that Linux is the only one that gets real and thorough exercise. And in such case it would be only appropriate that non-Linux-specific code paths are exercised on Linux. As for ossl_safe_getenv, I'm a bit torn. But I'm inclined to say that if there is need to secure more environment variable references, explicit OPENSSL_issetugid tests around the problematic references might be more appropriate. Because ossl_safe_getenv is "tri-state", which might be wrong in some circumstances. Which means that there would be calls to ossl_safe_getenv, but there also would be calls to OPENSSL_issetugid. In which case a unified approach would be more readable, because reader and developer would be excused from struggling with question which one to use when.

Andy Polyakov wrote:

dot-asm commented on this pull request.

@@ -82,7 +83,7 @@ char *_CONF_get_string(const CONF *conf, const char *section,
if (v != NULL)
return v->value;
if (strcmp(section, "ENV") == 0) {

           p = getenv(name);

           p = ossl_safe_getenv(name);

As already said, at this point config file is supposed to be "trusted". I.e. in setuid case it won't be one appointed by OPENSSL_CONF environment variable. Would it be appropriate to suppress this? Or in other words wouldn't it be appropriate to say that if system owner says "read environment variable" is trusted config, then it should be done unconditionally?

I cannot see how is such case someone would could use environment
variables. If configuration (even trusted) has at least one environment
it has to be ignored at all.
Otherwise code will produce unexpected failures.
As result in setuid programs we cannot have "trusted" config - only
default with documentation that environment variables are risky.
Abnormal failures is reason to vote against whole idea for "safe" use of
environment variables.

The problem with OPENSSL_issetugid is that there can be other ways how to elevate privileges than just setuid/setgid - particularly there are the capabilities which can be elevated on file with similar mechanism as setuid/setgid. Glibc secure_getenv handles that.

An attempt is made to use capabilities in OPENSSL_issetugid so it is doing more than checking the real and effective user and group IDs (on Linux).  Regardless, I'd prefer secure_getenv since our checking will likely become dated over time and is Linux specific whereas glibc might keep up better.

Travis failures are not relevant.

Does this need more discussion or can it be merged now that 1.1.1 is live?

Merged to master, 1.1.1 and 1.1.0.  1.0.2 will be a separate PR.
Thanks.

#7300 is the back port to 1.0.2-stable.
#7299 went weird when I clicked something wrongly.


Remove unneeded pipeline permissions.

Fixes #2989
Fixes #2990
This still needs some more testing with to make sure it doesn't break anything. Particularly when deploying on cloud providers.

Any update on this?

This just needs to be tested on some more cloud providers before we're confident it doesn't break things. We'll get it merged in before 1.8 ships though.

Ran a simple test off of this branch in GKE (the split benchmark with a small amount of data) and it seemed to run fine

I'm going to merge this branch, as at least basic tests seem to show that it works


mockito-kotlin lib ties you to Kotlin 1.0.3

Just FYI...if you want to use a different version of Kotlin for your main app, this will conflict.
I'd rather not use mockito-kotlin because of this...

Good point. When I have time I'll remove this dependency.

Thanks! 👍


Changing a user's password doesn't work

Previously with Stormpath i changed a user's password(after validating the current password) like this:
req.user.password = req.body.password;
req.user.save(function (err,res) {
    ...
});

This doesn't seem to work with Okta. I don't see any error in the response, its just the user object as usual.

Hi @passatgt , unfortunately that functionality has not been patched.  There is an endpoint in the Okta API to achieve this:
https://developer.okta.com/docs/api/resources/users.html#change-password
You may be able to make this call through the new Okta Node SDK (alpha preview):
https://developer.okta.com/okta-sdk-nodejs/jsdocs/GeneratedApiClient.html#changePassword__anchor
If you have troubles please let us know, and fallback to manual HTTP calls for now.
Hope this helps!

Thanks, i'll do a simple http call for now(i already do that to check for the validation of the current password), but would be nice to fix this so its even easier to use. I guess on the long run, this repo will be renamed / moved to okta?

Points for the Modest Mouse reference in the Okta Change-Password documents

Express-Stormpath change password example Before Okta
app.patch('/passwords', stormpath.authenticationRequired, function(req, res) {
      req.user.password = req.get('lf-password');
      req.user.save(function (err) {
        if (err) {
          return res.status(400).json({success: false, error: err});
        } else {
		  return res.status(200).json({success: true, message: 'Account password changed'});
		}
      });
    });

Express-Stormpath change password example with Okta
    app.patch('/passwords', stormpath.authenticationRequired, function(req, res) {

      // var request = require('request'); imported above

      var oldPass = req.get('lf-old_password');
      var newPass = req.get('lf-new_password');
      var userOktaID = req.user.id;

      var requestOptions = {
        method: "POST",
        url: oktaOrgURL + "/api/v1/users/" + userOktaID + "/credentials/change_password",
        headers: {
          "Accept": "application/json",
          "Content-Type": "application/json",
          "Authorization": "SSWS " + oktaAPIToken
        },
        json: true,
        body: {
          oldPassword: oldPass,
          newPassword: newPass
        }
      };

      request(requestOptions, function(error, response, body){
        if (error){  // If sending the request failed
          console.log(error);
          res.status(500).json({success:false, error: error});
        } else {
          if (!body.errorCode){ // If resetting the password failed
            return res.status(200).json({success: true, message: 'Account password changed'});
          } else {
            return res.status(400).json({success: false, error: body.errorSummary});
          }
        }
      });
    });

@isaacdre theres a simpler way to do this:
var changePasswordCredentials = {
   oldPassword: { value: req.body.old_password },
   newPassword: { value: req.body.password }
 };

 oktaClient.getUser(req.user.id).then(user => {

   user.changePassword(changePasswordCredentials).then(user => {
     return res.status(200).send({status: 'success', code: 'password_changed', message: 'Password changed successfully.'});

   }).catch((error) => {
     return res.status(400).send({status: 'fail', code: 'wrong_password', message: 'Wrong password.'});
   });

 }).catch((error) => {
   return res.status(400).send({status: 'fail', code: 'wrong_password', message: 'Wrong password.'});
 });


First batch of metrics to support from Gitlab

@kevin-kline Please see below for a list of metrics we want to support for our MVP for Gitlab:

Number of contributors
Number of commits
Number of reviewers
Merge review time (defined as from the first comment to merge)
Number of merge requests
MR review pass ratio (defined as the percentage of merged MRs vs all MRs)
Removed lines of code
Added lines of code
Absolute lines (defined as the diff between two snapshot of the codebase)
Accumulated lines (defined as the sum of added lines of code and removed lines of code during a time window)

Please Use this table。https://airtable.com/invite/l?inviteId=invnlKL0nPCiDE3bF&inviteToken=c578ba0043ee5da8ecfa888b7f14b82ded5d3905adc21bbb56e1841be77d55f7&utm_source=email

@kevin-kline The metric list for Gitlab has been updated, please see the first post in this thread.


By Default All Nodes should be collapsed to enhance Performance

I have been trying to get all compound nodes in collapsed form by default.
For achieving this I changed the cy.expandCollapse() function, by changing the following on line 1333 in cytoscape-expand-collapse.js -
 cy.nodes(':parent').data('expanded-collapsed', 'collapsed');
When I did the above I got an error in cueUtilities.js line 72 [Uncaught TypeError: Cannot read property 'options' of undefined] -

The strange part whoever is that when I tried to debug this using console.log, the changes were not being reflected, maybe because all the files are imported using dereq rather than normal require(). I still couldn't this part out. PS - Yes, I have cleared my cache, cookies and history. Also tried the same in Incognito mode as well, still the changes done weren't reflected.
Kindly let me know how can I achieve collapsed nodes by default on initialization itself.
My agenda behind this is to observe whether this improves my performance with large number of nodes(>3000) and edges(>15000). If you already done any similar experiment, kindly share your observations here.

@VarunWachaspati In that line "cy.nodes(':parent').data('expanded-collapsed', 'collapsed');" it is specified that all parent nodes are expanded by default. This line is not expanding the nodes and if you changed 'expanded' as 'collapsed' it will not collapse them ( Actually we will remove this line in the future please see #16 ). If you change a source file such as 'cueUtilities.js' you need to run one of the 'gulp build' or 'gulp' (build is the default command in our gulp file ) commands to reflect the changes to 'cytoscape-expand-collapse.js' file.
To collapse all nodes you can use the functions in our API. If you want to collapse all nodes bottom up recursively you can use one of the followings (I think that you do not have to change the extension codes for this purpose):

cy.nodes("parent").collapse()

cy.nodes("parent").orphans().collapseRecursively()

If you want to collapse just the top most level parent nodes you can use the following:

cy.nodes("parent").orphans().collapse()

@metincansiper Thanks for the swift reply. I am using cy.collapseAll(); as of now, but this still doesn't improve my performance of rendering for large graphs.
Closing the issue

@VarunWachaspati I have some questions:

What is the size (# nodes/edges) of your graph after all compounds collapsed?
When you say you're still not satisfied with the performance, are you referring to
o time it takes to perform collapse all, or
o time it takes to interact with the graph (where all compounds are collapsed), or
o time it takes to layout your graph (where all compounds are collapsed)?

@VarunWachaspati If it is something accepable for your case did you try to collapse just the top most level parent nodes by using 'cy.nodes(":parent").orphans().collapse()'? I think that this may be better in terms of performance.


Enhancement: Stricter Typing for tag Field in BrillPOSTaggedWord Interface

Description:
I would like to request an enhancement to the Natural NPM library regarding the definition of the tag field in the BrillPOSTaggedWord interface. Currently, the tag field is of type string, which is quite generic and allows any string value. However, it would be beneficial to have a stricter type definition that specifically accounts for all the possible Part-of-Speech (POS) tags.
Proposed Solution:
To address this, I suggest introducing a more specific type for the tag field by defining an explicit union type that includes all the valid POS tags. This would provide improved type safety and make it easier for developers to work with the library.
I have already created a sample type called POSTags with all the relevant POS tags defined as string literals. Here is an example:
/**
 * Represents Part-of-Speech (POS) tags.
* @typedef {'NN'   // Noun, singular or mass
 *           | 'VB'   // Verb, base form
 *           | 'JJ'   // Adjective
 *           | 'RB'   // Adverb
 *           | 'DT'   // Determiner
 *           | 'IN'   // Preposition or subordinating conjunction
 * ...
 */
type POSTags =
  | 'NN'
  | 'VB'
  | 'JJ'
  | 'RB'
  // ... (list continues with all valid POS tags)
  | 'BES'
  | 'HVS';

interface BrillPOSTaggedWord {
  token: string;
  tag: POSTags;
}

This enhancement would ensure that the tag field only accepts values that match the defined POS tags, reducing the risk of errors and improving the developer experience.
Impact and Benefits:

Improved Type Safety: With stricter typing, developers can catch potential errors related to incorrect or misspelled POS tags at compile-time rather than at runtime.
Enhanced Developer Experience: Clear and specific types provide better code understanding, enabling developers to work more confidently and efficiently with the Natural library.
Facilitated Integration: A well-defined type for the tag field simplifies the integration of Natural into TypeScript projects, as the expected types become more explicit.

Additional Context :
I believe that enforcing a more specific type for the tag field would be a valuable enhancement to the Natural library, especially for projects that utilize TypeScript and require stronger typing. It would align with the library's goal of providing reliable and efficient NLP functionalities.
I am willing to contribute to this enhancement by providing a pull request with the proposed changes, if it aligns with the project maintainers' vision.
Thank you for considering this enhancement request. I look forward to your feedback and discussing further improvements!

For starters, a great idea to offer more strict typing for tags, However, the set of tags is not static. It varies with the set of transformation rules that is loaded.


MY WEB UNDER ATTACK PLEASE RESPOND THIS ASAP, HOW TO USE PORT 80

Bismillah,
I'm using cloudflare for handle ddos attack and security
why I got this error when use port 80
streamsocket  unable to connect http://<IP_ADDRESS>:80 Permision denied
I'm sorry not compelete error log
Please i need help now
why i need to run on port 80 or 443 because, i've to setting port in cloudfare, which is not easy

I don't know where this log is from. It seems that there is no permission, you can use the root account to try.

Hi walko. Can I contact you regarding a security in webman ?


ci: setup codecov and add coverage badge

Fixes #25. Added codecov coverage as an GitHub action and add a badge in readme.md.
Need to update the CODECOV_TOKEN in the repo secrets after enabling it in codecov.

The uploader probably errored as go test did not generate the coverage files. Can use -coverprofile=coverage.txt -covermode=atomic flags.


Increase Hystrix timeout for KMS encrypt

KMS encrypt calls trigger the following Hystrix 500 error more than 10 times/day:
com.netflix.hystrix.exception.HystrixRuntimeException: KmsEncrypt timed-out and no fallback available.
Increasing the Hystrix KMS encrypt timeout should solve this problem.

Coverage remained the same at 53.053% when pulling d3d4e899b4b382829d10c6f0f13e7abeee065537 on bugfix/increase_kms_encrypt_timeout into 6b879a09db06458ad82da708148222646fe63b67 on master.


Create 205-Isomorphic-Strings.cpp

Create the C++ solution as the same solution in the YouTube video "https://www.youtube.com/watch?v=7yF-U1hLEqQ"

File(s) Modified: 205-Isomorphic-Strings.cpp
Language(s) Used: C++
Submission URL: _https://leetcode.com/problems/isomorphic-strings/submissions/871903786/_

Please provide the appropriate submission URL.

Updated the correct Submission URL.


Bevy 0.10.0 Frustum Culling is not working

Bevy version
0.10.0
[Optional] Relevant system information
Windows 11 (Desktop)
`AdapterInfo { name: "NVIDIA GeForce GTX 1080", vendor: 4318, device: 7040, device_type: DiscreteGpu, driver: "NVIDIA", driver_info: "516.40", backend: Vulkan }`

Fedora (Laptop)
`AdapterInfo { name: "AMD Radeon Vega 8 Graphics (RADV RAVEN)", vendor: 4098, device: 5597, device_type: IntegratedGpu, driver: "radv", driver_info: "Mesa 22.3.7", backend: Vulkan }`

What you did
Minimal Code Example:  https://github.com/NotoriousENG/bevy_cull_bug
What went wrong
I was expecting objects behind the far plane to be culled base off of the bevy 0.6.0 changelog https://bevyengine.org/news/bevy-0-6/
I'm wondering if it may be due to a frustum struct being empty? Not quite sure if it should be created as part of building a Camera3dBundle?
No objects are being culled, and I can still see beyond where I would expect the view to end.
Originally found it while working here:

https://github.com/NotoriousENG/goon_game

And posted to the rendering discussion on Discord:

https://discord.com/channels/691052431525675048/866787577687310356/1088250126448996432

Additional information
Please build and run the minimal code example to see my issue https://github.com/NotoriousENG/bevy_cull_bug
Using
const FAR: f32 = 10.00;

for a camera
let cam_transform = Transform::from_xyz(0.0, 5.0, -10.0).looking_at(Vec3::ZERO, Vec3::Y);
// camera
commands.spawn(Camera3dBundle {
    projection: PerspectiveProjection {
        far: FAR,
        ..default()
    }
    .into(),
    transform: cam_transform.clone(),
    ..default()
});

and cubes spawned such that
// cubes
for i in 0..20 {
    let z = i as f32 + 0.5;
    if z > cutoff_z {
        // these should be culled by the frustum culling?
        println!("cube {} is behind the far cutoff {} at {}", i, cutoff_z, z);
    }
    commands.spawn(PbrBundle {
        mesh: meshes.add(Mesh::from(shape::Cube { size: 0.5 })),
        material: materials.add(Color::hsl(i as f32 / 20.0 * 360.0, 1.0, 0.5).into()),
        transform: Transform::from_xyz(0.0, 0.0, z),
        ..default()
    });
}

All Cubes are behind where the far plane should be
cube 0 is behind the far cutoff -1.055727 at 0.5
cube 1 is behind the far cutoff -1.055727 at 1.5
cube 2 is behind the far cutoff -1.055727 at 2.5
cube 3 is behind the far cutoff -1.055727 at 3.5
cube 4 is behind the far cutoff -1.055727 at 4.5
cube 5 is behind the far cutoff -1.055727 at 5.5
cube 6 is behind the far cutoff -1.055727 at 6.5
cube 7 is behind the far cutoff -1.055727 at 7.5
cube 8 is behind the far cutoff -1.055727 at 8.5
cube 9 is behind the far cutoff -1.055727 at 9.5
cube 10 is behind the far cutoff -1.055727 at 10.5
cube 11 is behind the far cutoff -1.055727 at 11.5
cube 12 is behind the far cutoff -1.055727 at 12.5
cube 13 is behind the far cutoff -1.055727 at 13.5
cube 14 is behind the far cutoff -1.055727 at 14.5
cube 15 is behind the far cutoff -1.055727 at 15.5
cube 16 is behind the far cutoff -1.055727 at 16.5
cube 17 is behind the far cutoff -1.055727 at 17.5
cube 18 is behind the far cutoff -1.055727 at 18.5
cube 19 is behind the far cutoff -1.055727 at 19.5

Actual:

Expected:

This is intentional. Our perspective projection has an infinite far plane and apparently games no longer do far plane culling, instead you manage what is loaded/visible in your scene separately. Because the projection has no far plane, you can see as far as you have things loaded.
However, when identifying which meshes are affected by individual lights, entities are culled against the far planes of the light projections.

We still have frustum culling, we just don’t cull against any far plane for the camera view frustum. We still cull against the near, left, right, top, and bottom planes. Over time we will add occlusion culling so if objects block other objects, those blocked objects are culled too.
For lights the culling is for shadow mapping and it’s more complicated but the short version is that we try to cull meshes that don’t affect the shadow casting of the light into the scene. So if there are objects between the light and the visible scene where shadows can be cast, they should be rendered into the shadow map.

Thank you for your explanations! Especially for clearing up the shadow mapping. Knowing the other planes are culled is good too! :)
Is the far value of the view frustum unused then?
Could it be changed to be used for culling?

it's not really intuitive that bevy doesn't cull on the far plane

in Bevy, Unity, and Unreal replicating this example will have culling on the far plane
new users may think this is a bug, especially as this kills performance on weak hardware.

Regarding the ability to not have this far culling at the same time

Making it large or adding a property for infinite far plane would match current behavior?

We could possibly make it optional for perspective projections.


Sidebar resize

Here is my proposal to fix issue #14 .
This fix allows sidebar to close and open correctly AFTER it has been resized.

Thats usefull.

Landed in 3.1.0. :ship: :tada:  Thanks @teamrc! :cake:


Recursive update of all dependencies

adapt to most wgpu API changes

Not all tests pass because of semantic changes of wgpu which I currently do not know how to properly fix.
Maybe you have an idea.

This is great! Thank you for this, hopefully this makes it easier to get proper WebAssembly support.


ci.yaml validation stuck pending in PR check

Has been stuck awhile...
https://github.com/flutter/flutter/pull/114099

cc @vashworth

Reset the pre-submit checks to unblock.
Looking through the backend, there were no useful logs related to this stuck.
If the ci.yaml validation check was created correctly, we should be able to see If this check is stuck pending, push an empty commit to retrigger the checks in the Details link.
My guess is it hit uncaught exception when creating the check run: https://github.com/flutter/cocoon/blob/main/app_dart/lib/src/service/scheduler.dart#L285

This issue duplicates: https://github.com/flutter/flutter/issues/114087


chore: Allow count of elements in repeated-element array syntax to be any comptime Field

Description
WIP
Problem
Resolves #445
Summary
WIP
PR Checklist

[ ] I have tested the changes locally.
[ ] I have formatted the changes with Prettier and/or cargo fmt on default settings.

@jfecher, сan you check this and give me a hint?

@jfecher, closed by mistake, can you open it?

@jfecher
For simple literals like let arr = [1, 2];, should we indicate the size during desugaring to address type inference issues?
As an analogy in Rust, it's akin to using the turbofish for explicit type annotations:
slice.as_array::<2>();

This was blocked by the fact that we had no way to indicate the type (size cannot be inferred from the context), now it's blocked by the same thing, but there's an open pull request after which we can resume work, but we encounter another problem, that the method with desugaring will break the formatter.


Fix path under "Remove the Sidebar"

This resolves the issue mentioned here.

Good catch! Thanks for the PR.


Define lookup handlers for lisp-interaction-mode

Is this necessary? lisp-interaction-mode is derived from emacs-lisp-mode so it should inherit emacs-lisp-mode's hooks, no?

Yes, they do seem to share hooks.  I see both messages when entering lisp-interaction-mode with:
(add-hook! 'emacs-lisp-mode-hook
           (message "emacs-lisp-mode-hook"))
(add-hook! 'lisp-interaction-mode-hook
           (message "lisp-interaction-mode-hook"))

This doesn't seem to affect the lookup handlers though:
(progn
  (lisp-interaction-mode)
  +lookup-documentation-functions)
;=> (+lookup-dash-docsets-backend-fn +lookup-online-backend-fn)

(let (funs)
  (emacs-lisp-mode)
  (setq funs +lookup-documentation-functions)
  (lisp-interaction-mode)
  funs)
;=> (+emacs-lisp-lookup-documentation t)

(progn
  (lisp-interaction-mode)
  +lookup-documentation-functions)
;=> (+lookup-dash-docsets-backend-fn +lookup-online-backend-fn)

Fair enough. Thanks!

Thank you!  <3


Create Khronotab unit-tests

Non-issue. After the restructuring in e5c729bc, the file is mostly empty.


Force update plugins

Currently BRAT only updates a plugin if the manifest version has changed. Is it possible to add a command that updates a plugin unconditionally? It would be helpful for me as my plugins have a latest release that does not change the manifest version but the content of the JS file.

Great idea. Will keep in mind for next update.

I am working on the feature and wonder if it should be called: "Reinstall beta plugin" - because we are not really updating but rather grabbing the plugin and redownloading the files. It could even be "redownload beta plugin".
Any thoughts on how to name the command?

This will be available in the 1.7.1 update, which should be pushed out soon. I went with "Reinstall" plugin. Let me know if you think it should be named something different.

I am working on the feature and wonder if it should be called: "Reinstall beta plugin" - because we are not really updating but rather grabbing the plugin and redownloading the files. It could even be "redownload beta plugin".
Any thoughts on how to name the command?

"Reinstall" does sound better.


[csrng, dv] Verify CTRL.MUBI CM, allow randomizing FLAG0

This PR contains two commits:

Allow randomizing FLAG0, at the moment disabled except in the csrng_alert test.
Extend the csrng_alert test to also test invalid MuBi encodings in the FLAG0 field of the Application Interface Command Header for Instantiate and Reseed commands (previously untested).

This PR is related to #16238.

I am merging this as the PR doesn't touch RTL anyway (DV tests already passed) and had passed full CI before already.


Top Bar Position - Timeline Quick View Oddity - Solid Bar Appears at top of screen

When the Top Bar Position is set and a Timeline Quick View appears, a solid bar is shown on the top of the screen.
This behavior does not present when the Bottom Bar position is set.
See linked photos for reference:
https://imgur.com/a/u5akN
I use the bottom bar position daily, so no impact on my usage. But wanted to bring it to light!
Amazing watchface. My go-to on my PTR and now my P2HR!

@freakified: This issue was wrongly opened here.
Was opened here instead: TimeStyleBBPebble#11
Can be closed !


streamCallback crash

I noticed at times the app will crash when I try to start the stream.
Trying to pin it down.
this is the crash error:
Thread 11: EXC_BAD_ACCESS (code =1 , address=0x16cbcbeb8)
[NSIS(m_inputStream) setDelegate:SCB(m_streamCallback)];
Could this be because of poor network connection?

#348 Hey! I fixed it.


Arrangement and execution of the meeting (issue #412 related)

The goal is to execute a meeting covering basics of numerical simulation using ODESystem class

Issue can be closed