Add nothing sentinel value

Rendering nothing to a Part will cause it to clear. Useful to make sure that elements with ShadowRoots can trigger <slot></slot> fallback content:
import {html, nothing} from 'lit-html';

html`<my-element>${condition ? 'something' : nothing}</my-element>`

I was always fond of this solution. I'll take a look at implementing this

I like it, might make it much clearer than using '' to indicate that nothing should be rendered.
Just a thought about naming. Might empty be a better name? It'll match the :empty CSS selector, as well as saving two keystrokes 😉
I also sometimes see important constants being uppercased (e.g. EMPTY/NOTHING) in javascript, is that common or helpful for distinguishing between local variables and imported constants?

Thumbs up for nothing as it sounds reasonable. Regarding the empty, it sounds like adjective, not a noun (although, it can be probably used as a noun, too).

@web-padawan Good point. Might have made less sense than I thought. At least I didn't propose a verb 😄

this is pretty much the null object pattern, i'd disagree with it being called nothing though.
usually with this pattern, you'd call it NullSomething, such as NullPart

Note that there are two different sentinels that are "null"-like with different purposes.

noChange - which signals the renderer to make no changes to the current content and effectively do nothing
nothing - which signals the renderer to actively clear the current rendered content and render nothing.

These naming schemes are more descriptive than null, and they combine well with the render verb: render(nothing, somewhere), render(noChange, somewhere).

The name looks like psuedo-code to me, I still think there are better, more technical names that are used in other languages and libraries.
Ideally IMO we should just have a null part, surely. If we want to render nothing, we can then pass that static part to avoid creating a new part every time and to support any kind of cache behaviour. This is a pretty common pattern across all languages, as it saves you having to do null checks.
Wanting to avoid rendering (noChange) sounds like a separate feature request to me.

Passing new values should not create new parts (with the exception of nested templates or lists). There already exists a part for each dynamic value in a template, and passing new values directs these parts to render something. You don't have to create a new part to render a new value, and with the current implementation of lit-html there is no use for something like a static part.
The nothing value is a static sentinel that signals parts to clear their content, which is pretty much exactly what you suggest.
noChange is not a feature request, it is already supported.

Couldn't we just use null and use that to trigger the clear? Seems like that would be better DX?

@ruphin Of course, it is to render "nothing", i.e. a null part. The equivalent of render(null, somewhere) if we supported that.
I suggest we go with NullPart, if anything, or something similar rather than a plain english name.

It is reasonable to use null instead of a custom sentinel object.
My main concern with it is that null will have different behaviour in NodePart and AttributePart:
render(html`<div attr=${ null }>${ null }</div>`, document.body)
//

Having a distinct sentinel for this case makes the interaction more explicit, and avoids confusion. (Why does null remove this thing but not that thing?)

On the other hand, using null simplifies the implementation, and it has a little resonance with how innerHTML handles null:
document.getElementById('container').innerHTML = null;
//

At this moment null and undefined values in NodeParts render as '', so there is already some dissimilarity in how NodePart and AttributePart handle null. We could change this behaviour to clear instead of rendering '', but I suspect this will cause a slowdown in general use cases, as it will unnecessarily destroy and recreate text nodes when toggling between primitives and undefined / null. We could leave the current behaviour for undefined, and change null to clear instead, but the more you think about it the more it seems arbitrary and vague.
I will make a PR that implements nothing as originally suggested, and we can discuss the merit of different options based on that.

Note:

document.getElementById('container').innerHTML = null;
//

document.getElementById('container').innerHTML = undefined;
//
undefined

We already have an explicit check for null, so using null as a marker will not add any additional null checks whatsoever. The only performance impact is caused by inadvertently removing and creating TextNodes when switching between rendering null and other primitives, where the current implementation just changes the TextNode content.
Adding a sentinel like nothing has almost no impact to performance, since it will be the last value we test for in setValue, and almost all common rendered values will trigger before hitting that point in the chain. It does add more additional bytes to the total payload.

There is plenty precedent in the public API of this library where we use English language: removeNodes, noChange, createMarker, render. I don't see why this is suddenly a problem.

Where do we already check for null parts (or template results)? Out of interest.
Introducing the null object pattern generally results in simpler implementation but also becomes useful for removing the need for null checks which definitely do have a performance overhead. If we already need such checks, though, sure, it makes little difference but those are generally the two reasons for such a pattern.
nothing is a bit of an unconventional name for such a thing. Your examples all describe what they are, nothing describes nothing at all, it isn't clear what it is or why it even exists out of context. Context-dependant names aren't a great thing, it deserves a more descriptive name.
I'll stick with my suggestion for now, NullPart.

One phrasing I'd like to clear up is that we don't have "null Parts". Parts exist, usually, as a result of expressions, and value flow into them. So we just have null values, and with this proposal a nothing value.
I like nothing because it's clear. The DOM handles null and undefined in inconsistent ways. We could try to "fix" the DOM and normalize that behavior, but it'd be our opinion against the specs and implementations.

It holds no descriptive value, has no meaning even in context.
render(nothing)
What is nothing? Is it our variable, an imported one, a part, a template, it isn't clear at all. I really don't think we should be introducing names into the API which are so non-descriptive, it makes code a lot more difficult to follow. It looks simpler.. as a word, but it has very little meaning.
It would be nice if there was a slightly more descriptive name that could be agreed on.

We check for null when setting values that are primitives (strings, numbers, null, undefined, etc. Basically things that aren't objects or functions) because we render '' instead of null and undefined.

As mentioned in the issue, this feature is only to solve a very specific issue regarding fallback content in slots. Most users will never see or use nothing, and can continue using whatever they are using now, which can be null or '' or whatever they like. For the users that actually need it, it will be descriptive enough.

What do you mean with a new empty template?

if (foo) {
  return html``;
}

new template result per render, its something i've already seen people doing quite often. perf benefits would be achieved by using the null obj pattern and having a static empty result (which is essentially the same thing but instantiated once and used by ref).


Add 'i t' for inside tags

I just :+1: #418 and thought I'd give a go at actually implementing it.
Hopefully code is straight forward.
It's not exactly like vim right now, if you're not inside a tag it won't do anything, vim is smart enough to move to the next tag and then do the action. However that seems to be how the other motions in atom are done right now anyway...

@neiled Funny, I was just about to take another crack at this myself. I started a while back, but got all hung up on handling cases where < and > are used inside tags, but aren't tags themselves. But honestly, I'd much rather have the feature since it works in most cases than worry about that edge case.

Also, super great profile picture :+1:

Yeah, nice idea @neiled. It seems like this covers the majority of the use cases for i t. We can always make it fancier later.

Great! This might close #418 too perhaps @maxbrunsfeld?

And thanks @murphyrandle :wink:


Suppress advertising timeout warning when successful

Issue the following log only when there is an actual failure (err is non-zero).
BT_WARN("Failed to stop advertising: %d", err);

Fixes #53643.

Read the commit guidelines and fix the compliance issues.

Read the commit guidelines and fix the compliance issues.

@box-section ping


⚠ ClamAV has degraded performance

In 03dfaf8, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 75 ms

Resolved: ClamAV performance has improved in 032f3ec.


remove ghcr.io/opiproject/opi-sztp-server package

go to https://github.com/orgs/opiproject/packages/container/opi-sztp-server/settings and remove it

hi @glimchb , I am third year student of computer science stream of  IIT(BHU) Varanasi. I have basic experience of Go and shell.
I am interested in this project to contribute to per LFX mentee project. I am excited to be a part of this project.
I'd like to contribute but I'm unsure where to start.Can you help to guide a little?
Also for this issue #364 the link is not accessible to it .


Failure to build on Ubuntu - "undefined reference to  `g_time_zone_new_identifier'"

I have run
sudo apt-get install libayatana-appindicator3-dev

while building I am getting the following error:
/snap/flutter/current/usr/bin/ld: /lib/x86_64-linux-gnu/libayatana-ido3-0.4.so.0: undefined reference to
`g_time_zone_new_identifier'  ⬅ HERE

[  +10 ms] clang: error: linker command failed with exit code 1 (use -v to see invocation)
[  +62 ms] ninja: build stopped: subcommand failed.
[  +56 ms] Exception: Build process failed
Exited (sigterm)

cat etc/os-release
PRETTY_NAME="Ubuntu 22.04.1 LTS"
NAME="Ubuntu"
VERSION_ID="22.04"
VERSION="22.04.1 LTS (Jammy Jellyfish)"
VERSION_CODENAME=jammy
ID=ubuntu
ID_LIKE=debian
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
UBUNTU_CODENAME=jammy

Flutter Doctor -v
[✓] Flutter (Channel stable, 3.3.10, on Ubuntu 22.04.1 LTS 5.15.0-58-generic, locale en_US.UTF-8)
    • Flutter version 3.3.10 on channel stable at /home/gad/snap/flutter/common/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 135454af32 (5 weeks ago), 2022-12-15 07:36:55 -0800
    • Engine revision 3316dd8728
    • Dart version 2.18.6
    • DevTools version 2.15.0

[✗] Android toolchain - develop for Android devices
    ✗ Unable to locate Android SDK.
      Install Android Studio from: https://developer.android.com/studio/index.html
      On first launch it will assist you in installing the Android SDK components.
      (or visit https://flutter.dev/docs/get-started/install/linux#android-setup for detailed instructions).
      If the Android SDK has been installed to a custom location, please use
      `flutter config --android-sdk` to update to that location.

[✓] Chrome - develop for the web
    • Chrome at google-chrome

[✓] Linux toolchain - develop for Linux desktop
    • clang version 10.0.0-4ubuntu1
    • cmake version 3.16.3
    • ninja version 1.10.0
    • pkg-config version 0.29.1

[!] Android Studio (not installed)
    • Android Studio not found; download from https://developer.android.com/studio/index.html
      (or visit https://flutter.dev/docs/get-started/install/linux#android-setup for detailed instructions).

[✓] VS Code (version 1.74.3)
    • VS Code at /usr/share/code
    • Flutter extension version 3.57.20221221

[✓] Connected device (2 available)
    • Linux (desktop) • linux  • linux-x64      • Ubuntu 22.04.1 LTS 5.15.0-58-generic
    • Chrome (web)    • chrome • web-javascript • Google Chrome 109.0.5414.74

[✓] HTTP Host Availability
    • All required HTTP hosts are available

It looks like the glib library is missing, you can try installing it first.
sudo apt-get install libgtk2.0-dev

See: https://askubuntu.com/questions/58321/how-do-i-install-glib

Installed but the error persists:
[+1100 ms] [4/5] Linking CXX executable intermediates_do_not_run/ui
[        ] FAILED: intermediates_do_not_run/ui
[  +12 ms] : && /snap/flutter/current/usr/bin/clang++  -g  -B/snap/flutter/current/usr/lib/gcc/x86_64-linux-gnu/9 -B/snap/flutter/current/usr/lib/x86_64-linux-gnu -B/snap/flutter/current/lib/x86_64-linux-gnu -B/snap/flutter/current/usr/lib/ -L/snap/flutter/current/usr/lib/gcc/x86_64-linux-gnu/9 -L/snap/flutter/current/usr/lib/x86_64-linux-gnu -L/snap/flutter/current/lib/x86_64-linux-gnu -L/snap/flutter/current/usr/lib/ -lblkid -lgcrypt -llzma -llz4 -lgpg-error -luuid -lpthread -ldl -lepoxy CMakeFiles/ui.dir/main.cc.o CMakeFiles/ui.dir/my_application.cc.o CMakeFiles/ui.dir/flutter/generated_plugin_registrant.cc.o  -o intermediates_do_not_run/ui  -Wl,-rpath,/home/gad/Documents/mtg.studio.project/frontend/build/linux/x64/debug/plugins/screen_retriever:/home/gad/Documents/mtg.studio.project/frontend/build/linux/x64/debug/plugins/sentry_flutter:/home/gad/Documents/mtg.studio.project/frontend/build/linux/x64/debug/plugins/tray_manager:/home/gad/Documents/mtg.studio.project/frontend/build/linux/x64/debug/plugins/url_launcher_linux:/home/gad/Documents/mtg.studio.project/frontend/build/linux/x64/debug/plugins/window_manager:/home/gad/Documents/mtg.studio.project/frontend/linux/flutter/ephemeral:  plugins/screen_retriever/libscreen_retriever_plugin.so  plugins/sentry_flutter/libsentry_flutter_plugin.so  plugins/tray_manager/libtray_manager_plugin.so  plugins/url_launcher_linux/liburl_launcher_linux_plugin.so  plugins/window_manager/libwindow_manager_plugin.so  /home/gad/Documents/mtg.studio.project/frontend/linux/flutter/ephemeral/libflutter_linux_gtk.so  /snap/flutter/current/usr/lib/x86_64-linux-gnu/libgtk-3.so  /snap/flutter/current/usr/lib/x86_64-linux-gnu/libgdk-3.so  /snap/flutter/current/usr/lib/x86_64-linux-gnu/libpangocairo-1.0.so  /snap/flutter/current/usr/lib/x86_64-linux-gnu/libpango-1.0.so  /snap/flutter/current/usr/lib/x86_64-linux-gnu/libharfbuzz.so  /snap/flutter/current/usr/lib/x86_64-linux-gnu/libatk-1.0.so  /snap/flutter/current/usr/lib/x86_64-linux-gnu/libcairo-gobject.so  /snap/flutter/current/usr/lib/x86_64-linux-gnu/libcairo.so  /snap/flutter/current/usr/lib/x86_64-linux-gnu/libgdk_pixbuf-2.0.so  /snap/flutter/current/usr/lib/x86_64-linux-gnu/libgio-2.0.so  /snap/flutter/current/usr/lib/x86_64-linux-gnu/libgobject-2.0.so  /snap/flutter/current/usr/lib/x86_64-linux-gnu/libglib-2.0.so && :
[        ] /snap/flutter/current/usr/bin/ld: /lib/x86_64-linux-gnu/libayatana-ido3-0.4.so.0: undefined reference to `g_time_zone_new_identifier'
[   +1 ms] ninja: build stopped: subcommand failed.
[        ] clang: error: linker command failed with exit code 1 (use -v to see invocation)
[+26631 ms] Exception: Build process failed
Exited (sigterm)

Can it be that I am using libayatana-ido3-0.4 and the installation manual asked for libayatana-ido3-0.1?

I'm experiencing the same issue. Today I tried to build an example app for my Linux box (Kubuntu 22.04 LTS) and got the same error:
[        ] /snap/flutter/current/usr/bin/ld: /lib/x86_64-linux-gnu/libayatana-ido3-0.4.so.0: undefined reference to g_time_zone_new_identifier'
[        ] clang: error: linker command failed with exit code 1 (use -v to see invocation)
[   +1 ms] ninja: build stopped: subcommand failed.
[   +8 ms] Building Linux application... (completed in 3,0s)
[+5094 ms] Exception: Build process failed
`


Error executing lua ..../.local/share/nvim/lazy/nvim-cmp/lua/cmp/core.lua:143: bad argument #1 to 'pairs' (table expect ed, got boolean)

FAQ

[X] I have checked the FAQ and it didn't resolve my problem.

Announcement

[X] I have checked Breaking change announcement.

Minimal reproducible full config
-- nvimcmp.lua
local lazypath = vim.fn.stdpath("data") .. "/lazy/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
	vim.fn.system({
		"git",
		"clone",
		"--filter=blob:none",
		"https://github.com/folke/lazy.nvim.git",
		"--branch=stable",
		lazypath,
	})
end
vim.opt.rtp:prepend(lazypath)

require("lazy").setup({
    "hrsh7th/nvim-cmp",
    event = { "InsertEnter" },
    enabled = true,
    dependencies = {
        "L3MON4D3/LuaSnip",
        "saadparwaiz1/cmp_luasnip",
        "rafamadriz/friendly-snippets",
        "hrsh7th/cmp-nvim-lsp",
        "hrsh7th/cmp-nvim-lua",
        "hrsh7th/cmp-buffer",
        "hrsh7th/cmp-path",
        "hrsh7th/cmp-cmdline",
        "hrsh7th/cmp-emoji",
    },
config = function()
    local luasnip = require "luasnip"
    require('luasnip.loaders.from_vscode').lazy_load()
    luasnip.config.setup {}
    local cmp = require "cmp"
    local select = { behavior = cmp.SelectBehavior.Select }
    local mapping = {
        ["<C-n>"] = cmp.select_next_item(select),
        ["<C-p>"] = cmp.select_prev_item(select),
        ["<C-d>"] = cmp.mapping.scroll_docs(-4),
        ["<C-f>"] = cmp.mapping.scroll_docs(4),
        ["<C-y>"] = cmp.mapping.confirm({ select = false }),
        ["<C-Space>"] = cmp.mapping.complete(),
    }

    cmp.setup({
        completion = {
            completeopt = "menu,menuone,noinsert,noselect",
        },
        snippet = {
            expand = function(args)
                luasnip.lsp_expand(args.body)
            end,
        },
        mapping = mapping,
        sources = {
            { name = "nvim_lsp" },
            { name = "luasnip" },
            { name = "buffer" },
            { name = "path" },
            { name = "emoji" },
        },
    })
end
})

Description
I currently in middle of nuke and reconfig my neovim migrate from packer to lazy. then setup nvim-cmp first time is no error as expected, but after config further and restart go in command line or insert mode I got this error.
E5108: Error executing lua /home/kia/.local/share/nvim/lazy/nvim-cmp/lua/cmp/core.lua:143: bad argument #1 to 'pairs' (table expect
ed, got boolean)
stack traceback:
        [C]: in function 'pairs'
        /home/kia/.local/share/nvim/lazy/nvim-cmp/lua/cmp/core.lua:143: in function 'prepare'
        /home/kia/.local/share/nvim/lazy/nvim-cmp/lua/cmp/init.lua:328: in function </home/kia/.local/share/nvim/lazy/nvim-cmp/lua/
cmp/init.lua:324>
        ...ocal/share/nvim/lazy/nvim-cmp/lua/cmp/utils/feedkeys.lua:47: in function <...ocal/share/nvim/lazy/nvim-cmp/lua/cmp/utils
/feedkeys.lua:45>

Steps to reproduce

make minimal config form my  plugin  module nvimcmp.lua
open with nvim -u nvimcmp.lua
enter insert or command mode
got error

Expected behavior
no error.  running normal as 2 hour ago.
Actual behavior
E5108: Error executing lua /home/kia/.local/share/nvim/lazy/nvim-cmp/lua/cmp/core.lua:143: bad argument #1 to 'pairs' (table expect
ed, got boolean)
stack traceback:
        [C]: in function 'pairs'
        /home/kia/.local/share/nvim/lazy/nvim-cmp/lua/cmp/core.lua:143: in function 'prepare'
        /home/kia/.local/share/nvim/lazy/nvim-cmp/lua/cmp/init.lua:328: in function </home/kia/.local/share/nvim/lazy/nvim-cmp/lua/
cmp/init.lua:324>
        ...ocal/share/nvim/lazy/nvim-cmp/lua/cmp/utils/feedkeys.lua:47: in function <...ocal/share/nvim/lazy/nvim-cmp/lua/cmp/utils
/feedkeys.lua:45>

Additional context
NVIM v0.10.0-dev-1295+g1f864139b
I have try nuke my config and it still happen.

your config:
        ["<C-n>"] = cmp.select_next_item(select),
        ["<C-p>"] = cmp.select_prev_item(select),

correct config:
        ["<C-n>"] = cmp.mapping.select_next_item(select),
        ["<C-p>"] = cmp.mapping.select_prev_item(select),


Open vscode:// links on the extension page

Linkng to a vscode://publisher.extension/... URI in a readme file seems like a nice way of quickly demonstrating the user some features of your extension. But currently, such links do not seem to work; clicking them does nothing.

Please share proper steps to reproduce this issue (and fill in the issue template)
/needsMoreInfo


FFMmpeg::open() has 'media type' bug with MP3s (image metadata)

Q
A

Bug?
yes

New Feature?
no

Version Used
ecd50cd5f1f718223d5f698bdfc863421713dd82

FFmpeg Version
FFmpeg 2.8.10-0ubuntu<IP_ADDRESS>

OS
ElementaryOS Loki (Ubuntu 16.04)

Actual Behavior
calling $ffmpeg->open('audio.mp3'); returns an instance of FFMpeg\Media\Video instead of FFMpeg\Media\Audio when the mp3 file has an embedded image as part of it's metadata (id3 tag "cover image")
Expected Behavior
calling $ffmpeg->open('audio.mp3'); should return an instance of FFMpeg\Media\Audio object.
Steps to Reproduce
call $ffmpeg->open('audio.mp3'); using an audio file which has an id3 tag image aka "cover image" as part of its metadata
Possible Solutions
Not certain...
Better checks need to be made to differentiate video from mp3

can reproduce it.

Would the solution be to introduce an optional flag to allow forcing of the object type one wants returned?
Or would it be better to try detect audio more accurately?

I think it would be better to autodetect it based on the extension of the file. However, if the user is not rigorous about this, it may generate errors.
@jens1o, what do you think about auto-detection?

It's a orientation, but we shouldn't rely on it. May I create a pr with some checking?

If you can, please do, thanks a lot!

I'll do it as soon as the others are merged. Otherwise, the sums of pr aren't controllable anymore. Can you assign me to this issue in any way?

Unfortunately no, I tried but Github doesn't allow me to add assignees.

Is there any fix for this yet?

I'm happy to see a pr, I didn't had the time to invest it, and I also think it would mean a major break, so it is considered for 1.x for now.

There is a fix for this problem, I'd happy to see that more people would test this to see if there is any regression(while I couldn't find one).
See https://github.com/PHP-FFMpeg/PHP-FFMpeg/issues/402#issuecomment-464319454 for the patch.

Any possibility of movement on this one? It is a pretty significant bug that completely breaks working with audio files that have cover art meta data.
Is see @vibhavsinha proposed a fix that has not been touched in 2 years.
I would be willing to help with the issue if needed.

Any possibility of movement on this one? It is a pretty significant bug that completely breaks working with audio files that have cover art meta data.
Is see @vibhavsinha proposed a fix that has not been touched in 2 years.
I would be willing to help with the issue if needed.

I used his fix in a site and it worked. I did have to add a missing end brace per my comment on the pull request, but once I did that, it worked. I am back here because almost a year later, I ran into the same issue on a new site and had forgotten about the problem.

I have rebased my PR with the latest of master branch.
It is passing the test cases. Although ideally, we should have a test case to cover this scenario, but I am not sure how mocking works here.

I would appreciate this fix as well.


appengine_flexible/pubsub: simplify setup

Fixes #534.

Test failure caused by pending Go 1.11beta1 tests. All other Go versions passed.


received critical error from subsystem: error fetching asset proof: sql: no rows in result set

My tarod keeps crashing with the error:
received critical error from subsystem: error fetching asset proof: sql: no rows in result set

This happened after I tried minting an multiple assets in a row using commands that looked like this:
tarocli assets mint --type normal --name "minted currency" --supply 100000 --enable_emission false --meta "test minted" --skip_batch

My LND commit hash is: c03de26dc
My Taro commit hash is: ddb0e0d
I have built LND using the command:
git checkout master && make && make install tags="autopilotrpc chainkit chainrpc invoicesrpc peersrpc routerrpc signrpc walletrpc watchtowerrpc wtclientrpc"

Executing the commands:
lnd --bitcoin.active --bitcoin.testnet --debuglevel=debug --bitcoin.node=neutrino --neutrino.addpeer=faucet.lightning.community --rpclisten=<IP_ADDRESS>:10009

tarod --network=testnet --debuglevel=debug --lnd.host=localhost:10009 --lnd.macaroonpath=/home/ec2-user/.lnd/data/chain/bitcoin/testnet/admin.macaroon --lnd.tlspath=/home/ec2-user/.lnd/tls.cert --tarodir=/home/ec2-user/.taro --rpclisten=<IP_ADDRESS>:10029 --restlisten=<IP_ADDRESS>:8089

Produces the log:
2023-01-04 10:49:59.749 [WRN] CONF: open /home/ec2-user/.taro/taro.conf: no such file or directory
2023-01-04 10:49:59.750 [INF] CONF: Opening sqlite3 database at: /home/ec2-user/.taro/data/testnet/taro.db
2023-01-04 10:49:59.752 [INF] CONF: Attempting to establish connection to lnd...
2023-01-04 10:49:59.764 [INF] CONF: lnd connection initialized
2023-01-04 10:49:59.765 [INF] SRVR: Version: 0.1.99-alpha commit=v0.1.1-alpha-93-gddb0e0d, build=production, logging=default, debuglevel=debug
2023-01-04 10:49:59.765 [INF] SRVR: Active network: testnet3
2023-01-04 10:49:59.765 [INF] RPCS: Validating RPC requests based on macaroon at: /home/ec2-user/.taro/data/testnet/admin.macaroon
2023-01-04 10:49:59.765 [INF] RPCS: RPC server listening on <IP_ADDRESS>:10029
2023-01-04 10:49:59.765 [INF] RPCS: gRPC proxy started at <IP_ADDRESS>:8089
2023-01-04 10:49:59.766 [INF] GRDN: Starting ChainPlanter
2023-01-04 10:49:59.785 [INF] GRDN: Retrieved 3 non-finalized batches from DB
2023-01-04 10:49:59.786 [INF] GRDN: Launching ChainCaretaker(0327308cd3ab6eb3d3796fd9e8549c6cbf6584261314818f1600347580b1a5f2b0)
2023-01-04 10:49:59.786 [INF] GRDN: Launching ChainCaretaker(03186e54abfec586bad5a9fc5501b892e5df488eef32373cf505f20803560c79df)
2023-01-04 10:49:59.787 [INF] GRDN: Launching ChainCaretaker(02b993a1d1a92f2c2de89192e758cf52d17cb83d9be1be7f6066d20143b10dfcfa)
2023-01-04 10:49:59.787 [INF] GRDN: Starting asset custodian
2023-01-04 10:49:59.787 [INF] FRTR: Starting ChainPorter
2023-01-04 10:49:59.787 [INF] FRTR: Resuming delivery of 0 pending asset parcels
2023-01-04 10:49:59.787 [INF] RPCS: Starting RPC Server
2023-01-04 10:49:59.787 [INF] SRVR: Taro Daemon fully active!
2023-01-04 10:49:59.788 [INF] GRDN: BatchCaretaker(0327308cd3ab6eb3d3796fd9e8549c6cbf6584261314818f1600347580b1a5f2b0), advancing from state=BatchStateFrozen to state=BatchStateBroadcast
2023-01-04 10:49:59.788 [INF] GRDN: BatchCaretaker(0327308cd3ab6eb3d3796fd9e8549c6cbf6584261314818f1600347580b1a5f2b0): attempting to fund GenesisPacket
2023-01-04 10:49:59.788 [INF] GRDN: BatchCaretaker(0327308cd3ab6eb3d3796fd9e8549c6cbf6584261314818f1600347580b1a5f2b0): creating skeleton PSBT: (*psbt.Packet)(0xc000476500)({
 UnsignedTx: (*wire.MsgTx)(0xc000451b00)({
  Version: (int32) 2,
  TxIn: ([]*wire.TxIn) (cap=15) {
  },
  TxOut: ([]*wire.TxOut) (len=1 cap=15) {
   (*wire.TxOut)(0x2348e00)({
    Value: (int64) 1000,
    PkScript: ([]uint8) (len=34 cap=34) {
     00000000  00 00 00 00 00 00 00 00  00 00 00 00 00 00 00 00  |................|
     00000010  00 00 00 00 00 00 00 00  00 00 00 00 00 00 00 00  |................|
     00000020  00 00                                             |..|
    }
   })
  },
  LockTime: (uint32) 0
 }),
 Inputs: ([]psbt.PInput) {
 },
 Outputs: ([]psbt.POutput) (len=1 cap=1) {
  (psbt.POutput) {
   RedeemScript: ([]uint8) <nil>,
   WitnessScript: ([]uint8) <nil>,
   Bip32Derivation: ([]*psbt.Bip32Derivation) <nil>,
   TaprootInternalKey: ([]uint8) <nil>,
   TaprootTapTree: ([]uint8) <nil>,
   TaprootBip32Derivation: ([]*psbt.TaprootBip32Derivation) <nil>
  }
 },
 Unknowns: ([]psbt.Unknown) {
 }
})

2023-01-04 10:49:59.789 [INF] GRDN: BatchCaretaker(03186e54abfec586bad5a9fc5501b892e5df488eef32373cf505f20803560c79df), advancing from state=BatchStateBroadcast to state=BatchStateBroadcast
2023-01-04 10:49:59.789 [INF] GRDN: BatchCaretaker(03186e54abfec586bad5a9fc5501b892e5df488eef32373cf505f20803560c79df): extracted finalized GenesisTx: (*wire.MsgTx)(0xc00047e000)({
 Version: (int32) 2,
 TxIn: ([]*wire.TxIn) (len=1 cap=1) {
  (*wire.TxIn)(0xc0006fb860)({
   PreviousOutPoint: (wire.OutPoint) bb982a58ff3bebe8f9e3b993793112755fe393f45c9c42187a533e65f93fbe6b:1,
   SignatureScript: ([]uint8) <nil>,
   Witness: (wire.TxWitness) (len=1 cap=1) {
    ([]uint8) (len=64 cap=64) {
     00000000  47 3b 49 64 e8 c0 65 13  62 f8 ab e6 f4 6e 91 b8  |G;Id..e.b....n..|
     00000010  e1 36 8b 53 d2 6e c6 66  c2 53 34 c5 8a 88 ee 17  |.6.S.n.f.S4.....|
     00000020  23 8d 74 30 b5 a9 dd 6a  62 37 23 1f 14 15 cf 85  |#.t0...jb7#.....|
     00000030  4f 51 05 00 f8 86 30 03  e5 9c 85 91 d0 78 2e 32  |OQ....0......x.2|
    }
   },
   Sequence: (uint32)<PHONE_NUMBER>
  })
 },
 TxOut: ([]*wire.TxOut) (len=2 cap=2) {
  (*wire.TxOut)(0xc0006e39c0)({
   Value: (int64) 1000,
   PkScript: ([]uint8) (len=34 cap=34) {
    00000000  51 20 d1 3d 9e 19 f5 0a  20 b3 75 79 42 a8 27 70  |Q .=.... .uyB.'p|
    00000010  57 d5 7e 8d ff b5 99 7d  0f 14 41 c9 32 aa c5 48  |W.~....}..A.2..H|
    00000020  85 2c                                             |.,|
   }
  }),
  (*wire.TxOut)(0xc0006e39e0)({
   Value: (int64) 3114466,
   PkScript: ([]uint8) (len=34 cap=34) {
    00000000  51 20 87 d8 c6 4d a9 0f  1e f6 c1 ba b1 ad e2 3b  |Q ...M.........;|
    00000010  d3 63 d2 f4 69 6b d4 50  44 e5 85 d1 33 0e 72 e4  |.c..ik.PD...3.r.|
    00000020  12 9d                                             |..|
   }
  })
 },
 LockTime: (uint32) 0
})

2023-01-04 10:49:59.789 [INF] GRDN: BatchCaretaker(02b993a1d1a92f2c2de89192e758cf52d17cb83d9be1be7f6066d20143b10dfcfa), advancing from state=BatchStateFrozen to state=BatchStateBroadcast
2023-01-04 10:49:59.789 [INF] GRDN: BatchCaretaker(02b993a1d1a92f2c2de89192e758cf52d17cb83d9be1be7f6066d20143b10dfcfa): attempting to fund GenesisPacket
2023-01-04 10:49:59.789 [INF] GRDN: BatchCaretaker(02b993a1d1a92f2c2de89192e758cf52d17cb83d9be1be7f6066d20143b10dfcfa): creating skeleton PSBT: (*psbt.Packet)(0xc000476c80)({
 UnsignedTx: (*wire.MsgTx)(0xc00047e1c0)({
  Version: (int32) 2,
  TxIn: ([]*wire.TxIn) (cap=15) {
  },
  TxOut: ([]*wire.TxOut) (len=1 cap=15) {
   (*wire.TxOut)(0x2348e00)({
    Value: (int64) 1000,
    PkScript: ([]uint8) (len=34 cap=34) {
     00000000  00 00 00 00 00 00 00 00  00 00 00 00 00 00 00 00  |................|
     00000010  00 00 00 00 00 00 00 00  00 00 00 00 00 00 00 00  |................|
     00000020  00 00                                             |..|
    }
   })
  },
  LockTime: (uint32) 0
 }),
 Inputs: ([]psbt.PInput) {
 },
 Outputs: ([]psbt.POutput) (len=1 cap=1) {
  (psbt.POutput) {
   RedeemScript: ([]uint8) <nil>,
   WitnessScript: ([]uint8) <nil>,
   Bip32Derivation: ([]*psbt.Bip32Derivation) <nil>,
   TaprootInternalKey: ([]uint8) <nil>,
   TaprootTapTree: ([]uint8) <nil>,
   TaprootBip32Derivation: ([]*psbt.TaprootBip32Derivation) <nil>
  }
 },
 Unknowns: ([]psbt.Unknown) {
 }
})

2023-01-04 10:49:59.789 [INF] GRDN: Gardener for ChainPlanter now active!
2023-01-04 10:49:59.790 [DBG] GRDN: Subscribing to new on-chain transactions
2023-01-04 10:49:59.790 [INF] GRDN: Loading pending inbound asset events
2023-01-04 10:49:59.792 [INF] GRDN: Resuming 4 pending inbound asset events
2023-01-04 10:49:59.795 [ERR] SRVR: Shutting down because error in main method: received critical error from subsystem: error fetching asset proof: sql: no rows in result set
2023-01-04 10:49:59.795 [INF] RPCS: Stopping RPC Server
2023-01-04 10:49:59.796 [INF] SRVR: Shutdown complete

received critical error from subsystem: error fetching asset proof: sql: no rows in result set

Are you able to reproduce this on master w/ a clean install?

Hi @Roasbeef , no this error is not reproducible.

This error seems to occur when I am running tarod for a long time after I send myself an asset.
I almost always get this when I let tarod running over night.
Here is a new log:
[ec2-user@ip-172-31-85-57 ~]$ tarod --network=testnet --debuglevel=debug --lnd.host=localhost:10009 --lnd.macaroonpath=/home/ec2-user/.lnd/data/chain/bitcoin/testnet/admin.macaroon --lnd.tlspath=/home/ec2-user/.lnd/tls.cert --tarodir=/home/ec2-user/.taro --rpclisten=<IP_ADDRESS>:10029 --restlisten=<IP_ADDRESS>:8089 --batch-minting-interval=1m
2023-01-06 09:26:07.573 [WRN] CONF: open /home/ec2-user/.taro/taro.conf: no such file or directory
2023-01-06 09:26:07.574 [INF] CONF: Opening sqlite3 database at: /home/ec2-user/.taro/data/testnet/taro.db
2023-01-06 09:26:07.578 [INF] CONF: Attempting to establish connection to lnd...
2023-01-06 09:26:07.588 [INF] CONF: lnd connection initialized
2023-01-06 09:26:07.588 [INF] SRVR: Version: 0.1.99-alpha commit=v0.1.1-alpha-93-gddb0e0d, build=production, logging=default, debuglevel=debug
2023-01-06 09:26:07.588 [INF] SRVR: Active network: testnet3
2023-01-06 09:26:07.589 [INF] RPCS: Validating RPC requests based on macaroon at: /home/ec2-user/.taro/data/testnet/admin.macaroon
2023-01-06 09:26:07.589 [INF] RPCS: RPC server listening on <IP_ADDRESS>:10029
2023-01-06 09:26:07.589 [INF] RPCS: gRPC proxy started at <IP_ADDRESS>:8089
2023-01-06 09:26:07.589 [INF] GRDN: Starting ChainPlanter
2023-01-06 09:26:07.589 [INF] GRDN: Retrieved 0 non-finalized batches from DB
2023-01-06 09:26:07.589 [INF] GRDN: Starting asset custodian
2023-01-06 09:26:07.590 [INF] FRTR: Starting ChainPorter
2023-01-06 09:26:07.590 [INF] FRTR: Resuming delivery of 0 pending asset parcels
2023-01-06 09:26:07.590 [INF] RPCS: Starting RPC Server
2023-01-06 09:26:07.590 [INF] SRVR: Taro Daemon fully active!
2023-01-06 09:26:07.591 [INF] GRDN: Gardener for ChainPlanter now active!
2023-01-06 09:26:07.591 [DBG] GRDN: Subscribing to new on-chain transactions
2023-01-06 09:26:07.591 [INF] GRDN: Loading pending inbound asset events
2023-01-06 09:26:07.592 [INF] GRDN: Resuming 1 pending inbound asset events
2023-01-06 09:26:07.593 [ERR] SRVR: Shutting down because error in main method: received critical error from subsystem: error fetching asset proof: sql: no rows in result set
2023-01-06 09:26:07.593 [INF] RPCS: Stopping RPC Server
2023-01-06 09:26:07.593 [INF] SRVR: Shutdown complete

received critical error from subsystem: error fetching asset proof: sql: no rows in result set

Hi @snow884 , do you see any proof files on disk?

@snow884 we're taking a look at this, can you provide the requested info?

Hi @jharveyb ,
Thanks a lot for picking this up.
I am sorry I no longer have access to the environment where the issue occurred.
@dstadulis  I cannot provide the info.
The issue seems to be related to neutrino . When I use bitcoind I do not get this error anymore.

Could have been a follow-up error after https://github.com/lightninglabs/taro/issues/228 left the DB in an incomplete state. Or something around Neutrino (which I don't believe we are currently prioritizing to support). So going to close this with the intent to re-open if more people run into this.


Remove wrong 'const' qualifier for is_viewer pointer

is_viewer is commanded from pi, so it cannot have 'const' qualifier.
Enabling corresponding GCC warning as a error.

Thanks pavelkryukov =D


Remove Datum trait in mapper

Could you please add why you want to remove that trait in the description? The reason it was added was to abstract the data type.


Add Readme.md file to To - Do list

Hey @Ayushparikh-code @chicken-biryani
This PR will Resolve => Add Readme.md file to To - Do list webapp. #53
Please Review my PR

You can take @yellowberard 's help if you want. He created a similar PR https://github.com/Ayushparikh-code/Web-dev-mini-projects/pull/57

You can take @yellowberard 's help if you want. He created a similar PR #57

i saw his code ,he directly uploaded the file in the Readme itself and i was making the Readme in my Vs Code
That's why the url was diff
Hope It's Fine now @chicken-biryani


File System Singleton is not being respected

File system is being instantiated more than once in the code, breaking the test file emulator

This should be addressed by using the filesystem as a service through the service provider.


MLHR-1720 Implemented Inmemory Join Operator. Supported Inner, LeftOuter, RightOuter and FullOuter join types

Updated with the correct license

I will be working with @chaithu14  on this pull request.

Pull request and commit messages should reference Malhar JIRA.

Incorporated the review comments.

@chaithu14 please add @Unstable to all the class so that it can be integrated with ManagedState.
Also please fix the build failure.

And please rebase.

@chandnisingh Added Unstable annotation to all the classes and re-based the branch.

@chaithu14 classes here need to be unstable not evolving. Once they evolve after integration with managed state, we will mark them as evolving.
They cannot be both unstable and evolving at the same time. Please remove evolving annotation and then I will merge it.
Thanks,
Chandni

Removed the Evolving annotation and updated the branch.

Incorporated review comments.

@chaithu14 please create this against master

@chandnisingh devel-3 is same as master currently. We can complete this PR if it happens within the next few days.


Specify Black Version

Make the tests Pass

Sorry, I missed something. Why don't we keep black up to date?

I have the same question as @huguesdevimeux

I have the same question

I really don't know how to answer those question. As black is in beta stage this would be the best option for now. Maybe later once black becomes stable we shall keep it to latest version. Or we would have to change the codebase for each new version of black and spoils git blame.

It being in beta is actually a good reason. Alternatively, we can try to see what changed that made all our checks break. I think they may have changed the default line width. If that's all there is, we can just set a hard default and continue to use the new version.

Ok I think I found the reason. As you can see in black's changelog.
re-implemented support for explicit trailing commas: now it works consistently within any bracket pair, including nested structures (#1288 and duplicates)

This means that now black will reformat every single bracket pair that contains elements with a trailing comma. So if you run the latest version of black, you will see that it is converting this chunk
             self.put_start_and_end_on(
                 self.get_start(), tip.get_base(),
             )

into this chunk
             self.put_start_and_end_on(
                 self.get_start(),
                 tip.get_base(),
             )

And that is because of the trailing comma. For example, this code
             self.put_start_and_end_on(
                 self.get_start(), tip.get_base()
             )

does not get converted, because there is no trailing comma.
So most of the diffs we are seeing since the black version upgraded are because of this. So we have a few paths forward:

Fix the version to the previous version. I don't think this is the right thing to do, as we will be missing out on any bug fixes and new features, and someone will have to remember to upgrade black every once in a while.
Run the new version on the whole codebase and just commit the changes.
Remember to upgrade black on our local computers so that the trailing commas are dealt with one file at a time.
Go through the codebase and eliminate those pesky trailing commas at once.
Uninstall black altogether.

Remember, the black documentation says By using [black], you agree to cede control over minutiae of hand-formatting. So I think we either cede total control (which is either of the options 2 or 3), or we uninstall it and look for a different solution (option 5).

Maybe let's set up a GitHub Action to upgrade to latest version of black and also makes a PR when the new version, by formatting the code.

Also, here #165 gonna help us. We can specify the black version there and say users to do poetry run black . so we can maintain consistency.(It creates a virtual environment).

Maybe let's set up a GitHub Action to upgrade to latest version of black and also makes a PR when the new version, by formatting the code.

Yes, this seems good to me, along with

Honestly, I think the fix is to just ask everyone to upgrade their local black and enforce running it before submitting a PR.

No need to specify a version of black, let's keep it updated :D

Closing this in favour of discussion.

We have five alternatives as mentioned in an earlier comment, and it'd be great to reach a consensus soon! The failing black checks are starting to get annoying (see #385).

Just saying I am working on a GitHub Action which does this. So it's better to wait. As in the git blam it would be named github-action and not anybody's name.

@naveen521kk How exactly will that work? Will it push black's edits immediately to the repo? Also, should we just let all black checks fail until that's done?

Will it push black's edits immediately to the repo?

It will create a branch and then a PR.

Also, should we just let all black checks fail until that's done?

I will do it in a day or two.

Hmmm how often will it open a branch and PR? Because we were trying with the restyled bot which did the same thing after each PR and most of us found it really annoying...

@leotrs You can check #396 . It doesn't actually check for a PR. It just checks for master daily and if it fails it will make an PR.

Ok great. Once a day sounds a lot better. Thanks.

Ok so after trying a bot, and a GH action, and after having BIG RED black checks on all PRs for a few days, it seems that the best way to enforce this is to ask all devs to always run the latest version of black before submitting a PR. That's it.
Devs, please do not merge PRs with a red black check.


Potential addition: Godotons

I just finished a v0.2.0 of a new addon called Godotons which was designed for installing and updating arbitrary addons from git sources (github public supported right now, plans for private repos and gitlab are had)

It's starting to work pretty well and I was ready to start sharing it around and getting feedback.
https://github.com/Ducking-Games/godotons
Would this be appropriate to add to awesome-godot? I can open a PR if so.

Would this be appropriate to add to awesome-godot? I can open a PR if so.

Feel free to open a PR for this :slightly_smiling_face:


mention 'cargo miri test filter' in README

Also update the "unsupported" example to something that is actually unsupported.

@bors r+

:pushpin: Commit a760aab828c9279fbdc5615fd23e0c499fb1f549 has been approved by RalfJung

:hourglass: Testing commit a760aab828c9279fbdc5615fd23e0c499fb1f549 with merge 7b2e325728e800664e55a7b91bac8b8b8378ab74...

:sunny: Test successful - checks-actions
Approved by: RalfJung
Pushing 7b2e325728e800664e55a7b91bac8b8b8378ab74 to master...


adding Bitoreum (BTRM)

adding Bitoreum (BTRM) on zergpool & zpool

Thanks! I'll check this out and make a new release shortly

Looked into Bitoreum.  The commit history on the core client shows nothing about how it's different from Raptoreum and has no trail to audit such... It's also very new, lacks exchange support and liquidity. Can revisit later when it's more developed.


FDG-8620 Raw Data Chart Not show full data

https://federal-spending-transparency.atlassian.net/browse/FDG-8620
No change in test coverage.

Descoped the ticket as the behavior was added intentionally
https://data-transparency-bfs.slack.com/archives/CKM2LCAF6/p1723835535171159


Unit tests for testing all face detection models (positive/negative tests)

What?
This PR involves setting up and testing for each detection model.
There is a total of 14 tests, and they all run as intended:

(Note that the tests are relatively extensive)
The tests check for:

All detectors are loaded in correctly
All Face detectors are able to find a single face (Lenna)
All Faces dont find a face in an image without a face

Also

Added test.sh script for running tests
Added run.sh script for running the main program

Further improvements before PR merge (For reviewers)

Add output logs for all asserts. Currency only some of them have them. Need to find a naming convention for logging errors
Refactor constants module. Issue created for this => #34
We should save the results as pngs. Why? It is nearly impossible to be 100% sure what position the bounding box will have. The next iteration it might be 5 pixels higher. This is why I have used a range in the code: if the x position of this image is within this range, we can assume that the boundary box is set correctly. To be 100% we can also visually check the saved pngs. Issue created for this is #35

Used this article: https://www.dataquest.io/blog/unit-tests-python/


Dependency updates

I’m just starting to use Ohm, but I’m curious why the removal of space in function blocks, such as function() {return 0;} That doesn’t seem like conventional syntax. Also, the merge is named “Dependency updates” but has formatting. Personally, I believe that very short, very specific PRs make them easier to read, approve, and manage (history, reverting, etc.). Last, it’s good to be up to date with modules, but it’s nice to know what changed?


Workspace.open() support for splitting up/down

Closes https://github.com/atom/atom/issues/7059
Related to https://github.com/atom/markdown-preview/issues/99
Add ability to call atom.workspace.open() with up / down split properties.

+1
I just tried this. Built it, installed the "git plus" plugin, changed something invoked "Git Plus: Diff".
Works fine with "Git Plus" configured for right, up and down.
Doesn't work with "Git Plus" configured for "left" (it opens the diff in the current pane), but IMO fixing that is out of scope for this PR.

:+1:

Having some specs for this behavior would be great.

I haven't forgotten about this.. just been on vacation and I've been having trouble with some of the tests.. I might have to make some changes

@lpommers, ping?

I haven't look at this in a long time. I'll get this branch up to date...

This was added in another PR. Sorry for the mixup :blush:


Modal resets form fields

Im new with inertia and currently trying your package to implement the modal functionality.
Currently i can open the modal and perform a save action. If the save action gives a validation error the modal seems to flicker and resets the form fields values but shows the validation message.
Any idea where i should start looking at why this behaviour occurs? This does not occur in your example and works fine there.

Apparently, there's some issue related to Axios update. Please try downgrading it to v0.21.1
npm i<EMAIL_ADDRESS>

Also, make sure to update both npm and composer dependencies of the Modal package to the latest versions.

Will try asap thank you for the response

I dont know why, but running npm update and composer update fixed it!

Awesome, thanks!

I'm having the same problem with the following versions, which I believe are the latest as of right now:
composer.lock

based/momentum-modal: v0.1.6
inertiajs/inertia-laravel: v0.6.3

package-lock.json

axios: 0.27.2
momentum-modal: 0.1.11
@inertiajs/inertia: 0.11.0
@inertiajs/inertia-vue3: 0.6.0

I have tried to install axios 0.21.1 and 0.21.4 but it didn't seem to fix the problem.

Hi @lepikhinb
I am using axios: ^1.2.0 and i have similar problem. It would reload the slide-over page and errors would not appear.


Can not see active zone in lovelace card

Hi,
I followed your instruction.  Multi thermostat is working fine. Thanks.
But there is no active zone display in my lovelace card.
Any suggestions, please?
configuration.yaml
input_number:
min_living_temperature:
name: Min living temperature
initial: 22
min: 15
max: 25
step: 0.1
min_pokoj_sever_temperature:
name: Min sever temperature
initial: 20
min: 15
max: 25
step: 0.1
min_loznice_temperature:
name: Min loznice temperature
initial: 18
min: 14
max: 25
step: 0.1
climate:

platform: multizone_generic_thermostat
name: Home
heater: switch.sonoff_10004c1395_2
presets:
none:
report_zone_name_instead_preset_name: true
zones:
living:
friendly_name: obyvak
target_sensor: sensor.living_room_temperature
target_temp_sensor: input_number.min_living_temperature
pokoj_sever:
friendly_name: pokoj sever
target_sensor: sensor.pokoj_kluci_temperature
target_temp_sensor: input_number.min_pokoj_sever_temperature
loznice:
friendly_name: loznice
target_sensor: sensor.loznice_temperature
target_temp_sensor: input_number.min_loznice_temperature
open_window:
delta_temp: 2
min_delta_time: 60
delta_time: 180
zone_react_delay: 00:10:00
min_temp: 17
max_temp: 25
ac_mode: false
#target_temp: 22.0
cold_tolerance: 0.3
hot_tolerance: 0.4
min_cycle_duration:
minutes: 90
initial_hvac_mode: "heat"
precision: 0.1__

lovelace card code:
type: vertical-stack
title: Thermostat
cards:

type: thermostat
entity: climate.home
name: Domov
type: entities
entities:

input_number.min_pokoj_sever_temperature
input_number.min_loznice_temperature
input_number.min_living_temperature

Hi,
Have same issue with my setup.
HA Core 2022.11.1
latest verison of your integraton addon
I see that the Target temp value gates updated, but no zones at all.
if it that is only a cosmetic issue, I would not bother, but acctually its not working as expected either. perhaps its related.
will create separate issue, as it would be offtopic to this one.
thanks
Thomas

Hi, sorry for my english, I am from Czech Rupublic. I have same problem. Only "Idle" without zone name. Please, can you help me? Tahnks, Peter


Assemblies Missing From Code Coverage; Exhausted All Troubleshooting Steps

I have a solution with 4 assemblies, 2 of them MS test assemblies, 1 dll and 1 exe (console app).  The DLL and its test assembly show up in code coverage results, but the exe does not even though I have passing test runs on that assembly.  I added the sample .runsettings file and commented out all exclusions, leaving only include all exe and dll.
I have tried all of the relevant troubleshooting steps, but none of them solved my problem.

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

ID: c4d41fc7-bc52-8fe3-6b25-c1b2c3f89eb1
Version Independent ID: 9d141754-5f8f-d330-0974-75cbcf45a011
Content: Troubleshooting Code Coverage - Visual Studio
Content Source: docs/test/troubleshooting-code-coverage.md
Product: visual-studio-windows
Technology: vs-ide-test
GitHub Login: @Mikejo5000
Microsoft Alias: mikejo

@Good-man -- Mark, thank you for your question.  Please consider these resources:

https://social.msdn.microsoft.com/Forums/vstudio
https://www.stackoverflow.com
https://www.reddit.com/r/VisualStudio/

You are welcome to use the Report a Problem tool to report the issue.
cc @Mikejo5000 for awareness
I am closing this issue now.  You are welcome to @ mention me for any followup.  We hope to hear from you again.


Add method to compute normalized structure factors

It can be useful to have normalized structure factors for certain applications, and we have all the ingredients needed to make this happen in different places. I think it would make sense to add a function that computes normalized structure factors to rs.algorithms, or perhaps as a built-in method of rs.DataSet.

I'm closing this because a compelling use case hasn't come up. Can be re-opened if desired.


remove the error throwing in storage engine constructor

fixing the error found by @phavre because of the StorageEngine constructor changes:
https://github.com/driftyco/ionic/commit/07953672f53c793828f29d81101b46d6b88dfafa#commitcomment-15599683

haven't managed to test this on latest ionic 2.0 branch with ionic conference app since there is another error about "No provider for Http! (ConferenceApp -> ConferenceData -> Http)"


otf font not being loaded correctly

Here's my config:
{
      test: /\.(otf|eot|woff|woff2|ttf|svg|png|jpg)$/,
      loader: 'file-loader?limit=30000&name=[name]-[hash].[ext]'
    }

When I require a .ttf font everything works fine. Then trying to use .otf, no error is thrown, but the font simply doesn't load.
Any help would be greatly appreciated. Thanks!

Any update? Can you provide a standalone test case?

/cc @tnrich friendly ping. Maybe otf is corrupted?

Please re-open with an example to run.
Note that file-loader itself doesn't do much. It will write the otf file and return a path to it in the source. Without further information it's hard to say why it fails to load.


cask-repair gets confused when repairing cask cocktail

Cask cocktail version 12.2.1 currently has the wrong checksum.
Expected: e1d8b4529963e94b8a5d710ee3dd75f15423701aead815da271d624b2c653278   Actual: 1fe5d5524991d1e191b7654aca849612fbf674ed981df7c1ca8ff3e8ccab6565
I tried to repair it using cask-repair, which did not work as expected, and failed with an inappropriate error message. See what happened below:
Updating taps… Already up-to-date.
------------------------------------------------------------------------------------------------------------------------------------
cask 'cocktail' do
  if MacOS.version <= :mavericks
    version '7.9.1'
    sha256 'b8b5c37df3a2c44406f9fdf1295357d03b8fca6a9112b61401f0cca2b8e37033'

    url "https://www.maintain.se/downloads/sparkle/mavericks/Cocktail_#{version}.zip"
    appcast 'https://www.maintain.se/downloads/sparkle/mavericks/mavericks.xml'
  elsif MacOS.version <= :yosemite
    version '8.9.2'
    sha256 'acc7d191313fa0eb4109ae56f62f73e7ed6685f7d7d438d5138b85d68e40edd8'

    url "https://www.maintain.se/downloads/sparkle/yosemite/Cocktail_#{version}.zip"
    appcast 'https://www.maintain.se/downloads/sparkle/yosemite/yosemite.xml'
  elsif MacOS.version <= :el_capitan
    version '9.7'
    sha256 'ca6b4a264ca60a08ff45761f82b0b6161cbe3412bd6cbeedd5dbecebc8d26712'

    url "https://www.maintain.se/downloads/sparkle/elcapitan/Cocktail_#{version}.zip"
    appcast 'https://www.maintain.se/downloads/sparkle/elcapitan/elcapitan.xml'
  elsif MacOS.version <= :sierra
    version '10.9.1'
    sha256 'c41bdcff4e0a1bdf3b0b1dfa11e12de71acf64010c7dccfd337ec2f42ca7bd4f'

    url "https://www.maintain.se/downloads/sparkle/sierra/Cocktail_#{version}.zip"
    appcast 'https://www.maintain.se/downloads/sparkle/sierra/sierra.xml'
  elsif MacOS.version <= :high_sierra
    version '11.7'
    sha256 'e1d8b4529963e94b8a5d710ee3dd75f15423701aead815da271d624b2c653278'

    url "https://www.maintain.se/downloads/sparkle/highsierra/Cocktail_#{version}.zip"
    appcast 'https://www.maintain.se/downloads/sparkle/highsierra/highsierra.xml'
  else
    version '12.2.1'
    sha256 'e1d8b4529963e94b8a5d710ee3dd75f15423701aead815da271d624b2c653278'

    url "https://www.maintain.se/downloads/sparkle/mojave/Cocktail_#{version}.zip"
    appcast 'https://www.maintain.se/downloads/sparkle/mojave/mojave.xml'
  end

  name 'Cocktail'
  homepage 'https://www.maintain.se/cocktail/'

  app 'Cocktail.app'
end
------------------------------------------------------------------------------------------------------------------------------------
Type the new version (or leave blank to use current one, or use `s` to skip)
> 12.2.1

==> Downloading external files for Cask cocktail
==> Downloading https://www.maintain.se/downloads/sparkle/mojave/Cocktail_12.2.1.zip
######################################################################## 100.0%
==> Verifying SHA-256 checksum for Cask 'cocktail'.
==> Note: Running `brew update` may fix SHA-256 checksum errors.
Error: Checksum for Cask 'cocktail' does not match.
Expected: e1d8b4529963e94b8a5d710ee3dd75f15423701aead815da271d624b2c653278
  Actual: 1fe5d5524991d1e191b7654aca849612fbf674ed981df7c1ca8ff3e8ccab6565
    File: /Users/ralph/Library/Caches/Homebrew/downloads/f6eab049ffd189b9733a2c438213ae2304bf70c9501c8893b33be1bb82678620--Cocktail_12.2.1.zip
To retry an incomplete download, remove the file above.
If the issue persists, visit:
  https://github.com/Homebrew/homebrew-cask/blob/master/doc/reporting_bugs/checksum_does_not_match_error.md
error: Cannot delete branch 'master' checked out at '/usr/local/Homebrew/Library/Taps/homebrew/homebrew-cask'

There was an error fetching ./cocktail.rb. Please check your connection and try again.```

What‘s your macOS version?

On 17 Apr 2019, at 23:42, Vítor Galvão<EMAIL_ADDRESS>wrote:
What‘s your macOS version?

10.14.4

It was the cask itself that had a problem, because Travis isn’t running on Mojave.

Maybe the cask is broken - but as a user of the command who doesn't want to dig deeper, it still seems to me that the progress and error messages that result are confusing and unhelpful here.
For example, why should it be checking the checksum, when that is the broken thing I want to repair?
A more informative message as to why the repair has failed would also be useful.
Could repair-cask do some sanity checking of casks?

There are already plenty of checks in place. I’m not going to spend my time adding yet another one to complicate this already complex script when the problem was outside of it. Fix the problem where the problem is.


VS2022 correctly detected but still using 2019 for some tasks

Hi Jesse,
I got the VSbuild-sxs task properly working so it detects now VS2022 correctly. In the logs, I just saw, that it sill uses 2019 for calling e.g. CL.exe, Lib.exe and link.exe. 2022 is used for calling csc.exe for compiling. Not sure what went wrong here or if I have a general misunderstanding on my side, but shouldn't 2022 be used for the whole build over 2019?
"C:\agent\_work\_tasks\VSBuild-sxs_e611ef14-f538-5f42-8ffc-60c88f7f045c\1.199.0\ps_modules\MSBuildHelpers\vswhere.exe" -version [17.0,18.0) -latest -format json
ClCompile:          C:\Program Files (x86)\Microsoft Visual Studio\2019\Professional\VC\Tools\MSVC\14.28.29910\bin\HostX86\x64\CL.exe /c /I..\..\ /Zi /nologo /W3 /WX /diagnostics:column /O2 /Ob2 /Oy- /D WIN32 /D NDEBUG /D _LIB /D _MBCS /Gm- /EHsc /MT /GS /fp:precise /Zc:wchar_t /Zc:forScope /Zc:inline /Fo"x64\Release\\" /Fd"x64\Release\\vc142.pdb" /Gd /TC /FC /errorReport:queue DllMain.c Error.c Utils.c Win32.c
20>CoreCompile:          C:\Program Files\Microsoft Visual Studio\2022\Professional\MSBuild\Current\Bin\Roslyn\csc.exe /noconfig /unsafe- /checked- /nowarn:1616,1607,1701,1702 /fullpaths /nostdlib+ /platform:AnyCPU /errorreport:prompt /warn:4 /doc:bin\x64\Release\MiscellaneousCS.xml ...
Remark: We did not update the toolset for the C++-projects to v143 yet. They are still on v142.

I don't know how C++ evaluates what SDK to use. Can you run a build in diagnostics mode and MSBuild bedbug or binary jogging turned on and share the logs?

I think I figured out what happens. The C++ projects have a setting called VC++ Directories. Those have some SDK-depending Paths defined which resolve back to VS2019 which was delivering the Toolset v142. Once I update the Toolset for those projects to v143 which is delivered with VS2022, those Paths resolve to VS2022. Kind of a weird behavior for me because I don't know what happens if you don't have VS2019 installed anymore ;). Nevertheless, it looks like this is something internally in MSBuild and not caused by your pipeline task


url_for('static', filename='') error, i don't know where in error?

flask: 1.1.2
jinja2: 2.11.1

The error is probably a quote mismatch earlier in your template. The problem does not appear to be in the url_for call.

thanks, i'm check code


stuck

tried to install keybase on mac laptop, osx 10.11.6, latest
got this: http://i.imgur.com/JFQbTS8.png
restarted app, it did it again.  restarted laptop, it did it again.
no instructions, no help, no link, nothing online on the website.  <?>
where are the logs?  apparently there is one, but "keybase log send" doesn't say where it is
searched in ~/Library/Application Support/Keybase did not find anything helpful
what action is supposed to come next?
my log id: b4190f2227363c8c08621c1c

the issue is this:
1073 2017-02-13T20:45:27.921829 ▶ [DEBU keybase gpg_cli.go:353] 403 | running Gpg: /usr/local/bin/gpg --no-tty --armor --sign -u e3a091f223617b3613da61e1dc6dfc240b180120
1074 2017-02-13T20:45:27.931409 ▶ [DEBU keybase gpg_cli.go:332] 404 gpg: Sorry, no terminal at all requested - can't get input
1075 2017-02-13T20:45:27.931838 ▶ [DEBU keybase gpg_key.go:50] 405 - GPGKey Signing -> GPG error: exit status 2

haven't seen this issue before, but it's GPG, so I'm not surprised.  It's an incredibly inconvenient app to integrate with.  Does gpg work on the command line for you?

yes, it works command line. it only does, do not use it with a gui ever

where/how can that log be viewed?

gpg is installed using brew, from package gnupg
#> brew info gnupg
gnupg: stable 1.4.21 (bottled)
GNU Pretty Good Privacy (PGP) package
https://www.gnupg.org/
/usr/local/Cellar/gnupg/1.4.20 (53 files, 5.4M) *
Poured from bottle on 2016-03-08 at 14:23:47
From: https://github.com/Homebrew/homebrew-core/blob/master/Formula/gnupg.rb

so you know, typically when using this private key to sign something, the passphrase is required, and the command line interface offer the place to enter it
like,
'''
You need a passphrase to unlock the secret key for"
key details...
key details...
Enter passphrase: _'''
but if an external app is trying to call gpg and the key has to be unlocked, and you want to pass the passphrase back, you might use something like
gpg --no-version --no-verbose --batch --output - --passphrase-fd 0 --armor --detach-sign --textmode args args args ...
(or something like that)
which will allow gpg to read in the passphrase from STDIN

Try 'keybase login' from the command line. That should work in your case.
On Tue, Feb 14, 2017 at 2:27 AM Jonathan Dugan<EMAIL_ADDRESS>wrote:

so you know, typically when using this private key to sign something, the
passphrase is required, and the command line interface offer the place to
enter it
like,
'''
You need a passphrase to unlock the secret key for"
key details...
key details...
Enter passphrase: _'''
but if an external app is trying to call gpg and the key has to be
unlocked, and you want to pass the passphrase back, you might use something
like
gpg --no-version --no-verbose --batch --output - --passphrase-fd 0 --armor
--detach-sign --textmode args args args ...
(or something like that)
which will allow gpg to read in the passphrase from STDIN
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/keybase/client/issues/5862#issuecomment-279628384,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA05_-WAoqQtRGyzqzZo-mIFZvDY30Lcks5rcVd7gaJpZM4MAD8Y
.

worked!  thank you


Duplication of onKeyPress event when using physical keyboard

In my app, there is a sign in page with 2 input fields which are using a single virtual keyboard.
users might have a physical keyboard and might not (i have no control over this, depending on my client choice)
on initial navigation to the sign in page all works as expected.
However, after user logs out and redirected back to the sign in page, the physical keyboard key presses are duplicated (only with the physical keyboard, using the virtual directly is working as it should.)
any thoughts on why this behavior happens? it feels as if the onKeyPress event handler is being duplicated (when intercepting a physical keyboard click)
is there a way that the keyboard itself can identify a physical keyboard connected to the machine?

Hello @idoRosen25, I don't believe this is caused by simple-keyboard. To verify, you can use Chrome/Edge Dev Tools to add a breakpoint to your onKeyPress and see if it's called twice.
You can also provide a repro (or test repository) of the issue, and I can take a look.
Regards,
Francisco Hodge


Integrated code lifecycle: Provide Instructors more options to control container configuration

Checklist
General

[x] I tested all changes and their related features with all corresponding user types on a test server.
[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[x] I chose a title conforming to the naming conventions for pull requests.

Server

[x] Important: I implemented the changes with a very good performance and prevented too many (unnecessary) and too complex database calls.
[x] I strictly followed the principle of data economy for all database calls.
[x] I strictly followed the server coding and design guidelines.
[x] I added multiple integration tests (Spring) related to the features (with a high test coverage).
[x] I added pre-authorization annotations according to the guidelines and checked the course groups for all new REST Calls (security).
[x] I documented the Java code using JavaDoc style.

Client

[x] Important: I implemented the changes with a very good performance, prevented too many (unnecessary) REST calls and made sure the UI is responsive, even with large data (e.g. using paging).
[x] I strictly followed the principle of data economy for all client-server REST calls.
[x] I strictly followed the client coding and design guidelines.
[x] Following the theming guidelines, I specified colors only in the theming variable files and checked that the changes look consistent in both the light and the dark theme.
[x] I added multiple integration tests (Jest) related to the features (with a high test coverage), while following the test guidelines.
[x] I added authorities to all new routes and checked the course groups for displaying navigation elements (links, buttons).
[x] I documented the TypeScript code using JSDoc style.
[x] I added multiple screenshots/screencasts of my UI changes.
[x] I translated all newly inserted strings into English and German.

Changes affecting Programming Exercises

[x] High priority: I tested all changes and their related features with all corresponding user types on a test server configured with the integrated lifecycle setup (LocalVC and LocalCI).

Motivation and Context

We want instructors to be able to configure some docker flags. For now, this is restricted to disabling internet access when running the student code and to add env vars to the container.
Description

Disabling network access: To disable network access, we run this command docker network disconnect bridge <container-name> using the java docker API. However, some programming languages need to download dependencies to correctly run the submissions. Thus, after a container is started, we run a script to download dependencies. The script depends on the programming language. After that script is run, we execute the mentioned docker command and proceed with the build job as usual.
Environment variables: We parse the environment variables and pass them to the start container command.

Steps for Testing

Prerequisites:

1 Instructor

Testing disabling network access

Go to programming exercise creation(choose programming language other than Haskell or Swift)
Check the Edit Build Script checkbox
Check the Disable network access checkbox
Make sure that the tooltip is helpful and explains the feature correctly
Create exercise
Make sure that the solution build is successful (for languages other than Java, Javascript, Rust, Kotlin, you should see this string in the build logs No dependency download script needed for this programming language)

Testing env vars

Go to programming exercise edit (or create a new one)
Check the Edit Build Script checkbox
In the environment variables option, enter environment variables (it should be in the correct format as explained in the tooltip, env vars entered in the wrong format will be ignored)
In the build script, add this line echo $<env_var_key>
Save and run a buildjob for that exercise
Check that the value of the env var you added is present in the build logs

Testserver States

[!NOTE]
These badges show the state of the test servers.
Green = Currently available, Red = Currently locked
Click on the badges to get to the test servers.

Review Progress

Performance Review

[ ] I (as a reviewer) confirm that the client changes (in particular related to REST calls and UI responsiveness) are implemented with a very good performance even for very large courses with more than 2000 students.
[ ] I (as a reviewer) confirm that the server changes (in particular related to database calls) are implemented with a very good performance even for very large courses with more than 2000 students.

Code Review

[ ] Code Review 1
[ ] Code Review 2

Manual Tests

[ ] Test 1
[ ] Test 2

Performance Tests

[ ] Test 1
[ ] Test 2

Screenshots

Summary by CodeRabbit
Summary by CodeRabbit

New Features

Introduced a new configuration for Docker run settings, allowing users to manage network access and environment variables for build jobs.
Enhanced functionality to parse Docker flags from JSON strings and manage environment variables effectively.
Implemented lifecycle management in the programming exercise build configuration component for better user interaction.
Added detailed instructions for editing container configurations, including network access and environment variable management.

Bug Fixes

Improved validation logic for programming exercise settings to ensure compatibility with Docker configurations.

Documentation

Updated internationalization files to include new Docker flags and configuration options for both English and German.
Enhanced the programming exercise setup documentation to clarify new configuration options and validation processes.

Tests

Expanded test coverage for Docker flag handling and environment variable management in local CI integration tests.
Added tests for Docker flags parsing to ensure correct configurations in build jobs.

Is there a reason why the only configurable flag is the network? If I (for example) would like to specify how many cpus are used with the --cpu flag, I still wouldn't be able to do that in the UI, right?

@chrisknedl This is planned in a follow-up PR. Since we use CPU count to determine the build agent's capacity, allowing instructors to change the CPU count freely could lead to problems.

@BBesrour Will all docker flags in the application.yml config be added eventually? Wouldn't it be easier to just allow instructors to set them as key-value pairs instead of adding a new ui element for each of them (like the button for the network)?

@chrisknedl Yes, that's the plan! Instructors should be able to edit the performance flags (cpu, memory, etc) as a key-value pair.

Only allowing network yes/no would be problematic for our use-case:
We self-host a Maven cache to not have to bake in all dependencies into the container image. Then we start the container with a special --network artemis-restricted flag so that only network access to this Maven cache is allowed without allowing network to other hosts.


AZ Login Failed

If I follow this step on my laptop through Azure CLI plugin:

The Az Login error is shown during the pipeline execution:

If I run the same command and generate the json token from Azure Bash all works fine.

Hi @filippo-cala
we saw that you succeded in deploying the solution:
https://github.com/filippo-cala/Demo.Aks.Performance/actions/runs/2383469315
I guess there was an error during the copy paste of your secret?
We will go ahead and close this issue.


Remove grape and grape-swagger from dependencies

While this gem was constructed with grape and grape-swagger in mind, they are technically not necessary for non-test purposes.  This will address this issue.

But we do want to test compatibility, so I think you cannot remove the Gemfile part. See https://github.com/ruby-grape/grape-swagger-rails/blob/master/.travis.yml#L17 being built.

This is great, merging.


Add non-blocking buffered io for reads on non-pipelined commands

Taking pieces of: https://github.com/Shopify/dalli/pull/11/files to break it up.
This introduces a new file: BufferedIO, which will read non-blocking and fill up the buffer. This replaces the usual read and readline methods that we had previously.
The socket timeouts are also decreased after each operation to make sure that we don't see our timeouts go on forever if we continue to read from the socket.
I did NOT:

Add buffered writes
Change pipelined get/sets

There still needs to be more changes to update the above.

Oops I haven't added tests yet for the new file. Adding those now.

The socket timeouts are also decreased after each operation to make sure that we don't see our timeouts go on forever if we continue to read from the socket.

note this is good in terms of actually having timeouts that match expectations, but bad in terms of how dalli will close connections on timeouts to ensure the buffer and socket is clean... Since creating connections is a problem we have had, just noting this is something I read about when looking at deadlines...
if we end up making calls with absurdly small timeouts towards the end of a bunch of calls it might mean we time out more often... All good for now, but something to consider, this could be a place deadlines are better for say large multi_gets vs decreasing timeout for large collections.


README.md: improve docs gen tutorial

Improve the tutorial for walkingh through generating the .html documentation for keywords.
Main changes:

Add a reminder about virtual environment.
Simplify commands for generating the documentation.

@artur-rs please review before it's got stale as many other PRs here


Adds annotation in podSpec to trigger re-creation of pods on secret credential update

Changes
[1] Moves the secret checksum annotation from deployment's metadata.annotations to spec.template.annotations
[2] Adds an E2E to verify that at first DNS records are not created when wrong secret is supplied. When the wrong secret is updated with the right values, DNS records are created.

/lgtm

LGTM for the code changes. Let's think about the e2e test case for this feature to not fall into the same trap again.

/retest

Tests may be failing because the env variable changes (from 2 vars to 1 for the file) are not propagated to the desired deployment. This PR is supposed to fix this: https://github.com/openshift/external-dns-operator/pull/150.

Tested manually too:
02:32:58 $ oc -n external-dns get pods
No resources found in external-dns namespace.

02:33:07 $ oc -n external-dns get secret
NAME                       TYPE                                  DATA   AGE
builder-dockercfg-pdf6f    kubernetes.io/dockercfg               1      3h7m
builder-token-pwqlz        kubernetes.io/service-account-token   4      3h7m
builder-token-r2v8x        kubernetes.io/service-account-token   4      3h7m
default-dockercfg-nrgfx    kubernetes.io/dockercfg               1      3h7m
default-token-fvr4j        kubernetes.io/service-account-token   4      3h7m
default-token-tq696        kubernetes.io/service-account-token   4      3h7m
deployer-dockercfg-r8tmw   kubernetes.io/dockercfg               1      3h7m
deployer-token-5887f       kubernetes.io/service-account-token   4      3h7m
deployer-token-brdgr       kubernetes.io/service-account-token   4      3h7m

02:33:14 $ oc apply -f config/samples/aws/operator_v1alpha1_externaldns_openshift.yaml
externaldns.externaldns.olm.openshift.io/sample-aws created

02:33:19 $ oc -n external-dns get secret
NAME                                      TYPE                                  DATA   AGE
builder-dockercfg-pdf6f                   kubernetes.io/dockercfg               1      3h8m
builder-token-pwqlz                       kubernetes.io/service-account-token   4      3h8m
builder-token-r2v8x                       kubernetes.io/service-account-token   4      3h8m
default-dockercfg-nrgfx                   kubernetes.io/dockercfg               1      3h8m
default-token-fvr4j                       kubernetes.io/service-account-token   4      3h8m
default-token-tq696                       kubernetes.io/service-account-token   4      3h8m
deployer-dockercfg-r8tmw                  kubernetes.io/dockercfg               1      3h8m
deployer-token-5887f                      kubernetes.io/service-account-token   4      3h8m
deployer-token-brdgr                      kubernetes.io/service-account-token   4      3h8m
external-dns-credentials-sample-aws       Opaque                                3      3s
external-dns-sample-aws-dockercfg-8gtlk   kubernetes.io/dockercfg               1      3s
external-dns-sample-aws-token-mzc5z       kubernetes.io/service-account-token   4      3s
external-dns-sample-aws-token-s6sdn       kubernetes.io/service-account-token   4      3s

02:33:22 $ oc -n external-dns get pods
NAME                                       READY   STATUS    RESTARTS   AGE
external-dns-sample-aws-7cdcbb8b69-r7sfg   1/1     Running   0          7s

02:33:26 $ oc -n external-dns get pods external-dns-sample-aws-7cdcbb8b69-r7sfg -o yaml
apiVersion: v1
kind: Pod
metadata:
  annotations:
    externaldns.olm.openshift.io/credentials-secret-hash: ca286362386fad85434e29b75dd7a5b1759bdbf5c4d378580c176887547e9ce7

02:33:37 $ oc -n external-dns exec external-dns-sample-aws-7cdcbb8b69-r7sfg -- cat /etc/kubernetes/aws-credentials
[default]
aws_access_key_id = AAAAAAAAAAAAAAAA
aws_secret_access_key = aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa

2:34:34 $ oc -n external-dns get pods
NAME                                       READY   STATUS    RESTARTS   AGE
external-dns-sample-aws-7cdcbb8b69-r7sfg   1/1     Running   0          84s

02:34:43 $ oc -n external-dns-operator edit secret aws-access-key
error: secrets "aws-access-key" is invalid
secret/aws-access-key edited

02:36:18 $ oc -n external-dns get pods
NAME                                       READY   STATUS    RESTARTS   AGE
external-dns-sample-aws-6dff6959f6-7nx6t   1/1     Running   0          8s

02:36:32 $ oc -n external-dns exec external-dns-sample-aws-6dff6959f6-7nx6t -- cat /etc/kubernetes/aws-credentials
[default]
aws_access_key_id = WRONG
aws_secret_access_key = WRONG

02:39:36 $ oc -n external-dns get pods external-dns-sample-aws-6dff6959f6-7nx6t -o yaml
apiVersion: v1
kind: Pod
metadata:
  annotations:
    externaldns.olm.openshift.io/credentials-secret-hash: edfd61362b1c83fe22a862121cfc8cc0fd86968f5f914001083d715585aef9d7

/label qe-approved

/label docs-approved
/label px-approved


Added section: Provisioning access to resources

Added section to elaborate that Managed Identities can be used to provision access to other resources as well, with links to them.

Can we know how the host name for the SFTP Linked Service was give as Host: adflabsftp.westus2.cloudapp.azure.com


Consider using a better Accept header when fetching modules

Currently the following header is sent when fetching modules:
Accept: */*

Which makes it very difficult for a server to do content negotiation.  Either a header of:
Accept: application/javascript, application/typescript, text/javascript, text/typescript, text/jsx, text/tsx, application/json;q=0.9, text/plain;q=0.8, application/octet-stream;q=0.8

Or:
Accept: application/javascript, application/typescript, text/javascript, text/typescript, text/jsx, text/tsx, application/json;q=0.9, text/plain;q=0.8, application/octet-stream;q=0.8, */*;q=0.7

Would allow better negotiation of content for a web server.
Ref: https://github.com/denoland/deno/issues/13287

#42 - I picked the first one.


Unify accelerator and ep concept into system

Describe your changes

In this PR, the execution_providers is moved from engine to system config. typical config looks like

    "systems": {
        "local_system": {
            "type": "LocalSystem",
            "config": {
                "accelerators": [
                    {
                        "device": "gpu",
                        "execution_providers": [
                            "CPUExecutionProvider"
                        ]
                    }
                ]
            }
        }
    },
  "engine": {
        "host": "local_system",
        "target": "local_system",
}

Please note the above config is optional if we only use local system with default execution providers.
From the time being, the device is required if we would like to specify the accelerators. Should we deduce the device from EP if it is not provided?
Checklist before requesting a review

[ ] Add unit tests for this change.
[ ] Make sure all tests can pass.
[ ] Update documents if necessary.
[ ] Lint and apply fixes to your code by running lintrunner -a
[ ] Is this a user-facing change? If yes, give a description of this change to be included in the release notes.
[ ] Is this PR including examples changes? If yes, please remember to update example documentation in a follow-up PR.

(Optional) Issue link

Since you are updating accelerator, can we also update the engine run API output? Like current output node is a dict where key is an Accelerator object. When using Python API, it is difficult to get the output value directly for a specific device or ep for example, where you have to manually build an Accelerator object or for loop the whole result. One option is simplifying the key to a str <device>-<ep>, or other alternative solutions also fine.

Since you are updating accelerator, can we also update the engine run API output? Like current output node is a dict where key is an Accelerator object. When using Python API, it is difficult to get the output value directly for a specific device or ep for example, where you have to manually build an Accelerator object or for loop the whole result. One option is simplifying the key to a str <device>-<ep>, or other alternative solutions also fine.

The device-ep is a good suggestion to replace the AcceleratorSpec. One simple approach to switch to device-ep is to use something like str(AcceleratorSpec) instead of the original object. On the other hand, this is a separated improvement PR after current PR is merged.
@jambayk, @trajepl, what's your opinion to switch to device-ep from the original AcceleratorSpec?


Add BinaryData Serializer and Deserializer to JacksonAdapter

Description
Adds BinaryDataDeserializer and BinaryDataSerializer to JacksonAdapter to support cases when BinaryData is a property on a serializable type. Before this Jackson would attempt to reflectively access all fields on BinaryData and serialize them.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

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


added pagination for  every table AB#5547

Motivation
Pagination  implemented for every table in diem explorer
Test Plan

open   application  and  navigate into Events  tab
select  any event item  in dropdown  (like ,Mint Events, Burn Events,...)
you can see  Next  button  at  bottom  of the  table
click  on  next  , you can see  Previous  Button

/canary


Front-End: progress bar starting at 100 in the beginning

maybe set it up so it starts on hide and shows when you click join?

Solved.


Pass config.yaml variables in via environment

It would be nice to be able to reference an environment variable in the config.yaml, especially for sensitive data (postgres credentials).
I'd expect to be able to use the same syntax that docker-compose does, and be able support either of these:
  database:
    db_connect: '${DATABASE_URI}'
    db_connect_args:
      timeout: 120
      ssl: False
    db_pool_size: 30
    db_pool_max_overflow: 100

or
  database:
    db_connect: 'postgresql+pg8000://postgres:${DATABASE_PASSWORD}@anchore-db:5432/postgres'
    db_connect_args:
      timeout: 120
      ssl: False
    db_pool_size: 30
    db_pool_max_overflow: 100

Thanks Marc.
We are looking at how we can better handle this initial bootstrap process for both the DB password as well as the other credentials that are stored in the config file (admin password, anchore.io password).
Using an environment variable is a first step however we're also looking at supporting other secret management methods (swarm secrets, k8s secrets, etc)

Thanks @marccampbell. Are there any other secret-management systems you'd like to see available? If we were to do a plugin-based approach for things like docker-secret, vault, k8s secrets, etc are there any specific ones you'd want to see initially?

@marccampbell - We've added to master branch an improvement that allows environment variables to be used within the config.yaml.  There are a couple of constraints/rules:
a) it is a generic search/replace, so you can put whatever env variable you want in the config file of form ${ANCHORE...}, as long as it starts with 'ANCHORE'.  need to keep in mind that it is a straight replace - when adding ${ANCHORE...} strings into the config.yaml, should try to envision the result and ensure it is valid YAML
b) implementation includes an env file method as well - if ANCHORE_ENV_FILE=/path/to/foofile is set in the environment that executes anchore-engine, then /path/to/foofile will be opened and parsed, looking for "ANCHORE*=whatever" lines in the file which will be used for config.yaml search/replace
c) precedence is explicit environment variable > key/val in ANCHORE_ENV_FILE file
d) the config validator fails if you have a variable in the config (${ANCHORE...) after the load/search/replace pass, with an error message on startup indicating which variable is still in the resulting config after the search/replace pass
Thank you - let us know if this implementation fits the requirement!
-Dan


Beginner Skillmap Badges

The "Rescue Baby Dinos" badge is awarded when I've only completed the first Story learning path in the Beginner Skillmap.
Repro - Edge, incognito, signed into a user that doesn't have any Beginner Skillmap progress

Complete the first Story learning path
Open user profile, you will see the Baby Dino badge
Expected behavior - should have gotten the Story badge

Is this in beta?  We don't have separate badges for each path yet. Only the "beginner" badge.

Yes, this was in beta.  Can we get badges for the other paths?
It's weird to get a badge for completing Rescue the Baby Dinos when I did the Story path...

@soniakandah Can we get the badges for the other paths ASAP, please?

Here are the new beginner badges:

PR is ready!  I'm going to wait for approval on all of the markdown changes and the addition of the tutorial hat to the BG before I merge.

https://github.com/microsoft/pxt-arcade/pull/4571


Upgraded to resolve a crash in WinUI 2.6 with get_SystemBackdrop

…TR_READ_c0000005_Windows.UI.Xaml.dll!DirectUI::Window::get_SystemBackdrop
Summary of the Pull Request
What is this about:
WinUI 2.6 is causing a crash, fix is in 2.6.2
What is include in the PR:
Nuget upgrade
How does someone test / validate:
Quality Checklist

[ ] Linked issue: #https://github.com/microsoft/PowerToys/issues/13136
[ ] Communication: I've discussed this with core contributors in the issue.
[ ] Tests: Added/updated and all pass
[ ] Installer: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Docs: Added/ updated
[ ] Binaries: Any new files are added to WXS / YML

[ ] No new binaries
[ ] YML for signing for new binaries
[ ] WXS for installer for new binaries

Contributor License Agreement (CLA)
A CLA must be signed. If not, go over here and sign the CLA.

الغاء
احصل على Outlook for Androidhttps://aka.ms/AAb9ysg

From: Clint Rutkas @.>
Sent: Thursday, September 9, 2021 9:34:21 PM
To: microsoft/PowerToys @.>
Cc: Subscribed @.***>
Subject: [microsoft/PowerToys] Upgraded to resolve a crash in WinUI 2.6 with get_SystemBackdrop (#13137)
…TR_READ_c0000005_Windows.UI.Xaml.dll!DirectUI::Window::get_SystemBackdrop
Summary of the Pull Request
What is this about:
WinUI 2.6 is causing a crash, fix is in 2.6.2
What is include in the PR:
Nuget upgrade
How does someone test / validate:
Quality Checklist

Linked issue: ##13136https://github.com/microsoft/PowerToys/issues/13136
Communication: I've discussed this with core contributors in the issue.
Tests: Added/updated and all pass
Installer: Added/updated and all pass
Localization: All end user facing strings can be localized
Docs: Added/ updated
Binaries: Any new files are added to WXS / YML
*    No new binaries
*    YML for signinghttps://github.com/microsoft/PowerToys/blob/master/.pipelines/pipeline.user.windows.yml#L68 for new binaries
*    WXS for installerhttps://github.com/microsoft/PowerToys/blob/master/installer/PowerToysSetup/Product.wxs for new binaries

Contributor License Agreement (CLA)
A CLA must be signed. If not, go over herehttps://cla.opensource.microsoft.com/microsoft/PowerToys and sign the CLA.

You can view, comment on, or merge this pull request online at:
https://github.com/microsoft/PowerToys/pull/13137
Commit Summary

Upgraded to resolve a crash. [Watson Failure] caused by NULL_CLASS_PTR_READ_c0000005_Windows.UI.Xaml.dll!DirectUI::Window::get_SystemBackdrop

File Changes

M src/settings-ui/Microsoft.PowerToys.Settings.UI/Microsoft.PowerToys.Settings.UI.csprojhttps://github.com/microsoft/PowerToys/pull/13137/files#diff-4a4621b6eaf935e501a79f869e6ce60a8b450fe366137386d8d9896e8c9d8676 (4)

Patch Links:

https://github.com/microsoft/PowerToys/pull/13137.patch
https://github.com/microsoft/PowerToys/pull/13137.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/PowerToys/pull/13137, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AVQJAXVOVVWWWHQUGL373RTUBD423ANCNFSM5DXXNPVQ.

I think upgrading to WinUI 2.6.2 might introduce crashes on 19h1.. @jaimecbernardo knows the details.
It's the reason why https://github.com/microsoft/PowerToys/issues/12846 is blocked. There's a link to the WinUI issue there as well

I've just tested this PR.
Settings won't open, crashes at startup with an error in the event log when running in Windows 10.0.18363.
https://github.com/microsoft/microsoft-ui-xaml/issues/5435#issuecomment-903635572

closing PR due to 5435.  We need a 2.7 fix


Improved notebooks cluster management

An analyst can control the power of their compute environment so as to make tradeoffs between cost and power, without having to manage their own VMs directly.
Needs visual design.
Probably worth it to validate our mocks before implementing
"wireframes+" interaction + viz design for controls providing the ability to:

view the cost of their compute
play/pause/stop their compute
up/downsize their compute (including adding/removing spark workers)

Many of the actual controls are currently in the firecloud UI. It's about presenting them in a different way. One could have a 'runtime' element somewhere in the UI with a play/pause/stop button, the cost, and an up/downsize icon that opened up a form for specifying the size of runtime compute.
Needs eng design:
Some of the things we want to do (pause/unpause) have endpoints in leo
Up and downsizing the compute does not. For adding/removing spark workers, this can be done on the existing cluster. For changing the machine type (CPU/RAM) of the master/worker VMs, that will involve destroying the cluster and adding a new one. I think the best thing to do here is to warn the user that this is slow, snapshot the disk, create the new cluster, load the old disk (so the environment carries over) and then destroy the old cluster. We can put up a spinner for the 5min wait time.
External dependency:
Will likely need backend work from the leonardo team
Visual Design
https://projects.invisionapp.com/d/main#/console/13746874/286853469/preview
Eng Design
For this MVP, we will only consider up to two of the most recently created clusters (by createdDate in the Leo response). The states are:

one cluster, status "Creating" => cluster not ready
two clusters, one with status "Creating" => preparing new cluster (poll for changes)
two clusters, both with status "Ready" => new cluster ready
one cluster, status "Ready" => normal state

Leo does not appear to support adding/removing workers, but this should be tested by attempting to "create" an existing cluster with different values.
The UI needs only to create the new cluster when prompted by the user and display the status as specified above. The "current cluster" is the least recently created with a status of "Ready." On the user's action, the current cluster is deleted causing the new cluster to immediately become the current cluster.

We could break this ticket up and only do cost + play/pause now while we wait for the backend work on replacing a cluster

Designs in Invision!

@bradtaylor    will clarify some additional questions with Lou/Marty RE placement of widget

We're going to place the widget directly to the left of the user profile at the top level of nav, for now. We can revisit where on the page it lives with Marty + Lou once we actually get the thing in there


"xhr poll error"

hi. i'm try to connect my socket io server but it return ON ("error") with mess = "xhr poll error"
Ex : connect to "http://chat.socket.io/"
DEV please fix it . thank so much :D

Unable to reproduce error.


Print log file and line for better debuging

Changes proposed in this PR:

Add file and line to the logger
Delete log tag for linux distro (as it is obvious and consumes space)

Self evaluation:

Build test: [X]Passed [ ]Failed [ ]Skipped
Run test: [X]Passed [ ]Failed [ ]Skipped

Signed-off-by: Jihoon Lee<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.4.20191203. Thank you for submitting PR #394. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://nnsuite.mooo.com/.

:octocat: cibot: @zhoonit, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://nnsuite.mooo.com/nntrainer/ci/repo-workers/pr-checker/394-202008032130250.6725161075592-127596854a92ee43a33e71d1bd076fd301bcd9dd/.


Drop yaml libs use custom parser

WIP :raised_hand:
Simple regex parser implementation for user configs to replace PyYAML AND ruamel.yaml. Users on various platforms and Python versions seem to encounter various problems with both libs when installing Cookiecutter, which is a huge problem (see #557 :worried: ).
Please let me know your thoughts!
(Related to #568)

Known issue: Bad user configs do not cause an InvalidConfiguration error.
Won't update the tests until we are on the same page regarding the implementation. :point_left:

Hooray to saying goodbye to the overkill of heavy-duty formal YAML libraries! That said, config_parser is really close to being ConfigParser. How about something like simple_yaml instead? That is a more obvious name, and lends itself to being separated out into a different, pure Python YAML parser project. :smile:

Note: See gitter

@hackebrot Have you considered making this a separate library that we then include as a dependency?

Closing in favor of #621


Improve playbook to import/authorize ssh_key

To ssh as root to the vm hetzner, local, we use these commands to import our ssh_key as authorized key
sshpass -f pwd.txt ssh -o StrictHostKeyChecking=no<EMAIL_ADDRESS>-p 5222 "mkdir ~/.ssh && chmod 700 ~/.ssh && touch ~/.ssh/authorized_keys && chmod 600 ~/.ssh/authorized_keys"
sshpass -f pwd.txt ssh-copy-id -o StrictHostKeyChecking=no -i ~/.ssh/id_rsa.pub<EMAIL_ADDRESS>-p 5222

We should improve the existing playbook to :

Create a generic private/public key and host it on a github repo
Import the public ssh_key as authorized user within the target vm

Interesting articles :

http://blog.appliedinformaticsinc.com/how-to-manage-ssh-keys-using-ansible/
Authorize key
Generate ssh key and create user

I don't think this should be done for the following reasons:
For the case where we are creating VM for local development, the new_iso.sh script already grabs the host's keys (see https://github.com/snowdrop/cloud-native-infra/blob/9c358170b1c8157fa77f3172f0d16ef99a2f56da/virtualbox/build-centos-iso/cloud-init/new-iso.sh#L59) and imports them into the VM allowing the user to then perform password-less login into the VM without having to execute any additional command.
For the case where we want to install Openshift onto a cloud machine using our scripts, it would not be good to have a publicly accessible set of public/private keys, because then anyone could access the machine as a root user.

Feel free to look at https://github.com/eupraxialabs/tools and see if the ssh-import-id feature is something that can be worked into some of your use cases. See the section: "Special Tool in the Bag" The public image is at https://hub.docker.com/r/eupraxialabs/tools/
Clearly, you would want to parameterize the public key username in Ansible..
If we can be of any assistance, please let me know.

Other references: https://github.com/dustinkirkland/ssh-import-id

@davidjbrewer Thanks, I'll take a look when I have some time :)

@cmoulliard I propose we close this since we now have support for importing keys for GitHub.


Removing needs from held when deleting.

Fixes #2652

the pr does not fix the issue at hand, see comment above for details

Propagating all the necessary changes to the client might prove a challenge. If, for example, a persona is loaded that resides on a different node, we won't know about the deletion of an associated need with our current system.
We'd need to create an update channel to any node we have data from.


Default false for directories option

Most of the time, we use this library to iterate the files inside a path, but it also returns the directories always since true is the default value. It would be great to have directories: false as default value so that we don't have to pass { directories: false } everytime. It would be a better sensible default IMHO

I haven't thought too much about it, lots of the broccoli related tooling that leverages this lib used the directory entry to know to create subdirs and whatnot (avoiding the penalty of calling mkdir -p too often).
I do tend to share your perspective here though, that the cases where you want the directory entries are much fewer than the ones where you don't.

Although this may be true, even with a major version bump this change seems like an upgrade hazard, so I'm not sure its worth making the change.

Thanks, @stefanpenner @rwjblue that makes perfect sense.