Create OS X 10.7 and OS X 10.8 ISOs on newer macs

:bulb: Description
I would like the ability to create ISOs for OS X 10.7 and OS X 10.8 on newer macs.
:adhesive_bandage: Alternatives Solutions + Workarounds
The script createinstalliso seems to be able to do it, but maybe I'm missing a reason why it's not possible in Mist? It seems like those OSes are still running x86_64, so the architecture should not be an issue?
:information_source: Additional context

Could also be implemented in mist-cli. :white_check_mark:

@madsmtm if we are specifically speaking about Intel Macs, Mist is currently relying on the createinstallmedia command, which is not available in the Mac OS X Lion 10.7.5 and OS X Mountain Lion 10.8 installer app bundles.
In theory, the InstallESD.dmg found in each installer app bundle could be converted to an ISO, so without promising anything, I will investigate and report back 👍
Note: For Apple Silicon Macs, ISOs can only be created for macOS Big Sur 11 and later

Good news 🥳 converting the InstallESD.dmg within a Mac OS X Lion or OS X Mountain Lion installer to an ISO seems to do the trick!
See: #101


Use globalThis for default fetch

Use globalThis.fetch rather than window.fetch.  This enables automatic detection of fetch on Node.js 18+.
globalThis:

The globalThis property provides a standard way of accessing the global this value (and hence the global object itself) across environments. Unlike similar properties such as window and self, it's guaranteed to work in window and non-window contexts. In this way, you can access the global object in a consistent manner without having to know which environment the code is being run in. To help you remember the name, just remember that in global scope the this value is globalThis.

See: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/globalThis
Before, to use MedplumClient in a Node.js application, we recommended node-fetch:
import { MedplumClient } from '@medplum/core';
import fetch from 'node-fetch';

const medplum = new MedplumClient({ fetch });

Node.js 18 added a default fetch implementation.  Unfortunately, TypeScript's Node.js 18 type definitions do not include fetch, so you have to manually declare the type:
import { FetchLike, MedplumClient } from '@medplum/core';

declare global {
  var fetch: FetchLike;
}

const medplum = new MedplumClient({ fetch });

After this change, you don't need to do any of that.  You can simply use MedplumClient as you would expect:
import { MedplumClient } from '@medplum/core';

const medplum = new MedplumClient();

Coverage: 93.943% (+0.01%) from 93.932% when pulling 10229ef3d9b8a82050832eb855b4322cdfa0723a on cody-globalThis-fetch into 77fe4afc29222567963b28f9dd530f19ea36758d on main.


AttributeError: 'VectorizeFilter' object has no attribute 'filter_n'

I've downloaded the recent universe and executed this: pip install --upgrade universe
However, this issue still came up.....
[2017-10-23 13:43:20,102] Writing logs to file: /tmp/universe-4101.log
2017-10-23 13:43:20.105568: W tensorflow/core/platform/cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use SSE4.1 instructions, but these are available on your machine and could speed up CPU computations.
2017-10-23 13:43:20.105595: W tensorflow/core/platform/cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use SSE4.2 instructions, but these are available on your machine and could speed up CPU computations.
2017-10-23 13:43:20.105605: W tensorflow/core/platform/cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use AVX instructions, but these are available on your machine and could speed up CPU computations.
2017-10-23 13:43:20.105611: W tensorflow/core/platform/cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use AVX2 instructions, but these are available on your machine and could speed up CPU computations.
2017-10-23 13:43:20.105616: W tensorflow/core/platform/cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use FMA instructions, but these are available on your machine and could speed up CPU computations.
2017-10-23 13:43:20.109914: I tensorflow/core/distributed_runtime/rpc/grpc_channel.cc:215] Initialize GrpcChannelCache for job ps -> {0 -> <IP_ADDRESS>:12222}
2017-10-23 13:43:20.109959: I tensorflow/core/distributed_runtime/rpc/grpc_channel.cc:215] Initialize GrpcChannelCache for job worker -> {0 -> localhost:12223, 1 -> <IP_ADDRESS>:12224}
2017-10-23 13:43:20.110892: I tensorflow/core/distributed_runtime/rpc/grpc_server_lib.cc:316] Started server with target: grpc://localhost:12223
[2017-10-23 13:43:20,111] Making new env: PongDeterministic-v4
[2017-10-23 13:43:21,207] Trainable vars:
[2017-10-23 13:43:21,207]   global/l1/W:0 (3, 3, 1, 32)
[2017-10-23 13:43:21,209]   global/l1/b:0 (1, 1, 1, 32)
[2017-10-23 13:43:21,209]   global/l2/W:0 (3, 3, 32, 32)
[2017-10-23 13:43:21,209]   global/l2/b:0 (1, 1, 1, 32)
[2017-10-23 13:43:21,209]   global/l3/W:0 (3, 3, 32, 32)
[2017-10-23 13:43:21,209]   global/l3/b:0 (1, 1, 1, 32)
[2017-10-23 13:43:21,209]   global/l4/W:0 (3, 3, 32, 32)
[2017-10-23 13:43:21,209]   global/l4/b:0 (1, 1, 1, 32)
[2017-10-23 13:43:21,209]   global/rnn/basic_lstm_cell/kernel:0 (544, 1024)
[2017-10-23 13:43:21,209]   global/rnn/basic_lstm_cell/bias:0 (1024,)
[2017-10-23 13:43:21,209]   global/action/w:0 (256, 6)
[2017-10-23 13:43:21,209]   global/action/b:0 (6,)
[2017-10-23 13:43:21,209]   global/value/w:0 (256, 1)
[2017-10-23 13:43:21,210]   global/value/b:0 (1,)
[2017-10-23 13:43:21,210]   local/l1/W:0 (3, 3, 1, 32)
[2017-10-23 13:43:21,210]   local/l1/b:0 (1, 1, 1, 32)
[2017-10-23 13:43:21,210]   local/l2/W:0 (3, 3, 32, 32)
[2017-10-23 13:43:21,210]   local/l2/b:0 (1, 1, 1, 32)
[2017-10-23 13:43:21,210]   local/l3/W:0 (3, 3, 32, 32)
[2017-10-23 13:43:21,210]   local/l3/b:0 (1, 1, 1, 32)
[2017-10-23 13:43:21,210]   local/l4/W:0 (3, 3, 32, 32)
[2017-10-23 13:43:21,210]   local/l4/b:0 (1, 1, 1, 32)
[2017-10-23 13:43:21,210]   local/rnn/basic_lstm_cell/kernel:0 (544, 1024)
[2017-10-23 13:43:21,210]   local/rnn/basic_lstm_cell/bias:0 (1024,)
[2017-10-23 13:43:21,211]   local/action/w:0 (256, 6)
[2017-10-23 13:43:21,211]   local/action/b:0 (6,)
[2017-10-23 13:43:21,211]   local/value/w:0 (256, 1)
[2017-10-23 13:43:21,211]   local/value/b:0 (1,)
[2017-10-23 13:43:21,211] Events directory: /tmp/pong/train_0
[2017-10-23 13:43:21,338] Starting session. If this hangs, we're mostly likely waiting to connect to the parameter server. One common cause is that the parameter server DNS name isn't resolving yet, or is misspecified.
2017-10-23 13:43:21.380463: I tensorflow/core/distributed_runtime/master_session.cc:999] Start master session 29dafb83b925e4f3 with config:
intra_op_parallelism_threads: 1
device_filters: "/job:ps"
device_filters: "/job:worker/task:0/cpu:0"
inter_op_parallelism_threads: 2
[2017-10-23 13:43:21,750] Starting training at step=0
Exception in thread Thread-1:
Traceback (most recent call last):
File "/usr/lib/python2.7/threading.py", line 801, in __bootstrap_inner
self.run()
File "/home/icsl/Downloads/universe-starter-agent-master/a3c.py", line 93, in run
self._run()
File "/home/icsl/Downloads/universe-starter-agent-master/a3c.py", line 102, in _run
self.queue.put(next(rollout_provider), timeout=600.0)
File "/home/icsl/Downloads/universe-starter-agent-master/a3c.py", line 112, in env_runner
last_state = env.reset()
File "/home/icsl/gym/gym/core.py", line 104, in reset
return self._reset()
File "/usr/local/lib/python2.7/dist-packages/universe/wrappers/vectorize.py", line 46, in _reset
observation_n = self.env.reset()
File "/home/icsl/gym/gym/core.py", line 104, in reset
return self._reset()
File "/usr/local/lib/python2.7/dist-packages/universe/vectorized/vectorize_filter.py", line 30, in _reset
observation_n = [filter._after_reset(observation) for filter, observation in zip(self.filter_n, observation_n)]
AttributeError: 'VectorizeFilter' object has no attribute 'filter_n'

I have the same error with gym==0.9.4. With gym==0.7.4 I don't.


'Start Developing iOS Apps' Apple guide changes

Changes made in the iOS section:

Link to Start Developing iOS Apps Today (Objective-C) is dead, replaced with one alive found on Google.
Added the Swift version (only one currently available and updated, on Apple's website) of the guide (only available in HTML).

Thank you!


Remove filename validation

The current filename validation is too strict; furthermore, it is not currently necessary since the underlying file system is capable of raising an error if it receives an invalid file name.
Fixes #303

Correct.

thanks for the quick fix! do you have an expected build / version number for this?
Also am concerned that no tests caught this and there don't seem to be any added here. I'll be pinning versions/builds from now on but would be great to see some tests to catch regressions; I'd like to eventually be confident enough to pull latest again.


RCPT_TO / Return-Path

Comparing details from Logstash and Exchange Online I noticed that EOP reports the Return-Path as the from field (possibly wrongly? But that's another issue) and so I wondered why I'm only recording the To and From addresses. Checking the code I see no reference to rcpt_to or return-path, has anyone added this to their filter?
It's relevant to me for tracking bounce message processing. Bounce notifications are supposed to go to the Return-Path address rather than the from address. I'm aware this isn't always adhered to by mailservers.

Hi @dmgeurts
Your question is not really clear on this, but I'm assuming for the sake of clarity that:

This is about a Postfix server (sending end) sending to Exchange (receiving end)
You are on the sending end

Return-Path is a header that is added by the receiving server. Normally it is generated by using the RFC5321.MailFrom (e.g. the address that was sent during SMTP transfer in the MAIL FROM command). This header is therefor not available to the sending server. Apart from that, Postfix never cares about headers anyway, by default it only logs SMTP envelope data.
The Postfix server on the sending end will log the original sender in the qmgr logline when it starts processing the queued message, f.i. in https://github.com/whyscream/postfix-grok-patterns/blob/master/test/qmgr_0002.yaml, but not the actual MAIL FROM contents that the smtp program will send, AFAIK. Under normal circumstances this should be equal to the value in the qmgr logline, though.
NB I'm unsure how logs look like if you're rewriting the sender with Postfix config on the sending server. Feel free to contribute examples if you have them.

The reason that you're not seeing this in the logstash filter code is because the from is parsed using generic key-value parsing.

Nov 15 11:40:22 mailserver postfix/qmgr[15810]: B750520112<EMAIL_ADDRESS>size=2080, nrcpt=1 (queue active)
Nov 15 11:40:25 mailserver postfix/smtp[21481]: Untrusted TLS connection established to cust21691-2.in.mailcontrol.com[<IP_ADDRESS>]:25: TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)
Nov 15 11:40:26 mailserver postfix/smtp[21481]: B750520112<EMAIL_ADDRESS>relay=cust21691-2.in.mailcontrol.com[<IP_ADDRESS>]:25, delay=3.3, delays=0.13/0.01/2.2/0.95, dsn=2.0.0, status=sent (250 2.0.0 2AFAeNTj000812 Message accepted for delivery)
Nov 15 11:40:26 mailserver postfix/qmgr[15810]: B750520112: removed

In this SMTP conversation the remote server cust21691-2.in.mailcontrol.com is probably going to use<EMAIL_ADDRESS>as the return path.

@whyscream, my apologies!
Yes, your assumption is correct. I'm tracking emails forwarded from Postfix to O365. Postfix appears to be logging the rcpt-to address and O365 logs the from address.
Nov 18 19:05:17 host postfix/cleanup[3304557]: A82C61E1A28<EMAIL_ADDRESS>Nov 18 19:05:17 esh01 postfix/qmgr[2855631]: A82C61E1A28<EMAIL_ADDRESS>size=13780, nrcpt=6 (queue active)
Nov 18 19:05:18 host postfix/smtp[3304739]: A82C61E1A28<EMAIL_ADDRESS>relay=domain-com.mail.eo.outlook.com[<IP_ADDRESS>]:25, delay=1.3, delays=0/0.01/0.15/1.1, dsn=2.6.0, status=sent (250 2.6.0<EMAIL_ADDRESS>[InternalId=15298673512055, Hostname=AS4P190MB2021.EURP190.PROD.OUTLOOK.COM] 22586 bytes in 0.148, 148.174 KB/sec Queued mail for delivery)
Nov 18 19:05:18 host postfix/qmgr[2855631]: A82C61E1A28: removed

But O365 records:

I know that the service is configured with From: service+no-reply@ and<EMAIL_ADDRESS>I'm not complaining, but just noticed the difference and am wondering if there's anything that I could do to log both. But I guess that would have to start with Postfix actually logging both header fields.

I am still unsure what your issue is. Postfix is sending with<EMAIL_ADDRESS>in the RFC5321.MailFrom, so O365 will use that when bounces occur.
I have zero knowledge on O365 (and I would prefer to keep it that way 😉), but the fact that O365 shows the<EMAIL_ADDRESS>address as the "Sender" might just be because O365 shows you the E-mail header value and not the RFC5321.MailFrom. Thus, the log/interface you are looking at is not suitable for researching bounces.
The E-mail headers are just content, they can contain anything. If you're tracking bounces, the only thing you should be interested in is RFC5321 addresses (i.e. the SMTP envelope data).
That being said, you can configure Postfix to log E-mail header data using header checks. Note again that Postfix cannot reliably log the Return-path header as it is non-existent, or may be discarded and replaced when 0365 accepts the message. But the value that Postfix offers O365 as input for the Return-path data, is already logged by the qmgr.

Closing this issue, feel free to reopen.
Note that I haven't seen any question yet directly related to this project, as the subject was mostly the content of  log data, and not parsing it.


[Backport main] Add bwc version 2.7.1 (#7371)

Signed-off-by: GitHub<EMAIL_ADDRESS>Co-authored-by: opensearch-ci-bot<EMAIL_ADDRESS>(cherry picked from commit 5a5c2692af93df14d40efc639daea99699b0edad)
Description

Backports #7371 to main

Related Issues

#7375
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff
[x] Commit changes are listed out in CHANGELOG.md file (See: Changelog)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Gradle Check (Jenkins) Run Completed with:

RESULT: FAILURE ❌
URL: https://build.ci.opensearch.org/job/gradle-check/14784/
CommitID: 605a94d
Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green.
Is the failure a flaky test unrelated to your change?

#7327

https://build.ci.opensearch.org/job/gradle-check/14783/

Gradle Check (Jenkins) Run Completed with:

RESULT: FAILURE ❌
URL: https://build.ci.opensearch.org/job/gradle-check/14783/
CommitID: 605a94d
Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green.
Is the failure a flaky test unrelated to your change?

#7327

Codecov Report

Merging #7379 (605a94d) into main (7c3c411) will decrease coverage by 0.15%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff              @@
##               main    #7379      +/-   ##
============================================
- Coverage     70.69%   70.55%   -0.15%
+ Complexity    59631    59583      -48
============================================
  Files          4876     4876
  Lines        285816   285817       +1
  Branches      41162    41162
============================================
- Hits         202051   201647     -404
- Misses        67136    67575     +439
+ Partials      16629    16595      -34

Impacted Files
Coverage Δ

...ibs/core/src/main/java/org/opensearch/Version.java
77.33% <100.00%> (+0.11%)
:arrow_up:

... and 477 files with indirect coverage changes


Test

Coverage remained the same at 35.592% when pulling e9adb2a8135653154e651bf286677cae3752ed84 on gyuho:test into 6c90c79081e3914e6476e59b4aa504b516a26ad5 on kubernetes-sigs:master.


define general_holdposition enum

This may be a duplicate of fix holdposition, but I couldn't tell based on the description.
Like MOVE, PATROL, HOLDPOSITION has a new enum that appears to be the generalization of specialized commands (If I understand correctly, which is questionable here).
It looks like move and patrol enum values were changed, rather than adding new enums for the generalized values. I propose doing the same for HOLDPOSITION (change value from 18 to 3793)

Yep, this is mostly the dup of 'fix holdposition'.
A small portion of history :)
I thought about adding 'general' versions of the same commands as it allows to support older game versions. But so far it is unlikely that we can (should) support something older than 4.10.
From the other point of view, having both 'general' and 'basic' versions of the same command could be quite confusing for a developer.

so, if we want to learn from an old replay, will the commands be different?

Actually, a good question. Yes, might be different if the replay game version < 4.10.
It is a good idea to check the replays.

:) I can't get my replays working (issue #25), which is why I'm working on my scripted bot. If you can point me in the right direction for fixing 25 I would be glad to help. I'm just totally stumped by it.

Will be fixed together with #43 in the next release.

Fixed.


Handle did Rotation

https://didcomm.org/book/v2/didrotation

Didcomm protocol gives the possibility to parties to change their dids incase the suspect it has been tempered or for any other reason, and there is not level or specific message type for rotation so can be done at any level after and out of band invitation and should be handled


single file mode: EFM does not support NULL root directories

Neovim version (nvim -v)
v0.6.0-dev+575-g2ef9d2a66
Language server name/version
efm-langserver 0.0.37
Operating system/version
macOS Big Sur 11.3.1
:checkhealth
lspconfig: require("lspconfig.health").check()
========================================================================
  - INFO: efm: configuration checked.
  - INFO: clangd: configuration checked.
  - INFO: sumneko_lua: configuration checked.
  - INFO: pyright: configuration checked.

but this message was printed to cmdline:

Error detected while processing FileType Autocommands for "markdown":
E5108: Error executing lua .../pack/packer/start/nvim-lspconfig/lua/lspconfig/util.lua:288: table index is nil
stack traceback:
        .../pack/packer/start/nvim-lspconfig/lua/lspconfig/util.lua:288: in function 'add'
        ...ck/packer/start/nvim-lspconfig/lua/lspconfig/configs.lua:204: in function 'try_add'
        [string ":lua"]:1: in main chunk
Error executing vim.schedule lua callback: ...neovim/HEAD-2ef9d2a_1/share/nvim/runtime/lua/vim/lsp.lua:859: RPC[Error] code_name = unknown, code = 0, message = "parse \"\": empty url"
stack traceback:
        [C]: in function 'assert'
        ...neovim/HEAD-2ef9d2a_1/share/nvim/runtime/lua/vim/lsp.lua:859: in function 'cb'
        vim.lua:285: in function <vim.lua:285>

Steps to reproduce using "nvim -u minimal_init.lua"
local efm_prettier = {formatCommand = "prettier --stdin-filepath ${INPUT}", formatStdin = true}
lsp["efm"].setup {
    capabilities = require("cmp_nvim_lsp").update_capabilities(vim.lsp.protocol.make_client_capabilities()),
    init_options = {documentFormatting = true},
    filetypes = {
        "css",
        "html",
        "javascript",
        "javascriptreact",
        "json",
        "lua",
        "markdown",
        "python"
    },
    settings = {
        rootMarkers = {".git/"},
        languages = {
            css = {efm_prettier},
            html = {efm_prettier},
            javascript = {efm_prettier},
            javascriptreact = {efm_prettier},
            json = {efm_prettier},
            lua = {{formatCommand = "luafmt --stdin", formatStdin = true}},
            markdown = {efm_prettier},
            python = {{formatCommand = "autopep8 -", formatStdin = true}}
        }
    },
    on_attach = function(client, buffer_nr)
        if client.resolved_capabilities.document_formatting then
            utils.map('n','<c-f>', '<cmd>lua vim.lsp.buf.formatting()<cr>', { silent = true }, buffer_nr)
        end

        if client.resolved_capabilities.document_range_formatting then
            utils.map('v','<c-f>', '<cmd>lua vim.lsp.buf.range_formatting()<cr>', { silent = true }, buffer_nr)
        end
    end
}

lspconfig-setup ^
Just started getting this error after updating to neovim HEAD this morning. Have no changes in config, empty efm-langserver/config.yaml file  (this has worked fine for months).
Downgrading to neovim stable 0.5.1 also gives the same error, and lsp log gives:
[ERROR][2021-11-11 15:16:52] .../vim/lsp/rpc.lua:412	"rpc"	"efm-langserver"	"stderr"	"2021/11/11 15:04:46 efm-langserver: reading on stdin, writing on stdout\n"
[ERROR][2021-11-11 15:16:53] .../lua/vim/lsp.lua:734	"LSP[sumneko_lua]"	"on_error"	{  code = "SERVER_REQUEST_HANDLER_ERROR",  err = "...AD-2ef9d2a_1/share/nvim/runtime/lua/vim/lsp/handlers.lua:74: attempt to index local 'client' (a nil value)"}

NOTE: the second error line about indexing client nil value, only happens sometimes.
The EFM server doesn't attach to any buffer it is configured for because of this exit code.
Full Error is:
Error executing vim.schedule lua callback: ...neovim/HEAD-2ef9d2a_1/share/nvim/runtime/lua/vim/lsp.lua:859: RPC[Error] code_name = unknown, code = 0, message = "parse \"\": empty url"
stack traceback:
        [C]: in function 'assert'
        ...neovim/HEAD-2ef9d2a_1/share/nvim/runtime/lua/vim/lsp.lua:859: in function 'cb'
        vim.lua:285: in function <vim.lua:285>

Expected behavior
EFM should attach to buffer, and behave correctly
Actual behavior
EFM exits with error code (mentioned above). even :LspRestart doesnt help.
log files say:
[INFO][2021-11-11 15:30:58] .../vim/lsp/rpc.lua:258	"Starting RPC client"	{  args = {},  cmd = "efm-langserver",  extra = {}}
[DEBUG][2021-11-11 15:30:58] .../vim/lsp/rpc.lua:339	"rpc.send"	{  id = 1,  jsonrpc = "2.0",  method = "initialize",  params = {    capabilities = {      callHierarchy = {        dynamicRegistration = false      },      textDocument = {        codeAction = {          codeActionLiteralSupport = {            codeActionKind = {              valueSet = { "", "Empty", "QuickFix", "Refactor", "RefactorExtract", "RefactorInline", "RefactorRewrite", "Source", "SourceOrganizeImports", "quickfix", "refactor", "refactor.extract", "refactor.inline", "refactor.rewrite", "source", "source.organizeImports" }            }          },          dataSupport = true,          dynamicRegistration = false,          resolveSupport = {            properties = { "edit" }          }        },        completion = {          completionItem = {            commitCharactersSupport = true,            deprecatedSupport = true,            documentationFormat = { "markdown", "plaintext" },            insertReplaceSupport = true,            labelDetailsSupport = true,            preselectSupport = true,            resolveSupport = {              properties = { "documentation", "detail", "additionalTextEdits" }            },            snippetSupport = true,            tagSupport = {              valueSet = { 1 }            }          },          completionItemKind = {            valueSet = { 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 18, 19, 20, 21, 22, 23, 24, 25 }          },          contextSupport = false,          dynamicRegistration = false        },        declaration = {          linkSupport = true        },        definition = {          linkSupport = true        },        documentHighlight = {          dynamicRegistration = false        },        documentSymbol = {          dynamicRegistration = false,          hierarchicalDocumentSymbolSupport = true,          symbolKind = {            valueSet = { 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 18, 19, 20, 21, 22, 23, 24, 25, 26 }          }        },        hover = {          contentFormat = { "markdown", "plaintext" },          dynamicRegistration = false        },        implementation = {          linkSupport = true        },        publishDiagnostics = {          relatedInformation = true,          tagSupport = {            valueSet = { 1, 2 }          }        },        references = {          dynamicRegistration = false        },        rename = {          dynamicRegistration = false,          prepareSupport = true        },        signatureHelp = {          dynamicRegistration = false,          signatureInformation = {            activeParameterSupport = true,            documentationFormat = { "markdown", "plaintext" },            parameterInformation = {              labelOffsetSupport = true            }          }        },        synchronization = {          didSave = true,          dynamicRegistration = false,          willSave = false,          willSaveWaitUntil = false        },        typeDefinition = {          linkSupport = true        }      },      window = {        showDocument = {          support = false        },        showMessage = {          messageActionItem = {            additionalPropertiesSupport = false          }        },        workDoneProgress = true      },      workspace = {        applyEdit = true,        configuration = true,        symbol = {          dynamicRegistration = false,          hierarchicalWorkspaceSymbolSupport = true,          symbolKind = {            valueSet = { 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 18, 19, 20, 21, 22, 23, 24, 25, 26 }          }        },        workspaceEdit = {          resourceOperations = { "rename", "create", "delete" }        },        workspaceFolders = true      }    },    clientInfo = {      name = "Neovim",      version = "0.6.0"    },    initializationOptions = {      documentFormatting = true    },    processId = 52934,    trace = "off"  }}
[ERROR][2021-11-11 15:30:58] .../vim/lsp/rpc.lua:412	"rpc"	"efm-langserver"	"stderr"	"2021/11/11 15:30:58 efm-langserver: reading on stdin, writing on stdout\n"
[DEBUG][2021-11-11 15:30:58] .../vim/lsp/rpc.lua:446	"rpc.receive"	{  error = {    code = 0,    message = 'parse "": empty url'  },  id = 1,  jsonrpc = "2.0"}

Log file
No response

As far as I can tell, EFM uses it's own internal logic to resolve the root, but fails trying to parse the NULL root directory when we send it. Someone can file an issue upstream.

Closed via https://github.com/neovim/nvim-lspconfig/pull/1413
EFM will now only open if there is a detected git root directory that lspconfig can use to infer a project structure.
If you would like single file support for EFM, I strongly encourage you to file an upstream issue. Note, while it is allowed by the specification for servers to send NULL as rootDirectory or workspaceFolders, it is not required that servers handle this in any particular way.

This has been fixed in EFM master already, I asked for a new release: https://github.com/mattn/efm-langserver/issues/185


Version support matrix can add kubernetes

I just saw this in readme.md
The Ingress controller is tested on Kubernetes version 1.8 through 1.10.

I want to now kubernetes-ingress-controller is support Kubernetes version 1.11 1.12 1.13 1.14 1.15？

Hello @xbazhen,
It seems like the README is out of date.
I'll push a commit up to fix that.
We have tested KIC up to Kubernetes version 1.14 and have not seen issues yet.

Removed in https://github.com/Kong/kubernetes-ingress-controller/commit/d925c36dd4a5d21f27bbf301faf899708764ef85


Private Link Service availability for Azure Cache Premium?

Congrats on the recent GA of Azure Private Link for Azure Services.
What is the plan and availability of support for Azure Cache Premium?
Are there any workarounds available until the availability date?

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

ID: fbfc7ccc-7fd0-9bb7-7d39-e9cfa09736ce
Version Independent ID: c0e7f101-5707-63ff-8193-c74a3e1d9a75
Content: What is Azure Private Link?
Content Source: articles/private-link/private-link-overview.md
Service: private-link
GitHub Login: @malopMSFT
Microsoft Alias: allensu

@dataslicer , We are looking into this query and will update you as soon as possible.

@dataslicer , Appreciate your patience. AFAIK, there is no public announcement as of yet. Please do check Azure Updates page for news. There is no doc update required as of now and that it will be updated once available.
We are closing this issue for now. If there are further questions regarding this matter, please reply and we will gladly continue the discussion.


Rename String class to StringHelper for php 7 compatibility

In php 7 and above, classes were added for primitives like String and those keywords are now reserved. This has necessitated updating many projects to name their String classes something else. I've generally seen StringHelper and used that here for consistency.

Weird. Came back around to my project using a fork of this and had to make the same change just a little while ago, although I changed it to "Str". "StringHelper" is probably a better name.


Support for DefinedNames

Address features in #15
Adds support for DefinedNames for direct cell, cell ranges and column references.
Does not support DefinedNames for none cell references or formulas.
New to Cell class:

public string ColumnName
public IReadOnlyList<string> CellNames
public string CellName
This is an alias to the first entry in CellNames or the Column letter + row number
public int RowNumber

New to Row class:

public Cell GetCellByColumnName(string name)
Retrieves a cell in the row that is only in named column

New to Worksheet class:

public IEnumerable<Cell> GetCellsInRange(CellRange range)
Retrieves all cells in a specified range

New to FastExcell Class

internal IReadOnlyDictionary<string, DefinedName> DefinedNames { get; private set; }
public IEnumerable<IEnumerable<Cell>> GetCellRangesByDefinedName(string definedName, int? sheetId = null)
Retrieves ranges of cells by their defined name
public IEnumerable<Cell> GetCellsByDefinedName(string definedName, int? worksheetIndex = null)
Gets all cells by defined name
Like GetCellRangesByCellName, but just retreives all cells in a single list
public Cell GetCellByDefinedName(string definedName)
Returns cell by defined name
If theres more than one, this is the first one.
public IEnumerable<Cell> GetCellsByColumnName(string columnName, int rowStart = 1, int? rowEnd = null)
Returns all cells in a column by name, within optional row range
public int GetWorksheetIndexFromName(string name)
Retrieves the index for given worksheet name

Added new public class CellRange
Used by worksheet function GetCellsInRange
Constructor:
public CellRange(string columnStart, string columnEnd, int rowStart = 1, int? rowEnd = null)
Added new internal class DefinedName
This represents a defined name.
Added public class DefinedNameLoadException
For exceptions thrown when loading defined names - only thrown from DefinedName.cs

@mrjono1 Added for your consideration. I am still testing and reviewing code so no need to merge yet.

looking good, do you mind if I add some inline comments/suggestions

Comments/suggestions definitely welcome. It might be better to post here and reference file/line numbers.

Sibz are you all happy with the update? I am ready to merge it

Almost, just got back from work ... 1 more quick commit for cell.cs

That should do it :)

done and deployed, I'll add a section to the readme that acknowledges your input soon, thanks :)


Update stable version in repo from 1.0.3 to 1.1.0

Think about update stable version to 1.1.0
Version: stable=1.0.3 (132 days old) => latest=1.1.0 (15 days old)
Installs: stable=2248 (66.18%), latest=480 (14.13%), total=3397
Click to use developer portal
Click to edit
Do not close this issue manually as a new issue will be created if condition for update still exists.
Please drop a comment if any reason exists which blocks updating to version 1.1.0 at this time.
Note: This is an automatically generated message and not personally authored by bluefox!
@mcm1957 for evidence

Is there any reason to delay adding this adapter to stable any longer?
Please drop a commit if you know any reason to refrain from adding.
If there is no response until 16.8.2024 we could decide to update anyway.

This issue seems to be outdated.
This issue suggests to update the stable version of this adapter to 1.1.0 but the current stable version is already 1.1.0.
So this issue should be closed.
@mcm1957 for evidence


Add FLUSH PRIVILEGES support

The review comment is the most important, but make sure to look over the smaller comments as they're more informational.


feat(pixels): add my pixel (3,36)

Checklist

[x] I ran npm test locally and it passed without errors.

[x] I only edited the _data/pixels.json file.

[x] I entered the username in the pixels.json that I'm also using to create this pull request.
[x] I acknowledge that all my contributions will be made under the project's license.

I think I've fixed the overwrite issue? I'm not sure how to proceed to have this merged. Thanks for making a fun, simple project for us first-timers!


Adding to PSGallery

@chritea, So I started looking into the powershell gallery and I saw no requirements for pester. I was wondering where you saw or heard about this requirement. I still intend to create pester tests, just curious if I am missing something.

Added to PSGallery


Make test topic (ZTF data)

Make a non-expiring test topic with ZTF data. Probably simply requires marking one of the existing nightly topics as non-expiring.

Topic "ztf_test" created (hopefully with the right options to not expire) on the local ZTF mirror. Should contain a full copy of the data from the topic ztf_20200404_programid1.

Assigned to Roy to review.

Works great. See below.
$ hostname
lasair-dev-head
$ source activate lasair
$ python bin/ingest_no_database.py --logging INFO --maxalert 100000 --nthread 8 --group LASAIR-DEV1 --host <IP_ADDRESS> --topic ztf_test
Configuration = {'bootstrap.servers': '<IP_ADDRESS>:9092', 'default.topic.config': {'auto.offset.reset': 'smallest'}, 'group.id': 'LASAIR-DEV1'}
3: finished with 10844 alerts
5: finished with 11271 alerts
2: finished with 11064 alerts
6: finished with 11508 alerts
4: finished with 12097 alerts
0: finished with 11698 alerts
1: finished with 11571 alerts
7: finished with 11868 alerts
Run time 286.738100
$
Total is 91921
matches the ZTF monitor for 2020404

Thanks Roy.


Spike: Impact of duplicate result or order messages

Spike Goal
Ask each of our partner vendors (Epic, Oracle, Natus): What’s the impact of us sending a duplicate result or order message?
Timebox
3 days
Notes
For SFTP Ingest service:
-Chance this could happen if delete of processed results zip from CDPH fails (from their server)
-Or if our internal queue process fails in two ways.

First pass conversation with Manjula took place at Monday's (08/19/24) office hours.

Added to the CA SME call agenda for 08/21/2024.
Added to the AL SME call agenda for 08/22/2024.

Hudda will follow up with Bill on Oracle impact.  Neelima will follow up with Natus.

Hope to have answers from "Bill on Oracle impact. Neelima will follow up with Natus." during this week's SME call.

Per Bill and Neelima, this is a non-issue.  If duplicates (Orders or Results) are sent,  the original message will be replaced.  Per Bill multiple ACKS will be grouped together in the same origin message.

@JohnNKing let's discuss a low priority follow up story here?


context.createRequest didn't set "GetBody" properly.

The context package wrap the http.Request.Body with a internal type utils.nopCloser in createRequest and replace body with a new type context.contextReadCloser when we try to update values in context.
Unfortunately,  the GetBody method for http.Request didn't been set.
The std lib net/http set GetBody for us when we use http.NewRequest and this solution covered most cases. L788-L830 in Go1.8
	if body != nil {
		switch v := body.(type) {
		case *bytes.Buffer:
			req.ContentLength = int64(v.Len())
			buf := v.Bytes()
			req.GetBody = func() (io.ReadCloser, error) {
				r := bytes.NewReader(buf)
				return ioutil.NopCloser(r), nil
			}
		case *bytes.Reader:
			req.ContentLength = int64(v.Len())
			snapshot := *v
			req.GetBody = func() (io.ReadCloser, error) {
				r := snapshot
				return ioutil.NopCloser(&r), nil
			}
		case *strings.Reader:
			req.ContentLength = int64(v.Len())
			snapshot := *v
			req.GetBody = func() (io.ReadCloser, error) {
				r := snapshot
				return ioutil.NopCloser(&r), nil
			}
		default:
			// This is where we'd set it to -1 (at least
			// if body != NoBody) to mean unknown, but
			// that broke people during the Go 1.8 testing
			// period. People depend on it being 0 I
			// guess. Maybe retry later. See Issue 18117.
		}
		// For client requests, Request.ContentLength of 0
		// means either actually 0, or unknown. The only way
		// to explicitly say that the ContentLength is zero is
		// to set the Body to nil. But turns out too much code
		// depends on NewRequest returning a non-nil Body,
		// so we use a well-known ReadCloser variable instead
		// and have the http package also treat that sentinel
		// variable to mean explicitly zero.
		if req.GetBody != nil && req.ContentLength == 0 {
			req.Body = NoBody
			req.GetBody = func() (io.ReadCloser, error) { return NoBody, nil }
		}
	}

That's mean, even we sent our request, we still can get the request body if we want.
For gentleman, the request was built when dispatching, we can't get the request body before we sent it compared with http.NewRequest(although this behavior is useless ).
More importantly, we also can't get the request body !
That's mean, it's tricky if some jobs base on the request we sent, like storing the response by request fingerprint.
Here is a example to reproduce the problem.
package main

import (
	"bytes"
	"fmt"
	"io/ioutil"
	"net/http"

	"gopkg.in/h2non/gentleman.v1"
)

func main() {
	body := bytes.NewBufferString("Body Hell!")
	request, _ := http.NewRequest("POST", "http://httpbin.org/post", body)
	res, _ := http.DefaultClient.Do(request)
	// get request body after sent
	if body, _ := ioutil.ReadAll(res.Request.Body); body != nil {
		fmt.Println("the origin Body is empty:", string(body[:]))
	}
	bodyClone, _ := res.Request.GetBody()
	if body, _ := ioutil.ReadAll(bodyClone); body != nil {
		fmt.Println("Body still can be gotten by GetBody:", string(body[:]))
	}
	req := gentleman.NewRequest()
	req.Method("POST")
	req.URL("http://httpbin.org/post")
	req.BodyString("Body Hell!")
	fmt.Println("the req.Context.Request.GetBody is nil because it didn't be set in `createRequest`:", req.Context.Request.GetBody == nil)
	gres, _ := req.Do()
	if body, _ := ioutil.ReadAll(gres.Context.Request.Body); body != nil {
		fmt.Println("the Body is empty and we can'y get cached body by GetBody:", string(body[:]))
	}

Output:
the origin Body is empty:
Body still can be gotten by GetBody: Body Hell!
the req.Context.Request.GetBody is nil because it didn't be set in `createRequest`: true
the Body is empty and we can'y get a cache for GetBody:

#16

This issue is similar to #16

Alright, let's separate concerns here.
Regarding GetBody(), yes, this is indeed an issue and should be fixed. Let me know if you're interested in providing a PR.
Regarding reading the request body, and considering your example code, you're not using gentleman idiomatically. Let me explain details:
gentleman is a plugin-oriented, middleware-driven toolkit where everything is a composable entity that can be plugged into any outgoing HTTP traffic flow in both client or single HTTP request level. The reason of this is because in gentleman everything is defined in a lazy way based on some sort of observables hooks, so where when you call,  for instance, BodyString() method, you're not technically binding the body yet into the HTTP request object and that's why you can't introspect it.
In order to achieve that, you should use the gentleman approach and attach a custom middleware function or plugin that can observe the HTTP traffic. You can achieve that by observing both outgoing and incoming HTTP traffic flows. See supported middleware/plugin phases here.
You can also see some plugins for usage reference.


updating vote app plan to include lacework pods and traffic driving

This PR adds to the vote plan traffic and lacework pods.

@ipcrm I add the tags as suggested (good catch). I also updated the filename and project name to call out that his VoteApp is for EKS.


[3.0] No way to define tags at the top level OpenAPI object through annotations

There seems to be no way to define tags in v3 through annotations like in v2
@Tag(name ="tag", description="some tag")

@arthurdm I guess this is related also to #2317, please see my comment there

Closed as dup of #2317


Subcategory's subcategory's thread

I have a slight problem, what if I want to show the lastest active thread on the index fom a subcategory's subcategory?
(This means the nested level is 2)
It would show when im "in" the subcategory but not on the front page.

In thread.blade.php, something like:
@foreach ($category->subcategories as $subcategory)
    @foreach ($subcategory->subcategories as $subsubcategory)
        @foreach ($subsubcategory->threads as $thread)
            ...
        @endforeach
    @endforeach
@endforeach

...should work.

This is my current index.blade.php
https://gist.github.com/Teranode/b4f8c864d38b8b58cbae
how would i show it in the front?

Sorry, I meant to say index.blade.php
As long as you nest the loops as in my example then it should work.

i still dont get it

Well, you don't need to do it EXACTLY as in my example, just make sure you have a @foreach ($subcategory->subcategories as $subsubcategory) loop somewhere inside your @foreach ($category->subcategories as $subcategory) one, and then somewhere in your @foreach ($subcategory->subcategories as $subsubcategory) loop you can add a @foreach ($subsubcategory->threads as $thread) one.
I'm not going to edit your index view for you because I have no idea exactly where you want to show the threads, or how you want to format them.


docs: changed to use cache of most recent images

Since this is a minor documentation fix, we determined that an issue was unnecessary.

Current (cache is 404)

Changed

Thank you for the PR. It is an auto generated section and now cache is invalidated.


Force gifs list be final before view() method runs in FeedComponent

And discover a better way to unregister views from fluxxan.


[Feature Request] Support for Task List Items

Issue description
Support for Github task lists (aka check lists) as described at:
https://github.github.com/gfm/#task-list-items-extension-
Markdown to reproduce
- [x] foo
  - [ ] bar
  - [x] baz
- [ ] bim

Screenshot

<input checked="" disabled="" type="checkbox"> foo

<input disabled="" type="checkbox"> bar

<input checked="" disabled="" type="checkbox"> baz

<input disabled="" type="checkbox"> bim

Environment

Notepad++ X.Y.Z (x86/x64)
MarkdownViewer++ X.Y.Z
Windows 7/8/10 (x86/x64)

This feature would be really helpful in instructional documents like setup instructions.

+1 from my side. This would make this plugin complete for me...

It would be great if markdig would use a different class for checked list items. That way this issue could be mitigated by using some css that would e.g. strike through the list item or display a background that "shows" the checkbox. The current viewer result does not represent the intent of the markup.

What about https://github.com/LayoutFarm/HtmlRenderer ?

Hey @asherber
Thanks for pointing that repo out. Didn't now that fork.
Will have a look.
Thanks


[chromatic] auto accept main

### Tasks
- [ ] Auto accept snapshots of main branch

Since all snapshots of changes should be approved, everything on main is "the truth" so that snapshot be automatically accepted.
All snapshots of new pull requests are compared against main, so when the approved snapshot of main is that of a few versions ago you could receive false positives on your PR.

Story on hold until we have new allowance


NHSE x Microsoft Hack for Health

Content Suggestion:
The NHS England Data Science team, as well as a range of other analysts from across the organisation, attended an AI Hackathon at Microsoft, organised by the Data Science Team together with Microsoft and Kainos, with the key stakeholders being the NHS Websites Services Team. In this article, the author shares her experiences at the event.
Last week, the long awaited Hack for Health hosted at the beautiful Story Club, in Paddington London, by Microsoft and NHSE finally happened! At the hackathon, every team had either a Kainos or Microsoft representative, as well as a range of participants from across NHS England, including a strong Data Science Team presence, with several of our team members in each group. And unlike a usual hackathon, this one had stakeholders and well defined use cases, we were all working to one common goal.
Continue reading...

thank you!! Now added into december's edition


Add stable and pre-release build yml

@brettcannon I added test. I don't think we should wait for a build issue to add these tests. Since there some weird restrictions to the logic.


Adding Time Remaining Feature

It could be useful to show "Time Remaining" when the extension is clicked or at least have the option for this.

When the new version is accepted you will be able to see the remaining time when the extension icon is clicked.

Thanks! Love the extension btw, glad to see you're still updating it.


Migrate samples for Quota and RedisEnterprise

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
API changes are not detected in this pull request.


Bug: The system/View/Cell.php on return $instance->{$method}();

PHP Version
8.1
CodeIgniter4 Version
4.4.3
CodeIgniter4 Installation Method
Manual (zip or tar.gz)
Which operating systems have you tested for this bug?
macOS
Which server did you use?
apache
Database
MySQL
What happened?
File: system/View/Cell.php
the return method under or line 231.
return $instance->{$method}();
well this result in an error saying to few argument sent or not. but I have fixed it by writing it like.
return $instance->{$method}($params);
was this thing a bug or my mistake to correct it?
now seems working fine...
Steps to Reproduce
Download a fresh ci4 version 4.4.3,
create a cell (u can use the php spark make cell) like Cells/ClassCell
inside create method called ClassCell::MakeMethod(array $parma):string,
create a normal view file called normalView which is loaded from a controller as usual,
call the cell which load it's own from the view as
view_cell(ClassCell::MakeMethod, [whatever array or obj here])
this produce an error like too few argument or so...
Expected Output
I just wanted the CI4 to work without my correction... if that is bug, then good if not then correct me...
Anything else?
No response

Thank you for reporting.
It sounds like a bug, but Controlled Cell doesn't call the method.
<?= view_cell('AlertMessageCell', 'type=warning, message=Failed.') ?>

See https://codeigniter4.github.io/CodeIgniter4/outgoing/view_cells.html#creating-a-controlled-cell
@lonnieezell What do you think?

This is not a bug. It's just an OP's misconception of how Controlled Cells work.
Simple Cells !== Controlled Cells. The user guide is very clear here.
In Controlled Cells, parameters are always mapped to public variables of the class, which should be declared inside the class.

How many dynamic vars are we going to have/declare inside the class?.
it should be a bug michalsn...

How many dynamic vars are we going to have/declare inside the class?.

You need to declare public properties of the class - as many as needed.

it should be a bug michalsn...

No, it's not a bug. This is how Controlled Cells were designed and work. If you need a described behavior just use Simple Cells: https://codeigniter4.github.io/CodeIgniter4/outgoing/view_cells.html#simple-cells

Exactly that is the problem, I reported...
https://codeigniter4.github.io/CodeIgniter4/outgoing/view_cells.html#simple-cells
Unless it if fixed the Error is always thrown!

The documentation seems to be a bit difficult to understand the difference between Simple and Controlled Cells.
I sent a PR to improve: #8132

@MohammedNuru It seems you don't know the difference between  Simple Cells and Control Cells.
All you need to do is just to remove  extends Cell in the Cell class.

This is not a bug, but a misuse.


[7251] - Add POST /trainees/{trainee_id}/defer

Context
7251-endpoint-post-trainees-traineeid-defer
Changes proposed in this pull request

Add endpoint to defer trainees
Update docs for v0.1, v1.0

Guidance to review
Important business

[ ] Does this PR introduce any PII fields that need to be overwritten or deleted in db/scripts/sanitise.sql?
[ ] Does this PR change the database schema? If so, have you updated the config/analytics.yml file and considered whether you need to send 'import_entity' events?
[ ] Do we need to send any updates to DQT as part of the work in this PR?
[ ] Does this PR need an ADR?

NB: Please notify the #twd_data_insights team and ask for a review if new fields are being added to analytics.yml

I think the defer_date should be a required field, unless this has been confirmed by someone else as optional.

Yeah that was one thing I was unsure about. Should the rest of the validations be triggered always? I followed deferral_form.rb#L4

I think the defer_date should be a required field, unless this has been confirmed by someone else as optional.

Yeah that was one thing I was unsure about. Should the rest of the validations be triggered always? I followed deferral_form.rb#L4

@Reyko I think we can keep it simple here - check that defer_date is present, and is a date.

I think the defer_date should be a required field, unless this has been confirmed by someone else as optional.

Yeah that was one thing I was unsure about. Should the rest of the validations be triggered always? I followed deferral_form.rb#L4

@Reyko I think we can keep it simple here - check that defer_date is present, and is a date.

Refactored https://github.com/DFE-Digital/register-trainee-teachers/pull/4395/commits/b1ab53374ea23af3a1ece2125756499b1018866f


User data timing issues

Describe the issue
We need a better understanding of how long various parts of a user data collection are taking.  We've updated our logger to allow measurements.  This was incorporated on that last run, and we have statistics for every senate user
Preliminary results seem to indicate slow-downs on the server, maybe on the CDL fetches as well, though that's not super clear yet.

I updated the 17 `experts`` that took the longest to replicate in the current setup.
top='ehenrics fzhammoc dpland phymoore ekebreab hjcheng ffing ckedding ekneuman epontike eprebys gpfox jawdat clarson daccache fzebeler fanchert'

#save previous run
mkdir save
for i in $top; do mv $i* save; done

for i in $top; do echo $i;  experts cdl --debug-save-xml $i > $i.log.json; done


Model is not valid when has integer value as string

Documentation

[X] I have consulted the documentation, and my question isn't answered there.

Nuget Package
JsonSchema.Net.Generation
Package Version
5.4.3
How can I help?  Please provide as much context as possible.
How to achive model is valid when
public class NumberSchemaGenerator : ISchemaGenerator
{
    public bool Handles(Type type) => type.IsNumber();
    public void AddConstraints(SchemaGenerationContextBase context)
    {
        context.Intents.Add(new TypeIntent(SchemaValueType.String));
        context.Intents.Add(new TypeIntent(SchemaValueType.Number));
        context.Intents.Add(new TypeIntent(SchemaValueType.Integer));
    }
}

class MyClass
{
    public int id{ get; set; }
}

[Test]
public void JsonSchema_Test()
{
    var config = new SchemaGeneratorConfiguration();
    config.Generators.Add(new NumberSchemaGenerator());
    var schema = new JsonSchemaBuilder().FromType<MyClass>(config).Build();
    var jsonString = "{\"id\":\"123\"}";

    using var jDoc = JsonDocument.Parse(jsonString);
    var result = schema.Evaluate(jDoc);
    result.HasErrors.ShouldBeTrue();
}

We have specific case: all integer and long values are retrieved as string.
We get model is not valid when try to evaluate this model.
Thanks.
Code of Conduct

[X] I agree to follow this project's Code of Conduct

SchemaValueType is a flags enum.  You need to combine them in a single intent rather than adding the intent multiple times.
context.Intents.Add(new TypeIntent(SchemaValueType.String | SchemaValueType.Number));

Think of intents as keywords: they can only be added once.


Release new version of fil_actor_interface

Issue summary
The fil_actor_interface has changed significantly. Let's release a new major version.
Other information and links

Done in https://github.com/ChainSafe/fil-actor-states/commit/87ca7aad025d217b3bf9e1d32a537665a71640c8, apparently.


is it possible to insert new data in a specific row?

I want to insert new data in a specific row
Something like this:
db({id:key}).get()[0].insert(newInfo)
or
db({id:key}).insert(newInfo)

You can always add an 'order' field, and use it for sorting when you want data in order.
You might start with assigning 1, 2, 3, 4... and then realize that if you want to do an insert somewhere, you have to renumber everything above where you're inserting.
JavaScript numbers have enough precision, however, that you can step by something large (say, 10,000) in assigning ordering, and then just pick a number halfway between the two you want to insert between, so for example you'd insert order 25,000 between 20,000 and 30,000, and later you could insert 22,500 between 20,000 and 25,000, and so on.
Of course, you'll have to watch out for the (hopefully unusual!) case of having every integer between 'n' and 'm' assigned, and then recognize that and perhaps renumber everything.  But it'd be a rare case.


Win 8.1 listener state not listening [not supported]

After the most recent updates:

I'm seeing not supported:

Please help...
Thank you!

You can run update.bat to solve the issue. ( you should be connected to the Internet.)

I tried that before but just did it again.

[] Checking for updates...
[] Current update date: 2018.10.10
[] Latest update date:  2018.10.10
[] Everything is up to date.

It still doesn't work.

The easiest thing to do is this:

Download this ini file
Copy it over the top of "C:\Program Files\RDP Wrapper\rdpwrap.ini"
Find your copy of RDPWInst.exe and run it via cmd.exe as follows:
RDPWInst -r

If you are remotely connected, then It will disconnect your session, so connect again and you should be right.
Find your copy of RDPConf.exe and run it, it should show as all good; if it's running already, close it first as it won't have up to date status.
DO NOT RUN update.bat as it will only be able to get the ini file from the project and that one isn't fully up to date (it was last updated 16 months ago!)

Thank you!
Although things now look good here:

It doesn't actually seem to work:

gpedit.msc -- see section 8) at this link

Not quite... :(
In GP Editor, I don't have the Policies node:
Computer Configuration\Policies\Administrative Templates\Windows Components\Remote Desktop Services\Remote Desktop Session Host\Licensing
but I went to this:
Computer Configuration\Administrative Templates\Windows Components\Remote Desktop Services\Remote Desktop Session Host\Licensing
I did go ahead and change the "Set the Remote Desktop licensing mode" to "Per Device"
And I also set "Limit number of connections" to "999999" and I set the "License servers to use" as the IP address as shown in ipconfig under "IPv4 Address". (I don't have Server Manager being that I'm running 8.1 Pro, not a server version of Windows. (Is that the problem?...)
Similarly, in Reg Edit, I don't exactly have this:
HKEY_LOCAL_SYSTEM\SYSTEM\CurrentControlSet\Control\Terminal Server\RCM\GracePeriod
This is what I see:

So this is the closest I can get:
HKEY_LOCAL_MACHINE\SYSTEM\CurrentControlSet\Control\Terminal Server\RCM
(no HKEY_LOCAL_SYSTEM and no GracePeriod nodes)

Now what? :((((

Did you restart RDP with RDPWInst -r ?
And if that isn't enough, restart the machine to make sure the GPEDIT changes are active....

Sorry, I should have found a simpler reference without GPO, but at least you got the idea.

I tried both RDPWInst -r and restarting the computer but neither worked. I still get this:

When I run RDPWInst -r I see this:

This is a big problem for me... Please help... Thank you!

Should I undo the changes I made above to the Registry and GP?
What else might work? This didn't affect anyone else in this same way?

If the ini file I referenced above was used, then it has the empty line at the end.
Anybody else got any ideas?

Affinityv is completely correct:

If the ini file I referenced above was used, then it has the empty line at the end.
Anybody else got any ideas?

Just to start fresh, I rolled-back to a (Windows not data disk) backup from Feb. 7. I updated the RDPWrapper including using the INI file from @affinityv.
When I ran RDPWInst -r, I saw this error:

Although I now see this:

it still only lets me connect one user at a time...

Any suggestions please?...

So if RDPWrapper looks like it's configured correctly, what other (native Windows?) settings are there I should check? I double checked and see that RDP connections are enabled:

In Windows Firewall:

What else might be amiss?

Are there other things I need to set-up/verify as "prerequisites" for RDPWrapper to work?

This is a serious problem for how we use this computer.
I'm guessing this might not be directly a problem with the configuration of RDPWrapper... Can anyone please advise if there is anything I need to set-up/verify as "prerequisites" for RDPWrapper to work?

If the ini file I referenced above was used, then it has the empty line at the end.
Anybody else got any ideas?

so open it up and show us.

I experienced the very same problem with Windows 8.1. The machine is listening, but denies more than one RDP-connection. I suspect, Microsoft has just introduced a new method into termsrv.dll to thwart patching.

If the ini file I referenced above was used, then it has the empty line at the end.
Anybody else got any ideas?

so open it up and show us.

Wow. As silly as it sounds, what @bjgo wrote was the key. There was some corruption of the INI file. It works now.
Thank you all for your help!!!

@EliezerBee What and where was the corruption?  Was it in the file you download from my github repo area?

OK, this is a bit embarrassing...
I first downloaded the INI file from:
https://github.com/affinityv/INI-RDPWRAP/blob/master/rdpwrap.ini
I then opened it, and saw it looked right. But something still wasn't working. So at a later point, I once again downloaded the INI file, but the second time, I apparently clicked "Save link as" which then saved it as an HTML document. And the DLL didn't exactly know what to make of a bunch of HTML tags...

Okay, thanks @EliezerBee that explains it.  I'll keep adding INI sections when I learn about them; just added one now.

it doesn't seem to work anymore again :(

it doesn't seem to work anymore again :(

Okay, was it working for you recently?  What changed on your system that you know about?
What is the exact version of your termsrv.dll --- both release and whether it is 32 or 64 bit please.  And out of interest alone... just for curiosity, what is the version that WINVER.EXE provides?

I'm also having this problem :(((
From WinVer.exe:
Windows 8.1 Pro v6.3 (build 9600)
[image: image.png]
Please, please help...
On Tue, Nov 9, 2021 at 12:17 PM affinityv @.***> wrote:

it doesn't seem to work anymore again :(
Okay, was it working for you recently? What changed on your system that
you know about?
What is the exact version of your termsrv.dll --- both release and whether
it is 32 or 64 bit please. And out of interest alone... just for curiosity,
what is the version that WINVER.EXE provides?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/stascorp/rdpwrap/issues/998#issuecomment-964411431,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABKR7RTVUQC6XZPUNAKQV4LULFQUDANCNFSM4KV7UGWA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

I'm also having this problem :(((
From WinVer.exe:
Windows 8.1 Pro v6.3 (build 9600)

Please, please help...

Windows 8.1 Pro v6.3 (build 9600)

32 bit or 64 bit Windows?
And you now have the INI from the raw link?

64 bit
Should I download that INI from the raw link?
On Sun, Nov 14, 2021 at 2:07 PM affinityv @.***> wrote:

Windows 8.1 Pro v6.3 (build 9600)
32 bit or 64 bit Windows?
And you now have the INI from the raw link
https://raw.githubusercontent.com/affinityv/INI-RDPWRAP/master/rdpwrap.ini
?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/stascorp/rdpwrap/issues/998#issuecomment-968355065,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABKR7RX7ISZNTCW3BJVNA33UMAJJ5ANCNFSM4KV7UGWA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Should I download that INI from the raw link?

Yes, if you want to be sure it is complete and in the correct format.  Save as plain text file.

If that still doesn't work, I would be looking at your AV / Malware products, make sure you have exceptions.  Also make sure that the Windows registry is in tact for RDP.  This reg file in my repo might help you work that out.

And if all that fails, there might be a difference b/w the termsrv.dll file of Windows 8.1 and Windows Server 2012 R2 ... not sure about that; I expect it should be the same though.

@affinityv, thank you for the "raw link" advice. Now I'm here:

So there was an improvement. But when a second user tries to connect, they're told the other user has to give up their session first.
What's the next logical step?

What's the next logical step?

As an admin user RDPWInst.exe -r

Perfect!!! THANK YOU!
On Sun, Nov 14, 2021 at 2:44 PM affinityv @.***> wrote:

What's the next logical step?
As an admin user RDPWInst.exe -r
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/stascorp/rdpwrap/issues/998#issuecomment-968360218,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABKR7RXNJWWWWLSP5O434EDUMANSVANCNFSM4KV7UGWA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Perfect!!! THANK YOU!

So... all good now, multi users?

YES YES YES
On Sun, Nov 14, 2021 at 2:48 PM affinityv @.***> wrote:

Perfect!!! THANK YOU!
So... all good now, multi users?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/stascorp/rdpwrap/issues/998#issuecomment-968360751,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABKR7RRZTYSMIUTVERYZHVLUMAOCTANCNFSM4KV7UGWA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

What's the next logical step?

As an admin user RDPWInst.exe -r

I appreciate that. I had the same issue, and I could fix it with your advice. Thanks a million.


Use of vmaf & compressibility check

Hello,
Have you considered using VMAF and/or compressibility check in your framework?
The workflow that I have in mind is this one:

Choose a vmaf quality that is acceptable for you. 80%? 60%?

encode a small part of the original video (5% of the video in 30 seconds clips, spread equally to all video) in crf x

calculate vmaf of that clip, if the vmaf is high, then increase crf in order to decrease bitrate and go to step 1. If the vmaf is low, then decrease crf in order to increase bitrate and go to step 1. If the vmaf quality is ok, then you have your crf value for that video that wastes no bitrate.

Encode the whole video and inspect or vmaf it.

@dim-geo Sorry I took so long to reply. I've been traveling today.
Besides the fact that including a C library-based VMAF implementation in a Ruby-only Gem would not be practical, I don't have a lot of confidence in assessing video quality via some kind of objective algorithm. Especially for my target audience which are home transcoders.
Using VMAF, peak signal-to-noise ratios or a structural similarity indexes just take too much computation time and resources compared to judicious manual visual inspection.
I discuss this at length in the FAQ here:
https://github.com/donmelton/video_transcoding#how-do-you-assess-video-transcoding-quality
Of course, that doesn't mean I would prevent users from leveraging VMAF or other algorithms after transcoding. I just don't want to build those systems into my lightweight tools.
Does that make sense?

VMAF is a different kind of beast compared to PSNR and other 'objective' measurements.
VMAF is a perceptual video quality assessment algorithm.
The fact that is used & backed up by netflix, gives it more credit.
My main objection with it, is that it takes time to compute but it takes probably less personal time compared to rewatching a movie.
Another totally unrelated question and a suggestion:
1)How do you encode/handle grainy sources?
2)It would be beneficial I think to document all arguments/options of the tool in github page. Apologies if I din't find the page containing them.

@dim-geo As far as grainy sources go, I don't use any special filters or other handling for those. Typically you want to retain the grain so I rely on the encoder's default behavior.
Thanks for the suggestion about documentation. The problem though is that the README file is already too long. I think the right way to handle that is via a Wiki, and I do plan to create one in the future.


"Missing helper: "md"" after update to v0.6.0

Used the latest version of assemble and grunt-assemble for my private prohject, but after updating grunt-assemble from v0.5.0 to v0.6.0 I get this error when running it:
Running "assemble:pages" (assemble) task
Assembling tmp/travel_01_aus.html OK
Assembling tmp/blog.html ERROR
Warning: Missing helper: "md" Use --force to continue.
Aborted due to warnings.
What has changed that needs to be accomodated on my side for my project to run again?
Using node v7.7.2, assemble 0.23.0

assemble: {
pages: {
options: {
flatten: true,
layout: '<%= dir.assemble %>/layouts/default.hbs',
data: '<%= dir.assemble %>/data/.{json,yml}',
partials: '<%= dir.assemble %>/partials/.hbs'
},
files: {
'<%= dir.tmp %>/': ['<%= dir.assemble %>/pages/*.hbs']
}
}
},

Hi @rejas I anticipated there being issues and this is one of them. Check out the migration section in the readme that addresses missing "md" and "markdown" helpers.
There's a code snippet that you can put in <%= dir.assemble %>/helpers/markdown.js, then update the assemble config in your Gruntfile.js to register the helpers:
assemble: {
  pages: {
    options: {
      flatten: true,
      layout: '<%= dir.assemble %>/layouts/default.hbs',
      data: '<%= dir.assemble %>/data/*.{json,yml}',
      partials: '<%= dir.assemble %>/partials/*.hbs',
      helpers: '<%= dir.assemble %>/helpers/*.js'
    },
    files: {
      '<%= dir.tmp %>/': ['<%= dir.assemble %>/pages/*.hbs']
    }
  }
},

Thx, that worked flawlessly. Sorry for not looking at the migration guide first :-(

No problem. If you run into any other issues with migrating, please open an issue so we can fix it, find a workaround, or update the migration section.


Interactive Shapes

Is there a plan to add interactivity to shapes, more specifically, allowing users to move and transform shapes similar to what fabric.js does? If so, how would this be ideally implemented?
I'd be interested in contributing the functionality. Right now I am playing around with adding it through an InteractionMixin mixed in with RenderableMixin and enabling with a interactive property.

Hi @tarrencev
I want to add click event of particular shape of group in react native. I trying with difference option but i can not find any useful for that. can you please help for this issue.
sectionAngles.map(section => (
                <Group>
                  <Shape
                    key={section.index}
                    d={path(section)}
                    fill={section.data.color}
                    strokeWidth={1} />
              {/* <TouchableOpacity onPress = {() => console.warn("Hi...")}> */}
                  <ART.Text font={{ fontFamily: 'Helvetica Neue', fontSize: 20, }}
                    fill="#000"
                    x={path.centroid(section)[0]}
                    y={path.centroid(section)[1]}
                    ellipsizeMode={"middle"}
                    alignment="center"
                  >{section.data.itemName}</ART.Text>
                  {/* </TouchableOpacity> */}
                 </Group>

              ))


[SPARK-32608][SQL][3.0][FOLLOW-UP][test-hadoop2.7][test-hive1.2] Script Transform ROW FORMAT DELIMIT value should format value

What changes were proposed in this pull request?
As mentioned in https://github.com/apache/spark/pull/29428#issuecomment-678735163 by @viirya ,
fix bug in UT, since in script transformation no-serde mode, output of decimal is same in both hive-1.2/hive-2.3
Why are the changes needed?
FIX UT
Does this PR introduce any user-facing change?
NO
How was this patch tested?
EXISTED UT

Thank you, @AngersZhuuuu and all.


Add: Momo Shopping 2

Channel Name
Momo Shopping 2
Alternative Names (optional)
No response
Network (optional)
No response
Owners (optional)
No response
Country
TW
Subdivision (optional)
No response
City (optional)
New Taipei City
Broadcast Area
c/TW
Languages
zho
Categories (optional)
shop
NSFW
FALSE
Launched (optional)
No response
Closed (optional)
No response
Replaced By (optional)
No response
Website (optional)
https://www.momoshop.com.tw/main/Main.jsp
Logo
https://www.lyngsat.com/logo/tv/mm/momo_shopping_tw.png
Notes
No response

Please update your request:

link to the official website is incorrect

https://github.com/iptv-org/database/blob/master/CONTRIBUTING.md#channels

Fixed


how to inject ngAnimate??

amazing work!
i just don't know how to inject any third-part modules in AppGyer app
i tried in index.js:
angular.module('common', [
  // Declare here all AngularJS dependencies that are shared by all modules.
  'supersonic',
  'ngAnimate'
]);

tried in
angular
  // .module('example', ['angular-velocity'])
  .module('example', ['ngAnimate'])

none of those works , it drives me crazy ....

Did you run bower install ngAnimate and verify that you have the script included in your layout file? Check out the /dist folder and make sure the script exists.
<script src="/components/angular/angular.js"></script>
<script src="/components/ngAnimate/js/angular-animate.min.js"></script>

And in the controller:
angular
    .module('example', ['ngAnimate'])
    .controller('LearnMoreController', function($scope, supersonic) {
        // do things here
    });

You could also try this instead as you had it:
angular.module('common', [
    // Declare here all AngularJS dependencies that are shared by all modules.
    'supersonic',
    'ngAnimate'
]);

And then in your controller:
angular
    .module('example', [])
    .controller('LearnMoreController', function($scope, supersonic) {
        // do things here
    });

Notice the `.module('example', []) with empty array.

I'm closing this since this is an AngularJS-specific question and doesn't have to do with issues/features of Steroids/Supersonic per se. Please continue the discussion on https://forums.appgyver.com, if @justincolangelo's excellent answer didn't solve it already. :)

@justincolangelo  @harsaharsa
I tried those answer ,  none of those works
here and here is the source code
in my case i can't add any 3-part injection ...


Implement AWS Fargate support [Large build support]

This PR implements support for AWS Fargate with Git LFS support.
What it does?
With this feature you can run builds on the AWS Cloud on demand. Most importantly you can run builds for very large projects.
Why we need it?
GitHub Actions run on build machines provided by GitHub. You can see info about the specifications of the build machines here:
https://docs.github.com/en/actions/using-github-hosted-runners/about-github-hosted-runners
For game development and Unity projects the disc size in particular is quite small. You may well experience an error related to running out of disc space. You may also want to run the build on a server with more memory or CPU cores.
You can use this feature to send a build from any supported git platform to any of the supported remote build cluster types, now including AWS. This is an inexpensive and reliable way to scale to larger builds. There is virtually no limit to the disc size for the supported remote build cluster types.
To Do

docs (in progress)
steam deploy (estimating start Mon: 12th April 2021)

Planned for later

Align parameter features and code (local, k8s and aws all work a bit differently now)
AWS builds for Git using direct webhook (build any git based unity project)
Offer cache token parameter rather than assuming by branch/tag
Automated way to cache all docker images to AWS container registry (ECR)
Improved test coverage
Asynchronous build support
Unity test framework support
Make fargate task running more reusable as per our needs as a library or tool

Concept
You can run Game CI Unity builds on the supported platforms. You can also run the build on a remote build cluster such as Kubernetes or a Cloud Provider like AWS. This is a bit like self-hosting build servers except on demand.
The main reasons for doing this are reducing build machine costs, running builds too large for other options, nearly unlimited parallel builds and workflow improvements from further isolating the builds.
There can also be significant performance and cost improvements if you run several builds per commit (or any other event like a PR) because you never have to worry about blocking other builds with this approach.
Advantages

Configure CPU, memory/RAM and disc space as needed
Pay for usage
Scale up to as many builds as you want
No servers to manage or worry about

Disadvantage

cheap but costs aws fees
requires an AWS account
additional startup time to begin the build

Required Parameters
awsStackName : The base stack setup in the getting started section.
Container
remoteBuildMemory
remoteBuildCPU
Disc size is completely elastic there is no need to specify.
Build artifacts
Completed builds are zipped then uploaded to aws S3. Steam deployment is still work-in-progress.
Logs
Logs are streamed via Kinesis in buffers back to the GitHub runner to allow fast and stable log streaming, with no aws polling issues.
We also write the AWS Cloud Watch dashboard url to the GitHub output.
Cleaning up resources
Even in the event of an unexpected error we have a failsafe to make sure resources don't accidently remain active and charging you. This is accomplished with a lambda function that runs after a delay if the resources are still active.

Codecov Report

Merging #246 (9fb2cd1) into main (398eda6) will decrease coverage by 12.63%.
The diff coverage is 13.88%.

@@             Coverage Diff             @@
##             main     #246       +/-   ##
===========================================
- Coverage   73.16%   60.52%   -12.64%
===========================================
  Files          19       20        +1
  Lines         544      684      +140
  Branches       97      134       +37
===========================================
+ Hits          398      414       +16
- Misses        145      269      +124
  Partials        1        1

Impacted Files
Coverage Δ

src/model/build-parameters.ts
100.00% <ø> (ø)

src/model/aws.ts
7.46% <7.46%> (ø)

src/model/index.ts
100.00% <100.00%> (ø)

src/model/input.ts
100.00% <100.00%> (ø)

LGTM
Please make sure package-lock.json is removed in favour of yarn.lock by installing deps using yarn instead of npm.

Codecov Report

Merging #246 (600906e) into main (398eda6) will decrease coverage by 12.63%.
The diff coverage is 13.88%.

@@             Coverage Diff             @@
##             main     #246       +/-   ##
===========================================
- Coverage   73.16%   60.52%   -12.64%
===========================================
  Files          19       20        +1
  Lines         544      684      +140
  Branches       97      134       +37
===========================================
+ Hits          398      414       +16
- Misses        145      269      +124
  Partials        1        1

Impacted Files
Coverage Δ

src/model/build-parameters.ts
100.00% <ø> (ø)

src/model/aws.ts
7.46% <7.46%> (ø)

src/model/index.ts
100.00% <100.00%> (ø)

src/model/input.ts
100.00% <100.00%> (ø)


Add more detailed cluster compliance information

We want to add this recording rule so we can have more detailed information about cluster compliancy.

This would include the noisier pod-level checks as well, right? If so, is it possible to avoid sending those to grafana cloud?


NGP to AY8910 noise frequency conversion error

The NGP noise frequency seems to be a little different from the normal SN76489.

The corresponding AY8910 noise value should be 31, 15, 7, 0. But currently it is always 0.

Does this issue only occur on NGP conversion? Is it okay for other SN76489 vgms?

only occur on NGP conversion

NGP mode is realized with VGM's dual-chip support. Since vgm-conv does not support dual-chip, the conversion result only based only on the primary chip. I guess your VGM specifies the noise frequency in the secondary chip stream.


Add ip subnet info

This PR is for:

Server

Fixes ip subnet info bug
Steps to reproduce the bug
Changes to fix the bug
Affected branches

main

Checklist

[ ] Added unit test to verify the fix.

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


label information at ControlConstruct to be harmonized with Holder

If it would be decided not to add an instance to the name attribute at the Holder class for documenting the external label in Equipment Issue#15, it would make sense documenting the external label at the ControlConstruct and the LogicalTerminationPoint also as a separate attribute.
In this case...

§27, §28 of the TransmitterEquipment 1.0.0-tsp.200715.1225 would have to be deleted
ControlConstruct_Pac :: externalLabel : String [1] = "External label not yet defined." has to be added to the EquipmentAugment
LtpAugmentConfiguration :: externalLabel : String [1] = "External label not yet defined." has to be added to the LtpAugment (LtpAugment Issue#6)

Proposal to the 5G-xhaul call on 25th of May 2022:
In contrary to former considerations, there shall be no additional instance in the ControlConstruct::name list-attribute, but an explicit externalLabel attribute.
The following changes shall be made to the EquipmentAugment modelling:

A ControlConstructSpec class shall be added.
An abstraction of type Specify shall be added and connect the new ControlConstructSpec class with the existing ControlConstruct and contain the target "/CoreModel:ControlConstruct:ControlConstruct".
A ControlConstruct_Pac class shall be added.
A composite association [1/1] shall connect from ControlConstructSpec to ControlConstruct_Pac.
A ControlConstruct_Pac :: externalLabel : String = "External label not yet defined." attribute with comment "The externalLabel attribute is for the operator to persistently store an identifier from his documentation inside the device." shall be added.

Decision on the 5G-xhaul call on 25th of May 2022: Approved as proposed!

Has been updated and published in EquipmentAugment_1.0.0-tsi.240104.1330+yang.1


skull duplication

I am currently running

SpongeForge version: 1.12.2-2825-7.1.6-RC3705

Plugins/Mods:
Prism, GriefPrevention

Issue Description
https://github.com/prism/Prism/issues/132#issuecomment-491498302

Just tested this with the following
Mods: none
Forge: 1.12.2-<IP_ADDRESS>25
Sponge: 1.12.2-2825-7.1.6-RC3711
Plugins: Nucleus-1.9.3-S7.1-MC1.12.2-plugin
This issue requires a sponge plugin to be added before it occurs.
Simply placing a skull on the ground and breaking it will give you 2 skulls in return.

Sponge is capturing the entity spawn twice, I put a log message in PhaseContext::captureEntity and found that the method got called twice: https://gist.github.com/LXGaming/2b918d80b5c2e03ce080a2010d75f743 (Line 13 & 45)
This recently added Mixin is triggering the 2nd capture:
https://github.com/SpongePowered/SpongeCommon/blob/5f19dd24c9969574b90d36caa1552015a383d94a/src/main/java/org/spongepowered/common/mixin/core/server/management/MixinPlayerInteractionManager.java#L547-L581

I can't reproduce this on SpongeForge 3716 (using -Dsponge.shouldFireAll=true). I think it was fixed by 98f2f9ae2ea1febffd6dfe043a044145721b14ba.

this still happens without shouldFireAll, according to someone on Discord

It seem it's you can reproduce this just by going into survival not creative using 3716 .

Finally able to reproduce this. The problem causing this appears to be that BlockSkull expects the block and tile entity to be removed when harvestBlock() is called. However, Sponge re-adds the tile entity during harvestBlock() to fix https://github.com/SpongePowered/SpongeForge/issues/2709.

Fixed in https://github.com/SpongePowered/SpongeCommon/commit/6b2d3b3e9eeec202ac1bccc5e79cd628644d58ca.


The godot class heirarchy doesn't match how regular javscript classes work.

...which is understandable, I'm amazed all of of this works so well out of the box, fantastic work!
But anyway, the typescript definitions say that Node3D extends Node, and yet:
Node3D.prototype instanceof Node
(Node3D.prototype as any).__proto__ === Node.prototype

are both false, despite these normally being true in javascript:
class A {};
class B extends A {};
B.prototype instanceof A // is true
(B.prototype as any).__proto__ === A.prototype // is true

This means something like making an export function with default export_ arguments using reflect-metadata like:
export function Export<
  Key extends string,
  Type extends Node & {[key in Key]: Resource | Node}
>(): (target: Type, key: Key) => void;
export function Export(...args: Parameters<typeof export_>): (target: Node, key: string) => void;
export function Export(...args: any[]) {
  return (
    target: Node,
    memberName: string
  ) => {
    const type = Reflect.getMetadata("design:type", target, memberName);
    // If args were specified, use those
    if (args[0]) {
      return export_(args[0], args[1])(target, memberName);
    } else {
      // Otherwise automatically determine it
      if (type === Resource || type.prototype instanceof Resource) {
        return export_(
          Variant.Type.TYPE_OBJECT,
          {
            hint: PropertyHint.PROPERTY_HINT_RESOURCE_TYPE,
            hint_string: type.name
          }
        )(target, memberName)
      } else if (type === Node || type.prototype instanceof Node) {
        return export_(
          Variant.Type.TYPE_OBJECT,
          {
            hint: PropertyHint.PROPERTY_HINT_NODE_TYPE,
            hint_string: type.name,
          }
        )(target, memberName)
      }
    }

    // Typescript should ensure we never hit this, but error just in case
    throw new Error(`Failed to Export type ${type.name}`)
  }
}

won't actually work because the type.prototype instanceof Resource and type.prototype instanceof Node will return false for any subclasses of those types, unlike normal javascript class hierarchies.
Also this fake class heirarchy means we can't "polyfill" godot base classes with additional convenience methods for subclasses. A polyfill for Node3D will work on scripts that extend Node3D, but not on scripts that extend a type that extends Node3D, like CharacterBody3D.
i.e.
(Node3D.prototype as any).test = "does this work?"
(CharacterBody3D.prototype as any).test // is undefined

doesn't work, but it does in regular javascript:
class A {};
class B extends A {};
(A.prototype as any).test = "does this work?"
(B.prototype as any).test // is "does this work?"

In addition to generating the types with an inheritance hierarchy, could we also generate the actual classes (wherever that happens) with inheritance to match? Looking through the code here, I would have thought this does exactly that, but maybe I'm looking in the wrong place?
I figured out how to compile with VeryVerbose and run with --verbose and the JSB_LOG there is firing. Also thought this might be a v8 binding issue, so I tried compiling with use_quickjs=yes, but I have the same issue.
I see expose_godot_object_class seems to grab classes from native_classes_ if they're a godot class (at least I think?). Maybe those are different than the v8 classes somehow? I.e. Node3D extends a different Node than the one we can access from javascript? At least that's my hunch, because printing Object.keys((Node.prototype as any).__proto__) prints all the methods of a godot Object, so it's inheriting from something, just seemingly not the Object I can import from "godot".
This is all very new to me, sorry if I'm missing something obvious!

Whoops, I didn't realize when compiling with quickjs I also set dev_build=true, which changed the output binary name, so I was still running the v8 build when I tested this.
Switching to quickjs fixes these issues for me, so this seems to be a v8-specific issue.

Thanks for reporting it with such detailed info. The prototype issue has driven me crazy for a long time 😂, I'll check it out right away.

After dozens of attempts, it was finally fixed (56bb8ac). The problem was caused by incorrectly registering an ObjectTemplate into a FunctionTemplate (a nested enum within a class).

Amazing work, thank you!


Enhance Schema Dumper

Adds retention_policy support to migration_helpers
Adds continuous_aggregate and retention_policy support to schema_dumper
Updates pluralization to more closely match Rails conventions
Adds compatibility with Scenic gem

@jonatas we've started using this gem with an internal proof-of-concept, and we thought our enhancements might be useful to upstream.

Oh man! very happy to receive this PR! Thank you very much @JamesMasonRC!
I'll be reviewing and testing it in the next couple of days.
Can you please add some tests for the scenic views?Just to make it clear how it integrates and we keep it compatible!

@jonatas can do!
How would you feel about me adding in https://github.com/makandra/gemika to run the suite with & without scenic? The alternative would be writing a spec that requires scenic, validates that schema dumping works, then removes the ::Scenic constant. Using gemika feels cleaner, but it's also a bigger change.

@JamesMasonRC I like the idea. I think this is part of the ecosystem and as a library we're probably going to support several pg versions in a near future. So, let's add it, we can add only as a development dependency, so it will not hurt the production library!
And thanks for suggesting such nice improvement 🧠 🙇

@jonatas test suite updates are pushed. This could definitely use a more comprehensive test suite for the schema dumper, but this at least verifies the minimal functionality.

Hi @JamesMasonRC , running locally I got some error in the dependencies of scenic.

Gemfile.scenic

Could not find i18n-1.8.11 in any of the sources
Run `bundle install` to install missing gems.

Summary

- Gemfile         Ruby 2.7.1  Success
- Gemfile.scenic  Ruby 2.7.1  Failed

Some gemfiles failed

Running  bundle install --gemfile Gemfile.scenic makes it pass.
I'm adding this step to the bin/setup.

Great work @JamesMasonRC ! Thank you very much for upstreaming it!
Let me know how is going the usage and open an issue if you think we can improve it somehow!


Check delThis

Closets for example

It is needed to remove the delete this; statement. The functionality should be moved to ObjectFactory


Add Credo

Most of the refactoring was style except for the change on the QueueManager.
When adding a job that failed to the retry set or to the dead set the QueueManager will throw an error and die by using RetrySet.add! and DeadSet.add!
I also added it to run it on Travis
It closes #54

Coverage decreased (-0.2%) to 81.739% when pulling df999f708a498633b209a2eb51fb7cdd513b511a on credo into 59ed33bef0eab1565c96ddeb84018e5d7fbcc84f on master.


SCSS breakpoint mixins can break if a custom breakpoint is added in settings file

What should happen?
By adding a new 'custom' breakpoint in scss settings, you can then use the new breakpoint instead of using the foundation defaults.
So, add the following in settings.scss
$breakpoints: (
  small: 0,
  medium: 640px,
  newbreakpoint: 800px,
  large: 1024px,
  xlarge: 1200px,
  xxlarge: 1440px
);

we've added a breakpoint called 'newbreakpoint', which has a value of 800px, so we add it below medium, so they are in order of value.
If we use the following mixin in scss like so like this
.my-class {
 @include breakpoint(medium down) {
  margin: 16px;
 }
}

then the changes to any element with .my-class should take affect once the browser hits 640px
What happens instead?
What actually happens is the element is changed when the browser hits 800px, rather than 640px.
...
Possible Solution

If you place the custom breakpoint at the end of the list, then the problem goes away.
so you can do the following
$breakpoints: (
  small: 0,
  medium: 640px,
  large: 1024px,
  xlarge: 1200px,
  xxlarge: 1440px,
  newbreakpoint: 800px
);

...
Test Case and/or Steps to Reproduce (for bugs)

Test Case:

Explained above ^
Context

...
Your Environment

Foundation version(s) used:
Browser(s) name and version(s):
Device, Operating System and version:
Link to your project:

Checklist

[ ] I have read and follow the CONTRIBUTING.md document.
[ ] There are no other issues similar to this one.
[ ] The issue title and template are correctly filled.

If you place the custom breakpoint at the end of the list, then the problem goes away.

That is correct and expected as we use nth- helper methods in the scss code to generate the css from the list.
Normally a custom name in the middle makes not much sense as we already have good default names.

https://github.com/foundation/foundation-sites/blob/develop/scss/util/_breakpoint.scss
It is not a bug but it's not documented.
Do you want to provide a PR for the docs?


Checkbox and other components do not render <input> element with form attribute

Bug report
Current Behavior
In https://github.com/radix-ui/primitives/discussions/874, @benoitgrelard says...

That is correct, if it's outside of a form, there isn't any need for any native component anyway because bubbling only makes sense within forms.

That is not correct. See a simplified example of my case:

  <AbstractedCheckbox label="Accept terms and conditions" name="consented" form="terms-form" />
  <form id="terms-form" onSubmit={(event) => console.log(Object.fromEntries(new FormData(event.target)))}/>

// result -> {}
// result when form wraps checkbox -> { consented: "on" }

Forms are not always the parent of form fields. In the above case, Radix doesn't realize it's part of a form, and so doesn't render an <input> element, and the checkbox state doesn't end up in the FormData.
Expected behavior
An <input>/native element is rendered when the form attribute is present.
Suggested solution
This line in Checkbox.tsx (and in Switch and elsewhere)...
const isFormControl = button ? Boolean(button.closest('form')) : true;

should be changed to something like...
const isFormControl = button ? Boolean(button.closest('form')) || !!checkboxProps.form : true;

Or... just always render an <input>?  Not sure why it was decided to avoid it, every other library, headless or not, seems to always render an <input> and just visually and  accessibly hide it as appropriate.
Additional context
This is a perfectly valid feature of the HTML spec...
https://developer.mozilla.org/en-US/docs/Web/HTML/Element/input#form
https://html.spec.whatwg.org/multipage/form-control-infrastructure.html#attr-fae-form
... so I don't like having to justify why I would need it, but here is some context.
I tend to do it this because that way the form does not affect layout or styling. Yes I'm aware of display: contents but that does not prevent it from completely changing the CSS selectors required. As such, I tend to have an abstract form component that gets portaled to the body.
I'm sure there are many other cases why someone might need the input element there.
Workaround
As a very unpleasant workaround, to my <AbstractCheckbox> component I've had to track and sync a local copy of the checked state and add an invisible (visually and accessibly) <input> so the form can pick it up, like this (irrelevant stuff removed):
type Props = {
  label: ReactNode;
  value?: boolean;
  onChange?: (value: boolean) => void;
} & CheckboxProps;

const Checkbox = ({ label, tooltip, value, onChange, ...props }: Props) => {
  const id = useId();

  const [checked, setChecked] = useState(false);
  useEffect(() => {
    if (value !== undefined) setChecked(value);
  }, [value]);

  return (

      <Root
        {...props}
        id={id}
        checked={checked}
        onCheckedChange={(checked) => {
          if (onChange) onChange(!!checked);
          setChecked(!!checked);
        }}
      >
        <Indicator>
          <FaCheck />
        </Indicator>
      </Root>

        {label}

      {/* https://github.com/radix-ui/primitives/issues/2530 */}
      <input
        type="checkbox"
        form={props.form}
        name={props.name}
        style={{ display: "none" }}
        checked={checked}
        readOnly
      />

  );
};

export default Checkbox;

I'm planning to address this shortly. We'll need a more wholistic solution for all affected components, and I want to do some manual testing to ensure this doesn't impact click interactions for elements outside of its associated form.


Windows connection timeout

If you are using a windows system, you may see a connection timeout. This is a problem specific to windows cause by default settings. Please take a look at the following issues to edit the default timeout:

Oracle thread.
Stack exchange thread.
Microsoft thread.

This issue was posted FYI.


Added tests for flag feature

Implemented unit and integration tests for flag command
Updated applicant tests to include flag field

Resovles #145 and #119

Codecov Report

Merging #146 (8506cfd) into master (83768ad) will increase coverage by 2.67%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master     #146      +/-   ##
============================================
+ Coverage     70.67%   73.35%   +2.67%
- Complexity      516      537      +21
============================================
  Files            86       86
  Lines          1579     1580       +1
  Branches        180      180
============================================
+ Hits           1116     1159      +43
+ Misses          411      365      -46
- Partials         52       56       +4

Impacted Files
Coverage Δ

...ain/java/seedu/linkedout/model/applicant/Flag.java
62.50% <100.00%> (+19.64%)
:arrow_up:

.../main/java/seedu/linkedout/model/ModelManager.java
98.64% <0.00%> (+5.40%)
:arrow_up:

...ava/seedu/linkedout/model/applicant/Applicant.java
95.52% <0.00%> (+8.95%)
:arrow_up:

...linkedout/model/applicant/UniqueApplicantList.java
93.47% <0.00%> (+10.86%)
:arrow_up:

src/main/java/seedu/linkedout/model/Linkedout.java
92.59% <0.00%> (+11.11%)
:arrow_up:

...va/seedu/linkedout/logic/commands/FlagCommand.java
76.66% <0.00%> (+76.66%)
: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 83768ad...8506cfd. Read the comment docs.


F# interactive directives like #r or #load or #i docs

currently ionide does not display any help or hint in docs for F# interactive directives, while it does e.g. for #nowarn https://github.com/ionide/ionide-vscode-fsharp/discussions
https://github.com/fsharp/FsAutoComplete/issues/1225

closing as i found the code in fsautocomplete..


[Bug?]:  bug bug

What's not working?
****doscs
docs first issue docs
How do we reproduce the bug?
No response
What's your environment? (If it applies)
No response
Are you interested in working on this?

[ ] I'm interested in working on this

@pullflow-ai yes could you point me to a place in the docs?

@pullflow-ai hello?

@test-et-pullflow-ai
hello? Can you help me out ?

@pullflow-ai please help with this issue, can you check the docs?

@test-et-pullflow-ai any help?

@test-et-pullflow-ai any help?

hello

hello

@pullflow-ai are you there. ?

f

f


Update License file to follow 80-column rule.

Thanks!

On Apr 29, 2016, at 12:58 PM, Ben Zhang<EMAIL_ADDRESS>wrote:
Merged #175 https://github.com/damellis/sensors/pull/175.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub https://github.com/damellis/sensors/pull/175#event-646148079


DON'T MERGE YET: Update IncreaseUnlockTime component and contractUtils

Context

@oaksprout I can see the Approve modal being triggered in the main network.

@mohandast52 not sure what you mean?