Delayed crash after Animations with same name are added to the same AnimationLibrary

Tested versions
reproducible in v4.3.stable.nixpkgs [77dcf97d8]
System information
Godot v4.3.stable (77dcf97d8) - NixOS #1-NixOS SMP PREEMPT_DYNAMIC Thu Sep 12 09:13:13 UTC 2024 - X11 - Vulkan (Mobile) - integrated Intel(R) Graphics (ADL GT2) - 12th Gen Intel(R) Core(TM) i5-1240P (16 Threads)
Issue description
Low information core-dump when adding and hacking with Animation objects while making a multiplayer plugin.
================================================================
handle_crash: Program crashed with signal 11
Engine version: Godot Engine v4.3.stable.nixpkgs (77dcf97d82cbfe4e4615475fa52ca03da645dbd8)
Dumping the backtrace. Please include this when reporting the bug to the project developer.
[1] /nix/store/sl141d1g77wvhr050ah87lcyz2czdxa3-glibc-2.40-36/lib/libc.so.6(+0x40620) [0x7ffff7d0d620] (??:0)
[2] /nix/store/ss26dyx50lsqh5jdzyq621iq6vcdgxk0-godot4-4.3-stable/bin/godot4() [0x2cb27a4] (??:0)
[3] /nix/store/ss26dyx50lsqh5jdzyq621iq6vcdgxk0-godot4-4.3-stable/bin/godot4() [0x54229e] (??:0)
[4] /nix/store/ss26dyx50lsqh5jdzyq621iq6vcdgxk0-godot4-4.3-stable/bin/godot4() [0x4324cd1] (??:0)
[5] /nix/store/ss26dyx50lsqh5jdzyq621iq6vcdgxk0-godot4-4.3-stable/bin/godot4() [0x4325390] (??:0)
[6] /nix/store/ss26dyx50lsqh5jdzyq621iq6vcdgxk0-godot4-4.3-stable/bin/godot4() [0x4299b0a] (??:0)
[7] /nix/store/ss26dyx50lsqh5jdzyq621iq6vcdgxk0-godot4-4.3-stable/bin/godot4() [0x7cbce4] (??:0)
[8] /nix/store/ss26dyx50lsqh5jdzyq621iq6vcdgxk0-godot4-4.3-stable/bin/godot4() [0x68753c] (??:0)
[9] /nix/store/ss26dyx50lsqh5jdzyq621iq6vcdgxk0-godot4-4.3-stable/bin/godot4() [0x234258f] (??:0)
[10] /nix/store/ss26dyx50lsqh5jdzyq621iq6vcdgxk0-godot4-4.3-stable/bin/godot4() [0x2a0a4c5] (??:0)
[11] /nix/store/ss26dyx50lsqh5jdzyq621iq6vcdgxk0-godot4-4.3-stable/bin/godot4() [0x429d1f8] (??:0)
[12] /nix/store/ss26dyx50lsqh5jdzyq621iq6vcdgxk0-godot4-4.3-stable/bin/godot4() [0x22cfe18] (??:0)
[13] /nix/store/ss26dyx50lsqh5jdzyq621iq6vcdgxk0-godot4-4.3-stable/bin/godot4() [0x22cfdc4] (??:0)
[14] /nix/store/ss26dyx50lsqh5jdzyq621iq6vcdgxk0-godot4-4.3-stable/bin/godot4() [0x2326fbf] (??:0)
[15] /nix/store/ss26dyx50lsqh5jdzyq621iq6vcdgxk0-godot4-4.3-stable/bin/godot4() [0x42144f] (??:0)
[16] /nix/store/sl141d1g77wvhr050ah87lcyz2czdxa3-glibc-2.40-36/lib/libc.so.6(+0x2a27e) [0x7ffff7cf727e] (??:0)
[17] /nix/store/sl141d1g77wvhr050ah87lcyz2czdxa3-glibc-2.40-36/lib/libc.so.6(__libc_start_main+0x89) [0x7ffff7cf7339] (??:0)
[18] /nix/store/ss26dyx50lsqh5jdzyq621iq6vcdgxk0-godot4-4.3-stable/bin/godot4() [0x43e605] (??:0)
-- END OF BACKTRACE --
================================================================

Steps to reproduce
I was unintentionally adding Animations with the same name to the same AnimationLibrary due to the fact that they were not set as local_to_scene when they had been instantiated in a scene.
This seems to have created a bad pointer in the system which causes a crash on evaluation of $AnimationPlayer.current_animation_length after a delay.
Minimal reproduction project (MRP)
This is a very minimal project that reproduces the crash immediately
animationcrash.zip
extends Node2D

var a1 : Animation
var a2 : Animation
func _ready():
	var alib : AnimationLibrary = $AnimationPlayer.get_animation_library("liblib")

	print(alib, $AnimationPlayer2.get_animation_library("liblib"))

	var at = alib.get_animation("temp")
	a1 = at.duplicate()
	a1.track_insert_key(0, 0.0, Vector2(1,1))
	alib.add_animation("anim1", a1)
	$AnimationPlayer.play("liblib/anim1")
	$AnimationPlayer.pause()

	var at2 = alib.get_animation("temp")
	a2 = at2.duplicate()
	a2.track_insert_key(0, 0.0, Vector2(1,1))
	alib.add_animation("anim1", a2)
	print("Animation list ", alib.get_animation_list())
	prints(a1.length, a2.length, $AnimationPlayer.current_animation_length)

	# The Crash happens only on the delayed evaluation of current_animation_length
	await get_tree().create_timer(0.1).timeout
	prints(a1.length, a2.length, $AnimationPlayer.current_animation_length)

Thank you for reporting. However this is a duplicate of an already reported issue. We prefer to track issues in one place, so I am consolidating this with:

[ISSUE]

See the linked issue for details. If you think it may not be the same and we missed something, please comment here and this issue can be reopened.

Well, if you think this is the same as that other one, I'll subscribe to it so I can monitor what becomes of it, and check if this one disappears when it does.

If it's not please say so and this can be reopened, but with the information provided it sounds like the same issue (it's hard to tell without a stack trace showing the function names)

With a bit of work I've got a debug downloaded from https://github.com/Calinou/godot-debug-builds/releases and running on Windows.  The project has to be run from a PCK file since it is not the editor that crashes.
================================================================
CrashHandlerException: Program crashed
Engine version: Godot Engine v4.3.stable.custom_build (77dcf97d82cbfe4e4615475fa52ca03da645dbd8)
Dumping the backtrace. Please include this when reporting the bug to the project developer.
[0] Animation::get_length (C:\Users\hugol\AppData\Local\Temp\godot-debug-build-editor\scene\resources\animation.cpp:3703)
[1] AnimationPlayer::get_current_animation_length (C:\Users\hugol\AppData\Local\Temp\godot-debug-build-editor\scene\animation\animation_player.cpp:641)
[2] MethodBindTRC<AnimationPlayer,double>::call (C:\Users\hugol\AppData\Local\Temp\godot-debug-build-editor\core\object\method_bind.h:618)
[3] ClassDB::get_property (C:\Users\hugol\AppData\Local\Temp\godot-debug-build-editor\core\object\class_db.cpp:1556)
[4] Object::get (C:\Users\hugol\AppData\Local\Temp\godot-debug-build-editor\core\object\object.cpp:328)
[5] Variant::get_named (C:\Users\hugol\AppData\Local\Temp\godot-debug-build-editor\core\variant\variant_setget.cpp:314)
[6] GDScriptFunction::call (C:\Users\hugol\AppData\Local\Temp\godot-debug-build-editor\modules\gdscript\gdscript_vm.cpp:1180)
[7] GDScriptInstance::callp (C:\Users\hugol\AppData\Local\Temp\godot-debug-build-editor\modules\gdscript\gdscript.cpp:2032)
[8] Node::_notification (C:\Users\hugol\AppData\Local\Temp\godot-debug-build-editor\scene\main\node.cpp:217)
[9] Node2D::_notificationv (C:\Users\hugol\AppData\Local\Temp\godot-debug-build-editor\scene\2d\node_2d.h:37)
[10] Object::notification (C:\Users\hugol\AppData\Local\Temp\godot-debug-build-editor\core\object\object.cpp:873)
[11] Node::_propagate_ready (C:\Users\hugol\AppData\Local\Temp\godot-debug-build-editor\scene\main\node.cpp:273)
[12] Node::_propagate_ready (C:\Users\hugol\AppData\Local\Temp\godot-debug-build-editor\scene\main\node.cpp:262)
[13] Node::_set_tree (C:\Users\hugol\AppData\Local\Temp\godot-debug-build-editor\scene\main\node.cpp:3174)
[14] OS_Windows::run (C:\Users\hugol\AppData\Local\Temp\godot-debug-build-editor\platform\windows\os_windows.cpp:1662)
[15] widechar_main (C:\Users\hugol\AppData\Local\Temp\godot-debug-build-editor\platform\windows\godot_windows.cpp:181)
[16] _main (C:\Users\hugol\AppData\Local\Temp\godot-debug-build-editor\platform\windows\godot_windows.cpp:208)
[17] main (C:\Users\hugol\AppData\Local\Temp\godot-debug-build-editor\platform\windows\godot_windows.cpp:220)
[18] __scrt_common_main_seh (D:\a\_work\1\s\src\vctools\crt\vcstartup\src\startup\exe_common.inl:288)
[19] <couldn't map PC to fn name>
-- END OF BACKTRACE --
================================================================

Animation::get_length() is at https://github.com/godotengine/godot/blob/4.3/scene/resources/animation.cpp#L3703 and called from AnimationPlayer::get_current_animation_length()
https://github.com/godotengine/godot/blob/4.3/scene/animation/animation_player.cpp#L639
Therefore I believe that playback.current.from->animation is an invalid pointer.
It also crashes on 4.4dev3.
I don't think this issue should be closed. @AThousandShips

Will need some further testing then, not sure what differs between the two cases from your description but will see what the animation team says


Remove extra space from Makefile

Just to make bin/traceur.js and build/compiled-by-previous-traceur.js logs simetrics.

LGTM

Committed as 2e3f3c651763cadb35b58067d32ad8b65639e815


fix misspelled Kubernetes in Health Check Guide

found a typo in the health check guides should be Kubernetes not Kuberbetes

merged: f997bc0920ca153f0c6a4ddcb3c83d97446f683d


Question only    polybar in herbstluftwm

Are there any plans for making polybar work with hlwm.
I've tried several times and failed every time - but I'm just a simple enthusiast and not a programmer.
So basically I'm wondering if you're thinking of making polybar usable for hlwm too - and not jst ob . i3 and bspwm

I use polybar on xmonad, displaying workspaces with the xworkspaces-module (https://github.com/jaagr/polybar/wiki/Module:-xworkspaces). hlwm would need to be EWMH-compliant for it to work.

I use hlwm as my daily driver and it works without problems. What doesn't work for you? I don't use the xworkspaces module for my bar so I don't know how well it works, but I think hlwm is EWMH-compliant.

Closing due to inactivity


[BUG] Install silently fails with Anaconda, macOS

Describe the bug
There is a bug trying to install oiio with Anaconda. I'm not sure if you guys are in charge of distributing it through conda-forge or not, but figured I'd mention it.
I got it to install normally, once, on macOS Mojave a few weeks ago. It installed all the necessary dependencies (ilmbase, boost, etc). However, trying to install it in Big Sur the other day on another machine, and trying to install in Mojave in another Anaconda env fails presently, though it doesn't appear to. It finishes without error, Anaconda will list all the libraries as being installed, but the .so and other files are not in the env subfolders (or anywhere else on the system that I can find).
To Reproduce
Steps to reproduce the behavior:

Launch terminal, activate Anaconda env
conda install -c conda-forge openimageio
Package is not installed anywhere. Searching system files produces no results. Manual checks turned up empty.

Expected behavior
Package would be available in /conda_install_location/envs/MYENV/lib/python3.8/site-packages
Evidence
Nothing to show since it is a lack of the .so being compiled/written and no errors are thrown during install.
Platform information:

OIIO branch/version: 2.2.18
OS: macOS 10.14 and macOS 11.6
C++ compiler: clang/cmake I assume
Any non-default build flags when you build OIIO: didn't build it myself, used Anaconda

Sort-of workaround:
Since I got oiio installed recently (same version 2.2.18) I am able to copy the .so from my successful install to other envs and it works properly.

No, we are not actually directly involved in how the library is packaged up for Conda.
I think this is an issue that should be filed against conda, I don't think there's anything we can do about it (unless whoever is responsible for the conda package happens to be reading this issue here).

Okay. I'll try to find who manages it. Thanks.


don't reject @suppress {checkTypes} annotations

Rather than always disallowing {} in JSDoc, disallow it in the
specific cases we know that we already handle.

LGTM


fix: localize terminal find placeholder

Types

[x] 🐛 Bug Fixes

Background or solution
Resolve issue #2244
Changelog
wrap input placeholder inside localize

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Download the template repo into system default temp dir, to avoid the…

Download the template repo into system default temp dir, to avoid the unnecessary tmp folder auto-creation on Windows.

Thank you @tweea ✌


Client TUI improvements

fix issue #23, refactor client code

The 1m timeout is currently broken and needs to be updated.


Feat:  adding hacktober fest logos to the readme

Describe the feature
adding  hacktober fest logos to the readme & update  the readme
Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[X] I have starred the repository
[X] I'm a gssoc-24-extd contributor
[X] I want to work on this issue

assign me under gssoc 2k24

Good Idea @REHAN-18

@RamakrushnaBiswal done !!

Solved via #346


GPU memory out when training with BATCH_SIZE=8  in SECOND_STAGE

I used BATCH_SIZE=8 in FISRT_STAGE_EPOCHS  for training,there is no problem,but when I used BATCH_SIZE=8 in SECOND_STAGE_EPOCHS  for training,the error message is memory out.
My GPU is RTX 2070,where is wrong?

GPU内存太小了，我用的TitanXp 12G，都没法用batch=8进行第二阶段训练

GPU内存太小了，我用的TitanXp 12G，都没法用batch=8进行第二阶段训练
OK!

GPU内存太小了，我用的TitanXp 12G，都没法用batch=8进行第二阶段训练

OK，thank you for your reply.

I used batch_size = 6 and 2080 Ti. It shows out of memory!! How to solve it?

@Derekabc  set batch_size=2 or 4


2.0.0 needs to be released in RubyGems

Por favor.

+1

Any chance of this happening soonish?

Is 2.0.0 not ready for prime time?

Over a year since this is opened, should we just fork it and release that?

Very tempting. I have a private streetaddress I am currently using.

I won't have time to work on this for at least the next three months.  I'm ok transferring ownership of this repo and the gem to a volunteer.

Thanks for response @derrek. I nominate @zhon since hes already using it otherwise I don't mind doing it.

I am happy to contribute!

I don't mind doing it, except it will be a couple of weeks before I could get to it.
I also like moving it the street-address/street-address and adding a couple of other owners.

I think moving it and having multiple collaborators is a decent proposition given my current constraints.

@jsmestad would you like to collaborate on street-address as well?

I set up street-address-rb.  Need to do a quick bit of research to see if I  think its best to transfer this repo there or fork it into there.

let me know if i can help

Any update on this @derrek?

@drews256: I set up street-address-rb org and invited members.  Non of the invites were accepted.
That said I didn't follow through much.  Anyone on this thread willing to help maintain it? @zhon @jnmandal @jsmestad

I'll volunteer to help co-maintain it

@jaydorsey can you email me at derrek dot long at gmail

Will do. I got your invite last night and accepted

Hey @derrek I'm happy to help as well! I dont think I got an invite 🤔

@jnmandal invited

Alright the repo has been moved to https://github.com/street-address-rb/street-address
Slow but steady progress...

@derrek I'm still happy to help with this repo if you're interested. I sent you an email but understand you might be busy. Feel free to reach out when you're ready.

Seems @derrek is p. busy; @zhon, @jaydorsey what do you all see as the steps we need to take to ensure the 2.0.0 is robust enough for a release?
Currently I'm using a fork of master in production for my app with no issues (https://github.com/compost-connect/street-address). Anyone else doing something similar?

I have a small monkey patch to the released version (1.0.6) where I modify the way addresses are output. This works for me on Ruby 2.3. I have not even tested the master.
There are 3 unresolved patches we might want to look at.
I would like to see the test coverage metrics and see all the test passing.

@jnmandal I'm using a fork as well. The only changes I made are slight to the Gemfile to make it easier to use when you're pointing to the repo instead of using the gem.
I'd probably want to give it a full lookover including the outstanding PRs per @zhon
Also, I think a few of the issues (US/United States) could be quickly and easily addressed
I didn't want to jump into anything without chatting w/ @derrek first. Plus, he may be the only person with access to push to rubygems?

For @derrek to add a new owner, it only takes the following:
$ gem owner --add {{email}} {{gem}}

@zhon @jnmandal I chatted w/ @derrek briefly via email last week. He's swamped right now but is going to grant access to rubygems to one or more of us soon.
I'm currently using a fork of master. The only change I really made was to update the spec file so that the gemspec name is street_address to comport with what you commonly see in gems. IIRC, that made it slightly more intuitive to reference using it inside of a gemspec (e.g. gem/add_development_dependency 'street_address' vs StreetAddress)
Other than that, I figured I'd look at the 3 open PRs, run thru the issues list and curate it and see if there are any low hanging issues we can resolve immediately (the two for US/USA might be as easy as a regex/delete and 2 extra tests), and make sure the tests are passing.
I've been using this for a month or two now and for the most part is "just works" so I think other than the curation above a new release could be done relatively quickly.
Thoughts?

I like your thinking. I think we should have working release asap.

https://rubygems.org/gems/StreetAddress/ - 1.0.6 is still the latest.  A changelog for a major version bump would be good.

Bump. This is a nice gem - let's bring it back to life!

Is anyone using the github master branch in production?  I'd like to hear about it. If all is well we can push a 2.0 gem.  I held off releasing 2.0, a long time ago, because there were some huge regression issues then haven't been able to work in this space since.

No worries. Just happy to resuscitate the repo in some form, even if changed or slightly broken! I'm on an old version, not master. Want to take a stab at a changelog section in the README? Maybe I can help with testing.

I'm closing this issue, don't think it'll be resolved. Hopefully someone forks and maintains!

Any chance of release the 2.0 gem? I'm having version conflict when using this gem inside a custom gem that gets used in another project. The custom gem is working off of GitHub master (of this repo), but the project that is using that gem says
Could not find gem 'StreetAddress (~> 2.0.0)', which is required by gem 'property_service', in any of the sources.
It'd be great if StreetAddress 2.0.0 can be released to RubyGems.


[perfect-numbers] Description is not clear

The description isn't very clear, I mean it's not clear what to implement. I've been looking at the implementations and there's no consensus. Some of them implement the classification and the others just check whether or not the number is perfect.

Perfect numbers description

I would suggest doing something like what we have recently done for triangles. Triangles test data
Where there are 3 different functions that return a boolean value depending on if the number is  Perfect, Deficient, or Abundant

Looking at Triangles I see that metadata.md contains a clear statement what to do

Write a program that can tell you if a triangle is equilateral, isosceles, or scalene.

This statement is missing here.

@Insti this might be a good "first patch"

Naturally, one might ask: Is #481 sufficient to close this issue?
It certainly has helped greatly in this regard.
I think a little help in the README would be good - so perhaps https://github.com/exercism/x-common/blob/master/exercises/perfect-numbers/description.md could also say that we need to be able to determine whether a given number is perfect, abundant, deficient.
The sentence on the Aliquot sum also seems a bit out of place.

perhaps https://github.com/exercism/x-common/blob/master/exercises/perfect-numbers/description.md could also say that we need to be able to determine whether a given number is perfect, abundant, deficient.

#518 takes care of that.


package:flutter/src/widgets/framework.dart': Failed assertion: line 3571 pos 12: '!_dirty': is not : true

Steps to Reproduce

I created a global event listener on the home page, created OverlayEntry when the value is true, and displays it on other pages;
Now OverlayEntry is displayed, but this error is reported, not sure if it is my own problem. !

Logs

flutter: ══╡ EXCEPTION CAUGHT BY WIDGETS LIBRARY ╞═══════════════════════════════════════════════════════════
flutter: The following assertion was thrown while rebuilding dirty elements:
flutter: 'package:flutter/src/widgets/framework.dart': Failed assertion: line 3571 pos 12: '!_dirty': is not
flutter: true.
flutter:
flutter: Either the assertion indicates an error in the framework itself, or we should provide substantially
flutter: more information in this error message to help you determine and fix the underlying cause.
flutter: In either case, please report this assertion by filing a bug on GitHub:
flutter:   https://github.com/flutter/flutter/issues/new?template=BUG.md
flutter:
flutter: When the exception was thrown, this was the stack:
flutter: #2      Element.rebuild
package:flutter/…/widgets/framework.dart:3571
flutter: #3      BuildOwner.buildScope
package:flutter/…/widgets/framework.dart:2278
flutter: #4      _WidgetsFlutterBinding&BindingBase&GestureBinding&ServicesBinding&SchedulerBinding&PaintingBinding&SemanticsBinding&RendererBinding&WidgetsBinding.drawFrame
package:flutter/…/widgets/binding.dart:700
flutter: #5      _WidgetsFlutterBinding&BindingBase&GestureBinding&ServicesBinding&SchedulerBinding&PaintingBinding&SemanticsBinding&RendererBinding._handlePersistentFrameCallback
package:flutter/…/rendering/binding.dart:286
flutter: #6      _WidgetsFlutterBinding&BindingBase&GestureBinding&ServicesBinding&SchedulerBinding._invokeFrameCallback
package:flutter/…/scheduler/binding.dart:1012
flutter: #7      _WidgetsFlutterBinding&BindingBase&GestureBinding&ServicesBinding&SchedulerBinding.handleDrawFrame
package:flutter/…/scheduler/binding.dart:952
flutter: #8      _WidgetsFlutterBinding&BindingBase&GestureBinding&ServicesBinding&SchedulerBinding._handleDrawFrame
package:flutter/…/scheduler/binding.dart:864
flutter: #12     _invoke  (dart:ui/hooks.dart:219:10)
flutter: #13     _drawFrame  (dart:ui/hooks.dart:178:3)
flutter: (elided 5 frames from class _AssertionError and package dart:async)

[✓] Flutter (Channel stable, v1.5.4-hotfix.2, on Mac OS X 10.14.5 18F132, locale zh-Hans-CN)
    • Flutter version 1.5.4-hotfix.2 at /Users/dmlzj/www/flutter/source
    • Framework revision 7a4c33425d (10 weeks ago), 2019-04-29 11:05:24 -0700
    • Engine revision 52c7a1e849
    • Dart version 2.3.0 (build 2.3.0-dev.0.5 a1668566e5)

[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at /Users/dmlzj/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • ANDROID_HOME = /Users/dmlzj/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1136-b06)
    • All Android licenses accepted.

[✓] iOS toolchain - develop for iOS devices (Xcode 10.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 10.2.1, Build version 10E1001
    • ios-deploy 1.9.4
    • CocoaPods version 1.7.2

[✓] Android Studio (version 3.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 31.3.1
    • Dart plugin version 181.5656
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1136-b06)

[✓] VS Code
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.2.0

[✓] Connected device (2 available)
    • iPhone 6s Plus • 6429CE55-63CE-4D56-A211-AF805487B982 • ios • com.apple.CoreSimulator.SimRuntime.iOS-12-2 (simulator)
    • iPhone X       • 797028AE-1310-4EC8-8BBD-AC3591ABD83E • ios • com.apple.CoreSimulator.SimRuntime.iOS-12-2 (simulator)

My code :
class Index extends StatefulWidget {
  @override
  _IndexState createState() => _IndexState();
}

class _IndexState extends State<Index> {
  int tabIndex = 0;
  OverlayEntry overlayEntry;
  @override
  void initState() {
    bus.on('globalAudio', (arg) async{
      LogUtil.v(arg);
      if (arg) {
        await createGlobalAudio(context);
      } else {
        overlayEntry.remove();
      }
    });
    super.initState();
  }

  List<Widget> pageView = [HomeView(), ReadInnovation(), MyView()];
  @override
  void dispose() {
    super.dispose();
  }

  void selectTab(int index) {
    setState(() {
      tabIndex = index;
    });
  }

  // 创建全局音频窗口
  Future createGlobalAudio(BuildContext context) async{
    overlayEntry = new OverlayEntry(
      builder: (context) {
        // FocusScope.of(context).setFirstFocus(focusScopeNode);
        return Positioned(
            width: MediaQuery.of(context).size.width,
            height: 54,
            bottom: 180,
            // color: Colors.red,
            child: new Material(
              child: Padding(
                  padding: const EdgeInsets.all(0.0),
                  child: Column(
                    children: <Widget>[
                      DecoratedBox(
                        decoration: new BoxDecoration(
                          // border: new Border.all(
                          //     color: Color(0xFFFF0000), width: 0.5), // 边色与边宽度
                          // 生成俩层阴影，一层绿，一层黄， 阴影位置由offset决定,阴影模糊层度由blurRadius大小决定（大就更透明更扩散），阴影模糊大小由spreadRadius决定
                          borderRadius: BorderRadius.only(topLeft: Radius.circular(8), topRight: Radius.circular(8)),
                          boxShadow: [
                            BoxShadow(
                                color: Color.fromRGBO(0, 0, 0, 0.12),
                                offset: Offset(5.0, -5.0),
                                blurRadius: 10.0,
                                spreadRadius: 5.0),
                            // BoxShadow(
                            //     // color: Color(0x9900FF00),
                            //     offset: Offset(1.0, 1.0)),
                            BoxShadow(color: Color.fromRGBO(255, 255, 255, 0.9))
                          ],
                        ),
                        child: Row(
                          children: <Widget>[
                            Text('data', style: TextStyle(color: Colors.red),)
                          ],
                        )
                      )
                    ],
                  ),
                ),

            ),);
      },
    );
    //往Overlay中插入插入OverlayEntry
    Overlay.of(context).insert(overlayEntry);

    setState(() {});
  }

  @override
  Widget build(BuildContext context) {

    return Scaffold(
      body: pageView[tabIndex],
      bottomNavigationBar: BottomNavigationBar(
        selectedItemColor: Colors.red,
        items: const <BottomNavigationBarItem>[
          BottomNavigationBarItem(
            icon: Icon(Icons.home),
            title: Text('home'),
          ),
          BottomNavigationBarItem(
            icon: Icon(Icons.chrome_reader_mode),
            title: Text('read'),
          ),
          BottomNavigationBarItem(
            icon: Icon(
              Icons.people,
            ),
            title: Text('my'),
          )
        ],
        currentIndex: tabIndex,
        onTap: selectTab,
      ),
    );
  }
}

Am facing  this Error
'package:flutter/src/widgets/framework.dart': Failed assertion: line 2306 pos 12: '!_debugBuilding': is not true.
How can I fix this?

do not use flutter

Hi @dmlzj
Could you please run flutter upgrade --force.
If the issue persists, please provide your updated flutter doctor -v,
your flutter run --verbose
and a minimal reproducible code sample.
Thank you

@dmlzj
Closing as additional information isn't provided.
Could everyone who still has this problem please file a new issue with filling out template

While this error because you called setState more than one times in a lifecycle. Try to merges setState into a single call.


MISP: Pulls stuck randomly with error message

At the moment we see a problem in the MISP server that causes a server to get stuck after pulling some events from an external connected instance.
After some investigation, we found that there may be time problems that cause pulled events with a missing publishing timestamp to be inserted into the database, which then leads to an error in the MISP server and stops the pull job. Events that cause this error are displayed as unpublished in the MISP.
This error is marked as critical and a fix is planned for RC.2. This may delay the release of RC.2.

This is confirmed as bug in MISP (https://github.com/MISP/MISP/issues/3244)


MPO_700 does not spawn

Tested Distro: Rolling
Problem: Not able to spawn mpo_700
Log:

[spawn_entity.py-1] ValueError: Unicode strings with encoding declaration are not supported. Please use bytes input or XML fragments without declaration.

UTF-8 encoding was causing the issue! Rectified in recent commit!


Document what is required to use docker-compose setup

To be included in the documentation:

docker and docker-compose must be installed locally
Default Ram limits for docker (2GB - at least on OS X) are not enough for weaviate. The limits have to be increased to at least X GB (I will add an exact number later, after some trial and error)
The setup can be started with docker-compose -f docker-compose-dev.yml up from the project root. This spins up weaviate, as well as the demo importer. Weaviate is ready to be used as soon as the demo importer has finished with exit code 0.

Step 3 is not ideal yet as it is semi-manual, we are about to improve it, as part of #525. After that, we'll probably have a simple script that is blocking until the import has succeeded or finally failed.

Thanks @etiennedi, the RAM is also based on the size of the contextionary. I will make sure to add this to the docs.

Just FYI: The process I have described is to start up the dev-setup. Since we import a fixed set of demo-data, I'd assume the size of the contextionary is also fixed? (At least until the user imports their own data).

Ran some test on the memory requirements:

Memory Limit
Outcome

2 GB
❌ elasticsearch exits with 137

2.5 GB
✅  importer runs successfully

3 GB
✅  importer runs successfully

It seems with the current setup 2.5GB is the sweetspot. To have some headroom, I'd recommend mentioning 3GB in the docs.
It might also be worth explaining the symptoms: I.e.

"If the weaviate_index container fails with exit code 137 make sure your Docker limits are set toat least 3GB"

Alternatively, we could check if we could reduce the memory needs for Elastic by setting e.g. -Xmx, or some other JVM flags to detect the amount of memory available.
But this is probably the easier route yes.

Hi @moretea, friendly reminder for a suggestion/hint on how an end user currently is able to change the location of the Contextionary. Thx

"changing the location" is too imprecise. If the end-user wants to use a different contextionary, something like this would be sufficient:
FROM semi/weaviate
COPY ./new_contextoinary /location/of/contextionary/in/docker/image

Super, check, thanks

FMI: https://github.com/creativesoftwarefdn/weaviate/issues/549#issuecomment-433075246

Closing as this is part of the Getting Started Guides in the meantime.


Memory leak when using realtime charts

Codepen
The bug in question is illustrated by the realtime demo on the website: https://apexcharts.com/javascript-chart-demos/line-charts/realtime/
Explanation
Expected behaviour:
Be able to remove non-visible data points (avoiding memory leaks) while retaining the realtime animation
What is happening:
Two twings can happen:

Do nothing and the series array containing the datapoints will keep expanding forever (memory leak). The animation looks as expected, but after a few minutes the browser will start struggling due to the memory leak.
Remove non-visible data points from the series. This avoids the memory leak, but the realtime animation is not there anymore and instead you get weird animations when data points move in a vertical manner. The demo on the website solves this by "resetting" the data series every 60 seconds, but this causes the chart to show glitchy behaviour every 60 seconds.

I hope it is possible to fix this. In its current form the realtime chart is unfortunately not usable for anything other than demo purposes. That's a shame because ApexCharts is a really nice looking promising charting library and the realtime chart could be a killer feature.

Thanks for the detailed explanation.
When the data is appended, the SVG line paths gets extended to an unreasonable width if xaxis.range is provided. This puts a lot of stress on the animation as animating a line of 10000px would be very intense. Hence, I was resetting data every 60 seconds.
To overcome this situation, I tried an alternative way of keeping the line path short by not removing the datapoint but changing it to the first date of the range. That way, the animation is still smooth and we don't have to worry about resetting. If in case, the data becomes very large later on, we can reset the data completely at every 5 or 10 minutes.

The new method of resetting the previous data-points can be accessed here

Thanks for this update. Your new solution makes the realtime chart more workable, but it still requires to keep keep datapoints in memory that are not visible anymore.

Yes. That's why I suggested doing a reset at every 5 or 10 minutes.
Unfortunately, there's no other workaround yet.

Unfortunately, there's no other workaround yet.

@junedchhipa perhaps you can give some leads to investigate ?

We are having about the same problem and no turnaround seems to work.
Our company may be willing to fund fixing this bug. Please get in touch with me at<EMAIL_ADDRESS>Than you !
Stef


Forge version?

I know the annoyance of people asking for the other version, but i think out of all mods to be ported, this one is actually useful.
So at least consider it, just for this simple-ish project :)

Normally I wouldn't bother with Forge, but this mod is small enough that I might try it sometime.
In the meantime, you can try using Sinytra Connector, or any other compat tool. Again, since this mod is small, it might just work without issues.

Normally I wouldn't bother with Forge, but this mod is small enough that I might try it sometime. In the meantime, you can try using Sinytra Connector, or any other compat tool. Again, since this mod is small, it might just work without issues.

Hi, I'm creator of M-Tech modpack for 1.19.2. It has 1.2m downloads on curseforge.
I really need this mod.
Will you able to port it?

Did a quick test today and got it working on forge 1.20.1. I did not look into 1.19.2 yet, but I think I can make it work.

Any chance you release it?

My bad, I run into some troubles setting up the automatic publishing, put it off, and then completely forgot about it. Sorry about that!
Gonna publish it manually for now. Will look at backporting in a little bit, too.

Thanks! Really waiting 1.19.2 backport.

My bad, I run into some troubles setting up the automatic publishing, put it off, and then completely forgot about it. Sorry about that!
Gonna publish it manually for now. Will look at backporting in a little bit, too.

Any chance 1.19.2 forge backport?
I really need this mod in next major modpack udpate which close to release.


code injection with download from git-scm.com under link: 64-bit Git for Windows Setup

Running windows 10 64-bit and firefox
Trying to download git from page https://git-scm.com/downloads link to windows and the file may or may not start downloading or I click the link on the following page, 64-bit Git for Windows Setup, in either case this issue is present. This issue seems not to exist with the portable version for windows.
There is an unexpected file that installs while the expected file downloads. This unexpected file installs to a location other than the customary downloads location. That unexpected file installs in firefox folder at location:
C:\Users\username\AppData\Local\Mozilla\Firefox\Profiles########.default\cache2\entries
The expected file, Git-2.17.0-64-bit.exe, downloads to the expected downloads location.
This unexpected file is in bit format and is named with a key that is unique every download.
This unexpected file installs regardless, whether or not I proceed with the download or cancel the download through the dialog box associated with the download. This observation I made on may 23 2018 while attempting to document this questionable download behavior.
Steps To Reproduce:
Using windows 10 64-bit and firefox latest version as of this writing, version firefox 60.0.1 (64-bit)
Document the contents of folder
C:\Users\username\AppData\Local\Mozilla\Firefox\Profiles########.default\cache2\entries
go to https://git-scm.com/download/win click link: 64-bit Git for Windows Setup
go back to folder
C:\Users\username\AppData\Local\Mozilla\Firefox\Profiles########.default\cache2\entries
and note the addition of the latest file by date entry or a comparison of the contents of this folder as it exist after the download against the contents of the folder prior to the download.
Impact
Possible the unexpected file runs on next start-up of the firefox browser.

Hey @somestuff !
Thanks for reporting this issue. But that location seems to be where firefox stores cached data to possibly speed up navigation. A quick google search led me to this post: https://support.mozilla.org/pt-BR/questions/1169456


Device Profile "Assigned Firmware" not synced from CE Docker 3.4

I'm trying to perform OTA update from a Device connected to Edge.
The OTA firmware is visible in Edge and can be downloaded from the Edge UI.
However :

The device profile "assigned firmware" field has not synced from the Thingsboard CE.
Also , if I assign a Firmware directly to the device from the Thingsboard CE , it does not sync this field to Edge.
So, this means that when using the firmware url download , i get a 404 error, because the device does not have an assigned fw.

Workaround so far to be able to use the OTA firmware URL is to manually assign the firmware in the Edge UI.
(url : http://192.168.xx.xx:8080/api/v1/TOKEN/firmware?title=Tank_Level&version=1.0.27&size=0&chunk=1)
Am I missing something obvious here ?
Edge

CE

Edge Log when modifying the Assigned firmware of device profile on the CE
2022-08-04 20:44:49,479 [TB-Scheduling-1] INFO  o.t.s.c.t.s.DefaultTransportService - Transport Stats: openConnections [0]
2022-08-04 20:45:26,024 [grpc-default-executor-67] INFO  o.t.s.s.c.r.p.BaseCloudProcessor - Adding ATTRIBUTES_REQUEST/RELATION_REQUEST fd1ecb60-133d-11ed-908a-d922e38596b1 DEVICE
2022-08-04 20:45:28,129 [sql-log-1-thread-1] INFO  o.t.s.dao.sql.TbSqlBlockingQueue - Queue-0 [TS] queueSize [0] totalAdded [2] totalSaved [2] totalFailed [0]
202
CE log when modifying assigned firmware of device profile on  CE
2022-08-04 10:45:23,917 [rule-dispatcher-3-3] INFO  o.t.rule.engine.action.TbLogNode -
Incoming message:
{"id":{"entityType":"DEVICE_PROFILE","id":"7dafec70-0ca2-11ed-b7a7-5d88a11bd8fd"},"createdTime":1658812755127,"tenantId":{"entityType":"TENANT","id":"6d046130-f474-11ec-b746-1bfa669193d7"},"name":"Water Level","description":"","image":null,"type":"DEFAULT","transportType":"DEFAULT","provisionType":"CHECK_PRE_PROVISIONED_DEVICES","defaultRuleChainId":null,"defaultDashboardId":null,"defaultQueueName":null,"profileData":{"configuration":{"type":"DEFAULT"},"transportConfiguration":{"type":"DEFAULT"},"provisionConfiguration":{"type":"CHECK_PRE_PROVISIONED_DEVICES","provisionDeviceSecret":"ayrubtxsal0l09mg9jra"},"alarms":null},"provisionDeviceKey":"s6ymhzoxhan5l7k9kwwq","firmwareId":{"entityType":"OTA_PACKAGE","id":"70f1fb40-13d2-11ed-ad99-cdc5dc232715"},"softwareId":null,"externalId":null,"default":false}
Incoming metadata:
{"user

@joinke thanks for creating this ticket.
This is a bug in the 3.4 release.
The fix for this is going to be in the 3.4.1 release.
Please use a workaround at the moment and assign firmware to the device using Edge UI.
Sorry for any inconviences.
Pull request to this issue is going to be created today/tomorrow.

Many thanks for replying superfast and for proposing alternative/fix !

Fixed in the 3.4.1 release https://thingsboard.io/docs/edge/releases/#v341-august-19-2022

Hello @volodymyr-babak ,
I've tested this fix . The device firmware on the TB edge is now updated when I update the device profile in the TBCE. I can see that in the fw_tag, fw_version .. attributes. So, that solves the problem. One observation though , on the TB edge , in the device profile details , I can't see the Firmware details :
(both TB and TB edge are 3.4.1) (TB CE docker and TBEdge Ubuntu)
CE:

Edge :


[7.x.x] Bump Dependencies #4799348610

Bumps dependencies for product-is. Link : https://github.com/wso2/product-is/actions/runs/4799348610

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.jenkins-is-staging seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Mouse cursor becomes trapped in center of screen when dragging texture to "Animation Frames"

Godot version: v3.0.2.stable.official

OS/device including version: Windows 10 Home 64-bit

Issue description: Mouse cursor sometimes becomes trapped in center of screen when dragging xyz.png to "Animation Frames" while following tutorial "http://docs.godotengine.org/en/3.0/getting_started/step_by_step/your_first_game.html". I found a similar issue related to v2 stable branch but submitting here as this is slightly different....

Steps to reproduce:

Minimal reproduction project:

Having the same issue, but with drag and drop to sprite nodes on occasion.


Ollama messages are not being recorded in DB

I don't know if we're not storing them in DB or just not post-processing them correctly. Not being able to see them on dashboard endpoints.
/dashboard/messages
/dashboard/alerts

Closed via #331


pkg/trace/api: update dogstatsd proxy docs

What does this PR do?
Update the statsd handler documentation

Motivation

Additional Notes
Related to https://github.com/DataDog/datadog-agent/pull/13628

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


 I understand the singleton pattern but isn't it cleaner to just do Screen.xxx and TemplateFinder.yyy instead? I can't think about any immediate benefit of having an instance for both of those classes. The internal state of Config is stored in class level variables, TemplateFinder is just a utility in it's nature as well. There is no harm in leaving this as is if its a big ask, but just raising to discuss

I'm not opposed to that, but I don't really view it as being any different (functionally); we can achieve what you're saying by doing something like:
screen = Screen()
screen.grab()

Whether we store the singleton in a local variable or use it directly doesn't make a difference imo, but I can see how the code may look cleaner that way.
Another approach would be to break the singleton(s) completely and have everything managed in their respective files.
e.g.,
# screen.py
sct = mss()
monitor_roi = sct.monitors[0]

def grab() -> np.ndarray:
    img = np.array(sct.grab(monitor_roi))
    return img[:, :, :3]

I'd be in favor of this approach too because then all of the callers would have an even cleaner contract:
from screen import grab

def my_func_somewhere_else():
    img = grab()

What do you think @VladimirMakaev?
Personally, I like having the Screen() and TemplateFinder() because it (hopefully) reads / signifies that it's a singleton; whereas, if it's stored in a local variable now you have one layer of (possibly unnecessary) indirection.

Tested on Ezro's and my machines. Tested chicken, game recovery, and death as well. Working nicely.
Have not yet tested on VM.

Tested for hours on Ezro's and my machines. Tested chicken, game recovery, and death as well. Working nicely.
Have not yet tested on VM.

I've tested for 8+ hours on both VM and local.


can't modify frozen Hash (FrozenError)

Hey,
while entering rails conzole and/or rails server from (overridden) provider in folder /plugins/manageiq-providers-cisco_intersight (inside MiQ core) I get error, saying that hash is frozen:

Note that running the same commands (bundle exec rails s or bundle exec rails c) from the core does not give me an error. Any ideas about resolving this error?
Thanks a lot, Tjaž

Yep, that solved it. Thanks!


fix: kind cluster must access private registries

May fix #321

This is passing tests but...

the test chart is not modified so we should either generate some tests changes or force the helm install anyway
be sure our test case is using a private image

up @neilime ?


dosent work

Extension Name: vscode-icons
Extension Version: 7.27.0
OS Version: Windows_NT x64 6.1.7601
VSCode version: 1.28.0-insider

We have written the needed data into your clipboard. Please paste:

Please provide more information. :)

Maybe this can help you.


cap fps accuratly

I want to cap fps at exactly 72 fps, I can't do that in sodium. hook me up yeah?

A mod menu screen is currently being worked on to give more control over the mod. For such specific FPS limits, the vanilla bug where the FPS is kept at ~96.2% of the actual set limit will need to be fixed.

Added mod menu support with the ability to cap fps at any value. The vanilla bug still persists but for now you can compensate by increasing the fps limit a little over.


Create md-kaif-ansari.md

Notes for Reviewers
This PR fixes #
Signed commits

[x] Yes, I signed my commits.

🚀 Preview for commit 904569fa69241679cc95479cd11f513c5b36d903 at: https://67110bc8e331c31b20261e28--meshery-docs-preview.netlify.app


fix(docker): replace tini with dumb-init

Replace tini with dumb-init  as the use TINI_SUBREAPER  is causing issues where  PR_SET_CHILD_SUBREAPER is unavailable on some  platforms.

@abester Should we make this change across the other Dockerfiles we have?

@abester Do we have a write up of the underlying issue somewhere?  As @TelicentPaul notes we've used tini as the init process across pretty much all the platform containers so this will need fixing globally and it would be good to understand the problem so we aren't making the wrong fix and have to churn through another round of PRs

@abester Do we have a write up of the underlying issue somewhere? As @TelicentPaul notes we've used tini as the init process across pretty much all the platform containers so this will need fixing globally and it would be good to understand the problem so we aren't making the wrong fix and have to churn through another round of PRs

details of the problem: https://github.com/docker/for-mac/issues/6620#issuecomment-1694380189
its preventing the app from starting up when running in k8s on local desktop such as rancher-desktop , etc.

@abester Should we make this change across the other Dockerfiles we have?

I've done the PR for search, for now thats all that we need.


Unit test for LFN<->PFN consistency

Split off from #6437 from @mmascher
"I have a general comment. We all know that we have two implementations of the LFN<=>PFN conversion in CMS: one that is in Perl in phedex (lfn2pfn API), and one in python here in WMCore. Unfortunately we do not have anything that checks that the two are the same, and I know for sure that on some particular iinput the two were not returning the same result.
I think it would be nice to have a unit test that takes some lfn that are used in WMCore and CRAB3, all the sites, and checks that the phedex lfn2pfn API, and the WMCore API return the same information."
Maybe when we get around to mocking PhEDEx we can add this.

The other option is an integration test. On a first iteration it calls lfn2pfn, on another one it fetches the TFC from the phedex database and do the conversion using the WMCore implementation.


Research report

https://docs.google.com/presentation/d/19r1fZqs-j9NVleo2wNX1SgwD-x0t7qHiKizpfSdpYO8/edit#slide=id.g2ef72c1b34c_0_0


Uploading .hidx to the server

I got the following error message:
Latest server version: 4.0.1
Current local version: 3.1.0 (out-of-date)
[-] Your hmark is not up-to-date.

That's why I bypass the update check and run the tool. After generating .hidx , I uploaded them in https://iotcube.net/process/type/wf1. But it is not showing me any result. Rather asking me to download latest binaries.
I also downloaded [hmark for Linux x64](https://iotcube.net/tools/wf1/hmark_4.0.1_linux_x64.tar.gz) but when I try to run in on my Linux machine it is showing the following error:
[8512] Error loading Python lib '/tmp/_MEIdkjLT1/libpython3.8.so.1.0': dlopen: /lib/x86_64-linux-gnu/libm.so.6: version GLIBC_2.29' not found (required by /tmp/_MEIdkjLT1/libpython3.8.so.1.0)

I was able to solve it by switching to binary files of windows.


Update MongoDB Backup Agent to v<IP_ADDRESS>4

Codecov Report

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

@@           Coverage Diff           @@
##           master    #1674   +/-   ##
=======================================
  Coverage   98.46%   98.46%
=======================================
  Files          59       59
  Lines        2862     2862
=======================================
  Hits         2818     2818
  Misses         44       44

lgtm, but some tests are failing on travis. Can you retrigger it? Seems like an infra issue.


Fix stray double-quote in PATH

Spotted while fixing the various other recent breakages on the Windows images.

Included in #128


Bump to 0.8.4 version

We need to have this merged first

Published bb8 v0.8.4 at registry crates-io
[new tag]         v0.8.4 -> v0.8.4

Thanks!

Seems that the documentation is not providing all the details https://docs.rs/bb8/0.8.4/bb8/struct.State.html did we miss something for having the new Statistic struct being added in the documentation?

Ahh, we probably need to re-export Statistics in lib.rs.  (You can run cargo doc --open --no-deps to check the generated docs before submitting the PR.)

Yeps, give me some seconds

Here we go https://github.com/djc/bb8/pull/209


Access to maxViewMode is not possible yet.

I've seen that in the changes you've made to master, access to the maxViewMode is possible.  I need to use this for my project.  Will you be releasing a new version soon?

See comments on #1497. closing that one here.


Bug Report: custom character based on spookeez bops every 2nd beat

My custom character is based on the spookeez code and he goes left and right every 2 beats. It is like:
1
2 left
3
4 right
it should be
1 left
2 right
3 left
4 right
im bad at explaining things but i need help and i dont know how to fix it.

My custom character is based on the spookeez code and he goes left and right every 2 beats. It is like:
1
2 left
3
4 right
im bad at explaining things but i need help and i dont know how to fix it.

send the character code

idleToBeat is stupd

nvm fixed it

Then close the issue


[keyboard manager] A shortcut to toggle enable current shortcuts

I realize that a profiles feature is to be implemented for keyboard manager as seen on #1881 and would be an easy way to toggle between a profile with the actual shortcuts and a blank profile as an on/off toggle but that seems slightly far for now and having a way of quickly toggle it off would be a nice QoL change that doesn't necessarily overlap with profiles.
For a bit of context, I'm using shift+1~5 to send 6~0 while gaming as a sort of accessibility workaround and would like to be able to use the !@#$% symbols on chat without having to tab out and disable keyboard manager or setting another convoluted set of shortcuts to send those inputs again.
So:

shift+1 currently set to 6
press toggle shortcut(e.g. win+~)
can now type Hello! instead of Hello6
press toggle shortcut again
shift+1 is now back as 6

@lakitu13 The confusion with this could be that since Keyboard Manager in itself remaps keys/shortcuts, users could get confused about whether this shortcut (Win+~) would be affected by remaps. Would you expect it to be a shortcut that will be unaffected by any remaps? (for example if a user remapped Win to Ctrl, will pressing Win+~ actually toggle Keyboard Manager?)

Would you expect it to be a shortcut that will be unaffected by any remaps?

Yes! Calling it a "master shortcut" could make it easier to understand it would take priority over the "normal" shortcuts and key remapping. Once you choose your own "master shortcut" there could also be a confirmation prompt alerting the user if one of these keys is already in use as a remap/shortcut and vice versa.
Additionally on the interface, all the remaps and shortcuts could be grouped and indented to the master shortcut setting to help make it clear it has priority. Furthermore, this option could be hidden under an advanced settings, though it seems unnecessary in a set of tools meant for power users.
I believe it could look like this:

Heya, found this by googling. A master shortcut would be amazing, and would allow me to selectively remap keys to use in certain games, but still be able to toggle the remaps off when I want to use the keyboard normally to type in chat.

Closing as duplicate of https://github.com/microsoft/PowerToys/issues/4879
Even if the proposed implementation is different, the functionality is the same.

Closing as duplicate of https://github.com/microsoft/PowerToys/issues/4879
Even if the proposed implementation is different, the functionality is the same.


Implement methods

Need to implement methods in the new interpreter. I've got a basic class ready for it, but I want to put more consideration into this implementation than I did in the legacy interpreter.
We need to implement a register to store the value of return.
Example
def add(@first, @second)
   return @first + @second
end

@sum = add(50, 100)

if @sum == 150
   println "Passed"
else
   println "Failed"
endif

The basic flow will look something like:
// pseudocode
interpret():
   while (!end):
      if current() == "def":
         interpretMethodDefinition()
      next()

# pseudocode
interpretMethodDefinition():
   next() # skip "def"

   # grab name and parameters
   name = current()
   while current() != ")":
        add_parameter_token(current())

   # add body
   count = 1
   while count > 0:
      if requires_end_keyword(current()):
          ++count
      else if current() == "end":
          --count;
          if count == 0:
            break

   # if all is kosher, add object to map (map by name?)

Done.


Drop "aggregator" from names

Love it! 😍 Do you want to shorten Metric.MetricName to simply Metric.Name?

@olegsych @vitalyf007 let's also get back to use MetricTelemetry instead of AggregatedMetricTelemetry? Since there are no Aggregate anywhere now.

Not sure why I can't comment in line... So, I'll comment here. I'll implement Oleg's suggestion (MetricName => Name). As far as Sergey's suggestion, we can't really do it. MetricTelemetry already exists and I'd like to keep AggregatedMetricTelemetry as is for now.

@vitalyf007 any reasons to keep AggregatedMetricTelemetry instead of adding properties to MetricTelemetry?

The reason to keep MT and introduce AMT is two fold. First, back compat. Try to figure out how to add properties to existing MT such that none of the allowed usages are broken is hard. Usually it leads to situations when object is opened up to inconsistent state problems. MT represents two things and I want to deprecate it in favor of more clear object AMT.

I understand what you want to represent. But on the surface for the customer they are the same except Value is not Sum and other properties are not nullable. Usage of the class is exactly the same except one of constructors do not exists.
I just worry that creation of a new class is our attempt to make it clearer in our own heads.
@olegsych - do you have an opinion here?

@SergeyKanzhelev Regarding keeping MetricTelemetry vs introducing AggregatedMetricTelemetry I agree with the reasons @vitalyf007 cited. It's an important step toward cleaning up the confusion around MetricTelemetry. However, I feel we should only do it if we can commit to cleaning this up in the HTTP API and AA schemas within a reasonably short amount of time, ideally within one year. If that's not realistic, keeping the existing MetricTelemetry would be a lesser evil.

@olegsych we removed the term "Aggregation" from everywhere. We just saying "the metric". So we are sending the metric value as a MetricTelemetry object. This is my reasoning that the new type clears the situation in our mind, but confuses the customer. We do not expose metric kind in UI or AA. Maybe export shows it. But in general for the customer - metric got reported, value (most times aggregated) can be read in UI and AA.

@SergeyKanzhelev Good point. So instead of introducing AggregatedMetricTelemetry as a new type, we could also keep MetricTelemetry, add Sum property to it and deprecate its existing Value property. With the TelemetryClient.TrackMetric(string, double, IDictionary) method already deprecated, the confusion around whether metrics are pre-aggregated or not is also removed. @vitalyf007 I think I like this better than introducing a new telemetry class.


Create a Promises/A+ wrapper for m.deferred()/m.sync() exporting a Promise global

This is more or less a TODO for me. It'll help ease migration for some of the changes I'll be making to m.request() in light of #800 and #802.

On a related note, a modified version of the suite needs to make it into core as well. It's better to safely deprecate stable things than potentially broken things.

This is now done here


Forum footer and main footer look different

Forum footer:

I prefer the main footer (obviously):

So yes, we should probably try to make them at least similar.

I designed the main footer yay (for responsiveness, try it) but dunno how to set up the footer on the forums..
GrannyCookies is the one who managed the forum haxx, so I assigned him :package:

Good idea lols.

Also footer looks great!

When you assign me stuff, notify me on gitter, I don't check my github notifications :P
Working on it.

The code is in, but an admin needs to push to the non-osdev site for it to work.

All done. The CSS needs tweaking, but nothing major.
I'm also looking into allowing the team to run OSDEV builds on TeamCity independently.

@The-Cheese-Eater Great.

Cool!


Question: Can I have different AKS instances target a single Application Gateway?

I read this in the documentation:
For deployments that require more than one AGIC per cluster or multiple AGICs targeting one Application Gateway, please continue to use AGIC deployed through Helm.
However, it is not clear to me whether this means the multiple AGIC instances can be spread over AKS clusters, or whether they have to run on the same cluster.
I have a couple of clusters for which I would like to use the same Application Gateway, basically to reduce costs.

It seems this is possible:
https://docs.microsoft.com/en-us/azure/application-gateway/ingress-controller-install-existing#multi-cluster--shared-application-gateway
However, not very manageable if you ask me. You need to explicitly set which hostnames NOT to consider. Wouldn't it make more sense to have to specify which hostname I DO want AGIC to handle? I'm unable to know at all times which ingress hostnames are specified in the other cluster..


Wiki pages max size

Large wiki pages get cut at some point, havn't really checke the code, but i guess there is a maximum size for the markdown parser somewhere.
sorry if this is a duplicate, cause i remember reading about that somewhere, just could not find it in issues.

+1  major bug! If there is a limit, it should be very high, like 1MB or so

Example URL?

We do not have public git repositories, but all i can tell you is that if i save a big wiki page it is cut off at around a
-rw-r--r--  1 user  staff  101287 Aug  3 06:51 Max Size Test.md

The content can be downloaded at BIG MARKDOWN FILE

Demo on https://try.gogs.io as requested.

Ok i tested it at https://try.gogs.io/nananana/try/wiki/Home and the text is not cut, looks like i have to update my version of gogs. Or maybe it is the apache proxy.


switch to uuid-random instead of uuid package

Because usage of synchronous functions inside "uuid" package performance suffers, if knex is used in context of node based web-server. I tested "uuid-random" package which increase performance by 30%. (despite of my commit comment, new package does use synchr. functions, but in optimized way)

I'm not sure about using not very battle-tested library (1k downloads/week vs 10m for uuid). Is this a hot spot and bottleneck?

I avoided any synchronous function calls so far, and this is the only one alerted by node (WARNING: Detected use of sync API ...) in my app servers log (in dev-mode) so far. The best might be, to use something which create UUIDs without usage of synchronous functions ... but did not find a package that so far ...

I dont see any reason to change this. 30% difference in microbenchmarks is not valid reason.
If you are able to show some realworld benchmark, where the difference is significant that would be a good reason.

I think you can probably use https://github.com/ai/nanoid for that (I'm not not affiliated with author)
It gives nearly same performance increase (~25%, according to docs), also it uses less memory for IDs due to the length


Dashboards rename related changes

re-opened for #1173

Codecov Report

Merging #1192 (baf639e) into main (9328d08) will increase coverage by 0.09%.
The diff coverage is 90.38%.

@@             Coverage Diff              @@
##               main    #1192      +/-   ##
============================================
+ Coverage     64.51%   64.60%   +0.09%
- Complexity     3170     3182      +12
============================================
  Files           245      245
  Lines         17128    17147      +19
  Branches       3035     3038       +3
============================================
+ Hits          11050    11078      +28
+ Misses         4538     4524      -14
- Partials       1540     1545       +5

Impacted Files
Coverage Δ

...zon/dlic/auth/http/saml/HTTPSamlAuthenticator.java
69.72% <55.55%> (-0.28%)
:arrow_down:

...opensearch/security/filter/SecurityRestFilter.java
75.86% <75.00%> (+0.86%)
:arrow_up:

...pensearch/security/auditlog/impl/AuditMessage.java
73.57% <100.00%> (ø)

...earch/security/dlic/rest/api/AccountApiAction.java
82.35% <100.00%> (ø)

.../security/dlic/rest/api/ActionGroupsApiAction.java
100.00% <100.00%> (ø)

...nsearch/security/dlic/rest/api/AuditApiAction.java
63.82% <100.00%> (ø)

...curity/dlic/rest/api/AuthTokenProcessorAction.java
57.14% <100.00%> (ø)

...ch/security/dlic/rest/api/FlushCacheApiAction.java
63.33% <100.00%> (ø)

...security/dlic/rest/api/InternalUsersApiAction.java
82.35% <100.00%> (ø)

...earch/security/dlic/rest/api/MigrateApiAction.java
67.70% <100.00%> (ø)

... and 19 more

Continue to review full report at Codecov.

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


Fix/2529

Summary of changes
Closes #2529.
Pull Request Checklist

[x] Changes have tests
[x] News fragment added in changelog.d/.

Brilliant. Thanks!

Brilliant. Thanks!

I'll need to roll back this change as it's implicated in #2536.

I'll need to roll back this change as it's implicated in #2536.


Qiao continue part3-added delete images functions

Thanks.
But when I remove my image from my userProfile page, my userProfile image uploaded still in the admin browse images db.
Can you please take a look ?

But as an admin, I can still see the picture I removed.


Add TLS support for CTLog

Any idea whats wrong here?
https://github.com/securesign/fulcio/actions/runs/10093063453/job/27907996686#step:6:360


ui: Improve the display of information on /meeting/requests

Note the change to the PR title -  a bare 'ui: ' won't match what the CC engine is looking for.


clear keys list as well when the widget updates otherwise RangeError …

…occurs when number of items change in the list.
This occurs when list grows.

Thanks for contributing @ykhedar. Let me check.


Use correct function getTopmostReportId instead of removed one

Details
The PR https://github.com/Expensify/App/pull/21702 removed the function Navigation.getReportIDFromRoute(), but forgot to replace one spot where it was called, causing the app to crash. This removes the last occurrence of the function call.
Fixed Issues
$ https://github.com/Expensify/App/issues/22502
$ https://github.com/Expensify/App/issues/22502#issuecomment-1627616899 (/cc @c3024)
Tests

Sign out of the app
Go to /concierge (e.g. http://localhost:8080/concierge or https://staging.new.expensify.com/concierge)
Verify that you are taken to the login screen
Sign in with your credentials
Make sure the app doesn't crash
Note: it's possible you won't be redirected to the Concierge chat, as one would expect. This is a known issue that is being handled here.

[x] Verify that no errors appear in the JS console

Offline tests
N/A, you need to be online to sign in
QA Steps
Same as tests

[ ] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] If we are not using the full Onyx data that we loaded, I've added the proper selector in order to ensure the component only re-renders when the data it is using changes
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web
https://github.com/Expensify/App/assets/2229301/eb458096-6f07-4471-a8cc-0a24a3b047fc

Mobile Web - Chrome

Mobile Web - Safari

Desktop

iOS

Android

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/marcaaron in version: 1.3.38-6 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/francoisl in version: 1.3.38-7 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


1 fix issue of  https://github.com/kubeovn/kube-ovn/issues/1151

What type of this PR
Bug fixes

I tried to reimplement the func of ovn-leader checking. But I do not know if it is as your expectation.
would you give me  some comments?
Which issue(s) this PR fixes:
https://github.com/kubeovn/kube-ovn/issues/1151
Fixes #(1151)

@sxilinux the reason that we want to move it to go binary is that the ovn-is-leader.sh now is doing lots more than just readinessProbe. The leader detection and label pods work should be run in the main cmd rather than a readinessProbe.
What we need is to move this part of work to a new cmd and simplify the readinessProbe.

@oilbeater , thanks for your reply.
accodring to your description, my understanding is like that:
1 replace old new ovn_is _checker.sh by using go cmd( for convenience, we call the go binary as ovn_leader, ovn_leader does the same job as  ovn_is _checker.sh ). and at pod readiness process, insteadlly, we use  ovn_leader.
am i right?

@sxilinux ovn-leader 需要完成 leader 判断和切换以及 compact 相关的周期性任务，readiness 只需要完成检查 nothd，ovnnb 和 ovnsb 状态的任务即可，相当于把真正的健康检查工作和周期性的处理工作分开。周期性的工作放在 go binary 里，让然后容器启动后最后就执行这个二进制文件即可。健康检查可以还用之前的脚本，但只保留最基础的检查操作，或者将这部分工作也转移到 go 代码里，提供 http 接口进行检查

@oilbeater ，需求我理解了。我重新把代码调整下。

@oilbeater  i have remodified the code like what you said. would you review the codes?

1 i constructed kind-env in my local machine , did the test and  fixed code miss which will cause ci failture.

Please follow https://stackoverflow.com/questions/5189560/squash-my-last-x-commits-together-using-git and squash the 8 commits into one

ok. i will squash these commits into one

Hi, 16477a8 has squashed commits into one.

Thanks! @sxilinux


Enabling "Force hourly data update" during setup makes HA run out of memory

Version of home_assistant
2023.9.3 (Operating System 10.5, Supervisor 2023.09.2)
Version of the custom_component
v2.0.1 (via HACS)
Describe the bug
The bug is triggered by enabling "Force hourly data update" in the UI when configuring the integration. Without that option, the integration works fine.
First, the message "Unknown error occurred" appears in the UI. Memory consumption (measured by resident set size) slowly starts rising. About 30 seconds later, Home Assistant Core quickly starts consuming all available memory (from about 800M free to zero in <1 second) and is then forcibly terminated by the kernel. After Supervisor restarts Core, the integration config is gone. There is no error in the logs.
I don't think that this integration should require >800M of RAM for setup (about 3x the memory footprint of my normal HA Core), so I suspect that there is a major memory leak somewhere.
Debug log

2023-10-03 17:55:07.737 DEBUG (MainThread) [custom_components.dwd_weather] Setup with data {'data_type': 'forecast_data', 'station_id': '<redacted>', 'station_name': '<redacted>', 'wind_direction_type': 'degrees', 'hourly_update': True}
Oct 03 15:55:37 homeassistant kernel: oom-kill:constraint=CONSTRAINT_NONE,nodemask=(null),cpuset=/,mems_allowed=0,global_oom,task_memcg=/system.slice/docker-c7b337ad9212af91e5f199ce424af3791bb98d742caeded4f0675a8d687571ea.scope,task=python3,pid=46892,uid=0
Oct 03 15:55:37 homeassistant kernel: Out of memory: Killed process 46892 (python3) total-vm:3163072kB, anon-rss:1253000kB, file-rss:0kB, shmem-rss:0kB, UID:0 pgtables:3652kB oom_score_adj:-300
Oct 03 15:55:37 homeassistant systemd[1]: docker-c7b337ad9212af91e5f199ce424af3791bb98d742caeded4f0675a8d687571ea.scope: A process of this unit has been killed by the OOM killer.
23-10-03 17:56:31 ERROR (MainThread) [supervisor.misc.tasks] Watchdog found a problem with Home Assistant API!
23-10-03 17:56:31 INFO (SyncWorker_1) [supervisor.docker.manager] Restarting homeassistant

I will have a look into this. A bit more of RAM is needed, because the compressed file itself is already 43 MB. But this much is a little too much.

Afaik zip files can be processed stream-oriented (with low memory footprint).

The problem is, I have to parse the entire XML file and I am not sure if this will work with a stream.

I looked at your code and tested the memory usage; the ZipFile remains compact in memory (there are stream-oriented zip libraries, but the ~45 MB are not the problem.
For the stream-oriented XML you might want to try xml.etree.ElementTree.iterparse or its lxml equivalent (I would recommend to begin with the standard library version first to keep dependencies low and see how well that works).

Tinkered around a bit. This should give you a hint (the i.clear() part does the trick):
from zipfile import ZipFile
from xml.etree.ElementTree import iterparse

for (_, i) in iterparse(ZipFile(open('MOSMIX_S_LATEST_240.kmz', 'rb'), 'r').open('MOSMIX_S_2023100516_240.kml')):
  print(i)
  i.clear()

The python process consumes around 10 MB of memory on my MacBook.

Thanks for the hint. Unfortunately I am using lxml and I am not able to get it working with this. I created a branch stream if you would like to assist here.

I would like to, but I am really very busy and rusty when it comes to developing – still proficient in the Python language, but I fell up the corporate ladder away from development many years ago. It's really a problem for many HA users to have such a disk space and memory consuming add-on for a typical HA installation (especially it that's really unnecessary), but I am not having serious disk space, memory or CPU limitations in mine.
I haven't set up a serious Python development project (with a working dev environment directly on the git checkout, a build chain which runs the tests and builds the package, an installation to clean venv, test runner, several interpreter versions, …) in years. I just forked your code and couldn't get a project up and running with neither Eclipse/PyDev nor VS Code; it started with broken test files, I have forgotten how to run the tests with pip, etc. I'd really like to help, and if (sadly not when) I find the time and succeed to get something running, I might.
The outlined strategy should work, though. It looks to me as if lxml is mostly a drop-in replacement for xml (why did you use lxml in the first place?). But you will have to change the parsing strategy to an iterative one and drop all stations from the ElementTree on-the-fly. My little program above takes about 2s on my M1 MacBook.

I'll add this to my list, as I am also a bit busy at the moment.
I used lxml for the implementation of XPath.

Thanks for helping with this. Python isn't my main language, so there was some work to do ;) Best regards!
The new version v2.0.4 should resolve this.

Happy to help. Python used to be my main language. Now that I‘m into it I might help you tidy up the plain Python library (there is still quite a bit of repetitive, hard-tor-understand, entangled stuff left) weh I find some time. For the Home Assistant integration I might need a heads up how to properly test it.


Change url in Podspec to HTTPS one.

Hello.
Currently, in Podspec we have an SSH URL for sources. It breaks build on CI systems (sources can't be downloaded anonymously through SSH on GitHub).
Can you, please, change it to HTTPS one and upload the new version of the library?
I think s.source line in Podspec can be changed to:
s.source = { :git => 'https://github.com/blockfrost/blockfrost-swift.git', :tag => s.version.to_s }

Thanks.

Sure! Will release it today

@ypopovych it should be fixed already. @mmahut when you have a minute, pls release a new cocoapod version


test: Implement ModifiableLengthFieldTest #HSFDPMUW

I'll review this PR once I'm back from vacation by next week.


Clarify the difference between two kinds of SOUR.SOUR

#372 adds SOUR.SOUR to handle derivation and related provenance information. It does this in two places: SOURCE_RECORD and SOURCE_CITATION. We should add a Note in both places to cross reference the other example, or a technical FAQ in next minor describing how these differ.

I don’t understand SOUR.SOUR, this would seam to me to cause a loop!  Where can I see the proposal for this structure? Source_Records do not have a TYPE to identify what kind of source_record we are looking at, I’ve alway thought that sources need to have some kind of categorizing system (like Media_Records) to tell us the source_record is a census, grave, church book, bible, … now if we are using a Source_Record contain providence and derivation information which would use fields differently I’d want to know when listing sources. But I could be misinterpreting this, but I was still hoping for a new record type for providence and derivation information.

I really don’t understand this Subtag of SOUR._SOUR! What is being modeled here? Is this a book within a book, a set of books, documents or registers bound together to form a new book?
If so, normally the citation takes the form of:
Author Last Name, First M. "Chapter or Essay Title." In Book Title, edited by First M. Last Name
Which in GEDCOM can take two forms:

using the PAGE tag to document the inner book, with the source_record documenting the outer book.

using the source_record to document both the inner and outer book!

I normally take option 2.
0 @…@ SOUR
1 TITLE Chapter or Essay Title in Book Title
2 AUTH Author Last Name, First M.
2 PUBL Edited by First M. Last Name
3 CONT …
This “book in book” is very common in antiquarian texts where the original may be hard to review (either lost or not available for general review) and an editor transcribed it for us.  It is also common when translations are made and combined together with other translations.  We don’t get a chance to look at the original and someone nicely made them available to us.
IMHO We should not be telling our readers about a copy we did not see as proof that a “fact” we are asserting. The translation could have been wrong, the transcription (copy) may be incomplete or wrong.  Notes in the Editors Book (the book we read) source__record should remind the reader that this person included other people’s work in our cited source, but we were not able to view/review the original!

See https://github.com/dthaler/gedcom-citations/blob/main/source-derivation-extension.md which gives an example from Evidence Explained which this is intended to allow expressing in GEDCOM.

Everyone uses EE as THE authority, but other authorities exist such as Chicago/Turabain!  Following these should also be an option.
However I rather enter them this way:
0 @S1@ SOUR
1 TITL St. Paul Church Baptisms, December 1869-August 1893, in Baptismal Records, Wilmington, Delaware
1 AUTH Joseph M. Lalley Jr.
1 PUBL http://www.lalley.com : accessed 28 January 2007
But since the website is NOT the source, but the REPOsitory (just like Ancestry in not the Source) the better way would be:
0 @I1@ INDI
1 BAPM
2 SOUR @S1@
3 PAGE p. 214
2 DATA
3 DATE 28 January 2007
0 @S1@ SOUR
1 TITL St. Paul Church Baptisms, December 1869-August 1893
1 REPO @R1@
2 CALN http://www.lalley.com/
0 @R1@ REPO
1 NAME Baptismal Records, Wilmington, Delaware
The CALN could also be different if the "Baptismal Records, Wilmington, Delaware" is a subset of the website.
1 REPO @R1@
2 CALN Baptismal Records, Wilmington, Delaware, http://www.lalley.com/
0 @R1@ REPO
1 NAME
NOTE:  The bigger issue is that the Source_Citation and Source_Record are developed without enough fields and specifically without the ability to contain independent pieces of the written citation without regard to the citation style used.  A better way of working with "citations" is to have templates that overlay the data in the database so that ALL pieces of the citation can be found and positioned based on the style used.
When I did my Masters studies in Library Science we stored citation information following a robust and variable recording system call MARC.  I'm not suggesting that MARC be used here but an open ended citation record should be developed with a cataloging librarian as input to define all of the pieces used by EE to form a citation, footnote, endnote, etc, then build that citation record with all of the necessary fields and some form of TYPE to identify the citation we are recording.

NOTE: The bigger issue is that the Source_Citation and Source_Record are developed without enough fields and specifically without the ability to contain independent pieces of the written citation without regard to the citation style used. A better way of working with "citations" is to have templates that overlay the data in the database so that ALL pieces of the citation can be found and positioned based on the style used.

See https://github.com/dthaler/gedcom-citations/pull/3 for a proposal to do that.

When I did my Masters studies in Library Science we stored citation information following a robust and variable recording system call MARC. I'm not suggesting that MARC be used here but an open ended citation record should be developed with a cataloging librarian as input to define all of the pieces used by EE to form a citation, footnote, endnote, etc, then build that citation record with all of the necessary fields and some form of TYPE to identify the citation we are recording.

Would love to have you participate as part of the new citations team.

It has been a while since we discussed SOUR.SOUR and I am not sure if this concept has been eliminated!

@Norwegian-Sardines:
I try do understand your point using REPO, but it seems to do not fit to the situation:
We have an online archive (www.archion.de) showing scans of chuch books. Every page in those church book may be accessed by a link separately. By this I have:

the source: a church book
the citation: data taken from a part of one of the pages in the church book / scans in archion
I would like to add the link to the scan to the citation. However GEDCOM does not allow REPO in SOURCE_CIATTIONS, and the source itself is not specific as it is the whole book - with many scans..
Do you have a solution?

Albert,
I’m not sure I completely understand your question!
A written citation is nothing more than a statement in which we use to identify what supplied the information for an assertion, the source of the assertion. The term “source” means the actual artifact, book, document, website, etc. from which we obtained our information.
The Source_Record points to the REPOsitory where we found the source so the reader can go find it themself, this is (or should be) found using the link in the GEDCOM, Citation_Structure->Source_Record->Repository_Record.  When we talk about a “citation” in GEDCOM we are talking about the Citation_Stucture not the written form.
Do not confuse (as many people and software companies do) the artifact source with the colloquial use of the word “source”!
I’ve written some comments about this in a review for the citation group, and I’m in the process of writing a short summary with more information about citations as well!

The source (the church book) can be found in internet, so I can link SOUR via SOUR.REPO to the internet representation which contains in thousands of scans of this and other church books. Every scan has an own link, so I can refer to this scan by the special link for this scan. An event is documented on a page of the church book (I can refer to it by SOUR.PAGE), and in one of thousands of scans, where I have a link to show this special scan. You propose to handle the internet pages as REPO, but how do you refer to a special scan within this repo? I cannot do it with REPO.CALN, as it is not called in the source, but in the citation.
I try to show it by an example:
0 SOUR @S1@ SOUR
1 TITL Church book of St. Johann in ...
1 REPO @R1@
2 CALN www.archion.de
0 @R1@ REPO
1 NAME Archion
0 @I1@ INDI
1 NAME John Meyer
1 CHR
2 DATE 2 MAY 1888
2 SOUR @S1@
3 PAGE volume:Taufen 1885-1890, page:15, no:12
3 ???? url: www.archion.de/....special link to the scan of page 15 of Taufen 1885-1890 ...

I cannot use REPO in this structure, but
3 REPO @R1@
4 CALN url: www.archion.de/....special link to the scan of page 15 of Taufen 1885-1890 ...

would be a nice solution.
So far for I put it in PAGE, and do not use REPO:
3 PAGE volume:Taufen 1885-1890, page:15, no:12, url:www.archion.de/....special link to the scan of page 15 of Taufen 1885-1890 ...

.SOUR.PAGE already allows for an internet link in its design for v7 of GEDCOM.  A technical document was created outlining this at: https://github.com/dthaler/GEDCOM.io/commit/da067fbb23821bb29de7b5528f376bae1f3f0321


[DOC] Fix broken link in NewToGluten.md

What changes were proposed in this pull request?
this PR is fixing broken link for build-gluten-with-velox-backend in the NewToGluten.md file.
How was this patch tested?
NA

Thank you for fix


Compile failed on AlmaLinux(aarch64) with fpm SAPI

Build Command
bin/spc build bcmath --debug --build-fpm

Console Log
/root/static-php-cli/source/php-src/sapi/fpm/fpm/fpm_atomic.h:136:2: error: #error Unsupported processor. Please open a bug report (bugs.php.net).
  136 | #error Unsupported processor. Please open a bug report (bugs.php.net).
      |  ^~~~~
/root/static-php-cli/source/php-src/sapi/fpm/fpm/fpm_atomic.h:140:32: error: unknown type name ‘atomic_t’
  140 | static inline int fpm_spinlock(atomic_t *lock, int try_once) /* {{{ */
      |                                ^~~~~~~~
In file included from /root/static-php-cli/source/php-src/sapi/fpm/fpm/fpm.c:19:
/root/static-php-cli/source/php-src/sapi/fpm/fpm/fpm_scoreboard.h:23:17: error: unknown type name ‘atomic_t’
   23 |                 atomic_t lock;
      |                 ^~~~~~~~
/root/static-php-cli/source/php-src/sapi/fpm/fpm/fpm_scoreboard.h:52:17: error: unknown type name ‘atomic_t’
   52 |                 atomic_t lock;
      |                 ^~~~~~~~
make: *** [Makefile:393: sapi/fpm/fpm/fpm.lo] Error 1
make: *** Waiting for unfinished jobs....
In file included from /root/static-php-cli/source/php-src/sapi/fpm/fpm/fpm_scoreboard.h:13,
                 from /root/static-php-cli/source/php-src/sapi/fpm/fpm/fpm_children.c:26:
/root/static-php-cli/source/php-src/sapi/fpm/fpm/fpm_atomic.h:136:2: error: #error Unsupported processor. Please open a bug report (bugs.php.net).
  136 | #error Unsupported processor. Please open a bug report (bugs.php.net).
      |  ^~~~~
/root/static-php-cli/source/php-src/sapi/fpm/fpm/fpm_atomic.h:140:32: error: unknown type name ‘atomic_t’
  140 | static inline int fpm_spinlock(atomic_t *lock, int try_once) /* {{{ */
      |                                ^~~~~~~~
In file included from /root/static-php-cli/source/php-src/sapi/fpm/fpm/fpm_children.c:26:
/root/static-php-cli/source/php-src/sapi/fpm/fpm/fpm_scoreboard.h:23:17: error: unknown type name ‘atomic_t’
   23 |                 atomic_t lock;
      |                 ^~~~~~~~
/root/static-php-cli/source/php-src/sapi/fpm/fpm/fpm_scoreboard.h:52:17: error: unknown type name ‘atomic_t’
   52 |                 atomic_t lock;
      |                 ^~~~~~~~
make: *** [Makefile:395: sapi/fpm/fpm/fpm_children.lo] Error 1

System Info
# uname -a
Linux localhost.localdomain 5.14.0-284.30.1.el9_2.aarch64 #1 SMP PREEMPT_DYNAMIC Tue Sep 12 13:39:16 UTC 2023 aarch64 aarch64 aarch64 GNU/Linux

# cat /etc/os-release
NAME="AlmaLinux"
VERSION="9.2 (Turquoise Kodkod)"
ID="almalinux"
ID_LIKE="rhel centos fedora"
VERSION_ID="9.2"
PLATFORM_ID="platform:el9"
PRETTY_NAME="AlmaLinux 9.2 (Turquoise Kodkod)"
ANSI_COLOR="0;34"
LOGO="fedora-logo-icon"
CPE_NAME="cpe:/o:almalinux:almalinux:9::baseos"
HOME_URL="https://almalinux.org/"
DOCUMENTATION_URL="https://wiki.almalinux.org/"
BUG_REPORT_URL="https://bugs.almalinux.org/"

ALMALINUX_MANTISBT_PROJECT="AlmaLinux-9"
ALMALINUX_MANTISBT_PROJECT_VERSION="9.2"
REDHAT_SUPPORT_PRODUCT="AlmaLinux"
REDHAT_SUPPORT_PRODUCT_VERSION="9.2"

Other Info
Compile pure C program with __arch64__ successfully..
#include <stdio.h>
int main() {
#if (__arch64__ || __arch64)
    printf("ARM!\n");
#else
    printf("Not ARM!\n");
    return 0;
}

ARM!

缺 linux-headers 头文件吧

@jingjingxyk
Package kernel-headers-5.14.0-284.30.1.el9_2.aarch64 is already installed.

Is this a bug with aarch64-linux-musl too, or just the gcc wrapper?

Is this a bug with aarch64-linux-musl too, or just the gcc wrapper?

It seems that it only fails to compile on AlmaLinux-aarch64. It works fine on both alpine and debian of my arm64 architecture.

I meant if it also happened when you build musl-cross-compile (TARGET=aarch64-linux-musl) and then use aarch64-linux-musl to compile php-fpm.

I meant if it also happened when you build musl-cross-compile (TARGET=aarch64-linux-musl) and then use aarch64-linux-musl to compile php-fpm. I don't have an ARM64 machine, so I cannot test this. :/

I finally tested on clean debian 12 arm64 platform and use spc doctor --auto-fix, got the same error:

I can't see the configure command so I don't know if that's musl-gcc wrapper or aarch64-linux-musl compiler.
Maybe I will rent an aarch64 VM to test this.

@DubbleClick I uploaded the entire php-src and found no useful information so far.
php-src.tar.bz2.zip
GitHub only allows me to upload files smaller than 25MB, and not in bz2 format, so I can only compress it again.

It's musl-gcc wrapper. Can you try with the cplusplus branch at https://github.com/crazywhalecc/static-php-cli/pull/214?
It contains a lot of fixes and might just implicitly tackle this too. It seems to be missing various system headers, but compiling gcc with musl-cross-make will produce new ones.
/usr/bin/ld: /tmp/ccAzYpTF.o: in function `main':
conftest.c:(.text+0x8): undefined reference to `__dns_search

and I remember something about dns problems with aarch64/gcc for static builds.
Either way I will see to get access to an arm64 system that I can test on next week.

I spun up an arm64 almalinux (minimal) vm, installed php, composer and zip, copied spc over, ran spc doctor -> spc build snappy and it worked.
[13:36:03] [INFO] [EXEC] strip --strip-all php-fpm
[13:36:03] [DEBU] Running command with direct output: cd '/opt/static-php-cli/source/php-src/sapi/fpm' && strip --strip-all php-fpm
[13:36:03] [INFO] Deploying fpm file
[13:36:03] [INFO] [EXEC] cp '/opt/static-php-cli/source/php-src/sapi/fpm/php-fpm' '/opt/static-php-cli/buildroot/bin/'
[13:36:03] [DEBU] Running command with direct output: cp '/opt/static-php-cli/source/php-src/sapi/fpm/php-fpm' '/opt/static-php-cli/buildroot/bin/'
[13:36:03] [INFO] Build complete, used 219.747 s !
[13:36:03] [INFO] Static php-fpm binary path: /opt/static-php-cli/buildroot/bin/php-fpm
[13:36:03] [DEBU] Making new directory recursive: /opt/static-php-cli/buildroot/license
[13:36:03] [INFO] License path: /opt/static-php-cli/buildroot/license/
[root@aarch64 static-php-cli]# uname -a
Linux aarch64 5.14.0-284.11.1.el9_2.aarch64 #1 SMP PREEMPT_DYNAMIC Tue May 9 09:48:54 UTC 2023 aarch64 aarch64 aarch64 GNU/Linux```

@DubbleClick How did you install musl? I used the doctor from the default branch.

From the c++ branch pull request (#214 ) it worked right away.

I don't have much free time this week and may not be able to work on it. I'll be diving into it over the weekend and next week.

No problem, I will be able to get Alpine support done this week so all that's left will be MacOS support.


vencord: 1.7.3 -> 1.7.4

Description of changes
https://github.com/Vendicated/Vencord/compare/v1.7.3...v1.7.4
Quite a few fixes
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

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

I accidentally pushed to my master branch. Ignore this PR

See https://github.com/NixOS/nixpkgs/pull/300240


font for whole app not working

hello, I've gone over the docs and tried everything but the fonts still don't show.
This is what I've done so far:

Copied and pasted the fonts to assets/fonts (mind you, if I double click the ttf in Android Studio, it says "Unable to open font barlowregular.ttf")

Application class -> added
ViewPump.init(ViewPump.builder()
.addInterceptor(new CalligraphyInterceptor(
new CalligraphyConfig.Builder()
.setDefaultFontPath("fonts/barlowregular.ttf")
.setFontAttrId(R.attr.fontPath)
.build()))
.build());

Added to every activity ->   @Override
protected void attachBaseContext(Context newBase) {
super.attachBaseContext(ViewPumpContextWrapper.wrap(newBase));
}

thanks for any help!

You tried with another ttf, ideally finding a "mobile" friendly one.