lvim: Cannot find package "clangd"

LunarVim can't find a package named "clangd". This is what I get upon running lvim test.c:
Installation in progress for [clangd]
Error detected while processing BufReadPost Autocommands for "*":
Error executing lua callback: ...la/.tea/neovim.io/v0.9.1/share/nvim/runtime/filetype.lua:21: Error executing lua: ...la/.tea/neovim.io/v0.9.1/share/nvim/runtime/filetype.lua:22: BufReadPost Autocommands for "*"..FileType Autocommands for "*"..function <SNR>1_LoadFTPlugin[19]..script /Users/dracula/.tea/lunarvim.org/v1.3.0/share/lunarvim/site/after/ftplugin/c.lua: Vim(runtime):E5113: Error while calling lua chunk: ...ite/pack/lazy/opt/mason.nvim/lua/mason-registry/init.lua:79: Cannot find package "clangd".
stack traceback:
        [C]: in function 'error'
        ...ite/pack/lazy/opt/mason.nvim/lua/mason-registry/init.lua:79: in function 'get_package'
        ....org/v1.3.0/share/lunarvim/lvim/lua/lvim/lsp/manager.lua:121: in function 'setup'
        ...rvim.org/v1.3.0/share/lunarvim/site/after/ftplugin/c.lua:1: in main chunk
        [C]: in function 'nvim_cmd'
        ...la/.tea/neovim.io/v0.9.1/share/nvim/runtime/filetype.lua:22: in function <...la/.tea/neovim.io/v0.9.1/share/nvim/runtime/filetype.lua:21>
        [C]: in function 'nvim_buf_call'
        ...la/.tea/neovim.io/v0.9.1/share/nvim/runtime/filetype.lua:21: in function <...la/.tea/neovim.io/v0.9.1/share/nvim/runtime/filetype.lua:10>
stack traceback:
        [C]: in function 'nvim_cmd'
        ...la/.tea/neovim.io/v0.9.1/share/nvim/runtime/filetype.lua:22: in function <...la/.tea/neovim.io/v0.9.1/share/nvim/runtime/filetype.lua:21>
        [C]: in function 'nvim_buf_call'
        ...la/.tea/neovim.io/v0.9.1/share/nvim/runtime/filetype.lua:21: in function <...la/.tea/neovim.io/v0.9.1/share/nvim/runtime/filetype.lua:10>
stack traceback:
        [C]: in function 'nvim_buf_call'
        ...la/.tea/neovim.io/v0.9.1/share/nvim/runtime/filetype.lua:21: in function <...la/.tea/neovim.io/v0.9.1/share/nvim/runtime/filetype.lua:10>
Press ENTER or type command to continue

I'm not sure what the error above means because I do have clangd:
% clangd --version
Apple clangd version 14.0.3 (clang-14<IP_ADDRESS>.1)
Features: mac+xpc
Platform: x86_64-apple-darwin22.5.0

I can proceed but without syntax highlighting, even though nvim test.c works fine. Both were automatically installed with tea.
 % lvim --version
NVIM v0.9.1
Build type: RelWithDebInfo
LuaJIT 2.1.0-beta3
Compilation: /opt/tea.xyz/gx/cc/v0.1.4/bin/cc -O2 -g -Og -g -Wall -Wextra -pedantic -Wno-unused-parameter -Wstrict-prototypes -std=gnu99 -Wshadow -Wconversion -Wvla -Wdouble-promotion -Wmissing-noreturn -Wmissing-format-attribute -Wmissing-prototypes -Wimplicit-fallthrough -fdiagnostics-color=auto -fstack-protector-strong -DUNIT_TESTING -DINCLUDE_GENERATED_DECLARATIONS -I/var/folders/8b/n_gmhmk57ks85swq3mrdbrwc0000gp/T/968688f7/.deps/usr/include/luajit-2.1 -I/opt/gnu.org/gettext/v0.21.1/include -I/var/folders/8b/n_gmhmk57ks85swq3mrdbrwc0000gp/T/968688f7/.deps/usr/include -I/var/folders/8b/n_gmhmk57ks85swq3mrdbrwc0000gp/T/968688f7/build/src/nvim/auto -I/var/folders/8b/n_gmhmk57ks85swq3mrdbrwc0000gp/T/968688f7/build/include -I/var/folders/8b/n_gmhmk57ks85swq3mrdbrwc0000gp/T/968688f7/build/cmake.config -I/var/folders/8b/n_gmhmk57ks85swq3mrdbrwc0000gp/T/968688f7/src -I/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX12.3.sdk/usr/include -I/var/folders/8b/n_gmhmk57ks85swq3mrdbrwc0000gp/T/968688f7/.deps/usr/include -I/var/folders/8b/n_gmhmk57ks85swq3mrdbrwc0000gp/T/968688f7/.deps/usr/include -I/var/folders/8b/n_gmhmk57ks85swq3mrdbrwc0000gp/T/968688f7/.deps/usr/include -I/var/folders/8b/n_gmhmk57ks85swq3mrdbrwc0000gp/T/968688f7/.deps/usr/include -I/var/folders/8b/n_gmhmk57ks85swq3mrdbrwc0000gp/T/968688f7/.deps/usr/include -I/var/folders/8b/n_gmhmk57ks85swq3mrdbrwc0000gp/T/968688f7/.deps/usr/include

   system vimrc file: "$VIM/sysinit.vim"
  fall-back for $VIM: "/opt/neovim.io/v0.9.1/share/nvim"

Run :checkhealth for more info

Running the installation script in https://www.lunarvim.org/docs/installation somehow made the lvim test.c error go away.
% which lvim
/Users/dracula/.tea/.local/bin/lvim
% realpath $(which lvim)
/Users/dracula/.tea/tea.xyz/v0.36.0/bin/tea
% lvim --version
NVIM v0.9.1
Build type: RelWithDebInfo
LuaJIT 2.1.0-beta3
Compilation: /opt/tea.xyz/gx/cc/v0.1.4/bin/cc -O2 -g -Og -g -Wall -Wextra -pedantic -Wno-unused-parameter -Wstrict-prototypes -std=gnu99 -Wshadow -Wconversion -Wvla -Wdouble-promotion -Wmissing-noreturn -Wmissing-format-attribute -Wmissing-prototypes -Wimplicit-fallthrough -fdiagnostics-color=auto -fstack-protector-strong -DUNIT_TESTING -DINCLUDE_GENERATED_DECLARATIONS -I/var/folders/8b/n_gmhmk57ks85swq3mrdbrwc0000gp/T/968688f7/.deps/usr/include/luajit-2.1 -I/opt/gnu.org/gettext/v0.21.1/include -I/var/folders/8b/n_gmhmk57ks85swq3mrdbrwc0000gp/T/968688f7/.deps/usr/include -I/var/folders/8b/n_gmhmk57ks85swq3mrdbrwc0000gp/T/968688f7/build/src/nvim/auto -I/var/folders/8b/n_gmhmk57ks85swq3mrdbrwc0000gp/T/968688f7/build/include -I/var/folders/8b/n_gmhmk57ks85swq3mrdbrwc0000gp/T/968688f7/build/cmake.config -I/var/folders/8b/n_gmhmk57ks85swq3mrdbrwc0000gp/T/968688f7/src -I/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX12.3.sdk/usr/include -I/var/folders/8b/n_gmhmk57ks85swq3mrdbrwc0000gp/T/968688f7/.deps/usr/include -I/var/folders/8b/n_gmhmk57ks85swq3mrdbrwc0000gp/T/968688f7/.deps/usr/include -I/var/folders/8b/n_gmhmk57ks85swq3mrdbrwc0000gp/T/968688f7/.deps/usr/include -I/var/folders/8b/n_gmhmk57ks85swq3mrdbrwc0000gp/T/968688f7/.deps/usr/include -I/var/folders/8b/n_gmhmk57ks85swq3mrdbrwc0000gp/T/968688f7/.deps/usr/include -I/var/folders/8b/n_gmhmk57ks85swq3mrdbrwc0000gp/T/968688f7/.deps/usr/include

   system vimrc file: "$VIM/sysinit.vim"
  fall-back for $VIM: "/opt/neovim.io/v0.9.1/share/nvim"

Run :checkhealth for more info

I get the same error on ublue-os/silverblue-nvidia:38 (based on Fedora Linux 38):
[zeus@fedora ~]$ tea lvim hello.c

Installation in progress for [clangd]
Error detected while processing BufNewFile Autocommands for "*":
Error executing lua callback: ...us/.tea/neovim.io/v0.9.1/share/nvim/runtime/filetype.lua:21: Error executing lua: ...us/.tea/neovim.io/v0.9.1/share/nvim/runtime/filetype.lua:22: BufNewFile Autocommands for "*"..FileType Autocommands for "*"..function <SNR>1_LoadFTPlugin[19]..script /var/home/zeus/.tea/lunarvim.org/v1.3.0/share/lunarvim/site/after/ftplugin/c.lua: Vim(runtime):E5113: Error while calling lua chunk: ...ite/pack/lazy/opt/mason.nvim/lua/mason-registry/init.lua:79: Cannot find package "clangd".
stack traceback:
        [C]: in function 'error'
        ...ite/pack/lazy/opt/mason.nvim/lua/mason-registry/init.lua:79: in function 'get_package'
        ....org/v1.3.0/share/lunarvim/lvim/lua/lvim/lsp/manager.lua:121: in function 'setup'
        ...rvim.org/v1.3.0/share/lunarvim/site/after/ftplugin/c.lua:1: in main chunk
        [C]: in function 'nvim_cmd'
        ...us/.tea/neovim.io/v0.9.1/share/nvim/runtime/filetype.lua:22: in function <...us/.tea/neovim.io/v0.9.1/share/nvim/runtime/filetype.lua:21>
        [C]: in function 'nvim_buf_call'
        ...us/.tea/neovim.io/v0.9.1/share/nvim/runtime/filetype.lua:21: in function <...us/.tea/neovim.io/v0.9.1/share/nvim/runtime/filetype.lua:10>
stack traceback:
        [C]: in function 'nvim_cmd'
        ...us/.tea/neovim.io/v0.9.1/share/nvim/runtime/filetype.lua:22: in function <...us/.tea/neovim.io/v0.9.1/share/nvim/runtime/filetype.lua:21>
        [C]: in function 'nvim_buf_call'
        ...us/.tea/neovim.io/v0.9.1/share/nvim/runtime/filetype.lua:21: in function <...us/.tea/neovim.io/v0.9.1/share/nvim/runtime/filetype.lua:10>
stack traceback:
        [C]: in function 'nvim_buf_call'
        ...us/.tea/neovim.io/v0.9.1/share/nvim/runtime/filetype.lua:21: in function <...us/.tea/neovim.io/v0.9.1/share/nvim/runtime/filetype.lua:10>
Press ENTER or type command to continue

I don't have clangd, though.
[zeus@fedora ~]$ clangd
bash: clangd: command not found
[zeus@fedora ~]$ tea clangd
tea: command not found: clangd

Should I also try to run the official LunarVim script to see if that fixes it? The earlier error message was in the latest macOS (x86).

clangd appears to be a C LSP that's part of LLVM, which don't appear to build. Looks like lvim wants to configure it by default if you open a C file. We should probably look into building that, if feasible.

Not sure. I tried running their script but it can't find nvim, even under tea:
[zeus@fedora lvim]$ tea https://raw.githubusercontent.com/lunarvim/lunarvim/master/utils/installer/install.sh

      88\                                                   88\
      88 |                                                  \__|
      88 |88\   88\ 888888$\   888888\   888888\ 88\    88\ 88\ 888888\8888\
      88 |88 |  88 |88  __88\  \____88\ 88  __88\\88\  88  |88 |88  _88  _88\
      88 |88 |  88 |88 |  88 | 888888$ |88 |  \__|\88\88  / 88 |88 / 88 / 88 |
      88 |88 |  88 |88 |  88 |88  __88 |88 |       \88$  /  88 |88 | 88 | 88 |
      88 |\888888  |88 |  88 |\888888$ |88 |        \$  /   88 |88 | 88 | 88 |
      \__| \______/ \__|  \__| \_______|\__|         \_/    \__|\__| \__| \__|

--------------------------------------------------------------------------------
Detecting platform for managing any additional neovim dependencies
[ERROR]: Unable to find dependency [neovim]
Please install it first and re-run the installer. Try: sudo dnf install -y neovim
[zeus@fedora lvim]$ nvim --version
NVIM v0.9.1
Build type: RelWithDebInfo
LuaJIT 2.1.0-beta3
Compilation: /opt/llvm.org/v14.0.6/bin/cc -O2 -g -Og -g -Wall -Wextra -pedantic -Wno-unused-parameter -Wstrict-prototypes -std=gnu99 -Wshadow -Wconversion -Wvla -Wdouble-promotion -Wmissing-noreturn -Wmissing-format-attribute -Wmissing-prototypes -Wimplicit-fallthrough -fdiagnostics-color=auto -fstack-protector-strong -DUNIT_TESTING -DINCLUDE_GENERATED_DECLARATIONS -D_GNU_SOURCE -I/tmp/e14cfd95/.deps/usr/include/luajit-2.1 -I/usr/include -I/tmp/e14cfd95/.deps/usr/include -I/tmp/e14cfd95/build/src/nvim/auto -I/tmp/e14cfd95/build/include -I/tmp/e14cfd95/build/cmake.config -I/tmp/e14cfd95/src -I/usr/include -I/tmp/e14cfd95/.deps/usr/include -I/tmp/e14cfd95/.deps/usr/include -I/tmp/e14cfd95/.deps/usr/include -I/tmp/e14cfd95/.deps/usr/include -I/tmp/e14cfd95/.deps/usr/include -I/tmp/e14cfd95/.deps/usr/include

   system vimrc file: "$VIM/sysinit.vim"
  fall-back for $VIM: "/opt/neovim.io/v0.9.1/share/nvim"

Run :checkhealth for more info
[zeus@fedora lvim]$

Here is a simpler example:
[zeus@fedora lvim]$ cat test.sh
#!/usr/bin/env bash

nvim --version
[zeus@fedora lvim]$ bash test.sh
test.sh: line 3: nvim: command not found
[zeus@fedora lvim]$ tea test.sh
/var/home/zeus/develop/lvim/test.sh: line 3: nvim: command not found
[zeus@fedora lvim]$

Try calling tea nvim in your test.sh.

Try calling tea nvim in your test.sh.

That works

Should this issue be moved to the pantry repo?

Yes, I can move it. There are at least 3 distinct issues above:

lvim fails to execute lua scripts when executed standalone
lvim wants to install clangd even if it's already present
lvin wants nvim in the path (which might be fixed on darwin, where we make symlinks by default)


Add instructions about rsync, remove pigz.

Note: probably we should mention / investigate what happens if only one side has rsync installed.

Yup.

Closed by #41.


Web Worker support broken?

What is wrong?
ReferenceError: brain is not defined when the browser version of brain is loaded in a Web Worker
Where does it happen?

How do we replicate the issue?
I've prepared a minimal test case here: https://agreeable-apricot.glitch.me/
How important is this (1-5)?
2? I saw at one point this was fixed (#146), but that change doesn't seem to be in the current release.
Expected behavior (i.e. solution)
Running a traning in the browser on the main thread not great for performance, so if it's still possible to use this library in a worker, that would be ideal.
Other Comments
Thanks for making such a great library!

@robertleeplummerjr please check

Please try to access brain using self.brain

updating to the latest version of the library seems to have resolved the issue.


Add Aditya Saini to Contributors list

PR-accepted.

Keep contributing.


error executing l10ns interface

Hi Tinganho,
I'm getting a follow error, when i tried to run l10ns interface
Cannot find module './configurations/autoRoutes'
i've already reinstalled l10ns.
Thanks in advance,
Marcelo

I've just pushed out a new version<EMAIL_ADDRESS>Can you try it out?

Hi Tingan,
i ve made a complete uninstall, installed again, and now i'm getting the following error:
EACCES, permission denied '/usr/lib/node_modules/l10ns/interface/public/scripts/routers/composer.js'
fs.js:439
return binding.open(pathModule._makeLong(path), stringToFlags(flags), mode);
^
Error: EACCES, permission denied '/usr/lib/node_modules/l10ns/interface//public/scripts/configurations/cf.js'
at Object.fs.openSync (fs.js:439:18)
at Object.fs.writeFileSync (fs.js:978:15)
at /usr/lib/node_modules/l10ns/interface/core/configuration.js:124:10
at Glob. (/usr/lib/node_modules/l10ns/node_modules/glob/glob.js:96:7)
at Glob.emit (events.js:95:17)
at Glob.next (/usr/lib/node_modules/l10ns/node_modules/glob/glob.js:343:12)
at Glob._processEmitQueue (/usr/lib/node_modules/l10ns/node_modules/glob/glob.js:310:12)
at Glob.emitMatch (/usr/lib/node_modules/l10ns/node_modules/glob/glob.js:290:8)
at Glob._finish (/usr/lib/node_modules/l10ns/node_modules/glob/glob.js:230:8)
at Glob. (/usr/lib/node_modules/l10ns/node_modules/glob/glob.js:183:27)
thanks,
Marcelo

composer script is a compiled file, but I haven't seen it cause permission problems before. Which OS are you using?

Made an chmod -R 775 /usr/lib/node_modules/l10ns
and now is running
thanks
Marcelo

kubuntu 14.10

i've made an chmod -R 775 to the root of the package and now its working fine.
thank you

OK I will try to fix this later. I don't have a ubuntu image available. Ideally you shouldn't need to chmod things.

This should be fixed now.


Build out email copy for flow 1 (new signup verification flow)

See flow 1 here: https://docs.google.com/spreadsheets/d/1CjKb3Ytz4YSAqnsGt3Ft2tv7IcHWQaYK6UDTBuusBWw/edit#gid=0
re-send the existing Signup Verification email if the user has not verified their email.

TODO: @chenkc805 needs to understand what's the email verification dropoff rate. Asking analytics here: https://sourcegraph.slack.com/archives/CN4FC7XT4/p1694056725292899


Fix one_hot generates output with an additional dimension in TensorFlow backend (#19724)

Make it consistent with the behavior of the NumPy backend
refer to:
https://github.com/keras-team/keras/blob/master/keras/src/backend/numpy/nn.py#L445

problems here, need to reconsider


Change to char escapes

Currently, single char escapes are: 0, a, b, f, n, r, t, v, s and e. This will be reduced to only the most common: 0, n, r, t, s and e. So a, b, f and v will be removed.
Here are the escapes to remove:

a
b
f
v

Here are the escapes to keep:

0
n
r
t
s
e

Chars will be put back to maintain some kind of compatibility with Json. Json uses b and f. While at it, a and v should also be added.

Previous commit: f491d6a79bcc9b9edde6eed0e56c482227694781


[Bug] Cannot configure Loki's derived fields

Because of jsonData fields whitelisting I'm facing a problem similar to this issue.
Can I ask what is the rationale behind whitelisting jsonData? It seems that explicitly allowing each and every field is not a particularly scalable solution, especially regarding third-party datasource plugins.
In my case I need to be able to define something like this:
apiVersion: integreatly.org/v1alpha1
kind: GrafanaDataSource
metadata:
  name: loki-datasource
  namespace: monitoring
spec:
  name: loki.yaml
  datasources:
    - name: Loki
      type: loki
      uid: loki
      access: proxy
      url: http://loki:3100
      version: 1
      editable: false
      jsonData:
        derivedFields:
          - datasourceUid: tempo
            matcherRegex: (?:traceID|trace_id)=(\w+)
            name: TraceID
            url: $${__value.raw}

I've been wondering this myself.
It doesn't seem to serve much purpose beyond unmarshalling everything to map[string]interface{} and back again, unless there's something I'm missing.
If we generated the CRD manifests using these structs, I'd be a bit more on board with the whitelisting as it would be a stronger type guarantee, but this doesn't happen as far as I can tell so the benefit is not clear.

Resolved thanks for raising this issue @MatteoJoliveau :+1:
also CC @pb82  on @DeanBrunt comment.
Leaving open for discussion on that point.

Resolved thanks for raising this issue @MatteoJoliveau :+1:
also CC @pb82  on @DeanBrunt comment.
Leaving open for discussion on that point.


Add basic hotkey support in the admin UI

Closes #336

Coverage increased (+0.03%) to 87.769% when pulling 20b6266c059913b222ef407c84cbd65a056b0687 on feature/hotkey-nav into 67f0fd3317426ba82f1b29554295bde7c1864371 on master.


Set-PSReadlineOption -HistoryNoDuplicates borks value of -ViModeIndicator

My profile loads
Set-PSReadlineOption -EditMode Vi
Set-PSReadlineOption -ViModeIndicator Cursor

For the first time I set Set-PSReadlineOption -HistoryNoDuplicates and noticed my block cursor stopped working. Eventually I discovered (Get-PSReadlineOption).ViModeIndicator returned None.
Setting the order to
Set-PSReadlineOption -HistoryNoDuplicates
Set-PSReadlineOption -EditMode Vi
Set-PSReadlineOption -ViModeIndicator Cursor

is a temporary workaround.

Actually, after working with this some more, it appears that any call to Set-PSReadlineOption turns off the mode indicator even though the the option value remains unchanged:
ViModeIndicator                        : Cursor
Simply executing Set-PSReadlineOption -ViModeIndicator Cursor restores correct behavior.

I fixed the problem where the indicator mode was being reset to None.  Please give it a try and let me know if you still see it.
I also added an option called Prompt.  I haven't spent much time trying to get StatusLine working, so it is presently not available as a choice.

@srdubya Thanks Steve. :smile:
I apologize for not replying sooner -- I tested and then forgot. Everything looks good with respect to the mode indicator changing. I also like the Prompt option. I'm not sure I'll use it very often but it's kind of neat.


We need to change the text color and background color of the Delete button.

Describe the task

Go to KnowledgeCafe --> Library --> Book Category
The Delete button is now showing as normal text. So it doesn't know that it's a button.

Expected behavior

We need to change the text color and background color of the Delete button. As shown in the image below.

References
Bootstrap button:- https://getbootstrap.com/docs/4.0/components/buttons/

Working Fine.


how to do semantic segmentation on video using saved model from tensorflow-2?

Currently we have TF2 on production environment so wanted to do semantic segmentation on video using saved_model.pb (downloaded model with saved_model.pb)
Video segmentation worked for frozen graph using TF-1 but TF-2 dont support frozen graph so please provide script or any working example which will work on saved_model.pb.
System information
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Ubuntu 18.04
TensorFlow version (use command below): 2.5
Python version: 3.6
GCC/Compiler version (if compiling from source): 7.5
CUDA/cuDNN version: 11.2
GPU model and memory: T4 GPU

Hi @PritiDrishtic,
Thank you for reporting the issue.
Unfortunately, your issue is more related to TensorFlow support.
Please report the issue to the TensorFlow team.
Closing the issue, since it is not related to our library.
Cheers,


添加wxmodule和wxcomponent鉴权接口

1.添加WXAuthProcessor接口用于鉴权
2.WXBridgeManager的callNativeModule中调用WXAuthProcessor的onModuleAuth接口
3.WXDomObject中的parse方法中调用WXAuthProcessor的onComponentAuth接口
4.WXSDKInstance中的renderByUrl和setBundleUrl调用WXAuthProcessor的needAuth接口判断该instance是否需要鉴权

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.照烈 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.

ios版本的代码呢


error none type

i have this error
have pyhton pip dependences installed
what more should install for this program work like should be work WITHOUT errors
thanks
python ./dashproxy.py https://------.mpd -o. -d
Running dash proxy for stream https://-------.mpd. Output goes in .
mpd=[<Element '{urn:mpeg:dash:schema:mpd:2011}Period' at 0x0000023B3BC46310>]
Starting a downloader for Representation (period=0 adaptation-set=0 representation=0)
Traceback (most recent call last):
File "D:\Jonathan\Programas\Video_Encriptado\cmder_mini\dashproxy.py", line 289, in
main()
File "D:\Jonathan\Programas\Video_Encriptado\cmder_mini\dashproxy.py", line 286, in main
run(args)
File "D:\Jonathan\Programas\Video_Encriptado\cmder_mini\dashproxy.py", line 275, in run
return proxy.run()
File "D:\Jonathan\Programas\Video_Encriptado\cmder_mini\dashproxy.py", line 115, in run
self.refresh_mpd()
File "D:\Jonathan\Programas\Video_Encriptado\cmder_mini\dashproxy.py", line 129, in refresh_mpd
self.handle_mpd(mpd)
File "D:\Jonathan\Programas\Video_Encriptado\cmder_mini\dashproxy.py", line 155, in handle_mpd
self.ensure_downloader(mpd, rep_addr)
File "D:\Jonathan\Programas\Video_Encriptado\cmder_mini\dashproxy.py", line 173, in ensure_downloader
downloader.handle_mpd(mpd, self.get_base_url(mpd))
File "D:\Jonathan\Programas\Video_Encriptado\cmder_mini\dashproxy.py", line 204, in handle_mpd
segment_timeline = self.mpd.segment_timeline(self.rep_addr)
File "D:\Jonathan\Programas\Video_Encriptado\cmder_mini\dashproxy.py", line 77, in segment_timeline
return self.segment_template(rep_addr).find('mpd:SegmentTimeline', ns)
AttributeError: 'NoneType' object has no attribute 'find'

why noone answer???????????????????????????????????

Can you provide a manifest for which you tried to run the dashproxy? It looks that dashproxy can't find "SegmentTemplate" in Adaptation or Representation.

https://cdn-videos.flowlikemusic.com/files/videos/e125e3ef-912b-44ab-9a56-69593ab4d1cd/master.mpd

For this manifest I get different error:
Running dash proxy for stream https://cdn-videos.flowlikemusic.com/files/videos/e125e3ef-912b-44ab-9a56-69593ab4d1cd/master.mpd. Output goes in .
mpd=[<Element '{urn:mpeg:dash:schema:mpd:2011}Period' at 0x7f041f1054a8>]
Starting a downloader for Representation (period=0 adaptation-set=0 representation=0)
requesting master_hdinit.cmfv from https://cdn-videos.flowlikemusic.com/files/videos/e125e3ef-912b-44ab-9a56-69593ab4d1cd/master_hdinit.cmfv
Traceback (most recent call last):
File "dashproxy.py", line 289, in
main()
File "dashproxy.py", line 286, in main
run(args)
File "dashproxy.py", line 275, in run
return proxy.run()
File "dashproxy.py", line 115, in run
self.refresh_mpd()
File "dashproxy.py", line 129, in refresh_mpd
self.handle_mpd(mpd)
File "dashproxy.py", line 155, in handle_mpd
self.ensure_downloader(mpd, rep_addr)
File "dashproxy.py", line 173, in ensure_downloader
downloader.handle_mpd(mpd, self.get_base_url(mpd))
File "dashproxy.py", line 211, in handle_mpd
segments = copy.deepcopy(segment_timeline.findall('mpd:S', ns))
AttributeError: 'NoneType' object has no attribute 'findall'
Unfortunately it seems that dash-proxy does not support such SegmentTemplate (without SegmentTimeline) which is used in this manifest.
I will try to fix that.

@kevan1 : I provided a fix for your manifest on my branch: https://github.com/przemyslaw-gorszkowski-red/dash-proxy/tree/master_branch, please try to download your content with my modification

@przemyslaw-gorszkowski-red I'm trying to download the subtitles from this link but the script does not work
https://cdxaws-ak.akamaized.net/out/v1/55ceb00a9faf44258853629702fd6cb4/469c706849ff410385e5d65308566647/5e2a57fc0380466b9864849ce3f168ea/manifest.mpd
I can download the audio and the video from the link using
https://github.com/nilaoda/N_m3u8DL-CLI
which is not supporting downloading the subtitles, I only need the unencrypted subtitles. Could you try to modify the script? Thanks.

@Essam311 are you still interested in to get the script which is able to download subtitles from:
https://cdxaws-ak.akamaized.net/out/v1/55ceb00a9faf44258853629702fd6cb4/469c706849ff410385e5d65308566647/5e2a57fc0380466b9864849ce3f168ea/manifest.mpd ?
I fixed the existing script and will provide it this week on my fork.

From: Essam311 @.***>
Sent: 07 April 2021 02:29:17
To: Viblast/dash-proxy
Cc: Przemyslaw Gorszkowski; Mention
Subject: Re: [Viblast/dash-proxy] error none type (#17)
@przemyslaw-gorszkowski-redhttps://github.com/przemyslaw-gorszkowski-red I'm trying to download the subtitles from this link but the script does not work
https://cdxaws-ak.akamaized.net/out/v1/55ceb00a9faf44258853629702fd6cb4/469c706849ff410385e5d65308566647/5e2a57fc0380466b9864849ce3f168ea/manifest.mpd
I can download the audio and the video from the link using
https://github.com/nilaoda/N_m3u8DL-CLI
which is not supporting downloading the subtitles, I only need the unencrypted subtitles. Could you try to modify the script? Thanks.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Viblast/dash-proxy/issues/17#issuecomment-814517449, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALJAJVASAJI6BEFSTK5XXA3THORN3ANCNFSM4ZIC3CXQ.


Setting JS Rule on a new estimate table removes any estimate from tokens

When I create a new estimate table and apply a JS Rule of any of the provided samples, no estimate of any table are shown when hovering over a token.
To Reproduce
Steps to reproduce the behavior:

Configure settings to add a new health estimate table
Add a JS Rule to the new estimate table
Hover over token on scene
See no estimate shown above token

Expected behavior
The default estimate should persist if the new table and the JS Rule fail. If the JS rule is successful, estimates from the new table should be shown.

Game Specifics (please complete the following information):

Game System and version: dnd5e 2.2.2
Foundry: 10.303
Health Estimate: 31.2

I've added error handling to fix this issue, now there will be a warning on the console about this.
As for the JS Rule, this post made me realize that "actor" wasn't part of the JS Rule's scope, so all the samples I've posted weren't valid. I'm sorry for that. I've now added "actor" to the scope, so the samples are all valid and the one you've input for your JS Rule should work now.


Key Verification Support

Two major changes:

Separates Proxying from Creating a PeerConnection;
Adds a messaging API on PeerConnection over the control channel to go alongside heartbeats.

This change is

So, the problem with this is that you're adding features - sending messages - to PeerConnection which are already covered by DataChannel. The boilerplate implementations of both sendMessage and registerMessageHandler to both bridge.ts and churn.ts are a major smell.
Why don't you just create another datachannel and send the verification messages over that? You may not need to change PeerConnection or SocksToRtc interface at all (I certainly wouldn't object to improving those interfaces but don't go there unless you really need to since it'll require non-trivial changes to uProxy).

The PeerConnection already has a control DataChannel, and creating new channels on PeerConnections is a mess when the rest of uProxy's designed for them all to be proxy connections  I tried a new-datacannel-dispatcher approach before, for chat, and that got as much bounce back as this.  What do you want here?

Ah, I remember that now - RtcToNet assumes all new channels on the peerconnection are for proxying. But you could change that, and have it ignore channels that weren't created by SocksToRtc:

https://github.com/uProxy/uproxy-lib/blob/master/src/rtc-to-net/rtc-to-net.ts#L219

You'll need to examine how Pool (currently used by SocksToRtc) names channels (basically: p + number):

https://github.com/uProxy/uproxy-lib/blob/master/src/pool/pool.ts#L92

I think you might be able to avoid making any changes to PeerConnection.

Bah, Pool is a real mess isn't it. After discussing in person, seems like a getControlChannel-type method might be the most reasonable/expedient approach for now. Sorry...I agree it's too hard to make any change right now.

Thanks :smiley_cat:


[Bug]: EntraID Group mapping not working, when having more than 100 group memberships.

Plugin Version
6.0.1
PHP Version
8.2.19
Shopware Version
<IP_ADDRESS>
Installation method
Composer
Identity provider
Microsoft Azure OIDC
What happened?
Trying to configure a role assignment rule using the object ID of an EntraID group.
My user has more that 150 groups assigned (direct and indirect memberships)
In MicrosoftAzureOidcGroupsRule.php the code does not seem to honor @odata.nextLink tag to pull any more pages of the call made to https://graph.microsoft.com/v1.0/me/transitiveMemberOf. so only the first page will be checked and if the group id is not listed, the rule will not apply.
The call to MSGraph should be like (only return the group IDs makes it faster and much less traffic)
https://graph.microsoft.com/v1.0/me/transitiveMemberOf?$select=id
then, if the answer has @odata.nextLink tag, the code should iterate through all the pages returned before evaluating the membership.
Relevant log output
No response

Hi @ckupferschmid I just released an RC for 7.0.1. Could you test and confirm that it works with that fix? In my tests, it seemed to be fine.

Great, thank you.
sorry to ask, but how do I reference the rc of 7.0.1 using composer?

Hmm, does it work with Shopware <IP_ADDRESS>? Since 7.0.0 you only say it's compatible with Shopware 6.6.

We try to follow semver so I assume it would not. In the changelog we do have "Removed Shopware 6.5 compatibility" so I assume we make use of something new but I cannot confirm right now. Do we need to backport it for you or would you try to use the patch of the commit with https://github.com/cweagans/composer-patches/ ?

@JoshuaBehrens
Our project team would love to get a backport from your side in the 6.0 branch.
We do not have any experience with composer-patches.

@ckupferschmid I see. Let's what I can do about it

Moin @JoshuaBehrens
Any updates on this topic? Would help us.


[Snyk] Security upgrade django from 3.2.13 to 3.2.14

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `pip` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

requirements.txt

⚠ Warning
python3-openid 3.2.0 has requirement defusedxml<=0.4.1, but you have defusedxml 0.7.1.
python-jose 3.3.0 requires rsa, which is not installed.
paramiko 2.10.3 requires pynacl, which is not installed.
jsonschema 3.2.0 requires pyrsistent, which is not installed.
drf-oidc-auth 2.0.0 requires django, which is not installed.
django-helusers 0.7.0 requires Django, which is not installed.
deprecation 2.1.0 requires packaging, which is not installed.

Vulnerabilities that will be fixed
By pinning:

Severity
Priority Score (*)
Issue
Upgrade
Breaking Change
Exploit Maturity

741/1000  Why? Recently disclosed, Has a fix available, CVSS 9.1
SQL Injection SNYK-PYTHON-DJANGO-2940618
django:  3.2.13 -> 3.2.14
No
No Known Exploit

(*) Note that the real score may have changed since the PR was raised.
Some vulnerabilities couldn't be fully fixed and so Snyk will still find them when the project is tested again. This may be because the vulnerability existed within more than one direct dependency, but not all of the effected dependencies could be upgraded.
Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 SQL Injection

Codecov Report

Merging #253 (5185d06) into develop (5014a5d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #253   +/-   ##
========================================
  Coverage    93.44%   93.44%
========================================
  Files          118      118
  Lines         4086     4086
  Branches       360      360
========================================
  Hits          3818     3818
  Misses         200      200
  Partials        68       68

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 5014a5d...5185d06. Read the comment docs.

Duplicated #255


fix(cli): call setup hook in command

Types of changes

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

Description
It's overall a fix (where the setup hook is called) that will make TypeScript Runtime shine by not relying anymore on process.argv to find the rootDir in a not optimal (glitchy) way.
See : https://github.com/nuxt/typescript/pull/97
Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly. (PR: #)
[x] I have added tests to cover my changes (if not applicable, please state why)
[x] All new and existing tests are passing.

Codecov Report

Merging #6353 into dev will decrease coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##              dev    #6353      +/-   ##
==========================================
- Coverage   95.71%   95.71%   -0.01%
==========================================
  Files          79       79
  Lines        2663     2661       -2
  Branches      682      681       -1
==========================================
- Hits         2549     2547       -2
  Misses         98       98
  Partials       16       16

Flag
Coverage Δ

#e2e
100% <ø> (ø)
:arrow_up:

#fixtures
50.73% <0%> (+0.03%)
:arrow_up:

#unit
92.37% <100%> (-0.01%)
:arrow_down:

Impacted Files
Coverage Δ

packages/cli/src/run.js
100% <ø> (ø)
:arrow_up:

packages/cli/src/command.js
99% <100%> (ø)
:arrow_up:

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 c010add...3591802. Read the comment docs.

Codecov Report

Merging #6353 into dev will decrease coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##              dev    #6353      +/-   ##
==========================================
- Coverage   95.71%   95.71%   -0.01%
==========================================
  Files          79       79
  Lines        2663     2661       -2
  Branches      682      681       -1
==========================================
- Hits         2549     2547       -2
  Misses         98       98
  Partials       16       16

Flag
Coverage Δ

#e2e
100% <ø> (ø)
:arrow_up:

#fixtures
50.73% <0%> (+0.03%)
:arrow_up:

#unit
92.37% <100%> (-0.01%)
:arrow_down:

Impacted Files
Coverage Δ

packages/cli/src/run.js
100% <ø> (ø)
:arrow_up:

packages/cli/src/command.js
99% <100%> (ø)
:arrow_up:

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 c010add...3591802. Read the comment docs.


[FEATURE] Optional flight sound and visual effects

Not sure how difficult this would be, but I feel like it could add a ton to the experience. No worries if there is no interest.
Describe the solution you'd like
Optional Audio feedback for flight for example;

you can hear the air rushing past and have the sound change depending on velocity
slight air resistance sound effects for successful wing flaps, and braking

Noted. Will keep this request open
On Thu, Jul 27, 2023, 8:10 PM Maleus @.***> wrote:

Not sure how difficult this would be, but I feel like it could add a ton
to the experience. No worries if there is no interest.
Describe the solution you'd like
Optional Audio feedback for flight for example;

you can hear the air rushing past and have the sound change
depending on velocity
slight air resistance sound effects for successful wing flaps, and
braking

—
Reply to this email directly, view it on GitHub
https://github.com/Mattshark89/OpenFlight-VRC/issues/38, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHWYVDUIQGNPAF3UDBVDYSLXSMNQVANCNFSM6AAAAAA223ZTTQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

May as well mention I've been wanting to do something like this for a while, but haven't because I have no audio manipulation experience. I'd try and make the wind louder the faster you travel, etc.. Would love to try it in the future though, or see how others have done it in other instances.

so, this is going to be interesting to implement
Networking on this platform just..... sucks
I am looking into it though

using https://github.com/CyanLaser/CyanPlayerObjectPool I have managed to get per-player information sync working

This means I can properly work on the effects implementation now

just an update, pretty far deep into the implementation stages of this, pretty much borderline release candidate at this point, it just needs more testing

Finally merged! Release is 1.6.0


path to data

There is a folder called data in the repository. Not sure what this is.
Also, for the repository crowdastro-data, we should just use a soft-link to point to it. Then paths in the notebooks can be ../crowdastro-data.

https://github.com/chengsoonong/crowdastro/blob/master/docs/setup.md
Agreed on the soft link.

We should probably just wrap every data access with the crowdastro module. I can get rid of the dependency on gator_cache.sqlite now, so this will be much easier to do.

Should be resolved in the rework branch. Will merge soon, #58


Bug Report - Conduits spiking tick time to 100ms

Is there an existing issue for this?

[x] I did not find any existing issues.

Current Behavior
My TPS dropped to 8, checked the spark profiler and it's almost entirely conduits,

Persists through a server restart, see below:

Expected Behavior
No response
How to Test
No response
Local Environment

Operating System: Windows
Java version: 17
Minecraft Version: 1.20.1
Version: 6.2.0
Other Mods: https://github.com/ThePansmith/Monifactory

Any Additional Information?
The following is a .sparkprofile file, for viewing at https://spark.lucko.me/. Change the extension, etc.
GOnDKpQVJm.txt

looks like you have a big capacitor bank that is a bit too slow. If you don't need it you could either downsize it or disconnect it until we've upgraded the performance of them. sorry for the inconvenience

Forgot to update, removed the capacitors and the tps issues went away. Thank you!


use null prototype objects for languages/aliases

getLanguage currently returns values for object prototype method names. By swapping to objects with a null prototype, we can fix this weird behavior.

Discord uses getLanguage to validate that a language specified exists in highlightJS and retrieve metadata about the language for code block highlighting in chat. Because highlightJS returns prototype values instead of the highlight languages themselves, the result is a few different bugs in our clients which expect the return type to be only Language | undefined.

@night Thanks so much!


contract address didn't show a contract creation transaction

When there's a new contract was created on chain, supposely the transaction page should have the contract information together with owner information, but right now it's blank.
http://<IP_ADDRESS>:3000/tx/0xe97328dfe6fcb0d83e3a694fc702007cca5412390ebd759405371c9c8cbe7cd4

fixed


"Flatpak manifest detected" is always appearing every startup

I'm not sure if it is intended or a bug. If it is intended, there could be a don't show again, or automatically init a build every launch

it is a bug, because we show it before we have parsed the status of the pipeline

It shouldn't be hard to do, we need to add a new attribute to the state to tell if the state was loaded from the disk or not yet and wait for that to be set to something before deciding whether to show the dialog or not


Update LCE python interpreter docstring with efficiency warning

What do these changes do?
Updates the docstring of the python interpreter wrapper to warn about unoptimized performance for any platform other than ARM v8.
Related issue number
https://github.com/larq/larq/issues/607

@Tombana any idea what's wrong here? This seems unrelated to my changes...

@Tombana any idea what's wrong here? This seems unrelated to my changes...

Looks like a flaky test, I've restarted the tests now.

Looks like a flaky test, I've restarted the tests now.


suggest to archive http://www.biodiversity-exploratories.de/en/

http://www.biodiversity-exploratories.de/en/ contains many biodiversity datasets originating from German research groups.

fyi @iimog

an example of a dataset provided by @caterinap -
https://www.bexis.uni-jena.de/ddm/data/Showdata/20067


Kitematic stops and delete all the changes

Expected behavior
Actual behavior
Information about the Issue
I'm not sure how to define this issue.
But basically I'm doing this:
I installed Docker Kitematic 0.17.0 and I installed (a lot of times) Django (Official) using the GUI.
and I start a new Django Project.
All is good at this point.
When the image is running, I'm trying to add a new port (8000 to be specific) and the image is restarted and all my changes are deleted (but, not only my project changes, also the apt-get changes) and the image is restarted as the original.
It's very weird, and the only change that I'm doing is adding the port using the GUI
Steps to reproduce the behavior

...
...

Containers are note stateful (immutable) and thus aren't suppose to be modified - If you need to make modifications that persist:

Mount a directory where the data will be saved
Mount a volume where the data will be saved

When kitematic makes a  port change it re-creates the container in the background, as there is no way to dynamically change a container's config post-creation.


Effect runs every render when toggle is a dependency

I'm trying to run an enter animation when a component mounts.
In the docs, useEffect has a dependency on toggle, but toggle seems to change on every render. Is it not safe to use in the deps array?

I would expect the effect to run only once, despite toggle being a dependency.

hey @wouteralberts , the toggle function is stable across renders. The only exception is when the main hook is supplied with an onStateChange callback which is not stable. If that happens, you could create your onStateChange using React's useCallback.

Ah yep, that’s indeed what’s happening here. Makes sense, thanks!


ExportPathMap not working

I am trying to use exportPathMath to export dynamic routes. The exportPathMath seem to function correctly but the changes is not shown in sitemap.xml. URL is still "/shop/[projectID]".
Here ist my next.config.js:
const nextConfig = {
	exportPathMap: async () => {
		const response = await fetch('https://api.mywebsite.com/projects?modules.pagesection.title_contains=shop')
		const shopList = await response.json()

		const pages = shopList.reduce(
			(pages, shopItem) =>
				Object.assign({}, pages, {
					[`/shop/${shopItem.id}`]: {
						page: '/shop',
						query: { id: shopItem.id }
					}
				}),
			{}
		)

		const obj = Object.assign({}, pages, {
			'/': { page: '/' },
			'/indexlist': { page: '/indexlist' },
			'/shop': { page: '/shop' },
			'/information': { page: '/information' }
		})

		console.log('obj: ', obj) // prints correct routes
		return obj
	},
}

module.exports = withPlugins([ withPreact, withWebpack, linaria ], nextConfig)

My server.js:
sitemap({
	baseUrl: 'https://www.mywebsite.com',
	ignoredPaths: [ 'admin' ],
	pagesDirectory: __dirname + '/pages',
	targetDirectory: 'static/',
	nextConfigPath: __dirname + '/next.config.js',
	ignoredExtensions: [ 'png', 'jpg' ]
})

What am I doing wrong?

Hi, possibly the problem is withPlugins because returns a function and we expect an object.
You can configure manually without the auto configurators, to do that here https://nextjs.org/docs#customizing-webpack-config explains how.
Try it and tell me.

Thank you @IlusionDev for your answer, but sadly I do not understand concretely how to do this. However I change the nextConfig object it returns a function for the exportPathMap key, async or sync.
If I only module.export the object (exportPathMap & env) it works well :) Can you please write an example how I can combine this with exports of functions (withPreact, linaria)?
The only way that I managed to solve this was to create another file called next.config_for_pathmap.js and add that name to nextConfigPath.

No problem...I added support to next-compose-plugins (works with withPlugins) and you have not worry about nothing.
Everything works as always.
In a few minutes I'm going to release a new minor version with the change. Check it. 😸

That is awesome! Thank you a lot!


Add CI test for parsl.AUTO_LOGNAME [NEEDS stdout/stderr BEHAVIOUR TO BE BETTER DEFINED]

This new test is failing in the workqueue executor. I can't immediately see the cause. @dthain?

Further playing suggests that the workqueue executor does not like complicated pathnames: those with directory components in them, at least for stdout. See PR #1185. This might be the failure for this PR too, as parsl.AUTO_LOGNAME paths are absolute paths on the filesystem.

I think assumptions about shared filesystems need to be tightened up before this will work with workqueue - which is where it is failing at the moment. I'm leaving this PR open because it is something that definitely needs fixing and testing.

closing this because it's unlikely stderr/out handling will get tidied up as needed for this


[WIP] Migrate Gallery benchmarks out of flutter/flutter (flutter side)

Working in progress. Do not review yet.
Description
This PR moves Gallery web benchmarks out of Flutter proper and separates the dependencies for flutter/flutter and flutter/gallery, allowing the two repos to freely update dependencies in the future, while preserving the important benchmarks which give us tons of useful signals, not just performance, but stability also.
These three PRs (see References section below) behave in the following way:

web_benchmarks side — Creates the web_benchmarks package, the common package used by both non-Gallery web benchmarks and Gallery web benchmarks.

Once this PR is approved, we create a package (possibly publishing to pub.dev) and redirect all imports to that package.

gallery side — Adds Gallery benchmarks (browser side) to the flutter/gallery.
flutter side — Rewrite Gallery benchmarks in flutter/flutter to stop using pubspec.yaml dependencies, and start pulling flutter/gallery at runtime.

The three PRs must be landed in this order.
Currently, some dependencies / git references are pointing to my own branches. They will be moved to official branches once the corresponding prerequisites are landed.
References
web_benchmarks side — https://github.com/pennzht/web_benchmarks/pull/1
gallery side — https://github.com/flutter/gallery/pull/321
flutter side — https://github.com/flutter/flutter/pull/66351
Related Issues
Replace this paragraph with a list of issues related to this PR from our issue database. Indicate, which of these issues are resolved or fixed by this PR. There should be at least one issue listed here.
Tests
No tests are added here.
Tests are added for the web_benchmarks package.
Checklist
Before you create this PR, confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] All existing and new tests are passing.
[ ] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read Handling breaking changes.

[ ] No, no existing tests failed, so this is not a breaking change.
[ ] Yes, this is a breaking change. If not, delete the remainder of this section.

[ ] I wrote a design doc: https://flutter.dev/go/template Replace this with a link to your design doc's short link
[ ] I got input from the developer relations team, specifically from: Replace with the names of who gave advice
[ ] I wrote a migration guide: https://flutter.dev/go/breaking-changes-template Replace this with a link to a pull request that adds the migration guide to https://flutter.dev/docs/release/breaking-changes

Are there still plans for this or should we close it?

Since there has been no progress here for a few weeks and the PR is marked "draft", I'm going to close the PR for now. Please do not hesitate to create a new PR when this is ready to land! Thanks!


when the code and data will be released?

I’ve been following it for a long time. Please tell me when the code and data will be released. Thank you!

Hey there,
Thanks for the message.
I think it will take me one more week (I had a lot of work related stuff in January, so I could not focus on the code) - I hope to finish it by the 10th Feb (meanwhile, the main life2vec repository contains all the code, this repository will have a simplified version).
Just a note - we won't release the data, instead we are going to provide a randomly generated dataset.

Thank you very much for your reply, looking forward to seeing your great work and wishing you good health.

UPDATE: I am actively working on the code, hope to release during the weekend!

Still working! But the first batch of code should be online by Wednesday!


Generated property tag does not contain a possible null value on the array-casted JSON field

Versions:

ide-helper Version: dev-master
Laravel Version: 7.0.x
PHP Version: 7.4

Description:
I have a not-null JSON field in the table that casts to the array in the Eloquent model. Ide-helper adds a "@property array $field_name" that does not contain a possibly null value because a database field is not nullable. But the JSON can contain a "null" value as a string that will be cast into a null value through attribute casting. I think that the ide-helper should add an "array|null" docblock in such cases.
This issue breaks a PSALM checks logic during additional checks in the code.

I think that the ide-helper should add an "array|null" docblock in such cases.

But how do you define "such cases"?
It's a all-or-nothing, because when generating the code we can't know if your application is supposed to have to deal with this or not.
Suggested workaround: define a getter and the phpdoc/return type of the getter will be promoted to the @property and it will fix your PSALM checks.

But how do you define "such cases"?

You can look into the Eloquent Array mutator. It's using the simple json_decode that does not contain any post-processing code and it always can return a null as a value, so I think it's better to hardcode an "array|null" in the attributes that have cast from array or object mutators.
You can reproduce this issue:
The database has a not-nullable field "testfield" with the JSON or VARCHAR type:
$model->testfield = null; $model->save();
A field will contain a "NULL" string in the database and the returned output will be null but the database field is not nullable.


Revert doesn't revert until a new action has been dispatched

Not sure what we are doing wrong. However, when we press "revert" in the devtool all the actions disappear however nothing happens in our UI. It is first when we dispatch an action (any action) that the revert is visible.
What might be causing this?

The reducers seem to run but the mapStateToProps is never invoked. I'm using immutable js if that has any possible relation to the issue.

Seems to be related to batchedSubscribe(batchedUpdates) enhancer.
import { batchedSubscribe } from 'redux-batched-subscribe'
import { compose, createStore, applyMiddleware } from 'redux'
import { unstable_batchedUpdates as batchedUpdates } from 'react-dom' // eslint-disable-line camelcase

if (process.env.NODE_ENV === 'development' && process.env.IS_BROWSER) {
  const { devTools, persistState } = require('redux-devtools')
  enhancers = [
    //batchedSubscribe(batchedUpdates) // works without this line
  ]
  .concat(enhancers)
  .concat([
    devTools()
  ])
}

export default function(initialState) {
  return compose(...enhancers)(createStore)(appReducer, initialState)
}

Try putting it after devTools()?

Yes, putting it after devTools() solves the issue.


736 bug styling on dos opportunities page

Use a class name common across both G-Cloud and DOS searches for style - fixes record count size.
No longer import all of gov-uk elements - fixes heading size by ensuring that we no longer load the govuk-elements _reset.scss stylesheet. (Note that we do still load our own _reset.scss file, which is a bit different!)
https://trello.com/c/DG6GGfib/736-bug-styling-on-dos-opportunities-page

Looks good.


Learn.md

Learn repos should have a LEARN.md file to teach student how to build your project step by step. You can explain how to build your project with text, code snippets, images, or even short (5 minute) long video lessons. As the maintainer of a Learn repo, the LEARN.md file requires you to think critically about how to explain the building of your project and how to also make it engaging. We don't expect you to be an expert teacher, but we would like you to reflect on how difficult it was to get to your level of knowledge, and then provide friendly guidance to help other students to learn.
Reference: https://github.com/Recode-Hive/Stackoverflow-Analysis/blob/main/Learn.md

Hello, @Trisha-tech. Could you please assign this to me under gssoc'24 ?
Thanks


ExplicitJdbcTransaction leaves dangling “begin”

Hi Rob,
This issue is preventing me from upgrading to Ebean 6. It would be super if we could get this fixed as it would mean an end to all the wacky transaction trickery I’ve had to do to support auto commit in both true and false forms.
My JDBC and OpenJPA work requires auto commit true (I’ll call this ACT). Earlier Ebean requires it false (ACF). I had a little trickery in my data source that would flip it to ACF before giving the connection to Ebean (unless I was already in a transaction). This was working but not ideal.
JdbcTransaction has performRollback and performCommit but no performBegin to start a transaction. Instead it appears checkAutoCommit has been “repurposed” to start the transaction. With ACF you can select all day, no problem. As soon as you insert/update/delete you have to commit. There’s essentially always one implicit transaction. With ACT you need to issue the query “begin” when a transaction is to be started. This is presently done in checkAutoCommit.
The problem with this is when there is no explicit transaction and only “select” statements are issued, the “begin” query is not matched by any “commit” or “rollback”. This doesn’t seem to be a huge problem in general but it is causing a weird hang up when I pair a couple unit tests together. I have an update method that hangs forever. Not likely reproducible in an independent unit test.
My thinking is that JdbcTransaction needs a performBegin or some such that is called only when a transaction is to be started (implicit or explicit). Any subclass that uses ACF would do nothing while ExplicitJdbcTransaction would issue “begin”. ExplicitJdbcTransaction.checkAutoCommit should simply verify that auto commit is true.
What do you think?

The problem with this is when there is no explicit transaction and only “select” statements are issued, the “begin” query is not matched by any “commit” or “rollback”.

This somewhat specific to Postgres and Oracle (MVCC databases that I trust with read committed isolation level). Specifically in your case you are using PostgresPlatform and that has onQueryOnly = OnQueryOnly.CLOSE (rather than ROLLBACK or COMMIT).
If you look/breakpoint at TransactionManager.initOnQueryOnly() you will see this value checked against the isolation level etc and here it also allows explicit setting via system property.
String systemPropertyValue = System.getProperty("ebean.transaction.onqueryonly");

Now the reason this is like this generally is because usually "begin transaction" is implicit ... however, in the case of using ExplicitJdbcTransaction the "begin transaction" is no longer implicit but instead executes the "begin" statement.
So I believe what you are experiencing is that via ExplicitJdbcTransaction you get explicit "begin" executed but via onQueryOnly=OnQueryOnly.CLOSE ... we are not seeing effective closing of the transaction in the query only cases.
So to confirm the theory set system property via:
System.setProperty("ebean.transaction.onqueryonly", "COMMIT"); // or ROLLBACK if you like

... and run the tests again.

I have pushed up an expected fix for this issue.  Daryl, if you get a chance to review and perhaps test that would be great.

Setting the system property ebean.transaction.onqueryonly to either COMMIT or ROLLBACK gets me past the weird hanging problem. The begins that were dangling are now matched with commit/rollback. It looks like I need to use "COMMIT" in order manage my transactions in JDBC.
I can't build Ebean. It fails on what looks like a bunch of java.time stuff and ... we're still on Java 6. It looks like you are simply choosing ROLLBACK instead of CLOSE. It seems fine, but since I need COMMIT, it won't serve me in the end. Still, if you send me a built snapshot I can test that it gets past my hanging test.
For a bunch of select-only queries, would you say the begins/commits that wrap them are going to cost me a performance hit?

but since I need COMMIT, it won't serve me in the end.

I can change it to use COMMIT, no problem there.

For a bunch of select-only queries, would you say the begins/commits that wrap them are going to cost me a performance hit?

Yes but that is ok.

Hi Rob, thanks for sending me the snapshot. It fixes the problem (when I remove System.setProperty("ebean.transaction.onqueryonly", "COMMIT")).
I looked at the source, and the comment still states "rollback" instead of "commit" as the override.
Regarding the performance hit, you say it's "OK". Does that mean it's "not too bad"? It's not worth the effort to remove all begin/commit statements for on-query-only?
Thanks.

Cool, I'll fix that comment.

It's not worth the effort to remove all begin/commit statements for on-query-only?

The performance hit is relative to the cost of the queries.  If all the transactions just had a single cheap query like (select singleColumn from table where primaryKey = ?) then the relative overhead is high and you should look into removing those "begin"s ... but if those transactions have multiple queries (more complex graphs) or more complex/normal queries then the relative cost is much lower.
So the answer regarding performance is that this would require some time/effort into benchmarking the difference ... but there is a plan for a dashboard and if we had that now we could better answer that (rather than just say "do benchmarking").

Ok, I'm not sure what you mean by "you should look into removing those
"begin"s" since those are issued by ExplicitJdbcTransaction. It seems to me
that neither begin/commit/rollback should be issued for OnQueryOnly.

you should look into removing those "begin"s" since those are issued by
ExplicitJdbcTransaction

Yes, it would be more accurate to say ... you could then move away
from ExplicitJdbcTransaction
to the standard approach.  Given that is pretty much an all or nothing (all
transactions or no transactions for a given EbeanServer instance) that is
unlikely to be an option for you.
On 12 January 2016 at 09:45, dastultz<EMAIL_ADDRESS>wrote:

Ok, I'm not sure what you mean by "you should look into removing those
"begin"s" since those are issued by ExplicitJdbcTransaction. It seems to me
that neither begin/commit/rollback should be issued for OnQueryOnly.
—
Reply to this email directly or view it on GitHub
https://github.com/ebean-orm/avaje-ebeanorm/issues/524#issuecomment-170684520
.

This feature (ExplicitJdbcTransaction) does appear to do what it is intended to, certainly this issue of dangling begin is settled, but it does appear I can't use it. If I start a transaction in JDBC, the commit issued here does commit the transaction and a subsequent rollback does nothing. (Postgresql does not support nested transactions.)
I had suggested that for OnQueryOnly neither of begin or commit should be issued. If this were supported, I'd still have the issue with an "implicit" transaction - I start a Tx in JDBC, insert via Ebean (not in an Ebean Tx), issue a rollback in JDBC, the Tx is already committed by the implicit Tx Ebean creates for the insert.
I totally own all this transaction mixing nonsense, I'm simply not in a position to go "all Ebean". I will have to revert to the trickery I was doing before as it was in fact working. I don't know if you want to maintain ExplicitJdbcTransaction or remove it.
Sorry.

Hmmm, bummer.
> (Postgresql does not support nested transactions.)
In JDBC I'd expect to create save points via
java.sql.Connection.setSavepoint() to achieve nested transactions - I'm not
surprised nested "begin" is insufficient.
>  If this were supported, I'd still have the issue with an "implicit"
transaction - I start a Tx in JDBC, insert via Ebean (not in an Ebean Tx),
issue a rollback in JDBC, the Tx is already committed by the implicit Tx
Ebean creates for the insert.
I'm not sure I follow this bit (external code starts the transaction but
Ebean also thinks it is implicit). I believe externally the transaction is
started (say with "begin") but then the java.sql.Connection is still in the
pool such that Ebean thinks it is implicit.
I'm looking at the
test com.avaje.tests.transaction.TestExplicitTransactionMode. If you wish
to modify or add to that test to reproduce I'm happy to have a look there.
I presume you have investigated the impact of changing to use the methods
on EbeanServer that take a Transaction. In that case there is no implicit
transaction, Ebean will not ever do an implicit commit or rollback etc.
*> I will have to revert to the trickery I was doing before as it was in
fact working. *
Ok. I am happy to keep looking at it a bit more and especially if we can
modify TestExplicitTransactionMode to match your scenario.
Cheers, Rob.

In JDBC I'd expect to create save points via
java.sql.Connection.setSavepoint() to achieve nested transactions

That's interesting, I could look into intercepting begin/commit from Ebean
when in a JDBC transaction and either redirecting to save point logic or
ignoring altogether. So don't remove this feature yet. :-)

I'm looking at the
test com.avaje.tests.transaction.TestExplicitTransactionMode. If you wish
to modify or add to that test to reproduce I'm happy to have a look there.

I'll take a look but it doesn't build for me so that might be tough.

I presume you have investigated the impact of changing to use the methods
on EbeanServer that take a Transaction.

Yes, the challenge is that I have a lot of existing code that would all
need to be reworked. I can't just plug in a Transaction Manager to the
Ebean Server at a high level.
/D

I've decided to produce a DoNothingPreparedStatement when my JDBC connection is already in a transaction and Ebean wants to issue begin, commit, or rollback. There might be a corner case that is problematic, but I've got pretty good unit test coverage of various transaction patterns involving JDBC, Ebean and OpenJPA. So far it's working. (Please keep ExplicitJdbcTransaction. :-) )


Can't connect to random group server

When trying to connect to a random group server, running the bot in Chrome, using the following options:
var options = {
    options: {
        debug: true
    },
    connection: {
        random: "group",
        reconnect: false
    },
    identity: {
        username: "my_bot_name",
        password: "oauth:my_oauth"
    },
    channels: ["#my_channel"]
};

The Chrome console gives me the following errors:

Connecting to a random chat server gives:

The bot seems to log in, and it shows chat messages in the console, however listeners for chat messages don't trigger.

Twitch hasn't released a WebSocket server for Group chat yet, therefore it is not possible to connect to group chat on the browser.
https://github.com/Schmoopiie/tmi.js/wiki/Upgrading-from-twitch-irc-to-tmi.js#group-chat
However, I'll fix the issue you have when connecting to a random chat server.
Thanks!

I forced the protocol to be http instead of file. Keep me updated if it works :smile:

No errors on the chat servers now
Keep up the good work:+1:


Api feature

Used the  external API: API Ninjas
Created a new component for displaying a quote
Fetched data inside of the new component
Showed a loading state
Showed an error state
Imported the new component into the main component (next to the Calculator)
It is on the same page as the Calculator

@Cabraham1  Could you kindly point to me the part of the project requirement that states that the map method must be used? It seems I missed it.


Test case for bug #272

Should be moved to https://github.com/yiisoft/user

Pull requests can't be moved. Closing it here. Linked from the issue itself.


Make the indexToPoint and pointToIndex methods more generally available

Currently hidden under RogueTerminalEmulator.

Technically these functions live in two places (RogueTerminalEmulator, GridSquare), which I think is a sure sign that I'm not sure where they ought to live. Going to wait until I merge in the ui work and see how I feel then.

Closing for now.


 opens in current tab, but it seems like it should not

Oni Version: 0.3.4
Neovim Version (Linux only): 0.2.3-843-gf5b0f5e
Operating System: Ubuntu 16.04.3 LTS
Describe your issue
When hitting f12 while the cursor is on a symbol, it goes to that symbol's definition, but in the same tab as the one currently being opened.
Expected behaviour
This was bugging me (because i felt like it should have been going to those definitions in a new tab, much more practical imo), so i had a little peek in the code, and found this: https://github.com/onivim/oni/blob/master/browser/src/Editor/NeovimEditor/Definition.ts#L53
Based on the Enum definition here: https://github.com/onivim/oni/blob/master/browser/src/Editor/NeovimEditor/Definition.ts#L13, it looks like f12 should be opening in a new tab, but is not.
Steps to reproduce
Get your cursor over a symbol usage, then hit f12
Proposed solution
I feel like solving this might be quite easy: just change this line (https://github.com/onivim/oni/blob/master/browser/src/Editor/NeovimEditor/Definition.ts#L59-L60) to:
default:
  return 'tabe'

Makes sense to me! I've hit this before as well.

I feel like solving this might be quite easy: just change this line (https://github.com/onivim/oni/blob/master/browser/src/Editor/NeovimEditor/Definition.ts#L59-L60) to:

I think that would fix it for people who are using vim style tabs, but would potentally be awkward for people who use buffers option for their tabs. (That is every open buffer has its own tab).
We'd instead probably want to use this function : https://github.com/onivim/oni/blob/a34e1d64ecf529f2d31f5f0bbb9cc33c4d19980e/browser/src/Editor/NeovimEditor/NeovimEditor.tsx#L882-L905
Since it deals with that for us and does tabs for most users, and buffers for those who don't.

Good point.
I did try to test it out, but had some troubles making my build work, I'll give it a fair shot when I'm not at work

Actually, I like the current behavior - so will you mind, putting this
change under a togglable config?
On Thu, May 31, 2018, 19:58 Matthieu ARENE<EMAIL_ADDRESS>wrote:

Good point.
I did try to test it out, but had some troubles making my build work, I'll
give it a fair shot when I'm not at work
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/onivim/oni/issues/2272#issuecomment-393601253, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AE3Ijd-0RAmK9ME4EYjUa03hnVkGu7Qhks5t4CFQgaJpZM4UVE6f
.

If i ever get time to change that, I'll make sure to make it togglable, yes :)

@psxpaul Does #2504 fix this? If so, how would one configure it?

Yes, @badosu this should be fixed with my change. The default language.gotoDefinition command should use Oni.DefaultFileOpenOptions to determine where to open. You can change this by adding to your config.tsx:
Oni.DefaultFileOpenOptions.openMode = Oni.FileOpenMode.NewTab
The valid values are Edit, NewTab, VerticalSplit, HorizontalSplit, and ExistingTab`. My change also adds the following commands, which should behave as you expect (equivalent horizontal and vertical commands already exist):

language.gotoDefinition.openNewTab
language.gotoDefinition.openEdit
language.gotoDefinition.openExistingTab


Cannot create Experience Lab (tried Europe West and North)

I am using the Deploy to Azure button with Arm template. Had three tries that always ends with same error. I am using the SET Subcription that is whitelisted.
Getting error on dataLoad deployment.
 "code": "InvalidValuesForRequestParameters",
        "message": "Values for request parameters are invalid: signedExpiry."
See also attached image.

Jon Olav Abeland
<EMAIL_ADDRESS>

Thanks for submitting this. I've found the issue and should have a fix completed soon.


support default hostport connection info

There are cases where this might be deployed in an environment with only a single game server that needs to be interfaced with.
Supporting an environment variable to populate this, if not provided by the client would simplify api calls.

addressed in https://github.com/holysoles/ginrcon/commit/0269d50b4ede3907267329f2c06b921ddd186e81


CALENDAR GUI

Calendar GUI
Brief -A Calendar GUI using python and tkinter
Instructions

Create a new folder for your script and file/folder name should be appropriate.
Create a README.md in your folder for program Instructions
add requirements.txt if needed

Programming Language

[x] Python

Happy Coding

Can you please assign me this issue? I can get it done.

Sure go ahead :) you can get some idea from #679


Documentation for implementing your own EnvironmentRepository

I'm in the middle of implementing a new EnvironmentRepository and so far it's been easy enough extending from the base class and looking at the examples of existing implementations.
One issue I'm having is finding a way to add/extend spring-cloud-config-monitor so that my new EnvironmentRepository can also use the /monitor endpoint to refresh and automatically notify the consumers about new configuration. I tried creating a new BasePropertyPathNotificationExtractor but so far I can't get things wired so that the PropertyPathEndpoint can be called.
Documentation on how this is done or a sample bit of code would be extremely helpful to those extending spring-cloud-config. I suppose I can just create my own /my-monitor endpoint and publish the event like so?
this.applicationEventPublisher.publishEvent(new RefreshRemoteApplicationEvent(this, this.busId, service));

So you are creating a Bean for your BasePropertyPathNotificationExctractor and its not present in the list here?
https://github.com/spring-cloud/spring-cloud-config/blob/9c695648be6a88c232ed7792759fc9adfad82c2d/spring-cloud-config-monitor/src/main/java/org/springframework/cloud/config/monitor/EnvironmentMonitorAutoConfiguration.java#L54

Yes. It looks like I was able to register my extractor effectively, the issue might be further down the path. Now debugging the notifyByPath and the bus configuration.

OK let me know.

Ok I think I know what's happening, but I can't figure out the correct way of solving it.
My new NewDogPropertyPathNotificationExtractor is correctly added. The PropertyPathEndpoint's notifyByPath method correctly loops around and picks up my new extractor, the problem is that the new extractor uses BasePropertyPathNotificationExtractor's extract method that looks to see if the request contains a commits.
The update to the /monitor endpoint has no such information. Should I overwrite the extract method on my implementation or is there a way to trigger the refresh in another way (besides poking at the /monitor endpoint)?

I would just override the extract method in your implementation

If wanted to override the extractor to fire off events to a dog-service (so that it can perform its context refresh and ask for its configuration again) what would I need to return as a PropertyPathNotification?  So far I've made sure that the PropertyPathNotification has "dog-service" in its path. This made the guessServiceName populate the services set with: ["dog:service", "dog-service"] this triggered the following logs:
MyCustomPropertyPathNotificationExtractor : paths [dog-service]
 o.s.c.c.monitor.PropertyPathEndpoint.    : Refresh for: dog:service
 o.s.c.s.m.DirectWithAttributesChannel.   : Channel 'unknown.channel.name' has 1 subscriber(s).
 o.s.cloud.bus.event.RefreshListener.     : Received remote refresh request.
 o.s.cloud.bus.event.RefreshListener      : Refresh not performed, the event was targeting dog:service:**
 o.s.c.c.monitor.PropertyPathEndpoint     : Refresh for: dog-service
 o.s.cloud.bus.event.RefreshListener      : Received remote refresh request.
 o.s.cloud.bus.event.RefreshListener      : Refresh not performed, the event was targeting dog-service:**
c.g.c.a.l.MyCustomEnvironmentRepository   : Fetching configuration for application: dog-service, profile: kubernetes,development, label: null

I don't see where the spring-cloud-monitor would take the refresh event and try to call the implementation to fire a refresh event into the service bus. Can you point me to the part of the code that does this? Do I need to craft the path in a specific way to make sure the Config Service fires the event?

Actually I think its working!
Logs on the dog-service:
o.s.c.c.c.ConfigServerConfigDataLoader : Fetching config from server at : http://config-service:8080
o.s.c.c.c.ConfigServerConfigDataLoader : Located environment: name=dog-service, profiles=[kubernetes,development], label=null, version=null, state=null
o.s.cloud.bus.event.RefreshListener : Keys refreshed []
o.s.c.s.binder.DefaultBinderFactory : Retrieving cached binder: pubsub

My only concern is the double services caused by the guessServiceName

I would have to debug something to understand why there are two of the same service names in the list.  If you can provide a sample that would help.

I'm calling the /monitor endpoint like this:
curl -X POST http://localhost:8080/monitor -H "X-SOME-Event: push" -H "Content-Type: application/json" -d '{"updates":["dog-service"]}'
My extractor looks like this:
public class MyPropertyPathNotificationExtractor extends BasePropertyPathNotificationExtractor {

  @Override
  public PropertyPathNotification extract(
      MultiValueMap<String, String> headers, Map<String, Object> request) {
    if (Objects.nonNull(request) && request.get("updates") instanceof Collection) {
      Set<String> paths = new LinkedHashSet<>();
      List<String> updates = (List<String>) request.get("updates");
      if (!updates.isEmpty()) {
        paths.addAll(updates);
        log.info("paths {}", paths);
        return new PropertyPathNotification(paths.toArray(new String[0]));
      }
    }
    return null;
  }

  @Override
  protected boolean requestBelongsToMyRepoManager(MultiValueMap<String, String> headers) {
    log.info("Refreshing repo");
    return "push".equals(headers.getFirst("X-SOME-Event"));
  }
}

Basically the MyPropertyPathNotificationExtractor returns a PropertyPathNotification that has "dog-service" it its paths array.
From there the guessServiceName somehow adds ["dog:service", "dog-service"]. This is because in:
https://github.com/spring-cloud/spring-cloud-config/blob/8493efb8457b4107c4bfa63a0fc45b0ba6b98756/spring-cloud-config-monitor/src/main/java/org/springframework/cloud/config/monitor/PropertyPathEndpoint.java#L105
it goes into the while loop and then into the:
if ("application".equals(name)) {
    services.add("*:" + profile);
}

Since our applications are running in K8s we named them foo-service i.e cat-service, dog-service, duck-service, etc.

Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open the issue.


No module named 'botbuilder.ai.qna.dialogs' - Python QnA Sample 49

Version
botbuilder-ai - 4.9.1
Describe the bug
I was trying out the QnA Maker Sample - 49.qnamaker-all-features . I've configured my QnA KB and also the config.py with the necessary info. However the module botbuilder.ai.qna.dialogs does not seem to exist. I've manually verified for the class QnAMakermDialog and it does not exist

from botbuilder.ai.qna.dialogs import QnAMakermDialog

To Reproduce
Steps to reproduce the behavior:

Download the sample 49.qnamaker-all-features
Install the necessary requirements and configure QnAMaker.
Run python app.py in the folder

Expected behavior
The sample should've run successfully.
[bug]

@vijaysaimutyala Where are you seeing that import line?  The actual class name should be QnAMakerDialog (not QnAMakermDialog).

@tracyboehrer Sorry that might be a typo. You can look the import here I rectified the typo and still couldn't see the module. Below is the screenshot from my local env. I'm somehow missing the dialogs part I see on the Botbuilder-python repo

I tried updating botbuilder-ai, but its already at 4.9.1

A quick update. I created a fresh conda environment and installed the requirements of the sample 49 but I faced the same issue again. Also I noticed on the Samples homepage that the Python sample link is not updated yet.

@vijaysaimutyala This is a bug on our side.  We didn't provide an export for the dialogs package.

Thanks for checking @tracyboehrer Let me know if I can help in any way.

@vijaysaimutyala We are making the correction in the ai.qna package.  But it will require a patch release (4.9.2).  We are working though the timing on that.  This issue will automatically close out when the fix is merged though.

The fix is in our next planned release. Our next release is on ~Aug 15th, but it might slide a day or so depending.


[Feature Request] The ability to seek within a song

Would you be able to add the ability to seek within a song?
e.g. "!seek 1:00" which would make the bot skip to that point on the song.
Alternatively, would someone be able to tell me how to do this if it's already a feature?
I can't seem to find it anywhere.
Thanks!

#469

saw someone else had requested this, they just didn't use the word "seek" which is what I searched for, my bad


klint adds a line break after block assignment regardless of max_line_length

Expected Behavior
Block assignment should not break if it fits on its own line, based on max_line_length:
configure<GitPropertiesPluginExtension> {
  keys = buildList {
    add("git.branch")
    add("git.commit.id")
  }
}

Observed Behavior
klint adds a line break right after the assignment:
configure<GitPropertiesPluginExtension> {
  keys =
    buildList {
      add("git.branch")
      add("git.commit.id")
    }
}

NOTE: I'm using the Spotless plugin in a Gradle project. I haven't found how to configure --log-level, but I've raised an issue in the Spotless repository to see if that could be added.

Steps to Reproduce
Have Spotless (or probably just klint) installed and configured. In my case, I'm not using any additional configuration. In a Gradle build script paste this:
val keys = buildList {
  add("git.branch")
  add("git.commit.id")
}

And then run ./gradlew :spotlessApply
Environment
My .editorconfig looks like:
root = true

[*]
continuation_indent_size = 4
indent_size = 2
indent_style = space
max_line_length = 120

# Keep these unchanged
charset = utf-8
end_of_line = lf
insert_final_newline = true
trim_trailing_whitespace = true

Spotless is configured this way:
configure<SpotlessExtension> {
  encoding = StandardCharsets.UTF_8
  kotlinGradle {
    endWithNewline()
    indentWithSpaces(2)
    ktlint("1.1.0")
    trimTrailingWhitespace()
  }
  lineEndings = LineEnding.UNIX
}

I'm using Gradle 8.5 under Linux, and klint is set at version 1.1.0.

This behavior is caused by the multiline-expression rule. You can disable it in your .editorconfig.

Tried to locate multiline-expression but couldn't find it — the closest one is multiline-expression-wrapping.
Provided that's the correct one, I went ahead and placed this in the end of the .editorconfig file:
...

[*.{kt,kts}]
ktlint_standard_multiline-expression-wrapping = disabled

But still, it's doing exactly the same :grimacing:
In any case, if you don't think this shouldn't be the default behavior, feel free to close this issue. I would think that wrapping will make the most sense when lines are too long based on the defined max_line_length — with the few exceptions of some constructs that are always wrapped, like enum values, class annotations, etc.
Thanks for maintaining klint! :wave:

Tried to locate multiline-expression but couldn't find it — the closest one is multiline-expression-wrapping.

Yes, this is correct.

NOTE: I'm using the Spotless plugin in a Gradle project. I haven't found how to configure --log-level, but I've raised an issue in the Spotless repository to see if that could be added.

This could be a spotless problem. I don't know how they integrate with Ktlint. But with ktlint CLI it works, but you need to disable another rule as well because of rule dependencies:
root = true

[*.{kt,kts}]
ktlint_standard_multiline-expression-wrapping = disabled
ktlint_standard_string-template-indent = disabled

It works when the new rule, ktlint_standard_string-template-indent, is disabled in .editorconfig as well.
Thanks, @paul-dingemans!


update var to use key from inventory file

/assign @Atoms
Can you take a look if this is correct please ?


Add support for alternative emails

Is your feature request related to a problem? Please describe.
Some organizations use different emails for SDM and Slack. For those cases, we might want to use custom fields for alternate emails
Describe the solution you'd like
Add an environment variable that can be set for specifying a custom field to be used as the user email.
Additional context
User need registered in ticket 3064

The port to 1.0.x was cancelled, please refer to PR #130 for further details


webdataset.github.io cache / streaming info incorrect

https://webdataset.github.io/webdataset/desktop/ states:
"Note that caching happens in parallel with iterating through the dataset. This means that if you write a WebDataset-based I/O pipeline, training starts immediately; the training job does not have to wait for any shards to download first."
This doesn't seem to be accurate anymore:
https://github.com/webdataset/webdataset/issues/212#issuecomment-1360436384
"If you use caching, each individual shard is first downloaded completely, then the local copy of the shard is used as input for further reading. (Originally, we had caching and processing happening in parallel, but that makes error handling tricky.)"

Thanks, I'll update that.


[Feature request] Loading the model once for all

One of the advantage of an application that keeps running (like with a server) is that you can load the model once in the memory and make several pictures generations without having to load the model again in the memory.
This saves time.
So i would like to know if there is already a way to generate several little videos without having to reload the model again in the memory and if not (my feature request).
Make an API server that can accept requests from other software.

it is my version of the software or ???
I get this answer:
" No such option: -r"
How to use that ?

It works...
There is just one problem: the things seem to be generated with the same seed.
This makes this option pretty useless.
I think this is a bug.

yes but why include a --repeats command, when you can just have several seeds in the JSON (implied: i want to generate the content with these seeds) ?


Wrong Visual C++ install path set by kit

Brief Issue Summary
Kit detection seems to pick up wrong Visual C++ install path. The COM API might provide with the wrong version number.
Expected:

Select Kit
Work

Apparent Behavior:

Select Kit

[cmake] The CXX compiler identification is unknown
[cmake] CMake Error at CMakeLists.txt:5 (project):
[cmake]   The CMAKE_CXX_COMPILER:
[cmake]
[cmake]     C:/Kellekek/Microsoft/Visual Studio/Preview/VC/Tools/MSVC/14.13.26126/bin/Hostx64/x64/cl.exe
[cmake]
[cmake]   is not a full path to an existing compiler tool.
[cmake]
[cmake]   Tell CMake where to find the compiler by setting either the environment
[cmake]   variable "CXX" or the CMake cache entry CMAKE_CXX_COMPILER to the full path
[cmake]   to the compiler, or to the compiler name if it is in the PATH.
[cmake]
[cmake]
[cmake] Configuring incomplete, errors occurred!

CMake Tools Log
[cmake] The CXX compiler identification is unknown
[cmake] CMake Error at CMakeLists.txt:5 (project):
[cmake]   The CMAKE_CXX_COMPILER:
[cmake]
[cmake]     C:/Kellekek/Microsoft/Visual Studio/Preview/VC/Tools/MSVC/14.13.26126/bin/Hostx64/x64/cl.exe
[cmake]
[cmake]   is not a full path to an existing compiler tool.
[cmake]
[cmake]   Tell CMake where to find the compiler by setting either the environment
[cmake]   variable "CXX" or the CMake cache entry CMAKE_CXX_COMPILER to the full path
[cmake]   to the compiler, or to the compiler name if it is in the PATH.
[cmake]
[cmake]
[cmake] Configuring incomplete, errors occurred!

Visual Studio 15.9-preview4 fixed the issue.


Ability to fetch collection data from elasticsearch as a tree

Issue
https://github.com/wellcomecollection/platform/issues/4378
Description
This provides the ability to retrieve collection tree data from elasticsearch.
The functionality is not yet added to the API, as we will need to make a decision how this is provided (whether via a new endpoint or an existing one etc)

Having another look at this now.

@alexwlchan oops just saw your message! let me know if you have any changes and will put in a new PR


Chinese project name can not cordova build ios, but the English name of the project can cordova build ios

** BUILD FAILED **
The following build commands failed:
CompileC build/中国.build/Debug-iphonesimulator/中国.build/Objects-normal/i386/HCPLDPlugin.o 中国/Plugins/cordova-hot-code-push-local-dev-addon/HCPLDPlugin.m normal i386 objective-c com.apple.compilers.llvm.clang.1_0.compiler
Chinese project name can not cordova build ios, but the English name of the project can cordova build ios

I see it fails while building local-dev-addon. Have you tried to build the project without it? Just with chcp plugin?

cordova create china m.test.com 中国
cd china
cordova platform add ios
cordova plugin add cordova-hot-code-push-plugin
cordova plugin add cordova-hot-code-push-local-dev-addon
cordova build ios
＃The above will get an error
－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－
cordova create china m.test.com china
cd china
cordova platform add ios
cordova plugin add cordova-hot-code-push-plugin
cordova plugin add cordova-hot-code-push-local-dev-addon
cordova build ios
＃The above is normal, because it is the English name

Thanks for the way to reproduce it.
Issue lies in cordova-hot-code-push-local-dev-addon. If you remove it - project will compile just fine:
cordova create china m.test.com 中国
cd china
cordova platform add ios
cordova plugin add cordova-hot-code-push-plugin
cordova build ios

cordova-hot-code-push-local-dev-addon uses Swift code, and for some reason plugin can't inject bridging header into the project because of the chinese characters. Created an issue in that repo: https://github.com/nordnet/cordova-hot-code-push-local-dev-addon/issues/11 . Discussions should be continued in there.
Right now I don't have a fix for local-dev add-on. I think there are only two options now:

If you want to use it for development - rename project to English. Just for development time. When it's time to release - you will remove local-dev add-on anyway, and will rename the project. So it should build and work just fine.
Don't use cordova-hot-code-push-local-dev-addon at all, just the main plugin.

Will close this one, since the issue with the other plugin.


How do I disable controls?

Describe your problem here.
I want to make it so that, when you don’t dodge a projectile, it disables your controls for a bit. I’ve got the dodging part down, but I don’t know how to disable the player’s controls. Can anybody show me how?
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
No.

If you mean making the player not be able to hit notes you should do (im hoping you already know how to make a part of a script only work for a certain arrow type) setProperty('boyfriend.stunned', true/false)

Thank you! It worked perfectly!


Use patched go-git version

To work around object not found errors due to a newly introduced bug
in go-git v5.4.0.
Image available for testing: hiddeco/image-automation-controller:go-git-patch-776e182
May contain fix for #172

To test this on a cluster bootstrapped with Flux, change the image in your ./clusters/<name>/flux-system/kustomization.yaml like so:
apiVersion: kustomize.config.k8s.io/v1beta1
kind: Kustomization
resources:
  - gotk-components.yaml
  - gotk-sync.yaml
images:
- name: ghcr.io/fluxcd/image-automation-controller
  newName: docker.io/hiddeco/image-automation-controller
  newTag: go-git-patch-776e182

Closing in favour of: #174


Failed to legalize operation 'eir.logical.or'

failed to legalize operation 'eir.logical.or'

Affects the following files in OTP:

erts/preloaded/src/erts_dirty_process_signal_handler.erl
lib/et/src/et_viewer.erl
lib/inets/src/http_lib/http_response.erl

Fixed in #582


v0.2.9 is a branch and a tag

There is both a branch and a tag named v0.2.9.
In nixpkgs, this make it impossible to download the source archive for the tag via fetchFromGithub.
Could the branch be renamed or removed?

Deleted,

:+1:


failing test for View.add RecursionError

I hit this while writing fusion UPats in big graph #7556.
No asserts pre symbolic deletion 518c022c29104d79c7a50ec41af5b7e6404da317g.

i can check this one, also there's test_symbolic_shapetracker


Parallel scans

We sequentialise scan at the moment.  Clearly this won't do in the long run.  We need scans for parallelising all three levels of the CalibVolDiff benchmark.

Working on this for now.  Trying to figure out how to implement an inclusive variant of Blelloch's scan algorithm.

I cannot think of a way to do this that does not involve recursion.  How annoying.

Okay, this is the way it's gonna be (unless Cosmin tells me that it's stupid):

The input array has N elements.

Determine a workgroup size K and a number of workgroups W.  These
values are not related to N at all, but are based on the GPU (and
possibly the kernel itself).  This is the same thing we do for
reductions.  For now, let us assume that W is less than the maximum
workgroup size.

Each workgroup performs an inclusive scan on N / (K * W) elements,
producing N / K * W elements as output.  We now have an array
where each K*W-size chunk has been scanned.

For each chunk, take the last element.  This results in a size-W
array.  Compute the exclusive scan of this array.  This can be
done using a single workgroup if W is less than the maximum
workgroup size.

We now have a size-W array of "offsets".  Call this array
OFFSETS.  For any chunk i, use the scan operator to combine
OFFSETS[i] with all elements in the chunk.  This is just a map.

Enjoy your new inclusively scanned array.

Possible issues:

In step (2), each K-thread workgroup must scan N / (K * W)
elements.  This means every thread will have to process several
elements - most of the optimised GPU scan implementations I have
seen do not do this, which makes me uneasy.  Still, this should only
happen for large inputs, in which case we should have enough work to
saturate the hardware.  We must be careful to prevent branch
divergence.

Step (2) calls for an inclusive scan, but Blelloch's algorithm is an
exclusive scan.

I suppose that we can let W be greater than the maximum workgroup
size by sequentialising in step (3).  The most important thing is
that step (3) can be done within a single workgroup (or in
principle, a statically known number), as we otherwise end up with
recursion.

Step (3) can get away with doing an inclusive scan if we index with
i-1 (and special-case the first chunk) in step (4).

--
\  Troels
/\ Henriksen

Okay, this is the way it's gonna be (unless Cosmin tells me that it's stupid):

The input array has N elements.

Determine a workgroup size K and a number of workgroups W.  These
values are not related to N at all, but are based on the GPU (and
possibly the kernel itself).  This is the same thing we do for
reductions.  For now, let us assume that W is less than the maximum
workgroup size.

Each workgroup performs an inclusive scan on N / (K * W) elements,
producing N / K * W elements as output.  We now have an array
where each K*W-size chunk has been scanned.

For each chunk, take the last element.  This results in a size-W
array.  Compute the exclusive scan of this array.  This can be
done using a single workgroup if W is less than the maximum
workgroup size.

We now have a size-W array of "offsets".  Call this array
OFFSETS.  For any chunk i, use the scan operator to combine
OFFSETS[i] with all elements in the chunk.  This is just a map.

Enjoy your new inclusively scanned array.

Possible issues:

In step (2), each K-thread workgroup must scan N / (K * W)
elements.  This means every thread will have to process several
elements - most of the optimised GPU scan implementations I have
seen do not do this, which makes me uneasy.  Still, this should only
happen for large inputs, in which case we should have enough work to
saturate the hardware.  We must be careful to prevent branch
divergence.

Step (2) calls for an inclusive scan, but Blelloch's algorithm is an
exclusive scan.

I suppose that we can let W be greater than the maximum workgroup
size by sequentialising in step (3).  The most important thing is
that step (3) can be done within a single workgroup (or in
principle, a statically known number), as we otherwise end up with
recursion.

Step (3) can get away with doing an inclusive scan if we index with
i-1 (and special-case the first chunk) in step (4).

Cool stuff. This would also mean that if someone where to implement a flattening transformation that assumes parallel scans are fast, it would probably work :smile:

Yes!  Although I think you will need segmented scans too.  Do you intend to finish your work on flattening?

First succesful parallel scan executed.  This program (tests/soacs/scan1.fut):
fun [int] main([int] a) = scan(+, 0, a)

With an input size of 10.
It is not... great.  We assume that entire blocks of the input will fit in shared memory.  But it will suffice, I hope.  Currently, multidimensional scans are not supported.

Now tested with an input size of 3000 - still works.  (After fixing it.)

Works well for scalars now.  Still a little uncoalesced access going on.


[2.0-wip] Code Docs

Plz have a look at the docs for the code blocks. They need some style fix and a better documentation.

The styles look fine to me and as intended. I've added rendered examples of the code snippets though.


Feature/374 - Metadata Schema Properties - Extra Metadata Properties and Blacklist Metadata Properties

Added ability to specify, at a global level, via OSGi configuration

Extra metadata properties - this allows for the addition of properties that may not make sense to show up in the Metadata Schema (even as Read-only)
Blacklisted metadata properties - this allows for listing properties that should not show up as available Metadata schema properties to select from.

+1


Typos

StorPortInitilize --> You may want to fix the typos.

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

ID: f1e290fc-6386-21b8-0af3-242acb582a8e
Version Independent ID: 249f0dd4-053f-e9e0-e0a7-ba8a2308a4af
Content: StorPortInitialize function (storport.h) - Windows drivers
Content Source: wdk-ddi-src/content/storport/nf-storport-storportinitialize.md
Product: windows-hardware
Technology: windows-devices
GitHub Login: @lorihollasch
Microsoft Alias: loriwhip

Thanks for catching this, @imtiazdc  - I have fixed the typos!