Using with vim

Hi,
I tried to add fzf to vim via Vim Plug as described, but only :FZF! works.
:FZF simply does nothing (I am using tmux), neither are Open in splits mappings.
What am I doing wrong?
Also on a side note, I couldn't find a way to supply a list of ignored files and folders. Is there a way to ignore certain patterns?
Thanks.

Does fzf-tmux or fzf command work on your shell?

Also on a side note, I couldn't find a way to supply a list of ignored files and folders. Is there a way to ignore certain patterns?

fzf is a command line filter that can be used with anything, so it doesn't have the notion of ignored files and folders.
ps -ef | fzf
seq 1 100 | fzf

When the input is not specified (fzf w/o STDIN pipe), it will use find command as the default input. You can set $FZF_DEFAULT_OPTS environment variable to a command that lists the files as you like.

Yes both fzf and fzf-tmux work, and :FZF! works fine. I have copied the whole FZF part of (your config)[https://github.com/junegunn/dotfiles/blob/da378217ad008d422bc5b577802cad237a2930e1/vimrc#L1196-L1198] but no shortcuts do anything.

Hmm looks like they do work when not in tmux session. If that helps.

Can you try :call system('seq 100 | fzf-tmux') in vim on tmux session. Does it open a tmux pane below?
By the way, "Open in splits" mappings in the wiki page is no longer needed. As :FZF can now do it with CTRL-X/V/T.

:call system('seq 100 | fzf-tmux') does nothing

How about

:echo system('which fzf')
:echo system('which fzf-tmux')

Hmm... both not found.

Okay, create a new tmux window (which will load ~/.bashrc or ~/.zshrc), open vim there and try again.

I have added '/Users/me/.fzf/bin' to '~/.zshenv', that made it work. I guess it is better to install FZF with Homebrew rather than vim plugin, is it?

I have added '/Users/me/.fzf/bin' to '~/.zshenv', that made it work

No don't do that. The install script will download fzf binary and inject code to your ~/.zshrc that extends your $PATH, so manual update of your env files is not required. The reason :FZF! worked and :FZF didn't is that fzf-tmux script expects fzf to be in $PATH, but it wasn't as the updated ~/.zshrc was loaded on the tmux window you're running vim. As I mentioned earlier, if you open another window, the shell will have fzf and fzf-tmux in $PATH. So vim started in the window should run without a problem. I know this can be confusing, so I'll push a commit that makes fzf-tmux try to use fzf binary in the same directory if it's not found in $PATH.

I guess it is better to install FZF with Homebrew rather than vim plugin, is it?

No, not necessarily. I use vim-plug to manage fzf.

I opened a new Terminal window, killed tmux server and everything. It just didn't work until I've changed .zshenv

Do you see [ -f ~/.fzf.zsh ] && source ~/.fzf.zsh in your .zshrc?

You said:

Yes both fzf and fzf-tmux work

But you also said:

Hmm... both not found.

when I asked you about :echo system('which fzf'). I'm not sure what's going on.

[ -f ~/.fzf.zsh ] && source ~/.fzf.zsh in .zshrc - yes.
But as far as I know vim uses different shell that bypasses .zshrc or something. That's why I have a duplicate of $PATH in .zshenv.
fzf and fzf-tmux did work from terminal, just not from Vim, adding path to .zshenv fixed that.

fzf and fzf-tmux did work from terminal, just not from Vim, adding path to .zshenv fixed that.

If they work from terminal, and you started Vim from that shell, they should also work in Vim which is a subprocess of the shell. You'll notice that $PATH is exported inside ~/.fzf.zsh.
Anyway, with the previous commit, now both :FZF and :FZF! should run on a shell where fzf is not (yet) in $PATH. Thanks for the report.


Concat operator supports concating at dimension zero?

Hi
I noticed that concat operator only support concating at dimension one currently. For RNN, concating output of LSTMs before Softmax (i.e. concat all the batches from different locations among a sentence) will speed up the training process substantially.
Thanks;

Yes, this can be done by extending the https://github.com/dmlc/mxnet/blob/master/src/operator/channel_op_common.h to add a concat dimension argument.
Note that mshadow's concat and slice support along arbitrary dimension. But we need to add it :)
Are you interested in contributing? Also note although mshadow is templated, we can either enumerate dim with template, or do some reshape trick, to reshape to a 3D tensor and concat on dim 1. So it is general for any dimension

Sure, I'll try
Thanks;

thanks to @yang1fan2 this is now supported

#650


Add integration tests for self register

Added integration test cases to create self registered users using the REST-API

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/3034514673

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/3034514673
Status: failure

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/3042238466

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/3042238466
Status: failure

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/3044272702

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/3044272702
Status: failure

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/3046221064

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/3046221064
Status: failure

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/3050334991

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/3050334991
Status: success


[add-resource] New Theme

Hello ^^
I would like to add my new theme to the Grav Repository.
Here are the project details: https://github.com/koca/grav-theme-masonry
Thanks.

You will need to create a release within GitHub before it shows up in GPM.
https://help.github.com/articles/creating-releases/
The version number you tag the release with should match the version number in the blueprints.yaml

Alright, done :)

And now showing up in GPM.


View counter isn't aligned properly after YouTube's new design update.

With script enabled:

With script disabled:

I am using the 64-bit version of Firefox with the 46.0.1 update on Windows 7 64-bit.

same thing, hope project isn't dead so smb can fix this

Same

Fixed it by dragging all elements to the top layer, from the UI > Placement section in the settings. Using Chrome, hopefully works for you too. Cheers.

Thanks, that seem to have fixed the issue.

It does look a bit weird since the view counter is supposed to be above the line, but it'll do for now.

any idea for firefox?

If you're using the userscript version, I found a solution if you're willing to edit it. Go to line 3097 and delete the part that says #watch7-views-info{top:-3px!important;bottom:initial!important}. Save and reload the YT page for it to take effect. Works for me on Firefox 47b5

thanks, works, here is .xpi, if somebody need it
YouTubeCenter.zip


Ensure SNI flag set on multi-site hostname listener (Azure#426)

Closing in favor of https://github.com/Azure/application-gateway-kubernetes-ingress/pull/516


In sync nodes periodically go Offline and restart - with connected peers

It seems to be fairly widespread and persistent - I have nodes that are in sync and running fine, then go offline. Several others also experiencing per discord. Logs from two nodes and 5 incidents shared to logs@o1labs in google drive:
https://drive.google.com/drive/folders/1QAy7hQV7h_cUNgIJD5Ug3uS-qtttS8Sv?usp=sharing
I have also 1) captured netstat to show connections are open, and 2) started capturing coda client status every 5 mins just to see that nodes are going offline and restarting as expected (they are.)
Node 1 was 16vcpu/64G dedicated digital ocean (cpu peaks around 80%)
Node 2 was 32vcpu/64G dedicated digital ocean running only snark worker (cpu never about ~45%)
I have tried upping -max-connections to 100, and also -enable-peer-exchange true -- to no effect.
Other Issues that are about Nodes offline, but do not seem to be the same, listed in case related. Both are from prior testnet.
https://github.com/MinaProtocol/mina/issues/6882 - seems different because there is not a syncing problem.
https://github.com/MinaProtocol/mina/issues/6819 - may be related but insufficient information provided.

Likely related to https://github.com/MinaProtocol/mina/issues/7293

Likely related to https://github.com/MinaProtocol/mina/issues/7293

I am also experiencing this issue on all of my nodes

I am also experiencing this issue on all of my nodes

The same.

The same.

One of node thinks it is Synced at height 2.

One of node thinks it is Synced at height 2.

New logs captured with: attempt to proactively add peers when offline detected, and netstat while offline
Logs created using exported-logs here (shared w/<EMAIL_ADDRESS>The folder also includes 1) watchdog.txt generated by https://github.com/jrwashburn/mina-utils/blob/main/status-watchdog.sh which is what attempted to add new peers, 2) coda client status logged every 5 minutes, 3) netstat captured while in offline state.
https://drive.google.com/drive/folders/1YROXKQTyLd-BPNLRgHY04fJn5p3geSws?usp=sharing

New logs captured with: attempt to proactively add peers when offline detected, and netstat while offline
Logs created using exported-logs here (shared w/<EMAIL_ADDRESS>The folder also includes 1) watchdog.txt generated by https://github.com/jrwashburn/mina-utils/blob/main/status-watchdog.sh which is what attempted to add new peers, 2) coda client status logged every 5 minutes, 3) netstat captured while in offline state.
https://drive.google.com/drive/folders/1YROXKQTyLd-BPNLRgHY04fJn5p3geSws?usp=sharing

Same here, we've tried a bunch of scenarios for BP nodes (8CPU/ 32 RAM, 4 CPU / 64RAM etc), also a couple of flags such as -enable-flooding true -enable-peer-exchange true -max-connections 100.
We were thinking of running a bunch of BP nodes, then selecting those who will be in sync, then using isolate-network flag to connect to such sync nodes (ideally via internal cloud provider IP to minimize network overhead). But it won't be a blockchain then ...
I'm not an expert in blockchains but my gut feeling is that the more stable nodes will be present in blockchain the easier for new nodes to sync. In our case, lots of people have issues with syncing right now thus keeping the network in a shaken state.

Same here, we've tried a bunch of scenarios for BP nodes (8CPU/ 32 RAM, 4 CPU / 64RAM etc), also a couple of flags such as -enable-flooding true -enable-peer-exchange true -max-connections 100.
We were thinking of running a bunch of BP nodes, then selecting those who will be in sync, then using isolate-network flag to connect to such sync nodes (ideally via internal cloud provider IP to minimize network overhead). But it won't be a blockchain then ...
I'm not an expert in blockchains but my gut feeling is that the more stable nodes will be present in blockchain the easier for new nodes to sync. In our case, lots of people have issues with syncing right now thus keeping the network in a shaken state.

same....

same....

This can be closed; new versions are stable and this is not occurring.


Add h2spec to CI.

It would be really nice if one of the CI tests we ran involved running h2spec. This is a moderately tricky thing to do because it means we need to expand one of our example servers to be good enough to pass h2spec, but that's not the end of the world.

Done


strings.xml

Hello. How can i add a new images into strings.xml? Any sample for beginers?

No clue, can't remember. Haven't worked on this project for ages. I don't recommend using it for anything. Should probably update the readme to say it's deprecated. Maybe use Frames instead, since it's a looooot more recent.

 Closing.


hello ,how can i do it?

In PlatformIO by VSCodium ,I build blink is worked.
But ,when build this project, I got this:
src/Servo.h: In function 'void startServo()':
src/Servo.h:9:5: error: 'servo1' was not declared in this scope
9 |     servo1.attach(SERVO1_PIN); // attaches the servo
|     ^~~~~~
src/Servo.h:10:5: error: 'servo2' was not declared in this scope
10 |     servo2.attach(SERVO2_PIN); // attaches the servo
|     ^~~~~~
src/Servo.h:11:5: error: 'servo3' was not declared in this scope
11 |     servo3.attach(SERVO3_PIN); // attaches the servo
|     ^~~~~~
src/Servo.h:12:5: error: 'servo4' was not declared in this scope
12 |     servo4.attach(SERVO4_PIN); // attaches the servo
how can i do it? ,can you help me, I` m Davis. thank you.

Hi Dave. Please see step 5 of "Install Firmware to ESP8266" in readme.md. SERVO1_PIN is declared in configuration.h

@mvwtest , was your issue resolved?

closed due to inactivity


fail on router tu dapp interact

no working link tu dapp and wallet connect

url were on app.sustainlabs.ai


ImproperlyConfigured: GEOIP_DATABASE setting is defined, but file does not exist.

I am having problem when pushing new app with django-easy-timezones. Basicly, I need to call manage.py collectstatic to put files GeoLiteCityv6.dat and GeoLiteCity.dat to static directory. But when I run manage.py collectstatic, following error is raised.
remote: django.core.exceptions.ImproperlyConfigured: GEOIP_DATABASE setting is defined, but file does not exist.
remote: ln: creating symbolic link `/var/lib/openshift/user/app-root/runtime/repo//wsgi/static/media': No such file or directory

Neverending cycle.
I also have a suggestion. Instead of a need to update geoip database manually, how about using https://github.com/futurecolors/django-geoip ?

I don't think you want the dat file in static, then it'll be publicly
available..
On Wed, Jan 20, 2016 at 2:27 AM, Lukas Vojt<EMAIL_ADDRESS>wrote:

I am having problem when pushing new app with django-easy-timezones
Basicly, I need to call managepy collectstatic to put files
GeoLiteCityv6dat and GeoLiteCitydat to static directory But when I run managepy
collectstatic, following error is raised
remote: djangocoreexceptionsImproperlyConfigured: GEOIP_DATABASE setting is defined, but file does not exist
remote: ln: creating symbolic link `/var/lib/openshift/user/app-root/runtime/repo//wsgi/static/media': No such file or directory
Neverending cycle
I also have a suggestion Instead of a need to update geoip database
manually, how about using https://githubcom/futurecolors/django-geoip ?
—
Reply to this email directly or view it on GitHub
https://github.com/Miserlou/django-easy-timezones/issues/18.

Why not? That file is publicly available anyway.

..okay..
So, what is the problem?
d-e-t comes with the dat files included now, you shouldn't even need to
have them in your project.
On Wed, Jan 20, 2016 at 3:06 AM, Lukas Vojt<EMAIL_ADDRESS>wrote:

Why not? That file is publicly available anyway.
—
Reply to this email directly or view it on GitHub
https://github.com/Miserlou/django-easy-timezones/issues/18#issuecomment-173060687
.

oh, I did not know that. Ok, thanks. Probably I wrote this issue to early before thinking about it enough.

I am still missing those files:
django.core.exceptions.ImproperlyConfigured: GEOIP_DATABASE setting is defined, but file does not exist.

I installed this app like this: pip install django-easy-timezones==0.6.2. Then I deleted GEOIP_DATABASE and GEOIPV6_DATABASE variables, because you said they are optional.
But I am still getting ImproperlyConfigured error.
I can see that GEOIP files are in your pip package - https://pypi.python.org/pypi/django-easy-timezones
but pip somehow does not use them?
This is in my dist-packages(/usr/local/lib/python2.7/dist-packages) folder:
$ ls | grep easy_time
django_easy_timezones-0.6.2.dist-info
easy_timezones

$ ls easy_timezones/
__init__.py  __init__.pyc  middleware.py  middleware.pyc  signals.py  signals.pyc  tests.py  tests.pyc  urls.py  urls.pyc  utils.py  utils.pyc  views.py  views.pyc

Should not be those GEOIP files in easy_timezones folder?

You're right, they are supposed to be. This seems to be a packaging problem, but I'm not sure why that's happening. For now, just set the paths explicitly and I'll fix this in a few h.

Even I am getting same error.
Also I downloaded them from repo and added explicitly. Now I am getting Invalid database type, expected City error

Is this issue fixed?

why don't you disable this GeoLiteCity check?

This error is still present

I am also getting this error caused by line 24 of middleware.py:     raise ImproperlyConfigured("GEOIP_DATABASE setting is defined, but file does not exist.")
django.core.exceptions.ImproperlyConfigured: GEOIP_DATABASE setting is defined, but file does not exist.


scroll view bug: function only takes 2 parameters

https://github.com/readium/readium-shared-js/blob/develop/js/views/scroll_view.js#L841
            if (isRangeIsVisibleOnScreen(pageView, domRangeAsRange, 60)) {

...should match:
https://github.com/readium/readium-shared-js/blob/develop/js/views/scroll_view.js#L1232
    function isRangeIsVisibleOnScreen(range, percentVisible) {

For tracking purposes:
This bug was introduced at commit a51d0c123aa471eb27f4c356500947a97c9fa746 (27 November 2015):
https://github.com/readium/readium-shared-js/commit/a51d0c123aa471eb27f4c356500947a97c9fa746#diff-5183dc2e9103d4b256aa0077caf19765R935
This is scroll_view.js at revision a51d0c123aa471eb27f4c356500947a97c9fa746 (search for instances of isRangeIsVisibleOnScreen):
https://github.com/readium/readium-shared-js/blob/a51d0c123aa471eb27f4c356500947a97c9fa746/js/views/scroll_view.js#L935

Fixed here: https://github.com/readium/readium-shared-js/commit/4bd8bc9f2310adb7a4e951434cd800e006acc234
CC @JCCR (could you please check your fork? Thanks!)


User-specified node selector on pods

I'd like to be able to add a Spark configuration to specify a node selector for all pods in the Spark job.  This helps me select all pods from the job to do e.g. resource tracking per job.

This continues to build on the need for custom pod specifications as mentioned in #38. I suppose the feature set and room for customization is possibly greater in Kubernetes compared to all of the other cluster managers that are supported so far. The original design that aligns with the other cluster managers may be too limiting here.

Oh I think this is handled by #355


What is causing this crash?

cargo-clif.exe run -p app is crashing with:
thread 'rustc' panicked at src\archive.rs:23:9:
not implemented: creating dll imports is not yet supported
stack backtrace:
   Compiling ryu v1.0.18
   0:     0x7ff8c00a4e3d - std::backtrace_rs::backtrace::dbghelp64::trace
                               at /rustc/35b658fb1071d752d092751d9f17a9fa7b154ec4/library\std\src\..\..\backtrace\src\backtrace\dbghelp64.rs:91
   1:     0x7ff8c00a4e3d - std::backtrace_rs::backtrace::trace_unsynchronized
                               at /rustc/35b658fb1071d752d092751d9f17a9fa7b154ec4/library\std\src\..\..\backtrace\src\backtrace\mod.rs:66
   2:     0x7ff8c00a4e3d - std::sys::backtrace::_print_fmt
                               at /rustc/35b658fb1071d752d092751d9f17a9fa7b154ec4/library\std\src\sys\backtrace.rs:68
   3:     0x7ff8c00a4e3d - std::sys::backtrace::_print::impl$0::fmt
                               at /rustc/35b658fb1071d752d092751d9f17a9fa7b154ec4/library\std\src\sys\backtrace.rs:44
   4:     0x7ff8c00d54a9 - core::fmt::rt::Argument::fmt
                               at /rustc/35b658fb1071d752d092751d9f17a9fa7b154ec4/library\core\src\fmt\rt.rs:173
   5:     0x7ff8c00d54a9 - core::fmt::write
                               at /rustc/35b658fb1071d752d092751d9f17a9fa7b154ec4/library\core\src\fmt\mod.rs:1182
   6:     0x7ff8c009b341 - std::io::Write::write_fmt<std::sys::pal::windows::stdio::Stderr>
                               at /rustc/35b658fb1071d752d092751d9f17a9fa7b154ec4/library\std\src\io\mod.rs:1835
   7:     0x7ff8c00a4c16 - std::sys::backtrace::print
                               at /rustc/35b658fb1071d752d092751d9f17a9fa7b154ec4/library\std\src\sys\backtrace.rs:34
   8:     0x7ff8c00a7e18 - std::panicking::default_hook::closure$1
                               at /rustc/35b658fb1071d752d092751d9f17a9fa7b154ec4/library\std\src\panicking.rs:265
   9:     0x7ff8c00a7ac9 - std::panicking::default_hook
                               at /rustc/35b658fb1071d752d092751d9f17a9fa7b154ec4/library\std\src\panicking.rs:292
  10:     0x7ff8a2b7b115 - memchr
  11:     0x7ff8c00a859b - alloc::boxed::impl$50::call
                               at /rustc/35b658fb1071d752d092751d9f17a9fa7b154ec4/library\alloc\src\boxed.rs:2078
  12:     0x7ff8c00a859b - std::panicking::rust_panic_with_hook
                               at /rustc/35b658fb1071d752d092751d9f17a9fa7b154ec4/library\std\src\panicking.rs:804
  13:     0x7ff8c00a836f - std::panicking::begin_panic_handler::closure$0
                               at /rustc/35b658fb1071d752d092751d9f17a9fa7b154ec4/library\std\src\panicking.rs:663
  14:     0x7ff8c00a57af - std::sys::backtrace::__rust_end_short_backtrace<std::panicking::begin_panic_handler::closure_env$0,never$>
                               at /rustc/35b658fb1071d752d092751d9f17a9fa7b154ec4/library\std\src\sys\backtrace.rs:171
  15:     0x7ff8c00a7f86 - std::panicking::begin_panic_handler
                               at /rustc/35b658fb1071d752d092751d9f17a9fa7b154ec4/library\std\src\panicking.rs:661
  16:     0x7ff8c00fde54 - core::panicking::panic_fmt
                               at /rustc/35b658fb1071d752d092751d9f17a9fa7b154ec4/library\core\src\panicking.rs:74
  17:     0x7ff8bf82830c - <rustc_codegen_cranelift::archive::ArArchiveBuilderBuilder as rustc_codegen_ssa::back::archive::ArchiveBuilderBuilder>::create_dll_import_lib::h138e4a4d05900f0a
  18:     0x7ff8a138f428 - rustc_codegen_ssa[f52e3fb87beb71e3]::back::link::each_linked_rlib
  19:     0x7ff8a138a9f6 - rustc_codegen_ssa[f52e3fb87beb71e3]::back::link::link_binary
  20:     0x7ff8bf85a4c6 - <rustc_codegen_cranelift::CraneliftCodegenBackend as rustc_codegen_ssa::traits::backend::CodegenBackend>::link::hd19571cdbe1c7838
  21:     0x7ff89e895ad8 - <rustc_interface[b7ec6dbec7ff60f5]::queries::Linker>::link
  22:     0x7ff89e846bff - _wpgmptr
  23:     0x7ff89e841f4f - _wpgmptr
  24:     0x7ff89e84bd0b - _wpgmptr
  25:     0x7ff8c00b963d - alloc::boxed::impl$48::call_once
                               at /rustc/35b658fb1071d752d092751d9f17a9fa7b154ec4/library\alloc\src\boxed.rs:2064
  26:     0x7ff8c00b963d - alloc::boxed::impl$48::call_once
                               at /rustc/35b658fb1071d752d092751d9f17a9fa7b154ec4/library\alloc\src\boxed.rs:2064
  27:     0x7ff8c00b963d - std::sys::pal::windows::thread::impl$0::new::thread_start
                               at /rustc/35b658fb1071d752d092751d9f17a9fa7b154ec4/library\std\src\sys\pal\windows\thread.rs:52
  28:     0x7ff902f67344 - BaseThreadInitThunk
  29:     0x7ff90387cc91 - RtlUserThreadStart

error: the compiler unexpectedly panicked. this is a bug.

note: we would appreciate a bug report: https://github.com/rust-lang/rust/issues/new?labels=C-bug%2C+I-ICE%2C+T-compiler&template=ice.md

note: please make sure that you have updated to the latest nightly

note: please attach the file at `C:\Users\Fred\.cargo\registry\src\index.crates.io-6f17d22bba15001f\cc-1.1.2\rustc-ice-2024-07-13T13_09_51-10436.txt` to your bug report

note: compiler flags: --crate-type lib -C opt-level=3 -C embed-bitcode=no -C debug-assertions=on -C strip=debuginfo -C panic=abort -Z panic-abort-tests -Z codegen-backend=C:\clif\bin\rustc_codegen_cranelift.dll

note: some of the compiler flags provided by cargo are hidden

query stack during panic:
end of query stack
error: could not compile `cc` (lib)
warning: build failed, waiting for other jobs to finish...

src\archive.rs is not my file.
How to know what crate is causing this?
I'm on the latest rust, clif and windows dependencies as of today.

Looks like the cc crate has started using raw-dylib on Windows: https://github.com/rust-lang/cc-rs/pull/1137 cg_clif doesn't know how to build import libraries yet, and as such raw-dylib doesn't work.

 And now?

https://github.com/rust-lang/ar_archive_writer/pull/15 is a port of LLVM's import library writer to rust allowing it to be used by cg_clif. It was opened a while ago, but I didn't get around to reviewing it until a week ago. It is currently waiting on the author of the PR.

I've pushed a commit to get a regular fatal error rather than a panic.

Ok. In the meantime they maybe are reverting that change... https://github.com/rust-lang/cc-rs/issues/1156#issuecomment-2226916579

raw-dylib has been implemented in https://github.com/rust-lang/rustc_codegen_cranelift/pull/1524.


could not launch process: stub exited while attempting to connect: signal: abort trap

What version of Delve are you using (dlv version)?
Version: 1.0.0-rc.1
Build: f553c95eeb29be1694c12eb8769c3531bb628f5d
What version of Go are you using? (go version)?
go version go1.9 darwin/amd64
What operating system and processor architecture are you using?
macOS Sierra Version 10.12, 1.6 GHz Intel Core i5
What did you do?
dlv exec file
What did you expect to see?
The debug session should begin
What did you see instead?
could not launch process: stub exited while attempting to connect: signal: abort trap

dlv exec file

what's file what happens if you do lldb-server gdbserver :8181 -- file?

file is the go generated binary.

If I rutn lldb-server gdbserver :8181 -- generatedbinaryname, I get
using the default platform: host
Platform has no NativeProcessProtocol support
UNREACHABLE executed at /Library/Caches/com.apple.xbs/Sources/lldb/lldb-360.1.70/source/Host/common/NativeProcessProtocol.cpp:449!
Abort trap: 6

Looks like your lldb doesn't work.

@aarzilli what is the way to resolve it?

Not a bug (problem with user setup)


MediaSessionService can't be terminated

Hi. In my MediaBrowserServiceCompat i am runnin this code, in task removed:
override fun onTaskRemoved(rootIntent: Intent?) {
     super.onTaskRemoved(rootIntent)

            exoPlayer.stop()

            exoPlayer.clearMediaItems()
    }

Same as in uamp it triggers notification, which calls stopself().
onDestroy code :
 override fun onDestroy() {

        exoPlayer.removeListener(radioPlayerEventListener)
        exoPlayer.release()

        mediaSession.run {
            isActive = false
            release()
        }

        serviceJob.cancel()

    }

If exoPlayer is not playing when app is closed, everythin is fine. But if it is playing, then it prevents service to end. I wonder why.

To properly stop the service in onTaskRemove a few conditions must be met.

The service should not run in the foreground, so you should call player.pause before stopping the service.
The service should not have controllers being bound to it when stopping the service, so you should call session.release() to disconnect all controllers.

Same as in uamp it triggers notification, which calls stopself().

Please note there is a media3 branch in UAMP. It seems to me that UAMP does the following to make sure these conditions are met.
 override fun onTaskRemoved(rootIntent: Intent) {
        saveRecentSongToStorage()
        super.onTaskRemoved(rootIntent)
        // The choice what to do here is app specific. Some apps stop playback, while others allow
        // playback to continue and allow users to stop it with the notification.
        releaseMediaSession()
        stopSelf()
    }

Hi. Thanks for the answer.
It seems to me, that the code both implementations running is the same. Aparat that in media3 branch it does not handle notification. Or is it that I should release session before calling stopSelf()?
I tried it but still if exoplayer is playing, service does not stop.
It does terminate only if I pause player via RadioServiceConnection .transportControls before closing app. Maybe so because of the delay, I don't know.

Yes, exactly. From above:

The service should not run in the foreground, so you should call player.pause before stopping the service.

But then how to stop service if audio is still playing? Calling exoplayer.pause() in onTaskRemoved() does not make difference.

Sorry, I was confused. I was thinking you are using a Media3 service which isn't the case. If you are using MediaBrowserServiceCompat then your app does put the service into the foreground itself I think. The MediaBrowserService doesn't do that. If you look at the code of your app/service. You probably somewhere call service.startForeground(int, notification). If you are in this state when you are in onTaskRemoved the you need to remove the service from the foreground.
UAMP with MediaBrowserServiceCompat does this here.

Oh, okay. I think I am removing it already here though:
    override fun onNotificationCancelled(notificationId: Int, dismissedByUser: Boolean) {
        super.onNotificationCancelled(notificationId, dismissedByUser)

        radioService.apply {
            stopForeground(STOP_FOREGROUND_REMOVE)
            isForegroundService = false

            stopSelf()
        }
    }

Any new  thoughts on this matter?
I spotted another scenario with exoplayer when service will not be terminated.
It is if setting mediasource with stream which will trigger onPlayerError, or just putting phone in airplane mode for same result. If I pause exoplayer, and close an app, the service will still go on.


[ BB2-1133 ] Replace code to utilize the BB2 Node SDK

JIRA Ticket:
BB2-1133
User Story or Bug Summary:
This replaces code to utilize the SDK in the sample application.
This work goes along with this PR for updating the SDK.
NOTE: This replaces this PR and continues Nick's great work.
NOTE: Keeping this PR in draft mode until it is time to publish.
What Does This PR Do?
The following is a summary of the changes in this PR

Add cms-bluebutton-sdk dependency via .tgz file installation in server/package.json

Add imports for SDK in files where needed. The following imports are used:

import BlueButton from "cms-bluebutton-sdk";
import Environments from 'cms-bluebutton-sdk';
import { Errors } from 'cms-bluebutton-sdk';

Add BB2 SDK configuration in server/src/Server.ts

Replace Auth related code with SDK usage in server/src/routes/Authorize.ts

Add example of handling SDK exceptions in server/src/routes/Authorize.ts

Replace FHIR resource code with SDK usage in server/src/routes/Data.ts

Remove all code and files replaced by SDK usage.

Update Typescript configuration options in server/tsconfig.json

Copy server/yarn.lock from docker container installation to add to PR using:

docker cp <container-id>:/server/yarn.lock server/yarn.lock

Misc. cleanup.

PR Testing Procedure

Create a .tgz package file from the BB2-1133 SDK branch
Use these commands in a shell from the project root directory:

# REPO:  CMSgov/cms-bb2-node-sdk
git checkout dtisza1/BB2-1133-use-sdk-in-sample-app

# Clean up build & dist directories:
rm -rf build/ dist/

# Build
yarn build
yarn build:types

# Create tarball package .tgz file
yarn pack

This should produce a package .tgz tarball file named cms-bluebutton-sdk-v0.1.0.tgz.

Copy the .tgz package file in to this PR's branch.
Copy the cms-bluebutton-sdk-v0.1.0.tgz file in to the server/ directory.
Verify the following line is in the server/package.json file:
  "cms-bluebutton": "file:cms-bluebutton-sdk-v0.1.0.tgz",

Build and Run the sample application using the SDK:

# REPO:  CMSgov/bluebutton-sample-client-nodejs-react
git checkout origin/BB2-1133_use_node_sdk

docker-compose down

docker rmi bluebutton-sample-client-nodejs-react_server

docker rmi bluebutton-sample-client-nodejs-react_client

rm server/package-lock.json server/yarn.lock

docker-compose up -d

docker-compose logs -f # To see logs

Test the sample client in your browser via http://localhost:3000

What Should Reviewers Watch For?
If you're reviewing this PR, please check these things, in particular:

Are there any unhandled and/or untested edge cases you can think of?

Does this make any backwards-incompatible changes that might break end user clients?

Can you find any bugs if you run the code locally and test it manually?

Naming in general? Do the namings of variables, methods, classes, and other items make sense? Are they too verbose, or not verbose enough? Any redundancies? Are the comments descriptive enough?

Are audit logging items properly handled? Is sensitive information either not included or hashed?

What/how to review:

This is ready for review and testing.

I am running into some issues following the test outline. my package.json did not have the line:
"cms-bluebutton": "file:cms-bluebutton-sdk-v0.1.0.tgz",
So i had to add it in the dependencies. When i move to step 3 as a result of adding to package.json i get this error:
error: Your local changes to the following files would be overwritten by checkout:
	server/package.json
Please commit your changes or stash them before you switch branches.
Aborting```

Cool, I was able to work through the issues with your help. This all looks good so far!

Updated readme and pointed out SDK call in code comments. Please take a look. @dtisza1 @JFU-GIT

This work was implemented via this PR:  https://github.com/CMSgov/bluebutton-sample-client-nodejs-react/pull/33


Accomodate Gio changes

Fixes examples
Fixes books
Let the book tests also run when the book was not changed (don't think that fixes the problem we discussed though @bilelmoussaoui)

I don't believe the CI changes were acked by me. Please don't merge directly next time.

Sorry @bilelmoussaoui, I thought this PR already had one approval, but it indeed did not.

Anything needs fixes here still?

Not really, it's just the CI "solution" doesn't seems right to me; I will take care of that along with #270

Again, I said that I DON'T see that as the solution of the CI problem (see above 😉)

it's not a solution if you merge directly to master, which you should never do :-)


Zone attribute change breaks compatibility with Oracle 8 & 9 (probably RHEL 8 & 9 as well)

:speaking_head: Foreword
The change to the cookbook in v6.3.5 by adding --zone options to all firewall-cmd commands, conflicts with the --direct option and causes it to error out.
:ghost: Brief Description
      ================================================================================
       Error executing action `restart` on resource 'firewall[default]'
       ================================================================================

       Mixlib::ShellOut::ShellCommandFailed
       ------------------------------------
       Expected process to exit with [0], but received '2'
       ---- Begin output of firewall-cmd --zone=drop --direct --add-rule ipv4 filter INPUT 50 -i lo -m comment --comment 'allow loopback' -j ACCEPT ----
       STDOUT:
       STDERR: usage: see firewall-cmd man page
       --zone is an invalid option with --direct
       ---- End output of firewall-cmd --zone=drop --direct --add-rule ipv4 filter INPUT 50 -i lo -m comment --comment 'allow loopback' -j ACCEPT ----
       Ran firewall-cmd --zone=drop --direct --add-rule ipv4 filter INPUT 50 -i lo -m comment --comment 'allow loopback' -j ACCEPT returned 2

       Cookbook Trace: (most recent call first)
       ----------------------------------------
       /tmp/kitchen/cache/cookbooks/firewall/libraries/helpers_firewalld.rb:12:in `firewalld_rule!'
       /tmp/kitchen/cache/cookbooks/firewall/libraries/provider_firewall_firewalld.rb:108:in `block (2 levels) in <class:FirewallFirewalld>'
       /tmp/kitchen/cache/cookbooks/firewall/libraries/provider_firewall_firewalld.rb:107:in `each'
       /tmp/kitchen/cache/cookbooks/firewall/libraries/provider_firewall_firewalld.rb:107:in `block in <class:FirewallFirewalld>'

:pancakes: Cookbook version
6.3.5+
:woman_cook:  Chef-Infra Version
18.3.0
:tophat: Platform details
Oracle Linux 8
Oracle Linux 9
Most likely:
RHEL 8
RHEL 9
Steps To Reproduce
Try and deploy firewall rules on Oracle Linux 8 or 9.
:police_car: Expected behavior
No errors and running the same as v6.3.4

This is a HUGE issue!!! Without this being fixed, it prevents you from using this cookbook [Using Rocky Linux 9.4]

I tried downgrading to v6.3.4 and it still fails. Any workaround?

@breisig how are you applying v6.3.4?
For example, are you running that version against a cleanly created VM or are you rerunning the cookbook with a different version lock against an already failing VM?
If you are rerunning against an already failed VM, I would recommend purging the existing ACLs and firewall files like direct.xml to make sure the zone option is not being applied anywhere.
Also double check the Chef cache to see if the template it wants to apply actually contains the --zone option.

What I did was download the firewall code from git and checkout the 6.3.4 branch and uploaded it to Chef. I installed a FRESH Rocky Linux 9.4 linux VM with the latest updates applied and bootstrapped the server with the following recipe
# enable platform default firewall
firewall 'default' do
  action :install
  log_level :off
end

# open standard ssh port
firewall_rule 'ssh' do
  port     22
  command  :allow
  action :create
end

And it fails with the initial bootstrap of the server
Error: Mixlib::ShellOut::ShellCommandFailed: Expected process to exit with [0], but received '2'
     ---- Begin output of firewall-cmd --zone=drop --direct --add-rule ipv4 filter INPUT 50 -p tcp -m tcp -m multiport --dports 22 -m comment --comment 'ssh' -j ACCEPT ----
     STDOUT:
     STDERR: usage: 'firewall-cmd --help' for usage information or see firewall-cmd(1) man page
     --zone is an invalid option with --direct
     ---- End output of firewall-cmd --zone=drop --direct --add-rule ipv4 filter INPUT 50 -p tcp -m tcp -m multiport --dports 22 -m comment --comment 'ssh' -j ACCEPT ----
     Ran firewall-cmd --zone=drop --direct --add-rule ipv4 filter INPUT 50 -p tcp -m tcp -m multiport --dports 22 -m comment --comment 'ssh' -j ACCEPT returned 2

And I still get the error.

@breisig  sorry for being pedantic, need to double/sanity check.
When you bootstrap a VM, near the beginning, after possibly the installation of the client, you get a list of cookbooks and the versions that is being deployed/applied, please paste that into a comment.
This is to confirm it isn't still deploying a newer version of the cookbook that is stored on the Chef Server, because the operation you explained above, doesn't make it seem to me that you are actually locking the firewall cookbook version.
In version 6.3.4 of the firewall cookbook and earlier there is no --zone in the code used to generate the rule, it was only added after that.
https://github.com/sous-chefs/firewall/blob/6.3.4/libraries/provider_firewall_firewalld.rb

Sorry, I should of seen this. The newer cookbook was still around which the recipe defaulted to use it. I deleted it and now it work wth 6.3.4, Again, this is just a workaround.

@breisig it happens mate, that is why it is always a good idea to double/sanity check.
Tbh after some looking at the code, I see the following that needs to be changed:

The firewall_rule resource needs to make the zone attribute optional and no default value. (libraries/resource_firewall_rule.rb)
Then the command generation needs to have an if/else statement that tests whether zone is empty, if not empty use --zone, else use --direct

If I have time, I will try and submit a PR.


Compression behaviour for Azure CDN from Microsoft

There was confusion around transcoding and decompression behaviour for Azure CDN from Microsoft. Clarified.

@gball14 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

The edited update appears to be in master (in this repo): https://github.com/MicrosoftDocs/azure-docs-pr/blob/master/articles/cdn/cdn-improve-performance.md


Cannot obtain the same results as the paper

Training set:

Inference prompt: "A * backpack"
we get the following results:

How to solve this issue?

Hi @XiaominLi1997 ,
Just to make sure:

Are you using the LDM or the Stable Diffusion version? Our results are all using LDM (the paper predates Stable Diffusion).
Did you change the batch size or any other parameter? We used the original LDM code which has LR scaled by batch size, so you might get significant differences from that.
There's some rather significant seed sensitivity, so you might get very different results with just seed changes.

We re-ran things for a related issue here and got this cat checkpoint that you could try playing with.

Hi @XiaominLi1997 ,
Just to make sure:

Are you using the LDM or the Stable Diffusion version? Our results are all using LDM (the paper predates Stable Diffusion).
Did you change the batch size or any other parameter? We used the original LDM code which has LR scaled by batch size, so you might get significant differences from that.
There's some rather significant seed sensitivity, so you might get very different results with just seed changes.

We re-ran things for a related issue here and got this cat checkpoint that you could try playing with.

Thanks for your reply. I use Stable Diffusion version. I guess there are two possible reasons.

I set batch_size =1. I am not sure if this can degrade the generating performance during gradient descent.

When using Stable Diffusion version, I got the following warning.
Some weights of the model checkpoint at openai/clip-vit-large-patch14 were not used when initializing CLIPTextModel: ['vision_model.encoder.layers.0.layer_norm2.weight', 'vision_model.encoder.layers.23.mlp.fc2.bias', ... ...]
which means we did not load weights of openai/clip-vit-large-patch14 successfully.
How can I fix this error?
[Training setting]
python main.py
--base
configs/stable-diffusion/v1-finetune.yaml
-t
--actual_resume
/15764332239/projects/v1-5-pruned.ckpt
-n
toy
--gpus
0,
--data_root
./images/cat_statue
--init_word
toy
--no-test
[Inference setting]
python stable_txt2img.py
--n_samples
8
--embedding_path
/15764332239/projects/textual_inversion/logs/cat_statue2023-06-04T06-46-55_cat/checkpoints/embeddings_gs-6099.pt
--ckpt
/15764332239/projects/v1-5-pruned.ckpt
--prompt
"a * backpack."

The Stable Diffusion version doesn't work quite as well because SD's text encoder is much smaller and with lower dimensions. It means our learned 'words' have to be further away from real words, which leads to harder-to-edit results.
If you're trying to re-create paper results, you should use LDM (which the paper used) and not Stable Diffusion.
If you just want a text-based personalization method as a basis for future work, then you can certainly build on the SD version or use newer papers (for example https://github.com/NeuralTextualInversion/NeTI).
On getting better results with SD:
I would give the automatic1111 or diffusers implementation of Textual Inversion a try, since they have optimizations that should let you run bigger batch sizes on less memory. If you are only able to fit in a batch size of 1, I would look at using gradient accumulation steps (you can adjust this in the config) and / or increasing the learning rate.
I think the weight think is a warning and not an error. Check if it says something like "This IS expected if you are initializing..." and if so you can ignore it.

Thank you!

If you just want a text-based personalization method as a basis for future work, then you can certainly build on the SD version or use newer papers (for example https://github.com/NeuralTextualInversion/NeTI).

I wanna obtain a personalized embedding of the properties of a subject (object). I will have a try on NeTI.


Custom icons by protocol (URL scheme)

Idea
Currently, a number of URL schemes are special-cased to provide icons. However, there are still outstanding requests for more (#47, #55), and I was about to add one more (x-fantastical:). Presumably it will just keep happening, and it's not terribly sustainable to have you editing the code each time.
It would be great to have a UI in the settings to assign a custom icon by URL scheme (or possibly even by URL regex).

Glad to see this reported. Also #59 is another one. They will likely keep rolling in as OP stated.
Obsidian, DevonThink, Bear Notes, message:... The list goes on.

👍 for something like this. In our org we have a lot of internal links that are behind auth and the plugin is not able to fetch the favicons so they all show up the same as a generic globe icon.


feat(paged): add loading flag

Argh! Must have happened when I pasted in my changes. I reformatted.


proposal: databse/sql: add NullUint64

Hey! I maintain an entity framework for Go named ent, and we're having troubles handling nullable uint64 types as the official sql package doesn't support it. Using NullInt64 is not enough as some databases accept values that are greater than 2^63−1.
Since the sql.convertAssign function is private my 2 options are either copy its code or use the //go:linkname directive.

 }

Are you suggesting we add all the uint sizes, or just uint64? Why?

Hey @rsc and thanks for responding.
The NullUint64 is required for ent atm to support scanning null uint64 values because using NullInt64 returns a value out of range error if the value is greater than 2^63−1.
If you see any concern with adding the rest of the unit types, it can wait for the "generics" change, because they can be implemented externally with NullUint64.

Most SQL databases don't support uints; though some do. I would recommend we wait for a generic version to land. I was working on one when I encountered a compiler error at tip; once that is resolved I can look into it more.
I recommend against this proposal.

I don't think it matters if the database supports unsigned integers or not, it can also be defined as (numeric(20,0) check (c >= 0 and c <= 2^64-1)), and we should still be able to scan its value to the native uint64 type without using an external package.
Currently, Rows.Scan (convertAssignRows) does work with uint64, but if the column is nullable, we have no way to work around this except for using the sql.convertAssign function directly or copying it.
As I mentioned above, unlike NullByte and NullInt16 (that were added by me on 1.17), the NullUint64 type can't be implemented easily outside the sql package, and I don't think we should be blocked for at least a year until generics will be stable.

Thanks for the feedback @kardianos 🙏
I don't think it matters if the database supports unsigned integers or not, it can also be defined as (numeric(20,0) check (c >= 0 and c <= 2^64-1)), and we should still be able to scan its value to the native uint64 type without using an external package.
Currently, Rows.Scan (convertAssignRows) does work with uint64, but if the column is nullable, we have no way to work around this except for using the sql.convertAssign function directly or copying it.
As I mentioned above, unlike NullByte and NullInt16 (that were added by me on 1.17), the NullUint64 type can't be implemented easily outside the sql package, and I don't think we should be blocked for at least a year until generics will be stable.

Thanks for opening this @a8m, on behalf of the Ent community I join your proposal 🙏

@rsc I would prefer to wait for a generic Null. Unsigned types are not common in databases.


Pass a comment or key string through with the query and include it as comment in generated SQL

This helps when identifying the source of the SQL when looking at audit logs and performance data.

See release 3.2.0


need documentation on how to use this extension in VSCode

I created a self contained CQL measure and after right clicking on the file in VisualStudioCode and selecting "View ELM" I see the full ELM file without any CQL-to-ELM issues.
When I then right click and select "Execute CQL" it shows me the folders that I have to create on my Microsoft Windows laptop to place my content. I am getting an error as well:
Caused by: java.lang.IllegalArgumentException: Could not load source for library MyMeasure, version null.
at org.cqframework.cql.cql2elm.PriorityLibrarySourceLoader.getLibrarySource(PriorityLibrarySourceLoader.java:72)
at org.cqframework.cql.cql2elm.LibraryManager.translateLibrary(LibraryManager.java:208)
at org.cqframework.cql.cql2elm.LibraryManager.resolveLibrary(LibraryManager.java:184)
at org.opencds.cqf.cql.evaluator.engine.execution.TranslatingLibraryLoader.translate(TranslatingLibraryLoader.java:125)

I am unclear on what belongs in this folder:
No data found at c:\Users\revans\Documents\MyMeasures\input\tests. Evaluation may fail if data is required.
I placed dataFHIR Bundle with patient data and a single FHIR Measure in this folder, but maybe it is looking for a specific file name?
What belongs in this folder?

this one is obvious as I placed both of my two ValueSets in this folder:
Terminology path: c:\Users\revans\Documents\MyMeasures\input\vocabulary\valueset

since this is a measure my CQL has a parameter so I was expecting a right menu entry like "Execute CQL with parameters"
I am unclear how to specify a measure period like I do when I run CQL-ruler using PostMan from a local instance of https://github.com/DBCG/cqf-ruler

when I split up my single CQL into separate libraries it executes but I would like to know how to debug the CQL as I used to be able to do in the Atom plugin

Check out the user guide here:
https://github.com/cqframework/vscode-cql/wiki/User-Guide


Release notes did not pop up on update to 1.67.0 (user setup)

Does this issue occur when all extensions are disabled?: I don't know. It is too late.

VS Code Version: Version: 1.67.0 (user setup)
Commit: 57fd6d0195bb9b9d1b49f6da5db789060795de47
Date: 2022-05-04T12:06:02.889Z
Electron: 17.4.1
Chromium: 98.0.4758.141
Node.js: 16.13.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19044

Steps to Reproduce:

Open vscode
Update was installed but no changelog/release notes were opened.

Can you try installing 1.66, then 1.67 again and confirm that indeed you don't get a release notes prompt? Is it possible that the notes actually appeared but an extension might've closed the editor?

After reinstalling 1.66 and using the update function it is indeed showing the patch notes again.

Is it possible that the notes actually appeared but an extension might've closed the editor?
Possible


how to download large file?

hi, how can i download large file?

Hi @xyyc, what's preventing you from downloading a large file?

@xyyc I assume you've found your answer.


dirty data check

there may exists some dirty data in our own datasets, that will lead to training failure

Hi @yuzhj ,
Modifying the dataset might not be so general. We suggest implement an annotation filter pipeline and use that pipeline in your config. In this way, you only need to implement the pipeline once and apply it to all the datasets.


Update embedsignature directive to emit function annotations

This is for sure not enough, but at least it can handle "simple" annotations like non-compound compile-time constants and name nodes.

Should we start using the DeclarationWriter in Cython.CodeWriter for the expressions? I can't find any current usage, but a TreeVisitor seems the right approach to me.
We could extract an ExpressionWriter base class from it that excludes the declaration parts and could be used wherever we want to generate a text representation of Cython expressions.

@scoder I would need some help with that, I would like to get this working for the next release, but I'm a bit lost about how to get started.

It's used like this in TestUtils.py:
    def codeToLines(self, tree):
        writer = CodeWriter()
        writer.write(tree)
        return writer.result.lines

I would extract all visit_*() methods into the new base class (inheriting from TreeVisitor) that handle nodes allowed in a Cython expression, and then add new methods for nodes that are currently unsupported. Look at what's there already (NodeNode, IntNode, etc.), most nodes should be really straight forward.

@scoder Thanks! That helped a lot. I'll give it a look tomorrow.

@scoder PR updated. After a some thinking, it is not obvious to me that DeclarationWritter has much to share with the new ExpressionWriter I implemented, so I suggest to keep them separate for now. Please note the change in the testcase, are you OK with that? Otherwise, could you suggest a proper fix?

@scoder I think I've addressed most of your concerns. I'm still not sure about DeclarationWriter inheriting from ExpressionWriter, but such refactor should be easy to implement in a future PR.

Thanks! I'll let travis finish its run and then happily merge it.


Volume Short syntax

Both VolumeSource and PersistentVolume from various apiGroups (don't know if there is more than 1)

Here's what I'm planning to do for v1.Volume.
From:
name: somename
<source-type, e.g. emptyDir>:
  camelField0: val0
  camelField1: val1

To:
name: somename
type: <snake_case source-type, e.g. empty_dir>
camel_field0: val0
camel_field1: val1

sounds ok. I haven't gone throuhg every type to

merged a fix. may revisit later


MP - Microprofile Health documentation examples needs update to align with latest HealthCheckResponse to user builder

Environment Details

Helidon Version: 4.0.1
Helidon MP
JDK version: 21
OS: MacOs
Docker version (if applicable):

Problem Description
Documentation code uses HealthCheckResponse without using builder which is needed as per the Java Doc here: https://helidon.io/docs/latest/apidocs/io.helidon.health/io/helidon/health/HealthCheckResponse.html
Documentation: https://helidon.io/docs/latest/#/mp/health and https://helidon.io/docs/latest/#/mp/guides/health. Further the guides/health using Started annotation rather than Startup annotation.
Steps to reproduce
Documentation bug.

I think our pending PR https://github.com/helidon-io/helidon/pull/8125 deals with most of the aspects of this issue, but not the typo with the MP annotation.


I can't start a tutorial

Hi!
I can't start the https://fiware-tutorials.readthedocs.io/en/latest/pep-proxy.html tutorial.
$ docker -v
Docker version 20.10.7, build f0df350
$ docker-compose -v
docker-compose version 1.24.0, build 0aa59064
$ lsb_release -a
No LSB modules are available.
Distributor ID:	Ubuntu
Description:	Ubuntu 21.10
Release:	21.10
Codename:	impish
./services start
Stopping running containers
unknown shorthand flag: 'f' in -f
./services start legacy
Stopping running containers
WARNING: The host variable is not set. Defaulting to a blank string.
ERROR: Version in "./docker-compose/southport.yml" is unsupported. You might be seeing this error because you're using the wrong Compose file version. Either specify a supported version (e.g "2.2" or "3.3") and place your service definitions under the services key, or omit the version key and place your service definitions at the root of the file to use version 1.
For more on the Compose file format versions, see https://docs.docker.com/compose/compose-file/
docker-compose -f docker-compose/southport.yml config
WARNING: The host variable is not set. Defaulting to a blank string.
ERROR: Version in "./docker-compose/southport.yml" is unsupported. You might be seeing this error because you're using the wrong Compose file version. Either specify a supported version (e.g "2.2" or "3.3") and place your service definitions under the services key, or omit the version key and place your service definitions at the root of the file to use version 1.
For more on the Compose file format versions, see https://docs.docker.com/compose/compose-file/
Can I do something?

You have a very outdated version of docker-compose, so the obvious first step would be to upgrade Docker. The prerequisites page states use "Docker Compose 1.29 or higher" - docker-compose 1.24  is having difficulty with understanding version: "3.8" in the southport.yaml. Just to get it working you could reduce that to a lower number like "3.3" as suggested and see if that works for you - the docker-compose.yml files are not necessarily using 3.8 features, so that may work.

Thanks @jason-fox
I have resolved this issue by upgrading docker-compose to 1.29.2
I copied the linux command but didn't check it


Docker container for simulator

Since we are currently facing a few issues with the Noetic version of the simulator, it is a good idea to Dockerize the Melodic version of the repo for easy use on the ubuntu 20 machines, while running the simulator.

This is done under the master_containerized branch, will not pull to development as this is just a temporary fix until we figure out what is happening with the gazebo11 simulator


Add Gridding Cookbook to the gallery

This cookbook was removed in #142

It looks like the issue is still there.

After some local debugging, the reason turns out to be this line in the _gallery_info.yml file which has an empty tag:
tags:
  domains:
  packages:
    - xarray

etc.
Turns out our code doesn't know how to handle an empty set of tags.
A simple workaround would be to remove the domains field from the _gallery_info.yml file in the gridding cookbook repo. But our code shouldn't be that fragile. I'll see if there's a straightforward fix in the gallery code that we can implement here.

This should work.
The gallery code already had a check for empty tags but there was a bug in it. It was checking to see if the first item in a list was None, but there is no list at all in this case, just a plain old None.


styles/custom.less is overwritten

I wanted to tone down the yellow color for .status-modified in the treeview, so I override the coloring in my custom.less file.
li.status-modified {
    color: purple; //whatever color, purple just so it's obvious
}

When I do a reload of Atom, I notice a few things:

the yellow color persists!
unmodified files in a folder that is marked as modified are purple
modified files within the same folder are not purple as intended, but still yellow.

Oh, well clearly I have made a mistake with my CSS selector - it needs to be more specific than just li.status-modified so that it will be applying to files instead of the parent folder. Fair enough, I am not paying that much attention.
Open up ~/.atom/packages/atom-material-ui/styles/custom.less and... What's this? My customizations are gone!
Am I editing the wrong file?
I have noticed that if I inspect the generated <style> element for atom-material-ui via the developer console, my changes do show up the first time.
HOWEVER, the order of the rules is mangled so that the custom.less rules do not get applied at the end, which means they do not take priority over the default values unless they are marked !important.
I can create a separate issue for that part, if needed.
Am I just doing this all completely wrong? (I kinda hope so!)

@monkpit yep, you're doing it wrong :smile:. The custom.less is an internal stylesheet where I override the theme's color variables based on your settings. This file is generated every single time you start or reload Atom, that's why your changes won't persist.
The place for your own custom styles is in the user stylesheet located in ~/.atom/styles.less.
~ means your home directory and varies depending on your OS:

Windows C:\Users\<you>
OS X /Users/<you>
Linux /home/<you> (though it might differ from distro to distro)


Additional check in startup-script (no EXTERNAL IP/Cloud NAT) & functionality with Private Access

When installing Forseti without internet connection startup-script removes the Python modules and removes the installation. Due to missing internet connection afterwards it can't be installed/upgraded. Additional check in the script will be useful before removing the product.
Python3 modules
Before restart :
root@forseti-client-vm-88ca767:~# pip3 list | wc -l
104
After restart :
root@forseti-server-vm-88ca767:~# pip3 list | wc -l
80
Startup-script attached.
Question : if we reinstall the product and remove the startup script will it be safe or some part of the install process will attach it later again?
startup-script.zip

Hi @ttodo-softserve, thanks for submitting this issue. The startup script will get re-added if Terraform is run again to update/install.
Do you plan on adding a Cloud NAT? Any reason you did not add a Cloud NAT as part of the installation?
Forseti will not be able to work without one, since it has to make external calls to get Inventory data, amongst other things. If your process is to install Forseti, and then later go back and add a cloud NAT, then we can probably add in a check within the startup script to exit if there is no internet connection.

@gkowalski-google - thank you for your inputs! Appreciated.
We will be careful when using the TF or restart instances.
As for the Cloud NAT - it's a policy on organizational level, unfortunately.
I was thinking that using VPC network + assign private IP to CloudSQL will overcome missing external IPs and missing cloud NAT, leaving us with non-patched installation but still fully functional.
Managed to setup the internal VPC + CloudSQL with internal IP and during creation of inventory I got this partially failed (looks like permission issue but let me know if not) :
Please help me understand - what else will not work if there is no Cloud NAT, no external IPs + using VPC and Private CloudSQL ?
"status": "PARTIAL_SUCCESS",
"warnings": "GCP API Error: unable to get groups from GCP:\n<HttpError 403 when requesting https://www.googleapis.com/admin/directory/v1/groups?customer=C02uoh0ff&alt=json returned \"Not Authorized to access this resource/api\">\n{\n \"error\": {\n  \"errors\": [\n   {\n    \"domain\": \"global\",\n    \"reason\": \"forbidden\",\n    \"message\": \"Not Authorized to access this resource/api\"\n   }\n  ],\n  \"code\": 403,\n  \"message\": \"Not Authorized to access this resource/api\"\n }\n}\n\n\nGCP API Error: unable to get users from GCP:\n<HttpError 403 when requesting https://www.googleapis.com/admin/directory/v1/users?customer=C02uoh0ff&viewType=admin_view&alt=json returned \"Not Authorized to access this resource/api\">\n{\n \"error\": {\n  \"errors\": [\n   {\n    \"domain\": \"global\",\n    \"reason\": \"forbidden\",\n    \"message\": \"Not Authorized to access this resource/api\"\n   }\n  ],\n  \"code\": 403,\n  \"message\": \"Not Authorized to access this resource/api\"\n }\n}\n\n\n",

HI @ttodo-softserve, if the Forseti server cannot access the internet, then it will have the following issues:

Inventory: API calls to CAI or GCP APIs will fail, and this will prevent any inventory data to be collected.
Scanner: Should be fine, this between server and db
Notifier: Email, Slack, CSCC notifications will fail as these all rely on access to public internet

The error you mention above looks like it is due to not configuring G Suite to allow Forseti to connect. Please follow these steps.
As for the Cloud NAT policy, let me look into this some more to see if there is anything we can do. We are planning on releasing the Alpha version of the Real Time Scanner next quarter (hopefully early in the quarter). I'm not 100%, but it's possible that we would be able to use that for scanning without public internet access. This would scan resources as changes are made, but the periodic inventory/scans run by the cron job would still not work.

@gkowalski-google , I would like to provide more details of the current status - where MySQL is accessible via internal IP, Private access is ON on the default network.
Can you please advice which URL is Forseti trying to open in the scanner exception stack?
Any other comments are welcome - if the setup is with Private Access ON to run explain,inventory,model,scanner (notifier will be not OK as you mentioned).
I did a small script to check from the server instance access to the APIs :
for i in gcloud services list | egrep -v "NAME|TITLE"; do echo $i; curl -s https://$i/ > /dev/null; echo "Result $?"; done;
admin.googleapis.com
Result 0
appengine.googleapis.com
Result 0
bigquery-json.googleapis.com
Result 0
bigquerystorage.googleapis.com
Result 0
cloudapis.googleapis.com
Result 0
cloudasset.googleapis.com
Result 0
cloudbilling.googleapis.com
Result 0
clouddebugger.googleapis.com
Result 0
cloudresourcemanager.googleapis.com
Result 0
cloudtrace.googleapis.com
Result 0
compute.googleapis.com
Result 0
container.googleapis.com
Result 0
containerregistry.googleapis.com
Result 0
datastore.googleapis.com
Result 0
deploymentmanager.googleapis.com
Result 0
groupssettings.googleapis.com
Result 0
iam.googleapis.com
Result 0
iamcredentials.googleapis.com
Result 0
iap.googleapis.com
Result 0
logging.googleapis.com
Result 0
monitoring.googleapis.com
Result 0
oslogin.googleapis.com
Result 0
pubsub.googleapis.com
Result 0
securitycenter.googleapis.com
Result 0
servicemanagement.googleapis.com
Result 0
servicenetworking.googleapis.com
Result 0
serviceusage.googleapis.com
Result 0
sql-component.googleapis.com
Result 0
sqladmin.googleapis.com
Result 0
storage-api.googleapis.com
Result 0
storage-component.googleapis.com
Result 0
The ERRORs I see in the forseti log are :
2019-09-16 12:51:36,187 INFO [forseti-security][2.19.1] google.cloud.forseti.scanner.scanner_builder(_instantiate_scanner): Initializing the rules engine:
Using rules: /home/ubuntu/forseti-security/rules/blacklist_rules.yaml
2019-09-16 12:55:57,008 ERROR [forseti-security][2.19.1] google.cloud.forseti.services.scanner.service(_run_scanner): <urlopen error [Errno 110] Connection timed out>
Traceback (most recent call last):
File "/usr/lib/python3.6/urllib/request.py", line 1318, in do_open
encode_chunked=req.has_header('Transfer-encoding'))
File "/usr/lib/python3.6/http/client.py", line 1254, in request
self._send_request(method, url, body, headers, encode_chunked)
File "/usr/lib/python3.6/http/client.py", line 1300, in _send_request
self.endheaders(body, encode_chunked=encode_chunked)
File "/usr/lib/python3.6/http/client.py", line 1249, in endheaders
self._send_output(message_body, encode_chunked=encode_chunked)
File "/usr/lib/python3.6/http/client.py", line 1036, in _send_output
self.send(msg)
File "/usr/lib/python3.6/http/client.py", line 974, in send
self.connect()
File "/usr/lib/python3.6/http/client.py", line 1407, in connect
super().connect()
File "/usr/lib/python3.6/http/client.py", line 946, in connect
(self.host,self.port), self.timeout, self.source_address)
File "/usr/lib/python3.6/socket.py", line 724, in create_connection
raise err
File "/usr/lib/python3.6/socket.py", line 713, in create_connection
sock.connect(sa)
TimeoutError: [Errno 110] Connection timed out
2019-09-16 12:49:55,758 ERROR [forseti-security][2.19.1] google.cloud.forseti.services.inventory.base.resources(accept): GCP API Error: unable to get users from GCP:
<HttpError 403 when requesting https://www.googleapis.com/admin/directory/v1/users?customer=C02uoh0ff&viewType=admin_view&alt=json returned "Not Authorized to access this resource/api">
2019-09-16 12:49:55,428 ERROR [forseti-security][2.19.1] google.cloud.forseti.services.inventory.base.resources(accept): GCP API Error: unable to get groups from GCP:
<HttpError 403 when requesting https://www.googleapis.com/admin/directory/v1/groups?customer=C02uoh0ff&alt=json returned "Not Authorized to access this resource/api">

@ttodo-softserve Thank you for digging into this! We know this is something that needs to get documented for others to setup. We opened an issue recently to add a new Terraform example for a private deployment of Forseti.
As for the URL that the Blacklist Scanner is attempting to hit, it is this: https://github.com/forseti-security/forseti-security/blob/dev/rules/blacklist_rules.yaml#L3. I will add a note to the issue mentioned above to ensure we look into working around this. I think it's OK to ignore this error for now. If you want, you can disable this scanner in the Forseti config.yaml file and you should not see that error anymore.

@gkowalski-google - my bad for the users/groups exceptions. Did not recognize config is overwritten each time server runs. Now is OK - please ignore this.
Status so far (only internal and no internet connection) - inventory is collected, SQL is reachable. We have around 6000+ items in the inventory.
Issues :

SendGrid Email notification is not working as mentioned by you as well. Same setup with external IP/internet works - without has exception. Is it possible to use internal Google API for SendGrid?

BlackList module though disabled still throws exceptions :
- name: blacklist
enabled: false

_File "/usr/local/lib/python3.6/dist-packages/forseti_security-2.19.1-py3.6.egg/google/cloud/forseti/scanner/audit/blacklist_rules_engine.py", line 160, in get_and_parse_blacklist
data = urllib.request.urlopen(url).read().decode('utf-8')
File "/usr/lib/python3.6/urllib/request.py", line 223, in urlopen
return opener.open(url, data, timeout)
File "/usr/lib/python3.6/urllib/request.py", line 526, in open
response = self._open(req, data)
File "/usr/lib/python3.6/urllib/request.py", line 544, in _open
'_open', req)
File "/usr/lib/python3.6/urllib/request.py", line 504, in _call_chain
result = func(*args)
File "/usr/lib/python3.6/urllib/request.py", line 1361, in https_open
context=self._context, check_hostname=self.check_hostname)
File "/usr/lib/python3.6/urllib/request.py", line 1320, in do_open
raise URLError(err)
urllib.error.URLError: <urlopen error [Errno 110] Connection timed out>
Question - has SCC connector been tested without external IPs?

Just a note : I've tested and SCC connector is OK with only private IPs as per above thread.

Assigning to @blueandgold. @ttodo-softserve is trying to setup Forseti with no external IP or cloud NAT. The outstanding questions/issues are:

SendGrid Email notification is not working as mentioned by you as well. Same setup with external IP/internet works - without has exception. Is it possible to use internal Google API for SendGrid?
BlackList scanner is still throwing an exception when getting the blacklisted URLs, even though it is disabled. Why?

@ttodo-softserve , if you made changes to the forseti_conf_server.yaml file, such as disabling the a scanner, you have to tell the server to reload the config.  Please run this command on the Forseti Server/Client VM:
forseti server configuration reload

As for using SendGrid, with private IP, this is something that we are looking into in #2946.  I will have to look into this and follow-up with a solution.

@blueandgold - done it multiple times. Plus restart of the service.
It's minor issue, can be ignored for now.
How can all email notifications be disabled since SendGrid will not work anyhow without internet access?

Hi @ttodo-softserve, if want to disable email notifications, you can do update the notifier section of the forseti_conf_server.yaml file like below.  (note: you will have to update the source file that is stored in GCS, which is then pulled onto the Forseti server)
I will go ahead and close this issue.  But if you need any more help, please feel free to re-open.
Thanks.


inline declaration removal

GCC 5 is more strict about inline semantics: inline declaration without definition emits warning during compilation and will also cause an undefined reference error during linking.
Thus, removing the inline specifier from the declarations of uv__time_precise().
fixes #18
Notes:

Works on linux, speculatively applying the inline specifier removal to other targets as well.
Admittedly not sure why the inlines were there originally. The only headers that contain inline are the three uv_platforms.hs and only for uv__time_precise().

why the inlines were there originally : I think it was to reduce one function call while making libtuv from libuv for often called time access function.

can you rebase this as #17 is merged?

Sure, rebased.

@akiss77 , I'm so sorry I didn't see you have already rebased and I made a commit after your rebase. can you please rebase again?
I think as this PR is opened in my browser in an another tab, so there was no notification :(

@seanshpark no worries, here it goes :)


Add permissions on milestone step for teammates

Review by @jglick or @amuniz as current committers

👍

The minimum needed is me then. I can cut the release if needed.


Using tilde (~)

The following test fails
def test_gitignore_global_file(File):
    gitignore_global = File('~/.gitignore_global')
    assert gitignore_global.exists

This one succeeds
def test_gitignore_global_file(File):
    gitignore_global = File('/Users/me/.gitignore_global')
    assert gitignore_global.exists

In order to make the test succeed I have to use /Users/me instead of tilde.
Is it correct?

In the meanwhile, I found this workaround
from os.path import expanduser

def test_gitignore_global_file(File):
    path = expanduser('~/.gitignore_global')
    gitignore_global = File(path)
    assert gitignore_global.exists

Hi, thanks for reporting.
I think there is a bug here and the behavior could be different given the undelying backend or options (like sudo).

Python does not expand ~ by default.  For that matter, only shells do.
likewise by default Python will not expand '$HOME/.gitignore_global' which is valid to the shell and in many cases equivalent to the example given.
Should the File() module call both expanduser and expandvars from os.path?

Using expanduser cannot work on remote connections (ssh, docker etc). This must be done by the shell.

I had the same problem. My current workaround that works on remote connections looks like this:
import os

def test_gitignore_global_file(File, User):
    gitignore_global = File(os.path.join(User().home, '.gitignore_global'))
    assert gitignore_global.exists

But I think it makes the tests unneccesarily verbose. It would be nice if there was a shortcut for home-local paths.


Not compatible with .NET Standard 2.0

Getting the an error message when I try out the example XAML code.
 <CardView.CardViewContent>    </CardView.CardViewContent>
Type CardView not found in xmlns http://xamarin.com/schemas/2014/forms
I believe this error is being caused by the incompatibility, since Visual Studio is having to revert the package to the .NET Framework, instead of using .NET Standard

Hello @DiogoCosta1998 . Can you please create a new empty project using CardView and upload it so I check it?

Output.txt
CardViewApp.zip

CardViewApp.zip
Output.txt

Any update on this? I'd love to use this in .NET Standard 2.0.

I have just pushed a new package to Nuget that uses .Net Standard 2.0. to install: Install-Package XamarinForms.CardView.NetStandard
It might take up to an hour to get published, so if you can't directly install it give it up to an hour after the time of this post. Please give me your feedback if any error happened when trying it with .Net Standard.


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 82 ms

Resolved: ClamAV performance has improved in d8fae10.


Accept request timeout

Pull request template
Checklist

[x] I have read the CONTRIBUTING guide
[x] branch rebased on develop
[x] styleguide check npm run validate
[x] good naming
[x] keep simple
[ ] video or screenshot attached
[ ] testing instruction attached
[x] check your PR once again

Description
Решение проблемы https://github.com/swaponline/swap.react/issues/1395
Video or screenshot
What and how to test

Не то


Use hamcrest 1.3's TypeSafeDiagnosingMatcher

We don't need to preserve compatibility with 1.1 any more.
This backs out a change I made in
ef2223477951f70bfb1395f356a60bee68a9625c.

LGTM, other than the line length nits.  Please merge after you fix those.


fix(dApp): CI hotfix, SEO-title change & scss minor improvements

SEO-title changed to drop ETH environment mention
fixed major issue with the testnets CI
tiny scss refactoring
building optimized with webpack
module purgecss added

Closes ZKF-1079
Closes ZKF-1088

:tada: This PR is included in version 2.7.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


App deployed on the NB goes into error state

Is this caused by loss of connectivity? If so, this should be clear from the UI and it should either automatically resume normal state once problem resolved or allow user to acknowledge the issue and resume normal state.

This was caused by deployment state check putting deployment in error state when it fails to connect to the infrastructure


Pass seed groups directly to SybilGroupRank rather than finding seed groups by looking at seed users

https://github.com/BrightID/BrightID-AntiSybil/blob/8ebe6ccb407d487ce4f060976dedf71c8d137d4e/anti_sybil/algorithms/sybil_group_rank.py#L36
If we have for example 3 seed groups A, B, C and A=(a1, a2, a3) B=(b1, b2, b3) and C=(c1, c2, c3) and then (a1, b1, c1) make a group, my approach set this new group as a seed group. It may be problematic.
There is a subtle attack where members of different seed groups can choose to collude to create new seed groups.  It doesn't seem likely, but it's probably better to avoid it.

To sum up the bug, if all members of a group are in seed groups, that group will be a seed group even if it wasn't designated as one.


Add backwards compatibility tests for automation

Add backwards compatibility tests for the plugin to automate end-to-end upgrade paths for faster releases.
Developer documentation for implementing bwc tests and hooking them to CI: https://github.com/opensearch-project/opensearch-plugins/blob/main/TESTING.md#backwards-compatibility-testing

This issue is resolved by PR #681


Question about chunk size

I'm currently trying to make my dj engine (https://codeberg.org/obsoleszenz/librecdj/) sample accurate.
Until now I used SincFixedOut and whenever the chunk size from the audio host changed, i offloaded the creation of a new SincFixedOut on a worker thread.
Buf to make it sample accurate, i want to partition the buffer into smaller buffers, according to the midi events and their timestamp. The slices would be all of different sizes, meaning i would need multiple Resamplers. As this is a realtime context i can't heap allocate a new Resampler and I wonder if it would be possible to change the chunk size on an existing resampler?
Or am I overseeing something?

It would be possible to change the chunk size of the SincFixedIn/Out and FastFixedIn/Out. I think the easiest way to do this while avoiding allocations would be to only allow values equal to or smaller than what the resampler was created with. So you create a resampler with the largest chunksize you expect to need, and then reduce it to what is needed at the moment.
There could then be s trait method called set_chunksize or something like that, that changes the input or output chunksize depending on the resampler type (or an error if this isn't supported or if the new value is too large).

It would be possible to change the chunk size of the SincFixedIn/Out and FastFixedIn/Out. I think the easiest way to do this while avoiding allocations would be to only allow values equal to or smaller than what the resampler was created with. So you create a resampler with the largest chunksize you expect to need, and then reduce it to what is needed at the moment.
There could then be s trait method called set_chunksize or something like that, that changes the input or output chunksize depending on the resampler type (or an error if this isn't supported or if the new value is too large).

Only allowing chunk sizes <= the initial max_chunk_size would be perfect and all that i need. I actually started playing around with this idea, but i hit an issue in the internals. The needed_input_size became 0 after the first process call. I opened a PR here, but maybe makes sense to start over again: https://github.com/HEnquist/rubato/pull/81

That PR has a lot more changes than what is required! See https://github.com/HEnquist/rubato/pull/82
It's a draft and only implemented for SincFixedIn/Out so far but I think it works as it should. Can you try it?

That PR has a lot more changes than what is required! See #82
It's a draft and only implemented for SincFixedIn/Out so far but I think it works as it should. Can you try it?

Ouh nice! Thank you :) I will give it a spin but might need a couple of days.

That PR has a lot more changes than what is required! See #82
It's a draft and only implemented for SincFixedIn/Out so far but I think it works as it should. Can you try it?

Sorry, I didn't find time yet to play around with this further, have to fix my software to play a gig this week and I was a bit overoptimisitic to pull off the major change of a sample accurate audio engine ^^ So I hacked in a intermediate solution and come back to the proper way after the gig :) So sorry for no update...

I finally found the time and brain to implement sample accurate playback on my side and it works quite good so far :)
https://codeberg.org/obsoleszenz/librecdj/pulls/47/files#diff-d075ab60878d56dab23e87a12135b50c80e0fa83

Great, thanks! Then I'll include this in the next release.

This is included in the v0.16.0 release.


Add back the finalizer for resources in agent

revert: https://github.com/stolostron/multicluster-global-hub/pull/172
fix: https://github.com/stolostron/backlog/issues/26311
the finalizer is useful to clean the records in the database before deleting the resources in the regional hub cluster.
Signed-off-by: clyang82<EMAIL_ADDRESS>

/LGTM


fix(config): Don't double-warn about $CARGO_HOME/config

What does this PR try to resolve?
The core requirements for this bug are

You have a $CARGO_HOME/.config
Your project is inside of $HOME

We have a check to make sure we don't double-walk $CARGO/config but
that check is after we warn about there being no .toml extension.
To fix this, we just need to move that check outside of the file lookup.
This required changing the seen check from checking whether walked the
config file to checking if we've walked the config dir.  As we only have
one file per directory, this should work.
Fixes #14560
How should we test and review this PR?
test commit added the test, fix commit fixed the issue.
Additional information

@bors r+

:pushpin: Commit fe917f224decb340287d475a1f7eb43627647097 has been approved by epage
It is now in the queue for this repository.

:hourglass: Testing commit fe917f224decb340287d475a1f7eb43627647097 with merge 4b81a83267db2cef4eb34e151df7abbde93e1ff8...

:sunny: Test successful - checks-actions
Approved by: epage
Pushing 4b81a83267db2cef4eb34e151df7abbde93e1ff8 to master...


Unable to exec a created singularity image as a non-root user

Hi, I've previously been able to generate Singularity images using a much older version of the docker2singularity.sh script that I can then subsequently run as a non-root user.
Trying the latest master version, I can build the docker2singularity.sh docker container and run the tool to get a Singularity image file but I can only access the generated image as root due to the permissions of most of the files in the /.singularity.d/env directory. 10-docker.sh seems to have permissions set to 755 but all the other scripts have their permissions set to 700 resulting in the following error when trying to access the container as a non-root user:
/.singularity.d/actions/exec: .: line 7: can't open '/.singularity.d/env/01-base.sh'

I'm testing this using the alpine:3.7 docker image as follows (as root):
$ docker run -v /var/run/docker.sock:/var/run/docker.sock -v /tmp/test:/output --privileged -t --rm docker2singularity -n testimage alpine:3.7

The testimage.simg image file is created in /tmp/test and I can successfully run it using:
$ singularity exec /tmp/test/testimage.simg /bin/sh

If I switch back to a non root user, it fails to run giving the error shown above. I am also able to successfully run the docker command to create the image as a non-root user that is a member of the docker group but the permissions of the files in /.singularity.d/env still prevent the image from being run while not root.
I'm running singularity version 3.3.0-1.
Apologies if I'm missing something here, or if this is intended behaviour, I couldn't see any information about this in the docs or previous issues. Thanks.

You shouldn't need to run the docker container as root, but you would need the user added to the Docker group. But there is still the issue with the files permissions.
Could you try rebuilding the container, but commenting these two lines that copy actions and environment scripts? https://github.com/singularityware/docker2singularity/blob/master/docker2singularity.sh#L273
Older versions of Singularity wouldn't provide these, but (I think) it's likely the case that the build will handle it. Let me know if the files exist, and if that resolves the permissions, and I can do a fix.

Thanks for the quick response @vsoch.
I commented out lines 273 and 274 in the docker2singularity.sh script.
I hadn't noticed that the subsequent commands generate the 10-docker.sh script and copy this into $build_sandbox/.singularity.d/env/ so the first time I ran, the run fails at step 5 with:
(5/10) Setting ENV variables...
cp: can't create '/tmp/testimage.build/.singularity.d/env/10-docker.sh': No such file or directory

I added a line to create this directory after the two commented out lines:
mkdir $build_sandbox/.singularity.d/env; chmod 755 $build_sandbox/.singularity.d/env

The script now runs successfully, creates the testimage.simg file and I can run a terminal in the singularity container without having to be root.
I can also confirm that within the singularity container, /.singularity.d/env contains 6 scripts in addition to 10-docker.sh (i.e. 01-base.sh, 90-environment.sh, etc.)
There are also 5 scripts in the /.singularity.d/actions directory. It looks like these directories are therefore being populated correctly without the copy commands in script.
If you don't mind updating this, that's great, otherwise I'm happy to raise a PR for the changes.

Woot, good work @jcohen02 I'm so glad that fixed it!
If you have the bandwidth, I'd greatly appreciate a PR to the branch that corresponds with the version you are working with (e.g., https://github.com/singularityware/docker2singularity/tree/v3.3.0). Once that goes through, I'll open another PR for that branch to master.

Thanks @vsoch, great that this was straightforward to resolve. PR created against v3.3.0.

Fixed with #68 #69 thanks @jcohen02 !


Improve metadata for social media preview cards

Add missing og:url and og:type fields
Localize description field if a city link is shared.

per @Ceshion let's at least wait post-migration to a framework before adding this

We can also add additional context to the actual page title~
https://github.com/defund12/defund12.org/pull/1525/files#diff-822e9cf54ee310f9a7ebd0d16c9ae066R7
-  <title>{{ site.title }}</title>
+  <title>{% if page.title %}{{ page.title | escape }} on Defund12.org{% else %}{{ site.title }}{% endif %}</title>

Just did two small fixes. Got frustrated trying to learn react and Gatsby. Someone else take over if you want!

@bingbongle @Ceshion There is still two small tweaks here that are probably good.

It looks like this was added via: https://github.com/defund12/defund12.org/pull/1596


Elastic license change

Link to product page on endoflife.date
https://endoflife.date/tags/elastic
No incorrect details, but https://www.elastic.co/blog/elasticsearch-is-open-source-again is now under AGPL again, which might be worth mentioning.
Unclear at this point if this applies retrospectively to the interim releases, or to newer releases after the announcement etc.

from license perspective, it would be definitely a new major release, and we should call it out when the new release comes out.

We could add an info note about the license change. Do you have a link for license change in next major?
Hashicorp and Redis were careful about calling out the version numbers that would carry the change, I wish there was something similar here.

The FAQ says 8.16+:

The 2024 change, where we added AGPLv3 alongside SSPL, is expected to take place before the 8.16 release is generally available.


Plot polylines

Hi, and thanks for a great open source contribution!
This isn't really an issue, but I was just wondering if you might have an example of plotting a polyline from a set of coordinates on top of a network plot?

Here's an example of using matplotlib to plot more "stuff" on top of a network: https://github.com/gboeing/osmnx/blob/master/examples/07-example-plot-network-over-shape.ipynb
And here's an example of doing something similar with leaflet/folium: https://github.com/gboeing/osmnx/blob/master/examples/11-plot-routes-folium-web-map.ipynb

Hi @gboeing,
Following your example 7, I've been trying to draw lines as polygons on top of a base map, but with no luck. I'm certain that the coordinates I have are inside of the boundary, and I get no error when adding them. Do I need to re-project them before adding, or is there something wrong with the format I'm adding them as?
def plot(geometries):
  # get the place shape
  gdf = ox.gdf_from_place('Copenhagen Municipality,Denmark')
  gdf = ox.project_gdf(gdf)

  # get the street network, with retain_all=True to retain all the disconnected islands' networks
  G = ox.graph_from_place('Copenhagen Municipality,Denmark', network_type='drive', retain_all=True)
  G = ox.project_graph(G)

  fig, ax = ox.plot_graph(G, fig_height=10, show=False, close=False, edge_color='#777777')

  # Add shape from gdf
  for geometry in gdf['geometry'].tolist():
    if isinstance(geometry, (Polygon, MultiPolygon)):
      if isinstance(geometry, Polygon):
        geometry = MultiPolygon([geometry])
      for polygon in geometry:
        patch = PolygonPatch(polygon, fc='#cccccc', ec='k', linewidth=3, alpha=0.1, zorder=-1)
        ax.add_patch(patch)

  # Add lines:
  for geometry in geometries:
    if isinstance(geometry, (Polygon, MultiPolygon)):
      if isinstance(geometry, Polygon):
        geometry = MultiPolygon([geometry])
        for polygon in geometry:
          patch = PolygonPatch(polygon, fc='#148024', ec='#777777', linewidth=10, alpha=1, zorder=2)
          ax.add_patch(patch)

  plt.savefig('images/cph.png', alpha=True, dpi=300)

plot(geometries)

geometries containes polygons like these:
POLYGON ((55.6949875 12.558462, 55.6938796 12.5584122, 55.6929711 12.5585957, 55.6921317 12.5579927, 55.6916918 12.5564539, 55.6909246 12.5553629, 55.6901215 12.554119, 55.6891181 12.5531433, 55.6881469 12.5526575, 55.687502 12.5538862, 55.6866445 12.5530816, 55.6856769 12.5524416, 55.6848185 12.5515929, 55.6838506 12.551074, 55.6829915 12.5504047, 55.6821492 12.5498124, 55.6812104 12.5492503, 55.680311 12.5486803, 55.6792187 12.547724, 55.6783172 12.5472156, 55.6774282 12.5466767, 55.6765291 12.5461124, 55.6755652 12.5453961, 55.6747743 12.5445313, 55.6738159 12.5439029, 55.673417 12.5454132, 55.6733398 12.5470051, 55.6731045 12.5486561, 55.6726013 12.5501493, 55.6727833 12.5520672, 55.6716717 12.5525378, 55.6706619 12.5528382, 55.6698239 12.5521737, 55.6949875 12.558462))
POLYGON ((55.6699977 12.5521733, 55.6693768 12.5509383, 55.6684025 12.5511539, 55.6677405 12.5500371, 55.6668188 12.5501435, 55.6658323 12.550075, 55.665264 12.5487917, 55.6649187 12.5473085, 55.6645313 12.5457653, 55.6699977 12.5521733))

Your best bet for "how-to" questions is the user community at StackOverflow.


feat: add metrics generator

:tada: This PR is included in version 0.2.0-alpha.25 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Broadened catch clause when extracting es query body

What does this PR do?
A user reported that our instrumentation of the elasticsearch client triggers an error when using hiberantesearch:
java.lang.UnsupportedOperationException: Not implemented! Expected to produce content only over produceContent(), or writeTo(OutputStream) if blocking calls are acceptable for your use case.
at org.hibernate.search.elasticsearch.util.impl.GsonHttpEntity.getContent(GsonHttpEntity.java:164)
at co.elastic.apm.agent.esrestclient.ElasticsearchRestClientInstrumentationHelper.createClientSpan(ElasticsearchRestClientInstrumentationHelper.java:105)
at co.elastic.apm.agent.esrestclient.v6_4.ElasticsearchClientAsyncInstrumentation$ElasticsearchRestClientAsyncAdvice.onBeforeExecute(ElasticsearchClientAsyncInstrumentation.java:68)

This error is not caught by our current instrumentation. With this PR, we simply make our code more defensive to only fail the recording of the query and not the entire adivce.
I would suggest not implementing any special case handling for hibernatesearch, as the issue has been fixed there since version 6:
https://github.com/hibernate/hibernate-search/commit/8d603b112f8ce634e5c06ca72a96acc1653efffe
Checklist

[ ] This is an enhancement of existing features, or a new feature in existing plugins

[ ] I have updated CHANGELOG.asciidoc
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] Added an API method or config option? Document in which version this will be introduced
[ ] I have made corresponding changes to the documentation

[ ] This is a bugfix

[ ] I have updated CHANGELOG.asciidoc
[ ] I have added tests that would fail without this fix

[ ] This is a new plugin

[ ] I have updated CHANGELOG.asciidoc
[ ] My code follows the style guidelines of this project
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] I have updated supported-technologies.asciidoc
[ ] Added an API method or config option? Document in which version this will be introduced
[ ] Added an instrumentation plugin? Describe how you made sure that old, non-supported versions are not instrumented by accident.

[ ] This is something else

[ ] I have updated CHANGELOG.asciidoc

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Start Time: 2023-01-30T13:04:34.657+0000

Duration: 37 min 43 sec

Test stats :test_tube:

Test
Results

Failed
0

Passed
2696

Skipped
36

Total
2732

:green_heart: Flaky test report
Tests succeeded.
:robot: GitHub comments
Expand to view the GitHub comments

To re-run your PR in the CI, just comment with:

/test : Re-trigger the build.

run benchmark tests : Run the benchmark tests.

run jdk compatibility tests : Run the JDK Compatibility tests.

run integration tests : Run the Agent Integration tests.

run end-to-end tests : Run the APM-ITs.

run windows tests : Build & tests on windows.

run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)


Develop

提交了DeeplabV3和 GAN综述两个的markdown，两项issue均由本人另一个账号申请，后续提交与修改均通过此账号，辛苦您替换一下！感谢！

 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.You have signed the CLA already but the status is still pending? Let us recheck it.

1.图片放到images相应的目录下：https://github.com/PaddlePaddle/awesome-DeepLearning/tree/master/docs/images
2.文件夹命名不要有中文，仿照我们的命名方式
3.在本地利用sphinx编译没有错误了之后再提交


add OasisV3MatchingMarketExchangeWrapper

Pull Request Test Coverage Report for Build 8745

38 of 38 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 8728:
0.0%

Covered Lines:
1579

Relevant Lines:
1579

💛  - Coveralls


VIRUS FOUND

Just put the built files con VirusTotal
https://www.virustotal.com/gui/file/6a0b2aa3a188d2c7bf090c6560b9252d398072044b36cb33aeac528fa41c919d

Hi! Thanks for the issue.
I checked this. These 3 positives are false positives from little-known antiviruses; there is nothing dangerous in this file. In any case, you can download the source code and compile the binary yourself.

If anyone has experience on how to add an exception to virustotal or what needs to be done to remove this false positive trigger, I am open to advice.


Add Liqoctl pipeline

Description
This PR adds the steps to build liqoctl and publish it with each release. It also fixes several step conditions which were broken and update the create release action, since the one currently used is deprecated.
How Has This Been Tested?

[x] Manually

/test

/test

/test

/test

/merge


Pattern matching: aligned topic text with code

Fixes dotnet/docs#5171
Also fixes triangle area.

This won't compile:

GeometricUtilities.cs(34,25): error CS0077: The as operator must be used with a reference type or nullable type ('Square' is a non-nullable value type)
GeometricUtilities.cs(38,25): error CS0077: The as operator must be used with a reference type or nullable type ('Circle' is a non-nullable value type)

What you need to do to fix that is to switch from as to a cast in the following code:
https://github.com/dotnet/samples/blob/6d0ec951f98f1890df7104ef4b6568147ce72b12/csharp/PatternMatching/GeometricUtilities.cs#L29-L46

@svick thank you very much! I've fixed the code.