[Forge 1.20.1] Crash when Opening Preview Screen

When going to create a world and clicking onto the preview tab the game crashes. I have a pretty big mod list so if needed I can slowly go through them all and see if another mod is causing the crash, but heres the log for now:
latest.log

Looks like the incompatible mod is WorldEdit. Could you please confirm if this is the case? If there is another crash without WorldEdit, please attatch the new log file again.

Could you please check if this fixes the issue:
world_preview_1.0.1.tar.gz

GitHub closed this issue automatically with my commit. Please reopen if there are still problems.


Error on an old device ?

Hi
I've got an old device (no lock in the app and can't group it in heatzy app)
since the last update can't control from HA
here are the logs if any help :
Logger: homeassistant.core
Source: custom_components/heatzy/climate.py:115
Integration: Heatzy (documentation, issues)
First occurred: 18:20:00 (3 occurrences)
Last logged: 18:22:20
Error executing service: <ServiceCall homeassistant.turn_off (c:0fe37c73f17271f4fc771d77133f7ac2): entity_id=['switch.chauffage_coin']>
Error executing service: <ServiceCall homeassistant.turn_on (c:ff2114a420c7a2aa927ebe527649f213): entity_id=['switch.chauffage_coin']>
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/core.py", line 1511, in catch_exceptions
await coro_or_task
File "/usr/src/homeassistant/homeassistant/core.py", line 1530, in _execute_service
await handler.job.target(service_call)
File "/usr/src/homeassistant/homeassistant/components/homeassistant/init.py", line 121, in async_handle_turn_service
await asyncio.gather(*tasks)
File "/usr/src/homeassistant/homeassistant/core.py", line 1495, in async_call
task.result()
File "/usr/src/homeassistant/homeassistant/core.py", line 1530, in _execute_service
await handler.job.target(service_call)
File "/usr/src/homeassistant/homeassistant/helpers/entity_component.py", line 209, in handle_service
await self.hass.helpers.service.entity_service_call(
File "/usr/src/homeassistant/homeassistant/helpers/service.py", line 663, in entity_service_call
future.result()  # pop exception if have
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 896, in async_request_call
await coro
File "/usr/src/homeassistant/homeassistant/helpers/service.py", line 700, in _handle_entity_call
await result
File "/usr/src/homeassistant/homeassistant/components/template/switch.py", line 190, in async_turn_off
await self._off_script.async_run(context=self._context)
File "/usr/src/homeassistant/homeassistant/helpers/script.py", line 1260, in async_run
await asyncio.shield(run.async_run())
File "/usr/src/homeassistant/homeassistant/helpers/script.py", line 363, in async_run
await self._async_step(log_exceptions=False)
File "/usr/src/homeassistant/homeassistant/helpers/script.py", line 381, in _async_step
await getattr(self, handler)()
File "/usr/src/homeassistant/homeassistant/helpers/script.py", line 584, in _async_call_service_step
await service_task
File "/usr/src/homeassistant/homeassistant/core.py", line 1495, in async_call
task.result()
File "/usr/src/homeassistant/homeassistant/core.py", line 1530, in _execute_service
await handler.job.target(service_call)
File "/usr/src/homeassistant/homeassistant/helpers/entity_component.py", line 209, in handle_service
await self.hass.helpers.service.entity_service_call(
File "/usr/src/homeassistant/homeassistant/helpers/service.py", line 663, in entity_service_call
future.result()  # pop exception if have
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 896, in async_request_call
await coro
File "/usr/src/homeassistant/homeassistant/helpers/service.py", line 700, in _handle_entity_call
await result
File "/config/custom_components/heatzy/climate.py", line 115, in async_set_preset_mode
await self.coordinator.heatzy_client.control_device(
AttributeError: 'HeatzyClient' object has no attribute 'control_device'

I’ve the same problem

had to revert back to 4.6 to be able to control each heatzy

Fix with 5.3


faster_rcnn_resnet152'

can you please provide us with a link to download faster_rcnn_resnet152 backbone please

Hi Tekaya.
Sorry, my guess is as good as yours. I googled it and got a repository with a couple of models.
https://github.com/onnx/models
Best regards
Daniel


Completely remove OSG GUI

This PR completely removes the OSG gui. Leaves Magnum as the only GUI available. This is still WIP..
EDIT: This also replaces Sdl2Application with GlfwApplication which seems to be more stable.

@Aneoshun can you check if we need this with Magnum WindowlessGLApplication (just running the magnum_contexts example should suffice)? In my case, I didn't need it, but I want to be sure before removing it..

@Aneoshun @jbmouret let me know if this still works in Mac. Make sure that you update your installations (see the new installation.md in the remove_osg branch) and use the latest magnum master..
If this is still working on Mac, then I think we should be able to merge..

It does compile, but does not work. Apparently the shader compilation fails:
[robot_dart WARNING]: "Shadows are not working properly on Mac! Disable them if you experience unexpected behavior.."
Platform::GlfwApplication: warning: the executable is not a HiDPI-enabled app bundle
GL::Shader::compile(): compilation of fragment shader 2 failed with the following message:
ERROR: 3:187: storage modifier qualifier 'const' must precede precision qualifiers
ERROR: 3:192: storage modifier qualifier 'const' must precede precision qualifiers
ERROR: 3:197: storage modifier qualifier 'const' must precede precision qualifiers
ERROR: 3:204: Use of undeclared identifier 'finalAmbientColor'
ERROR: 3:262: Use of undeclared identifier 'finalDiffuseColor'
ERROR: 3:270: Use of undeclared identifier 'finalSpecularColor'
ERROR: 3:273: Use of undeclared identifier 'diffuseReflection'
Assertion Magnum::GL::Shader::compile({vert, frag}) failed in ../src/robot_dart/gui/magnum/gs/phong_multi_light.cpp on line 35
Abort trap: 6

I pulled, and rebuilt Corrade, Magnum, Magnum-plugin, Magnum-integration and robot_dart.

fyi:
OpenGL vendor string: ATI Technologies Inc.
OpenGL renderer string: AMD Radeon Pro 560X OpenGL Engine
OpenGL version string: 2.1 ATI-3.0.68
OpenGL shading language version string: 1.20

OpenGL vendor string: ATI Technologies Inc.
OpenGL renderer string: AMD Radeon Pro 560X OpenGL Engine
OpenGL version string: 2.1 ATI-3.0.68
OpenGL shading language version string: 1.20

This is a very old version of OpenGL (2.1). It's not possible to support it. We need OpenGL version bigger than 3.1...

According to: https://support.apple.com/en-us/HT202823
my MBP shoudl support OpenGL 4.1 and OpenCL 1.2...

According to: https://support.apple.com/en-us/HT202823
my MBP shoudl support OpenGL 4.1 and OpenCL 1.2...

Then why does it say OpenGL 2.1? I will check on a mac computer but this seems to me as a driver installation bug? Not related to robot_dart..

Hahah oh my ... this is endless :see_no_evil: You'll get 2.1 on Mac if you don't request core GL 3.2+ at least -- their driver either support the old pre-GL3 compat profile or core GL3+, but not both. So this seems to be one such case.
There was some change in Catalina that might have broken this (I heard some reports that GL3.3 at least needs to be requested, otherwise you get 2.1, previously 3.2 was enough) -- did this work with Sdl2Application or is Sdl2App affected the same way? Random patch that might make things different:
diff --git a/src/Magnum/Platform/GlfwApplication.cpp b/src/Magnum/Platform/GlfwApplication.cpp
index ee9f3a17d..f76711c6e 100644
--- a/src/Magnum/Platform/GlfwApplication.cpp
+++ b/src/Magnum/Platform/GlfwApplication.cpp
@@ -384,7 +384,7 @@ bool GlfwApplication::tryCreate(const Configuration& configuration, const GLConf
            compatibility contexts (which are the default). Unlike SDL2, GLFW
            requires at least version 3.2 to be able to request a core profile. */
         glfwWindowHint(GLFW_CONTEXT_VERSION_MAJOR, 3);
-        glfwWindowHint(GLFW_CONTEXT_VERSION_MINOR, 2);
+        glfwWindowHint(GLFW_CONTEXT_VERSION_MINOR, 3);
         glfwWindowHint(GLFW_OPENGL_PROFILE, GLFW_OPENGL_CORE_PROFILE);
         glfwWindowHint(GLFW_OPENGL_FORWARD_COMPAT, glFlags >= GLConfiguration::Flag::ForwardCompatible);
         #else

Well, actually, I am not sure that I tested the shadows when we were still on SDL2, so I can't tell.
I tried your patch and I don't see any difference.
The version info that I pasted earlier come from glxinfo, which might be the most accurate source of info on OSX (e.g., it starts a XQuartz server...).

Oh! What's the startup log from the engine, then? Does it show GL 2.1 as well? I don't trust glxinfo, especially on platforms where X11 is not native. GLFW uses CGL there, not GLX.

This is what the executable gives when I comment Corrade::Utility::Debug magnum_silence_output{nullptr};   in graphics.hpp
git/robot_dart$ ./build/meshes
[robot_dart WARNING]: "Shadows are not working properly on Mac! Disable them if you experience unexpected behavior.."
Platform::GlfwApplication: warning: the executable is not a HiDPI-enabled app bundle
Renderer: AMD Radeon Pro 560X OpenGL Engine by ATI Technologies Inc.
OpenGL version: 4.1 ATI-3.0.68
Using optional features:
    GL_ARB_ES2_compatibility
    GL_ARB_separate_shader_objects
    GL_ARB_texture_filter_anisotropic
    GL_ARB_texture_storage
    GL_ARB_vertex_array_object
    GL_EXT_debug_label
    GL_EXT_debug_marker
Using driver workarounds:
    no-layout-qualifiers-on-old-glsl
GL::Shader::compile(): compilation of fragment shader 2 failed with the following message:
ERROR: 3:187: storage modifier qualifier 'const' must precede precision qualifiers
ERROR: 3:192: storage modifier qualifier 'const' must precede precision qualifiers
ERROR: 3:197: storage modifier qualifier 'const' must precede precision qualifiers
ERROR: 3:204: Use of undeclared identifier 'finalAmbientColor'
ERROR: 3:262: Use of undeclared identifier 'finalDiffuseColor'
ERROR: 3:270: Use of undeclared identifier 'finalSpecularColor'
ERROR: 3:273: Use of undeclared identifier 'diffuseReflection'
Assertion Magnum::GL::Shader::compile({vert, frag}) failed in ../src/robot_dart/gui/magnum/gs/phong_multi_light.cpp on line 35
Abort trap: 6

(I haven't reverted the patch introduced above)

In the shader, I am requiring OpenGL 3.2 and some additional features. Maybe that's the issue @mosra ? Should I require 3.3?

Great! Then GL 2.1 was just a false alarm (and glxinfo is thus wrong). I expect it to work the same with or without the patch, considering I didn't get any recent reports about apps/examples refusing to work on Mac.
The remaining error is that const can be used for non-compile-time constants only since GLSL 4.20, and here you can get 4.1 at most. So removing the consts from here should make things work: https://github.com/resibots/robot_dart/blob/d8a81f77dbf275dbdd72d76d8180da1a2e11a4e1/src/robot_dart/gui/magnum/resources/PhongMultiLight.frag#L175-L189
@costashatz what's the reason for fully disabling magnum's startup log? it provides very useful info for debugging ;)

@costashatz what's the reason for fully disabling magnum's startup log? it provides very useful info for debugging ;)

When running things in parallel with multiple contexts, I get the log information too many times! Of course, I could just disable it for each executable that needs to run in the cluster.. I will think about adding it back.. :)

@Aneoshun did 388b2d5 fix your issues?

@costashatz Yes it does! I now have a robot, with shadows and ground!

@costashatz Yes it does! I now have a robot, with shadows and ground!

Great! Then I will merge soon! @jbmouret let me know if you have any comment/issue..

The waf configure does not work for me anymore.
./waf configure    --controller=/Users/jmouret/git/resibots/hexapod_common/hexapod_controller --magnum_install_dir=/usr/local/include
Setting top to                           : /Users/jmouret/git/resibots/robot_dart
Setting out to                           : /Users/jmouret/git/resibots/robot_dart/build
Checking for 'clang++' (C++ compiler)    : /usr/bin/clang++
Checking for 'clang' (C compiler)        : /usr/bin/clang
Checking boost includes                  : 1_71
Checking boost libs                      : ok
Checking for Eigen                       : /usr/local/include/eigen3
Checking for DART includes (including io/urdf) : 6.10.0 in /usr/local/include
Checking for DART libs (including io/urdf)     : ['dart', 'dart-utils', 'dart-utils-urdf']
DART: Checking for Assimp                      : /usr/local/include
DART: Checking for Bullet Collision libs       : libs: ['LinearMath', 'BulletCollision', 'dart-collision-bullet'], bullet: /usr/local/include/bullet
DART: Checking for Ode Collision libs          : Not found
Checking for hexapod_controller includes       : /Users/jmouret/git/resibots/hexapod_common/hexapod_controller/include
Checking for Corrade includes                  : /usr/local/include
Getting Corrade configuration                  : ['BUILD_DEPRECATED', 'TARGET_UNIX', 'TARGET_APPLE']
Checking for Corrade components                : ['CorradePluginManager', 'CorradeUtility', '/usr/local/bin/corrade-rc']
Checking for Magnum includes                   : /usr/local/include
Checking for Magnum lib                        : ['Magnum']
Getting Magnum configuration                   : ['BUILD_DEPRECATED', 'BUILD_MULTITHREADED', 'TARGET_GL']
Magnum: Checking for OpenGL includes           : /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/System/Library/Frameworks/OpenGL.framework/Versions/A/Headers
Magnum: Checking for OpenGL lib                : ['GL']
Magnum: Checking for MagnumGL lib              : ['MagnumGL']
Checking for Magnum components                 : Not found
Magnum needs to be configured! Cannot proceed!
Magnum needs to be configured! Cannot proceed!
AVX compilation of dart                        : yes
AVX compilation of dart-utils                  : yes
AVX compilation of dart-utils-urdf             : yes
-march=native (AVX support)                    : yes
['-Wall', '-O3', '-g', '-faligned-new', '-march=native', '-std=c++14']
'configure' finished successfully (0.932s)

=> There is an issue with the magnum components (but magnum, corrade, etc. are OK). Any idea what is wrong? (the old 'magnum' branch binaries still work!). Also, would it be possible to print which component is missing? I guess only one of them is not detected.

=> There is an issue with the magnum components (but magnum, corrade, etc. are OK). Any idea what is wrong? (the old 'magnum' branch binaries still work!). Also, would it be possible to print which component is missing? I guess only one of them is not detected.

We no longer are using Sdl2, but Glfw3. Can you update your installation of Magnum following these instructions: https://github.com/resibots/robot_dart/blob/remove_osg/docs/installation.md#installing-magnum?

Thanks! It works now.

Thanks! It works now.

Merging then... Thanks for all the feedback!


Skia->Engine roller blocked on displaylist failures

Example failure: https://github.com/flutter/engine/pull/47467
log
FAILED TESTS (10/710):
      34 ms: /Volumes/Work/s/w/ir/cache/builder/src/out/host_debug/display_list_unittests DisplayListTest.SingleOpsMightSupportGroupOpacityBlendMode (try #1)
      38 ms: /Volumes/Work/s/w/ir/cache/builder/src/out/host_debug/display_list_unittests DisplayListTest.SingleOpsMightSupportGroupOpacityBlendMode (try #2)
      26 ms: /Volumes/Work/s/w/ir/cache/builder/src/out/host_debug/display_list_unittests DisplayListTest.SingleOpSizes (try #1)
      30 ms: /Volumes/Work/s/w/ir/cache/builder/src/out/host_debug/display_list_unittests DisplayListTest.SingleOpSizes (try #2)
      26 ms: /Volumes/Work/s/w/ir/cache/builder/src/out/host_debug/display_list_unittests DisplayListComplexity.DrawTextBlob (try #1)
      27 ms: /Volumes/Work/s/w/ir/cache/builder/src/out/host_debug/display_list_unittests DisplayListComplexity.DrawTextBlob (try #2)
      26 ms: /Volumes/Work/s/w/ir/cache/builder/src/out/host_debug/display_list_unittests DisplayListTest.SingleOpDisplayListsNotEqualEmpty (try #1)
      25 ms: /Volumes/Work/s/w/ir/cache/builder/src/out/host_debug/display_list_unittests DisplayListTest.SingleOpDisplayListsNotEqualEmpty (try #2)
      29 ms: /Volumes/Work/s/w/ir/cache/builder/src/out/host_debug/display_list_unittests DisplayListTest.SingleOpDisplayListsCompareToEachOther (try #2)
      34 ms: /Volumes/Work/s/w/ir/cache/builder/src/out/host_debug/display_list_unittests DisplayListTest.SingleOpDisplayListsCompareToEachOther (try #1)
SUMMARY:
  /Volumes/Work/s/w/ir/cache/builder/src/out/host_debug/display_list_unittests DisplayListComplexity.DrawTextBlob passed 0 / 2 times.
  /Volumes/Work/s/w/ir/cache/builder/src/out/host_debug/display_list_unittests DisplayListTest.SingleOpDisplayListsCompareToEachOther passed 0 / 2 times.
  /Volumes/Work/s/w/ir/cache/builder/src/out/host_debug/display_list_unittests DisplayListTest.SingleOpDisplayListsNotEqualEmpty passed 0 / 2 times.
  /Volumes/Work/s/w/ir/cache/builder/src/out/host_debug/display_list_unittests DisplayListTest.SingleOpSizes passed 0 / 2 times.
  /Volumes/Work/s/w/ir/cache/builder/src/out/host_debug/display_list_unittests DisplayListTest.SingleOpsMightSupportGroupOpacityBlendMode passed 0 / 2 times.

example failure
[411/710] DisplayListTest.SingleOpDisplayListsCompareToEachOther (29 ms)
[INFO:test_timeout_listener.cc(76)] Test timeout of 300 seconds per test case will be enforced.
�[0;33mNote: Google Test filter = DisplayListTest.SingleOpDisplayListsCompareToEachOther
�[m�[0;32m[==========] �[mRunning 1 test from 1 test suite.
�[0;32m[----------] �[mGlobal test environment set-up.
�[0;32m[----------] �[m1 test from DisplayListTest
�[0;32m[ RUN      ] �[mDisplayListTest.SingleOpDisplayListsCompareToEachOther
../../flutter/display_list/display_list_unittests.cc:653: Failure
Value of: listA->Equals(*listB)
  Actual: true
Expected: false
DrawTextBlob(variant 1 ==? variant 2)
�[0;31m[  FAILED  ] �[mDisplayListTest.SingleOpDisplayListsCompareToEachOther (1 ms)

cc @flar

The SingleOpDisplayListsCompareToEachOther hasn't changed in over a year: https://github.com/flutter/engine/blame/0b6d9e13f1b28b70dd94f18bb7f3aa2a14f6a387/display_list/display_list_unittests.cc#L625

Here are the 2 commits in that roll: https://skia.googlesource.com/skia.git/+log/817fca468240..bca43ec489e5
One has to do with canvas kit, the other has to do with text.  It seems unlikely that those changes would cause failures.

The whole display_list portion of the repository hasn't changed since oct 20: https://github.com/flutter/engine/commits/0b6d9e13f1b28b70dd94f18bb7f3aa2a14f6a387/display_list

This failure sounds more linked to the text change:
[ RUN      ] DisplayListTest.SingleOpsMightSupportGroupOpacityBlendMode
../../flutter/display_list/display_list_unittests.cc:1041: Failure
Expected equality of these values:
  display_list->can_apply_group_opacity()
    Which is: true
  expect_for_op
    Which is: false
{
  receiver.drawTextBlob(TestBlob1, 0, 0);
}
../../flutter/display_list/display_list_unittests.cc:1054: Failure
Expected equality of these values:
  display_list->can_apply_group_opacity()
    Which is: true
  expect_with_kSrc
    Which is: false
{
  receiver.setBlendMode(kSrc);
  receiver.drawTextBlob(TestBlob1, 0, 0);
}
[  FAILED  ] DisplayListTest.SingleOpsMightSupportGroupOpacityBlendMode (0 ms)

I was able to verify locally that bca43ec489e5fa23c08f598b5747a6e83ef04277 is causing the failures.

I reached out to @kjlubick and he said that this is because flutter engine is using skia's BUILD.gn files directly.  Specifically he changed the SK_DEFAULT_TYPEFACE_IS_EMPTY for skia and surprisingly to him it changed flutter's behavior.

Fix incoming in https://skia-review.googlesource.com/c/skia/+/771703

@flar made a good point offline that those tests were flimsy to be testing with a default font anyway and filed an issue to address that: https://github.com/flutter/flutter/issues/137565

This isn't showing up anymore.  However there is another issue now manifesting.  I'll file a new issue.  Thanks for the help on this one everyone.


Expose spec values?

In Pulumi or even Terraform, it's possible to make references to .spec body. Looks like it's not possible to do this with cdk8s. For example:
const deploy = new KubeDeployment(this, 'deployment', {
    spec: {
        replicas: 2,
        selector: {
            matchLabels: label,
        },
        template: {
            metadata: { labels: label },
            spec: {
                containers: [
                    {
                        name: 'hello-kubernetes',
                        image: 'paulbouwer/hello-kubernetes:1.7',
                        ports: [{ containerPort: 8080 }],
                    },
                ],
            },
        },
    },
});

const svc = new KubeService(this, 'service', {
    spec: {
        type: 'LoadBalancer',
        ports: [{ port: 80, targetPort: IntOrString.fromNumber(8080) }], // We're kinda hardcoding port value here
        selector: label,
    },
});

instead we could do something like this if .spec is available
const svc = new KubeService(this, 'service', {
    spec: {
        type: 'LoadBalancer',
        ports: [{ port: 80, targetPort: deploy.spec.template.spec.containers[0].ports[0] }],
        selector: label,
    },
});

This serves 2 purposes,

Make implicit dependency between resources. In this case service is dependent on deployment.
Avoid hardcoding values as much as possible.

Is this technically possible to achieve this? Or it's not possible at all?

@shinebayar-g See https://github.com/cdk8s-team/cdk8s/issues/918#issuecomment-1378423591

Duplicate of https://github.com/cdk8s-team/cdk8s-core/issues/36


Op 7699 - bug fix

We prefer small, well tested pull requests.
Please refer to Contributing to Spinnaker.
When filling out a pull request, please consider the following:

Follow the commit message conventions found here.
Provide a descriptive summary for your changes.
If it fixes a bug or resolves a feature request, be sure to link to that issue.
Add inline code comments to changes that might not be obvious.
Squash your commits as you keep adding changes.
Add a comment to @spinnaker/reviewers for review if your issue has been outstanding for more than 3 days.

Note that we are unlikely to accept pull requests that add features without prior discussion. The best way to propose a feature is to open an issue first and discuss your ideas there before implementing them.
https://devopsmx.atlassian.net/browse/OP-7699

This PR  triggered this build in this execution, please merge to master only if this succeeds:  https://spin.prod.opsmx.com/#/applications/gitprtrigger/executions/details/01FHTD9Z3XD32RAYSMCKS6DQF7


ascii-magician plugin

Repository URL
https://github.com/yakirk/ascii-magician-plugin
New Repository Name
ascii-magician-plugin
Description
A Jenkins plugin that adds a touch of magic to your build output by printing random ASCII art of Jenkins in the console.
The Ascii Art Magician plugin is a fun and easy way to add some personality to your Jenkins builds.
GitHub users to have commit permission
@yakirk
Jenkins project users to have release permission
kadkoda
Issue tracker
GitHub issues

Security audit, information and commands
The security team is auditing all the hosting requests, to ensure a better security by default.
This message informs you that the security team was notified about the request and will soon participate in this issue to assist.
The team is usually starting by a quick superficial audit and if it's not sufficient, they are planning a deeper audit.
Commands
Security team only:

/audit-ok => the audit is complete, the hosting can continue :tada:.
/audit-skip => the audit is not necessary, the hosting can continue :tada:.
/audit-required => the superficial audit was not sufficient, a deeper look is necessary :mag:.
/audit-findings => the audit reveals some issues that require corrections :pencil2:.

Anyone:

/audit-review => the findings from the audits were corrected, this command will ping the security team to review the findings :eyes:.
	It's only applicable when the previous audit required changes.

Only one command can be requested per comment.

(automatically generated message)

From a security perspective, everything is fine 👍
/audit-ok

Hey @yakirk,
looks good so far, but I would recommend moving from main to a less generic name. Preferably your plugin name.
https://github.com/yakirk/ascii-magician-plugin/tree/main/src/main/java/io/jenkins/plugins/main

Hey @NotMyFault - Thanks!
I've fixed the recommendation.

/hosting host


Change shell option from "--url" to "--from" in "catalog add" command

This should be done to be consistent with usage is other commands, from can be a name (translated to a url) or a url.
Check docs to look for usage.

I think it is consistent now the way it is.
catalog add --name mycatalog --url https://blah.com/mycatalog
project add --name mycoolapp --url http://blah.com/mycoolapp

boot new --from jpa
boot new --from http://blah.com/jpa
boot add --from scheduling
boot add --from http://blah.com/scheduling

--from   is 'name or URL'
--url    is 'URL', name is not allowed.

Use the option name --from when accepting a name or a url and use --url when accepting only a url


does not work well with zsh...

I really love this pretty app..
but in my zsh , when I input a command like puthon, it told me
puthon
zsh: command not found: puthon

so thefuck seems did not work... Could you tell  me how can I configure zsh ?

Yes.

I am using Mac
wmn@cratos ~/code/src/github.com/carlhuda/janus [master] ± % uname -a
Darwin cratos.local 14.1.0 Darwin Kernel Version 14.1.0: Thu Feb 26 19:26:47 PST 2015; root:xnu-2782.10.73~1/RELEASE_X86_64 x86_64
wmn@cratos ~/code/src/github.com/carlhuda/janus [master] ± % git --version
git version 2.3.5
wmn@cratos ~/code/src/github.com/carlhuda/janus [master] ± % zsh --version
zsh 5.0.7 (x86_64-apple-darwin13.4.0)
wmn@cratos ~/code/src/github.com/carlhuda/janus [master] ± % which zsh
/usr/local/bin/zsh

For git brnch, it only works if git config --global help.autocorrect is set to 0
wmn@cratos ~/code/src/github.com/carlhuda/janus [master] ± % git config --global help.autocorrect
1
wmn@cratos ~/code/src/github.com/carlhuda/janus [master] ± % git brnch
WARNING: You called a Git command named 'brnch', which does not exist.
 See 'git --help'.

Is it repeats with 1.23+?

works now for me!

Add alias in .zshrc; works for me:
alias fuck='$(thefuck $(fc -ln -1 | tail -n 1)); fc -R'


Init React App

Initialize React app in project

Wait one, second, this should probably be in another folder. We probably need a client and server folder.

Wait one, second, this should probably be in another folder. We probably need a client and server folder.

yeah this actually crossed my mind after making the PR 😅


How to use Youtube-dl to download dynamic m3u8?

Checklist

[x] I'm asking a question
[x] I've looked through the README and FAQ for similar questions
[x] I've searched the bugtracker for similar questions including closed ones

Question

Edit: Sorry for skipping the form. Never happened again.
Hi guys,
First of all, let me appreciate your incredible work. Been using it for a while and it's really useful.
I'm trying to download a video using Youtube-dl. Sadly the m3u8 changes every 20 seconds. Fortunately, I managed to renew the m3u8 link using PHP and Now the question is, possible to make something like a func/script in order to pass those m3u8 urls to Youtube-dl and then expect Youtube-dl to download all the segments?
To be more clear, I can renew the m3u8 links but still looking for a way to ask Youtube-dl to replace the m3u8 link automatically after being expired.

I've just edited the issue. Please open it again. Thank you.


After the IOU is created, the Tax rate changes to 0%

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: v1.4.69-0
Reproducible in staging?: Y
Reproducible in production?: Y
Found when executing PR : https://github.com/Expensify/App/pull/41264
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Logs: https://stackoverflow.com/c/expensify/questions/4856
Issue reported by: Applause-Internal team
Action Performed:

Open https://staging.new.expensify.com/
Log in to your HT account
Create a WS
Enable "Taxes"
Set "Tax Rate 1" as Workspace currency default
Navigate to the WS room
Create a manual IOU with default currency and Tax Rate 1
Navigate to the details of the created IOU

Expected Result:
After the IOU is created, the Tax rate must remain the same as when the IOU was created
Actual Result:
After the IOU is created, the Tax rate changes to 0%
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
https://github.com/Expensify/App/assets/115492554/666c613e-a334-4107-ae30-0ff6a03f61ad

View all open jobs on GitHub

We think this issue might be related to the #collect project.

CC: @MonilBhavsar is this on your radar?

Yes, we're already working on this issue in this PR https://github.com/Expensify/App/pull/40443
Let's put this on HOLD

Sounds good! Bumping down to Weekly.

We've merged the PR. This is fixed on main branch


Allow updates to existing winget manifests

We recently received notification from a customer that installing
microsoft/git via winget was broken due to a signing issue. This happened
because we accidentally deployed an incorrect SHA for the microsoft/git
version <IP_ADDRESS>.0 manifest on microsoft/winget-pkgs then replaced
the asset on GitHub with a new version with a new SHA.
When we tried to re-run the update-winget workflow
to correct the SHA, we discovered some issues with the action that make it
impossible to update existing manifests. This PR updates the action to allow
us to both create new winget manifests and update existing ones.
Here is an example of a test PR opened to update an existing manifest with these changes.
And here is an example of a test PR that creates a new manifest with these changes.

I'm seeing some weirdness where compiling with npm run all is including formatting changes to main.ts. I am not sure whether including them matters, but I'm leaving them in this PR for now to be safe. @mjcheetham if you think I should remove, I'm more than happy to do so though.


cursorWordPartLeftSelect and cursorWordPartRightSelect keyboard shortcut not working in renaming file

VSCode Version: Version: 1.31.0 (system setup)
Commit: 7c66f58312b48ed8ca4e387ebd9ffe9605332caa
Date: 2019-02-05T22:35:56.624Z
Electron: 3.1.2
Chrome: 66.0.3359.181
Node.js: 10.2.0
V8: 6.6.346.32
OS: Windows_NT x64 10.0.10586
OS Version: Windows 10 Pro

Steps to Reproduce:
I set my keyboard shortcut for cursorWordPartLeftSelect  and cursorWordPartRightSelect.
Then I rename my PascalCase file and use the keyboard shortcut and it didn't work.
Does this issue occur when all extensions are disabled?: Yes

The rename input box is a native browser input, which does not implement sub-word navigation.


Add support for Powerpoint, Word doc, PDF, Outlook email, ZIP, GZ, BZ2

Fixes https://github.com/ebyhr/presto-flex/issues/16

Please remove the redundant first commit and resolve conflicts.

found some bugs in the readers


Fix ui defects

Fixing of OSC UI defect #418
Description
Fixed the OSC UI defect for displaying selection type option parameters correctly,
Motivation and Context

fixes issue https://github.com/opensecuritycontroller/osc-core/issues/418
How Has This Been Tested?
Tested manually
Screenshots (if appropriate):
Types of change

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

Checklist:

[X] My code follows the Code style guidelines of this project
[ ] My unit test follow the Unit test guidelines
[X] Unit test coverage percentage is maintained(increased/remains the same but not decreased)

@pravinrajr9 not sure how this fixes the original issue. The original issue is about the text Vm -> VM

@arvindn05 I added this parameter assuming it could reflect the changes immediately since everything was fine in the code. But it worked fine for older version of osc-ui, where I was able to reproduce the issue.  I am not able to reproduce this defect in latest versions of osc-ui. We can close this bug.  It may be a good practice to add this parameter since we want listener to get called immediately for workload selection type, this pull request may be irrelevant to this defect as it can not be reproduced in newer osc versions.

https://vaadin.com/forum/#!/thread/2983812/3070078
@pravinrajr9 Immediate has the cost of as soon as the field changes value, it is sent to the server.
I think here the combo box results in openstack calls which are expensive. Not having it immediate is better in this case as we make the call only after the user moves away from the combobox.

@pravinrajr9 Please close the pull request as it seems this is not needed for the bug fix. You can mark the bug as non-issue and post a screenshot there as collateral.


PR testing not working

I recently changed the version branches of the releases in the rosdistro and the tracks.
But PR testing is still not working.
Dummy PR to cross check that is here
Any suggestions on what I might be missing here ?

I think that you were using the wrong verbiage I retried with @ros-pull-request-builder retest this please and it triggered.  As documented here: http://wiki.ros.org/buildfarm/Pull request testing

The other two triggers weren't hit as the PR was already existing before the job reconfigured at 11pm on the 6th. As well the commits were at 9pm on the 6th.


Cannot sopport  element?

I converted a SVG file to excalidraw JSON, but the  elm were all lost. But i floud ExcalidrawTextElement in the code. So how i need to do to support the  elm? Tks.

my problem too, is there any solution?


dependency volume changes when source code is changed

Problematic line. I'm working on WSS with mongodb, when I make a change on source code of WSS, mongodb's data volume also changes and I lose all the previous data.
/cc @antho1404

I was thinking about this one too when I worked on the influxdb service and yes it can be annoying. In some cases it's actually a good thing to have this all automatic, but in some other cases it's really annoying so maybe it should be the responsibility of the user and have a parameter "bind" or something like that.

Also the line you are pointing is really risky to remove because what happen if 2 services have the same name of dependency and same name of volume eg: db -> db that might be a big problem because data will be shared

I think the update option is not really related and I think this issue will solve the problem https://github.com/mesg-foundation/core/issues/275 but until then I really think we could either force to bind the volume in the .mesg folder based on the name like we had before or even make an option for that.
The final solution is really to work on this "alias/ID" for service that will always be static and we can use that as identifier for the volume and this will not change anymore.

@antho1404 please confirm and close this issue in favor of forum proposal.

This is still a bug that can be solved by the service ID feature but it should be kept open as reminder

can't we for now revert to the initial system that was doing the binding all the time of the different volumes on the host disk based on the name of the service (with dash).
This will solve the problem and we can later on use the alias/id of the service.
What do you think ?

@antho1404 you mean using the service name in the yaml instead its hash as volume name?

yes
but even this might not be enough because with this we still rely on docker to create volumes but we cannot bind these volumes to the actual host so maybe using the name instead of the hash AND binding this to the host in the ~/.mesg/services/{serviceName}/{dependencyName}/{volumePath}

I agree, but I'd like to keep Docker volumes instead of binding volumes to host machine for isolation.
In future we can have new features like volume plugins, providing data backups to users or easily moving core, its whole stack and data volumes to another machine. Having data in Docker volumes may give us more flexibility for these kind of use cases.
Is there a reason to the bind volumes to host machine?

I totally agree that binding volumes is not an ideal solution and docker volumes are better but there is some cases where binding can be helpful.
One example is to inject configurations in the service.

We have for example @krhubert who is working on the integration with tendermint and needs to have the genesis file sent to the service (workaround is to have it created in the service with a special script)
The ethereum service that require an endpoint and/or a network to connect to that could be sent with volumes
Like you said doing some backup or using existing tools on their host machine

I agree that I would like to keep docker volumes here and not binding, maybe we can have an option here and it would be an undocumented feature for really advanced services. Services shouldn't be configured by the user anyway.
What do you think about that?

I totally see your point and sharing networks/files from host machine to services is an actual need. On the other hand we still should be using data volumes.
In this case we need to both support sharing networks/files and having data kept in docker volumes.
I suggest to get some insights from docker-compose too see how they handle this cases.
For data volumes they have this syntax:
https://docs.docker.com/compose/compose-file/#volume-configuration-reference
version: "3"

services:
  db:
    image: db
    volumes:
      - data-volume:/var/lib/db
  backup:
    image: backup-service
    volumes:
      - data-volume:/var/lib/backup/data

volumes:
  data-volume:

This one demonstrates how network/files are shared:
https://github.com/libresh/compose-nginx/blob/master/docker-compose.yml#L18-L20
Note that ./nginx is an actual folder in their repo.
nginx-gen:
  ...
  volumes_from: [nginx]
  volumes:
    - ./nginx/templates/nginx.tmpl:/etc/docker-gen/templates/nginx.tmpl:ro
    - /var/run/docker.sock:/tmp/docker.sock:ro

I'm not so sure about exposing an undocumented advanced feature if we can avoid its need. It should be our last choice.

The latest messages are off-topic and the conversation continued on the PR #571

this one should be solved by the PR #583 volumes will be binded to the alias and not the hash anymore so whatever the version of the service, the alias stays the same and so the volume


A youki member to join containers member

Hi, @rhatdan, @saschagrunert
I'd like to invite @Furisto to the containers member. WDYT?
He is a active contributor of  containers/oci-spec-rs, containers/youki and `containers/containrs . He is youki's number one contributor. Of course, he has interested in containers member. There is no particular change, but I do not think there is any room for denial in my opinion for him to become a member of this group.

+1 from my side

Hi, @rhatdan. WDYT?

Thank you everyone! 🙏

@lsm5 Thanks!! Nothing for now.


Roll engine 18d74fe45b9c..fad1b23c42b9 (1 commits)

git@github.com:flutter/engine.git/compare/18d74fe45b9c...fad1b23c42b9
git log 18d74fe45b9c..fad1b23c42b9 --first-parent --oneline
2019-11-27<EMAIL_ADDRESS>Roll fuchsia/sdk/core/linux-amd64 from xyyOR... to mSEnz... (#14043)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

Auto-roller completed checks. About to merge.


torizon-core-common: add Alsa UCM configuration

Due to BSP changes, ALSA configuration files were reworked into a new recipe.
Removing old config:
https://git.toradex.com/cgit/meta-toradex-nxp.git/commit/?h=kirkstone-6.x.y&id=54721dc86ece6b41d88f5e9de4262b575efc5393 https://git.toradex.com/cgit/meta-toradex-ti.git/commit/?h=kirkstone-6.x.y&id=d5d48dc1cc1ab54b3a498a0d5ea19646cab381ee
Adding new UCM config:
https://git.toradex.com/cgit/meta-toradex-bsp-common.git/commit/?h=kirkstone-6.x.y&id=5183100ea2bb3fe29702f96657e05f4e905433b5

From what I can understand this new alsa-ucm-conf-tdx recipe is only providing configs for the verdin boards right? So optimally we should only be installing this when building for those machines right?
Though I see the BSP installs this recipe unconditionally in their reference images so maybe it's not a big deal: https://git.toradex.com/cgit/meta-toradex-demos.git/commit/?h=kirkstone-6.x.y&id=d7a89bf699d730e3f6081c4da12346d78f6fd8b6
Your thoughts?

From what I can understand this new alsa-ucm-conf-tdx recipe is only providing configs for the verdin boards right? So optimally we should only be installing this when building for those machines right?
Though I see the BSP installs this recipe unconditionally in their reference images so maybe it's not a big deal: https://git.toradex.com/cgit/meta-toradex-demos.git/commit/?h=kirkstone-6.x.y&id=d7a89bf699d730e3f6081c4da12346d78f6fd8b6
Your thoughts?

Makes sense. I've updated the commit.

From what I can understand this new alsa-ucm-conf-tdx recipe is only providing configs for the verdin boards right? So optimally we should only be installing this when building for those machines right?

Even though I agree this would be optimal, it seems to me this should be a concern for the alsa-ucm-conf-tdx recipe. Otherwise, on every update of the recipe we'd need to re-evaluate our decision on only including the package for verdins and this would probably make us miss it when the BSP adds configs for other machines; well, unless we know there won't be configs for other machines...

From what I can understand this new alsa-ucm-conf-tdx recipe is only providing configs for the verdin boards right? So optimally we should only be installing this when building for those machines right?

Even though I agree this would be optimal, it seems to me this should be a concern for the alsa-ucm-conf-tdx recipe. Otherwise, on every update of the recipe we'd need to re-evaluate our decision on only including the package for verdins and this would probably make us miss it when the BSP adds configs for other machines; well, unless we know there won't be configs for other machines...
P.S.: @EdTheBearded Sorry, I'm commenting this after you updated the PR.

It's ok.
I've changed it back as well. As you said, this isn't a recipe specific for Verdin (although the installation of the configs should), and if they add more devices in the future, we're already covered.

Tagging @hiagofranco who worked on the ALSA stuff as far as I know, maybe he can help provide some background if needed.

Hi all,
this was done only to Verdin for now because this was the modules being affected by the read-only-rootfs breaking Yocto builds because of different codecs between Dahlia and Dev boards.
> cat recipes-bsp/alsa-state/alsa-state.bbappend

FILESEXTRAPATHS:prepend := "${THISDIR}/alsa-state/:"

# make this machine specific, as we have different codecs with different settings
PACKAGE_ARCH:tdx = "${MACHINE_ARCH}"
> ls recipes-bsp/alsa-state/alsa-state
 apalis-imx6   apalis-imx8   colibri-imx6   colibri-imx7   colibri-imx7-emmc

The alsa bbappend is still there under meta-toradex-nxp layer and it will make Apalis and Colibri work for now. In the future, we will also remove these files and add UCM files for all boards, which you will get for free.
This is current on our backlog and being tracked under ELB-5712. Please feel free to add any comments or any suggestions, or any reccomendations for me, thanks!

@hiagofranco Thank you for clarifying! Guess we'll go ahead with the merge then.


Split up the best practice guides

Contributions

[X] I have read the contribution docs, and understand what's expected of me.

Link to the page on docs.getdbt.com requiring updates
https://docs.getdbt.com/docs/guides/best-practices
What part(s) of the page would you like to see updated?
I think this page should be split up into:

Best practices for projects
Best practices for workflows

We can add the pro-tips in these guides or possibly create new guides for super users:

Using workflows like a pro (not married to this title though I do appreciate that it feels more accessible than "Advanced blah blah blah")
Using projects like a pro

Additional information
No response

Closing because guides are categorized


Delete a contact using Python

Code sample to delete a contact using Nylas Python SDK.

Hey @relaxedtomato I would like to contribute to this. Can you please assign it to me!

@SouvikDcoder There you go 😉

Done and merged.


websocket panel blank

hey,
first of all - the test page (http://janodvarko.cz/test/websockets/) works perfectly.
in a site I am developing, the websocket panel is just blank.
My socket is using wss (secure websocket), and I tried canceling it, but that did not resolve the issue.
My socket is also not using a standard port, may that be the issue?
I also thought that different domains in the client and the socket could be an issue, but the test page also has a different domain.

started working. have no idea what happed :)

Hi, thanks for posting. If it happens again, please do the following:

Open the browser console (Ctrl+Shift+J)
Close (if open) and re-open devtools in the browser (F12)
Clear the browser console
Click Web Sockets in devtools
Post any errors in the browser console to a new issue.


[feature] enable dynamic tune emitPath of file

if (config.emitPath) {
        // support functions as emitPathto generate them dynamically
        url=    typeof config.emitPath=== "function"
                 ? config.emitPath(url)
                 : url;
    }

before emit

@Delagen can your explain use case?

@evilebottnawi for example to tune paths, that not child of context folder, or resolved somewhere else. I fork local copy of this module to strip node_modules from generated urls

@Delagen why your strip node_modules from generated urls? In production or dev?

@evilebottnawi I want to strip module path from production bundles, but save original filenames

@Delagen can your provide minimum reproducible test repo? Without configuration it's hard to say how you can solve this problem and how it will be solved correctly

For example:
tree
src
node_modules/common-lib

webpack context is src
when file required from common-lib
it placed as /node_modules/common-lib/..../file.html for example
and as more as ../../../ in relative path from webpack context, the more // in urls

@Delagen it is expected behavior, just use name options for production env (for avoid this situation), for dev env it is normal.

@evilebottnawi I want to preserve some path info in production environment.
Filename can be the same, but I don't want to include hash in filename

@Delagen also your can use outputPath as function (https://github.com/webpack-contrib/file-loader/blob/master/src/index.js#L58) and do what your want with paths

@evilebottnawi ok, but it's be tricky to detect needed files and what to do with urls in generated code?

@Delagen sorry for delay on answer, problem still exists? outputPath and publicPath can be function and you can control and url and place where emit file(s). But i recommend don't do it in dev env (development experience) and use [name], [path] and etc in prod env.

@evilebottnawi Works now


Non-trivial merge to 0.23

Just re-triggered it and same deal. Any idea what's going on here?

Ah, I know what's going on. The ci thing which segregates JVM/JS builds in the matrix went missing. So now it's trying to do both at the same time and Scala.js linking step is extremely memory-intensive. So GH is killing the job.

Nice green checkmark for a build so broken that didn't run at all.


Muted parts in playlist not downloadable / error in web player

Within a few minutes of TwitchLeecher trying to download this vod (https://www.twitch.tv/videos/286102401), it gives a Status: Error with the following log:
https://pastebin.com/xHKhvcbp
The video itself seems bugged on Twitch. When I try to watch it there, only the "red" parts play the video (muted of course), and the moment a non-red part tries to play, the screen goes black with the words "2000: Network error" on it.
It seems like Twitch has butchered the vod by applying it's copyright mute filter to the video.
Is there anything that can be done to retrieve the footage of this video?
Thanks.

This seems to be a serious issue Twitch introduced a couple weeks ago. More and more videos show this behavior and more and more users contact me about it. The source of the problem is that the Twitch video playlist contains files marked as muted like so:
1.ts
2.ts
3-muted.ts
4-muted.ts
5.ts
6.ts

Some of the X-muted.ts files cannot be downloaded! This is NOT an issue of Twitch Leecher. As @pooiyx  stated, even the web player throws an error at those positions in the video. So Twitch screws up their videos apparently.
A temporary fix would be to just ignore the files that cannot be downloaded and are marked as muted. But this would result in videos that suddenly skips while watching.
We have 2 options in the end:

Do nothing and wait for Twitch to solve what they messed up
Skip the muted parts and therefor cut the downloaded video at those positions

I want to leave an additional note: In the end it's the streamers fault. By using non-free music in his stream he violates the copyright law resulting in muted videos. It is not TL's or Twitches  job to undo this misbehaviour. Twitch is just accommodating. They could very easily delete those streams instead of muting the parts. I receive many e-mails that indicate I am the one responsible and should fix it. This is simply wrong.

@Franiac after a small investigation on this issue, I noticed that info in playlist is erroneous, some muted chunks are not marked as muted. That causes TL to fail.
Some data:
In this playlist, for example, on Line 1286 is a chunk 638.ts that returns 403, but it can download the muted part 638-muted.ts.
Can another option be, to try to download the muted chunk when the original download fails?

@igodorogea So basically Twitch not only makes muted files not downloadable but also messes up the names of the muted and non-muted parts. It gets better and better.
What I am interested in right now is the following: Are those videos with a corrupt playlist playable in the web player? Because if not, I don't see any point in fixing those issues on my end. Don't get me wrong, but if Twitch itself cannot play back those videos, why should I put work into it.
Yesterday there popped up muted files in the playlist that are not downloadable...
Today we have non-muted parts that are actually muted...
I ask: What do we have tomorrow?
The thing is I would have already got onto the Twitch forums and told them how bad they are developing their stuff but I cannot do this because I don't know if they actually like what TL does or not 😅

I have a big problem with "do your best" software. They are all bad because the user does not know what the software does in a certain context. That is one of the worst behaviors a software can have. So my point stands: Twitch messes up their videos and I am not here to fix what they messed up because I cannot guarantee that the outcome is usable.
Under those circumstances I will close the issue now since I don't see me or us coming up with the "break through idea" how to deal with it other than erroring out like the web player also does.

I can tell you how I solved the problem myself: I went through to the twitch VOD's URL, looked for muted sections that didn't load, and manually recorded them. Then I went through and downloaded the sections of the VOD, using TL, with those parts removed. Then I manually concatenated them; when I upload the monstrosity to youtube, I'll have to include "<10 minute jump, sorry!" somewhere in the middle of the 10 hour vod.
The VOD I was downloading was the fifth of 9, and I didn't want to miss a giant multi-hour section in the middle over 10 minutes of deleted footage. The point of the anecdote is, at the moment, the solution to this problem is exceedingly tedious, and TL can probably do this task better and more efficiently than a human. Without this feature, people will still find a way, but damn, if there's a better way.


MSID: 97962 Version: 1 DOI: 10.1101/2<IP_ADDRESS>9236

MSID: 97962
Version: 1
Preprint DOI: https://doi.org/10.1101/2<IP_ADDRESS>9236
Step 1. Awaiting reviews
Editorial to post reviews via hypothesis
Useful links:

DocMap: https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=97962
New model tracking: https://docs.google.com/spreadsheets/d/1_fHaoOy7hjyocptKtVJRijeNpUY4hBS7Ck_aVmx6ZJk/
Reviews on sciety: https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>9236

For trouble shooting (e.g. no Docmaps available):

DocMap issue addressing: https://miro.com/app/board/uXjVNCwK6EI=/
Explore DataHub DocMaps API: https://lookerstudio.google.com/reporting/4c2f0368-babb-4beb-b5b3-497e7e7b0f08/page/ejphD
Unmatched submissions and preprints: https://lookerstudio.google.com/u/0/reporting/9f86204f-3bf7-477c-9b18-5c5ef141bf69/page/p_gxi57ha93c
Unmatched manuscripts spreadsheet: https://docs.google.com/spreadsheets/d/15QcK8w-ssB7109RQEDtFpJPZ0J5HTGxoHa_2TtpMBbg/edit#gid=1336081641

Step 2. Preview reviewed preprint
Production QC content ahead of publication
Instructions:

QC preview: https://prod--epp.elifesciences.org/previews/97962v1
Update ticket with any problems (add blocked label)
When QC OK, add QC OK label to ticket and add publication date and time to https://docs.google.com/spreadsheets/d/1amAlKvdLcaDp5W8Z8g77NmkwbMF5n_u89ArSqPMO8jg
Move card to next column
(At end of the day post link in #enhanced-preprint and ask for PDF to be generated)

Useful links:

Preprint DOI :  https://doi.org/10.1101/2<IP_ADDRESS>9236
Confirm reviews returned by EPP: https://prod--epp.elifesciences.org/api/reviewed-preprints/97962/v1/reviews
To update the MECA path in the docmap: https://docs.google.com/spreadsheets/d/1mctCQuNFBjSn97Lihy7_vBO6z7-N-oqyLv4clyi6zHg

Step 3: Awaiting search reindex
This step adds the reviewed preprint to the homepage: https://elifesciences.org
The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.
Useful links:

Jenkins pipeline to reindex search can be triggered sooner or monitored here: https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 4: Published! PDF requested
Waiting for PDF to be generated
Useful links:

PDF tracking: https://docs.google.com/spreadsheets/d/106_XeDjmuBae7gexOTNzg60lapeqjl2aRn9DzupGyS8/

Step 5: Introduce PDF to data folder and git repo
Upload PDF to relevent folder in git repo https://github.com/elifesciences/enhanced-preprints-data/
Step 6: Done!

[ ] Kettle is on!

warn: [mixed-citation-other-publication-flag] mixed-citation in reference (id=c11) has a publication-type='other'. Is that correct?
warn: [mixed-citation-other-publication-flag] mixed-citation in reference (id=c13) has a publication-type='other'. Is that correct?
warn: [mixed-citation-other-publication-flag] mixed-citation in reference (id=c14) has a publication-type='other'. Is that correct?
warn: [mixed-citation-other-publication-flag] mixed-citation in reference (id=c15) has a publication-type='other'. Is that correct?
warn: [mixed-citation-other-publication-flag] mixed-citation in reference (id=c21) has a publication-type='other'. Is that correct?
warn: [mixed-citation-other-publication-flag] mixed-citation in reference (id=c33) has a publication-type='other'. Is that correct?


draft updated version of https://www.finos.org/pure-alloy

@grizzwolf and @aitana16 to draft updated version of https://www.finos.org/pure-alloy that will go live when the documentation page goes live (consider developer.gs.com to ensure they're aligned and to avoid duplicated content)

@aitana16 and @grizzwolf will start working this week on this.

hi @grizzwolf adding the specific action items discussed on our call earlier today below:

[ ] add link to the documentation site: https://alloy.finos.org/
[ ] add this Question to the FAQ section: "What's the modeling scope of the pilot project?"
Answer: The current modeling scope encompasses an effort to extend FX options from ISDA's CDM model. Additional areas being discussed include CML Reference data, an RFQ in the context of interest rate swaps and ESG data.
[ ] harmonize timelines: Phase 1 should be "Q1-Q2 2020" instead of Q1 2020

@aitana16 I've started to make the edits listed above that we discussed last week (see https://app.hubspot.com/content/2419532/edit/20928950602/content). FYI that I think the timeline on the Alloy documentation site also should get updated to reflect that Phase 1 is from January 2020 to May 2020.

@aitana16 @grizzwolf I've made this edits -- can one of you review to check if I didn't catch any typos. See https://www.finos.org/alloy.
@aitana16 When you get a chance can you now update the alloy.finos.org site to match the timeline -- see how I've just used the quarters instead.

@brooklynrob I've tried to make some edits here https://app.hubspot.com/content/2419532/edit/20928950602/content  but my current configuration does not allow me to save edits to landing pages. / cc @grizzwolf
Specifically, the answer to the last question says that the next general meeting is set for february 20th. Instead of posting specific dates, and having to update them constantly, I'd redirect to the FINOS Programs Calendar https://calendar.google.com/calendar/embed?src=finos.org_fac8mo1rfc6ehscg0d80fi8jig%40group.calendar.google.com&ctz=America%2FNew_York


ng-pivot-options does not work

Hi I am using the following example that you have given , but the ng-pivot-options is not working .
When I remove the  ng-pivot-options="options" from the div , only then the data is displayed
following is my code.
(scripts are included in the same order)

$scope.data = [
{ color: "blue", shape: "circle" },
{ color: "red", shape: "triangle" }
];
                   $scope.options = {
                       rows: ["color"],
                       cols: ["shape"]
                   };

Hi @amolkulkarni-Tieto
Thanks for reporting this issue, are you getting any errors output on your browser's console?  Later today I'll try to replicate your scenario to check if I can solve any compatibility issues that may be causing the problem.
In the meantime, please take a look at this previous thread and check if the solution also works for you.
https://github.com/ErickRuano/angular-pivot/issues/3

Hi I have looked at the issue you have mentioned earlier in https://github.com/ErickRuano/angular-pivot/issues/3 , but it did not solve my issue. The only issue i see in console is a warning which says that Angular was loaded twice ,

@ErickRuano this seems to be same issue as #6, @amolkulkarni-Tieto check if you have v.1.2.1
if you have installed using bower the version is not up-to-date.
You can try giving direct git url in bower.json to install latest version of angular-pivot

Hi @ErickRuano , I am getting the below error when I try to run my angular-app, will appreciate your help


Call GC in-between tests

I have checked to see that only maximum of 100 fibers are ever in the system at the same time.
But GC is not always triggered in-between tests which results in GC memory keep accumulating.
This fixes the OOM crash I get in my system with 8G of physical memory.

I'm a little worried about the stop-the-world GC behavior. If the threads are paused then some of the tests might time out if the collection cycle is too long.

I'm a little worried about the stop-the-world GC behavior. If the threads are paused then some of the tests might time out if the collection cycle is too long.

Calling GC once for each test could help with that as well. More we delay it, higher the risk of a long collection cycle, right?

Calling GC once for each test could help with that as well. More we delay it, higher the risk of a long collection cycle, right?

You're right. Anyway we can always increase the timeouts. And if the collection takes too long, it means we are allocating too much anyway.

I am confused by this, because I think the runtime should call GC.collect before throwing OOM. I made some comment on slack, let me repeat it here:
"
I remember under Debian I had to increase the maximum GC memory for dlang through a command line argument, and then the tests ran normally. Under Ubuntu I never had this problem.
It is more like a workaround until someone profiles the
btw I had 32GB memory and still got OOM without the command line parameter
"

I java we have something called weak reference, so an object can become 'strongly unreachable' but that weak reference can be converted back to 'strong reference' and then it becomes 'strongly reachable' and cannot be GC'd anymore, I don't think D has weak references

maybe you manually calling the GC caused somehow to reduce the number of parallel tests, and it is just by coincidence that you no longer get OOM

D's GC is derived from the Boehm GC, which was developed for languages without support for GC (C/C++).
It has evolved quite a bit since its inception, but it's still far from fully utilizing the full type information it could.
Note that we could disable this hack and try to run it concurrently.

I am confused by this, because I think the runtime should call GC.collect before throwing OOM. I made some comment on slack about this, let me repeat it here:
"
I remember under Debian I had to increase the maximum GC memory for dlang through a command line argument, and then the tests ran normally. Under Ubuntu I never had this problem.
It is more like a workaround until someone profiles the
btw I had 32GB memory and still got OOM without the command line parameter
"

From the top of my head, in POSIX fibers allocate stack space with mmap. It may be not triggering GC.

I am confused by this, because I think the runtime should call GC.collect before throwing OOM. I made some comment on slack about this, let me repeat it here:
"
I remember under Debian I had to increase the maximum GC memory for dlang through a command line argument, and then the tests ran normally. Under Ubuntu I never had this problem.
It is more like a workaround until someone profiles the
btw I had 32GB memory and still got OOM without the command line parameter
"

From the top of my head, in POSIX fibers allocate stack space with mmap. It may be not triggering GC.

what was the actual error message you got? core.exception.OutOfMemoryError?

Apparently, this is where the druntime aborts.  Each fiber uses mmap to allocate a stack and creates a mapping, which is limited per process. These mappings are only destroyed when Fiber is deconstructed.

Apparently, this is where the druntime aborts.  Each fiber uses mmap to allocate a stack and creates a mapping, which is limited per process. These mappings are only destroyed when Fiber is deconstructed.

As expected, setting vm.max_map_count parameter in Linux to a higher value also fixes the issue.  vm.max_map_count  defaults to 65530 on my machine.

This wont be needed when https://github.com/Geod24/localrest/pull/103 is merged.


Installation ( curl net installer ) fails with unprivileged user.

How am I supposed to install the tools without jailing my default shell user and without using sudo?
the documentaion doesn't say anything about it ..and from what I see in the installer shell script, there is no option for nonroot users~
The installation just exits with code 1:
$ curl -sL http://git.io/vtf5N | bash
Something where wrong :(

curl -sL http://git.io/vtf5N | bash -s -- --me

thanks, that helped :) damn.. some stuff I still need to learn about bash :/
hmm why isn't this option listed in the documentation? I'm for sure not alone who isn't creating an extra user and dedicated server for that..

The -s argument tells bash to process standard input as a script.
The -- argument tells bash to stop processing arguments - any following arguments are passed to the script.
The --me argument is passed to the netinstall.sh script, which in turn passes it to the install.sh script, to which it means to perform a user-install.

thanks for the explanation!
hmm while configuration I couldn't find the argument for enabling of bRawSockets which is to be passed via commandline parameter to enable UDP protocol~
I guess, it's not yet planed in v1.4 ~ so I need to modify the script till then?

ark_bRawSockets=""

This was added by hotfix #185

wow..damn ok, the repo search didn't show it up :/
ok, thanks a lot, I think, it will work for now :)


requestSubmit on safari incorrectly sets the submitter property on the SubmitEvent

I've been using Turbo Streams with a custom idiomorph action (though this bug occurs without idiomorph, see the sandbox below for a minimal example) and came across a bug with Turbo & Safari (tested on version 17) that might affect the upcoming refresh feature.
I have a form with multiple submit buttons that have different names, and some stimulus actions that submit the form via requestSubmit(). Clicking a button submits the form to the server and re-renders it with idiomorph., On Safari the requestSubmit form submissions will include the name/value of the previously used submit button.
So, if I submit the form with a submit button that is effect=add_distribution, then net time I submit the form with requestSubmit (regardless of whether I pass a submitter in or not, I tried adding a hidden submit button and specifying that), the previous effect=add_distribution parameter is sent to the server.
Firefox/Chrome work correctly and respect the submitter specified (or not) to requestSubmit
Anyway, for now I'm using the polyfill from Turbo regardless of whether the browser supports requestSubmit natively or not and that fixes the problem.
Just thought I'd bring it to your attention since people will be enjoying less javascript in Turbo 8 with idiomorph and refresh.
Here's a code sandbox showing the issue. If you comment out Turbo, the console shows the correct submitter, but with Turbo enabled, it uses the previous submitter.

I see you've opened https://bugs.webkit.org/show_bug.cgi?id=265077, that's great!

I think this is a dupe of https://github.com/hotwired/turbo/pull/933. It just has not made it in a release yet.

Excellent! I can't believe I missed the PR, I searched for existing issues.
I'll give the master branch a try..

You've got https://github.com/hotwired/dev-builds/tags as well.

I also had opened this about a month ago but got no attention haha https://github.com/hotwired/turbo/issues/1050

I think it will be fixed in the next version of turbo, for now you can grab their polyfill for requestSubmit and use that.


Annoying Drawer Border on Android

It appears that react navigation drawer was not tested for dark theme(s) as you can see weird border blinks in the the video below.

Here's a code snippet to reproduce this issue:
import React from 'react';
import {Text, View, StatusBar, StyleSheet, Button} from 'react-native';
import {createDrawerNavigator} from 'react-navigation-drawer';
import {createStackNavigator} from 'react-navigation-stack';
import {createAppContainer} from 'react-navigation';

const Sidebar = ({navigation}) => (
  <View style={styles.container}>
    <Text style={styles.text}>Sidebar</Text>
    <Button title="Close Sidebar" onPress={() => navigation.closeDrawer()} />
  </View>
);

const Home = ({navigation}) => {
  return (
    <View style={styles.container}>
      <Button title="Open Sidebar" onPress={() => navigation.openDrawer()} />
    </View>
  );
};

const Routes = createStackNavigator({Home: {screen: Home}}, {mode: 'card', headerMode: 'none'});

const Drawer = createDrawerNavigator(
  {DefaultRoutes: Routes},
  {
    contentComponent: Sidebar,
    drawerType: 'slide',
    drawerWidth: 300,
  },
);

const AppContainer = createAppContainer(Drawer);

const App = () => {
  return (
    <>
      <StatusBar backgroundColor="#121212" barStyle="light-content" />
      <AppContainer />
    </>
  );
};

const styles = StyleSheet.create({
  container: {
    flex: 1,
    justifyContent: 'center',
    alignItems: 'center',
    backgroundColor: '#121212',
  },
  text: {
    fontSize: 18,
    color: '#eeeeee',
    marginBottom: 14,
  },
});

export default App;

and a gist:
https://gist.github.com/binodnp/1f28059e6013ab17f5cb7d3867ceed8a
Is there any workaround for this?

workaround: https://github.com/react-navigation/drawer/issues/71#issuecomment-501909140

add overlayColor to transparent in contentComponent.
like this
const Drawer = createDrawerNavigator(
{DefaultRoutes: Routes},
{
contentComponent: Sidebar,
drawerType: 'slide',
drawerWidth: 300,
overlayColor: 'transparent'
},
);

This issues is not related to #71 and therefore the workaround doesn't work.

I fixed it by changing drawerType to drawerType: 'back',. At least its a quick workaround.


Ensemble builder

Initial Commit with an empty file, mainly to test the check-api

Should we wait for the dataset class to be merged and then make changes to this PR before merging it? Otherwise, for me the changes are good.

We need the changes in this PR for SMAC. It also fixes the backend
When building an ensemble we use smac output, so we do not care about the data manager.
Nevertheless, I am reading the datamanager to make any time test performance, but is a 1 line change when the dataset/datamanager is ready

We need the changes in this PR for SMAC. It also fixes the backend
When building an ensemble we use SMAC output, so we do not care about the data manager.
Nevertheless, I am reading the data manager to make any time test performance, but is a 1 line change when the dataset/data manager is ready
Okay then, from my side it can be merged. Let's wait for Lucas to review it


Maximum call stack size exceeded when use a recursive function inside subscribe

Hi, I'm trying to transform some code from amqp to nats, I start with the basic Fibonacci request/reply
server
import {connect} from 'ts-nats';

function fibonacci(number) {
  // eslint-disable-next-line max-statements-per-line
  if (number === 0 || number === 1) { return number; } else { return fibonacci(number - 1) + fibonacci(number - 2); }
}

// ...
(async function() {
  try {
    const nc = await connect();
    console.info('server running.');
    await nc.subscribe("fibo", (err, msg) => {
      console.info("receiving message");
      if (err || !msg.reply) {
        // handle the error
      } else {
        // eslint-disable-next-line radix
        const reqNumber = parseInt(msg.reply);

        console.log(msg.data);
        const response = fibonacci(reqNumber);
        console.info(`response is ${response}`);
        nc.publish(msg.reply, response.toString());
      }
    }, {max: 100});
    // Do something with the connection
  } catch (ex) {
    // handle the error
  }
})();

client
import {connect} from 'ts-nats';

// ...
(async function() {
  try {
    const nc = await connect();

    console.info("msg 2");
    const msg = await nc.request('fibo', 0, "6");
    console.info(`msg: ${msg.data}`);

  } catch (ex) {
    // handle the error
  }
})();

the Fibonacci code is not tail recursive but I've tested this even with big numbers and works ok, I never get an error related to the call stack when use it...
when I run this code I get this
Exception in PromiseRejectCallback:
/Users/Admin/Downloads/proj/js/typescript/vertex_rpc/src/prueba.ts:4
    return new (P || (P = Promise))(function (resolve, reject) {
           ^

RangeError: Maximum call stack size exceeded
Exception in PromiseRejectCallback:
/Users/Admin/Downloads/proj/js/typescript/vertex_rpc/src/prueba.ts:7
        function step(result) { result.done ? resolve(result.value) : adopt(result.value).then(fulfilled, rejected); }
                                                                                ^

RangeError: Maximum call stack size exceeded
(node:39229) UnhandledPromiseRejectionWarning: RangeError: Maximum call stack size exceeded
    at /Users/Admin/Downloads/proj/js/typescript/vertex_rpc/src/prueba.ts:1:1
    at /Users/Admin/Downloads/proj/js/typescript/vertex_rpc/src/prueba.ts:8:37
    at new Promise (<anonymous>)
    at __awaiter (/Users/Admin/Downloads/proj/js/typescript/vertex_rpc/src/prueba.ts:4:12)
    at fibonacci (/Users/Admin/Downloads/proj/js/typescript/vertex_rpc/src/prueba.ts:14:12)
    at /Users/Admin/Downloads/proj/js/typescript/vertex_rpc/src/prueba.ts:5:76
    at Generator.next (<anonymous>)
    at /Users/Admin/Downloads/proj/js/typescript/vertex_rpc/src/prueba.ts:8:71
    at new Promise (<anonymous>)
    at __awaiter (/Users/Admin/Downloads/proj/js/typescript/vertex_rpc/src/prueba.ts:4:12)
(node:39229) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:39229) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

do you know what could be the issue here?...thank you so much

try implementing without recursion.


0205 - Isomorphic Strings

Description of the Problem
Given two strings s and t, determine if they are isomorphic.
Two strings are isomorphic if the characters in s can be replaced to get t.
All occurrences of a character must be replaced with another character while preserving the order of characters. No two characters may map to the same character but a character may map to itself.
Code
class Solution:
    def isIsomorphic(self, s: str, t: str) -> bool:

Link To The LeetCode Problem
LeetCode

I want to work on this

@Vinit-Dantkale
Waiting for your PR 😸

Hey @vJechsmayr , I might be unavailable. Please feel free to close or assign someone else (or put an Available tag so someone else can take it).
Thanks 👍


Applying styles to allowed class

I'm trying to use sanitize-html on an input field but need to allow certain classes to be entered and to have their styles applied after sanitization. I'm using Astro, and simply including the class within the scoped style tag isn't working. My guess is that the scoping is messing with things. Is there a way to apply a style within the sanitize function, perhaps within transformTags? I've looked through the docs and only see an implementation for elements, so I'm wondering if this is possible. I'm using Node v19.2.0 on MacOS X

It sounds to me like you are asking for two separate things. Santize is great for making sure that HTML is well formed and can transform the input HTML automatically. It sounds like you want to take the additional step of using the sanitized input. This sounds more like a case for JavaScript manipulation of the DOM, or... Maybe I'm wrong and you can add some additional details.


Getting list of child from a list

I am just trying to print all discovered printers name after clicking on "Add a printer or scanner" but xpath - List printerList = SettingSession.findElementsById("SystemSettings_Printer_Add_ListView"); does not work. I noticed that FindElementsByAccessibilityId isn't available with Java instead it takes FindElementsById (in bold below)
Here is the code -
public void DiscoverPrinter(String ptr_name) throws InterruptedException, IOException
{
// Click on "Add a printer or scanner" & wait until listing is over
		try {
			SettingSession.findElementByName("Add a printer or scanner").click();
			}catch(Exception e){
	    	e.printStackTrace();
	    	log.info("Error clicking on -> \"Add a printer or scanner\"");
			}
		log.info("Clicked on \"Add a printer or scanner\"");
		Thread.sleep(500);

		do {
			log.info("Printers & Scanners Discovery is IN PROGRESS....");
			Thread.sleep(1000);
			}while(SettingSession.findElementsByName("Searching for printers and scanners").size()!=0);

		log.info("Printers & Scanners Discovery COMPLETED");

		List<WebElement> printerList = **SettingSession.findElementsById**("SystemSettings_Printer_Add_ListView");
		log.info("Number of Discovered Printers => "+printerList.size());
		for(WebElement sPriner:printerList)
		{
			log.info(sPriner.getText());
		}

Below is the properties details from inspect

Changing SettingSession object from RemoteWebDriver type to WindowsDriver type makes findElementsByAccessibilityId available but still below code is not getting the list of child printers
List printerList = SettingSession.findElementsByAccessibilityId("SystemSettings_Printer_Add_ListView");
log.info("Number of Discovered Printers => "+printerList.size());
for(WebElement sPriner:printerList)
{
log.info(sPriner.getText());
}

Please reopen if you are still experiencing this issue.


Additional Findings from Fuzzing

Prerequisites

[X] Can you reproduce the problem in a MWE?
[X] Are you running the latest version of AngleSharp?
[X] Did you check the FAQs to see if that helps you?
[X] Are you reporting to the correct repository? (there are multiple AngleSharp libraries, e.g., AngleSharp.Css for CSS support)
[X] Did you perform a search in the issues?

Description
To continue the issues found in #1179 we use this thread.
All issues found will be fixed in 1.1.2.
Steps to Reproduce
See individual posts here.
Expected Behavior
Should work.
Actual Behavior
Causes a parser panic (either crash, indefinite loop, or other non-desired behavior).
Possible Solution / Known Workarounds
Issue with tree construction due to invalid input / edge case / problem in the HTML5 error handling area.

First case reported by @SyrasX:

<template>
<s><object><svg><template></object></object>

First case reported by @SyrasX:

<template>
<s><object><svg><template></object></object>

Another similar input that also timeouts:
<template>
<template>
<object><svg><template></object></object><e>

Completed in 1.1.2.


Input Widgets Don't Seem to Be Correctly Rendered Over Preview

What system are you running Yazi on?
macOS
What terminal are you running Yazi in?
iTerm2
Yazi version
0.1.5
Did you try the latest main branch to see if the problem has already been fixed?
Tried, but the problem is still present
Describe the bug
Part of the preview is rendered over the input widget, while part of the input widget doesn't go away after cancelling the input.
https://github.com/sxyazi/yazi/assets/109451175/99a6d625-7c06-4058-92d6-c7c9d7c769af
https://github.com/sxyazi/yazi/assets/109451175/53c6a43a-3418-41e8-803e-cc04cadd6201
Expected Behavior

The input widget should be on top of the preview.
The preview should be restored after the input widget is canceled/submitted.

To Reproduce
Open Yazi in terminal windows with less columns, so that the input widget overlaps the preview. Open and cancel the input widget.
Configuration
No response
Anything else?
No response

This is a known bug, and I haven't found the time to fix it yet, in case someone wants to create a PR for it. Here are some ideas:

When the Input component is visible=true, get the position of the Input component:
https://github.com/sxyazi/yazi/blob/7c49d9c57a8eb5f1c3263037b9424b55f2306735/yazi-fm/src/input/input.rs#L19-L20

Implement an image_clear method for the Adaptor, which takes the position of the Input component and clears the image in that area, finely like the existing image_hide:
https://github.com/sxyazi/yazi/blob/7c49d9c57a8eb5f1c3263037b9424b55f2306735/yazi-adaptor/src/adaptor.rs#L182-L195

Yazi supports 3 image protocols: Kitty, iTerm2, and Sixel, there's no need to implement all of them. Any constructive PRs are highly welcome!

Thanks for your suggestion! I tried a bit but failed for now. It seems we need to rewrite the Iterm2::encode function to get rid of the cells in the Input component. Using crossterm::terminal::clear may not work because they don't provide function to reset a single cells (I'm not 100% sure though).
Another workaround is to draw the cells within the Input component. However, the rendering for this area is handled by paragraph from ratatui::widgets:: now. Maybe we can implement our own render function. This seems to be a easier way for our goal, and it doesn't involve passing through the Input information from Apps all the way down to Adaptors. Personally, I would prefer this one.
Will give it another try when I have time!

For the iTerm2/Sixel protocol, it doesn't have a true concept of "clear". It just:

Simply move the cursor to the corresponding cell, by Term::move_to()
Using a blank character (" ") to fill it: stdout.write_all(" ".repeat(width))

It's exactly the same as what image_hide does now, except that it doesn't currently clear the input area when it's visible:
https://github.com/sxyazi/yazi/blob/a0ba8537188cd5c29593b1c53a3569ac7f1a0bd7/yazi-adaptor/src/sixel.rs#L22-L32


Result Saving and Manual Commits broken on Update

System information:

Windows 10 Pro 1909
DBeaver <IP_ADDRESS>002151353
No Additional Extensions

Connection specification:

Database: Oracle
Driver: oracle12c
No Tunnels and Proxys, direct connection.

Describe the problem you're observing:
After the latest update, somehow manual commiting broke; transaction counter stays on none or one forever; result editing automatically saves without even asking.

Include any warning/errors/backtraces from the logs
The Log keeps showing a FrameworkEvent Error from Apache Felix.

hi @dev-whiteflag ,
unfortunately, cannot reproduce an issue. Is issue reproducible for UPDATE query only?

No, the problem persists independently if it is a SELECT or a UPDATE query.

Data editor is working properly again, don't know what happened. But manual committing isn't working still. The short video you asked is in the zip, as Github doesn't allow any video uploading.
DBeaver 6.3.5 - Oracle - CPO Script 2020-02-20 10-29-22.mp4.zip

Data editor is working properly again, but manual committing isn't still. I've tested with another version of DBeaver (<IP_ADDRESS>810232047) and is working properly. There's a zip with the short video you asked.
DBeaver 6.3.5 - Oracle - CPO Script-14 2020-02-20 10-36-38.mp4.zip

On another note, doesn't seems to be a problem with Oracle Database, because still happens when I change databases.

Issue was fixed on version 7.0.0. Thanks!


Force initial carouselIndex

This issue is described in #296
The solution is a bit convoluted but the point is that the provided initial index was overridden because slides were not loaded yet.

Please, merge this fix quickly. It's really need for me.

@EAndreyF : did you try this PR ?

No. Change for my is in 488: currentSlides && currentSlides.length > 0 && newValue >= currentSlides.length or just currentSlides && currentSlides.length && newValue >= currentSlides.length
Because I initialize my slider without images. If this PR is hard to test, I could make simple PR for my changes.

+1

+1 this solved my problem

+1

+1

This fix doesn't work.

            {{$index}} should be {{$ctrl.vehicleIndex}}

I define in the controller this.vehicleIndex = 2 but the $index keeps being initially set to 0:


DadJoke Commit - and Update Throw

Commit Dad Joke to Dev
Re-read how you had things set up in GitHub. Submitting Pull request to Dev from Dev instead of to Dev from Master.
Screenshot below - added in comments worked in dev so unless the comments break it... should work just fine.

I'm assuming this is what you meant for me to do :)
My bad, I was syncing with Master making changes and then pulling as opposed to syncing with Dev, making changes and pulling.

Closing this for since it has conflicts


🐛 - Reactions fail to recognize occasionally

Issue

Sometimes, the user will react, but it won't register, and soon the command will cancel.
The user is not able to register another command until the command cancels, signifying that the command is still running, and not canceled.

There is no traceback

Context
Related code:
https://github.com/LeptoSpira/offthedialbot/blob/619fdef49d00b2311cc8381498c2575b7bf39d3d/offthedialbot/commands/signup.py#L104-L108
Steps to reproduce:

$signup

(follow the steps in signup)

Hit the ✅

Additional Context:
One possible cause of the issue is the modified timeout length

(Hopefully) No longer relevant

(Hopefully) No longer relevant


Updations in prerequisites #768

#768
Checklist

[x] I have made a material change to the repo (functionality, testing, spelling, grammar)
[x] I have read the [Contribution Guide] and my PR follows them.
[x] I updated my branch with the master branch.

Short description of what this PR does:
On the website its mentioned that Send up to 40,000 emails for 30 days, then send 100 emails/day free forever but it has not been mentioned in the README.

 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.

Hello @Rishabh04-02,Thanks again for the PR! It's HACKTOBERFEST! We want to show our appreciation by sending                                              you some special Hacktoberfest swag. If you have not already, could you please fill out                                              this form so we can send it to you? Thanks!                                             Team SendGrid DX

Hello @Rishabh04-02,Thanks again for the PR! We appreciate your contribution and look forward to                                              continued collaboration. Thanks!                                             Team SendGrid DX


tab complete

make tab complete that will compare the current input text to a list of all commands

you should make it put the autocomplete as the accent color (which is dark grey by default, so it would look quite nicely)


DataProducts link is not working

[Enter feedback here]
When clicking the link for 'data product' mentioned in the line 'Establish a data product architecture', it opens up a page with '404- Content not found'.

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

ID: 5f10895e-4e1d-1cdd-2d57-32880e0cc45a
Version Independent ID: 3518d0a5-ffd1-dfa7-6d6b-de2a9ef170d6
Content: Analytics end-to-end with Azure Synapse - Azure Example Scenarios
Content Source: docs/example-scenario/dataplate2e/data-platform-end-to-end.yml
Service: architecture-center
Sub-service: example-scenario
GitHub Login: @fabragaMS
Microsoft Alias: fabraga

Thanks for reporting - this issue is under review. This is a Microsoft Internal DevOps Bug ID # 533579

@ravkirn
I have assigned the issue to the content author for further review.
https://docs.microsoft.com/en-us/azure/cloud-adoption-framework/scenarios/data-management/architectures/data-landing-zone-data-products

PR submitted.
https://github.com/MicrosoftDocs/architecture-center-pr/pull/4945

Thanks @fabragaMS for doing that last year.  Closing this down.


MM-26971 Fix GM duplicates when created and already exists in a category

Summary

Check if the channel exists in another category before adding it to DM category in a team, If it does not then add it to DM category

Ticket Link
https://mattermost.atlassian.net/browse/MM-26971

@Willyfrog Please prioritize reviews for v5.26,

QA review is done on the webapp PR.

/cherry-pick release-5.26


Working with Flutter timelines

Proposed title of article
Working with Flutter timelines
Introduction paragraph (2-3 paragraphs)::
You can think of timelines as progress indicators which shows the current level of a larger process. Timelines are useful when one needs to calculate percentage of completion of a task.
In this tutorial we will be building an example of task completion application in fluttuer to explain how uselful this can be.
Key takeaways:

Undertanding Flutter timelines
Flutter timeline_tile dependency
Using timeline dependency to build a flutter application.

References:
Please list links to any published content/research that you intend to use to support/guide this article.
Templates to use as guides

How To Guide Template
Software Review Template
Tutorial Template

@mia-roberts
Good afternoon and thank you for submitting your topic suggestion.
Your topic form has been entered into our queue and should be reviewed (for approval) as soon as a content moderator is finished reviewing the ones in the queue before it.

Hi @mia-roberts
This sounds like a really interesting topic. Please make sure you provide more original content than what is already there in the pub.dev registry.
:rocket:

@mia-roberts


first request with a recreated user returns a 401 error

Describe the bug
when a user is deleted and recreated the first request with that user returns a 401 error, all next requests succeed
Steps to reproduce
Steps to reproduce the behavior:

start ocis with PROXY_ENABLE_BASIC_AUTH=true bin/ocis server
create user curl -k -X POST https://localhost:9200/ocs/v2.php/cloud/users -d userid="jasmine" -d<EMAIL_ADDRESS>-d username="jasmine" -d password="1234" -u admin:admin
use the new user, e.g. send a PROPFIND: curl -k -X PROPFIND https://localhost:9200/remote.php/webdav -u jasmine:1234 -v
delete the user: curl -k -X DELETE https://localhost:9200/ocs/v2.php/cloud/users/jasmine -u admin:admin
create it again: curl -k -X POST https://localhost:9200/ocs/v2.php/cloud/users -d userid="jasmine" -d<EMAIL_ADDRESS>-d username="jasmine" -d password="1234" -u admin:admin
use the new user, e.g. send a PROPFIND: curl -k -X PROPFIND https://localhost:9200/remote.php/webdav -u jasmine:1234 -v => Response is HTTP/1.1 401 Unauthorized
use the user again: curl -k -X PROPFIND https://localhost:9200/remote.php/webdav -u jasmine:1234 -v => Response is as expected HTTP/1.1 207 Multi-Status

Expected behavior
the first request after user creation should be already successful
Actual behavior
the first request always fails with HTTP 401
Setup

BRANCH=master
PROXY_ENABLE_BASIC_AUTH=true

Additional context
this seems not to be a timing issue but always the first requests fails, also when using other endpoints

needs to be reevaluated after spaces are used to create and delete user homes (personal spaces)

re-tested oCIS 1.19.1. The issue no longer occurs.
Everything works as expected.
TODO:

[ ] check owncloud-sdk tests

Task completed! Closing here


Fixed bad method name in auth docs

The method name in ParserInterface is parseToken() not parse().

For some reason the github fork button for equip took me to my fork of spark...

👍

👍
Thanks @aminemat!


Why does the devices have some number referenced at the end?

when i do sc -l
i get
[ ] Gemalto Ezio Shield 01 00
[*] ACS ACR 38U-CCID 02 00
But what does 01 00 and 02 00 even mean

any thoughts on this ?

@martinpaljak

This comes from libccid/pcsc-lite


feat: generate models by referencing the Connection Definition

This PR fixes #11540
Signed commits

[ ] Yes, I signed my commits.

@Kevin222004
Thank you for your contributions! If you are ready, Let's discuss this during the Meshery Development call today at 9:00 AM CT | 7:30 PM IST. Adding it as an agenda item to the meeting minutes || meeting link.

Fixed by https://github.com/meshery/meshery/pull/12146


Scala Strict Deps toggling via cmd line flag

Scala strict deps can now be turned on (error/warn) via cmd line flag which makes it actually usable!
There are still some question marks about the performance which is why this is still turned off by default for now.
Details:
upgraded bazel to 0.5.3 since that exposes strict java deps cmd line flag to skylark
many BUILD files were removed since the toggling can be done in the test via the cmd line flag and the same BUILD files can be reused.
weird error mode test is no longer needed since validation is done by bazel on the cmd line flag
error mode test is not needed since it's tested by other tests
warn doesn't have a unit-test so is still needed
off/default drive skylark code so they're needed as e2es

It's a little weird to use --strict-java-deps rather than scala, but I guess skylark does not give you the ability to add new flags.
Seems fine to me.
👍

Exactly. I opened an issue to add a "strict_jvm_deps" flag. No real hopes
on it getting in but I'm trying.
On Sat, 5 Aug 2017 at 21:31 P. Oscar Boykin<EMAIL_ADDRESS>wrote:

It's a little weird to use --strict-java-deps rather than scala, but I
guess skylark does not give you the ability to add new flags.
Seems fine to me.
👍
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bazelbuild/rules_scala/pull/257#issuecomment-320462390,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABUIF5CImPF_6fyYfOGBgtnSIxpYVY0Hks5sVLT5gaJpZM4Os9Vw
.


The dependency list for the CP4D module needs to be cleaned up

There are conflicts in the dependencies for cp4d.  This includes common services, and catalog sources.
We also need to make sure that module.yaml for each of the dependent modules correctly lists its dependencies.  Currently CP4D instance, CP4D operator do not list dependencies.

Issues
https://github.com/cloud-native-toolkit/terraform-gitops-db2u-operator/issues/14
https://github.com/cloud-native-toolkit/terraform-gitops-cp-db2-oltp/issues/8
https://github.com/cloud-native-toolkit/terraform-gitops-cp-db2/issues/6
https://github.com/cloud-native-toolkit/terraform-gitops-cp-db2-schema/issues/3
https://github.com/cloud-native-toolkit/terraform-gitops-cp4d-operator/issues/11
https://github.com/cloud-native-toolkit/terraform-gitops-cp4d-instance/issues/3

Fixed all db2 dependency except db2 schema module which is currently broken

All of this issues got resolved except the db2 instance and schema module is pending.
as part of CP4D 4.5 release (June), they will release the instance operator which can help to automate