:terminal redraw delayed if cursor blinking is disabled

After running command inside terminal redraw is sometimes delayed. It can be forced by moving cursor. It happens on Windows 10 with gGvim 8.0.0823
Steps to reproduce:

gvim --clean "+set guicursor=a:blinkon0"
:terminal
vim

It doesn't happen always, repeat step three until it happens.
Examples
With cursor blinking disabled

With cursor blinking enabled

Version
VIM - Vi IMproved 8.0 (2016 Sep 12, compiled Jul 31 2017 20:16:37)
MS-Windows 32-bit GUI version with OLE support
Included patches: 1-823
Compiled by hp@HP-PC
Huge version with GUI.  Features included (+) or not (-):
+acl                +ex_extra           +multi_byte_ime/dyn -tag_any_white
+arabic             +extra_search       +multi_lang         +tcl/dyn
+autocmd            +farsi              +mzscheme/dyn       -termguicolors
+balloon_eval       +file_in_path       +netbeans_intg      +terminal
+browse             +find_in_path       +num64              -tgetent
++builtin_terms     +float              +ole                -termresponse
+byte_offset        +folding            +packages           +textobjects
+channel            -footer             +path_extra         +timers
+cindent            +gettext/dyn        +perl/dyn           +title
+clientserver       -hangul_input       +persistent_undo    +toolbar
+clipboard          +iconv/dyn          -postscript         +user_commands
+cmdline_compl      +insert_expand      +printer            +vertsplit
+cmdline_hist       +job                +profile            +virtualedit
+cmdline_info       +jumplist           +python/dyn         +visual
+comments           +keymap             +python3/dyn        +visualextra
+conceal            +lambda             +quickfix           +viminfo
+cryptv             +langmap            +reltime            +vreplace
+cscope             +libcall            +rightleft          +wildignore
+cursorbind         +linebreak          +ruby/dyn           +wildmenu
+cursorshape        +lispindent         +scrollbind         +windows
+dialog_con_gui     +listcmds           +signs              +writebackup
+diff               +localmap           +smartindent        -xfontset
+digraphs           +lua/dyn            +startuptime        -xim
+directx            +menu               +statusline         +xpm_w32
-dnd                +mksession          -sun_workshop       -xterm_save
-ebcdic             +modify_fname       +syntax
+emacs_tags         +mouse              +tag_binary
+eval               +mouseshape         +tag_old_static
   system vimrc file: "$VIM\vimrc"
     user vimrc file: "$HOME\_vimrc"
 2nd user vimrc file: "$HOME\vimfiles\vimrc"
 3rd user vimrc file: "$VIM\_vimrc"
      user exrc file: "$HOME\_exrc"
  2nd user exrc file: "$VIM\_exrc"
  system gvimrc file: "$VIM\gvimrc"
    user gvimrc file: "$HOME\_gvimrc"
2nd user gvimrc file: "$HOME\vimfiles\gvimrc"
3rd user gvimrc file: "$VIM\_gvimrc"
       defaults file: "$VIMRUNTIME\defaults.vim"
    system menu file: "$VIMRUNTIME\menu.vim"
Compilation: cl -c /W3 /nologo  -I. -Iproto -DHAVE_PATHDEF -DWIN32  -DFEAT_CSCOPE -DFEAT_TERMINAL -DFEAT_NETBEANS_INTG -DFEAT_JOB_CHANNEL   -DFEAT_XPM_W32   -DWINVER=0x0501 -D_WIN32_WINNT=0x0501 /MP -DHAVE_STDINT_H /Ox /GL -DNDEBUG /arch:IA32 /MD -DFEAT_OLE -DFEAT_MBYTE_IME -DDYNAMIC_IME -DGLOBAL_IME -DFEAT_MBYTE -DFEAT_GUI_W32 -DFEAT_DIRECTX -DDYNAMIC_DIRECTX -DDYNAMIC_ICONV -DDYNAMIC_GETTEXT -DFEAT_TCL -DDYNAMIC_TCL -DDYNAMIC_TCL_DLL=\"tcl86.dll\" -DDYNAMIC_TCL_VER=\"8.6\" -DFEAT_LUA -DDYNAMIC_LUA -DDYNAMIC_LUA_DLL=\"lua53.dll\" -DFEAT_PYTHON -DDYNAMIC_PYTHON -DDYNAMIC_PYTHON_DLL=\"python27.dll\" -DFEAT_PYTHON3 -DDYNAMIC_PYTHON3 -DDYNAMIC_PYTHON3_DLL=\"python36.dll\" -DFEAT_MZSCHEME -I "C:\Users\hp\Desktop\racket-x86\include" -DDYNAMIC_MZSCHEME -DDYNAMIC_MZSCH_DLL=\"libracketxxxxxxx.dll\" -DDYNAMIC_MZGC_DLL=\"libmzgcxxxxxxx.dll\" -DFEAT_PERL -DPERL_IMPLICIT_CONTEXT -DPERL_IMPLICIT_SYS -DDYNAMIC_PERL -DDYNAMIC_PERL_DLL=\"perl526.dll\" -DFEAT_RUBY -DDYNAMIC_RUBY -DDYNAMIC_RUBY_VER=24 -DDYNAMIC_RUBY_DLL=\"msvcrt-ruby240.dll\" -DFEAT_HUGE /Fd.\ObjGXOULYHTRZi386/ /Zi
Linking: link /RELEASE /nologo /subsystem:windows /LTCG:STATUS oldnames.lib kernel32.lib advapi32.lib shell32.lib gdi32.lib  comdlg32.lib ole32.lib uuid.lib /machine:i386 gdi32.lib version.lib   winspool.lib comctl32.lib advapi32.lib shell32.lib  /machine:i386  msvcrt.lib oleaut32.lib user32.lib  /nodefaultlib:lua53.lib  /STACK:8388608  /nodefaultlib:python27.lib /nodefaultlib:python36.lib   "C:\TCL_x86\lib\tclstub86.lib" WSock32.lib .\xpm\x86\lib-vc14\libXpm.lib /PDB:gvim.pdb -debug

xtal8 wrote:

After running command inside terminal redraw is sometimes delayed. It
can be forced by moving cursor. It happens on Windows 10 with gGvim
8.0.0823
Steps to reproduce:

gvim --clean "+set guicursor=a:blinkon0"
:terminal
vim

It doesn't happen always, repeat step three until it happens.

Delayed by how much?  In the gif it seems to be quite long, perhaps
related to 'updatetime'?
--
[clop clop]
ARTHUR:  Old woman!
DENNIS:  Man!
ARTHUR:  Man, sorry.  What knight lives in that castle over there?
DENNIS:  I'm thirty seven.
ARTHUR:  What?
DENNIS:  I'm thirty seven -- I'm not old!
The Quest for the Holy Grail (Monty Python)
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

Delay is about 4 seconds. It is lower if I reduce updatetime and higher if I increase it.

xtal8 wrote:

Delay is about 4 seconds. It is lower if I reduce updatetime and
higher if I increase it.

I suspect the Win32 GUI implementation of gui_mch_wait_for_chars() does
not do the screen update.  It does call parse_queued_messages() though.
--
WOMAN:   King of the who?
ARTHUR:  The Britons.
WOMAN:   Who are the Britons?
ARTHUR:  Well, we all are. we're all Britons and I am your king.
The Quest for the Holy Grail (Monty Python)
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

@xtal8 Could you please try to start vim -u none -N in the terminal?

With vim -u NONE -N i can't reproduce entering delay, but exiting delay is still sometimes present.

After many tries, I've been able to reproduce starting delay also. It happens much less frequently, though.

Yasuhiro Matsumoto wrote:

@xtal8 Could you please try to start vim -u none -N in the terminal?

With a recent Vim "vim --clean" works better.
--
The question is:  What do you do with your life?
The wrong answer is: Become the richest guy in the graveyard.
(billionaire and Oracle founder Larry Ellison)
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

Does this problem still happen?  There were some recent pages to make redrawing work better.

I tried 8.0.0871. It's still happening.

Now it happens much more frequently if I start vim inside :terminal with --clean option than without it. With  8.0.0823 it was other way around.

@xtal8 Could you please try this patch?
diff --git a/src/gui_w32.c b/src/gui_w32.c
index 863975c7f..df4abd418 100644
--- a/src/gui_w32.c
+++ b/src/gui_w32.c
@@ -2080,13 +2080,15 @@ gui_mch_wait_for_chars(int wtime)

 	    parse_queued_messages();

-	    if (pPeekMessage(&msg, NULL, 0, 0, PM_NOREMOVE)
-		|| MsgWaitForMultipleObjects(0, NULL, FALSE, 100, QS_ALLINPUT)
-								!= WAIT_TIMEOUT)
+	    if (pPeekMessage(&msg, NULL, 0, 0, PM_NOREMOVE))
+	    {
+		process_message();
+		break;
+	    } else if (MsgWaitForMultipleObjects(0, NULL, FALSE, 100,
+			QS_ALLINPUT) != WAIT_TIMEOUT)
 		break;
 	}
-#endif
-
+#else
 	/*
 	 * Don't use gui_mch_update() because then we will spin-lock until a
 	 * char arrives, instead we use GetMessage() to hang until an
@@ -2094,6 +2096,7 @@ gui_mch_wait_for_chars(int wtime)
 	 * returning as soon as it contains a single char -- webb
 	 */
 	process_message();
+#endif

 	if (input_available())
 	{

Yasuhiro wrote:

@xtal8 Could you please try this patch?

This patch only changes the GUI, but I also see a problem in a terminal.
Please check if this change is also needed after including patch 8.0.0924.
--
Living on Earth includes an annual free trip around the Sun.
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

patch 8.0.0924 doesn't fix this issue.

The cause of this issue is the calling GetMessage even though PeekMessage return false.

Included the patch, looks like it is fixed now.

Thank you @mattn, it works great now.


Special characters throw field-verbose-name-gettext-case

If a verbose_name contains special characters, they fail the islower() / isupper() / isdigit() check defined here:
https://github.com/kalekseev/django-extra-checks/blob/07eb62d3dc92431c43ec21f2b6da56946b2a8b52/src/extra_checks/checks/model_field_checks.py#L102-L104
In my particular example, this field throws the field-verbose-name-gettext-case error:
class Order(models.Model):
    details = models.TextField(
        _("order details / item list"),
    )

Perhaps adding the islower() check for the whole sentence will help?
            if (
                value
                and isinstance(value, str)
-                and not all(
+                and not (value.islower() or all(
                    w.islower() or w.isupper() or w.isdigit() for w in value.split(" ")
-                )
+                ))
            ):
                yield self.message(
                    "Words in verbose name must be all upper case or all lower case.",
                    hint=f'Change verbose name to "{value.lower()}".',
                    obj=field,
                )

Thank you for the report, fix released.

Perhaps adding the islower() check for the whole sentence will help?

Text like "abc / XYZ" is valid but won't pass this check.

Thank you for the report, fix released.

Thank you very much! That was quick 👍🏻

Perhaps adding the islower() check for the whole sentence will help?

Text like "abc / XYZ" is valid but won't pass this check.

Yeah that's why I OR'ed it with the old check and put the NOT over the brackets. This way, if the whole string is lowercase with special characters, it will work for the first half of the OR, otherwise the second half would be initiated. In any case, I find your fix much cleaner, so it doesn't matter 😅


Standard _Service bus:   "Messages content" option is missing from service bus trigger.

Describe the Bug with repro steps
1.Create a new Stateful Logic App(Standard) , select service bus trigger under the built-in connectors.
2.Select "When messages are available in Service Bus queue"
3. Add connection string of service bus.
4. Enter the queuename(test1) in the Queue name parameter.
5. Turn the Split on to off state and click on "Done"
6. Add "Send message" action from the service bus built-in action as below and use the 2nd queue(test2) as the queue input parameter and content from the "messages content" from the trigger. It will automatically add a foreach loop as it’s a batch trigger.
Expected : Content from "Messages content" option should be available from trigger.
Actual : Content from "Messages content" option is missing from trigger.
What type of Logic App Is this happening in?
Standard (Portal)
Are you using Preview Designer or GA Designer
Preview
Workflow JSON
{
  "definition": {
    "$schema": "https://schema.management.azure.com/providers/Microsoft.Logic/schemas/2016-06-01/workflowdefinition.json#",
    "actions": {
      "Send_message": {
        "type": "ServiceProvider",
        "inputs": {
          "parameters": {
            "entityName": "test2"
          },
          "serviceProviderConfiguration": {
            "connectionName": "serviceBus",
            "operationId": "sendMessage",
            "serviceProviderId": "/serviceProviders/serviceBus"
          }
        },
        "runAfter": {}
      }
    },
    "contentVersion": "<IP_ADDRESS>",
    "outputs": {},
    "triggers": {
      "When_messages_are_available_in_a_queue": {
        "type": "ServiceProvider",
        "inputs": {
          "parameters": {
            "queueName": "test1",
            "isSessionsEnabled": false
          },
          "serviceProviderConfiguration": {
            "connectionName": "serviceBus",
            "operationId": "receiveQueueMessages",
            "serviceProviderId": "/serviceProviders/serviceBus"
          }
        }
      }
    }
  },
  "connectionReferences": {
    "serviceBus": {
      "api": {
        "id": "/serviceProviders/serviceBus"
      },
      "connection": {
        "id": "/serviceProviders/serviceBus/connections/serviceBus"
      },
      "connectionName": "serviceBus",
      "authentication": {
        "type": "ManagedServiceIdentity"
      }
    }
  },
  "parameters": {}
}

Screenshots or Videos

Browser
Edge
Additional context
Environment Details:
URL: https://aka.ms/emastaging-flowstip1
Version: 2.30529.1.1

@vgouth - Content is messages content, did you try to add that and see if forloop is added?

@preetriti1 - I tried to add Content option as well, Still unable to see foreach loop added.

Issue not repro in stating environment. Closing this issue


Need to run "npm run stylua-custom"

          Need to run "npm run stylua-custom"

Originally posted by @luttje in https://github.com/luttje/glua-api-snippets/issues/16#issuecomment-1656987585

The issue doesn't seem to be that the command needs to be run. My code in #16 didn't take into account that some deprecation warnings would be multi-lined resulting in. https://wiki.facepunch.com/gmod/table.GetFirstKey creating this deprecation warning:
---[SHARED AND MENU] Returns the first key found in the given table
---
---[(View on wiki)](https://wiki.facepunch.com/gmod/table.GetFirstKey)
---@param tab table Table to retrieve key from
---@return any #Key
---@deprecated Instead, expect the first key to be 1.

Non-numerically indexed tables are not ordered and do not have a first key.
function table.GetFirstKey(tab) end

Oh thanks so much for catching onto this.
I'd love to help you out, but I'm not going to be able to for another two weeks or so. If you make PR's I'll merge them ASAP.
Thanks for your continued contributions


ReportStream ORM Conversion Tests

DevEx/OpEx
Implement tests in ReportStream to avoid ORM conversion failures in the future. See: https://github.com/CDCgov/trusted-intermediary/issues/815
Discussed during Engineering Block on 2/2/2024
Tasks

[ ] Add TI-Staging tests to alert us if something breaks
[ ] Pact testing

[ ] Add RS GitHub checks
[ ] Discuss Pact testing with RS

[ ] Something. Something. Something. Something.

Additional Context
Add any other context or screenshots about the work here.

My understanding is ReportStream has ORM conversion tests, but they the expected output was updated incorrectly. Replacing this effort with https://github.com/CDCgov/trusted-intermediary/issues/1255.


[test] increase coremark test timeout

This addresses #14330.
Signed-off-by: Timothy Trippel<EMAIL_ADDRESS>

After increasing the timeouts, the test not passes.


[nodejs] bootstrap steps failing

./.github/workflows/scripts/e2e-nodejs-push.sh: line 30: cd: nodejs/e2e-delegator-generic-workflow_dispatch-main-default-slsa3: No such file or directory

https://github.com/slsa-framework/example-package/actions/runs/5584039910/jobs/10205002714

This should be fixed by https://github.com/slsa-framework/example-package/commit/575bdc4566a9fb0bca9cff80d9bd88688ff36997 but need to check with @laurentsimon if this was intentionally set.


Can't see Toggler in Android Studio settings

Hi,
I'm using the latest version of Android Studio with Kotlin. The plugin works, but can't configure settings of it because Settings/Preferences -> Tools -> Toggler doesn't exist.

I hope you can solve this.

Hey!
Thanks for letting me know 👍. I'll look into the issue soon and let you know once I know more.

Hey @Cico-source, I just tried to replicate the above mentioned issue, but wasn't able to.
Using the following setup the Toggler configuration page did show up under the Tools section.

Freshly installed Android Studio Bumblebee

2021.1.1 Patch 2
Build #AI-211.7628.21.2111.8193401

Windows 10

Could you provide me with additional specifics about your setup so that I can try and reproduce the issue? Thanks!

Hi,
I'm using excatly the same setup. I tried the plugin on my paralelly installed Android Studio Canary edition and there it works normally. I'll be reinstalling Android Studio in a week, so I'll let you know whether it solved the problem.

Hey @Cico-source! Any update yet on the issue?

Hi, I'm really sorry for the delay. When I just reinstalled Android Studio, problem remained the same. But when I did a complete reinstallation where all my settings and plugins were removed, it worked.

@Cico-source Alright, thanks for the update 👍. I'll close the issue for now, it can be reopened if the same problem were to occur in the future.

I've noticed that when I install HighlightBracketPair plugin and restart Android Studio, Toggler disappears from settings.

I've noticed that when I install HighlightBracketPair plugin and restart Android Studio, Toggler disappears from settings.

Thanks for letting me know! I'll try and reproduce the issue soon.

Just an update:
I looked into the issue. @AprilViolet their HighlightBracketPair plugin once installed and once the IDE has been restarted does indeed remove Toggler's configuration page from the Tools section of the IDE's preferences.
From a brief look I wasn't able to find the cause. I'll put this on the backlog for now.

Hey @Cico-source! I revisited the issue today and was able to find the cause.
The identifier of Toggler's configuration menu was set to a default value which originates from the JetBrains plugin template. This identifier should be unique and because the HighlightBracketPair plugin also still uses the default value a conflict occured. I'll inform them about the issue aswell.
I've made the necessary changes to resolve the issue in 9835d4c236cf6b30e2c7b985e23b7b3476173e82 and rolled out version 1.2.8 which includes the fix. The new version is available in the releases section and will be available on the marketplace or as an update in your IDE after approval by JetBrains in about 2 business days.
The configuration menu now successfully shows up on my machine in Android Studio (I used Android Studio Bumblebee | 2021.1.1 Patch 2 on MacOS).
Once again, thanks for making me aware of the issue. If you have any feedback, then do let me know.


Can't load pretrained seer/swav checkpoints (RG_Y_128GF)

Instructions To Reproduce the 🐛 Bug:
Title
Issue when trying to load pretrained weights for Swav RegNet 128 GF

what changes you made (git diff) or what code you wrote
I followed the exact instructions here
https://colab.research.google.com/github/facebookresearch/vissl/blob/v0.1.6/tutorials/Using_a_pretrained_model_for_inference_V0_1_6.ipynb#scrollTo=NCwpxr5lNBQy

only change was to the config itself, as follows
cfg = [
'config=home/jrbb/xxx/xxx/models/RegNet_128gf.yaml',
'config.MODEL.WEIGHTS_INIT.PARAMS_FILE=/home/jrbb/xxx/xxx/swav/swav_RGNT128_pretrain.torch', #renamed chkpt
'config.MODEL.FEATURE_EVAL_SETTINGS.EVAL_MODE_ON=True',
'config.MODEL.FEATURE_EVAL_SETTINGS.FREEZE_TRUNK_ONLY=True',
'config.MODEL.FEATURE_EVAL_SETTINGS.EXTRACT_TRUNK_FEATURES_ONLY=True',
'config.MODEL.FEATURE_EVAL_SETTINGS.SHOULD_FLATTEN_FEATS=False'
]
All installations worked without errors

what exact command you run:
cfg = compose_hydra_configuration(cfg)
what you observed (including full logs):

Exception has occurred: MissingConfigException
Could not load config/home/jrbb/xxx/xxx/models/RegNet_128gf.yaml.
Available options:
resnet50_synthetic
File "/home/jrbb/vissl/vissl/utils/hydra_config.py", line 125, in compose_hydra_configuration
return compose("defaults", overrides=overrides)
File "/home/jrbb/xxx/xxx/training/utils.py", line 800, in build_dataset
cfg = compose_hydra_configuration(cfg)

please simplify the steps as much as possible so they do not require additional resources to
run, such as a private dataset.

Expected behavior:
If there are no obvious error in "what you observed" provided above,
please tell us the expected behavior.
No errors
Environment:
Provide your environment information using the following command:

sys.platform         linux
Python               3.9.12 (main, Jun  1 2022, 11:38:51) [GCC 7.5.0]
numpy                1.19.5
Pillow               9.1.1
vissl                0.1.6 @/home/jrbb/vissl/vissl
GPU available        True
GPU 0                Tesla V100-SXM2-16GB
CUDA_HOME            /usr/local/cuda
torchvision          0.13.1+cu102 @/home/jrbb/.conda/envs/py39/lib/python3.9/site-packages/torchvision
hydra                1.0.7 @/home/jrbb/.conda/envs/py39/lib/python3.9/site-packages/hydra
classy_vision        0.7.0.dev @/home/jrbb/.conda/envs/py39/lib/python3.9/site-packages/classy_vision
tensorboard          2.4.0
apex                 unknown
cv2                  4.5.5
PyTorch              1.12.1+cu102 @/home/jrbb/.conda/envs/py39/lib/python3.9/site-packages/torch
PyTorch debug build  False

PyTorch built with:

GCC 7.3
C++ Version: 201402
Intel(R) Math Kernel Library Version 2020.0.0 Product Build 20191122 for Intel(R) 64 architecture applications
Intel(R) MKL-DNN v2.6.0 (Git Hash 52b5f107dd9cf10910aaa19cb47f3abf9b349815)
OpenMP 201511 (a.k.a. OpenMP 4.5)
LAPACK is enabled (usually provided by MKL)
NNPACK is enabled
CPU capability usage: AVX2
CUDA Runtime 10.2
NVCC architecture flags: -gencode;arch=compute_37,code=sm_37;-gencode;arch=compute_50,code=sm_50;-gencode;arch=compute_60,code=sm_60;-gencode;arch=compute_70,code=sm_70
CuDNN 7.6.5
Magma 2.5.2
Build settings: BLAS_INFO=mkl, BUILD_TYPE=Release, CUDA_VERSION=10.2, CUDNN_VERSION=7.6.5, CXX_COMPILER=/opt/rh/devtoolset-7/root/usr/bin/c++, CXX_FLAGS= -fabi-version=11 -Wno-deprecated -fvisibility-inlines-hidden -DUSE_PTHREADPOOL -fopenmp -DNDEBUG -DUSE_KINETO -DUSE_FBGEMM -DUSE_QNNPACK -DUSE_PYTORCH_QNNPACK -DUSE_XNNPACK -DSYMBOLICATE_MOBILE_DEBUG_HANDLE -DEDGE_PROFILER_USE_KINETO -O2 -fPIC -Wno-narrowing -Wall -Wextra -Werror=return-type -Wno-missing-field-initializers -Wno-type-limits -Wno-array-bounds -Wno-unknown-pragmas -Wno-unused-parameter -Wno-unused-function -Wno-unused-result -Wno-unused-local-typedefs -Wno-strict-overflow -Wno-strict-aliasing -Wno-error=deprecated-declarations -Wno-stringop-overflow -Wno-psabi -Wno-error=pedantic -Wno-error=redundant-decls -Wno-error=old-style-cast -fdiagnostics-color=always -faligned-new -Wno-unused-but-set-variable -Wno-maybe-uninitialized -fno-math-errno -fno-trapping-math -Werror=format -Wno-stringop-overflow, LAPACK_INFO=mkl, PERF_WITH_AVX=1, PERF_WITH_AVX2=1, PERF_WITH_AVX512=1, TORCH_VERSION=1.12.1, USE_CUDA=ON, USE_CUDNN=ON, USE_EXCEPTION_PTR=1, USE_GFLAGS=OFF, USE_GLOG=OFF, USE_MKL=ON, USE_MKLDNN=OFF, USE_MPI=OFF, USE_NCCL=ON, USE_NNPACK=ON, USE_OPENMP=ON, USE_ROCM=OFF,

CPU info:

Architecture         x86_64
CPU op-mode(s)       32-bit, 64-bit
Byte Order           Little Endian
CPU(s)               8
On-line CPU(s) list  0-7
Thread(s) per core   2
Core(s) per socket   4
Socket(s)            1
NUMA node(s)         1
Vendor ID            GenuineIntel
CPU family           6
Model                79
Model name           Intel(R) Xeon(R) CPU E5-2686 v4 @ 2.30GHz
Stepping             1
CPU MHz              2627.823
CPU max MHz          3000.0000
CPU min MHz          1200.0000
BogoMIPS             4600.13
Hypervisor vendor    Xen
Virtualization type  full
L1d cache            32K
L1i cache            32K
L2 cache             256K
L3 cache             46080K
NUMA node0 CPU(s)    0-7

ASIDE: is there a better way to load pretrained weights? Previously I tried using the torchvision model for regnet128 with the VISSL checkpoint provided, but so many of the keys differed by more than the prefix that it seemed futile to continue down that path
Thanks :)

Will update this and then close
Seems like you need to reference the models in the vissl directly in the config - uploading your own to a different path does not work
So this
cfg = [
'config=home/jrbb/xxx/xxx/models/RegNet_128gf.yaml'
]
becomes this
cfg = [
'config=pretrain/swav/models/regnet128Gf.yaml'
]
My mistake


Plain eval is used without transpilation. Ts handlers do not work

Describe the bug
Plain eval is used without transpilation:
https://github.com/deep-foundation/js-docker-isolation-provider/blob/7d4ffde2b77ede4c50540c2710fe5fdabfc89ab2/index.ts#L31
Example Of Non-Working Typescript Handler
Handler Code
import { DeepClient } from<EMAIL_ADDRESS>import { Link } from<EMAIL_ADDRESS>import { DeepClientInstance } from "@deep-foundation/deeplinks/imports/client";

async (options: {
  deep: DeepClient;
  data: {
    newLink: Link<number>;
  };
}) => {
    return 5
};

Error
{
  "stack": "SyntaxError: Cannot use import statement outside a module\n    at eval (<anonymous>)\n    at memoized (/node_modules/lodash/memoize.js:62:23)\n    at makeFunction (file:///index.js:29:16)\n    at file:///index.js:61:20\n    at Generator.next (<anonymous>)\n    at file:///index.js:7:71\n    at new Promise (<anonymous>)\n    at __awaiter (file:///index.js:3:12)\n    at file:///index.js:56:33\n    at Layer.handle [as handle_request] (/node_modules/express/lib/router/layer.js:95:5)",
  "message": "Cannot use import statement outside a module"
}

Expected Behavior
No response
Current Behavior
No response
Steps to reproduce
No response
Environment
No response

Workaround 1
Use require or await import.


Compatiblity with xarray>2024.2.0 (DataArray object to construct a variable is ambiguous)

When processing RCM HH-HV file:
Traceback (most recent call last):                                                                                File "/home/datawork-cersat-public/cache/project/sarwing/commits/xsar_2024v7v30_1v2_0v2v3vpost13_1v0v8_1v0v6_1v0v6_1v2v2_1v0v3_1v0v1_1v2v0/grdwindinversion/bin/SAR_L1-to-L2_wind_processor", line 10, in <module>
    sys.exit(processor_starting_point())
  File "/home/datawork-cersat-public/cache/project/sarwing/commits/xsar_2024v7v30_1v2_0v2v3vpost13_1v0v8_1v0v6_1v0v6_1v2v2_1v0v3_1v0v1_1v2v0/grdwindinversion/
lib/python3.10/site-packages/grdwindinversion/main.py", line 50, in processor_starting_point
    out_file, outputds = makeL2(input_file, out_folder, config_file,
  File "/home/datawork-cersat-public/cache/project/sarwing/commits/xsar_2024v7v30_1v2_0v2v3vpost13_1v0v8_1v0v6_1v0v6_1v2v2_1v0v3_1v0v1_1v2v0/grdwindinversion/
lib/python3.10/site-packages/grdwindinversion/inversion.py", line 768, in makeL2
    xr_dataset, dual_pol, copol, crosspol, copol_gmf, crosspol_gmf, model_vv, model_vh, sigma0_ocean_cross, dsig_cross, sensor_longname, out_file, config = pr
eprocess(
  File "/home/datawork-cersat-public/cache/project/sarwing/commits/xsar_2024v7v30_1v2_0v2v3vpost13_1v0v8_1v0v6_1v0v6_1v2v2_1v0v3_1v0v1_1v2v0/grdwindinversion/
lib/python3.10/site-packages/grdwindinversion/inversion.py", line 700, in preprocess
    xr_dataset = xr_dataset.assign(nesz_cross_final=(
  File "/home/datawork-cersat-public/cache/project/sarwing/commits/xsar_2024v7v30_1v2_0v2v3vpost13_1v0v8_1v0v6_1v0v6_1v2v2_1v0v3_1v0v1_1v2v0/grdwindinversion/
lib/python3.10/site-packages/xarray/core/dataset.py", line 7145, in assign
    data.update(results_data_vars)                                                                                File "/home/datawork-cersat-public/cache/project/sarwing/commits/xsar_2024v7v30_1v2_0v2v3vpost13_1v0v8_1v0v6_1v0v6_1v2v2_1v0v3_1v0v1_1v2v0/grdwindinversion/lib/python3.10/site-packages/xarray/core/dataset.py", line 5703, in update                                                                                merge_result = dataset_update_method(self, other)
  File "/home/datawork-cersat-public/cache/project/sarwing/commits/xsar_2024v7v30_1v2_0v2v3vpost13_1v0v8_1v0v6_1v0v6_1v2v2_1v0v3_1v0v1_1v2v0/grdwindinversion/lib/python3.10/site-packages/xarray/core/merge.py", line 1055, in dataset_update_method                                                                           return merge_core(                                                                                File "/home/datawork-cersat-public/cache/project/sarwing/commits/xsar_2024v7v30_1v2_0v2v3vpost13_1v0v8_1v0v6_1v0v6_1v2v2_1v0v3_1v0v1_1v2v0/grdwindinversion/
lib/python3.10/site-packages/xarray/core/merge.py", line 699, in merge_core                                                                                collected = collect_variables_and_indexes(aligned, indexes=indexes)
  File "/home/datawork-cersat-public/cache/project/sarwing/commits/xsar_2024v7v30_1v2_0v2v3vpost13_1v0v8_1v0v6_1v0v6_1v2v2_1v0v3_1v0v1_1v2v0/grdwindinversion/lib/python3.10/site-packages/xarray/core/merge.py", line 358, in collect_variables_and_indexes
    variable = as_variable(variable, name=name, auto_convert=False)                                                                                File "/home/datawork-cersat-public/cache/project/sarwing/commits/xsar_2024v7v30_1v2_0v2v3vpost13_1v0v8_1v0v6_1v0v6_1v2v2_1v0v3_1v0v1_1v2v0/grdwindinversion/
lib/python3.10/site-packages/xarray/core/variable.py", line 136, in as_variable                                                                                raise TypeError(
TypeError: Variable 'nesz_cross_final': Using a DataArray object to construct a variable is ambiguous, please extract the data using the .data property.

Solved the issue using xarray 2024.2.0 and numpy 1.26.0

Fixed by https://github.com/umr-lops/grdwindinversion/pull/87


Create basic README

Created basic README.md with basic information about setup bot locally for development.

Thanks!


no CPU / RAM / Network usage

hi !
Your script works great with veeam information, but server informations (ram, cpu, network) stay greyed :/
I use veeam entreprise 11, do I need to do something to have this stats ?

thx !

Hello,
Thanks so much, to achieve that, you need to install the telegraf agent on
the server, full procedure can be found on the blog
https://jorgedelacruz.uk/2020/01/07/looking-for-the-perfect-dashboard-influxdb-telegraf-and-grafana-part-xix-monitoring-veeam-with-enterprise-manager-shell-script/
Thanks, let me know :)
On Fri, May 21, 2021, 7:17 AM eglyn @.***> wrote:

hi !
Your script works great with veeam information, but sevrer information
(ram, cpu, network) stay greyed :/
I use veeam entreprise 11, do I need to do something to have this stats ?
[image: image]
https://user-images.githubusercontent.com/24936778/119091180-f6ec7e80-ba0c-11eb-897d-5e99ea127492.png
thx !
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/jorgedlcruz/veeam-enterprise_manager-grafana/issues/8,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAMBAHDUBFZ5WHDAYPZ3ZTLTOX3IRANCNFSM45IPW7KA
.

thx ! it works !, I missed this step :)


fix: when multi container use device allocate fail

Fixes: https://github.com/Project-HAMi/HAMi/issues/212#issuecomment-2007150458
Kubelet is a loop to allocate GPU devices for multiple container requests using GPU resources, currently in PodAllocationTrySuccess method will check whether need release node lock, but hami.io/devices-to-allocate this annotation key is not use, so check failed, because current key is use hami.io/vgpu-devices-to-allocate.

Test result:

test.yaml

apiVersion: v1
kind: Pod
metadata:
  name: gpu-pod
spec:
  containers:
    - name: ubuntu-container
      image: chrstnhntschl/gpu_burn
      args:
      - "6000"
      resources:
        limits:
          nvidia.com/gpu: 1
          nvidia.com/gpumem: 500
          nvidia.com/gpucores: 3
    - name: ubuntu-container1
      image: ubuntu:18.04
      command: ["bash", "-c", "sleep 86400"]
      resources:
        limits:
          nvidia.com/gpu: 1
          nvidia.com/gpumem: 1000
          nvidia.com/gpucores: 10
    - name: ubuntu-container2
      image: ubuntu:18.04
      command: ["bash", "-c", "sleep 86400"]
      resources:
        limits:
          nvidia.com/gpu: 1
          nvidia.com/gpumem: 2000
          nvidia.com/gpucores: 30

run kubectl apply -f test.yaml result.

Thanks :)


Set the ^UnitTestRoot node when building the container

This is a follow-up to #415. It switches to using an iris.script file for the sequence of ObjectScript commands that need to be run when the container is being built, and adds the one that completes integration of InterSystems Testing Manager.

Pinging @isc-rkuszews as a potential reviewer.

Pinging @isc-rkuszews @daimor about getting this merged.

I'm not a maintainer anymore


HBASE-28109 Avoid NPE when starting up HMaster

There's a chance for rs to be NULL when starting up the HMaster. Then we ran into NPE exception
2023-09-18 14:17:40,452 ERROR [master/hmaster:16000:becomeActiveMaster] master.HMaster: Failed to become active master
java.lang.NullPointerException
        at org.apache.hadoop.hbase.master.HMaster.isRegionOnline(HMaster.java:1229)
        at org.apache.hadoop.hbase.master.HMaster.waitForMetaOnline(HMaster.java:1218)
        at org.apache.hadoop.hbase.master.HMaster.finishActiveMasterInitialization(HMaster.java:968)
        at org.apache.hadoop.hbase.master.HMaster.startActiveMasterManager(HMaster.java:2193)
        at org.apache.hadoop.hbase.master.HMaster.lambda$run$0(HMaster.java:528)
        at java.lang.Thread.run(Thread.java:750)

This PR add a simple check and make HMaster keep waiting instead of directly shutting down.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 32s
Docker mode activated.

Is this a fresh new cluster? Otherwise I do not think the region state for meta could be null...

@Apache9 Yes, it a fresh cluster start up.


Feature Request: Modern and valid code and markup

Description
A design system that is inclusive, modern and adaptive should conform to standards and improvements. This complies both to standards when it comes to best practices for accessibility in design, and also in code. It is often better to use solutions, such as markup, that are natively supported by browsers than to create new ones.
Additional Information
One such issue is in the Accordion component.
As I see it rendered, it contains markup consisting of divs and custom scripting to handle its functionality.
HTML 5 contains native accordion tags. The tags are supported by all modern browsers, and more than 97 % of all Norwegian users' browsers can handle the tags.
Example:

Expand me!
I'm expanded

<details>
<summary>Expand me!</summary>
I'm expanded
</details>

Awesome! Thanks for the suggestion :)
Our goal is to be as true to native html as possible so this would fits in nicely:)
We are planning on using the dialog element for our dialog/modal once we start developing that.
Don't think I have seen this element used in accordions before, but deffo interested to have a closer look at this!
Not sure when we will be able to have a look at this as we currently have a working Accordion and are missing other components entirely. Either after we reach our critical mass of components dubbed v1 or in between issues, well have to see :)
I am curious how easy it is to style and tweak these elements 🤔  Have you had any experience implementing this?

I am curious how easy it is to style and tweak these elements
There are some examples in the MDN link I provided. Also, the detailstag support the toggle event, so it would probably be easy to extend functionality on that if needed.


Add ConfigureAwait(false) to DCC

Not sure how I missed these, guess rider didn't show them for some reason.

99% sure it won't trigger on private methods, since the whole ConfigureAwait pattern is there to guard against consumers of an API that may not necessarily be following expectations. Not sure it's required here yet, haven't reviewed, but something to consider.

It does trigger rider now for some reason, underlining the entire massive block >_>. It triggered as a compiler warning if you use local-framework, but it was too late by that point.

Is the rule setup differently/incorrectly on the osu-framework project then?

Doesn't seem to be. Looks like the compiler warning is there in o!f, so discard what I said above:


Ripple effect animation speed

I was wondering if there is any way to change the ripple effect speed in LayoutRipple. Is there any documentation for this project?

To change the speed of the ripple you can go to line number 55 in ButtonFloat.java
and edit this line:
rippleSpeed = Utils.dpToPx(2, getResources());
change the 2 to a larger number and you will see the effect speed up =)

Thanks for the help rpdave

This is possible via xml?


c#

Type of issue
Typo
Description
[Insira comentários aqui]
Page URL
https://learn.microsoft.com/pt-br/dotnet/csharp/fundamentals/program-structure/top-level-statements
Content source URL
https://github.com/dotnet/docs/blob/main/docs/csharp/fundamentals/program-structure/top-level-statements.md
Document Version Independent Id
27a40713-4a2d-ca9f-8e9e-6f568ad8acdb
Article author
@BillWagner
Metadata

ID: 843bd4d7-cbb2-fca7-d4d2-f3ef027c167e
Service: dotnet-csharp
Sub-service: fundamentals

closing as likely opened by mistake


fix  uncle blokcs are orphaned

From the perspective of the integrity of the program code, conitnue is missing when confirming the uncle block, which leads to the wrong entry of if block.ConfirmationProgress > 0.75 , cause uncle blocks to become orphan blocks

As I currently do not have an ETC pool, could someone comment on how effective this fix is?

I think the logic makes sense, even though I never encountered that issue. But maybe I just got lucky.


Ignore some auto-generated DER files

24 hours has passed since 'approval: done' was set, but this PR has failing CI tests.  Once the tests pass it will get moved to 'approval: ready to merge' automatically, alternatively please review and set the label manually.

Travis failure is unrelated.

Merged
80627240638673eb605f48486b2651712690985f Ignore some auto-generated DER files

I literally pushed this at the same moment you did. You won. :-)

:rofl:


slack badge not loading in README on github

It is trying to load from http://slack.noms.io which is in DNS, but isn't responding to ping or HTTP requests.

I started slackin-nomsdb on the EC2 slack instance which fixed this for now. Not sure why it was down.

Maybe it can't handle the cod,e which would be ironic.


Progress report throws console error

See: http://sharing.beakernotebook.com/gist/anonymous/c0cc63591e9ac28bb603
Two possible bugs found with this notebook: One occurs when running the first cell of the notebook as is:
TypeError: Cannot assign to read only property 'status' of 4
    at Object.bkHelper.receiveEvaluationUpdate (http://<IP_ADDRESS>:8801/eb04f18/beaker/app/dist/beakerApp.js:2099:37)
    at Object.defineProperty.value [as print] (http://<IP_ADDRESS>:8801/eb04f18/beaker/app/dist/beakerApp.js:5353:20)
    at foo (eval at <anonymous> (http://<IP_ADDRESS>:8801/eb04f18/beaker/plugin/evaluator/javaScript.js:171:31), <anonymous>:5:12)
    at eval (eval at <anonymous> (http://<IP_ADDRESS>:8801/eb04f18/beaker/plugin/evaluator/javaScript.js:171:31), <anonymous>:17:3)
    at wrappedCallback (http://<IP_ADDRESS>:8801/eb04f18/beaker/app/dist/beakerVendor.js:46305:81)
    at http://<IP_ADDRESS>:8801/eb04f18/beaker/app/dist/beakerVendor.js:46391:26
    at Scope.$eval (http://<IP_ADDRESS>:8801/eb04f18/beaker/app/dist/beakerVendor.js:47434:28)
    at Scope.$digest (http://<IP_ADDRESS>:8801/eb04f18/beaker/app/dist/beakerVendor.js:47246:31)
    at Scope.$apply (http://<IP_ADDRESS>:8801/eb04f18/beaker/app/dist/beakerVendor.js:47538:24)
    at http://<IP_ADDRESS>:8801/eb04f18/beaker/app/dist/beakerVendor.js:49049:36

The second occurs if the type of the second cell is changed to R:
TypeError: Cannot read property 'result' of undefined
    at Scope.$scope.getOutputResult (codecelloutput-directive.js:34)
    at Object.get (angular.js:10903)
    at Scope.$digest (angular.js:12642)
    at updateLoadingStatusMessage (mainapp.js:66)
    at Object._impl.updateStatus (mainapp.js:570)
    at Object.bkHelper.updateStatus (helper.js:183)
    at doNext (evaluatejobmanager.js:163)
    at evaluateJob.then.job.finished (evaluatejobmanager.js:180)
    at _fulfilled (q.js:798)
    at self.promiseDispatch.done (q.js:827)

It looks like you cannot beaker.print('....') in a callback or deferred function.


Failed update database or manage migration using CLI

failed update database or manage migration using CLI
To Reproduce
Steps to reproduce the behavior:

follow quick start guide to create new project
update database using CLI with command "dotnet ef database update"
error message "unable to create object of type "BlazorHeroContext". for different patterns supported at design time , see  https://go.microsoft.com/fwlink/?linkid=851728"

Expected behavior
im expected this running well like using Package manage console (PMC) with command : "update-database"
Desktop (please complete the following information):

OS: Windows 64
Sql server : 2008 R2
Net Core : 5

@iammukeshm is this closed because only Visual Studio 2019 is supported and not dotnet ef database update? I was hoping to do my development on linux instead of windows if possible


Wildcard support in jar filters

Since I know my annotations exist only our in-house jars, I could benefit from saying:
new FastClasspathScanner("org.company.product", "jar:org-company-product-*").scan()

This is a good suggestion, I'll try to put this into the next release. How urgent is this for your current usage? It might take me a few days to get to this, given that I just got back from traveling.

This is now implemented in git, can you please test for your usecase and let me know if it works for you?

Closing, please re-open if it doesn't work for you.


Decouple reliance on docker sock?

I want to deploy this in a Kubernetes environment. I know I will be making lots of changes but it seems the base image expects a bind to the docker sock.
Could you help understand where this requirement comes in so I might remove it?
I see the base image the demo-kuma is built on is "louislam/dockge". But I believe I only need the node server. I'll be working on this in the meantime, but hope to hear back thanks!

Got it working by removing all the docker install steps from the base image.
Then replaced all the docker-compose create and delete commands with k8 api calls. I have session affinity managed at Cloudflare and run one demo-kuma instance on each of my clusters.
I really wish I could get it working with path prefixes instead of one domain per demo. But I gave up.


How do I load a local audio file?

The documentation mentions that files can be loaded from either a http or file url. I've not been able to load local files successful and the example doesn't seem to show how to achieve this?

how do you get file? from document picker?
which platform was you using? (android or ios)

it should be
 await load({
      uri: `file://{audio.fileUrl}`,
      key: audio.fileUrl,
      volume: audio.volume / 100,
      loop: false,
  });

play({
 uri: `file://${audio.fileUrl}`,
 key: audio.fileUrl,
})


Maked mobile version

Thank you！

I've finally made the mobile version!
(And sorry about having a mistake about the past simple of 'make'.)

I haven't test on my smartphone, so there may be some bugs. When this happens, please put the code back to the last commit.


Docs update (more extensions, new theme)

Use sphinx_book_theme instead of furo as it requires no custom css to look good for our purposes (and dark mode is coming soon)
Remove scanpydoc dependency
Remove nbsphinx in favor of myst-nb
Add typed returns custom extension like in scanpy
Use same autosummary class template as in scvi-tools (single page for classes with methods/attr tables)

See here for a live example.
Fixes #74 by using a different theme.

@grst it should pass but I can't figure out how to make it ignore rendering the class.rst file I added

To make this PR simpler I removed linkcode

@adamgayoso, question about the theme:
Weren't a number of the cases that we needed CSS for related to the dark theme?


Update EKS-A to support Flux v2.0.0

Synced with @taneyland offline and no EKS-A code changes required for this Flux version. Closing


previousVersionFile configured by a string property instead of a function #84

I created this PR to allow others testing it, but it's not thoroughly tested and shouldn't be merged!
Added releasing.previousReleaseVersion.
Removed interface RemoteUrlResolver, two different string properties, that you can set directly on task, are used instead - previousVersionPomUrl and previousVersionSourcesJarUrl

@szczepiq
I tested it with mockito on a few cases:

adding a System.out statement to one of src/main/java .java files -> release needed (sources jar modification)
adding a compile time dependency on guava to .gradle file -> release needed (pom modification)
adding a System.out statement to one of src/test/java .java files -> release NOT needed
adding an arbitrary dir to .gitignore -> release NOT needed

All tests (except .gitignore) done on mockito-core (root) and mockito-android projects.
Before each test I removed project .gradle dir and did ./gradlew clean assertReleaseNeeded. For comparing sources.jar it's useful to turn on "info" level of logging.
As soon as this PR is merged I will create a pull request in Mockito with the changes in configuration.
I think it's OK to enable it now. The sooner we start testing it on real products the better, but if you had a minute to see for yourself if it complies with your expectations it would be great!

I tested with 'example' project and it failed with exception: https://gist.github.com/szczepiq/e13e16392dc66d0cc2e0fbe8ce5c1015

I tested with Mockito project and found potential issues (failing comparePublications task). I'll unhook from the workflow and merge. There's a lot of good stuff in this PR that I don't want to wait :)


Transfer repository from youtalk to dynamixel-community

Hi @ijnek and @ROBOTIS-Will
Thank you for discussing releasing dynamixel_hardware package on ROSCon 2022. It was so exciting.

Separate this repository into two repositories: dynamixel_hardware and dyanamixel_hardware_examples to resolve https://github.com/youtalk/dynamixel_control/issues/36
Transfer their repositories to a new organization dynamixel_community.
Make release repositories dynamixel_hardware-release and dyanamixel_hardware_examples-release on the organization.
Release dynamixel_hardware package first!

What do you think? I want to add you to the organization's co-owners.

This is great and I would be happy to be a co-owner. Thanks @youtalk!

The release is ongoing! https://github.com/ros/rosdistro/pull/35400


update Vietnamese

Hi @sunner
just update new prompt to vietnamese

Thank you very much. The PR will be included in next release.


FT: Node v6

Switch to node v6

what about this?  https://github.com/scality/S3/issues/249

@LaurenSpiegel still not start to work on this :) We have node v6 with binary encryption everywhere for the moment :)

please update readme as well and have lots of end to end fun :)

Once this is forward ported to master, the update will have to be made in this branch: https://github.com/scality/S3/pull/591/files#diff-c945a46d13b34fcaff544d966cffcabaR162

@LaurenSpiegel updated :)

The readme too...

@LaurenSpiegel damn ^^ will modify it ^^

updated :)

@ironman-machine
SCALITY_METADATA_BRANCH=ft/node-v6
SCALITY_INTEGRATION_BRANCH=ft/node-v6
SCALITY_VAULT_BRANCH=ft/node-v6
try

Hello @alexandre-merle
Starting end to end procedure using the following payload:
{"SCALITY_VAULT_BRANCH": "ft/node-v6", "REPO_NAME": "S3", "DEFAULT_BRANCH": "master", "SCALITY_METADATA_BRANCH": "ft/node-v6", "SCALITY_INTEGRATION_BRANCH": "ft/node-v6", "SCALITY_S3_BRANCH": "ft/node-v6"}
Please follow http://ci.ironmann.io/gh/scality/Integration/10016 for CI status.

Thank you, @alexandre-merle!!


Compatability with networkx is broken

Networkx main has dropped support for Python3.8 and 3.9. We should consider following the Scientific-Python's community spec: https://github.com/networkx/networkx/pull/7028
Reasoning:
This makes it very hard to support both dowhy and networkx related features using Python packaging like poetry because:
python>=3.8, <3.12 is incompatible with python>=3.10, <3.12
Networkx is core for graph-related functionality in pywhy-graphs too, so tight-integration with networkx:main is desirable. Moreover dowhy relies on graph-related functionality since it is a package for graph-based causal inference tasks.
For that matter, major SciPython package like numpy and scipy have dropped support for Python3.8

@adam2392 Thanks for bringing up this issue and the PR to drop python 3.8 support.

I would definitely wait for dropping 3.8 support and restrict the networkx version. Even 3.8 is not that commonly used yet. So, taking out that support would probably break the dependency for many people.

I would definitely wait for dropping 3.8 support and restrict the networkx version. Even 3.8 is not that commonly used yet. So, taking out that support would probably break the dependency for many people.

I'm actually surprised by that because Python 3.7 lost support earlier this year already for a few months, so if a new bug/security-issue is found, it won't even be fixed, which is scary to me.
I'm just wondering along the lines: "is there a way to push our users towards a more healthy/less-risky usage of Python?"

Yea, here's the official timeline of support for python versions.  3.8 is already in security-fix-only mode and is going to end-of-life in about 11 months.
https://devguide.python.org/versions/
I agree that keeping track with networkx and other major dependencies would be good.  Do we know if Numpy, Pandas or other ML libraries that EconML and causal-learn depends on switching over on a schedule too?

I think numpy, pandas, scipy have all removed support for Python3.8 at this point on main. I think scikit-learn is in the process as well too.
Their general strategy is support older versions of X (e.g. Numpy) that have Python3.8 compatibility only for critical bug-fixes, but otw, all new features must be Python3.9+ compatible.
Since Python3.8 is technically still supported in version 0.10.x, and if v0.11.x is on a release schedule that overlaps w/ Python3.8 eol, it seems reasonable to deprecate python3.8 support in v0.11.xdev, while still maintaining support for 3.8 via v0.10.x and below. Up to you all! Just wanted to bring it some attention because this issue came up in pywhy-graphs, because I maintain extremely tight integration w/ networkx.

Ok, we might aim at 0.11 release in December.

Now that we are past the 0.11 release, do we want to revisit python 3.8 support in main?  @adam2392 @bloebp @amit-sharma

Yes, we can drop 3.8 support for 0.12+


Test: Add a test suite for api/news/regions:regionId

What would you like to share?
Add a test suite for the /:regionId endpoint.
Steps:

Add a new describe block for /api/news/sources inside /__tests__/integration-tests/routes/news.test.js
Add test suite for all possible scenarios

Additional information

Note
All axios requests are mock inside the __mocks__ folder
Fell free to add more mock data inside that file

Hi there, I'm interested in this issue. Please assign this to me.


fix: ArgoCD server resources limit/request changes in ArgoCD CR is not reconcilied

Signed-off-by: iam-veeramalla<EMAIL_ADDRESS>What type of PR is this?

/kind bug

What does this PR do / why we need it:
Users can modify argocd server container resource limit/request in the ArgoCD CR.  The operator should apply/reconcile the changes to the cluster.   However, reconciler is not checking for changes in ArgoCD CR against the running state due to a bug.
This PR adds a check in the reconcile function to check for changes in ArgoCD CR on argocd server container resource limit/request against the cluster and set the "changed" flag accordingly.
Have you updated the necessary documentation?
NA
How to test changes / Special notes to the reviewer:

Create ArgoCD CR in the foo namespace.
Record the resource limit/request on the running argocd server pod
Update the ArgoCD Server container resource request/limits to different values
Verify that the resource request/limits changes have been applied to the cluster not updated on the container.

LGTM
thanks @iam-veeramalla !


Updated the User and Behavior Analytics workbook

Change(s):

Updated the existing User and Entity Behavior Analytics workbook to read from Anomalies, instead of BehaviorAnalytics; to get anomalies for IPs and Hosts (on top of Users, which were already there), to get incidents with entities that match anomalies.

Reason for Change(s):

Replacing BehaviorAnalytics with Anomalies, as that is the actual source for anomalies
Adding further information to make the workbook more valuable

Version Updated:

Yes, to 2.0 as these are big changes

Testing Completed:

Yes

Checked that the validations are passing and have addressed any issues that are present:

Yes

Guidance <- remove section before submitting

Before submitting this PR please ensure that you have read the following sections and filled out the changes, reason for change and testing complete sections:
Thank you for your contribution to the Microsoft Sentinel Github repo.

Details of the code changes in your submitted PR.  Providing descriptions for pull requests ensures there is context to changes being made and greatly enhances the code review process.  Providing associated Issues that this resolves also easily connects the reason.

Change(s):

Updated syntax for XYZ.yaml

Reason for Change(s):

New schema used for XYZ.yaml
Resolves ISSUE #1234

Version updated:

Yes
Detections/Analytic Rule templates are required to have the version updated

The code should have been tested in a Microsoft Sentinel environment that does not have any custom parsers, functions or tables, so that you validate no incorrect syntax and execution functions properly.  If your submission requires a custom parser or function, it must be submitted with the PR.

Testing Completed:

Yes/No/Need Help

Note: If updating a detection, you must update the version field.

Before the submission has been made, please look at running the KQL and Yaml Validation Checks locally.
https://github.com/Azure/Azure-Sentinel#run-kql-validation-locally

Checked that the validations are passing and have addressed any issues that are present:

Yes/No/Need Help

Note: Let us know if you have tried fixing the validation error and need help.

References:

Guidance for Detection checks
General contribution guidance
PR validation troubleshooting

Pull request to update the User and behavior analytics workbook to version 2.0.0


Milestones Type bug

fixes #128
Currently the UI does not persist the milestone type on postback so the user would have to reselect type if they do not choose DECEASED or DROPPED on post. This behavior was dictated by the client side validation. To address this bug, I added MilestoneType as a property that is sent to the API and stored. So MilestoneType is now returned with its stored value instead of dictated by JS. I also updated the JS to remove this functionality.
This will need to be run with PR # 72 from the API
To test:

Go to Participation -> Create Milestone
Select 0 for question: Which milestone type are you reporting?

Click save and force validation

Check Q: Which milestone type are you reporting? - You will see that the milestone type persist correctly

Ran into an interesting error message from the GitHub linter about our javascript file form the Prettier package. It doesn't provide further documentation on the formatting errors, but it tells you the command you need to run using an npm package. The package can run from your local machine and not the project, so it can be used without adding node files to the project.
because I have Node on my machine, I ran the prettier command on the file and pushed up the changes for the test to pass. Other devs may have to do this in the future

@ashleybot Thanks for catching that. When I tested the form for when MILESTONE TYPE = 1 I entered 99 for the day on that date input so that funky validation didn't kick on. When I went back to test I could see that it was wonky. I got that validation resolved for CHANGEDY. I noticed issues for  ACONSENT as well so I got the validation resolved for that input. I didn't see any code for validation for questions 3 and 3a so left it alone but if there is some validation needed I will update it.

If not this is ready for another review.

@ashleybot Im getting failure for package release. I updated API to preview 4.0.1-preview.3 and that includes all the changes that Marvin has made to the A1. So when I update my package references in the Web app to 4.0.1-preview.3, I am missing the definition to the properties that Marvin had taken out for the A1. I believe Marvin has to have his PR for the Web project merged into main, and then I need to merge main into my feature branch to get this package error to pass.


Babel deoptimized the styling the styling - template.js more then 100kb

I've been searching on how to deal with this, but haven't managed to find a solution.
[BABEL] Note: The code generator has deoptimised the styling of "PATH/templates.js" as it exceeds the max of "100KB".

Currently I don't know what problem it produces (performance, or something else). It appears twice in the log when I run any of the tasks.

Hmm, try to change this line to:
 { 'name':babelify, 'options': { 'compact': 'true' }},

That works, great. Thanks! :)

That works, great. Thanks! :)


allow queryParams to optionally be null

Closes #637
This PR adds the ability have nullable typed query params.
val myThing: MyThing? = context
    .queryParam<MyThing>("my_thing")
    .getOrNull()

Great, thank you !

This has been released as part of 3.1.0 now: https://javalin.io/news/2019/06/23/javalin-3.1.0-released.html


[BUG] Link for Popular TT songs not work

This link on description isn't working :
https://filebin.net/klylrens0uk2pnrg/drive-download-20240209T180019Z-001.zip

https://www.swisstransfer.com/d/d1bbfb04-b546-472b-a864-1c324f1e5bc9

This link dosent work too

Can someone reupload please?


PDF Document viewer - current document disappears every time

There is a vexing flaw in the PDF Document viewer: whenever you move away the viewer (close the tab) and opens it again the current document disappears and you have to invoke it again by choosing from the pinboard - EVEN IF THERE IS ONLY ONE DOC IN THE INBOARD.
Which is irritating and time consuming.

The problem happens ,whenever you jump to another taxon, which hard-forces the viewer to close.
And the document restarts to slowly load every single time...

@mjy is this issue pointed out by @adrik29 something we can address or is it just that the PDF Viewer software we use inside TW has issues we can do nothing about?

I think its possible saving the current state of the pdf viewer using local Storage, but i'm not so sure if we can't avoid to download the pdf again.

same as #368

@adrik29 I pushed some changes for this in the development branch. You will see the changes in the next production update.


Fixing IdfPassageScorer

Fixed the bugs in IdfPassageScorer with the correct implementation.
Removed stopwords from the both the question and the answers.

Remove src/main/scala/WmdPassageScorer.scala from commit?


watch: "user." drops from intelisense menu

I had a "user." entry on the watch panels intelisense menu but then it dropped off after I restarted the bot:

@Stevenic can you give us some repro steps to better diagnose this?

Is this because there is only one "user" property and once it's added the "user" menu is removed from the picker?
https://user-images.githubusercontent.com/39317542/125499275-0139e7a5-f912-4bec-80d6-38124a7176fc.mov

That could be... I'll close for now


Feature: Cut over to v2 API

Related: https://github.com/open-sauced/api/issues/478

The AI service should cut over to the new v2 API when we make that release. Example:
https://github.com/open-sauced/ai/blob/82794666d87004bb4b53390135bf0230563e806c/.env.production#L2
For the most part, for the AI service, this should just be a 1:1 replacement of v1 with v2 but may require abit more investigation to make sure it works with the v2 API before we make the cut over.

Closed by: https://github.com/open-sauced/ai/pull/304


TVR-13514 update AgencyProposalSegment swagger definition

Hi @quancheng-zhao. It looks like this Swagger is throwing a few errors for us. You can use the SAP API Hub validator: https://openapi-validator.internal.cfapps.sap.hana.ondemand.com/ to see what errors we're seeing. Thank you.

Hello @jonikamelcher I cannot access the SAP API Hub validator, but I fix the problem from official swagger online editor. Could the last version pass in API validator? Thank you

:shipit:


Darwin-x64 nightly build isn't running

Nightly build for macOS can't be launched.
In Info.plist defined, that Executable name is Electron, but for nightly build executable is named Camunda Modeler. That causes error "App is Damaged Can't Be Opened”.
Simple renaming solves the problem.

Thanks for reporting, we'll look into this.


[release-1.16] tracing: Update proxyConfig.Tracing merge logic (#42518)

Please provide a description of this PR:
fixes: #44016

@zirain: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio_release-1.16
4f60cf45c1d5993f9112271fa7d20c2d14926a28
link
true
/test unit-tests-arm64

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@zirain: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio_release-1.16
4f60cf45c1d5993f9112271fa7d20c2d14926a28
link
true
/test unit-tests-arm64

unit-tests_istio_release-1.16
4f60cf45c1d5993f9112271fa7d20c2d14926a28
link
true
/test unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@zirain: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio_release-1.16
4f60cf45c1d5993f9112271fa7d20c2d14926a28
link
true
/test unit-tests-arm64

unit-tests_istio_release-1.16
4f60cf45c1d5993f9112271fa7d20c2d14926a28
link
true
/test unit-tests

gencheck_istio_release-1.16
4f60cf45c1d5993f9112271fa7d20c2d14926a28
link
true
/test gencheck

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


fix(transformer/async-generator-functions): incorrect transformation for for await if it's not placed in a block

Found in https://github.com/oxc-project/monitor-oxc/actions/runs/11681867380/job/32527898715

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#7148  👈
#7147
#7135
#7138
#7134
#7132
#7131
#7125
#7113
#7107
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @Dunqing and the rest of your teammates on  Graphite

Merge activity

Nov 6, 12:50 AM EST: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.


Do not copy WAL for lagged slots

Problem
See https://neondb.slack.com/archives/C026T7K2YP9/p1702813041997959
Summary of changes
Do not take in account invalidated slots when calculate restart_lsn position for basebackup at page server
Checklist before requesting a review

[ ] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.
[ ] Do we need to implement analytics? if so did you add the relevant metrics to the dashboard?
[ ] If this PR requires public announcement, mark it with /release-notes label and add several sentences in this section.

Checklist before merging

[ ] Do not forget to reformat commit message to not include the above checklist

Unfortunately original approach with checking invalidated field doesn't work. It is not set!
Postgres set wal_status to extended based on comparison of restart LSN with current WAL position.
It doesn't not actually invalidate slot.
So I have to ignore max_slot_wal_keep_size parameter and perform this check in page server.
It may be better to move this parameter to config, but it requires so much changes, that right now it is just hardcoded.


Fix Faker::Vehicle Array Access

Just a little fix accessing the array.
I don´t know if this is the best way to fixed, feel free to comment.
Fixs #627

See PR #644

@rui-ribeiro Your PR is right. But test cases related to Vehicle model is faild. The reason behind it is, when you access this
manufacture = fetch_all('vehicle.manufacture').sample
In your app, it returns hash whose keys are symbols. And when you run test cases it returns hash whose keys are strings. Strange behavior.

Yes, exactly that. I don´t know why it behaves like that, must me some require or extension.
That´s why i referenced PR #644 because of the conversion of the hash for indifferent hash.

Another solution for this problem is, You can use or( || )  in you code.
For Example: In vin method you can write like
vehicle_identification_number = (manufacture['wmi'] || manufacture[:wmi]).split('').concat( Array.new(14) { c.sample } )
It will solve both problems. thanks


Add --help to "docker volume inspect --help" output

Closes #16146
While in there, modified the testing infrastructure for the help text
so that we can test commands with nested commands - like "volume".
Signed-off-by: Doug Davis<EMAIL_ADDRESS>

LGTM

LGTM


Idea: Bring back all_or_nothing

The conflict resolution is not atomic. Even if all conflicts are deleted along with updated document by a single _bulk_docs request, the data might be corrupted as result. There is a problem with validation function. If the validation fails for the updated document, conflicts are deleted, but main document is not updated. If the sender is unable to solve such situation immediately, the data from the conflicted documents are lost.
Please bring back all_or_nothing, or a similar feature. It should just process validation for the all updated documents before they are processed normally. The implementation should be easy, the validation functions should be available on all nodes, so there is no barrier to validate these documents on any of these nodes.

This would require a distributed transaction and is not something we anticipate bringing to CouchDB.
Sent from my iPhone

On 26 Mar 2018, at 10:56, Ondrej Novak<EMAIL_ADDRESS>wrote:
The conflict resolution is not atomic. Even if all conflicts are deleted along with updated document by a single _bulk_docs request, the data might be corrupted as result. There is a problem with validation function. If the validation fails for the updated document, conflicts are deleted, but main document is not updated. If the sender is unable to solve such situation immediately, the data from the conflicted documents are lost.
Please bring back all_or_nothing, or a similar feature. It should just process validation for the all updated documents before they are processed normally. The implementation should be easy, the validation functions should be available on all nodes, so there is no barrier to validate these documents on any of that nodes.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Nonsense. Where you see requirement of distributed transaction? I only need to perform validation which doesn't need to be distributed. Please reopen.

Because the docs in the request will map to different shards on different nodes. We can’t do the old school all or nothing  semantics without some consistency protocol.
The current all or nothing semantics is as good as it’ll get.
Sent from my iPhone

On 26 Mar 2018, at 19:25, Ondrej Novak<EMAIL_ADDRESS>wrote:
Nonsense. Where you see requirement of distributed transaction? I only need to perform validation which doesn't need to be distributed. Please reopen.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

This is totally untrue or not understood correctly. So let's change the name of the feature: "pre-validation".
Any of the shard just received the _bulk_docs request:
As the first, It performs validation through all validation functions. This must be at least done on all shards, so the validation functions must be available on all shards as well. So the validation functions must be also available on the current shard. The validation functions are called for the all documents in _bulk_docs request - one by one - there is no mapping to shards. It just perform validation.
Now, if there is a validation error, whole request is rejected.
After passing the "pre-validation", the request continues normally.
(pre-validation feature is opional similar to all_or_nothing]

Perhaps we're talking past each other.
CouchDB's original all_or_nothing semantic was that all the updates would be applied to the database or none would be. This was possible because there was a single file and an atomic operation to update the b+trees.
When we added clustering, this semantic could no longer be honored without a consistency protocol / distributed transaction.
The current all_or_nothing behaviour is that the changes will be written even if they introduce conflicts. Not an obvious semantic.
Your point about validate_doc_update functions is separate, the all_or_nothing behaviour worked whether there were VDU's or not. Perhaps there's a case for not processing a _bulk_docs request if any item fails the validation but it runs against our explicit choice of availability (for reads and for writes) over consistency, and the long-held position that the unit of consistency is the individual document. It's also not guaranteed to be the case that all the writes would succeed if the validate_doc_update functions pass for every document in the _bulk_docs request.
In your original comment;
If the validation fails for the updated document, conflicts are deleted, but main document is not updated. If the sender is unable to solve such situation immediately, the data from the conflicted documents are lost.

I don't recognise that behavior for CouchDB. CouchDB does not discard successful writes after the fact, so if you want to expand on what you observed there perhaps there's something to learn.

Okaj, let's assume that this is about a new feature.

We have a document with conflicts.  There is main revision and the one or more conflicted revisions (deleted conflicts are not included)
Let there is some automate process which is able to merge the most of conflicts
The automate process creates a new main document and then a set of delete requests for conflicted revisions. The conflicted revision is deleted by simple putting _deleted flag into it.
The _bulk_docs request is prepared and send to couchdb

question of the ordering? Is it better to put main revision as the first or as the last? With clustered database, this is hard to decide. In my code, i always put the main revision as the last, because if the changes are applied in the order - it is single document mapped to single shard(?) - the changes feed will contain deleted conflicts as the first and then the updated main revision.

Now, we have VDU. The purpose of  VDU is also to detected malformed merged documents.
however, the VDU rejects the malformed document, but it cannot reject deletion of the conflict.
As the result, the conflicted revisions are deleted (and lost after compaction). The main revision is not updated, so updates are also lost.

The workaround is to reconstruct conflicted documents back using the data temporary stored in the memory of that automated process -  but this is very ugly solution. It would be better to call VDU on the main revision and test it before the write
Hmm, other workaround is to send main revision as the single request and delete conflicts only if the requests passes. This has other issues. It needs two request. The new revision of the main document can be replicated and it still looks like conflicted, because conflicts has not been deleted yet. There can be also another process (for instance on other replica) which will try to merge conflicts with already updated revision. So this is the reason, why I am sending  updated document as the last item
To improve the work with conflicts i have just another suggestion, the field "_merged", but this will need an another ticket and looks more complicated than above suggestion.

We have considered a (fundamental) change within CouchDB that would allow a revision to have multiple parents. This would change conflict resolution from what you describe (posting a new merged revision and then deleting the unwanted siblings) to a single doc update, which (should it pass all the VDU's) would apply the 'merged' update and resolve all the conflicts.
Are we describing the same problem now?

Sounds promising. I agree. It describing the same problem and your proposal would solve it definitely.

Actually I have come up with a similar thought when thinking about the problem. The purpose of the "_merged" field would be to specify multiple parents. Once the document with such field is received, the database would "purge" opened revisions that are mentioned in the list. This would also solve problem with multiplication of deleted conflicts that are still considered as opened.
The main benefit of such idea is that removing conflicts along with the document update would be atomic and no more deleted conflicts would be left opened.


Weighthooks

where will the weight hooks go in ACEfit.jl?

As optional input to the fitting routines, I think

default_weighthooks in ACE1pack ...

seems tentatively addressed by #8 but keep upen until finalized and merged.


issue-300: Sync funtion accepts context, created corresponding tests

New SyncWithCtx function that can accept context as an additional parameter
Created new tests for this function by reusing the existing ones

Thanks for the review @at-wat , I've addressed your comments.


Valgrind error (might be an Oryol issue)

==980== Conditional jump or move depends on uninitialised value(s)
==980==    at 0x4C31D52: __memcmp_sse4_1 (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==980==    by 0x47A063: Oryol::_priv::glVertexAttr::operator!=(Oryol::_priv::glVertexAttr const&) const (glVertexAttr.h:35)
==980==    by 0x476A52: Oryol::_priv::glRenderer::applyMeshes(Oryol::_priv::pipeline*, Oryol::_priv::mesh**, int) (glRenderer.cc:376)
==980==    by 0x477007: Oryol::_priv::glRenderer::applyDrawState(Oryol::_priv::pipeline*, Oryol::_priv::mesh**, int) (glRenderer.cc:475)
==980==    by 0x45EA43: Oryol::Gfx::ApplyDrawState(Oryol::DrawState const&) (Gfx.cc:153)
==980==    by 0x489841: Oryol::_priv::debugTextRenderer::drawTextBuffer() (debugTextRenderer.cc:138)
==980==    by 0x4890BB: Oryol::Dbg::DrawTextBuffer() (Dbg.cc:87)
==980==    by 0x44B5AF: VoxelTest::OnRunning() (Main.cc:183)
==980==    by 0x4A5263: Oryol::App::onFrame() (App.cc:180)
==980==    by 0x4A4F67: Oryol::App::StartMainLoop() (App.cc:93)
==980==    by 0x44A8FF: main (Main.cc:43)
==980==

Hmm strange, I've added a few more asserts in glRenderer::applyMeshes(), but they don't trigger. Also Address Sanitizer on OSX runs cleanly.
I'm seeing uninitialized value messages on OSX before on every call into a system DLL though. I wonder if these are false positives...

yes, driver dlls e.g. are full of errors - but this is not in the driver, this is in glVertexAttr.h.

But looking at the code I didn't seeing any obvious error(?) The glVertexAttr.h constructor initializes all members, and the mesh pointer inside glRenderer::applyMeshes() should always point to a valid mesh object (that's what the new asserts ensure), clang static code analysis and address sanitizer also comes out clean, at least in Oryol code, so I'm not sure what to fix :)
There are a few static code analyzer warnings in stb_rect_pack, stb_true_type, imgui and turbobadger (turbobadger actually looks pretty bad, one potential null-pointer access and 10 potential memory leaks).

Argh, I know what it is... the compiler will add padding bytes after those 7 uint8's, and these aren't initialized with anything. I'll remove the memcpy with a proper member-by-member-comparison...

Ok, can you check with this fix please?
https://github.com/floooh/oryol/commit/8ea76c604cd8cd787157e0e5488b34fe29e2f8de
Thanks for catching this, I checked for other memcmp() and didn't find any other suspicious uses.

fixed - thanks a lot


fix(app): Update robot rename rule

Overview
This will fix #10214
The new rule is that allows users to use a-z, A-Z and 0-9.
regex
/^([a-zA-Z0-9]{0,35})$/
https://regexr.com/

Changelog

Updated RenameRobotSlideout regex
Updated the test for a new regex pattern
Cleaned up device_settings

Review requests

Risk assessment
low

user-facing text strings look good 👍


Set qt to zero for dry EDMF cases and add GABLS

Purpose

To-do

Content

[ ] I have read and checked the items on the review checklist.

This seems like an easy way to make EDMF run dry case with qt=0. @trontrytel Let me know what you think.

looks good to me. But it also looks like Bomex box breaks from --nh_poly 1

looks good to me. But it also looks like Bomex box breaks from --nh_poly 1

Changing the domain size (which has a similar effect to changing hyperviscosity) make Bomex work but break Rico:( I am a bit unsure about using hyperdiffusion for poly order 1, and also it doesn't seem to help reduce ci time much, so I will just keep the default poly order 3 for now.

bors r+

bors r+


Update dropdown.md to show of dict options and selected_key

📝 Summary
I had to read the docs like 8 times to figure out what selected_key does, at first I thought it was a vestigal name for value. So here is an update to the interactive example (so cool) that shows off more features.
Also the name value for the argument to pick the default choice of the dropdown takes the key from the options dict, so the name is misleading.
🔍 Description of Changes

📋 Checklist

[x] I have read the contributor guidelines.
[x] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on Discord, or the community discussions (Please provide a link if applicable).
[x] I have added tests for the changes made.
[x] I have run the code and verified that it works as expected.

📜 Reviewers

@akshayka OR @mscolnick

Thanks @ggggggggg !


Restrict naming rule can be letters only for X*, S*, SX* and Z*

All of naming rule in current spec only said: name beginning with a letter, it a little too loosely, I purpose it should be more restrict, the name can be only consists of letters.

Standard extensions can also be named using a single “Z” followed by a name beginning with a
letter and an optional version number.

Non-standard subsets are named using a single “X” followed by a name beginning with a letter
and an optional version number.

Standard supervisor instruction-set extensions are defined in Volume II, but are named using “S”
as a prefix, followed by a supervisor subset name beginning with a letter and an optional version
number.

Agreed, I'm updating the text to replace "name beginning with a letter" with "alphabetical name"


[FEATURE]: Create the LinkPagination component for the Storybook

Is your feature request related to a problem?
This component is part of a reusable design system for Fonoster's dashboard. It helps achieve consistency and reduces repetitive implementation work.

Describe the solution you'd like
The LinkPagination should follow the design in the Figma file and be implemented as a standalone React component with the following props: onClick, disabled.

It should use MUI styles for styling
The component must have basic tests using

Describe alternatives you've considered
NA
Additional context
Before working on this item, please check the CONTRIBUTING document. Details about the LinkPagination component can be found here.
Acceptance Criteria:

[ ] Matches Figma design
[ ] Storybook story with documentation
[ ] Works in light/dark themes
[ ] Includes basic unit tests

All yours, @parz3val.

I've opened a pr for this. Sorry for delay, I was a bit busy. #696


factory: start proxy before assign vm to a sandbox

Right now proxy is always started after we assign the guest vm to a sandbox. But then we will have to wait again for the proxy to start up. We can move the creation of proxy to vm factory so that new vms are created with configured proxy type, and then wait for both proxy and vm/agent in parallel. This will save about 300ms startup time for vm factory.

FYI, the PR is verified in kata-containers/tests#578 and it all passed.

@jodh-intel when we close connection to the agent, it needs sometime to cleanup and restart listening on the communication( serial or vsock) port. That time can be saved if we sleep a bit to wait for the agent to come around and start listening again. The sleep is only done when creating new vm templates and saves time for every new vm that are created from template, so it worth the invest.

//recheck

@bergwolf would be nice if you could reduce the coverage drop ;)

UT added and CI green. @sboeuf @amshinde PTAL.

@sboeuf @amshinde Are you happy with the PR now? Please merge it if so.

@bergwolf also, Travis seems completely stuck, could you re-push this PR to unblock it?

@sboeuf Thanks for reviewing. Repushed and opened #728.

@amshinde PTAL

@jodh-intel @egernst @WeiZhang555 PTAL, we need another approval to this PR if you think it's ok to be merged. And please merge it as Travis is stuck for no reason, otherwise the CI is all green.

Not familiar with this part, so I think I should trust @sboeuf 's judgement :smile:
LGTM


Allow F12 to be bound to both go to definition and open link

I tried to get this working with this but I don't think we have a context key for this:
{ "key": "f12", "command": "editor.action.openLink", "when": "!jumpToCursorSupported"}

I think ideally this is what I'm after:
{ "key": "f12", "command": "editor.action.openLink", "when": "linkAtCursor"}

I've been after this for a while as it would make using markdown much nicer imo.
cc @lostintangent (tried to get it working for wikilens)

Actually I think both of these do what I'm after:
{ "key": "f12", "command": "editor.action.openLink", "when": "editorLangId == 'markdown'" },
{ "key": "f12", "command": "editor.action.openLink", "when": "!editorHasDefinitionProvider" },

Should we add one of these to core?


Remove deadlock that was caused by calling pthread_rwlock_wrlock()

I have try to build current master with musl instead of glibc and I have found that fuzz test will get stucked. Issue seems to be deadlock caused by calling pthread_rwlock_wrlock() on same thread. I guess glibc is not following POSIX so strictly, because by description this may cause this behaviour "The calling thread may deadlock if at the time the call is made it holds the read-write lock (whether a read or write lock)."
CLA: trivial
Signed-off-by: Matus Kysel<EMAIL_ADDRESS>

Agreeing that this is trivial.

Yup

Merged to master.  Thanks for the contribution.


parser_cli

System information

**OS Platform and Distribution: Linux Ubuntu 20.04LTS
**Apollo installed from: 软件包安装
**Apollo version: 8.0

Steps to reproduce the issue:
I try to run the command:
bazel build //modules/drivers/gnss/test:parser_cli
But an error occurres after running：
ERROR:/apollo_workspace/modules/drivers/gnss/test/BUILD:15:10:Linking modules/drivers/gnss/test/parser_cli fatled:(Exit 1):gcc failed:error executing command /usr/bin/gcc @bazel-out/k8-fastbuild/bin/modules/drivers/gnss/test/parser_cli-2.params

What other files need to be modified for this file to compile successfully?

use buildtool build rather than bazel build:
buildtool build -p modules/drivers

use buildtool to build, rather than native bazel


Does not work with symbols used as a shared library

@dennishansen This plugin works with any symbol or instance in your document, regardless of the location of your symbol master (local or in a library). The plugin requires you have something to select though, and there is a use case where you use a library symbol as an override, which doesn't give you anything local in your document to select. In that situation, I usually add another version of that instance to my document somewhere, select it and run the plugin, then you will be able to locate those stubborn instances only used as overrides, take action, and then delete the new one you added to run the plugin. Hope this helps!

That's a good idea, thanks!


Warn users if legacy google-app-engine plugin is installed

And prompt to remove the old JB plugin.
Depends on #767. Needed only until old plugin is debundled from IJ.

this may be relevant to this too (same kinda of warning thing):
https://github.com/GoogleCloudPlatform/gcloud-intellij/pull/651


[REQ] add guardrails for parsing OS version

What kind of request is this?
Improvement of existing experience
What is your request or suggestion?
For example, for ubuntu if a scanner return X.Y.Z, copa tries to pull in the ubuntu:X.Y.Z image which doesn't exist
related to #437
Are you willing to submit PRs to contribute to this feature request?

[X] Yes, I am willing to implement it.

Hii @sozercan I am interested in this issue. Can you assign it to me.

On first looking, I got this idea
We can change like this
   case strings.Contains(osType, "ubuntu"):
        // For Ubuntu, extract the major and minor version (e.g., 22.04)
        versionParts := strings.Split(osVersion, ".")
        if len(versionParts) >= 2 {
            osVersion = strings.Join(versionParts[:2], ".")
        } else {
            return "", "", fmt.Errorf("invalid Ubuntu version format: %s", osVersion)
        }
        return "ubuntu", osVersion, nil

here. By defining 	osVersion := osData["VERSION_ID"] in getOSType function. This can easily remove this issue. WDYT  @sozercan @ashnamehrotra

Most of work on this issue with this approach is complete, If you all agree then i would like to raise a PR.

Hi @h4l0gen, that sounds good. We recently merged a getOSVersion() function in #570, which can be used here.

/assign

Hii @ashnamehrotra I have used getOSVersion() function as you suggested and raised a PR, PTAL whenever you find time :)

In semantic versioning terms, if major.minor.patch is being passed in, are we only wanting to return major.minor and discarding the patch versioning?
For example, if we pass in Alpine 3.17.5, we discard the patch versioning and return 3.17?

@MiahaCybersec yes, for the tooling images as described in #437 this is what we want to follow.

After rather extensive testing I think we already have this implemented.
With Debian, if I pass in 12.6, 11.1, etc. we always use major-slim.
With Ubuntu, all images I pass in return major.minor as the tooling image as expected.
With RPM images, including when we pass Mariner in to be patched, we always use 2.0 and patch is always discarded.

closing with #736


Added attributes map option

When specified, only the listed attributes are included.
Hopefully it's TS-esque

@ShadowManu I believe this is now ready for review

Please rebase this branch on master to merge it!


add another "location" level

levels are: Project has multiple Areas has multiple Locations has multiple Sections.

also known as grouping. not relevant at the moment.

also known as grouping. not relevant at the moment.


Configure Azure Pipelines

This pipeline configuration builds the application and produces a build for the Release configuration.

 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.:x: vijayhotti sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Update (New) Admin UI to 2024-05-08

Contains various bug fixes as well as code enhancements. For a full list of changes see
https://github.com/opencast/opencast-admin-interface/releases/tag/2024-05-08
Your pull request should…

[x] have a concise title
[ ] close an accompanying issue if one exists
[ ] be against the correct branch
[ ] include migration scripts and documentation, if appropriate
[ ] pass automated tests
[ ] have a clean commit history
[ ] have proper commit messages (title and body) for all commits

Rebased onto the current develop


Set default parameters

Make functions similar to this empymod.utils.get_minimum() and empymod.utils.set_minimum(), but for setting the defaults for emg3d.solve().
E.g.,
emg3d.utils.set_defaults(
    sslsolver=True,
    linerelaxation=False,
    verb=-1,
)

Having this global option saves a lot of typing later on in large examples.

Closing this in favour of simulations. Many runs should be done with simulations, and the solver-setting is then an parameter/attribute of the simulation.


mbed compile can access the network, through pip

Mbed CLI calls out to Mbed OS build tools delivered with Mbed OS, and is responsible for making sure that they have all of their requirements. The following subcommands could update Mbed OS, which would change the build tools requirements. Therefore the following subcommands should install requirements:

mbed new
mbed import
mbed add
mbed deploy
mbed update

The following subcommands are not supposed to interact with the network to allow offline operation, and currently might access the network through pip:

mbed compile
mbed test
mbed export

These commands should not auto-install, so that offline work is not interrupted.
@screamerbg What do you think?

@theotherjimmy Agreed. Thanks for highlighting this. I'll issue a PR later today to address this.

ARM Internal Ref: MBOTRIAGE-1653

#833 Resolved this.


roll engine to 97e7c6e66122447c681fafcd92bcd42f41d02c61

97e7c6e66 (upstream/master) Roll src/third_party/skia fd98c2c87164..afea9c32acd9 (1 commits) (flutter/engine#5916)
1a35c5166 Roll src/third_party/skia f4bf7fc35d83..fd98c2c87164 (1 commits) (flutter/engine#5915)
8cb0f6d26 Roll src/third_party/skia 6cfa29738295..f4bf7fc35d83 (2 commits) (flutter/engine#5914)
dcd2956f8 Initial support for more finely-grained a11y features on Window (flutter/engine#5901)
83da72745 Roll src/third_party/skia 1935aa3d27cd..6cfa29738295 (7 commits) (flutter/engine#5912)

There are two other competing engine rolls: https://github.com/flutter/flutter/pull/20116 and https://github.com/flutter/flutter/pull/20104. Do you care about this particular sha to get benchmarks etc?

Nope, will close


Adding the accuracy based on testing dataset in addition to validation datasets

Signed-off-by: Omar Shrit<EMAIL_ADDRESS>

@zoq this one can be merged as well.


Reveal hidden Dealer card

Dealer should reveal the hidden card after player surrender / bust. I believe this is a common practice and helps to keep more accurate count.

Done.


Point users to Arraylist in vector section (tiny change).

Previous language may imply Zig doesn't have C++ style vectors, when that functionality indeed exists in ArrayLists.
This confused me when I was looking for vectors in Zig, hope it doesn't bite anyone else! Feedback welcome

Thanks!


Wrong or unnecessary translation of "Closure"

VSCode Version: Code 1.11.1 (d9484d12b38879b7f4cdd1150efeb2fd2c1fbf39, 2017-04-06T13:58:12.954Z)
OS Version: Windows_NT ia32 6.1.7601
Extensions:

Extension
Author
Version

jshint
dbaeumer
0.10.15

githistory
donjayamanne
0.2.0

Theme-Dreamweaver
gerane
0.0.2

Theme-PastelsonDark
gerane
0.0.2

Theme-TwoStones
gerane
0.0.2

hopscotch
idleberg
0.1.4

crayon-vscode
jansenfuller
0.1.3

jscs-formatter
moose87
0.1.1

jscs
ms-vscode
0.1.9

Steps to Reproduce:

Debug a node.js application with the integrated node.js debugger
Hit a breakpoint
=> In the left panel there are several sections displayed i.e. for breakpoints and variables including the section for local, global and closure scople. The german translation "Abschluss" of the word "Closure" doesn't fit here it is actully more confusing. than it helps. In fact i would recommend to not translate this word to german because it's commonly used as it is "Closure".

I tried hard to find the translation inside the project to file a pull request but i couldn't find it.

Thank you for the feedback.
We just started having a process for users contributing to our translations. All the details about this can be found here https://github.com/Microsoft/Localization/wiki/Visual-Studio-Code-Community-Localization-Project
Can you please follow those steps and contribute to our translations?

Just read our process for the translation issues -> reopening, adding tags and assigning appropriate people.
@ckosmowski you can still contribute using transifex but in case you do not feel like it we have an alternative solution

ping @danyeh @khoiph1

@isidorn I don't want these issue appear in the inbox again

Okay i'll have a look at the translation process.

We are discussing and fine tuning the process with the translation team. Here are some things to keep in mind:
Transifex has very limited issue tracking capabilities (to my knowledge). You can basically only add comments to translated strings. It is a translation platform not a good issue tracking system.
Since VS Code's issue tracking system is GitHub I think the best we can do for now is to have these issues in GitHub (we did for the internal translation as well). I think it will raise the bar as well to file translation issue since it requires the creation of a user id to comment on strings. And you are treated as a translator afterwards :-(
The idea how to handle this is as follows:

we have a translation representative from the translation team which we will assign these issue to (the inbox tracker does so)
the issues will be label as i18n and marked as help wanted so that they don't show up as bugs.
we need to educate translators in Transifex to look at these isssues.

@egamma FYI.

I now created an transifex account and joined the correct team.  Still i can't really find the wrong translation.  There is a resource called "vscode-node-debug" which sounds like what i am looking for but there are no german translations at all.
Is the german translation of missing keys generated automatically so that i can't find the wrong value in the resources ?

@ckosmowski I apologize, we've created vscode-node-debug resource only today for the Node debug adapter (this is an extension, not the tool). Therefore, you don't see translations for it yet, because they have not been recycled from the old translation memory we have. However, you can still create a translation suggestion for this string, even without an existing translation being present.
We will update vscode-node-debug resource with latest translations as soon as possible. VS Code as a standalone tool has all existing translations already there, if you wish to contribute.

Thanks @dbaeumer

I reverted German translation to 'closure'.
https://www.transifex.com/microsoft-oss/vscode-extensions/translate/#de/$/111444286?q=closure
Thanks @ckosmowski for the feedback.


depends on non-NOTIFYable propertie, TypeError: Cannot read property 'data' of undefined

hello!

I would like a color graph.
i have a problem like the picture. Please tell me how to fix it

If you don't have a radiometric lepton, that graph won't show up.

Is it related to GPP, DSP, PT1 FW version?
How do I use my radiometric lepton?

500-0726-01 is a Lepton 3.0. Lepton 2.5 and 3.5 are the radiometric parts.


PODNet - NME accuracy for ImageNet

Hi Arthur @arthurdouillard ,
I see that your paper PODNet reports PODNet-CNN and PODNet-NME accuracies from CIFAR-100.
But for ImageNet, it reports only PODNet-CNN accuracy. Could you please point out why? If there are available, can you please provide the numbers?
Thank you!

Hey,
I don't have all results, but a PODNet-NME with POD-channels (see table 3b) has for avg on ImageNet100 50/25/10/5 steps: 53.89 / 63.33 / 70.30 / 73.13.
So the results with POD-spatial should be higher but I don't think I ever run them. Nor on ImageNet1000.
Overall, you should expect better perfs than UCIR (both NME and CNN) but it is worse than PODNet-CNN.
I've found on large scale datasets learning the classifier end-to-end was better than relying on doing a KNN on the embedding space.
Does that answer your question? So I couldn't give more results.

Yes, that answers. Thank you very much @arthurdouillard


links

https://github.com/fl64/bookmarks/blob/master/GO.md
https://appliedgo.net/spotlight/6-repos-that-help-you-get-better-in-go/
https://yourbasic.org/golang/

https://github.com/avelino/awesome-go?tab=readme-ov-file#contents

https://www.golangcode.com/


feat(RHTAPREL-806): update operator toolkit refrence

This commit updates the operator-toolkit refrence
as a result of migration to konflux-ci from redhat-appstudio
more details parent EPIC:
https://issues.redhat.com/browse/RHTAPREL-800

Do Not Merge/Hold preparation PR, Can be merged once repo is migrated to konflux-ci!