Adds detection of BusyBox v1.30.0+ with updated timeout arguments

This fixes #71 - adds support for BusyBox 1.30.0+ and maintains support for versions below.
The problem
BusyBox below v1.30.0 had a non-standard implementation of timeout.  BusyBox v1.30.0+ updated timeout to match coreutils.  This library was previously checking if timeout belongs to busybox and assuming it uses the non-standard flags.

See mirror/busybox@c9720a7 as noted by @socketpair here: https://github.com/vishnubob/wait-for-it/pull/6

To fix the problem without breaking support for older versions of BusyBox, this adds a version check of BusyBox to determine which version of the flags to use.
Note: the changes use bash's regular expression's BASH_REMATCH to get the job done.  For Alpine users, this means you'll have to install bash and execute wait-for-it using bash.
The last time I contributed to wait-for-it, it was not compatible with ash, but things may have changed.  If requiring bash is a problem, just say so.
I've only tested on Alpine 3.1 and 3.10 w/ bash, but welcome additional testing and feedback.
Fixes #71

encounter the same issue

This doesn't seem to work in Alpine 3.6, you noted this should preserve backwards comparability
Output:
/ # time ./wait-for-it.sh localhost:2020
wait-for-it.sh: waiting 15 seconds for localhost:2020
./wait-for-it.sh: line 62:   130 Terminated              timeout $WAITFORIT_BUSYTIMEFLAG $WAITFORIT_TIMEOUT $0 --child --host=$WAITFORIT_HOST --port=$WAITFORIT_PORT --timeout=$WAITFORIT_TIMEOUT
wait-for-it.sh: timeout occurred after waiting 15 seconds for localhost:2020
Command exited with non-zero status 143
real    0m 15.01s
user    0m 0.04s
sys     0m 0.00s

This works and is backwards compatible, but it breaks the current tests that got merged into upstream after this was made


[WIP] spec: run tests on Electron as well

This is a WIP while I figure it out, opening the PR to trigger builds

Sorry forgot about this, closing out as #848 was merged


[vcpkg] Add option cmake-args

to pass additional args to the cmake script execution within vcpkg
things users/maintainers might want to pass:
--trace/--trace-expand
CMAKE_MESSAGE_LOG_LEVEL
CMAKE_MESSAGE_INDENT
CMAKE_MESSAGE_CONTEXT_SHOW/ --log-context (If vcpkg would use CMAKE_MESSAGE_CONTEXT)
PORT_DEBUG=ON

/azp run

This should probably imply/require --editable.

@ras0219 This is not passed to the vcpkg_configure_cmake.
This is added to [DEBUG] CreateProcessW("E:\vcpkg_cache\downloads\tools\cmake-3.17.2-windows\cmake-3.17.2-win32-x86\bin\cmake.exe" <alot of cmake_args> -P "E:\qt_zstd\scripts\ports.cmake")

Sure, but the point is that these options can change the behavior of the package build -- that means they shouldn't be used as a basis for a cached binary tree.

Then there needs to be a whitelist for cmake-args which don't imply --editable. (Mainly those vars mentioned in the very first comment.)

We had a discussion about this at today's meeting, and our opinion is that:

The option needs to be renamed --x-cmake-args
This needs to imply --editable for now, and we can special case stuff afterwards.

Once those are in, we're up to merge.

@strega-nil Done

Thanks @Neumann-A


Rename-many; include sometimes allows items through which do not match

This must be fix before next release.

This is an example that reproduces this problem:
$dir = 'C:\Users\Plastikfan\dev\github\PoSh\Loopz\Elizium.Loopz\docs'
gci $dir | remy -Include '^(Get|Show)' -Pattern '(?<verb>[\w]+)' -Paste '[${verb}]' -WhatIf

This is supposed to only include docs which begin with 'Get' or 'Show', but all the other documents are processed too.

Actually, there is already a test case that exhibits this issue:

The Include parameter is specified with 'data', but other items are still let through this filter, namely: 'loopz.application.t1.log' and 'loopz.application.t2.log'.

This was caused by the following code:
    [regex]$anchoredRegEx = if ([RegexEntity]$ae = $bootStrap.Get('Anchored')) {
      $ae.RegEx;
    }
    else {
      $null;
    }

    [regex]$includedRegEx = if ([RegExEntity]$ie = $bootStrap.Get('Include')) {
      $ie.RexEx;
    }
    else {
      $null;
    }

    [regex]$exceptRegEx = if ([RegExEntity]$ee = $bootStrap.Get('Except')) {
      $ee.RexEx;
    }
    else {
      $null;
    }

and is mysteriously fixed by not using direct class member access, but by using the get_ function that is automatically generated for class members. I don't what it is about this scenario that needs this type of member access instead of direct access to the member, as I have done this before many times without issue. In fact, this is the first circumstance that I have found where use of the get_ member function is required in order to access a member:
    [RegExEntity]$ae = $bootStrap.Get('Anchored');
    [regex]$anchoredRegEx = ${ae}?.get_RegEx();

    [RegExEntity]$ie = $bootStrap.Get('Include');
    [regex]$includedRegEx = ${ie}?.get_RegEx();

    [RegExEntity]$ee = $bootStrap.Get('Except');
    [regex]$exceptRegEx = ${ee}?.get_RegEx();


Fill pa, pasuperct and pacommwct

Part of #929
To fill the gaps, we will implement a backscraper
pa
Between May 28, 2021 and November 16, 2021 we have 0 documents. There are 1261 documents in that time period, some of which are not opinions
pasuperct
Between May 28, 2021 and November 17, 2021 we have 0 documents. We are missing around 1504 "Non Precedential" documents and 164 Precedential opinions (source)
pacommwct
Between May 28, 2021 and August 23, 2021 we have 0 documents. We are missing 49 precedential opinionsf

I have updated this source to use the API, instead of the RSS Feed, both for the present scraper and for the back scraper
This source returns OpinionClusters from it's API.. It would be an easy candidate to start returning clusters. For example (and more on the example files):
    {
        "Author": null,
        "BoardDocketNumber": null,
        "Caption": "In the Interest of: N.E.M., Appeal of: N.E.M. - No. 9 EAP 2023",
        "CourtDocketNumber": null,
        "CourtType": 3,
        "DispositionDate": "2024-03-21T00:00:00",
        "Keywords": null,
        "UserIdentifier": "E.D. Prothonotary",
        "UploadDate": "0001-01-01T00:00:00",
        "PostedToday": false,
        "Postings": [
            {
                "Id": 88531,
                "AuthorId": "Donohue, Christine",
                "OpinionId": 80242,
                "FileName": "J-41B-2023mo -<PHONE_NUMBER>59675150.pdf",
                "ProcessedDate": "2024-03-21T00:00:00",
                "PostingTypeId": "mo",
                "PublicationTypeId": null,
                "RenderedDate": "2024-03-21T00:00:00",
                "SortOrder": 0,
                "FileVersion": 1,
                "Author": {
                    "Id": 0,
                    "AuthorName": "Justice Christine Donohue",
                    "AuthorCode": "Donohue, Christine",
                    "Selectable": true,
                    "SortOrder": 1430
                },
                "PostType": {
                    "Id": 0,
                    "PostingTypeCode": "mo",
                    "PostingTypeId": "Majority Opinion",
                    "SortOrder": null
                },
                "PublicationType": null
            },
            {
                "Id": 88533,
                "AuthorId": "Dougherty, Kevin M.",
                "OpinionId": 80242,
                "FileName": "J-41B-2023co -<PHONE_NUMBER>59675223.pdf",
                "ProcessedDate": "2024-03-21T00:00:00",
                "PostingTypeId": "co",
                "PublicationTypeId": null,
                "RenderedDate": "2024-03-21T00:00:00",
                "SortOrder": 0,
                "FileVersion": 1,
                "Author": {
                    "Id": 0,
                    "AuthorName": "Justice Kevin Dougherty",
                    "AuthorCode": "Dougherty, Kevin M.",
                    "Selectable": true,
                    "SortOrder": 1440
                },
                "PostType": {
                    "Id": 0,
                    "PostingTypeCode": "co",
                    "PostingTypeId": "Concurring Opinion",
                    "SortOrder": null
                },
                "PublicationType": null
            }
        ],
        "Id": 80242
    },

Commands to fill the gap
docker exec -it cl-django python /opt/courtlistener/manage.py cl_back_scrape_opinions --courts juriscraper.opinions.united_states.state.pa --backscrape-start=05/27/2021 --backscrape-end=11/17/2021

docker exec -it cl-django python /opt/courtlistener/manage.py cl_back_scrape_opinions --courts juriscraper.opinions.united_states.state.pasuperct --backscrape-start=05/27/2021 --backscrape-end=11/18/2021

docker exec -it cl-django python /opt/courtlistener/manage.py cl_back_scrape_opinions --courts juriscraper.opinions.united_states.state.pacommwct --backscrape-start=05/27/2021 --backscrape-end=08/24/2021

After running the backscrapers, for the time periods mentioned above

pa from 0 to 197
pasuperct from 0 to 1530
pacommwct we have 41 new precedential opinions

The backscraper is buggy, it should consume the paginated results; but only consumes the first page. Luckily, this is mostly fixed by overriding the days_interval using the optional argument; however, some days still have more than 20 results and need pagination
The following days have more than 20 opinions, and thus need the backscraper to paginate. For the high volume courts, I think we surely have gaps I haven't detected, due to the lack of pagination
pa
INFO Backscraping for range 2021-09-22 2021-09-22
INFO Backscraping for range 2021-11-17 2021-11-17
pasuperct
INFO Backscraping for range 2021-11-16 2021-11-16
INFO Backscraping for range 2021-11-05 2021-11-05
INFO Backscraping for range 2021-10-12 2021-10-12
INFO Backscraping for range 2021-10-08 2021-10-08
INFO Backscraping for range 2021-09-16 2021-09-16
INFO Backscraping for range 2021-09-10 2021-09-10
INFO Backscraping for range 2021-08-19 2021-08-19
INFO Backscraping for range 2021-08-17 2021-08-17
INFO Backscraping for range 2021-08-16 2021-08-16
INFO Backscraping for range 2021-08-06 2021-08-06
INFO Backscraping for range 2021-07-30 2021-07-30
INFO Backscraping for range 2021-07-27 2021-07-27
INFO Backscraping for range 2021-07-20 2021-07-20
INFO Backscraping for range 2021-07-13 2021-07-13
INFO Backscraping for range 2021-07-09 2021-07-09
INFO Backscraping for range 2021-07-08 2021-07-08
INFO Backscraping for range 2021-07-01 2021-07-01
INFO Backscraping for range 2021-06-21 2021-06-21
INFO Backscraping for range 2021-06-15 2021-06-15
INFO Backscraping for range 2021-06-10 2021-06-10
INFO Backscraping for range 2021-06-09 2021-06-09
INFO Backscraping for range 2021-06-08 2021-06-08
INFO Backscraping for range 2021-06-04 2021-06-04
INFO Backscraping for range 2021-05-28 2021-05-28


Add a README.md for Problem Of The Day | 22 January 2023

Comment below if you're interested in working on this issue and would like to be assigned to it.

Shall I work on this issue?


Please help to improve cpython performance in xormask

Sorry for my poor english

https://github.com/aaugustin/websockets/issues/466#issuecomment-423814482
would you please patch this for cpython users ?
    from websockets.speedups import apply_mask
    class XorMaskerShifted1(object):

        __slots__ = ('_ptr', '_mskarray','_mask')

        def __init__(self, mask):
            assert len(mask) == 4
            self._mask = mask

        def process(self, data):
            return apply_mask(data,self._mask)

no, won't merge;

I guess it's native code and only in "websockets"
I care more about PyPy
we use wsaccel on cpy, but the actual solution would be to finish work on https://github.com/crossbario/autobahn-python/tree/master/autobahn/nvx


Occasional console related exception

There appears to be some sort of race condition that can occur that results in the following exception and stack trace:
System.InvalidOperationException: Stack empty.
   at System.ThrowHelper.ThrowInvalidOperationException(ExceptionResource resource)
   at System.Collections.Generic.Stack`1.Pop()
   at Colors.Net.ColoredConsoleWriter.Write(Object ovalue)
   at Colors.Net.ColoredConsoleWriter.WriteLine(Object value)
   at Colors.Net.ColoredConsole.WriteLine(Object value)
   at WebJobs.Script.Cli.Diagnostics.ConsoleTraceWriter.Trace(TraceEvent traceEvent)
   at Microsoft.Azure.WebJobs.Script.CompositeTraceWriter.Trace(TraceEvent traceEvent)
   at Microsoft.Azure.WebJobs.Script.CompositeTraceWriter.Trace(TraceEvent traceEvent)
   at Microsoft.Azure.WebJobs.Host.CompositeTraceWriter.InvokeTraceWriters(TraceEvent traceEvent)
   at Microsoft.Azure.WebJobs.Host.CompositeTraceWriter.Trace(TraceEvent traceEvent)
   at Microsoft.Azure.WebJobs.Host.Loggers.TraceWriterFunctionInstanceLogger.LogFunctionStartedAsync(FunctionStartedMessage message, CancellationToken cancellationToken)
   at Microsoft.Azure.WebJobs.Host.Executors.FunctionExecutor.LogFunctionStartedAsync(FunctionStartedMessage message, IFunctionOutputDefinition functionOutput, IReadOnlyDictionary`2 parameters, CancellationToken cancellationToken)
   at Microsoft.Azure.WebJobs.Host.Executors.FunctionExecutor.<ExecuteWithLoggingAsync>d__1a.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.Azure.WebJobs.Host.Executors.FunctionExecutor.<TryExecuteAsync>d__2.MoveNext()

I can't reliably reproduce it, but I only ever see it when there are multiple concurrent function executions.
@ahmelsayed Perhaps the issue is that the ColoredConsoleWriter in Colors.Net is not thread safe?

See above. If this is indeed a colors issue you should be able to write a unit test that hammers the API on multiple threads and repros pretty easy.


[EN] Add default response for HassGetState

Let's say somebody has the following 2 entities:

Alice time to home (sensor.alice_time_to_home, provided by Waze, UOM minutes)
Bob time to home (sensor.alice_time_to_home, provided by Waze, UOM minutes)
and defines a custom sentence under conversation:, along the lines of:

config/custom_sentences/en/time_to_home.yaml
language: ro
intents:
  HassGetState:
    data:
      - sentences:
          - "how much longer [un]till {person_to_sensor:name} gets home"
          - "when will {person_to_sensor:name} be home"

lists:
  person_to_sensor:
    values:
      - in: "Alice"
        out: "Alice time to home"
      - in: "Bob"
        out: "Bob time to home"

This will fail because there is no default response. If we were to add a response: one, the response would be Alice is 10 minutes instead of Alice time to home is 10 minutes.
As such, this PR introduces a very simple 10 minutes response for the above scenario, where people to add query sentences to default intents can use for HassGetState.

Looks good, thanks!


[Bug] Very long command output lines are split in two

error.log content:
ERROR: [Synth 8-690] width mismatch in assignment; target has 64 bits, source has 32 bits [C:/Projects/HDL/FPGA/Models/Model_2.0/FPGA/Latest/ZU7_COMPONENT_PLUS/COMPONENT_PLUS_V004_000/COMPONENT/COMPONENT.srcs/sources_1/new/ACQ_COMPONENT.vhd:4085]
ERROR: [Synth 8-285] failed synthesizing module 'ACQ_COMPONENT' [C:/Projects/HDL/FPGA/Models/Model_2.0/FPGA/Latest/ZU7_COMPONENT_PLUS/COMPONENT_PLUS_V004_000/COMPONENT/COMPONENT.srcs/sources_1/new/ACQ_COMPONENT.vhd:226]
ERROR: [Synth 8-285] failed synthesizing module 'Top_COMPONENT' [C:/Projects/HDL/FPGA/Models/Model_2.0/FPGA/Latest/ZU7_COMPONENT_PLUS/COMPONENT_PLUS_V004_000/COMPONENT/COMPONENT.srcs/sources_1/new/Top_COMPONENT.vhd:167]

When executing this command:
type error.log

then the log lines are split in two in output:
ERROR: [Synth 8-690] width mismatch in assignment; target has 64 bits, source has 32 bits [C:/Projects/HDL/FPGA/Models/Model_2.0/FPGA/Latest/ZU7_COMPONENT_PLUS/COMPONENT_PLUS_V004_000/COMPONE
NT/COMPONENT.srcs/sources_1/new/ACQ_COMPONENT.vhd:4085]
ERROR: [Synth 8-285] failed synthesizing module 'ACQ_COMPONENT' [C:/Projects/HDL/FPGA/Models/Model_2.0/FPGA/Latest/ZU7_COMPONENT_PLUS/COMPONENT_PLUS_V004_000/COMPONENT/COMPONENT.srcs/sources_
1/new/ACQ_COMPONENT.vhd:226]
ERROR: [Synth 8-285] failed synthesizing module 'Top_COMPONENT' [C:/Projects/HDL/FPGA/Models/Model_2.0/FPGA/Latest/ZU7_COMPONENT_PLUS/COMPONENT_PLUS_V004_000/COMPONENT/COMPONENT.srcs/sources_
1/new/Top_COMPONENT.vhd:167]

[Process exited 0]

So for example error parsing with errorformat is broken. We should expect just tree output lines, not six, due to the split.
nvim version:
NVIM v0.8.0-1210-gd367ed9b2
Build type: RelWithDebInfo
LuaJIT 2.1.0-beta3
Compilato da runneradmin@fv-az177-603

Os is Windows 10 Pro.

This is an artifact of how the terminal works in neovim. It relies of libvterm, and currently there is no way to distinguish a "hard wrap" (intentional line break) from a "soft wrap" (line break due to exceeding the terminal's width). There's been some talk about this on both the neovim and vim side, but no fixes yet.
Another option for you: I've added the ability to use the jobstart strategy without using the neovim terminal. This should fix all line wrapping issues, but will no longer run your programs in a pty so you'll lose fancy features that some commands provide (progress bars, colorized output, etc). You can enable this for all your neovim tasks by setting the default strategy
require("overseer").setup({
  strategy = { "jobstart", use_terminal = false },
})

Or you can set it on a case-by-case basis by passing it to the task directly, wherever you're using a task definition. For example, here's how you would do it from a custom template:
overseer.register_template({
  name = "My task",
  builder = function()
    return {
      cmd = { "ls" },
      strategy = { "jobstart", use_terminal = false },
    }
  end,
})

You can also do this using overseer.add_template_hook

It now seems to work, but now I have to run this command:
cmd = vim.fn.expandcmd("Invoke-Expression -Command \"C:\\Xilinx\\Vivado\\2020.2\\bin\\vivado.bat -mode batch -source synth_design.tcl -nojournal -nolog -tempDir vivado_deletable -verbose PRODUCT\\PRODUCT.xpr\"")

instead of;
cmd = vim.fn.expandcmd("vivado -mode batch -source synth_design.tcl -nojournal -nolog -tempDir \"vivado_deletable\" -verbose \"PRODUCT\\PRODUCT.xpr\"")

It seems that with your suggestion, PowerShell is used, instead of normal CMD.

I don't know why the behavior would be different; termopen() is supposed to have the same behavior as jobstart() with regards to the shell. You could try two things:

pass the command as a table instead of a string. e.g.

cmd = {"vivado", "-mode", "batch", "-source", "synth_design.tcl", "-nojournal", "-nolog", "-tempDir", "vivado_deletable", "-verbose", "PRODUCT\\PRODUCT.xpr"}

set shell=cmd.exe. This determines which shell is used to run string commands (see :help jobstart). cmd.exe should be the default value, so it's likely that you're setting it to powershell somewhere.

I don't know why the behavior would be different; termopen() is supposed to have the same behavior as jobstart() with regards to the shell. You could try two things:

pass the command as a table instead of a string. e.g.

cmd = {"vivado", "-mode", "batch", "-source", "synth_design.tcl", "-nojournal", "-nolog", "-tempDir", "vivado_deletable", "-verbose", "PRODUCT\\PRODUCT.xpr"}

It worked but I had to use vivado.bat instead of vivado.

set shell=cmd.exe. This determines which shell is used to run string commands (see :help jobstart). cmd.exe should be the default value, so it's likely that you're setting it to powershell somewhere.

In my case it is set to:
shell=pwsh.exe -NoLogo

I also noticed that every time a command from the template is run, the task name is very long:

See the string after the Running word. I suppose the task name is simply copied from the expanded cmd string.
I wonder if there is a config to set that task name so that it can be customizable, as it happens for example with cmd or cwd parameters in the register_template function.
So that it could become something like this:

Yeah there are two name fields you may care about. The template name is what will show up in the list when you call :OverseerRun
overseer.register_template({
  name = "Run my task",
  builder = function() end
})

You can also pass a name to the task, which will be what shows up in the task list
overseer.register_template({
  name = "My task",
  builder = function()
    return {
      name = string.format("[Vivado] - Synth - %s", vim.fn.expand('%')),
      cmd = {"vivado.bat", "-mode", "batch", "-source", "synth_design.tcl", "-nojournal", "-nolog", "-tempDir", "vivado_deletable", "-verbose", "PRODUCT\\PRODUCT.xpr"},
      strategy = { "jobstart", use_terminal = false },
    }
  end,
})


"Mining process is having issues" on init, init takes way too much.

There's this user running a rig with 4 7990 vidcards, making up for 8 GPUs @ 600/1000MHz.
Leaving aside the hashrate is easily 4x lower than expected, this bug is about:
after init,
icon turns yellow block and reports "Mining process is having issues".
Some elaborations
This is due to building the kernels in way too much time. 801 uses a watchdog-based monitoring approach. It was assumed no system would take so much time in initializing.
It is obvious this assumption proved to be false. Albeit this particular system is not in the target demographics, the reported bug can affect intended users.
As a side note
The mining process will eventually start (at reduced performance) but the block will stay yellow. "yellow" could truly be correct or not; it has to be investigated as well.
partial clinfo
  Max image 2D height:                           16384
  Max image 3D width:                            2048
  Max image 3D height:                           2048
  Max image 3D depth:                            2048
  Max samplers within kernel:                    16
  Max size of kernel argument:                   1024
  Alignment (bits) of base address:              2048
  Minimum alignment (bytes) for any datatype:    128
  Single precision floating point capability
    Denorms:                                     No
    Quiet NaNs:                                  Yes
    Round to nearest even:                       Yes
    Round to zero:                               Yes
    Round to +ve and infinity:                   Yes
    IEEE754-2008 fused multiply-add:             Yes
  Cache type:                                    Read/Write
  Cache line size:                               64
  Cache size:                                    16384
  Global memory size:<PHONE_NUMBER>
  Constant buffer size:                          65536
  Max number of constant args:                   8
  Local memory type:                             Scratchpad
  Local memory size:                             32768
  Kernel Preferred work group size multiple:     64
  Error correction support:                      0
  Unified memory for Host and Device:            0
  Profiling timer resolution:                    1
  Device endianess:                              Little
  Available:                                     Yes
  Compiler available:                            Yes
  Execution capabilities:
    Execute OpenCL kernels:                      Yes
    Execute native function:                     No
  Queue properties:
    Out-of-Order:                                No
    Profiling :                                  Yes
  Platform ID:                                   00007FFC73B1CFB0
  Name:                                          Tahiti
  Vendor:                                        Advanced Micro Devices, Inc.
  Device OpenCL C version:                       OpenCL C 1.2
  Driver version:                                1445.5 (VM)
  Profile:                                       FULL_PROFILE
  Version:                                       OpenCL 1.2 AMD-APP (1445.5)
  Extensions:                                    cl_khr_fp64 cl_amd_fp64 cl_khr_
global_int32_base_atomics cl_khr_global_int32_extended_atomics cl_khr_local_int3
2_base_atomics cl_khr_local_int32_extended_atomics cl_khr_int64_base_atomics cl_
khr_int64_extended_atomics cl_khr_3d_image_writes cl_khr_byte_addressable_store
cl_khr_gl_sharing cl_ext_atomic_counters_32 cl_amd_device_attribute_query cl_amd
_vec3 cl_amd_printf cl_amd_media_ops cl_amd_media_ops2 cl_amd_popcnt cl_khr_d3d1
0_sharing cl_khr_d3d11_sharing cl_khr_dx9_media_sharing cl_khr_image2d_from_buff
er cl_khr_spir cl_khr_gl_event

  Device Type:                                   CL_DEVICE_TYPE_GPU
  Vendor ID:                                     1002h
  Board name:                                    AMD Radeon HD 7900 Series
  Device Topology:                               PCI[ B#15, D#0, F#0 ]
  Max compute units:                             32
  Max work items dimensions:                     3
    Max work items[0]:                           256
    Max work items[1]:                           256
    Max work items[2]:                           256
  Max work group size:                           256
  Preferred vector width char:                   4
  Preferred vector width short:                  2
  Preferred vector width int:                    1
  Preferred vector width long:                   1
  Preferred vector width float:                  1
  Preferred vector width double:                 1
  Native vector width char:                      4
  Native vector width short:                     2
  Native vector width int:                       1
  Native vector width long:                      1
  Native vector width float:                     1
  Native vector width double:                    1
  Max clock frequency:                           1000Mhz
  Address bits:                                  32
  Max memory allocation:<PHONE_NUMBER>
  Image support:                                 Yes
  Max number of images read arguments:           128
  Max number of images write arguments:          8
  Max image 2D width:                            16384
  Max image 2D height:                           16384
  Max image 3D width:                            2048
  Max image 3D height:                           2048
  Max image 3D depth:                            2048
  Max samplers within kernel:                    16
  Max size of kernel argument:                   1024
  Alignment (bits) of base address:              2048
  Minimum alignment (bytes) for any datatype:    128
  Single precision floating point capability
    Denorms:                                     No
    Quiet NaNs:                                  Yes
    Round to nearest even:                       Yes
    Round to zero:                               Yes
    Round to +ve and infinity:                   Yes
    IEEE754-2008 fused multiply-add:             Yes
  Cache type:                                    Read/Write
  Cache line size:                               64
  Cache size:                                    16384
  Global memory size:<PHONE_NUMBER>
  Constant buffer size:                          65536
  Max number of constant args:                   8
  Local memory type:                             Scratchpad
  Local memory size:                             32768
  Kernel Preferred work group size multiple:     64
  Error correction support:                      0
  Unified memory for Host and Device:            0
  Profiling timer resolution:                    1
  Device endianess:                              Little
  Available:                                     Yes
  Compiler available:                            Yes
  Execution capabilities:
    Execute OpenCL kernels:                      Yes
    Execute native function:                     No
  Queue properties:
    Out-of-Order:                                No
    Profiling :                                  Yes
  Platform ID:                                   00007FFC73B1CFB0
  Name:                                          Tahiti
  Vendor:                                        Advanced Micro Devices, Inc.
  Device OpenCL C version:                       OpenCL C 1.2
  Driver version:                                1445.5 (VM)
  Profile:                                       FULL_PROFILE
  Version:                                       OpenCL 1.2 AMD-APP (1445.5)
  Extensions:                                    cl_khr_fp64 cl_amd_fp64 cl_khr_
global_int32_base_atomics cl_khr_global_int32_extended_atomics cl_khr_local_int3
2_base_atomics cl_khr_local_int32_extended_atomics cl_khr_int64_base_atomics cl_
khr_int64_extended_atomics cl_khr_3d_image_writes cl_khr_byte_addressable_store
cl_khr_gl_sharing cl_ext_atomic_counters_32 cl_amd_device_attribute_query cl_amd
_vec3 cl_amd_printf cl_amd_media_ops cl_amd_media_ops2 cl_amd_popcnt cl_khr_d3d1
0_sharing cl_khr_d3d11_sharing cl_khr_dx9_media_sharing cl_khr_image2d_from_buff
er cl_khr_spir cl_khr_gl_event

  Device Type:                                   CL_DEVICE_TYPE_GPU
  Vendor ID:                                     1002h
  Board name:                                    AMD Radeon HD 7900 Series
  Device Topology:                               PCI[ B#15, D#0, F#0 ]
  Max compute units:                             32
  Max work items dimensions:                     3
    Max work items[0]:                           256
    Max work items[1]:                           256
    Max work items[2]:                           256
  Max work group size:                           256
  Preferred vector width char:                   4
  Preferred vector width short:                  2
  Preferred vector width int:                    1
  Preferred vector width long:                   1
  Preferred vector width float:                  1
  Preferred vector width double:                 1
  Native vector width char:                      4
  Native vector width short:                     2
  Native vector width int:                       1
  Native vector width long:                      1
  Native vector width float:                     1
  Native vector width double:                    1
  Max clock frequency:                           1000Mhz
  Address bits:                                  32
  Max memory allocation:<PHONE_NUMBER>
  Image support:                                 Yes
  Max number of images read arguments:           128
  Max number of images write arguments:          8
  Max image 2D width:                            16384
  Max image 2D height:                           16384
  Max image 3D width:                            2048
  Max image 3D height:                           2048
  Max image 3D depth:                            2048
  Max samplers within kernel:                    16
  Max size of kernel argument:                   1024
  Alignment (bits) of base address:              2048
  Minimum alignment (bytes) for any datatype:    128
  Single precision floating point capability
    Denorms:                                     No
    Quiet NaNs:                                  Yes
    Round to nearest even:                       Yes
    Round to zero:                               Yes
    Round to +ve and infinity:                   Yes
    IEEE754-2008 fused multiply-add:             Yes
  Cache type:                                    Read/Write
  Cache line size:                               64
  Cache size:                                    16384
  Global memory size:<PHONE_NUMBER>
  Constant buffer size:                          65536
  Max number of constant args:                   8
  Local memory type:                             Scratchpad
  Local memory size:                             32768
  Kernel Preferred work group size multiple:     64
  Error correction support:                      0
  Unified memory for Host and Device:            0
  Profiling timer resolution:                    1
  Device endianess:                              Little
  Available:                                     Yes
  Compiler available:                            Yes
  Execution capabilities:
    Execute OpenCL kernels:                      Yes
    Execute native function:                     No
  Queue properties:
    Out-of-Order:                                No
    Profiling :                                  Yes
  Platform ID:                                   00007FFC73B1CFB0
  Name:                                          Tahiti
  Vendor:                                        Advanced Micro Devices, Inc.
  Device OpenCL C version:                       OpenCL C 1.2
  Driver version:                                1445.5 (VM)
  Profile:                                       FULL_PROFILE
  Version:                                       OpenCL 1.2 AMD-APP (1445.5)
  Extensions:                                    cl_khr_fp64 cl_amd_fp64 cl_khr_
global_int32_base_atomics cl_khr_global_int32_extended_atomics cl_khr_local_int3
2_base_atomics cl_khr_local_int32_extended_atomics cl_khr_int64_base_atomics cl_
khr_int64_extended_atomics cl_khr_3d_image_writes cl_khr_byte_addressable_store
cl_khr_gl_sharing cl_ext_atomic_counters_32 cl_amd_device_attribute_query cl_amd
_vec3 cl_amd_printf cl_amd_media_ops cl_amd_media_ops2 cl_amd_popcnt cl_khr_d3d1
0_sharing cl_khr_d3d11_sharing cl_khr_dx9_media_sharing cl_khr_image2d_from_buff
er cl_khr_spir cl_khr_gl_event

  Device Type:                                   CL_DEVICE_TYPE_CPU
  Vendor ID:                                     1002h
  Board name:
  Max compute units:                             8
  Max work items dimensions:                     3
    Max work items[0]:                           1024
    Max work items[1]:                           1024
    Max work items[2]:                           1024
  Max work group size:                           1024
  Preferred vector width char:                   16
  Preferred vector width short:                  8
  Preferred vector width int:                    4
  Preferred vector width long:                   2
  Preferred vector width float:                  8
  Preferred vector width double:                 4
  Native vector width char:                      16
  Native vector width short:                     8
  Native vector width int:                       4
  Native vector width long:                      2
  Native vector width float:                     8
  Native vector width double:                    4
  Max clock frequency:                           3016Mhz
  Address bits:                                  64
  Max memory allocation:<PHONE_NUMBER>
  Image support:                                 Yes
  Max number of images read arguments:           128
  Max number of images write arguments:          8
  Max image 2D width:                            8192
  Max image 2D height:                           8192
  Max image 3D width:                            2048
  Max image 3D height:                           2048
  Max image 3D depth:                            2048
  Max samplers within kernel:                    16
  Max size of kernel argument:                   4096
  Alignment (bits) of base address:              1024
  Minimum alignment (bytes) for any datatype:    128
  Single precision floating point capability
    Denorms:                                     Yes
    Quiet NaNs:                                  Yes
    Round to nearest even:                       Yes
    Round to zero:                               Yes
    Round to +ve and infinity:                   Yes
    IEEE754-2008 fused multiply-add:             Yes
  Cache type:                                    Read/Write
  Cache line size:                               64
  Cache size:                                    32768
  Global memory size:<PHONE_NUMBER>0
  Constant buffer size:                          65536
  Max number of constant args:                   8
  Local memory type:                             Global
  Local memory size:                             32768
  Kernel Preferred work group size multiple:     1
  Error correction support:                      0
  Unified memory for Host and Device:            1
  Profiling timer resolution:                    339
  Device endianess:                              Little
  Available:                                     Yes
  Compiler available:                            Yes
  Execution capabilities:
    Execute OpenCL kernels:                      Yes
    Execute native function:                     Yes
  Queue properties:
    Out-of-Order:                                No
    Profiling :                                  Yes
  Platform ID:                                   00007FFC73B1CFB0
  Name:                                                 Intel(R) Core(TM) i7-377
0K CPU @ 3.50GHz
  Vendor:                                        GenuineIntel
  Device OpenCL C version:                       OpenCL C 1.2
  Driver version:                                1445.5 (sse2,avx)
  Profile:                                       FULL_PROFILE
  Version:                                       OpenCL 1.2 AMD-APP (1445.5)
  Extensions:                                    cl_khr_fp64 cl_amd_fp64 cl_khr_
global_int32_base_atomics cl_khr_global_int32_extended_atomics cl_khr_local_int3
2_base_atomics cl_khr_local_int32_extended_atomics cl_khr_int64_base_atomics cl_
khr_int64_extended_atomics cl_khr_3d_image_writes cl_khr_byte_addressable_store
cl_khr_gl_sharing cl_ext_device_fission cl_amd_device_attribute_query cl_amd_vec
3 cl_amd_printf cl_amd_media_ops cl_amd_media_ops2 cl_amd_popcnt cl_khr_d3d10_sh
aring cl_khr_spir cl_amd_svm cl_khr_gl_event

(One question is why the user clipped off the head of the output... perhaps he just didn't know about output redirection, that's curious).

Cannot be reproduced using device replication. It looks like the driver caches kernel builds on the same device. I cannot blame it.
I have a suspect but that'll need to be validated by third parties.


Appointment Time Formatting in Outside Trips

Column is currently formatted as mm/dd/yyyy rather than just the time.

Appt Time and DO time in Sent trips is also formatted incorrectly in HH:MM:SS time, should be just HH:MM

Note: Formatting is now copied from current sheet, rather than transferred from source sheet. This will preserve column formatting, but requires column formatting to be set correctly. May need some tweaking to work when transferring data to an empty sheet.


[Message Formatting] HeroCard buttons not working on Facebook Messenger channel?

Bot Info

App ID: 7792dce2-616d-466e-a2d9-bcadcb22dde3
SDK Platform: .NET
SDK Version: <IP_ADDRESS>
Active Channels: WebChat, Facebook Messanger, Skype
Deployment Environment: Azure Bot Service

Issue Description
I am showing a carousel with 2 herocards with buttons. In skype everything works correctly, but when I click on the button into the Facebook Messanger channel, the bot doesn't give me a response. I am sure it's an issue of the button because if I send a message typing the same text of the button everything works.

Hi @aleinnocenzi - Can you share your code that creates the hero card with buttons?

@aleinnocenzi can you share please your code? I will try to repro and provide you with a solution.

What ActionType are you using?

Hi am having the same issue, am using HeroCard with ActionType = AmBack, please tell me which cards can I use on facebook messenger channel.

Facing the same issue with heroCards (in carousel Layout) only for facebook channel.
Below is the code for creating a hero card.
new builder.HeroCard(session)             .title(striptags(truncate(data[product].title, 80)))             .subtitle("Rs." + data[product].final_price + "")             .text(entities.decodeHTML(striptags(truncate(data[product].details, 80))))             .images([                 builder.CardImage.create(session, data[product].img),             ])             .buttons([                 builder.CardAction.openUrl(session, data[product].scandidUrl, "Buy Now")             ])
building message using carousel layout (as shown below).
new builder.Message()             .attachmentLayout(builder.AttachmentLayout.carousel)             .attachments(cards);


[Alignment] valign

<columns small="12" large="12" valign="top">

What did you expect to happen?
For the valign attribute to be passed along to the compiled code
What happened instead?
the attribute is stripped

What version of Foundation for Emails are you on? This was introduced in 2.1. I see it working on the docs so I'll close this here.


chore: update avro, prroto plugin versions

📚  View the storybook of this branch here

@kamiiiel looks like the artifact failed to land on nexus or something


Document fix in default initializer

A minor change to the documentation for the default initializer behavior in case of None.
Initializers can be usually be "specified" as arguments in Link.__init__ and I think this change makes it a bit clearer what it means.

LGTM. Jenkins, test this please.

Jenkins CI test (for commit 679479c4df664fac49d94cccee1a74017271256e, target branch master) failed with status FAILURE.

Jenkins issue is unrelated. LGTM. Merging.


Fixed Marathon-LB OSS doc

What are your changes?

Reworked the 2 tutorials
Renamed the tutorial from Quick Start to Deploying an Externally Load Balanced App with Marathon-LB
Renamed the tutorial from Using Marathon-LB to Deploying an Internally and Externally Load Balanced App with Marathon-LB
Added a new install topic, that both tutorials link to as prereq

What type of changes does your doc introduce?

[x] Bug fix
[ ] New feature

What is the urgency level of this change?

[ ] Blocker
[x] High
[ ] Medium

I have completed these items:

[ ] I have tested all commands and code snippets.
[ ] I have built locally and tested for broken links and formatting.
[x] I have made this change for all affected versions (e.g. 1.7, 1.8, and 1.9).
[x] My doc follows the style guide: https://github.com/dcos/dcos-docs#contributing.

CC: @pyronicide @surdy

Looks good. The names of the MLB tutorials now do more closely reflect the updated content.  :+1:

@emanic can you give this a sanity check? In addition to the Marathon-LB changes I've rearranged some other TOC items as well. I have confirmed that this works technically, and it's been reviewed by eng and support. thanks!

Some conflicts


multiline comments

Hi!
It seems hacky to specify comments that span more than one line. Two naive attempts fail:
If I insert "\n" into the comment string, then the serialization would break at that spot, but not insert another "# "
If I would just add another & comment [...] combinator, then the last one is taken and the first is ignored.
It works in the end by not only breaking with newline, but also adding "# " manually.
To illustrate with the example from the docs:
configParser' :: IniSpec Config ()
configParser' = do
  section "NETWORK" $ do
    cfHost .=  field "host" string
      & comment ["The desired hostname\n# some other comment line:"]

Ah! I don't think I explained this well in the docs, but the reason the argument to comment is a list is specifically to support multi-line comments. So the intended way of adding a multi-line comment would be with
configParser' :: IniSpec Config ()
configParser' = do
  section "NETWORK" $ do
    cfHost .=  field "host" string
      & comment ["The desired hostname", "some other comment line:"]

I think in addition to documenting that better I should also implement support for multiple comment combinators, because it seems like something which might work, and I might as well build the library to support that.


OpenSsl:154 - Failed to load netty-tcnative;OpenSslEngine will be unavailable, java.lang.illegalArgumentException: Failed to load any of given libraries :[netty_tcnative_windows_x86_64, netty_tcnative_x86_64, netty-tcnative

Query/Question
Its a old gradle groovy project with some java runners one of it is to retrieve emails
i am trying to retreive mails using it getting errors as below
OpenSsl:154 - Failed to load netty-tcnative;OpenSslEngine will be unavailable, java.lang.illegalArgumentException: Failed to load any of given libraries :[netty_tcnative_windows_x86_64, netty_tcnative_x86_64, netty-tcnative
at io.netty.util.internal.NativeLibraryLoader.loadFirstAvailable
could you please suggest a azre version without netty or how to resolve this issue
Why is this not a Bug or a feature Request?
A clear explanation of why is this not a bug or a feature request?
Setup (please complete the following information if applicable):

OS: windows
IDE: eclipse
Library/Libraries: com.azure.identity 1.11.1

Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[ ] Query Added
[ ] Setup information Added

Thanks for filing this issue @Jaffar321.
This guide goes over how you can replace the default HttpClient implementation used by the SDKs:
https://learn.microsoft.com/azure/developer/java/sdk/http-client-pipeline#replace-the-default-http-client
At this time, the only other available GA HttpClient we provide is OkHttp. Version 1.11.17 would align with the version of azure-identity you've included in this issue.

Instead I removed azure identity and added msal4j for generating token and passed token in iauthenticationprovider and using same for graph client
I tried few steps commented but was not successful
Thank you for support

Thanks


Create subtract() function in calculator.py

subtract(x, y): Returns the difference of x - y

Looks Great!


Update app.d

update the import module name;

Thanks! Once we get the ImGui backend we'll have to switch the hello world app to that, but I forgot to adjust this when I removed our backend, and it looks like Luna forgot to adjust it when she readded the module but changed the name. We mostly use Inochi-Creator for testing right now when everything is pre 1.0 and in flux.


[BUG] Senha para descompactar jogo

Describe the bug
Após baixar o jogo quando vou tentar descompactar é pedido um senha.
Steps to Reproduce

Baixar Raft (Versão Super Raft Boat Together ou Raft)
Tentar descompactar arquivo zipado

Expected behavior
No response
Screenshots

Operating System
Windows 10
Hydra Version
v2.1.3 "Leviticius"
Additional Information
No response
Before opening this Issue

[X] I have searched the issues of this repository and believe that this is not a duplicate.
[X] I am aware that Hydra team does not offer any support or help regarding the downloaded games.

As stated when you opened your issue, we don't provide any support or help regarding the games you download

If you need further help, I'd advise you to join the discord server managed by the community (https://discord.gg/hydralaunchercommunity)


Rename development database to be more consistent

Suffix _development to the name of the development database to match the pattern used on other databases in this app and the forms-admin app.
What problem does the pull request solve?
The name of the development database is currently a little surprising.
Checklist

[x] I've used the pull request template
[ ] ~I've linked this PR to the relevant issue (if mission work)~
[ ] ~I've written unit tests for these changes (if code change)~
[ ] I've updated the documentation in (If any documentation requires updating)

[ ] README.md
[ ] Elsewhere (please link)

@danworth is this going to affect the AWS stuff?

@danworth is this going to affect the AWS stuff?

Thanks for checking Laurence. We override the DB config by setting the DATABASE_URL env var which takes precedence over the config files. Also we set RACK_ENV to production in all AWS and PaaS deployments so AFAIK it wouldn't be using the development config settings anyway.
https://github.com/alphagov/forms-deploy/blob/ffaf19399a01afff1cb99343a2533f91af7744c8/infra/modules/forms-api/main.tf#L23.


Add Contributors to the repository

Description
@allcontributors please add @greg-neustroev for code, documentation, and maintenance

@allcontributors please add @greg-neustroev for code, documentation, and maintenance

@all-contributors
please add @g-moralesespana for ideas and planning.
please add @abelsiqueira for code review.
please add @datejada for code, documentation, code review, and maintenance.
please add @clizbe for project management and ideas.

@all-contributors
please add @abelsiqueira for code review.
please add @datejada for code, documentation, code review, and maintenance.

@all-contributors
please add @datejada for code, documentation, code review, and maintenance.

@all-contributors please add @clizbe for project management and ideas.

@allcontributors please add @greg-neustroev for code, documentation, code review, and maintenance.


How can I fine-tune custom mesh data?

Could you provide information on data preprocessing or the necessary code?

Hi, we have uploaded the fine-tuning scripts and pre-processed ShapNet data to huggingface. Welcome to check it out!
Unfortunately, I could not find local copy of the pre-processing file for now. If you would like to pre-process the data, please remember to drop duplicate faces after discretizing the vertice coordinates.


after importing vue-video-player, npm run dev, it reports error

[Vue warn]: Failed to mount component: template or render function not defined.
I looked around on google and find out it's because the vue-video-player module I got is runtime-only version, not standalone version...
how could I get a standalone version?
and how can I use this runtime version if there's no standalone one?
Thanks!

I'm sorry, I do not understand what you mean by the standalone version, what is its use scene?

Thank you for replying!
I don't know why, I just run npm install --save vue-video-player and I'm importing it like this:

and then the video player is not showing and when I check the console of the browser, it reports
[Vue warn]: Failed to mount component: template or render function not defined.
found in
--->
 at /Users/HOZ/Projects/vue_epg_op_service/src/components/VideoPlay.vue
I'm using Vue2 + vuex + webpack3.4.1

Please read the README.md documentation carefully, if you are not serious enough, no one can help you!

so sorry about that! I'm new to frontend and Vue, so I don't know what's wrong with my code. I read the readme over and over, one thing I have to point out, import { videoPlayer } from 'vue-video-player' doesn't work, I have to remove '{' and '}' so that vue can recognize this component.
could you please kindly point out what's wrong with my code? Thanks a lot!

global use

import Vue from 'vue'
import VueVideoPlayer from 'vue-video-player'
Vue.use(VueVideoPlayer)

component use

import { videoPlayer } from 'vue-video-player'
export default {
  components: {
    videoPlayer
  }
}

Thank you and it's amazing that it works, although the layout is a mass but I think I can work it out!
I thought if I named the component as VueVideoPlayer and use the tag  is certainly ok, but it's not!!!  I don't know why.... , and what does { } do when "import { videoPlayer } from 'vue-video-player" ?  what's the difference between the import without {}?
Could you explain to me?
Thank you very much!


DO NOT MERGE: Show corrected cherry pick for v1.0.17 release

DO NOT MERGE!!!
This PR is just for review of my previous cherry-pick from #27 that left the swagger.yaml in a bad state where components/schemas/Instance is missing from the end of the file.  The changes in this PR should become the new master branch.
Steps to recreate this PR from the existing master branch
# Create a branch from the pre-v1.0.17 master branch
git checkout -b cherry-pick/v1.0.17 18e523f99a0a9ab7c0baeac02f89088a4c93cce1

# Start the git cherry pick from 2 upstream commits before the 18e523f
#   18e523f appears to be an out of order commit due to the 1.0.14 hotfix
git cherry-pick -x -m 1 210489150ad93b0d34e565fa3c86fe420a27e775^..546fd5ff54ef7c0cf3ea08fc5c5166a9d21a4f7e

# Accept all of incoming conflicts and skip empty commits

...
# Pull in the commit that pulls in the package-lock.json generated when node-gyp=3.8.0
git cherry-pick -x -m 1 9cce9ec63aec2f91c0ac3a8e535cf57b0b05acbd

/hold

Closing in favor of the squashed commits in #30


Improve results.py coverage

We need to improve the results.py coverage.
Currently, it is in 23.01%.

Hi there!
I have marked this issue as stale because it has not had activity for 45 days.
Consider the following options:

If the issue refers to a large task, break it in smaller issues that can be solved in
less than 45 days;
Label the issue as wontfix or wontfix for now and close it.

Hi there!
I have marked this issue as stale because it has not had activity for 45 days.
Consider the following options:

If the issue refers to a large task, break it in smaller issues that can be solved in
less than 45 days;
Label the issue as wontfix or wontfix for now and close it.


[message] update comment/documentation

A GitHub outage earlier this week caused inconsistencies that required a force-push to master.  Can you remove 0f2e87c from this PR?


Add .meta() to zod types

Inspecting zod objects directly can be really useful for developing certain types of libraries. Of course tons of libs are already doing it - everything in the zod-to-x section of the documentation for example.
What if it were possible to attach arbitrary information, (or even just a string) via a meta function call? Having an explicit "meta" field could allow library developers to have their code react to zod schemas with much higher levels of customization, which could in turn allow us to improve developer experience in new ways.
The example I have in mind is this - Lets look at a tRPC procedure with a zod schema as an input validator:
getPosts: t.procedure.input(z.object({
  id: z.string().meta("The id of the post."),
  search: z.string().meta("Returns posts with this term in the body").
})).query(() => {
  //...
}),

Here we're attaching a description to each input parameter. Now we can do things like generating documentation directly from our validation schemas. Anyone who's developing a library that inspects zod schemas would have much more flexibility in the types of APIs they'd be able to develop.
It doesn't have to be a string, it could be any other typed object. One solution could be .meta<MetaType>(), or maybe an optional interface for generating a meta enabled client (probably a lot more difficult to create?) export const z = createZodClient<Meta>()
Selfishly, I'm developing a tool that inspects TRPC router and generates a manual testing / documentation UI automatically, which generates forms based on input zod schemas. With a meta function like this, it would allow developers to create documentation per input field with a minimal amount of effort. For me I'd just want a string but IDK if there are use cases for more complex objects or not
Would really love to add something like this, thoughts?

I was just checking out a lib called https://github.com/StefanTerdell/zod-to-json-schema and it seems they are converting json schemas's description with using .describe(). .describe() seems to be undocumented but already there. The drawback is that it only seems to support strings atm. It would indeed be very cool if we can define our own meta types.

Hi all,
This doesn't seem like a big lift. However, I wonder how prevalent is this use for integrating into other APIs? I certainly see the benefit (esp since other libraries are already weaponizing the .describe()), but I mean specifically the integration path: adding objects to a schema?
If we're going to work on making API integration accessible, I'd love to look at the ecosystem at large and see how other top-shelf libraries open themselves up to extensibility. Is it enough to add arbitrary objects to a schema? Or perhaps is there another introspection utility (maybe off of schema._def) that could be built and would be better suited long term?

It supports .describe() to add a description
But you can add when are creating the schema too like that:
const zodNumber = z.number({
  description: 'My zod number',
})

For me I miss the ability to access if is required or not in the .transform() or in the .refine()
I think zod could fix that by creating a obj meta and add things like required, description to solve that

Yeah I understand the opinions in https://github.com/colinhacks/zod/issues/273. It could very easily lead to anti-patterns however I can also see the benefits of adding a meta field. At the moment I'm using https://github.com/asteasolutions/zod-to-openapi which extends Zod by binding to methods.
Would be very useful to be able to add metadata which is quite closely related. eg. examples, description

I'm sure it could be used in pretty dumb ways, but potentially in pretty cool ways as well.
As someone who's built and worked on zod-introspecting packages and who uses zod schemas constantly (for form validation, automatic form construction, and trpc input validation), I can at least attest that having it being natively supported anywhere the schema._def as being very convenient during introspecting, and .meta() being probably the most natural or end-user-developers.
Personally, I'd prefer a way to have .meta to be able to be any object but also be typed upfront, to avoid having to explicitly type each call to .meta, that seems like a great DX and would support any extensions. (Having to do .meta<Type>() would be pretty tedious). I do really like the idea of just being able to create a custom zod instance that comes with a typed meta.
But even just some extra pre-defined meta fields like @samchungy is suggesting would be pretty useful on their own without opening the door to too much craziness.
the typical zod user would never use it anyways... (it's never going to footgun people who are just using it with trpc or react-hook-form, for example.) Would only affect devs that are leveraging some library that is already integrated to use the zod meta

@AndreiLucas123 each successive .optional(), et al. call "wraps" the underlying schema in a chain of nested children. So, its all in there but you would have to recurse manually to get to any of it - or build an API around it proper.
In any case it seems @colinhacks came down on one side under #273 (and I can't say I really disagree). Less is more and all that. FWIW, I don't think I'd ever see any other name brand "utility" packages (think of io-ts, lodash, caolan's async etc) catering to downstream package plugins and so on - especially when wrapping them is very easy.
Although! There could be great value in documenting the available internals of zod so that other devs can easily introspect/extend the schemas.

@maxArturo I think docs are a great idea here! I sure could've used them. I may write a short-but-potentially-useful guide up soon
Anyways - closing this, thanks for the discussion. Doesn't seem super clear whether meta() is a good idea or not

I wish to add some thoughts on this subject, should I do it in this issue, on #273 or create another one?
🤔


[Bugs] TNGF dies instead of cleanly exiting on terminate

Describe the bug
TNGF panics on free5gc exit with the message:
[FATA][TNGF][NGAP] panic: runtime error: invalid memory address or nil pointer dereference
goroutine 7 [running]:
runtime/debug.Stack()
	/usr/local/go/src/runtime/debug/stack.go:24 +0x5e
github.com/free5gc/tngf/internal/ngap/service.listenAndServe.func1()
	/home/xxxx/free5gc/NFs/tngf/internal/ngap/service/service.go:50 +0x38
panic({0x8e0920?, 0xd90730?})
	/usr/local/go/src/runtime/panic.go:914 +0x21f
github.com/free5gc/tngf/internal/ngap/service.listenAndServe(0x0?, 0x0?, 0x0?)
	/home/xxxx/free5gc/NFs/tngf/internal/ngap/service/service.go:121 +0x665
created by github.com/free5gc/tngf/internal/ngap/service.Run in goroutine 1
	/home/xxxx/free5gc/NFs/tngf/internal/ngap/service/service.go:36 +0xe5
./run.sh: line 69: wait: pid 15589 is not a child of this shell
The free5GC terminated successfully!

To Reproduce
Steps to reproduce the behavior:

Install free5GC v3.4.3
Run free5GC using ./run.sh -tngf and wait for it to fully load
Send SIGTERM (kill) free5GC
TNGF will crash

Expected behavior
TNGF should shutdown correctly, together with the other NFs.
Screenshots
N/A
Environment (please complete the following information):

free5GC Version: v3.4.3
OS: Ubuntu 20.04.6 LTS
Kernel version: 5.4.0-196-generic
go version: go1.21.8 linux/amd64
c compiler version (Option): 9.4.0

Trace File
Configuration File
free5gc-tngf-config-folder.zip
PCAP File
N/A
Log File
free5gc-logs.zip
Additional context
N/A

Hi, @oliveiraleo,
We will trace this issue, thank you!
Best regards,


Fix opensuse provisioning support

task/selinux.py, orchestra/remote.py: disable SELinux for opensuse/suse distros
nuke/actions.py: added support for sle/opensuse distros
Signed-off-by: Roman Grigorev<EMAIL_ADDRESS>

Can one of the admins verify this patch?

ok to test

@susebot run deploy

@susebot run deploy

Ti be honest

Would be nice to have a bit more of a description (in the commit msg) what is broken and why this is needed (see also https://github.com/ceph/ceph/blob/master/SubmittingPatches.rst#id7 )

commit message is updated

suse:tier0 in sepia for this PR http://pulpito.ceph.com/kyr-2019-12-09_19:46:45-suse:tier0-nautilus-none-basic-smithi/

suse:tier0 in sepia for this PR http://pulpito.ceph.com/kyr-2019-12-09_19:46:45-suse:tier0-nautilus-none-basic-smithi/

@ryg- the run above has used following file:
$ cat nautilus-overrides.yaml
overrides:
  install:
    ceph:
      packages:
        deb: []
        rpm: []
    repos:
    - name: nautilus
      url: https://download.opensuse.org/repositories/filesystems:/ceph:/nautilus/openSUSE_Leap_15.1/
    - name: distro
      url: http://download.opensuse.org/distribution/leap/15.1/repo/oss/
    - name: update
      url: http://download.opensuse.org/update/leap/15.1/oss/

And the command:
teuthology-suite -m smithi -k none --ceph-repo https://github.com/ceph/ceph --ceph nautilus --suite-repo https://github.com/suse/ceph --suite-branch ses6-downstream-commits --filter opensuse --suite suse:tier0 -d opensuse -D 15.1 -v $PWD/nautilus-overrides.yaml --newest 100 --limit 1 -t refs/pull/1369/merge

This I had to apply the patch from the #1373

@susebot run deploy


My second revup foo

Topic: foo2
Relative: foo

#
head
base
diff
date
summary

0
f72e72ca
ea97da75
diff
Apr 25 15:20 PM
1 file changed, 0 insertions(+), 0 deletions(-)

Reviews in this chain:
└https://github.com/em-cxu/revup-tutorial/pull/1 My first revup foo
　└https://github.com/em-cxu/revup-tutorial/pull/3 My second revup foo


Sort cells

Is there have any function or property for sorting bubbles-cells?

Hey @Elshad !
Not sure what you mean? If you sort your data source, then cells will display in that order. 😄

Sorry for my bad English. I mean when we add date to bubble, does it cause sorting by date?

Nope, you determine the order 😄


Error finding uses old .Net instead of project .Net

My project uses .Net Framework 4.5 and as such I have access to List<T>.Zip. But I get an error in vim that no such method exists, so it must be that it uses an older version.
I can compile the project just fine with msbuild, so it must be an issue with OmniSharp I gather. I tried to modify the omnisharp.json file like so:
{
   "MsBuild": {
      "UseLegacySdkResolver": true,
      "ToolsVersion": 4.5,
      "Platform": 4.5,
   },
   "sdk": {
      "4.5"
    }
}

but that doesn't seem to have any effect. I can see in the logs the following:
{"Event":"ProjectConfiguration","Body":{"ProjectId":"ac4fbce5885b4e84","TargetFrameworks":["v4.5"],"References":["c3f49fb8fd06f5e4","ca01f9c6c9519128","13bbf2cbcdb96469","6001a9b860f7e43","6a5eaa177b0b8d54"],"FileExtensions":["-1257278712"]},"Seq":23,"Type":"event"}

So it looks like the right version is taken (also without the changes in the omnisharp.json file), but for some reason OmniSharp complains. What could be the issue?

@Rakete1111 would you mind saying what your resolution was? It might help people finding this issue in future.

It wasn't an issue with OmniSharp, but with my code (missing an include due to my bad knowledge of C#). sorry

OK, thanks for reporting back.


Add the ability to add your own custom onIndexChange callback

Motivation
react-native-tab-views has the ability to have an onIndexChange callback. But right now this library is stopping you from using it because it has its own onIndexChange method. I think it would be good to expose this API to the users to that users can add an onIndexChange to the createMaterialTopTabNavigator.
The main reason to have this is to be able to easily react to navigation changes within the created navigator. This is useful for example tracking user behaviour. I made sure it triggers after the navigation so that the state of the navigation would add up to the index that is being send in to the onIndexChange callback.
The same result is hard to reach in a different way. There is onSwipeStart, onSwipeEnd and tabBarOnPress but those will also be triggered when the location is not changed by the swipe or press, and before the navigation actually finished.
Test plan
It would look something like this:
createMaterialTopTabNavigator({
  ... // other config
  onIndexChange () {
    // This will be triggered _after_ the index has changed and navigation is up to date
  },
});

I'm not sure if this is the best way to do it, but I wanted to open a PR to discuss this suggestion anyway.

@satya164 thank you for the fast reply.
That seems like a reasonable approach, but how would that work when using the createMaterialTopTabNavigator like so:
createMaterialTopTabNavigator(
  {
    start: startScreen,
    someOtherPage: someOtherPageScreen,
  },
  {
    initialRouteName: 'start',
    swipeEnabled: true,
    animationEnabled: true,
  },
);

@brentvatne it doesn't make sense to have it as an option imo. there's nothing you can achieve by having this option in createXNavigator which is totally static that you can't use by using onNavigationStateChange inside a component. If there's really a need for this, it should be a feature of the navigator instead of exposing the option from the library as the library's onIndexChange doesn't really make sense in context of React Navigation where the way to identify a route is routeName

@satya164 That's basically how I solved my case for this. I use a global onNavigationStateChange to detect a change of tabs. The "problem" with this approach is that it doesn't keep the navigator config in one place.
In my case the materialTopTabNavigator is nested within stackNavigators and a bottomTabNavigator. So even when with changes in the stackNavigator the check for tab switch will be triggered, it's just not acted upon.
With the purpose of clean code I think this PR could be an improvement. But I leave it up to you to merge it. You're right that it's already possible and that this perhaps becomes more of a "nice to have" than a "must have".

The "problem" with this approach is that it doesn't keep the navigator config in one place.

An event listener is not really a config imo. Event listeners are tied to the rendered instance, so specifying a listener during navigator creation isn't very useful. Say you have multiple instances of the navigator rendered, there's no way to distinguish which one triggered the listener.

Apart from that, this library is purely used as a wrapper around react-native-tab-views to add some specific behaviour and I feel that as many props should be passed on as possible to keep functionality in place.

I agree to some extent. However, exposing anything regarding the internal state management such as onIndexChange directly leads to a leaky abstraction. Anything regarding state management should be exposed in terms of how react-navigation does things instead of how the underlying implementation works to make the code consistent. I'm totally up for exposing this functionality, but not with the proposed API.
I think if this is a necessary feature, we should expose a prop on all navigators which allow you to listen to such events from a parent component. Can you describe your use case? What do you want to do in the callback?

Closing due to no response


Review actions that should persist their state into settings

[x] zoom - setting
[x] sidebar location - setting
[ ] status bar visibility - setting
toggle whitespace - not a setting
toggle wordwrap - not a setting
toggle control charactes - not a setting

I pushed:

sidebar location (https://github.com/Microsoft/vscode/issues/13338)
zoom (https://github.com/Microsoft/vscode/issues/13334)

I am not 100% convinced we should blindly write to settings for word wrap, white space etc. because other editors seem to apply this only for the current file and not all files. If I have 2 files side by side open and I click on word wrap for the left file (because it could be a markdown file), I would NOT expect word wrap to be enabled for the other file which could be any source file where word wrap does not make sense.

Make sense to distinguish settings that are temporary and per buffer only.
To me toggle status bar falls into the same category as the sidebar location.
Let's have an inventory at the top in the description.

I'd like to add

[ ] diff editor, show leading/trailing whitespace changes
[ ] diff editor, sideBySide vs inline diff

Please persist toggle whitespace, and toggle indent guides (#13385).

Yes, it is very helpful to have different settings per buffer (preferably by file type).

Reopening to track the work on the diff editor related commands.

I added test plan and release notes for zoom, status bar visibility and sidebar location.

@bpasero Can this be closed or is there anything pending to be done?


Update vendored packages

First, the dependencies are split between those of the sample directory and the rest. Then, all the dependencies are updated to the latest release version of each directly imported package.

Somehow, CI check didn't run for this PR...

Somehow, CI check didn't run for this PR...

I am not sure why CircleCI did not work and it seems that there is no way to start a CI job manually....
I confirmed that this PR is built successfully in my environment.

@Naoya-Horiguchi

this is a big (~75k lines) change, so reviewers are not expected to read line-by-line. Could you share how you generated this change

You are right, this looks like a massive change, I should have provided some hint how to reproduce it. Nevertheless, the actual effect is recorded in the vendor.json files, that one can inspect. To reproduce the full change, one can remove all the files from the vendor directories, except vendor.json, and issue govendor sync command.
The first patch was a semi-manual move of parts of vendor/vendor.json to sample/vendor/vendor.json, followed by govendor sync. The second patch was achieved by removing all the vendored packages and adding the missing direct dependencies back, specifying the latest release version, with govendor fetch name/of/the/package@version.

@ynamiki

Go module, which was experimentally introduced in Go 1.11 might provide better (and standard) way for managing dependencies. I will check it later.

Thanks, that looks interesting. We might want to adopt this approach when it is stable.

I was a bit too fast merging this request. Now, make lint fails for some reason: https://circleci.com/gh/hyperledger-labs/minbft/77#tests/containers/0. I'm wondering if that was the reason the CI didn't appear in the PR check results. Anyway, it seems like CircleCI silently ignores failures in linting stage. That might not be what we want.

Now, make lint fails for some reason
CircleCI silently ignores failures in linting stage

Fixed that in #55


[Event Request] Codeunit 101 Cust. Entry-SetAppl.ID - OnLoopTempCustLedgEntryOnSetApplId

Please add a new publisher event to the function SetApplId as follows:
[External] SetApplId(VAR CustLedgEntry : Record "Cust. Ledger Entry";ApplyingCustLedgEntry : Record "Cust. Ledger Entry";AppliesToID : Code[50])
.
.
IF TempCustLedgEntry.FINDSET THEN
  REPEAT
//>> Start New Publisher Call
    OnLoopTempCustLedgEntryOnSetApplId(TempCustLedgEntry);
//<< End New Publisher Call
    CustLedgEntryToUpdate.COPY(TempCustLedgEntry);
    CustLedgEntryToUpdate.TESTFIELD(Open,TRUE);
    CustLedgEntryToUpdate."Applies-to ID" := CustEntryApplID;
.
.

New Publisher Event:
LOCAL [IntegrationEvent] OnLoopTempCustLedgEntryOnSetApplId(VAR TempCustLedgEntry : Record "Cust. Ledger Entry")

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


NoMethodError in Raven.capture_exception(exception)

When I called Raven.capture_exception(exception), it caused a NoMethodError:
NoMethodError - undefined method `read_attribute_for_serialization' for #<Raven::Event:0x007fb3812d9128>:
  active_model_serializers (0.9.3) lib/active_model/serializer.rb:88:in `block (2 levels) in attributes'
  active_model_serializers (0.9.3) lib/active_model/serializer.rb:160:in `block in attributes'
  active_model_serializers (0.9.3) lib/active_model/serializer.rb:159:in `attributes'
  active_model_serializers (0.9.3) lib/active_model/serializer.rb:284:in `serializable_object'
  active_model_serializers (0.9.3) lib/active_model/serializable.rb:23:in `block in serializable_object_with_notification'
  activesupport (4.2.3) lib/active_support/notifications.rb:166:in `instrument'
  active_model_serializers (0.9.3) lib/active_model/serializable.rb:55:in `instrument'
  active_model_serializers (0.9.3) lib/active_model/serializable.rb:22:in `serializable_object_with_notification'
  active_model_serializers (0.9.3) lib/active_model/array_serializer.rb:43:in `block in serializable_object'
  active_model_serializers (0.9.3) lib/active_model/array_serializer.rb:42:in `serializable_object'
  active_model_serializers (0.9.3) lib/active_model/serializable.rb:16:in `block in as_json'
  activesupport (4.2.3) lib/active_support/notifications.rb:166:in `instrument'
  active_model_serializers (0.9.3) lib/active_model/serializable.rb:55:in `instrument'
  active_model_serializers (0.9.3) lib/active_model/serializable.rb:10:in `as_json'
  activesupport (4.2.3) lib/active_support/json/encoding.rb:35:in `encode'
  activesupport (4.2.3) lib/active_support/json/encoding.rb:22:in `encode'
  activesupport (4.2.3) lib/active_support/core_ext/object/json.rb:37:in `to_json_with_active_support_encoder'
  actionpack (4.2.3) lib/action_controller/metal/renderers.rb:116:in `block in <module:Renderers>'
  active_model_serializers (0.9.3) lib/action_controller/serialization.rb:53:in `block (2 levels) in <module:Serialization>'
  actionpack (4.2.3) lib/action_controller/metal/renderers.rb:45:in `block in _render_to_body_with_renderer'

@chadwtaylor Can you provide any additional details here? It looks like you're trying to render a Raven::Event with ActiveModelSerializers?
Some controller code or other context would be helpful.

I wonder if this is related to #372

Closing b/c I'm 99% certain this is #372. Re-open if you've got a reproduction.


Error: Failed to apply catalog: Found 1 dependency cycle:

Affected Puppet, Ruby, OS and module versions/distributions

Puppet: PE 2017.1
Ruby: ruby-<IP_ADDRESS>8
Distribution: RHEL 7.3
Module version: 2017-01-12 - Release 5.1.0

How to reproduce (e.g Puppet code you use)
class profile::splunk_forwarder {
include ::splunk::forwarder
}
splunk::params::src_root: 'puppet:///software'
splunk::params::version: '6.5.1'
splunk::params::build: 'f74036626f0c'
splunk::forwarder::forwarder_output:
'tcpout_defaultgroup':
section: 'default'
setting: 'defaultGroup'
value: 'default-autolb-group'
tag: 'splunk_forwarder'
'tcpout_default-autolb-group':
section: 'tcpout:default-autolb-group'
setting: 'server'
value: ''
tag: 'splunk_forwarder'
What are you seeing
Error: Failed to apply catalog: Found 1 dependency cycle:
(File[/opt/staging/splunk/splunkforwarder-6.5.1-f74036626f0c-linux-2.6-x86_64.rpm] => Staging::File[splunkforwarder-6.5.1-f74036626f0c-linux-2.6-x86_64.rpm] => Package[splunkforwarder] => File[/opt/staging/splunk/splunkforwarder-6.5.1-f74036626f0c-linux-2.6-x86_64.rpm])
Try the '--graph' option and opening the resulting '.dot' file in OmniGraffle or GraphViz
What behaviour did you expect instead
Splunk Forwarder should install from the software location served from my puppet master and then outputs should be configured.
Output log
Any additional information you'd like to impart

Just ran into same issue, and my fix is to avoid using splunk_forwarder as profile name. Apparently this will generate a tag which then conflicts with the dependency setup in the module. Hope this helps someone.

Thanks @starchx for the information on why this happens!


refactor: validate request parameters

Add validation of the request parameters in API endpoints, and do slightly refactor of the endpoints to improve code readability.
Issue #13.

Converted to draft because I need to check failures of the unit tests. Until then, it can be reviewed if you wish to do.

@mzeevi Can you please review and approve, if all good?

/approve


feat(minifier): normalize Infinity into f64::Infinity

#8148  👈 (View in Graphite)
main

How to use the Graphite Merge Queue
Add either label to this PR to merge it via the merge queue:

0-merge - adds this PR to the back of the merge queue
hotfix - for urgent hot fixes, skip the queue and merge this PR next

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

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


Changelog 2390

What is the purpose of this change? What does it change?
This is the changelog entry for PR #2391 / Bug #2390
Was the change discussed in an issue or in the forum before?
Closes #2390.
Checklist

[x] I have read the Contribution Guidelines
[ ] I have added tests for all changes in this PR
[ ] I have added documentation for the changes (in the manual)
[x] There's a new file in changelog/unreleased/ that describes the changes for our users (template here)
[ ] I have run gofmt on the code in all commits
[ ] All commit messages are formatted in the same style as the other commits in the repo
[x] I'm done, this Pull Request is ready for review

This is a copy of PR #2497 because for some reason I couldn't fix up that original PRs branch.

Thanks!


Administration Control

After some come and go argument, we've concluded that BerryPteroEgg will require an administrator control over the egg download, allowing the administrators to have specific versions of MCC be able to download, this may or may not also provide opportunities for administrations to also allow other applications to be usable on their platforms.

Apparently, it's better to keep it the way we do right now instead of having to handle additional pterodactyl related coding which could become a serious issue later on.


rename --subsampling_off to --skip_rasusa or --skip_subsampling

Description of feature
Really minor issue.
Mostly for consistency with the usual convention of naming these types of options as --skip_<tool>

--subsampling_off renamed to --skip_subsampling.


Please call out explicitly that your handler code will be called only once if you use a random handler key name

Type of issue
Missing information
Description
In the documentation about custom handlers it is written that for the handler key names you can use a control's property, or you can use a random name (like MyCustomization as in the documentation).
The consequences of this are not clear from the article.
If you do not use a property name but a random name then it will be called only when the control is created. The code is not run when properties are updated.
If you want to run the action code from e.g. AppendMapping when a property is updated, then the key should be nameof(ClassName.PropertyName`)
Page URL
https://learn.microsoft.com/en-us/dotnet/maui/user-interface/handlers/customize?view=net-maui-8.0
Content source URL
https://github.com/dotnet/docs-maui/blob/main/docs/user-interface/handlers/customize.md
Document Version Independent Id
70227b04-808f-1370-186b-10cfef525959
Article author
@davidbritch
Metadata

ID: 659665da-2ff3-3550-2e0c-20902c15c3e6
Service: dotnet-mobile
Sub-service: dotnet-maui

Hi @hansmbakker
Yes, you're right. It's a fair comment so I'll get the doc update.d


Enable inter-documentation links to be clickable

Summary
Details and comments
Extracts a partially unrelated change from #766

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 85.302%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
17480

Relevant Lines:
20492

💛  - Coveralls


A better landing page

Right now, it is just a simple title and login button. It will be cool if there's animation or some nice graphic to showcase Unfork's features. Feel free to propose!

I would love to work on this.

@JumaCodes hello, thanks for offering! May I know what you have in mind for enhancing it?

This issue is still available to whoever wants to pick it up!

I can try and brew up a mockup soon-
Now that I have Daisy figured :rofl:

This, with inspiration from this and with an inverted colour scheme?

This looks nice! You can start working on it if you like. May I ask what the top right button is for?
Also FYI #39 is in progress, so you will need to add a section for that app.


Make applyPatches patched callback async

On applyPatches() functions the loadFile() option accept a callback that must be called with the content of the file to be patched, but the patched() option don't, forcing to implement it as a sync function. Ironically, just in the next line there's a setTimeout(processIndex, 0) to process the next index!
What I propose is to add a callback argument to patched() function so it can both notify when it has finished and can start processing the next index, and also if there was an error while writting the patch. A more advanced option would be to use async.js for that, that's already a dependency and would make the code a bit cleaner.

Closing in favor of #130


Packaging

Android

[ ] Run 'npm install' from Gradle instead of having to do it manually

iOS/MacOS
Win32
UWP
Linux

We decided that this is not needed for the core repo. Dependent repos should create packages.


add connpool for message

Issues associated with this PR
Your PR should present related issues you want to solve.
Solutions
You should show your solutions about the issues in your PR, including the overall solutions, details and the changes. At this time, Chinese is allowed to describe these.
UT result
Unit Test is needed if the code is changed, your unit test should cover boundary cases, corner cases, and some exceptional cases. And you need to show the UT result.
Benchmark
If your code involves the processing of every request, you should give the Benchmark Result.
Code Style

Make sure Goimports has run
Show Golint result

Codecov Report

Merging #1216 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1216      +/-   ##
==========================================
- Coverage   56.04%   56.02%   -0.02%
==========================================
  Files         298      298
  Lines       26941    26941
==========================================
- Hits        15100    15095       -5
- Misses      10413    10417       +4
- Partials     1428     1429       +1

Impacted Files
Coverage Δ

pkg/module/http2/transport.go
78.47% <0.00%> (-0.29%)
:arrow_down:

pkg/module/http2/server.go
81.70% <0.00%> (-0.08%)
:arrow_down:

Continue to review full report at Codecov.

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

cc @Dennis8274

@taoyuanyuan , host now use atomic.LoadPointer to load && store, should not get a partial value


Fix type annotation

Fix for incorrect type annotation in quart.json.loads. From object_: str to object_: Union[str, bytes]

fixes #206

Checklist:

[ ] Add tests that demonstrate the correct behavior of the change. Tests should fail without the change.
[ ] Add or update relevant docs, in the docs folder and in code.
[X] Add an entry in CHANGES.rst summarizing the change and linking to the issue.
[ ] Add .. versionchanged:: entries in any relevant code docs.
[x] Run pre-commit hooks and fix any issues.
[x] Run pytest and tox, no tests failed.

Thanks


Fix MISUSE error. It comes from uninitialized flag_count variable.

@Shaienn definitely right, sorry about that...

@Shaienn did you use the code ? Does it actually works ?

Just started to implement :) Should works, why not?

Actually I didn't have the time nor the opportunity neither the necessity to use the code I wrote, the test runs just fine, but if you find bugs it is perfectly normal...
Really sorry about that...


Add watch-mode for assisting in application filter creation

We should have an operational mode specifiable on the commandline that make roflcoptor not daemonize or block any messages. It would output log messages to stdout or to a file. The log messages in this case will contain all commands the client sends and all responses the servers sends.
This would be useful for creating filter rules for applications.

https://github.com/subgraph/roflcoptor/commit/3317124e81929c00ee6bc374f9a3ee899309f9b4
done!


"Collapse multiple instances of the same line" is no longer working

I normally don't check the .lrc files. I just realize that this function is no longer working since ver. 1.1

Ah, yikes. This was removed in 1a963fd49 because it made the code simpler and I figured nobody cares about the disk space savings. I totally forgot that this was a user-visible setting.
I'm leaning towards just removing the option and keeping the current (never-collapse) behaviour in the interest of simplicity. If anybody desperately wants to keep the collapse behaviour then now is your chance to make that known (but no promises).

It really not because of disk space savings.  Though not very often, I reckon that some of us may wish to edit the lyrics manually to correct typos in one or two lines.  If same lines are grouped together, I just need to correct one line and there is no need to look for similar lines manually.  This can be done via an external editor but it's nice if we can do it with the app.  Anyway, it's not something critical and it's up to you.


[FEAT] Include SQL db

Need to consider adding a SQLite db using EF Core framework to support storage of keys and cache data

This is already implemented in alpha-de branch. Will submit a pull request to solve


Azure studio zip file (Apple chip - M1 mbp 14") can't be opened

Azure Data Studio Version:
OS Version:

Steps to Reproduce:

From this page:
https://learn.microsoft.com/en-us/sql/azure-data-studio/download-azure-data-studio?view=sql-server-ver16&tabs=redhat-install%2Credhat-uninstall
click on Apple Silicon for Mac.
Try to unzip get the error message:

unable to expand .....zip file, it is in an unsupported format. I've try the Archive tool, unzip utility on the command line. Nope.
Thanks.

@jdipallo I usually just double click on the downloaded file and the OS will extract the content. could you please try it?


Create new session for tab migration

CATEGORY
Choose one

[X] Bug Fix
[ ] Enhancement (new features, refinement)
[ ] Refactor
[ ] Add tests
[ ] Build / Development Environment
[ ] Documentation

SUMMARY

Friday we turned on the SQLLAB_BACKEND_PERSISTENCE at Lyft, and noticed a deadlock when migrating SQL Lab tabs from localStorage to the backend:
sqlalchemy.exc.OperationalError: (_mysql_exceptions.OperationalError) (1213, 'Deadlock found when trying to get lock; try restarting transaction')
[SQL: INSERT INTO tab_state (created_on, changed_on, extra_json, user_id, label, active, database_id, `schema`, `sql`, query_limit, latest_query_id, autorun, template_params, created_by_fk, changed_by_fk) VALUES (%s, %s, %s, %s, %s, %s, %s, %s, %s, %s, %s, %s, %s, %s, %s)]
[parameters: (datetime.datetime(2019, 11, 16, 1, 39, 45, 903405), datetime.datetime(2019, 11, 16, 1, 39, 45, 903410), '{}', '1963', 'dim_applicants', 1, 7, None, 'select * from events_annotated.event_ride_requested limit 10', None, None, 0, None, 1963, 1963)]

Note that there was no data loss, and the tabs were migrated successfully on a refresh.
I tried to repro this locally, migrating 20 tabs to a MySQL database, but I wasn't able to hit the deadlock. Looking at the code, it seems that we reuse the same session across all requests, so I refactored it to create a new session for each request.
@mistercrunch do you think this would fix the problem? I'm not super familiar with SQL Alchemy, but from what I read this should work.

TEST PLAN

I wasn't able to repro the original bug, but ran this code to ensure that it still works.
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

REVIEWERS
@mistercrunch

Codecov Report

Merging #8594 into master will increase coverage by 0.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #8594      +/-   ##
==========================================
+ Coverage   65.67%   65.69%   +0.01%
==========================================
  Files         474      474
  Lines       23587    23589       +2
  Branches     2574     2574
==========================================
+ Hits        15491    15496       +5
+ Misses       7927     7924       -3
  Partials      169      169

Impacted Files
Coverage Δ

superset/views/sql_lab.py
55.84% <0%> (-0.74%)
:arrow_down:

superset/utils/core.py
88.81% <0%> (+0.16%)
:arrow_up:

superset/dataframe.py
94.48% <0%> (+0.78%)
:arrow_up:

superset/db_engine_specs/sqlite.py
66.66% <0%> (+9.99%)
:arrow_up:

Continue to review full report at Codecov.

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

This sounded weird so I did some googling. This turned up:

If you are using InnoDB or any row-level transactional RDBMS, then it is possible that any write transaction can cause a deadlock, even in perfectly normal situations. ... If you catch one, the normal thing to do is just attempt to execute the failed query again.

https://stackoverflow.com/questions/2596005/working-around-mysql-error-deadlock-found-when-trying-to-get-lock-try-restarti/2596101#2596101
So apparently this is mostly MySQL specific (I'm assuming you are using MySQL?), i.e. the recommended course of action is to try the update, catch the deadlock exception and try again. As I hope this isn't common for other backends, perhaps this should be wrapped in a method in the MySQL spec. Not ideal, and might end up cluttering the codebase, so perhaps we need to think this through before making any changes..

@villebro thanks for the investigation!

So apparently this is mostly MySQL specific (I'm assuming you are using MySQL?), i.e. the recommended course of action is to try the update, catch the deadlock exception and try again. As I hope this isn't common for other backends, perhaps this should be wrapped in a method in the MySQL spec. Not ideal, and might end up cluttering the codebase, so perhaps we need to think this through before making any changes..

I think it's not only MySQL-specific, but configuration specific. It happened in our production MySQL database, but when I tested on my laptop using a local MySQL I couldn't repro it.
I'm going to try reproing against our staging DB from my laptop.

@betodealmeida bump - is this still required?


[AMBARI-23481] Selected Hosts dropdown is disabled after navigating a…

…way and coming back to hosts page.
What changes were proposed in this pull request?
Making sure selectedHosts gets updated while navigating back from host pages.
How was this patch tested?
21533 passing (23s)
48 pending

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/1659/
Test FAILed.
Test FAILured.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/1663/
Test PASSed.


[wip] Feature/router config

Issues associated with this PR
Your PR should present related issues you want to solve.
Solutions
You should show your solutions about the issues in your PR, including the overall solutions, details and the changes. At this time, Chinese is allowed to describe these.
UT result
Unit Test is needed if the code is changed, your unit test should cover boundary cases, corner cases, and some exceptional cases. And you need to show the UT result.
Benchmark
If your code involves the processing of every request, you should give the Benchmark Result.
Code Style

Make sure Goimports has run
Show Golint result

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@612003f). Click here to learn what that means.
The diff coverage is 78.4%.

@@            Coverage Diff            @@
##             master     #995   +/-   ##
=========================================
  Coverage          ?   55.86%
=========================================
  Files             ?      244
  Lines             ?    23580
  Branches          ?        0
=========================================
  Hits              ?    13172
  Misses            ?     9210
  Partials          ?     1198

Impacted Files
Coverage Δ

pkg/config/v2/config.go
40% <ø> (ø)

pkg/configmanager/dumper.go
17.24% <ø> (ø)

pkg/admin/server/apis.go
39.44% <0%> (ø)

pkg/config/v2/route.go
75.9% <100%> (ø)

pkg/config/v2/upstream.go
70.83% <100%> (ø)

pkg/configmanager/configmanager.go
97.65% <100%> (ø)

pkg/admin/server/server.go
89.65% <100%> (ø)

pkg/plugin/factory.go
60% <60%> (ø)

pkg/plugin/plugin.go
69.23% <69.23%> (ø)

pkg/plugin/server.go
7.4% <7.4%> (ø)

... and 4 more

Continue to review full report at Codecov.

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

#1013 tracking this pr, if this pr getting merged, i will know


[Bug] Export-DbaScript return 2 spaces if object doesn't exist

Before submitting a bug report:

[X] Running latest release of dbatools
[X] Verified errors are not related to permissions
[ ] Is this bug with Copy-DbaDatabase? You can replicate it using Backup-DbaDatabase ... | Restore-DbaDatabase ...

NOTE: Copy-DbaDatabase will not work in every environment and every situation. Instead, we try to ensure Backup & Restore work in your environment.

Environmental data

### PowerShell version:

Name                           Value
----                           -----
PSVersion                      5.1.14393.2879
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.14393.2879
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

### dbatools Module version:

Name    : dbatools
Path    : C:\Program Files\WindowsPowerShell\Modules\dbatools\0.9.811\dbatools.psd1
Version : 0.9.811

SQL Server:

Microsoft SQL Server 2016 (RTM-GDR) (KB4019088) - 13.0.1742.0 (X64)   Jul  5 2017 23:41:17   Copyright (c) Microsoft Corporation  Enterprise Edition: Core-based Licensing (64-bit) on Windows Server 2016 Datacenter 6.3 <X64> (Build 14393: )

Errors Received

no errors

Steps to Reproduce
/*
    Any T-SQL commands involved or used to produce test objects/data.
*/

$options = New-DbaScriptingOption
$options.IncludeIfNotExists = $true
$options.PrimaryObject = $false
$options.DriChecks = $true
$options.DriForeignKeys = $true
$options.DriWithNoCheck = $true
foreach ($Table in Get-DbaDbTable -ServerInstance $server -Database $database)
{if ($Table.name -NotLike 'MS*')
{$Table | Export-DbaScript -Path objects.sql -ScriptingOptionsObject $options -NoPrefix -Append}
}

Expected Behavior
creating object script on each row in file

Actual Behavior
if some tables don't have objects I want to script, function return 2 spaces. And file becomes look like zebra - code, some pages with spaces, code again

thanks for the report, we'll take a look when we have a moment

Issue was being caused by using a default value for the Prefix of "" even when -NoPrefix is specified.
Will submit fix along with some other updates to Export scripting functions

awesome! very much appreciated, @sqllensman

this will be resolved in dbatools 1.0, due in 20 days👍


Bug with dragging elements

Discovered an interesting bug. The problem is that for the google chrome browser in example dragging an item to the left differs from dragging the item to the right. To drag the left element, it's enough that the mouse pointer simply hovers on the right element, since the right one should be dragged over half of the left one.
However, if you set the width of the parent container to an even value, then dragging (right and left) is the normal

learned the code. The error in my case arises because the width of the first container is greater than the second. So while the drag logic is connected for half the element, then I'm watching this.
It may be worth adding a condition if the difference is greater than 1 in this case(function _onDragOver, line 775, in developed sortable.js), since chromium for these cases makes the width of the first larger than the subsequent ones by 1.

On quick look, its looks like the cursor position is being considered while dropping cards on right side. Whereas, while dropping on left side, card's beginning coordinates are being considered (which is the correct way)

Solved in 1.8.0-rc1.


Switch jackson dependency to 2.6.x

Jackson 2.7 does not work with Spring 4.2.3:
http://stackoverflow.com/questions/34721851/spring-4-2-3-and-fasterxml-jackson-2-7-0-are-incompatible

+1


PubSub : repeatedly extend the acknowledgement deadline for backlogged pull messages

For the asynchronous subscriber, backlogged messages have automatic acknowledgement deadline processing.
" Because the client library repeatedly extends the acknowledgement deadline for backlogged messages "
This feature would be useful for messages that are being processed via synchronous (pull.)

We've implemented our own wrapper around PubSub sync functionality that starts ScheduledExecutorService and performs automatic ack prolong action if the task is not yet acknowledged.

closing this and will track in #3500