Add InferenceObjects as a chain_type

This PR is a prototype of adding InferenceObjects.InferenceData as a chain_type for sample. Here's a working example:
julia> using Turing, InferenceObjects

julia> @model function foo()
           x ~ Normal()
           y ~ filldist(Normal(), 2)
           z = similar(y, 2, 3)
           for i in axes(z, 1), j in axes(z, 2)
               z[i, j] ~ Normal()
           end
       end;

julia> idata_prior = sample(foo(), Prior(), 1_000; chain_type=InferenceData)
Sampling 100%|█████████████████████████████████████████████████████████████████| Time: 0:00:00
InferenceData with groups:
  > prior
  > sample_stats_prior

julia> idata_post = sample(foo(), NUTS(), MCMCThreads(), 1_000, 4; chain_type=InferenceData)
┌ Info: Found initial step size
└   ϵ = 0.9
┌ Info: Found initial step size
└   ϵ = 3.2
┌ Info: Found initial step size
└   ϵ = 1.6
┌ Info: Found initial step size
└   ϵ = 1.6
Sampling (4 threads) 100%|█████████████████████████████████████████████████████| Time: 0:00:02
InferenceData with groups:
  > posterior
  > sample_stats

julia> idata = merge(idata_post, idata_prior)
InferenceData with groups:
  > posterior
  > sample_stats
  > prior
  > sample_stats_prior

julia> idata.posterior
Dataset with dimensions:
  Dim{:draw} Sampled{Int64} Base.OneTo(1000) ForwardOrdered Regular Points,
  Dim{:chain} Sampled{Int64} Base.OneTo(4) ForwardOrdered Regular Points,
  Dim{:y_dim_1} Sampled{Int64} Base.OneTo(2) ForwardOrdered Regular Points,
  Dim{:z_dim_1} Sampled{Int64} Base.OneTo(6) ForwardOrdered Regular Points
and 3 layers:
  :x Float64 dims: Dim{:draw}, Dim{:chain} (1000×4)
  :y Float64 dims: Dim{:y_dim_1}, Dim{:draw}, Dim{:chain} (2×1000×4)
  :z Float64 dims: Dim{:z_dim_1}, Dim{:draw}, Dim{:chain} (6×1000×4)

with metadata Dict{String, Any} with 3 entries:
  "start_time" => 1.66929e9
  "created_at" => "2022-11-24T11:29:24.947"
  "stop_time"  => 1.66929e9

julia> idata.sample_stats
Dataset with dimensions:
  Dim{:draw} Sampled{Int64} Base.OneTo(1000) ForwardOrdered Regular Points,
  Dim{:chain} Sampled{Int64} Base.OneTo(4) ForwardOrdered Regular Points
and 12 layers:
  :lp               Float64 dims: Dim{:draw}, Dim{:chain} (1000×4)
  :n_steps          Int64 dims: Dim{:draw}, Dim{:chain} (1000×4)
  :is_accept        Bool dims: Dim{:draw}, Dim{:chain} (1000×4)
  :acceptance_rate  Float64 dims: Dim{:draw}, Dim{:chain} (1000×4)
  :log_density      Float64 dims: Dim{:draw}, Dim{:chain} (1000×4)
  :energy           Float64 dims: Dim{:draw}, Dim{:chain} (1000×4)
  :energy_error     Float64 dims: Dim{:draw}, Dim{:chain} (1000×4)
  :max_energy_error Float64 dims: Dim{:draw}, Dim{:chain} (1000×4)
  :tree_depth       Int64 dims: Dim{:draw}, Dim{:chain} (1000×4)
  :diverging        Bool dims: Dim{:draw}, Dim{:chain} (1000×4)
  :step_size        Float64 dims: Dim{:draw}, Dim{:chain} (1000×4)
  :step_size_nom    Float64 dims: Dim{:draw}, Dim{:chain} (1000×4)

with metadata Dict{String, Any} with 3 entries:
  "start_time" => 1.66929e9
  "created_at" => "2022-11-24T11:29:24.947"
  "stop_time"  => 1.66929e9

Note that all of this is type piracy, and it would probably be much cleaner to move this code into a DynamicPPLInferenceObjects.jl glue package that Turing instead depends on. However, we currently need the metadata function in order to retrieve sampling statistics and logevidence. Is there any way that functionality could be moved to DynamicPPL?
Relates https://github.com/TuringLang/MCMCChains.jl/issues/381

Also, the getparams functionality would need to be in DynamicPPL or AbstractMCMC.

  :z Float64 dims: Dim{:z_dim_1}, Dim{:draw}, Dim{:chain} (6×1000×4)

It seems getparams still flattens z. Is there a way to access an unflattened form?

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 31 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-1.7%) to 79.503%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/inference/Inference.jl
0
31
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-1.7%

Covered Lines:
1152

Relevant Lines:
1449

💛  - Coveralls

Note that all of this is type piracy, and it would probably be much cleaner to move this code into a DynamicPPLInferenceObjects.jl glue package that Turing instead depends on. However, we currently need the metadata function in order to retrieve sampling statistics and logevidence.

For demonstration purposes, I've coded up a more complete example of what I mean at https://github.com/sethaxen/DynamicPPLInferenceObjects.jl

Ah, this has evolved substantially since this PR. https://github.com/TuringLang/DynamicPPL.jl/pull/465 supersedes this PR by adding an InferenceObjects extension to DynamicPPL. That was held up by trying to get predict as part of DPPL or APPL's API (https://github.com/TuringLang/DynamicPPL.jl/issues/466 and https://github.com/TuringLang/AbstractPPL.jl/pull/81). Personally I agree with https://github.com/TuringLang/AbstractPPL.jl/pull/81#pullrequestreview-1357854683 that it makes more sense to add it to DPPL.
I do think that work can be done fairly quickly, but I need to wrap up a few things first before resuming.


M> Translating RChain Vision Video

Had a quick discussion about building the Global RChain Community presence with @edeykholt and he suggested translating the RChain Vision video found on the RChain Coop website. I think that would be a great idea and we should set a budget for this and other future translations.
@sportshark Would you like to engage members of the Chinese community to provide an accurate translation of the video into Chinese? This would fit well into the RChain Members Chinese site  that you had created.
@patrick727 Do you happen to have the transcript for the RChain Vision video that you worked on previously? That would make for a more accurate translation.

@entropee i got german subtitles for rchain vision, https://www.youtube.com/watch?v=JyI9XvObRCc

I could do subtitles for this video in Greek.

优酷Youku: RChain Vision video
Bilibili: Rchain Vision video
爱奇艺 Iqiyi : Rchain Vision video
Done by @zsluedem as cited here : https://github.com/rchain/Members/issues/289#issuecomment-362889679

Since we decided to not translate video subtitles anymore, should this be closed? @dckc

Point me to the record of the decision? I don't recall whether it was binding.
Regardless, if the goal is achieved and noone plans to do more work, an issue should be closed.
See also https://github.com/rchain/bounties/wiki/Bounty-Task-Guides#pruning-issues regarding pruning issues where the goals aren't met.

@Mervyn853 Okay, so what I made out of this is that it would be better if you created a blog article about this video, highlighting the important content, BUT it's up to you, you can still do the subtitle translation.

What is the status on this? @Mervyn853


Neovim theme background colors are being stripped when scrolling

From @jasin on November 30, 2017 15:25
This is a similar problem to the one listed here, but since someone decided to lock that thread I'll start my own.
#1706
@zadjii-msft
Simply put, I am running ubuntu from the windows store
lsb_release -a returns 16.04.3 LTS
You can see the problem in neovim, when the file is scrolled down the background color is stripped. I have no idea if this problem has been fixed in 16.04.3 LTS or if this is only fixed in the insider version??  Any sort of tips on how I can further look into the problem I'd happy to look into it and report back.

Copied from original issue: Microsoft/WSL#2710

From @rodrymbo on November 30, 2017 17:47
You could fire up sshd and log in to it with PuTTY or similar and see if the same background problem continues. If it works okay in a terminal but not in the console, then this is a console issue...
Work continues to make the console work more and more like a terminal...

From @zadjii-msft on November 30, 2017 19:12
@jasin What windows build# are you on? There was a bit of churn in that area recently. I think I already fixed that, but if not, knowing the specific build number will help me track it down.

From @zadjii-msft on December 1, 2017 17:16
@jasin could you post what the following python script looks like when you run it in the console?
import sys
import time

def csi(seq):
    sys.stdout.write('\x1b[{}'.format(seq))

def write(s):
    sys.stdout.write(s)

if __name__ == '__main__':
    print('Clearing the line paints to the end?')
    print('\E[42mfoo\E[43m\E[KTest')
    csi('42m')
    write('foo')
    csi('43m')
    csi('K')
    write('Test')
    csi('m')
    print('')
    print('\E[42mfoo\E[43m\E[K\\nTest')
    csi('42m')
    write('foo')
    csi('43m')
    csi('K')
    write('\nTest')
    csi('m')
    print('')
    print('\E[42mfoo\E[43m\E[1KTest')
    csi('42m')
    write('foo')
    csi('43m')
    csi('1K')
    write('Test')
    csi('m')
    print('')
    print('\E[42mfoo\E[43m\E[1K\\nTest')
    csi('42m')
    write('foo')
    csi('43m')
    csi('1K')
    write('\nTest')
    csi('m')
    print('')
    print('\E[42mfoo\E[43m\E[2KTest')
    csi('42m')
    write('foo')
    csi('43m')
    csi('2K')
    write('Test')
    csi('m')
    print('')
    print('\E[42mfoo\E[43m\E[2K\\nTest')
    csi('42m')
    write('foo')
    csi('43m')
    csi('2K')
    write('\nTest')
    csi('m')
    print('')
    print("\x1b[48;2;64;128;255mX\x1b[48;2;128;128;255m\x1b[KX")
    csi('m')
    print('')

You can just paste that into a file like gh2710.py and run it with python gh2710.py

From @jasin on December 1, 2017 11:18
@zadjii-msft Windows Version 10.0.16299 Build 16299

From @jasin on December 1, 2017 19:0

Hope this helps

From @zadjii-msft on December 1, 2017 19:7
Welp, that's exactly how that's supposed to look. Shoot. This must be some other weirdness then.
Just to help investigate, are there any specific plugins you have installed? Have you modified your neovim configuration at all? (I'm not really familiar with neovim, but for normal vim I'm talking about the .vimrc file)
I'll file a bug on my end to make sure this gets tracked down.

From @jasin on December 1, 2017 20:10
Well I am not sure I fully understand the problem so it seems.
I was using neovim during an ssh session to my FreeBSD box. That is where I do my development. I decided to quickly install the neovim package from the unstable ppa and I do not see the issue locally that I am experiencing during an ssh session to my FreeBSD box. I do run zsh shell on that server box.
To note, I did run the py scrypt on my FreeBSD box through ssh session and you said that looked ok, so this must specifically be a neovim/FreeBSD issue over SSH. But that is as far as I am willing to comment. I'm not sure I know enough to look into it further without some possible clues.

From @jasin on December 1, 2017 20:39
Further investigation shows this is not just limited to neovim, I see the issue in vim as well over SSH to FreeBSD, but slightly different, opening up a file now shows me this without even having to scroll. I have also ruled out the shell, makes no difference, didn't think it would, but I double checked using bash shell

From @jasin on December 1, 2017 20:57
Here you go
jasin:tyr~$ infocmp
#       Reconstructed via infocmp from file: /usr/local/share/misc/terminfo.db
xterm-256color|xterm with 256 colors,
        am, bce, ccc, km, mc5i, mir, msgr, npc, xenl,
        colors#0x100, cols#80, it#8, lines#24, pairs#0x7fff,
        acsc=``aaffggiijjkkllmmnnooppqqrrssttuuvvwwxxyyzz{{||}}~~,
        bel=^G, blink=\E[5m, bold=\E[1m, cbt=\E[Z, civis=\E[?25l,
        clear=\E[H\E[2J, cnorm=\E[?12l\E[?25h, cr=\r,
        csr=\E[%i%p1%d;%p2%dr, cub=\E[%p1%dD, cub1=^H,
        cud=\E[%p1%dB, cud1=\n, cuf=\E[%p1%dC, cuf1=\E[C,
        cup=\E[%i%p1%d;%p2%dH, cuu=\E[%p1%dA, cuu1=\E[A,
        cvvis=\E[?12;25h, dch=\E[%p1%dP, dch1=\E[P, dim=\E[2m,
        dl=\E[%p1%dM, dl1=\E[M, ech=\E[%p1%dX, ed=\E[J, el=\E[K,
        el1=\E[1K, flash=\E[?5h$<100/>\E[?5l, home=\E[H,
        hpa=\E[%i%p1%dG, ht=^I, hts=\EH, ich=\E[%p1%d@,
        il=\E[%p1%dL, il1=\E[L, ind=\n, indn=\E[%p1%dS,
        initc=\E]4;%p1%d;rgb\:%p2%{255}%*%{1000}%/%2.2X/%p3%{255}%*%{1000}%/%2.2X/%p4%{255}%*%{1000}%/%2.2X\E\\,
        invis=\E[8m, is2=\E[!p\E[?3;4l\E[4l\E>, kDC=\E[3;2~,
        kEND=\E[1;2F, kHOM=\E[1;2H, kIC=\E[2;2~, kLFT=\E[1;2D,
        kNXT=\E[6;2~, kPRV=\E[5;2~, kRIT=\E[1;2C, kb2=\EOE, kbs=^H,
        kcbt=\E[Z, kcub1=\EOD, kcud1=\EOB, kcuf1=\EOC, kcuu1=\EOA,
        kdch1=\E[3~, kend=\EOF, kent=\EOM, kf1=\EOP, kf10=\E[21~,
        kf11=\E[23~, kf12=\E[24~, kf13=\E[1;2P, kf14=\E[1;2Q,
        kf15=\E[1;2R, kf16=\E[1;2S, kf17=\E[15;2~, kf18=\E[17;2~,
        kf19=\E[18;2~, kf2=\EOQ, kf20=\E[19;2~, kf21=\E[20;2~,
        kf22=\E[21;2~, kf23=\E[23;2~, kf24=\E[24;2~,
        kf25=\E[1;5P, kf26=\E[1;5Q, kf27=\E[1;5R, kf28=\E[1;5S,
        kf29=\E[15;5~, kf3=\EOR, kf30=\E[17;5~, kf31=\E[18;5~,
        kf32=\E[19;5~, kf33=\E[20;5~, kf34=\E[21;5~,
        kf35=\E[23;5~, kf36=\E[24;5~, kf37=\E[1;6P, kf38=\E[1;6Q,
        kf39=\E[1;6R, kf4=\EOS, kf40=\E[1;6S, kf41=\E[15;6~,
        kf42=\E[17;6~, kf43=\E[18;6~, kf44=\E[19;6~,
        kf45=\E[20;6~, kf46=\E[21;6~, kf47=\E[23;6~,
        kf48=\E[24;6~, kf49=\E[1;3P, kf5=\E[15~, kf50=\E[1;3Q,
        kf51=\E[1;3R, kf52=\E[1;3S, kf53=\E[15;3~, kf54=\E[17;3~,
        kf55=\E[18;3~, kf56=\E[19;3~, kf57=\E[20;3~,
        kf58=\E[21;3~, kf59=\E[23;3~, kf6=\E[17~, kf60=\E[24;3~,
        kf61=\E[1;4P, kf62=\E[1;4Q, kf63=\E[1;4R, kf7=\E[18~,
        kf8=\E[19~, kf9=\E[20~, khome=\EOH, kich1=\E[2~,
        kind=\E[1;2B, kmous=\E[M, knp=\E[6~, kpp=\E[5~,
        kri=\E[1;2A, mc0=\E[i, mc4=\E[4i, mc5=\E[5i, meml=\El,
        memu=\Em, oc=\E]104\007, op=\E[39;49m, rc=\E8,
        rep=%p1%c\E[%p2%{1}%-%db, rev=\E[7m, ri=\EM,
        rin=\E[%p1%dT, ritm=\E[23m, rmacs=\E(B, rmam=\E[?7l,
        rmcup=\E[?1049l, rmir=\E[4l, rmkx=\E[?1l\E>,
        rmm=\E[?1034l, rmso=\E[27m, rmul=\E[24m,
        rs1=\Ec\E]104\007, rs2=\E[!p\E[?3;4l\E[4l\E>, sc=\E7,
        setab=\E[%?%p1%{8}%<%t4%p1%d%e%p1%{16}%<%t10%p1%{8}%-%d%e48;5;%p1%d%;m,
        setaf=\E[%?%p1%{8}%<%t3%p1%d%e%p1%{16}%<%t9%p1%{8}%-%d%e38;5;%p1%d%;m,
        sgr=%?%p9%t\E(0%e\E(B%;\E[0%?%p6%t;1%;%?%p5%t;2%;%?%p2%t;4%;%?%p1%p3%|%t;7%;%?%p4%t;5%;%?%p7%t;8%;m,
        sgr0=\E(B\E[m, sitm=\E[3m, smacs=\E(0, smam=\E[?7h,
        smcup=\E[?1049h, smir=\E[4h, smkx=\E[?1h\E=,
        smm=\E[?1034h, smso=\E[7m, smul=\E[4m, tbc=\E[3g,
        u6=\E[%i%d;%dR, u7=\E[6n, u8=\E[?%[;0123456789]c,
        u9=\E[c, vpa=\E[%i%p1%dd,

From @zadjii-msft on December 1, 2017 20:43
@jasin can you post the output of infocmp on your FreeBSD machine?

From @msundin on December 29, 2017 3:38
I don't know why #1706 is locked? It is still very valid and describes the bug well.

From @msundin on December 27, 2017 19:31
I can confirm this bug in Ubuntu 16.04 after Fall Creators update and via SSH to FreeNAS (FreeBSD). When scrolling, the background color (in this case grey) disappears where there is no text. Both in Vim and Neovim. This makes it impossible to use sophisticated color schemes in Vim/Neovim:

From @sus007 on January 5, 2018 11:18
I too confirm this bug in my system:

Operating System and version: Windows 10 Pro Insider Preview / Version: 1709 / OS-BUILD: 17025.1000

Ubuntu 16.04.03 LTS

VIM- Vi IMproved 8.0 (2016 Sep 12, compiled Jan 02 2017)

I've tried every suggested methods of #1706 , still while scrolling down the entire color-scheme sort of fades out, turns colorless as we can see below.

From @zadjii-msft on January 4, 2018 22:17
Okay yea I'm seeing this again.
To limit the repro variables for myself:

it repro's in normal vim
switch to a colorscheme with a RGB bg for text - these will make it really obvious:

let g:solarized_termcolors=256
colo solarized

:redraw! will get the whole buffer looking right, and then any scroll up down will only have the right bg filling the top or bottom line

From @zadjii-msft on January 11, 2018 22:51
I got this figured out. I was off-by-one on copying the RGB attributes of the last run of characters.
Before:

After:

gnome-terminal (for reference):

I'm gonna add some tests and get this code reviewed. Thanks for all the persistence and patience folks!

As of today, with the latest Windows Insiders build, the fade away issue while scrolling has been resolved but still there's this ugly grey bar masking every text in the document while the color stays intact.

Operating System and version: Windows 10 Pro Insider Preview / Version: 1709 / OS-BUILD: 17074.1002
Ubuntu 16.04.03 LTS
VIM- Vi IMproved 8.0 (2016 Sep 12, compiled Jan 02 2017)

As a temporary workaround, setting term to tmux-256color in .vimrc fixes this for me.

Happily I found this bug listing.  It seems to be an xterm default rendering issue.  I added the following to my .bashrc:
(line 115):  export TERM=screen-256color
The result is vim properly renders background colours all the way to the end.  Using default xterm-256color I get the results of the gif above.  Interesting note:  This seems to be related to new line characters.  Using delete/backspace on a line and causing the text to shift left (specifically left) caused the background to render all the way to the right again on that line specifically.

I'll add it to my backlog and come back and investigate. I thought I fixed this for Spring Creator's Update, but I don't really know when that's actually getting released....

It may well have been resolved in the spring update, I'm not running a
preview build. Here's hoping for a quick test cycle though.
On Fri, Apr 20, 2018, 13:02 Mike Griese<EMAIL_ADDRESS>wrote:

I'll add it to my backlog and come back and investigate. I thought I fixed
this for Spring Creator's Update, but I don't really know when that's
actually getting released....
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/console/issues/70#issuecomment-383159611,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABNRVnitb6foR4VEkQcpII2oE94Qo5X7ks5tqhSTgaJpZM4SHvdT
.

@jasin For the record, 16299 is still Fall Creator's Update. "Spring Creator's Update" (which is I guess just "April Update" now, don't ask me I'm not in marketing) should have a build number that's something like "17134" or greater.
This issue has kinda evolved from one manifestation of the bug to the next similar, yet different incarnation. When I get to it, I'll let you know.

I can't seem to get this to repro anymore, in any fashion. If someone can get me a repro (color scheme/.vimrc) that this still repros on Insider's with, I'll be happy to re-open, but I think it's fixed now.

Operating System and version: Windows 10 Home / Version: 1809 / OS-BUILD: 17763.194
Arch Linux on WSL (Linux Pablo-OMEN 4.4.0-17763-Microsoft #194-Microsoft Mon Dec 03 17:58:00 PST 2018 x86_64 GNU/Linux)
vim package version: vim 8.1.0542-1

I suppose you can't repro completely because I'm not using Ubuntu on WSL (which would be normal). But here is the screenshot of the bug.

:redraw! fix it perfectly.
Here my .vimrc (which is basically the default one) and the colorscheme (base16)
https://gist.github.com/Pablo1107/0afb0cb31bb8cb8df6aacd0b8243a762

Issue is still here:

I already put export TERM=screen-256color into my bashrc, and put set t_Co=256 into my vimrc. :redraw! does nothing. I've already source'd both my bashrc and vimrc.

To reproduce the above, install vim-gtk on Debian (latest windows stable), then try installing the summerfruit256 colorscheme

In standard Vim, putting this into the vimrc did the trick:
if &term =~ '256color'     " Disable Background Color Erase (BCE) so that color schemes     " work properly when Vim is used inside tmux and GNU screen.     set t_ut= endif

Thanks @ndrewxie!
This makes it seem like a tmux (tmux/tmux#109) and screen[1] issue moreso than a conhost issue.
[1] from man 1 screen:
       bce [on|off]

       Change  background-color-erase  setting.  If  "bce"  is  set  to  on,  all  characters cleared by an erase/in‐
       sert/scroll/clear operation will be displayed in the current background color.  Otherwise  the  default  back‐
       ground color is used.

When I have bce off in my .screenrc, I see the same issue:

when I set bce on, however:

I'm on 1903 (Release Preview), using Ubuntu 18.04 in WSL, neovim and "gruvbox" colorscheme. I can still reproduce the issue. The only solution apparently is to put " let g:gruvbox_termcolors=16" in my init.vim, but this completely change all the colors

I can reproduce this with a minimal vim:
vim -Nu NONE +"colorscheme jellybeans" +"help help" and scrolling around inside the help window and some parts of the background are green instead of dark grey.
Where jellybeans is this specific version. (Also occurs with zellner, xoria256, sandydune, etc. but couldn't repro with any colors shipped with vim7. Probably because vim7 doesn't ship with schemes using 256 colors.)
I'm on Windows 1909 (OS Build 18363.657). (Public version, not insider.)
Using WSL: Ubuntu 14.04.5 LTS
VIM - Vi IMproved 7.4 (2013 Aug 10, compiled Nov 24 2016 16:43:18)
Included patches: 1-52
Extra patches: 8.0.0056
&term == xterm-256color
&t_Co == 256
Running a terminal as bash.exe ("Microsoft Bash Launcher")
Strangely, I only noticed this recently.  Maybe after my Windows Update on Friday?
Either of these lines will make the problem go away, but at the cost of reduced colours:
set term=xterm
set t_Co=16

I'm using the bash.exe that came with installing WSL through the Windows Store (2 years ago). I'm not using "Windows Terminal (Preview)" from the Windows Store (testing that, the issue doesn't occur because &term is win32 and &t_Co is 16 -- so it doesn't support 256 colours). The WSL repo asks to report issues here, so I assume my experience is relevant to this bug?

This would be a great issue for @Kayla<EMAIL_ADDRESS>& team to explore. Thanks for filing David.
Rich.
From: David Briscoe<EMAIL_ADDRESS>Sent: Wednesday, March 11, 2020 15:35
To: microsoft/terminal<EMAIL_ADDRESS>Cc: Rich Turner<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [microsoft/terminal] Neovim theme background colors are being stripped when scrolling (#70)
I can reproduce this with a minimal vim:
vim -Nu NONE +"colorscheme jellybeans" +"help help" and scrolling around inside the help window and some parts of the background are green instead of dark grey.
Where jellybeans is this specific versionhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fidbrii%2Fdaveconfig%2Fblob%2F3d03c751a6af0908191170e4a791016daedd1590%2Fmulti%2Fvim%2Fcolors%2Fjellybeans.vim&data=02|01|richturn%40microsoft.com|2dfa3ce66c6244d0991e08d7c60c78a9|72f988bf86f141af91ab2d7cd011db47|1|0|637195629170393194&sdata=xNbW0uw2K6e9v1o55muQoZTjRxiGARrofrcOPaSRgOQ%3D&reserved=0. (Also occurs with zellner, xoria256, sandydune, etc. but couldn't repro with any colors shipped with vim7. Probably because vim7 doesn't ship with schemes using 256 colors.)
I'm on Windows 1909 (OS Build 18363.657). (Public version, not insider.)
Using WSL: Ubuntu 14.04.5 LTS
VIM - Vi IMproved 7.4 (2013 Aug 10, compiled Nov 24 2016 16:43:18)
Included patches: 1-52
Extra patches: 8.0.0056
&term == xterm-256color
&t_Co == 256
Running a terminal as bash.exe ("Microsoft Bash Launcher")
Strangely, I only noticed this recently. Maybe after my Windows Update on Friday?
Either of these lines will make the problem go away, but at the cost of reduced colours:
set term=xterm
set t_Co=16
I'm using the bash.exe that came with installing WSL through the Windows Store (2 years ago). I'm not using "Windows Terminal (Preview)" from the Windows Store (testing that, the issue doesn't occur because &term is win32 and &t_Co is 16 -- so it doesn't support 256 colours). The WSL repo asks to report issues herehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2FWSL%2Fblob%2Fmaster%2FREADME.md%23reporting-issues-in-windows-console-or-wsl-text-renderinguser-experience&data=02|01|richturn%40microsoft.com|2dfa3ce66c6244d0991e08d7c60c78a9|72f988bf86f141af91ab2d7cd011db47|1|0|637195629170403150&sdata=2H4EkK1R1m0voU3e4P%2BiBTZrtox%2F2SIqW6ttKg8i2bU%3D&reserved=0, so I assume my experience is relevant to this bug?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2Fterminal%2Fissues%2F70%23issuecomment-597913799&data=02|01|richturn%40microsoft.com|2dfa3ce66c6244d0991e08d7c60c78a9|72f988bf86f141af91ab2d7cd011db47|1|0|637195629170403150&sdata=xHiwixwxICxBYPcfLQQ7s6JTQP2hQLNeUeQI0dNu8eY%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAAHK3HXX4EUOHUAHAKO3AA3RHAG2HANCNFSM4EQ665JQ&data=02|01|richturn%40microsoft.com|2dfa3ce66c6244d0991e08d7c60c78a9|72f988bf86f141af91ab2d7cd011db47|1|0|637195629170413107&sdata=mSLR8eXtCM%2BYEK0qyQqE19FM3FwKVlBjsToKR7l8IRY%3D&reserved=0.

@idbrii since this issue is closed and you seem to be having some slightly different symptoms, would you mind filling a new one so that we might track it appropriately? Thanks.


Is a fully qualified attribute name needed in an iot-hub device query?

I have a question about attribute notation in iot-hub-device-query-language.
azure-docs/articles/iot-hub/iot-hub-devguide-query-language.md
Is there a way to write an attributename in a query like
WHERE myProp = 'abc'
instead of
WHERE properties.reported.myProp = 'abc'
Thank you.

@mwotruba Thanks for the feedback. We are actively investigating and will get back to you soon.

@mwotruba From all examples and SDK references, the fully qualified attribute must be used.
Including the corrected URL here for reference: IoT Hub query language for device twins, jobs, and message routing

Please let me know if you have any additional questions.
Regards,
Mike

@mwotruba We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


Trigger GitHub CI workflow dispatch instead of Travis CI

This PR changes deployment trigger from Travis to GitHub. Now only a single route /trigger is exposed. When called with
{"repository": "...", "branch": "..."}

it will look inside naucse, find the corresponding repository and check if it should be updated (same as before). If yes, the build will be triggered via a Github Actions CI workflow in the pyvec/naucse.python.cz repository (https://github.com/pyvec/naucse.python.cz/pull/707).

I've added one more simplification, and a rudimentary "front page".
@Kobzol, does this look OK?
@mikicz, if you want to take a look, please do, but I don't think it's necessary. Thanks for running the hooks!

@encukou It looks OK, thanks!

@hroncok Would it be appropriate to use @pyvecpusher's token for naucse-hooks (GHA)?


Wrong cluster size when zooming out

I have implemented the clustering and it works fine, when I use the default MKAnnotationViews, so when I am not overriding the method mapView:ViewForAnnotation:
But I want to have different colored annotations for different sized clusters. I have implemented the method mapView:viewForAnnotation: as following:
    var annotationView = mapView.dequeueReusableAnnotationViewWithIdentifier("AnnotationView")
    if annotationView == nil {
        annotationView = MKAnnotationView(annotation: annotation, reuseIdentifier: "AnnotationView")
    }
    annotationView?.image = UIImage(named: "OneLocationAnnotationImage")!

    if annotation.isKindOfClass(CCHMapClusterAnnotation) {
        let clusterAnnotation = annotation as! CCHMapClusterAnnotation
        let count = clusterAnnotation.annotations.count
        if count == 1 {
            annotationView?.image = UIImage(named: "OneLocationAnnotationImage")!
        } else if count < 3 {
            annotationView?.image = UIImage(named: "LowDensityAnnotation")!
        } else if count < 5 {
            annotationView?.image = UIImage(named: "LowMediumDensityAnnotation")!
        } else if count < 7 {
            annotationView?.image = UIImage(named: "HighMediumDensityAnnotation")!
        } else {
            annotationView?.image = UIImage(named: "HighDensityAnnotation")!
        }

        return annotationView
    }
    return annotationView

When I now zoom in, the clustering works fine, but when I zoom out, the image is not returning to the one at the beginning, which should be the case. So the question is, if I am doing the right approach to get to the result I want?

The annotation views are recycled when zooming in and out. This means that the image needs to be updated in both mapView:viewForAnnotation: as well as mapClusterController:willReuseMapClusterAnnotation:. I suggest you have a look at MapViewController that comes with the iOS example – this class implements both methods to update the image displayed for the annotation views.

Thank you, that was the hint I needed :) Now the clustered annotations work properly!
The CCHMapClusterController is really well done and easy to implement, but there are two things I would change, because then it is even easier to use:
I had to search for the ClusterAnnotationView class in the example project, maybe you could include them in the Framework as "CCHClusterAnnotationView.h/m" and you could create an header importing all .h files, so that when you are using Swift you only have to import one file in the bridging header.


[WIP] doc: add docs for hertz client retry

What type of PR is this?

docs
What this PR does / why we need it (en: English/zh: Chinese):

en: add docs for hertz client retry
zh:新增 hertz client retry 文档
Which issue(s) this PR fixes:

refer to https://github.com/cloudwego/cloudwego.github.io/pull/369 and improve it.


dev(narugo): add ai-created

Codecov Report

Merging #28 (e0fbedd) into main (1a98793) will increase coverage by 0.06%.
The diff coverage is 100.00%.

:exclamation: Current head e0fbedd differs from pull request most recent head eb7e990. Consider uploading reports for the commit eb7e990 to get more accurate results

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main      #28      +/-   ##
==========================================
+ Coverage   98.16%   98.22%   +0.06%
==========================================
  Files          36       37       +1
  Lines         816      847      +31
==========================================
+ Hits          801      832      +31
  Misses         15       15

Flag
Coverage Δ

unittests
98.22% <100.00%> (+0.06%)
:arrow_up:

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

Impacted Files
Coverage Δ

imgutils/validate/__init__.py
100.00% <100.00%> (ø)

imgutils/validate/aicheck.py
100.00% <100.00%> (ø)

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


Use CURL_WRITEFUNC_ERROR to correctly indicate write callback error

Hello!
Recently curl introduced CURL_WRITEFUNC_ERROR macro to make error handling in write callbacks more robust. I found out that there's one place in Newsboat where that could be used so here goes the PR ;)

This seems pretty straightforward to me, but I think that this will bring more clutter to the code. Perhaps it'd better to wait for CI to obtain 7.87.0 version of curl since this issue is not that critical.

We declare (but don't actually check, oops) compatibility with curl 7.32.0, so even when CI gets a newer version, we still won't be able to use the new constant. I'm hesitant to bump the requirement just for the new constant though.
Let's just close this then. Thanks for the pull request though! Perhaps I could interest you in one of the issues that are fairly easy to start with? :)


MorphToSelect

Package
filament/filament
Package Version
v3.2.69
Laravel Version
v10.48.8
Livewire Version
No response
PHP Version
8.1
Problem description
The select is created successfully, but after selecting the model it gives an error:
Call to a member function post() on null
I looked for examples of use on github and re-read the documentation several times - I don’t see the error
class AutoPostCalendarWidget extends FullCalendarWidget
{
    public Model | string | null $model = AutoPost::class;

    public function getFormSchema(): array
    {
        return [
        \Filament\Forms\Components\MorphToSelect::make('post')
            ->types([
                \Filament\Forms\Components\MorphToSelect\Type::make(CraigslistPost::class)
                    ->titleAttribute('title'),
                    \Filament\Forms\Components\MorphToSelect\Type::make(Domain::class)
                    ->titleAttribute('id'),
            ])->searchable(),
            ];
        }

namespace App\Models;

use Illuminate\Database\Eloquent\Factories\HasFactory;
use Illuminate\Database\Eloquent\Model;
use Illuminate\Database\Eloquent\Relations\MorphTo;

class AutoPost extends Model
{
    use HasFactory;

    protected $guarded = false;

    public function post(): MorphTo
    {
        return $this->morphTo();
    }
}

<?php

use Illuminate\Database\Migrations\Migration;
use Illuminate\Database\Schema\Blueprint;
use Illuminate\Support\Facades\Schema;

return new class extends Migration
{
    public function up(): void
    {
        Schema::create('auto_posts', function (Blueprint $table) {
            $table->id();

            $table->morphs('post');

            $table->date('start_at');
            $table->date('end_at');
            $table->time('time_at');

            $table->timestamps();
        });
    }

    public function down(): void
    {
        Schema::dropIfExists('auto_posts');
    }
};

Expected behavior
Displaying a list of records of the selected model
Steps to reproduce

php artisan migrate:fresh --seed
log in as admin@admin.com:admin
Go to the Auto Post Calendar page
Click the New auto post button
select model Craigslist post

Reproduction repository
https://github.com/dbrnjh47/bag
Relevant log output
No response

Donate 💰 to fund this issue

You can donate funding to this issue. We receive the money once the issue is completed & confirmed by you.
100% of the funding will be distributed between the Filament core team to run all aspects of the project.
Thank you in advance for helping us make maintenance sustainable!

I used the saade/filament-fullcalendar plugin
Still, he is to blame for the problem, I attach the solution
// https://github.com/saade/filament-fullcalendar/issues/122
```
protected function getFormModel(): Model|string|null
{
return $this->event ?? PostScheduled::class;
}
public function resolveEventRecord(array $data): PostScheduled
{
    return PostScheduled::find($data['id']);
}


Is it necessary to add selinux label at  container-selinux ?

What is the problem you're trying to solve
add selinux at  container-selinux
Describe the solution you'd like
add selinux at  container-selinux
Additional context
add selinux at  container-selinux

Yes. Help wanted.
https://github.com/containers/container-selinux/blob/main/container.fc

As I understand it nerdctrl is a client server tool like docker and talks to the container engine.  So it does not need to be labeled as a container engine.  Since it will not need special privs to create namespace, cgroups, SELinux ...

nerdctl will use /var/lib/nerdctl/ to save some information ,so I think it is necessary to add some selinux label @rhatdan


Error in the generation of the C++ code should result in a cmake error

E.g. when i have an Error in the python code:
AttributeError: 'VectorProducer' object has no attribute 'name'

I still get a build file
-- Add build target for file analysis_emb.cxx.
-- Add test for target analysis_emb
-- Configuring done
-- Generating done
-- Build files have been written to: /work/sbrommer/ntuple_prototype/build

@stwunsch there is propably an easy for this this

Yep, it's easy! Can you give me a diff, which produces invalid code and should fail?

Does not happen anymore, can be closed


[ci] Test the dec tool builds

Just realized dec tool was still broken because of the header, so I added it to the CI to avoid this in the future.

Thanks! Should have done this a while ago.


Null check via helper function doesn't register as null check to Flow.

As the title tries to explain, if you are doing a null check via a helper function, Flow doesn't recognize it as a null check. So if you try to access the property that you're null checking, a type error is thrown.
/* @flow */

function func(val:string){}

class Struct {
	prop:?string;
}

// This doesn't work.
function checkEmpty(struct:Struct){
	if(empty(struct.prop))throw new Error('prop is empty.');
	func(struct.prop);
}

// This doesn't work.
function checkNegateEmpty(struct:Struct){
	if(!empty(struct.prop)){
		func(struct.prop);
	}
}

// This works
function manualEmptyCheckThrow(struct:Struct){
	if(struct.prop == null)throw new Error('prop is empty');
	func(struct.prop);
}

// This works
function manualEmptyCheck(struct:Struct){
	if(struct.prop != null){
		func(struct.prop);
	}
}

The use case is that I want to check if a value is empty. The real logic for such a function is:
function empty(val){
	return (
		val === undefined ||
		val === null ||
		typeof val.length !== 'undefined' && val.length == 0;
	);
}

But I simplified it for the example above.

It's not currently possible to write a function that refines types like this, but there is an open issue for this which has seen a recent proposal. I'm closing this to merge with #34 instead. Thanks for the very detailed issue!


Refactor and top/bottom shadows refactoring

Simplify code and make it work with flexible container. + Implement top/bottom shadows. + readme update

This looks amazing to me! Thanks for the great job 💃


Separate FAKE_ADDRESS and NOT_ADDRESS

Before this commit, a string FAKE_ADDRESS was sometimes used
as a random address and sometimes as an oddly-sized string
that's not an address.  The real value of FAKE_ADDRESS was
an oddly-sized string that's not an address.
This commit separates this constant into two: FAKE_ADDRESS and
NOT_ADDRESS. FAKE_ADDRESS is a random address while NOT_ADDRESS
is an oddly-sized string that's not an address.
This closes #1166.

Thank you!


Using rice in the fun/bin/food script messes up terminal colors

My guess is that some color code is not properly escaped.  Calling anything else with the same script fixes the problem.

nice catch. i added \e[0m to the end of each. thanx

nice catch. i added \e[0m to the end of each. thanx


waveglow

Hi, how can i use waveglow?

Try this:
radtts_waveglow.zip

glow and denoiser taken from https://github.com/NVIDIA/waveglow
inference.py adjusted to load and use waveglow in place of hifigan


Prepare  show&tell presentation on Schema Service

Please link the presentation or document to this ticket.
Please make a note here on the length of the presentation and when you'd be ready for present it

1 day


Playground content created via document.createNode() does not clear on loading a different playground

Step 1: Load the following playground:
https://playground.babylonjs.com/#2ARI2W#152
Step 2: Change the playground iteration (#152 in the URL) to a different value (or remove it), and navigate to the new URL
Expected: All content created within the playground editor is cleared and replaced with content from the new playground
Actual: "document"-based calls are not reverted, leaving them present in all future playgrounds until page is properly refreshed (since modifying the playground iteration doesn't refresh the page)

It is up to the user to clear that when the engine is disposed (or at least the scene with scene.onDispose handler).
How do you propose to track what the user created? We can probably tag our elements I guess

Ideally we expect the users to open a chat on the forum first so we can have all our community to contribute to the discussion.

will be very hard to track the entire DOM. We could hard-refresh all DOM elements? Add some state change to the parent and redraw everything. Just a thought?

Honestly I'm more incline to do nothing :)

Honestly I'm more incline to do nothing :)

This is probably the cleanest solution :-)


[firebase_crashlytics] Support for deobfuscation of Dart reports

I can't even find any info on how to find dart obfuscation maps to deobfuscate reports manually

Do we have to write some tool to map obfuscated logs onto readable ones?

I've found at least how to generate maps
On android go to your_flutter_project/android/gradle.properties and add line
extra-gen-snapshot-options=--obfuscate,--save-obfuscation-map=obfuscation_maps/android_dart_symbols.json

For iOS, I've found this guide on medium: https://medium.com/@waytoa/obfuscating-flutter-code-and-generate-map-files-for-ios-fdaf4a377ec4

@rostopira
Yeah, thanks, I got that! The problem is how to reverse the obfuscation process given the obfuscated stacktrace and the obfuscation map to output the human readable log.

I've wrote simple script for deobfuscation:
const fs = require('fs');

if (process.argv.length < 3) {
    console.log(
        "Dart crash reports deobfuscation tool\n" +
        "Usage: node deobufscate.js map.json report.txt\n"
    );
    return;
}

const mapData = fs.readFileSync(process.argv[2], 'utf-8');
const mapJsonArray = JSON.parse(mapData);
let map = {};
let characters = new Set();
for (let i = 0; i < mapJsonArray.length; i += 2) {
    const obfuscated = mapJsonArray[i+1];
    const deobfuscated = mapJsonArray[i];
    if (obfuscated === deobfuscated) {
        // Obfuscation map contains pairs like "[]=":"[]=" and "toString":"toString"
        // Skip them
        continue;
    }
    for (const char of obfuscated)
        characters.add(char);
    map[obfuscated] = deobfuscated;
}

if (process.argv.length < 4)
    return;

let report = fs.readFileSync(process.argv[3], 'utf-8');
let start = 0;
while (start < report.length) {
    if (!characters.has(report[start])) {
        start++;
        continue;
    }
    let end = start;
    while (characters.has(report[end]))
        end++;
    const replacement = map[report.substr(start,end-start)];
    if (replacement) {
        report = report.substr(0, start) + replacement + report.substr(end);
    }
    start = end + 1;
}
console.log(report);

@rostopira
This issue has been moved to https://github.com/FirebaseExtended/flutterfire/issues/1150. Any further collaboration will be done there.


game start for all

game start button should start the game for both players, not just yourself.

game now starts when all players click 'Ready!'


Filter columns directly from dropdown in list views

As ...
Nelly - Network Engineer
I want ...
Be able to quickly filter columns in list view by selecting a dropdown from the column header similar to excel/sheets. Specifically the ability to choose unique values in the list - deduplicated by display name.
Ex:

So that ...
List views can be filtered quickly and use existing UX patterns that feel natural to users who use table views in other applications.
I know this is done when...
I can select one or more items from a dropdown that originates from the list vew column header and the list filters on the item(s).
Optional - Feature groups this request pertains to.

[ ] Automation
[ ] Circuits
[ ] DCIM
[ ] IPAM
[ ] Misc (including Data Sources)
[ ] Organization
[ ] Plugins (and other Extensibility)
[ ] Security (Secrets, etc)
[ ] Image Management
[X] UI/UX
[ ] Documentation
[ ] Other (not directly a platform feature)

Database Changes
none
External Dependencies
No response

Idea for #1565


Fix resource leaks

I had a look over the source code and noticed a few resource leaks (with findbugs).
The second commit documents that this plugin does not compile with Java 7 and because Gradle 3 requires at least Java 7 I set this up to require Java 8.
The third commit cleaned up lots of warnings (eclipse likes to do these changes on it's own).
If you dislike some of the changes / disagree with them you should be able to just cherry-pick the others.
I can also update this PR of course.
The source code doesn't have a consistent code style. There is a Gradle plugin that can enforce a consistent code style: https://plugins.gradle.org/plugin/com.diffplug.gradle.spotless. Spotless can also use an eclipse formatter profile.
WDYT about formatting the source, and applying spotless with your favorite formatting definition?

That looks great,
thanks very much.
Very happy to take all kinds of improvements. :-)
(It was a very quick hack together of a solution initially)


Can we make "public"

No. It contains executable files in inst/extdata of which I do not know the origin and whether it is allowed to use them.

Well I was too fast but it should be ok:

"This codes documented by the report are public domain, open-source software and can be downloaded from the Internet at URL http://water.usgs.gov/software/ground_water.html/."
USGS (2008): UCODE_2005 and Six Other Computer Codes for Universal Sensitivity Analysis, Calibration, and Uncertainty Evaluation, page 5


Parallelize operations in groups of pods

What type of PR is this?
/kind feature
What this PR does / why we need it:
Paralelize updates to pod objects. These operations can be slow, as they require a connection with the apiserver.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?

NONE

E2E tests passed 2/5 times

/assign @tenzen-y


Are the species inputs fed to the model together?

Hi :)
I have a confusion between the figure that illustrates the model and the code. The figure shows that the model input is the genes from multiple species fed simultaneously to the model. However, what I got from the code is that each species's genes are fed individually to the model one by one.
Could you clarify this point?
Thanks!

Hi,
An individual cell will only have genes from its species. The insight from SATURN / Figure 1 is that while this is true, we can map different species genes to a shared feature space, the space of macrogenes. Thus, for cells from different species, although they have different genes, they can have the same set of features. Does that help explain?


Call to function "yamldecode" failed

Hello,
I am testing the RBAC module and the spec.yml file is in the same location as other .tf files. While running terraform plan, I am receiving below error.  Is there anything I should be trying out?
I given below location in plan output:
var.spec_file_path
  The path to the RBAC specification file.

  Enter a value: ~/Documents/snowflakerbac_1/spec.yml

 Error: Error in function call
│
│   on .terraform/modules/rbac/locals.tf line 4, in locals:
│    4:   spec = yamldecode(file(var.spec_file_path))
│     ├────────────────
│     │ var.spec_file_path is "spec.yml"
│
│ Call to function "yamldecode" failed: on line 7, column 12: did not find
│ expected key.

Hi, got your open issue. we are going to look.

@vpasnur would you be able to provide the contents of your spec.yml file?

@vpasnur The error implies to me one of two issues:

The file path isn't correct and the project isn't able to load the values
The format of your spec.yml doesn't match the spec outlined here

Feel free to share the spec.yml and we can review it.

Hello All,
Attached is my spec.yml file. It's in the same location as other TF files.
Let me know if any modifications are needed to this spec.yml file. Thank you
On Fri, Mar 3, 2023 at 11:48 AM Mustafa EL-Hilo @.***>
wrote:

@vpasnur https://github.com/vpasnur The error implies to me one of two
issues:

The file path isn't correct and the project isn't able to load the
values
The format of your spec.yml doesn't match the spec outlined here
https://github.com/Infostrux-Solutions/mm-foodmarket-snowflake

Feel free to share the spec.yml and we can review it.
—
Reply to this email directly, view it on GitHub
https://github.com/Infostrux-Solutions/terraform-snowflake-rbac/issues/33#issuecomment-1453809622,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A6GS5LNBNJZL37PMA36WWVLW2IOE5ANCNFSM6AAAAAAVMFWBCQ
.
You are receiving this because you were mentioned.Message ID:
@.***
com>

@vpasnur do you mind pasting the spec.yml file here as the email reply wont attach it to here

Yeah. Sure. Here is the info.
databases:
database_name:
privileges:
database:
read:
roles:
- ACCOUNTADMIN
schemas:
read:
roles:
- SYSADMIN
        roles:
          role_name:
            grant_to:
              roles:
                - SYSADMIN

roles:
role_name:
grant_to:
roles:
- SYSADMIN
warehouses:
warehouse_name:
privileges:
roles:
- SYSADMIN
On Mon, Mar 6, 2023 at 12:20 PM Mustafa EL-Hilo @.***>
wrote:

@vpasnur https://github.com/vpasnur do you mind pasting the spec.yml
file here as the email reply wont attach it to here
—
Reply to this email directly, view it on GitHub
https://github.com/Infostrux-Solutions/terraform-snowflake-rbac/issues/33#issuecomment-1456568257,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A6GS5LK64W5PQ37LXJ3IFITW2YMFJANCNFSM6AAAAAAVMFWBCQ
.
You are receiving this because you were mentioned.Message ID:
@.***
com>

The spacing for this spec.yml doesn't seem correct to me.. spacing is very important in yml.
databases:
  database_name:
    privileges:
      database:
        read:
          roles:
            - ACCOUNTADMIN
      schemas:
        read:
          roles:
            - SYSADMIN
      roles:
        role_name:
          grant_to:
            roles:
              - SYSADMIN
roles:
  role_name:
    grant_to:
      roles:
        - SYSADMIN

warehouses:
  warehouse_name:
    privileges:
      roles:
        - SYSADMIN

Thanks for pointing that out. i used yaml formatter and noticed a tab space
given extra.
Can I know where I should define database name, schema and table name? I am
currently getting below errors.
Error: Invalid function argument
│
│   on locals.tf line 65, in locals:
│   65:     if contains(keys(spec.privileges), "database")
│     ├────────────────
│     │ spec.privileges is null
│
│ Invalid value for "inputMap" parameter: argument must not be null.
╵
╷
│ Error: Invalid function argument
│
│   on locals.tf line 104, in locals:
│  104:     if contains(keys(spec.privileges), "schemas")
│     ├────────────────
│     │ spec.privileges is null
│
│ Invalid value for "inputMap" parameter: argument must not be null.
╵
╷
│ Error: Invalid function argument
│
│   on locals.tf line 143, in locals:
│  143:     if contains(keys(spec.privileges), "tables")
│     ├────────────────
│     │ spec.privileges is null
│
│ Invalid value for "inputMap" parameter: argument must not be null.
On Mon, Mar 6, 2023 at 2:22 PM Mustafa EL-Hilo @.***>
wrote:

The spacing for this spec.yml doesn't seem correct to me.. spacing is
very important in yml.
databases:
database_name:
privileges:
database:
read:
roles:
- ACCOUNTADMIN
schemas:
read:
roles:
- SYSADMIN
roles:
role_name:
grant_to:
roles:
- SYSADMIN
roles:
role_name:
grant_to:
roles:
- SYSADMIN
warehouses:
warehouse_name:
privileges:
roles:
- SYSADMIN
—
Reply to this email directly, view it on GitHub
https://github.com/Infostrux-Solutions/terraform-snowflake-rbac/issues/33#issuecomment-1456816606,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A6GS5LMSDC3RH7PVJWRVSETW2Y2ONANCNFSM6AAAAAAVMFWBCQ
.
You are receiving this because you were mentioned.Message ID:
@.***
com>

databases:
  database_name:
    privileges:
      database:
        read:
          roles:
            - ACCOUNTADMIN
      schemas:
        read:
          roles:
            - SYSADMIN
      roles:
        role_name:
          grant_to:
            roles:
              - SYSADMIN
roles:
  role_name:
    grant_to:
      roles:
        - SYSADMIN

warehouses:
  warehouse_name:
    privileges:
      roles:
        - SYSADMIN

In this example, the database name is database_name, the role name is role_name and warehouse is warehouse_name. Just replace database_name, role_name and warehouse_name with the values you want


Add GPU accelerated array_contains operator

Closes https://github.com/NVIDIA/spark-rapids/issues/1372
Depends on https://github.com/rapidsai/cudf/pull/7125 and https://github.com/rapidsai/cudf/pull/7039
This Draft/WIP PR adds support for array_contains on the gpu. I have skipped some input check methods from the Spark implementation since right now we only support basic typed lists in LHS, it would be nice to get reviews on that decision, refer to (https://github.com/apache/spark/blob/master/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala#L1120) . I hope I have got the Expr checks right given we want to support only basic typed lists. Appreciate any corrections if needed. This patch also needed the hasNans check since we have incompat behavior with Spark. Marking this as draft since we have moving pieces.

build

build


[Question] Async ConsumerFunc

I noticed that the ConsumerFunc does not support async, so the client doesn't wait for the consumer function to finish before processing more messages. The problem is that my CPU becomes overwhelmed when receiving 1,000,000 messages in a short amount of time.
Is there a way to solve this problem ? The only temporary fix I have is to enqueue all the asynchronous calls in a queue, but that seems a bit overkill and not very efficient.
const consumer = await this.client.declareConsumer({
            ...
 },  (message) => {
        this.worker.enqueue(async () => {
           ...
        })
 })

Thank you,

Hello, sorry for the delay.
We'll get back to you in the following days

I am also interested in this because I am trying to replace a single consumer on a queue that needs to respond to a single message at a time in order with potentially async operations (like updating a database). I'm using a classic queue with prefetch of 1 and manual acknowledgements at the moment, but want to use a partitioned superstream instead (I already tried a consistent hash exchange but its lack of partition stability on restarts was a deal breaker).
While investigating superstreams, I was concerned to make sure I could keep the ordered delivery guarantee and I couldn't find much on that in the stream library documentation so digging in the code I spotted this https://github.com/coders51/rabbitmq-stream-js-client/blob/d8083e5b165a9ace3d926a46712f9a6b3d911fae/src/client.ts#L554-L556 which looked like it would not wait to deliver the next message in a chunk if the handler function wanted to do something async.

@tunniclm i ended up patching the library code locally. Maybe not the best idea but that works.
Here is the patch (https://pnpm.io/cli/patch) :
<EMAIL_ADDRESS>

@AmauryD Unfortunately, I'm still seeing overlap in message processing I think because the delivery of new message chunks is overlapping. I suspect this is due to the default credit policy of creditsOnChunkReceived(2, 1) (start requesting 2 chunk and request a new one each time one is received) which can result in new chunks being delivered/start processing before the prior chunk is finished processing.
I'm going to try testing with a policy of creditsOnChunkCompleted(1, 1). It looks like I can't configure this when declaring a super-stream consumer, so I'm going to hard-code it in my client library for the test.

@AmauryD Unfortunately, I'm still seeing overlap in message processing I think because the delivery of new message chunks is overlapping. I suspect this is due to the default credit policy of creditsOnChunkReceived(2, 1) (starts requesting 2 chunks and then requests a new one each time one is received) which can result in new chunks being delivered/start processing before the prior chunk is finished processing.
I'm going to try testing with a policy of creditsOnChunkCompleted(1, 1). It looks like I can't configure this when declaring a super-stream consumer, so I'm going to hard-code it in my client library for the test.

I forgot to mention it earlier, but yes, I also added:
{
            creditPolicy: creditsOnChunkCompleted(1, 1),
}

to my Consumer.


Argument parsing rework

Pull request checklist:

[x] CHANGELOG.md was updated, if applicable

Description
Makes use of QCommandLineParser and parses user input adding few dank features:

Shows detailed description with --help:

Handles joining only channels from command line, for example: chatterino --channels pajlada,zneix,mm2pl will start chatterino, but only with those 3 channels open in each split. I think that comma as split character is fine since it can't be a part of twitch username.
To make sure this doesn't overwrite existing configuration in window-layout.json, saving in WindowManager is disabled when application was invoked with --channels argument.

Small TODO:

[x] Add documentation
[x] Rework handling --channels - it works, but it still uses strings to parse JSON, maybe there's a better solution?

Notes to take a look at before merging:

Everything works fine on linux, however on Windows: --help gives you dialog window with help message, but --version doesn't show anything and quits - that was also a thing before, but we can fix it by using native addVersionOption method, however that will only output app name + version without commit, removing functionality added in #1739 which I don't wanna do.
Due to nature of how help message works, --crash-recovery is now shown in --help menu. I don't think it's a big issue but I can't seem to find an option to hide that either.

Closing a stale issue?

I suppose this should also disable settings from being changed and saved?

I suppose this should also disable settings from being changed and saved since they are not synced between instances?

Agree, it is ignoring saving layout to window-layout.json if the application was started with -c flag.

Another solution would be to use IPC to add a new window to the original chatterino instance

Not sure what do you mean here, this doesn't create separate instances of chatterino - it only generates its own layout based on channels supplied in parameter while ignoring layout in window-layout.json.

Agree, it is ignoring saving layout to window-layout.json if the application was started with -c flag.

I mean it might be good to extend this to the regular settings as well, not just the layout of the windows.

Not sure what do you mean here, this doesn't create separate instances of chatterino - it only generates its own layout based on channels supplied in parameter while ignoring layout in window-layout.json.

Right now (unless I am mistaken) running chatterino with --channel x will open a new instance of chatterino instead of communicating to the running instance that it should open the channels in a new window.

I changed the way how you specify channels - it's now platform:channelName, where platform is just a single letter. I don't know how IRC channel integration in Chatterino works, so I didn't implement those, but that can be easily added if there'll be need for it.
Also changed tab delimeter character to ; because of following:
As for opening more splits in one tab, I thought of a following idea: --channels t:foo,t:bar;t:asd,t:test would open 2 tabs with 2 splits each:

containing Twitch channels foo and bar
containing Twitch channels asd and test

This could be done, but handling multiple vertical and horiziontal splits in one tab is horrible and at this point I think it's a bit overcomplicated, so I don't know if I should try implementing multiple splits in one tab with more parsed JSON or just drop that altogether, wait for eventual rework of WindowManager.cpp and then maybe even remove parsed JSON.

would prefer to use a character other than -, which could be part of a username atm.
maybe : instead?
and if no prefix is supplied, having Twitch be the default seems sane for this Twitch chat client

I tried this and it works well, I use it in a script that runs streamlink and chatterino.

Yeah, it is working fine with current code however I should get rid of JSON parsing and it is possible because #1964 is merged now. I just gotta rework that one bit and it will be good to go. I have to find some time in the week and I hope I can make it before this weekend. Thanks for approving though :)


Remove scipy restriction

Removes a requirement restriction added in #835 in order to get the docs to build.  The underlying cause of the docs failing has been resolved so we can remove this restriction.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #837   +/-   ##
=======================================
  Coverage   98.15%   98.15%
=======================================
  Files         117      117
  Lines       10851    10851
=======================================
  Hits        10651    10651
  Misses        200      200

Continue to review full report at Codecov.

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


Editor 3D Viewport mouse controls broken on Windows

Tested versions
e5b4ef8
System information
Windows 10
Issue description
Bisect and PR https://github.com/godotengine/godot/pull/84363 broke the (right) mouse input in the Editor 3D Viewport.
E.g. right mouse camera view does no longer work or has janky focus loss.
If you hold the right mouse button to move the camera it only moves for a split-second and then starts to drop input.
Steps to reproduce
Use the (right) mouse on Windows to interact with the Editor 3D Viewport.
Minimal reproduction project (MRP)
N/A

@RandomShaper


Can I just update base image version to bump up Golang with ansible operator

Type of question

General operator-related help

Question
What did you do?
My ansible operator is built based on 0.17.2, I'm using base image quay.io/operator-framework/ansible-operator:v0.17.2
I need to upgrade golang to 1.15.7+ to solve security issue, Can I update base image version to v1.5.0 directly? will rest of code work with v1.5.0 base image?
Do I have to restructure my operator for golang upgrade?  Is there easy way to do it?
Operator type:
/language ansible

@niuxiaojuan-github There have been quite a few changes in the ansible base image from 0.17.x to 1.5.0. You may have to migrate your project to be compatible with the post 1.0+ changes. Here is the migration guide which maybe helpful https://sdk.operatorframework.io/docs/building-operators/ansible/migration/

+1, that will definitely require manual migration steps.


Added Azure Data Factory Custom Policies

This PR fixes

Added Azure Data Factory Custom Policies

@jtracey93
Azure Data Factory

Policy Name
Policy Area
Description

Deny-DataFactory-ApiVersion
Resource Management
Denies old API version for data factory V1.

Deny-DataFactory-IdentityType
Authentication
Enforces use of system assigned identity for data factory.

Deny-DataFactory-IntegrationRuntimeManagedVirtualNetwork
Network Isolation
Denies Integration Runtimes that are not connected to the Managed Virtual Network.

Deny-DataFactory-ManagedPrivateEndpoints
Network Isolation
Denies external private endpoints for linked services.

Deny-DataFactory-PublicNetworkAccess
Network Isolation
Denies public access to data factory.

Deploy-DataFactory-ManagedVirtualNetwork
Network Isolation
Deploy managed virtualnetwork for data factory.

@krnese I would enforce the following on the Corp Management Group:
Deny-DataFactory-PublicNetworkAccess


How to programmatically hide a notification?

I am creating a notification using...
$.notify 'Getting Iteration Data...', {
  position: "top center",
  className: "info",
  clickToHide: true,
  autoHide: false,
  arrowShow: false
}

But I can not figure out how to hide this notification after the AJAX request has completed. Let me know how I can do this. Thanks

This works, but it'd be nice to have a wrapper for this in the library to avoid leaking implementation details.

I think the current method with triggering notify-hide on the wrapper is a bit clumsy too, and not very well documented.
If you're calling .notify() on an element, the hide event should also be bound to that element.
Also would be nice to be able to do something on the lines of $el.notify(false) or $el.notify('hide') or $el.notify.hide() or something to that degree, which would seem more like the expected behaviour (as far as I believe).


Change data format

Id suggest to save plain data as JSON files, so it will be easy to load it in any environment.
now it have to be patched to read as JSON.

A good point. I'll make it in JSON the next release.


DataFrame summary without ReportBuilder fixes #221

#221  fixes

#221  fixes

Can I use text-table gem to create table for distribution in vector.

Well, looking closer, I see most of my notes are related to pre-existing code... But still, the library becomes mature now, and so we are struggling for better code quality with each commit. Report building was one of the parts sitting here since ancient times, and the idea of rewriting it is not only drop ReportBuilder, but also make the code shiny and readable.

I will try to implement as much as good as I can make.I will try to improve on my next commits. And I will make a new pull request with new branch when changes are final so that these confusing commits doesn't show up on commit history.

ohh okay i will make the changes. Are vector_summary and vector_indent better ?

Well, we are already inside Vectors method, why we need any prefix at all?..

Closing this. Send a new PR with fixes.


JENKINS-35347: Add support for V3 auth.

If user provides identity of the form "tenant:username" then V2 is used as before.
If user provides identity of the form "project:username:domain" then V3 is used.
Note: This is (heavily!) based on https://github.com/jenkinsci/openstack-cloud-plugin/pull/124 by FROSADO.
I've added help-text to the configuration UI, and I've changed the code layout to minimize code duplication and number of lines changed, but functionally it's as PR124.
I've carried out basic "test connection" tests against our (in-house) old v2-auth OpenStack system and our new v3-auth OpenStack system and this new code is able to connect to both and e.g. list images to populate drop-down lists.

Note: The unit test that failed is unreliable - the failure is not related to these code changes.

This didn't work for me, please see #139 bug for details, clearly we need to improve the documentation and use current parameters names (instead of old ones).

@ssbarnea, the issue is unrelated to this PR as noted at the issue.

@olivergondza I closed this issue. It was a mistake to raise it because the trunk code is properly documented, the issue is still true with the latest release but next one will address it.


Danger Swift installed through Homebrew - danger-swift pr; [!] Unknown option: --process Did you mean: --pry?

I wanted to test danger-swift installed from Homebrew, through this guide:
https://danger.systems/swift/guides/getting_started.html
But, after running
danger-swift pr https://github.com/danger/swift/pull/318

I get this error:
➜  ~ danger-swift pr https://github.com/danger/swift/pull/318
[!] Unknown option: `--process`
Did you mean: --pry?

Usage:

    $ danger pr

      Run the Dangerfile locally against Pull Requests (works with forks, too). Does
      not post to the PR. Usage: danger pr <URL>

Options:

    --clear-http-cache                  Clear the local http cache before running
                                        Danger locally.
    --pry                               Drop into a Pry shell after evaluating the
                                        Dangerfile.
    --dangerfile=<path/to/dangerfile>   The location of your Dangerfile
    --verify-ssl                        Verify SSL in Octokit

If I try to run it with local and passing the Dangerfile to it:
➜  ~ danger-swift local —dangerfile Dangerfile.swift
[!] Unknown option: `--process`
Did you mean: --pry?

Usage:

    $ danger local

      Run the Dangerfile locally. This command is generally deprecated in favor of
      `danger pr`.

Options:

    --use-merged-pr=[#id]   The ID of an already merged PR inside your history to use
                            as a reference for the local run.
    --clear-http-cache      Clear the local http cache before running Danger locally.
    --pry                   Drop into a Pry shell after evaluating the Dangerfile.
➜  ~

What am I missing?

Do you have Danger ruby installed on your machine? If yes I think that is the issue

Do you have Danger ruby installed on your machine? If yes I think that is the issue

You were right! Indeed I had a ruby version installed too:
➜  ~ which danger
/Users/*****/.rvm/gems/ruby-2.6.3/bin/danger

Fixed with:
gem uninstall danger

Uninstalling Ruby danger did the trick, if anyone else has the same issue.
Thank you. 😃

Could make it use danger-js which should always be installed when danger-js is installed and not clash

Is it there also if you install danger with Brew?

Confirmed, that's a no - it is a zip file with just a 'danger' executable. Shame. Doesn't mean it can't try use danger-js first before falling back to danger, but I bet that's already in there.

No, it is not, this is a great idea, will have a look at adding it :)


[Issue #98] Added option to use bucket name directly as host

Added checkbox option under Advanced section in Settings > Media.
When checked, bucket name will be used as direct host of images instead of storage.googleapis.com/
This option is only meant for cases where the bucket is set up for static hosting (e.g. image subdomain as a CNAME record).

@ewsopp Maybe we should support this in setup assistant.

@alimuzzaman lets add this in, unless you see some problem.

Keep it out of setup assistant (for now), but add into the settings area.

Great!


Adds updateTags for subnet group.

Issue #, if available:
Customers cannot update tags for subnet group after creation.
Description of changes:
Add some functions so that tags of subnet group can be updated now.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

/lgtm

/lgtm cancel

/lgtm


macOS Monterey and anaconda crash

OS: macOS Monterey 12.3.1
PyQt version: 5.12.3

I work with Anaconda and tried to install labelImg the same way that is explained for windows+Anaconda
Everithing seems to work but when I try to execute the program I have the following errors:
Traceback (most recent call last):
File "/Users/ericleroy/GitHub/labelImg/labelImg.py", line 1660, in
sys.exit(main())
File "/Users/ericleroy/GitHub/labelImg/labelImg.py", line 1656, in main
app, _win = get_main_app(sys.argv)
File "/Users/ericleroy/GitHub/labelImg/labelImg.py", line 1647, in get_main_app
win = MainWindow(args.image_dir,
File "/Users/ericleroy/GitHub/labelImg/labelImg.py", line 210, in init
self.dock.setFeatures(self.dock.features() ^ self.dock_features)
TypeError: unsupported operand type(s) for ^: 'DockWidgetFeatures' and 'DockWidgetFeatures'
Do you have any idea?

I have the exact same error, but in Windows 10 with conda

I also get the same error in Windows 10 with conda:
python 3.10.0
labelImg 1.8.6
pyqt5 5.12.3
lxml 4.8.0


Use as a node\browserify npm module

There's an option to install as an npm-module, but did anyone actually tried to use it from as a node-module from browserify? It's not possible since it's not exporting anything from the root index.js

This should work:
var client = require('cross-storage/lib/client.js');
var hub    = require('cross-storage/lib/hub.js');

I'll addan index.js to lib and export the two though.

var client = require('cross-storage/lib/client.js');
gives you an empty Object as a result since client.js is not exporting anything via module.exports.
You can check out this article to prepare the library code for any kind of usage with global, AMD and CommonJS styles http://oli.me.uk/2013/07/21/exporting-through-amd-commonjs-and-the-global-object/

hah, oops. I'm familiar with AMD in a browser environment (https://github.com/danielstjules/blankshield/blob/master/blankshield.js#L150-L168 https://github.com/danielstjules/hoops/blob/master/lib/hoops.js#L191-L213), just woke up and figured I'd already done the work. :)
I'll have a fix for this throughout the week.

Great, thanks! :)

So sorry for the late reply on this: https://github.com/zendesk/cross-storage/commit/81ad623f89940520a3bf6e593fe8079e52c5e6e5

Closing, please see https://github.com/zendesk/cross-storage/releases/tag/0.7.0 :)


Bidirectional text (e.g. Hebrew with quotes around a part), in a category name fails to load

Switch to Hebrew
Try to add the translate extension, nothing happens

Alternately, If you first add the translate extension, then switch to Hebrew, the translate blocks category disappears:

Moved from LLK/scratch-vm#1390
/cc @ericrosenbaum
Copied from @picklesrus comment
This turns out to be a problem related to the characters in the translation of the category label into Hebrew.
Here's text that makes it fail: "message": "תרגום \"גוגל\""
The escaped quotes are confusing the category menu label - everything that comes after it in the toolbox xml definition gets cut off.
I plugged the that message into a category name in the default toolbox in the scratch-blocks playground and it fails similarly. It seems to be a problem with the scratch-blocks toolbox category name handling escaping in RTL.  Plugging the same text into labels on blocks works fine though.

Here's the code in label fields that forces text to LTR:
  if (this.sourceBlock_.RTL && text) {
    // The SVG is LTR, force text to be RTL.
    if (this.sourceBlock_.editable_ && this.sourceBlock_.type === 'math_number') {
      text = '\u202A' + text + '\u202C';
    } else {
      text = '\u202B' + text + '\u202C';
    }
  }

Applying this fix in toolbox.js probably goes here or here.
The problem has only showed up for category labels, but I imagine it's also a problem for flyout buttons and labels.  To fix, looks like we need something similar at or around this line in flyout_button.
This line may also be relevant (where the toolbox creates the XML for a category label inside the flyout).


mfa/login issues

I've been having difficulty lately with getting any type of automated mfa working.  Sometimes the image captcha shows up, and sometimes it doesn't.  I'm trying to use email with a gmail/imap but I don't know if thats possible because of ssl?  Is there anyone who has this working today?
Thanks.

@clueo8 I can describe my setup, which is working so far.
In the mint 2-step verification, you can set an app like Google Authenticator. This will give you two things: a QR code and a "manual setup" code. Scan the QR code with your Google Authenticator app. Copy the manual setup code. This will be all uppercase letters. Pass this as mfa_token. Set your mfa_method to soft-token.
Here's a Python code sample:
    mint = mintapi.Mint(
<EMAIL_ADDRESS>        'my_password',
        mfa_method='soft-token',
        mfa_token='XXXXXXXXX',
    )

If you need to log in manually, you will need to use your Google Authenticator app. If mintapi needs to log in, it can use the manual setup code to automatically log in.

@nickodell Can you share how you get past the email notifications. I was able to get the authenticator working however its also asking for an email code prior to that and I don't know how to turn that off or bypass it within the mintapi

@nickodell that helped a ton thanks for the walkthrough


Change default gravity for CPUParticles2D

Keep it consistent with GPUParticles2D.

I think it might be GPUParticles2D which is not consistent with the rest of the 2D physics:
$ rg gravity.*9
Area2D.xml
62:             <member name="gravity" type="float" setter="set_gravity" getter="get_gravity" default="980.0">

Area3D.xml
60:             <member name="gravity" type="float" setter="set_gravity" getter="get_gravity" default="9.8">

ProjectSettings.xml
1381:           <member name="physics/2d/default_gravity" type="float" setter="" getter="" default="980.0">
1386:                   # Set the default gravity strength to 980.
1390:                   // Set the default gravity strength to 980.
1453:           <member name="physics/3d/default_gravity" type="float" setter="" getter="" default="9.8">
1458:                   # Set the default gravity strength to 9.8.
1462:                   // Set the default gravity strength to 9.8.

CPUParticles2D.xml
168:            <member name="gravity" type="Vector2" setter="set_gravity" getter="get_gravity" default="Vector2(0, 980)">

CPUParticles3D.xml
194:            <member name="gravity" type="Vector3" setter="set_gravity" getter="get_gravity" default="Vector3(0, -9.8, 0)">

ParticlesMaterial.xml
183:            <member name="gravity" type="Vector3" setter="set_gravity" getter="get_gravity" default="Vector3(0, -9.8, 0)">

@akien-mga You're right. Your way makes more sense.

There is a counter-proposal for this that was opened a while ago: https://github.com/godotengine/godot-proposals/issues/2533


Update the logic for performing upgrades between releases

The following issues were identified in trying to upgrade a 0.8.0-rc0 deployment to 0.8.0.  All the issues are relatively straight forward, so putting them all together into one issue.
Issue: How to force an upgrade:

Decision is to have both --force-upgrade and --from-version

Issue: What if the from-version is not in the upgrade yaml?  It would be nice to have it fallback to the one previous to the --from-version or in secure storage.

Decision: Make the code smarter and have it find the first version equal to or greater than the from version (from args or storage) and apply all the upgrades including and to the current version.  This becomes the literal from to use.
Further -- if there is more than one identical update -- run it only once. See the example above -- all of the updates are to resave-ConnRec so it really only needs to be done once.

Issue: Once a version is in storage, cannot move to a newer version without running the upgrade command.

Decision: Change to only stop if there is an upgrade required between versions -- and if not, update the storage version.

Issue: On a new deployment, there is no version put into storage.

Decision: Add to the create storage the additional of the version record.

Issue: No version in storage in existing deployments.

Decision: Recommend that the deployment have an upgrade --from-version... command.

Issue: No warning currently produced when version in storage is not found.

Decision: Add the warning, with details about what to do.

Issue: Can we run an upgrade automatically when deploying a new version. Current behaviour is to abort the startup.

Option: Auto-upgrade.

Risk: Multiple instances try to run the upgrade could cause unexpected side effects.

Same as the next problem -- must be run offline
Need a semaphore or equivalent if the upgrade must be run once and only once.

Risk: Some updates MAY require running the upgrade offline.

Flag in YML file to say MUST be run as an explicit upgrade.

Issue: The current expectation is that in the YML, the version for the change is the "from" version -- e.g. when upgrading from a designated release.

Decision: The version associated with an upgrade command should be the version in when the upgrade should be run. E.g. Any version being upgraded to that version must run the upgrade steps.  So:

If the entry is v0.8.0: Update_Connections -- that command should be run from every pre-0.8.0 version when upgrade to 0.8.0 or beyond.  Further, if going from 0.7.5 to 0.9.0, they would run the 0.8.0 upgrade.

@shaangill025 -- if you think any of these are "too big" for what is an urgent update, please feel free to propose we separate out the change into a separate issue to be done in a later release.  We can decide as a group how urgent it is and how therefore we will handle it.
Thanks


wg_engine: upgrade wgpu-native library

see issue https://github.com/thorvg/thorvg/issues/2909
updeted wgpu_native support to v22.1.x.x
Please, update your webgpu headers and binaries for all your native platforms
See: https://github.com/thorvg/thorvg/wiki/WebGPU-Raster-Engine-Development

@SergeyLebedkin Could you please confirm testing on Linux or WSL on your end?
Crashed on ubuntu (wsl) with v<IP_ADDRESS>
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
thread '<unnamed>' panicked at library/core/src/panicking.rs:221:5:
panic in a function that cannot unwind
stack backtrace:
   0:     0x7f44b81eca05 - std::backtrace_rs::backtrace::libunwind::trace::h649ab3318d3445c5
                               at /rustc/eeb90cda1969383f56a2637cbd3037bdf598841c/library/std/src/../../backtrace/src/backtrace/libunwind.rs:116:5
   1:     0x7f44b81eca05 - std::backtrace_rs::backtrace::trace_unsynchronized::hf4bb60c3387150c3
                               at /rustc/eeb90cda1969383f56a2637cbd3037bdf598841c/library/std/src/../../backtrace/src/backtrace/mod.rs:66:5
   2:     0x7f44b81eca05 - std::sys::backtrace::_print_fmt::hd9186c800e44bd00
                               at /rustc/eeb90cda1969383f56a2637cbd3037bdf598841c/library/std/src/sys/backtrace.rs:65:5
   3:     0x7f44b81eca05 - <std::sys::backtrace::BacktraceLock::print::DisplayBacktrace as core::fmt::Display>::fmt::h1b9dad2a88e955ff
                               at /rustc/eeb90cda1969383f56a2637cbd3037bdf598841c/library/std/src/sys/backtrace.rs:40:26
   4:     0x7f44b820f2bb - core::fmt::rt::Argument::fmt::h351a7824f737a6a0
                               at /rustc/eeb90cda1969383f56a2637cbd3037bdf598841c/library/core/src/fmt/rt.rs:173:76
   5:     0x7f44b820f2bb - core::fmt::write::h4b5a1270214bc4a7
                               at /rustc/eeb90cda1969383f56a2637cbd3037bdf598841c/library/core/src/fmt/mod.rs:1182:21
   6:     0x7f44b81eaa4f - std::io::Write::write_fmt::hd04af345a50c312d
                               at /rustc/eeb90cda1969383f56a2637cbd3037bdf598841c/library/std/src/io/mod.rs:1827:15
   7:     0x7f44b81edb51 - std::sys::backtrace::BacktraceLock::print::h68d41b51481bce5c
                               at /rustc/eeb90cda1969383f56a2637cbd3037bdf598841c/library/std/src/sys/backtrace.rs:43:9
   8:     0x7f44b81edb51 - std::panicking::default_hook::{{closure}}::h96ab15e9936be7ed
                               at /rustc/eeb90cda1969383f56a2637cbd3037bdf598841c/library/std/src/panicking.rs:269:22
   9:     0x7f44b81ed82c - std::panicking::default_hook::h3cacb9c27561ad33
                               at /rustc/eeb90cda1969383f56a2637cbd3037bdf598841c/library/std/src/panicking.rs:296:9
  10:     0x7f44b81ee1b1 - std::panicking::rust_panic_with_hook::hfe205f6954b2c97b
                               at /rustc/eeb90cda1969383f56a2637cbd3037bdf598841c/library/std/src/panicking.rs:800:13
  11:     0x7f44b81edfe3 - std::panicking::begin_panic_handler::{{closure}}::h6cb44b3a50f28c44
                               at /rustc/eeb90cda1969383f56a2637cbd3037bdf598841c/library/std/src/panicking.rs:667:13
  12:     0x7f44b81ecec9 - std::sys::backtrace::__rust_end_short_backtrace::hf1c1f2a92799bb0e
                               at /rustc/eeb90cda1969383f56a2637cbd3037bdf598841c/library/std/src/sys/backtrace.rs:168:18
  13:     0x7f44b81edca4 - rust_begin_unwind
                               at /rustc/eeb90cda1969383f56a2637cbd3037bdf598841c/library/std/src/panicking.rs:665:5
  14:     0x7f44b7553455 - core::panicking::panic_nounwind_fmt::runtime::h907a0444fa61a6dc
                               at /rustc/eeb90cda1969383f56a2637cbd3037bdf598841c/library/core/src/panicking.rs:112:18
  15:     0x7f44b7553455 - core::panicking::panic_nounwind_fmt::h4c4dc67d0bbc166c
                               at /rustc/eeb90cda1969383f56a2637cbd3037bdf598841c/library/core/src/panicking.rs:122:5
  16:     0x7f44b75534e2 - core::panicking::panic_nounwind::hb98133c151c787e4
                               at /rustc/eeb90cda1969383f56a2637cbd3037bdf598841c/library/core/src/panicking.rs:221:5
  17:     0x7f44b75536a6 - core::panicking::panic_cannot_unwind::he9511e6e72319a3e
                               at /rustc/eeb90cda1969383f56a2637cbd3037bdf598841c/library/core/src/panicking.rs:309:5
  18:     0x7f44b78f2ef7 - wgpuSurfaceConfigure
                               at /tmp/wgpu-native/src/lib.rs:3775:1
  19:     0x7f44b875d4c2 - _ZN10WgRenderer6targetEP15WGPUSurfaceImpljj
                               at /home/hermet/thorvg/build/../src/renderer/wg_engine/tvgWgRenderer.cpp:344:25
  20:     0x7f44b875d4c2 - _ZN10WgRenderer6targetEP15WGPUSurfaceImpljj
                               at /home/hermet/thorvg/build/../src/renderer/wg_engine/tvgWgRenderer.cpp:321:6
  21:     0x7f44b875d4c2 - _ZN10WgRenderer6targetEP16WGPUInstanceImplP15WGPUSurfaceImpljjP14WGPUDeviceImpl
                               at /home/hermet/thorvg/build/../src/renderer/wg_engine/tvgWgRenderer.cpp:314:11

@hermet
yes. checked on wsl (ubuntu 24.04), windows and macos (intel silicon). It works
Please, make sure, than sdl2 window created with SDL_WINDOW_SHOWN param

@hermet yes. checked on wsl (ubuntu 24.04), windows and macos (intel silicon). It works Please, make sure, than sdl2 window created with SDL_WINDOW_SHOWN param

@SergeyLebedkin tested with your PR. it doesn't work.

the current issue seems that the given surface instance is not prepared properly yet in the target() call point:
bool WgRenderer::target(WGPUInstance instance, WGPUSurface surface, uint32_t w, uint32_t h, WGPUDevice device)

The surface generation by the SDL window may need to be ensured before being passed to it...

Hello, @SergeyLebedkin.
I am using mac os system. I simply replaced dylib and headers with latest version(v<IP_ADDRESS>).
I tried to perform clean build, encountered a build error.
Seems like it has crash with mac os built-in C++ header (MacTypes.h:548:41) Sharing full error log here: log.txt

This looks an existing problem. Not a version issue.

Hello, @SergeyLebedkin.
I am using mac os system. I simply replaced dylib and headers with latest version(v<IP_ADDRESS>).

Double check on macos 15.0.1 (Intel Silicon) It works fine

Hello, @SergeyLebedkin.
I am using mac os system. I simply replaced dylib and headers with latest version(v<IP_ADDRESS>).

Double check on macos 15.0.1 (Intel Silicon) It works fine

@SergeyLebedkin please try with test build:
meson setup builddir -Dtests=true ...

Hello, @SergeyLebedkin.
I am using mac os system. I simply replaced dylib and headers with latest version(v<IP_ADDRESS>).

Double check on macos 15.0.1 (Intel Silicon) It works fine

@SergeyLebedkin please try with test build: meson setup builddir -Dtests=true ...

check on macos with -Dtests=true. v19 and v22 have the same errors

@SergeyLebedkin Once conflicts resolved, i'll test again :)

@tinyjin resolved

Could you squash this commit?

Its too big for me. and its just a merge conflics resolver

Note: X11 Window
thread '<unnamed>' panicked at src/lib.rs:582:5:
Error in wgpuSurfaceConfigure: Validation Error

Caused by:
  Invalid surface

stack backtrace:
   0:     0x7fb0fb960895 - <std::sys_common::backtrace::_print::DisplayBacktrace as core::fmt::Display>::fmt::hd736fd5964392270
   1:     0x7fb0fb9847ab - core::fmt::write::hc6043626647b98ea
   2:     0x7fb0fb95e6cf - std::io::Write::write_fmt::h0d24b3e0473045db
   3:     0x7fb0fb96066e - std::sys_common::backtrace::print::h45eb8174d25a1e76
   4:     0x7fb0fb961a09 - std::panicking::default_hook::{{closure}}::haf3f0170eb4f3b53
   5:     0x7fb0fb9617aa - std::panicking::default_hook::hb5d3b27aa9f6dcda
   6:     0x7fb0fb961ea3 - std::panicking::rust_panic_with_hook::h6b49d59f86ee588c
   7:     0x7fb0fb961d84 - std::panicking::begin_panic_handler::{{closure}}::hd4c2f7ed79b82b70
   8:     0x7fb0fb960d59 - std::sys_common::backtrace::__rust_end_short_backtrace::h2946d6d32d7ea1ad
   9:     0x7fb0fb961ab7 - rust_begin_unwind
  10:     0x7fb0fb4b3283 - core::panicking::panic_fmt::ha02418e5cd774672
  11:     0x7fb0fb555f25 - wgpu_native::handle_error_fatal::ha412e282d23de10b
  12:     0x7fb0fb56f0ed - wgpuSurfaceConfigure
  13:     0x7fb0fbdac407 - _ZN10WgRenderer16surfaceConfigureEP15WGPUSurfaceImplR9WgContextjj
                               at /home/hermet/thorvg/build/../src/renderer/wg_engine/tvgWgRenderer.cpp:396:25
  14:     0x7fb0fbdac407 - _ZN10WgRenderer16surfaceConfigureEP15WGPUSurfaceImplR9WgContextjj
                               at /home/hermet/thorvg/build/../src/renderer/wg_engine/tvgWgRenderer.cpp:374:6
  15:     0x7fb0fbdac407 - _ZN10WgRenderer6targetEP14WGPUDeviceImplP16WGPUInstanceImplPvjji
                               at /home/hermet/thorvg/build/../src/renderer/wg_engine/tvgWgRenderer.cpp:352:29
fatal runtime error: failed to initiate panic, error 5
Aborted (core dumped)

see: https://github.com/gfx-rs/wgpu-native/issues/448#issuecomment-2556549353


Synthesis in Vivado

Hello, I'm trying to synthesize this core in Xilinx Vivado, but it's complaining about this code:
reg cycen; reg [4:0] cycgo; always @(posedge cycen or cycgo1 or cycgo2 or cycgo3 or cycgo4 or cycgo5)begin 	cycgo <= {cycgo5, cycgo4, cycgo3, cycgo2, cycgo1}; end
with the error message of "mixed level sensitive and edge triggered event controls are not supported for synthesis "
What was the intention here, to have cygo1:5 group combinatorially, or have them latched on the edge of cycen?

Hi there,
It was my first verilog project, I was a beginner so there are many code segments which are not done properly.
My intention there was to get then latched at the edge of cycen.
You can make cycgo1:5 edge triggered to make it synthesizable.

Thanks!


API key visible to the client

These components are fantastic, but I foresee a big issue when embedding them in websites that have public access to all users. Whilst I understand that an API key is required to contact your services, using it as a value inside a web component property/attribute allows malicious users to get a hold of it and use it for their needs.
Are there any recommended patterns to avoid this, or perhaps a possibility to generate a single-use token.
Thankyou!

If you would like to upvote the priority of this issue, please comment below or react on the original post above with :+1: so we can see what is popular when we triage.
@OvidijusParsiunas Thank you for opening this issue. 🙏
Please check out these other resources that might help you get to a resolution in the meantime:

Check the issue tracker - bugs and feature requests for Google Maps Platform APIs and SDKs
Open a support case - Get 1:1 support in Cloud Console.
Discord - chat with other developers
StackOverflow - use the google-maps tag

This is an automated message, feel free to ignore.

@OvidijusParsiunas Thanks for your interest in these components and for raising the issue! We'd like to note that the Maps JavaScript API key is a client-side key that's intended to be embedded on your website, as seen in this basic example from the documentation.
For recommended patterns to prevent abuse on the API key, please see security best practices and in particular setting application restrictions so that only specific referrer websites/IP addresses are able to make calls using the API key.
If the guidance on API key restrictions above do not satisfy your use case, consider opening a new issue for OAuth support.


Fixes for Data Tables transformer

Fixed:

Dates are now formatted correctly as ISO strings
Reference labels are now formatted correctly.

Coverage decreased (-0.01%) to 95.801% when pulling 717f95dc5cf53ffd887d138aa30f0b59f2aa98fb on serializing-timestamps-datatables into 3ed230aa5a90bf59ca0cb34ce338296274c752b5 on master.


Update chain_state.rs (ChainStateInner struct changes)

Minor noticed errors in comments in the ChainStateInner struct

ACK 50cdcd7. Thank you for your PR!


Updating Readme From Branch example

Updates
I updated the readme file.Please review and accept.

Thoughts
I probably shouldn't, but okay :+1:


Proactive sample (ASP.Net Core 2)

Inline with Documentation sample (with a few modifications)
@JonathanFingold : Note the constructor of the bot and what's being injected.  I find this to be a bit more inline with our new configuration story.  Same thing with the Accessors.

remove the work Introduction.

Refers to: csharp_dotnetcore/17.Proactive-Messages/README.md:42 in 60393da. [](commit_id = 60393daf5e9ba8b31f508807965fa47df9b76597, deletion_comment = False)


feature request: shift click selects several datasets

it would be great, besides the select all option if you want to select a group of consecutive datasets, make use of shift + click

There has been no activity on this issue for more several months. We are closing it. If you think this still needs to be addressed please open a new issue.


fig3b as 26

We need to put this (or something similar) in the first subplot.

Maybe JC could improve the cartoon, I can send him the affinity designer file and upload it in place of subfigure 1.

Maybe @jclagarde you could improve the cartoon. I can send you the affinity designer file I have for this. Let me know if you have time and are interested! Have you figured out how to place the cartoon on the figures?

Yeah, we just finished that this afternoon. The other figures have cartoons, but @jclagarde is working on scaling them so the text is bigger.

@shak360 could you review this when you got a chance?

Codecov Report

Merging #408 into master will decrease coverage by 0.27%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #408      +/-   ##
==========================================
- Coverage   89.14%   88.86%   -0.28%
==========================================
  Files          20       20
  Lines        1317     1320       +3
==========================================
- Hits         1174     1173       -1
- Misses        143      147       +4

Flag
Coverage Δ

#unittests
88.86% <0.00%> (-0.28%)
:arrow_down:

Impacted Files
Coverage Δ

lineage/Analyze.py
11.32% <0.00%> (-0.33%)
:arrow_down:

lineage/states/stateCommon.py
93.82% <0.00%> (-1.24%)
:arrow_down:

Continue to review full report at Codecov.

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

Maybe @jclagarde you could improve the cartoon. I can send you the affinity designer file I have for this. Let me know if you have time and are interested! Have you figured out how to place the cartoon on the figures?

@Farnazmdi Yeah, I'd be glad! My email is<EMAIL_ADDRESS>feel free to send it whenever :) and yeah I figured out how to place the cartoons on the figures

If you can add that change @Farnazmdi , it should be good to go.

@Farnazmdi Looks good to me, feel free to merge


Is there a batch_norm layer in your "example/alexnet.yaml"

I read the file, and I find there's an annotation that tells me there's a batch_norm layer. However, according to the code, you haven't explicitly declare the batch_norm variable. Therefore, it seems that the batch_norm layer will not be initialized by default.

Yes you are correct that the comment indicates batch norm and bias but only has bias in the actual network. We will edit that comment.

The fix is lined up for v1.5.0 release.


Installation fails on macbook

install.sh fails on my MacBook.  There are no error messages.
The source of the failure is the silent failure of wget followed by a non-error exit code for install. Adding verbosity to wget invocation in install.sh exposes this error:

--2020-06-09 10:17:56--https://github.com/condaforge/miniforge/releases/latest/download/Miniforge3-MacOSX-x86_64.sh
Resolving github.com (github.com)... <IP_ADDRESS>
Connecting to github.com (github.com)|<IP_ADDRESS>|:443... connected.
OpenSSL: error:1407742E:SSL routines:SSL23_GET_SERVER_HELLO:tlsv1 alert protocol version
Unable to establish SSL connection.

This seems to indicate that wget is linked against an version of SSL that is not accepted by the remote host.  Re-installation of wget on my MacBook (via brew) hangs forever and does not install (no error messages since it hangs forever).

Ok, seems like your wget is broken. There isn't much we can do in emirge in this situation. What is your MacOS version? Where did you get wget from?

Try curl?

I think this was fixed by reinstalling wget.

Indeed, wget (among other things in /usr/local) was (are) dorked up after OS upgrade.  De-wedged the wget re-installation and all is well (after MattD's help).

Closing this issue.


WIP: translate to fr foundational user

/check-cla

@douniaHas Do you need help to translate it? Don't hesitate :)

/remove-lifecycle stale


[Q] What is the dummy.txt used for?

Hello, I can see I have a dummy.txt in the storage/whisper folder which shows in the grafana when looking at the graphite data source. I was wondering where it came from since I don't produce such metric so I found it here: https://github.com/graphite-project/carbon/blob/8af0b4f9eb62e2224926019edbcf1ea2cf0df31f/setup.py#L61
I am interested to know why is that there and why it needed?

The only purpose is so that the install process creates those directories. You might see a ".keep" file used for the same purpose by some other software projects on linux systems.
I think that the first step towards getting rid of the need for these "dummy.txt" files is https://github.com/graphite-project/carbon/pull/835

Hi @ploxiln, thanks for the explanation. What was weird was that dummy.txt is it looks like a file, but indeed is a directory. However, your answer answers my question so I am closing this


clean up Makefile, adjust 'clean' target

remove deprecated targets
ensure 1 empty line between targets

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@497e15f). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #9729   +/-   ##
=========================================
  Coverage          ?   42.24%
=========================================
  Files             ?      597
  Lines             ?    78209
  Branches          ?        0
=========================================
  Hits              ?    33041
  Misses            ?    41119
  Partials          ?     4049

Continue to review full report at Codecov.

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

One more change:

make clean now cleans everything, introduced make clean-go to just clean build and integration tests.

Maybe we should also update the documentation.

Reverted controversial clean change, should be good to go now.


Fix default APB_IMAGE

automation-broker doesn't exist.

Coverage remained the same at 38.907% when pulling 84d71206a1dce278a17ff12d6d16a00217e1d0fe on eriknelson-patch-1 into e653dddeb5896f278bfea3398f4ddca92722b7cc on master.


new type of Display integrated into Dashboard and Switch Panel

Please review this code

May you update the codebase, test your commits and resubmit the pull request?