Use ReviewNB for rich notebook diffs?

I found this tool that might make reviewing PRs in this repo easier: https://www.reviewnb.com
It allows for rich diffs of notebooks (instead of having to visually compare JSON or downloading the notebooks to your computer and manually diffing).

Enabled by @kozbo and working fine. Thanks!


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 86 ms

Resolved: ClamAV performance has improved in 5f476fe.


searchInput type to also accept Integers

vue-autosuggest version: 1.4.2-1
node version:latest
npm (or yarn) version:latest

Relevant code or config

What you did:
What happened:

 Invalid prop: type check failed for prop "searchInput". Expected String, got Number.

Reproduction repository:

Problem description:
Hi, thanks for the lib. Currently, property searchInput type checks for String. This throws a vue warning when an integer is passed and selected. Is there a way to get around this warning or perhaps adding a more generic type to searchInput to accept both String and Integers.
https://github.com/Educents/vue-autosuggest/blob/master/src/parts/DefaultSection.js#L7
Suggested solution:
Allow both Integer and String
The way I am dealing with this is via converting each item inside of suggestion array to string.
    getSuggestions (obj, key) {
      let data = []
      for (let item of _.uniq(_.map(obj, key))) {
        data.push(String(item))
      }
      return [{data: data}]
    },

Where are you overriding searchInput? This value is an internal property so if it is overridden it should be a string. Here is a codesandbox where I tried to reproduce your problem:
https://codesandbox.io/s/yw977zn14j
In the codesandbox renderSuggestion and getInputValue I am returning ints and I am not getting the warning so not sure why you are. If you could reproduce your usage case in codesandbox I could take a look.

Hi thanks for the sandbox.
Yes the searchInput is internal property and I am not overriding it all.
Steps to reproduce the warning

focus on input
select int
click elsewhere on dom
focus on input (warning)

thank you for the clarification! seems like unwanted behavior. will get a patch in soon.

using v-model causes the same error now!


Yarn compatibility

Is your feature request related to a problem? Please describe.
I am using yarn in my project and I cannot use this cli if npm package manager is not installed
Describe the solution you'd like
Make compatible with yarn. When you execute ```hjs new my-name-proyect``, it will ask you for the package manager.

1.9.0


Input validation against the openapi spec

Currently the input is pretty much unvalidated. If the json body or query params of the request can get parsed into the golang struct it's seen as valid.
So on the one hand I think we need to expand our openapi spec to describe parameters and content better, and then validate the request based on that spec.
Potentially using https://pkg.go.dev/github.com/getkin/kin-openapi/openapi3filter ?

Fixed in #169


⚠ Avatar API Endpoint has degraded performance

In d9832a8, Avatar API Endpoint (https://avatar.roblox.com/v1/avatar-rules) experienced degraded performance:

HTTP code: 200
Response time: 1673 ms

Resolved: Avatar API Endpoint performance has improved in edf0d4e after 11 minutes.


Update TODO.txt

Isn’t it already done? Or are you making your own icon bounce?

Isn’t it already done? Or are you making your own icon bounce?

mine

Isn’t it already done? Or are you making your own icon bounce?

mine

Ok, once you are done with it, tell it to me for me to push the code

@XdgamerZ38 I might close theses PRs due to them being outdated as hell

if you don't mind @Wither362

Really, do what you want

Really, do what you want

alright


(Bug) invitee names not shown

@codev911 thanks we are lookiing into that.
@serdiukov-o-nordwhale lets discuss this

@iLystopad lets test this asap so we can release a hotfix

@sirpy The issue is still reproduced. The name of the invitee is not displayed. Also, the name of the newly invited user is not displayed.
Checked on the QA 1.22.13-2
Device: Desktop // Windows 10 // Chrome 88.0.4324.150
Attachment:

@iLystopad
the users in the example you tested have been deleted so their info is not available.
please test with registered accounts (you can simply perform the signup again with the deleted accounts)

@sirpy The issue is reproduced with newly invited user. The name of the newly invited user is not displayed on the Invite screen.
Checked on the Dev 1.22.13-2 // QA 1.22.13-2
Device: Windows 10/ Google Chrome 88.0.4324.150
Attachment:
https://www.dropbox.com/s/g7q00yjurl7yyh0/The name of new invited user is not displayed on the invite screen.mp4?dl=0


404 errors on GDrive sync should be marked as failure to stop syncing

This fixes #1289. @rafatower CR this, please.

:+1:

@rafatower CR again this, please. I'm testing this improved error message at staging and I'll free it just afterwards.

@Kartones CR this, please

perfect, I see better this approach than the original of leaving as success but with retried times


Sol rates drops after a few hours.

I think after about 2-3h the GPUs or CPU just stop working or slowes down.  Sol/s rate drops from 26 to 20 for example when i restart. it is back to max. Test this with 0.2 and 0.3a
I noticed this on all my systems.
RIG: Windows 10 64 AMD 16.10.3 with 8 AMD GPUS. I7 4790K Drops from 121 to 116
SYS1: windows 7 64: xeon e5-2690 Nvidia Quadro M5000 Drops from 89 to 70 Driver 369.26
SYS2: windows 7 64: xeon X5687 Nvidia Quadro k5000 and K5200 Drops from 26 to 20 Driver 369.26
SyS3: windows 7 64: xeon X5687 Nvidia Quadro 5000 Drops from  17 to 14 Driver 369.26
ALso I nave noticed that http://coinsforall.io/ miner is 20% faster than yours. for example just the amd cards does 123 Sols, Excluding CPU with this but your does 120 including the CPU of 23 Sol/s

I have noticed this as well, with a 10-25% dropoff in hashrate after 8 hours.

I am also seeing this on 2 systems that both have a i5 2400s with same mobo. Fresh install from same usb stick (linux mint 18). Installed the miner right after the fresh installs and nothing else. 1 stays ~14 for hours while the other one starts out around 13-14 but slowly drifts down to 10 within 3 hours and 6-8 before I finally just reboot it and it goes back up to normal.


started building models

extracted working directory into root and deleted Develop folder, added schema with tables, set the sequelize connection.

Hey Justin, I should have specified in my initial commit - I started with committing the schema for review to make sure the overall structure was agreeable prior to writing all the models and their associations, mainly to make it easier to visualize before modularizing the tables in the models folder. Since you agreed to the overall schema structure, I've finished building out the models in their respective files to complete that task. I also added the Develop folder back and placed all subfolders back inside. I had not experienced any issue deploying to Heroku before without a Develop folder, but if the group prefers having it, I have no qualm with that. Let me know what you think of this update when you have a chance. Thank you.


I'd like to join this project's community

Your public repo
https://github.com/billmcgraw2024/gov4git-identity-public.git
Your public branch
main

approve

 Follow up
On Friday, 29-Nov-24 17:37:47 UTC by Gov4Git dev
@billmcgraw2024 was added to the community.


adding order book storage test case

added test cases for following classes

JsonAdapterOrder
JsonOrderBookStorage
JsonSerializableOrderBook

Codecov Report

Merging #262 (af4418b) into master (ec0c554) will increase coverage by 2.37%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master     #262      +/-   ##
============================================
+ Coverage     59.17%   61.54%   +2.37%
- Complexity      705      734      +29
============================================
  Files           134      135       +1
  Lines          2680     2723      +43
  Branches        319      332      +13
============================================
+ Hits           1586     1676      +90
+ Misses         1000      948      -52
- Partials         94       99       +5

Impacted Files
Coverage Δ

...n/java/seedu/address/storage/JsonAdaptedOrder.java
100.00% <100.00%> (+100.00%)
:arrow_up:

...edu/address/storage/JsonSerializableOrderBook.java
100.00% <100.00%> (+80.95%)
:arrow_up:

.../main/java/seedu/address/ui/TotalOrdersWindow.java
0.00% <0.00%> (ø)

...java/seedu/address/logic/commands/FindCommand.java
100.00% <0.00%> (ø)

.../seedu/address/logic/commands/FindTaskCommand.java
0.00% <0.00%> (ø)

.../seedu/address/logic/parser/FindCommandParser.java
100.00% <0.00%> (ø)

...seedu/address/logic/commands/FindOrderCommand.java
0.00% <0.00%> (ø)

...du/address/logic/parser/FindTaskCommandParser.java
0.00% <0.00%> (ø)

...u/address/logic/parser/FindOrderCommandParser.java
0.00% <0.00%> (ø)

...ain/java/seedu/address/logic/commands/Keyword.java
100.00% <0.00%> (ø)

... and 4 more

Continue to review full report at Codecov.

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


STYLE: Allow conversion from nullptr to WeakPointer, not from zero

Small WeakPointer improvements:

Defaulted default-constructor (allowing the compiler to consider it
"constexpr" and "noexcept")
Removed the other user-defined special functions (Rule Of Zero)
Removed redundant user-defined operator=(ObjectType*)
Added converting constructor, WeakPointer(std::nullptr_t).

No longer allowed implicit conversion from the integer zero.
Included GoogleTest unit tests.

No longer allowed implicit conversion from the integer zero.

Where does this feature come from?

Thanks for your approval, @dzenanz

No longer allowed implicit conversion from the integer zero.

Where does this feature come from?

With this PR, WeakPointer would have a converting constructor from nullptr_t, as well as the existing one from ObjectType *. This would make implicit conversion from the integer zero ambiguous.
Without this PR, conversion from the integer zero would be performed by the existing converting constructor, from ObjectType *.

OK, I think the PR looks fine, but I leave it up to any of you to press Rebase-and-merge   :-)


Update sbt-protoc to 1.0.3

Updates com.thesamet:sbt-protoc from 1.0.0-RC4 to 1.0.3.
GitHub Release Notes - Changelog - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.thesamet", artifactId = "sbt-protoc" } ]

labels: sbt-plugin-update, semver-pre-release

Superseded by #34.


Feature Request: Instant Read

I think it'd be nice if there was an option to jump straight to images when you click on the thumbnail, instead of taking it to the gallery page with read and download. Then maybe add an option to go from the viewer to the gallery? Thank you!
When I hold onto the thumbnail for 1~2 second, options for download and favorites show up. I think it'd be nice if there a third option, read. What I originally proposed would be more preferable, but I think this would be great too. Thanks!

Also, I can't seem to load favorites, neither from exhentai nor from e-hentai. I'm keep getting the sad panda error message, and I've tried to tweak the views to minimal on both e-hentai and exhentai but it does not work.
I'm currently using the version 1.5.0. Thank you!

That‘s a good idea. Maybe I should exchange the actions between on clicking thumbnails and entries in downloading page.
As for the sad panda, please log out, then log in.

It worked. Thank you so much!!


GitRepo in stalled condition Git Updating when using SSH

Rancher Server Setup

Rancher version: v2.6.6 and v2.6.2
Installation option: Helm Chart

If Helm Chart, Kubernetes Cluster and version: RKE1, - Rancher 2.6.2 with k8s 1.21.5 and Rancher 2.6.6 with k8s 1.23.6

Proxy/Cert Details: none

Information about the Cluster

Kubernetes version: 1.21.13 and 1.23.7
Cluster Type: Downstream

If downstream, what type of cluster?: Custom

User Information

What is the role of the user logged in? Admin

Describe the bug
When adding a GitRepo the GitRepo goes into Active state and I can visually see it gets the correct commit ID from the repository but after some time it goes into the Git Updating state and it stays there indefinitely
To Reproduce

Create a Git repository with fleet bundles and make sure to use your SSH credentials. (the fleet-examples repo can be used for this purpose)
Copy the SSH url and configure a GitRepo with it. Use your ssh public and private keys. This should match the public key configured in your Git server (ex. Github)
Apply the configuration now check the commit ID is shown in the GitRepo entry in the Rancher UI
After some time it will go into the Git Updating state

Result

The GitRepo condition Stalled: True (view Yaml -> conditions)
There isn't any logs about it in the controller or fleet agent.

Expected Result

Fleet clones the fleet repo properly and deploys the bundles to the downstream cluster.

Screenshots

Additional context
Rancher version v2.6.6 has fleet v0.3.9
Rancher version v2.6.2 has fleet v0.3.7
GitRepo Conditions
status:
  commit: 763fe7f7defce8f7878469efe5a2b213e4432cbd
  conditions:
  - lastUpdateTime: "2022-07-11T15:40:23Z"
    status: "True"
    type: Ready
  - lastUpdateTime: "2022-07-11T16:45:33Z"
    status: "True"
    type: Accepted
  - lastUpdateTime: "2022-07-11T15:40:24Z"
    status: "True"
    type: ImageSynced
  - lastUpdateTime: "2022-07-11T15:40:24Z"
    status: "False"
    type: Reconciling
  - lastUpdateTime: "2022-07-11T15:40:47Z"
    reason: Stalled
    status: "True"
    type: Stalled
  - lastUpdateTime: "2022-07-11T16:45:33Z"
    status: "True"
    type: Synced

@isaax Can you expand on what this is blocking? We're trying to tidy up/deprecate the confusing status/blocker label

@richard-cox After the last analysis by Mario it seems this is only occurring when using the UI. Using/Setting ssh repositories with kubectl works. I don't think we need the blocker label anymore.

While working on ui automation for ssh private git repos, we noticed that tests were passing in Rancher 2.8-head using Fleet 0.9.3-rc2 while failing in Rancher 2.7-head using Fleet 0.8.3.
As it turned out we noticed this issue keeps occurring on Rancher 2.7 (latest check done in v2.7-bd0bb772b60a447d352789b3d7c4368ffe50eecf-head ):

This works well when a carriage return is added at the end of the line as @manno pointed out on the private key (on public key is optional).
On 2.8-head  (latest check on v2.8-10907fed8e43c1ff3124271f74eab63e47723619-head) it is working well without the extra line (and with the extra line as well). I guess this is because I don't see Tekton being used:

I discussed this with Mario offline and the documentation points out on the info panel here to use the extra line on private keys. Since this is no longer needed for 2.8, I guess we could :

leave it as it is and always add the extra line

point out that is only needed for older versions in 2.7

fix this in the affected versions.

BTW: Automated tests for 4 private repo providers (Github. Gitlab, Azure, Bitbucket) using ssh are currently in place here.  Currently adds an extra line line on both keys to ensure it works on all versions. This is subject to be adapted depending on what is decided for this issue.


Getting vim-airline working in Windows 10 with latest nvim and nvim-qt releases.

environment

vim: Neovim
vim-airline: Master
OS: Windows 10

actual behavior
Disclaimer: New to VIM, new to Neo VIM. (Like yesterday new)
Errors coming from highlighter.vim. That made it unuseable, anytime a buffer or window changed I was flooded with red error messages rendering neovim useless for me, but I really wanted this plugin.
When using Neovim-QT (Which from my understanding counts as a GUI and not a Terminal) it seems that airline still tries to treat it as an old windows terminal and ends up calling a function airline#msdos#round_msdos_colors(rgblist) which would have an index error on a:rgblist[2]. I found that I was able to simply change:
return airline#msdos#round_msdos_colors(rgb) in highlighter.vim to return rgb
Which brought me to the next series of errors, that seemed to be some formatting issues of the hi command undoubtedly from my changes above I figured. However by removing the if s:is_win32term condition and body altogether it fixed the problems and airline seems to be working flawlessly now.
Although I can essentially guarantee my fixes aren't the appropriate fixes to fix this problem I hope they can help you guys figure out whats up. Here's my vimrc(Yes it's messy yes it's nooby):
set background=dark

call plug#begin('~/AppData/Local/nvim/plugged')

Plug 'vim-airline/vim-airline'
Plug 'vim-airline/vim-airline-themes'

Plug 'scrooloose/nerdtree', { 'on':  'NERDTreeToggle' }
Plug 'flazz/vim-colorschemes'
Plug 'kien/ctrlp.vim'
Plug 'terryma/vim-multiple-cursors'

call plug#end()

filetype plugin on
filetype indent on

set autoread

let mapleader = ","
let g:mapleader = ","

let g:enable_bold_font = 1

set history=700

set wildmenu

set tabstop=4

set title

set number

set cursorline

set noswapfile
set nobackup
set nowb

set softtabstop=0 noexpandtab

set shiftwidth=4

set noerrorbells
set vb t_vb=

let g:airline#extensions#tabline#enabled = 1

NERDTree D:\

colorscheme BlackSea

map <Leader>c :e C:/Users/dennmat/AppData/Local/nvim/init.vim <CR>
map <c-s> :w <CR>

nmap <C-h> <C-w>h
nmap <C-j> <C-w>j
nmap <C-k> <C-w>k
nmap <C-l> <C-w>l

inoremap jj <Esc>

" Copy to system clipboard
vnoremap <C-c> "+y
" Paste from system clipboard with Ctrl + v
inoremap <C-v> <Esc>"+p
nnoremap <Leader>p "0p

" Toggle between last 2 buffers
nnoremap <leader><tab> <c-^>

But from very little knowledge in any and all of this, my best guess is it seems like it's trying to treat nvim-qt as a windows terminal with limited color pallet instead of as a gui.
Perhaps there was an easy flag I could have set somewhere but I couldn't find it or mention of it anywhere online.
Also there is a possibility that this could be a nvim or nvim-qt issue(in that it's not reporting what it is properly or something) if so let me know and I'll move this issue over there
Thanks, if you look at this great, if not that's cool to.
Thanks for maintaining and writing this cool plugin :)

I don't understand. In nvim, doesn't has("gui_running") not return 1? It should, if it is a gui.
I also don't understand, what other changes you made. can you show me a diff please?

I ran this: :echom has("gui_running") the result was a 0 so I'm assuming that this then is a nvim-qt/nvim issue. If so sorry for taking up your time, still new to all this!
Anyway, if you're still curious, my git diff (I just commented stuff out instead of removing, as I was experimenting)
diff --git a/autoload/airline/highlighter.vim b/autoload/airline/highlighter.vim
index 58a481b..95f3974 100644
--- a/autoload/airline/highlighter.vim
+++ b/autoload/airline/highlighter.vim
@@ -13,7 +13,7 @@ function! s:gui2cui(rgb, fallback)
     return a:rgb
   endif
   let rgb = map(split(a:rgb[1:], '..\zs'), '0 + ("0x".v:val)')
-  return airline#msdos#round_msdos_colors(rgb)
+  return rgb "airline#msdos#round_msdos_colors(rgb)
 endfunction

 function! s:get_syn(group, what)
@@ -57,10 +57,10 @@ function! airline#highlighter#exec(group, colors)
     return
   endif
   let colors = a:colors
-  if s:is_win32term
-    let colors[2] = s:gui2cui(get(colors, 0, ''), get(colors, 2, ''))
-    let colors[3] = s:gui2cui(get(colors, 1, ''), get(colors, 3, ''))
-  endif
+  "if s:is_win32term
+  "  let colors[2] = s:gui2cui(get(colors, 0, ''), get(colors, 2, ''))
+  "  let colors[3] = s:gui2cui(get(colors, 1, ''), get(colors, 3, ''))
+  "endif
   let old_hi = airline#highlighter#get_highlight(a:group)
   if len(colors) == 4
     call add(colors, '')

Thanks

Hi Matthew!
On Sa, 17 Sep 2016, Matthew Dennis wrote:

I ran this: :echom has("gui_running") the result was a 0 so I'm
assuming that this then is a nvim-qt/nvim issue. If so sorry for
taking up your time, still new to all this!

Please report that than to the neovim-qt project.
Best,
Christian
Reich ist man nicht durch das was man besitzt, sondern durch das, was
man bereit ist zu entbehren!
-- Epikur von Samos (341-271 v.Chr.)

Hello,
I had this same issue and I want you to re-open this issue to deal with it.
I wanted to use vim-airline with neovim-qt on Windows 10 and everything looked fine. But when I type 'vs' to open vertical split, lots of error messages poured.

I found this page which says that this issue is the fault of neovim-qt and is closed.
But I found some historical processes and I think perhaps vim-airline has to deal with it.
First, NeoVim GUI only opens after init.vim. Therefore gui_running in init.vim is always 0.
We can configure vim-airline in ginit.vim to avoid this issue, but it is not good solution.
Vim-airline can be used in tui, so there is no reason that it is placed in ginit.vim only.
Second, NeoVim removed telling lie about gui_running. Therefore, lots of plugins using gui_running modified thier checking conditions.
Third, after that, there was an issue for this change, and a PR was made for this issue and merged.
Forth, but 's:is_win32term' in highlighter.vim just check 'gui_running', so when neovim-qt run, is_win32term returns true, then s:gui2cui is executed, then airline#msdos#round_msdos_colors() is executed, then lots of crashes occurs.
I fixed this issue by appending "&& !has('nvim')" in the line of definition of s:is_win32term.
However, I don't know all structures of this project and other environments, so I don't believe this is the correct action which is enough to be merged in this project.
Therefore, I would like this issue to be re-opened and some neat solutions to be found and to be merged to this project.
Thanks.

if the length of the color list is not 3 items, I think it is neovims fault. Please discuss with them then.

Well, did you mean that on Windows, even if it is GUI environment, 'is_win32term' is true and the function 'gui2cui' must be called? (Even working well without them)
I don't know much about the difference between GUI and CUI management in vim but it looks little curious.

no. is_win32term is only supposed to be true, when running in the terminal.

Okay, then the problem is that neovim_qt is GUI, but gui_running is not set when vim-airline is loaded.
TL;DR I just want vim-airline to change its win32 CUI/GUI distinguish algorithm for neovim-qt, which is missed in this change.

Here are some backgrounds

nvim vs nvim-qt
Nvim is CUI but nvim-qt is GUI client. If nvim run with vim-airline, it works fine. The problem is nvim-qt, which is GUI environment.
init.vim vs ginit.vim
There are two configuration file(like vimrc, gvimrc), init.vim and ginit.vim.
When init.vim is loaded, gui_running is not set. After that it is loaded completely, then ginit.vim is loaded.
Nvim loads just init.vim, and nvim-qt load init.vim and ginit.vim consecutively.
Problem
The problem is that vim-airline is loaded in init.vim when nvim-qt is run. Because it is not in ginit.vim, gui_running is not set. But the environment is GUI.

There are some options

Move to vim-airline loading from init.vim to ginit.vim
We can choose the option that moving the vim-airline loading from init.vim to ginit.vim. It looks working because no crash occurs. But some feature is not working, such as NORMAL text region at the left bottom is disappeared.
However I think vim-airline is not GUI-only plugin, so It can work properly when it is loaded in init.vim.
Change neovim side to set 'gui_running' when init.vim is loaded on nvim-qt
NeoVim decided not do that. Look at this [RFC] ui: revert "gui_running" hack #4155. This decision can seem somewhat onreasonable or selfish, because lots of plugins with colorscheme use this variable to check CUI/GUI. But vim-airline already handled this issue. Move to next.
Change vim-airline's terminal checking algorithm
Vim-airline already handled this issue. Neovim true color support #991
But 'is_win32term' is missed. It might be changed properly but it didn't. It just check 'gui_running' to distinguish CUI and GUI which is not working on nvim-qt.

I hope you could understand what I mean. I understand that you doesn't need to support every vim environments. I just want to notice a point that enlarges your coverage and makes more the users of vim-airline be happy.
Anyway, It is very very rare case that Windows 10 + NeoVim QT + Vim-Airline....
Thank you for reading and answers even though it is a closed issue.
I am sorry if my sentence seemed impolite. I am not a English native.

Are you sure, that vim-airline is loaded before ginit.vim is sourced? That sounds wrong and I believe this is different than for Vim. I think there it first loads vimrc, then gvimrc and plugins are loaded last. I would be fine with changing the is_win32term logic to include something like has("nvim") but how can we then distinguish between a nvim running in the terminal and nvim-qt? or perhaps we shouldn't make that variable static? Hm, does this patch change anything:
diff --git a/autoload/airline/highlighter.vim b/autoload/airline/highlighter.vim
index 16ae66e..eee256b 100644
--- a/autoload/airline/highlighter.vim
+++ b/autoload/airline/highlighter.vim
@@ -3,7 +3,6 @@

 scriptencoding utf-8

-let s:is_win32term = (has('win32') || has('win64')) && !has('gui_running') && (empty($CONEMUBUILD) || &term !=? 'xterm')

 let s:separators = {}
 let s:accents = {}
@@ -64,7 +63,8 @@ function! airline#highlighter#exec(group, colors)
     return
   endif
   let colors = a:colors
-  if s:is_win32term
+  let is_win32term = (has('win32') || has('win64')) && !has('gui_running') && (empty($CONEMUBUILD) || &term !=? 'xterm')
+  if is_win32term
     let colors[2] = s:gui2cui(get(colors, 0, ''), get(colors, 2, ''))
     let colors[3] = s:gui2cui(get(colors, 1, ''), get(colors, 3, ''))
   endif

I thought that the 'gui_running' is set when ginit.vim is loaded, but I am wrong. I tested your suggested code and force vim-airline to be loaded in ginit.vim using vim-plug. There is no 'gui_running' even after vim loaded ginit.vim.
I have same concern that how we can distinguish between nvim and nvim-qt.
The comment of this issue of neovim-qt says that 'g:GuiLoaded' can distinguish those. Your suggestion + checking 'g:GuiLoaded' solves the errors, but it can't distinguish nvim and nvim qt. According to my experiment, running nvim in terminal also sets g:GuiLoaded.
All of these things, 'nvim', 'gui_running', 'g:GuiLoaded' can not be used to distinguish nvim and nvim-qt.
At last, I think the approach is very different between gvim and neovim. I thought there might be a simple solution, but now I think I was wrong.
Thank you for your attention.

I left a comment there. Hopefully those guys know how to properly distinguish it then. If this ever gets resolved properly, feel free to open a new issue (or leave a comment here)

According to my experiment, running nvim in terminal also sets g:GuiLoaded

@ybbarng this should only happen if you are loading the gui shim manually in your init.vim, usually nvim-qt loads that for you. Are forcing this in init.vim?

I think the approach is very different between gvim and neovim. I thought there might be a simple solution, but now I think I was wrong.

Correct. Neovim allows for a GUI to attach/detach at any time and you can have multiple interfaces open, which means a global gui_running option would not make sense.  gui_running in nvim is ALWAYS false. Since s:is_win32term uses has('gui_running') then it is no surprise that it does not work as intended.
Internally nvim always starts in a state where there is no GUI. The GUI then attaches to neovim. Processing of init.vim happens before the GUI attaches. Processing of ginit.vim happens after the GUI attaches.
In a nutshell there is currently no standard way to identify whether the a GUI is about to be attached from within init.vim. g:GuiLoaded is nvim-qt specific, and only tells you that the gui plugin was loaded not whether a GUI is attached. There are already open issues upstream to address this, but a viable PR has yet to be merged.
Defering plugin loading to ginit.vim works for the GUI but disables the plugin for the terminal which is not ideal for most people. For a dirty workaround in init.vim you can use nvim-qt -- --cmd "let g:DelayPluginsForGUI=1" and use that variable for the checks. I certainly dont mind merging a similar solution in nvim-qt, but this is only useful for init.vim hacks.
PS:  keep in mind that in Windows, the terminal UI is not available in the latest stable nvim release, only in the master branch. So any specific issues concerning wrong colors or state really should be pushed upstream.

@equalsraf IIRC, that problem exists for as long as neovim exists and is very unsatisfying. Can't you simply lie to the plugins, if you know that there will be a GUI attached anytime soon? Or perhaps, have another has('nvim_gui_is_starting') so that the decision can be detected and left to the plugin writers what to do?

If the problem here is only color support I think we are over complicating things.
If the option termguicolors is set then you are in 24 bit terminal or in a GUI. AFAIK all nvim GUIs set this. This can can be used init.vim, the GUI ensures nvim sets this prior to sourcing it.
So adding something like this to s:is_win32term
&& !(has('nvim') && &termguicolors )

This is not fully accurate, the option is not set by default even when a terminal supports colors. But there is no working terminal UI for nvim in Windows right now so its hard to be sure if it will have support.
It may happen that the nvim windows TUI ends up supporting all the colors and then I'm sure someone will complain later to fix it :D, but I think this as correct as it can be right now.

okay, good suggestion. I changed your condition (57e564b22748e5ffcd7a459) slightly to:
&& !(exists("+termguicolors") && &termguicolors)

which makes it work for Vim as well.


Add wbs

Bremerhaven wbs training

@adlan-b Regrettably, I have to close the request without merging, because this organization is a training center, not a college, nor a university. Please ensure that you are familiar with this repository rules located at the bottom of this page.
Please apply for an appropriate licensing option (which may also be free) at https://www.jetbrains.com/community/education/#courses.
Thank you for your understanding!


Sorting Algorithms in Python

Aim

What is the objective of the Script
To add few basic py files that has the code to sort a list.

Details

What the features will your script have
It has a function(def) that has the implementation for sorting algorithms in python.

@varshith-k what extra value will it add to the repo? all the sorting algorithms are already implemented in the best way possible.

@kaustubhgupta are all sorting algorithms added to the repo?

Is there a need to add sorting algorithms here?

I thought they might be useful python scripts.

@antrikshmisri @santushtisharma10 @Kushal997-das thoughts?

It's fine you can close this issue.

@varshith-k I think you should pick up something else as there are a lot of sorting algorithms already present in the codebase


Possible final fields with default accessors needs to be reviewed and updated

Checkstyle rule:
Fields should be 'final' when:

Initialized and never reassigned in constructor or other methods.
Only assigned in constructor and no other method.

Next cases below has one of this rules and fields are not final because they have default access to enable direct assignation from same package classes.
Please review each case and decide if this default access is required or if it can be changed to private and then adding a default access method in the class to have that field updated (encapsulate fields).
Each file is currently suppressed. Please remove checkstyle suppress after updating the file
Reference info:
Originally posted by @conniey in https://github.com/Azure/azure-sdk-for-java/pull/5009/files
Cases
1 KeyCreateOptions.java  (KV)
2 LocalKeyCryptographyClient.java (KV)

Editing bug to split apart KV and Identity work separately

As a note, KeyCreateOptions was renamed to CreateKeyOptions.

Created a PR (#13323) to address this.


jQuery(element).change() not call complete\incomplete methods in inputmask

jQuery(element).change() not call complete\incomplete methods in inputmask

@FRAGnatt ,
What is the issue?  Can you better explain.

When I have element with mask for example
jQuery('#myElem').inputMask({
'mask':'#{1,3}',
'oncomplete' :function(){console.log(1)}})
When After inputMask is completed, I want to revalidate my inputmask.
In previously version I used jQuery('#myElem').change(). And my callback was called. But in version 3.2 I had problem with it.
Actually it's a bad that a new version with bugs and not compatible with old

@FRAGnatt ,
Hmmm I never had an event binding to the change event, ...  In which version was this behavior?
Concerning your other remarks.
It is impossible for me to test all cases with every release.  I only see that the unittests pass.  Be welcome to contribute to the unittests!

Hmmmm. Sorry, I think it is mistake my old code. At now I test jQuery().change() at version 3.1.63 And it's not work. Sorry, I think I made a mistake when I wrote this issue.
Can you help me? How I can trigger event to validate my input with inputmask ?  Not validate him, not use method isComplete. I want trigger my callback from the outside. The event  onincomplete or oncomplete, or maybe onkeydown

@FRAGnatt ,
Try with 	$(selector).trigger("setvalue.inputmask");


Bridge Discord and Gitter through #legacy Discord channel

While the vast majority of Codewars users have welcomed Discord with open arms, there are still a tiny percentage of users who refuse to switch to Discord for various reasons such as privacy concerns or usability, etc. While that percentage is tiny, it includes some of the best known power users on Codewars such as @Voileexperiments and @JohanWiltink . Nevertheless, without a proper bridging mechanism between Discord and legacy Gitter, the community remains split into two with no reliable method of inter-communication.
In the past few weeks, @XRFXLP has managed to relay Discord messages to Gitter and vice-versa through the use of two separate bots. He concluded that it could not be reliably achieved with a single bot due to race issues(?). This experiment demonstrates that such a bridge is feasible to implement.
The question now is: do we consider it worth the time and effort to create such a bridge? If so, perhaps we can build the Discord to Gitter relaying with this existing bot and create a separate Gitter bot for the reverse direction. What do you think?

I'm not interested. Discord messages are not public, but sending them to Gitter makes them public. A much more serious privacy concern than not wanting to share their Discord tag. It must be opt-in.
What's worth bridging anyway?
I was thinking Discord and GitHub Discussions to be used like this:

Discord (sync, private): casual chats, help, ephemeral, thinking out loud
GitHub Discussion (async, public): Q&A, discussing more concrete ideas (can be based on discussions in Discord)

We can ask for their input on GitHub, and they can start one.

For the split, it's not ideal, but it's not like we've split in half. The migration was definitely worth these small downsides. I'm keeping Gitter until it becomes a pain, mainly for the past messages. I wish Gitter had "archive" option, so I can archive all the rooms except for the main one.

Discord messages are not public, but sending them to Gitter makes them public. A much more serious privacy concern than not wanting to share their Discord tag. It must be opt-in.

Besides not wanting to share my discord tag (my discord is for an entirely different purpose), note that discord uses a different set of redentials: Github discussions, CW (if your account's tied to github account) and gitter are all tied to a single github account. When there are two sets of separate credentials all kinds of problem arose, which is left as an exercise to the reader but I can assure that it's a lot.


kdeApplications.mbox-importer: fix hydra build [18.03]

Motivation for this change
Backport of #39018.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

Success on aarch64-linux (full log)
Attempted: kdeApplications.mbox-importer
Partial log (click to expand)
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/xy1lx2czs4m9pj97982kyk5wla7gc1jj-mbox-importer-17.12.2
shrinking /nix/store/xy1lx2czs4m9pj97982kyk5wla7gc1jj-mbox-importer-17.12.2/bin/mboximporter
strip is /nix/store/lvx1acn1ig1j2km8jds5x3ggh3f2wa8v-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/xy1lx2czs4m9pj97982kyk5wla7gc1jj-mbox-importer-17.12.2/bin
patching script interpreter paths in /nix/store/xy1lx2czs4m9pj97982kyk5wla7gc1jj-mbox-importer-17.12.2
checking for references to /build in /nix/store/xy1lx2czs4m9pj97982kyk5wla7gc1jj-mbox-importer-17.12.2...
postPatchMkspecs
postPatchMkspecs
/nix/store/xy1lx2czs4m9pj97982kyk5wla7gc1jj-mbox-importer-17.12.2

Success on x86_64-linux (full log)
Attempted: kdeApplications.mbox-importer
Partial log (click to expand)
copying path '/nix/store/5n7yim6mmv5d0w9xxpida1r8r1vwwydk-akonadi-contacts-17.12.2' from 'https://cache.nixos.org'...
copying path '/nix/store/b48f800s5dy5ixf1g1330pjwb5gy1xj0-libkdepim-17.12.2' from 'https://cache.nixos.org'...
copying path '/nix/store/y0x32qq5jcs1p4jfn1yznszlmxck0l5q-mailimporter-17.12.2' from 'https://cache.nixos.org'...
copying path '/nix/store/9z348kd2gs5pr771q3i9s349n5shc9dg-pimcommon-17.12.2' from 'https://cache.nixos.org'...
copying path '/nix/store/qr2pvvygrpd65jg7c9xi3maizd8vjmzd-kdepim-apps-libs-17.12.2' from 'https://cache.nixos.org'...
copying path '/nix/store/lh9j612bk9alx2fqjmf2ld3vlsrjfy0j-libgravatar-17.12.2' from 'https://cache.nixos.org'...
copying path '/nix/store/hz35l78skfnqdnw1615p48jhiqs7fdjl-messagelib-17.12.2' from 'https://cache.nixos.org'...
copying path '/nix/store/ng2b3j8ds6fhkq5xlbaykiwzr6mv8zm6-mailcommon-17.12.2' from 'https://cache.nixos.org'...
copying path '/nix/store/r420vvdypycx53ckg1ds3d5v3bly4dkp-mbox-importer-17.12.2' from 'https://cache.nixos.org'...
/nix/store/r420vvdypycx53ckg1ds3d5v3bly4dkp-mbox-importer-17.12.2


create basic unit test case for qos

create basic unit test to verify scheduler module .

Qos.bess basic script is developed,, output can e verified by prints


[Idea]: Advanced "No Permission" (maybe bug)

Describe the feature
hello, i have a problem
when someone dont have permission to /mvtp when it used it shows all the permission user needs
how i can disable that? so it just says "you dont have access to do this..."
not like it shows what permission user need.
forget about 'displaypermerrors' cuz it disables actual "No Permission" message too
So the feature is "disable showing list of permissions needed while also enable actual 'No permission' message"
How is the feature useful to all Multiverse users?
Ofc it prevent to get annoyed :/
Agreements

[X] I have searched for and ensured there isn't already an open issue regarding this.

In the config.yml, toggle displaypermerrors to false.

In the config.yml, toggle displaypermerrors to false.

As i said... No 'displaypermerrors' cuz it disable the actual "no permission" message too...
Did y read it all?

No i didnt read it all...
But with a little bit of thinking, it is achievable.....
displaypermerrors](displaypermerrors: 'false'
Then with your perms plugin, give the perm multiverse.teleport,self.w a false.
There are ways this can be achieved in the short/medium term...

fixed. tysm


Diagram breaks when I have connector defined inside a service

Please see the below code I get a spinner when I open the diagram.
Affected version v3.3.2
import ballerinax/github;
import ballerina/http;

configurable string gitHubPat = ?;

github:ConnectionConfig config =  {
    auth: {token: gitHubPat}
};

type RepoStars record {
    string name;
    int stars;
};

# A service representing a network-accessible API
# bound to port `9090`.
 service / on new http:Listener(9090) {
    github:Client gc;

    function init() returns error? {
        self.gc = check new github:Client(config);
    }

    # A resource for generating greetings
    # + orgName - the input string name
    # + max - number of repos to return, defaults to -1, meaning all
    # + return - string name with hello message or error
    resource function get repos(string orgName, int max = 5) returns RepoStars[]|error {
        stream<github:Repository, github:Error?> repos = check self.gc->getRepositories(orgName, true);
        RepoStars[]? repoNames = check from var item in repos
            order by item.stargazerCount descending
            limit max
            select { name: [item.name](http://item.name/), stars: item.stargazerCount ?: -1};
        return repoNames ?: [];
    }
}

Related to https://github.com/wso2-enterprise/choreo/issues/16074

There will always be cases when the diagram breaks. If we can figure out a way to gracefully say "Sorry I can't draw that - please raise an issue  with the sample code" type thing that'll be perfect.


mj_loadXML error: b"XML Error: Schema violation: unrecognized attribute: 'autolimits'\n\nElement 'compiler'

I got this error when I was running extract _ OBS _ from _ raw _ datasets.sh. After checking, my mujoco_py version is <IP_ADDRESS>. I want to know the correct version of mujoco_py when running this document.

My best guess is that you need to upgrade your mujoco version (e.g. pip install -U mujoco). Depending on your version of robosuite, you shouldn't be using mujoco_py at all


Add version: pnpm.pnpm.7 version 7.5.0

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[ ] Does your manifest conform to the 1.1 schema?


How to configure EquirectangularDepthSensor on the latest version of habitat sim?

Habitat-Lab and Habitat-Sim versions
Habitat-Lab: v0.3.0
Habitat-Sim: v0.3.0
Question:
I want to use EquirectangularDepthSensor data in the experiment, but it seems that I cannot successfully complete the configuration. May I ask how to configure EquirectangularDepthSensor on the latest version of habitat sim?
(EquirectangularDepthSensor seems to be supported on previous versions (such as v0.2.0) of habitat lab and habitat sim.)
Previous failed attempts:
I configure it in the yaml file using the following method:
simulator:
agents:
main_agent:
sim_sensors:
rgb_sensor:
width: 512
height: 512
equirect_depth_sensor:
width: 481
height: 241
habitat_sim_v0:
gpu_device_id: 0

But when the code runs, it will generate the following error message:
omegaconf.errors.MissingMandatoryValue: Missing mandatory value: habitat.simulator.agents.main_agent.sim_sensors.equirect_depth_sensor.type
full_key: habitat.simulator.agents.main_agent.sim_sensors.equirect_depth_sensor.type
reference_type=SimulatorSensorConfig
object_type=SimulatorSensorConfig
Exception ignored in: <function Simulator.del at 0x7f7bab357c10>
Traceback (most recent call last):
File "/home/zhaishichao/anaconda3/envs/habitat4/lib/python3.9/site-packages/habitat_sim-0.3.0-py3.9-linux-x86_64.egg/habitat_sim/simulator.py", line 528, in del
File "/home/zhaishichao/anaconda3/envs/habitat4/lib/python3.9/site-packages/habitat_sim-0.3.0-py3.9-linux-x86_64.egg/habitat_sim/simulator.py", line 131, in close
AttributeError: 'HabitatSim' object has no attribute '_initialized'

❓ Questions and Help

I have completed the configuration of EquirectangularDepthSensor in the latest version of habitat, thank you.

I have completed the configuration of EquirectangularDepthSensor in the latest version of habitat, thank you.

Can you share your Python and Yaml code?
Thx


Add order query params on API definitions returning collections

Tradegecko API offers the order argument in order to sort returned collections (See documentation here).
This pull request update the service description in order to use it.

Thanks!

I've just tagged as 1.1.0


Converting double sigma from beta to Unicode uses terminal sigma for first sigma

Example:
using GreekBetaCode
s = "QA/LASSAN"
U(s) == "θάλαςσαν"

Thank you, @neelsmith , for the valuable test and for reporting the issue!!
I'll assign myself to fix the issue.


Autostyle in the editor doesn't restore previous style properly

This doesn't seem to happen in deep-insights standalone or with the embed.

Also the reinstantiation seems to be mighty quick, could it be because of the blank cached tiles you mentioned in the demo @javisantana ?

This was probably something to do with my local tiler as it's not giving problems in production, and I can't get it to do the same thing again now.


Error: 5 NOT_Found: Requested Entity was not found.

When I upload a file via my Firebase storage bucket I can see that the function is triggered.  However I am continuously getting the following error when I look at the Function's Log:
callCustomModel
error occurred { Error: 5 NOT_FOUND: Requested entity was not found.
at Object.exports.createStatusError (/user_code/node_modules/@google-cloud/automl/node_modules/grpc/src/common.js:87:15)
at Object.onReceiveStatus (/user_code/node_modules/@google-cloud/automl/node_modules/grpc/src/client_interceptors.js:1188:28)
at InterceptingListener._callNext (/user_code/node_modules/@google-cloud/automl/node_modules/grpc/src/client_interceptors.js:564:42)
at InterceptingListener.onReceiveStatus (/user_code/node_modules/@google-cloud/automl/node_modules/grpc/src/client_interceptors.js:614:8)
at callback (/user_code/node_modules/@google-cloud/automl/node_modules/grpc/src/client_interceptors.js:841:24)
code: 5,
metadata: Metadata { _internal_repr: { 'grpc-server-stats-bin': [Object] } },
details: 'Requested entity was not found.' }
callCustomModel
AutoML API Error:  { Error: 5 NOT_FOUND: Requested entity was not found.
at Object.exports.createStatusError (/user_code/node_modules/@google-cloud/automl/node_modules/grpc/src/common.js:87:15)
at Object.onReceiveStatus (/user_code/node_modules/@google-cloud/automl/node_modules/grpc/src/client_interceptors.js:1188:28)
at InterceptingListener._callNext (/user_code/node_modules/@google-cloud/automl/node_modules/grpc/src/client_interceptors.js:564:42)
at InterceptingListener.onReceiveStatus<EMAIL_ADDRESS>

For anyone else who runs into this issue, I was able to resolve by correcting the const automl_model value in the index.js file.  I'm not sure if it's standard but my automl_model was the ID under the Model  page.  It wasn't the name or the Dataset ID.


No get method for @Id field?

I've read the documentation and the conversation in this closed issue here: https://github.com/dizitart/nitrite-database/issues/125
I understand the difference between the Id annotated field and the database internal NitriteId. But what I am still wondering is, why is there no convenience method to simply get a document by its defined document id?
I also could not find any sample on how to find a document by its id. Can you provide an example and add it maybe to the readme?

To get the NitriteId stored in a document use:
document.getId()

And to find a document by it's NitriteId, you can use:
collection.getById(nitriteId)

Let me know if you are searching for something else.

Sorry - I just realized that I formulated my question a little bit misleading. What I want to achieve is the following.
I have a user object where I defined the email as @Id. When I want to check if a specific mail adress is already registered in my database, I have to use the find method with an additional FindOption to check for its existence - here a sample:
repository.user.find(ObjectFilters.eq("mail", mail), FindOptions.limit(0, 1)).first()
Would be nice to simply do the following instead:
repository.user.get(mail)

Currently there is no such api as you desire. Even if I expose such a function, the inner working be exactly like - user.find(ObjectFilters.eq("mail", mail), FindOptions.limit(0, 1)).firstOrDefault().
Moreover, @Id is not mandatory in repository. You can use object in repository where there is no id field. In that case, what this api will return?
As I said in issue #125, the @Id field does not have any special treatment inside the repository. For your convenience, you can create such an utility method, or if you are using kotlin, you can create an extension method for such requirement.

Ok. I understand the current functionality. But I have to disagree from a general architectural perspective. What you offer is a NoSQL db without a simple get functionality. What is the point to have an Object collection without @Id field? Especially since you will have an internal _id value anyway? Why hide something from the developer which they will need in the end anyway? I call that a big flaw in your design. You offer a query mechanism without a simple get. I would highly recommend to include that as an enhancement in your next release.
I tried to use this DB for a very simple use case and failed to do so. I will go back to use SQLite or H2 instead

I am not claiming the design is suitable for every scenarios, but IMO the functionality you want will be better served by a key-value store.
Nitrite is a document store like mongo, do you see any such query api in mongo? ObjectRepository is just a built-in ODM layer on top of NitriteCollection. The reason I left the @Id field optional, is because of the use case where you don't have any control over the entity object and you can't modify it.
I'll try to see what I can accommodate in ObjectRepository in future with respect to the get call by id field without breaking existing scenarios, but at the end of the day you are always free to choose any solution that fits your requirement.


[Feature Request] Suppress "No such fusion" message

With multiple people spamming random combinations, the console becomes pretty useless to try to parse.  Should hide this message and only display successful Fusions.

Matt fixed it.


Ispn 14877 Possibility to configure CLI REST client using properties

https://issues.redhat.com/browse/ISPN-14877

Thanks for the contribution. Could you also add a note to connect.adoc 's description section ?

Closing in favour of https://github.com/infinispan/infinispan/pull/11398


Only create InputEventMouseMotion event on Windows if mouse moves

In Windows, a window can receive WM_MOUSEMOVE events even if the mouse coordinates don't change, "For example, if the cursor is positioned over a window, and the user hides the window, the window receives WM_MOUSEMOVE messages even if the mouse did not move."[1]. Similarly, WM_MOUSEMOVE events are received when a mouse button is released. This results in unexpected InputEventMouseMotion events being generated: see #20357.
This PR checks to see if the mouse actually moved before adding an InputEventMouseMotion event. Since this was fixed for WM_INPUT for when mouse_mode is MOUSE_MODE_CAPTURED in #22368, this PR applies 33dd2c8de to the the WM_MOUSEMOVE and WM_POINTERUPDATE Windows events as well as the Wacom WT_PACKET event.
Fixes #20357.

We reviewed this in a PR-review meeting. The issue @bruvzg mentioned needs to be addressed. We guessed we could maybe track those changes using windows[window_id].last_pressure or windows[window_id].last_tilt, but it needs to be tested.


Bug fix/http2 statistics

Scope & Purpose
Fixes the request statistics for HTTP2. As mentioned in #11523, for these requests no statistics were gathered at all.

[x] Bug-Fix for devel-branch (i.e. no need for backports?)
[x] The behavior change can be verified via automatic tests

AFAIK http2 support is new, so I suppose this fixes does not have to be backported?
Testing & Verification
This change is already covered by existing tests, such as api-statistics-spec-grey.
Perhaps this test can be ungreyed now that statistics for all request types should work correctly?

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/10015/

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/10077/

Starting new Jenkins run after pulling in current devel:
http://<IP_ADDRESS>:8080/view/PR/job/arangodb-matrix-pr/10105/

Build failed on MSVC with narrowing warning:
  H2CommTask.cpp
C:\vm02-windows\oskar\work\ArangoDB\arangod\GeneralServer\H2CommTask.cpp(521,42): error C2220: the following warning is treated as an error [C:\vm02-windows\oskar\work\ArangoDB\build\arangod\arangoserver.vcxproj]
C:\vm02-windows\oskar\work\ArangoDB\arangod\GeneralServer\H2CommTask.cpp(501): message : while compiling class template member function 'void arangodb::rest::H2CommTask<arangodb::rest::SocketType::Tcp>::sendResponse(std::unique_ptr<arangodb::GeneralResponse,std::default_delete<arangodb::GeneralResponse>>,arangodb::RequestStatistics::Item)' [C:\vm02-windows\oskar\work\ArangoDB\build\arangod\arangoserver.vcxproj]
C:\vm02-windows\oskar\work\ArangoDB\arangod\GeneralServer\H2CommTask.cpp(804): message : see reference to class template instantiation 'arangodb::rest::H2CommTask<arangodb::rest::SocketType::Tcp>' being compiled [C:\vm02-windows\oskar\work\ArangoDB\build\arangod\arangoserver.vcxproj]
C:\vm02-windows\oskar\work\ArangoDB\arangod\GeneralServer\H2CommTask.cpp(521,42): warning C4244: 'argument': conversion from 'uint64_t' to 'int32_t', possible loss of data [C:\vm02-windows\oskar\work\ArangoDB\build\arangod\arangoserver.vcxproj]
  C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.26.28801\bin\HostX64\x64\clcache.py Finished. Exit code 2

Restarting test:
http://<IP_ADDRESS>:8080/view/PR/job/arangodb-matrix-pr/10108/

Thanks @mpoeter !

IMHO the fix itself feels a bit hackish - it is not immediately clear where the current statistics item resides (in the CommTask map, in the handler, in the stream?), but it was the simplest solution with the fewest code changes.
A cleaner solution would require more extensive design changes - something I might take on in the future.

@mpoeter : you are right, but the fix already helped a lot and is definitely a big improvement compared to what there was before.
Much appreciated!


Add the ability to remove a vertex by double-clicking a point while in edit or draw mode.

Tested.Reintegrate

I'm not able to push the fix because my computer is in quarantine.

merged into the dev branch. closing this issue.


caseInsensitive maybe more descriptive than expectsLowerCase

Thanks, but case-insensitive is clear enough.
https://en.wikipedia.org/wiki/Case_sensitivity

@fnlctrl As case-insensitive is more clear, why not use caseInsensitive instead?

Oops.. Misread changes/original, sorry.
expectsLowerCase is also clear enough, and since it's just source code (not documentation), there's no practical to change it.

Understand, but I am reading the source code now 😄


Copy downstream metadata to upstream

https://github.com/envoyproxy/envoy/issues/20536
Add a new dynamic metadata namespace "envoy.common.down_up",  you can add your own dynamic metadata into this namespace if you want to propagate your downstream dynamic metadata to the upstream stream info.

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

@zhangbo1882  Please also add unit tests

/retest

@kyessenov could you help review? Thanks.

@zhangbo1882 please avoid force-pushing
@kyessenov is this PR ready for a senior-maintainers round?

@adisuissa @zhangbo1882 Sorry I was on and off the last few days. I think that we still need to resolve sharing of metadata between different downstreams for the same upstream, but we could benefit from senior maintainer on how to design that (e.g. should we use a hash again or just aggregate in a bag like this PR proposes).

/wait-any
cc @mattklein123 re design question above

in progress


Expose DynEnergyInOther for containers to Prometheus

What would you like to be added?
Kepler exposes idle power of containers of host components other than CPU and DRAM (container.IdleEnergyInOther) to Prometheus, but it doesn't do so dynamic power (container.DynEnergyInOther) . It should be exposed as well.
https://github.com/sustainable-computing-io/kepler/blob/f94a6651894f875f556f7494934055dc681ad8ce/pkg/collector/prometheus_collector.go#L765-L770
Why is this needed?
Both idle and dynamic power attributed to containers are part of power of a container. To visualize container power correctly in dashboard, it is required that both of them are exposed.

@tiwatsuka can you create a PR for this?

closed by #815


build.sh: Add TypeScript bindings to the tgz file.

This makes the tgz consistent with the npm package. Fixes #3268.
This also removes a stale comment in the script (we stopped packaging the old HTML docs a while back).
Going forward, we would like to use the TS declarations as documentation because they must be kept up to date anyway, and they already contain type information. This was discussed in #3250.

Thank you for your hard work!


How to do a search on the server pages?

How can I implement a search if I have a splitting into server pages (server-side pagination)? At the moment, information search is performed on the page that is active, and the remaining pages are simply ignored by the filter. Is there any documentation or example?

Currently the filter only seachs an "in memory" array, if you need server side pagination you need another considerations, like the request URL and other things that are out of scope of this project.
I currently use this project with server side pagination, but I check if the result array has less than my page and fetch more rows, this works fine for a few rows, but it's not scalable.


Search local platforms

QrAndBarcodeScanner is a well implemented app, but, to be useful to everyone, it needs to be able to search local platforms; ebay.co.uk and amazon.co.uk, here in the UK.

Hello. Thank you for your feedback. I guess it will be better to implement the option to add arbitrary URLs for searching barcodes, so users can add and use their own favorite websites. Hopefully I will implement this in one of the future releases.


Disable keyboard (Left, Right, Down & Up) arrow keys in MatCalendar

Hi Team,
Current Behavior:
If i use any of the arrow kyes (Left, Right, Top & Down) the selection moves in the calendar.
Expected Behavior:
I want to disable the keyboard events for calendar.

We don't have any plans to add a feature like this.


Fix #2539

What's this PR do/fix?
Issue #2539
Are there unit tests? If not how should this be manually tested?
No unit tests, but I've tested with the issue demo repo.
What are the relevant issues?
Maybe #2274 , #1881 , #2294

@draco1023 thanks for the PR! However the existing tests don't seem to be passing. Perhaps add a test of your own as well and describe what your problem was and how this fixes the problem

@draco1023 🎗

@dilipkrish Sorry, I don't have much time recently.
My PR always goes deep when Map and its subtype is met, but the schema perfers $ref, which caused ci tests failed.

@draco1023 not a problem. In general, even if the aren't any new tests, its should minimally pass the ci build to get pulled in.
Also there is no UI that will support this as the spec itself doesnt support this scenario.

@dilipkrish The figures above show the main differences between swagger2/declaration-bugs-service.json (left side) and the result using the PR (right side).
Since ApplicationLang and Language both implement the Map interface, and in my opinion, there is no reason to distinguish between Map's (or Collection's) implemention class and interface.

@draco1023 thats fine, but couple of things

Regardless, the test need to pass, so arguably you could make the change to the contract files but I dont believe this is the right fix.
What if you had nested one level deeper? It doesnt seem correct that you'd inline all the models. The way that it renders currently, it preserves the types for e.g. Language.


SEO broken link: Update choose-aspnet-framework.md

@tdykstra 	remind me of the meta-data we're supposed to add.

@tdykstra 	aren't we supposed to add
#metadatax: seohack2018
or something close to that? So they can track the results?
I'll find it.


ViewState.lookAt returns InvalidUpVector when camera is looking directly down

Describe the bug
When trying to setup the view to look straight down (the equivalent of StardardViewId.Top),  ViewStatus.InvalidUpVector is returned.
Looking at the lookAt implementation, its due to the upVector being the same as targetPoint to eyePoint vector.
(Side note - this scenario seems problematic not only for lookAt - e.g. in the Camera Visualizer sample, the camera plane fails to be constructed for the same scenario)
Scenario ilustrated using Camera Visualizer sample:

To Reproduce
Example of lookAt parameters used:
view3d.lookAt({
    eyePoint: [
      20.40488933620268,
      3.509489688064754,
      48.1222892966759
    ],
    targetPoint: [
      20.40488933620268,
      3.509489688064754,
      2.1124999999999687
    ],
    upVector: [
      0,
      0,
      1
    ],
    lensAngle: 90
  });

In the example above, the parameters were gathered after applying StandardViewId.Top:
eyePoint = view.getEyePoint();
upVector = view.getUpVector(eyePoint);
lensAngle = view.getLensAngle();
targetPoint = view.getTargetPoint();

Expected behavior
ViewState is sucessfully updated.
Additional context
I’m not sure if I can safely assume that upVector is the same as targetPoint to eyePoint vector only in this specific scenario, and thus its safe to just set StandardViewId.Top.
Note that the same issue appears if the view matches StandardViewId.Bottom.
What is the proper way to setup a “looking down” or "looking up" view from LookAtUsingLensAngle args?

getUpVector (undocumented) gives you the up vector in world space at a given position. Within the project extents, this is unit Z.
LookAtArgs.upVector (as documented) is the camera's up (y) vector in view space.


Disable pylint error.

Merging to remove 1 CI error.  Will continue to look into others:

shotgun is still flaky.
hist dask is flaky.
Somehow a new error on osx with dask shows up with index error.  Maybe an upgraded numpy or dask has different behaviours on different platforms.
github action osx failure.  Seems to be internal error in new libomp.


Add warning- valid special characters in the service provider name

We have restricted special characters in the SP name. After this fix we are allowing only ". - _ and space". This restriction is there is SOAP service and DCR Rest service as well.
Hence added a warning in the following docs:
[(https://docs.wso2.com/display/IS570/Adding+and+Configuring+a+Service+Provider)]
[(https://docs.wso2.com/display/IS560/Adding+and+Configuring+a+Service+Provider)]
[(https://docs.wso2.com/display/IS550/Adding+and+Configuring+a+Service+Provider)]
[(https://docs.wso2.com/display/IS540/Adding+and+Configuring+a+Service+Provider)]
[(https://docs.wso2.com/display/IS530/Adding+and+Configuring+a+Service+Provider)]
[(https://docs.wso2.com/display/IS570/Using+the+Service+Provider+API)]
[(https://docs.wso2.com/display/IS560/Using+the+Service+Provider+API)]
[(https://docs.wso2.com/display/IS550/Using+the+Service+Provider+API)]
[(https://docs.wso2.com/display/IS540/Using+the+Service+Provider+API)]
[(https://docs.wso2.com/display/IS530/Using+the+Service+Provider+API)]

Closing this issues since the changes have been made already.


HHH-18679 throw exception when columnValues is null while addKeyColumn

Currently, the code will throw NPE when annotating the id with @Generated(writable = true) WITHOUT sql attribute
This fix is about throwing a dedicated exception instead of NPE as we can not handle any further.

Note: I'm a new contributor, so my code might be bad. Please give me advices/suggestions if the PR is so stupid, I will do my best to fix it. Thank you so much

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license
and can be relicensed under the terms of the LGPL v2.1 license in the future at the maintainers' discretion.
For more information on licensing, please check here.

https://hibernate.atlassian.net/browse/HHH-18679

Hey @anaconda874, thank you so much for opening the PR and trying to solve this!
The NPE thrown by Hibernate of course is an issue, and while your idea of validating this case and generating a better error message is definitely on the right track (though I would not create a new exception and just use IdentifierGenerationException), I think we have the tools to actually allow @Generated(writable = true) on an identifier property to work.
I was looking at your issue as well and I've created https://github.com/hibernate/hibernate-orm/pull/9078, with this change we should be able to understand when a generated id is writable and read the value from the persisted entity instead of letting the database generate one.

Don't worry @anaconda875, your contribution is still very much appreciated - and let's see what the others think.
Note: I used @ColumnDefault in the test just for a quick way to have a database-generated value, realistically one would have some sort of trigger / database function generating the identifier value on the fly.

Note: I used @ColumnDefault in the test just for a quick way to have a database-generated value, realistically one would have some sort of trigger / database function generating the identifier value on the fly.

Hi @mbladel , may I ask a question? If there are "some sort of trigger / database function generating the identifier", then why we need writable = true?
The document of sql says that Determines if the value currently assigned to the annotated property is included in SQL insert and update statements

If there are "some sort of trigger / database function generating the identifier", then why we need writable = true

Well, as you said (and the documentation you posted is about the writable flag) the user might choose to persist an entity with a value that was already assigned to the property annotated with @Generated. The same can be true for identifiers, mainly useful for testing where you might want to persist some data with a known id value to deterministically produce a desired outcome.

Closing this as superseded by https://github.com/hibernate/hibernate-orm/pull/9078.
Once again, thank you @anaconda875 for the effort, your PR looked great for a first attempt. We'd be more than happy to review more of your work going forward, and don't be afraid to reach out to us if you need any help.


Improve T2C documentation

specification to become a part of the documentation
specific features for Ergoaktiv columns

Isn't this done @tomasmacek ?

Not yet


Checkout.js disable AMEX cards in payment method

How to disable, like example, AMEX card in payment methods?

You can set:
funding: {
    disallowed: [ paypal.CARD.AMEX ]
}

bluepnume Also I try few times on LinkedIn and Facebook contact you. You, for mistake, got one payment, can you please make it charge back?

Good morning,
all fine, customer get money back. Thanks a lot.

Среда, 18 июля 2018, 1:14 +02:00 от Daniel Brain<EMAIL_ADDRESS>I've refunded the payment -- can you please check?
—
You are receiving this because you authored the thread.
Reply to this email directly,  view it on GitHub , or  mute the thread .

--
Mihails Atamanskii


The engine "node" is incompatible with this module

cypress-testing-library version: 2.3.3
node version: 6.13.0
npm (or yarn) version: 1.10.1

Relevant code or config
see below
What you did:
$ npm install --save-dev cypress-testing-library
$ yarn install

What happened:
After running yarn install when cypress-testing-library was installed, I got the following message:
...
error<EMAIL_ADDRESS>The engine "node" is incompatible with this module. Expected version "> 6". Got "6.13.0"
error Found incompatible module
info Visit https://yarnpkg.com/en/docs/cli/install for documentation about this command.
...

Reproduction repository:
working on it
Problem description:
see above
Suggested solution:
The problem can be circumvented by running yarn install --ignore-engines but it seems like the solution would be to change the value of engines.node in package.json from "> 6" to ">6" since that's the format used in the relevant npm documentation.

Dang - my theory about the space was incorrect. scours yarn github issues to point blame somewhere

I must be tired because I'm just realizing this: >6 is exclusive of 6.*, not just greater than 6.0. Carry on...


Load cross origin files

Allow for niivue to load textures and nii files from remote sources

@cdrake, for nii files, this is already possible (loadFromURL), but the server you're loading from must have a wildcard CORS policy (something like a public AWS S3 bucket). I guess textures would work the same way. What sort of textures would we load remotely?

@hanayik - Anything that might be shared from a different site, whether it
was a font or a graphic we wanted to include in the view.
On Tue, Dec 7, 2021 at 2:18 PM Taylor Hanayik @.***>
wrote:

@cdrake https://github.com/cdrake, for nii files, this is already
possible (loadFromURL), but the server you're loading from must have a
wildcard CORS policy (something like a public AWS S3 bucket). I guess
textures would work the same way. What sort of textures would we load
remotely?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/niivue/niivue/issues/82#issuecomment-988200449, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHVPFL3NPHE5UJ6BQ7JLILUPZMXFANCNFSM5DEQLYEA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

--
Regards,
Chris


tests: driver: flash_default limited on tier one boards

remove mcux test scenario, which is overlapped with default limit test to boards listed in the boards list
fixing: #62067

@nashif this is because it is this case that make the flash crashed that blocks the issue in #62067, I should rephase the issue, and resubmit a PR for this


Use Lime for ranking scenario

Hello, I'm using Lime with xgboost for ranking with over 3k features. Does Lime work in Learning to Rank field? Or should I reconstruct it as a classification problem?

I haven't really thought about what explanations for ranking would look like, but if you can set it up as a classification problem it should work : )


ytboob.com

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm reporting a new site support request
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that none of provided URLs violate any copyrights or contain any DRM to the best of my knowledge
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[ ] I've read about sharing account credentials and am willing to share it if required

Region
No response
Example URLs
-Single video - https://ytboob.com/full-nude-3/
-Playlist-ish links

https://ytboob.com/?filter=latest
https://ytboob.com/?filter=most-viewed
https://ytboob.com/?filter=longest
https://ytboob.com/?filter=popular
https://ytboob.com/page/2/
https://ytboob.com/page/3/

Provide a description that is worded well enough to be understood
mp4 is hosted on vidhost.me, and is easily curled.
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp-nightly-builds [65cf46cdd] (win_exe)
[debug] Python 3.10.11 (CPython AMD64 64bit) - Windows-7-6.1.7601-SP1 (OpenSSL 1.1.1t  7 Feb 2023)
[debug] exe versions: ffmpeg 7.0-full_build-www.gyan.dev (setts), ffprobe 7.0-full_build-www.gyan.dev
[debug] Optional libraries: Cryptodome-3.21.0, brotli-1.1.0, certifi-2024.12.14, curl_cffi-0.5.10, mutagen-1.47.0, requests-2.32.3, sqlite3-3.40.1, urllib3-2.3.0, websockets-14.1
[debug] Request Handlers: urllib, requests, websockets, curl_cffi
[debug] Extractor Plugins: AGB (YoutubeIE), Youtube_AgeGateBypassIE
[debug] Loaded 1838 extractors
[generic] Extracting URL: https://ytboob.com/full-nude-3/
[generic] full-nude-3: Downloading webpage
WARNING: [generic] Falling back on generic information extractor
[generic] full-nude-3: Extracting information
[debug] Looking for embeds
ERROR: Unsupported URL: https://ytboob.com/full-nude-3/
Traceback (most recent call last):
  File "yt_dlp\YoutubeDL.py", line 1634, in wrapper
  File "yt_dlp\YoutubeDL.py", line 1769, in __extract_info
  File "yt_dlp\extractor\common.py", line 742, in extract
  File "yt_dlp\extractor\generic.py", line 2553, in _real_extract
yt_dlp.utils.UnsupportedError: Unsupported URL: https://ytboob.com/full-nude-3/

The HTML5 video in the single video page is found by the upstream Generic extractor but not by the re-factored HTML5 extractor here.

is this site just reposted content from youtube/other video sharing sites?

@bashonly Yes. It is intended as a "mirror" of NSFW videos that are either removed from YT, or almost certainly going to be removed from YT due to the obvious ToS violation. See r/bannedyoutube , r/ytnsfw and some other subreddits to find the u/ytboob bot that's automating this.


[Sema] Context validation checks for access-level on imports

Notify of a possible unintentional public imports by raising errors on default imports when other imports of the same target module have an explicit access level. This diagnostic won't be necessary when (or if) we consider default imports to be internal in Swift 6.
Raise an error on public import of a non-resilient module from a resilient module, and silence it for any access level below public. Keep this diagnostic as a warning for @_implementationOnly import.
Raise an error on public import of a private module from a public module, and suggest a fixit inserting the internal keyword before the import.
Report conflicting use of @_exported and non-public imports.

This follows the parser changes in #63912. Exportability checks and module interface support will be added next.

@swift-ci Please smoke test

@swift-ci Please smoke test

@swift-ci Please smoke test

@swift-ci Please smoke test


Key management

Encryption solutions for data-at-rest will be protecting data for potentially very long periods of time. In many ways, the key management system may well be the single most important component of your storage security solution. As we’ve discussed, you will likely need to maintain keys for many years. You need assurance that the keys used to encrypt data will be available whenever and wherever authorized access to data is required. At the same time, the keys need to be secured so that they themselves aren’t compromised (resulting in a data breach).
Data at Rest

Rule - Build support for changing algorithms and keys when needed
If keys are compromised or an external authority expires them, key changes will be needed. Application polices or emergency needs will force application administrators to rotate keys and potentially rekey data at some point. It's best to be prepared to rapidly handle this need when necessary. Including a key version and encryption algorithm version with the encrypted data is a useful, proactive feature. For instance, including a simple prefix string, such as "{1,1}...", prior to the encrypted data could indicate algorithm version 1, key version 1. This allows for an "online" change to the encryption algorithm and key without re-encrypting all existing data all at once.


Order Meals By Importance

Meals that change day to day should be at the top of the food list.
Commonly start "Hot Traditional ..."
Entrees should be above sides.

Closed with ea941c1


Updated ouroboros-consensus-diffusion to ouroboros-network-<IP_ADDRESS>

Closes #974

Esgen is going to add something to the Bootstrap Peers IER.

Rendered Bootstrap Peers IER

@coot wrote on Slack

I would write a sentence that in the pre-syncing state the node will wait for Honest Availability Assumption. And maybe expand what it actually means for chain-sync clients.

FTR: We want to merge this in order to unblock the release train for Node 8.11; we can merge improvements to the docs in a follow-up PR.


Add explicitly a message in order to process for fonctional test

Hello, thank you for your amazing lib.
Is there a way to explicitly add a message in a queue in order to process this message ?
In my case, an other service push a message in messenger. My main app receive and process the message. I would like to realize a functional test (no access to the first app, no shared rabbitmq in ci). I am no way to populate the queue without special hack for test.
for exemple, i would like to do this with phpunit:
$this->messenger()->queue()->addMessage(new MyMessage('42'));
$this->messenger()->process();
//check in db|file|other the process was successfully executed

Do you think it's the right way or completely dumby ?
Thank you for your help !
[EDIT]
It's work with
$transport = self::$container->get('messenger.transport.my_message');
$transport->send(new Envelope($new MyMessage(42), [new AmqpStamp("my_message")]));
$this->messenger('my_message')->queue()->assertContains(MyMessage::class, 1);
$this->messenger('my_message')->process();

I was going to suggest what you did in your edit. TestTransport::send() is marked as internal - maybe I should remove that?

Thank you,  but i don't know all about the impact of this change... you are the expert !

No impact really, I just marked everything that I didn't expect to be used by end-users as @internal.

@fpondepeyre, what do you think of #14?


Adding new unique id to userId$ stream does not set new user profile

Expected behavior
I may be misunderstanding the expected behavior, but my understanding is that with the setup
  final _user$ = StreamController<String>.broadcast();

    _mixpanel = MixpanelAnalytics(
...
      userId$: _user$.stream,
...
    );
...

any time I call
    _user$.add(newUser);

The stream should trigger an event to provide a new user profile that uses this new user id.
The use case for this is an app where a user can have multiple accounts, and each account needs a distinct analytics profile.
Actual behavior
In practice, however, it does not- I set up the initial user profile (eg. _user$.add("aaaaaaa"); and mixpanel correctly tracks events with this profile. When I add a new user ID (eg. _user$.add("bbbbbbb"); Mixpanel still reports that the events belong to profile aaaaaaa. This happens both when using single and batch MixpanelAnalytics.
I think a workaround to this problem is to dispose and re-initialize my _mixpanel variable each time the user profile changes. Please let me know if this is how the plugin was designed, or if this is indeed a bug. Thanks!

Second this!

Hi @NateWilliams2, thanks for reporting this. The plugin should behave as you are expecting. We'll review this and come back to you asap.

Ok, I see why you're getting this behavior. I guess your adding the value to the stream just before sending the event. Take a look at the StreamController docs. As you can see there:

If sync is false, the event will always be fired at a later time, after the code adding the event has completed.

So, either you set the sync argument to true, or you listen to the stream and just add the event once the change it's done.
Let me know if that solves your question and feel free to close the issue if it does.
Take into account that we have just released vesion 1.4.0 which adds a new setter called userId. This is not a stream so you won't be having this sync issues.
//cc @Oriol-GG fyi

Thanks for the explanation!
I'll try out one or both of these solutions and report back if I'm still having issues.


[C#] Update to C#9

Add latest to csprojs to pick up C#9
Remove "is {}" in favor of "is not null"
Use "Type x = new()", which is particularly useful for field definitions

Fixed Linux builds


Revise ALG2-congenital disorder of glycosylation

Mondo term (ID and Label)
MONDO:0011933 ALG2-congenital disorder of glycosylation
MONDO:0014543 congenital myasthenic syndrome 14
Suggested revision and reasons
After giving a lumping and splitting presentation on ALG2 to the Congenital Disorders of Glycosylation (CDG) GCEP on Nov. 1st, 2023, it was decided amongst the expert panel to lump congenital myasthenic syndrome 14 and ALG2-congenital disorder of glycosylation together. We are requesting that congenital myasthenic syndrome 14 be added as a child term under ALG2-congenital disorder of glycosylation.
Your nano-attribution (ORCID)
0009-0001-2576-9655
https://orcid.org/0009-0001-2576-9655

Thanks @gleon98, we'll look into this as soon as we can. Thanks for the ticket.

Hi @gleon98 - sorry for the delay. When I added this parent to congenital myasthenic syndrome 14, it caused an error in Mondo - this new parentage makes congenital myasthenic syndrome 14 a type of 'metabolic disease', which is defined as:
A congenital disorder (due to inherited enzyme abnormality) or acquired (due to failure of a metabolically important organ) disorder resulting from an abnormal metabolic process.
I wanted to confirm that 'congenital myasthenic syndrome 14' is a type of metabolic disorder?
Thanks!

No worries thank you for the update, I was waiting to speak with the expert panel yesterday about this, and they agreed to this change. It would be considered a metabolic disorder!

thanks @gleon98!


Change components to LIT

dont know if these will actually be visible via editor tooltips, but for checking whats possible we need a jsdoc for these

there are a few things still todo:

Confirmationstatus name
X Icon
Onclick on select component

looks fine:
the select component should either be done with slots or like the filter container


Make devmode paths base themselves on BAY_HOME, not the pwd

I fixed the pep8 failure in master, the build should be green if you rebase.


Fixed Session Service Provider registry

The service provider was registering the wrong parameters
to the session listener instead of the actual session class
This fixes #1342
Signed-off-by: RJ Garcia<EMAIL_ADDRESS>

@fabpot. Those are updated.
out of curiosity with doing conditionals on one line, how would you format a bunch of conditions so that the line isn't so long.
$long_var_name == 1 || $long_var_name_b == 1 || ($long_var_name_2 == 2 || $long_var_name3 == 2) && $one_more_long_var_name ==5

Everyone has a large screen nowadays, so there is no need to cut lines at 80 chars.

Thank you @ragboyjr.


Instagram

What is this PR trying to achieve?
Adding an Instagram post type.
What does it look like?

No JS:

For interface components

[x] Browser testing - Have you checked the component in our supported browsers
[x] No javascript - Have you checked the component with javascript disabled
[x] A11y testing - Have you run pa11y against the component and fixed/verified all errors, warnings and notices? (iFrame title from Instagram empty – outwith our control).

Have the following been considered/are they needed?

[x] Tests?

This is getting very close to components being fully fledged pieces of content that can import their own styles, DOM and scripts... Let's make sure we think about this if it comes up again.

You have conflicts