Add diagnostics for live preview mode

Adds diagnostics for the live preview mode (:StartPreview).
When in live preview mode, any definition errors are shown as diagnostics on the file being previewed (so they appear however the user has their diagnostics configured). When loading a colorscheme with errors normally, the errors are sent via vim.notify so that they are still surfaced to users. Currently, 2 types of errors are shown: invalid color values (ex: #21212 instead of #212121), or unknown attribute names (ex: Comment { f = "red" } instead of Comment { fg = "red" }).

(This example shows the diagnostics as they appear via lsp_lines.nvim)
I also realized that we weren't wasn't properly resetting the highlights on :StopPreview, so I made it so that the active colorscheme is saved when calling :StartPreview, and then we call :colorscheme <previous colorscheme> on :StopPreview.

@snejus would you be able to test out the live preview mode on this branch? I'm hoping that this will solve the earlier issue you had with errors in insert mode!

Error executing vim.schedule lua callback: ...ker/start/polychrome.nvim/lua/polychrome/diagnostics.lua:134: attempt to call field 'pack' (a nil value)
stack traceback:
        ...ker/start/polychrome.nvim/lua/polychrome/diagnostics.lua:134: in function 'to_range'
        ...ker/start/polychrome.nvim/lua/polychrome/diagnostics.lua:165: in function <...ker/start/polychrome.nvim/lua/polychrome/diagnostics.lua:145>
        ...ker/start/polychrome.nvim/lua/polychrome/diagnostics.lua:235: in function 'create_diagnostics_from_error'
        ...ker/start/polychrome.nvim/lua/polychrome/diagnostics.lua:263: in function ''
        vim/_editor.lua: in function <vim/_editor.lua:0>

See my :checkhealth output, if that helps
==============================================================================
CopilotChat: require("CopilotChat.health").check()

CopilotChat.nvim [core] ~
- OK nvim: NVIM v0.10.1
  Build type: Release
  LuaJIT 2.1.1723675123
  Run "nvim -V1 -v" for more info

CopilotChat.nvim [commands] ~
- OK curl: curl 8.9.1 (x86_64-pc-linux-gnu) libcurl/8.9.1 OpenSSL/3.3.1 zlib/1.3.1 brotli/1.1.0 zstd/1.5.6 libidn2/2.3.7 libpsl/0.21.5 libssh2/1.11.0 nghttp2/1.62.1 nghttp3/1.4.0
  Release-Date: 2024-07-31
  Protocols: dict file ftp ftps gopher gophers http https imap imaps ipfs ipns mqtt pop3 pop3s rtsp scp sftp smb smbs smtp smtps telnet tftp
  Features: alt-svc AsynchDNS brotli GSS-API HSTS HTTP2 HTTP3 HTTPS-proxy IDN IPv6 Kerberos Largefile libz NTLM PSL SPNEGO SSL threadsafe TLS-SRP UnixSockets zstd
- OK git: git version 2.46.0

CopilotChat.nvim [dependencies] ~
- OK plenary: installed
- OK copilot: copilot.vim
- WARNING tiktoken_core: missing, optional for token counting. See README for installation instructions.
- OK treesitter[markdown]: installed
- OK treesitter[diff]: installed

==============================================================================
conform: require("conform.health").check()

conform.nvim report ~
- Log file: /home/sarunas/.local/state/nvim/conform.log
- OK dprint ready (markdown, dockerfile, toml)
- WARNING eslint unavailable: Formatter config missing or incomplete
- OK gofmt ready (go)
- OK latexindent ready (tex)
- OK prettier ready (graphql, typescript, javascript, json, htmldjango)
- OK ruff_format ready (python)
- OK ruff_organize_imports ready (python)
- OK shellharden ready (bash, sh)
- OK sqlfluff ready (sql)
- OK stylua ready (lua)
- WARNING tidy unavailable: Formatter config missing or incomplete
- OK trim_newlines ready (_)
- OK trim_whitespace ready (_)
- WARNING tslint unavailable: Formatter config missing or incomplete
- OK yamlfmt ready (yaml)

==============================================================================
mason: require("mason.health").check()

mason.nvim ~
- OK mason.nvim version v1.10.0
- OK PATH: prepend
- OK Providers:
    mason.providers.registry-api
    mason.providers.client
- OK neovim version >= 0.7.0

mason.nvim [Registries] ~
- OK Registry `github.com/mason-org/mason-registry version: 2024-08-20-lawful-medaka` is installed.

mason.nvim [Core utils] ~
- OK unzip: `UnZip 6.00 of 20 April 2009, by Info-ZIP.  Maintained by C. Spieler.  Send`
- WARNING wget: not available
  - ADVICE:
    - spawn: wget failed with exit code - and signal -. wget is not executable
- OK curl: `curl 8.9.1 (x86_64-pc-linux-gnu) libcurl/8.9.1 OpenSSL/3.3.1 zlib/1.3.1 brotli/1.1.0 zstd/1.5.6 libidn2/2.3.7 libpsl/0.21.5 libssh2/1.11.0 nghttp2/1.62.1 nghttp3/1.4.0`
- OK gzip: `gzip 1.13`
- OK tar: `tar (GNU tar) 1.35`
- OK bash: `GNU bash, version 5.2.32(1)-release (x86_64-pc-linux-gnu)`
- OK sh: `Ok`

mason.nvim [Languages] ~
- WARNING luarocks: not available
  - ADVICE:
    - spawn: luarocks failed with exit code - and signal -. luarocks is not executable
- WARNING Composer: not available
  - ADVICE:
    - spawn: composer failed with exit code - and signal -. composer is not executable
- OK Go: `go version go1.23.0 linux/amd64`
- OK Ruby: `ruby 3.2.4 (2024-04-23 revision af471c0e01) [x86_64-linux]`
- OK PHP: `PHP 8.3.10 (cli) (built: Jul 31 2024 13:54:49) (NTS)`
- WARNING javac: not available
  - ADVICE:
    - spawn: javac failed with exit code - and signal -. javac is not executable
- WARNING java: not available
  - ADVICE:
    - spawn: java failed with exit code - and signal -. java is not executable
- WARNING julia: not available
  - ADVICE:
    - spawn: julia failed with exit code - and signal -. julia is not executable
- OK python: `Python 3.8.19`
- OK node: `v18.20.4`
- OK cargo: `cargo 1.80.1 (376290515 2024-07-16)`
- OK npm: `10.8.2`
- OK RubyGem: `3.4.19`
- OK pip: `pip 24.0 from /media/poetry/virtualenvs/rich-tables-clqIuwIl-py3.8/lib/python3.8/site-packages/pip (python 3.8)`
- OK python venv: `Ok`

mason.nvim [GitHub] ~
- OK GitHub API rate limit. Used: 0. Remaining: 5000. Limit: 5000. Reset: Sun 25 Aug 2024 09:04:32 AM BST.

==============================================================================
nvim: require("nvim.health").check()

Configuration ~
- OK no issues found

Runtime ~
- OK $VIMRUNTIME: /usr/share/nvim/runtime

Performance ~
- OK Build type: Release

Remote Plugins ~
- OK Up to date

terminal ~
- key_backspace (kbs) terminfo entry: `key_backspace=\177`
- key_dc (kdch1) terminfo entry: `key_dc=\E[3~`
- $COLORTERM="truecolor"

External Tools ~
- OK ripgrep 14.1.0 (/usr/bin/rg)

==============================================================================
nvim-treesitter: require("nvim-treesitter.health").check()

Installation ~
- OK `tree-sitter` found 0.22.6 (b40f342067a89cd6331bf4c27407588320f3c263) (parser generator, only needed for :TSInstallFromGrammar)
- OK `node` found v18.20.4 (only needed for :TSInstallFromGrammar)
- OK `git` executable found.
- OK `cc` executable found. Selected from { vim.NIL, "cc", "gcc", "clang", "cl", "zig" }
  Version: cc (GCC) 14.2.1 20240805
- OK Neovim was compiled with tree-sitter runtime ABI version 14 (required >=13). Parsers must be compatible with runtime ABI.

- OK nvim-treesitter installed.

Checking external dependencies ~
- OK rg: found ripgrep 14.1.0
- OK fd: found fd 10.1.0

===== Installed extensions ===== ~

Telescope Extension: `fzf` ~
- OK lib working as expected
- OK file_sorter correctly configured
- OK generic_sorter correctly configured

==============================================================================
vim.lsp: require("vim.lsp.health").check()

- LSP log level : WARN
- Log path: /home/sarunas/.local/state/nvim/lsp.log
- WARNING Log size: 156327 KB

vim.lsp: Active Clients ~
- GitHub Copilot (id: 1)
    Root directory: nil
    Command: node /home/sarunas/.local/share/nvim/site/pack/packer/start/copilot.vim/dist/language-server.js --stdio
    Settings: {
      ["github-enterprise"] = {
        uri = vim.NIL
      },
      http = {
        proxy = vim.NIL,
        proxyStrictSSL = vim.NIL
      }
    }
    Attached buffers: 3

vim.lsp: File Watcher ~
- file watching "(workspace/didChangeWatchedFiles)" disabled on all clients

vim.lsp: Position Encodings ~
- No buffers contain mixed position encodings

==============================================================================
vim.treesitter: require("vim.treesitter.health").check()

- Nvim runtime ABI version: 14
- OK Parser: bash                 ABI: 14, path: /home/sarunas/.local/share/nvim/site/pack/packer/start/nvim-treesitter/parser/bash.so
- OK Parser: c                    ABI: 14, path: /home/sarunas/.local/share/nvim/site/pack/packer/start/nvim-treesitter/parser/c.so
- OK Parser: cpp                  ABI: 14, path: /home/sarunas/.local/share/nvim/site/pack/packer/start/nvim-treesitter/parser/cpp.so
- OK Parser: csv                  ABI: 14, path: /home/sarunas/.local/share/nvim/site/pack/packer/start/nvim-treesitter/parser/csv.so
- OK Parser: diff                 ABI: 14, path: /home/sarunas/.local/share/nvim/site/pack/packer/start/nvim-treesitter/parser/diff.so
- OK Parser: dockerfile           ABI: 14, path: /home/sarunas/.local/share/nvim/site/pack/packer/start/nvim-treesitter/parser/dockerfile.so
- OK Parser: editorconfig         ABI: 14, path: /home/sarunas/.local/share/nvim/site/pack/packer/start/nvim-treesitter/parser/editorconfig.so
- OK Parser: git_config           ABI: 13, path: /home/sarunas/.local/share/nvim/site/pack/packer/start/nvim-treesitter/parser/git_config.so
- OK Parser: git_rebase           ABI: 14, path: /home/sarunas/.local/share/nvim/site/pack/packer/start/nvim-treesitter/parser/git_rebase.so
- OK Parser: gitcommit            ABI: 14, path: /home/sarunas/.local/share/nvim/site/pack/packer/start/nvim-treesitter/parser/gitcommit.so
- OK Parser: gotmpl               ABI: 14, path: /home/sarunas/.local/share/nvim/site/pack/packer/start/nvim-treesitter/parser/gotmpl.so
- OK Parser: graphql              ABI: 13, path: /home/sarunas/.local/share/nvim/site/pack/packer/start/nvim-treesitter/parser/graphql.so
- OK Parser: ini                  ABI: 14, path: /home/sarunas/.local/share/nvim/site/pack/packer/start/nvim-treesitter/parser/ini.so
- OK Parser: javascript           ABI: 14, path: /home/sarunas/.local/share/nvim/site/pack/packer/start/nvim-treesitter/parser/javascript.so
- OK Parser: jq                   ABI: 13, path: /home/sarunas/.local/share/nvim/site/pack/packer/start/nvim-treesitter/parser/jq.so
- OK Parser: json                 ABI: 14, path: /home/sarunas/.local/share/nvim/site/pack/packer/start/nvim-treesitter/parser/json.so
- OK Parser: latex                ABI: 14, path: /home/sarunas/.local/share/nvim/site/pack/packer/start/nvim-treesitter/parser/latex.so
- OK Parser: ledger               ABI: 14, path: /home/sarunas/.local/share/nvim/site/pack/packer/start/nvim-treesitter/parser/ledger.so
- OK Parser: lua                  ABI: 14, path: /home/sarunas/.local/share/nvim/site/pack/packer/start/nvim-treesitter/parser/lua.so
- OK Parser: markdown             ABI: 14, path: /home/sarunas/.local/share/nvim/site/pack/packer/start/nvim-treesitter/parser/markdown.so
- OK Parser: python               ABI: 14, path: /home/sarunas/.local/share/nvim/site/pack/packer/start/nvim-treesitter/parser/python.so
- OK Parser: query                ABI: 14, path: /home/sarunas/.local/share/nvim/site/pack/packer/start/nvim-treesitter/parser/query.so
- OK Parser: regex                ABI: 14, path: /home/sarunas/.local/share/nvim/site/pack/packer/start/nvim-treesitter/parser/regex.so
- OK Parser: requirements         ABI: 14, path: /home/sarunas/.local/share/nvim/site/pack/packer/start/nvim-treesitter/parser/requirements.so
- OK Parser: rst                  ABI: 14, path: /home/sarunas/.local/share/nvim/site/pack/packer/start/nvim-treesitter/parser/rst.so
- OK Parser: sql                  ABI: 14, path: /home/sarunas/.local/share/nvim/site/pack/packer/start/nvim-treesitter/parser/sql.so
- OK Parser: toml                 ABI: 14, path: /home/sarunas/.local/share/nvim/site/pack/packer/start/nvim-treesitter/parser/toml.so
- OK Parser: vim                  ABI: 14, path: /home/sarunas/.local/share/nvim/site/pack/packer/start/nvim-treesitter/parser/vim.so
- OK Parser: vimdoc               ABI: 14, path: /home/sarunas/.local/share/nvim/site/pack/packer/start/nvim-treesitter/parser/vimdoc.so
- OK Parser: xml                  ABI: 14, path: /home/sarunas/.local/share/nvim/site/pack/packer/start/nvim-treesitter/parser/xml.so
- OK Parser: yaml                 ABI: 14, path: /home/sarunas/.local/share/nvim/site/pack/packer/start/nvim-treesitter/parser/yaml.so
- OK Parser: zathurarc            ABI: 14, path: /home/sarunas/.local/share/nvim/site/pack/packer/start/nvim-treesitter/parser/zathurarc.so
- OK Parser: c                    ABI: 14, path: /usr/share/nvim/runtime/parser/c.so
- OK Parser: lua                  ABI: 14, path: /usr/share/nvim/runtime/parser/lua.so
- OK Parser: markdown             ABI: 14, path: /usr/share/nvim/runtime/parser/markdown.so
- OK Parser: markdown_inline      ABI: 14, path: /usr/share/nvim/runtime/parser/markdown_inline.so
- OK Parser: query                ABI: 14, path: /usr/share/nvim/runtime/parser/query.so
- OK Parser: vim                  ABI: 14, path: /usr/share/nvim/runtime/parser/vim.so
- OK Parser: vimdoc               ABI: 14, path: /usr/share/nvim/runtime/parser/vimdoc.so

Huh, interesting, looks like table.pack doesn't exist in your Lua, but does in mine, despite my LuaJIT version being older.

Errors are gone!

Diagnostic virtual text shows immediately in normal mode, but the behaviour is a bit more interesting in insert mode, since it shows only once I leave it. This may have to do with my configuration though!

Yeah, diagnostics don't update immediately in insert mode unless you have vim.diagnostic.config({ update_in_insert = true }). That's a global setting though, so most people may leave it set to false. That's fine though, my main goal with this was finding a way to keep the preview updates happening on TextChangedI, while still allowing for errors to be reported asynchronously.
I'm going to make a few small changes to this and then get it merged, thanks for testing this out!


Upgrade to Edge.js 5.9.1

We're still on 4.0.0, and we should attempt to move to latest. Obviously, we'll need to carefully test for regressions.

https://github.com/Azure/azure-webjobs-sdk-script/commit/96783969299f720224a521ab0803e084477e0f04


Docs: WFS and Style

add docs for WFS
add docs for Style including Labels
make mandatory properties fat and remove column "mandatory". This makes the docs less verbose.
Update the version number
TODO:
[ ] add remark that fat is mandatory

Good to have mandatory attrs in bold! Is it an idea to also add a default column for non-mandatory attrs (where applicable)? This is often expressed in the 'meaning' text.

@justb4 I like the idea of having a column for default values. I do not see this change in this PR, but I will open an issue.

This is good to go, thanks @JakobMiksch! I'll merge now.


Add support for primary and secondary contact information

This PR adds support for adding a secondary telephone number on all pages with contact information or updates the labels of the ones that already supported this.

There are a lot of commits since I created a commit for each page. I might reduce this by combining all related pages into one commit.


Update aws-java-sdk-bundle to 1.12.548

About this PR
📦 Updates com.amazonaws:aws-java-sdk-bundle from 1.12.530 to 1.12.548
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.amazonaws", artifactId = "aws-java-sdk-bundle" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "com.amazonaws", artifactId = "aws-java-sdk-bundle" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #2890.


ReceiveDisabled option missing in Service bus queue statuses

Hi,
Is your feature request related to a problem? Please describe.
Missing ReceiveDisabled option in Service bus queue status updated. While it's existing in Azure portal, Powershell command and Service bus documentation, it was deleted in one of the PR for Azure CLI:
https://github.com/Azure/azure-cli/pull/11566.
Official documentation about Service bus:
https://docs.microsoft.com/en-us/azure/service-bus-messaging/entity-suspend#queue-status.
Is it scheduled to delete it also in other components? (Azure portal for example). I need to use that option for setting my queue status and now I am not sure if it's good to rely on this queue status in production.
Describe the solution you'd like
Revert the setting back or clarify the purpose of deletion just in Azure CLI.

Hi, do you have any update about my question?

+1 here

service bus service team should take a look

@rangarnow, thanks for bringing this to our attention. As per my understanding, Receivedisabled holds true for Service bus queues.
I would triage internally with team and keep you posted with additional updates.


How to check if user is online in Azure SignalR serverless mode

Hi,
My goal is to create a chat app that can save a message when the user is offline and distribute the message the next time the user comes back online.
The missing piece in my Azure Function is how to check if a user is online before sending him the message? Note that this question is not about dealing with the offline and distribution challenge. That's not a problem. Only how to find out somehow that I need to store my message and distribute later.
My code so far is very classic and works:
public static class Talk
    {
        [FunctionName("Talk")]
        public static async Task<IActionResult> Run(
            [HttpTrigger(
                AuthorizationLevel.Anonymous,
                "post",
                Route = "talk")]
            HttpRequest req,
            [SignalR(HubName = "simplechat")]
            IAsyncCollector<SignalRMessage> questionR,
            ILogger log)
        {
            try
            {
                string json = await new StreamReader(req.Body).ReadToEndAsync();
                dynamic obj = JsonConvert.DeserializeObject(json);
                var jObject = new JObject(obj);

                req.Headers.TryGetValue("x-ms-client-principal-name", out var sender);

                var recipient = obj.recipient.ToString();

                await questionR.AddAsync(
                    new SignalRMessage()
                    {
                        UserId = recipient,
                        Target = Constants.MessageName,
                        Arguments = new[] { jObject }
                    });
...

Is there a way to verify a user is online in there?
A way to see that the message was not delivered would also do the trick.
I tried using the following to get access to the missing pieces but this throws an exception. Can this be potentially used from inside an Azure Serverless Function?
var serviceHubContext = await StaticServiceHubContextStore.Get("AzureSignalRConnectionString").GetAsync("simplechat");
                    log.LogInformation("C# HTTP trigger function processed a request.");
                    await serviceHubContext.Clients.All.SendAsync("newMessage", "LOL!");

I prefer not to keep a user list myself through on/disconnect because it's not completely reliable and there must be an easier way, right?
Maybe using the rest api is an option but I have no idea how to do that in case someone has an exemple.
Thanks a bunch for your lights.
BTW: Can you mix ... : ServerlessHub stuff and static Azure Functions? Like for example using this to distribute the message:
public class SignalRTestHub : ServerlessHub
{
    [FunctionName("SignalRTest")]
    public async Task SendMessage([SignalRTrigger]InvocationContext invocationContext, string message, ILogger logger)
    {
        ...
    }
}

I tried but it seems only the code in there is ignored.
If someone has an Azure ServerlessHub sample and Xamarin clients, please share.
Thanks.

We do have REST API for checking if the user has any connected connections https://github.com/Azure/azure-signalr/blob/dev/docs/swagger/v1.md#get-check-if-there-are-any-client-connections-connected-for-the-given-user We haven't integrated this feature into the Mangement API yet (https://github.com/Azure/azure-signalr/issues/922), for now, you can have a REST API call to get the result. (cc @wanlwanl )

Thanks for your advice @vicancy much appreciated.
I've done that and it seems to be working quite nicely.
It would be really nice to get this feature added to the Management API. It's quite some code including non trivial code to generate the access token. This or some way to verify that my message was/can be delivered or not.
(Also an example of how to call the Management API from inside Azure Functions would be nice.)
If anyone is interested by the code. I used code from this sample https://github.com/aspnet/AzureSignalR-samples/tree/master/samples/Serverless
I added this method to ServerHandler.cs (+ some minor code adjustment IIRC)
public async Task<bool> IsUserOnline(string userId)
        {
            string url = GetSendToUserUrl(_hubName, userId);
            if (!string.IsNullOrEmpty(url))
            {
                Client.DefaultRequestHeaders.Authorization =
                   new AuthenticationHeaderValue("Bearer", _serviceUtils.GenerateAccessToken(url, _serverName));

                var response = await Client.GetAsync(url, HttpCompletionOption.ResponseHeadersRead);
                if (response.IsSuccessStatusCode)
                {
                    return true;
                }
            }

            return false;
        }

From inside my azure Function I call this like that:
var server = new ServerHandler(connectionString, "simplechat");
                if (await server.IsUserOnline(recipient))
                {
                    await SendMessageToClient(questionR, "Message delivered");
                }
                else
                {
                    await SendMessageToClient(questionR, $"User {recipient} is not online.");
                }


Windows IIS preset

Ref https://github.com/nuxt/framework/pull/6180

@Heinrichv Would you like to work on thi to create the iis preset and docs? We can after that list it in nuxt framework docs.


Generate metatron credentials with IP address

This makes the generation of metatron credentials a callback, versus
the previous, which was a generation "ahead of time". Although,
this introduces a higher chance of error, and latency at container
startup, it lets us pass the IP address of the container into
the metatron init script.

Pull Request Test Coverage Report for Build 922

0 of 49 (0.0%)  changed or added relevant lines in 2 files are covered.
4 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.03%) to 25.934%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

executor/runtime/docker/docker.go
0
15
0.0%

executor/runner/runner.go
0
34
0.0%

Files with Coverage Reduction
New Missed Lines
%

executor/runner/runner.go
1
57.73%

launchguard/client/client.go
3
72.11%

Totals

Change from base Build 912:
-0.03%

Covered Lines:
2408

Relevant Lines:
9285

💛  - Coveralls

This relates to TITUS-1757


Create tokens programmatically

I'm looking for a way to create user accounts without a password and initiate the reset procedure without the extra work on front-end or user side – it's supposed to be an atomic operation from my API's POV. Such functionality would also be helpful when creating mass or time-based password resets. Unfortunately, there's quite some logic hidden in the ResetPasswordRequestToken's post that prevents one from doing so.
What do you think about extracting and exposing it for the programmatic use? I can imagine two ways it could be done at the moment:

create a public function of email and/or register a new signal handler
follow the UNIX philosophy mentioned in the README file a one step further and extract the core resetting functionality to a separate package so it could be used without DRF and only keep a thin layer of DRF-specific usage here.

I've already seen #97 which partially covers this but it also introduces a completely new concept of "registration tokens" which are not needed for the use cases presented above.
If we reach an agreement on the scope of the change, I'd be happy to work on it and submit a PR.

Relates to https://github.com/anexia-it/django-rest-passwordreset/issues/141


Complete yarn plugin

Add available properties
Fix TODOs in the yarn plugin
Clean up code
Review code and check understanding

Extracted https://github.com/spdx/tools-ts/issues/170 to separate the extension of the plugin with licenses.


Picking out order

Tweaks picking docs slightly for formatting and to add additional context about the ordering of Over and Out events. Also shifts Out to trigger before Over in the global event ordering.
Because of how focus is tracked, we must send all Over and Out events at the same time, in a block. Originally I had Over precede Out in the global event order, because this seemed natural. However, the effect of this, when a pointer moves between entities, is to have the new entity receive Over before the old entity received Out, which several users found confusing.
The new ordering (out before over globally, over before out locally per entity) should make it much easier to write hover state cleanup code.

I'd love to have some tests to verify that order, but that's a ton of extra work and I won't block on it.

ooh good idea. I think this will make sense to do as part of the next winit upgrade, because the way this pointer stuff interacts with their pointer stuff will be complex and not entirely owned by us.

@NthTensor could you look into the conflict and fixing the order in the comment?

Yep, thanks for the poke.


Bump jetty-maven-plugin from 9.4.40.v20210413 to 11.0.3

Bumps jetty-maven-plugin from 9.4.40.v20210413 to 11.0.3.

Release notes
Sourced from jetty-maven-plugin's releases.

11.0.3
Changelog

#3764 DeprecationWarning Decorator
#5684 Review disabled tests
#5798 jetty-runner startup error with jetty-10
#5817 Provide more filtering for CustomRequestLog
#6049 Default provider [files] section always executed
#6084 GzipHandler: NPE in setDeflaterPoolCapacity and setInflaterPoolCapacity
#6098 jetty-cdi is missing from jetty-bom
#6099 Cipher preference may break SNI if certificates have different key types
#6105 HttpConnection.getBytesIn() incorrect for requests with chunked content
#6106 WebSocket/CDI integration is broken in Jetty 10
#6125 Do not allow override of jakarta.* container classes by webapps per Servlet 5.0 Section 15.2.1
#6132 Ambiguous segment in URI in DELETE /a/projects/foo/branches/refs%2Fheads%2Ftest request after upgrade from 10.0.0 to 10.0.2
#6153 jetty-maven-plugin does not correctly pass JVM arguments for external deployMode
#6159 Jetty with Conscrypt unable to handle any HTTPS requests when connected by IP rather than hostname.
#6166 WebSocket MessageInputStream.read() spends a lot of time in ByteBuffer.compact()
#6205 OpenIdAuthenticator may use incorrect redirect
#6208 HTTP/2 max local stream count exceeded
#6224 make jetty-jspc-maven-plugin @threadsafe
#6227 Better resolve race between AsyncListener.onTimeout and AsyncContext.dispatch
#6238 jetty-keystore Invalid manifest header Bundle-SymbolicName: ""
#6250 Lazily allocate HTTP2Stream data queue
#6251 Use CyclicTimeout for HTTP2Streams
#6254 Total timeout not enforced for queued requests
#6263 Review URI encoding in ConcatServlet & WelcomeFilter
#6277 Better handle exceptions thrown from session destroy listener
#6280 Copy ServletHolder class/instance properly during startWebapp
#6287 Class loading broken for WebSocketClient used inside webapp

11.0.2
Changelog
:warning: Important Security related Changes

CVE-2021-28165 - #6072 - jetty server high CPU when client send data length > 17408
CVE-2021-28164 - #6101 - Normalize ambiguous URIs
CVE-2021-28163 - #6102 - Exclude webapps directory from deployment scan

Other Changes

#4275 - Path Normalization/Traversal - Context Matching
#5828 - Allow to create a WebSocketContainer passing HttpClient
#5832 - Ctrl-C after jetty:run produces NoClassDefFoundError
#5835 - Review Durable Filters, Servlets and Listeners
#5977 - Cache-Control header set by a filter is override by the value from DefaultServlet configuration
#5994 - QueuedThreadPool "free" threads
#5996 - ERROR : No module found to provide logback-impl for logback-access{enabled}
#5999 - HttpURI ArrayIndexOutOfBounds

... (truncated)

Commits

d97980b Updating to version 11.0.3
2d08ed6 Merged branch 'jetty-10.0.x' into 'jetty-11.0.x'.
cd73338 Remove WebSocketComponents & HouseKeeper on Server restart. (#6218)
28562d6 Merged branch 'jetty-10.0.x' into 'jetty-11.0.x'.
802d32d Fixes #6207 - Make ALPN optional in HTTP2Client over TLS
0e2a809 Merge remote-tracking branch 'origin/jetty-10.0.x' into jetty-11.0.x
455e798 Merge pull request #6293 from eclipse/jetty-10.0.x-6287-WebSocketClientClassL...
462aff3 Merged branch 'jetty-10.0.x' into 'jetty-11.0.x'.
67e2b4a Fixes #5306 - Default jetty.*.acceptors should be 1. (#6236)
ddbf8e2 Merge remote-tracking branch 'origin/jetty-10.0.x' into jetty-11.0.x
Additional commits viewable in compare view

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.

Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

@dependabot rebase will rebase this PR
@dependabot recreate will recreate this PR, overwriting any edits that have been made to it
@dependabot merge will merge this PR after your CI passes on it
@dependabot squash and merge will squash and merge this PR after your CI passes on it
@dependabot cancel merge will cancel a previously requested merge and block automerging
@dependabot reopen will reopen this PR if it is closed
@dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
@dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
@dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
@dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Author: dependabot[bot]. Timestamp: 2021-06-09T06:49:45Z

Superseded by #10.


[Bug]: Error in syntax in SignupForm.jsx & LoginForm.jsx files

🐞 Bug Report
📋 Description

the  and  tag in line ending not correct leading to error during npm run dev
⚙ Steps to Reproduce
Steps to reproduce the behavior:

correct the order of  and  tag as shared in the screen shot

✔ Expected Behavior

❌ Actual Behavior

📷 Screenshots

SignupForm.jsx
Actual code erro:

Updated code:

LoginForm.jsx
Actual code erro:

Updated code:

🖥 Environment (if applicable):

OS: (e.g., Windows, macOS, Linux)
Browser: (e.g., Chrome, Safari, Firefox)
Version: (if relevant)

🔧 Additional Information

Kindly assign me the issue under [gssoc-ext][hacktoberfest-accepted] tag.

🐞 Bug Report
📋 Description
the and tag in line ending not correct leading to error during npm run dev
⚙ Steps to Reproduce
Steps to reproduce the behavior:

correct the order of  and  tag as shared in the screen shot

✔ Expected Behavior
❌ Actual Behavior
📷 Screenshots
SignupForm.jsx
Actual code erro:
Updated code:
LoginForm.jsx
Actual code erro:
Updated code:
🖥 Environment (if applicable):

OS: (e.g., Windows, macOS, Linux)
Browser: (e.g., Chrome, Safari, Firefox)
Version: (if relevant)

🔧 Additional Information
Kindly assign me the issue under [gssoc-ext][hacktoberfest-accepted] tag.

Sure.

can i work on it if this issue is open

I am working on it

I am working on it

It's been so long

i have the solution. Will commit it today

is it fine

the issue is still not fixed


Feature/register and volunteer form

There's a register page you can access from home, and a form you can access with '/form-volunteer'.
I tried to add a google icon to the register page, but our material-ui version doesn't support it, therefore, we must see how to proceed about this later.
For now everything works and there isn't any conflict between my files and develop.

Form seems alright but the text clips with the header.

Proportions change a lot depending on whether the page's fully open or just on a partial window


Jumpstart ArcBox - Overview

Scenario which you are working on
A link to the Jumpstart scenario you are working on.
Describe the bug
Arcbox deployment failed on ubuntuCAPIDeployment and ubuntuRancherDeployment steps, regardless of region selected.
{"code":"DeploymentFailed","message":"At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/DeployOperations for usage details.","details":[{"code":"VMExtensionProvisioningTimeout","message":"Provisioning of VM extension installscript_CAPI has timed out. Extension provisioning has taken too long to complete. The extension last reported "Plugin enabled".\r\n\r\nMore information on troubleshooting is available at https://aka.ms/VMExtensionCSELinuxTroubleshoot"}]}
{"code":"DeploymentFailed","message":"At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/DeployOperations for usage details.","details":[{"code":"VMExtensionProvisioningError","message":"VM has reported a failure when processing extension 'installscript_k3s'. Error message: "Enable failed: failed to execute command: command terminated with exit status=2\n[stdout]\n\n[stderr]\n"\r\n\r\nMore information on troubleshooting is available at https://aka.ms/VMExtensionCSELinuxTroubleshoot "}]}
To Reproduce
Steps to reproduce the behavior:
Click Deploy button, let deployment run.
Expected behavior
Completed with no errors
Screenshots

Additional context
Add any other context about the problem here.

Thank you @vanessabruwer. Can you please share the installCAPI.log output?

ssh to the ArcBox-CAPI-MGMT VM
sudo -s
_cat /var/lib/waagent/custom-script/download/0/a/installCAPI.log

I'll need to redeploy, as I removed all resources post failure. Will revert with log file once done (may take a few days)

Due to no response, the release of ArcBox 2.0 and unable to repro, I will go ahead and close this for now. Please comment if further assistance is required.


A new plugin for oh-my-zsh: easy zsh named directory management

This plugin allows the user to create, change, and delete named directories easily. It also provides an autocompletion function, and a function called folder_name() that can be included in PROMPT to show the user all the names of the current folder (not including default names such as '~' or '~user').
It stores the bookmarks in the file $HOME/.zshbookmarks by default, but this is customizable.
I have used this function for years since I first wrote it, and I think the community could benefit from it :)
Happy to change anything.

@MikeDacre, what is the difference between your plugin and jump plugin?

@popstas Wow. I thought no one cared about this at all, it has been 18 months since I submitted this.
From looking at jump it seems that it is essentially the same idea, cdbk just has a nicer ui and is able to parse out hardcoded named directories from ~/.zshrc.
I didn't know jump existed when I submitted cdbk 18 months ago, feel free to drop the pull request, I don't really care that much. I wrote cdbk six years ago, and I am not switching to jump, particularly given the extra features of cdbk and the fact I use it every day. However, considering the liveliness of this pull request, my assumption is that interest in cdbk is essentially zero, so just delete the pull request if you want.

You can add your plugin to the external plugins list.


Future chaining does not respect ignored futures

import 'dart:async';

void main() async {
  final c = Completer<void>();
  Future.error("a").whenComplete(() {
    c.complete();
    return Future.error("b");
  }).ignore();
  await c.future;
}

$ out/ReleaseX64/dart /tmp/test.dart
Unhandled exception:
b

Ack. I'd have expected it to be the "a" error which wasn't ignored, if anything.
This'll be fun to debug. :sob:

And for the record, @mraleph was completely correct, it was the chaining which broke the ignore.


ReactiveProperty without a default value

Hello,
as far as I understand, there are two ways of initializing a ReactiveProperty:

with a default value
with a source observable

When given a default value, the property will propagate this value immediately to its subscribers. When given a source observable, the property will not propagate anything until the source observable produces something.
I find myself bridging my reactive properties with non-UniRx event systems, and I'd really like a way to create a reactive property without a source observable and without a default value, which will not propagate any values until something is first set to it.
I could probably define a helper subject, then initialize my reactive property with this subject and then set the value into the subject when it comes but it feels like a lot of ceremony for something that should be easy to do.
Think it'd be useful?

One way we get around that is to use ReplaySubject.  If you create a new ReplaySubject with a buffer size of 1 then it don't emit anything until the first OnNext call, and every OnNext call after that will send the newest value.

Thanks, that's clever and we might use that. Would still prefer to see this in ReactiveProperty :)

Agree. There should be option to disable emit if no default value was passed

I ended up commenting line 193 on ReactiveProperty.cs. There should be an option to disable that OnNext on subscribe, in fact it should be disabled by default. I can see situations where the current value is needed, but most of the time all I want is the next one.
`public IDisposable Subscribe(IObserver observer)
{
if (isDisposed)
{
observer.OnCompleted();
return Disposable.Empty;
}
        //JMN: This is a non desired behavior. When subscribing to a ReactiveProperty it should not emit the current value
        // raise latest value on subscribe
        // observer.OnNext(value);

        // subscribe node, node as subscription.
        var next = new ObserverNode<T>(this, observer);
        if (root == null)
        {
            root = last = next;
        }
        else
        {
            last.Next = next;
            next.Previous = last;
            last = next;
        }
        return next;
    }`


Factbox OnLookUp not being executed

1. Describe the bug
The OnLookUp trigger is not executed on the Factboxses.
2. To Reproduce
Create any simple fact box with a simple message example in it
field(TestText; TestText)
{
Caption = 'TestText';
ApplicationArea = all;
Lookup = true;
            trigger OnLookup(var Text: Text): Boolean
            begin
                Message('The OnLookup Works!');
            end;
        }

3. Expected behavior
When you click "TestText"  - you see the expected message ('The OnLookup Works!).
4. Actual behavior
When you click "TestText"  - you do not see the expected message.
5. Versions:

AL Language: Version 12.6
Visual Studio Code: Version: 1.73.1 (user setup)
Business Central: 23.1
List of Visual Studio Code extensions that you have installed: -


Syncthing security issue

Issue description
The Syncthing project recently released a fix for a security vulnerability, the symlink attack described here:
https://github.com/syncthing/syncthing/issues/4286
The issue was fixed in Syncthing 0.14.35. However, this release contained a serious bug, so 0.14.36 should be used instead.

cc @peterhoeg

0.14.36 was already in master but now 17.03 has been updated too. Thanks for spotting this @lfam!


limit matchers to only those required (json/xml)

this avoid potentially confusing results if the expected/actual result is not
valid JSON (as the matcher library tries to sniff the type of content being
matched)

I think it would be nice to extract this logic to separate class. Just to make it more readable. Are you willing to do it?

@lchrusciel PR updated.

Nice! I would love to merge this, but unfortunately it has some conflicts. @rodnaph can you rebase?

Ah sorry, didn't notice.  Rebased.

And the tests failed. Probably because of non-static call from a static method.

@lchrusciel Build passing now.

Thanks Rhodri!


Referencing other keys only works on outer keys in the global scope

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[x ] Bug report
[ ] Performance issue
[ ] Feature request
[ ] Documentation issue or request
[ ] Support request
[ ] Other... Please describe:

Current behavior
Referencing another key in a nested key doesn't work (pipe returns blank).
Referencing another key in a scoped file doesn't work (pipe returns blank).
Example demonstrated in StackBlitz: https://stackblitz.com/edit/ngneat-transloco-sjxkeq
The "some.nested.test" key will not fully render on the Home Page.
The "test" key will not render on the Admin Page (a scoped page)
Expected behavior
Nested keys can reference other keys in the file (ideally any key in the file, not just other keys in the same nest).
Scoped definitions should be able to reference other keys in the way that the documentation reads.
Minimal reproduction of the problem with instructions
Example demonstrated in StackBlitz: https://stackblitz.com/edit/ngneat-transloco-sjxkeq
What is the motivation / use case for changing the behavior?
Documentation states this feature exists but doesn't state any limitations.
Environment

Angular version: 8.2.14

Browser:
- [x] Chrome (desktop) version XX
- [ ] Chrome (Android) version XX
- [ ] Chrome (iOS) version XX
- [ ] Firefox version XX
- [ ] Safari (desktop) version XX
- [ ] Safari (iOS) version XX
- [ ] IE version XX
- [ ] Edge version XX

For Tooling issues:
- Node version: XX
- Platform:

Others:

It should be:
"test": "Result2: {{some.nested.key}}"

The second one I believe should be:
"test": "{{adminPage.title}}"

Both of the issues I was having resulted from not knowing that I needed to reference the scoped variable from within itself. Thank you @ArielGueta !

@ArielGueta Thanks for clarifying.
@devinvoss I updated the Reference Other Keys section in the docs 👍


[Website] Split Showcase and add mini-Showcase to homepage

We're revamping the Showcase, and in this first PR we're focusing on two things:

We're splitting the list of featured apps. Those in the "pinned" list will be fairly static and PRs that add/remove apps from this list will be discouraged. Apps in the "featured" list will be held to the same standards established this past summer (funded startup or public company + engineering blog post on RN / news article from established company).

We're displaying the pinned apps in a small showcase at the bottom of the homepage.

Note that pinned and featured apps are displayed together in the main showcase without any separation. Pinned apps will appear first in an arbitrary order, followed by featured apps in alphabetical order. Future PRs may address how apps are displayed in the main Showcase itself.
We want to also make sure people understand some of these apps are not built entirely using React Native. Some apps are native apps with individual RN views added in.
Full Showcase with pinned apps + featured apps:

Mini-showcase on the homepage:

cc @lacker, @skevy, @brentvatne, @lelandrichardson

@hramos if you'd like to use Airbnb on the "who's using React Native" page, i've gotten approval. You can use this Icon image: https://a2.muscache.com/airbnb/static/icons/apple-touch-icon-180x180-bcbe0e3960cd084eb8eaf1353cf3c730.png

@lelandrichardson great, I'll go ahead and add it. Do you have any preference on what the icon should link to? It looks like https://www.airbnb.com/mobile might work.

@hramos that works for me!

Added Airbnb. We're still missing infoLinks for Instagram and Airbnb, but we can make an exception since these apps will be pinned to the homepage.

@hramos it's not the greatest infolink, but you could use this potentially: https://www.youtube.com/watch?v=tUfgQtmG3R0

Works for me

@facebook-github-bot shipit

Waiting on the internal diff to land...


Refactor event name resolving

@mattwynne As discussed some weeks ago I finished the refactoring of the Event::Bus:

Added more tests for Bus + NameResolver
Refactored the the test suite to make use of context to show the code behaves in different contexts/situations

Maybe we should really move that event bus thing to a separate library. This would make sharing the code much easier and we could use rubygems + version requirement(s) to make aruba and cucumber depend on the correct interface. WDT? What about cucumber-ruby-common or just event-bus.

@mattwynne I prepared a ruby gem for this: https://github.com/dg-ratiodata/event-bus

I prepared a PR to integrate the gem instead of the changes made in this PR. Either #932 or #933 should be merged.

Thanks @dg-ratiodata. I'm pretty snowed under for the next few weeks so I may not have a chance to grok this code for a while. Don't let that hold you up using the event bus pattern for aruba.

@mattwynne Ok. I am going to use the gem for aruba.

Pinging @tooky on this as well.

Having read this, I think that my preferred refactoring (if we're going to bother extracting a resolver at all) would be to extract three strategy objects so that we can do away with the whole case statement.

Superseded by #933

Let's move the discussion here https://github.com/fedux-org/event-bus/pull/4.


Added Python resources

I will have to looking into this closely.  Seems like you were behind few commits and merged the whole branch into yours.
This shouldn't really affect anything but I have to see all the changes once.
Give me some time please @jackmcbride 🙂

@codingonHP can you please start looking into al the old PRs and merge them? 🙏
If there are any conflicts, please try to fix or assign it to the user who has worked on this change.

@vicky002 Sure, let me see what I can do.

Hey @jackmcbride if you get time, please have a look at this :)

closing due to inactivity.


Question: Dynamic query, Criteria Builder or Elasticsearch

Hello Guys,
I am developing a complex filter screen where I have multiples fields that must be combined to perform the query with pagination (limit and offset). In my particular case I cannot get the entire list and work with stream transform due the fact my List is paginated.
Floor has any alternative like  pass the entire query (dynamic query) to the dao instead just field parameters? Or something like criteria builder or may elastic search?

You can use sqflite directly (via $FloorMyDatabase.database.rawQuery(), see below), but apart from that this is pretty much what #315 tries to implement (some kind of raw query which is not parsed and checked and uses simple string replacements for the arguments)
Using a raw query right now:
AppDatabase _database = await $FloorAppDatabase.databaseBuilder('myDb.db').build();
List<Map<String,dynamic>> output = await _database.database.rawQuery(
    'SELECT * FROM Page LIMIT $count OFFSET $offset'
  )
print(output);

@mqus the simples raw queries have worked, thank you! I am closing this question.


Make nature of height constraint more explicit

If the multiplier is zero, you're not really constraining to the other anchor, you're just using the constant.
Using a multiplier of anything other than 0 causes it to crash because they do not share an ancestor.

Actually, it appears that this causes it to crash because they do not share an ancestor. So the constraint method does not seem to work at all.

Yes you are right, the height anchor should be constant 0 initially, and it shouldn't be constrained with a multiplier because they have no common ancestor. it looks good to me, will merge it now , Thanks @gsbernstein!


Update build procedures. Add rpm and deb package scripts.

Better folder layout, cleans up/re-arranges files/folders in the repo.
Adds build scripts for deb and rpm packages. Packages build, but they're as-of-now untested.
Also adds build script for macOS package.
Cleans up local install and uninstall procedures.

Tested on macos and bionic. Updated wiki as well after reproducing a build on a fresh install of Ubuntu 18.04.


[Feature request] A more powerful custom origin calculation method depending on other headers

Currently this middleware only allows you to dynamically resolve the CORS header value depending on the origin of the incoming request, but sometimes this isn't enough and more information about the request is necessary to make the decision.
Something like this would be nice:
cors({
  origin: (req: express.Request) => {...}
})

Here's a use case:
An app has header-based-authentication (a token is passed along with each request) and an Access-Control-Allow-Origin: * value, cause CSRF attacks are kind of impossible - there's no cookies to be forwarded and an attacker would have to somehow steal the token from some other origin's localStorage or something (which is impossible AFAIK) to be able to use it with an API call.
But then there's a plan to also enable session-based-authentication for the same app. This changes security constraints, however, because now a Access-Control-Allow-Origin: * value is def going to allow CSRF attacks to happen, cause even if the API call is invoked from attacker.com the cookie is going to be forwarded alongside the request and will allow the attacker to use the API with the browser user's cookie.
One solution to that problem (the best one IMO) is to calculate a Access-Control-Allow-Origin value differently depending on which authentication method is used. If there's an Authorization header, then Access-Control-Allow-Origin: * is fine, but if there's a session cookie header instead, then Access-Control-Allow-Origin will have to be more limited and only allow specific approved origins.
If needed I can do a PR for this, just let me know if you accept the premise and will merge it once it's done

I don't think that is feasable with CORS. Have you verified? Because either the Cookie header or the Authorization header would be included in the CORS OPTIONS request by the web browser, which is the point at which the Origin header would need to be responded to. This is because the pre-flight request (OPTIONS method) only contains a small subset of headers that are defined in the CORS spec.

@dougwilson Am I understanding you correctly - you're skeptical cause not all headers will be available for checking in that newly proposed origin calculation callback?
If so, I still think it's good enough, cause you can at least check some headers. The most important ones are the authorization & cookie headers, so as long as those are there, that's already very helpful.

I'm saying that AFAIK neither of those exact headers would be available, so your solution wouldn't work no matter if this was added or not. I'm asking if you verified that or not.

@dougwilson I haven't verified this, no, it was my assumption that you'd get these headers like you would with a normal request. With this new information coming to light, the feature request might not make any sense anymore

So the way CORS itself works (unrelated to this module) is that it's a "double-opt-in" system. The idea is that the client has no idea if the web server supports CORS and so it needs a way to probe that. What they did was create a "pre-flight" request whenever the original request was not a "safe request" -- that is, if the request is not one of the "simple verbs" and/or it contains auth-like headers like Cookie, Authortization, etc. So when that is the case, the web browser will first make that "pre-flight request" which is the veb OPTIONS and includes none of the body content and headers from the original request, and that request is where your decision needs to happen on what the Access-Control-Allow-Origin and etc. headers contain. If it doesn't contain the Origin then the web browser will fail and never make the original request.
Your example wants to decide between two cases that are both "non-safe" and thus they would both use a pre-flight, at which point, it would not be possible to tell which is which case but you need to supply a Access-Control-Allow-Origin to get the original request to be made, of course.
I hope that makes sense.

It does make sense, thank you


add support for more tabColor options and split RGB #1283

This commit renames TabColor into TabColorRGB (but keeps an alias
for backwards compatibility), as well as adds support for more tab color
options (Theme, Indexed and Tint).
Signed-off-by: Thomas Charbonnel<EMAIL_ADDRESS>Add support for more TabColor options
Description
The following changes were made:

Rename TabColor into TabColorRGB
Add TabColorTheme as well as an enum for all its values
Add TabColorIndexed as well as a default const for when it is unset
Add TabColorTint as well as a default const for when it is unset

Related Issue
#1283
Motivation and Context
The current version of Excelize only supports TabColor of type "RGB". Excel can set tab colors based on other dimensions (see here).
How Has This Been Tested
By adding tests to the existing tests for worksheet options in sheetpr_test.go
All the tests passed on Linux (Fedora 34, go1.16.13)
Types of changes

[ ] Docs change / refactoring / dependency upgrade
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

Regarding the documentation, i'm not sure if it's updated automatically when i changed the example tests, is that enough?

Regarding the documentation, i'm not sure if it's updated automatically when i changed the example tests, is that enough?

Yes, the example code will be verified by running them as tests.

Codecov Report

Merging #1285 (1dff95c) into master (ebea684) will increase coverage by 0.00%.
The diff coverage is 100.00%.

:exclamation: Current head 1dff95c differs from pull request most recent head 2cfc0e0. Consider uploading reports for the commit 2cfc0e0 to get more accurate results

@@           Coverage Diff           @@
##           master    #1285   +/-   ##
=======================================
  Coverage   98.47%   98.47%
=======================================
  Files          31       31
  Lines       23417    23450   +33
=======================================
+ Hits        23059    23092   +33
  Misses        239      239
  Partials      119      119

Flag
Coverage Δ

unittests
98.47% <100.00%> (+<0.01%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

sheetpr.go
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us.

LGTM, thanks for your contribution.

I have added a new exported ColorMappingType used for color transformation, using it instead of integer data type theme color enumeration, and using the ColorMappingType prefix instead of the enumeration with TabColorTheme, because these enumerations not only can be used for the tab theme color, which also can be used for the drawing markup language and themes, so I think we should rename them.

Okay looks good 👌
Thomas Charbonnel

On Aug 6, 2022, at 3:29 PM, xuri @.***> wrote:

I have added a new exported ColorMappingType used for color transformation, using it instead of integer data type theme color enumeration, and using the ColorMappingType prefix instead of the enumeration with TabColorTheme, because these enumerations not only can be used for the tab theme color, which also can be used for the drawing markup language and themes, so I think we should rename them.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.


add request time in default logger

I think it will be very useful if the default logger recored the request time, or the log cannot tell us anything about time.
before this changed, the default logger output looks like:
"GET http://<IP_ADDRESS>:3000/ HTTP/1.1" from <IP_ADDRESS>:53322 - 200 7B in 12.29µs

now it will be:
2016/12/15 - 11:40:12 "GET http://<IP_ADDRESS>:3000/ HTTP/1.1" from <IP_ADDRESS>:53322 - 200 7B in 12.29µs

@chenjie4255 thanks for bringing this to my attention, there has certainly been a regression here. I will fix it in a few minutes.

fixed by 18893e620db10241dd2066014f5b8ef60b2d413e

Just to clarify: The underlying log prints the timestamps. See https://golang.org/pkg/log/#pkg-constants


Enhancement: Generate QR Code For Invite Link

It would be awesome if the app could generate a QR code and display it in a dialog when clicking the 'Copy Invite Link' button. This would make it very easy to demo this application during a presentation or workshop so that a guest can scan the code and quickly join the session.

@recursivecodes - I've implemented a basic version of this in the following branch: https://github.com/aws-samples/amazon-ivs-real-time-basic-web-demo/pull/4.
Please check out the branch and let me know if it works for you use-case. Thanks!

Merged #4


Don't quote Object.prototype keys

Keys on Object.prototype, such as constructor and toString, are not
reserved words and don't need to be quoted when they are used as object
keys.

Coverage increased (+0.01%) to 98.438% when pulling 40ef3f193f8db9e4f376747b6652c3904789b76e on rhendric:not-reserved-words into 062a91045d60da933745d495a66df98ca07bc0a2 on blakeembrey:master.

LGTM. Could probably starting moving some of this library to ES2015 some time to support Object.create(null) or Set.


None of my relationships are working in 5.4

Laravel Version: 5.4 Release
PHP Version: 7.0.8
Database Driver & Version: MYSQL 5.7.13

Description:
BadMethodCallException
Error: Call to undefined method Illuminate\Database\Query\Builder::save()
As for the associate method and manually updating my model attributes, these are silent and appear to work as normal however they are not inserted into the database.
Steps To Reproduce:
Setup any one-to-many relationship and attempt to save.

 Using the save() on the wrong Model. Geezus


how to use IN clause with variable ?

Hello, a new question;) How to use multiple values in a variable ?  for example, i dont know how to use  IN clause with variable , this querie doesnt return datas :

Hello, we are very interested in this feature because we will be able to use your plugin with all our graphs. We have tried to use ${hostM:druid:json} and ${hostM:lucene} because we use an ElasticSearch datasource to build the variable hostM in Grafana.

Hello @philippesg ,
We do use filters like AND contract in (${contract:sqlstring}) and it's working fine, I'd be curious to see the generated query. Feel free to join on https://join.slack.com/t/grafadruid/shared_invite/zt-ryrcolyp-GQCiKcOF0WhnK~iqpkrqLA so I can try to help you in a more interactive way :)

FYI, I just tested that in latest version to be 100% sure:

Hello @jbguerraz, thank you for your help. it's worked. I just see that it's my data is in upper case and i must applied lower() function.


Full logout from mobile application

Question
I have a mobile application which uses an Authorization Code + PKCE flow against an identity server implementation. Everyhting works good either for login as well as for logout. My  client uses reference token configuration.
After logging out however there is still some time (few minutes) while the token remain valid.
My application has been assessed by a QSA which says that this practice does not adhere to OWASP recommendations

The user's session should be terminated after clicking the logout button. Please follow the session management recommendations from OWASP: https://cheatsheetseries.owasp.org/cheatsheets/Session_Management_Cheat_Sheet.html

Is there anything I can do  to mitigate this behaviour?

This may be related to the clock skew with JWT tokens:
https://github.com/IdentityServer/IdentityServer4.AccessTokenValidation/issues/90

In case of JWT there is indeed a clock skew - 5 mins by default.
Reference tokens should be immediate - unless the API does some caching of the validation result.

Thanks a lot for very quick answer.
I am getting a little bit confused. Should I call the revocation endpoint before logging out to invalidate the access and refresh token?
I have read about signout procedure in documentation however, as my application is a native mobile app developed with xamarin, I do not really understand if it fall under Front/Back channel or Session Management.
Could anyone give a bit of guidance on  the right process to follow?

Revoking tokens is a good idea during logout. This is unrelated to session management.

Many thanks!
I  have been able to solve the issue by invoking the revocation endpoint through IdentityModel. Here is a sample code:
var logoutUri = await _oidcClient.PrepareLogoutAsync( request );
using (var client = new HttpClient()) {
     var endSessionResponse = await client.GetAsync( logoutUri ).ConfigureAwait( false );
     var disco = await client.GetDiscoveryDocumentAsync( AuthConstants.AuthorityUri );
     var accessTokenRevocationRequest = new TokenRevocationRequest() {
         RequestUri = new Uri( disco.RevocationEndpoint ),
         ClientId = AuthConstants.ClientId,
         Token = accessToken,
         TokenTypeHint = AuthConstants.ACCESS_TOKEN_NAME
     };
     var accessTokenResponse = await client.RevokeTokenAsync( accessTokenRevocationRequest );
     var refreshTokenRevocationRequest = new TokenRevocationRequest() {
         RequestUri = new Uri( disco.RevocationEndpoint ),
         ClientId = AuthConstants.ClientId,
         Token = refreshToken,
         TokenTypeHint = AuthConstants.REFRESH_TOKEN_NAME
     };
     var refreshTokenResponse = await client.RevokeTokenAsync( refreshTokenRevocationRequest );
}

Also noted that when revoking the refresh token first also the access token is revoked.


feat: Add Social Media and GitHub Logos to Footer or Start Page

https://twitter.com/furter_samuel/status/1348738584296247305?s=20

don't know how to place it.. if you find a good way, just do it.

don't know how to place it.. if you find a good way, just do it.

#223

#223


Write a Java program to convert hours to weeks

Description
Write a Java program to convert hours to weeks
1 week = 7 days = 168 hours
Input  : 840
Output : 5

How to contribute

Save the solution in ConvertHoursToWeeks.java file
Add ConvertHoursToWeeks.java file in convert-hours-to-weeks folder

!assign
i want to contribute

Hey @yashgoel2211,
This issue has been assigned to you for more than 15 days.
We are unassigning you from this issue.
If you are still interested in contributing to this issue, please get auto-assigned on the issue by commenting !assign again.
Thanks for your contributions :)

!assign


[Fix-11903] [Worker] org.apache.dolphinscheduler.plugin.task.api.ProcessUtils:[66] - kill task failed

Fix #11903

Codecov Report

:exclamation: No coverage uploaded for pull request base (2.0.8-prepare@d31f961). Click here to learn what that means.
The diff coverage is n/a.

@@               Coverage Diff                @@
##             2.0.8-prepare   #13345   +/-   ##
================================================
  Coverage                 ?   20.71%
  Complexity               ?     1601
================================================
  Files                    ?      540
  Lines                    ?    23045
  Branches                 ?     2754
================================================
  Hits                     ?     4773
  Misses                   ?    17783
  Partials                 ?      489

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Detection of NOT operator while using stars

Hi,
I got a query like S1-5 and when setting star: true in a query, it gets a query like this: *S1*-*5*.
In my opinion it has to get a query like this: *S1-5*. Am I wrong or could this please get fixed?

Hi Alex
Hyphens are special characters in Sphinx's query syntax (the logical NOT), which is why Thinking Sphinx treats it as a query term separator. If you want to change the behaviour of this, you'll need to take a slightly different approach to wildcarding your query:
query = "S1-5"
query = ThinkingSphinx::Query.wildcard query, /[\p{Word}-]+/ # default is /\p{Word}+/
Model.search query

Or also, a slightly simpler approach: you can pass the regex pattern in as the :star option:
Model.search "S1-5", :star => /[\p{Word}-]+/

Hi Pat,
thank you very much, works great :)

Great to hear … and I only just noticed the title of this issue, where you'd already noted that the logical NOT was part of it. Sorry for explaining what you already knew!


HyperAnalysis End Members

In the end member tab, would it be possible to have the oprion of curve smoothing (similar to the plotting tool in the Sediment plugin) and also the option to choose between the continuum removed and the not-continuum removed displaying of data?

Should now be available via 7f2e422.


THRIFT-4666: Attempt to work around dlang client pool test failure

If someone who's dlang-knowledgeable could take a look at this I'd really appreciate it. I think this fixes the sporadic dlang build failures.


Rewrite security subsystem

Closes: https://github.com/metarhia/impress/issues/581

Landed in ef5200e, 92dc076 and d0a4951


python312Packages.dash: 2.17.1 -> 2.18.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python312Packages.dash is: Python framework for building analytical web applications
meta.homepage for python312Packages.dash is: https://dash.plot.ly/
meta.changelog for python312Packages.dash is: https://github.com/plotly/dash/blob/refs/tags/v2.18.0/CHANGELOG.md
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.18.0 with grep in /nix/store/szmqhlfjgshgvayvzf1hnnykwkrsfij3-python3.12-dash-2.18.0
found 2.18.0 in filename of file in /nix/store/szmqhlfjgshgvayvzf1hnnykwkrsfij3-python3.12-dash-2.18.0

Rebuild report (if merged into master) (click to expand)

9 total rebuild path(s)

9 package rebuild(s)

First fifty rebuilds by attrpath

das
picosnitch
python311Packages.dash
python311Packages.dash-bootstrap-components
python311Packages.meshtastic
python312Packages.dash
python312Packages.dash-bootstrap-components
python312Packages.meshtastic

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/szmqhlfjgshgvayvzf1hnnykwkrsfij3-python3.12-dash-2.18.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python312Packages.dash https://github.com/r-ryantm/nixpkgs/archive/ba7864c4ec08622ed41fde5065dbcb6b223d286e.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/ba7864c4ec08622ed41fde5065dbcb6b223d286e#python312Packages.dash

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/szmqhlfjgshgvayvzf1hnnykwkrsfij3-python3.12-dash-2.18.0
ls -la /nix/store/szmqhlfjgshgvayvzf1hnnykwkrsfij3-python3.12-dash-2.18.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

14 packages built:

das
das.dist
python311Packages.dash
python311Packages.dash-bootstrap-components
python311Packages.dash-bootstrap-components.dist
python311Packages.dash.dist
python311Packages.meshtastic
python311Packages.meshtastic.dist
python312Packages.dash
python312Packages.dash-bootstrap-components
python312Packages.dash-bootstrap-components.dist
python312Packages.dash.dist
python312Packages.meshtastic
python312Packages.meshtastic.dist

Maintainer pings
cc @antoinerg @TomaSajt for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

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

I'm away from home this week so I cannot test this now.
My main question would be whether or not we also need to update the yarn deps.


Added some compiler directives.

These will tell GUI programs which compiler to use.

Which language to use

and which is the root file.

@kanster, this will fix the issue were you have to have the thesis.tex open in the editor in-order to compile.

tested, all good


Navigate between screens with Compose: Android Basics with Compose

Hello!
In the OrderSummaryScreen function, the following starting value can't be added to the parameter onSendButtonClicked:
onSendButtonClicked: (String, String) -> Unit = {},
This is because {} does not take the two String parameters. I might have missed something, but I got an error message when building.
I solved the problem by replacing {} with {first, second -> Unit}.
I hope this message is helpful, and it's not just that I am too clumsy. :D
David
PS: Thank you for providing such a great material!

Hi @DavidBuzzTheStone
Thank you for reporting this issue that you experienced.
It is not you and the lesson is being updated so future students do not encounter it.
Glad you were able to figure out your own work around which is an important skill as a developer.
Please do let us know of any other issues you experience so we can address those too.
Glad you are enjoying the course!
Regards.


Reexport AbstractFFTs symbols?

It's kind of unfortunate to have to do using FFTW, AbstractFFTs to be able to get all of the FFT functionality you need, and it's less discoverable for users. It seems like it would be more convenient from a usability perspective to just do @reexport using AbstractFFTs here. Thoughts, @stevengj?

👍


Throws, explicits and trivial cleanups

Reviewers: @seelabs @nbougalis

Current coverage is 62.02%

Merging #1520 into develop will increase coverage by +0.03% as of baf6d61

@@            develop   #1520   diff @@
=======================================
  Files           888     888
  Stmts         62871   62869     -2
  Branches          0       0
  Methods           0       0
=======================================
+ Hit           38979   38994    +15
  Partial           0       0
+ Missed        23892   23875    -17

Review entire Coverage Diff as of baf6d61

Powered by Codecov. Updated on successful CI builds.

:+1:

:+1:

Merged as ed9f5639a8e02d1931dd45f13fa2fa21827f1a4d


⚠ GitLab Server has degraded performance

In 3cf518e, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 11352 ms

Resolved: GitLab Server performance has improved in 777f659.


Schedule for early 2020

Rooms are booked; now for speakers!
@KirstieJane, @sgibson91 and I got some ideas together yesterday:

Data management
Pytest
Visualisation - @KirstieJane
GitHub bots
LaTeX (need to work out a specific topic!) - @LouiseABowler
Conda/virtualenv environments
Binder - @sgibson91
Reproducible research in R (tidyverse?)

More ideas are of course very welcome; drop us a message or post below if you would either like to suggest a topic, or lead a session of your own. We'd really like to hear about specific projects at the Turing, so if you've got a reproducible project that you'd like to show off, or get some feedback on one you're currently working on, let us know!
All sessions are on Mondays, 12-1:15pm (more realistically, 12:15-1pm + clear-up time) unless marked.

Date
Room
Speaker
Topic
Notes

2nd March
David Blackwell
@sgibson91
Binder
Until 1pm

16th March
David Blackwell

30th March
David Blackwell

27th April
Lovelace

18th May
David Blackwell

@sgibson91, I've put you down for a session on the 2nd March based on what was on the board yesterday - is that still okay with you? And feel free to modify the title above, I've left it very vague!

Date's in my diary so that's all good. How about something like "From Zero to Binder: Creating reproducible computational environments in the cloud"?

Looks like a great selection already, how about code style/formatting and how to automatically enforce it (see e.g. https://pep8speaks.com and https://black.readthedocs.io/en/stable/index.html)?

Would it be possible to upload the tutorials/presentation from the reproducibles lunches as well? It'd be great to have them as a reference for later on.

Link to Binder tutorial: https://bit.ly/zero-to-binder-tutorial (Sorry, I can't edit the top post, at least not from my phone)


[BUG]cant change language

Describe the bug
no matter how I change the ISO code in usr_config.json
still can only show en manga

if possible, would you mind pasting the contents of the usr_config.json file here?

I have tried adding extra langauges (for example russian)

and i am able to view russian-translated manga.

if you are migrating from an old version of this app, make sure you edit the correct config file. It will be in "AppData/Roaming/mangadesk" folder.

if you are migrating from an old version of this app, make sure you edit the correct config file. It will be in "%AppData%/mangadesk" folder (on Windows).

oh thats the one, thanks for your reply so much, sorry for not making the issue clear


[Feature Request] el-carousel transform not in one direction when there are 2 el-carousel-item

Existing Component
是
Component Name
el-carousel
Description
when there are two el-carousel-item, it will transform from right to left, and then from left to right.Maybe always transform in one direction(from right to left) is much better?

In general, one direction is better.

Sorry, we have no plan to support this.

Could be supported by Merge Request ?


scorecard started reducing score for vulnerabilities in unrelated packages that aren't imported

Describe the bug
Scorecard used to show 0 vulnerabilities but it suddenly started showing 29 vulnerabilities (e.g. in packages that are not imported directly or indirectly).  This reduces the score by 10 points under "Vulnerabilities".
For example, a CBOR codec library isn't using any networking packages.  However, scorecard is showing it as being affected by issues innet/http such as GO-2022-0969.

Warn: Project is vulnerable to: GO-2022-0969

GO-2022-0969 is a bug in net/http, which isn't directly or indirectly imported by the CBOR codec.
Reproduction steps
Steps to reproduce the behavior:

view score at https://securityscorecards.dev/viewer/?uri=github.com/fxamacker/cbor/v2
expand "Vulnerabilities" (it shows 29 vulnerabilities like GO-2022-0969 which is in net/http)
view imports at https://pkg.go.dev/github.com/fxamacker/cbor/v2?tab=imports
notice that net/http is not imported by fxamacker/cbor or fxamacker/cbor/v2

Expected behavior
Report 0 vulnerabilities from net/http for projects that don't use or import anything that uses net/http.
Additional context
Screenshot

I didn't investigate remaining vulnerabilities after seeing the first one is a false alarm.

Thanks for the issue. We use osv-scanner as a library for vulnerability detection. Luckily they support call analysis for Go, so we should be able to ignore these uncalled vulnerabilities.

My hope is to have #3893 merged soon. There was an upstream issue that needed to be fixed. Based on the currently implementation, the output when running on the above CBOR library would be:
{
      "details": [
        "Warn: Project is vulnerable to: GO-2023-1840"
      ],
      "score": 9,
      "reason": "1 existing vulnerabilities detected",
      "name": "Vulnerabilities",
      "documentation": {
        "url": "https://github.com/ossf/scorecard/blob/main/docs/checks.md#vulnerabilities",
        "short": "Determines if the project has open, known unfixed vulnerabilities."
      }
    }

Where GO-2023-1840 affects:

before go1.19.10
from go1.20.0-0 before go1.20.5

Since your Go directive specifies 1.17 as the minimum required Go version, #3893 would still penalize it. I realize this isn't a 100% fair/accurate outcome for a library, compared to an application, but Scorecard doesn't have any way of handling the distinction currently.

@spencerschrock thanks for fast response and details! :smile:

A slightly different fix was used, for now we are filtering all Go stdlib vulns, in part due to the issues I mentioned above the go directive being a minimum version, not the version a binary will eventually use.
Note: this will still take some time (~10 days) for the change to propagate to the API/badge.


feat: add C implementation for math/base/special/ellipe

Resolves #772
Description
Added C implementation for @stdlib/math/base/special/ellipe.

What is the purpose of this pull request?

This pull request:

adds c implementation for @stdlib/math/base/special/ellipe

Related Issues

Does this pull request have any related issues?

This pull request:

resolves #772

Questions

Any questions for reviewers of this pull request?

No.
Other

Any other information relevant to this pull request? This may include screenshots, references, and/or implementation notes.

No.
Checklist

Please ensure the following tasks are completed before submitting this pull request.

☑ Read, understood, and followed the contributing guidelines.

@stdlib-js/reviewers

This is ready for review now. @Pranavchiku

@kgryte @Planeshifter @Pranavchiku Any other changes to be made ?

@gunjjoshi Thanks for working on this. We need to fix the indentation in a few files.

I have made the changes suggested by you, along with fixing the indentation.

Just an FYI: I had to push a patch due to bugs introduced in this PR: 89da7ae
For future PRs, be very careful about copying and pasting.

Definitely I'll keep that in mind. Apologies for this one. Thanks for correcting this.


Feature request: add hubble-ui support?

cilium-cli current supports enabling Hubble with cilium hubble enable. This deploys hubble-relay but not hubble-ui. What do you think about adding cilium hubble-ui enable (or cilium hubble enable --ui)?

Fixed by https://github.com/cilium/cilium-cli/pull/163


Increase speed

Speed increase is very important.
Make sure to remove anything that might reduce the app's speed.

Done.
As much as possible.


Hive table level white/black listing

https://github.com/linkedin/WhereHows/issues/1045
Current version of Whitehows has database level blacklisting/whitelisting. We have use cases where under certain database, we might have lots of hive tables that store intermediate/template results, we don't want them to appear on the datasets.
Proposed solution:
we could use ALTER HIVE table using Hive's TBLPROPERTIES (key/value pair)to indicate whether to include/exclude tables. TBLPROPERTIES('wherehows' = 'true/false')
logic:
if hive.table_white_list_enabled=True (disregard the value of hive.table_black_list_enabled)
Only tables with TBLPROPERTIES('wherehows' = 'true') will be INCLUDED
if hive.table_black_list_enabled=True
tables with TBLPROPERTIES('wherehows' = 'false') will be EXCLUDED
-- example of adding TBLPROPERTIES:
ALTER TABLE [table_name] SET TBLPROPERTIES ('wherehows' = 'true');
-- example of removing TBLPROPERTIES:
ALTER TABLE [table_name] UNSET TBLPROPERTIES ('wherehows' );
#example HIVE_METADATA_ETL.job
hive.table_white_list_enabled=True
by default hive.table_white_list_enabled and hive.table_black_list_enabled will be False, so process as current logic.

Coverage remained the same at 26.911% when pulling c3c66af59b29617aabc9e49da49d784672bf2ed6 on richardxin:master into 50f48c15ae0230b7934a2d203ca138193cdc4214 on linkedin:master.

Codecov Report

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

@@           Coverage Diff           @@
##           master    #1046   +/-   ##
=======================================
  Coverage   30.47%   30.47%
=======================================
  Files         178      178
  Lines        4647     4647
  Branches     1231     1231
=======================================
  Hits         1416     1416
  Misses       3231     3231

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 50f48c1...c3c66af. Read the comment docs.

Thanks for the contribution!


Rafael abstracts

Assume the figures for baeysian fwi are palceholders?

Assume the figures for baeysian fwi are palceholders?

Yes


Added migration quide

@avalanche123 is this PR acceptable? Easy migration guide could save some time to other users.

Hey @zhil! As per this project's README file:

This project is no longer actively maintained, please find one of the populate forks. Thanks!

So, I'm affraid it doesn't accept new PRs.

@straube well, this PR is actually adding detailed migration guide, would be nice to include it :)

I'm not one of the project owners. I just wanted to give you a heads up. :)


Spectator View Calibration App Crashing

I am trying to get the Spectator View Calibration App working with a WebCam (USE_OPENCV = TRUE).  When I run the app I crash after capturing the image in void CalibrationApp::TakeCalibrationPicture().  It looks like it's getting a memory corruption.  If I just return without calling HasChessBoard then the app continues to work and I can see the video.  The crash will happen trying to free the corners vector if a ChessBoard was not found or a crash (AccessViolation) trying to write the PNG if a chess board was found.
I also needed to change
HRESULT OpenCVFrameProvider::Initialize(ID3D11ShaderResourceView* colorSRV, ID3D11Texture2D* outputTexture)
and add
videoCapture->set(cv::CAP_PROP_FRAME_WIDTH, FRAME_WIDTH);
videoCapture->set(cv::CAP_PROP_FRAME_HEIGHT, FRAME_HEIGHT);
Otherwise it would always acquire at 640x480.  Not sure if this really means something else if going wrong.

I am using the chessboard image provided which is 6x4.  The app also crashes when I chessboard was not detected.

Are you building release or debug?
Can you provide a callstack

I am building Debug x64.
When I chessboard is visible I get
Exception thrown at 0x00007FF95B8F7310 (opencv_world310.dll) in Calibration.exe: 0xC0000005: Access violation reading location 0x00000299ABE7F000.
With this shortened stack
opencv_world310.dll!00007ff95b8f7310()	Unknown
opencv_world310.dll!00007ff95b8ecc2b()	Unknown
opencv_world310.dll!00007ff95b8ec8a6()	Unknown

Calibration.exe!CalibrationApp::TakeCalibrationPicture() Line 196	C++
Calibration.exe!CalibrationApp::TakeCalibrationPictureAtInterval::__l8::() Line 222	C++
Calibration.exe!std::_Invoker_functor::_Call<void (void) & __ptr64>(CalibrationApp::TakeCalibrationPictureAtInterval::__l8::void (void) & _Obj) Line 1377	C++
Calibration.exe!std::invoke<void (void) & __ptr64>(CalibrationApp::TakeCalibrationPictureAtInterval::__l8::void (void) & _Obj) Line 1445	C++
Calibration.exe!std::_Invoke_ret<void,void (void) & __ptr64>(std::_Forced<void,1> __formal, CalibrationApp::TakeCalibrationPictureAtInterval::__l8::void (void) & <_Vals_0>) Line 1462	C++
Calibration.exe!std::_Func_impl<void (void),std::allocator,void>::_Do_call() Line 214	C++
Calibration.exe!std::_Func_class::operator()() Line 280	C++

The cachedColorMap looks reasonable

  cachedColorMat	{flags=1124024344 dims=2 rows=720 ...}	cv::Mat
  flags<PHONE_NUMBER>	int
  dims	2	int
  rows	720	int
  cols	1280	int

I tried a few more things last night and it turns that that running the Release build works.  Not sure if it is expected that only the Release build would work or not but this was not obvious to me.
I do think that
videoCapture->set(cv::CAP_PROP_FRAME_WIDTH, FRAME_WIDTH);
videoCapture->set(cv::CAP_PROP_FRAME_HEIGHT, FRAME_HEIGHT);
needs to be added for the capture to work correctly, at least in my case.

Updated - enforce Release for the calibration application, and add cap properties for width and height.
This cap update does not appear to be guaranteed, since some cameras do have an upper bound (Open CV does not attempt to scale the frame up in this case)
In this case, the previous behavior will still be there: the update loop logs that the actual frame dimensions do not match the expected dimensions.

@ccifra Thanks, also trying to use a webcam (Logitech 920c 1920x1080) with Spectator View. I am using the updated version that includes your changes. However, I still get reported that the resolution is 640x480, and the image from the webcam is also showing in a strange way, suggesting an error in the image stream format. What camera and resolution are you using?


[BUG] Issue title...UserWarning: Grad strides do not match bucket view strides. This may indicate grad was not created according to the gradient layout contract, or that the param's strides changed since DDP was constructed.  This is not an error, but may impair performance.

Describe the bug
A clear and concise description of what the bug is.
when i execute the script the "dustrbuted_train.sh", i encounter warnings "UserWarning: Grad strides do not match bucket view strides. This may indicate grad was not created according to the gradient layout contract, or that the param's strides changed since DDP was constructed.  This is not an error, but may impair performance."
what is that?
To Reproduce
Steps to reproduce the behavior:

model = convnext.fb_in1k
dataset = imagenet21k_winter ( the number of class : 19167)

Expected behavior
A clear and concise description of what you expected to happen.

Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

OS: Ubuntu 22.04.3 LTS
This repository version [e.g. pip 0.3.1 or commit ref]
PyTorch version w/ CUDA/cuDNN  [e.g. from conda list, 1.7.0  py3.8_cuda11.0.221_cudnn8.0.3_0]

Additional context
Add any other context about the problem here.

@TheDarkKnight-21th it's expected, not ideal perhaps but not an issue, and altering the ops used to remove the warning will in fact make performance lower in most scenarious based on past analysis.


Add heroku access to log buckets

Related https://github.com/dandi/dandi-archive/issues/1424

@satra @yarikoptic When looking into this PR I realized master isn't building cleanly anymore, it looks like a regular terraform plan wants to make a lot of changes to S3 buckets. Is it possible manual changes were made through the GUI recently?

I did no changes in GUI any time not only recently.

neither did i.


Capslock handling in text box

Fixes https://github.com/ppy/osu/issues/3120
Also, how would I write a test for this?
(There is a chance that this might not compile since I do not have a working compiler)

I will try to do it for windows also.

On windows everything works perfectly as far as I tested (spamming with/without caps lock in framework textbox tests).

Oh ok! What about on linux? (Not that I can do anything about it since I do not have a linux machine)

Also, in that case, should I make CapslockDown a MacOSGameWindow specific thing?

Ok I will close it.
@ekrctb Upon inspecting the MacOSKeyMap.cs file in the openTk repo, I realize certain keys (including capslock) is not handled. Is there a reason for this?

I know nothing about MacOS APIs and I can't test it because I don't have a Mac. So I don't know.
Please setup a working compiler and test yourself if you going to fix the issue.

How would I test osu framework with a custom opentk patch?

OpenTK is referenced like https://github.com/ppy/osu-framework/blob/c1a9cd47551171619d825db2bb667d935046c3c1/osu.Framework/osu.Framework.csproj#L27. You could change this to your local version of OpenTK. Please consult google or other people for general problems.

In macOS, modifier keys (such as caps lock) trigger a flagsChanged event rather than a keyDown/keyUp, which is why OpenTK does not detect it, and why there is a hack in MacOSGameWindow.
If you're going to "fix" it, either add the missing case to the switch in MacOSGameWindow, or correctly implement flagsChanged in OpenTK to handle all the modifier cases.  Honestly I would just do the quick fix in osu!framework, as CocoaNativeWindow needs a serious rewrite which will likely happen with OpenTK 4.0.

Does this do the job? https://github.com/ppy/opentk/pull/26

@swoolcock It is based on the idea that characters take modifier keys into account (while the old charactersIgnoreModifier ignores modifier keys


Deposit of Ethereum not shown, not deposited

I just deposited 3 Ether which has been taken off from my ledger and on etherscan it shows "success" however, on Loopring I do not see anything. hash: 0x0ce66fe0bcdd146e1a392e5659dc69119ff5be198e2acdd28be79b54bd8798c3
Please help. Thx,
Oliver

I'm seeing/having the same issue for a deposit I made about 15 minutes ago.

After 30 confirmations you should be able to see your balance into
Loopring. It is not there yet?
El 14/7/20 a las 12:43 p. m., olivero09 escribió:

I just deposited 3 Ether which has been taken off from my ledger and
on etherscan it shows "success" however, on Loopring I do not see
anything. hash:
0x0ce66fe0bcdd146e1a392e5659dc69119ff5be198e2acdd28be79b54bd8798c3
Please help. Thx,
Oliver
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Loopring/dexwebapp/issues/178, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADMEOQ2CRMRM534IESDLOTR3SDK3ANCNFSM4OZWJDVA.

After 30 confirmations you should be able to see your balance into Loopring. It is not there yet? El 14/7/20 a las 12:43 p. m., olivero09 escribió:
…
I just deposited 3 Ether which has been taken off from my ledger and on etherscan it shows "success" however, on Loopring I do not see anything. hash: 0x0ce66fe0bcdd146e1a392e5659dc69119ff5be198e2acdd28be79b54bd8798c3 Please help. Thx, Oliver — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#178>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AADMEOQ2CRMRM534IESDLOTR3SDK3ANCNFSM4OZWJDVA.

I am afraid it is not yet shown, usually I see the deposit and I can see the countdown of the confirmation but with my last deposit nothing is shown.

The deposit appeared just now. I closed the brave browser logged back into my account and the deposit appeared. I closed the tap and logged in and out before but it seems I had to completely close the browser to get the result or it was a coincidence.


If one key is having other key value as a prefix, getManyBytes function is returning wrong values

Sample test case :
package main_test
import (
"testing"
"github.com/tidwall/gjson"
"github.com/stretchr/testify/assert"
)
func Test_GJson(t *testing.T) {
result := gjson.GetManyBytes([]byte(sampleJSON), "name", "name1", "address", "addressDetails")
expectedValues := []string{"FirstName", "FirstName1", "address1", "address2"}
assert.Equal(t, expectedValues[0], result[0].String())
assert.Equal(t, expectedValues[1], result[1].String())
assert.Equal(t, expectedValues[2], result[2].String())
assert.Equal(t, expectedValues[3], result[3].String())
}
const sampleJSON = {                "name": "FirstName",                "name1": "FirstName1",                "address": "address1",                "addressDetails": "address2", }

Explanation  : This test case is failing. It gives "FirstName" as a value for both keys "name" and "name1". If one key is present as a prefix in other key, GetManyBytes function is returning wrong values.

@Poorva17 Thank you very much for reporting this issue.


[Animated] Animating a View height to zero causes craziness

I'm using a spring animation on the height property of a view. How do you make the spring "not spring", aka, not go to a negative number, when it reaches zero?

There are configuration properties on interpolate in the Animated library called extrapolateLeft and extrapolateRight that can be set to "clamp". Something like this:
// in your event handler to trigger the spring
Animated.spring(this._animValue, springConfig).start();

// in render()
<Animated.View style={{
  someProp: this._animValue.interpolate({
    extrapolateRight: 'clamp',
    <other fields>
  })
}} />

https://github.com/facebook/react-native/blob/master/Libraries/Animated/src/Interpolation.js#L106
In the future, please use resources like Stack Overflow for questions since we try to use the GitHub issue tracker primarily for core bug fixes.


Display issue on referential choise

Referencial choises: Units, UD, Relative Z-sum and Quantitative Z-sum are displayed many times in my case. Almost twice my screen size. I can't take a screenshot :(

Oh this is a weird bug.
Do you have any log or use case so I can try to reproduce them ?

It happends when I reopen an account on the same CuteCoin session.

https://github.com/ucoin-io/cutecoin/commit/12e835b70d3b16a5061705c1c0fa5977e349ddea


Implement 'hotel open'

This will open http://localhost:2000 in your local browser. It respects the config file, but always uses localhost (in case you set host to <IP_ADDRESS>).
osx, linux and windows supported.

As tempting as it is to accept this PR, I want to keep the CLI to a minimum.
Sorry.


Is it possible to choose the command to execute when connecting to a task?

I'm new to ECS and was looking for a way to help manage my clusters.  I recently saw a k8s TUI that looked really good and thought that something like that for ECS would be handy.  Then I discovered this project and it looks pretty good to me.
One thing I would like is the ability to choose a different "shell" when connecting to a task.  It looks like the project defaults to "sh" but some of my containers use "bash", which is much more friendly.

Thank you for reaching out and sharing your interest in this project! I'm glad to hear that you find the tool useful as you navigate ECS.
Regarding your request to select different shells when connecting to a task, it's a great suggestion. Currently, the project defaults to using "sh", but I understand the need for flexibility. While this functionality isn't available in our current implementation, I agree that it would be a valuable feature to include. I will look into how we might adjust the implementation to accommodate this capability in a future update.
Thank you again for your input. it's essential for improving the tool, and I appreciate your feedback!

This should be fixed now that my change from #154 is merged in

I just checked out the new version and the "cmd" flag works great!
One thing that might make it better is to include the option to set it via and environment variable, similar to the AWS_PROFILE flag.  It's not a blocker but it would make it easier to utilize.
Again, thanks for all your work.  I really like this utility and use it all the time.  :+1:


Add warning in knife node from file

Description
Unlike knife environment from file since this functionality doesn't update,
there should be a confirmation warning before overriding a node in knife node from file if the node exists.

Can you be more specific about what you think is or isn't happening? All of the from file commands are pretty literally just PUTing whatever data you give which does a full object overwrite.

@coderanger I could have sworn that at more than one time in past after running node from file on a node that was existing, all the node attributes outside of the template were erased and the chef-client runs were failing.
In replying to your question, I tried to replicate again the issue I saw on a different type of server but I am not able to anymore.
I am closing this PR for now and will open it again if and when I encounter the issue.


Corrected Powershell profiles location and added test for the same.

Resolve #1190
For windows :
AllUsersAllHosts = $PSHome\profile.ps1
AllUsersCurrentHost = $PSHome\PowerShell_profile.ps1
CurrentUserAllHosts = ~\Documents\PowerShell\profile.ps1
CurrentUserCurrentHost = ~\Documents\PowerShell\PowerShell_profile.ps1
For Linux :
AllUsersAllHosts = $PSHOME/profile.ps1
AAllUsersCurrentHost = $PSHOME/PowerShell_profile.ps1
PCurrentUserCurrentHost =
/home/user/.config/powershell/PowerShell_profile.ps1
$CurrentUserAllHosts = /home/user/.config/powershell/profile.ps1

Hi @charub, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;

Hi @charub, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Charu Bassi). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

@charub the discussion conclusion is that we don't need to change the profile file names for powershell core. Please see the comments in #1190 for details.
Give the discussion conclusion, do you think we need to change anything to resolve issue #1190 ?

@daxian-dbw : As per the discussion we want to keep the same profile file names. So this change is no longer required.