Couldn't find pattern: ^diff --git \zs.+\ze

as soon as I run this
git diff | vim - -R +Diffurcate

I get this error
Error detected while processing command line..function diffurcate#Run:
line   11:
Couldn't find pattern: ^diff --git \zs.\+\ze
Press ENTER or type command to continue

Could you post the output of git diff in that particular project, so I can try to figure out why the pattern doesn't match?

@AndrewRadev I get same error:

Here is output of git diff:

@azamat-sharapov Could you post the full git output as text or attach it as a file, so I can run it locally? I could retype it from the image, but the error might be somewhere down below, or it might be some hidden characters, so ideally git diff > output.diff and uploading that here would be perfect.

@AndrewRadev here it is output.diff.zip (github doesn't allow .diff file upload, so I compressed it).

I see, this works fine for me, but I think I understand why it might not work now -- I think you might have set nomagic somewhere in your config. Could you try it out now?

@AndrewRadev didn't work, unfortunately..
Error detected while processing function diffurcate#Run:
line   11:
Couldn't find pattern: \m^diff --git \zs.\+\ze
Press ENTER or type command to continue

Here is my .vimrc:

.vimrc
set nocompatible   " Disable vi-compatibility
set hidden

" Swap files out of the project root
set backupdir=~/.vim/backup//
set directory=~/.vim/swap//

syntax on
filetype plugin indent on

" Change <leader>
let mapleader = ","
let g:mapleader = ","

call plug#begin()

" color scheme
Plug 'morhetz/gruvbox'

Plug 'Shougo/context_filetype.vim'

" fuzzy search
Plug '/usr/local/opt/fzf'
Plug 'junegunn/fzf.vim'

" better in-buffer search
Plug 'junegunn/vim-slash'

" file tree
Plug 'scrooloose/nerdtree', { 'on': 'NERDTreeFind' }
Plug 'PhilRunninger/nerdtree-visual-selection'

" Plug 'jiangmiao/auto-pairs'
" Plug 'Raimondi/delimitMate'
" Plug 'michaeljsmith/vim-indent-object'
Plug 'urxvtcd/vim-indent-object'
Plug 'cohama/lexima.vim'
Plug 'tpope/vim-repeat'
Plug 'tpope/vim-surround'
Plug 'tpope/vim-eunuch'
Plug 'tpope/vim-unimpaired'
Plug 'tpope/vim-abolish'
Plug 'vim-scripts/Tabmerge'
Plug 'AndrewRadev/splitjoin.vim'
Plug 'chaoren/vim-wordmotion'
Plug 'svermeulen/vim-cutlass'
Plug 'vim-scripts/delview'
Plug 'vim-scripts/dbext.vim'
Plug 'ekalinin/Dockerfile.vim'
Plug 'AndrewRadev/diffurcate.vim'

" comment plugin
Plug 'tyru/caw.vim'

" multi cursor support
Plug 'mg979/vim-visual-multi', {'branch': 'master'}

" git plugin
Plug 'tpope/vim-fugitive'

" inserts end keyword for ruby
Plug 'tpope/vim-endwise'

" HTML/CSS plugin
Plug 'mattn/emmet-vim'

" ruby syntax highlighting, indentation, omnicompletion, etc.
Plug 'vim-ruby/vim-ruby'

" rails power tools
Plug 'tpope/vim-rails'

" lint engine
Plug 'dense-analysis/ale'

Plug 'neoclide/coc.nvim', {'branch': 'release'}
Plug 'exu/pgsql.vim'
Plug 'editorconfig/editorconfig-vim'
Plug 'othree/html5.vim'
Plug 'jvirtanen/vim-hcl'
Plug 'hashivim/vim-terraform'
Plug 'fatih/vim-go', { 'do': ':GoInstallBinaries' }
Plug 'posva/vim-vue'
Plug 'pangloss/vim-javascript'
Plug 'leafgarland/typescript-vim'

" Better JSON
Plug 'elzr/vim-json'

" github AI pairprogrammer
Plug 'github/copilot.vim'

" Plug 'joereynolds/vim-minisnip'

call plug#end()

" UI
set t_Co=256
set background=dark
let &t_8f = "\<Esc>[38;2;%lu;%lu;%lum"
let &t_8b = "\<Esc>[48;2;%lu;%lu;%lum"
set termguicolors

let g:gruvbox_contrast_dark='hard'
let g:gruvbox_hls_cursor='red'
let g:gruvbox_invert_selection=0
" let g:gruvbox_italic=0
colorscheme gruvbox
highlight Search cterm=underline
highlight SpecialKey ctermfg=grey guifg=grey25

set nocursorline
set showmode                    " always show what mode we're currently editing in
set showcmd                     " Show (partial) command in the status line
set nowrap                      " don't wrap lines
set tabstop=2                   " a tab is two spaces
set smarttab
set tags=tags
set softtabstop=0               " when hitting <BS>, pretend like a tab is removed, even if spaces
set shiftwidth=3                " number of spaces to use for autoindenting
set shiftround                  " use multiple of shiftwidth when indenting with '<' and '>'
set backspace=indent,eol,start  " allow backspacing over everything in insert mode
set autoindent                  " always set autoindenting on
set copyindent                  " copy the previous indentation on autoindenting
set number                      " always show line numbers
set incsearch                   " realtime search
set timeout timeoutlen=400 ttimeoutlen=0
set novisualbell                " don't beep
set noerrorbells                " don't beep
set autowrite                   " Save on buffer switch
set history=200
set nojoinspaces                " no spaces on line join
set foldmethod=indent
set foldlevel=99
set scrolloff=5                 " some padding on working line
set backupcopy=yes              " otherwise webpack --watch won't work
set listchars=tab:>-,trail:~,extends:>,precedes:<,space:·
set nolist
set pastetoggle=<leader>I
set ignorecase
set smartcase " insensetive case when given text is all lowercase
set diffopt+=internal,algorithm:patience
set updatetime=300

" fix ctrl-up/down
map <ESC>[1;5A <C-Up>
map <ESC>[1;5B <C-Down>
map! <ESC>[1;5A <C-Up>
map! <ESC>[1;5B <C-Down>

" Fast saves
nmap <leader>w :w<cr>

" Fast quit
nmap <leader>q :q<cr>

" Easy escaping to normal model
imap jj <esc>
imap jk <esc>

" Easier window navigation
nmap <C-h> <C-w>h
nmap <C-j> <C-w>j
nmap <C-k> <C-w>k
nmap <C-l> <C-w>l

" Resize vsplit
nmap <C-v> :vertical resize +5<cr>
nmap 25 :vertical resize 40<cr>
nmap 50 <C-w>=
nmap 75 :vertical resize 120<cr>

" Create split below
nmap :sp :rightbelow sp<cr>

" When I forget sudo
cmap w!! w !sudo tee % >/dev/null

" autocmd cursorhold * set nohlsearch
" autocmd cursormoved * set hlsearch
" Make double-<Esc> clear search highlights
" nnoremap <silent> <Esc><Esc> <Esc>:nohlsearch<CR><Esc>

" Paste and indent
nnoremap <leader>p p=`]

" Quickly edit/reload the vimrc file
nmap <silent> <Leader>ev :e $MYVIMRC<CR>
nmap <silent> <Leader>sv :so $MYVIMRC<CR>

" Go to tab by number
noremap <leader>1 1gt
noremap <leader>2 2gt
noremap <leader>3 3gt
noremap <leader>4 4gt
noremap <leader>5 5gt
noremap <leader>6 6gt
noremap <leader>7 7gt
noremap <leader>8 8gt
noremap <leader>9 9gt
noremap <leader>0 :tablast<cr>

" Alternate file
nmap <Leader>6 <C-^>

" vim-cutlass
nnoremap x d
noremap x d
nnoremap xx dd
nnoremap X D

set statusline+=%#warningmsg#
set statusline+=%*
set statusline+=%F

" fix alt+n mapping in Vim CLI
if !has('gui_running')
  exec "map \en <A-n>"
endif

" Auto save and load folds
augroup AutoSaveFolds
	autocmd!
	" view files are about 500 bytes
	" bufleave but not bufwinleave captures closing 2nd tab
	" nested is needed by bufwrite* (if triggered via other autocmd)
	autocmd BufWinLeave,BufLeave,BufWritePost ?* nested silent! mkview!
	autocmd BufWinEnter ?* silent! loadview
augroup end

" nerdtree toggle
let g:NERDTreeWinPos = 'right'
let g:NERDTreeWinSize=50
let g:NERDTreeMinimalMenu=1
map <Leader>t :NERDTreeFind<CR>

" fzf
nmap <Leader>b :Buffers<CR>
nnoremap <silent> <expr> <Leader><Leader> (expand('%') =~ 'NERD_tree' ? "\<c-w>\<c-w>" : '').":GFiles --cached --others --exclude-standard\<cr>"
nnoremap <silent> <Leader>ag :Ag <C-R><C-W><CR>
xnoremap <silent> <Leader>ag y:Ag <C-R>"<CR>
nnoremap <silent> <Leader>` :Marks<CR>

" query, ag options, fzf#run options, fullscreen
autocmd VimEnter *
\ command! -bang -nargs=* Ag
\ call fzf#vim#ag(<q-args>, '', { 'options': ['--bind', 'ctrl-a:select-all,ctrl-d:deselect-all'] }, <bang>0)

" minisnip
" let g:minisnip_trigger = '<C-j>'

" auto-pairs
" let g:AutoPairsFlyMode = 0

" coc
" inoremap <expr> <cr> pumvisible() ? "\<C-y>" : "\<C-g>u\<CR>"
" inoremap <silent><expr> <cr> pumvisible() ? coc#_select_confirm() : "\<C-g>u\<CR>"
" inoremap <silent><expr> <cr> pumvisible() ? coc#_select_confirm() : "\<C-g>u\<CR>\<c-r>=coc#on_enter()\<CR>"
" inoremap <silent><expr> <NUL> coc#refresh()
inoremap <expr> <Tab> pumvisible() ? "\<C-n>" : "\<Tab>"
" inoremap <expr> <S-Tab> pumvisible() ? "\<C-p>" : "\<S-Tab>"

" Use tab for trigger completion with characters ahead and navigate.
" inoremap <expr> <Tab> coc#pum#visible() ? coc#pum#next(1) : CheckBackspace() ? "\<Tab>" : coc#refresh()
inoremap <expr> <S-TAB> coc#pum#visible() ? coc#pum#prev(1) : "\<C-h>"

" Make <CR> to accept selected completion item or notify coc.nvim to format
" <C-g>u breaks current undo, please make your own choice.
" inoremap <silent><expr> <C-Space> coc#pum#visible() ? coc#pum#confirm() : "\<C-g>u\<CR>\<c-r>=coc#on_enter()\<CR>"

nmap <silent> gd <Plug>(coc-definition)
nmap <silent> gy <Plug>(coc-type-definition)
nmap <silent> gi <Plug>(coc-implementation)
nmap <silent> gr <Plug>(coc-references)

function! CheckBackspace() abort
	 let col = col('.') - 1
	 return !col || getline('.')[col - 1]  =~# '\s'
endfunction

" Github Copilot
imap <silent><script><expr> <C-J> copilot#Accept("\<CR>")
let g:copilot_no_tab_map = v:true

"\'coc-tsserver',
let g:coc_global_extensions = [
	\'@yaegassy/coc-volar',
	\'coc-tailwind-intellisense',
	\'coc-go',
\]

" == ale
let g:ale_fix_on_save = 1
let g:ale_disable_lsp = 1
let g:ale_javascript_eslint_executable = 'eslint_d'
let g:ale_javascript_eslint_use_global = 1
let g:ale_fixers = ['prettier']
let g:ale_linters_explicit = 1

" == vim-wordmotion
let g:wordmotion_spaces = '_-.:'
let g:wordmotion_prefix = ''
"let g:wordmotion_mappings = {
"\ 'w' : '<C-w>',
"\ 'b' : '<C-b>',
"\ 'e' : '<C-e>',
"\ 'ge' : 'g<C-e>',
"\ 'aw' : 'a<C-w>',
"\ 'iw' : 'i<C-w>',
"\ }

" == pgsql syntax
let g:sql_type_default = 'pgsql'

" vim-json
let g:vim_json_warnings = 0
let g:vim_json_syntax_conceal = 0

runtime macros/matchit.vim

Hm, this is odd. I tried to set up your vimfiles locally, but I'm not getting that error (though I did have to remove CoC, it wasn't installing properly locally and it kept erroring out).

A problem, but not that same error
Seems to work after that error

There's definitely some problem, which I think has to do with the a/, b/ prefixes. Locally, when I git diff, I don't really get those. I wonder if it's a configuration of some kind. Do you know under what circumstances they show up?
I'll experiment some more in the coming days, see what I can figure out.

I've pushed another attempt at a fix. Could you try it out now, let me know if there's any difference?

Still no difference, unfortunately. My git version is 2.30.1 (Apple Git-130) and here is .gitconfig just in case:

.gitconfig
[color]
  ui = true
[core]
  filemode = false
  editor = vim
[user]
  email =<EMAIL_ADDRESS>  name = "Azamat"
[alias]
  st = status -sb
  co = checkout
  ll = log --oneline --all --decorate --graph
[push]
  default = simple
[diff]
[merge "ours"]
  driver = true
[init]
  defaultBranch = main

Anything else I can provide?

Let's try this step by step. First, I'd like you to try directly searching for that pattern by pressing / to start a search and pasting \m^diff --git \zs.\+\ze . Note that it ends in a space. If that doesn't work, we can try removing some parts of it, although it would be really weird if it doesn't.
If it does work, I'd like you to move to the start of the file with gg0 and then use : to enter command-line mode and paste echo search('\m^diff --git \zs.\+\ze ', 'Wc') and let me know what number it prints.

When searching, I got E486: Pattern not found: \m^diff --git \zs.\+\ze .
For the second command, I got 0.

Let's try a few one by one, see which ones hit. It's strange to me that they don't work, since I used your config as well:

\mdiff --git \zs.\+ze
\m^diff --git .\+
diff --git

Now that I think about it, I didn't ask which Vim/Neovim version you're using? I could try compiling that version in particular, maybe something subtle happened with the regexes that I didn't know about.


With Postgres connection mode set to "Simple", can't add or delete columns from table

Description
When you create a Postgresql connection, and set its Connection view to "Simple", DBeaver doesn't let you add or remove columns in the table view.
DBeaver Version
Community Edition 23.2.0
Operating System
Ubuntu
Database and driver
PostgreSQL 12.14
PostgreSQL JDBC Driver 42.5.2

Steps to reproduce
No response
Additional context
No response

Hello @DeflateAwning
Yes, that's expected behavior for the "Simple View". Other object creation also is not available via UI.
You can choose a custom or advanced navigator view or change your tables via SQL Editor.

Is there a way to show only tables (or only tables and views) in the sidebar without decreasing the capabilities of DBeaver overall?

Unfortunately, there is no way to do it like this.
Please use custom mode or advanced to change metadata via UI.


GUI

What is the GUI library used in Bluesky. If i want to contribute to Bluesky as a programmer what are the prequisites that I shold have in Python.

Hi @anandatc,
BlueSky uses pyQt5 (soon also pyQt6) for windows and widgets, and OpenGL >=3.3 to draw the radar screen.


Fields for schema

See also https://github.com/opengeospatial/ogcapi-features/issues/838
https://schemas.opengis.net/ogcapi/tiles/part1/1.0/openapi/schemas/tms/propertiesSchema.yaml

uom (same as uom in https://github.com/opengeospatial/ogcapi-features/issues/838 ) ?
uomURI (same as uom in https://github.com/opengeospatial/ogcapi-features/issues/838 ) ?
uomSymbol ?
observedProperty
observedPropertyURI (same as definition in https://github.com/opengeospatial/ogcapi-features/issues/838 )

For "axes" in the range (from  https://github.com/opengeospatial/ogcapi-features/issues/838)

referenceFrame
localFrame
axisId

Coverages-specific

fieldIndex (integer starting at 0)
spatialResolution (when differing per field e.g., panchromatic band, scene classification layer, see #142)

Encoding info

nilValues / nodata (from  https://github.com/opengeospatial/ogcapi-features/issues/838)
significantBits, scale, offset, data type (encoding specifc or not... see https://github.com/opengeospatial/coverage-implementation-schema/issues/1#issuecomment-742654021)

Statistics

Min, Max (already covered by JSON Schema?), Average, StdDev...

Do we need a distinct ogc-x field extension for categories, or can this achieved with a JSON schema construct? (e.g., having an integer enum, and associated desription?)

For units of measures, Features - Part 5 already specifies x-ogc-unit which defaults to UCUM and x-ogc-unit-lang (to allow for alternatives such as QUDT).
We could raise the issue that this is more a vocabulary than a "language"? x-ogc-unit-vocab?
Should x-ogc-unit be x-ogc-uom?
Does this address the needs of uom, uomURI, and uomSymbol identified above?
Do we need additional fields for semantic lookup vs. for display to the user?
We should clarify how QUDT should be identified? A full URI , or only the identifier?
Topic for the sprint in a couple week!

In Features - Part 5, it was clarified that UCUM for example really is a language rather than a vocabulary, since you can combine different codes together to create new units, and therefore language is the more general term. A note was added there to clarify that.
The decision to use unit rather than uom is because unit is readily understandable by the uninitiated.
So x-ogc-unit and x-ogc-unit-lang should address all of the uom, uomURI and uomSymbol mentioned above.
There is also a new x-ogc-definition which was added to the generic Schemas in Features - Part 5 (currently requirement 8).

A  - The keyword "x-ogc-definition" SHALL be used to identify the semantic definition for the property.
B - The value of the keyword "x-ogc-definition" SHALL be a URI.

so this addresses the semantic aspect (observedProperty and observedPropertyURI).
Regarding statistics, while the standard JSON Schema properties might define the minimum/maximum values, it would not necessarily indicate the statistics of the current dataset. It might therefore make sense to define custom fields here for these.
referenceFrame, localFrame, axisId might be more relevant to Connected Systems, but if these are defined there they could also be used if relevant for Coverages.
fieldIndex has been defined in Features - Part 5 as x-ogc-property-seq.
In addition to making sure the draft reflects all these decisions so far (x-ogc-unit, x-ogc-unit-lang, x-ogc-definition, x-ogc-property-seq), this would leave the following to define:

x-ogc-statistics : { min, max, average, stddev }
x-ogc-encodingInfo : { nodata, significantBits, scale, offset, dataType }
x-ogc-spatialResolution (when differing per field e.g., panchromatic band, scene classification layer)

I recommend to reopen this issue.
In the current draft of Features part 5 I see the adoption of x-ogc-definition and x-ogc-unit but I do not see x-ogc-nilValues or equivalent to specify nodata values, what is essencial for grid coverages. I was not able to spot anything like this in the current draft of ogc api coverage.
Can we do something about it?

@joanma747 We could, but we need to discuss it first! :) That's why you should make sure to join the meeting tomorrow 10:00 AM Eastern ;)

"title": "No data"

Is there a possibility to link to external lists ?
Can we have a URI for each "classification" value?

Look at "observation characteristics" properties that comes from OMS for other "things" to add. https://umltool.ogc.org/index.php?m=7&o=CCBF8590-E16B-4e08-B49A-8804CBF4B7A6

SWG 2024-12-04:

There is on-going discussion about how to add extra information about enum values, with 2 potential approach discussed in this Features - Part 5: Schemas PR: https://github.com/opengeospatial/ogcapi-features/pull/953#discussion_r1790243723.
We will discuss in Common setting up a wiki page listing the x-ogc-* fields (see also https://github.com/opengeospatial/ogcapi-features/issues/945)
Regarding nodata, we had proposed x-ogc-nilValues that could support multiple null values -- also consider the possibility to use NaN as discussed in Testbed 20-GIMI
x-ogc-statistics should be defined
Regarding encoding info e.g., significantBits, scale, offset, dataType, the challenge is this information might not be part of the logical schema but specific to a particular format
We should consider x-ogc-spatialResolution in relation to issue #142

@jerstlouis On the OMS ObservationCharacteristics, this element can be applied at all levels, e.g. individual Observation, Timeseries, Collection of Observations.
In the Coverage context, it could be used to provide additional metadata on the individual fields. I recently did an example in the context of the TSML update where I extended a CovJSON Timeseries with OMS concepts in the parameters

@KathiSchleidt @joanma747 So which are the extended fields that we should define?
Tomorrow in Common we're playing to set up a wiki page to keep track of them (see https://github.com/opengeospatial/faster-directions/issues/1#issuecomment-2436510127 regarding how difficult this has become).
From your example we would have:

x-ogc-definition -- This has already be defined and I believe would correspond to observedProperty
x-ogc-unit and x-ogc-unitLang -- This has already been defined as corresponding to unit
x-ogc-observingProcedure
x-ogc-observer
x-ogc-featureOfInterest
x-ogc-host -- Could we somehow qualify or rename this -- host has a rather specific meaning on the web?

Would other things be relevant?

@jerstlouis for OMS:host, how about we change this to sosa:Platform?
SOSA is the semantic twin of OMS under W3C, currently we're updating SOSA based on the recent OMS update.


[Action, Green Reviews WG] Define access control management process for Green Reviews infra

Description

Now that Green Reviews WG has got access to the dedicated infrastructure on Equinix we need to establish a process to give access to this infrastructure to those who need it.  Once the process it outlined we can put it in writing.
For now we can have a dedicated section for this in the CNCF TAG ENV – WG Green Reviews Design Document

Who should get access and how?
For how long?
How to ask for access in a transparent and lightweight manner?
Clean up access when not needed?
Other concerns?

Input

Need input on the questions provided above so that we can define the process in writing.
Outcome

Formally defined process for access control management and requesting access to the Green Reviews WG infrastructure.

@RobertKielty can this be part of CLOwarden? or do we need to manage infra access via equinix console / api "manually"

CLOWarden is used to govern access to just GitHub repos at the moment.
I like the way you're thinking though! Long term, the vision is to create plugins to have CLOWarden control access to multiple services. We're not there yet though.

Hey @Callisto13 (who is currently doing awesome stuff at Equinix)!
We got access to an Equinix project. Our immediate next step is to start provisioning a Kubernetes cluster using Cluster API. We're looking at potentially using the Equinix Metal Cluster API provider (CAPEM). The configuration will live in this tooling repo: https://github.com/cncf-tags/green-reviews-tooling
No further questions at the moment but wanted to add you to this thread and ask in advance in case you have any pro tips!

👋 hey @nikimanoledaki
I'm afraid I don't have any hot tips for CAPEM, but it looks about the same as any other CAPI provider 😄 .
As for Auth to your hardware, I am guessing y'all have a project under the CNCF org? That would give you 2 access options:

Full console access, where you invite people to join the org as a "limited collaborator" with access to that project only.
Programmatic access, where you create a "bot" limited collaborator user with access to that project, then issue project level API keys for users.

Neither of these have expiry options, so you'd have to manage when to revoke access. Or write some code around adding and automatically removing people, which should be fairly trivial.
(A third way would be to create the machines you know you want to use and have a nice automated way of adding people's SSH keys, that way they would not need an Equinix account. This is assuming you either keep public IPs on the devices or have a VPN or tunnel etc. This would limit them to the servers only and gives you slightly more granular access for people who don't need any more than that.)
Lmk if you have any hardware questions 😀

Thank you @Callisto13 for outlining our auth options! 🎉 The 2 WG co-chairs (@guidemetothemoon and I) as well as 2 TAG leads (@mkorbi & @leonardpahlke) were added by the CNCF folks to the dedicated project for the TAG Environmental Sustainability. So 4 of us have (1) full console access for now - unsure if/how we would extend access. Perhaps we could create new roles to extend access to more individual contributors. For everything else, we will most likely go with (2) project-level API keys added to the repository and try to use IaaS tooling as much as possible.
With regard to the hardware, the first CNCF Project that we will test is Falco, which is a security- and eBPF-based tooling. We will be using Kepler, which is also eBPF-based. We have a few kernel-level requirements. We're starting with the Falco requirements outlined here. If you have any pointers for getting started on which hardware to provision and how to do so given these requirements, we would be very grateful! The state of our infrastructure decisions is outlined here - we're still very much in the planning phase.

Do we really need to grant other access to Equinix setup? Perhaps read access for some logs. If we define the deployment infra in the repo (with OpenTofu or similar 😉) and deploy via CI/CD - anyone can propose config changes, we just need to review and approve the PRs.

Update - here is the process that we are using at the moment:

WG Chairs created a Project API Key in the Equinix Project.

Project-level API keys can be added to a project to allow access to the Equinix Metal API that is not tied to a particular user. Project API keys do not have access to the entirety of the API; some endpoints can only be used by personal API tokens.

WG Chairs privately share the Project ID (from project settings) as well as the API key token, which has read/write access, with individual contributors.

This has helped to unblock individual contributors with manual testing. We will update this once we have consensus on the IaC component and start implementing it.

Update
This issue is mostly completed and documented here.

Clean up access when not needed?

We should revoke access when offboarding a TAG lead - not sure if offboarding is documented since the TAG is still relatively new and we haven't gone through that process yet but just something to keep in mind! cc TAG Chairs @leonardpahlke @catblade @mkorbi
Adding the API key as a secret to the green-reviews-tooling repository is tracked in this POC: https://github.com/cncf-tags/green-reviews-tooling/pull/6

Closing this issue since we're tracking specific access-related issues in the WG repo https://github.com/cncf-tags/green-reviews-tooling


Persona stats

Beginning of looking at stats and how users can be defined by personas

Super helpful! Here are some thoughts:

Computational with  any combo of Project management / Project leadership / Project management (12 of these)
Project management and/or admin (7)
any combo of Project management / Project leadership / Project management
Comp only ("analyst") (13)
resultsTidy %>% filter(KindOfWork == "Computational work")
Clinical (2)
resultsTidy %>% filter(str_detect(KindOfWork, "Clinical"))
Any education (6)
resultsTidy %>% filter(str_detect(KindOfWork, "education"))

⚠ broken url errors ⚠ There are broken url errors that need to be addressed. Click here ➡ for broken url errors! ❗ Add errors that aren't errors to the resources/ignore-urls.txt file of this repo. If you are having troubles see this guide Comment updated at 2024-08-21-15:50:35 with changes from e08d0c1

this is weird - the link works for me 🧐

⚠ broken url errors ⚠ There are broken url errors that need to be addressed. Click here ➡ for broken url errors! ❗ Add errors that aren't errors to the resources/ignore-urls.txt file of this repo. If you are having troubles see this guide Comment updated at 2024-08-21-15:50:35 with changes from e08d0c1

this is weird - the link works for me 🧐

The link worked for me too! @cansavvy pinging you on this, but maybe it's a known OTTR behavior?

⚠ broken url errors ⚠ There are broken url errors that need to be addressed. Click here ➡ for broken url errors! ❗ Add errors that aren't errors to the resources/ignore-urls.txt file of this repo. If you are having troubles see this guide Comment updated at 2024-08-21-15:50:35 with changes from e08d0c1

this is weird - the link works for me 🧐

The link worked for me too! @cansavvy pinging you on this, but maybe it's a known OTTR behavior?

Which link are we talking about?

https://www.ukbiobank.ac.uk/enable-your-research/about-our-data
Since it's an outside link, I don't realy want to add it to the ignore file

https://www.ukbiobank.ac.uk/enable-your-research/about-our-data
Since it's an outside link, I don't really want to add it to the ignore file

Sometimes servers are down temporarily or somebody sneezed or something and if you retry it will work. If not, then I think adding it to the ignore file is fine. ¯_(ツ)_/¯
It is not related to any known OTTR bugs as far as I can tell

That all makes sense -- thank you for checking and explaining!

@kweav I went ahead and ran styler to format the code. Let's try to use styler or something similar for future additions to keep things as readable as possible. Once installed, you can run the styler through Add-ins, so it should be very quick!

I think this one is ready @kweav !


example data

Hi,
Thanks for the great work.
Have you provided the data (example data) ?
I want to look into .txt files!
currently, data folder is empty.

Please refer to our data preparation instruction. https://github.com/yangchris11/samurai?tab=readme-ov-file#data-preparation

Yes, I see this.
I mean the content of each file . a folder which includes these different files then I can open each e.g., nlp.txt and look into its content.


Move pkg overrides into separate flakes

[x] Move into separate flakes

Originally posted by @christianharke in https://github.com/christianharke/nixcfg/pull/1#r797960147

Done


Add subject device detail

I agree, also, (in 301) additional data of the patient could be added to the tile in the future, so data there is not limited to device status.

Cool, there's one detail though this panel opens on click and not on hover, since it opens a panel and we have to load the individual sensors data, so to go to the details is a 2-click process. Pressing once on the row to open the detail and then on the "OPEN SUBJECT DATA" button.

What if the panel is open by default? Then we could show the patient information on hover and the patient view on click?

Because we don't get all the data from the API, both subject details and the sensors data, it will be awkward to show a constant loading as the user moves the mouse up and down the table.
Click/press/tap is more intentional and permanent, there's less of an immediate feedback expectation from users and the data persists on screen until the user clicks again.

Didnt think of that. Okay, any arguments against the 2 click approach? I like @mpgxvii idea to use the panel as a means to display other data in the future, also.

@kerzmaximilian the argument against is that it takes longer for a researcher to go to the details, but this can be run through researchers when it's implemented to validate, it's easy enough to change at a later time.
Also really like @mpgxvii idea, which adds an argument for having the 2 clicks, since there's more information that needs "processing" by researchers! 👍

@mpgxvii @kerzmaximilian made the changes, please review.


Create delete canonical_url rake task

This is used to manually refresh the canonical url if the journal url changes.


Update dependencies, refactor code, improve testing

Adds more complete testing and better validation of content in the yaml configuration files

Many iterative fixes to bump versions to the latest, fix build/ci problems with those changes.


Improved: Slim down use of ${groovy: } (OFBIZ-13146)

Improved: Slim down use of ${groovy: } (OFBIZ-13146)
With the framework evolution, we used on many place the scriptlet ${groovy: } on screen rendering where something it's now unecessary.
This issue convert some of them on a other solution to slim down this usage

Hi Nicolas,
Thanks for the good work (tested on Flat Grey for "---------- Last system notes ----------")
Should the file framework/common/src/main/groovy/org/apache/ofbiz/common/GetUiLables.groovy
not named
framework/common/src/main/groovy/org/apache/ofbiz/common/GetUiLabels.groovy ;)

Thanks Jacques I'll have a look to many change on many subject :D


[New Feature Thread] Yearly Wrap Up per User Details

This is a documentation thread
Per user at the end of the year, we could do what Spotify started with a total count for the year per user. We could do something similar with trees, by having a series of slides that state

how many trees you planted
C02 calculations
type of arborist
more

Can i solve this issue?
Please assign this issue to me. so i will contribute in this issue.


KAFKA-16059: Fix thread leak KafkaAPIsTest

KafkaApisTest creates instance of KafkaApis.scala which created reaper threads for purgatory but we never call close and hence, these threads are leaked.
This PR closes the instance of KafkaApis in @AfterEach.
This leak is similar to the one detected in https://github.com/apache/kafka/pull/15084

The test changed in this PR is successful - https://ci-builds.apache.org/job/Kafka/job/kafka-pr/job/PR-15093/3/testReport/kafka.server/ControllerApisTest/

Thanks folks for fixing this stuff :)


shopware store version vs. composer installation

If the plugin is installed via composer with the shopware-store url
composer require store.shopware.com/adyenpaymentshopware6

then the plugin is listed 2 times in the plugin:list command
Shopware Plugin Service
=======================

 -------------------------------------- ------------------------------------------ --------------------------------------------------------- --------- ----------------- ----------------------- ----------- -------- ------------- ----------------------
  Plugin                                 Label                                      Composer name                                             Version   Upgrade version   Author                  Installed   Active   Upgradeable   Required by composer
 -------------------------------------- ------------------------------------------ --------------------------------------------------------- --------- ----------------- ----------------------- ----------- -------- ------------- ----------------------
  AdyenPaymentShopware6                  Adyen Payment                              store.shopware.com/adyenpaymentshopware6                  4.1.3                       Adyen                   Yes         Yes      No            Yes
...
 AdyenPaymentShopware6                                                                                4.1.3                                               No          No                     Yes

Seems that this is not causing any problems, but looks wrong...
I think the problem is, that the composer version contains a vendor directory with a composer/installed.php file and composer loads this too
Versions
Shopware version: <IP_ADDRESS>
Plugin version: 4.1.3
To Reproduce
Install the plugin via composer
Expected behavior
plugin:list shows the adyen plugin only once

Hello @0x4,
Thank you for reaching out to us regarding this issue,
Please note that we attempted to reproduce the reported issue in a local environment but we were unsuccessful.

After executing bin/console plugin:list command, the plugin is listed only once, as evidenced by the screenshot above. We tried with PHP8.2 and 8.3 but the result was the same.
Could it be possible that you previously installed the Adyen plugin via zip, uninstalled it, and then installed the Adyen plugin with composer?
Could you try to reproduce the issue on your end again and let us know if you can do so?
Thank you,
Kind regards.

Hello @0x4,
Thank you for the feedback.
Note that our team was able to reproduce the issue after running plugin:refresh command after the installation.
Our team will proceed with fixing the issue.
We will let you know as soon as we have some valid information to share.
Kind regards.

Hello @0x4,
Please note that our team is still working on this issue.
As soon as we have some valid information to share, we will let you know.
Thank you,
Kind regards.


fix(ci): move job to new runner

What does this PR do?
Move this job to new k8s runners

Motivation
Maintenance

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/trigger-ci --variable RUN_ALL_BUILDS=true --variable RUN_KITCHEN_TESTS=true --variable RUN_E2E_TESTS=on --variable DEPLOY_AGENT=true --variable RUN_UNIT_TESTS=on

/merge


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 59 ms

Resolved: ClamAV performance has improved in 503e915.


DEFAULT_PER_PAGE is not work

In gitlab api the max per page has a limit value, so if the item size larger than the value, the api with DEFAULT_PER_PAGE can not fetch the all items except fetch items with many times.

  For projects it would look something like this:
// Get a Pager instance that will page through the projects with 20 projects per page
Pager<Project> projectPager = gitlabApi.getProjectsApi().getProjectsPager(20);
List<Project> allProjects = new ArrayList<>(projectPager.getTotalItems());

// Iterate through the pages and append each page of projects to the list
while (projectsPager.hasNext())
    allProjects.addAll(projectsPager.next());

@ccl0326
Did the above answer your question?

@gmessner yep, it can answer my question.


Handshapes: search signs by handshape features

Expand present search form for phonology by adding the handshape fields

Querying is working now. I'll get the fields in the results adjusted to show all fields.
I have the page so that you can select Search handshape, or "pull-down" to Search sign.
(I didn't do Search sign yet.)
So we want to search for signs that use the (found) handshape(s)?
I made this search form be for Handshapes (it's basically  the List View of Handshapes, with Query functionality.)
So then I need to also adjust the Search sign query to include fields for Handshapes. (Yes, that's what this issue says.)
I guess multiple types can't be in the same query result list. So searching for signs as part of the page shown above probably doesn't make sense. Oh wait, I guess it would be a list of signs instead.
I am wondering if the user would want to search of "part of" the Finger selection?
(So say for Thumb and Index. Rather than TI.)
[Does that make sense to be able to search for a subset of Selected Fingers? That was that Boolean for Each Finger modelling of this field. I can convert back and forth to the choice list.]

Yes, we want both to search for handshapes, and to search for signs by handshape features.
Now we're at it, it would be good to think about a way to make this search more flexible than to select one out of all values for a field. In case the number of values is limited (say, 1-15), something with checkboxes to select one or multiple values could be good. Alternatively, we could use regular expressions (as in the gloss searches), and have help popups listing the values. Could you discuss this with @vanlummelhuizen ?

Don't think I understand your question about searching for "part of", could you explain?

I have a field Number of selected fingers (Quantity) because one of the papers I looked at to see how they modelled handshapes had used Quantity. I changed it say quantity to fit better in the results list. I'll delete this if it's not needed.

There is no Dutch translation for Handshape since it's either Strong or Weak hand.
The Handshape column (see screenshot) is what it is in the language being used.

Hmmm. I just noticed that it's either Strong hand or Weak hand having a particular Handshape (as the Query facility is currently implemented for Signs.) So we probably want it to be either? (Either Strong hand or Weak hand matches Handshapes with the requested fields.)

I think we'd want to be flexible here: by default searches should search for either hand, but a checkmark or something should enable "Only strong hand" and "Only weak hand". I know I know, I keep making up requirements as you go :-)
As for the "Number of selected fingers", I actually quite like adding that! It's not strictly speaking a phonological concept, but it will come in handy for all kinds of questions I expect. Thanks!
The Dutch term for "Handshape" is "Handvorm" (Sterke hand, zwakke hand); but we'll have to do another run of translations soon anyway, so please add {% trans %} tags as much as possible.

I had to zoom out so it would fit. In this example, I made it so the finger selection can be per finger.
I'm making it so that when editing a handshape, you can edit each fingers of the Finger selection.
But when displaying the handshape, it uses the abbreviated forms TIMRP, TIM, etc.
I had to add the choice Either, so it can match selected and not selected. (Choosing Eitther removes the finger selection for that finger from the query so it will match anything for that finger. To get partial matches.)

That looks quite doable, thanks for that, but you were also talking about searching for an exact number of fingers, no? Like in, "one SF but it doesn't matter which". That might come in handy, and it would be fine if we'd need to specify this number explicitly in a field, there's only so many handshapes to describe.

Yes! The Quantity would be useful for that. (It can be computed from the finger selection if it's not set.)
I'm also working on querying the signs based on the handshape fields.

oh right! I missed it, sorry

@susanodd @ocrasborn This is live now! #345 #346 #347

https://signbank.science.ru.nl/dictionary/handshapes/

@susanodd , could you set the default number of search hits that are displayed to 50 rather than 5?

Great to see this live now, btw!

Fixed. (Pushed to github master branch.)


Bump VHD to 2023.02.01

What type of PR is this?

What this PR does / why we need it:
Bumps the VHD version (for weekly Agentbaker release)
Which issue(s) this PR fixes:

Fixes #
Requirements:

[ ] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

GPU E2E fails due to quota limitation - skipping it here


feat(video): add component

PR Checklist

[x] Fix linting errors
[x] Label has been added

Change information
Added new video component

We need some changes here:

Create a branch under the repo of fork, not main repo. (I'll close this PR, you can create new PR from fork repo.)
Video component require at least one testacse.
Can we quote a free, open source, with out copyright video in the example? It's better to put the video in the public folder. (same origin)
video.tsx looks too large, can we try to abstract it?
Video component seems to need to forward ref, users may need to control the DOM manualy.


Epub conversion adds furigana as letters, not markups

I've tried both add and override functionalities combined with horizontal and vertical writting and i always get the furigana written after the kanji with same size letters instead of markups. See the image below:

I'm interested in this project, i will try to check the code if i get some free time.

I think this depends on the reader application.
The generate EPUB file contains markups for the furigana, which are correctly rendered with Calibre's viewer for example.
On Kindle however, it would not render the markup, so I would always convert to AZW3 or MOBI instead. (From what I read, it is not fully EPUB-compatible but does a conversion to their format under the hood)
Which reader are you using?

I was using https://epub-reader.online/ and kindle desktop reader app. The epub comes from a kindle azw3 drm removed file and converted to epub. I will try with calibre viewer and converting to other ebook formats after furiganising and see how it goes.

I have tried everything and the only variant that works is the html output. With all other variants, on add furigana and any of the 3 sentence alignment methods, i get horizontal with markups like this:

This is using mobi on calibre, but same thing for epub or any other.

I've found the error. The problem was that the epub files i was trying to convert are 2.0, and also, the ones that the furiganalyser outputs is 2.0 and that version does not support ruby tags. Converting the file to 3.0 with sigil worked for me, not it displays the furigana in any reader. There is an extension called ebooklib that you can use to make sure the header contains something like:
<package xmlns="http://www.idpf.org/2007/opf" unique-identifier="isbn" version="3.0" xml:lang="ja" ...
so it is epub3.

Great, thank you for diving into the issue!
So one way to fix this would be have furiganalyse upgrade the uploaded epub from 2.0 to 3.0.
I will give it a try.


Use bat as a library to remove system dependency

Let's use bat as a library and just always use it instead of falling back to cat if the user doesn't have bat installed.
https://docs.rs/bat/latest/bat/

Looks like it doesn't have a way to render to string. It can only print it to STDOUT.


db v3

Hello @patrickcleeve2! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file fibsem/db/v3/app.py:

Line 7:80: E501 line too long (86 > 79 characters)
Line 13:80: E501 line too long (90 > 79 characters)
Line 17:1: E303 too many blank lines (3)
Line 39:1: E303 too many blank lines (9)
Line 41:80: E501 line too long (95 > 79 characters)
Line 43:1: E302 expected 2 blank lines, found 1
Line 48:1: E305 expected 2 blank lines after class or function definition, found 1
Line 62:9: E303 too many blank lines (2)
Line 63:80: E501 line too long (85 > 79 characters)
Line 63:86: W291 trailing whitespace

In the file fibsem/db/v3/models.py:

Line 5:1: E303 too many blank lines (3)
Line 21:80: E501 line too long (89 > 79 characters)
Line 32:1: W293 blank line contains whitespace
Line 41:80: E501 line too long (89 > 79 characters)
Line 56:1: E303 too many blank lines (4)
Line 61:80: E501 line too long (89 > 79 characters)
Line 81:80: E501 line too long (89 > 79 characters)
Line 100:80: E501 line too long (89 > 79 characters)
Line 120:80: E501 line too long (89 > 79 characters)
Line 136:1: E302 expected 2 blank lines, found 1
Line 172:74: W292 no newline at end of file

In the file fibsem/db/v3/util.py:

Line 4:1: E302 expected 2 blank lines, found 1
Line 4:80: E501 line too long (82 > 79 characters)
Line 10:1: E302 expected 2 blank lines, found 1
Line 28:1: W391 blank line at end of file


How to auto-focus first window of current display when a floating window is closed?

Dear yabai developer and enthusiasts,
The following line in the configuration file seems not working..
yabai -m signal -add event=window_destroyed action="yabai -m window --focus first"

Thank you in advance!

yabai -m window --focus first will attempt to focus the asp-managed window on screen. If you simply want the most recently used window on screen, use this signal:
yabai -m signal --add event=window_destroyed active=yes action="yabai -m window --focus \$(yabai -m query --windows --space | jq '.[0].id')"

@koekeishiya Thank you! I just realized I was missing a hyphen in the --add option. Your suggested command is also very helpful!